Implement provider for chrome debugging protocol (closes #1417)

\cc @inikulin @AlexanderMoskovkin

Feature summary
Usage
testcafe 'cdp:config' test.js
E. g.
testcafe 'cdp:local;headless;device.name=iPhone 4' test.js
testcafe 'cdp:local;headless;device={name=iPhone 4;orientation=horizontal}' test.js
Config
local:      Indicates that a local browser instance should be started by provider. Also triggered automatically by specifying path or arguments.
args:      Optional user-specified arguments
incognito:  Indicates that a local browser instance should be started in incognito mode,
headless:  Indicates that a local browser instance should be started in incognito mode,
host:      Specifies a host to connect via CDB. May be specified with port, e.g. localhost:9222 ,
port:      Specifies a port to connect via CDB,
path:      Path to the chrome browser,

userDataDir:    Specifies temporary directory for storing user profile to allow multiple running Chrome instances,
noTempUserData: Don't create temporary user profile directory,

emulation: Should enable DevTools Device Emulation,

device: {
    screen: {
        width:   Device screen width,
        height:  Device screen height,
        density: Device screen physical/logical pixel ratio,
    },

    mobile:      Specifies that device is mobile,
    orientation: 'vertical' or 'horizontal',
    name:        Use named device preset from DevTools configuration,

    userAgent:      Specifies device useragent string,

    touch:          Device has touch features
}

Why we need separate cdp alias, can we use chrome instead, e.g. chrome:headless?

FPR

remote: Indicates that a local browser instance should not be started by provider.

Why we need this. Isn't specifying host enables remote debugging

If you've started browser with CDB manually (on the port 9222), you can just write remote instead of host=localhost:9222. Also, you can use remote to connect Chrome on Android if you have a device connected by USB.

noCdb: Start local browser instance without Chrome Debugging Protocol support (no headless, no emulation)

Why we need this in the first place. isn't it the same as testcafe chrome test.js?

I would like that CDB will be enabled by default, because it gives better screenshots: even if Chrome is in background, and without browser UI, on all platfroms (currently we don't include Chrome UI in screenshots only on Windows).

args: Optional user-specified arguments

AFAIK we already have the way to specify args for providers, so why we need this?

You can specify arguments in the old way if you don't use any other options. Without it it will be harder to extract provider-specific options from user-supplied args. So if you don't plan to use emulation/headless/etc. you can still use
testcafe 'chrome --my-arg --my-arg2'

But if you are using emulation, you will use args option (live example from V. Gein):
testcafe "chrome:args=--disable-infobars --disable-session-crashed-bubble --kiosk;device.name=iPhone
 4"

host: Specifies a host to connect via CDB. May be specified with port, e.g. localhost:9222 , port: Specifies a port to connect via CDB

Having a host is just enough, moreover it makes sense to name it `remote, e.g.:

CDP requires a free TCP port, so if you have port 9222 occupied, you may want just to start Chrome on another port:
testcafe 'chrome:port=9222'

vs
testcafe 'chrome:remote=localhost:9222'

@AndreyBelym Still looks a bit messy for me. Let's ask others: \cc @DevExpress/testcafe

It is difficult to read and understand for me.
We have two feature requests: device emulation and headless mode.
We are implementing their using Chrome running via Chrome debug protocol.
So, browser provider name should tell about this.
chrome:cdp;mobileEmulation={name=iPhone 4;orientation=horizontal}
chrome:cdp;headless
chrome:cdp;args={port:2222}

cdp suffix indicates that chrome run via CDP instead of normal run via command line
mobileEmulation and headless mode are most interesting options and they should be placed in top config level.
arg - for all parameters as host, port and etc.
I use mobileEmulation term as in ChromeDriver documentation

I would like that CDB will be enabled by default, because it gives better screenshots: even if Chrome is in background, and without browser UI, on all platfroms (currently we don't include Chrome UI in screenshots only on Windows).

I don't think so. It is a big change. It's need to measure but I propose that Chrome running via CDP is slower than Chrome running via command line.

testcafe "chrome:args=--disable-infobars --disable-session-crashed-bubble --kiosk;device.name=iPhone

Some of these options we can add by default. For example, --disable-infobars.
Same as in https://github.com/DevExpress/testcafe-browser-tools/commit/9966ffa7f97498888623e65bd5c57d05ca4f6fa3
Aso I don't understand why we need incognito mode option.

I agree with @miherlosev in most points. Let's discuss it together today

Summary updated

After one more review we want to propose more plain syntax for this features:
The common pattern is the following:
chrome[:pathToBrowser][:headless][:emulation][:arg1=value;arg2=value]
path - for the case if you want to run headless or device emulation mode for a portable browser;
headless - enable headless mode;
emulation - enable device emulation;
Available arguments:

for device emulation: width=<number>, height=<number>, density=<number>, mobile=<bool>, orientation=<vertical|horizontal>, userAgent=<string>, touch=<bool>.
cdp args: cdpPort=<int>

Here are some examples:
// Default headless mode
"chrome:headless"

// Device emulation
"chrome:emulation:device=iphone 6"
"chrome:emulation:width=100;height=200;density:1;mobile=false;orientation=vertical;userAgent=abc;touch=true"

// Headless device emulation
"chrome:headless:emulation:device=iphone 6;cdpPort=9223"

// Headless with cdpPort setting
"chrome:headless:cdpPort=9223"

// Headless/device emulation for a portable version
"chrome:path/to/chrome:headless:cdpPort=9223"

What do you think? /cc @DevExpress/testcafe

Fine

FPR

@testcafe-build-bot \retest

FPR


Git source crashes on ability "plus_high_five" but deployed version does not.

Tried updating constants but that didn't work (it has it as high_five).
So I went with adding a default value for when abilitiesByName fails out.
this.definition = abilitiesByName[this.refname] || {dname: this.refname};
Example match id:<PHONE_NUMBER>
I'd like to have my code match your implementation can you commit what you did?

Good catch! There's a few new Dota+ ablities that are not handled properly.
The reason it works on the deployed version is that abilities are not yet introduced there, there's no fix as of yet. Will fix and push shortly!

I've applied your fix in 7dd3c117c53258b3b244178c2fc899b3253755bf, thanks! In addition, 99d913f2f48d1c02e3cf7576f698c78ad1b3f4b6 prevents these Dota Plus abilities from showing as regular hero abilities.
Have also deployed the latest version / commit.

Awesome! Thanks, I've pulled these into my tree.


Requester should check the size of the returned certificate chain

https://github.com/DMTF/libspdm/blob/1e2350efd9c08b8be9d8624fdaf592a8176243bb/library/spdm_requester_lib/libspdm_req_get_certificate.c#L243-L244
total_responder_cert_chain_buffer_length is the size of the certificate reported through the CERTIFICATE response fields. Its value should be checked against the value returned in the certificate chain's Length field.

This has been checked in https://github.com/DMTF/libspdm/blob/main/library/spdm_crypt_lib/libspdm_crypt_cert.c#L1134


A11y_Azure Tools Storage Explorer_Add a resource via Azure Active Directory_Connection Summary_AI4D : Name "To get missing image description...." is assigned to decorative image.

GitHubTags: #C&AI_Aug2020_BMAzure Tools Storage Explorer;#A11yMAS;#A11YTCS;#DesktopApp;#Azure Tools Storage Explorer;#WCAG4.1.2;#Win32;#A11ySev2;#Benchmark;#AI4D;#FTP;#WCAG1.3.1;#NVDA;#JAWS;#BM-Azure Tools Storage Explorer-win32-Aug2020;
Environment Details:
Application Name: Azure Tool Storage Explorer
Application version: 1.14.2
Windows Version - Windows10
Repro Steps:

Launch Storage Explorer
Navigate to "Open Connect Dialog" pane present on left side of screen and hit enter.
Select "Add a resource via Azure Active Directory(Azure AD)" radio button and hit enter on Next button.
"Attach with azure AD" screen opened and navigate to Next button and hit enter.
Enter the required details in edit boxes and hit enter on "Next" button.
"Connection Summary" pop up will appear.
Open AI4D and Check the name assigned to graphic label below Tenant label.

Actual:
Name "To get missing image description...." is assigned to decorative image graphic label below Tenant label.
Screenreader is reading name for decorative image graphic label below Tenant label.
Expected:
Name should not be assigned to decorative image graphic label below Tenant label.
User Impact:
Screen reader users will get incorrect information and get confused
Attachment
https://microsoftapc-my.sharepoint.com/:i:/g/personal/v-prraha_microsoft_com/EQFa_5Q-wtpIlVK1XNXcAn0BHNCwXwsnW-ZbeSWmYpMuZA?e=CSP6jH

The image in question is content of a ::before pseudo-element, and screen readers are generating a name for it. Recommendations I'm seeing say we shouldn't use pseudo-elements even for decorative content.

@craxal We are not able to find the exact control. Could you please help us.

As per the confirmation on to the mail the product has been changed and this test case does not apply anymore. So, closing the bug. Thanks
Github Tags:#Closed;


TypeError: 'type' object is not subscriptable

While mypy checks completes successfully, the actual python interpreter (Python 3.10.1) rejects parameter of type asyncpg.Pool[asyncpg.Record] with error TypeError: 'type' object is not subscriptable.
Is:
pool: asyncpg.Pool[asyncpg.Record]
a correct way to declare a parameter?

Since the generic types are only in stubs, you will need to either us from __future__ import annotations at the top of the file or you'll need to surround it in quotes. For more information, you'll want to check out the section on using classes that are generic in stubs but not at runtime in the mypy docs (but it applies to all type checkers).


⚠ Bofrost.fr has degraded performance

In b44f622, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 857 ms

Resolved: Bofrost.fr performance has improved in b41f044 after 6 minutes.


SOLR-16149: remove Lucene/Solr language in the Solr Ref Guide

https://issues.apache.org/jira/browse/SOLR-16149

Oops, didn't see #799 whilst multi-tasking.


Version 0.17.2

v0.17.2 (2020-10-25)
Features

Allow the http module to be used without a tcp listener (c45353d2)
Format seq expressions without seq (5c0cec2d)
Compile block expressions as monadic sequences (bce59737, closes #884)
std:

add Option assertions to std.test (28e5053f)
add modulo functions to int and float (92f188ab)

Bug Fixes

Recognize raw string literals without any # (4d66fbb3, closes #885)
Prevent zero-argument functions from being created in Rust (e91ea06d, closes #873)
Give tuple fields a span (2a1c2c71)
xor_shift_new inconsistent description (591b64b3)

bors r+


Keep the catalogue docs next to the catalogue

Definitely think we should have these, but I’ve cancelled the build until we get the Travis logic in.


Consider normalizing equivalent inference parameters

I would find it very helpful if common inference parameters, such as temperature or max. tokens, had normalized argument names. These could then be automatically mapped to the required configuration names for each provider.
For example:

AWS provider uses topP and maxTokens
OpenAI provider uses top_p and max_tokens

This discrepancy necessitates the use of if-else statements to handle different providers, which seems counter to the framework's goal of streamlining integration.
While I am not a professional developer, here’s an idea for implementation:
# aisuite/providers/__init__.py
NORM_KWARGS_AWS = {
    "max_tokens": "maxTokens",
    "temperature": "temperature",
    "top_p": "topP",
    "stop_sequences": "stopSequences",
}

# aisuite/providers/aws_provider.py
from aisuite.providers import NORM_KWARGS_AWS

for norm_key, value in kwargs.items():
    key = NORM_KWARGS_AWS.get(norm_key, norm_key)
    ...

See: https://github.com/viictorjimenezzz/aisuite
Thank you for considering this request!

I think some kind of translator module if we create, will help in this aspect. The translator module will have the state or the fields of the config.

One good thing would have been if we could track the config classes of each Provider 😅. Then hard coding normalized fields won't be required. But yeah, I don't we can use it here as these providers are being used as packages, And we have no control over the Python environment paths (As they will be user-specific)

Then whenever we call the Provider class and with the set of configs, the translator module will parse the user-given configs to the proper field name w.r.t. the Provider class. Also, for field data type validation can be easily performed by Pydantic.

This is my initial approach. ConfigTranslator will use the providers_config.yml which will contain the mapping between different Providers' config and the NORM_KWARGS @viictorjimenezzz mentioned before. And as we keep adding more Providers we can update the YAML file.


After updating the Numba from 0.43 to 0.51.2, the parallel code is displaying the error

I have recently updated Numba in Anaconda at my machine from 0.43 to 0.51.2. The code which was running successfully with parallel=True is not running with prange. When I have replaced prange with range, the code executes but sequentially. Also, the sequential execution is taking much greater time with parallel=True in comparison to parallel=False.
The error message that I am getting in correspondence to my my program using parallel=True and prange is
TypingError: Internal error at <numba.core.typeinfer.CallConstraint object at 0x000001F463923C18>.
Failed in nopython mode pipeline (step: convert to parfors)
maximum recursion depth exceeded while calling a Python object
During: resolving callee type: type(CPUDispatcher(<function case1_flagi1_j1 at 0x000001F462EBC620>))

Enable logging at debug level for details.

Thanks for the report - can you provide an executable sample of code that allows reproducing the error please?

Thanks for the report - can you provide an executable sample of code that allows reproducing the error please?

I have faced it in the following code
@njit(cache=True,parallel=True)
def case1(edge_list,signs,i,j,rows,ppp,nnn,ppn_I,ppn_II,ppn_III,ppn_IV,pnn_I,pnn_II,pnn_III,pnn_IV):
	a1,a2,a3,a4,a5,a6,a7,a8,a9,a10=0,0,0,0,0,0,0,0,0,0
        for k in prange(j+1,rows):
                if (edge_list[i,1]==edge_list[k,0] and edge_list[j,1]==edge_list[k,1]) or (edge_list[i,1]==edge_list[k,1] and edge_list[j,1]==edge_list[k,0]):
                    if np.sum(signs[i]+signs[j]+signs[k])==3:
			a1+=1
                    elif np.sum(signs[i]+signs[j]+signs[k])==1:
                        x=np.zeros((3,1),dtype=np.int8)
                        if edge_list[i,1]==edge_list[k,0]:
				x[0]=signs[i]
                        elif edge_list[j,1]==edge_list[k,0]:
				x[0]=signs[j]
                        if x[0,0]==1 and x[1,0]==1 and x[2,0]==-1:
				a3+=1
                        elif x[0,0]==1 and x[1,0]==-1 and x[2,0]==1:
				a4+=1
                        elif x[0,0]==-1 and x[1,0]==1 and x[2,0]==1:
				a5+=1
                    elif np.sum(signs[i]+signs[j]+signs[k])==-1:
                        x=np.zeros((3,1),dtype=np.int8)
                        if edge_list[i,1]==edge_list[k,0]:
				x[0]=signs[i]
                        elif edge_list[j,1]==edge_list[k,0]:
				x[0]=signs[j]
                        if x[0,0]==-1 and x[1,0]==-1 and x[2,0]==1:
				a7+=1
                        elif x[0,0]==-1 and x[1,0]==1 and x[2,0]==-1:
				a8+=1
                        elif x[0,0]==1 and x[1,0]==-1 and x[2,0]==-1:
				a9+=1
                    elif np.sum(signs[i]+signs[j]+signs[k])==-3:
			a2+=1
        return ppp,nnn, ppn_I,ppn_II,ppn_III,ppn_IV,pnn_I,pnn_II,pnn_III,pnn_IV

@njit(cache=True)
def signed_directed_triangles_numba1(edge_list):
    edge_list=edge_list[:,0:3]
    rows_edge_lists=edge_list.shape
    rows=rows_edge_lists[0]
    signs=np.zeros((rows_edge_lists[0],1),dtype=np.int8);
    for ii in range(0,rows_edge_lists[0]):
        if edge_list[ii,2]>=0: 		            signs[ii]=1;
        elif edge_list[ii,2]<0:			    signs[ii]=-1;
    edge_list=edge_list[:,0:2]
    extracted_triangles=np.zeros((rows_edge_lists[0]-2,10),dtype=np.uint16);
    for i in range(0,rows_edge_lists[0]-2):
        flag_i1_j1=0
        ppp,nnn=0,0
        ppn_I,ppn_II,ppn_III,ppn_IV=0,0,0,0
        pnn_I,pnn_II,pnn_III,pnn_IV=0,0,0,0
        for j in range(i+1,rows-1):#rows_edge_lists[0]-1):
            if edge_list[i,0]==edge_list[j,0]:
                flag_i1_j1=1
		ppp,nnn,ppn_I,ppn_II,ppn_III,ppn_IV,pnn_I,pnn_II,pnn_III,pnn_IV=case1(edge_list,signs,i,j,rows,ppp,nnn,ppn_I,ppn_II,ppn_III,ppn_IV,pnn_I,pnn_II,pnn_III,pnn_IV)
        extracted_triangles[i]=np.array([ppp,nnn,ppn_I,ppn_II,ppn_III,ppn_IV,pnn_I,pnn_II,pnn_III,pnn_IV],dtype=np.uint16)
    triangles_each_type=np.sum(extracted_triangles,axis=0)
    print(triangles_each_type)
    return

edge_list=np.array([[6,2,4,1],[6,5,2,2],[1,15,1,4],[4,3,7,4],[13,16,8,5],[13,10,8,6],[7,5,1,7],[2,5,-1,8],[2,100,-10,9],[13,100,10,10],[16,10,-6,11],[16,100,-4,12],[5,100,-3,13],[13,200,-4,14],[100,200,3,15],[7,300,1,16],[7,400,1,17],[5,300,-5,18],[5,400,3,19],[111,222,-4,20]],dtype=np.int16)
signed_directed_triangles_numba1(edge_list)

I have also discussed about this error on https://numba.discourse.group/t/re-execution-of-the-same-jitted-function-causes-re-compilation-for-same-type-of-data/300 and Mr.Luk-f-a has also agreed about this error. He advised me to post at here. I have slightly changed the code from there because the code that I posted there was not for parallel execution and the code that I have written above is for parallel execution and to avoid race conditions.
I have uninstalled on my machine the Anaconda with the latest update of Numba as I am not able to roll back the version 0.43.1 and re-installed March 2019 version of Anaconda (having Numba version 0.43.1) such that the above code is working there.
I have one more query about the above code on version 0.43.1 of Numba that it is working sequentially (without any parallel=True) option in 2.7 seconds, while, with parallel=True, it is executing in 9.14 seconds. Due to this inefficiency, I have updated Numba on my machine and I got the above error in version 0.51.2 for the above code. Earlier, I have also installed the latest version of Anaconda but, in that version, I am also getting the same error where I have also noted that the version of Numba is 0.51.1 and hence I have re-installed March 2019 version of Anaconda with 0.43.1 version of Numba.

Thanks for the report. This is a bug. Reproducer:
from numba import njit, prange
import numpy as np

@njit(parallel=True)
def foo():
    A = B = 0
    for k in prange(1):
        if k == 2:
            A += 1
        else:
            x = np.zeros((1, 1))
            if x[0, 0]:
                B += 1

foo()

this gets stuck here when working out the reductions:
https://github.com/numba/numba/blob/5dcb2f1252f30f30060725171facee0217685394/numba/parfors/parfor.py#L3524-L3529
Because defs contains {'A.3': Var(A.2, issue6414.py:9), 'A.2': Var(A.3, issue6414.py:9)} which is a mutually referent structure. I imagine that this is SSA related as these are just versions of the same variable.
CC @DrTodd13


Next/20200602/v4

#241 and #229
Replaces #240
(CI expected to fail for master)

#229 breaks this for 5.0


[BUG] verbose flag does not work in splunk_eventgen cmd line tool

Describe the bug
client cmd line tool parameter -v/--verbose does not work
To Reproduce
Steps to reproduce the behavior:

use . pip to install eventgen
splunk_eventgen -v generate

Expected behavior
the log level should be debug and all log should be flushed to stderr
Actual behavior
get an error in log

2019-04-20 11:03:23 eventgen        INFO     MainProcess Key 'verbose' in stanza 'global' may not be a valid setting
and the log level is info, no debug log

Screenshots
n/a
Sample files and eventgen.conf file
any file is ok
Do you run eventgen with SA-eventgen?
No
If you are using eventgen with pip module mode (please complete the following information):

python version: 2.7
OS: macos 10.14
Virtual Env is used: yes
Eventgen Version 6.3.4

Additional context

Fixed.


dleyna: purge recipes that can be found in meta-oe

As of krogoth branch of meta-openembedded dleyna recipes can be found in
the upstream repository. Meta-bistro dleyna recipes are not being actively
used and as such can be removed.
Signed-off-by: Gordan Markuš<EMAIL_ADDRESS>

It's a no-brainer I guess after @DunderRoffe 's comments are addressed.

Ok as long as it is intentional. I misinterpreted the commit message thought dlenya recipes ment all of them not the specifics that do not exist upstream. Maybe the short log should say "remove dlenya recipes / services" instead as we do not really care even if they are upstream?

@DunderRoffe thanks for the explanation, I'll rewrite the commit message to make it more clear.

As of krogoth branch of meta-openembedded dleyna recipes can be found in
the upstream repository.

I had to read it many times to be sure what you mean. An ',' after meta-openembedded, would make it much more readable. :)

Meta-bistro dleyna recipes are not being actively
used and as such are not being maintained.

Same here: after "such".

Once the PELUX platform decides
to use dleyna once again the appendments will be found in the same place.

Same here: after "again".
appendments? That's not a word. :)
Also if you could specify the "same place", that would be even better.
Yes, I'm a nitpicker. :D

@zeenix fixed. An appendment is indeed a word.

@zeenix fixed. An appendment is indeed a word.

Thanks. Interesting. For some reason it's marked red here.

Sorry, I'm a bit confused on this part:

the appendments will be found in the same place:
layers/multimedia-layer/recipes-multimedia/dleyna/

You mean, they'll still be added to the same place? You're removing them so they won't be anywhere to be found? If you mean in the upstream, I don't think the location in upstream is important enough to be mentioned here and will only confuse idiots like me. :)

Quote: "Once the PELUX platform decides
to use dleyna once again, the appendments will be found in the same place:
layers/multimedia-layer/recipes-multimedia/dleyna/"
I am not sure how to make it more clearer :).
Once we start using dleyna as we probably will, we might want to append the recipes, which we will, the appendments will be found in that path. So from that commit message you could historically see why the content changed so drastically.

I am not sure how to make it more clearer :).

How about: "found in that path" -> "re-added to the same location" and rest the same as you wrote in your comment? Also, git is not very picky on path and more about content so I'm not sure this point is important.

@zeenix as per our discussion I've removed the last part to minimize the confusion.


Get infura key from infura.io

In order to talk to eth mainnet or testnet (rinkeby) we will need an infura project ID.
steps:

sign up for free on infura.io
paste project id in this issue

b75d6ca3e25c45d384c975ae8d468f05


Correctly mark module names as non-recommended

What's in this pull request?

[CodeCompletion] This pull request fixed our module completion bug where we erroneously stroke module names that have not been imported to a source file. rdar://25416494
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci test

@swift-ci test


[update]app.go-init-genesis

SupplyModule was missing in SetOrderInitGenesis

This pull request fixes this issue. Now I'm closing it.
#154


Don't enable API keys for things like Google Maps and email unless there is a valid configuration in the install script

Currently the install script will let you just leave some lines blank or fill in "None" and Docassemble will error out on certain screens that depend on the APIs that get "turned on".
We can improve that by being more explicit in the install script.


Use TableOperations.modifyProperties in more places

The changes in this PR make use of the new modify properties method introduced in #2799

Your suggestions/comments should be addressed @cshannon, if you would like to take another look.


Add world submenu entries and checkboxes

closes #187

We already wrote some tests and added the submenu items to the first layer search.
What is still missing (and most of the complexity of this issue) is to add the submenu entries as items with child items in Algernon (where you can tab in)

Still missing:

Add 4 window menu entries under line back in (probably best as separate actions on the window type?)
Maybe add actions toggle, enable and disable?
Fix tests

Unnamed can be found (the project name in the middle)

Discussion: Filtering the window items: Our idea was to filter / remove the items in the NamedWorldMenuEntry. Because they are dynamically generated, we would have to "inject" / pass a filter block. This feels a bit ugly, because it has some "zig-zag": The ALGWorldMenuType looks for the window-entry -> passes the filter block to the entry -> the entry filter the items. Any other ideas?

Regarding the bug:

We think it might have something to do that on every change in a menu a new menu instance is generated. We probably store the old instance somewhere in the code or whatever - these are then not up to date and don't have submenus anymore which is why the items are not found again until a menu item is opened.

Bugs should now be fixed

It's just another CI problem. All test pass on my machine.

Concerning the latest commit: I think that it's a good idea not to cleanup the morphs after updateMenu, because

This destroys the owner field of submorphs. That's why we had to catch the exception dirtily in the first place.
When you open a dynamic world menu, then open algernon, the list gets cleared. This now no longer happens.

allEntries exists on ALGNamedWorldMenuEntry and ALGWorldMenuType - are both necessary?


Could marker object point to correct level on the y axis (with positioning inBar)?

First of all, thank you for creating this python framework for the javascript TV library. I'm a hobby coder and this has made my life much easier.
Description
The feature/enhancement would be to have an implementation for a marker with y-coordinates, or a line series where marker data (time, value, color, etc.) could be set by the user, and the line would be invisible and markers visible. This is discussed in the below forum:
https://github.com/tradingview/lightweight-charts/issues/127
In the below screenshot, the current behavior sets markers relative to the bar position, which can be inaccurate for use case of tracking events.
Ideally, if a transaction or event occurred within the bar at some y-value / price (e.g., @ 540.59). The marker should be anchored to that y-value.

Thank you for your continued work in this framework!

Hey @maximbelyayev,
As the issue you linked references plugins in relation to implementing this feature, I'm marking this as a duplicate.
Louis


When installing the plugin generates the following problem

https://github.com/schoenkaft/cordova-plugin-background-mode/commit/6fb1fb77da5eb76bc542c546e90d89693103416a


Parse RTF bodies

go-pst should support decoding RTF bodies by default.

References [MS-OXRTFCP]: Rich Text Format (RTF) Compression Algorithm


Can't have case as the last expression

I found out that whenever I have case expressions as the last expression I get the following error:
CallStack (from HasCallStack):
  error, called at src/parserlib/CommonParserUtil.hs:203:11 in genlrparser-<IP_ADDRESS>-94uLv9mXElrJeWoLxfW0Mm:CommonParserUtil

Example file: regression_1.rl
data F = { F Int };

main : Int = case (F 5) {
  F i => i
};

@tzbob Thanks for pointing this out.
You should omit the last semicolon after the case expression. Semicolons are only allowed between two toplevel things.
But interestingly, deleting the semicolon uncovered a bug in the typechecker. So I fixed it, and the new version is available in the commit 1ca9041
Thanks!!


Stickers and Wallpapers dont work

i installed the plugin, then uninstalled ClaudiaIDE since i had it, i changed to "Rias Crimson" theme and the colors work, but no sticker or wallpaper even though they are set to "true" in the doki settings, i tried manually linking them by finding the pictures and setting the path manually but that does not seem to work either.
Me setting paths manually:
https://imgur.com/a/aIxlEvg

That's a bit curious. Could you answer these questions for me, please?

What version of Visual Studio are you using?
What version of the plugin did you install?

That's a bit curious. Could you answer these questions for me, please?

What version of Visual Studio are you using?
What version of the plugin did you install?

VS 2022
Doki V2.2.0
should be the correct versions.
also, some new info i found, if i open the IDE/.cs file through Unity Engine OR by a finding the file by itself on my desktop, the sticker and wallpaper WONT appear.
BUT, if i open the IDE/.cs file through the VA 2022 launcher (just searching up Visual Studio 2022 Current on the windows search bar or something, and then click on my file which is under my "open recent" tab) then the stickers and wallpapers WILL appear.
Opening file through here will work:
https://imgur.com/a/Y8jUqel
and opening through both of these methods do not work:
https://imgur.com/a/soPorTl

Thanks for the supporting information!
I am fairly confident it is related to this issue https://github.com/doki-theme/doki-theme-visualstudio/issues/6#issuecomment-897197288
When I get around to updating this plugin in the near future, I'll look harder into solving this issue. Thanks for your patience!

Thanks for the supporting information! I am fairly confident it is related to this issue #6 (comment)
When I get around to updating this plugin in the near future, I'll look harder into solving this issue. Thanks for your patience!

seems like your right.


Lep 184 project boot

Nice! LGTM, but I'll leave to @LAA-NeilH to review

@starswan could you please list down the setup commands in the description

@ripan I think this adds what you're after: https://github.com/ministryofjustice/cfe-crime/pull/5 That PR is a branch off this branch, but can be merged in either order


Bug: Second ID missing in player games comparison results in avalanche of errors

Running the new function to check if two ID's have shared games creates a large number of error warning popups if the second ID input field is empty.

I can catch that exception, so no error pops up in the next version - You need to provide two player IDs to make use of that 'Check' method.

Also happens when both fields are empty, I've noticed :)
Didn't intend to trigger either of the two, but misclicked a button.


Use PNID with the same NNID name

I'd like to use my old NNID account with Pretendo, so I won't lose my associated Mii and my savegames. I thought about adding the Pretendo account, backing up my data, copying it to the NNID profile, and then injecting the PNID account. Unfortunately, you can't add another account with the same name. Any suggestions on how to achieve this?

Do you mean the Network ID usernames are the same? My tool was designed specifically for this use case.
If you currently have a Nintendo Network ID attached, it takes a bit of setup, but everything will be able to work under one user.

Backup the currently attached Nintendo Network ID, through Wii U Account Swap.
Unlink the account, also on Wii U Account Swap. This effectively turns the user into a local user.
Configure Inkay to connect to Pretendo Network through the Wii U Plugin System menu.
Connect your Pretendo Network ID to the user, through the Wii U Menu itself.
Backup your Pretendo Network ID, through Wii U Account Swap.

From here on out, you should be able to use Wii U Account Swap to go between the two Network IDs. Inkay is automatically configured to connect to the correct server when you perform a switch.

I followed the steps above but instead of linking it I accidentally overwrote the account. Is there anyway I can recover any save data?

This would probably be better off in a separate issue, but are you able to describe what you mean by overwriting? You can go back and forth between a Pretendo and Nintendo backup. If that's what you mean, you should be able to just swap to the other one? Did you tamper with the backups?

Hello,
I've solved the issue, turned out I just needed to reboot. Which then the Wii U decided to show my local user.


[BUG] Tasks don't get deleted when they should.

Describe the bug 🐛
When deleting topics they get deleted until you close the app and open it again, then they appear again.
To Reproduce 🐣
Steps to reproduce the behavior:

Create a topic
Delete it using x
Close app
Reopen it

Expected behavior  🤔
They should be deleted
Screenshots 🧐
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):  🤖

OS: Ubuntu 21.10
WM: i3
Terminal: Alacritty
DE: Regolith

Work as intended. Kudos.


Fix UniqFieldMixin for CheckConstraint compatibility for Django 2.2+

This PR fixes logic of UniqFieldMixin to work with Django 2.2+ django.db.models.Model.Meta.constraints that have not only instances of UniqueConstraint but also CheckConstraint.
Closes #1522

Coverage decreased (-71.5%) to 0.0% when pulling 0ac7e284e84191245d6857e3bd03df6190a40f16 on radekwlsk:topic-1522 into 3d4a7d3f36357f1dac93ed79ce82751c6d4a0a1e on django-extensions:master.

Thanks !


Update setPassword.sh : escape quotes in order to protect passwords which start with a number

escape quotes in order to protect passwords which start with a number

@gvenzl this can be reviewed/merged without an OCA. Which I think it should be.


qt5-base install error

I updated harfbuzz to the new version (#7740). But after that, when I try to install qt5-base for x64-windows-static triplet, I get this error:

ERROR: Feature 'system-harfbuzz' was enabled, but the pre-condition 'features.harfbuzz && libs.harfbuzz' failed.

Any help?

One of the configure.json in the source path of qt5-base is probably doing a test on the harfbuzz library and is missing the information that it needs to link the dependent libraries of harbuzz.
You probably need to turn on verbose in configure_qt.cmake in qt5-base to somehow be able to debug it.

@Neumann-A Many thanks, I'll do your suggestion.

Related to https://github.com/microsoft/vcpkg/pull/10146

This issue should be fixed now, harfbuzz has upgraded to 2.6.6.

@PhoebeHui I just attempted to update my vcpkg packages and OPs exact same error happened to me again, even though harfbuzz has arrived at version 2.7.2. So the error has either not been fixed or it has been reintroduced.


Add explanatory note to Windows ALPN version check

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (6225ebb) 80.09% compared to head (dc755e9) 80.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #507   +/-   ##
=======================================
  Coverage   80.09%   80.09%
=======================================
  Files          28       28
  Lines        5943     5943
=======================================
  Hits         4760     4760
  Misses       1183     1183

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Translate tasks/configure-pod-container/assign-pods-nodes-using-node-affinity in Korean

This is a Feature Request

What would you like to be added

Translate tasks/configure-pod-container/assign-pods-nodes-using-node-affinity in Korean
Why is this needed
No translation tasks/configure-pod-container/assign-pods-nodes-using-node-affinity in Korean
Comments

Page to Update: https://kubernetes.io/docs/tasks/configure-pod-container/assign-pods-nodes-using-node-affinity/

/language ko
/assign


Unable to chain fill_in?

Slightly related to a previous issue, I'm trying to create a helper that steps through user sign in's.
Esteem.UserHelper.sign_in/3 is only ever invoked from another module which is using AcceptanceCase
defmodule Esteem.UserHelper do
  use Wallaby.DSL

  def sign_in(session, email, password) do
    session
    |> visit("/sessions/new")
    |> fill_in("Your email", with: email)
    |> fill_in("Password", with: password)
    |> click_button("Submit")
  end
end

When it runs, I get the following error
  1) test user signs out and is redirected to login page (Esteem.UserSessionsTest)
     test/acceptance/user_sessions_test.exs:16
     ** (FunctionClauseError) no function clause matching in Wallaby.DSL.Actions.fill_in/3
     stacktrace:
       (wallaby) lib/wallaby/dsl/actions.ex:59: Wallaby.DSL.Actions.fill_in(%Wallaby.Session{base_url: "http://localhost:65372/", id: "195cd7d0-3f74-11e6-b0d4-d76992d84dcf", screenshots: [], server: #PID<0.467.0>}, "Password", [with: 'test'])
       (esteem) test/support/user_helper.ex:8: Esteem.UserHelper.sign_in/3
       test/acceptance/user_sessions_test.exs:18

It seems that the second fill_in (in this case, "Password") is getting a clause error.. and I honestly have no idea why.
Here is another test in the suite that works just fine, so I don't know why chaining fill_in's would be an issue
defmodule Esteem.UserSignsUpTest do
  use Esteem.AcceptanceCase, async: true

  test "user signs up", %{session: session} do
    session
    |> visit("/")
    |> fill_in("First name", with: "Luke")
    |> fill_in("Last name", with: "Rollans")
    |> fill_in("Email", with<EMAIL_ADDRESS>    |> fill_in("Password", with: "123456")
    |> fill_in("Organisation name", with: "Marvellous Mutants")
    |> click_button("Submit")

    notice =
      session
      |> find(".alert-info")
      |> text

    assert notice == "Signed up successfully!"
  end
end

I am really sorry to post again, but I just don't know how to proceed.. and hopefully this might help someone else in future :)

No worries @lukerollans :)
It looks like your password argument in your sign_in helper is coming in as a char list:
(wallaby) lib/wallaby/dsl/actions.ex:59: Wallaby.DSL.Actions.fill_in(%Wallaby.Session{base_url: "http://localhost:65372/", id: "195cd7d0-3f74-11e6-b0d4-d76992d84dcf", screenshots: [], server: #PID<0.467.0>}, "Password", [with: 'test'])

fill_in expects a binary and enforces that with a guard clause. Hence why you would be getting a "no clause" error. I suspect that if you change your password to a string it'll solve the problem. If that doesn't work could you post the html as well so I can try running things locally?

Definitely shows how new I am to Elixir :) Thanks again!


Tests de-registration of device

Need to add support for device de-registration and ensure it works well.
@screamerbg @dhwalters423

As far as I can tell, there is no API call to deregister a device. The only way you can deregister a device is for the registration timeout to lapse.
We could test this by setting the registration timeout to something very low (like a few seconds).


Implement current_time method

What is the goal of this pull request?
Implement current_time method for DurableOrchestrationContext
What does this pull request change?
Initializes current_time field with the first OrchestrationStarted event in constructor.
Updates current_time field with the next OrchestrationStarted in history.
What work remains to be done?
N/A
Do you consider it adequately tested?
NO

@all-contributors please add @dmolokanov for code, test, design, and ideas.


fix(test): set env vars if the job is running from PR check

Description
The job ci/prow/appstudio-hac-e2e-tests is PR check, but is running with the main branch, not with the branch from a PR. This PR should fix it.
Issue ticket number and link
https://issues.redhat.com/browse/HAC-5623

/lgtm

/approve


Opaque structures no longer supported by locator API

Recent changes to the locator API prevent the locator from being initialised with an existing service. Now, the service may only be initialised via in-place construction. Overall, the changes to the locator API are great for encapsulation and simplicity, but has made it difficult (or impossible) to register an opaque structure as a service.
Consider the following example from the previous API, in which a shared_ptr pairs the creation and destruction of an SDL_Window (an opaque C-struct) via create/destroy functions. The shared_ptr is then assigned to the locator and the user can rest assured that the underlying type will be destroyed appropriately when the service is reset.
std::shared_ptr<SDL_Window> window {
    SDL_CreateWindow(
        window_title,
        SDL_WINDOWPOS_CENTERED,
        SDL_WINDOWPOS_CENTERED,
        window_size.x,
        window_size.y,
        0),
    &SDL_DestroyWindow,
};

entt::service_locator<SDL_Window>::set(window);

I don't believe that the new locator API can support using an opaque structure as a service type anymore, as there is no way to attach the custom create/destroy functions, unless this can be achieved in a relatively simple way via a custom allocator?
I could create a wrapper class for the opaque type that will invoke the create/destroy functions, and register this as a service instead, but this feels more cumbersome and adds another layer of indirection.
Do you have any thoughts on this?

Recent change

Define recent 😅 do you mean the changes from February? 🙂

Do you have any thoughts on this?

I think you're right actually. It's likely a matter of making the service_handle public and using reset to inject it.

Looking into this. What about a construct function that forwards its arguments directly to the shared_ptr constructor?
I kinda prefer this version rather than exposing the shared_ptr directly, mainly because we can still change the implementation details in future without having to also affect the API.

I agree that it would be best not to expose the underlying storage type in case you wish to change it in the future, but what do you imagine the signature of a construct function to be?
If it simply took variadic arguments and passed them to the shared_ptr constructor, then the user would need to understand what the underlying storage type is, and any changes to this type would need to be communicated.
template <class... Args>
void construct(Args&&... args); // It is unclear exactly what needs to be provided here.

If the function explicitly took a raw pointer and deleter function, then the user would not need to know how it is stored internally, only that the locator becomes the owner of the pointer and it will be deleted appropriately.
template <class Deleter = std::default_delete<T>>
void construct(T* service, Deleter deleter);

But, this would have a side-effect of restricting what underlying storage type you can use, i.e. you would not be able to use a unique_ptr because the deleter is part of the template signature.

Fair enough. I feel that not exposing the shared_ptr isn't an option. 🤔
The loader model has the same problem after all. You still have to return a shared_ptr and therefore to know about the implementation details.
Any other idea or suggestion?

At least with the loader model you have the explicit result_type alias, so the user can't mistakenly assume that the internal type is anything else. I personally don't mind any of these solutions, I just like to play devils advocate. I also think it is fair enough if you would prefer not to support opaque structures directly as you would typically expect a C++ user to create raii wrappers for C library constructs anyway.


Made method FlipTableConverters.fromResultSet() public per Issue #11.

Hi,
Fixed issue #11 . Thanks for the AWESOME library!

Hi, can this be merged?

+1


Added missing include to fix build break.

ioBucketizer.cpp would not build without .

@DanPear you know more about the VS build environment, any idea what Centis ran into? Otherwise, obviously harmless.


Installed with HomeAssistant integration, it shows that it cannot be connected when configured

How do you have frigate itself installed?

How do you have frigate itself installed?

and is the addon currently running?

and is the addon currently running?

yes

Same here, the installation method listed in the documentation seems to fail with the current HA version (at least for me). what worked is following these instructions https://www.youtube.com/watch?v=pqDCEZSVeRk&ab_channel=EverythingSmartHome

having the same issue

ConfigurationWhen configuring the integration, you will be asked for the URL of your Frigate instance which is the URL you use to access Frigate in the browser. This may look like http://<host>:5000/. If you are using HassOS with the addon, the URL should be one of the following depending on which addon version you are using. Note that if you are using the Proxy Addon, you do NOT point the integration at the proxy URL. Just enter the URL used to access Frigate directly from your network.

Addon Version
URL

Frigate NVR
http://ccab4aaf-frigate:5000

Frigate NVR (Full Access)
http://ccab4aaf-frigate-fa:5000

Frigate NVR Beta
http://ccab4aaf-frigate-beta:5000

Frigate NVR Beta (Full Access)
http://ccab4aaf-frigate-fa-beta:5000

I'm also having the same issue connecting to frigate running on the same docker bridge. this is without running it as an addon but as an additional service in my docker compose.

Hy HASS logs show a connection timeout. I get the same thing after some time with a perfectly functioning setup. I can still access Frigate just fine at the address it's configured for

same issue here as well.. Updates to 0.13 and 5.0 beta integration. Sometimes it works with : IP-OF-FRIGATEHOST:5000 but then after a while it stops working
This error originated from a custom integration.

Logger: custom_components.frigate.api
Source: custom_components/frigate/api.py:51
Integration: Frigate (documentation, issues)
First occurred: September 19, 2023 at 21:21:21 (720 occurrences)
Last logged: 13:43:39

Error fetching information from http://ccab4aaf-frigate-fa-beta:5000/api/stats: 502, message='Bad Gateway', url=URL('http://ccab4aaf-frigate-fa-beta:5000/api/stats')
Timeout error fetching information from http://ccab4aaf-frigate-fa-beta:5000/api/stats:

everything worked fine before beta.. but that is beta.. some things moght nt work :) wiating for solution patiently.

same here. I see same error in log but component, addon and all sensor are working fine.

This is quite unlikely to be an issue with the integration (where this issue is filed). Check your Frigate logs for issues (it may be crashing), and check your address is correctly specified.

address is correct ( to be sure I deleted integration and re added using url from docs page (https://docs.frigate.video/integrations/home-assistant/ - in my case Frigate Full Access). The new integration founds the cam, the sensor to switch on off motion sensor works. Only this annoying error in HA logs.

Hello I'm facing the same issue
The addon Is running well, but when I try to install or add the integration frigate the addon is discovered but never loads correctly the cameras or entities...
I'm currently using this version
Frigate version: 0.12.1

Same.

I'm also having this issue, beta or not beta, with the latest HA update - something is up, unable to connect

Hi All,
The issue is with the full access beta, - the issue does not exist with the regular beta or regular (non-beta) install.

Having the same issue with Full access (no beta) addon:
This error originated from a custom integration.

Logger: custom_components.frigate.api
Source: custom_components/frigate/api.py:227
Integration: Frigate (documentation, issues)
First occurred: 18:47:00 (4 occurrences)
Last logged: 18:47:52

Error fetching information from http://ccab4aaf-frigate-fa:5000/api/stats: Cannot connect to host ccab4aaf-frigate-fa:5000 ssl:default [Name has no usable address]
Error fetching information from http://ccab4aaf-frigate-fa:5000/api/stats: 502, message='Bad Gateway', url=URL('http://ccab4aaf-frigate-fa:5000/api/stats')

same here

Hello, i have the same issue, if i install add-on 012.1, I can't install the integation Frigate 4.0.0
do you known the procédure for install the version 4.0 integration frigate ?
thanks

I had this issue as well, and I had to do the following:
The version I was running (0.12.1) was throwing error when trying to connect version 5.0.0 beta, but got the following error
Using a Frigate server (http://ccab4aaf-frigate-fa:5000) with version 0.12.1-367d724 <= 0.12.1 which is not compatible -- you must upgrade: https://github.com/blakeblackshear/frigate/releases
So I upgraded the add-on to 0.13 beta, and after figuring out the right connection string, the integration connected and added the entities.

Ok, I solved this problem, indeed when installing the add-on version v0.13.0 and the version integration 5.0.3 beta, I didn't see that we could have a choice!
thank you

Still not working for me.  I’m running 0.13.0 (Beta) and 5.0.0 beta in HACS. Still cannot connect with the integration. What am I missing?

I cannot start the addon because it says, "Port '8555' is already in use by something else on the host."  What else would be using that port, how do I find out, and what are my options to start this add-on?

Most likely it is WebRTC camera or go2rtc. Go2rtc can just be removed and WebRTC card should be removed, frigate added, and then can readd the WebRTC camera integration and it will use frigate as the source.

o2rtc can just be removed and WebRTC card should be removed, frigate added, and then can readd the WebRTC camera integra

I don't have any addon WebRTC or go2rtc installed and I still have this issue, unable to load the integration Frigate ...

Most likely it is WebRTC camera or go2rtc. Go2rtc can just be removed and WebRTC card should be removed, frigate added, and then can readd the WebRTC camera integration and it will use frigate as the source.

In the logs appears this:
2024-01-02 00:11:52.377 ERROR (MainThread) [custom_components.frigate] Using a Frigate server (http://ccab4aaf-frigate:5000) with version 0.12.1-367d724 <= 0.12.1 which is not compatible -- you must upgrade: https://github.com/blakeblackshear/frigate/releases
But I don't have any update available...

You are using a pre release version of the integration which requires the pre release frigate 0.13 (currently in RC)

You are using a pre release version of the integration which requires the pre release frigate 0.13 (currently in RC)

Ok, sorry and How can I try to solve this ?
Thank you

Either downgrade the integration version or update frigate

Either downgrade the integration version or update frigate

Done, solved thank you !

I was experiencing the same problem on and off, but I always managed to solve the problem by matching the right versions of the integration and Frigate add-on.
After updating HA to core version 2024.1.0 this morning, I can not get the integration to connect to Frigate - no matter what combination I am trying.
Frigate add-on runs perfect no matter if it is ver. 12 og ver. 13-rc1.
Log says this:

Error fetching information from http://ccab4aaf-frigate:5000/api/stats: Cannot connect to host ccab4aaf-frigate:5000 ssl:default [Name has no usable address]
Error fetching information from http://ccab4aaf-frigate-beta:5000/api/stats: 500, message='INTERNAL SERVER ERROR', url=URL('http://ccab4aaf-frigate-beta:5000/api/stats')

I was experiencing the same problem on and off, but I always managed to solve the problem by matching the right versions of the integration and Frigate add-on. After updating HA to core version 2024.1.0 this morning, I can not get the integration to connect to Frigate - no matter what combination I am trying. Frigate add-on runs perfect no matter if it is ver. 12 og ver. 13-rc1. Log says this:

Error fetching information from http://ccab4aaf-frigate:5000/api/stats: Cannot connect to host ccab4aaf-frigate:5000 ssl:default [Name has no usable address]
Error fetching information from http://ccab4aaf-frigate-beta:5000/api/stats: 500, message='INTERNAL SERVER ERROR', url=URL('http://ccab4aaf-frigate-beta:5000/api/stats')

Same here, only difference I updated HA OS to 11.3 which stopped the addon from being able to connect to frigate.
Frigate 0.12.1 and addon 4.0.1
Log says this:
Error fetching information from http://ccab4aaf-frigate:5000/api/stats: Cannot connect to host ccab4aaf-frigate:5000 ssl:default [Name has no usable address]
Error fetching information from http://ccab4aaf-frigate-beta:5000/api/stats: 500, message='INTERNAL SERVER ERROR', url=URL('http://ccab4aaf-frigate-beta:5000/api/stats')
I then tried updating core to 2024.1.0 but that did not fix it.
I went back to 2023.12.4 and everything works again.

Totally agree with you VagnPetersen. Same here, update HA OS 11.3 and core to 2024.1.1.
Couldt not connect integration 5.0 RC1 to add on 0.13 RC1

I found a work arround to update both the addon and the integration.
First I backupped frigate.db
I stopped frigate 0.13 BETA addon and installed the Frigate 0.12.1 addon and started this.
I then installed the 4.0.1 integration and restarted, I then could connect to the frigate addon.
Then updated the integration to v5.0.0-rc1 and once again restarted HA.
After this the integration was still connected.
I then stopped 0.12.1 addon and deleted the newly created frigate.db and restored the original frigate.db.
I started the addon.
Then the integration said connection failed but a reload of it made it connect again

frigate 0.12 and integration 5.0 are not compatible, it says this in the release notes

frigate 0.12 and integration 5.0 are not compatible, it says this in the release notes

yes I know that, but updating HAOS to 11.3 and core to 2024.1.x results in the integration (5.0RC1) and addon(0.13.0) not being able to connect anymore.
So Im trying every possible combination to maybe find more info on this problem.

yes, it is a problem with HA OS 11.3 https://github.com/blakeblackshear/frigate/issues/9218

Oh great. Thanks!

I got the integration to work when adding and not using http://ccab4aaf-frigate:5000 as URL.
I changed ccab4aaf-frigate to homeassistant.local and that worked (also the IP of the HA server itself works)

Sadly that trick did not work for me :(

Just updated to OS 11.4.
Everything is working again :)

going to close this, anyone facing a similar issue should make their own issue


camera streaming receiver occurred NO GPU COMPATIBLE

My test environment are like below that.
[Windows]
CPU : Intel(R) Core(TM) i5-4690 CPU @ 3.50GHz
RAM : 16Gb
GPU : GeForce GTX 970 (4GB)
ZED : 2.8.2
CUDA : 10.0.130
[Jetson Nano]
CPU : ARMv8 Processor rev 1
RAM : 4Gb
GPU : NVIDIA Tegra X1
ZED : 2.8.2
CUDA : 10.0.166
I just set up camera sender at Jetson and set up camera receiver at Windows. The camera receiver occurred NO GPU COMPATIBLE when I try to connect sender in Jetson. However, all tools working are fine with Windows system only. I couldn't imagine what I am wrong?? Please just let me know what can I do for fix.
Thanks a lot

Hi,
Can you run the receiver with InitParameters::sdk_verbose option at true, so that you can have more info in the console output.
One of the explanation can be that you are using HEVC (H265) codec for streaming and the GT970 does not support H265 decoding (--> NO GPU COMPATIBLE).

Hi,
Can you run the receiver with InitParameters::sdk_verbose option at true, so that you can have more info in the console output.
One of the explanation can be that you are using HEVC (H265) codec for streaming and the GT970 does not support H265 decoding (--> NO GPU COMPATIBLE).

Thanks you for reply, I used default sample files. It has written sdk_verbose option at true already. I try to some option that change streaming codec and camera resolution. Unfortunately, it doesn't work until now.
ZED (Init) >> Video mode: HD720@60
Failed to query video capabilities: Inappropriate ioctl for device
Opening in BLOCKING MODE
NvMMLiteOpen : Block : BlockType = 4
===== NVMEDIA: NVENC =====
NvMMLiteBlockCreate : Block : BlockType = 4
H264: Profile = 77, Level = 50
 streaming is in progress.... ```

I will try to other GPU though I should use GT970 also. If you have any idea, Please let me know.

Thanks again.

Hi,
About the InitParameters::sdk_verbose, I was more talking about the receiver (on windows) to see why the GPU is not compatible/detected.
Did you try on localhost as well ? (Windows to Windows, same PC) ?

I try from Jetson NANO to Windows. I think I have two issue. One is that I couldn't change port. Other is I can see below error coded.
Failed to query video capabilities: Inappropriate ioctl for device
I just guess theses issue make problem.

This line
Failed to query video capabilities: Inappropriate ioctl for device
is not important (just a warning from the encoder)
What do you mean by "you cannot change port" ? Inside the sample ?
make sure you have this line : https://github.com/stereolabs/zed-examples/blob/master/camera streaming/sender/src/main.cpp#L56

I know how to change port in sample. It works fine on Windows. However, it doesn't work on Jetson. You can see below image. I set port to 8052 but it didn't change.

As you mention the warning is not important, I really don't know why 'camera streaming' sample doesn't work. Sample works fine on Windows, same PC but it doesn't work with Jetson NANO.

Today I tested sender and receiver on one Jetson NANO. I have same problem.
Jetson [Sender] --- [Receiver] Jetson         <== NO GPU COMPATIBLE
Jetson [Sender] --- [Receiver] Windows     <== NO GPU COMPATIBLE
Windows [Sender] --- [Receiver] Windows <== OK
I doubt ZED SDK for Jetpack 4.2

If you stream on port 8052 , then the receiver should be on <IP_ADDRESS>:8052.
But prefer a higher port to avoid conflicts. For example :
./ZED_Streaming_sender 40000 && ./ZED_Streaming_Receiver <IP_ADDRESS>:40000
Note that until a client (receiver) has not requested a connection, the sender will not send packet, therefore it does not appears in the network event list.

Dear,
I also have same problem.
Following your code for "sender", when i run this line: err = zed.enableStreaming(stream_params);, it raise one error as : "Cannot load library HEVC". After that, it raise "NO GPU COMPATIBLE
"
I run this code on Jetson nano with newest SDK (2.8.4).
Anyone can fix it ?
Thanks!!!


Update ft_is_negative.c

Just being pedantic

Thanks for the input :) I merged it.

^^. I'm in the midst of my piscine, I decided not to resort to github at
all, so wish me luck hehe
On Thu, Aug 31, 2023, 11:20 AM Mr. Galt @.***> wrote:

Thanks for the input :) I merged it.
—
Reply to this email directly, view it on GitHub
https://github.com/Mr-Galt/42-Piscine/pull/1#issuecomment-1700498879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAOTDHZFQMHJVLYYIXGIZQ3XYA3KZANCNFSM6AAAAAA4D2NSZM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Migrate rustdocs links

