C interface of this repo

I am searching the Github for a good C implementation of the Aho-Corasick algorithm, but currently no one is suitable. I have used pyahocorasick in my python program before, and I am familiar with its interfaces. Since pyahocorasick is realized on the basis of cython, so I wonder whether it is possible for this repo to expose C interface as well...

Hi, I was thinking about that, but the current implementation is too coupled with Python C API. I would rather rewrote everything in C++ and then expose Python and C interfaces.

Hi, I was thinking about that, but the current implementation is too coupled with Python C API. I would rather rewrote everything in C++ and then expose Python and C interfaces.

Wow, that is really cool. I am looking forward to it. Thanks.

Just wanted to double check if there are there any plans for a C or C++ API? The best implementation of the algo I know is pyahocorasick, so it would be really great to have it available in C or C++ as well!

I considering only C++ (and then some C wrapper), as C++ template would easily let to experiment with different memory layouts, allocators, data types etc. As a side effect, it would allow having a more robust Python solution.


Options to disable Ip fallback when determine machine id

As the title said, I want the option to disable fallback to getting Ip as machine code.
https://github.com/houseme/snowflake-rs/blob/64e015cbb05881e0d03c2c616693c5d438bb11ba/src/builder.rs#L114-L116
For example:
[features]
default = []
ip-fallback = ["dep:pnet_datalink"]

The reason is because compiling pnet on Windows require some setup and maybe someone (like me) opt to choose the less of a hassle way of just not wanting it.

Welcome to contribute PR @HigherOrderLogic


add spring boot starter for azure keyvault

add spring boot starter for azure keyvault. sample code and readme will in next PR
@ZhijunZhao @aovechki

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@aovechki fixed listSecrets PagedList result loading issue.

Codecov Report

Merging #38 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   50.34%   50.34%
=======================================
  Files          24       24
  Lines         435      435
  Branches       43       43
=======================================
  Hits          219      219
  Misses        195      195
  Partials       21       21

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 7478274...8d99ed5. Read the comment docs.

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@aovechki pls help to review and approve the PR. no outstanding issue so far, but pr is in request change status. thanks.


Add Shift + Up/Down key combination to recall previously entered chat messages.

This is a simple yet super cool feature on another OSRS client. It's extremely helpful in crowded areas where sometimes your messages will not go through, even if you type them out multiple times. All you would need to do is hold Shift and the Up/Down key and you could alternate through your old messages that were entered. Sorry if this is in the wrong forum area.

This feature is basically auto-typing. If I recall correctly, we aren't interested in implementing this.

Yeah, I guess we should close this as wontfix


Vapor clang dependencies for clang-10 missing

Vapor clang dependencies for clang-10 missing
https://github.com/vapor/vapor/issues/2546

@rickwierenga were you able to answer the clang dependencies missing for clang-10 ?

@rickwierenga were you able to answer the clang dependencies missing for clang-10 ?

@rickwierenga This issue was closed, and was stale artifacts. https://github.com/vapor/vapor/issues/2546

@rickwierenga This issue was closed, and was stale artifacts. https://github.com/vapor/vapor/issues/2546


Cannot stop stuck build

I cannot stop a stuck build that has been running for multiple days via drone cli. I have tried restarting the drone container but no success
drone build stop "<repo>" "<commit>"
This is the error I got
strconv.Atoi: parsing "<commit>": invalid syntax

You can only stop build numbers and not commits.  Or am I missing something? I've checked for the command in help function. I tried to stop with build number but instead I get this error client error 400: Cannot cancel a non-running build. I have confirmed that the test is running by doing drone build info <repo> <build-number>

If this command doesn't exist you are not using the latest cli version.

Solved. Was running v0.7. As stated in the http://docs.drone.io/cli-installation/ but GitHub release was ahead to v0.8. and upgraded drone server to 0.8.3.


Can not reproduce. This seems to be fixed after #394 or #233 .
Current behaviour: if user is already logged in it doesn't have any back history(thirdroom related) else back history is /preview > /login  > oidc > /

I found the problem

else back history is /preview > /login > oidc > /

In this case going back to /login will get redirected back to / as user is already logged in


Dont support uint in arguments

Hello!
i tried use this:
package main

import (
    "github.com/codeskyblue/go-sh"
    "log"
    "fmt"
)

func main() {
    port := uint(1)

    session := sh.NewSession()
    session.ShowCMD = true
    out, err := session.Command("/Users/Ivan/Downloads/redis-3.2.3/src/redis-server", "--port", port).Output()
    if err != nil {
        log.Fatal("ERROR: ", err)
    }
    fmt.Println("output is", string(out))
}

And have result:
/usr/local/go/bin/go run /Users/Ivan/Documents/GolangSpace/src/examples/test.go
[golang-sh]$ /Users/Ivan/Downloads/redis-3.2.3/src/redis-server --port

*** FATAL CONFIG FILE ERROR ***
Reading the configuration file, at line 2
>>> 'port'
Bad directive or wrong number of arguments
2016/09/09 15:55:25 ERROR: exit status 1
exit status 1

This means redis-server not support --port argument.

@codeskyblue No.
Please look print your command "[golang-sh]$ /Users/Ivan/Downloads/redis-3.2.3/src/redis-server --port"
But in code:
out, err := session.Command("/Users/Ivan/Downloads/redis-3.2.3/src/redis-server", "--port", port).Output()
For my project, i use this fix (just convert uint to string):

And its work.

Yep, I remember it is because the inject lib, so only string is working.

@codeskyblue Okay, maybe allow only []string? Or convert by sprit?

sorry I dont very understand?

Maybe need to write another lib

I met this too, Command() doesn't non-string type, will discard it


OCPBUGS-6395: Upgrade golang/x/net to v0.4.0 to fix CVE-2022-41717

Signed-off-by: Alexander Greene<EMAIL_ADDRESS>

/retest

/lgtm


WFLY-18635 Drop support for scattered cache

https://issues.redhat.com/browse/WFLY-18635

@rhusar That is already case within the resource description.


Support grid view if possible

Kindle support grid view to lookup on eBooks.
is it possible to add a grid view though tags on eBook?

it's only supported in reflowable kindle. meaning we can't use fixed layout to allow it.


Support for bracket comments in cmake syntax highlighting

I would like to request the ability to syntax highlight CMake bracket comments as "comment". That is, currently if I do this:
# This is a comment
This is not a comment

Much like how linguist sees the #-comment as a comment, so does Vim. However, this:
#[[ This is a bracket comment in CMake.
It lasts until the end of the bracket ]]
This is not a comment

is not seen as a comment by Vim (and I guess by linguist...time for another FR! :smile:).
It would be nice if vim could display that as a "block comment" much like how it does with C-code:
/* This is a nice multiple
 * line comment */
This isn't

I did try looking at the c.vim to see if I could figure out how to support it but this:
https://github.com/vim/vim/blob/dc9ef26845c6bf5ba63bfa8d00fc8a9bdc3b2de5/runtime/ftplugin/c.vim#L29-L30
and this chunk:
https://github.com/vim/vim/blob/dc9ef26845c6bf5ba63bfa8d00fc8a9bdc3b2de5/runtime/syntax/c.vim#L214
broke my mind like the dot in Jeremy Bearimy.
NOTE: I know that the docs say that bracket comments are only supported since CMake 3.0 but that was 5+ years ago. I guess one could try to be smart and detect if cmake_minimum_required is set to something higher than 3.0, but I think it's a fair assumption to think 3.x is now standard enough.

please contact the syntax file maintainer of cmake for your enhancement request. (see the top of the file).

@chrisbra Done. Forgot about that in this day of GitHub!

Which Vim version do you use? Can you check the header of your cmake syntax file $VIMRUNTIME/syntax/cmake.vim? Mine is from 2017 Aug 30 and multi-line comments are properly highlighted.
Nevertheless the author could update the current syntax file. There's a new one from 2019 May 29.

@bfrg Huh. You might be right. I just tried it at home (with a homebrew vim) and it works.
  Each one has the same cmake.vim. I guess my desktop has a few more patches...

@mathomp4 You can check with :scriptnames which cmake syntax file is loaded (after you opened a CMakeLists.txt file). Maybe you have an older syntax file on your system which doesn't support multi-line comments. For example, my Distro (Gentoo Linux) puts the cmake syntax file that is shipped with CMake in /usr/share/vim/vimfiles/syntax/cmake.vim. Files in $VIM/vimfiles are sourced before the ones in $VIMRUNTIME, see :h 'runtimepath'. This means whenever a syntax file in $VIM/vimfiles is sourced, the ones shipped with Vim won't be used.

