Hope to support 4K video

Hope to support 4K video

@Vxider John's screensaver just displays the videos that Apple created for their Apple TV screensaver. If Apple starts to stream 4K aerial videos to the Apple TV, then John's screensaver will automatically pick them up.

Closing as 4K video will be supported automatically if Apple ever released any 4K videos.


BLACK CAMERA on iOS - Stopped working

One week ago I've tested the repo and the demo on different devices and all of them were working. Today all of a suudden, the iOS devices I have access to, get a black camera when accessing the camera.
All the other devices seem to work fine, do we have a workaround for this?

This sounds like a permissions problem. Did you get an update this week?
Can you wipe any of those devices? Maybe there's a cookie or localstorage cached that's messing with things.

I've tried opening the demo https://cozmo.github.io/jsQR/ with icognito mode to make sure it isn't a cookie problem.  It happened on a iPhone X and iPhone 7 at the same time.
The demo https://cozmo.github.io/jsQR/ basically shows a black screen on safari and a "loading camera" forever on chrome.
That's actually a problem with accessing the camera with iOS but I guess there isn't a better place to comment this issue?

on iPhone 6  works, camera opens, but it is very slow, about 4 fps :D
any ideas ?

I have had some problems with iOS, i use zebracrossing and jsqr parallel. This is how my video element looks:  <video className={classes.scanner} playsInline autoPlay muted ref={this.videoRef} /> Maybe you can try the playsInline, autoPlay and muted attributes @Guttz. You also need to set the contraints for the getUserMedia to video: { facingmode: 'environment' }.
For the lagg on iOS @TheAlexandru i added a timeout of 300ms to the recursive frameRequestCallback function call, this helped some framerate, but not ideal since it now only scans every 0.3 seconds. I hope this helps out a bit...

A few thoughts

Chrome will never work on iOS - iOS blocks access to getUserMedia inside of webviews, which is what chrome uses.
@TheAlexandru for performance I'd make sure to benchmark rendering video to canvas without jsQR, since jsQR can eat up a fair number of resources. If the FPS looks better there you may want to try running jsQR in a separate thread via webworkers, and potentially downscaling the video first. The tradeoffs of these solutions would have the evaluated on a case by case basis.
As for the original black screen issue - I can't reproduce this on an iPhone running the iOS 12.3.1, which is all I have access to. I can reproduce this by disabling access to the camera by clicking the camera control button in safari. Might be a good place to start

However as mentioned in the README jsQR is designed to have no webcam specific implementation details. Rather the API is designed to take in image data and output any QR codes found within. Handling reading image data from the webcam and passing it to jsQR is the responsibility of the consumer, and as such I can't provide much additional help here and am going to close the issue since it's not a jsQR issue I do not believe. Please comment/re-open if you disagree.


refactor to clear up ServerReactiveSocket logic

not a fix, just making it clearer what is returned

I'm in the middle of cleaning up the server/client setup and will be sending a pull request for this. Can we wait to make more changes like this until after that?

I'll hold off on this for now

covered by https://github.com/rsocket/rsocket-java/pull/285


systemd: Add script to recover from rescue or emergency mode

Several services can trigger rescue or emergency mode on failure. If either mode is triggered after an OTA update the device will require manual intervention. This is not desirable for Torizon OS as updates should be able to recover on their own during failure.
Add a script that will trigger a reboot that will eventually cause a rollback if emergency or rescue mode is triggered while in the middle of an update.
Related-to: TOR-3390

do we need this on master/scarthgap? If so, could you please cherry-pick this change?

Probably we do. The rescue/emergency behavior would be the same on these versions as well. I'll need to rebase the patches first though. I know the part of the code-base this patch affects has changed slightly in the latest versions of systemd

PRs created for master and scarthgap:

https://github.com/torizon/meta-toradex-torizon/pull/94
https://github.com/torizon/meta-toradex-torizon/pull/95


evolution doesn't work

I'm trying to use evolution on a fairly fresh NixOS. It says 16.03pre75806.77f8f35 (Emu). I installed evolution with nix-env -i evolution. I'm not running gnome; I have .config/evolution and .local/share/evolution directories from another system.
When starting it, I get the following terminal output:
** (evolution:3935): WARNING **: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.

(evolution:3935): Gtk-WARNING **: Failed to register client: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.gnome.SessionManager was not provided by any .service files

and then a bunch of critical assertion failures, as you do with Gtk programs.
Instead of loading up my email accounts it greets me to "Welcome to Evolution" and acts as if it has never seen me before, prompting me for my email accounts. If I enter my info it occasionally indicates that it's unable to talk to mail servers like when I press the button to make it ask my mail servers for supported authentication methods. When I get through the whole thing and press Finish, I get a dialog window saying "The name org.gnome.evolution.dataserver.Sources5 was not provided by any .service files" and cannot proceed.
Then I run nix-env -i evolution-data-server and restart evolution and get the following terminal output:
** (evolution:3982): WARNING **: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.

(evolution:3982): Gtk-WARNING **: Failed to register client: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.gnome.SessionManager was not provided by any .service files

** (evolution:3982): CRITICAL **: em_utils_folder_is_sent: assertion 'CAMEL_IS_FOLDER (folder)' failed

It does seem to start up normally, then, and lists my email accounts and seems generally aware of my settings. However it still is unable to connect to mail servers, with those blue banner bars at the top going 'Failed to connect account "<my account name here>".', 'The reported error was "Could not connect to imap.gmail.com: Network is unreachable".'
I'm confident I am not having network issues because it happens with multiple mail servers, to which I can connect fine otherwise.

I think a lot of the GNOME stuff only works when GNOME is enabled currently. @lethalman

Yes it should be enough for evolution to be wrapped with necessary gio modules. As for evolution-data-server, it's probably enough to put it in propagatedUserEnvPkgs and avoid the nixos module altogether.

If I run evolution with GIO_EXTRA_MODULES={glib_networking}/lib/gio/modules set and evolution-data-server in profile, evolution gets my accounts and is able to connect.

gnome-keyring is apparently also needed. Without gnome-keyring in my profile, evolution prompts me for a password rather often, like a dozen times in a row on startup and then once per connection, or something. I'm not checkboxing "Add this password to your keyring", but the expected behavior is that evolution only prompts me once per start.
Terminal output:
(evolution:30086): e-data-server-ui-WARNING **: credentials_prompter_store_credentials_cb: Failed to store source credentials: The name org.freedesktop.secrets was not provided by any .service files
** Message: Remote error from secret service: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.secrets was not provided by any .service files

Apparently we also need the gio modules from dconf for evolution to remember its window size across restarts, the default window is rather tiny.

On the Contacts view:

Unable to open address book 'Personal'
This address book cannot be opened.  Please check that the path $HOME/.local/share/evolution/addressbook/system-address-book exists and that permissions are set to access it.
Detailed error message: Unable to connect to 'Personal': The name org.gnome.evolution.dataserver.AddressBook9 was not provided by any .service files

Stderr:
(evolution:14811): evolution-util-WARNING **: name_selector_get_client_cb: Unable to connect to 'Personal': The name org.gnome.evolution.dataserver.AddressBook9 was not provided by any .service files

Also, in the Calendar view:

Failed to open calendar 'Contacts : Birthdays & Anniversaries'
Unable to connect to 'Birthdays & Anniversaries': The name org.gnome.evolution.dataserver.Calendar7 was not provided by any .service files

Also, this line shows up a whole lot on stderr:
(evolution:16370): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

Would it be appropriate to have evolution unconditionally propagate evolution-data-server and gnome-keyring? Propagating the entire output of evolution-data-server seems really coarse, but I'm not sure if there's a middle ground.

The problem here is quite controversial, however I think being e-d-s and gnome-keyring single-instance applications, it's doable to propagated them.
I don't care about propagating the entire e-d-s output, we're talking about evolution, not a simple unix tool.

I can't reproduce this issue with recent version of nixpkgs. I've tested Evolution on Debian (not NixOS) and it works just fine. I'm think the reporter has fixed the issue himself: #17926.
I'm closing it. If the issue still occurs please feel free to reopen it.

I can reproduce (at least part of) this on 19.03pre169108.36f31600749 (Koi).
The settings are not modifiable and I get a lot of
(evolution:16370): dconf-WARNING **: failed to commit changes to dconf: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name ca.desrt.dconf was not provided by any .service files

Same result as fgaz here. The settings can not be saved with this error:

The name org.gnome.evolution.dataserver.Sources5 was not provided by any .service files

And the exact same dconf warning over & over again.

I ran into this as well.

The name org.gnome.evolution.dataserver.Sources5 was not provided by any .service files

I also get "The name org.gnome.evolution.dataserver.Sources5 was not provided by any .service files" after the last email account configuration dialog, so I can't save the settings.

This appears to be fixed for me. Not sure if something changed in nixpkgs or if I enabled some option that did the trick (maybe xdg.portal.* or services.gnome.{gnome-settings-daemon,evolution-data-server,gnome-keyring}.enable).
@ben0x539 @jschwartzenberg @sheyll @raboof is this fixed for you?

sorry can't say, i stopped using email a while back

@raboof is this fixed for you?

I'm still getting "The name org.gnome.evolution.dataserver.Sources5 was not provided by any .service files" when I nix-shell evolution and try to go through the account configuration dialog, on nixos-unstable ff377a78794d412a35245e05428c8f95fef3951f

I just discovered that since a year ago there's a programs.evolution.enable option: https://github.com/NixOS/nixpkgs/blame/386234e2a61e1e8acf94dfa3a3d3ca19a6776efb/nixos/modules/services/desktops/gnome/evolution-data-server.nix#L37-L38
I think it can be expected that programs that have a programs.*.enabled option fail to launch if that's not enabled.
Maybe that could be better documented? There could be a section for evolution in the nixos manual, or even a generic section similar to the systemd one: https://nixos.org/manual/nixos/stable/index.html#sect-nixos-systemd-nixos

I just discovered that since a year ago there's a programs.evolution.enable option: https://github.com/NixOS/nixpkgs/blame/386234e2a61e1e8acf94dfa3a3d3ca19a6776efb/nixos/modules/services/desktops/gnome/evolution-data-server.nix#L37-L38

Hmm, that is a NixOS option though, so I guess it can't be used from nix-shell, home-manager or when using Nix and nixpkgs on other Linux distro's, right?

I think it can be expected that programs that have a programs.*.enabled option fail to launch if that's not enabled. Maybe that could be better documented? There could be a section for evolution in the nixos manual, or even a generic section similar to the systemd one: https://nixos.org/manual/nixos/stable/index.html#sect-nixos-systemd-nixos