Now that our crates are published on crates.io, our docs are also published on docs.rs.
We should replace all substrate.dev/rustdocs links with the corresponding docs.rs links. For example https://docs.rs/sc-service/ (which points to latest published version) or https://docs.rs/sc-service/0.8.0-alpha.3/sc_service/ which points to a specific version.
context: https://github.com/paritytech/substrate/issues/5652

Noting that we have some team-internal discussion about the best platform for the Substrate Rustdoc. We should leave this issue open to track the proper migration of all Rustdocs, but we may not want to use docs.rs in particular because it does not allow searching across (Substrate) crates.


[chipmunk 3]: columns view: colors, sizes, visibility

In columns view we should allow:

change a color of column
hide/show column
save/restore a state

duplicate #1852


& background invocation operator should -WorkingDirectory when starting a job

PR #10324 is merged and now we can use -WorkingDirectory when starting a job for background invocation operator.
The following code can be replaced:
https://github.com/PowerShell/PowerShell/blob/f69f30ba49c13ddd601724b35634bf4d1f44e31d/src/System.Management.Automation/engine/runtime/Operations/MiscOps.cs#L529-L532
Steps to reproduce

Expected behavior

Actual behavior

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-daily.20190906
PSEdition                      Core
GitCommitId                    7.0.0-daily.20190906
OS                             Microsoft Windows 10.0.18978
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Please assign me to the issue. I submitted a PR for this issue (https://github.com/PowerShell/PowerShell/pull/16180).


Incorrect indentation of CSS inside SVG

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.36.0
What version of eslint-plugin-svelte are you using?
2.21.0
What did you do?
Using no special configuration beyond extending from plugin:svelte/recommended, when I add a style tag to an SVG, the CSS properties are not indented inside each rule.
<svg>
  <style>
    .test {
      fill: red;
    }
  </style>
</svg>

The svelte/indent rule shows a fixable error, "Expected indentation of 4 spaces but found 6 spaces."
What did you expect to happen?
<svg>
  <style>
    .test {
      fill: red;
    }
  </style>
</svg>

What actually happened?
<svg>
  <style>
    .test {
    fill: red;
    }
  </style>
</svg>

Link to GitHub Repo with Minimal Reproducible Example
https://ota-meshi.github.io/eslint-plugin-svelte/playground/#eJyrVkrOT0lVslKyKS5Lt4vJU1CwKS6pzEkFMxUU9EpSi0sUqiEcBYW0zJwcK4Wi1BRriEgtWIM+TAeQBTREqRYAXrgaFw==
Additional comments
I would expect this is a bug, but if it's intended, perhaps there could be a way to adjust this behavior in the svelte/indent rule's options.

Thank you for this report.
I think it's a bug in the svelte/indent rule.


Add Docker build

We plan to enable running catalyst-api separately to catalyst, therefore it needs its own Docker image.
Note that it I didn't include w3 tool, because Web3 Storage has shut down, so we should remove the support for Web3 Storage.
Design Doc: https://www.notion.so/livepeer/Zero-Impact-Catalyst-API-Deployments-c2b15232f3a2450ba3e4803130ecd2be?pvs=4#5ce68dfca1c341bfb6b21361d36d19df

@hjpotter92 also I see the build failed because the Docker Hub registry does not exist. How to create it?
buildx failed with: ERROR: failed to solve: failed to push livepeer/catalyst-api:pr-1323: push access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed


Unable to rekey private folder

Not sure what to do here. It asked for an existing device to be turned on, an Android client, which I did, but that didn't do anything. I then generated a new paper key from that mobile device to be used on the client. That rekeyed all my other conversations & etc but not my private one. Any ideas? I was migrating OS installations so all of my stuff is on there...
Log ID: fc5d2e35cc739527ee510c1c

@strib Any info on what I should include from my phone? I had the app open for a while and it never did anything.

Hmm thanks.  I can't find the logs on our end.  Can you please write some kind of feedback message when you send them?  Like "for strib" would be fine.  Thanks!

@strib Sent!

Got it thanks, will look now.

@strib Will do! Hopefully it works :pray:

Thanks. I'll keep monitoring it on my end, and if it doesn't seem to be working after a while, I'll ask for more logs.

Ok, seems like it still hasn't worked.  Assuming your app has indeed been open this whole time, I guess there's a deeper issue.  Can you please send logs again from your phone, with the same message as before?

Yep it's been open for awhile, will do.
Sent with "for strib"

Looks like you might have hit another bug in our code, this time an internal deadlock.  I'm still investigating the logs (thank you!), but you might see more progress if you kill the app and re-open it fresh.  Then again navigate into the folder and leave it open for a bit.

Cool, at least it's noticed! When I restart the app and go to "Hamburger Menu > Files > private > jaredallard" it doesn't open immediately. I wait 3-5 seconds and "An error occured." pops up. If I tap it again it will open the file list and show "loading" seemingly forever.

Sigh, ok.  Can you please send more logs after that?  I wonder if it hit the same crazy problem again.

@strib Sent!

Ok great.  The good news is that I figured out the issue and a fix is in review now: keybase/kbfs#1972.
The bad news is that it will require a new app release from us to let you upgrade and get past this bug.  Our next release is scheduled for early next week.
If this is more urgent than that, I can ask around about making a one-off release for you to try out.  Off the top of my head I don't know how to do that, but I can probably find out.

@strib If you can get a one-off release for me that would be awesome since I have all my stuff on there, ssh keys, gpg, etc.... but I get it if you're unable too! Thanks for looking into this

I'm afraid our Android release story is way too complicated to make a one-off release.  (I was hoping it would be similar to our macOS story, but it is not.)  So I think you'll have to wait until next week.  Hopefully it will be out Monday.
If you happen to be an Android developer, you can probably build your own apk from our open-source repos, but that's a long shot.
Sorry for the inconvenience.  In the meantime, treat your Android phone with the utmost care, it contains the only key to your private folder right now!  At least you have a paper key now, so if this happens in the future you can recover using that.

Closing since this was resolved! Thanks :)