@bfrg Ah. Okay. :scriptnames helped a lot! There was a random old cmake.vim in my ~/.vim directory. Not sure why it's there as Vim has had cmake syntax for a while.
Now the issue seems to be that on my cluster at work the older system has Vim 8.0 which doesn't seem to handle bracket comments. But Vim 8.1 on the new system does! I guess I just need to either annoy them to install a newer version or build it myself. 😄


openapi for healthy checking , ipam plugin in , CLI

https://github.com/spidernet-io/spiderpool/pull/177
done


How can I run this demo

Thank you for your interest in our work. I'm going to spend some time implementing the demo. I'll let you know when it's finished.


What shall I do with Tss2_Tcti_Device_Init() Failed to open device file /dev/tpmrm0 error?

When I start keylime_agent, on CentOS7
the error appears. what shall I do?
[root@localhost ~]# /usr/local/bin/keylime_agent
Using config file /etc/keylime.conf
2021-09-29 14:19:01.683 - keylime.tpm - INFO - TPM2-TOOLS Version: 3.2.2
Traceback (most recent call last):
File "/usr/local/bin/keylime_agent", line 5, in
from keylime.cmd.agent import main
File "/usr/local/lib/python3.6/site-packages/keylime/cmd/agent.py", line 9, in
from keylime import keylime_agent
File "/usr/local/lib/python3.6/site-packages/keylime/keylime_agent.py", line 53, in
tpm_instance = tpm(need_hw_tpm=True)
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 236, in init
self.__startup_tpm()
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 393, in __startup_tpm
retDict = self.__run(['tpm2_startup', '-c'])
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 383, in __run
raise Exception("Command: %s returned %d, expected %d, output %s, stderr %s" % (cmd, code, expectedcode, retout, reterr))
Exception: Command: ['tpm2_startup', '-c'] returned 1, expected 0, output [], stderr [b'ERROR:tcti:src/tss2-tcti/tcti-device.c:281:Tss2_Tcti_Device_Init() Failed to open device file /dev/tpmrm0: No such file or directory \n', b'ERROR: tcti init allocation routine failed for library: "device" options: "/dev/tpmrm0"\n', b'ERROR: Could not load tcti, got: "device"\n']

What kernel version are you running?
We are now using the kernel resource manager for the TPM. If you are running a Kernel older than 4.12 this feature is no available.
Instead install tpm2-abrmd and export TPM2TOOLS_TCTI=tabrmd:bus_name=com.intel.tss2.Tabrmd before starting the agent.

It is Linux 3.10.0-693.el7.x86_64. I use tpm_emulator.service as tpm. can It work?

It is Linux 3.10.0-693.el7.x86_64. I use tpm_emulator.service as tpm. can It work?

yum install tpm2-abrmd
export TPM2TOOLS_TCTI=tabrmd:bus_name=com.intel.tss2.Tabrmd
[root@localhost ~]# /usr/local/bin/keylime_agent
Using config file /etc/keylime.conf
2021-09-29 15:59:52.318 - keylime.tpm - INFO - TPM2-TOOLS Version: 3.2.2
Traceback (most recent call last):
File "/usr/local/bin/keylime_agent", line 5, in
from keylime.cmd.agent import main
File "/usr/local/lib/python3.6/site-packages/keylime/cmd/agent.py", line 9, in
from keylime import keylime_agent
File "/usr/local/lib/python3.6/site-packages/keylime/keylime_agent.py", line 53, in
tpm_instance = tpm(need_hw_tpm=True)
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 236, in init
self.__startup_tpm()
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 393, in __startup_tpm
retDict = self.__run(['tpm2_startup', '-c'])
File "/usr/local/lib/python3.6/site-packages/keylime/tpm/tpm_main.py", line 383, in __run
raise Exception("Command: %s returned %d, expected %d, output %s, stderr %s" % (cmd, code, expectedcode, retout, reterr))
Exception: Command: ['tpm2_startup', '-c'] returned 1, expected 0, output [], stderr [b'ERROR: Could not dlopen library: "tabrmd"\n', b'ERROR: Could not load tcti, got: "tabrmd"\n']

I don't know what kind of emulator tpm_emulator.service is which package contains that service.
I would recommend either to use swtpm integration into QEMU/libvrit if that is your hypervisor.
If you want to run a virtual TPM for testing on the same system use swtpm.
Some information can be found here: https://keylime-docs.readthedocs.io/en/latest/installation.html#tpm-2-0-support


[ACR] Update to 2017-03-01 version

Update getCredentials to listCredentials to support multiple login credentials.
Refine regenerateCredential to support regenerate the specified login credential.
Add Sku to registry properties as a required property.
Rename GetProperties to Get.
Change CreateOrUpdate to Create, add registry create parameters.

Generated from swagger https://github.com/Azure/azure-rest-api-specs/blob/master/arm-containerregistry/2017-03-01/swagger/containerregistry.json

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as a link to the swagger spec, used to generate the code.
[x] The project.json and AssemblyInfo.cs files have been updated with the new version of the SDK.

/cc @SajayAntony

/cc @JasonRShaver can we get this merged. RPs have been already updated.

@azuresdkci test this please

@cormacpayne Are there more comments or can we get this merged so we can publish the package? Thanks.


How do I implement this on my data set?

Thank you very much for your help! I want to write a dataset about my dataset, my data only contains 224*224 image and mask, may I ask which dataset should I refer to?

You can refer to the implementation of COCO or CAMO, which represent two types of annotation.


Jest ignores obsolete file snapshots if there are inline snapshots

🐛 Bug Report
Jest ignores obsolete file snapshots if there are inline snapshots in this test.
To Reproduce
Steps to reproduce the behavior:
test('test', () => {
	expect('foo').toMatchSnapshot();
});

$ jest
 PASS  __tests__/test.js
  ✓ test (3ms)

 › 1 snapshot written.
Snapshot Summary
 › 1 snapshot written from 1 test suite.

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   1 written, 1 total
Time:        2.244s
Ran all test suites.

Change the above test to use inline snapshots:
test('test', () => {
	expect('foo').toMatchInlineSnapshot();
});

$ jest
 PASS  __tests__/test.js
  ✓ test (3ms)

 › 1 snapshot written.
Snapshot Summary
 › 1 snapshot written from 1 test suite.

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   1 written, 1 total
Time:        2.302s
Ran all test suites.

Now both a file and an inline snapshots exists. The file snapshot is completely ignored, Jest doesn't report that it's obsolete and doesn't remove it with jest -u.
Expected behavior
Jest should fail on obsolete snapshots and remove them with jest -u.

Thanks for reporting! This is a great point, I'm also able to reproduce.
I agree that it should mark the snapshot file as obsolete if it is not used anymore.

/cc @azz

I've never used toMatchInlineSnapshot, when I tried it yesterday tests even failed until I added prettier and update jest. Was toMatchInlineSnapshot just added recently? if someone tell me its commit link?
Thanks,

https://github.com/facebook/jest/pull/6380 & https://github.com/facebook/jest/releases/tag/v23.3.0


Please support newer versions of FastAPI

I like the module, but unfortunately it only works with FastAPI 0.70-0.71 Would be nice when the latest versions are supported as well

Thanks for the feedback @wavermartijn.  I will take a look at getting the package to support newer versions.  If you have a PR you would l like to submit, sent it over.

@wavermartijn I did push an update that I think should let newer version of FastAPI work with the module.  I haven't tested it yet due to time.

Hi.
Can you update your project dependencies and rebuild package ?
https://python-poetry.org/docs/basic-usage/#updating-dependencies-to-their-latest-versions
Now I can't install fastapi-healthcheck without pain.

Let me take a look and see what I can do. I will see about getting a github action to build the project once a week if I can to avoid this problem going forward.

@jtom38 Perhaps we can close this issue since we updated the requirements. Wdyt?

@panosangelopoulos I think with this PR we can close it. https://github.com/jtom38/fastapi_healthcheck/pull/17
That change should let the package continue to be used with newer versions of FastAPI till it hits v1.0.0.  At that point a review of FastAPI will be needed again to validate that the package still works or if it needs any updates.

@jtom38 It makes sense! I just approved your PR #17 .

https://github.com/jtom38/fastapi_healthcheck/releases/tag/v0.2.6 was released and this should work with newer versions of FastAPI until v1.0.0.  At that time, the package will need to be validated to make sure its working as expected.

Correction.  Please use https://github.com/jtom38/fastapi_healthcheck/releases/tag/v0.2.7 or newer as v.0.2.6 didn't generate a package.
https://pypi.org/project/fastapi-healthcheck/0.2.7/