I agree just documenting that it cannot be used completely standalone, and needs some context to be in place (pointing to the NixOS option as an easy way to achieve that), would be useful and 'sufficient'. I guess the nixpkgs manual would make most sense for that? Though I don't see a section where it would fit particularly well...

I'm running nixos, but cannot get evolution to work. I'm not running gnome (but just lightdm+i3).
To successfully run evolution, I had to set the following in my configuration.nix:
  programs.dconf.enable = true;
  services.gnome.evolution-data-server.enable = true;
  services.gnome.gnome-settings-daemon.enable = true;
  programs.evolution.enable = true;

It would be great if just doing programs.evolution.enable = true; pulls in the required dependencies.

I get this same error if I do nix shell nixpkgs#evolution. Any way to solve it with just nix shell?

replaced by #222431

I'm not sure if offloading work to the user is the best way to approach this task.  It sounds like evolution doesn't launch with certain misconfiguration.  Why not fix the derivation to not permit this misconfiguration?

It would be great if just doing programs.evolution.enable = true; pulls in the required dependencies.

It is the case now
https://github.com/NixOS/nixpkgs/blob/575f3027caa1e291d24f1e9fb0e3a19c2f26d96b/nixos/modules/services/desktops/gnome/evolution-data-server.nix#L49-L55


fairseq-hydra-train is broken for legacy model architectures

🐛 Bug
The version of hydra-core which is prescribed in the fairseq/setup.py includes a bug from OmegaConf which breaks legacy model support in fairseq-hydra-train. The most up-to-date version of hydra-core allowed currently is 1.0.7, which has the extremely strange behavior that for a DictConfig object (which is what fairseq-hydra-train casts the args of the command into) overrides the behavior of __getattr__ so that getattr(dict_config_object, "key_not_present_in_dict_config_object", default_value) always returns None instead of default_value. This breaks the mechanism for modifying architectures through the register_model_architecture system.
Proposed fix
Allow new versions of hydra-core and omegaconf in setup.py. I upgrade to hydra-core==1.2.0 and omegaconf==2.2.3 and the bug was successfully resolved.
To Reproduce
For example, in the bart_large architecture, none of the hyperparameter changes are propagated, so that even if you specify arch=bart_large, the model actually created is just bart_base.

Using this config.yaml file in your current working directory:

task:
  _name: span_masked_lm
  data: /path/to/data/binary

criterion: cross_entropy

dataset:
  batch_size: 16
  ignore_unused_valid_subsets: true

optimizer:
  _name: adam
  weight_decay: 0.01
  adam_betas: (0.9,0.98)
  adam_eps: 1e-06

lr_scheduler:
  _name: inverse_sqrt
  warmup_updates: 100000

optimization:
  clip_norm: 0.1
  lr: [1e-5]
  max_update: 5000000
  update_freq: [4]

model:
  arch: bart_large

execute fairseq-hydra-train -m --config-dir ./ --config-name config will successfully launch and train, but will train only a 6-layer bart.base model because none of the architecture changes were allowed to propagate through the DictConfig object.

Expected behavior
The above example should train a 12-layer bart_large model as arch: bart_large is set.
Environment

fairseq Version (e.g., 1.0 or main): d81fac8163364561fd6cd9d82b6ee1ba502c3526 (Aug 29 2022)
PyTorch Version (e.g., 1.0): 1.12.1
OS (e.g., Linux): Ubuntu 18.04
How you installed fairseq (pip, source): source
Build command you used (if compiling from source): python -m pip install -e . --upgrade
Python version: 3.8
CUDA/cuDNN version: 11.4
GPU models and configuration: 4x 16GB Tesla V100
Any other relevant information:

Had the same issue and updated OmegaConf to version 2.1.0 following the discussion at https://github.com/omry/omegaconf/discussions/746.
They fixed this inconsistency in that version and I tested it and everything worked fine for BART pre-training.
It's a minor update if they follow semantic versioning, and in theory it shouldn't be a problem to update fairseq and hydra.


imp(Protobuf)!: alignencode{,_length_delimited}_vec with prost::Message to return Vec<u8>

Closes: #1323

[X] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[X] Added entry in .changelog/

Codecov Report

Merging #1324 (d42d3d8) into main (9854a66) will decrease coverage by 0.1%.
The diff coverage is 94.0%.

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

@@           Coverage Diff           @@
##            main   #1324     +/-   ##
=======================================
- Coverage   59.7%   59.7%   -0.1%
=======================================
  Files        283     283
  Lines      26903   26894      -9
=======================================
- Hits       16087   16081      -6
+ Misses     10816   10813      -3

Impacted Files
Coverage Δ

tendermint/src/proposal/sign_proposal.rs
40.5% <0.0%> (ø)

tendermint/src/proposal.rs
90.4% <33.3%> (ø)

proto/src/lib.rs
100.0% <100.0%> (ø)

proto/tests/unit.rs
95.0% <100.0%> (+0.1%)
:arrow_up:

tendermint/src/block/header.rs
99.2% <100.0%> (-0.1%)
:arrow_down:

tendermint/src/public_key.rs
76.3% <100.0%> (ø)

tendermint/src/public_key/pub_key_request.rs
97.3% <100.0%> (ø)

tendermint/src/validator.rs
72.6% <100.0%> (-0.4%)
:arrow_down:

tendermint/src/vote.rs
83.5% <100.0%> (ø)

tendermint/src/vote/sign_vote.rs
96.8% <100.0%> (ø)

... and 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Wrong Dockerfile in docker-library

docker pull still gets you 2016.11 as latest from the official repo:
https://github.com/docker-library/official-images/blob/master/library/rakudo-star

There's an outstanding pr to update the docker library with 2017.0: https://github.com/docker-library/official-images/pull/2599

Ah I see. Maybe master or a tag could have rakudo_version=latest, with other branches/tags pointing to specific releases?

Will try to take a look. Meanwhile, there are also unofficial images
updated to 2017.01 https://hub.docker.com/r/jjmerelo/alpine-perl6/

I think we'll either need to give them a well defined use case for https://github.com/perl6/docker/pull/6 or take it out, as we seem to be at a bit of an impasse with it.

Does anyone mind if I roll back #6 for now, just to get the new image built?

↑ @JJ

I think the markdown changes there are fine -- I was planning to comment
later today recommending to roll back the Dockerfile change but you beat me
to the punch. 😇

In the interest of getting the new image out there, I'm going to revert #6.  We can always bring it back if needed.


Insert pyproject.toml sections in a pre-defined order rather than just appending

Motivation
At the moment, new config sections in pyproject.toml just get appended to the end of the file. However, this might not be the most principled or intuitive way.
Summary of desired feature
Create a new algorithm for determining the order of entries in the pyproject.toml dictionary before dumping, e.g. alphabetical order.

The value in doing this is diminished now since pyproject-fmt is now supported, so adding this would be re-inventing the wheel (specifically the component which decides where to put new sections, which is quite clever and opinionated logic which we would want to be 100% in alignment with).


combined state level visuals, modified description with better explaination and updated with latest data.

Overall combined state level visuals and updated with latest data till March.

Need to put in the Disclaimer and Legends..
Description along the graph and it has to be indented..
Have fetched the progress from module progress csvs wont activity timestamp data suffice?


Pulling manifest fails with error "read: connection reset by peer"

What is the issue?
After upgrading from 0.1.27 to 0.1.30, I can no longer pull models when connected to a corporate network:
$ ollama pull codellama

pulling manifest
Error: pull model manifest: Get "https://ollama.com/token?nonce=VV5GsyYSIqo_4gO3ILCHrA&scope=repository%!A(MISSING)library%!F(MISSING)codellama%!A(MISSING)pull&service=ollama.com&ts=1712050312": read tcp <IP_ADDRESS>:56764-><IP_ADDRESS>:443: read: connection reset by peer

What did you expect to see?
The model codellama should be pulled and saved locally.
Steps to reproduce

Connect to corporate network
Run ollama pull ... or ollama run ... for any model not present locally

Are there any recent changes that introduced the issue?
It used to work in December.
I downgraded and tested two Ollama versions from the past - both 0.1.24 and 0.1.27 are able to pull models on the corporate network. The issue appeared in version 0.1.28.
OS
Linux
Architecture
x86
Platform
No response
Ollama version
0.1.30
GPU
No response
GPU info
No GPU
CPU
Intel
Other software
No response

I captured the traffic with Wireshark:
250	31.770536460	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	336	Client Hello
251	31.817286137	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	1350	Server Hello
252	31.817286328	<IP_ADDRESS>	<IP_ADDRESS>	TCP	1350	443 → 56764 [ACK] Seq=1285 Ack=271 Win=524032 Len=1284 TSval=1780638996 TSecr=743213469 [TCP segment of a reassembled PDU]
253	31.817286341	<IP_ADDRESS>	<IP_ADDRESS>	TCP	1350	443 → 56764 [ACK] Seq=2569 Ack=271 Win=524032 Len=1284 TSval=1780638996 TSecr=743213469 [TCP segment of a reassembled PDU]
254	31.817286355	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	554	Certificate, Server Key Exchange, Server Hello Done
255	31.817352245	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	56764 → 443 [ACK] Seq=271 Ack=1285 Win=64128 Len=0 TSval=743213516 TSecr=1780638996
256	31.817357662	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	56764 → 443 [ACK] Seq=271 Ack=2569 Win=63232 Len=0 TSval=743213516 TSecr=1780638996
257	31.817359342	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	56764 → 443 [ACK] Seq=271 Ack=3853 Win=62208 Len=0 TSval=743213516 TSecr=1780638996
258	31.817363546	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	56764 → 443 [ACK] Seq=271 Ack=4341 Win=61824 Len=0 TSval=743213516 TSecr=1780638996
259	31.818124543	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	192	Client Key Exchange, Change Cipher Spec, Encrypted Handshake Message
262	31.855122846	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	117	Change Cipher Spec, Encrypted Handshake Message
263	31.855603585	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	489	Application Data
264	31.893609863	<IP_ADDRESS>	<IP_ADDRESS>	TCP	66	443 → 56764 [RST, ACK] Seq=4392 Ack=820 Win=523776 Len=0 TSval=1780639072 TSecr=743213554
265	31.894373918	<IP_ADDRESS>	<IP_ADDRESS>	TLSv1.2	97	Encrypted Alert

Details for the frame 264:

A configuration change was implemented in the aforementioned corporate network, so I am no longer able to reproduce this issue and help testing any potential changes.

Can the author teach me how to downgrade

The details depend on your OS and how you install Ollama, but in general under Releases you can find old Ollama versions, e.g. https://github.com/ollama/ollama/releases/tag/v0.1.27.