Fixed typo in differences between astro and jsx

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)
New or updated content
Changes to the docs site code

Description

Closes # Closes no open issues
What does this PR change? Give us a brief description.
In this section the phrase reads "...but there a couple of key differences between .astro files and JSX." I added the word "are" so that it then reads as "...but there are a couple of key differences between .astro files and JSX."

Prepare for typo fixes! And make them double! Thanks for another amazing improvement, @Njong392 :purple_heart:


Line wrapping at a non-assignment operator

Hey there,
I am having hard time to enforce the following rule:
"When a line is broken at a non-assignment operator the break comes before the symbol." from
https://android.github.io/kotlin-guides/style.html
Setting formatting->android to true in detekt-config.yml doesn't seem to work, detekt keeps changing the following line to:
    return a_very_long__logic_condition && second_very_long__logic_condition
            && third_very_long__logic_condition

to
    return a_very_long__logic_condition && second_very_long__logic_condition &&
            third_very_long__logic_condition

Is there a way to change it?
Any help would be greatly appreciated :)
Cheers,
Tomek

Detekt provides a wrapper over ktlint.
Can you try this out with ktlint's command line interface?
Please run ktlint's command line interface over your specific code.
Please report back whether it behaves the same or not.

Hey @schalkms, thanks for your response. As suggested, I just tried ktlint's command line interface and it formatted my code the same way as detekt (I guess that was expected, since detekt is wrapping ktlint).