python3Packages.pillow: 9.0.0 -> 9.0.1

Motivation for this change
https://github.com/python-pillow/Pillow/releases/tag/9.0.1
Fixes:

CVE-2022-22817: PIL.ImageMath.eval in Pillow before 9.0.0 allows evaluation of arbitrary expressions, such as ones that use the Python exec method.

Partial fix NixOS 21.11 (7f6df6405f7c3e987ca011ca5d8c206f8bdc8c86), but needs to be updated to https://github.com/python-pillow/Pillow/pull/6009

CVE-2022-24303: In show_file, use os.remove to remove temporary images.

No public CVE listing yet.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Built python39Packages.pillow on macos 10.15
Tested on nixos against staging-next due to gfortran woes on staging. Built for python 3.9 & 3.10 there with pillow kivy tilequant mahotas pydicom pilkit cairosvg piexif pytesseract bokeh wordcloud faker pikepdf imageio python-mapnik imageio for python 3.10.
Wonder if we should do as in #157872 and add a couple of the more heavily used or sensitive of these to passthru.tests.

Wonder if we should do as in #157872 and add a couple of the more heavily used or sensitive of these to passthru.tests.

Yea, that would be great.
But we can do that in another PR

I'll do the backport...


Update scalatest to 3.1.0-SNAP10

Updates org.scalatest:scalatest from 3.1.0-SNAP9 to 3.1.0-SNAP10.
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 @scala-steward in the comments below.
Have a nice day!

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

Codecov Report

Merging #1122 into master will increase coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1122      +/-   ##
==========================================
+ Coverage   84.09%   84.37%   +0.27%
==========================================
  Files          77       77
  Lines        2508     2508
  Branches      183      183
==========================================
+ Hits         2109     2116       +7
+ Misses        399      392       -7

Impacted Files
Coverage Δ

.../core/shared/src/main/scala/io/circe/Decoder.scala
90.53% <0%> (+0.29%)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
91.25% <0%> (+1.25%)
:arrow_up:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
90.36% <0%> (+1.8%)
:arrow_up:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


Osram Smart+ plug, only turning off with Hue integration