详细信息取决于您的操作系统和安装 Ollama 的方式，但通常在“版本”下，您可以找到旧的 Ollama 版本，例如 https://github.com/ollama/ollama/releases/tag/v0.1.27。

Thank you so much!Solved,It's because of the version problem.,Download version 0.1.27 no problem.

Since the security vuln and subsequent patch in 0.1.34 I now get this error message again.


TabNine was unable to install javascript semantic completion backend

Not sure what it means in https://tabnine.com/semantic

But when I start typing, it shows the below message

Is it not working properly? It doesn't show the % like the videos from the blog post
The help page said

If the above steps don't work, check that the language server works when you run it in your terminal.

Which terminal and what language server does this mean???
I can get to the TabNine::config page and find the 2 files in C:\Users\jason_yu\AppData\Roaming\TabNine
But I am not sure what is the equivilance to this step for javascript or any other language other than GO

Assume for the sake of these instructions that you're trying to get go-langserver to work. Use "which go-langserver" in your terminal to find the absolute path to the language server.

which is not working in Sublime's console or cmd or powershell and I am not sure what is the "javascript-langserver"

Sublime Text: Version 3.2.1, Build 3207
OS: Windows 10 1903
npm: 5.6.0

Are you using TabNine Cloud? That's how you see the %s.
The language server is flow. You can look up instructions for how to install it. The windows equivalent of which is where, see https://stackoverflow.com/questions/304319/is-there-an-equivalent-of-which-on-the-windows-command-line

Tried installing flow and use where flow and added it to the TabNine.toml file
[language.javascript]
command = "C:\\Users\\jason_yu\\AppData\\Roaming\\npm\\flow"
args = ["lsp"]
install = [["npm", "install", "-g", "flow-bin"]]

But after TabNine::restart, it still show the same error

I had this issue as well, but I'm on a mac so your solution may not be as easy. I was able to get around it with the Fix Mac Path package, but maybe there is a similar package for windows? Anywho, the issue basically came down to tabnine being unable to find the correct binary from the PATH when it opened a shell behind the scenes. If you ensure your node binary path is added to your PATH in one of your shell's initialization files (~/.profile for me), then stuff should work. You may also want to try restarting sublime after changes have been made.
FWIW, tabnine's suggestion of adding the full path for command did not seem to work for me either.