Please report this in ktlint's repo.
There's nothing we can do about this for now.


Build and watch theme.scss.css?

Problem
Well this is kind of weird but build or watch seems to do nothing with styles.
Watch: Styles cannot be applied because gulp is watching for theme.scss.css and even if I remove .css it does the same, gulp is still watching for theme.scss and because there is no source maps browsers cannot translate/read scss.
Build does not compile theme.scss
It is me or anyone else has the same problem?

@salasmx SCSS watching works as follows:

The watch:css watches theme.scss
The watch:dist task watches all files in dist/ for changes.
Whenever theme.scss is changed, watch:css triggers a rebuild and the new version output as dist/assets/theme.scss.liquid. watch:dist then triggers an upload of this new version to the servers.

Not sure what is wrong with your setup given the details you provided. Can you please elaborate?

@t-kelly I've created a new slate theme and now everything works like a charm.
Thank you.
btw. it would be nice some scss error log on terminal/console.

https://github.com/Shopify/slate/issues/127 😉


task: create and integrate notification store

Task description

[ ] notification zustand store
[ ] integrate to notifications [small] [large]

Related Issues
No response

@Codebox124 kindly share ur update


Deprecating Int8DynActInt4WeightQuantizer

Summary:
Added torchao API int8_dynamic_activation_int4_weight I ran some benchmark with eager mode and there was some accuracy loss and some slowdowns in compile.
But later I found we only care about performance on executorch. So was trying to benchmark et perf and accuracy. but I can't get the env setup correctly for executorch to run the experiments
Specifically there were some issues when I'm trying to run the following after I installed executorch from pip
python3 torchchat.py export llama3.1 --quantize torchchat/quant_config/mobile.json --output-pte-path llama3.1.pte