Home Assistant release with the issue: 0.103.0b0
Last working Home Assistant release (if known):  Was working last christmas ;)
(Both HA and Hue bridge are upgraded since)
Operating environment (Hass.io/Docker/Windows/etc.): Hass.io
Integration: https://www.home-assistant.io/integrations/hue
Description of problem:
Osram Smart+ plug only turns off via HA's Hue integration.  Both on and off works as it should from the Hue app.
On/off via API calls also work's as expected.
curl -H "Accept: application/json" -X PUT --data '{"on":true}' http://<IP>/api/<username>/lights/32/state
[{"success":{"/lights/32/state/on":true}}

and

curl -H "Accept: application/json" -X PUT --data '{"on":false}' http://<IP>/api/<username>/lights/32/state
[{"success":{"/lights/32/state/on":false}}

Osram Smart+
Model: Plug 01
SW: V1.04.12
Hue
Version 3.33.0 (10997)
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
hue:
  bridges:
    - host: <IP>
      allow_unreachable: true
      allow_hue_groups: false

Traceback (if applicable):

Additional information:

hmm that is what we send too. Are you just calling the light.turn_on service, or also passing in params  ?

To my knowledge I do not pass any parameters.
Turning on via HA
2019-12-09 09:48:09 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: domain=light, service=turn_on, service_data=entity_id=light.onoff_plug_1>
2019-12-09 09:48:09 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.onoff_plug_1, old_state=<state light.onoff_plug_1=off; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:47:42.079605+01:00>, new_state=<state light.onoff_plug_1=on; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:48:09.455852+01:00>>
2019-12-09 09:49:05 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.onoff_plug_1, old_state=<state light.onoff_plug_1=on; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:48:09.455852+01:00>, new_state=<state light.onoff_plug_1=off; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:49:05.032676+01:00>>

Turning on via Hue app
2019-12-09 09:54:02 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.onoff_plug_1, old_state=<state light.onoff_plug_1=off; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:49:05.032676+01:00>, new_state=<state light.onoff_plug_1=on; friendly_name=OnOff plug 1, supported_features=40 @ 2019-12-09T09:54:02.065122+01:00>>

Sniffed the traffic to the Hue bridge, with a error.
hue/sat/bri not available.
sudo ngrep -q 'HTTP' 'host <IP>' -d ens160 -W byline  | grep "/lights/25/state"
PUT /api/<username>/lights/25/state HTTP/1.1.
[{"success":{"/lights/25/state/on":true}},{"error":{"type":6,"address":"/lights/25/state/hue","description":"parameter, hue, not available"}},{"error":{"type":6,"address":"/lights/25/state/sat","description":"parameter, sat, not available"}},{"error":{"type":6,"address":"/lights/25/state/bri","description":"parameter, bri, not available"}},{"success":{"/lights/25/state/alert":"none"}}]


Post-start subscription to Tickers

I'm looking at the historic csv price handler
I'm sure that for nearly all backtests, that one is going to specify all the tickers initially. However, the class does provide for the ability to subscribe to more tickers after instantiation.
For instance (don't mind my super-sophisticated stock tickers):
from mkt_data_mgr import MarketDataMgr
from event import TickEvent
import queue

eq = queue.Queue()
tickers = ['dum', 'dum2']
mkt = MarketDataMgr('/Users/ari/documents/development/backtester/', eq, tickers)
mkt.subscribe_ticker('dum3')
while mkt.continue_backtest:
    try:
        event = eq.get(False)
    except queue.Empty:
        mkt.stream_next()

I realize that I have adapted things and technically this is no longer strictly speaking qstrader code. But, it mostly is, and the subscribe_ticker method is the same. I think you'll find the same results if anyone runs within qstrader.
Anyway the results of this are that ticker 'dum3' does not show up in the stream. And this makes sense if we look at the init code.
        if init_tickers is not None:
            for ticker in init_tickers:
                self.subscribe_ticker(ticker)
        self.tick_stream = self._merge_sort_ticker_data()

It shows quite clearly that it is not sufficient to subscribe, but one must also merge it into the tick stream. Somehow this needs to be integrated from the outside. So, either there ought to be a "subscribe_multiple_tickers" or "subscribe_initial", or, alternatively, make it so that subscribe_ticker iteratively adds the new tickers to the tick_stream.
Please let me know what you think. I've never contributed to coding an opensource project and I'm not sure of the protocol. I don't mind coding it, but want to make sure that there is interest in this change/fix. I have some others, too. Ive changed the execution handler to be more like an exchange with an order book and "matching engine" since I'm working with tick data. I don't mind posting it, but my code is now a bit different than qstrader. If there is interest, I can either post mine, or maybe shift my project to be a fork of qstrader.

Hi gRittyKitty,
Indeed you are correct - this is definitely an issue.
I mentioned in a few other issues today that we're actually working on a "ground up" redevelopment of QSTrader, and one such reason is the need for subscribing tickers beyond the initial backtest start date.
This is clearly required for strategies such as long-term equities momentum, where certain ETFs would not have been available prior to the backtest initiation date.
We're handling this in the new version by completely separating the iteration over data from the event-loop. This means that new tickers can be added/removed easily via some additional logic, without affecting the event looping process.
We're likely to have the first version released by the end of 2017 or start of 2018, but this may to far in the future for your needs? If so, we can certainly discuss trying to append in your proposed changes into the current codebase.
Cheers,
-Mike.

Hi Mike,
I've already implemented these features. It ended up demanding a bit more, too. Once you have limit orders, then you have the potential for unfilled orders (I guess you could do IOC orders, but I did not think about that then). Once you have unfilled orders, there needs to be a way to cancel. Once you cancel without filling, then you need a way to let the strategy know whether it is in or out of a trade (my actions are asymmetric), so I created StrategyEvents for the portfolio_mgr to "talk" to the strategy_mgr.
I've also implemented some logging features, a trading start time, trading end time, and a liquidate portfolio function.
It did not seem like too much until I see it written here. Anyway, like I said, I'm happy to share if there is interest.
Ari


Add a UnityAot MSBuild option for Mono's System.Private.CoreLib

IL2CPP is currently using the Mono System.Private.CoreLib implementation, but it contains some AOT unfriendly code paths.  We instead want to be able to use the NativeAot version of ArraySortHelper.
This adds an UnityAot property that we can use to conditionally include files.  It also adds an UNITY_AOT compilation define that we can use.

What would it look like if we moved all the logic into a System.Private.CoreLib.props file somewhere under the unity root directory and then imported just the props file in System.Private.CoreLib.csproj ?
If it's doable, the advantages would be

Easier to review our customizations at a glance
Minimizes the changes to System.Private.CoreLib.csproj which will reduce the odds of conflicts.

What would it look like if we moved all the logic into a System.Private.CoreLib.props file somewhere under the unity root directory and then imported just the props file in System.Private.CoreLib.csproj ?

@mrvoorhe - I've done that - it does clean some things up a bit.

Maybe hold off merging until you can confirm this PR didn't break Test - Runtime & Pal - Linux x64 Debug.  The last run on main was in late September and it was green.  I started a new run
https://unity-ci.cds.internal.unity3d.com/project/1674/branch/unity-main/jobDefinition/.yamato%2Fruntime-and-pal-tests.yml%23test_-_runtime__pal_-_linux_x64_debug?nav=branches

The job passed on unity-main https://unity-ci.cds.internal.unity3d.com/job/43049025 so it appears this PR is somehow breaking something?

The job passed on unity-main https://unity-ci.cds.internal.unity3d.com/job/43049025 so it appears this PR is somehow breaking something?

I reran this and it passed.


Libraries of BIG tech companies

This topic must be filled with corresponding resources.

Add libraries like: Netflix OSS, etc.


Bug fix on MinInt operator, some mapping and float utils

The MinInt operator didn't work, so I fixed that
also included some personal utils I made during my last project :)

I recently reimplemetned [Layer2d] to add some features that might be helpful for you:


[Feat] Nonautoregressive Methods

Description
Contributed by @Furffico, @tycbony, and @henry-yeh from DeepACO and GLOP. Adding new support for non-autoregressive (NAR) methods, including edge embedding methods for various environments, anisotropic GNN encoder with edge-gating mechanism, non-autoregressive decoder, and related utils functions.
Motivation and Context
NAR methods are an important extension for RL4CO model support, aside from various previous autoregressive methods.
Types of changes

[x] New feature (mostly just adding new functionality)
[x] Example (update in the folder of examples)
[x] Breaking change (there are some changes to basic classes, even though most of them are simple code-level optimization, just be careful about hidden bugs)

Checklist

[ ] This change requires an extension of the documentation.
[x] This change has provided a testing.

I have modified the code following the review above. Please review the changes.


CAMEL-18793: add prettyBody to simple language

Description
Working on that issue about pretty print mode

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[X] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[X] I checked that each commit in the pull request has a meaningful subject line and body.

[X] I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

There is org.apache.camel.util.xml.XmlPrettyPrinter to use for XML

There is org.apache.camel.util.xml.XmlPrettyPrinter to use for XML

Struggling with cyclic reference between camel-support and camel-xml-jaxp. Some advice?

There is org.apache.camel.util.xml.XmlPrettyPrinter to use for XML

Struggling with cyclic reference between camel-support and camel-xml-jaxp. Some advice?

I was able to handle this extracting org.apache.camel.util.xml.XmlPrettyPrinter to a new package and removing his dependency with camel-core-model. To do that I had to copy/paste org.apache.camel.util.StringHelper.padString to org.apache.camel.util.xml.XmlPrettyPrinter
A bit tricky but works 😅

I was able to handle this extracting org.apache.camel.util.xml.XmlPrettyPrinter to a new package and removing his dependency with camel-core-model. To do that I had to copy/paste org.apache.camel.util.StringHelper.padString to org.apache.camel.util.xml.XmlPrettyPrinter

Normally I would frown copying/pasting, but I think it may be OK here. After this is merging, we can replace both of them with " ".repeat(level * blanks) as the JDK's repeat is about 2 times faster than the one we have (likely due to using intrinsics under the hood)

Also the csimple-language.doc should be updated.

Also the csimple-language.doc should be updated.

I couldn't figure out what to add to simple-language.doc

Also the csimple-language.doc should be updated.

I couldn't figure out what to add to simple-language.doc

Ah yeah we just refer to simple for those functions


[docs] 📝 FAQ overhaul for linking to individual questions

Executive Summary
Fixes #2268. Individual questions can be linked to.
Overview

Reformat "Contents" to use the .. contents:: directive
Reword "Critical" into "Critical Issues"
Reformat "Critical" section to define "critical issues" first
Reformat FAQ sections to follow a new format (below)
Reformat FAQ sections so individual questions have links

All sections now follow a new format (below).
A "frequently asked question" may also include a possible
cause and a solution (if the two are not obvious from
the context):
FAQ Section Title
=================

.. contents::
  :local:
  :backlinks: none

1. Question 1
-------------

**Possible Cause**: This is likely due to...

**Solution**: Fix with...

Some questions were renamed to try and make their contents more explicit. (e.g.: Python Package: Question 1). Many questions could begin with "I see an error message like this." Having more-explicit names could be helpful if the FAQ index gets longer, or if a person reads the index/greps for error messages.
Question 1:

- I see error messages like this when install from GitHub using python setup.py install
+ Error: setup script specifies an absolute path when installing from GitHub using python setup.py install

Alternatives / Discussion Points
All feedback is appreciated! The following are some specific points that might warrant further discussion:

Long questions are autogenerated with long links (e.g. FAQ.html#why-is-early-stopping-not-enabled-by-default-in-lightgbm). An alternative might be FAQ.html#general-12, but this might require setting more things manually.
Each question is a subsection. This causes the side bar to have a large number of entries in the General section (see the preview image). This has been a known issue with sphinx_rtd_theme for some time. An extension exists for limiting the depth, but would introduce a dependency.
The current formatting makes it a little hard to tell where one section ends and another begins (e.g. R to Python, see image below).

Preview Images

Long side bar:

R/Python border

thanks @hayesall ! I'll look more closely tomorrow

In addition, can you please elaborate what exactly is required to make FAQ links short? Maybe one example?

Long questions are autogenerated with long links (e.g. FAQ.html#why-is-early-stopping-not-enabled-by-default-in-lightgbm). An alternative might be FAQ.html#general-12, but this might require setting more things manually.

This can be accomplished through :ref: tags.
For example:
**Contents**

- :ref:`general-1`
- :ref:`general-2`

.. _general-1:

1. Where do I find more details about LightGBM parameters?
----------------------------------------------------------

(answer)

.. _general-2:

2. Another Question
-------------------

(answer)

All occurrences of general-1 are replaced with the title ("1. Where do I..."), and the html hyperlink is: FAQ.html#general-1

Still failing a check-docs test, looking into it.

Still failing a check-docs test, looking into it.

It's linkchecker issue. Never mind! I'll re-run the tests till it pass.

This can be accomplished through :ref: tags. ...

Ah, I see. Thanks!
It requires manual edition of contents section too each time new entry is added, right?..

It requires manual edition of contents section too each time new entry is added, right?..

Yep.

It's linkchecker issue.

Sphinx's make linkcheck is also raising an error on my local machine (below). Maybe GitHub tweaked something on their backend and now the links aren't behaving the same?
Warning, treated as error:
/Users/hayesall/LightGBM/docs/FAQ.rst.rst:289:broken link:
https://github.com/microsoft/LightGBM/blob/master/python-package/README.rst#build-from-sources
(Anchor 'build-from-sources' not found)
make: *** [linkcheck] Error 2

Sphinx's make linkcheck is also raising an error on my local machine (below). Maybe GitHub tweaked something on their backend and now the links aren't behaving the same?

We are using old LinkChecker 8.6              Copyright (C) 2000-2013 Bastian Kleineidam linkchecker on the CI. Maybe this is the reason? Because on CI the error is about YouTube:
URL        `https://www.youtube.com/watch?v=iqzXhp5TxUY'
Name       `here'
Parent URL file:///home/travis/build/microsoft/LightGBM/docs/_build/html/Parallel-Learning-Guide.html, line 262, col 119
Real URL   https://www.youtube.com/watch?v=iqzXhp5TxUY
Check time 0.531 seconds
D/L time   0.568 seconds
Size       1KB
Info       Redirected to
           `https://www.google.com/sorry/index?continue=https://www.youtube.com/watch%3Fv%3DiqzXhp5TxUY&q=EgRoxoM6GKiegeoFIhkA8aeDS518C8SE-j-34lK4u8ejzutjLzF1MgFy'.
Warning    [http-empty-content] No Content
Result     Valid: 204 No Content

I remember we pin old version exactly due to anchors false positive errors.


[WFLY-8767]Changing the max-pool-size of the datasource pool should i…

possible fix for :
https://issues.jboss.org/browse/WFLY-8767

retest this please

retest this please

Retest this please


Why does hasValidIdToken not take the clockSkewInSec property into account?

Problem
When the local system time is set 20 minutes to the future, then processIdToken returns an error with the message 'Token has expired'. The property clockSkewInSec can be used to allow the current time to deviate when validating id_token's iat and exp values, the default value is 10 minutes, so when we set clockSkewInSec to 25 minutes the processIdToken executes without any error, however the method hasValidIdToken doesn't take the clockSkewInSec into account, which means that the token is not valid.
To Reproduce
Steps to reproduce the behavior:

Set the local system time 20 minutes to the future
Then we get the message 'Token has expired'

Question
Why does the method hasValidIdToken not take the clockSkewInSec into account?

I think the PR that introduced it was not complete enough. To be honest, given the property name I'd expect the skew also be applied to other tokens and times.

Thank you for your response, do you know how long it could take for the bug to be fixed?

In the end about 2/3rds of the fixes come from community members like yourself, so first someone would have to fix these things, and then it would have to get merged and released.
I'll be honest and say that there's quite a few PRs open for some time already, and the maintainer hasn't had time to create new releases. It's open source and volunteering work, so this happens and should be okay IMHO.
If you need a quick fix your best bets are to either (a) fork and fix it yourself, or (b) look into using alternative libraries.

It's now considered.
Btw: There is now also an exchangable DateProvider. The default implementation returns Date.now()


[Fixed]: "Latest News " is displaying the blue processing screen , un…

…less User tapped on that - ios App v1.0.1.

In Carousel, Scroll will be disable when get single url.
Show loading indicator in Webview.

Description:

Linked issues:
#730
Screenshots:

How to test:

Open the App
Open Hamburger menu
select "Latest News "
Blue screen will display , with a processing sign
It will display this blue screen continuously unless user tap/swipe on the blue screen .

Hello @Patrick-Erichsen, I am able to reproduce this error with my iPhone 6+ device (iOS version 12.4.1). But not able with my iPhone simulators running iOS 13.3. I have uploaded two videos to my google drive which both shared me from my tester team. They are able to reproduce this error with their devices. Please check below link.
https://drive.google.com/drive/folders/1ryx3kscOTgOaC_X5rBrzxDiCmIw2kh6v?usp=sharing
In first video, its not happening but in second video issue is persisting.

It could be 12.x ios, can you initialize a 12.0 simulator and try again? It would be great if we can reproduce it on a simulator too.

@tstirrat I tried this on an iPhone 6 emulator running iOS 12.4 and was not able to reproduce the error. Could you give it a try? I had to download iOS 12.4 and then I ran yarn install:pod && react-native run-ios --simulator="iPhone 6 Plus".

Hello @ShreyaaSahu They are not able reproduce #730 issue on their simulator. Could you please explain that how to get this issue. You were tested this issue with your device right?

Hi All ,
I have tested this in latest release v1.0.65 ( iOS 13.3 , Device model XR ) . This issue is persistent .
All the steps for reproducing this issue are explained in issue #730 .

@ShreyaaSahu  @eyganeshv this is what I am seeing on each device:
iPhone 11 simulator running iOS 13.3: https://drive.google.com/file/d/1Tz4IFiMmanisikZK6IS_3yd7oedvb4gB/view?usp=sharing
iPhone 6+ simulator running iOS 12.4:
https://drive.google.com/file/d/1MZyBubynjMvNEDpaQTXCCVBB0NvlB6YQ/view?usp=sharing

Hello @Patrick-Erichsen, If already same code from my slide thn you can do close this issue otherwise do merge.

Well it sounds like @ShreyaaSahu is still able to replicate. I want to try and get on other dev to confirm that they are not able to replicate before we close the issue.

@eyganeshv Are you okay to close this out? I am still unable to reproduce.
We also have some folks working on a larger UI/UX refactor for the entire news screen, so this work will likely be overwritten very shortly with the work in that new refactor.

@Patrick-Erichsen Yes, You can close this.


Integration into Existing Project

I'm working with a Hosting Control Panel called VestaCP. They have a paid for addon for a file manager however I would like to integrated Angular-FileManager into it for the end users. Is there away to have it automatically default to the users ~ based on the username they are signed into the control panel with?

Hi @vstock28021 this project started for the same requirement that you have.
The answer is YES, but you will need to extend the backend connector (i guess you will use ftp)
To be used as an example I've created a demo connector with ftp.. obviously is not 100% completed.. but maybe can be a help to start
https://github.com/joni2back/angular-filemanager/blob/master/bridges/php/handler.php


Added and loaded a new spam domain section

Added and loaded a new spam domain section. These domains have have been found to be the root domains of spam either in forums, e-mail, or other online communications. The ones loaded so far contain social engineering attempts, but the intent of the section may vary over time.

Thanks @ScriptTiger!  Merging.


Fix package-manager issue

This PR updates the contributing.md and adds the package manager field to our root package.json. The idea here is to help contributors who have the wrong version of yarn installed (e.g v2 or v3).

Related to #3189


Change gpuMemoryLimit setting used when undefined

For non-mobile devices, the gpuMemoryLimit used a setting of "none" (no limit) when it was not defined.  This changes that setting to TileAdmin.nonMobileUndefinedGpuMemoryLimit (6,000 MB) instead.
Some very large reality data sets and imodels could on occasion cause a WebGL context loss with no limit imposed, so this attempts to set a generous limit that should prevent most of these problems for the common case of the gpuMemoryLimit being undefined.  If desired, the limit can still be set to "none".
See https://github.com/iTwin/itwinjs-backlog/issues/1047

@aruniverse can we backport this to 4.4?
cc @danieliborra

@mergifyio backport release/4.4.x


Application crash when unexpected environment variables are present

Describe the bug
I'm not entirely sure which one is causing it, but the presence of some Kubernetes standard environment variables can cause RSS Bridge to crash.
To Reproduce
Steps to reproduce the behavior:
Run the docker container with a Kubernetes deployment and service called "rssbridge". This will create the following environment variables automatically (exact values will vary, however I suspect it's the names that are causing the issue
RSSBRIDGE_PORT_80_TCP=tcp://<IP_ADDRESS>:80
RSSBRIDGE_PORT_80_TCP_PROTO=tcp
RSSBRIDGE_PORT=tcp://<IP_ADDRESS>:80
RSSBRIDGE_PORT_80_TCP_PORT=80
RSSBRIDGE_SERVICE_HOST=<IP_ADDRESS>
RSSBRIDGE_SERVICE_PORT=80
RSSBRIDGE_PORT_80_TCP_ADDR=<IP_ADDRESS>

The following error will then be generated on every URL
Exception TypeError: Argument 2 passed to Configuration::setConfig() must be of the type string, null given, called in /app/lib/Configuration.php on line 109 at lib/Configuration.php line 181

Stacktrace

#0 index.php:7
#1 lib/RssBridge.php:15
#2 lib/RssBridge.php:34
#3 lib/Configuration.php:109
#4 lib/Configuration.php:181

Expected behavior
RSS-Bridge should work

Those env vars are not related to rss-bridge.
rss-bridge parses all env variables which are prefixed by RSSBRIDGE_. I'd call this a namespace collision and the kubernetes should prefix their envs with KUBERNETES_ or something like that. Anyways, I pushed a fix that will make the error go away.

Those env vars are not related to rss-bridge.

Oh I agree, this is very much an odd edge condition rather than an outright fault in rss-bridge.

Anyways, I pushed a fix that will make the error go away.

Thank you :)


Fix transformers test skipping all tests.

Around the release of torch 2.1, something changed that began skipping all of the tests that should have been running in nv-transformers.
Sample passing build before: https://github.com/microsoft/DeepSpeed/actions/runs/6399875223/job/17372640478#step:8:1486
Sample passing build after (skips all tests): https://github.com/microsoft/DeepSpeed/actions/runs/6476276918/job/17584733821#step:8:1355

Closing in favor of #4388


[FEATURE] Skip the test from Mocha execution using grep option

Testdeck Version
latest
Description
Use annotation to skip the test at the time of compilation from the Mocha.
Defining situation:
I am working on an application that is built for a SaaS product that has 60+ features/modules and 3000+ test cases. It requires verifying the feature or sub-feature we run the test to target very specific test cases. We use the Mocha grep option which helps to choose only run those tests only. We have added tags, some settings value dynamically in the title which evaluates dynamically before Mocha compilation. for example:
// file - helper.ts
export default getUserPoint(){
  // Get the customer user points from backend dynamically
  const points = 100; // assume it is 100
  return points;
}

export default isApplicableForCurrentClient(settings){
  // fetch the current client (for which saas product is running) settings from backend dynamically if the current feature is enabled or not for given settings
  const enabled = false; // assuming it is false for now
  return enabled;
}

// file - test.spec.ts
describe('feature-name: about sub-feature', function () {
    it(`should show the offer if the user coin points greater than ${getUserPoint()} #coins #points isSettingEnabled=${isApplicableForCurrentClient('point-settings')}`, function () {
      // ...
    });
});