Still having issues with this on Windows. I installed flow-bin and I can find it in both Msys Git which and Cmd where. I can run flow and flow lsp and it seems to be working fine. But no matter what I do I can't get it to work in TabNine. I get the log below when filtering for lsp:: (I changed a few of the paths):
[2019-10-28][11:13:07][lsp::session][INFO] Spawning controller for route Route { language: "javascript", root: AbsolutePath { path: "\\\\?\\C:\\Users\\xxx.xxx\\Workspace\\tb2\\xxx\\xxx", uri: "file:///C:/Users/xxx.xxx/Workspace/tb2/xxx/xxx" } }
[2019-10-28][11:13:07][lsp::language_server_transport][INFO] Starting language server: `flow lsp`
[2019-10-28][11:13:07][lsp::low_level_language_server][INFO] Calling LSP method `DidOpenTextDocument`
[2019-10-28][11:13:07][lsp::low_level_language_server][INFO] Calling LSP method `Completion`
[2019-10-28][11:13:07][lsp::language_server_transport][ERROR] install error: Os { code: 2, kind: NotFound, message: "The system cannot find the file specified." }
[2019-10-28][11:16:00][lsp::session][INFO] Spawning controller for route Route { language: "javascript", root: AbsolutePath { path: "\\\\?\\C:\\Users\\xxx.xxx\\Workspace\\tb2\\xxx\\xxx", uri: "file:///C:/Users/xxx.xxx/Workspace/tb2/xxx/xxx" } }
[2019-10-28][11:16:00][lsp::language_server_transport][INFO] Starting language server: `flow lsp`
[2019-10-28][11:16:00][lsp::language_server_transport][ERROR] install error: Os { code: 2, kind: NotFound, message: "The system cannot find the file specified."

I'm having the exact same problem

[language.javascript]
command = "C:\Users\thero\AppData\Roaming\npm\node_modules\flow-bin\flow-win64-v0.118.0\flow"
args = ["lsp"]
install = [["npm", "install", "-g", "flow-bin"]]

I modified as such, still no love. Still tries to install it.

I get a new error after renaming the TOML file. Claiming that there's an error.
TabNine found a config file at C:\Users\thero\AppData\Roaming\TabNine\TabNine.toml but was unable to parse it:
Error { inner: ErrorInner { kind: InvalidHexEscape('s'), line: Some(1), col: 15, message: "", key: [] } }

```My TOML file is complete as below

> ```
> [language.javascript]
> command = "C:\Users\thero\AppData\Roaming\npm\node_modules\flow-bin\flow-win64-v0.118.0\flow"
> args = ["lsp"]
> install = [["npm", "install", "-g", "flow-bin"]]
> ```
>
> I modified as such, still no love. Still tries to install it.

Which would be the s in javascript if I'm reading it right.

[language.javascript]
command = "C:\Users\thero\AppData\Roaming\npm\node_modules\flow-bin\flow-win64-v0.118.0\flow"
args = ["lsp"]
install = [["npm", "install", "-g", "flow-bin"]]

I modified as such, still no love. Still tries to install it.
EDIT: After escaping the \ with \ it seems to work... novice mistake.

That is how I solved TabNine command execution problem.
Just add the full path of the executable to the config file.
But the point is, you have to use \ and escape it.
Finally it looks like this.


Open only on Calendar Icon

@eternicode This may be a duplicate ticket of #456, but I'm not sure if it's a new issue since that was from a year or two back). I'm having an issue, and am not sure if #377 solved it or if the fix is still in the works... I'm using the "showOnFocus" method (set to False- have tried in both JS and HTML), but rather than only focusing when clicking the calendar icon, it eliminates focus on everything. Am I using the method incorrectly? Code sample below.
HTML:

    <input id="date" type="text" maxlength="14" class="form-control our-date-picker" data-provide="our-date-picker" data-date-autoclose="true" data-date-toggle-active="true" data-date-show-on-focus="false"/>

Javascript:
$('#sandbox-container input').datepicker({
    orientation: "top right",
    showOnFocus: "false"
});

can you provide a jsfiddle?

We got it as close to how it is currently working in ours as we could; if you set the data-date-show-on-focus to true, it works the same as ours (except that ours also opens on clicking the calendar). On setting it to false, it works the same as ours. We can also use the methods in our Javascript, but for some reason couldn't replicate this on jsfiddle; let me know if this helps! Thank you so much!
https://jsfiddle.net/tn7d8nfx/9/

The behaviour of showOnFocus="false" seems to be inconsistent between using the data-date-show-on-focus attribute, and setting $.fn.datepicker.defaults.showOnFocus in JS. If set using the attribute, the calendar is still displayed the first time the input receives focus.
https://jsfiddle.net/DonDavio/kke1jbbj/8/

I've been seeing what DonDavio describes in my project. Setting the defaults via $.fn.datepicker.defaults on page load and using lazy init like the addon example with data-provide="datepicker". First time the input is focused the picker shows. Any subsequent focus works as should be expected. It's like the defaults aren't being applied until after the datepicker has been initialized and shown.

I am having this same issue, the picker shows the first activation even though I'm asking it not to with data-date-show-on-focus="false"

Still having the exact same problem. Anybody knows if this is already fixed or not?


inner join and join should be marked as inner

SELECT * FROM table JOIN otherTable ON table.ID = otherTable.FK

equal to
SELECT * FROM table INNER JOIN otherTable ON table.ID = otherTable.FK

But after parsing through jsqlparse, the net.sf.jsqlparser.statement.select.Join object will not be marked as inner and there is no other attribute to mark the type.

LEFT JOIN equal to LEFT OUTER JOIN, and LEFT JOIN in net.sf.jsqlparser.statement.select.Join be marked as left，LEFT OUTER JOIN be marked as left and outer.
Should JOIN also have a separate attribute for marking?

Greetings.
Please use a recent Version of JSQLParser 4.5 or 4.6-Snapshot because your example works without any problem.
You can try it here.

Hello, there is no problem with parsing, it just means that the type of Join cannot be easily distinguished.
For INNER JOIN, the attribute inner is marked true:

but for JOIN, no attribute is marked:

Currently, I can only confirm the type of the join by obtaining the firstToken of the ASTNode:
 join.getASTNode().jjtGetFirstToken().toString().equalsIgnoreCase("JOIN")

expect an attribute to mark the JOIN.

Greetings.
The class Join has a public method isInner() which definitly works as intended, because otherwise the De-parsers would fail:
public String toString() {
        StringBuilder builder = new StringBuilder();

        if (isSimple() && isOuter()) {
            builder.append("OUTER ").append(rightItem);
        } else if (isSimple()) {
            builder.append(rightItem);
        } else {
            if (isNatural()) {
                builder.append("NATURAL ");
            }

            if (isRight()) {
                builder.append("RIGHT ");
            } else if (isFull()) {
                builder.append("FULL ");
            } else if (isLeft()) {
                builder.append("LEFT ");
            } else if (isCross()) {
                builder.append("CROSS ");
            }

            if (isOuter()) {
                builder.append("OUTER ");
 ---->>  } **else if (isInner())** {           <------------------
                builder.append("INNER ");
            } else if (isSemi()) {
                builder.append("SEMI ");
            }

            if (isStraight()) {
                builder.append("STRAIGHT_JOIN ");
            } else if (isApply()) {
                builder.append("APPLY ");
            } else {
                builder.append("JOIN ");
            }

            builder.append(rightItem).append((joinWindow != null) ? " WITHIN " + joinWindow : "");
        }

        for (Expression onExpression: onExpressions) {
            builder.append(" ON ").append(onExpression);
        }
        if (usingColumns.size()>0) {
            builder.append(PlainSelect.getFormatedList(usingColumns, "USING", true, true));
        }

        return builder.toString();
    }

I think, I do understand your actual question now better: You want to de-parse to INNER JOIN even when only JOIN (without INNER) has been provided.
Answer is: No, because it would depend on the RDBMS is INNER JOIN had exactly the same meaning as JOIN and its not guaranteed. Although you can implement your own Deparser, which translates any JOIN into a full INNER JOIN.

Thank you for your reply.
JOIN really should not be marked as inner after parsing, but can we add an attribute to distinguish it? For example, add a singel attribute.

but can we add an attribute to distinguish it

Not needed. The absence of any other attribute set clearly indicates what is going on. Please follow the logic of the toString() method shown above.

Ok, I understand, then I still use ASTNode#firstToken to judge whether it is only JOIN (without INNER).
Thanks for your help.


Support other id types besides binary_id

Rewrite all migrations (we need to start with a fresh database in the demo) to change id type in users table to bigserial more easily
Support other id types besides :binary_id (tested with :bigserial: default Ecto migration primary key type)
Format schemas

This must be tested carefully. There may be some side effects.

Awesome! :-)
Maybe we should change on of the demo resources to integer IDs?

Awesome! :-)
Maybe we should change one of the demo resources to integer IDs?

Changed film_reviews primary key type to bigserial

I would have chosen a resource where we have all of the CRUD actions. Once we have tests for the demo in place, all actions would be tested with integer IDs.

I would have chosen a resource where we have all of the CRUD actions. Once we have tests for the demo in place, all actions would be tested with integer IDs.

Done. I changed users table primary key to bigserial. I rewrote all migrations to make this change. Therefore, this also contains the changes from #376
We have to start with a fresh database in our demo application.
This is not a breaking change for Backpex, but one for the demo. Should we add https://github.com/naymspace/backpex/labels/breaking-change to this PR?
This is ready for review now 🚀
@pehbehbeh

We have to start with a fresh database in our demo application.

I will do that manually. Maybe we can reset the demo database on each deploy. 🤔

Should we add breaking-change to this PR?

No, not relevant to Backpex users.


P2  |  Part 1 Add answers to questions about refactoring

Link to the different refactoring plans. Answer the questions under 3.5.3 Task 3: Refactoring plan as well as the git commands showing the actual refactorings.

Closed by #80


cleanup unused variable and typedef warnings

This has changes to clean up C++ output that gives warnings about variables initialized but unused. These come from harmless bits of unnecessary code from code like x[,1]. Fixes #1399.
This also introduces a way to remove the warnings about unused typedefs. We hadn't done this before because it is a bit of a pain when creating the C++ code to see how to do that. But instead I post-processed the final result based on text processing to identify if a typedef is not used and then remove it. I also added a nimbleOption to toggle this, so that if it causes problems we can turn it off.
Let's see what testing shows.

@paciorek The test failure here is in test-car. I can't reproduce it on my machine, so I'm stuck.  It is not at first glance obviously related to changes on this branch.  Are you able to reproduce it?

Sorry failure was a CAR hiccup I had fixed on devel but not pushed. I am rerunning the failed test set.

Easiest path forward is just to merge. I think devel is fine, but having trouble rerunning tests such that they recognize that.


⚠ Bofrost.at has degraded performance

In ee9322c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2388 ms

Resolved: Bofrost.at performance has improved in a44f67f after 8 minutes.


Database (4): Saving POIs

[x] Save the data to database instead of a json file

Done and approved by customer.


No undefined types with defined initialState for useLocalStorage and useSessionStorage

Description
The type of the returned value from useLocalStorage is currently typed as T | undefined, but when providing an initialValue, it should be unnecessary to check for undefined, since, if it is, you should get the initialValue of type T instead.
To solve this I have added two TypeScript overloads for useLocalStorage:

The case when you provide initialValue, value is T
The case where you don't, value is T | undefined.

Additionally:

I changed the type of the setter parameter type to not allow undefined, since that's a no-op anyways.
Changed the remove callback to set state back to initialValue instead of undefined.
Wrote a test for remove as I couldn't find any existing ones.

I have tested to install this package locally (via yarn pack) to check if the types look correct in a project using it, and as far as I can see, it does.
One thing I'm a bit uncertain of, is if the set callback should allow undefined, when initialValue is undefined. It would make sense, but since that's a no-op anyways, it kind of doesn't. So my conclusion is that set should never allow undefined, unless undefined actually does something. It could for example be a second way to remove/unset the value (i.e. do exactly the same as remove), but then you might as well just use remove. So, yeah...
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as before)

The typings of the useLocalStorage hook and callbacks have been improved.
This might cause some minor needs for adjustments in typings where the hook is used.
That remove now sets state back to initialValue rather than undefined.
This might cause need for adjustment if someone has provided an initialValue other than undefined, and relies on the value being set to undefined rather than returned to the provided initialValue after calling remove.
The useSessionStorage hook signature is now equal to useLocalStorage.
This means the last parameter should now be an options object instead of a boolean, but I've made it backwards-compatible so that true gives same result as { raw: true }

Checklist

[x] Read the Contributing Guide
[x] Perform a code self-review
[x] Comment the code, particularly in hard-to-understand areas
[x] Add documentation
[x] Add hook's story at Storybook
[x] Cover changes with tests
[x] Ensure the test suite passes (yarn test)
[x] Provide 100% tests coverage
[x] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[x] Make sure types are fine (yarn lint:types).

Additionally...
After I initially created this PR, I decided to do some more minor fixes:

Moved my also not merged fix #1448 into this PR (stale state fix for updater function)
Added tests for previously untested serializer & deserializer options
Replaced useSessionStorage with a copy of useLocalStorage so they now have the same features and signature.

Example of why the current types are annoying:

Here, there's no reason why value should be undefined, since I provide the hook with an initualValue that is guaranteed to be a boolean.

I legit need this right now lol merge please! I'm having to sprinkle in a non-null assertion #ugly

@gongjoonamu If you find yourself sprinkling many of those non-null assertions, if you haven't already done similar, here's a temporary "workaround" to keep the #ugliness is one place. So when this is released, you can just delete this, and "redirect" the imports back to the react-use one.
import { Dispatch, SetStateAction } from 'react';
import { useLocalStorage as useLS } from 'react-use';

// TODO: Remove when https://github.com/streamich/react-use/pull/1438 (hopefully) gets released
export default function useLocalStorage<T>(
  key: string,
  initialValue: T
): [T, Dispatch<SetStateAction<T>>, () => void] {
  return useLS<T>(key, initialValue) as [T, Dispatch<SetStateAction<T>>, () => void];
}

Added another commit now, which replaces the current useSessionStorage with a copy of useLocalStorage (with storage replaced) because of the following:

It had the same typing issue as useLocalStorage
It did not have the same features, like custom serializer and such
It did not have any tests at all

Added another commit now, which replaces the current useSessionStorage with a copy of useLocalStorage (with storage replaced) because of the following:

It had the same typing issue as useLocalStorage
It did not have the same features, like custom serializer and such
It did not have any tests at all

Would be nice if someone could review this soon. 🙏🚀

Would be nice if someone could review this soon. 🙏🚀

Realized I had another PR with a fix related to useLocalStorage too (#1448), so decided to move that commit here and apply the fix for useSessionStorage as well.
I also fixed some eslint warnings, and added some tests for the serializer & deserializer options which were previously untested.

Realized I had another PR with a fix related to useLocalStorage too (#1448), so decided to move that commit here and apply the fix for useSessionStorage as well.
I also fixed some eslint warnings, and added some tests for the serializer & deserializer options which were previously untested.

Looks like this MR also will fix the return type of useSessionStorage?
Currently it is returning [T, (value: T) => void], and I am running into the problem that Typescript is complaining that I can't use the functional update for the setState function.
Whereas is should be, [T, Dispatch<SetStateAction<T>>], or since you use the copy of useLocalStorage [T, Dispatch<SetStateAction<T>>, () => void]

Looks like this MR also will fix the return type of useSessionStorage?
Currently it is returning [T, (value: T) => void], and I am running into the problem that Typescript is complaining that I can't use the functional update for the setState function.
Whereas is should be, [T, Dispatch<SetStateAction<T>>], or since you use the copy of useLocalStorage [T, Dispatch<SetStateAction<T>>, () => void]

Looks like this MR also will fix the return type of useSessionStorage?
Currently it is returning [T, (value: T) => void], and I am running into the problem that Typescript is complaining that I can't use the functional update for the setState function.
Whereas is should be, [T, Dispatch<SetStateAction<T>>], or since you use the copy of useLocalStorage [T, Dispatch<SetStateAction<T>>, () => void]

Yeah, this PR makes them the same api and typings.

Looks like this MR also will fix the return type of useSessionStorage?
Currently it is returning [T, (value: T) => void], and I am running into the problem that Typescript is complaining that I can't use the functional update for the setState function.
Whereas is should be, [T, Dispatch<SetStateAction<T>>], or since you use the copy of useLocalStorage [T, Dispatch<SetStateAction<T>>, () => void]

Yeah, this PR makes them the same api and typings.

Are there any issues with this PR?
I needed to switch a useLocalStorage using a custom (de)serializer with useSessionStorage and was disappointed to find that it was missing that functionality and I couldn't use it.

Noticed some merge-conflicts had popped up here since last time, so I have rebased and fixed them all now.

@streamich any chance you or another maintainer can look/merge?

I closed my PR #1506 in favor of this one.
I would use useLatest or similar approach for serializer/``deserializeror evensetitself to maintain referential stability similar toReact.useState`.

Any update on getting this merged?

My current project would benefit from this. Any chance getting this merged?

Bump... Hoping this can get merged and released some time soon

Any updates? I am having same issue here. undefined even though second param will be always defined

@streamich are you able to review this? It would help out a lot of us to get this merged in


Union all issues

Hi,
We're using DBT and clickhouse. Trouble begins when using UNION ALL , which is a known clickhouse issue: https://github.com/ClickHouse/ClickHouse/issues/30532
As stated in the above it's a very common pattern for BI/DBT.
DBT has a special union macro (https://github.com/dbt-labs/dbt-utils/blob/main/macros/sql/union.sql) which is very useful.
Bottom line, would be very useful to have dbt-clickhouse Materialize a table that uses this macro as a series of INSERTs of each unionized relations, rather than a single CREATE TABLE as SELECT.

Thanks for reaching out @royxact. We plan to start working on dbt utils soon and we'll definitely consider your suggestion when implementing the union macro.

Thanks @royxact, did you try changing max_threads to limit the degree of parallelism?
Anyway, we plan to start working on dbt utils and I'm pretty sure we could add that feature to the union macro too.

Hi @guykoh ,
Thank you very much for your suggestion (:
We've tried max_threads=1 as a workaround, and that keeps the memory usage under control. But - performance would be limited to a single thread. Sure we could try to tune it, but in the end every relation added to UNION ALL still consumes some memory (see https://github.com/ClickHouse/ClickHouse/issues/23245).
Maybe it's better to solve https://github.com/ClickHouse/ClickHouse/issues/30532 and https://github.com/ClickHouse/ClickHouse/issues/23245 , rather than fixing the DBT macro.
But, I think reaching a point where a CREATE TABLE as SELECT of complex generated queries, with multiple JOINS plus UNION ALL, works perfectly without the need to spend time tuning it for parallelism and join types -  might take some more time.
That's the reason I think it's worth investing in improving the macro, to make clickhouse very useful with DBT, with "no brainer" scalability, right now in the short term.
Thanks!


[BugFix]Fix desc external catalog table not have [partition key] for partitio…

…n tables
What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #15394
Problem Summary(Required) ：

Fix desc external catalog table not have [partition key] for partitio…
desc iceberg_catalog_01.test.test_iceberg_par;
+-------+------------------+------+-------+---------+---------------+
| Field | Type             | Null | Key   | Default | Extra         |
+-------+------------------+------+-------+---------+---------------+
| dt    | VARCHAR(1048576) | Yes  | false | NULL    | partition key |
| name  | VARCHAR(1048576) | Yes  | false | NULL    |               |
| id    | BIGINT           | Yes  | false | NULL    |               |
+-------+------------------+------+-------+---------+---------------+
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 2 / 3 (66.67%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/ShowStmtAnalyzer.java
1
2
50.00%
[309]

:large_blue_circle:
com/starrocks/common/proc/ExternalSchemaProcNode.java
1
1
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5


bug: no matches found: embedchain[community]

🐛 Describe the bug
Looks like the community edition can't be installed for some people.
no matches found: embedchain[community]
This happened to discord user andyy with z shell.

Closing this issue since there is no community edition right now.


Hide/unhide vault

It will be great if you add toggle in the 3 dot menu to hide/unhide vault from Albums. So that no one say their is a secret vault on our Phone. Otherwise some will ask what is this vault or what's inside  this vault.

Not going to do that. Content is already hidden. If you're worried about people seeing there's a lock, you could be also worried that they see the hidden items in the app settings.

Otherwise some will ask what is this vault or what's inside this vault.

You don't have to answer them.


Where is the code

Your work seems very interesting, we would like to try it. When will the source code be available?
Regards.

I have uploaded the code. However, I still have to run a few tests in the next couple of days ...


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg b619c92...3c00bb0:
  > 3c00bb0 Update community files (# 2426)
  > fc93ac7 upgrade to latest dependencies (# 2424)
  > 51be315 Update community files (# 2419)
bumping knative.dev/hack 82bfca1...b9cb198:
  > b9cb198 Update community files (# 151)
  > 0e30607 Update kntest path after https://github.com/knative/test-infra/pull/3087 (# 150)
  > 7331bb1 rename function (# 149)
  > 9b303d6 Update community files (# 148)
bumping knative.dev/networking 8ea3045...4337f03:
  > 4337f03 upgrade to latest dependencies (# 619)
  > 2538710 Update community files (# 618)
  > a70f23c Update community files (# 617)
  > bd21fc9 add component label to reduce invocations (# 616)
  > 58bc12b upgrade to latest dependencies (# 615)
  > c9e7082 Add `activator-ca` and `activator-name` keys in `config-network` (# 608)
  > 9ce389c Update community files (# 614)

/lgtm
/approve


Add definition on how to use the Money object.

In several services I encountered a wrong usage of the Money object, where the amount was actually a Cent value that needed to be converted into a Euro value, before it could be used. That is the wrong way to do it and can create very confusing situations. This PR adds a small paragraph about the correct usage.

It's nice to see that this sparked a discussion. :) I tried to incorporate your suggestions. While I like the idea of having a central location for the Money object definition, I agree that this should be done separately, because it does not help with the problem I want to address.

I feel like this could be helpful for some discussions around the topic in general: https://github.com/zalando/jackson-datatype-money/blob/master/MONEY.md

👍

:+1:


Add support for Properties file format

I needed to generate a file similar to what was generated by the buildnumber-maven-plugin. Unfortunately there wasn't anything similar for Gradle, but I found your plugin and modified the code so it can generate properties files too. You can see an example output below. If you're happy with it please merge it into the code.
#Created by build system. Do not modify
#"2020-11-24T14:46:27Z"
version="<IP_ADDRESS>.2.7.0-SNAPSHOT"
revision=1018
name="registry-client"
timestamp=1606229187761
group="com.registries"
sha="779af3619473de02c5709440c2c7ed7ecc9d7d3b"
git_date="2020-11-23T19:11:06Z"
git_branch="rewrite-build-no"
build_date="2020-11-24T14:46:27Z"
dirty=1

@lessthanoptimal

thanks! merging the PR


Documentation Transition: MkDocs + Markdown + PlatyPS

Documentation should be easy to write, contribute to, and publish and allow for auto-generated reference documentation.
Expected Behavior
Contributors to documentation should be able to write that documentation in markdown, the de facto documentation markup standard for PowerShell. They should also be able to update the documents without having to fiddle too much with configuration. The documentation should be deployable to ReadTheDocs or another static site via CI and include testing. The documentation should also support exporting the existing reference documentation for public functions so that that documentation is not being updated in multiple places.
Current Behavior
Currently, the docs are manually written in ReStructured Text (RST) and published to ReadTheDocs. No reference documentation is included.
Possible Solution
Use mkdocs for documentation, allowing the contributors to write docs in markdown, theme and publish to readthedocs with minimal effort. Reference documentation can be generated from the module using PlatyPS.
Context
This would make contributing to documentation easier.
Your Environment

Module version used: Vester 1.0.1
Operating System: Windows 10.0.14393
PowerShell version: 5.1.14393.693

We discussed this in Slack yesterday, and @michaeltlombardi may have time to assist with this transition.
I agree with the move from RST to Markdown. The one time in March I got motivated to do a full docs refresh, I got frustrated learning RST syntax, and the lack of an easy local preview option on my Win10 box.
I think our neglect in updating them speaks for itself: Any move toward ease of refresh (and a healthier doc update cadence) is welcome.

Closed by #158.


Add a flag that allows for compact output

As of now, all commit messages seem to be spaced out:
    ci(CircleCI)!: Renamed main build job

    Renamed main build job from `build-base` to `base`

    BREAKING CHANGE: The `build-base` is now named `base`

    123

Allowing the user to generate a more compact message with -c/--compact would be neat:
    ci(CircleCI)!: Renamed main build job
    Renamed main build job from `build-base` to `base`
    BREAKING CHANGE: The `build-base` is now named `base`
    123

This could be taken further by introducing some kind of template where a user can define their own template and formatting (commits.template):
{type}{if has_scope: ({scope})}{if is_breaking: !}: {short_msg}
{long_msg}
{if is_breaking: BREAKING CHANGES: {breaking_changes}}
{if are_affected: issues_affected}

Just an idea :)

Are you alright with adding structopt for command-line arguments parsing?
It's relatively lightweight (when compared to clap)

I've worked with both structopt and clap. Since structopt does rely on clap anyway and has been merged into the 3.0-alpha I'd rather use that version of clap.

I am working on #12, so this could be solved with it. Not sure if I want to add a flag for it or just provide an alternative template inside the repo that people can use, as my main goal is to follow the conventional commits specification.

While it makes sense for the spec to separate the title, body, and footer, on shorter commit messages it's a pity to have 2 newlines for 3 lines of content. I agree that it's important to follow a specification for the commits, but if only the commits were pretty to look at as well!

So I have thinking about this for a while and I will probably include a compact version of the current template without the new lines. However, the help screen will have a note that it is not compliant with the specification.

That sounds great!


i cant launch terminal on win 7

Issue Type: Bug
The terminal process "C:\Windows\system32\cmd.exe" failed to launch (exit code: 259).
VS Code version: Code - Insiders 1.51.0-insider (89c002ab02f87102d91efc83c191ef1174756c6a, 2020-10-15T23:17:08.035Z)
OS version: Windows_NT x64 6.1.7601
Extensions: none

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693


feat: reduce number of dependencies by using lodash isEqual

I could be wrong but I am pretty sure Lodash's isEqual already does a deep check and pretty much does the same thing.
I like small bundle sizes 😄

Idk why the lockfile updated vue-demi again 😦

Ouch you're right I checked bundlephobia and it increased a lot aha

It seems to be my fault, the version was not included in the commit but in the npm build

I will do it nevermind thanks a lot @DRoet


bring 2 commits from stable branche into master

Somehow, those got merged directly in stable branches. Those commits should have been merged first in master, then in stable-3.2 and/or stable-3.1 to ease the tracking and avoid potential regression.

jenkins test dev-container_podman

jenkins test dev-docker_cluster

jenkins test dev-lvm_osds_container

jenkins test dev-container_podman

jenkins test dev-docker_cluster

jenkins test dev-container_podman

jenkins test dev-lvm_osds_container

jenkins test dev-lvm_osds

jenkins test dev-lvm_osds


fix(behavior_path_planner): fix geometric pull out lane id for overlapping (#2828)

Description

cherry-pick
https://github.com/autowarefoundation/autoware.universe/pull/2828
Related links

Tests performed

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55
Is It OK to merge into beta/v0.7.0-odaiba1?

@tkimura4
OK!

Thanks!


Help

What is your feature request?
A description of your feature request. References or screenshots appreciated.
How would this feature help you?
A clear and concise description of what this feature would help with, or do.

Hi @byzak4 did you mean to provide more details on what you need help with?

closing by lack of response


Colab video output split screen?

I tried the colab notebook and it worked great, but it splits the video in half with the original on the left and upscaled on the right. I just want the full upscaled video. Is there a way to disable this (or do I have to buy the app?)

Nevermind, removing --montage disabled it


paper-spinner with core-tooltip.css

don't work paper-spinner with core-tooltip.css

Please post a reduced test case.
On Jan 2, 2015 12:40 PM, "BoaventuraThomas"<EMAIL_ADDRESS>wrote:

don't work paper-spinner with core-tooltip.css
—
Reply to this email directly or view it on GitHub
https://github.com/Polymer/polymer/issues/1066.

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


HTML Element type to have children field?

thinking we should add the children field to the html element type. I can make  a PR if you agree.
The use case would be when a parent element is clicked we want to do something to 1 or more children.
One solution would be use elm-pub-sub, but that seems like some extra indirection for some things.
What are your thoughts?

Great idea.
I wanted HtmlElement to mirror the structure of HtmlElement in JavaScript as much as possible, but I've only been adding properties as use cases arise.
How far nested would we go? Would we return every descendant of the node? This could be a pretty big payload.

Hm upon further investigation HtmlElment from the DOM Api does not have children
It looks like it is part of Element which specific elements inherent from like HTMLDivElement
Not sure the best way to handle it. There might be some insight on the PureScript DOM library

No I think you were right: https://developer.mozilla.org/en-US/docs/Web/API/ParentNode/children

I guess the real question is the DomPorts Api HtmlElement the same as the DOM Api, or is that really a HtmlElement

That's a good point, there might be some renaming in order.
It already contains fields that only sub-types of HtmlElement will have. (Such as value from HTMLInputElement and pathname from HTMLAnchorElement.) So there's some inconsistency. I'm open to ideas! :)

Maybe we stick with it just to add the children and we can plan on how we want to resolve the DOM API to DomPorts types?
that will solve the use case I talked about now, but we an revisit the Api with a little more thought.


refactor(diff-blocks): convert diff-blocks to styled component

My first converted component!
I've found the need to define a new config.fontsStyled object with converted styles rules.
Is that ok?
Also renamed styles to avoid colors in names.

Coverage increased (+0.4%) to 34.943% when pulling 9545fbe4cdeb1b078f22c8f54dd54a2f019fd22e on machour:diff-blocks-styled into 6567aa3ed1cde26c94c0ca783149d6a59f98be3d on gitpoint:master.


Adding delegate-lookup support to ArrayContainer

Hi Jeremy,
Following the release of container-interop 1.1 that adds the delegate-lookup feature, I just stumbled on your ArrayContainer class and could not resist the temptation to add support for this feature in your class.
Quite simple really. Hope you will like it and accept this pull request :)
Ho, and unit tests are here too!

Cool. I don't have time this week to take a close look at this, but I'll get to it eventually.
@mnapoli Could you take a look at this too? Thanks.

:+1: great!


⚠ Thumbnails API Endpoint has degraded performance

In cdee147, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 5687 ms

Resolved: Thumbnails API Endpoint performance has improved in a5c52a5 after 9 minutes.


[URGENT] add this to your stories

It's the final day for OpenCode'18, we have all the goodies with us.Please use the below pic as your whatsapp, facebook, instagram stories to promote OpenCode'18.Let spread the word.
Let it stay for 24 hrs, 30 points for each platform.Post screenshot here.
Removal of the story after posting its screenshot here will result in a penalty.
Use the image provided below.In the caption you must atleast include #OpenCode18

#489 Added to story

Fb, insta and whatsapp 😛😛😛

Facebook, Insta, WhatsApp, Snapchat, Twitter and LinkedIn


Setting up development build

Hi,
I'm trying to get everything setup locally to contribute and I noticed that after running yarn install It's failing with dependency issues.
Running:
node: 16
yarn 3
macosx 13.1
fresh clone of the repo following the read me.
Removing yarn.lock fixes things but I'm assuming we don't want that?

In addition, I'm getting all of these typescript errors when building

I'm on the develop branch and this still seems to be an issue.

please run
yarn install
yarn build:all
then navigate to packages/extension then run yarn watch
please reopen if the issue still exists


Integrate Charts and Conditional Rendering

Tutorials: https://developers.sap.com/tutorials/ui5-webcomponents-react-charts.html
Dear Developers,
I think the commit npm install @ui5/webcomponents-react-charts --save have some problems. I am trying to do the hands on, but my computer cannot render the Barchart and the LineChart.
But if I use npm install @ui5/webcomponents-react-charts, the chart can be rendered.
Best regards,
Songyu

Hi @songyu-Ren
the npm command should have nothing to do with the actual rendering of the chart. Npm is used here to install the @ui5/webcomponents-react-charts package. Before version 5 of npm you had to use the --save parameter to automatically add the package name to the dependencies (package.json), for newer versions this parameter isn't necessary anymore as npm install <package-name> adds it even without the parameter.
I kept it in the Tutorial, as it doesn't hurt to add the parameter and is easier for users with an old npm version.
So your rendering problem should have another root cause.

Dear Lukas,
Thank you so much for helping. I am going to check my root.
Best regards,
Songyu

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Completion removes previously written text, messes up indentation

See https://asciinema.org/a/a5sox8i2y6cpdyfwust7k8j2z for what I mean

Please create minimal vimrc less than 30 lines and upload the reproduce ways from Vim starting.  I will test it.

Minimal vimrc:
set nocompatible

" Pathogen
runtime bundle/vim-pathogen/autoload/pathogen.vim
filetype plugin indent on
call pathogen#infect()

Installed plugins, however:

https://github.com/rust-lang/rust.vim
pathogen
racer-vim

Removing rust.vim fixes the issue.

Actually removing rust.vim breaks everything, I get "omifunc not defined". Weird...

I was too fast, the problem is reproducible with just racer and pathogen and a minimal vimrc. I've edited the previous post.
(This message is just for email users, it will be deleted)

New screencast: https://asciinema.org/a/9yoda7n0g5gwh6mvk83pl0zro

I have tested it in minimal vimrc.
But I cannot reproduce it.
I think you have used old racer or vim-racer.  Closing.

I'm so sorry, that was indeed the case. Sorry for bothering you!
On Wed, Sep 07, 2016 at 07:07:23AM -0700, Shougo wrote:

I have tested it in minimal vimrc.
But I cannot reproduce it.
I think you have used old racer or vim-racer.  Closing.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/racer-rust/vim-racer/issues/59#issuecomment-245291517


Playground: Problem with accessibility of the navigation bar

The hover state of links in the navigation bar has a low color contrast against the background (#8c6b2b v #000, which only passes AA for large text).

Repro:
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

Expected behavior:
Link text should either have a brighter color or be bolder with the color #8c6b2b when hovered.
Actual behavior:

This should be fixed when we switch back to blue next week

This should be fixed when we switch back to blue next week

The blue color used, as I remember, had an even lower color contrast against the #8c6b2b color.

this should be fixed by https://github.com/orta/typescript-play/commit/4ff7ef15212bdbea65b9e6bc950ff0dda0542f8f


Bump Scala to 2.12.20

Adds support for JDK21+:

https://github.com/scala/scala/releases/tag/v2.12.18
https://github.com/scala/scala/releases/tag/v2.12.19
https://github.com/scala/scala/releases/tag/v2.12.20

Or see: https://endoflife.date/scala#jdk-compatibility

Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?

@pan3793 I don't have full visibility on that, but it doesn't like it, looking at Usages in Maven Central: https://mvnrepository.com/artifact/org.apache.parquet/parquet-scala

+1 for retiring the parquet-scala module, or even dropping it directly.
there is only one class in the parquet-scala module, users are easy to copy and embed it if we drop the parquet-scala module.
additionally, it's rare to publish a scala lib that only supports a single scala binary version, most scala libs support 2~3 scala binary versions in each version

@pan3793 I agree, are you interested in raising a [DISCUSS] on the mailinglist?

@Fokko https://lists.apache.org/thread/scdq9t2gvvs4glhq0qx4qcvfp62j793s

Closing this in favor of https://github.com/apache/parquet-java/pull/3063


Function caller helpers - create definition from attributes, invoke the function as specified by LLM

As discussed in #344
Added FunctionCallingHelper, related tests and playground demo. Modified OpenAI.Utilities to use the OpenAI library from the solution instead of Nuget.
XML comments, xUnit tests are added.
example usage:
async Task ExerciseFunctionCalling(IOpenAIService openAIService)
{
	var calculator = new Calculator();
	var req = new ChatCompletionCreateRequest();
	req.Functions = FunctionCallingHelper.GetFunctionDefinitions(calculator);
	req.Messages = new List<ChatMessage>
	{
		ChatMessage.FromSystem("You are a helpful assistant."),
		ChatMessage.FromUser("What is 2 + 2 * 6?"),  // GPT4 is needed for this
		//ChatMessage.FromUser("What is 2 + 6?"),  // GPT3.5 is enough for this
	};

	do
	{
		var reply = await openAIService.ChatCompletion.CreateCompletion(req, Models.Gpt_4_0613);

		if (!reply.Successful)
		{
			Console.WriteLine(reply.Error?.Message);
			break;
		}

		var response = reply.Choices.First().Message;

		if (response.FunctionCall != null)
			Console.WriteLine($"Invoking {response.FunctionCall.Name} with params: {response.FunctionCall.Arguments}");
		else
			Console.WriteLine(response.Content);

		req.Messages.Add(response);

		if (response.FunctionCall != null)
		{
			var functionCall = response.FunctionCall;
			var result = FunctionCallingHelper.CallFunction<float>(functionCall, calculator);
			response.Content = result.ToString();
		}

	} while (req.Messages.Last().FunctionCall != null);
}

class Calculator
{
	[FunctionDescription("Adds two numbers.")]
	public float Add(float a, float b) => a + b;

	[FunctionDescription("Subtracts two numbers.")]
	public float Subtract(float a, float b) => a - b;

	[FunctionDescription("Performs advanced math operators on two numbers.")]
	public float AdvancedMath(float a, float b, AdvancedOperators advancedOperator)
	{
		return advancedOperator switch
		{
			AdvancedOperators.Multiply => a * b,
			AdvancedOperators.Divide => a / b,
			_ => throw new ArgumentOutOfRangeException(nameof(advancedOperator), advancedOperator, null)
		};
	}

	public enum AdvancedOperators
	{
		Multiply, Divide
	}
}

Output:
Invoking AdvancedMath with params: {
  "a": 2,
  "b": 6,
  "advancedOperator": "Multiply"
}
Invoking Add with params: {
  "a": 2,
  "b": 12
}
The result of 2 + 2 * 6 is 14.

Hi, This is high level PR, Thank you. I have small changes please let me know if they are okay. Also I have a question/suggestion.

Perhaps we should introduce a new exception type for when OpenAI returns an imaginary method or parameter, preventing us to find&invoking the method.
Seems like Type field is not in use, do we need it?

Thank you, I'm glad you like it, @kayhantolga.
The Type was supposed to be an override (like Name), but seems like I forgot to implement.
As for a separate exception, I agree.
I'll get the PR amended with these changes.


Seperate 32bit and 64bit update streams

Change the updater logic to download 64bit / 32bit builds depending on the current install.  Related to #16

Has the 64 bit version gone live yet? When I download 1.8.0 fresh from GitHub, I'm still running the 32 bit version.

Not yet, its a multi stage process.  1.8.0 has some of the underlying logic in the updater to allow for multiple build uploads.  Next update will check for x86/x64 and the update after that will actually have x86/x64 releases.  It's the simplest way I could come up with to move from a single update stream to 2

Awesome, thanks so much!
On Fri, Nov 20, 2015 at 5:21 PM, Samuel Attard<EMAIL_ADDRESS>wrote:

Not yet, its a multi stage process. 1.8.0 has some of the underlying logic
in the updater to allow for multiple build uploads. Next update will check
for x86/x64 and the update after that will actually have x86/x64 releases.
It's the simplest way I could come up with to move from a single update
stream to 2
—
Reply to this email directly or view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/39#issuecomment-158554716
.


Got wrong line numbers

Check this: http://preview.daimaduan.com/paste/f4612fa76b
line number shows 16, but actual line number is 19. I understand it's because highlightjs auto wrap words and lines, but how to make them match?

Hi,
Currently script does not supported line wrapping.
Your using default styles from bootstrap for pre & code.
I suggest this CSS for fix your issue:
pre {
	overflow: auto;
	word-break: normal;
	word-wrap: normal;
}
pre code {
	white-space: pre;
}

Best regards,
Yauheni

Hi @wcoder , it did fix my issue, but the way we fix it, add a scroll bar then.
If only there's a better way, :dancer:

Hi,
I found the solution. Will add in next version of plugin... I hope soon.

that's great!
On Thu, Nov 5, 2015 at 12:29 AM, Yauheni Pakala<EMAIL_ADDRESS>wrote:

Hi,
I found the solution. Will add in next version of plugin... I hope soon.
—
Reply to this email directly or view it on GitHub
https://github.com/wcoder/highlightjs-line-numbers.js/issues/4#issuecomment-153783462
.

--
Best Regards,
David Xie
Founder of ScriptFan technology community - http://scriptfan.com
Manager of Xi'an GDG (Google Developer Group)
http://about.me/davidx

Everything happens for a reason. Attitude determines everything!

@wcoder how's going with your solution? :dancer:

Hi,
I tried to solve the problem a couple of weeks ago, the idea is the following:

wrap each line of code in a block element;
use pseudo-elements to specify the line number.

Difficulties arose with the first item on the list is due to the implementation of the highlight.js.
I keep thinking on a solution. If you have any ideas, please let me know.

The should work, and as I can think over, this may be the best solution. Just one question, can you know that how many lines does highlight.js auto wrapped for a line? I mean, let's say, there's a line which has 160 characters, and it's wrapped into 3 lines according to the width, but can you know it's 3 lines?
If you can, that's perfect, problem solved.

highlight.js don't uses the wrapper lines by default. Used only wrapping words in inline elements and added line breaks.

oh, didn't know.

Hi @wcoder any update on this?


No license

Your repository currently contains no license info - who is allowed to use it?

Thanks! I'll make sure to add it. It will be MIT.
Med vänlig hälsning,
David Uvenman
	_____________________________

From: Samuel<EMAIL_ADDRESS>Sent: lördag, juni 25, 2016 14:53
Subject: [DavidUv/jqTreeContextMenu] No license (#7)
To: DavidUv/jqTreeContextMenu<EMAIL_ADDRESS>Your repository currently contains no license info - who is allowed to use it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

License is now added. It is MIT.

Thanks! :-)


CSV plugin picking unknown remote_addr and throwing 500 Internal server error

Hi,
The csv plugin was working fine until sometime back. Since starting this week, the plugin is throwing strange error.
There has been no change in grafana version or the plugin version since many weeks.
Query data error
Object
status:500
statusText:"Internal Server Error"
data:Object
message:"Query data error"
traceID:"00000000000000000000000000000000"
config:Object
url:"api/ds/query"
method:"POST"
data:Object
requestId:"Q1541"
retry:0
headers:Object
hideFromInspector:false
message:"Query data error"
In grafana logs, i can see below entries. The remote_addr it refers here is entirely different IP. I am using a local csv read option but this IP is not my local machine IP. What and where is it fetching this IP from ? Any clue or pointers ?
a: Failed to query data: rpc error: code = Unavailable desc = error reading from server: EOF" remote_addr=XX.XX.XX.XXX traceID=00000000000000000000000000000000
logger=context traceID=00000000000000000000000000000000 userId=1 orgId=1 uname=admin t=2022-07-08T05:01:21.39+0000 lvl=eror msg="Request Completed" method=POST path=/api/ds/query status=500 remote_addr=XX.XX.XX.XXX time_ms=6 duration=6.192578ms size=75 referer="http://XX.XX.YY.YYY:3000/d/X8CXJ0Xnz/dashboard?orgId=1&from=now-30m&to=now" traceID=00000000000000000000000000000000
Workaround which works for temporary moment is :

Edit Panel, do any minor modification to properties and undo changes, graph will be visible.
Apply changes in panel, graph is still visible
But as soon as you change time or click refresh on top of grafana right corner, all graphs are lost and same error repeats.

PS : I am loading 10-20 CSV files from a fixed backend location to plot different graphs. As i said, this was working flawlessly till sometime back. Not exactly sure what happened but these are errors I have noticed.
Primarily I suspect the remote_addr. How is the remote_addr fetched inside the plugin code ?

From my setup I can infer that the remote_addr is the IP of the client on which one is viewing grafana.
Having said that I am too facing these random "Internal Server Error 500"
In my use case, the csv file is very small (143 kb), and there seems to be no co-relation with response times either.
Although it works with few manual refreshes, but this is not production grade. I would not like my users to be confused at random.

Would it be possible for the request to be made from the grafana server instead of from user browser?

Which grafana version are you experiencing this?

This is a duplicate of #214


Capturing mouse click events

After spending hours reading documentation and looking into examples of bqplot use cases I still can not answer a simple question: how to capture mouse position from a mouse click. Please help.
Also, if the bqplot team believes that the quote from documentation presented below (and there are plenty of those) helps users in any way they are missing something very important here.
Quote:
class bqplot.scales.OrdinalScale(**kwargs)
An ordinal scale.
A mapping from a discrete set of values to a numerical range.

Hi,
If any user believes that the documentation is not clear, constructive suggestions/contributions will be welcomed.  :-)
I'm not sure about the mouse position. I don't think its been implimented.  You are free to submit an issue.

I'm a mechanical engineer and not a software developer. So i understand what you are saying. I learnt bqplot last year using the examples in https://github.com/bloomberg/bqplot/tree/master/examples. The examples are really good. I believe there is a section where it describes how to change the size of the markers.  If you want to get creative with interactive plots (and you don't want to write JavaScript), bqplot is really good and it is worth learning. Hopefully the documentation will improve over time. Thanks for the feedback.

My 2c from a engineer turned recent software developer:
A combination of the tutorial notebooks and the docs (http://bqplot.readthedocs.io/) was how I learnt, first copy and pasting the examples from notebooks and gradually requiring the greater detail in the docs, understanding which kwargs belong to Axes, Scales, Marks etc. Learning the library from the docs alone would be challenging for me.
How did you end up setting the marker size? The parameter name 'default_size' might be the cause of some confusion, as it is 'default' for the plot you are making, not a bqplot-wide default that requires setting.
Searching the docs for 'scatter marker size' leads straight to the list of kwargs for the Scatter mark, and 'default_size' is listed. Explaining how to implement this in an API call feels a little out of scope. I originally thought the docs to be a bit parsimonious, but in fact they explain the API concisely, and leave any implementation to the notebooks.

@andymcarter  Exactly, I also use 'default_size' to set the marker size. Should it be this way? The name default_size becomes a misnomer if you can change the variable behind it and breeds confusion. Besides, why not give an example of how such a basic thing as setting marker size can be accomplished?
Thanks for pointing me to the examples. They have been helpful, but definitely they are far from covering majority of typical use cases.

@sergeyfridrikh
To set size in the scatter plot, see example below which is taken from the scatter.ipynb in the examples/marks folder.  Search for the sec1_returns variable which is a numpy array of sizes for each scatter point....

from __future__ import print_function
import numpy as np
import pandas as pd
from bqplot import (
    Axis, ColorAxis, LinearScale, DateScale, DateColorScale, OrdinalScale,
    OrdinalColorScale, ColorScale, Scatter, Lines, Figure, Tooltip
)
from ipywidgets import Label

## Get Data

price_data = pd.DataFrame(np.cumsum(np.random.randn(150, 2).dot([[1.0, -0.8], [-0.8, 1.0]]), axis=0) + 100,
                          columns=['Security 1', 'Security 2'], index=pd.date_range(start='01-01-2007', periods=150))
size = 100
np.random.seed(0)
x_data = range(size)
y_data = np.cumsum(np.random.randn(size) * 100.0)
ord_keys = np.array(['A', 'B', 'C', 'D', 'E', 'F'])
ordinal_data = np.random.randint(5, size=size)

symbols = ['Security 1', 'Security 2']

dates_all = price_data.index.values
dates_all_t = dates_all[1:]
sec1_levels = np.array(price_data[symbols[0]].values.flatten())
log_sec1 = np.log(sec1_levels)
sec1_returns = log_sec1[1:] - log_sec1[:-1]

sec2_levels = np.array(price_data[symbols[1]].values.flatten())

sc_x = LinearScale()
sc_y = LinearScale()
sc_y2 = LinearScale()

sc_size = LinearScale()
sc_opacity = LinearScale()

scatter2 = Scatter(x=sec2_levels[1:], y=sec1_levels, size=sec1_returns,
                   scales={'x': sc_x, 'y': sc_y, 'size': sc_size, 'opacity': sc_opacity},
                   default_size=128, colors=['orangered'], stroke='black')

ax_y = Axis(label='Security 1', scale=sc_y, orientation='vertical', side='left')
ax_x = Axis(label='Security 2', scale=sc_x)

Figure(axes=[ax_x, ax_y], marks=[scatter2])


Awesome

Awesome - A Curated List of Awesome Lists
https://github.com/sindresorhus/awesome

Cool. I had it in my icebox for a while but I wasn't sure if it would fit here, but yeah, it does.
Added 9926b3d01fb9abffa26e95cebaf441285581f224

@sindresorhus :+1:

Awesome!!!

:+1:


[Bug]: Overwrite between Tweakscale Redist and Tweakscale

Is there an existing issue for this?

[X] I have searched the existing issues

Operating System
Windows 10
CKAN Version
v1.32.0
Game Version
Steam KSP <IP_ADDRESS>90
Did you make any manual changes to your game folder (i.e., not via CKAN)?
No response
Describe the bug
I was not expecting this error to occur, also had this occur on an earlier version of CKAN, so that's not the issue. Might be a packaging error with the files.
Steps to reproduce
Attempt to download Tweakscale Redist while having the regular Tweakscale installed
Let it download and attempt to install the file.
 We tried to overwrite a file owned by another mod!" error.
Relevant log output
No response

Which file?

Which file?

went ahead and put that into the thing, didn't know that that was a requirement.

CKAN hasn't installed that file for TweakScale since #9114. Are you trying to install an old version of TweakScale or something?

CKAN hasn't installed that file for TweakScale since #9114. Are you trying to install an old version of TweakScale or something?

no, i have the TweakScale file already, I just don't know why some mods need it.

What version of TweakScale do you have installed or are trying to install?

Try uninstalling TweakScale and then installing it again.

alright, I'll see if that makes it not install that file.

That shouldn't matter, you can delete it or not as you please.

That shouldn't matter, you can delete it or not as you please.

hmmm

I don't think TweakScale is going to be re-installed anytime soon on my KSP instance...

That screenshot looks fine to me. I don't know what problem you're trying to highlight.

well it managed to install properly... weird.

Glad you got it working! 🎉

guess it didn't realize that that KSP Recall mod was a dependency or smth... then again I had it manually installed so maybe that was the issue.


Cleanup delay pass to smf

add support for cleanup_delay_usec_per_file CR parameter
should have been in last PR but I missed
change the CI to actually test it

Codecov Report

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

@@            Coverage Diff            @@
##             master     #680   +/-   ##
=========================================
  Coverage          ?   53.48%
=========================================
  Files             ?        8
  Lines             ?      316
  Branches          ?        0
=========================================
  Hits              ?      169
  Misses            ?      147
  Partials          ?        0

Flag
Coverage Δ

gha
53.48% <0.00%> (?)

python-3.9
53.48% <0.00%> (?)

unit
53.48% <0.00%> (?)

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

Continue to review full report at Codecov.

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

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_smallfile
Pass
0
00:04:10

ready to merge


Add image descriptions for the relay and block producer network images

This PR gives 2 images functional descriptions for visualy impaired coders.

Great idea


[WIP] Prepare migration to mozaik >= 1.1.0

[x] Upgrade babel
[x] Move all code to es6
[x] Add code-coverage
[x] Use ava to run tests

Changes Unknown when pulling 2224095f34eafecce6e2a73d58cca64c03831666 on mozaik-1.1.x into ** on master**.


Implementation of containsAnyGreekCharacters

Currently, the onlyGreek method offers this functionality but it will return a string with empty spaces, if the supplied string contains any of them.
This is fine, as onlyGreek is intended to do so (based on it's current unit tests). However, adds an extra step for use cases that we just need to know if a string contains any greek characters or not.
Also, refactored the greekTimeLiteralToEnglish to use the new method instead of the internal check.

@gbrandtio every contribution is welcome !  ;)


Add background task to unfreeze and remove clawback (and other) addresses from ASA

Won't be doing this part in favor of #250


Test 0.10 on AWS

v 0.10
Basic functionality

[ ] Basic install doc/example instructions all good
[ ] Setup multiple backup storage locations and test
[ ] backup, restore, delete, schedule

Cross-cluster scenarios

[ ] backup in Cluster A, restore into Cluster B

Upgrade from 0.9.x to 0.10:

[ ] Install v 0.9.8
[ ] take a bunch of backups
[ ] Upgrade to v 0.10
[ ] Config multiple backup storage locations
[ ] backup, restore, delete, schedule

@carlisia should this be closed at this point?

Yes


[CSM] Simplify Security Profile config checks

What does this PR do?

This PR removes a config check we added on the Security Profile and Activity Dump managers and ensures we check the profile directory values only when both are enabled.
Motivation

These config checks prevent us from using hard coded profiles without the activity dump manager in various situations (tests, demo, etc).
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Merged in https://github.com/DataDog/datadog-agent/pull/27679


Dependency resolution in Gradle 5.6.2

The dependency resolution docs indicates that  Gradle will pick the highest version of a dependency, but the resolution might fail if the version is too high.
What does "too high" mean in the context of dependency resolution?  I inferred that "too high" meant "not a major" version in the the semver terminology, but I was wrong.
If I add these dependencies to any Gradle subprojects:
  implementation 'org.webjars.npm:dompurify:1.0.4'

and
  implementation 'org.webjars.npm:dompurify:2.0.7'

Then, Gradle will resolve dompurify to 2.0.7.
Does this mechanism work the exact same way if dompurify were a transitive dependency?  In other words, if I bring a hypothetical dependency that depends on dompurify and that dependency version differs from the version I used in my direct dependency, would Gradle still resolve to the highest version?
Expected Behavior
Dependency conflict.
Current Behavior
Pick the highest version and hope everything works after.
Steps to Reproduce
Create any project and add two dependencies that have different major version.

resolved via different forum.


Allow JSONClient to throw timeout exception

JSONClient eagerly catches any exceptions and ends up throwing a RuntimeException, attempting to parse a JSON response into a known Hoist exception along the way:
https://github.com/xh/hoist-core/blob/develop/src/main/groovy/io/xh/hoist/http/JSONClient.groovy#L106
I would like to support the case where we handle SocketTimeoutExceptions as routine so we don't get giant stacktrackes. This could either be a built-in case handled by this class, or we could reconsider this general exception approach to allow users of JSONClient more control over how it catches and throws.

took a look at this and definitely understand the case you are trying to handle.  A couple of quick thoughts:

I don't think socket timeouts should be 'routine', with our current understanding of routine exceptions.  However, I can see why you would almost never want a giant stacktrace resulting from a failed HTTP request.  It seems to me that we need to ask ourselves -- when do we want a giant stacktrace in our logs?  Why aren't we more aggressively putting things elsewhere in our default exception handling.

Right now, the "cause" of the exception would be the underlying SocketTimeout, but I agree that that is not really optimal for the end-user. I can't recall any good reason right now for not returning the underlying exception and will look at returning it.

Thanks for looking at this - agree with you on both points re. this not really being a "routine" exception but also us very rarely (if ever) wanting large stacktraces in our primary logs.
My thinking around "routine" was, loosely, that these exceptions didn't represent a programming error, or a fatal / unrecoverable condition, but were likely transient and could in some applications occur with some amount of frequency.  But I can see and agree how that's still not quite how we wanted to define routine, which is maybe more "exceptions that we might deliberately or commonly throw as part of a standard validation or other process."
I am pretty sure this arose from me looking to troubleshoot a client app that was repeatedly timing out calling a back-end data source - the logs at that point were almost unusable due to the volume of Runtime exceptions and the length of the traces.
Appreciate any help with it - ATM


Enable Secure RBAC

This patch enables the Secure RBAC policies to include both the Keystone default roles, and Keystone scoped tokens.
[1] https://docs.openstack.org/designate/latest/admin/policy.html#enabling-keystone-default-roles-and-scoped-tokens

/recheck

/retest

/lgtm

/approve


[Deployment]: [Prepare SDC Prod environment]

Describe the bug
Prepare infra for production and verify a test deployment. Prepare Database, File Storage, Whitelisting, Access, etc.
Pre Production Deployment Work

Setup DB with master slave architecture
Setup Minio
Setup Elastic Search
Setup Ingress
implement HPA and cronjob in UAT

@suresh12 @Pasam6 all activities being done should be as per the installation guide. will be a good way to make sure that nothing is missed out from the guide including things like whitelisting, access, external integration settings, etc.

Implemented HPA with 2 pods as minimum number of pods for a service and trigger a new pod when 80% of resources like memory and cpu is crossed.

@Pasam6 is it a new when we hit 80%. I think we may want to do something like new pod if we remain above 75% for 2-3 min. Else we may end up creating a new pod even for sudden spikes.

@atulgupta2024 This is implemented,
stabilizationWindowSeconds: 120 sec for scaling up and 60sec for scaling down.
Regarding memory requests for each service should i consider the actual service consumption in uat / sdc  and provide double the resource

@Pasam6 yes, that can be good value to start with

Setup DB with master slave architecture : Need to make the failover automatic and create a virtual ip which maps to both master & slave nodes.
Setup Minio: Need to create virtual ip and use rsync to make the data replicated into standby minio
Setup Elastic Search: Completed
Setup Ingress: Completed
implement HPA and cronjob in UAT : Implemented HPA for services in  pucar-prod namespace. Regarding cronjob , developer has to create an api for scheduler then only we can implement cronjob .

Production deployment is completed and tested @suresh12


Request for Drag and drop feature.

Area - UI
Version - 1.1.0
Requirement:
To Drag and drop individual test cases in the folder section and also to drag and drop a single line of code inside the test case.

@NandhaSrinivasan
Add comment for this ticket whether this issue is fixed or this is expected


Check for logged in user on editTrack, deleteTrack and createTrack methods

Currently in the API, for the track-related methods, we check to see if a user can edit the related event, but we don't check first of all if they are logged in.
This isn't a problem from a security point of view, however it does mean we have to perform some additional DB reads to fetch the event etc, whereas we could quit immediately for anyone who isn't logged in - this is what happens for other methods, such as deleteTalk etc,

Joind.In uses JIRA for managing issues - please claim at: https://joindin.jira.com/browse/JOINDIN-703

Hi @liam-wiltshire,
may I take care of this?

Please do!! - if you can claim it on Jira as well that'd be great!!

Well, it seems my works is not needed anymore. I'd have preferred to check it before implementing it 😓

My apologies @micheleorselli  - we've had quite a few contributions recently, and while we are trying to make sure we match them up to JIRA issues, I think this one slipped through.
I hope you didn't spend too long on it, and obviously I'm sorry for any time you have spent  - Joind.in is going through a bit of a transition at the moment, old maintainers stepping back, new maintainers finding their feet, but we are working on getting better!
Thanks,
Liam

No problem at all @liam-wiltshire, we're talking about few lines of code :-)
Now I have a fully configured vm, maybe I will look for other issues 👍

Should this issue be closed now then ?


Can the latex template print the corresponding author's email in the paper's header?

In the example paper.md, the corresponding author adds their email: https://github.com/openjournals/inara/blob/main/example/paper.md?plain=1. As far as I can tell, the GitHub action does not print the corresponding author's email anywhere. It doesn't seem like the latex template is accessing this field. Is it possible to add this to the GitHub action? Some preprint servers ask authors to have at least one email on the preprint. Thanks!

Good idea, thanks! Done.

The corresponding author needs to have both the email and the corresponding: true field.
authors:
  - name: John Doe
    affiliation: [1, 2]
    orcid: 0000-0001-2345-5678
    email<EMAIL_ADDRESS>    corresponding: true

If you have time, a PR to add this to the docs would be greatly appreciated.

Re: adding to docs - Happy to. I will try to get to it soon.
Re: markdown header - I have something that looks like that:
authors:
  - name: Adam Pollack
    corresponding: true
    email<EMAIL_ADDRESS>    orcid: 0000-0001-6642-0591
    affiliation: 1

and I have the following github action
on: [push]

jobs:
  paper:
    runs-on: ubuntu-latest
    name: Paper Draft
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Build draft PDF
        uses: openjournals/openjournals-draft-action@master
        with:
          journal: joss
          # This should be the path to the paper within your repo.
          paper-path: joss_submission/paper.md
      - name: Upload
        uses: actions/upload-artifact@v3
        with:
          name: paper
          # This is the output path where Pandoc will write the compiled
          # PDF. Note, this should be the same directory as the input
          # paper.md
          path: joss_submission/paper.pdf

but I don't see the email show up in the pdf header (see below).
Sorry if I'm missing something super obvious.

I'm actually not sure why it's like that, and whether that's an oversight or intentional behavior. Could you open an issue for that?