"pip install --target" produces silently broken installation with latest setuptools

When installing zope.deprecation and zope.interface via pip install --target, a strange warning occurs and the installation is left in an incomplete state.
This bug could be caused by PIP or by Setuptools. However, since the only difference between my working and my non-working system is the setuptools version, while the pip versions are equal, I decided to file the bug for setuptools. I tested only Python 2.7, not Python 3.
This is the smallest installation command that reproduces the bug:
rm -rf testdir/
pip install --target testdir/ zope.deprecation==4.2.0 zope.interface==4.4.2

Everything works fine on a system with latest pip 9.1.0 and older setuptools 33.1.1:
Collecting zope.deprecation==4.2.0
  Using cached zope.deprecation-4.2.0-py2.py3-none-any.whl
Collecting zope.interface==4.4.2
  Using cached zope.interface-4.4.2-cp27-cp27mu-manylinux1_x86_64.whl
Collecting setuptools (from zope.deprecation==4.2.0)
  Using cached setuptools-36.0.1-py2.py3-none-any.whl
Installing collected packages: setuptools, zope.deprecation, zope.interface
Successfully installed setuptools-36.0.1 zope.deprecation-4.2.0 zope.interface-4.4.2

However, on a system with latest pip 9.1.0 and latest setuptools 36.0.1, this ends with a strange warning:
Collecting zope.deprecation==4.2.0
  Using cached zope.deprecation-4.2.0-py2.py3-none-any.whl
Collecting zope.interface==4.4.2
  Using cached zope.interface-4.4.2-cp27-cp27mu-manylinux1_x86_64.whl
Collecting setuptools (from zope.deprecation==4.2.0)
  Using cached setuptools-36.0.1-py2.py3-none-any.whl
Installing collected packages: setuptools, zope.deprecation, zope.interface
Successfully installed setuptools-36.0.1 zope.deprecation-4.2.0 zope.interface-4.4.2
Target directory /.../testdir/zope already exists. Specify --upgrade to force replacement.

Despite the warning, pip exits with a success code (exit code 0) and shows "Successfully installed" for zope.deprecation as well as zope.interface. But zope.interface is not fully installed. There is just the dist-info but not the Python package itself, i.e. the directory interfaces/ in testdir/zope/ is missing:
$ ls -1 testdir/
easy_install.py
easy_install.pyc
pkg_resources
setuptools
setuptools-36.0.1.dist-info
zope
zope.deprecation-4.2.0.dist-info
zope.deprecation-4.2.0-py2.7-nspkg.pth
zope.interface-4.4.2.dist-info
zope.interface-4.4.2-py2.7-nspkg.pth
$ ls -1 testdir/zope/
deprecation

Moreover, when following the advice of the warning and adding --upgrade, the issue is still not solved:
rm -rf testdir/
pip install --upgrade --target testdir/ zope.deprecation==4.2.0 zope.interface==4.4.2

Now, the warning vanished and pip indicates success:
Collecting zope.deprecation==4.2.0
  Using cached zope.deprecation-4.2.0-py2.py3-none-any.whl
Collecting zope.interface==4.4.2
  Using cached zope.interface-4.4.2-cp27-cp27mu-manylinux1_x86_64.whl
Collecting setuptools (from zope.deprecation==4.2.0)
  Using cached setuptools-36.0.1-py2.py3-none-any.whl
Installing collected packages: setuptools, zope.deprecation, zope.interface
Successfully installed setuptools-36.0.1 zope.deprecation-4.2.0 zope.interface-4.4.2

However, the situation is just flipped: zope.interface is installed but zope.deprecation is not. In other words, zope.deprecation has been replaced with zope.interfaces:
$ ls -1 testdir/
easy_install.py
easy_install.pyc
pkg_resources
setuptools
setuptools-36.0.1.dist-info
zope
zope.deprecation-4.2.0.dist-info
zope.deprecation-4.2.0-py2.7-nspkg.pth
zope.interface-4.4.2.dist-info
zope.interface-4.4.2-py2.7-nspkg.pth
$ ls -1 testdir/zope/
interface

In total, I see the following issues here:

pip install --target produces an incomplete, broken installation

Success is indicated despite the broken installation
There is just a warning, not an error message
The warning message is misleading, as pip install --upgrade --target doesn't solve the issue (even though the warning message is technically almost correct)

pip install --upgrade --target produces an incomplete, broken installation

Success is indicated despite the broken installation
There is neither a warning nor an error message

I cannot reproduce this on OS X with either PyPy 5.7.1 or CPython 2.7.13 using pip 9.0.1 and setuptools 36.0.1. I tested both in and out of a virtual environment.
$ mktmpenv -p /opt/local/bin/python2.7
Running virtualenv with interpreter /opt/local/bin/python2.7
New python executable in //tmp-4581a0f14a1224d2/bin/python
Installing setuptools, pip, wheel...done.
This is a temporary environment. It will be deleted when you run 'deactivate'.
[tmp-4581a0f14a1224d2 (TMP)]$ python -c 'import setuptools; print(setuptools.__version__)'
36.0.1
[tmp-4581a0f14a1224d2 (TMP)]$ pip install --target TD zope.deprecation==4.2.0 zope.interface==4.4.2
Collecting zope.deprecation==4.2.0
  Using cached zope.deprecation-4.2.0-py2.py3-none-any.whl
Collecting zope.interface==4.4.2
  Using cached zope.interface-4.4.2-cp27-cp27m-macosx_10_12_x86_64.whl
Collecting setuptools (from zope.deprecation==4.2.0)
  Using cached setuptools-36.0.1-py2.py3-none-any.whl
Installing collected packages: setuptools, zope.deprecation, zope.interface
Successfully installed setuptools-36.0.1 zope.deprecation-4.2.0 zope.interface-4.4.2
[tmp-4581a0f14a1224d2 (TMP)]$ ls -l TD TD/zope
TD:
total 16
-rw-r--r--  1 jmadden staff  126 Jun 27 07:06 easy_install.py
-rw-r--r--  1 jmadden staff  367 Jun 27 07:06 easy_install.pyc
drwxr-xr-x  6 jmadden staff  204 Jun 27 07:06 pkg_resources
drwxr-xr-x 62 jmadden staff 2108 Jun 27 07:06 setuptools
drwxr-xr-x 12 jmadden staff  408 Jun 27 07:06 setuptools-36.0.1.dist-info
drwxr-xr-x  4 jmadden staff  136 Jun 27 07:06 zope
-rw-r--r--  1 jmadden staff  349 Jun 27 07:06 zope.deprecation-4.2.0-py2.7-nspkg.pth
drwxr-xr-x 10 jmadden staff  340 Jun 27 07:06 zope.deprecation-4.2.0.dist-info
-rw-r--r--  1 jmadden staff  529 Jun 27 07:06 zope.interface-4.4.2-py2.7-nspkg.pth
drwxr-xr-x 11 jmadden staff  374 Jun 27 07:06 zope.interface-4.4.2.dist-info

TD/zope:
total 0
drwxr-xr-x 10 jmadden staff  340 Jun 27 07:06 deprecation
drwxr-xr-x 32 jmadden staff 1088 Jun 27 07:06 interface

Of course, the resulting target directory can't actually be used to import zope:
[tmp-4581a0f14a1224d2 (TMP)]$ PYTHONPATH=TD python
Python 2.7.13 (default, May  5 2017, 09:16:47)
[GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.38)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import setuptools
>>> setuptools
<module 'setuptools' from '//tmp-4581a0f14a1224d2/TD/setuptools/__init__.pyc'>
>>> import zope
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: No module named zope
>>>

Some more details on my setup:

Everything works fine on Debian/stable and Debian/testing

Using the system-provided setuptools
Upgraded PIP to 9.0.1 by hand

The issue occurred on the following CentOS 7 setup:

Install a fresh CentOS 7 based on CentOS-7-x86_64-Minimal-1611.iso
No EPEL packages were used, just yum install python-devel from vanilla CentOS 7
Install PIP via https://bootstrap.pypa.io/get-pip.py

Side note: Regarding the missing __init.py__, this is a separate issue where I didn't want to go too much into distracting details. In this setup, the __init__.py needs to be created afterwards via touch zope/__init__.py. However, when zope.interfaces and zope.deprecation are installed system-wide, they do work out of the box, despite the missing zope/__init__.py. I haven't yet got around to raise an issue about that. Also, I don't really know if that is a CPython, PIP or setuptools issue.

Sounds like an issue similar to what I ran into. My issue was trying to install subpackages under a common package hierarchy that already exists at the time of installation. example package names: dog.german, dog.british. I added a temporary fix for my use case: using --root [location] instead of -t to install in a predetermined location. This creates the sites-package sub directories, but that location is static and I copy over the dir from final location for further use.


Add2and6 button aqara opple switch