Once the above code compiles then it will look like below:
// file - test.spec.ts
describe('feature-name: about sub-feature', function () {
    it(`should show the offer if the user coin points greater than 100 #coins #points isSettingEnabled=false`, function () {
      // ...
    });
});

// configure wdio.config.ts for mocha ops with grep
...
mochaOpts: {
  ui: 'bdd',
  timeout: 60000,
  grep: 'isSettingEnabled=true'
},
...

As per the above grep settings, it will only run those tests which are applicable to the current client and customer. It enables running the test for many different types of clients and its customer without setting up the test data every time in e2e test. it will pull the real settings and run the test accordingly.
The best approach of Mocha is, these tests which don't match the grep will be excluded at the mocha compilation itself. it will not be skipped at execution time hence it saves lots of test run execution time. It helps get the verification report quickly.
I know, if we add all these in the test title still in TestDeck then it can be achieved but sometimes it makes my test case title ugly and loses the readability. I want the same efficiency with the help of the TestDeck annotation feature without losing the test title readability.
Like below:
// file - test.spec.ts
@suite('feature-name: about sub-feature')
class MySuite{

@addUserPoints() // fetch the points and replace the pattern in the title `<replace-with-point>`
@tags(['#coins', '#points'])
@isSettingEnabled('point-settings') // 'setting-key-name', add title like `isSettingEnabled=true | false`
@test 'should show the offer if the user coin points greater than <replace-with-point> '() {
  // ...
  }
}