Error:
Traceback (most recent call last):
  File "/data/users/jerryzh/torchchat/torchchat.py", line 17, in <module>
    from torchchat.cli.cli import (
  File "/data/users/jerryzh/torchchat/torchchat/cli/cli.py", line 14, in <module>
    import torch
  File "/home/jerryzh/.conda/envs/torchchat/lib/python3.10/site-packages/torch/__init__.py", line 368, in <module>
    from torch._C import *  # noqa: F403
ImportError: /home/jerryzh/.conda/envs/torchchat/lib/python3.10/site-packages/torch/lib/../../nvidia/cusparse/lib/libcusparse.so.12: undefined symbol: __nvJitLinkComplete_12_4, version libnvJitLink.so.12

Test Plan:
test instruction from Jack: https://docs.google.com/document/d/1eRAoY1Jq4SR5A7iAYC71maSZAPzsBJmr9VuZPhR5ZYA/edit?tab=t.0#bookmark=id.otk3jomaciya
Reviewers:
Subscribers:
Tasks:
Tags:

@vmpuri jic you missed this


0xF45ba58c415eD6C77cfC157f42f300f9Fd980985

I can not get it to mount to my account

can you describe your operation in detail?


running the app

hello,
I'm trying to run your app
after install the dep I have this issues :
./volatile.py
File "./volatile.py", line 16
def load_data(tickers: list):
^
SyntaxError: invalid syntax

Looks like you didn't get create and activate a virtual environment? Try following the installation instructions.


Failures in app don't reflect failures in nested scatters

So when a workflow has nested scatters, the failures of tasks in the nested scatters DO show up in the failures table on the app, however the summary of the failed jobs icon does not reflect them!
The icon shows 1 failed job:

But actually there were actually 6 and 1 was a re-tried failure (for 7 total failed jobs).

I would've expected the icon to show 6 failed calls (and not reflect the re-try as a separate failure but that would be better than what it's doing now).

