Image generation fails to arrive to the canvas

Any workflow will fail with the following error
TypeError: arguments did not match any overloaded call:
  start(self, msec: int): argument 1 has unexpected type 'float'
  start(self): too many arguments

Additionally, live preview isn't showing anything, generation appears to be in the ComfyUI/Output folder.
To Reproduce
Steps to reproduce the behavior:

Go to txt2img dock
Set it up and generate
Wait for gen (check live)
See error

Screenshots

TypeError
Python 3.11.3: /usr/bin/python3
Fri Jul 28 12:09:38 2023

A problem occurred in a Python script.  Here is the sequence of
function calls leading up to the error, in the order they occurred.

 /media/Makai/home/.local/share/krita/pykrita/krita_comfy/client.py in on_history_received(history_res={'fda29e90-398b-4d4f-b5ea-867de55cb34c': {'outputs': {'SaveImage': {'images': [{'filename': 'ComfyUI_00113_.png', 'subfolder': '', 'type': 'output'}]}}, 'prompt': [0, 'fda29e90-398b-4d4f-b5ea-867de55cb34c', {'3': {'class_type': 'KSampler', 'inputs': {'cfg': 7.0, 'denoise': 1.0, 'latent_image': [...], 'model': [...], 'negative': [...], 'positive': [...], 'sampler_name': 'euler', 'scheduler': 'simple', 'seed':<PHONE_NUMBER>6670115194, 'steps': 15}}, '6': {'class_type': 'CLIPTextEncode', 'inputs': {'clip': [...], 'text': 'a cat girl, best quality'}}, '7': {'class_type': 'CLIPTextEncode', 'inputs': {'clip': [...], 'text': 'worst quality, low quality'}}, 'CheckpointLoaderSimple': {'class_type': 'CheckpointLoaderSimple', 'inputs': {'ckpt_name': 'Anime/based64_v3.inpainting.safetensors'}}, 'ClipSetLastLayer': {'class_type': 'CLIPSetLastLayer', 'inputs': {'clip': [...], 'stop_at_clip_layer': -2}}, 'EmptyLatentImage': {'class_type': 'EmptyLatentImage', 'inputs': {'batch_size': 1, 'height': 512, 'width': 512}}, 'SaveImage': {'class_type': 'SaveImage', 'inputs': {'filename_prefix': 'ComfyUI', 'images': [...]}}, 'VAEDecode': {'class_type': 'VAEDecode', 'inputs': {'samples': [...], 'vae': [...]}}, 'VAELoader': {'class_type': 'VAELoader', 'inputs': {'vae_name': 'blessed2.vae.pt'}}}, {'client_id': '7d0f3bad-00ee-42f1-9d80-60202621b7d9'}, ['SaveImage']]}})
  256                 timer = QTimer()
  257                 timer.timeout.connect(check_if_populated)
  258                 timer.start(0.05)
  259
  260         if status == STATE_DONE or skip_status_check:
timer = <PyQt5.QtCore.QTimer object>
timer.start = <built-in method start of QTimer object>
TypeError: arguments did not match any overloaded call:
  start(self, msec: int): argument 1 has unexpected type 'float'
  start(self): too many arguments
    __cause__ = None
    __class__ = <class 'TypeError'>
    __context__ = None
    __delattr__ = <method-wrapper '__delattr__' of TypeError object>
    __dict__ = {}
    __dir__ = <built-in method __dir__ of TypeError object>
    __doc__ = 'Inappropriate argument type.'
    __eq__ = <method-wrapper '__eq__' of TypeError object>
    __format__ = <built-in method __format__ of TypeError object>
    __ge__ = <method-wrapper '__ge__' of TypeError object>
    __getattribute__ = <method-wrapper '__getattribute__' of TypeError object>
    __getstate__ = <built-in method __getstate__ of TypeError object>
    __gt__ = <method-wrapper '__gt__' of TypeError object>
    __hash__ = <method-wrapper '__hash__' of TypeError object>
    __init__ = <method-wrapper '__init__' of TypeError object>
    __init_subclass__ = <built-in method __init_subclass__ of type object>
    __le__ = <method-wrapper '__le__' of TypeError object>
    __lt__ = <method-wrapper '__lt__' of TypeError object>
    __ne__ = <method-wrapper '__ne__' of TypeError object>
    __new__ = <built-in method __new__ of type object>
    __reduce__ = <built-in method __reduce__ of TypeError object>
    __reduce_ex__ = <built-in method __reduce_ex__ of TypeError object>
    __repr__ = <method-wrapper '__repr__' of TypeError object>
    __setattr__ = <method-wrapper '__setattr__' of TypeError object>
    __setstate__ = <built-in method __setstate__ of TypeError object>
    __sizeof__ = <built-in method __sizeof__ of TypeError object>
    __str__ = <method-wrapper '__str__' of TypeError object>
    __subclasshook__ = <built-in method __subclasshook__ of type object>
    __suppress_context__ = False
    __traceback__ = <traceback object>
    add_note = <built-in method add_note of TypeError object>
    args = ("arguments did not match any overloaded call:\n  s...ed type 'float'\n  start(self): too many arguments",)
    with_traceback = <built-in method with_traceback of TypeError object>

The above is a description of an error in a Python program.  Here is
the original traceback:

Traceback (most recent call last):
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/client.py", line 258, in on_history_received
    timer.start(0.05)
TypeError: arguments did not match any overloaded call:
  start(self, msec: int): argument 1 has unexpected type 'float'
  start(self): too many arguments

Desktop:

OS: Linux
ComfyUI commit revision: 1141029a4ace2e21a7787a8ca6b3bf65cb6457bd
Extension commit revision: b4230543a1e5d096a43a69ae2ecdc5ae90399f07