npm test

  feature-name: about sub-feature
    ✓ should show the offer if the user coin points greater than 100

  1 passing (1s)
  0 failing

I appreciate it if you guide how to achieve it using or enhancing TestDeck.
Techstack

webdriver.io - latest
Mocha - latest
Allure Reporter - 7.x.x
TestDeck - latest
Nodejs - 18.x.x

Reference links:

https://mochajs.org/#-grep-regexp-g-regexp
http://www.petecorey.com/blog/2016/07/25/mochas-grep-flag/

For parameterising the test name, see https://testdeck.org/pages/guide/parametrised_tests.
As for conditionally running individual tests or suites, see

https://testdeck.org/pages/guide/advanced_suite#skipping-suites---
https://testdeck.org/pages/guide/advanced_tests#skipping-tests---

@Amitesh I know it has been a long time, but have you succeeded with the above provided information? May we close the issue?

For what it is worth, it is already provided and the responsibility of the test engineer to apply proper means to its test suites and test cases.


Reports dashboard

Updated the code for generating the summary report. Fixed some hacks and improved code quality.

sorry wrong project.


Read user name and password from ~/.netrc

I wouldn't like to paste my password into the _config.yml where it's publicly visible but it's also frustrating to type it out for each glynn call. I already have the password saved in ~/.netrc because that's where other programs like vim and lftp look for it. Could glynn check that file for a user name and password defined for its host, before prompting me to enter them?

Sure. The netrc library will allow reading from ~/.netrc and fetch the credentials.
I'm not using glynn these days and don't have visibility on when I'd be able to implement this feature. It should be fairly simple though. Feel free to do it.


AAD Connect new server build and ADFS settings

Documentation is not clear about post installation steps required with respect to ADFS. If AAD Connect is installed in Staging mode on a new Server using existing database then what steps need to be done for ADFS when there is existing federation setup with AAD already and when these must be done.
Documentation states that 'User Signin method' needs to be changed for versions lower than <IP_ADDRESS>. However, if we want to change sign-in method to Federation with ADFS, then wizard requires details of existing ADFS servers. If these details are provided while in Staging mode, will it break user-sign-ins to AAD.
How the existing AAD Connect and ADFS federation setup will be impacted if a user proceeds with changing the user sign-in method to Federation and running through the wizard.

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

ID: a0dbe321-8471-e781-ccd7-1bacdb72395e
Version Independent ID: a4c74473-cc49-a31b-8916-0a7497e9cc42
Content: Install Azure AD Connect by using an existing ADSync database
Content Source: articles/active-directory/hybrid/how-to-connect-install-existing-database.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@SinghGurd , Thanks for raising the issue. Could you please provide the document that you are referring to?

Oops...It's about https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-install-existing-database

Any update on this one.

@SinghGurd Apologies for the delayed response. Adding ADFS Servers to the AD Connect when operating in staging mode will not affect the behavior of ADFS and Azure authentications with ADFS. Changing the sign-in method will change the domain state to federated. If your domain is already federated then there will be no impact at all.
If you make any changes(SSL cert change ) to the ADFS servers from the AD connect tool, then the behavior will be impacted. Let me know if you have any further questions.

Thanks Manoj...Do you know of a blog/documentation that walks through using an existing ADFS farm? Existing documentation runs through creating a new ADFS farm https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-install-custom#configuring-federation-with-ad-fs.
It also states "Azure AD Connect can be used to manage only one AD FS farm. If you have existing federation trust with Azure AD configured on the selected AD FS farm, the trust will be re-created again from scratch by Azure AD Connect."
Would be great to have the process documented where AAD Connect is re-build and then configured to use an existing ADFS farm.

@SinghGurd The document you referenced talks about  "create-a-new-ad-fs-farm-or-use-an-existing-ad-fs-farm" your scenario as well.
Can you take a look again and me let me know if it covers your scenario.

Thanks @ManojReddy-MSFT; that article doesn't cover our scenario in detail. We want to know 100% what AAD Connect does to ADFS and AAD when user sign-in method is changed to Federation and domain credentials and ADFS details are provided in the wizard.
I have outlined our scenario below again

Domain federated with AAD using ADFS. Existing AAD Connect in place and working fine.
New AAD Connect has been installed on a new server using database backup of existing AAD Connect

We now want to promote new AAD Connect as production and for that we are require to re-run the configuration wizard --> Change user-sign-in method to Federation --> Provide domain admin credentials --> Provide ADFS details. We have found no documentation which states 'what AAD Connect does to ADFS and Azure AD when this wizard is run and what impact we can expect on users"

@SinghGurd My sincere apologies for the delayed response.
I will try to explain the changes that AAD Connect does at each stage for your scenario

When you provide ADFS/ WAP details, this is just to let the AD connect know that these servers are already handling the ADFS and WAP role and will not have any behavior change at this stage.  AD Connect will try to connect to these servers and will validate that it can perform any management task in the future if needed. So behavior for the users is not impacted at all.

If you rerun the wizard and perform any management task, like changing the SSL certificate, then AAD connect will try to update it in all servers and in this scenario behavior of the servers will be impacted.

Hope this clarifies. Let me know if you still have any questions regarding this.

@ManojReddy-MSFT Thank you for the details.

Hi @ManojReddy-MSFT I'm facing a scenario that I think is not covered in the documentation you refer to.
Originally my customer installed AAD Connect with a new AD FS farm. Time has passed and they wanted to upgrade the farm to a newer version. The team that handles the platform now did not know about how it was installed originally so they went ahead with a rolling upgrade of their AD FS farm, decommissioning the original servers after that.
Now the people administering AAD Connect are stuck in the wizard because it tries to connect to the first server in the original farm, so it looks like AAD Connect was not aware of the rolling upgrade of AD FS.
We don't know how to fix this situation, we guess the best way would be to just reinstall AAD Connect into a new instance and forget about the integration, but not sure if this will break anything.  My inner nerd asks me to find a way to reconnect the AAD Connect and the AD FS farm before doing anything, but I can't find documentation on how to handle and existing installation, only a new one.
Maybe you could point me to the right documents?


EAGLE-229 EAGLE-228 Fix incompatible alert library

https://issues.apache.org/jira/browse/EAGLE-229
https://issues.apache.org/jira/browse/EAGLE-228

Not completed yet, don't merge


Logout and endemulation

Odd though... If I logout, then paste the local harvest URL in again, I get a CAS error.


Capability to run without a shell.

Appears to work fine if not running under a shell. Useful for clean, from scratch containers.
Ignore if there's a reason the command needs to run under a shell.
Regards.

Hey Eduardo!
Using shell allows running command chains and defining env variables with a single Procfile line. Like this: NODE_ENV=test yarn install & yarn test.