@vortexing -- Can you point us to the WDL that was used to produce this? Just want to try to recreate this.


Fix Java OOM error in test_sparse_int64 by forcing int64 as index dtype

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

build

Latest code has removed index_dtype from model class.

build


refactor category show action

and remove n+1 queries

[ ] create index in profiles table (published)
[ ] delete topic cloud on profiles index (see screenshot)


Link to source on draggable-and-rotate.mdx links to draggable.mdx instead of draggable-and-rotate.mdx

https://cobe.vercel.app/docs/showcases/draggable-and-rotate

Fixed in https://github.com/shuding/cobe/commit/7caacde2282ae98e63b26575a511953155c7a9ea

yeah, also fixed in #75 when i found the issue…


"No module named pip" when already installed.

I am on windows
Occures when trying to install requirements.txt:
py -3 -m pip install -r requirements.txt

u need to download pip

What is pip? you can pass the download link


[PE-D][Tester C] Inspecting Invalid Contact error message not correctly formatted

Inspecting invalid contact as shown above has displayed a %s formatter.  Perhaps your team could consider removing the "%s" or format the invalid contact name correctly into the formatter? :)

Labels: severity.Low type.FunctionalityBug
original: linuschui/ped#3

Duplicate #139


Entropy fix

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updated entropy calc.


