sc 214640 : log cleanup and req tracing [from lucene-solr]

Description
Apply changes from https://github.com/fullstorydev/lucene-solr/pull/102
Solution
Apply changes for the above branch, details:
Changes included in this PR:

Add log marker based on path (upstreamed in https://issues.apache.org/jira/browse/SOLR-16582)
Add info message on search start (not upstreamed, see discussions in https://github.com/apache/solr/pull/1237 https://issues.apache.org/jira/browse/SOLR-16586)
Simplified message on sub-shard requests -(not upstreamed as this is quite specific to FS usage. Solr community might not like the fact that full query is no longer printed for handler processing in the data node - for us it's okay as it's easy for us to see ALL logs across nodes including QA node from BQ)
Enable handling of rid from external systems - (not upstreamed, Solr is using the OpenTracing context at here which is in theory more generic. However, for FS, it's easier for us to use rid as our request id generated from alexa/anita does not comply to OpenTracing trace ID format, hence it's easier for us to "workaround" it by using rid which has no format restriction)
Adding rid to the solr/server/resources/log4j2.xml , use the %notEmpty{} expression to avoid empty rid tags

Changes skipped in this PR:

In SearchHandler, generate/tag rid (request id) earlier and include such in MDC context. Very similar change was already implemented in 9.0 by another contributor https://issues.apache.org/jira/browse/SOLR-15790 .

@patsonluk is it better to create separate PR for each issue?

@hiteshk25 I have broken it down for upstream PRs (those proposed to apache community).
As for local merges (within fullstory-solr) from our own 8.x to 9.x, can we do a 1-1 mapping for PR? (one PR in 9.X for each PR in 8.x)

LGTM( just see if you can verify logs!)

Thanks! Tested locally (by running solr in cloud mode directly, ie bin/solr -p 9999 -c -a "-Xdebug -Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=19999") to ensure:

Marker is working (adding /replication and /get to ignore and issued http://localhost:9999/solr/test/get). Added jira issue to use AsyncLogger to be consistent with usage in 9.x https://fullstory.atlassian.net/browse/SAI-4295
Log on start of query such as Start External Search Query...
Use a autogenerated rid if not provided (localhost-n), if provided an rid from as query param, then such rid will be used and logged


Stats are not being displayed

https://vetherasset.org/stats
Stats are not loading for me since yesterday, it just spins infinitely.

It works, that could have been API server side issue.


change title on README

Changes suggested:
Suggested to change the color of shapes

Changes suggested:
Suggested to change the color of shapes

Left reviews and line comments
Created a file and made a commit
Opened a pull request

Left reviews and line comments
Created a file and made a commit
Opened a pull request

I added the new commit

I added the new commit

I make the comment

I make the comment

X    Left reviews and line comments
X   Created a file and made a commit
X    Opened a pull request

X    Left reviews and line comments
X   Created a file and made a commit
X    Opened a pull request


Update ffmpeg links approved_links.txt

ffmpeg2theora website is no longer available and ffmpeg makes a really good job converting to the format. I updated the PlayVideo doc to recommend ffmpeg and I'm updating the approved links here.
https://github.com/adventuregamestudio/ags-manual/wiki/Multimedia#playvideo

Thanks!


Fix management command options

Commit fe7c908368f7c79883c76c21e87a1eef70691007 introduced a couple of small errors into process_tasks while adding Django 1.10-style option parsing. The result was that --log-file was overwriting --queue and there was no way to specify --log-std. This fixes those issues.

Thank you for the fix, @danieljcrabtree :)

You're welcome!


AWS Resource Discovery

Add permissions and env variables to support resource discovery in AWS.

Codecov Report

Merging #153 (83f4afb) into develop (1f871a0) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #153      +/-   ##
===========================================
+ Coverage    32.74%   32.76%   +0.02%
===========================================
  Files           43       43
  Lines         3329     3330       +1
===========================================
+ Hits          1090     1091       +1
  Misses        2136     2136
  Partials       103      103

Flag
Coverage Δ

unittests
32.76% <100.00%> (+0.02%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/provider/pulumi/aws/aws.go
63.59% <100.00%> (+0.16%)
:arrow_up:

pkg/provider/pulumi/aws/lambda.go
71.15% <100.00%> (ø)

Continue to review full report at Codecov.

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


Boolean function to check if a case involves depletion or not

We have some plugin-side changes that need to know if the model is being depleted or not (selection of additional data files, preparation of additional tallies, etc.) Currently, we have a lot of patterns like
hasBurnup = sum(getBurnSteps(cs)) > 0

which involves a lot more work than we actually need for this check. Nearly all the routines related to burnup require building (in some fashion) the entire schedule first (see _getStepAndCycleLengths).
It would be useful to have something like armi.utils.checkHasBurnup(cs) -> bool. This is something that could be implemented outside of ARMI, but I believe it would be more valuable to have it tied directly to the other burnup schedule parsing routines in ARMI.
Maybe converting _getStepAndCycleLengths and other methods to be a generator would work? Then, at least we could write
hasBurnup = any(getStepLengths(cs))

assume the problem has depletion is there is one non-zero burnup step?

Various thoughts that occur to me:

hasBurnup = sum(self.cs["buGroups"]) > 0
test if there is an interface at the DEPLETION point in the Interface stack

But, sure, that's a one-line helper utility.  NBD.
The PR is up here


Endpoint Security

Information

QPPCT-505

Changes proposed in this PR:

Spring Security configuration with JWT filter
All CPC+ endpoints will now filter based off if they are valid orgs

Checklist

[x] All JUnit tests pass (mvn clean verify).
[x] New unit tests written to cover new functionality.
[x] Added and updated JavaDocs for non-test classes and methods.
[x] No local design debt. Do you feel that something is "ugly" after your changes?
[x] Updated documentation (README.md, etc.) depending if the changes require it.

SonarQube analysis reported 2 issues

 1 major
 1 minor

Watch the comments in this conversation to review them.

When reviewing please keep in mind that I chose not to write tests for the SecurityConfig class. The reason being is that setting up all the mocks for this unit tests  would take much effort and I believe that this could easily be replaced with Integration tests once we get those Integration tests fixed. I read this up from a stack overflow issue here https://stackoverflow.com/questions/43663688/spring-security-httpsecurity-configuration-testing . Please let me know what you think :)

SonarQube analysis reported 1 issue

 1 minor

Watch the comments in this conversation to review them.

@saquino0827: In the past I've only done integration tests using MockMvc to test the Spring Security setup, so I think this is an ok plan.

I will close this branch for now as we are pending environment variable changes.


Sast/script service v2 fix caching on windows

Background
On windows we see:
[DBG] Could not move C:\Users\agentuser\AppData\Local\Octopus\Temp\bda9d58b-4b0f-4069-846e-9abbe3c8e7b7\extracted\build\netcoreapp2.2\Tentacle.win-x64 to D:\buildAgent\tempDir\buildTmp\TentacleBinaryCacheTentacleBinaryCachecbb708fc0e124ef8bdf8c7c8929f42ce\net48\5.0.12 tentacle may not be cached locally. System.IO.IOException: Source and destination path must have identical roots. Move will not work across volumes.
[15:35:07 ](https://build.octopushq.com/buildConfiguration/TeamFireAndMotion_OctopusTentacleNet48_IntegrationTestNetFramework/7975151?buildTab=log&focusLine=1674&logView=linear&linesState=1674)    [DBG]    at System.IO.DirectoryInfo.MoveTo(String destDirName)
[15:35:07 ](https://build.octopushq.com/buildConfiguration/TeamFireAndMotion_OctopusTentacleNet48_IntegrationTestNetFramework/7975151?buildTab=log&focusLine=1675&logView=linear&linesState=1675)    [DBG]    at Octopus.Tentacle.Tests.Integration.Support.TentacleBinaryCache.<GetTentacleVersion>d__4.MoveNext() in C:\BuildAgent\work\639265b01610d682\source\Octopus.Tentacle.Tests.Integration\Support\TentacleBinaryCache.cs:line 42
[15:35:11 ](https://build.octopushq.com/buildConfiguration/TeamFireAndMotion_OctopusTentacleNet48_IntegrationTestNetFramework/7975151?buildTab=log&focusLine=1676&logView=linear&linesState=1676)    [DBG] Downloading https://f.feedz.io/octopus-deploy/dependencies/packages/Tentacle.Binaries.win-x64/5.0.12/download to C:\Users\agentuser\AppData\Local\Octopus\Temp\bda9d58b-4b0f-4069-846e-9abbe3c8e7b7\4c5e45619635407b8c2fc54bf2cafe5d
15:35:11     [DBG] Downloading https://f.feedz.io/octopus-deploy/dependencies/packages/Tentacle.Binaries.win-x64/5.0.12/download (36064767 bytes)

How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

This was pulled in with another PR


CSS variables mixed with SCSS variables don't emit proper CSS in 4.8+

// styles.scss
$theme-color: #177380;
body {
  --theme-color: $theme-color;
}

Run:
node-sass styles.scss

Output in 4.7.4 (expected):
body {
  --theme-color: #177380; }

Output in 4.8.1-4.8.3:
body {
  --theme-color: $theme-color; }

$ npm -v
5.6.0
$ node -v
v9.11.1

Please search issue before opening a new one.
body {
  --theme-color: #{$theme-color}; }

The old behaviour was a bug that violated the Sass language spec.

Thanks, apologies for not finding the existing issue!

Please search issue before opening a new one.
body {
  --theme-color: #{$theme-color}; }

The old behaviour was a bug that violated the Sass language spec.

Still not working for me


--set-num-threads parameter throws error

The following error occures when trying to use the --set-num-threads parameter:
RuntimeError: set_num_threads expects an int, but got str
Besides this I find it strange to use "set" in a commandline option. Maybe we can just skip that and use --num-threads instead.
Definition of Done:

[x] The --set-num-threads parameter has been renamed to --num-threads
[x] The value of --num-threads has been casted to integer by using python's "int()" method

Additional Information
I couldn't notice a speed improvement by using 10 Threads (M2 Max). Maybe pytorch just isn't able to use the threads in that environment.

thanks!


collector: added log conversion to entries

Added functionality to convert pdata Logs to Cloud Logging entries. Also
added necessary configuration and self observability.

/gcbrun


Can't use firefox impersolate

Describe the bug
Can't use any impersolate for Firefox
To Reproduce
r = requests.get("https://tls.browserleaks.com/json",
                 impersonate="ff102",
                 proxies=proxies,
                 headers = header)

Expected behavior
Expect some JA3 hash for firefox
Versions

OS: Windows 11
cffi==1.16.0
curl-cffi==0.5.10
dpkt==1.9.8
pycparser==2.21
pyja3==1.0.0

Additional context

Sync session

Error
raise RequestsError(f"impersonate {impersonate} is not supported")
curl_cffi.requests.errors.RequestsError: impersonate ff102 is not supported

duplicate of https://github.com/yifeikong/curl_cffi/issues/59

Close as dup of #59


No Endpoints under state

I run version 2.8.0 and get no endpoints under state. Cars in Profile BMW IX I20 2024 and BMW I3S 2020

solved after some wait time


Errors on startup after updating

I assume this is a problem with a possible update of zsh, but I ran a brew upgrade and now I am seeing this whenever I open a new terminal.
/opt/homebrew/Cellar/zplug/2.4.2/repos/zplug/zplug/autoload/init.zsh:16: colors: function definition file not found
polling.zsh:17: add-zsh-hook: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::get_interfaces:49: regexp-replace: function definition file not found
__zplug::core::core::prepare:67: compinit: function definition file not found
[zplug] ERROR: The loading of zplug was discontinued.
__zplug::core::load::from_cache:24: compinit: function definition file not found
[zplug] These repos have failed to load:
- zsh-users/zsh-autosuggestions
- MichaelAquilina/zsh-you-should-use
/Users/mitchell/.zshrc:67: compinit: function definition file not found
/Users/mitchell/.zsh/funcs:10: command not found: compdef
/Users/mitchell/.zsh/funcs:159: command not found: compdef
/dev/fd/12:57: add-zsh-hook: function definition file not found
/dev/fd/12:58: add-zsh-hook: function definition file not found

This is what I currently have with regard to zshrc
export ZPLUG_PROTOCOL="SSH"
export ZPLUG_HOME="$brew_prefix"/opt/zplug
source $ZPLUG_HOME/init.zsh

zplug 'zplug/zplug', hook-build:'zplug --self-manage'
zplug "zsh-users/zsh-autosuggestions"
zplug "zsh-users/zsh-completions"
zplug "zsh-users/zsh-syntax-highlighting"
zplug "lib/history", from:oh-my-zsh
zplug "lib/key-bindings", from:oh-my-zsh
zplug "plugins/colored-man-pages", from:oh-my-zsh
zplug "plugins/fzf", from:oh-my-zsh
zplug "MichaelAquilina/zsh-you-should-use"

if ! zplug check --verbose; then
    printf "Install? [y/N]: "
    if read -q; then
        echo; zplug install
    fi
fi

zplug load

I think this may have been a weird combo of needing to restart my computer as well as Alacritty having been updated, but not restarted.

Confirmed this weird behaviors with zplug + kitty terminal emulator. Fixed by reboot my computer.


Interface not working

Builded with OpenWRT 15 and 18, same for both, onlt Dashboard pages working, everything else showing white pages.

Yes. It's work in progress.
On Sat, Feb 16, 2019 at 2:32 AM Ventsislav Ivanov<EMAIL_ADDRESS>wrote:

Builded with OpenWRT 15 and 18, same for both, onlt Dashboard pages
working, everything else showing white pages.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zhaojh329/vuci/issues/8, or mute the thread
https://github.com/notifications/unsubscribe-auth/AbWFD_hq8tbd7luXNlO5Tcfqtqy05DsYks5vNv0ugaJpZM4a-M7s
.

The project looks dead, anyway looking good.

After half a year of busy time, I was free recently and started to develop.


Scale to many CPU cores

What is the best way to achieve best throughput (only forward runs) on a multi-cpu/many-core server?
Description
I am using CPP package. My problem is "embarrassingly parallel" - each forward pass can be run independently.  It looks to me that mxnet doesn't scale well with number of CPUs.
My goal is to maximize overall throughput, not to minimize each forward passes' computation time.
I can achieve it by running multiple processes (one per core). Maybe I missed something, but I couldn't find a way to do it with multiple threads.

One practical approach is to launch many instances (mxnet itself) and each one uses several cores, say 4,8,16, for your inference.  So you can get maximize the overall throughput a lot.
BTW, MKLDNN backend will be much faster now. You need to specify the thread number and bind thread to physical cores explicitly.
https://github.com/apache/incubator-mxnet/blob/master/MKLDNN_README.md

@mxnet-label-bot : [C++, Windows, Question]

Hi pengzhao.
In that table from MXNET-11 does "28 core, 1 instance" means running 1 process with 28 threads?
Thank you.

Hi pengzhao.
Can you elaborate how to do it? Create multiple executors and call them from multiple threads? Which version of mxnet? Thanks you.

@qw42 , you can use master branch or release version >= 1.2.0.
A simple experiment by following commands.
export OMP_NUM_THREADS=${CORES_NUM}
mem_num=0
COMMAND="numactl -C 0-$[${CORES_NUM}-1] -m ${mem_num} python XXX.py"
for ((i=1;i<${INSTANCES};i++))
        do
              if [ $[${CORES_NUM}*$i] -ge 28 ];then
               mem_num=1
               fi
              CAFFE_COMMAND+=" & numactl -C $[${CORES_NUM}*$i]-$[${CORES_NUM}*$i+${CORES_NUM}-1] -m ${mem_num} python XXX.py"
        done
echo ${CAFFE_COMMAND} > run_instance.sh
bash -x run_instance.sh

@qw42 did you have a chance to try and could this approach help for your case？

Thanks for the feedback. It needs the FW level supports or you can write your own code for your targets.


Unable to find type [DuckDB.NET.Data.DuckDBConnection] Error with PowerShell 5.1 , version 0.14 of PSDuckDB on Windows 11

Install-Module -Name PSDuckDB
.
.
Welcome to PSDuckDB! 08/16/2024 08:38:29
Connected to an in-memory database
Unable to find type [DuckDB.NET.Data.DuckDBConnection].

@Dolphi2 thanks for trying the module. I need to update the manifest. Sorry - PS 7.x only


feat(Usage Reports): Update Account Summary

PR summary
Updates Account Summary to reflect changes to the API
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
Updates Account Summary to reflect changes to the API
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
Integration tests:
Running Suite: UsageReportsV4 Suite
===================================
Random Seed:<PHONE_NUMBER>
Will run 80 of 80 specs

••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••
Ran 80 of 80 Specs in 6.900 seconds
SUCCESS! -- 80 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok      github.com/IBM/platform-services-go-sdk/usagereportsv4  8.071s

Examples:
Ran 80 of 80 Specs in 8.151 seconds
SUCCESS! -- 80 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok      github.com/IBM/platform-services-go-sdk/usagereportsv4  9.335s

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


Why does Union of types not resolve to this generic function definition?

I have the following example.
from typing import Union, TypeVar, Dict, Sequence

IdentifierSymbol = TypeVar("IdentifierSymbol", str, int)

def f(ids: Sequence[IdentifierSymbol]) -> Dict[int, IdentifierSymbol]:
    return dict(zip(range(len(ids)), ids))

def g(ids: Union[Sequence[int], Sequence[str]]) -> int:
    x = f(ids) # Problem here
    return 1

PyLance complains with the following in the line x=f(ids):
Argument of type "Sequence[int] | Sequence[str]" cannot be assigned to parameter "ids" of type "Sequence[IdentifierSymbol@f]" in function "f"
  Type "Sequence[int] | Sequence[str]" cannot be assigned to type "Sequence[IdentifierSymbol@f]"
    TypeVar "_T_co@Sequence" is covariant
      Type "str" is not compatible with constrained type "int" PylancereportGeneralTypeIssues

I am not sure how to interpret this, but it sounds like the problem is that int is not compatible with str. But why is this an issue in this case? I have either a sequence of strs or a sequence of ints, where does the compatibility between int and str come in?
Using overload instead of generic types still doesn't resolve. Any hints as to how to do this correctly?

The type constraint solver needs to solve for IdentifierSymbol, and it does so by considering all of the assignments to it. In your example, the type of ids is Sequence[int] | Sequence[str], so both int and str are being assigned to IdentifierSymbol. When multiple distinct types are assigned to a TypeVar, the constraint solver needs to find a common type that satisfies both. Pylance/pyright solve this by creating a union of the two types, so in this example, IdentifierSymbol is solved as str | int. Mypy takes a different approach and "joins" the two by finding a common ancestor. In this case, the common ancestor is object. Both Pylance/pyright and Mypy emit an error because the type determined by the constraint solver (int | str or object) do not match the constraints imposed by IdentifierSymbol, which says that the type must be either a str or an int.
I know this is a bit unintuitive, but it's how constraint solving works in the Python type system.
I can see a few workarounds:

Remove the constraints int and str from IdentifierSymbol. This workaround works in both Pylance/pyright and in Mypy.
Replace the constraints int and str with a bound=Union[int, str]. This works fine in Pylance/pyright, but Mypy doesn't seem to like it for reasons I don't understand.

Here's one more potential workaround that uses a new type feature that will appear in Python 3.10. It's documented in PEP 647.
from typing import Any, Type, TypeGuard, Union, TypeVar, Dict, Sequence

IdentifierSymbol = TypeVar("IdentifierSymbol", int, str)
T = TypeVar("T")

def is_sequence_of(ids: Sequence[Any], type: Type[T]) -> TypeGuard[Sequence[T]]:
     return len(ids) == 0 or isinstance(ids[0], type)

def f(ids: Sequence[IdentifierSymbol]) -> Dict[int, IdentifierSymbol]:
    return dict(zip(range(len(ids)), ids))

def g(ids: Union[Sequence[int], Sequence[str]]) -> int:
     if is_sequence_of(ids, int):
         x = f(ids)
     elif is_sequence_of(ids, str):
         x = f(ids)
     return 1

Thanks! We're currently still on an older version of Python, but hope to be able to explore this feature when we migrate.
It seems that it also works if I try to define g as def g(ids: Sequence[IdentifierSymbol]) -> int, but then PyLance doesn't like that I am using the TypeVar only once (presumably because it's not happy that I am making it seem like a generic).

I'm not sure there's any other work to be done here; this is a product of how the type system works.


iPad Pro 9.7" WIFI Error Says Unsupported

When I enter my ECID and model for my 9.7" WIFI iPad Pro it goes to a screen that says the device is not supported (on the tsssaver.1conan.com website), but when I enter my ECID into the box to take me to my previously saved blobs it takes me right to my old blobs (verified good, used to upgrade to 10.2 from 9.3.3). I'm not sure if this is an issue with the source code or the site itself, so I apologize if this is not the appropriate place to post this issue. -Thanks!

This is an issue with the website's source code which is newer than the code here.
Please email me or message me on reddit instead. (That said, I'll try fixing that ASAP)


Finalising Our Idea

#1 From That We Can mostly say we are doing project two unless anyone has a specific objection please comment right now.
I would like to start talking about the specificity of the project and assigns individuals to certain parts and tasks of the project.

[ ] Choose Scope of Application

[ ] Choose What Platforms this Application would be one

[ ] Choose The Technologies Needed to Achieve This(Including Languages we will be using)

[x]  Study Each Individuals Team member's strengths and weaknesses to see what our team can do.

[ ] Do a full feasibility study of the application in question

[ ] Dividing and Assigning Work to certain parts of the project.

[x]  Construct a Timeline for the project and assign milestones we need to meet

What should I do, senpai?
((It was a compliment btw...))


I don't see qrcode in 0.4.31

[22042]: [2017-11-5 19:41:15] [homebridge-tplink-smarthome.TplinkSmarthome] homebridge-tplink-smarthome v3.10.0, node v8.4.0, homebridge v0.4.31
Nov 05 19:41:15 raspberrypi homebridge[22042]: [2017-11-5 19:41:15] [homebridge-tplink-smarthome.TplinkSmarthome] Configuring cached accessory: [Enchufe Despacho] 80067D13E0B63549B3C421437CC2A3EF170F269C 620ddee9-
Nov 05 19:41:15 raspberrypi homebridge[22042]: [2017-11-5 19:41:15] [homebridge-tplink-smarthome.TplinkSmarthome] Configuring cached accessory: [Enchufe Salon] 8006D6898FE186D00ACAC4BA2882F84117F6ED0D 1aa747c6-602
Nov 05 19:41:15 raspberrypi homebridge[22042]: Scan this code with your HomeKit app on your iOS device to pair with Homebridge:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:15 raspberrypi homebridge[22042]:
Nov 05 19:41:16 raspberrypi homebridge[22042]: [2017-11-5 19:41:16] Homebridge is running on port 50080.
Nov 05 19:41:16 raspberrypi homebridge[22042]: [2017-11-5 19:41:16] Camara Dahua is running on port 46563.
Nov 05 19:41:16 raspberrypi homebridge[22042]: [2017-11-5 19:41:16] Camara Habitacion is running on port 44843.

I have seen the same thing happen when running Homebridge in Docker on a Synology NAS - the user interface only shows plain text logs so the QR code gets scrambled.
Would it be worth continuing to display the pairing pin below the QR code output, perhaps alongside instructions of how to enter it manually in iOS 11? This would also keep it backwards compatible for those still running iOS 10.

Hello,
I want back the display of the code in text form for the console.
Thank you

Same issue here... The QR code is not visible is many cases!
Please add an option to switch between the new and the old?
Would be very helpful. It breaks the Homebridge add-on for Home Assistant (since Hass.io relies on Docker with a web interface showing the logs).

Wow, just read a lot about the packages that generates the QR code.
Seriously? Adding a package that basically hasn't received any commits or other maintenance for years? With rendering issues open for Windows 10 & MacOS? And a couple of PR's that are left to collect dust?
What does that say about this project?

Maybe @KhaosT can tell us a little more?

I'm really sorry about how the recent change of QR code breaks on some platforms. I should be more careful with that one. Do you guys think I should just switch back to printing the code, or try find another package for printing QR code in terminal?

I think just printing the code in the same way it was in the previous version under the new QR code would satisfy everyone.
I don't think we'll ever have much luck getting the QR code to print correctly in plain text terminals like Synology or Hass.io use.

Okay I added back the normal code form in 0.4.32. Once again, really sorry about that.

With 0.4.32 i got this:
Nov 14 10:01:15 raspberrypi homebridge[31374]: Setup Payload:
Nov 14 10:01:15 raspberrypi homebridge[31374]: X-HM://0023ISYWYXMAP
Nov 14 10:01:15 raspberrypi homebridge[31374]: Scan this code with your HomeKit app on your iOS device to pair with Homebridge:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]:
Nov 14 10:01:16 raspberrypi homebridge[31374]: Or enter this code with your HomeKit app on your iOS device to pair with Homebridge:
Nov 14 10:01:16 raspberrypi homebridge[31374]: ┌────────────┐
Nov 14 10:01:16 raspberrypi homebridge[31374]: │ undefined │
Nov 14 10:01:16 raspberrypi homebridge[31374]: └────────────┘
Nov 14 10:01:16 raspberrypi homebridge[31374]: [2017-11-14 10:01:16] Homebridge is running on port 42837.

I really like this new QR-Code, but agree that the pin might still be needed.
Is there some way to generate the code from another application, i would like tp thow it on a webpage.

@cis2131 Yeah, you can create yourself. See the Setup Payload part with a URL like X-HM://0023ISYWYJMRW, simply create an QR code based on that and iOS will scan it.

@Krocko try again with 0.4.33?

@KhaosT now the pin is visible. Trank you.

Thanks, but is there a way to get it after start (other than searching trough the logfile), or to konw the value before start?.

@cis2131 Once it's there, it remains the same until you reset the accessory. Technically you can put the setupID in the bridge configuration file and then manually compute the setup payload in advance...

Is that possible with current version?

@cis2131 for how to compute setup payload, see here (https://github.com/nfarina/homebridge/issues/1588#issuecomment-341158722)


Add support repetitive arguments to operand

Fixes https://github.com/jaegertracing/jaeger-operator/issues/1415
Signed-off-by: Ruben Vargas<EMAIL_ADDRESS>

CI seems to be failing.

Let me know when this gets ready to be reviewed.

This is ready for review

@pavolloffay Could you please review?
Thank you!


Topology-edit check if a child is removed

Throw an error if this happens
Don't clone and don't update
Ignore this validation when the graph is already incorrect

Fixes #531

@ChaelKruip could you maybe validate this error message:
https://github.com/quintel/etmoses/pull/531/files#diff-c6a4af8568e53f47391a0aacbe40d6e9R102

@ChaelKruip could you maybe validate this error message:

I think it is fine. Is that the type of validation you want?

It's more to check if the error message is 'good'. If it's understandable to the user. I would say it is but maybe you have a different opinion about it.


pipenv should inform about Python version mismatch

If a dependency requires a newer Python version than configured in the current Pipfile, then pipenv fails to install the depencencys - which is totally fine - but didn't give a reason why the installation failed. Here an example of the pipenv output:
thomass@bwpm-G3MW424H6N cdk-base-stacks % pipenv install --dev
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
Building requirements...
Resolving dependencies...
✔ Success!
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✘ Locking Failed!

CRITICAL:pipenv.patched.pip._internal.resolution.resolvelib.factory:Cannot install -r /var/folders/6p/p41kzv952szcvs_h3hnk9fz40000gn/T/pipenvr0fs9l8crequirements/pipenv-xon0274m-constraints.txt (line 4) because these package versions have conflicting dependencies.
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/resolver.py", line 821, in _main
[ResolutionFailure]:       pre, clear, verbose, system, write, requirements_dir, packages, dev
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/resolver.py", line 776, in resolve_packages
[ResolutionFailure]:       requirements_dir=requirements_dir,
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/resolver.py", line 757, in resolve
[ResolutionFailure]:       req_dir=requirements_dir,
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/utils/resolver.py", line 1097, in resolve_deps
[ResolutionFailure]:       req_dir=req_dir,
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/utils/resolver.py", line 883, in actually_resolve_deps
[ResolutionFailure]:       resolver.resolve()
[ResolutionFailure]:   File "/Users/thomass/Library/Python/3.8/lib/python/site-packages/pipenv/utils/resolver.py", line 688, in resolve
[ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  You can use $ pipenv install --skip-lock to bypass this mechanism, then run $ pipenv graph to inspect the situation.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

It would be really nice if pipenv would detect, that the Python version of the current project is too low for the dependencies and print out that information. Otherwise it is really difficult - or even luck to figure that out.

@ThomasSteinbach This is probably a non-trivial enhancement request -- we are essentially relying on the pip resolver to resolve packages that match the markers from the Pipfile -- in this case resolution was impossible.  It would be hard to know the reason resolution was not possible is because of a general version mismatch vs a new version does exist but isn't compatible with your python marker.

I believe you get this level of output with the -v or --verbose flag in recent releases.

I don't understand why this issue was closed, it is a clear regression. I do not get the desired output when running with verbose mode.
Just the fact that dependency mismatch makes pipenv print an exception and a stack trace is a strong sign that it does not handle this correctly anymore. It used to just print out what was wrong and exit gracefully as @sireliah showed.


Update Issue - 11275978

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 77.0

Operating System
Windows 10

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


WIP: Add documentation for the Adapter trait

I think it's important that this trait in particular is well-documented. Right now I'm working based on demos, but I chose to relate it to GraphQL since it's currently the query language of choice.
I might be a bit off with descriptions here, and I think the query_hint and vertex_hint may need more explanations, since they are ignored in the demos from what I can see. But according to #131 these are subject to change soonTM :small_airplane:

Tbh I'm thinking of completely revamping this trait.
#131 will remove the explicit "hint" arguments in favor of a &QueryInfo value. If we then adopt the (in my mind, better) nomenclature from BasicAdapter, then there's basically nothing left here that won't get changed.
What do you think? Are the names of BasicAdapter better and easier to understand? If we were to adopt them here in a breaking change, would that be an improvement?

Sorry about the CI, have added  "editor.formatOnSave": true now...
I have tried, based on demo-hackernews, to provide adequate examples and explanations for some of the adapter methods. However, I might be off at some places. But perhaps it is a good idea for a complete newbie to attempt to write the docs, because it makes it painfully obvious what is hard to understand...
--- Above this line was before you posted your comment :)
Right now I think this trait is quite difficult to understand, but looking at BasicAdaptor I realize that I should have looked there first!
If nothing else, BasicAdapter has a lot better method names, and just that makes it easier to use. It might be worth a breaking change, because right now the demos just ignore query_hint and vertex_hint, which makes them basically magic.
For me at least, with very limited background in graph theory and graph databases, but some knowledge of GraphQL, I think it might be helpful to explain concepts in the adaptors by describing them in a GraphQL context (like "a starting vertex is like a field of the Query type in a GraphQL schema"), like I have tried to do in this PR.

Right now I think this trait is quite difficult to understand, but looking at BasicAdaptor I realize that I should have looked there first!

This is the fault of insufficient docs 😅 Let's try to figure out how to best direct the next person to that easier-to-implement trait. It's easier to implement because it can't do everything that the "real" one under the hood can do, but it's enough for most cases.

If nothing else, BasicAdapter has a lot better method names, and just that makes it easier to use. It might be worth a breaking change, because right now the demos just ignore query_hint and vertex_hint, which makes them basically magic.

Yes, agreed. I realized a design like #131 is necessary because the hint parameters are technically all you need but in practice are just waaay too tedious to use. After I've put in a few more iterations on #131, I'd love your help and feedback there on naming and documentation. It's super useful to have a second pair of eyes on these things — thanks for taking the time!

I think it might be helpful to explain concepts in the adaptors by describing them in a GraphQL context (like "a starting vertex is like a field of the Query type in a GraphQL schema"), like I have tried to do in this PR.

I understand the sentiment here. I have two concerns, and a concrete proposal which I'll name first.
I think we may be better off with an "Learning Trustfall for GraphQL users" page somewhere in the docs, instead of inlining the GraphQL analogies and comparisons into the main documentation.
Here's why:

Trustfall isn't actually GraphQL: it uses an analogous syntax for the most part, but its semantics are quite different and more powerful. When seeing the syntax, people very familiar with GraphQL are already tempted to (incorrectly) say "ah, this is just another GraphQL federation implementation" -- the briefest of looks at the HackerNews thread on Trustfall is sufficient to confirm this. We need to discourage that leap as hard as we can, rather than encouraging it by comparing and analogizing to GraphQL across the docs.
People not familiar with GraphQL shouldn't have to learn it first before understanding our docs. Trustfall needs to be friendly to people from a SQL background, or Excel background, or dataframes background. Explaining Trustfall concepts by comparing them with analogous GraphQL concepts makes GraphQL a "preferred" background for learning Trustfall, which not only doesn't help everyone else but actively hurts: "oh no, another thing I have to learn before I can understand this."

By having documentation free of concepts from other languages, and having separate "Learning Trustfall for X users" pages in the docs, we can let the readers self-select into the right groups and then we can make the right analogies to them:

In the SQL case: "The @optional directive is like LEFT JOIN in SQL: it means it's okay if the data on the other side doesn't exist. In that case, any selections from inside the @optional block have a value of null."
In the GraphQL case: "A starting vertex is one available as a field on the Query type of the schema."
etc.

What do you think?

Yes, agreed. I realized a design like https://github.com/obi1kenobi/trustfall/pull/131 is necessary because the hint parameters are technically all you need but in practice are just waaay too tedious to use. After I've put in a few more iterations on https://github.com/obi1kenobi/trustfall/pull/131, I'd love your help and feedback there on naming and documentation. It's super useful to have a second pair of eyes on these things — thanks for taking the time!

I'd be happy to help! Right now I'm looking intro using Trustfall myself, so I'm very interested in seeing improvements.

I think we may be better off with an "Learning Trustfall for GraphQL users" page somewhere in the docs, instead of inlining the GraphQL analogies and comparisons into the main documentation.

Reading your response I would say I agree. I think the reason I have to reason around it like that myself is that when there is nothing to latch onto, it is much easier to map it to something you already know.
If anything, I think the docs (or README, or whatever) could use some images of operations. I saw some on your blog, but they don't explain in much detail what actually happens inside the adapters/engine, and how that would be different from GraphQL. This might be just me though.
Another question is: How much does one want to steer the adapter implementations? Looking at the demos, there are definitely patterns that occurs several times. Are all users to write their own implement_property! when most of them look the same? Could there be room for a trustfall_adapter_macros (or is there one that I have missed?)

In the SQL case: "The @optional directive is like a SQL LEFT JOIN: it means it's okay if the data on the other side doesn't exist. In that case, any selections from inside the @optional block have a value of null."
In the GraphQL case: "A starting vertex is one available as a field on the Query type of the schema."

Just those two are very useful: Take something the users know, and explain new concept with old ones. Perhaps we shouldn't explain things using only GraphQL, but implementing these in the docs (where people go first) might come a long way.

If anything, I think the docs (or README, or whatever) could use some images of operations. I saw some on your blog, but they don't explain in much detail what actually happens inside the adapters/engine, and how that would be different from GraphQL. This might be just me though.

The README definitely needs several layers' worth of rework. Some images would definitely be nice, but I think some of the "getting started" docs might be more useful now that many people are already at least somewhat familiar with the project and what it does.
Ultimately it's just a question of sequencing the work in maximum-impact order. I think good images are a bit difficult to make, and probably not the immediate highest priority compared to "getting started" docs that point to BasicAdapter instead of Adapter, for example.

I'd also love to know about the domain where you're thinking about using Trustfall, if you'd be up for a chat? I hang out on the rust-lang Zulip and you can also find me on Discord if you'd prefer that. No pressure, obviously.


Mongodb file download fix

We used the mongodb persistence in our try-out of the bays components and found the following issues when downloading files:

file names are constructed containing the AAS ID (e.g. "http://smart.festo.com/id/instance/99920200616214522000030025") - this may not result in a valid filename
when fetching files from mongodb collection they are not stored in a dedicated folder

Fixes:

replace characters in filenames that may cause troubles by "_"
use a temporary directory (previously generated) to save files fetched from mongodb

Thank you very much for providing this PR! Would it be possible to sign the ECA?

Thank you very much for providing this PR! Would it be possible to sign the ECA?

Done!

After some internal discussions, we realized that the issue is more complex than it seems. There are the following points that need to be considered:

Backwards compatibility: If we change the way how filenames are generated, a file stored in the old way is not findable anymore. Thus, the logic needs to be adapted in a way that it still works with the old names, e.g., checking for the new filename on retrieval and if it does not exist, checking for the old filename. Storing a file always should utilize the new filename.
Filename collision: By applying the regex, ids "id:" and "id/" would both be changed to "id_". Thus, if same idShorts are used, files may get overwritten. A possible solution for this would be to apply the regex as proposed by you and attaching the identifier-Hash to the resulting filename.

Would it be possible to update the PR the consider both issues?

Is this PR still active? If no, I will close it in a few days

Is this PR still active? If no, I will close it in a few days

I am currently working on a solution that takes the named issues from your earlier comment into account, maybe I can provide a proposal this week.

Thank you very much for updating the PR! Feel free to ping me when you're done with your implementations, I will then gladly review it again

Thank you very much for updating the PR! Feel free to ping me when you're done with your implementations, I will then gladly review it again

Ping ;-) ... the last commit fixed the failing test, so I am finished now - thnx for reviewing again 👍

Looks good to me. Thank you very much for your contribution!


Integration with Updated CAD-to-DAGMC

CAD-to-DAGMC had a recent update that slightly changes its syntax. ParaStell's main branch was updated to reflect this change thanks to PR #77, but the oo_version branch will need the same changes.

Closed for reasons discussed in #80


[prod] Add JNDI DataSources (for ISPN) and validation rule pattern test

Porting validation rule test from indy-1.7.x branch (1cf4bebaf8b5fc95e0a6960419434552b22763fd)
Porting 3 commits from indy-1.7.x which add support for JNDI DataSources, which allow us to configure a single connection pool for all ISPN caches to use, instead of creating one connection pool per cache, which puts a lot of extra load on the DB.

This supercedes #1163 since we should be making an effort to get back to master branch. I'll still merge the other PR in case we need to release a hotfix in the meantime though.


feat: implement image shape support radius

实现 Image 图形，支持设置裁切半径

Implement Image shape and support setting clip radius property

issue: https://github.com/antvis/G6/issues/6098


Fix endpoints of some models

https://platform.openai.com/docs/models/model-endpoint-compatibility

Already in development branch


Removed extraneous dev note

@mcleblanc @jillre : PR passed acceptance checklist. Please advise re: writer assignment for further review.

@riverar : Thanks for your continued interest in our docs.


added note about orphaned users

#5411

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@greg-lindsay - PR has been copy edited and is ready for final review. This is based on issue #5411
Thank you.
cc: @kenwith


feat(genesis-applier): create genesis_applier package

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#44  👈
#43
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


System.NullReferenceException: 'Object reference not set to an instance of an object.'

Hi,
In the latest version of the code I'm getting:
System.NullReferenceException: 'Object reference not set to an instance of an object.'
in
private async Task Authenticate(string username, string password, string endpoint)
in the
public partial class Client : ISwiftClient, IDisposable

This is not being called:

I'm using this code to call the SwiftClient and the code works with version 1.2.2 of the nuGet package

I'm using this code:
public class OpenStackSwift : IDisposable, IFileServer
{

    SwiftClient.Client client;
    string containerId;

    // Flag: Has Dispose already been called?
    bool disposed = false;

    public OpenStackSwift(string username, string password, List<string> endPoints, string containerId)
    {
        this.containerId = containerId;
        client = new SwiftClient.Client()
        .WithCredentials(new SwiftCredentials
        {
            Username = username,
            Password = password,
            Endpoints = endPoints
        })
        .SetRetryCount(6)
        .SetRetryPerEndpointCount(2)
        ;
    }

    public void UploadFile(System.IO.Stream stream, string fileName)
    {
        stream.Position = 0;
        using (MemoryStream ms = new MemoryStream())
        {
            stream.CopyTo(ms);
            Task<SwiftResponse> task = Task.Run(() => client.PutObject(this.containerId, fileName, ms.ToArray()));
            SwiftResponse response = task.Result;
        }
    }

Hello @jluzardo1971
I am also getting the same error. Do you have any comments to resolve this ?. Your input will help a lot.
Thanks in advance.


spike: look into migrating to using pydantic-sqlalchemy

Acceptance Criteria

Investigate using pydantic-sqlalchemy to minimize the amount of models and schemas used.

Tech Notes

At the moment sqlalchemy models are being used for interacting with the database and table creation, where pydantic models are used for data validation and within the logic of the code.
Ideally instead of having a schema and db model for the same thing, we would just use a single schema that can also be used by the db

After making changes to crud.py to return the application layer data classes, I don't think this is really needed as much anymore.
These changes can be found in #307 and #309


Improve fake data performance

This speeds up fake data a bit, and extracts a few functions out to improve readability

Ready for review


RPS mode data size feature

I see this data size is only applicable in RPS and HTTP invoker.
So, a couple of questions. Do we have or can we add comparable data movement for trace execution? The other question would be to find some dataset to make it realistic.
And what about RPC mode? We have a message in protobuf that we don't use at all; we can use it to model data movement.

@leokondrashov: There is an opportunity to add data movement for trace execution. This paper gives some insight into distribution and releases the trace. I think this could be nice project for a student to take care of -- look into the data, find a way to match functions in the execution trace with data trace, and add such a feature in loader.
Regarding the RPC mode, I can add support for it in gRPC. I think this will perform poorly with large data payloads (like 1MB and bigger), so I expect significant overhead there. What do you think?

The payloads are quite small in this paper, up to 10s KB in the majority of cases, so I don't see a problem with adding it to gRPC.
The other question is also how to interpret these numbers, they are not only about inputs that we send directly with the requests, but also outputs. Originally, it was intended more as a storage workload characterization, not data moved through HTTP with request/response.
Previously, @JooyoungPark73 was doing some work on that front with the Azure Blob trace you mentioned. I think we should have some discussion about that.


create connection without pairing

can i connect two devices without pairing.please help me out

With the current API, no.


uniformBucketEnabled.rego referencing deprecated config

terrascan version: v1.2.0
Operating System: Mac OS X 10.15.7

Description
Terrascan is referencing the deprecated config for google_storage_bucket called bucket_policy_only.  Terraform updated their logic to now use uniform_bucket_level_access instead, so the rego file here needs to be updated to say:
package accurics

uniformBucketEnabled[api.id]
{
     api := input.google_storage_bucket[_]
     not api.config.uniform_bucket_level_access == true
}

What I Did
Terraform Resource:
resource "google_storage_bucket" "my_bucket" {
  project = "my_project"
  name    = "my_name"

  uniform_bucket_level_access = true
}

Command ran + response:
terrascan scan
results:
    violations:
        - rule_name: uniformBucketEnabled
          description: Ensure that Cloud Storage buckets have uniform bucket-level access enabled.
          rule_id: accurics.gcp.IAM.122
          severity: MEDIUM
          category: Identity & Access Management
          resource_name: $sample_name
          resource_type: google_storage_bucket
          file: main.tf
          line: 100

I'm happy to update this and make a PR with your approval.

Hey @jdyke,
Thank you for bringing this to our notice. We would be very happy if you raise a PR for this!
Please let us know if you face any issues while raising a PR here, we would be happy to help!

Hey @jdyke,
Thank you for bringing this to our notice. We would be very happy if you raise a PR for this!
Please let us know if you face any issues while raising a PR here, we would be happy to help!


[v2] markup different client and server

Version
2.0.0-4
I get
Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
(client) ="1">
<div class="sc-hqyNC jPUbry" data-r
is this something i must investigate more or is this from v2?
i did not have this in 1.4.3

Are you using https://github.com/styled-components/babel-plugin-styled-components? I believe it is still a WIP but it should ensure that you get deterministic class names for your components.

Work fine thank you, i still get that message, but anyway good to have "speaking classes" now I see clearly wich Element is causing it.

Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
(client) ss="page__Container-bodknx eXHyXF" data-
(server) ss="page__Container-1tu8z7d fRbQnJ" data

I'm having this issue too, it seems like the class names are generated in a different way on the client and the server?

Are you running the babel plugin on both the client and the server?

@mxstbr yes, the human friendly class names appear both on the client and the server generated HTML.
 (client) ead__StyledMasthead-1qcthel fGzUif masth
 (server) ead__StyledMasthead-tu6v5q BzWWz masthea

I'm also experiencing this issue when using SSR with next.js + styled-components v2.0.0-4 regardless of whether I use the babel plugin or not? No such issue with same code and 1.4.
Instead, figure out why the markup being generated is different on the client or server:
 (client) class="A__A-11k92nv bvKGfC" data-reactid
 (server) class="A__A-11k92nv cDagqo" data-reactid

Experiencing the same issue as above when using babel-plugin-styled-components.
However, the issue does appear slightly different if I remove the plugin - the error only shows up if I hot-reload some component and then refresh the browser (without restarting the server). If I restart the server then error disappears.
Btw @mxstbr, awesome work with styled components. Big fan!

The unpublished commit https://github.com/styled-components/babel-plugin-styled-components/commit/e0b1b4c248f4864c65a1fd368bfc71b36d7407c9 fixes this issue for me. @mxstbr @geelen any reason it wasn't published yet?

FYI @mxstbr, I have this issue with "preprocess": true using the babel plugin (v1.5.0). I know it is experimental, so setting the value to false fixes the issue. I'm using next.js.
This is my .babelrc:
{
  "presets": ["next/babel"],
  "plugins": [
    [
      "styled-components",
      { "ssr": true, "displayName": false, "preprocess": false }
    ]
  ]
}

Unfortunately we hit a wall with preprocess and that will most likely never be stable/usable. We have some further plans and experiments in this direction, but they won't have anything to do with the Babel plugin.
I'd recommend not using that option as it's unfinished and most likely very broken. We'll probably remove it in a future major version.


fix use configuration

fix misc typing

I merge this to facilitate development, but we have to revisit the order of module imports - the env variable solution should ultimately not be necessary in my eyes.


maps with markercluster

Are there any plans to allow the use of maps with Leaflet.markercluster?  Or is there a known alternative to leaflet-image that works with markercluster?

Hi @atdiff, you posted this issue three times - I'm going to close the other two issues, #77 and #78. Apologies if the expectation was that GitHub issues is a chat-like forum, or the Comment button appeared to be broken - I know there was a GitHub outage recently.
I do recommend you search for existing issues before posting a new one: before you asked this question three times, it had been asked four times before.
There aren't any plans to add Leaflet.markercluster support. Markercluster uses HTML for its output mechanism, and Canvas cannot rasterize HTML. The route to support would be changing Markercluster to support Canvas or SVG, and that's more work on this project than I can justify at this point, given the other things in my schedule. I'd happily accept a PR implementing support, and I'm sure the markercluster folks would as well.
In the meantime, there are options like html2canvas or, if you don't need browser compatibility, phantomjs, which can rasterize marker clusters.

Sorry about the duplicated posts.  It acted like it posted but then didn't appear, so it was very odd.


Add support for multiple sequences in FASTA

It would be really useful if you could provide support for multiple sequences per FASTA file:
#################################################################################
Beginning BACPHLIP pipeline
Traceback (most recent call last):
  File "/global/homes/s/snayfach/.conda/envs/bacphlip/bin/bacphlip", line 8, in <module>
    sys.exit(main())
  File "/global/homes/s/snayfach/.conda/envs/bacphlip/lib/python3.8/site-packages/bacphlip/command_line.py", line 8, in main
    bacphlip.run_pipeline(args.input_file, force_overwrite=args.force_overwrite, local_hmmsearch=args.local_hmmsearch)
  File "/global/homes/s/snayfach/.conda/envs/bacphlip/lib/python3.8/site-packages/bacphlip/bacphlip.py", line 225, in run_pipeline
    six_frame_translate(input_file_path, six_frame_file, force_overwrite=force_overwrite)
  File "/global/homes/s/snayfach/.conda/envs/bacphlip/lib/python3.8/site-packages/bacphlip/bacphlip.py", line 85, in six_frame_translate
    nt_record = check_genome_fasta_reqs(nt_records)
  File "/global/homes/s/snayfach/.conda/envs/bacphlip/lib/python3.8/site-packages/bacphlip/bacphlip.py", line 65, in check_genome_fasta_reqs
    raise Exception("Input fasta file appears to contain more than one sequence record. "
Exception: Input fasta file appears to contain more than one sequence record. Currently bacphlip only supports single contig inputs.

Currently in progress, will try and update you when this feature is pushed. Thanks for the feedback

This should now be possible by providing the --multi_fasta run-time flag. Please see updated docs and note that all sequence records in the multifasta must have unique id's (as parsed by biopython, which is used to extract record id's) in order to write relevant files. Thanks for the feedback and let me know if you have any issues, still writing a few tests for this functionality but it seems stable in my hands.


Weird issue in rendering HTML

Hey,
I'm writing a scraping library, which also leverages selectolax as parser.
But the problem is our test failes with this library, due to strange behaviour. perhaps bug? :)
As shown in picture, we're passing that HTML to the parser, but

Why table is empty? we didn't structured  tag like that. UL comes in table, but in your code that didn't happen. Not sure why you modify the input
Why there's a  tag insert by this library? I could see you have some attr for head, but can't you initialize it as None if there's no head in HTML?

Can you post the original HTML snippet (as text)?

I don't see any problems here. It's not allowed to have ul tags inside table. If you open such HTML in Chrome, it most likely will be transformed in a similar way. That's how malformed HTML is handled by parsers.

I don't see any problems here (visually, looking at the screenshot). It's not allowed to have ul tags inside table. If you open such HTML in Chrome, it most likely will be transformed in a similar way. That's how malformed HTML is handled by parsers and browsers.

Hmm I understand what you're saying, but is there a way we can escape it?
I'm writing a library, called FastCrawler.
What we want to do is to offer a pydantic interface for data models. like
from fastcrawler import CssField, SelectoLaxProcessor, BaseModel

class Person(BaseModel):
    name: str = CssField(query="td:nth-child(1)::text" , processor=SelectoLaxProcessor)
    id: int = CssField(query="td:nth-child(2)::text" , processor=SelectoLaxProcessor)

class PersonTable(BaseModel):
    persons: list[Person] = CssField(query="table tr" , processor=SelectoLaxProcessor)

so I need to iterate over HTML elements, and parse them accordingly. I wouldn't except the string i'm giving is different than what real there is.

No, the parser works the same way as browsers, so it's not possible, unfortunately. Some old browsers won't even display part of this HTML.


Logo for SAC is very tiny

Look into the logo in the header for obs because it’s tiny

@charlotteguard we can only worry about the height of the logo and let the width increase while keeping aspect ratio, which helps for "short and wide" logos but shifts layout of the header - WDYT? We could offset the title from the left instead of trying to center it, maybe?

Looks like that's what we do on the Forecast view header.


new release and prebuilt binaries?

The latest releases does not include a prebuilt binary and the go install command fails with:
~> go install github.com/TorchedSammy/Circleload@latest
go: downloading github.com/TorchedSammy/Circleload v0.3.1
go: github.com/TorchedSammy/Circleload@latest: <EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/TorchedSammy/circleload
	        but was required as: github.com/TorchedSammy/Circleload

This seems to be fixed in the latest branch but it needs a release.

I got it built from source and can also say that the chimu mirror is broken but kitsu works

@TorchedSammy please do! it would be awesome for this project to be able to be used again ^-^

I'd love to help contribute/maintain this if there's anything I can do, I've worked with Go before so it should be easy.

I've made some code changes and all the mirrors work now (hopefully). To fix the original issue, just run go install github.com/TorchedSammy/circleload@master.
When I feel not lazy I'll work on some better readme and a changelog for a new release.


Custom DNS situation

There needs to be some mention whether custom DNS servers are supported and if so how. If not, then some explanation of how to manage this.

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

ID: 8c7387c7-c5ea-584a-1463-eeb562b2cf4d
Version Independent ID: 798fcb07-735e-c79f-6ede-485c671d2364
Content: Scenarios to use a virtual network - Azure Container Instances
Content Source: articles/container-instances/container-instances-virtual-network-concepts.md
Service: container-instances
GitHub Login: @tomvcassidy
Microsoft Alias: tomcassidy

@mattcowen
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@tomvcassidy
Can you please check and add your comments on this doc update request as applicable.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


Add bison.

Some toolchains failed. To exclude broken please send pull request to pkg.bison.

Please rebase/squash to resolve conflicts.

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2426
https://travis-ci.org/ingenue/hunter/builds/276973941?utm_source=github_status&utm_medium=notification

Testing

Some toolchains failed.
Excluding instructions:

https://docs.hunter.sh/en/latest/creating-new/create/cmake.html#excluding-tests

Actually this is an interesting problem in iOS build:
Performing install step for 'bison-universal'

cd /Users/travis/build/ingenue/hunter/_testing/Hunter/_Base/48efcab/76d000c/df2ea98/Build/bison/Source/universal && /Users/travis/build/ingenue/hunter/_ci/cmake/bin/cmake -P /Users/travis/build/ingenue/hunter/_testing/Hunter/_Base/48efcab/76d000c/df2ea98/Build/bison/Source/universal/autotools-merge-lipo.cmake

CMake Error at autotools-merge-lipo.cmake:234 (message):

  lipo execution failed: fatal error: lipo: can't figure out the architecture

  type of:

  /Users/travis/build/ingenue/hunter/_testing/Hunter/_Base/48efcab/76d000c/df2ea98/Build/bison/Build/multi-arch/armv7/bin/yacc

If you look at bin/yacc in any bison installation, you will see it is a shell script to reroute to bison binary. I wonder what the specific issue is here.

Actually this is an interesting problem in iOS build

iOS universal build is not supported well for autotools.

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.2435
https://travis-ci.org/ingenue/hunter/builds/277475809?utm_source=github_status&utm_medium=notification

No upload since package is not cacheable. Released: https://github.com/ruslo/hunter/releases/tag/v0.19.106


vips_colourspace: no known route from 'cmyk' to 'srgb'

Unable to convert one image.
Log:
vips_colourspace: no known route from 'cmyk' to 'srgb'

Here is the same issue:
https://github.com/imgproxy/imgproxy/issues/39
I installed yesterday with
curl -Ls https://github.com/imgproxy/imgproxy/archive/master.tar.gz \
  | tar -xz --strip-components 1 -C .
CGO_LDFLAGS_ALLOW="-s|-w" /usr/local/go/bin/go build -o /usr/local/bin/imgproxy

I don't know if you need informations about the host:
uname -a
Linux serv47797592-02.secure-node.at 4.19.0 #1 SMP Fri Nov 20 21:47:55 MSK 2020 x86_64 GNU/Linux

OS: linux, Debian GNU/Linux 10 (buster) (chroot) - I think it runs RHEL underneath
Processor: 8 cpus, 64bit, little endian, Intel(R) Xeon(R) E-2286G CPU @ 4.00GHz
CC version: cc (Debian 8.3.0-6) 8.3.0
Git version: git version 2.20.1

/usr/local/go/bin/go version
go version go1.16.6 linux/amd64

I add the image as attachment.

Hey Thomas!
It seems like you're using an old version of libvips. I'd recommend using the official Docker image instead of building imgproxy from the source. I spend a lot of time keeping the dependencies updated and patched so everything works as intended. If you build imgproxy from the source, you'll have to deal with all the issues I already fixed in the image.

Thank you!
It works with docker.


Missing ClaimReview and MediaReview Fields

We are missing a bunch of MediaReview and ClaimReview fields. See the note below:
ClaimReviews

worstRating appears to be empty for all but 1 entry.
itemReviewed.name appears to be all empty, which should be “location of statement” in FactStream
I think all the “appearance” (“source of statement” in FactStream) is duplicated in itemReviewed.firstAppearance — we don’t have the “first” flag in FactStream so I think that field should be empty. Also appears we’re maybe outputting an extra set of brackets around appearance (not sure if that matters)
ratingExplanation looks good.

MediaReviews

Missing originalMediaLink
Missing startTime and endTime
We’re still outputting something to creator.Url, which I don’t think we should be as we don’t have that input capability in FactStream. That field appears to be duplicating itemReviewed.Url, it should only appear in the latter.
Duplicating originalMediaContextDescription in itemReviewed.mediaItemAppearance.description; I think we should only keep it in originalMediaContextDescription

We need to accept these and also put them into the blueprints, and probably views

Platonic MediaReview:
{
      "id": "d69e315c-5f98-4641-8e75-453b72df4d24",
      "@context": "https://schema.org/",
      "@type": "MediaReview",
      "author": {
        "@type": "Organization",
        "name": "PolitiFact",
        "url": "http://www.politifact.com/",
        "image": null,
        "sameAs": null
      },
      "datePublished": "2022-09-30",
      "itemReviewed": {
        "@type": "MediaReviewItem",
        "creator": {
          "@type": "Person",
          "name": "Instagram posts",
          "url": ""
        },
        "startTime": null,
        "endTime": null,
        "interpretedAsClaim": {
          "@type": "Claim",
          "description": "Footage NASA shared from the DART spacecraft just before it crashed itself into an asteroid isn't real."
        },
        "mediaItemAppearance": [
          {
            "@type": "VideoObjectSnapshot",
            "accessedOnUrl": "https://www.instagram.com/p/Ci_mPJsgsE6/",
            "description": ""
          }
        ]
      },
      "mediaAuthenticityCategory": "Original",
      "originalMediaContextDescription": "The replay is 10 times faster than reality, except for the last six images, which are shown at the same rate that the DART spacecraft returned them. ",
      "originalMediaLink": "https://www.nasa.gov/feature/dart-s-final-images-prior-to-impact",
      "url": "https://www.politifact.com/factchecks/2022/sep/29/instagram-posts/nasas-footage-dart-spacecraft-crashing-itself-aste/"
    }```


[Suggestion] No redo function

If I am undoing several changes (which works GREAT) and I undo one or two steps too far, I would like to redo.
In the array you use to hold the history, can you do an undo by moving a pointer back from the end, allowing redo to move the pointer forward again, and when you change something then truncate the array and append the latest change?

Why doesn't that already work? Does the redo button not work?

vs.

Oops. Sorry.


Conflict after changes to Delly2 DAG in WGS SV Export

Issue
After changes to Delly DAG in WGS SV Calling, it no longer outputs an merged file, *.delly2.merge_genotypes*.
Error:
11-29 21:51 snakemake.logging WARNING  Building DAG of jobs...
MissingInputException in rule wgs_sv_annotation_vcf_sv_filter  in line 71 of /path/to/snappy-pipeline/snappy_pipeline/workflows/wgs_sv_annotation/Snakefile:
Missing input files for rule wgs_sv_annotation_vcf_sv_filter:
    output: work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.tbi, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.md5, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.tbi.md5
    wildcards: mapper=bwa, caller=delly2, index_ngs_library=<LIBRARY_NAME>
    affected files:
        /path/to/wgs_sv_calling/work/bwa.delly2.merge_genotypes/out/bwa.delly2.merge_genotypes.bcf.csi
        /path/to/wgs_sv_calling/work/bwa.delly2.merge_genotypes/out/bwa.delly2.merge_genotypes.bcf
11-29 21:51 snakemake.logging ERROR    MissingInputException in rule wgs_sv_annotation_vcf_sv_filter  in line 71 of /path/to/snappy-pipeline/snappy_pipeline/workflows/wgs_sv_annotation/Snakefile:
Missing input files for rule wgs_sv_annotation_vcf_sv_filter:
    output: work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.tbi, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.md5, work/bwa.delly2.annotated.<LIBRARY_NAME>/out/bwa.delly2.annotated.<LIBRARY_NAME>.vcf.gz.tbi.md5
    wildcards: mapper=bwa, caller=delly2, index_ngs_library=<LIBRARY_NAME>
    affected files:
        /path/to/wgs_sv_calling/work/bwa.delly2.merge_genotypes/out/bwa.delly2.merge_genotypes.bcf.csi
        /path/to/wgs_sv_calling/work/bwa.delly2.merge_genotypes/out/bwa.delly2.merge_genotypes.bcf

To Reproduce
Required relevant config parts to reproduce the behavior:
  wgs_sv_annotation:
    tool_ngs_mapping_variant_calling: bwa
    tool_variant_calling: gatk_hc
    tools_ngs_mapping: [bwa]
    tools_wgs_sv_calling: [delly2]

  wgs_sv_calling:
    path_ngs_mapping: ../ngs_mapping
    tools:
      dna:
      - delly2
      - manta

  wgs_sv_export:
    release: GRCh37  # REQUIRED: default 'GRCh37'
    path_refseq_ser: ...

Called with cubi-tk snappy kickoff
Expected behavior
The correct file name convention should be set in wgs_sv_annotation.
From snappy-pipeline/snappy_pipeline/workflows/wgs_sv_annotation/__init__.py:
class VcfSvFilterStepPart(BaseStepPart):

     ...

    def get_input_files(self, action):
        """Return path to pedigree input file"""
        assert action == "run"

        @dictify
        def input_function(wildcards):
            if wildcards.caller == "popdel":
                name_pattern = "internal.concat_calls"
                key_ext = {"bcf": ".vcf.gz", "csi": ".vcf.gz.tbi"}
            else:  # delly2
                name_pattern = "merge_genotypes"
                key_ext = {"bcf": ".bcf", "csi": ".bcf.csi"}

            tpl = "work/write_pedigree.{index_ngs_library}/out/{index_ngs_library}.ped"
            yield "ped", tpl.format(**wildcards)
            # TODO: make work for non-Delly?
            tpl = (
                "work/{mapper}.{caller}.{index_ngs_library}/out/"
                "{mapper}.{caller}.{index_ngs_library}"
            )
            # SVs
            wgs_sv_calling = self.parent.sub_workflows["wgs_sv_calling"]
            for key, ext in key_ext.items():
                yield "sv_" + key, wgs_sv_calling(
                    tpl.replace("{index_ngs_library}", name_pattern) + ext
                ).format(**wildcards)

Additional context
Changed output and removed Delly dependency from WGS SV Annotation: #188
WGS SV Export and Annotation have a dependency in kickoff based on PopDel usage, not Delly. But I assume still need to be called...?

CC @Nicolai-vKuegelgen


Issue in window.onload() due to an undefined variable

Minor issue in landing page causing the date when the stats last updated not showing.
Very minor issue but will be patched in next few hours or days.

Fixed in 3.5.1


Display Boost and Favorite count

On my instance, the boost and favorites count isn't showing up in the home feed, even though I'm using the nightly version.
My instance (Buddyverse.xyz):

mementomori.social :

This is not part of the Mastodon Bird UI. How could CSS style change any functionality? I have changed that to the fork of my instance. A Mastodon issue related to this can be found here.


Update to latest SDK

Blocked on the next SDK update.
This also requires an explicit dependency on structopt, which I feel like shouldn't be passed onto the users.
closes #8

I agree, forcing the users to use structopt or an explicit pattern like CLI one here is not good, that's why we did not enforce this in the design at the beginning, and instead used envs to pass args to the gadget from the manager.
cc @tbraun96

Is this ready for merge?

I need to change the sdk version first


habridge and weaved?

Has anyone tried to use the habridge with weaved/remot3.it?  The weaved installer doesn't seem to see anything listening on port 80, but the habridge does work locally.  I did notice this from netstat -lptn.  I'm not up to speed on tcpip v6, but maybe there is some issue here?  Or I have something configured incorrectly on the RPI?
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      522/sshd
tcp6       0      0 :::80                   :::*                    LISTEN      6411/java
tcp6       0      0 :::22                   :::*                    LISTEN      522/sshd
sshd seems to be on both tcp and tcp6, but the habridge is only on tcp6.

That should not be the case as I force the ha-bridge to be IPV4 only and IPV6 needs to have a setting passed to enable it. I would look at the config for your PI as you can disable IPV6 from starting as it is on by default.

To me it looks like just the opposite - IPV6 is enabled but IPV4 is not in the ha-bridge.  The Raspberry Pi seems to be enabled for both as can be seen in the support for sshd on both.  I'm using the 4.1.4 pre-built version.  I'll look for anything interesting in the logs.  Any ideas?  Thanks.

https://www.raspberrypi.org/forums/viewtopic.php?f=36&t=138899

This is a non-issue, I think.  Despite weaved not seeing this, it works anyway.  I was confused because my work location doesn't let some traffic through the firewall, and this is one it blocked.  Once I tried remote access to the ha-bridge from a non-work location via weaved, it worked fine.  Moral of the story: only work on the ha-bridge at home...


Add support to ROS discovery plugin for ROS2 Python-only packages tha…

…t do not contain CMakeLists.txt. Second attempt that fixed some bugs from first attempt.

Codecov Report

Merging #297 (fd84286) into master (e3d728e) will decrease coverage by 3.31%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##            master     #297      +/-   ##
===========================================
- Coverage   100.00%   96.68%   -3.32%
===========================================
  Files           50       50
  Lines         2729     2744      +15
===========================================
- Hits          2729     2653      -76
- Misses           0       91      +91

Impacted Files
Coverage Δ

...ick_tool/plugins/discovery/ros_discovery_plugin.py
100.00% <100.00%> (ø)

...tatick_tool/plugins/tool/uncrustify_tool_plugin.py
65.21% <0.00%> (-34.79%)
:arrow_down:

statick_tool/plugins/tool/xmllint_tool_plugin.py
77.77% <0.00%> (-22.23%)
:arrow_down:

statick_tool/plugins/tool/cppcheck_tool_plugin.py
84.04% <0.00%> (-15.96%)
:arrow_down:

...tatick_tool/plugins/tool/perlcritic_tool_plugin.py
85.18% <0.00%> (-14.82%)
:arrow_down:

...tatick_tool/plugins/tool/clang_tidy_tool_plugin.py
85.91% <0.00%> (-14.09%)
:arrow_down:

statick_tool/plugins/tool/cccc_tool_plugin.py
91.08% <0.00%> (-8.92%)
:arrow_down:

statick_tool/statick.py
95.96% <0.00%> (-4.04%)
:arrow_down:

statick_tool/discovery_plugin.py
96.29% <0.00%> (-3.71%)
:arrow_down:

Continue to review full report at Codecov.

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


Feature: Move all endpoints into the static Endpoint class

Currently, each client has the endpoint typed as a string. While this works, I would like to get them centrally located in a single Endpoint.cs class so it's one place to go and chance it versus changing it in each place.

@wh1337 we prob can close this if moving to refit, since routes are passed through attributes there


Release: welcome rework

:tada: This PR is included in version 0.20.0 :tada:
The release is available on:

v0.20.0
GitHub release

Your semantic-release bot :package::rocket:


Quick pick focus color is incorrect

Since the changes in the list/tree widget focus work, the quick pick has a different focus color

Is this #117001?

Yeah: https://github.com/microsoft/vscode/issues/117001#issuecomment-782081399
Unfortunately I don't see a clear way forward to get this fixed without having themes adopting the change. We will make sure they get notified in the release notes.


Question on get_knn_log_prob

Hi, I am reading the code. In knnlm.py, there is a line (https://github.com/jxhe/efficient-knnlm/blob/main/fairseq/knnlm.py#L267):
index_mask = torch.eq(torch.from_numpy(self.vals[knns]).long().cuda().squeeze(-1), tgt[knn_mask].unsqueeze(-1)).float()

May I know what is the purpose of this line? Is the tgt means the prediction target tokens? If so, why is the target available during testing?
Thanks!

Closing this question. Found the answer after reading the code again.
This is to calculate what is the knn probability of the ground truth label, and it is only used for validation. The metric used is perplexity so this is enough for it.


Alt-tabbing on windows causes selected text to be deleted

Video:
https://github.com/SpartanJ/ecode/assets/48155462/b7291f88-3771-4d17-a1c2-10a8caab089e

Thanks. That's annoying, i'll have to test it on Windows, it does not happen in the other OSes, but Windows is by far the least tested OS since I don't personally use Windows.

Fix was introduced in the latest version due to some changes for proper IME support, there seems to be a bug in SDL2 but it is easy to solve. Fix is here. Will be available in next release.


Support 1.17.12, 1.18.9, 1.19.2

Support 1.17.12, 1.18.9, 1.19.2, which is targeted to release on 9/16

Verified that K8s versions v1.17.12, v18.9, v1.19,2 and v1.16.15 are supported in master-head(2b260f05fc348dd)
Automation runs succeeded
It was also verified that ingress-controller version is updated to rancher/nginx-ingress-controller:nginx-0.35.0-rancher1  for clusters with all supported K8s versions


merge package template from Kronos-Tools/npm-package-template

package.json

chore(engines): update<EMAIL_ADDRESS>from template

Coverage remained the same at 78.327% when pulling 01d8ebca82995fe989bf41277283c2e9271ee7f7 on template-sync-1 into 430b15891825d7bca877de4e5621353d1a2219ae on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   83.92%   83.92%
=======================================
  Files           1        1
  Lines         168      168
=======================================
  Hits          141      141
  Misses         27       27

Continue to review full report at Codecov.

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


Show left nav properties with errors as red

If user clicks 'Save' and properties have errors, they will display in red to indicate they have errors.

Is this intended to be "missing required properties" or "properties with errors"? (Most errors are missing required, but there are other errors as well.)

OK, updating to make slightly more accurate.

That sounds good. Thanks!


How to add a robot?

This is the post for saving the question and answer shown here
About the Jrl library, if I want to add new robots into it, for example, ur3, what is the correct step to do that? Here is my solution based on my understanding:

Implement the class Ur3() in robots.py
Download related urdf files from some projects like urdf_files_dataset or generate them using ROS based on your README.md
Generate the capsule folder using calculate_capsule_approximation.py

Are these steps enough? Should I use calculate_ignorable_link_collision_pairs.py and calculate_rotational_repeatability.py?

I notice that you modified the urdf file and remane it to "xx_formatted.urdf". Is there anything that I should be concerned?

The idea behind the xx_formatted.urdf is to keep the original urdf file (from a ros package, online source, etc), and make the Jrl specific edits to the xx_formatted.urdf version. This is so that it's clear what's been changed from the original.


A record instead of CName??

Is it possible to have the script create an A record instead of a CName record
For instance, instead of two records one would get created.  For example:
servername.domain.com   A  <IP_ADDRESS>
This is because I am routing traffic between two VPC across regions.  And AWS cannot resolve CNames across regions.

The script will always create an A record.  The domain name is inherited from the DHCP option set or you can use the zone tag.  The script is not written to allow vanity host names.  If you want a vanity host name, you need to create a CNAME record.  That said, it wouldn't be that hard to modify the script to use the value of a tag as the host name.

I am looking at how I would do that.  Any pointers?

I am afraid your code is too complicated for me to figure this out.  Any chance that this feature is on your roadmap?  I found someone else that has done a lambda script for updating an IP address for A records, but it does not create or delete them.  It seems that no one else is working on this.


[Refactor/State] Maybe add a raf on subscribe listener in state

dont think its nessecary just make sure the input from data sources is batched


use tcmalloc allocator

Motivation
switch to tcmalloc allocator.
Note that libtcmalloc must be installed on the target (build) machine. On ubuntu systems this means installing libgoogle-perftools-dev

I suggest we close this for now, as our tests have shown even though its a big improvement, it doesn't solve our core memory issue.

@vicsn I agree, even more so now we found the jemalloc options that give similar results.


issues running CAFEH : nonsensical results.

Hello,
I'm trying to apply CAFEH to detect multi tissue eQTL across ~15 cell states and I'm encountering several issues at the moment.
1/ First CAFEH  seems to multithread without warning the user.
which can be solved in my case by setting the following environment variables
```
export OMP_NUM_THREADS=1
export OPENBLAS_NUM_THREADS=1
export MKL_NUM_THREADS=1
export VECLIB_MAXIMUM_THREADS=1
export NUMEXPR_NUM_THREADS=1

2/ Second when running CAFEH on permuted data, i find many genes for which one or more components  have  a p_active>0.99 (and even pip>.99 in many instances), which seems to suggest that it's not specific.
3/  for genes where a strong eQTL can be detected unambiguously by standard mapping approaches (P<10^-30), CAFEH does NOT include the peak eQTL SNP in its 95% CI, despite detecting a single component in some of the tissues where the eQTL is active.
I suspect that I must be doing something wrong (or the code is malfunctional ?).
I'd be very happy to exchange on this in order to find what went wrong in my analyses.
toy data to reproduce these issues can be found here.

Hi @mrotival

I'm not sure why that might be happening, virtually everything is implemented with basic linear algebra operations in numpy. Is it the default behavior of numpy to multithread?
Having access the example you run would be useful for me to trouble shoot. Unfortunately, I can't seem to access the toy data you provided.
Are you using individual level data or summary stats here? This type of behavior might arise from using summary stats with a mismatched reference LD matrix.

Dear Karl.

not sure, but it might be, as using multiple cores is an easy way to
speed up computation. Here it was only a problem because it used the number
of cores available on the node, rather than what's available in the cluster
allocation. on a local computer this would not have been an issue.
In any case, this can be solved pretty easily by setting environment
variables adequately.
I'll try to send the data again ASAP.
everything I've done was with individual data.

Best;
Maxime
Le jeu. 6 janv. 2022 à 02:40, Karl Tayeb @.***> a
écrit :

Hi @mrotival https://github.com/mrotival

I'm not sure why that might be happening, virtually everything is
implemented with basic linear algebra operations in numpy. Is it the
default behavior of numpy to multithread?
Having access the example you run would be useful for me to trouble
shoot. Unfortunately, I can't seem to access the toy data you provided.
Are you using individual level data or summary stats here? This
type of behavior might arise from using summary stats with a mismatched
reference LD matrix.

—
Reply to this email directly, view it on GitHub
https://github.com/karltayeb/cafeh/issues/2#issuecomment-1006215492, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLHS7ZAVWWJ4XT2WS2MLE3UUTXHXANCNFSM5JZPYFRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

here is a wetransfer link with the unzipped toy data
https://we.tl/t-tJaWFjVjKd
Le ven. 7 janv. 2022 à 21:14, Rotival Maxime @.***> a
écrit :

Dear Karl.

not sure, but it might be, as using multiple cores is an easy way to
speed up computation. Here it was only a problem because it used the number
of cores available on the node, rather than what's available in the cluster
allocation. on a local computer this would not have been an issue.
In any case, this can be solved pretty easily by setting environment
variables adequately.
I'll try to send the data again ASAP.
everything I've done was with individual data.

Best;
Maxime
Le jeu. 6 janv. 2022 à 02:40, Karl Tayeb @.***> a
écrit :

Hi @mrotival https://github.com/mrotival

I'm not sure why that might be happening, virtually everything is
implemented with basic linear algebra operations in numpy. Is it the
default behavior of numpy to multithread?
Having access the example you run would be useful for me to
trouble shoot. Unfortunately, I can't seem to access the toy data you
provided.
Are you using individual level data or summary stats here? This
type of behavior might arise from using summary stats with a mismatched
reference LD matrix.

—
Reply to this email directly, view it on GitHub
https://github.com/karltayeb/cafeh/issues/2#issuecomment-1006215492,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLHS7ZAVWWJ4XT2WS2MLE3UUTXHXANCNFSM5JZPYFRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @mrotival
I was taking a look at your example and a few things come to mind.
First I would recommend normalizing your expression (mean 0, variance 1) before fitting CAFEH. The default initialization of the priors were chosen to make sense in this setting. The variance of expression seems to vary quite a bit across cell types so this may be causing some trouble. I'll update the documentation and provide a warning at runtime to encourage users to normalize their inputs, I'm considering making this the default behavior.
With that in mind, I permuted the sample labels of y 50 times, fit CAFEH, and recorded the number of components CAFEH detected (p_active > 0.95 in at least one study). With normalized inputs, CAFEH does not seem to perform as poorly on the simulations, but it's still not perfect.

As far the permutation analysis goes, and what could be driving the remaining false positives I think we need to think about
(1) the population structure in the data (as indicated by your covariates) and
(2) correlated measurement error among celltypes/studies should
I'm not totally sure what role (1) might play but I wouldn't be surprised if it leads to some weird stuff. I guess the right thing to do would be to permute X AND the covariates. (2) is a limitation of CAFEH, as it assumes measurement error in each study is independent


Unresolved reference: embedding

Steps to reproduce
in android studio only it's now showing on vs code
Create new project and it show error on
file path
->testapp/android/app/src/main/kotlin/com/example/testapp/MainActivity.kt
Expected results
show error
Unresolved reference: embedding
Unresolved reference: FlutterActivity
Actual results
no error my old code also show error after update
my version
Flutter 3.16.8 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 67457e669f (2 days ago) • 2024-01-16 16:22:29 -0800
Engine • revision 6e2ea58a5c
Tools • Dart 3.2.5 • DevTools 2.28.5
Code sample
Code sample
package com.example.testapp

import io.flutter.embedding.android.FlutterActivity

class MainActivity: FlutterActivity() {
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

If you open the android project in android studio while your flutter project is active (or open), this is what happens. The android project wouldn't have the necessary dependencies it needs to resolve the references which is why you get that error.
If you want to open the android project, there should be a button in your toolbar that states Open for editing in android studio. Click that and it will resolve the references.
This should not affect running your application. If it does, kindly file a new issue and provide adequate details to reproduce.
Closing this since you are trying to open to projects that are incompatible and this is an IDE issue and not a flutter issue.
Thank you


Response cookies character '+' replace by blank

Respoonse cookies don't need CGI::escape()
when cancel CGI::escape() int the function parse_cookie, then the cookie content character '+' replace by blank

Is this still relevant in rest-client 1.8.0, which revamps cookie parsing?

Feel free to reopen with more information.


Ability to rename attribute data-id.

Currently in our application we use an id attribute on every element (except Text node).
We would like to reuse this name in your extension.
Is there any possibility you'd be willing to make the name of the data-id^1 attributes configurable?

Yup, this seems pretty trivial to do. PR welcome, otherwise @mary-ext is out this week but I can entreat her to tackle it when she gets back!


grammar fix

arabic correction

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

language- arabic

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