Fix/updates - Move to React 16 and bootstrap tables 2

Move to React 16 and bootstrap tables 2 - resolve #2

Coverage increased (+0.2%) to 70.845% when pulling e47e3ecdd876102a4d76d79b7669a83d46f6f51c on fix/updates into c79daa09870c3cde919208e3436b91b1178b1709 on master.


Error in updating color in Visual Block mode

Issue
When I try to switch to visual block mode, editor throws an error saying attempting to  concatenate a nil value.
Invalid expression: luaeval(require("galaxyline").component_decorator')("ViMode")

This is the command I am using
vim.api.nvim_command('hi GalaxyViMode guifg='..mode_color[vim.fn.mode()])

Also I tried checking what the vim.fn.mode() returns. In block mode it is returning '^V'. I also tried to do a check to see if it is returning null of the string '^V' but nothing works.

to input ^V character you need to enter input mode and press <c-v> twice.

@wh1tevs
Thanks, it solved the issue. But I have a doubt. Why writing '^V' manually didn't work and doing what you told did.

@Tinku10 because ^V as text is a two separated symbols, but ^V character is a special control symbol who just looks like ^V
:h i_CTRL-V
:h i_CTRL-V_digit
:h digraph-table

@wh1tevs Thanks, got it.

@Tinku10 because ^V as text is a two separated symbols, but ^V character is a special control symbol who just looks like ^V
:h i_CTRL-V
:h i_CTRL-V_digit
:h digraph-table

Thank you man, I encountered the same issue, and I've been struggling over it for a while!


feat: add service status protocol

Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[x] I have added tests to cover my changes.
[x] I have filled out the description and linked the related issues.

Description
Closes ethersphere/bee-backlog#54

This PR could not come any sooner 🙏


Sponsors section?

The commits I have done recently on re-start is related to my work on http://www.quantum-ba.com/, so what about adding a section of "sponsors" and/or "companies using re-start" so we give them some credit? :-)

Sure!
that would be awesome 😄

Done, I wanted to do a pull-request from online editor and since I had push requests it did it directly on master, sorry :-P


Fix casing of module locations to match that used by Rollup.

Changes
Snowpack uses require.resolve to resolve the absolute location of installed modules.  On case-insensitive filesystems, require.resolve will (intentionally) use the cwd case in its result.  However, Rollup uses the canonical case for module IDs internally, so if it arrives at an entry point module via a different dependency path it will see it as a "new" module and duplicate it in the installation output.
This PR ensures that Snowpack canonicalizes the case of the module locations before passing them to Rollup.  Note that fs.realpathSync.native is needed, as fs.realpathSync explicitly doesn't canonicalize casing.  This make the method dependent on the libc that Node was linked against, with two implementation-dependent limitations called out in the docs:  the number of symlinks followed may be limited, and, if linked against musl libc, the procfs file system must be mounted on /proc.  Neither of these seems like a big issue to me.
Testing
I tested this manually on my Windows machine but I'm not entirely sure how to set up a regression test for it.  AFAICT, all the tests are run in both Linux (case-sensitive) and Windows (case-insensitive) environments, and any test for this issue would only work in the latter and break in the former.

This change is

Thanks for filing! Before we go down this route, is there anything that we can do on our end to sanitize the input before we send it to Rollup? Or, is this really a bug in Rollup (that it has this case-sensitivity issue)?
I'm fine to pursue the path you've given here, but would have a slight preference for avoiding adding new limitations/complexity when this is an issue that affects a smaller number of users.
Also if you can, I'd love a few more details on how you've tested the fix. It sounds like this is an issue that you were originally seeing on your case-sensitive Windows machine, but are you saying that our Windows CI environment isn't case-sensitive?

It's hard to say where exactly the bug is but I think Snowpack is the only place where we're likely to be able to effect a fix.  Rollup needs to treat module paths as case-sensitive since it might be on a case-sensitive file system where "foo/bar.js" and "Foo/bar.js" are in fact different modules.  Node has, for whatever reason, decided to use the current cwd case as authoritative throughout its own libraries; this doesn't make sense to me but I assume they've got their reasons and, given that this was an intentional decision a while back, we're unlikely to convince them otherwise.  Rollup is using canonically cased paths for keying its internal maps -- I suspect they either use their own file system traversal libraries, or fix up the results obtained from Node's libraries.  I doubt it would be productive to try to convince them to use cwd-flavored casing, not least because it's easy to canonicalize a path, but hard to "cwd-canonicalize" one.
Hence, it's up to Snowpack to make sure that any paths we feed into Rollup have canonical casing, no matter how they were obtained.  The changes I propose here are one way to do this, by fixing the paths at the "source".  Another approach would be to canonicalize the paths as we build Rollup's option objects.  The latter is cleaner, in that we only touch one spot in the code.  I went with the former as it may prove more robust going forward should the module paths make their way into Rollup through some new code flow (e.g., through a newly added plugin or some such).  I'm happy to go with either one, though.  (Or both, for good measure!)
I'm not sure why you say that this would affect a small number of users -- anyone on a case-insensitive file system is potentially affected, which means all Windows-based developers and most Mac-based ones I believe (though perhaps Mac shells forcefully canonicalize cwd?).  The solution also doesn't introduce any limitations or external complexity, and fairly minimal internal complexity.
I observed the issue because my repository was checked out into C:\Code\workspace... but my shell's cwd was C:\code\workspace\....  On Windows, these are effectively equal paths, but Rollup doesn't know that and was treating modules on the first path (as deduced via transitive dependencies) as different from those on the latter path (as provided by Snowpack's entry points).  To reproduce this in a regression test you'd need to cd to a test directory while using the "wrong" case; obviously this would only work on a Windows platform (or Mac, but you don't appear to test on those), since on Linux the badly cased directory wouldn't be found.  I'm not sufficiently familiar with Snowpack's test framework to figure out how to limit a test to a given platform, and how to artificially make it run with a non-canonical cwd.

I'm not sure why you say that this would affect a small number of users

It sounded like we wouldn't be able to test this in CI, did I misread that in your original post? That's where a lot of my caution here is coming from. Can you think of a way that we could write the test that would cover this behavior on either our Windows or Linux CI runner? It's fine if running the test in a case-insensitive env is essentially a no-op, as long as it's running at least once in the case-sensitive env.

All right, I wrangled together a test -- PTAL.  Note that it's only effective in case-insensitive environments.  Please let me know what else I can do to convince you that this is not some esoteric corner case.  Thanks.


Bug: select and cascader choose option show value not label

https://github.com/Bennnis/T-UI/pull/32


Feature Request: Select All

Thanks for the great library.
Is it possible to expose selectall functionality? I can achieve this by using the following on click, but it seems like programmatic control over range selections is possible with the library.
document.execCommand("selectAll", false, undefined);

Something like Edit.select(range)?

I'm a little hesitant to include this. The logic exists but you have found a valid way to select all content natively without any quirks that need to be smoothed over.
In general, I'm struggling to find use-cases for selecting a specific range beyond selecting everything, which is easily possible. I think selecting a specific range may not be a common use-case. I can be convinced otherwise, but currently I'm on the fence on implementing this 😅

The reason select all might want to be included is that while the above "works", it's a little buggy in terms of UX and I'm not sure we could rely on it to work in a production setting. The execcommand's success relies on the field being the currently focused field, and if there are other ui updates scheduled (popups etc, other events) then I've noticed just in dev mode it's behavior isn't reliable.


Shorten timeout at COMMIT phase when justified

When justified by a strong quorum, shorten the timeout at COMMIT phase.
Fixes #242

Code change looks good but I think we need to address synchronous sends to self (#316) to remove the race with alarms in order for the fuzz to pass.

Hmm, still failing

Hmm, still failing

What changed?

What changed?

I suspect either changes to storage power or this is simply a fluke. I have captured https://github.com/filecoin-project/go-f3/issues/340 to get to the bottom of it.


Feature GitHub api integration

I have added the missing env key ... Please make a new PR .
Thanks.

Okay thanks!


Dot Plots not grouping correctly

The dot plot below is being grouped by Sex when it should be grouped by test :(

I think that the dot plots are rarely used and have a few issues. Considering removing them altogether.


Rewrite Compute sample for new API

Found in #5582
Compute sample should use the new API.

@donmccasland Please confirm we own compute.

@danoscarmike - should have been ping'd Dan.
I'm presuming this should also go to @rsamborski for vis and to loop in his Compute team and @Sita04

@Sita04 please take a look at this issue


Update cats-effect to 2.3.0

Updates org.typelevel:cats-effect from 2.3.0-M1 to 2.3.0.
GitHub Release Notes - 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!

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

labels: library-update, semver-pre-release

Codecov Report

Merging #321 (ffa259b) into master (7561093) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #321      +/-   ##
==========================================
- Coverage   84.56%   84.48%   -0.08%
==========================================
  Files         106      106
  Lines        1276     1276
  Branches       25       25
==========================================
- Hits         1079     1078       -1
- Misses        197      198       +1

Impacted Files
Coverage Δ

...ore/src/main/scala/net/BufferedMessageSocket.scala
92.30% <0.00%> (-3.85%)
:arrow_down:

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 7561093...ffa259b. Read the comment docs.


Ranker tag not displayed

Describe the bug
When I following the instruction in https://labelstud.io/tags/ranker.html to create a ranker tag, I found it not be displayed in the interface.
To Reproduce
Steps to reproduce the behavior:
1.Save the example JSON below as a file called example_ranker_tag.json.
2.Upload it as a task on the Label Studio UI.
3.Set up a project with the given labeling configuration.
Expected behavior
Show Ranker tag normally for the RLHF annotation
Screenshots

Environment (please complete the following information):

Ubuntu 18.04.6
Label Studio Version 1.7.1

Additional context
Really important for building a RLHF annotation platform

@Homura1212 Unfortunately Ranker tag was broken long time ago and we are working on redesigned of it. I agree, this is a very important thing now.

can you please refer my discussion similar to this please #4150


tests: add unit tests for all the core interfaces and cache related stuff

self-explanatory...

integration > unit


integrate sound system

Audio for sfx already included using http://www.superflashbros.net/as3sfxr/ to design and export sounds.


Add media-src 'self'; to CSP

For various browser bug reasons, some medias doesn't load in some browsers without content-security-policy header having media-src 'self'; in it. This adds it.
Signed-off-by: Sami Olmari<EMAIL_ADDRESS>Tested-off-by: Sami Olmari<EMAIL_ADDRESS>

Also tested by few other peoples that indirectly got this issue reported to me and all confirms this fix working.


Issue Reporter window is blank

On the latest Insiders, the issue reporter is blank for me.

Open the Issue Reporter ("Help: Report Issue" command).
:bug: Issue Reporter doesn't load.

/confirm

Doesn't happen with OSS built from main right now.

it still happens building from sources, this is the commit ed63059a4752d031535182b1d6eb37719483fbbd that broke it
created a PR fixing it https://github.com/microsoft/vscode/pull/148456


[Patterns] Language tests

I believe we have the desired level of test coverage for patterns, based on co19.


Option to make exponent divisible by 3 (for metric units reading)

Problem Statement
When working with metric units it's usefull to see scientific notation with exponent divisible by 3. Number 3,3898e-4 is not as clear as 338.98e-6 is clearly 338.98µ units.
Evidence or User Insights
You can try "1/2950=" in standard or scientific mode to see result. It was discussed in #226 also.
Proposal
For calculations in metric units it's better to see results in format similar to T, G, M, k, m, µ, n, etc. units.
Goals
User should be able to set exponent divider to make scientific notation easily readable. Default value should by 1 to preserve actual state with ability to enable predefined value 3 (or even other value) to read metric results more easily.
Non-Goals
It's not meant to make any kind of units conversion.
Low-Fidelity Concept
If exp>0 then exp=int(exp/div)*div
if exp<0 then exp=int((exp-div+1)/div)*div
Requested Assignment
If possible, I would like to implement this.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Instead of calling it "metric units reading", I think the preferred term for numbers in scientific notation with exponents divisible by 3 is 'engineering notation'. I believe an ENG/SCI toggle button is the most appropriate user interface integration for this feature.

Instead of calling it "metric units reading", I think the preferred term for numbers in scientific notation with exponents divisible by 3 is 'engineering notation'. I believe an ENG/SCI toggle button is the most appropriate user interface integration for this feature.

Engineering units are also feets, pounds, etc. which are not using metric system. Better to use MET/SCI toggle.

Thanks for the idea! It should be pretty straightforward to expose this in Scientific mode, but a couple details to figure out. Some comments:

Regarding naming, we believe we should move forward with engineering notation.

We believe the best approach might be updating this display mode UI to be clearer about switching between display modes: automatic (AUTO, default), scientific notation (SCI), and engineering notation (ENG), but are open to other suggestions.

Perhaps we could leverage the pattern we have in Programmer Mode to cycle between modes:

This change may make it easier to address #226 (if we go with the approach above, and can introduce a 4th display mode that turns off auto).

It's called Engineering Notation and often abbreviated ENG mode on calculators.
Usually there is the option between standard notation (showing integer values for lower values), scientific notation (one digit before the comma) and engineering notation (up to three digits before the comma). Here is an example from the HP Prime calculator:

Engineering Notation from Wikipedia

Engineering notation (often named "ENG" display mode on scientific calculators) differs from normalized scientific notation in that the exponent n is restricted to multiples of 3. Consequently, the absolute value of m is in the range 1 ≤ |m| < 1000, rather than 1 ≤ |m| < 10. Though similar in concept, engineering notation is rarely called scientific notation. Engineering notation allows the numbers to explicitly match their corresponding SI prefixes, which facilitates reading and oral communication. For example, 12.5×10−9 m can be read as "twelve-point-five nanometres" and written as 12.5 nm, while its scientific notation equivalent 1.25×10−8 m would likely be read out as "one-point-two-five times ten-to-the-negative-eight metres".

Exactly what i am lookng for! Even better if the user could input Engineering Notation with alphanumerics as well. Just like: 1k, 5.3M, 2u5

I implemented engineering notation using the advice @grochocki gave. It could be extended to add an 'OFF' option to address #226 as well.
Would I be able to make a pull request for my implementation to be reviewed and merged?
https://github.com/microsoft/calculator/assets/76914681/9cd69755-b0ef-4b25-930a-375d05eeb3b6


Automatically insert reference during completion if there's a matching variable

This pattern comes up a lot when using e.g. ash.
struct CreateInfo {
    /* ... */
}

fn main() {
    let create_info = CreateInfo { /* ... */ };
    foo<complete>
}

fn foo(create_info: &CreateInfo) {
    /* ... */
}

Current:

Desirable:

Thanks for the suggestion! This was added in #5846, so I'm going to go ahead and close this issue.


Wrong balance display due to data caching

Describe the bug

[x] Wrong balance display due to data caching
[ ] Still loading when open NFT Tab

To Reproduce
Video: Video Detail: https://www.loom.com/share/c9a6a9f9b99d4ebd9d34a27d39e9da94
Desktop (please complete the following information):

OS: macOS Monterey 12.4
Browser Chrome
Version 103.0.5060.134 (Official Build) (arm64)

Version test: https://drive.google.com/file/d/1bBF9j7Qpfx6zlGS6td4sszldf776ElMY/view?usp=sharing

validation passed! (Crowdloans)


Pause container for a long amount of time

I am using a stack with mesos, marathon and docker.
I have a docker container running a registry on -p 31800:5000
The registry data dir is about 18GB.
To backup the data dir, I use :
docker pause $cid
tar cpzf registry.tar.gz /srv/registry
docker unpause $cid

For some reason, the container fail to start after that.
Reason is that the port 31800 is already allocated, I don't know if this error comes from docker itself, or if marathon crash the app.
Is there anyway to get this work ?

@kopax I'm afraid we can't help you here since your issue is with the docker engine.  I suggest you post a question in the docker channel on IRC.
If port 31800 is used, lsof should be able to tell you which app is using it.


Unknown HCLK_R bits

After running BUFGMUX example on Nexys Video with limited grid I obtained the bitstream, which is not working correctly on the hardware. The diff_fasm step returns no differences between the FASMs generated by the vivado and symbiflow. In addition, a bitstream generated by the vivado in the diff_fasm step works correctly.
Here is the output of thebit2fasm.py script used on vivado diff_fasm bitstream:
{ unknown_bit = "00401b9a_43_4", unknown_segment = "0x00401b80", unknown_segbit = "26_1380" }
{ unknown_bit = "00401b9a_55_20", unknown_segment = "0x00401b80", unknown_segbit = "26_1780" }
{ unknown_bit = "00401b9a_98_14", unknown_segment = "0x00401b80", unknown_segbit = "26_3150" }

This leads to HCLK_R_X137Y130 in the xc7a200tffg1156-1 tilegrid.json.
The features related to the HCLK_R are generated by the 058-pip-hclk fuzzer.

@rw1nkler Can you please create a PR that replicates this issue?  Thanks.