AMS AG added drvier for sensor CCS811 and ENS210

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Automatic Restart on Mac?

I know this isn't really an issue but is there any way to make bots restart automatically after crash?

use daemon ...

@drees5 yes, if a bot crashes you can automatically restart it using a simple bash script which was mentioned by @MeGaPk . All you need to do is put a bash script file in PokemonGo-Bot folder and run it.
#!/bin/bash
while true
do
./pokecli.py
echo ">pokecli exited... restarting...";
sleep 5;
done
Now save save this file asstart.sh and put it in PokemonGo-Bot . Now start the program in the terminal using ./start.sh all the time

https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Restarting-bot(s)

many times the bot just freeze until I press cmd+c and start it again, is it possible to restart bot every hour automatically?

@Philipp59 are you talking about Dev ?


build(deps): add constraints to avoid vulnerable transitive dependencies

WHAT
constraints:

net.minidev:json-smart to version 2.4.10
org.yaml:snakeyaml to version 2.0

WHY
to avoid hi-level vulnerabilities detected by veracode
FURTHER NOTES
Closes #258

Why do we need to add this in multiple files? Can't we define this on root level of the project?

Why do we need to add this in multiple files? Can't we define this on root level of the project?

having them underneath the "parent" dependency that caused the vulnerability will make them more visible when it will be possible to remove them (because of upgrades).


Changed boolean representation of X-B3-Sampled from true/false to 1/0…

… inline with spec and other implementations.
Resolves #68

@adriancole  X-B3-Sampled with value 1/0 is correct?

1|0 is more often sent than true|false. Usually, the server will accept either true|false or 1|0

I'll add a change log entry for this one inline with the other pull request comment.

@adriancole Is the right approach to converge on a single standard or support both true/false and 1/0?  From my perspective a single agreed representation simplifies expectations, testing & implementation but I'm new to this particular project/ecosystem (thinking the handling of multiple header values (true+1) as you cannot assume what the receiving server will interpret as true).  Either way, given the promise of tracing across polyglot services there appears to be a gap in the documentation covering the http header specification that's inappropriate for swagger.  Perhaps somewhere on the main OpenZipkin readme?  Let me know, happy to take a stab.

I don't like the idea of making true|false a standard for this header.
We've already documented 1|0 here http://zipkin.io/pages/instrumenting.html
and these headers are confusing enough already.
At the moment behavior of "true"|"false" is undefined... there exist
libraries who accept it, but that doesn't mean we should standardize on
that and then incur the busy work of chasing up people who don't accept it
to now accept it. Moreover, it complicates the story about emitting the
header, as now we have 2 options for what is right to emit! Just becomes a
mess.
Bottom-line, if the idea of accepting what was formerly accepted by this
library (true|false) creeps you out, I'd just say don't accept it. The last
thing we need is more complex if statements ported to standard docs or
integration tests.

in case timezone's hinder us.. here's another option. Even if we have
the standard as 1|0, we could add an implementation note to the web
site like this..
Note: Some older tracers set the value "true" or "false" as opposed to
"1" or "0". Some servers accept both variants for transition purposes.
Basically a paragraph break could keep this behavior optional to
support (ie. non-standard), while at the same time document its
existence in case people want to know about it without asking. There
are other places where implementation notes exist, for example
"unknown" as a service name.

I think we're on the same page here.  I'd like to change the header to support 1|0 in alignment with other implementations and document the change (represented by current branch).  My original problem was that zipkin-tracer was not continuing tracing from other services (e.g. sleuth).  This would correct that and make zipkin-tracer compatible with a wider range of zipkin trace implementations and also track to the specification documentation you highlighted (which I'd missed earlier)

great.. I'm glad my blathering didn't lead to nowhere :)

LOL not at all.  I think the current pull request lines up with the conversation unless I've got my wires crossed :-)

A different object sounds good, keeps the logic isolated and it is very easy to add a comment explaining why it exists.

@jcarres-mdsol Looking further at this approach.  There's already a Http->obj abstraction in the form of ZipkinEnv.  The work on the Faraday wrapper didn't seem to warrant a separate object so I ended up moving this to a separate method and changing the method->header map to a constant.