Added support for the 2 and 6 Button version of the Aqara Opple Wirless Switch
I know this is THE worst approach for handling different devices with seperate classes but I am loaded with work and this is the quickest (but dirtiest) fix for supporting these devives, will try to change that when i got time (which will be hopefully in march lol). Sorry!
Regards,
Abel

Thanks @tr1ng0 , I can see if I can reduce the code by using some internal trick

Thanks @tr1ng0 , I can see if I can reduce the code by using some internal trick


Doppel solution

Required items, please complete
Change(s):

Created a Doppel folder inside Solutions, with subfolders for Data Connectors and Workbooks.
Added the following files:

Data Connectors:

Connector_Doppel.json
DeployToAzure.json

Workbooks:

Doppel.json
Preview images for the workbooks.

Logos:

doppel.svg

Sample Data:

Doppel_Logs.json for sample data ingestion.

Reason for Change(s):

Initial commit: To submit Doppel Data Connectors and Workbooks for review by the Microsoft Sentinel team as part of the integration process.

Version Updated:

No
It does not involve analytic rule or detection templates that would require a version number.

Testing Completed:

Yes
Tested locally using a Microsoft Sentinel environment.
Validated that the Data Connectors and Workbooks function as expected when imported into Sentinel, and that the Sample Logs are correctly processed.

Checked that the validations are passing and have addressed any issues that are present:

Yes, all validations have passed.
There were no issues with the workbooks or data connectors after testing.

Hello @anantm-metron,

Update the branch from master

Please create a input file which will located in Data folder.
This will file will consist basic details of the Solution and listed content in the Solution.

Add Solutionmetadata file which will consist metadata details of the Solution.
For input file and solution metadata please refer - https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Agari

Add workbook metadata to this Location - https://github.com/Azure/Azure-Sentinel/blob/master/Workbooks/WorkbooksMetadata.json

Workbook images added are B&W, Add colorful image. For each image added, add it with white and black background.

Can you specify what kind of Connector is this. Is this a custom connector or of kind CCP.
I'm confused about the azuredeploy file you added - if possible please specify

Hello @anantm-metron, I have added some permission in data connector
Please check once and needed please act

Hello @anantm-metron, Please create a custom table schema for mentioned table in workbook
for ref - https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables


PR: Rename Path by Current Working Directory label option (Find in Files)

Fixes #4693

@andfoy, please rebase this one onto 3.x, you did it against master.

@ccordoba12 Something went wrong with my local branch while I was performing the rebase. I will open this PR from another branch

Ok, no prob ;-)


[bazel] Add rules for sw/otbn/crypto

@jadephilipoom Since you're now our resident OTBN expert, I'm hoping you can take a look at this. Thanks!


instance.on('progress') not working

[x] Issue does not already exist
[ ] Issue observed on https://plyr.io

I need to get the progress of an audio-instance for a custom radial progress-bar.
Expected behaviour
The Event playerInstance.on('progress', (event) => { console.log(event) }) should be fired every XX seconds while playing a media-file.
Actual behaviour
Event is fired only 2 times.
Environment

Browser: Chrome
Version: 65
Operating System: macOS
Version: 10.13.3

Players affected:

[ ] HTML5 Video
[x] HTML5 Audio
[ ] YouTube
[ ] Vimeo

Steps to reproduce

See codepen

Relevant links
https://codepen.io/CinKon/pen/mxEQPJ

Also I'd prefer a getCurrentProgress method :)

You probably meant to use timeupdate https://developer.mozilla.org/en-US/docs/Web/Events/timeupdate
Also, adding event listeners with a generic event listener method such as on is pretty much the gold standard of event listening, and a lot more efficient than your suggestion.

Yeah, that's how i do it now. But since plyr offered this event in the docs, i thought it might be a good idea to use this in the first place ;)