The live preview thing is expected. It used to do something when it was an Automatic1111 plugin, the tab is useless right now (but I've been too lazy to remove it).
As for the issue, can you tell me what version of Krita are you using? I can't reproduce but I've noticed the failing function is a built in function, not really part of the plugin's code.

Sorry, I forgot.

Krita: 5.1.5-10
Distro: ArchLinux

I pushed some changes. Can you git pull, restart Krita and try again?

The error changed.

UnboundLocalError
Python 3.11.3: /usr/bin/python3
Fri Jul 28 12:51:52 2023

A problem occurred in a Python script.  Here is the sequence of
function calls leading up to the error, in the order they occurred.

 /media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py in cb(response={'info': {}, 'outputs': ['iVBORw0KGgoAAAANSUhEUgAAAgAAAAIACAIAAAB7GkOtAAAA...o+5jtiQSQU0aH21N//z+JVRBPBJ528QAAAABJRU5ErkJggg==']})
  298
  299             if self.cfg("hide_layers", bool):
  300                 for layer in layers[:-1]:
  301                     layer.setVisible(False)
  302
layer undefined
layers undefined
UnboundLocalError: cannot access local variable 'layers' where it is not associated with a value
    __cause__ = None
    __class__ = <class 'UnboundLocalError'>
    __context__ = None
    __delattr__ = <method-wrapper '__delattr__' of UnboundLocalError object>
    __dict__ = {}
    __dir__ = <built-in method __dir__ of UnboundLocalError object>
    __doc__ = 'Local name referenced but not bound to a value.'
    __eq__ = <method-wrapper '__eq__' of UnboundLocalError object>
    __format__ = <built-in method __format__ of UnboundLocalError object>
    __ge__ = <method-wrapper '__ge__' of UnboundLocalError object>
    __getattribute__ = <method-wrapper '__getattribute__' of UnboundLocalError object>
    __getstate__ = <built-in method __getstate__ of UnboundLocalError object>
    __gt__ = <method-wrapper '__gt__' of UnboundLocalError object>
    __hash__ = <method-wrapper '__hash__' of UnboundLocalError object>
    __init__ = <method-wrapper '__init__' of UnboundLocalError object>
    __init_subclass__ = <built-in method __init_subclass__ of type object>
    __le__ = <method-wrapper '__le__' of UnboundLocalError object>
    __lt__ = <method-wrapper '__lt__' of UnboundLocalError object>
    __ne__ = <method-wrapper '__ne__' of UnboundLocalError object>
    __new__ = <built-in method __new__ of type object>
    __reduce__ = <built-in method __reduce__ of UnboundLocalError object>
    __reduce_ex__ = <built-in method __reduce_ex__ of UnboundLocalError object>
    __repr__ = <method-wrapper '__repr__' of UnboundLocalError object>
    __setattr__ = <method-wrapper '__setattr__' of UnboundLocalError object>
    __setstate__ = <built-in method __setstate__ of UnboundLocalError object>
    __sizeof__ = <built-in method __sizeof__ of UnboundLocalError object>
    __str__ = <method-wrapper '__str__' of UnboundLocalError object>
    __subclasshook__ = <built-in method __subclasshook__ of type object>
    __suppress_context__ = False
    __traceback__ = <traceback object>
    add_note = <built-in method add_note of UnboundLocalError object>
    args = ("cannot access local variable 'layers' where it is not associated with a value",)
    name = None
    with_traceback = <built-in method with_traceback of UnboundLocalError object>

The above is a description of an error in a Python program.  Here is
the original traceback:

Traceback (most recent call last):
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py", line 300, in cb
    for layer in layers[:-1]:
                 ^^^^^^
UnboundLocalError: cannot access local variable 'layers' where it is not associated with a value

I made a new push. Update, restart and try again. A new message will probably show up, can you show me what it says?

AssertionError
Python 3.11.3: /usr/bin/python3
Fri Jul 28 13:27:30 2023

A problem occurred in a Python script.  Here is the sequence of
function calls leading up to the error, in the order they occurred.

 /media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py in cb(response={'info': {}, 'outputs': ['iVBORw0KGgoAAAANSUhEUgAAAgAAAAIACAIAAAB7GkOtAAAA...vgcdcq9H2l9LZ4LSssP8HTWIq5TMbw0AAAAAASUVORK5CYII=']})
  296             except Exception as e:
  297                 self.client.images_received.disconnect(cb)
  298                 assert False, e
  299                 return
  300
e undefined
AssertionError: Raw data size: 1048576, Expected size: 16777216
    __cause__ = None
    __class__ = <class 'AssertionError'>
    __context__ = AssertionError('Raw data size: 1048576, Expected size: 16777216')
    __delattr__ = <method-wrapper '__delattr__' of AssertionError object>
    __dict__ = {}
    __dir__ = <built-in method __dir__ of AssertionError object>
    __doc__ = 'Assertion failed.'
    __eq__ = <method-wrapper '__eq__' of AssertionError object>
    __format__ = <built-in method __format__ of AssertionError object>
    __ge__ = <method-wrapper '__ge__' of AssertionError object>
    __getattribute__ = <method-wrapper '__getattribute__' of AssertionError object>
    __getstate__ = <built-in method __getstate__ of AssertionError object>
    __gt__ = <method-wrapper '__gt__' of AssertionError object>
    __hash__ = <method-wrapper '__hash__' of AssertionError object>
    __init__ = <method-wrapper '__init__' of AssertionError object>
    __init_subclass__ = <built-in method __init_subclass__ of type object>
    __le__ = <method-wrapper '__le__' of AssertionError object>
    __lt__ = <method-wrapper '__lt__' of AssertionError object>
    __ne__ = <method-wrapper '__ne__' of AssertionError object>
    __new__ = <built-in method __new__ of type object>
    __reduce__ = <built-in method __reduce__ of AssertionError object>
    __reduce_ex__ = <built-in method __reduce_ex__ of AssertionError object>
    __repr__ = <method-wrapper '__repr__' of AssertionError object>
    __setattr__ = <method-wrapper '__setattr__' of AssertionError object>
    __setstate__ = <built-in method __setstate__ of AssertionError object>
    __sizeof__ = <built-in method __sizeof__ of AssertionError object>
    __str__ = <method-wrapper '__str__' of AssertionError object>
    __subclasshook__ = <built-in method __subclasshook__ of type object>
    __suppress_context__ = False
    __traceback__ = <traceback object>
    add_note = <built-in method add_note of AssertionError object>
    args = (AssertionError('Raw data size: 1048576, Expected size: 16777216'),)
    with_traceback = <built-in method with_traceback of AssertionError object>

The above is a description of an error in a Python program.  Here is
the original traceback:

Traceback (most recent call last):
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py", line 292, in cb
    layers = [
             ^
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py", line 293, in <listcomp>
    insert(name if name else f"{mode} {i + 1}", output)
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py", line 260, in insert
    assert expected == size, f"Raw data size: {size}, Expected size: {expected}"
           ^^^^^^^^^^^^^^^^
AssertionError: Raw data size: 1048576, Expected size: 16777216

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/media/Makai/home/.local/share/krita/pykrita/krita_comfy/script.py", line 298, in cb
    assert False, e
AssertionError: Raw data size: 1048576, Expected size: 16777216

Are you selecting the whole canvas? If not, can you try again but with the whole canvas selected?

I had nothing selected. It works when I select something.

Yes, that error is a product of a data check the previous mantainers had put in the code and sometimes (mostly always) it kicks when you don't have anything selected. Honestly I don't know exactly what purpose does it have and I'm debating with myself if I should just remove it.
In the meantime, I recommend to always select before generating something, unless you're upscaling or importing your last ComfyUI generation (which I made to skip the data check). If you want to work with the whole canvas, simply make a whole selection on it. Remember to set upscaler in Common Options tab if you want to work with bigger sizes and don't have base/max size disabled, otherwise the image will be plain resized and lose a lot of quality.

Thanks!

Thx for info about - having to select canvast for imgToImg to work. Btw, it would be great if plugin would detect that no pixels were selected, and it would select it for us @JasonS09


SW-2606 Add timeZone fields to search API

Allow clients to query the time zones of organizations, facilities, planting
sites, and users.

Current dependencies on/for this PR:

main

PR #877

PR #878

PR #879   👈

This comment was auto-generated by Graphite.


repl does not work anymore in release mode

Description
I’m not sure this was ever supposed to be working, however running the REPL in release mode does not work anymore.
Reproduction
cd your-swift-library
swift run -c release --repl

Expected behavior
In the REPL, one should be able to import the module and use it.
On macOS it is possible to import it, but it’s empty. On Linux the module is not found at all.
Environment
Swift version 6.0 (swift-6.0-RELEASE)
Target: aarch64-unknown-linux-gnu
and
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0
Additional information
Using the REPL in release mode might seem weird but it allows running Swift scripts  importing arbitrary modules compiled in release mode.
It is used by https://github.com/xcode-actions/swift-sh

Actually it works for my project. I had to fix the REPL invocation which seems incorrect (missing the /Modules path component in the -I option).


Unable to test with PhantomJS

Description
We have a Ruby on Rails application with integration tests using Poltergeist (PhantomJS) but we have some incompatibility issues.
One of the issues is that PhantomJS does not support ES6 features like Object.assign, to fix that we added a polyfill and it works.
The point is the following error with Ramda.js:
TypeError: reduce: list must be array or iterable

I don't know if it is related to lack of ES6 support from PhantomJS and we are unable to find what cause the error but we found that is related to Ramda :(
My Setup

Operating System: Ubuntu 16.04
Project Version: 3.1.2
NodeJS: 6.11.2
PhantomJS: 2.1.1

[x] I have tested with the latest version
[ ] I can simulate the issue easily

Current Behavior
Crashes when tries to run Pagarme-js lib with PhantomJS
Expected Behavior
Run tests using PhantomJS and be able to identify any ES6 feature that is used by Pagarme-js but isn't supported by PhantomJS or IE11, for example. No problem if we add polyfills to the ES6 features.

That's weird because in ramda's homepage (http://ramdajs.com/) they use PhantonJS to run tests

could you isolate the issue in a separate repo so I can clone it and check it out?

It's really weird this Ramda error. I'll try to create the repo!

I tried to reproduce as much as I can of my current environment at this repo: https://github.com/lucasrcdias/pagarmejs-phantomjs
I just want to point that I had an error with startsWith from ES6 with this repo and I added the startsWith polyfill.
In the end the error is the same:
TypeError: reduce: list must be array or iterable

I don't know if it helps but I searched for reduce in source files and it's used in:
validations/validate/card/brand.js
validations/validate/card/cardNumber.js
validations/validate/card/cnpjAndCpf.js -> I didn't validate cnpj or cpf.
I wanted to help but I'm not familiar with Ramda.js and functional programming :/


Allow to configure the head section of pages in Citation (aka SEO)

Informative purpose : for the front end part, https://github.com/nfl/react-helmet seems to fill the need :)


[wifid] iw command is missing (v0.3.5)

When I updated v0.3.5, the spr stopped working, so I proceeded with the full reset. (Both wired and wireless went wrong)
After that, I kept seeing wifid-related error messages on the admin screen, so when I checked the container log, there was a message as below.
maybe the 'iw' package is missing?
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
/scripts/startup.sh: line 33: iw: command not found
wlan0: interface state UNINITIALIZED->COUNTRY_UPDATE
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     6  100     6    0     0      3      0  0:00:02  0:00:01  0:00:01     3
wlan1: interface state UNINITIALIZED->COUNTRY_UPDATE
/scripts/startup.sh: line 33: iw: command not found

quick remedy:
docker compose exec -it wifid
apt update && apt install -y iw
then goto admin wifi menu -> "restart all wifi devices"

nice, thank you always for the super great spr.

fixed in v0.3.6


Switch from term to file and back

I noticed in the second gif that you switched from the term, opened the file, and then switched back to the term. How would one also achieve this functionality?

See nvr -h. That's the -l option, that simply uses :wincmd p to switch to the previously accessed window before opening the files.
Obviously this only works when nvr is run from within :terminal. :-)

Ahh thank you!! ❤

Hmm so I'm attempted to follow the same logic within the gif and can't recreate one part.
It looks like you enter the changelog in the current pane that held the terminal, with: nvr CHANGELOG.md. After that a shortcut must have been used to switch back to the term.
This is the flow I'm looking for: Open File in current terminal pane, edit/save, return to terminal in that same pane

You want everything to happen in the same window?
Well, terminal buffers are a bit special in some regards, but still buffers. So, after doing nvr CHANGELOG, you could simple write and delete that buffer with :w | bd, then it should switch back to the terminal buffer automatically. You just write and switch to the previous buffer with :w | bp.
If the problem is that your terminal buffers disappears after switching buffers, you might have to :set hidden.

This is perfect, I need to read more into buffers. Thank you!!


Update Script Tag Behavior

Current Situation
Currently the script tag always expects scripts to be structured as a effect hook...
// this is run on every render
() => {
  // this is run once the script is loaded and each time its content changes
  return () => {
    // this is run when the script is unloaded (i.e. it's removed from the tree) or just before its content changes
  }
}

Proposed Actions
The script tag should behave as a normal script by default
Utilize one of the following designs

A parameter to utilize a destructor.

It could possibly be a kwarg called destructor="...", which is the name of a JavaScript function within this script's scope that will be called on unload
Alternatively, could be a boolean flag such as returns_destructor to denote the script is a singleton function with a destructor

Instead of assuming the script is like an effect, we should be able to check if the script evals to a function, and if it does, we'll call it like an effect

Work Items

[ ] Develop a method to create normal scripts
[ ] Consider adding support for a src=... tag, if it does not exist

@rmorshea Was this resolved in #632 ?

Yup. Good catch.

Confirmed that it works great for my sign out (window.location.reload) button scenario!
https://github.com/Archmonger/Conreq/commit/28f4883575948754d6d6c6f1a4442020a20a61ec


💡 Rofi File Browser

Summary
rofi-file-browser-extended-git available in the AUR is a cool project to use rofi as a file browser.

https://aur.archlinux.org/packages/rofi-file-browser-extended-git
https://github.com/marvinkreis/rofi-file-browser-extended

set $rofi-file-browser rofi -show file-browser-extended -file-browser-resume -file-browser-follow-symlinks -show-icons -font "$gui-font" -lines 10 -width 35

bindsym $mod+Shift+a exec $rofi-file-browser

Drawbacks
No drawbacks really
Implementation idea
No response
Will you be able to contribute a PR for this?

[X] Yes I will be able to get that feature a lot quicker.

I'm not entirely convinced. I guess this isn't supposed to replace pcmanfm, but to be an additional tool?

well pcmanfm does have many extra features and the given package is a third party AUR one...i'll close this.


sdk-folder/third-party/portaudio/lib/.libs/libportaudio.a: No such file or directory

Hi!,please help me!!
I run this commond to build the SDK："cd /{HOME}/sdk-folder/sdk-build && cmake /{HOME}/sdk-folder/sdk-source/avs-device-sdk -DSENSORY_KEY_WORD_DETECTOR=OFF -DGSTREAMER_MEDIA_PLAYER=ON -DPORTAUDIO=ON -DPORTAUDIO_LIB_PATH=/{HOME}/sdk-folder/third-party/portaudio/lib/.libs/libportaudio.a -DPORTAUDIO_INCLUDE_DIR=/{HOME}/sdk-folder/third-party/portaudio/include && make"
But is a error  has occurred, it said:"libportaudio.a:No such file or directory", but my libportaudio.a is exists!
the log as follow:
//================
[ 96%] Linking CXX executable SampleApp
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
c++: error: ~/alexa/sdk-folder/third-party/portaudio/lib/.libs/libportaudio.a: No such file or directory
SampleApp/src/CMakeFiles/SampleApp.dir/build.make:359: recipe for target 'SampleApp/src/SampleApp' failed
make[2]: *** [SampleApp/src/SampleApp] Error 1
CMakeFiles/Makefile2:8010: recipe for target 'SampleApp/src/CMakeFiles/SampleApp.dir/all' failed
make[1]: *** [SampleApp/src/CMakeFiles/SampleApp.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
foryou@foryou-virtual-machine:~/alexa/sdk-folder/sdk-build$
//=================
Tell us about your environment:

What version of the AVS Device SDK are you using?
  newest SDK

Tell us what hardware you're using:

[ ] Raspberry Pi

Tell us about your OS (Type & version):

[ ] Linux-Ubuntu

Hi every please help me!
think you!!!

@cr1108x,
you forgot a $. {HOME} should be ${HOME} that is why there is no home directory on your portaudio directory lines.
Ryan

Thanks @ryan-esty.  @cr1108x - I'll close this for now - please re-open if you have further issues running those commands.

Thank you very much!!!


HMA364015: Edited the Readme file

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Raviteja Emandi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Potential segfault on Arduino Nano boards when running NoteRequest

Steps to reproduce:

Connect a Notecard to an Arduino Nano
Install the note-arduino library
Add the following code in a new sketch

#include <Notecard.h>
#define serialDebug Serial
#define serialNotecard Serial1

#define NOTE_PRODUCT_UID "com.my.test"
void setup() {
  serialDebug.begin(115200);
  while (!serialDebug) {
    ; // wait for serial port to connect. Needed for native USB
  }
  serialDebug.println("Starting...");
  NoteSetDebugOutputStream(serialDebug);
  NoteInitSerial(serialNotecard, 9600);
  J *req = NoteNewRequest("hub.set");
  JAddStringToObject(req, "product", NOTE_PRODUCT_UID);
  JAddStringToObject(req, "mode", "continuous");
  if (!NoteRequestResponse(req)) {
    serialDebug.println("Failed to send request!");
  }
}
void loop() {
  // put your main code here, to run repeatedly:
}

Upload the sketch and open the Serial Monitor.

Problem: In some environments, NoteRequestResponse doesn't return. The same is true of NoteRequest. This was observed in Linux but has not yet been reproed on Windows or Mac.

Reproduced on Mac with an Arduino Nano on 8/26

This has been fixed is is about to be merged in a PR. Due to other library changes, the sample to re-test has changed...
#include <Notecard.h>
#define serialDebug Serial
#define serialNotecard Serial1

Notecard notecard;
#define NOTE_PRODUCT_UID "com.my.test"

void setup() {
  serialDebug.begin(115200);
  while (!serialDebug) {
    ; // wait for serial port to connect. Needed for native USB
  }

  serialDebug.println("Starting...");

  notecard.setDebugOutputStream(serialDebug);
  notecard.begin(serialNotecard, 9600);

  J *req = notecard.newRequest("hub.set");
  JAddStringToObject(req, "product", NOTE_PRODUCT_UID);
  JAddStringToObject(req, "mode", "continuous");
  if (!notecard.requestAndResponse(req)) {
    serialDebug.println("Failed to send request!");
  }
}
void loop() {
  // put your main code here, to run repeatedly:
}


Origin header from client not being propagated causing CORs issues in browser

In the old Helius architecture, it appears that the Access-Control-Allow-Origin header provided by the RPC proxy was simply propagated back in the response from Helius, preventing CORs issues from occurring on browser-based client applications.
In the new backend architecture, however, the above is no longer the case. Instead, the new architecture now takes the Origin header as provided by browser-based clients and adds it as the Access-Control-Allow-Origin header.  In other words, the RPC proxy must either relay Origin headers provided by browser-based client applications, provide an Origin header when proxying browser-based client requests to Helius, or override response headers provided by Helius to include CORs headers (e.g., Access-Control-Allow-Origin) if no Origin is provided.
https://github.com/helius-labs/helius-rpc-proxy/blob/fb22b5ef35b17e09a277dbb1e9d82e9ecd942653/src/index.ts#L44-L52

Was this ever implemented? I dont see it in the code, but i see the last update is on the same date as this issue..


Missing Test Connection result in error using the custom connector in PBI service (with personal gateway)

please add the test connection function as defined here:
https://learn.microsoft.com/en-us/power-query/samples/trippin/9-testconnection/readme
example:
FortifyOnDemand = [
TestConnection = (dataSourcePath) => {"FortifyOnDemand.Contents"},
Authentication = [

here a commit on my branch
https://github.com/alex-zak/FortifyPowerQueryConnectors/commit/4070dd358b8e827660cbd29557eb682806b179c2

Hello @alex-zak . Appologies for the very delayed reply. I've since performed a large refactoring of this project.
Can you explain to be very quick that adding TestConnection does? After reading the documentaiton that you pointed out, it's not immediately clear to be what's going on.


Add Pep validation and name to output

This is most definitely draft quality code.

I think I will store the PepManager on the plugin

I would remove the whole thing about embeds/no embeds, since we can now get more useful information. Also if we put the result inside an embed, we can use masked links so the result is less ugly.

I think you're right tbh
Cc @trag1c

I would remove the whole thing about embeds/no embeds, since we can now get more useful information. Also if we put the result inside an embed, we can use masked links so the result is less ugly.

I think you're right tbh Cc @trag1c

:+1:

I also agree with trag1c's comment
Pep -> PEP PepManager -> PEPManager

You don't actually have to agree since Pep and PepManager don't follow PEP8 lol

Also, since we now have access to more detailed information, I think that all responses should be embedded. The Pep class could have two methods:
__str__: Returns the one-line information, using a masked link (instead of putting the links inside parenthesis)
embed: Returns a detailed embed of the Pep
Then auto PEP replies would return a single embed containing a line for each individual pep, and the command can return the detailed embed.

You don't actually have to agree since Pep and PepManager don't follow PEP8 lol

In that case ig they should be left as-is probably.

You don't actually have to agree since Pep and PepManager don't follow PEP8 lol

In that case ig they should be left as-is probably.

what? Pep and PepManager DON'T follow PEP8

what? Pep and PepManager DON'T follow PEP8

Then why did you say I didn't have to agree lol. Go with whatever follows pep8

what? Pep and PepManager DON'T follow PEP8

Then why did you say I didn't have to agree lol. Go with whatever follows pep8

because it doesn't matter whether you agree or not, renaming it is sort of a requirement in this case

I agree with @trag1c

Also, since we now have access to more detailed information, I think that all responses ...

I am against this because of how bulky embeds are. The button takes up enough space already.

I agree with @trag1c

Changed my mind. The Pep class looks like constant when its capital and I dont like that tbh.

I agree with @trag1c

Changed my mind. The Pep class looks like constant when its capital and I dont like that tbh.

Dude that's like naming your functions with camelCase???

For the class renaming discussion,
instead of Pep or PEP what about something like PEPObject or PEPItem?

Dude that's like naming your functions with camelCase???

I don't understand what you want me to do

Dude that's like naming your functions with camelCase???

I don't understand what you want me to do

"Rename Pep and PepManager to PEP and PEPManager"

Dude that's like naming your functions with camelCase???

I don't understand what you want me to do

"Rename Pep and PepManager to PEP and PEPManager"

If you don't like PEP looking like a constant then pick a different name as suggested by @Endercheif

Also, since we now have access to more detailed information, I think that all responses ...

I am against this because of how bulky embeds are. The button takes up enough space already.

I think it looks great.

I think it looks great.

yeah its better than i though actually

yeah its better than i though actually

Should I just push my changes

yes

Run black please

merging moment

I wonder who's gonna merge the fastest

me

wait for ci to pass

:why:

me

:trollface:


Add a test for using a PSK with QUIC

Check that we can set and use a PSK when establishing a QUIC connection.
Fixes https://github.com/openssl/project/issues/83
This also fixes a bug in the SSL_CIPHER_find() function which crashed when passed a QUIC SSL object.

@hlandau - feedback addressed. Please take another look.

This pull request is ready to merge

Merged to the master branch. Thank you.


Menu improvement

@RemoveIt has done a great work at #75
So now we can do some CSS to give the menu a better look. I suggest this design:

wow, already done?

How server location would be determined?

we can use a css framework
my list:
http://purecss.io/
http://imperavi.com/kube/
http://www.gumbyframework.com/
http://foundation.zurb.com/
http://getskeleton.com/
http://materializecss.com/
http://tuktuk.tapquo.com/
http://semantic-ui.com/
http://concisecss.com/
http://getuikit.com/
http://material-ui.com/#/
https://www.muicss.com/
and the framework used by agar.io:
http://getbootstrap.com/

@RemoveIt we can put a config or a variable in the code to tell the server location.

@Ariamiro Bootstrap is much better, but I'm afraid that it will increase the client size. If we use the simple design, we can write CSS manually. It much faster and keep the game size small. What do you think?

If this menu is only thing that needs styling we can stay without css framework

@huytd
like this:
var ServerLocation = US
var US = United States
we can use a style from a framework.

Currently, most of our code is in HTML5 Canvas, only this menu and the Chatbox are HTML elements for now.
@Ariamiro yes, just something like:
var serverLocation = "United States";
// or
var serverLocation = "US West";

you can see my css in: https://agar-io-clone-d3vont3ch.c9.io/
i am editing the css

Cool.
But you can give the #startMenu some padding.
Then the textbox and button width: 100% and box-sizing: border-box
Don't need the margin :)

ok.

My panel currently looks like this: https://agar-poznan.c9.io :)

good job.

Great :D we should put some instruction in the panel as @empty206 did

My game will be earmarked in Polish, so I took Polish descriptions.
English:
Available commands:
'-help'
'-dark'
'-ping'
2015 | All rights reserved.

i am going to restart my menu, you can use the empty206 menu if you want.

What about being able to "watch" the game behind the menu, with some transparency? Just so it's not that dark gray background?

With the current code, the game is not connected to server when the menu displayed yet :D
I pushed a commit. The menu now look like this.
If anyone want to improve it more, feel free to reopen this issue.


How about generalize it to any popular base?

See:
https://github.com/cryptocoinjs/base-x
Often in some domain areas users required Base58 (Bitcoin, Ripple and others crypto addresses)
Base85 also have standard (RFC 1924)

Thanks for the suggestion. I'm closing this a duplicate of #7; feel free to continue the discussion there, though for either of these to be included you'd need to make a case beyond "there is at least area which uses it".


I forgot how to deploy new version to github pages

^

Gonna leave this up cause I am going to forget how to this again later literally just run
npm run deploy
I already did the setup for this before a long time ago


Confirm delete of environments

On the manage environments dialog, the (X) delete button doesn't confirm deleting, which creates a bad user experience when it is accidentally clicked on.

We're working on a UI overhaul, and hope to release it soon. Stay tuned.

@rprince You can undo deletion of environments, collections, requests & folders in Postman 4.0. This should help you recover any accidental deletes.


Updating workflows in stable

Codecov Report

Merging #81 (935a279) into stable (c495060) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           stable      #81   +/-   ##
=======================================
  Coverage   95.07%   95.07%
=======================================
  Files          26       26
  Lines        1442     1442
=======================================
  Hits         1371     1371
  Misses         71       71

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 c495060...935a279. Read the comment docs.


fix(http1): only send 100 Continue if request body is polled

Before, if a client request included an Expect: 100-continue header,
the 100 Continue response was sent immediately. However, this is
problematic if the service is going to reply with some 4xx status code
and reject the body.
This change delays the automatic sending of the 100 Continue status
until the service has call poll_data on the request body once.
Closes #838

Do you have a plan to make the same thing possible with HTTP/2? According to HTTP specifications, an h2 server, like h1, must return a 100 continue response to Expect: 100-continue. Also, in practical terms beyond the spec, 100 continue allows for more precise control of body transmission timing than RST_STREAM. I need it right now and I'm thinking of creating a patch if there's no ongoing implementation or concerns. If you have any suggestions, please let me know.


Curious inconsistent render when using memoized selector direct reference

What is the current behavior?
When I have a selector created with reselect or re-reselect:
const getSomething = createSelector(state => state.value, value => value + 1);

There is seemingly a difference in re-rendering behavior when passing that selector directly to useSelector vs. wrapping it:
// What I want to do, seems fine per the docs:
const value = useSelector(getSomething);

// What I have to do instead:
const value = useSelector(state => getSomething(state));

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Demo repo: https://github.com/exogen/test-redux-selector
Run:
$ git clone git@github.com:exogen/test-redux-selector.git
$ cd test-redux-selector
$ yarn
$ yarn test

All the code lives in this file: https://github.com/exogen/test-redux-selector/blob/master/src/App.test.js
Here's what to look for when running the test:

The final flushed output of the render (as output using @testing-library/react) is consistent.
However, in the re-render output you'll see Found inconsistency? true because an inconsistent render was detected. This shows that you can see inconsistent state in effects, where you might be making decisions based on comparing certain state values etc. (this is why it's a problem in my real code)
You can also see the actual values during render via the console.log output.
I'm not sure why there are 3 renders in the first place, it seems like there should be two: the initial render, then the render with updated store. Instead, there's a 3rd render with inconsistent state from memoization in between.

I wonder, could it be because of my usage of redux-mock-store here? I'm re-rendering with a new store instance, not the same store instance with updated state – a bit weird, but also doesn't seem like it should cause inconsistency.
What is the expected behavior?
I would expect useSelector(selector) and useSelector(value => selector(value)) to have equivalent behavior.
Which versions of React, ReactDOM/React Native, Redux, and React Redux are you using?
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>
Thanks! 🙇

Hmm.  For this test at least, I would advise against providing a different store entirely.  Instead, try dispatching an action that changes the value, and also wrap that behavior in act(), like:
act(() => {
    store.dispatch({type: "INCREMENT"});
});

Does that change the rendering results at all?

Yes, that works!
I wonder if it's just unsafe to use redux-mock-store with rerendering altogether then, as it intentionally doesn't supply any way to update state.
One reason I use it is so that I don't have to specify a reducer or dispatch any actions in my tests, for when I just want to see what a component does with various states as input. What's the recommended pattern for easy testing like this, should I mock useSelector instead so I don't need a store at all?

Looking at the redux-mock-store docs, it says:

A mock store for testing Redux async action creators and middleware. The mock store will create an array of dispatched actions which serve as an action log for tests.  Please note that this library is designed to test the action-related logic, not the reducer-related one

I think the answer here is that you really should just create a "real" Redux store, even if it's got a test-only reducer in it, because the point is to have a Redux store that can update its state.  You're not trying to actually check what actions were dispatched and assert on those.

I do use that functionality as well and it's nice to just be able to use similar test code everywhere, but it should be just as easy to replicate by doing:
store.dispatch = jest.fn();

and spying on its calls, so no big loss there.

Right, I'm more saying that for this particular test, you'd want a real store instead of a mock one.

Closing as this appears to be resolved.


[ctre] Update to 3.4.1

Describe the pull request

What does your PR fix?
Updates CTRE to 3.4.1

Which triplets are supported/not supported? Have you updated the CI baseline?
Just an update

Does your PR follow the maintainer guide?

Yes

The 'paraview' breaks due CI basline error,just wait CI error fixed.

/azp run

Thanks for the update!


test: add line break scenario

There was a gap in the coverage when it comes to rendering with line break. This PR aims to improve that.

In two snapshots, totals row is exceeding the table wrapper toward right, seems like something is not going correct in those cases:

In two snapshots, totals row is exceeding the table wrapper toward right, seems like something is not going correct in those cases:

Yes that is a bug that should be fixed. But will not do it as part of this PR


Logos of games

Icons Request
Icons name:

Overwatch
League of Legends
Counter Strike : Global Offensive
Unreal tournament
Quake
Rocket League
Starcraft 2
DOTA 2
Super smash bros. ultimate

Use case: Develop a platform of gaming.

I'm not sure some of the game logos can be translated to icons...
Em... em...

I understand. You can find some icons of game logos in icons8.com. Maybe it can help you create one.
https://icons8.com/icon/set/unreal/ios
https://icons8.com/icon/set/overwatch/ios
https://icons8.com/icon/set/league-of-legends/ios
https://icons8.com/icon/set/dota-2/ios

OK, I can try some game logos.
The problem is Remix Icons are based on a 24*24px grid, so there are only a few pixels can be used, while the game logos contain many details.

 Even if you can't do it, it's really cool of you to try.

Maybe it's just me, but I feel like this is not the right repo for game icons. All the other icons are for making web applications, more or less.
Game icons may be a good idea, I just think that it's the wrong place for it.

I agree @jenstornell there are millions of games and even if you picked the top few they might involve some very complicated shapes for a very small audience, however I would add probably the following

Unreal (https://cdn.worldvectorlogo.com/logos/unreal-1.svg)
Unity (https://images.techhive.com/images/article/2015/03/unity-logo-100571261-large.jpg)

If any game related icons were to be added


Provide educates metrics for workshops

Currently we only have access to Google Analytics/DataStudio and Amplitude. Those are great for seeing how many hits the landing page for each workshop gets.
They do not provide any metrics about what folks are doing within the workshop like:

How far do they get?
Are there steps they get stuck on?

Having the workshop data would allow us to iterate to make the workshops better without direct feedback (which we would also like via #742).

We did just merge #746, which collects events around when workshops are started, finished, abandoned, and an event as they progress through the workshop (ie. If they move on to the next step, an event is emitted to say they're 7% through the workshop). Does this combined with #742 achieve what you're looking for here?

@BrianMMcClain Quite possibly.
Where might I find a report that includes those metrics so I can take a look and answer that question?

Ideally a dashboard broken down by workshop would be ideal.
Some ideas for metrics by workshop:
% completion
step that most people fall out of the workshop at.

We have this now.


Documentation needs more examples.

The documentation is not covering all scenarios. It would be great if you can add more details with example codes

Hello @XOneto9  - I'm embarrassed to say I'm not sure what I did that you agree with :) Could you point me at whatever it is you are referencing so I can refresh my memory?

@mattmassicotte
OMG.. I’m so sorry it is mistake to refers you,
I think it is mistype on the writing my opinion :(
Raaaaally soorrrrrry about that;;

@XOneto9  Oh, really don't worry about it! Mystery solved :)

Just some thoughts on what could be useful as documentation. I've come from SnapKit (too cumbersome) via Stevia (too many APIs - not easy to know which one to use) to TinyConstraints, and I managed to convert my simple App (3 ViewControllers, about 6 or so views in each) to using TinyConstraints in just over an hour, so I'm very impressed! The Example App is excellent and shows what can be done. I must admit though, it's a bit over my head at the moment.
The docs at the moment give a good overview of what is possible. I think what would be very helpful is recipes, here's two for example;

How to align two (or more) views by their bottom edge or baseline (or any other edge). Is it possible to have an array of views all aligned by the same edge? If it can only be done by viewA.top == viewB.top, viewB.top == viewC.top, that's fine. It's just not obvious at the moment (the ==, or equation based API comes from Stevia, which is very good, but it has three API's: visual layout, chainable and equation-based and it's not clear when to use each one).
How to evenly space out some views (eg 4 buttons on the vertical/horizontal axis).

Thanks for a great framework, whilst writing this out, I realised that something I was wracking my brain with (collection of a image-label-button view into its own subclass of UIView to allow it to be used repeatedly) could be done using the 'stack' method, and so I implemented that - works very well! (I couldn't find it being used in the Example App though).
Hope this feedback is constructive to you.

If you're looking for an in depth video documentation here it is: https://www.youtube.com/watch?v=ucbaEkEJKrc&list=PL_csAAO9PQ8ZHY2VzOi7LJ80CtfAgCgHP


chart.yaml missing in resource file

Hi,
Half way the lecture (https://youtu.be/aD5_OA3IEaA?si=HbuoUOlnU_WIj1Uh) you add the chart.yaml inside the /infrastructure/base/nginx/kustomization.yaml file.
However it is not inside the repository here. (or perhaps this is removed later in a lecture as it is redundant in that case, please close this.
Many thanks for this valuable lessons on flux! enjoying it a lot!!
Wes

ok.. sorry I was to early :-) there is a short cut for it :)


Test Tweedie normalization with Wright-Bessel

With y > 100 or power = 1.01, scipy.special.wright_bessel returned infinities, so I couldn't test those.
Closes #481. @lorentzenchr

Just as info, I did not open an issue in scipy because of the following:
from scipy.special import wright_bessel

y = 6
power = 1.999
dispersion = 1
alpha = (2 - power) / (1 - power)  # -0.0010010010010008906
x = (((power - 1) / y) ** alpha) / ((2 - power) * (dispersion ** (1 - alpha)))  # 1001.7961657008032

wright_bessel(-alpha, 0, 0.7 * x)

gives
2.7524526105751842e+304

And note that
import numpy as np
np.finfo(np.float64).max

gives
1.7976931348623157e+308

Conclusion: no obvious flaw in wright_bessel


A large number of workflows causes ES index to fail

Bug Report
This is somewhat speculative, but it appears that a large number of published workflows can cause an ES reindex operation to fail. Due to #2351 , it's hard to tell exactly what is going on, but I:

Ran code that generated and published an additional ~3,000 workflows. 12,000 workflows total -- the rest are unpublished.
Ran the suite of perf tests, which ended up crashing the webservice
Restarted Dockstore with docker-compose
Tried to reindex -- I get a 500 error:

{
  "code": 500,
  "message": "There was an error processing your request. It has been logged (ID e36bcf1b77097f61)."
}

┆Issue is synchronized with this Jira Bug
┆Fix Versions: Dockstore 1.11
┆Issue Number: DOCK-663
┆Sprint: Backlog
┆Issue Type: Bug

After restoring the database without the generated test workflows, the es reindex worked. So, probably the number of workflows, although I suppose something could have been in a funny state given the crash.

Locally, generated 14,322 workflows, of which ~3,500 were published, and it worked.
Our next problem some day will be that we won't be able to load all the workflows into memory, but that's a ways off, and in any case this fixes it on the ES side.


refactor: Replace floodsub with gossipsub

This PR replaces floodsub with gossipsub. Left it as a draft as a starting point to get a bit more insight as to if this is the right starting point to proceed from there. Will be going back after to comment on everything as well that needs commenting or if comments needs adjustments to reflect changes.
The only issue right now, which I may just be looking over (due to my lack of time in debugging it) is that while I am able to subscribe to the topic, no messages are being published to it nor is the stream receiving anything in the topic.
This [Draft] PR supersede #489
Checklist (can be deleted from PR description once items are checked)

[ ] New code is “linted” i.e. code formatting via rustfmt and language idioms via clippy
[ ] There are no extraneous changes like formatting, line reordering, etc. Keep the patch sizes small!
[ ] There are functional and/or unit tests written, and they are passing
[ ] There is suitable documentation. In our case, this means:

[ ] Each command has a usage example and API specification
[ ] Rustdoc tests are passing on all code-level comments
[ ] Differences between Rust’s IPFS implementation and the Go or JS implementations are explained

[ ] Additions to CHANGELOG.md files

Should we go on and pass through the gossip result? Eg Gossipsub::publish returns Result<MessageId, PublishError>. Should Pubsub::publish and IpfsEvent::PubsubPublish also use the same return? Although the test is failing on a single test (which I will have to dig into soon or probably just change the test although I shouldnt need to change anything, will find out here shortly), testing it code wise it works just fine once the peers are connected to eachother.


SublimeLinter 4 and improvements

Hi!
This Pull Request is a follow up of the Pull Request #2 of @schuylr. It brings various fixes and improvements:

Support for SublimeLinter 4 (current)
SublimeText Project is no longer required, the linter will detect its context also on open folders and files
Updated Readme.md including examples

I hope to hear from you to bring this package forward. Please let me know if you are okay with my changes or if you expect any adjustments.
Thank you very much!

Just a little heads-up. This package seems to be abandoned. We intend to replace its entry in Sublime Text package control with a fork that will be made compatible with SublimeLinter 4.

Beside the code review I have a question.  This linter adapter registers for a lot of languages, are these supported out of the box actually, or downloaded and updated on the fly? For example they download some code on first request.  (Iirc haskell is a GB on my machine.)
The docs state that piping in the source code is supported.  Why is this here a saved-file only linter?

how we actually take/transition the repo

We need to change the package control entry to point to your fork (so create a PR there!) and then in your fork you need to create a release that's > 1.0.0 (so 2.0.0 or 1.1.0, your pick).

how we actually take/transition the repo

We need to change the package control entry to point to your fork (so create a PR there!) and then in your fork you need to create a release that's > 1.0.0 (so 2.0.0 or 1.1.0, your pick).

Hi @kaste
As far as I understand, codeclimate does not install engines automatically. From the documentation:

As part of setting up your project, we recommend running codeclimate engines:install from within your repository before running codeclimate analyze, and after adding any new plugins to your configuration file.

I did a set of simple performance tests and think it is not that good to run the codeclimate CLI on each random file that will be opened in SublimeText.
I suggest removing all selectors from the defaults and letting the user decide which selectors they want to use and do these settings globally or in a SublimeText Project File.
From my point of view, we're now talking about changes that go beyond this pull request. It is no longer just about compatibility with SublimeLinter 4, but about further interventions. Therefore, I have decided to leave this pull request as it is and make the changes in a new branch in my repository. I've added you as a maintainer to my repository and as a reviewer to the corresponding pull request, which you can find here.

Hi @kaste
As far as I understand, codeclimate does not install engines automatically. From the documentation:

As part of setting up your project, we recommend running codeclimate engines:install from within your repository before running codeclimate analyze, and after adding any new plugins to your configuration file.

I did a set of simple performance tests and think it is not that good to run the codeclimate CLI on each random file that will be opened in SublimeText.
I suggest removing all selectors from the defaults and letting the user decide which selectors they want to use and do these settings globally or in a SublimeText Project File.
From my point of view, we're now talking about changes that go beyond this pull request. It is no longer just about compatibility with SublimeLinter 4, but about further interventions. Therefore, I have decided to leave this pull request as it is and make the changes in a new branch in my repository. I've added you as a maintainer to my repository and as a reviewer to the corresponding pull request, which you can find here.

Hi @braver, thank you very much for your update. I will file my Pull Request once @kaste has proven the changes here.

Hi @braver, thank you very much for your update. I will file my Pull Request once @kaste has proven the changes here.

Hi @braver, my fork is now ready to replace the original package. I filed my pull request to overtake the responsibility.
@codeclimate, I'm open to giving you back the control over this package in reverting the package control index. If you are not interested, I would appreciate a deprecation notice or something similar at your repository's README.md.
Thank you very much for developing the codeclimate CLI. It is an excellent tool and helps me every day to improve my code.

Hi @braver, my fork is now ready to replace the original package. I filed my pull request to overtake the responsibility.
@codeclimate, I'm open to giving you back the control over this package in reverting the package control index. If you are not interested, I would appreciate a deprecation notice or something similar at your repository's README.md.
Thank you very much for developing the codeclimate CLI. It is an excellent tool and helps me every day to improve my code.


ZXCapture stops working after showing UIImagePickerController

I've encountered a strange issue with using ZXCapture.
After showing a UIImagePickerController in camera mode (i.e. with sourceType = UIImagePickerControllerSourceTypeCamera) no QR code is getting captured with ZXCapture, my delegate method isn't  #called.
I'm using the same code as in BarcodeScanner example with some parts removed. It works fine otherwise.
Are there any known issues with using the two?

After some debugging I found that it still captures QR codes but only through a very small area of the screen in the top left corner.
Still trying to figure out why. scanRect is not changing across captures (I checked) so I'm kind of confused...

Adding this line:
self.session.sessionPreset = self.sessionPreset;

to the start method of ZXCapture appears to fix this issue.


Any plans to release for Linux?

yes, and also on MacOS but I need to install the whole thing, and I've never installed Linux and software yet. I also need to figure how to compile it on both of those machine too ...

Great! Hit me up if you need some help with the Linux version.


[scatter] allow select country for Europe

also populate place type options after place has been selected

For now, the place type options are general to the selected place type which means some of these options may not be real options for the selected place.
eg.

Lay audience question:  will we at some point make it like map tool, which seems to know better:

Lay audience question: will we at some point make it like map tool, which seems to know better:

I think we definitely should and we could, but scatter is a little harder to tell what child places are possible for a selected place. For example, with Europe, it looks like there actually are stat vars for AdministrativeArea1/2 in Europe


Sonic the Wizard Werehog: Watch missayisha.in

Sonic the Wizard Werehog requests the watch of the watch_keyword missayisha\.in. See the MS search here and the Stack Exchange search here.

missayisha\.in has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Being used in the "website" field of a spam user.

!!/approve

!!/approve


[libconfig] Export correct cmake config

Fix https://github.com/microsoft/vcpkg/issues/30906

Note: Export correct cmake config files and usage:
libconfig provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(libconfig CONFIG REQUIRED)
    target_link_libraries(main PRIVATE libconfig::libconfig)

    find_package(libconfig++ CONFIG REQUIRED)
    target_link_libraries(main PRIVATE libconfig++::libconfig++)

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

From the upstream code structure, it was originally intended to export the cmake configuration for libconfig and libconfig++ .
https://github.com/hyperrealm/libconfig/blob/4f13b7f5152427b5e511a09617ca2d9ff63f7800/lib/CMakeLists.txt#L144

I think the patch is completely invalid. I still wait for an answer to #30906 (comment).

CMakeLists.txt:
cmake_minimum_required(VERSION 3.0)
project(test)
add_executable(main main.cpp)
set(CMAKE_FIND_DEBUG_MODE 1)
find_package(libconfig CONFIG REQUIRED)
target_link_libraries(main PRIVATE libconfig::libconfig libconfig::libconfig++)

CMake Debug Log at vcpkg/scripts/buildsystems/vcpkg.cmake:852 (_find_package):
  The internally managed CMAKE_FIND_PACKAGE_REDIRECTS_DIR.

    E:/usage/build/CMakeFiles/pkgRedirects

  <PackageName>_ROOT CMake variable [CMAKE_FIND_USE_PACKAGE_ROOT_PATH].

    none

  CMAKE_PREFIX_PATH variable [CMAKE_FIND_USE_CMAKE_PATH].

    E:/usage/vcpkg/installed/x64-windows/debug
    E:/usage/vcpkg/installed/x64-windows

  CMAKE_FRAMEWORK_PATH and CMAKE_APPBUNDLE_PATH variables
  [CMAKE_FIND_USE_CMAKE_PATH].

  Env variable libconfig_DIR [CMAKE_FIND_USE_CMAKE_ENVIRONMENT_PATH].

    none

  CMAKE_PREFIX_PATH env variable [CMAKE_FIND_USE_CMAKE_ENVIRONMENT_PATH].

    none

  CMAKE_FRAMEWORK_PATH and CMAKE_APPBUNDLE_PATH env variables
  [CMAKE_FIND_USE_CMAKE_ENVIRONMENT_PATH].

    none

  Paths specified by the find_package HINTS option.

    none

  Standard system environment variables
  [CMAKE_FIND_USE_SYSTEM_ENVIRONMENT_PATH].

    C:/Program Files/Microsoft MPI/Bin
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.5
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.5/libnvvp
    C:/VulkanSDK/<IP_ADDRESS>/Bin
    C:/Windows/System32
    C:/Windows
    C:/Windows/System32/wbem
    C:/Windows/System32/WindowsPowerShell/v1.0
    C:/Windows/System32/OpenSSH
    C:/Program Files/Microsoft SQL Server/130/Tools/Binn
    C:/Program Files/TortoiseGit
    C:/Program Files/Git/cmd
    C:/Program Files (x86)/NVIDIA Corporation/PhysX/Common
    C:/Program Files/NVIDIA Corporation/Nsight Compute 2021.3.1
    C:/Users/test/AppData/Local/Programs/Python/Python38-32/Scripts
    C:/Program Files/dotnet
    C:/Program Files (x86)/IncrediBuild
    C:/Program Files/CMake
    C:/Program Files/Microsoft SQL Server/150/Tools/Binn
    C:/Program Files/PuTTY
    C:/Users/test/.cargo
    C:/Users/test/AppData/Local/Microsoft/WindowsApps
    C:/Users/test/.dotnet/tools

  CMake User Package Registry [CMAKE_FIND_USE_PACKAGE_REGISTRY].

    none

  CMake variables defined in the Platform file
  [CMAKE_FIND_USE_CMAKE_SYSTEM_PATH].

    C:/Program Files
    C:/Program Files (x86)
    C:/Program Files (x86)/test

  CMake System Package Registry
  [CMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY].

    none

  Paths specified by the find_package PATHS option.

    none

  find_package considered the following locations for libconfig's Config
  module:

    E:/usage/vcpkg/installed/x64-windows/debug/libconfigConfig.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig-config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfigConfig.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig-config.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfigConfig.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfig-config.cmake
    E:/usage/vcpkg/installed/x64-windows/share/libconfig/libconfigConfig.cmake

  The file was found at

    E:/usage/vcpkg/installed/x64-windows/share/libconfig/libconfigConfig.cmake

Call Stack (most recent call first):
  CMakeLists.txt:5 (find_package)

-- Configuring done
CMake Error at CMakeLists.txt:6 (target_link_libraries):
  Target "main" links to:

    libconfig::libconfig++

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

-- Generating done

A find_package call is missing for an IMPORTED target.

find_package(libconfig++ CONFIG REQUIRED)
Cf. https://github.com/microsoft/vcpkg/issues/30906#issuecomment-1515790645

you must call find_package(libconfig++) if you want to use the C++ API.

find_package considered the following locations for libconfig++'s Config
  module:
...
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++Config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++-config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++Config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++-config.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfig++Config.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfig++-config.cmake
    E:/usage/build/CMakeFiles/pkgRedirects/libconfig++Config.cmake
    E:/usage/build/CMakeFiles/pkgRedirects/libconfig++-config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++Config.cmake
    E:/usage/vcpkg/installed/x64-windows/debug/libconfig++-config.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfig++Config.cmake
    E:/usage/vcpkg/installed/x64-windows/libconfig++-config.cmake
...
  The file was not found.

Do not load share.
Looks like CMAKE_ PREFIX_ PATH did not pass share.

... but you had E:/usage/vcpkg/installed/x64-windows/share/libconfig/ for libconfig.
But wait, you are right, CMake won't consider E:/usage/vcpkg/installed/x64-windows/share/libconfig for package name libconfig++...
It works the other way: CMake would consider E:/usage/vcpkg/installed/x64-windows/share/libconfig++  for package name libconfig...
So as a more simple fix, we might just pass PACKAGE_NAME libconfig++ to vcpkg_cmake_config_fixup.
But in the end, upstream must say how they want to resolve the inconsistency.

But in the end, upstream must say how they want to resolve the inconsistency.

Submit the simplest repair patch and upstream confirmation is required

Closing this PR. Feel free to open a new PR if you are still working on this or there is more progress. Thanks!


Patch links, and fix a couple punctuation errors

While I was preparing to get swift docs setup, I noticed that links are broken; So I fixed them.

So looking back on this, I realize that changing the links would cause unintended behaviour on the website end. Is there a good way to have the links work on both the deployment and the repository? If not, I would be happy to undo the link modifications in this PR and refocus it on punctuation and grammatical touch-ups.

Crud, thought I had cherry picked out the links - I may just have broken links in prod... one sec


Rename findAny to findFirst

Because findAny can return either lucas or cooper.

@piotrek1543 Thanks for the review.
I was wondering whether to fix the test to pass both cases of lucas and cooper, but I thought it may not be good that the return value of the method is indefinite in the first place.
@svtk I'll leave it up to you.


update links (uk, uz lang)

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
https://github.com/yiisoft/yii2/issues/19287

Thanks


Add new symbols introduced in iOS 11.1 🧡  ⏏

This pull request adds the two new symbol emojis introduced in iOS 11.1:

[x] 🧡 Orange Heart
[x] ⏏ Eject Button

@muan: I think I've followed all the existing conventions when adding these new emojis, so I'm going to go ahead and merge these changes and open one more pull request to add the remaining new emojis. If you see anything that needs additional attention, please let me know and I'll happily take care of it. 🙇


Option for pause

Given the game's fast-paced nature and real life's unpredictability, one may need to pause sometimes without breaking the streak.
Conventiently, there's a master key – [Space] – the use of which is already congruent with "affect gameplay". Adding the option to use it to pause while mid-round should benefit the player and have no drawbacks (aside from a slight increase in infrastructure difficulty, which is worthwhile).
On mobile devices (#24), the same direction of swiping that starts the round over could be used for pausing.
During pause, keys must be hidden to prevent unfair play.
Special background effect may be added to clearly indicate paused state. Proposed effect: diagonal stripes moving at a moderate speed.

The barbercone effect for pause has been tested and declined. It used a surprising amount of GPU, which goes against the innate philosophy of password being a simple game in design.


[Pending - fixing build issues] Upgrade Microsoft.VisualStudio.SDK version to 17.0 preview

Currently Microsoft.VisualStudio.SDK 15.0 references to old PIA assemblies which are incompatible with VS 2022.
To support latest Visual Studio 2022 need to upgrade the VS SDK to 17.0.
Fixed couple API changes due to the SDK version upgrading.
To support both VS 2022 and previous version of VS, refactor the projects:

Create a shared project Sarif.Viewer.VisualStudio.Core.
Move all C-Sharp code files into shared project.
Update existing VSIX project Sarif.Viewer.VisualStudio to only build up to VS 2019 and update VS SDK reference to 16.0
Add a new VSIX project Sarif.Viewer.VisualStudio.2022 which references to VS SDK 17.0
Both 2 VSIX projects reference to shared project Sarif.Viewer.VisualStudio.Core.
Both 2 VSIX projects build .vsix files, 1 for VS 2022 and another for previous version.

Close this sending #432 instead.


Audio device with 6 channels

Thank you for the informative tutorial.
Do you happen to know if there is a way to setup the layout/platform in such a way that macOS would see an audio device with 6 output channels (5.1 audio) rather than 2 output channels (Left/Right Stereo)

It works the same, just more outputs you have to rout. Have a look at Desktop Layouts which support 5.1 audio mappings for reference. Layout 17 of ALC1220 for example:


cannot install sungrow add-on

Failed to install add-on
Unknown error, see supervisor logs
Logger: homeassistant.components.hassio
Source: components/hassio/websocket_api.py:138
integration: Home Assistant Supervisor (documentation, issues)
First occurred: 12:26:06 AM (2 occurrences)
Last logged: 12:28:21 AM
Failed to to call /addons/ba22da74_gosungrow/install -

Hello is this integration still supported? can not install in HA 2024 8.1


The chart has too much space between donut chart and legends

Bug report
Codepen
https://codepen.io/berkbaski/pen/ZEKQqop
Explanation
Some android devices have too much space between donut chart and legends. The iOS devices I tried never had this problem. I don't know how to fix this.
Should be like this (iOS)

This is how it works on problematic devices.
Device info: Huawei Mate 20 Lite
Native browser

Chrome

Could someone add labels?

Could someone help me?

solution?


PMID:18344406 increased activation of S-phase DNA damage checkpoint, histone acetylation

[x]  increased activation of S-phase DNA damage checkpoint

child of FYPO:0000007 | abnormal S-phase DNA damage checkpoint

normal histone H4-K16 acetylation during vegetative growth
increased histone H3-K56 acetylation during vegetative growth

normal  histone H3-K56 acetylation during vegetative growth


[FEAT-REQ] Add the lib in the React Native Directory

Hello,
I think it's a good idea to add the library in the React Native Directory.
Now expo with SDK 52 check all the libs againt this directory to know if they support the newArchitecture and fail with a warning if the lib is not found in the directory.
Check can be disabled for a specific lib but I think it's a good thing to enable it, because add the library in the directory is very easy
How to : https://github.com/react-native-community/directory/?tab=readme-ov-file#how-do-i-add-a-library

Hey there! 👋
Thanks for bringing this up and sharing the link! This definitely sounds like a good idea, especially with Expo SDK 52 checking for libraries in the React Native Directory. I’ll check it out and work on adding the library soon.
Really appreciate you pointing this out! 🙏

Hey there! 👋
I had to do an MR on the directory, so I added the package at the same time.  : https://github.com/react-native-community/directory/pull/1389

Hey there! 👋
Thank you so much for taking the time to raise the PR and for adding the package to the directory! 🎉 I’m really glad to see it’s already been merged. This is super helpful and much appreciated! 🙏
I’ll go ahead and close this issue now. Thanks again for your contribution and support! 😊


Adding support for accounts from configured networks

[x] add m.accounts: string[] to the deployment builder for easy access inside a module (no more hardcoding required!)
[x] updated examples to remove hardcoded accounts in favor of m.accounts
[x] add a from option to relevant deployment actions to allow specifying which account to send the transaction from
[x] added docs and tests for the new features
[x] (also fixed the --force flag)

fixes #135

I am going to merge this PR to reduce the number of open PRs that will be affected by the tsconfig refactor PR.


Feature/separates indexer ingress

Purpose

How does the solution address the problem

Changes made

prnet cleaned-up successfully


Serve immediate 404 for static files (no fallthru)

Need to take this for a proper spin, but I'm quite excited by this improvement!
closes #5887

upgrade express to 4.13.*, which has serveStatic 1.10.0, and in turn has the new 'fallthrough' option
change local-file-store and middleware serving /public/ and /shared/ files to use the new fallthrough: false option
404s are now served directly, without slashes or uncapitalise getting triggered

This is ready for primetime.
There are some additional todos:

handle the overridable static files like favicon & robots.txt better (sort of covered by #4869)
make this same optimisation for theme static files

I'll raise these as separate issues, with an idea of what to do.

This test failure seems to be genuine :-/.

The error I saw was the same as the one reported in #6550
Uncaught TypeError: Cannot read property \'get\' of undefined at http://localhost:7357/assets/ghost.js, line 8731\n'
It's an ongoing issue but not related to this PR


Update request: azure-functions-core-tools 4.0.5455 → 4.0.5611

Package name: azure-functions-core-tools
Latest released version: 4.0.5611

Current version on the unstable channel:  4.0.5455
Current version on the stable/release channel: 4.0.5348

[x] Checked the nixpkgs pull requests

Notify maintainers
@Detegr @mdarocha - I've communicated with @mdarocha via email.

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Fixed by https://github.com/NixOS/nixpkgs/pull/361077


Update doobie-core, doobie-h2, ... to 0.13.0

Updates

org.tpolecat:doobie-core
org.tpolecat:doobie-h2
org.tpolecat:doobie-refined

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

Files still referring to the old version number
The following files still refer to the old version number (0.10.0).
You might want to review and update them manually.
project/plugins.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.tpolecat" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #253.


[v2] Uncaught Invariant Violation: Maximum update depth exceeded

I'm sorry I can't give you a clean reproduction on this; it only happens when using Cypress to automate my tests, and even then, only in particular circumstances involving a lot of prior tests having been run.
The React exception message looks like so:

(Ignore the latter paragraphs; the React exception message is the entirety of the first paragraph.)
My suspicion is that my application's heavy use of MobX and controlled props is causing a feedback loop, as evidenced by this snippet of my own application code:

I can't explain why the feedback loop only happens while running automated tests ... perhaps you or quill have got some idempotency/loop-detection type stuff and it's being foiled in that environment.
I am currently rewriting the offending components of my app to be simpler and uncontrolled, which will likely work around whatever issue is occurring. I wanted to raise this issue with you, however, since ReactQuill v2 is in its beta period and you might have an "ah-hah" realization about how to prevent this feedback loop.
If I have any more time to dedicate to it, I will try to create a standalone repro for you.

Are you passing the Quill delta back into the component instead of the HTML? There is a warning about that.

Hi @xeger, could you take a closer look at your initial self.value you’re passing to the component’s value?
I encountered a similar problem with a re-render loop. Turns out it is related to line-breaks, especially between paragraphs and lists. I could reproduce the issue with the following minimal example:
import React, {useState} from 'react'
import ReactQuill from 'react-quill'

export const ReactQuillTest = () => {
  const [testText, setTestText] = useState("
Lorem

ipsum

")

  return <>
    <ReactQuill value={testText} onChange={(html) => {console.log(testText, html); setTestText(html)}} />

{testText}

  </>
}

Here, the component keeps inserting

 before the
:

Lorem

ipsum

Lorem

ipsum

Lorem

ipsum

…

BTW: What solved it for me was to use ReactQuill in uncontrolled mode. Documentation mentions controlled mode caveats.

Interesting; I copy-pasted @smnscp's reproduction into my source file in place of my own ReactQuill invocation, and the problem didn't reproduce.
There are some similarities between me and you: I'm using ReactQuill in controlled mode; I am both consuming and value='ing the Quill HTML content, as opposed to the deltas. However, to @alexkrolick's point, my value= prop assignment is guarded by a function isContentBlank that reads the editor's delta to make a decision:
const isContentBlank = editor =>
  editor.getLength() === 1 ||
  (editor.getContents().ops.length === 1 && isBlank(editor.getText()));

So: upon rendering ReactQuill I'm reading both the protected editor's deltas and text, and I am consuming its HTML (and feeding that back to the editor's value as an HTML string, or sometimes a blank string).
If I stub isContentBlank to always return false, the issue stops reproducing. So, it has something to do with using ReactQuill in controlled mode and consuming its delta and text output.
If I change the ordering so that editor.getHTML is called first, the problem still reproduces.
Perhaps most curious of all: the problem only reproduces during the first iteration of my tests in a given browser process. If I re-run the tests, the problem stops reproducing -- even though Cypress is entirely reloading the React app in its embedded testing iframe.
So, this issue seems to involve a race condition that is impacted by V8 behavior -- JIT compilation perhaps, or caching of some sort at any rate.
Here is the relevant stack trace FWIW:
    at checkForNestedUpdates (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:23118:7)
    at scheduleUpdateOnFiber (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:21389:3)
    at Object.enqueueSetState (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:13100:5)
    at ReactQuill.Component.setState (webpack-internal:///./node_modules/react/cjs/react.development.js:325:16)
    at ReactQuill.onEditorChangeText (webpack-internal:///./node_modules/react-quill/lib/index.js:371:18)
    at ReactQuill.eval (webpack-internal:///./node_modules/react-hot-loader/dist/react-hot-loader.development.js:714:25)
    at Emitter.ReactQuill._this.onEditorChange (webpack-internal:///./node_modules/react-quill/lib/index.js:86:96)
    at Emitter.emit (webpack-internal:///./node_modules/quill/dist/quill.js:8666:27)
    at Emitter.emit (webpack-internal:///./node_modules/quill/dist/quill.js:1938:99)

Hello. The management of value has been changed. Among other things, it might fix this issue, as it avoids triggering many renders.
It is experimentally located in this branch: https://github.com/zenoamaro/react-quill/tree/fix/direct-value-change
Any chance you could try it (eg. using npm link)? It is a big enough change that I'd prefer to test it out before releasing it. Thanks!

Hi @zenoamaro, I did a quick testing of that branch. My above ReactQuillTest component did work as expected without re-render loop. Thanks for the fix!

Thanks! Closing this

It's unclear whether the fix is on master since the referenced branch is deleted.

We're running 2.0.0-beta.2 and are having the same problem.

@shinzui What was the source of the problem? We're facing the same issue – the quill adds characters to the original value and creates an infinite loop

I'm having this issue trying to build a blog.. sometimes when I edit a post it breaks with react-dom.development.min.js:1 Uncaught Error: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate.


Create new event onPreferencesChanged

New event created to allow the application to react when preferences are updated.

Hi @lvcabral
Thanks for the PR!
So -if I understand correctly- this would provide the preferences window to pass on a handler which gets triggered when the preferences are updated.
However, right now there's no use case that the preferences window needs to be updated of preferences changes I think?
What is your use case that you want this new event exposed on the preferences window?

My use case is have configurable themes on the settings screen, I allow my users to pick 4 options: purple, dark, light and system, to accomplish that I had to make 2 changes:

This one to allow register to the event
A custom index.html (see code below)

I first pushed this change, and if approved I would ask for a way to customize the asset.html or somehow have a way to inject a script on this file, just like we can customize the `

Interesting! Looks fine from here.

Looks cool! There is a feature request open to enable custom themes, but I like the workaround you found.
Not sure if there are many more use cases for this event other than setting the theme of the prefs window, but I'll definitely approve it.
Would you mind creating an issue for this, referencing it to this PR, and adding a line to the Changelog.md? I'll merge it afterwards.

Updated the changelog (the linter added lines between the titles to match .md standard) and the name of the event to match your naming standard (...Changed -> ...Updated)

I won't punish you by holding up this PR any longer, but we do need to document the API in the readme a LOT better than it is currently.

@lacymorrow do you want me to update the Readme too? This was not clear in your message above.


How can i uninstall cargo-shuttle?

How can i uninstall cargo-shuttle?
Despite the presence of .cargo/cargo-shuttle
terminal command $ cargo uninstall cargo-shuttle  doesn't succeed:
error: package ID specification cargo-shuttle did not match any packages

You can simply delete the cargo-shuttle (binary) file. You can find where it is (if it's on PATH) with which cargo-shuttle.

Should we add something to the docs about this so the issue can be closed?


Plunker template for Covalent

Similar to how material2 and many repos do it, let's create a plunker template similar to:
http://plnkr.co/edit/o077B6uEiiIgkC0S06dd
there's some questions and prereqs around generating and hosting JS and CSS on CDN

The common libraries can be loaded from cdnjs
    <script src="https://cdnjs.cloudflare.com/ajax/libs/typescript/2.1.1/typescript.min.js"></script>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/core-js/2.4.1/core.min.js"></script>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/zone.js/0.7.2/zone.min.js"></script>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/systemjs/0.19.41/system.js"></script>
    <script src="https://cdnjs.cloudflare.com/ajax/libs/hammer.js/2.0.8/hammer.min.js"></script>

And then we can get the umd libraries from angular, material and even our own umd with https://unpkg.com/ and load it in the systemjs.config.js file like this:
    // Angular specific mappings.
    '@angular/core': <EMAIL_ADDRESS>    '@angular/common': <EMAIL_ADDRESS>    '@angular/compiler': <EMAIL_ADDRESS>    '@angular/http': <EMAIL_ADDRESS>    '@angular/forms': <EMAIL_ADDRESS>    '@angular/router': <EMAIL_ADDRESS>    '@angular/platform-browser': <EMAIL_ADDRESS>    '@angular/platform-browser-dynamic': <EMAIL_ADDRESS>    '@angular/material': 'https://unpkg.com/<EMAIL_ADDRESS>    '@covalent/core': 'https://unpkg.com/<EMAIL_ADDRESS>
The rest is just platform.css and prebuilt theme loading from https://unpkg.com (which we need to generate first a prebuilt theme.

And once we have a prebuilt theme
<link href="https://unpkg.com/<EMAIL_ADDRESS>rel="stylesheet">

This will be our base plnkr url for the time being and will be updated every release.
http://plnkr.co/edit/7uZQn4mLNJkL6XN9WSNd
It will help us greatly in bug replication, demos and idea sharing.


3 Layers FB Game

Layer 1 - Background
Layer 2 - User profile picture
Layer 3 - png image like hat or whatever ..

![dscf0081_copy_1282611008]
(https://cloud.githubusercontent.com/assets/17341325/14957881/6db810d2-1090-11e6-9f69-866f90c81488.png)

So you want a new "layer" and for each layer we have, you'll want to upload a couple of photos (to choose from randomly?), I'm I right?

Actually it will be only one picture every time.
and yeah ill upload a couple of images, for example 5 different hats and the (idiot) player will see only one hat on his head (randomly)


Preloading in button mode

Hi,
I am using the button solution, so loading on click because I want the user to have the possibility to reach the footer.
For loading time it would be great to preload the next page and after clicking the content is directly appended.

I am closing this issue as its been inactive for over year.


"failed to build" on the newest OpenWrt v23.05.0

Re-open an issue here, should relate the makefile:
Hi, I've tried under openwrt v23.05.0, cannot pass:  (gcc-12.3.0 and many other changes)
got one critical error:
touch /home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/.configured_68b329da9893e34099c7d8ad5cb9c940
rm -f /home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/.built
touch /home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/.built_check
make -C /home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/Common -f Common.mk CXX=ccache mipsel-openwrt-linux-musl-g++ AR=mipsel-openwrt-linux-musl-gcc-ar ArchiveOutputSwitch="r "
make[3]: Entering directory '/home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/Common'
make[3]: *** No rule to make target 'mipsel-openwrt-linux-musl-g++'.  Stop.
make[3]: Leaving directory '/home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/Common'
make[2]: *** [Makefile:55: /home/jack/Desktop/openwrt-23050/build_dir/target-mipsel_24kc_musl/serial2mqtt-0.0/.built] Error 2
make[2]: Leaving directory '/home/jack/Desktop/openwrt-23050/package/serial2mqtt-openwrt'
time: package/serial2mqtt-openwrt/compile#0.17#0.05#0.19
    ERROR: package/serial2mqtt-openwrt failed to build.
make[1]: *** [package/Makefile:120: package/serial2mqtt-openwrt/compile] Error 1
make[1]: Leaving directory '/home/jack/Desktop/openwrt-23050'
make: *** [/home/jack/Desktop/openwrt-23050/include/toplevel.mk:232: package/serial2mqtt-openwrt/compile] Error 2

How to fix, please help, thanks.
refer to the same issue.

I've no OpenWrt platform at my disposal, but looking at the error messages , it seems the build is looking for a command mipsel-openwrt-linux-musl-g++ that it misses. Maybe you need installing the compiler set ?

It looks that OpenWrt questions are best answered by @halfbakery  , since he made the port.

@vortex314 , thanks.


[FEATURE] Fill out missing specs of this plugin in the API Spec Repo

Many endpoints of this plugin have yet to be described in the OpenSearch API Spec repo. Without proper specs, we won't be able to generate clients that can serve these endpoints.
For more details, please check the Developers Guide and the related issue on the spec repo:
https://github.com/opensearch-project/opensearch-api-specification/issues/232

[Catch All Triage - Attendees 1, 2, 3, 4, 5]

Thanks for filing the issue.


border-radius 50% does not work

Challenge Make Circular Images with a Border Radius has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.98 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

In your case, you are defining  .border-radius { border-radius: 50%;  } at the top of the CSS declarations. Which is then overwritten by border-radius: 10px; inside .thick-green-border definition.

@itzsnowing @santosh thanks a lot for your comments, we request you to please head over to the Help Chat for providing assistance with regards to coding. Issue Tracker is strictly for bug reports only.
@k-ryoko Closing this as not a bug with the codebase, but if you feel the instructions are not clear, the comment below.
Thanks everyone and happy coding!


Add Litmus Gulp task

Fixes #2... Will add readme instructions as a separate PR that will be part of #1

Thanks @m-bodmer


Loom

What is the vendor's name?
Loom
What is the vendor's pricing site?
https://www.loom.com/pricing
What is the base pricing? Use the lowest tier that looks sane for a small business customer, not free or personal tiers.
$8 per user/month
What is the minimum pricing for SSO support?
$12 per user/month
Does this pricing info come from a quote or other non-public source?
via quote, have used this service with my company and payed 12 dollar per user per month for SSO
Are there any caveats we should list in the footnotes?
burn them!

Hey @Mtthsvnlphn ,
as this repository was not maintained for a long time, I decided to start a new initiative under https://ssotax.org/. The data is already way more up2date than in this repository! Loom is already available there. Moreover I integrated feedback from the community like a Friends of SSO page.


spotify token asks for config twice

When running spotify token it asked for my ID and Secret twice. See screenshot.

Oh thanks for reporting @rebz , I see why this is happening.
So by default we check if creds are set or not, this is done here:
https://github.com/ersel/spotify-cli-mac/blob/master/index.js#L53
So regardless what command you're trying to run, we check if creds are set or not. If they are not set, we call the setTokens function to let user set tokens.
In your case, you ended up setting tokens because of else block on Line 52. Once creds were set, your action was resolved and CLI ended up asking for your tokens again (because you run spotify token, this would not have happened if you had run spotify mute for instance...)
So there are 2 things we should do I think:
1 - Remove the misleading line from README.md that you need to run spotify token to set your creds, running just spotify should suffice on the first ever run.
2 - We should short-circuit the app within the else block on Line 52, if user has run spotify token.
This would involve checking for command line args then short circuiting by doing: 	return process.stdout.write('\n');
Hope that clarifies what happened, I would be happy to accept a PR addressing above if you have time to look into it.

This has been fixed. Thanks @QuadDepo


Retire the gdx-lwjgl3-glfw-awt-macos extension

Retire the gdx-lwjgl3-glfw-awt-macos extension in favour of LWJGL's own solution:
if (SharedLibraryLoader.isMac) {
    Configuration.GLFW_LIBRARY_NAME.set("glfw_async");
}

This fixes #6896, #6989, #7059. It also seems to fix #6987 and #6992, but those crashes were hard to reproduce since they only seemed to happen very rarely. I couldn't reproduce #6872, but it might be fixed by this as well.

When I tried this in gdx-liftoff, it didn't work any better than the AWT extension. I believe that was in the early days of LWJGL 3.3.3, and things may have changed, but you should try building a .jar version of the LWJGL3 tests and running it on Mac, resizing the window(s), moving them, etc. If it works for you, could you post that JAR here for other Mac users to test? I think so far the best solution is to do what Guacamole does and restart the JVM with the appropriate arguments for the platform. gdx-liftoff bundles StartupHelper with new LWJGL3 projects for this reason, which is mostly the same as the startup code in Guacamole. StartupHelper does also include some support for running on Windows when your username isn't all ASCII (which is a problem because LWJGL3 and libGDX copy DLLs into a temp directory by default, and that is normally inside the user's home dir, but libraries on Windows seem to only be load-able from ASCII paths).
Here's the gdx-liftoff commit where I reverted the code that would be added here, because there were still problems:
https://github.com/libgdx/gdx-liftoff/commit/ee645963f56e4ea0020b61d40bb395e13f861f2a#diff-ca301a9d8cc4f9ff71a5578872faca1a77eeacbe59089cd6d661249b69311c9d

Since LWJGL 3.3.1 (when this was introduced and when liftoff seems to have used it) LWJGL has pushed a couple of fixes for their async lib which seem to have resolved the initial issues. The jar file of the libGDX tests is too big to upload it here, so anyone else who wants to try this will have to clone libGDX and adapt the tests with the code snippet in the PR description above. On my end (M1 macbook) everything works quite well.
Using the -XstartOnFirstThread argument (either manually or by restarting the JVM) still has one advantage: the window continues to be rendered when it is resized.

I've just tried using LWJGL async GLFW implementation with a non-trivial app (https://github.com/EsotericSoftware/spine-runtimes/tree/4.2/spine-libgdx/spine-skeletonviewer).
While it starts up and renders correctly, resizing the window will crash the app.
➜  spine-libgdx git:(4.2) ✗ java -jar spine-skeletonviewer/build/libs/spine-skeletonviewer.jar
FATAL ERROR in native method: Thread[AppKit Thread,5,system]: No context is current or a function that is not available in the current context was called. The JVM will abort execution.
        at org.lwjgl.opengl.GL11C.glViewport(Native Method)
        at org.lwjgl.opengl.GL11.glViewport(GL11.java:7968)
        at com.badlogic.gdx.backends.lwjgl3.Lwjgl3GL20.glViewport(Lwjgl3GL20.java:839)
        at com.badlogic.gdx.backends.lwjgl3.Lwjgl3Graphics.renderWindow(Lwjgl3Graphics.java:101)
        at com.badlogic.gdx.backends.lwjgl3.Lwjgl3Graphics.access$000(Lwjgl3Graphics.java:45)
        at com.badlogic.gdx.backends.lwjgl3.Lwjgl3Graphics$1.invoke(Lwjgl3Graphics.java:80)
        at org.lwjgl.glfw.GLFWFramebufferSizeCallbackI.callback(GLFWFramebufferSizeCallbackI.java:44)
[1]    96295 abort      java -jar spine-skeletonviewer/build/libs/spine-skeletonviewer.jar

I'll see if I can make this work, then we can get rid of glfw-awt-macos.

I've investigated this. gdx-lwjgl3-glfw-awt-macos is as broken as LWJGL3's glfw_async. The brokenness mostly concerns resizing windows, which will eventually result in native crashes ala:
[AGXG13XFamilyCommandBuffer renderCommandEncoderWithDescriptor:]:888: failed assertion `A command encoder is already encoding to this command buffer'

To lower the maintenance burden, ripping out gdx-lwjgl3-glfw-awt-maco is fine in my book. Users can switch to adding this to their main():
import org.lwjgl.system.Configuration;

...

static public void main(String[] argv) {
   if (SharedLibraryLoader.isMac) {
      Configuration.GLFW_CHECK_THREAD0.set(false);
      Configuration.GLFW_LIBRARY_NAME.set("glfw_async");
   }
   ... initialize your LWJGL3 app as before ...
}

This also means we can revert this and upgrade to LWJGL 3.3.4:
https://github.com/libgdx/libgdx/pull/7444

Soooo... We can merge this and then revert #7444? Or is something else missing? As I understand it, 054b25b8ced6fa54ba0c8d959b6a08a8d54de9d9 should've fixed the problems this PR would've created

Yes, we can revert revert! I let you do the honors.


Activity for adding a stolen car

https://material.io/guidelines/components/buttons-floating-action-button.html on main activity?

I'm thinking about using your detail fragment as the last step in the car adding process as a summary.

Can be - You can show the fragment once the car is added.
On 03/22/2017 10:57 AM, Róbert Ďuriančík wrote:

I'm thinking about using your detail fragment as the last step in the
car adding process as a summary.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/VaclavStebra/stolen-car-database/issues/3#issuecomment-288349960,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD0IYA9NUu0QCuNwLLNiou769I1kLhu2ks5roPBtgaJpZM4MaQkW.

step - Basic info + optional info
step - Photo
step - Location
step - Summary

Sounds good.
I will try to finish my currently assigned tasks by the end of this week
so you can use it.
On 03/22/2017 10:59 AM, Róbert Ďuriančík wrote:

step - Basic info + optional info
step - Photo
step - Location
step - Summary

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/VaclavStebra/stolen-car-database/issues/3#issuecomment-288350424,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD0IYNf77tJ1og9P5tUgTPwB6xTSFst0ks5roPDmgaJpZM4MaQkW.


For #806: Replace Select and Condition by cactoos analogues.

For #806:

Replaced Select and Condition by Filtered;
Corrected tests that needed;
Corrected java version so we can use cactoos.

Job #849 is now in scope, role is REV

This pull request #849 is assigned to @g4s8/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@yegor256 please take a look

@g4s8 please take a look

Codecov Report

Merging #849 into master will decrease coverage by 0.15%.
The diff coverage is 77.77%.

@@             Coverage Diff              @@
##             master     #849      +/-   ##
============================================
- Coverage     77.19%   77.03%   -0.16%
- Complexity      998      999       +1
============================================
  Files           224      222       -2
  Lines          4731     4673      -58
  Branches        380      366      -14
============================================
- Hits           3652     3600      -52
+ Misses          918      913       -5
+ Partials        161      160       -1

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/takes/rs/RsWithStatus.java
85.18% <100%> (-0.27%)
9 <2> (+2)

src/main/java/org/takes/rs/RsWithoutHeader.java
83.33% <100%> (-1.29%)
1 <0> (ø)

src/main/java/org/takes/rq/RqWithoutHeader.java
81.81% <100%> (-1.52%)
1 <0> (ø)

.../main/java/org/takes/facets/fallback/FbStatus.java
76.92% <50%> (-2.03%)
9 <3> (+2)

src/main/java/org/takes/http/MainRemote.java
53.33% <0%> (-1.67%)
6% <0%> (-1%)

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 341fb1b...dfa1106. Read the comment docs.

The architect of the project has changed; @yegor256/z is not at this role anymore; @paulodamaso/z is the architect now

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 21min)

@elenavolokhova/z please review this job completed by @g4s8/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

Code review was too long (29 days), architects (@paulodamaso) were penalized, see §55

The job #849 is now out of scope

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @paulodamaso/z

@0crat quality good

Order was finished, quality is "good": +20 point(s) just awarded to @g4s8/z

Quality review completed: +8 point(s) just awarded to @elenavolokhova/z