@jcarres-mdsol Is this active? I run into the same issue with a polyglot environment.

any way we can pick this back up? the polyglot example is broke on account of it. https://github.com/hawkular/hawkular-apm/tree/master/examples/polyglot-zipkin

Hi Adrian,
This is good to go from my standpoint, as per the last comments.  Let me
know if there's anything I need to do to get this unstuck.

Stuart

On Thu, Nov 3, 2016 at 12:10 AM, Adrian Cole<EMAIL_ADDRESS>wrote:

any way we can pick this back up? the polyglot example is broke on account
of it. https://github.com/hawkular/hawkular-apm/tree/master/
examples/polyglot-zipkin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin-ruby/pull/69#issuecomment-258061082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUlxespEgNRyKo4SNRDh4a8LX8FrdONks5q6V7PgaJpZM4I56Kh
.

looks like a rebase is all that's needed, right @jcarres-mdsol ?

Only one comment, then just need to rebase and it is good to merge!

Np.  I'll take a look at this later today.
On Thu, Nov 3, 2016 at 7:49 AM, Jordi Polo Carres<EMAIL_ADDRESS>wrote:

Only one comment, then just need to rebase and it is good to merge!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin-ruby/pull/69#issuecomment-258122183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUlxQs82aHQPSaIIGx6uh6FHUFGfXNWks5q6cowgaJpZM4I56Kh
.

So, I've been thinking about this and going back to the comments of Adrian there, I think this PR does two things:

When discovered a true/false force it to 1/0
Accept both true/false and 1/0

The problem with the first point is that a service with this tracer will break a system which is full of tracers which only accept true/false. In particular if you have rails services, you will be forced to upgrade all of them to 0.19 or 0.18 will break the chain.
So this PR becomes more complicated but to avoid breaking any system the best thing is to propagate whatever you get. If you get true/false, propagate those, else propagate 1/0
If a new one needs to be created you can default to 1/0 so all your system will be 1/0 from origin to end.
Else, this would be a 1.0 kind of release and a difficult one to upgrade to as everyone needs to upgrade at the same time, as the contract between services have changed.

could you solve this with a flag/option? currently it is unusable in a
system that is written correctly.

For example, that way you can upgrade the libraries and eventually
change the option in such a site. You could also "double-check" ex. it
is very odd to propagate an id and also have sampled=0/false. You
could use a heuristic that double-checks until library updates
converge.

How about this. When we receive 'X-B3-Sampled', we interpret it as a boolean (eg either "true" or 1 means True) but leave the actual value alone, and propagate it in whatever ruby client the same way as it was received. For example, we pass it opaquely as it was sent to us. This should solve the brown field.
Then, the problem is simplified to the first emission of 'X-B3-Sampled'. In this case, we could have a configuration option, which says to write either true|false or 1|0 when sampling a trace. This setting could be set to write_sampled_as=true_false to retain historical behavior. At some point, all the libraries will be updated, then they can change to write_sampled_as=1_0.
New installations would default to write_sampled_as=1_0 which makes them compatible with all the other propagators.

Sounds reasonable.
However I could see a case where you do want to transform a value to
correctly work with another service.
Having control over both the pass through representation and initial sample
value would handle all possible use cases.
Thoughts around the default stance(s)?  It seems to me that the
write_sampled_as=1_0 is more standards compliant at this time.
On Fri, Nov 4, 2016 at 4:19 AM, Adrian Cole<EMAIL_ADDRESS>wrote:

How about this. When we receive 'X-B3-Sampled', we interpret it as a
boolean (eg either "true" or 1 means True) but leave the actual value
alone, and propagate it in whatever ruby client the same way as it was
received. For example, we pass it opaquely as it was sent to us. This
should solve the brown field.
Then, the problem is simplified to the first emission of 'X-B3-Sampled'.
In this case, we could have a configuration option, which says to write
either true|false or 1|0 when sampling a trace. This setting could be set
to write_sampled_as=true_false to retain historical behavior. At some
point, all the libraries will be updated, then they can change to
write_sampled_as=1_0.
New installations would default to write_sampled_as=1_0 which makes them
compatible with all the other propagators.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin-ruby/pull/69#issuecomment-258368798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUlxQmwnDW29PA4pKnULPgQBcyGVh9rks5q6uqXgaJpZM4I56Kh
.

I've reimplemented this PR in a backwards compatible way, can you take a look at #79 ?

closing this as #79 provides similar functionality. Thanks for the inspiration and pushing for this!


PE-D documentation reports fixes

Removed mentions of dob and po in UG
Closes #119 #120 #122 #125 #127 #129
Fix Rapporbook typo
Closes #118

Codecov Report
Base: 77.21% // Head: 77.21% // No change to project coverage :thumbsup:

Coverage data is based on head (89352ad) compared to base (74609dc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #130   +/-   ##
=========================================
  Coverage     77.21%   77.21%
  Complexity      786      786
=========================================
  Files           108      108
  Lines          2199     2199
  Branches        249      249
=========================================
  Hits           1698     1698
  Misses          410      410
  Partials         91       91

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

LGTM


⚠ Samsung has degraded performance

In 6b11119, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2328 ms

Resolved: Samsung performance has improved in 14617f6.


Speed-Soft.Time-Sync.Server version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

The setup contains two versions: Server and Client - Both can be installed by one setup (the reason for the same hash). In the installer there is an argument "/Type=Client" and "Type=Server". This is the difference.


Feature request: Cargo templates

Some history: There was an effort to do this that got reverted pending an RFC:

https://github.com/rust-lang/cargo/issues/396
https://github.com/rust-lang/cargo/pull/3004
https://github.com/rust-lang/cargo/pull/3878

This is a new tracking issue.
Related: #3506

Also relevant: #5114, initialising an examples directory for lib crates.

See https://github.com/rust-lang/rfcs/pull/2549#issuecomment-433712561.

See also #12769 proposing adding educational comments to Cargo.toml generated by cargo new/init.

I created an interactive configurator for Cargo projects (focused on performance aspects of configuration), might be a bit relevant: https://github.com/Kobzol/cargo-wizard.


A stack overflow problem occurred when executing wasm files.

qjs.zip
Steps to Reproduce
iwasm qjs.wasm

Then I got the following error.

 I submitted PR #1300 to fix it, could you please test again?

Thanks a lot.


Protractor 6: Browser getter ignoreSynchronization to log error

Since its been deprecated.

Also do this for other methods removed. (log as warning not error)


Moonblade generational arena

We could really benefit from some kind of manually-coded simple garbage collection arena vs. cloning and the Cow complexity.

The generational arena is a looooot of hassle...
We should clone the stored values. And wrap non-scalar values in a Rc.


Move seahorse_util to separate file

this makes the generated lib.rs much easier to read
it also sets a pattern for adding future library files as needed.

Seahorse generates what is essentially a regular valid Anchor Rust project to be compiled by the Anchor CLI. Currently, all the code is thrown into a single lib.rs file. This includes code that is in a module seahorse_util. This module contains a bunch of seahorse boilerplate, helper functions, and traits for properly generating valid working Rust code from the python source.
This Seahorse boilerplate clutters the lib.rs file unnecessarily, it is trivial to put it in an adjacent file and then just "import" it via use. An additional benefit is that this makes it easier to test that Seahorse outputs are correct, since we now can generate much smaller and simpler lib.rs files that do not include all the Seahorse boilerplate.
I haven't ever used the Solana playground but I'll take a look and respond with more thoughts once I do


Improved the documentation indentation

Improved the documentation indentation

Current coverage is 34.08% (diff: 100%)

Merging #925 into development will not change coverage

@@           development       #925   diff @@
=============================================
  Files                7          7
  Lines             1021       1021
  Methods            131        131
  Messages             0          0
  Branches           157        157
=============================================
  Hits               348        348
  Misses             673        673
  Partials             0          0

Powered by Codecov. Last update 7f8753d...ec817fa


Does @uidotdev/usehooks have test coverage?

Sorry to open this as an issue, and first of all great work with the website and these useful hooks!
I could not find any tests in this repository and I want to make it clear whether these hooks have test coverage. If so, it would be useful to add a link somewhere and if not it would be good to add a disclaimer about it.

No test coverage.


Improve CI

In this branch I plan to improve the CI

[x] Add a new github workflow dedicated to CI
[x] Add a job checking compatibility with older Zig versions
TODO: document why the ci/compat.sh script needs to use grep.
[x] Ensure all .zig files have conforming code formatting
Now the Eowyn workflow will ensure it.
See also #222.
.
I wanted to add a quote like "May the compiler have mercy upon us." in bd777dc.
Something like "follow the orthodoxy" but I'm not very good at it.
.
Most of the time, the code formatting was not conforming in order to keep comments aligned.
I think this causes only problems (see Go, when a simple change may cause a huge code reformatting),
and using zig fmt: off is probably a very bad idea for newcomers.
[x] Make the patch files consistent
See also #244.
.
It was interesting that the headers of the two patches in unified format was--- exercises/080_anonymous_structs.zig
+++ answers/080_anonymous_structs.zig

.
Is the old answers directory used when creating a new exercise?

The build runner exits with exit code 1, ignoring the child exit code.  Zig 0.6.0 works correctly, instead.

But the question is, back to which version and for what?

But the question is, back to which version and for what?

In the job I used Zig 0.6.0, 0.7.0, 0.8.0, 0.9.0 and 0.10.0.  That is, all the old stable releases.
I have updated the bash script, and now it works.
The problem, however, is that a new version of the Zig compiler has been released, and it broke the Eowyn workflow.
I wlll update the build.zig file to the new API.

@chrboesch Can you run the github action again?  Thanks.

deprecated; use std.Build.installArtifact

@perillo Maybe you should update your branch, before continuing.

@perillo Maybe you should update your branch, before continuing.

You are right.  I assumed that when running the action my branch would be rebased to main automatically.  Sorry for the noise.

Everything is fine, thank you!

Yeah, thank you for all the great work, @perillo !

It was fun.  Ziglings uses the Build system in a creative way, so I learned a lot about how it works.
But there are still several changes in my TODO file.