Events like progress are merely re-dispatched and set to bubble (media events don't by default) so if it fires twice, that's because it's only fired twice on the underlying media element. Are you on a fast connection or have the file cached? The spec says:

While the load is not suspended (see below), every 350ms (±200ms) or for every byte received, whichever is least frequent, queue a task to fire a simple event named progress at the element.

Anyway, you should try v3 as it's way better than v2. I'm about to merge to master but you can grab it from the beta branch or npm.
I can add a progress getter on the instance which I guess would return between 0 and 1 (for percentage).

OK, should've read the description more accurate... My bad

It happens to the best of us!

i read this wrong the first time, too.

progress...  Sent periodically to inform...

was enough to confuse me.

was enough to confuse me.

It's literally "borrowed" from:
https://developer.mozilla.org/en-US/docs/Web/Guide/Events/Media_events


setOptions not available

Hello Rik !
It seems that we can't setOptions use this way :
import FilePond, { registerPlugin, setOptions } from 'svelte-filepond';
import es_ES from 'filepond/locale/es-es.js';
// ...

setOptions(es_ES)

It seems to be a missing feature from the original Filepond library. Should it be done this way ?

It's indeed not exported, with the framework you should use the component to interact with FilePond. I'd just spread the props on the component instead.


VM: Implement, ABS, NOT opcode

Closes #222

Codecov Report

Merging #233 into vm will decrease coverage by 0.22%.
The diff coverage is 32.14%.

@@            Coverage Diff             @@
##               vm     #233      +/-   ##
==========================================
- Coverage   52.84%   52.62%   -0.23%
==========================================
  Files          76       76
  Lines        2581     2609      +28
==========================================
+ Hits         1364     1373       +9
- Misses       1098     1114      +16
- Partials      119      122       +3

Impacted Files
Coverage Δ

pkg/vm/vm_ops.go
100% <ø> (ø)
:arrow_up:

pkg/vm/stack/boolean.go
0% <0%> (ø)
:arrow_up:

pkg/vm/stack/stack.go
49.25% <0%> (-3.98%)
:arrow_down:

pkg/vm/stack/Int.go
74% <0%> (-10.1%)
:arrow_down:

pkg/vm/vm_ops_maths.go
53.42% <60%> (+1.7%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

100 points dauTT
50 points decentralisedkev for review

https://neoscan.io/transaction/81db58e8e3fcae21d4d614c008935de2fcb2d53bdf05ce5b4a40e13c3a110314


Add dark mode support

Enhance the user experience in low light environments by implementing a dark mode in the application.

By default, the dark mode will automatically follow the current Windows light scheme.

This is only possible when running Windows 10 or above.

Requires #211.


MAINT: remove accidental print statement from loggamma code.

Came in in gh-6781.

Note that this is also fixed in https://github.com/scipy/scipy/pull/6804.

okay thanks, let's get that in then.


PLOT/HLIM gets axis labelling wrong.

Reported by Ansley B Manke on 19 Mar 2007 19:31 UTC
Reported by Alexandre Ganachaud and Billy Kessler
yes? DEFINE AXIS/Y/units=deg
yax={-21.98,-21.54,-20.97,-20.60,-20.10,-19.40,-18.68,-17.96,-17.11,-16.36,-15.93}
yes? LET yvar = sin(y[set view upper; plot/hlim=-22.5:-15/line/symbol yvar
yes? set view lower; plot/hlim=-22.5:-15.5/line/symbol yvar
The lower plot should start at -22.5, with the first point at -21.98. Instead it
has the axis labelled starting at -23. and the first point near -22.5
Note this does not have to do with the irregular spacing or the fact that this
example axis has negative latitudes.
yes? DEFINE AXIS/Y=16:22:0.5/units=deg yreg
yes? let y2 = sin(ygy=yreg
yes?))
yes? set view upper; plot/hlim=15:22.5/line/symbol y2
yes? set view lower; plot/hlim=15.5:22.5/line/symbol y2
This time the second axis should start at 15.5, with the first data point at 16,
but instead the axis labelling starts at 16 which puts the first point at 16.5
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1498

Comment by ansley.b.manke on 9 Dec 2011 22:05 UTC
I no longer see this behavior, though I can verify it with older executables. Closing the ticket.

Attachment from<EMAIL_ADDRESS>on 19 Mar 2007 19:32 UTC
the first plot
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1498_GIT_772/b1.gif

Attachment from<EMAIL_ADDRESS>on 19 Mar 2007 19:32 UTC
the second plot, northern hemisphere
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1498_GIT_772/b2.gif


S3 output set 'endpoint' to allow use with non-amazon servers

The S3 input has the SQS functionality that does not really translate to other services, but the S3 output works great with google gcs (w/ interoperability api, aka S3) and minio.
it would be extremely easy to add to this minimally, eg:
@@ -47,6 +47,7 @@ type AmazonAWSCredentialsConfig struct {

 // AmazonS3Config contains configuration fields for the AmazonS3 output type.
 type AmazonS3Config struct {
+       Endpoint    string                     `json:"endpoint" yaml:"endpoint"`
        Region      string                     `json:"region" yaml:"region"`
        Bucket      string                     `json:"bucket" yaml:"bucket"`
        Path        string                     `json:"path" yaml:"path"`
@@ -113,6 +114,9 @@ func (a *AmazonS3) Connect() error {
        if len(a.conf.Region) > 0 {
                awsConf = awsConf.WithRegion(a.conf.Region)
        }
+       if len(a.conf.Endpoint) > 0 {
+               awsConf = awsConf.WithEndpoint(a.conf.Endpoint)
+       }
        if len(a.conf.Credentials.ID) > 0 {
                awsConf = awsConf.WithCredentials(credentials.NewStaticCredentials(
                        a.conf.Credentials.ID,

The unfortunate part is the code says Amazon alot, but the above is all that is necessary.

Hey @iluminae, sounds good. I'm happy to tackle this next time I revisit AWS configs. I'm thinking of reducing the code duplication across aws components in Benthos, I'll make sure to add an endpoint field when I do it.

Implemented: https://github.com/Jeffail/benthos/commit/e56253b5684d5b0eddb1d3d4094c157bd19d506c
Released: https://github.com/Jeffail/benthos/releases/tag/v0.34.3


Standardize quotation marks in memory of hereNT

hereNT's final blog post was about the lack of standardization in single and double quotation marks in the obligatory line of code atop of every PHP file to prevent requesting the file directly. Of course, hereNT was not complaining about the lack of standardization, but simply noting with bemusement that such a quirk existed because no one bothered to standardize it because it worked anyway.
In a tribute to fix them as lazily as possible, I did the following to change all single quotes found on the C5_EXECUTE line to single quotes:
find . -iname '*.php' -print0 | xargs -0 perl -pi -e '/C5_EXECUTE/ && s/\"/'"'"'/g'

A summary of the changes (860 of them) can be found by piping git diff through Awk:
awk '/^-/{minus = $0; getline plus; print minus " => " plus; }' \
| sort                                                          \
| uniq -c                                                       \
| sort -nr

A little manual clean up to make the results more readable:
| count | change                                                       |
|-------+--------------------------------------------------------------|
|   506 | -defined('C5_EXECUTE') or die("Access Denied.");             |
|       | +defined('C5_EXECUTE') or die('Access Denied.');             |
|-------+--------------------------------------------------------------|
|   204 | -<?php defined('C5_EXECUTE') or die("Access Denied."); ?>    |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.'); ?>    |
|-------+--------------------------------------------------------------|
|    85 | -<?php defined('C5_EXECUTE') or die("Access Denied.");       |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.');       |
|-------+--------------------------------------------------------------|
|    23 | -<?php defined('C5_EXECUTE') or die("Access Denied.");?>     |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.');?>     |
|-------+--------------------------------------------------------------|
|    17 | -    defined('C5_EXECUTE') or die("Access Denied.");         |
|       | +    defined('C5_EXECUTE') or die('Access Denied.');         |
|-------+--------------------------------------------------------------|
|     3 | -defined('C5_EXECUTE') or die(_("Access Denied."));          |
|       | +defined('C5_EXECUTE') or die(_('Access Denied.'));          |
|-------+--------------------------------------------------------------|
|     3 | -defined('C5_EXECUTE') or die(_("Access Denied."));          |
|       | +defined('C5_EXECUTE') or die(_('Access Denied.'));          |
|-------+--------------------------------------------------------------|
|     3 | -<?php  defined('C5_EXECUTE') or die("Access Denied."); ?>   |
|       | +<?php  defined('C5_EXECUTE') or die('Access Denied.'); ?>   |
|-------+--------------------------------------------------------------|
|     3 | -<?php  defined('C5_EXECUTE') or die("Access Denied.");      |
|       | +<?php  defined('C5_EXECUTE') or die('Access Denied.');      |
|-------+--------------------------------------------------------------|
|     2 | -defined('C5_EXECUTE') or die("Access Denied."); ?>          |
|       | +defined('C5_EXECUTE') or die('Access Denied.'); ?>          |
|-------+--------------------------------------------------------------|
|     2 | -defined("C5_EXECUTE") or die("Access Denied.");             |
|       | +defined('C5_EXECUTE') or die('Access Denied.');             |
|-------+--------------------------------------------------------------|
|     2 | -<?php defined('C5_EXECUTE') or die(_("Access Denied.")); ?> |
|       | +<?php defined('C5_EXECUTE') or die(_('Access Denied.')); ?> |
|-------+--------------------------------------------------------------|
|     2 | -<?php defined('C5_EXECUTE') or die("Access Denied."); ?>    |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.'); ?>    |
|-------+--------------------------------------------------------------|
|     1 | -defined('C5_EXECUTE') or die("Access Denied.");             |
|       | +defined('C5_EXECUTE') or die('Access Denied.');             |
|-------+--------------------------------------------------------------|
|     1 | -<?php defined('C5_EXECUTE') or die("Access Denied."); ?>    |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.'); ?>    |
|-------+--------------------------------------------------------------|
|     1 | -<?php defined('C5_EXECUTE') or die("Access Denied."); ?>    |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.'); ?>    |
|-------+--------------------------------------------------------------|
|     1 | -<?php defined('C5_EXECUTE') or die("Access Denied.");  ?>   |
|       | +<?php defined('C5_EXECUTE') or die('Access Denied.');  ?>   |
|-------+--------------------------------------------------------------|
|     1 | - defined('C5_EXECUTE') or die("Access Denied.");            |
|       | + defined('C5_EXECUTE') or die('Access Denied.');            |
|-------+--------------------------------------------------------------|

Normalization of single/double quotes is included in #7355 (rebased as #7765): see https://github.com/concrete5/concrete5/pull/7765/files#diff-1dfe8d980e61c43af20946618d6db6dcR399 (somebody takes care of useless stuff, even if I'm Italian I'm not that lazy :wink:).
Furthermore, 6 years ago I removed the useless _() calls, but after all this time we still have them in the core (maybe because a lot of developers is still using some starting point template that uses them).
BTW, I love the many reasons why this #7780 has been created, so it's obviously a :+1: for me.

Love this. Can it be expanded to also remove the long dead _() function.

Love this. Can it be expanded to also remove the long dead _() function.

See https://github.com/lrcg/concrete5/pull/1

Thanks @JohntheFish and @mlocati

Can be a little dangerous if you don't check first and verify after, but I use something like this:
find . -iname '*.php' -print0 | xargs -0 perl -pi -e '/C5_EXECUTE/ && s/(_(([^)]+)))/$1/g'

Bump. Just because it hasn't been resolved doesn't mean it should be automatically stale.

Just because it hasn't been resolved doesn't mean it should be automatically stale.

Line any other issue

This has to go against release/9.0.0, not develop. This is never going to merge cleanly from develop into release/9.0.0.
I'm genuinely wondering how we might review each and every line of such a pull request. I guess if we decided to dedicate a day to it? 🤷♂

Bump

Bump again

Bump again

Let's add Status:Proposal label to this issue. Issues with Status:Proposal label never considered as Stale.

Let's add Status:Proposal label to this issue. Issues with Status:Proposal label never considered as Stale.

Bump

Is this now fully resolved in both 8.5.7 and v9 ?

@JohntheFish from recent digs through 8 & 9, it is not fixed yet...


Authentication causing undesirable behaviour when there are multiple replicas

This issue is a: (mark with an x)

[X] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
When I am using authentication with multiple replicas, the container app is only successfully authenticating against one of the replicas, which is causing undesirable behaviour in my application
Steps to reproduce

Have a container app with two replicas
Have ingress enabled
Have a custom domain configured for the container app
Have authentication set up with a redirect uri for the custom domain

Expected behavior [What you expected to happen.]
I can login to the application successfully and no errors due to authentication occur
Actual behavior [What actually happened.]
I can intermittently login, but received a lot of errors when doing so, the errors I receive are:
'react_devtools_backend_compact.js:2367 TypeError: Failed to fetch dynamically imported module:'
'React Router caught the following error during render TypeError: Failed to fetch dynamically imported module:'
'Access to manifest at ... has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.'
'login.windows.net/.../oauth2/v2.0/authorize? ... net::ERR_FAILED 302 (Found)'
Screenshots

Additional context
This issue is resolved when I enable 'Session Affinity' and I can successfully login to my application with no issues

/cc @njucz
I think this needs either token store or session affinitly.

@Jade-Codes can you please provide more details on how you configured auth for your app? It’s unclear if you are using the built-in authentication feature in Container Apps or something else. Thanks.

@anthonychu yes I am using in built container apps authentication with a redirect.
I am using the built in authentication as documented here:
https://learn.microsoft.com/en-us/azure/container-apps/authentication
With Azure AD

Hi @Jade-Codes may I confirm that your app is trying to get the info of authenticated user? If that's the case, you need to set token store or session affinity. If your app is just showing info which is unrelated with login user, I think there should be no such issue.

Hi @njuCZ no my app doesn't know anything about the authentication nor did I want it to for this iteration, I am not using anything in the code to reference the authentication. Do I need to?

@njuCZ I have yes, I am using a react application and using the deployment image from here:
https://github.com/IQEngine/IQEngine

@Jade-Codes I changed to use your image, it still works without enabling session affinity

Can you directly ping me your container app resource id so I can take a check?

@njuCZ what's your name sorry? I can ping you via Teams

zhcheng

After investigating with @njuCZ I can determine the issue was due to not having a configured secret for the application which resolves my issue. I am closing this ticket.


boot2docker.iso build in windows7 with  virtualbox boot2docker failed on step 45. COPY .git /git/.git.

I tried three times and always failed on "step 45 COPY .git  /git/.git."
the VM system said:“lstat .git:no such file or directory"
I have intalled the docker in windows7.
I want to build my customised boot2docker, so I download and unpack the github boot2docker.zip to sharefolder
between windows7 and virtualbox boot2docker.
I login in boot2docker, go to the sharefolder directory and run
"docker build -t boot2docker  . && docker run --rm boot2docker > boot2docker.iso"
And it fail as  above description.
Anyone know how to solve the problem?

ZIP does not contain .git folder.
You have to use git clone instead of downloading ZIP.


GitHub Actions to Run Solidity Linter

Use solhint and prettier to lint all solidity files and add a GitHub Actions check.
The .solhint.json config file sets the max line length is set to 99 in accordance with the Solidity PEP8 standard.

GitHub Actions to run Solidity lints


add liquality

Please remove the .env file too

Yes, but I think it's better to revert all the commits, implement the required changes, like adding the .env to the .gitignore file, commit them and push the changes with force (-f). Because right now the .env file is in the commit history and we better remove it.


Update scalafmt-core to 3.3.2

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

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

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #90.


make deferred approvals text clear

@shashban : Thanks for your contribution! The author, @alexhomer1, has been notified to review your proposed change.


Run IP check only for changed files.

- Do not require `check_ip_addresses.pl` to pass.
- Make `check_ip_addreses.pl` run only for changed files.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.102%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1729

Relevant Lines:
3256

💛  - Coveralls


Improved 2D platformer template

This upgraded platformer_stock_B.py dedicates an entire level editor to easily build your own levels, an asset system (both static and animated art), and improved collision.
I've also provided some free assets to help get you started in learning how the editor works.
Details and guides on how to use the editor and main base template will be provided soon!

Changed old folder/file paths to be "psC" instead of "psB"


TypeError [ERR_UNKNOWN_ENCODING]: Unknown encoding: utf8mb3

TypeError [ERR_UNKNOWN_ENCODING]: Unknown encoding: utf8mb3
at stringSlice (buffer.js:630:9)
at Uint8Array.toString (buffer.js:666:10)
at Parser.parseString (/var/www/api/kameazy/realtime/node_modules/mysql/lib/protocol/Parser.js:281:28)
at Object.exports.readMysqlValue (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/common.js:446:25)
at readRow (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/rows_event.js:111:16)
at UpdateRows._fetchOneRow (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/rows_event.js:143:13)
at UpdateRows.RowsEvent (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/rows_event.js:62:27)
at new UpdateRows (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/rows_event.js:135:13)
at BinlogHeader.parse (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/lib/packet/binlog_header.js:47:23)
at Protocol._parsePacket (/var/www/api/kameazy/realtime/node_modules/mysql/lib/protocol/Protocol.js:272:12)
{ Error: Packets out of order. Got: 178 Expected: 7
at Parser._tryReadPacketHeader (/var/www/api/kameazy/realtime/node_modules/mysql/lib/protocol/Parser.js:470:15)
at Parser.write (/var/www/api/kameazy/realtime/node_modules/mysql/lib/protocol/Parser.js:33:29)
at process._tickCallback (internal/process/next_tick.js:61:11)
--------------------
at Protocol._enqueue (/var/www/api/kameazy/realtime/node_modules/mysql/lib/protocol/Protocol.js:144:48)
at Immediate._start (/var/www/api/kameazy/realtime/node_modules/@rodrigogs/zongji/index.js:240:31)
at runCallback (timers.js:705:18)
at tryOnImmediate (timers.js:676:5)
at processImmediate (timers.js:658:5) code: 'PROTOCOL_PACKETS_OUT_OF_ORDER', fatal: true }

I am getting the same problem after we updated our MySQL server from version 7 to version 8.0.29. Both packets out of order error and the unknown encoding: utf8mb3. Not sure what direction to take with solving this.

@Athu135 Hello i found a small go around for this problem. Requires edit to the @rodrigogs/zongii/lib/common.js file.
Im creating a pull request now for the fix.

@sfabara Hello listening to this gives a relieve. Could please let me know on how to do it. It would be of great help.

@Athu135 https://github.com/sfabara/zongji I forked the zongji repo and added the fix - if you want to see for yourself it was just a change a change on lines:

443
466
of @sfabara/zongii/lib/common.js file.

@sfabara It work for me, thanks !

@Athu135 https://github.com/sfabara/zongji I forked the zongji repo and added the fix - if you want to see for yourself it was just a change a change on lines:

443
466
of @sfabara/zongii/lib/common.js file.

Excellent, thank you! I was wondering if I'd have to change the collation in every database & every table on my server just to watch the MySQL binlog... with this fix, I do not!


Add 'public' field to ActionAdmin list display

Coverage remained the same at 92.31% when pulling 2f3139b2dfa2662daa7e57b221836ff2923c5fc9 on ryangallen:patch-1 into a267e6aee4a49d6f26fc60c9f2f8e97d719376ed on justquick:master.


net/http: TestLinuxSendfile hanging on linux-arm64 releaselet

TestLinuxSendfile hangs reliably on the linux-arm64 releaselet. When I run master on the releaselet the test gets skipped because it detects strace is not working. Will move the skip logic to Go 1.8 branch.

CL 38633 OK for Go 1.8.5.

go1.8.5 has been packaged and includes:

CL 38633

The release is posted at golang.org/dl.
— golang.org/x/build/cmd/releasebot, Oct 26 21:07:54 UTC


fleet:functional-test: make Fleetctl() start print stdout and stderr on failures

If functional tests fail in Fleetctl() start for some units, then just print directly as much information as possible. This will save some time when errors are just obvious ones.

LGTM


On building and installing with CC_SYSTEM_BUILD=yes, cc-runtime still looks for images under /usr/local/share instead of /usr/share

build output shows assets path as /usr/share/defaults, but the binary looks for /usr/local/share instead.

cc-runtime version ?
Are you building in the normal way?:$ make build-cc-system && sudo -E PATH=$PATH make install-cc-system

Can you paste the output from make plus what do you see in config-generated.go?

@amshinde  - can you still reproduce this?  If not, can we close?

I am unable to recreate so I'm closing this for now. Feel free to re-open if it can be recreated (and please attach the info requested above in that case).

@jodh-intel I recently tried this on a new machine with the same results.
$ make build-cc-system && sudo -E PATH=$PATH make install-cc-system
cc-runtime - version 3.0.0-beta.2 (commit b37a883b8df69c79a253c52e7e6ae9be594823fe)

• Summary:

        Clear Containers system build     : yes

        binary install path (DESTTARGET)  : /usr/local/bin/cc-runtime
        config install path (DESTCONFIG)  : /etc/clear-containers/configuration.toml
        hypervisor path (QEMUPATH)        : /usr/bin/qemu-lite-system-x86_64
        assets path (PKGDATADIR)          : /usr/share/clear-containers
        proxy+shim path (PKGLIBEXECDIR)   : /usr/libexec/clear-containers
        pause bundle path (PAUSEROOTPATH) : /var/lib/clear-containers/runtime/bundles/pause_bundle

     BUILD   pause
cc-runtime - version 3.0.0-beta.2 (commit b37a883b8df69c79a253c52e7e6ae9be594823fe)

• Summary:

        Clear Containers system build     : yes
 binary install path (DESTTARGET)  : /usr/local/bin/cc-runtime
        config install path (DESTCONFIG)  : /etc/clear-containers/configuration.toml
        hypervisor path (QEMUPATH)        : /usr/bin/qemu-lite-system-x86_64
        assets path (PKGDATADIR)          : /usr/share/clear-containers
        proxy+shim path (PKGLIBEXECDIR)   : /usr/libexec/clear-containers
        pause bundle path (PAUSEROOTPATH) : /var/lib/clear-containers/runtime/bundles/pause_bundle

     BUILD   pause
     INSTALL install
     INSTALL install

I then tried running a docker container with the installed runtime:
$ sudo docker run -it debian bash
docker: Error response from daemon: oci runtime error: Config file /usr/local/etc/clear-containers/configuration.toml does not exist.

Runtime version:
$ cc-runtime version
cc-runtime  : 3.0.0-beta.2
   commit   : b37a883b8df69c79a253c52e7e6ae9be594823fe
   OCI specs: 1.0.0-rc5

The toml file has been correctly installed under /etc/clear-containers but the runtime still looks for it under /usr/local/etc/clear-containers.

Looks like the issue was caused due to an an earlier run of make. It is necessary to do a make clean for the system build and install to work correctly. The makefile should regenerate the binaries for system builds.


Server should reject connections from extensions that are too old

The server should reject connections from clients that are not at least the current minor version number.  Breaking protocol version should come with an increase in minor version number.

Did this on the client side.  Currently requires an exact match on the version number.  We can relax it to just a major.minor match when things stabilize.  I don't want to burn through minor version numbers yet.


Portable Teleporter crash opening the GUI after teleporting

The portable teleporter crashes when opening the GUI after teleporting:
http://pastebin.com/Pd4kcdTy
The log is from a development environment using commit 63f751c7103faa784ac42f8b100fb449804beda1 but also happens while playing a normal game. Note that while I have Industrialcraft 2 installed, this crash is without the quantum suit or any other armor or even inventory item. The first time the GUI loads fine and displays the destinations. After teleporting, if you open the GUI, it displays briefly but the list is empty and the game crashes.

I can't reproduce this on my end.  Is this a one-time occurrence or you or does it happen consistently?

So far it just happend once. I've removed the logistical transporters with MCEdit and replaced them.
Server starts just fine now. Is this some strange multipart issue?

This happens very consistently for me. I'll try and upload a video of what's going on...

Don't worry about it, just found and fixed the issue :)

Yup, that fixed it :D


No Refresh when creating new File

Hi Guys!
If I create a new file in my workingdirectory ist doesn't show up. I have to refresh the list and then there is my new file. In a previous version the refresh was automatically there when I create a new file. The same thing applies to folders.
My version of VSCode is 0.10.5

@FinnStutzenstein which OS? And where do you create the file, inside or outside VSCode?

My OS is Windows 8.1 and I create the file with VSCode. I forgot to mention that my workingdirectory is a samba network share mounted as a network drive. When I create a file on my harddrive, it shows up directy in Code.

Ok, it is likely that file watching fails for network shares.

/duplicate https://github.com/Microsoft/vscode/issues/59146


Provide documentation on how to update pipelines for a new or existing collection

When creating a new collection, need documentation on how pipelines can be updated to add/update/delete tasks.

@mtamboli could you add this requirement to issue 56 and close this one?

@mtamboli - is going to check and get back to us.

@mtamboli is the doc required for this issue addressed in the draft for https://github.com/kabanero-io/docs/issues/56

Discussed with Vijai and others on this. Closing this one and we will use issue 56.


how to build mpifileutils as rpm now

Hi guys,
Glad to see v0.9 release is out now, I found that all the build scripts are removed since cmake&spack ci introduced, I can easy to build mpifileutils with predefined scripts as RPM before, but seems this feature is gone, so how to build rpm now?

+1 for an RPM build process! I had a heck of a time building this on a RHEL7 box, and I'd rather not have to install Spack and compile gcc again when I already have yum.


DDB structured parsing

Improve DDB expressions support.
Currently DDB expressions are mostly handled in an adhoc manner. This pull request is the start of going towards a structured approach. In order to action expressions the following steps will be taken:

Expression gets parsed into a tokenlist (tokenized)
Tokenlist get transformed to expression tree (AST) -> This commit
The AST gets validated (full semantic correctness)
AST gets processed to perform the update

This pull request provides 1 & 2 for the update expression. Code for each point resides in a commit of its own to ease review.  Publishing this PR as review of this part can already be done and at this stage it will already provide benefit as it will be able to throw Validation Exceptions that now silently pass and will be raised when running against AWS DynamoDB.
This approach should both work for the UpdateExpression and the ConditionExpression.  The ConditionExpression is already more structured and closer to this approach but can be converted further.
Going for the structured approach should allow easier bug fixing and adding of enhancements since each code part has clearly identified responsibilities. In particular I started this work to tackle https://github.com/spulec/moto/issues/2806 .

Coverage decreased (-0.2%) to 94.132% when pulling 6809f50291cd036e33331d0c9a45f5834829c7ae on pvbouwel:ddb_full_parsing into 48288d8bb12a719c042b65fe445cd3f161e2112b on spulec:master.

I will add user facing tests.  It seems like my second commit broke Py27 compatibility. I wanted to check what precisely broke but when I go to job log in travis CI I always seem to get There was an error while trying to fetch the log..  Not sure if that is the right way to retrieve why that build failed. Is there a way for me to retrieve the errors of that build?

No, that should be the right way! Not sure why it's not showing anything.
You might have to run it locally.

The extra commits add the validation of the AST.  It caused some tests to fail. Checked against AWS DynamoDB and the test were invalid.  I've changed them to pass against AWS that is in the separate commit e6b51a2. I had to add a small commit as well that makes sure setup.py packages reserved_keywords.txt in order to make sure this file is available as a data file.
Only step remaining to switch fully to the structured parsing is implementing the executor which will take the validated AST and executes all the actions of the UpdateExpression. When enabling that it will be possible to perform code cleanup.

Thanks @pvbouwel. Is this ready to merge now?

This by itself should be good to merge. It will already improve error messaging a lot imho. I think it's still a bit too forgiving for some updates but that is something that I will test and tackle when looking at using the structured parsing for actually performing the updates (ETA next weekend). But merging should not be blocked by the implementation of the executor.

After merging this PR, I now get the following error when installing moto:
ERROR: Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/usr/local/reserved_keywords.txt'
This is a misconfiguration of setuptools as I highlighted in https://github.com/spulec/moto/pull/2885#discussion_r415150276. The data_files content needs to be updated to install the file in the correct package-specific location. @pvbouwel can you please take a look?


[Star Wars Saga Edition] Bugfix (Initiative) and Update (data attributes for translation)

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Added a lot of data attributes for translation, especially on the NPC sheet.
Removed extraneous colons
Fixed the Grapple header on the NPC tab
Fixed the Initiative as Decimal erroneous calculation and updated the translation json file
Changed the default Grapple mod on the PC sheet to Dex, since that is most likely to be higher than Str.
Changed the update date.

Character Sheet Info Roll20 Internal Use only.


Devcontainer not compiling

Describe the bug
Devcontainer is not compiling via Devpod
To Reproduce

Go to Devpod
Create workspace -> Point to project root directory and .devcontainer/devcontainer.json file
Set environment to Docker & Open With VSCode
Open with VSCode

Expected behavior
Expected to open with VSCode
Screenshots

Desktop (please complete the following information):

OS: MacOS

Additional context
Possibly related issue...
https://github.com/devcontainers/features/issues/831

@msolorio Here are my intermittent updates with devcontainers from today. Feel free to look through, else I'll get back to it tomorrow

Following the steps to reproduce (set file path to local repo), I was able to have the DevContainer compile and open vscode.

@msolorio What laptop are you using?

Using OS: macOS Sonoma 14.1.1

Interesting, I'm on Sonoma 14.6

I'm on the intel chip

Ah, I'm on the Apple Silicon

https://github.com/docker/for-mac/issues/7006#issuecomment-2124954348

@msolorio Solved by upgrading to Docker Desktop 4.33. Now using 4.34 and it still works


Support passing certificate at runtime

Currently, our image allows passing in a certificate at --initialize,
and will reuse that cert when it starts later on. If no cert was
provided, on will be auto-generated when the database starts.
This works well as long as the certificate doesn't change, but we'll
soon be replacing the certificate for *.aptible.in, so our image
should ideally start accepting a renewed certificate on restart.

cc @fancyremarker

👍


Sora CPP SDK の 2024.8.0-canary.26 の変更に対応

This pull request primarily focuses on updating the codebase to replace absl::optional and absl::nullopt with std::optional and std::nullopt respectively. These changes span across multiple files and functions to ensure consistency and compatibility with the updated Sora C++ SDK.
Key changes include:
SDK and Dependency Updates:

Updated Sora C++ SDK to version 2024.8.0, which includes changes from absl::optional to std::optional and from absl::nullopt to std::nullopt.

Decoder Changes:

Replaced absl::optional<int> with std::optional<int> in DynamicH264Decoder::Decode.
Updated the call to callback_->Decoded to use std::nullopt instead of absl::nullopt.

Encoder Changes:

Changed NumberOfThreads function to use std::optional<int> instead of absl::optional<int>.
Updated ScalabilityModeFromTemporalLayers to return std::optional<ScalabilityMode> and use std::nullopt. [1] [2]
Modified DynamicH264Encoder class to use std::optional for scalability_modes_ and encoder_thread_limit_ members.

Audio Handling Changes:

Updated SoraAudioSinkImpl::OnData and SoraAudioSourceInterface::OnData to use std::optional<int64_t> instead of absl::optional<int64_t>. [1] [2]
Changed SoraAudioSourceInterface::Add10MsData and SoraAudioSource::OnData to use std::optional<int64_t> and std::nullopt. [1] [2] [3]
Modified SoraAudioStreamSinkImpl::OnData to use std::optional<int64_t> instead of absl::optional<int64_t>.

These updates ensure that the codebase is aligned with the latest standards and improves overall maintainability and compatibility.

確認ありがとうございます。こちらマージします。


Fix session chat bug

Description
The main purpose of this pull request is to fix the session chat bug where the keyboard hides the input.
Test Plan
N/A
Motivation and Context
We want the current user to be able to see what they're typing when they're typing on the keyboard.
How Has This Been Tested?
I've manually tested the performance of the app by going through the containers in question and seeing how the new implementation is performing.
Types of Changes

Documentation
[x] Bug fix
New feature
Breaking change

Bug Fixes

#139 (492ba58 by @therealaldo)

Checklist

[x] My code follows the code style of this project

 Session chat


Typing the winget command tries to execute winget.exe in another user folder

Brief description of your issue
When I type winget in any command prompt always results in an error saying it can't find the file winget.exe. Reading the path to the exe reveals that is trying to reach for it in the wrong user folder
Screenshots are in Italian

Typing the right path by hand results in winget functioning properly

I have two users on this PC, my folder is "utente"
I checked Path and also updated winget manually but it didn't fix the issue, first time seeing it
Steps to reproduce
Typing any winget command results in this error
Expected behavior
The full list of arguments should pop up
Actual behavior
It's giving me an error saying it can't winget.exe in the wrong file path
Environment
Gestione pacchetti Windows v1.4.11071
Copyright (c) Microsoft Corporation. Tutti i diritti sono riservati.

Windows: Windows.Desktop v10.0.19045.3031
Architettura di sistema: X64
Pacchetto: Microsoft.DesktopAppInstaller v1.19.11071.0

Hi @volo34,
Could you check your Path variable $env:path, and see if C:\Users\utente\AppData\Local\Microsoft\WindowsApps is among the entries? It sounds like this might be missing.
(For better readability, try $env:path -split ";")

Hi @volo34,
Could you check your Path variable $env:path, and see if C:\Users\utente\AppData\Local\Microsoft\WindowsApps is among the entries? It sounds like this might be missing.
(For better readability, try $env:path -split ";")

It's there

Can you try:
Reset-AppxPackage -Package Microsoft.DesktopAppInstaller_1.20.1511.0_x64__8wekyb3d8bbwe

Can you try:
Get-AppxPackage *appInstaller* | Reset-AppxPackage

Even tried this

For some reason, Reset-AppxPackage doesn't exist

I think you first need to import the commands. I also play with Powershell sometimes.

I think you first need to import the commands. I also play with Powershell sometimes.

How can I do that?

Sometimes they like to hide it there.

But Windows 10 or later is very different because you need to remove-appx and add-appx

Can you check this folder have 4 files under this directory, C:\Users\utente\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe
winget.exe
python.exe
python3.exe
WindowsPackageManagerServer.exe
This Windows 11 Setup migrating the variable on Windows 10 21H2 causing the correct path on winget in standard user is C:\Users%USERNAME%\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe

⚠Important⚠ Please do not delete this folder C:\Users%USERNAME%\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe because it contains some important files to load the winget commands.

Can you check this folder have 4 files under this directory, C:\Users\utente\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe?
winget.exe python.exe python3.exe WindowsPackageManagerServer.exe
This Windows 11 Setup migrating the variable on Windows 10 21H2 causing the correct path on winget in standard user is C:\Users%USERNAME%\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe on Windows 11 22H2.

Is it normal that they are all 0 KB?

Can you check this folder have 4 files under this directory, C:\Users\utente\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe?
winget.exe python.exe python3.exe WindowsPackageManagerServer.exe
This Windows 11 Setup migrating the variable on Windows 10 21H2 causing the correct path on winget in standard user is C:\Users%USERNAME%\AppData\Local\Microsoft\WindowsApps\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe on Windows 11 22H2.

Is it normal that they are all 0 KB?

Yes, it is true.

This still happens every time I update Microsoft.DesktopAppInstaller via the Microsoft Store. The error lasts for some days, sometimes even weeks, then it starts working again. Haven't found what causes this

I think most likely could be registry value in the path.


Create separate music_assistant.log for tag/pil media scan loggings

as discussed on https://discord.com/channels/753947050995089438/753947050995089442/986926012170371102 it would be great to have a dedicated MA log, containing the id3 tag errors and other music file related issues.
That way we could keep the home_assistant.log much cleane, while it is flooded currently with things like:
2022-06-16 11:17:15 WARNING (MainThread) [music_assistant.music.file] /media/music/nas/mp3/Wagner/Walkure - Stein, Bayreuth 1970, 11597-3/Walkuere - Disc 2/02.  Was verlangst du.mp3 is missing ID3 tag [albumartist], using Various Artists as fallback
2022-06-16 11:17:16 WARNING (MainThread) [music_assistant.music.file] /media/music/nas/mp3/Wagner/Walkure - Stein, Bayreuth 1970, 11597-3/Walkuere - Disc 2/05.  Ein andres ist's.mp3 is missing ID3 tag [albumartist], using Various Artists as fallback

and have it persist restarts, so we can use the file as a 'guide' to actually fix the tagging errors.
next to that, there's a multitude of:

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 79, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 220, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 137, in handle
    result = await result
  File "/config/custom_components/mass/panel.py", line 91, in get
    data = await mass.metadata.get_thumbnail(url, size=size)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/controllers/metadata/__init__.py", line 235, in get_thumbnail
    thumbnail = await create_thumbnail(self.mass, path, size)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/helpers/images.py", line 30, in create_thumbnail
    img_data = await prov.get_embedded_image(path)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/music_providers/filesystem.py", line 850, in get_embedded_image
    return await self.mass.loop.run_in_executor(None, _get_data)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/music_providers/filesystem.py", line 847, in _get_data
    tags = TinyTag.get(file_path, image=True)
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 194, in get
    tag.load(tags=tags, duration=duration, image=image)
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 208, in load
    self._parse_tag(self._filehandler)
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 695, in _parse_tag
    self._parse_id3v2(fh)
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 730, in _parse_id3v2
    frame_size = self._parse_frame(fh, id3version=major)
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 771, in _parse_frame
    frame_id = self._decode_string(frame[0])
  File "/usr/local/lib/python3.9/site-packages/tinytag/tinytag.py", line 841, in _decode_string
    raise TinyTagException('Error decoding ID3 Tag!')
tinytag.tinytag.TinyTagException: Error decoding ID3 Tag!

which, just like the PIL errors:
File "/config/custom_components/mass/panel.py", line 91, in get
    data = await mass.metadata.get_thumbnail(url, size=size)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/controllers/metadata/__init__.py", line 235, in get_thumbnail
    thumbnail = await create_thumbnail(self.mass, path, size)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/helpers/images.py", line 47, in create_thumbnail
    return await mass.loop.run_in_executor(None, _create_image)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.9/site-packages/music_assistant/helpers/images.py", line 43, in _create_image
    img.thumbnail((size, size), Image.ANTIALIAS)
  File "/usr/local/lib/python3.9/site-packages/PIL/Image.py", line 2475, in thumbnail
    im = self.resize(size, resample, box=box, reducing_gap=reducing_gap)
  File "/usr/local/lib/python3.9/site-packages/PIL/Image.py", line 2043, in resize
    self.load()
  File "/usr/local/lib/python3.9/site-packages/PIL/ImageFile.py", line 251, in load
    raise OSError(
OSError: image file is truncated (59 bytes not processed)

just litter the current HA log, without really being useful at all, because there's no reference to the offending issue.
Suggestion/requiest would be:
keep these errors out of HA, make them a bit more friendly to read, and have them persist in a dedicated log.
if t all possible, in a separate folder, and Not in de /config folder which I personally like to keep as tidy and write-free as possible.
thanks for considering!

I really meant Pil loggins...
File "/usr/local/lib/python3.9/site-packages/PIL/ImageFile.py",

Moved to discussions, raise your votes!


When launched and without delay immediately given a search term, psearch gives "Broken Pipe Error 32"

When launching PySearch and immediately after hitting the shortcut (PySearch is not running yet) entering a prompt for PySearch to search for, the FZF UI will break with a "broken pipe error [32]" message.
This is fixed by reloading with CTRL + R.

Fixed a while ago


feat(mockServer): set preserveResolvers default to true

Description
Motivation and Context
https://github.com/americanexpress/parrot/issues/253
How Has This Been Tested?
Tested locally with an OneApp module, but I'm unsure whether sharing screenshots of our local development screens in a public github PR breaches any American Express HR violation. I'm on Web and Mobile Slack!
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (adding or updating documentation)
[ ] Dependency update

Checklist:

[ ] My change requires a change to the documentation and I have updated the documentation accordingly.
[x] My changes are in sync with the code style of this project.
[x] There aren't any other open Pull Requests for the same issue/update.
[ ] These changes should be applied to a maintenance branch.
[ ] This change requires cross browser checks.
[ ] This change impacts caching for client browsers.
[ ] I have added the Apache 2.0 license header to any new files created.

What is the Impact to Developers Using parrot?

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


Adapt to recent openssl download reorganization

What this PR does / why we need it:
Subject says it all.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
Note that travis will fail eventually but it should only be later in the docker build of cc-builder due to the known issue that dev image is built from master, not current branch (see #269).  Also note that the peer image currently still won't build but that should also be after the openssl build relevant part during the attempt to do a make plugins which is currently not yet working ...

Hmm, the PR actually had some fixup commits, but i guess it's now a bit later for that ...


Fix: Use NodeId where NodeId is meant, not u64

Note: It would be ideal to wrap NodeId into a newtype to prevent wrong usage in the future, but that's a breaking change.
Further, NodeId should ideally be a generic argument of Raft type, but that's even bigger breaking change.
The fix in suite.rs is not perfect, since it's still using a literal (as are all the tests).
This change is in preparation for another PR that introduces a possibility to use NodeId as a GUID/128-bit entity (e.g., you could also code IPv6 address there directly).

@mergify update

To be honest, making NodeId a generic type is just what I was planning to do! :DDD
Thanks for this clean patch!

To be honest, making NodeId a generic type is just what I was planning to do! :DDD

Yeah, but the number of generics then keeps increasing. I was thinking about a different approach: instead of adding generics, reduce them to one, which would provide all necessary types as associated types. Then it's extremely easy to add further required types (one-liner for the user to adapt to the change) and also to set defaults (once the relevant Rust feature is complete; then no effect for the user), so you could even think about providing entire set of runtime features that can be implemented more optimally by the user.

Thanks for this clean patch!

Yeah, but the number of generics then keeps increasing. I was thinking about a different approach: instead of adding generics, reduce them to one, which would provide all necessary types as associated types. Then it's extremely easy to add further required types (one-liner for the user to adapt to the change) and also to set defaults (once the relevant Rust feature is complete; then no effect for the user), so you could even think about providing entire set of runtime features that can be implemented more optimally by the user.

Hey man, to me, one trait with associated types looks like a great idea!


Update git package

On all nodes

Might have to follow this approach: https://stackoverflow.com/a/74028195/6416660
As default package is too old or check if a newer version exists on snap then add there.

This was completed in prod.  Double check the code base to see if prod and code differ.

Is there a PR for this?

@steven-basart
Bump.

Yeah should close:
https://github.com/centerforaisafety/cerberus-cluster/blob/74f3c857a3844dfd8d42844a9333e317b03b189a/playbooks/roles/cais-all/tasks/ol-7.yml#L55
It's already been added to main a while ago and is in sync with prod.


Fix Error : Pick Image CameraOnly on Android Q

Environment.getExternalStoragePublicDirectory is now deprecated on
Android Q.
On Android Q cameraOnly is always getting error
"Failed to create image file" with blank white space screen.

Hi @AjiSantosoIND , I think i'm gonna include this commit to my others PR because there's CI things that needs to be fixed first.
Thanks anyway 👌


Add binary and varbinary SQLDataType

Faster comparison and sorting of fields using binary and varbinary types in MySQL comparing to char and varchar. In case of binary/varbinary, comparison and sorting is based on the numeric values of the bytes in the values.

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

Codecov Report

Merging #124 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #124     +/-   ##
=========================================
- Coverage   85.25%   85.16%   -0.1%
=========================================
  Files          43       43
  Lines        3575     3579      +4
=========================================
  Hits         3048     3048
- Misses        527      531      +4

Flag
Coverage Δ

#SwiftKuery
85.16% <0%> (-0.1%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/SwiftKuery/SQLDataType.swift
85.71% <0%> (-14.29%)
: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 039375f...f4c1b1d. Read the comment docs.

Hey @Mladen-K , do the 3 SQL platforms (PostgresSQL, MySQL, and SQLite) that we support use the same notation? What I mean is does the Binary Type in Swift always translate to binary in SQL for the  3 platforms.

In PostgreSQL, there are both data types, although they are represented as BIT and BIT VARYING. Without testing on working PostgreSQL database, I can't tell for sure wether the generated query will generate required fields with "binary" and "varbinary". SQLite doesn't support binary types...

I believe that instead of hardcoding a string in SwiftKuery, ideally we should add a case to the QueryBuilder and then override the default binary in Swift-Kuery-PostgreSQL to BIT and the same for varbinary. There is also the need to research how we handle types not being available in all three platforms, and try to take the same approach for SQLite

Ok, I'll see what I can do.

Thanks

http://www.sqlitetutorial.net/sqlite-data-types/ seems useful.

Hey @Mladen-K , any news on this?

@EnriqueL8, Just haven't had time to check it out. Sure will do in next few days.

Thanks!


Fix user guide

Fixes #117 for inconsistent command
Attempt to fix #143, fix #133, fix #140, fix #132 for unclear index instruction

Pull Request Test Coverage Report for Build 418

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 60.088%

Totals

Change from base Build 414:
0.0%

Covered Lines:
1647

Relevant Lines:
2741

💛  - Coveralls


Add files via upload

i have done some of the work to the cpd-app
1-login/register
2-firebase
3-splash screen
4-home screen
4-sideMenu
5-links to different pages in the sidemenu

i try but it says the file size is too big so i cannot upload

Are you adding the files via upload on GitHub?
The best way would be to commit the files to the repository from your local repo. For that, you'll need to initialise a repo with git init command.
Once that's done add all the project files to the staging area with git add . command and make a commit with git commit -m "commit message".
When that's done the code needs to be pushed into the GitHub repo. First, you'll need to add the remote repository, use git remote add origin https://github.com/szpatrichard/cpd-medical.git. Verify the new remote URL using git remote -v.
You can now push all the changes in your local repo to the remote repository specified in the origin. If your default branch name is main, then there will be an error as the main branch in the remote repository is write-protected as a security rule. To successfully push your commit, you need to specify which branch on the remote repository you want to push to: git push origin main:your-branch-name.

I uploaded the app to the following link
https://github.com/Zubaidullah/cdp-group-project
On Fri 28 Apr 2023 at 4:14 p.m., Richard Szilagyi @.***>
wrote:

Closed #1 https://github.com/szpatrichard/cpd-medical/pull/1.
—
Reply to this email directly, view it on GitHub
https://github.com/szpatrichard/cpd-medical/pull/1#event-9125949469, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGISTEZEGPPO7CCQYP2NYCTXDPNENANCNFSM6AAAAAAXMOVB6E
.
You are receiving this because you authored the thread.Message ID:
@.***>


Device Pixel Ratio and Retina Support?

I have a feeling that the web API is not taking into account the device pixel ratio. When I try the demo, the animation appears blurry on my retina Mac. This is also noticeable on the rive.app website.
Potential Fix
A simple fix would be to double the canvas context and then downscale the canvas itself. I think the API could somehow do this behind the scenes.
Workaround
What I can do right now is to set a rather large size on the canvas and then use CSS transforms to downscale it. But this comes with other issues.
CodePen:
Here you can visually compare the rendering quality / edges of a an unscaled and scaled canvas.
https://codepen.io/getflourish/pen/XWNOojM
Visual Comparison:
scale: 1.0 (left) vs scale: 0.5 (right)

Thanks for highlighting this; we're seeing the same thing ourselves and are working on a fix for this in the runtime.

@getflourish You can onLoad of the Rive instance when created, call .resizeDrawingSurfaceToCanvas() to correctly set the width/height attributes accordingly with devicePixelRatio taken into account.
More here: https://help.rive.app/runtimes/overview/web-js/faq#why-is-my-animation-blurry-at-runtime

Thanks!

Hello. I am having the same problem while on an angular app. Unfortunately, I cannot find Rive object on RiveModule via ng-rive. So I cannot call .resizeDrawingSurfaceToCanvas(). Is there any workaround this? Am I missing something? Thanks.

I am also experiencing this issue with Angular on my MacOS. Is there any advice what to do for pixelated graphics with ng-rive ?


Handle non-existing ignore files.

rsync exits with error code if file passed to --ignore-from does not exist.
This PR handles that and also refactors mainframer.sh into bunch of functions for easier understanding.

PAUSEMERGE
Actually, let's merge test framework #66 first and then I'll add tests for this PR.

Rebased and added tests for ignore rules!

RESUMEMERGE, @ming13 PTAL


Thick lines on jelo, loje, laso and mute

Some thick lines have been forgotten on the top of jelo, loje and laso.
And at the bottom of mute.

Left-over artifacts from converting the original non-uniform strokes to uniform ones, thanks for pointing these out! I cleaned up these glyphs but there are probably more with similar issues.


[QUESTION] How To Resolve Real Objects Keys?

Here is original code:
// index.js
const client = require('client.js'); // trouble in module what call from client.js
if(client.active) console.log("I active!");
else console.log("I didn't active!)
// client.js
module.exports = new class Kruzya{constructor() {this.active = true}};

Here is settings:
{
    "renameProperties": true,
    "renamePropertiesAutoExclude": true,
}

Хер ис обфускейтед коде:
const a0_0x5a0d73 = require('client.js');
// I delete if else code because this is useless
a0_0x5a0d73['_0x1e4856'] // undefined
a0_0x5a0d73['active']  // true

Yep, one more bug-feature-as-designed. But I have idea how we could to do. I would create a second issue becase that only question, not a resolve of error.
Temp fix:
{
    "reservedNames": ["active"] // this should fix this issue, but we didn't get obfuscated prop, so this is bad :c
}

For the same multifile names names cache should be implemented.
https://github.com/javascript-obfuscator/javascript-obfuscator/issues/629


Target net60

@dustinsoftware thank you for the pr, but what does this get you since it already targets net standard? Just curious

Are you looking for the nupkgs in particular? I probably need to update keys if you are.

Yeah dotnet tool install will pull from nuget so those need an update..
On Thu, Apr 7, 2022 at 22:37, Hans van Bakel @.***>
wrote:

Are you looking for the nupkgs in particular? I probably need to update
keys if you are.
—
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/pull/290#issuecomment-1092385476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCFVI77VI2QOSJM2TZTLVD6LW3ANCNFSM5ROOURAQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'll see if I can get those back publishing over the weekend
On Fri, Apr 8, 2022, 04:52 Masters @.***> wrote:

Yeah dotnet tool install will pull from nuget so those need an update..
On Thu, Apr 7, 2022 at 22:37, Hans van Bakel @.***>
wrote:

Are you looking for the nupkgs in particular? I probably need to update
keys if you are.
—
Reply to this email directly, view it on GitHub
<
https://github.com/hvanbakel/CsprojToVs2017/pull/290#issuecomment-1092385476
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAHGCFVI77VI2QOSJM2TZTLVD6LW3ANCNFSM5ROOURAQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/pull/290#issuecomment-1092781178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7QKPBRFQYBH6QW67MGLLLVEAMY7ANCNFSM5ROOURAQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


[QST] Test the performance of casting array to string

What is your question?
This is a follow-on issue of https://github.com/NVIDIA/spark-rapids/pull/4221.
Currently, casting array to string is a little complicate because we need to consider some corner cases, especially when legacy = true. We need to test the performance impact, such as:

compare the performance when legacy is enabled and disabled
compare the performance of CPU version and GPU version
profile the code and try to optimize it.

see the performance report in #4221


Option Group Header css selector?

I am unable to track down the CSS selector that allows you to style the option group headers. Please help!
Thanks!

.Select-option-group-label.


Fix responsiveness of Approvals page.

A simple overflow property should help for now.

A simple overflow property should help for now.

Some commits in #63 fix the issue by simply making NavTwo fixed.


Blame timeout

To avoid heavy blame operations from taking too much time, a timeout should be implemented that aborts the blame command if it exceeds a configurable amount of time.
Suggested configuration  vairable and its default value:
blameTimeout = 20000 ms or 20 s

Will try to write a script that creates a nice commit history to test with. And I only wrote this comment to get a green dot :)


TCPClient stay alive , send message and receive

Hello , i'm trying to do smart Home application which will connect to TCP Server
i was using socket on Unity but i decided to move to xamarin
i tried your plugin it's great , just when i connect to server and send message , i want to stay connected and send the message
not after i disconnect the message be sent , because i need to keep the client connected to the server for the server sending update sometimes to the client
how it can be done ?

ok i got it , it's my fault since i don't understand too much i forgot to use
StreamWriter _streamWriter;
then i do that afte rconnect
_streamWriter = new StreamWriter(client.GetStream());
        _streamWriter.WriteLine("text");
        _streamWriter.Flush();

thank you , this answer for anybody had the same miss understanding like me

Yes - the socket was likely being flushed on disconnect - you could call Flush yourself (as you have done) or specify a 0 buffer size if you're happy with the implications of that 👍


Use ol.format.GMLBase#srsName instead of undeclared srsName_

The variable was changed from private to protected in 8182396

Nice catch LGTM


Run jake without global installation

Andrew, thanks for this PR!
Unfortunately, your variant doesn't work on Windows (the error message is '.' is not recognized as an internal or external command). But looks like local jake creates both Windows batch and Unix bash scripts in node_modules/.bin directory, which is added in PATH env variable by npm. So
"scripts": {
    "build": "jake"
}

should work fine without global jake for both platforms!
What do you think about this variant?

Oh, crap! Always forget about windows. Thank you, fixed that in merging branch.

Merged! Thanks again!


[Bug] Scrollbar in Firefox

To Reproduce
Create a  quite long content note.
Switch to preview mode and scroll to the bottom until you see the very last line.
Expected behavior
You should see the last line and, if possible, the last blank line.

Actual behavior
But, you will see only a half of the last line.

Notes
This issue occurs on Firefox. There is no problem with the chromium based browsers. Maybe just  a little hack of CSS.

for the class .previewer.previewer_direction-ltr
if we use padding-bottom: 0px; bottom: 39px;
it fixes the issue. for chrome.

I will create a pull request to fix this.


Add IntAct to import chain

Added for historical tracking purposes:
Query (CM):
Do you have a conversion of IntAct already? E.g. for
http://www.ebi.ac.uk/intact/complex/details/EBI-767678
with has_part relationships to the UniProt classes
Response (DN):
No, we do not have such a translation. In fact, we don't yet even have an import of the (possible not-in-PRO) proteoforms described there. The first step we must take is to develop an accurate mapping between IntAct proteoforms and PRO proteoforms (good headway made on this, using Reactome as a test case). Then, we'd create the PRO terms for those proteoforms (I do not yet even have an estimate of the upper bound for this). With those in hand, we would then tackle the issue of placing the IntAct complexes in a hierarchy.
Follow-up (CM):
OK, for our purposes we don't need the hierachy, just the parts in OWL (not even necessarily closed) and a parent of 'complex'

I understand the "parts in OWL" need (you mean the proteoforms, as we call them). But not sure what is meant by "parent of 'complex'"

E.g.
http://wwwdev.ebi.ac.uk/intact/complex/details/EBI-8790722
==>
SubClassOf 'GO:protein complex' AND
  has_part some PR:P31376 AND
  ...
  has_part some PR:Q03388

Understood. Thanks.


Fix for snap-1517

Persistent server's status is not toggeled from "waiting" to running even if they are already "running".
As a fix, in FabricServiceImpl#endNotifyWaiting, changed the status to running. For this, added a new function CacheServerLauncher#runningForced().
There exists a similar function CacheServerLauncher#running() that is called from but that does not change the status to "running" if the current status is "waiting". So added a function runningForced() instead.
This is not related to large volume testing. The server always remains in "waiting" state (when checked with snappy-status-all.sh) if it had to wait for buckets from other nodes
Patch testing
Tested manually by starting the cluster and checking the the status with snappy-status-all.sh.
Will run store precheckin prior to merge.
ReleaseNotes changes
NA
Other PRs
NA

LGTM. Minor comment is that instead of duplicating the method we can pass a boolean and do it. That should be preferrable.
If you prefer to stick with this way then at least change the method name to indicate the toggle because of endWaitNotification. May be something like:
endWaiting.

Agree with comment of @kneeraj that this can be a boolean in running() instead of duplicating most of the code.

Ok, will add a boolean parameter to running()


Swap 2 numbers using Bit Manipulation techniques

@HarshCasper  I would like to contribute to this issue under DWOC
I can do in c++ and java

Can you please assign this issue to me? I would love to work on this issue under DWOC.
Thank you

I would like to solve this issue using java

I would like to do it. kindly assign me for this issue.

I would like to contribute as a participant of GSSOC'21


Detect JSON API media type as a json content view

In practice, Ember.js applications support JSON API out the box, so it would be awesome if HTTP requests with a Content-Type: application/vnd.api+json header (as defined in the spec) are detected and parsed as JSON 😃
IANA assignment here:
https://www.iana.org/assignments/media-types/application/vnd.api+json
More about JSON API:
https://github.com/json-api/json-api

Awesome, thanks for the great PR! :smiley: :cake:
Will merge once tests pass.


feat:  add time tracking to generate-app [FTL-8779]

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Increment agreement step counter when receiving a nil voteset

Fixes #75

Codecov Report

Merging #76 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
+ Coverage   58.52%   58.64%   +0.12%
==========================================
  Files         116      116
  Lines        6102     6103       +1
==========================================
+ Hits         3571     3579       +8
+ Misses       1933     1929       -4
+ Partials      598      595       -3

Impacted Files
Coverage Δ

pkg/core/consensus/agreement/collector.go
92.3% <100%> (+6.59%)
:arrow_up:

pkg/core/consensus/agreement/broker.go
79.71% <100%> (+0.92%)
:arrow_up:

pkg/core/consensus/selection/filter.go
93.54% <0%> (+16.12%)
:arrow_up:

Continue to review full report at Codecov.

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


fix(interactive-mode): Fix filtering of options based on prompt property

This PR fixes issue #13 by properly filtering the options using the prompt property.
It also updates the example and performs minor tweaks in the README.

Thanks @iamturns!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: