Upgrade from 1.22 to 1.23 throwing exec format error when using helm upgrade on AKS

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
When trying to upgrade from 1.22 to 1.23, we are getting an exec format error on some of our AKS clusters. We are trying to run 4 istiod pods, some come up healthy and others fail with the above error. We are running this from a github action. Confirmed that the images and nodes are running the same architecture. The upgrade times out at this step in our github workflow helm upgrade -i istiod-${DASHED_NEW_ISTIO_VERSION} after upgrading istio base because the pods fail to come up healthy.

Version
istioctl-1.23.2-linux-amd64.tar.gz used by our runners as a binary

Additional Information
No response

What is the image with the sha256? The only times i have seen this, a user incorrectly pushed only the amd64 image to their own registry, making it no longer a multi-arch image.

What is the image with the sha256? The only times i have seen this, a user incorrectly pushed only the amd64 image to their own registry, making it no longer a multi-arch image.

Here's the manifest for the istio/pilot:1.23.2 image in our container registry. We did a docker pull istio/pilot:1.23.2 and then docker tag istio/pilot:1.23.2 <container registry>/istio/pilot:1.23.2 and then docker push <container registry>/istio/pilot:1.23.2. Those were the original commands that were failing with the same error so we tried again but using docker pull --platform linux/amd64 istio/pilot:1.23.2 to make sure the arch was compatible.
{
"schemaVersion": 2,
"mediaType": "application/vnd.docker.distribution.manifest.v2+json",
"config": {
"mediaType": "application/vnd.docker.container.image.v1+json",
"size": 4037,
"digest": "sha256:a342234ebb356711afdfba00daa6e4178e194049dd5c92d8c4d74a26ec1ec2e6"
},
"layers": [
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 30567306,
"digest": "sha256:eb993dcd6942ffcb7633f2cb271bd4b0a275fc9bdc8f5100c5b4d24694cacf50"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 23713757,
"digest": "sha256:5dc5876572ae6ff8a1c35532f8ac40be475ab0cb183a59a22a266f8f0a480418"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 4483,
"digest": "sha256:68b3f62a8929f5520bbc6376ce777bfb01a9ac8617fb9216cdfae228d3cd6600"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 24896416,
"digest": "sha256:c0543a4b59a07709a9672587689ac16c25ed28b2d1512db8f63c83764c2ea944"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 3910,
"digest": "sha256:9fa4f9ea149ec376410b8d283308350539da673a1c40e204c8052199de8c0e13"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 1669,
"digest": "sha256:95298a1c1064f195d7b815f71fd0e80c1d8e5ef88482f9ec4689cd019f94ba9e"
}
]
}

Yeah that doesn't work. Docker will pull the platform of where you ran it on. I use crane cp to do this, probably there are other ways as well

FYI: crane index filter "${imageName}" --platform linux/amd64 --platform linux/arm64 -t "${MIRROR_REGISTRY}/${imageName}" if there're too much platforms.

It seems that when we pushed the original image after using docker pull istio/pilot:1.23.2 which had the incorrect arm architecture, the images were cached on our nodes. After pulling the correct image with docker pull --platform linux/amd64 istio/pilot:1.23.2 and pushing that to our container registry, we updated the pull policy from IfNotPresent to Always which fixed the issue.


Defined new revisions for golang dep: github.com/cloudflare/cfssl

Ran component through the scanners since I noticed it had not been harvested yet. PR matches curation site. Closing.


Run Course Dashboard test and store results

Implemented here: https://github.com/openedx/tutor-contrib-aspects/pull/657
Waiting for beta tests for reports

Results and remediations are here: https://github.com/openedx/tutor-contrib-aspects/issues/800


To make it more accessible we should have the option to use ul and li as the tags for the slider and the slider items

To make it more accessible we should have the option to use ul and li as the tags for the slider and the slider items

Please report this issue to Swiper officials.


Customizable sort formats

This PR proposes experimental _sort_formats behavior extension by introducing the notion of criterion.
The main problem with sorting now is that it's impossible to tweak the default sorting behavior without copy pasting sorting key routine or providing field_preference that is also flawed since it does not support calculated conditions. Thus the aim of this improvement is to be able to easily customize the default sorting behavior with minimal overhead and maximum convenience.
For this to happen the notion of criterion is introduced. Criterion is a either a field from formats dictionary or a synthetic expression calculated based on some data from formats dictionary. Criterion is referenced as a plain string denoting it's name, criteria is either a list or a tuple of criterions.
Currently following criteria are supported for sorting customization:
Field criteria:

format_id
width
height
tbr
abr
vbr
fps
filesize
filesize_approx
language_preference
quality
source_preference

Synthetic criteria:

Synthetic criterion
Explanation

preference
Calculated based on preference field with some corrections

proto_preference
Calculated based on protocol field and prioritizes direct HTTP(s) URLs

ext_preference
Video extension preference, calculated based on ext field and value of prefer_free_formats setting

audio_ext_preference
Audio extension preference, calculated based on ext field and value of prefer_free_formats setting

With this approach sorting can be customized in several ways.
Way 1: Hardcode criteria preference
Use only specified criterions when sorting formats removing any other default criteria from consideration. This is achieved by passing criteria to criteria_preference in order of preference.
Example: Override default criteria with set of only 5 criteria in exactly this preference:
# Bitrates are completely broken. Single m3u8 may contain entries in kbps and bps
# at the same time without actual units specified. This lead to wrong sorting.
self._sort_formats(formats, criteria_preference=('preference', 'height', 'width', 'fps', 'format_id'))

Way 2: Tweak relative criteria preference
Tweak criteria relative preference between several criteria keeping all other default criteria inplace. This is achieved by passing relative critea list to criteria_relative_preference.
Example: Prefer width over height, all other default criteria stay as before
self._sort_formats(formats, criteria_relative_preference=('width', 'height'))

Way 3: Exclude criteria from consideration
Remove criteria from consideration completely. This is achieved by passing criteria list to exclude to criteria_exclusion.
Example: Remove tbr from consideration, all other default criteria stay as before:
self._sort_formats(formats, criteria_exclusion=('tbr',))

Way 2 and Way 3 may be combined.
This solves #6018, #8001.

@dstftw Could this branch be revised and eventually merged?


Problems with nullptr in not cpp11 project.

Hello, I have a problem where try to compile my test project without cpp11 standard supporting.
Now using a new version of gcc is enough to using nullptr in your test projects, but if I don't use -std=c++11 I got a message that I fool.
May be you can use "__cplusplus" directive to check is cpp11 standard is used? Or may be better to get a choise to user by something like "#define CATCH_CONFIG_NO_CPP11FEATURES"?
[kovtyukhrd@kovtyukhrd ~ ]$ gcc --version
gcc (GCC) 5.1.0

+1
gcc --version
i586-alt-linux-gcc (GCC) 5.1.1 20150618 (ALT Linux 5.1.1-alt2)

vpashka, what whay to solve this problem you prefer?

I mean, I have the exact same problem and her decision, I do not know yet.

What is the actual message you're getting?

I'm sorry .. I think there was some internal error. Now g++5 normal compiles my tests.

Either way I have now added the ability to disable C++11 feature use - individually (using, e.g. CATCH_CONFIG_CPP11_NO_NULLPTR) or en-masse (CATCH_CONFIG_NO_CPP11).
Is your issue resolved?

It might also be worth revisiting the logic that (dis|en)ables nullptr for gcc (IIRC, currently it's based solely on the gcc version (__GNUC__ et al)rather than the language version).
Specifically it doesn't check __cplusplus to see if C++11 is enabled, so if you're using a reasonably recent version of the compiler with an over-a-decade-old version of the language, you'll see these kind of problems. In fact, I'd consider using only __cplusplus >= 201103 to determine whether to use nullptr or not; I doubt there are any vendors who define __cplusplus as 201103 or greater who don't support nullptr, and if there are any, then you can point any related issues back to them where they belong...

Are there any versions of GCC that support nullptr but don't set __cplusplus to >= 201103?
The __GXX_EXPERIMENTAL_CXX0X__ check in there looks suspicious in that regard. I think I "borrowed" that bit from boost, at some point (I don't have boost sources on this machine to check).

Hello, I'd find something on this issue:
AFAIUI GXX_EXPERIMENTAL_CXX0X is was used in gcc compilers with version less than 4.7, and after 4.7 version we can check a __cpluslpus. My version is g++ (GCC) 5.1.0 and they probably, remoove the GXX_EXPERIMENTAL_CXX0X support, because they are not experimental now.

After seeing https://github.com/philsquared/Catch/pull/471 I think I follow @PureAbstract's comment now.
We currently have this guard:
#if __GNUC__ > 4 || (__GNUC__ == 4 && __GNUC_MINOR__ >= 6 && defined(__GXX_EXPERIMENTAL_CXX0X__) )
#   define CATCH_INTERNAL_CONFIG_CPP11_NULLPTR
#endif

I think if we just drop the first part (__GNUC__ > 4 ||) everything should work (because it will drop down to the later #if (__cplusplus >= 201103L) catch-all)?
That seems to make sense. I'm going to make a change on that basis.

Closing this, as it is very old and has been significantly changed since.


Problem in installing jvarkit

Hi,
I am installing jvarkit: http://lindenb.github.io/jvarkit/Biostar84452
On running:
./gradlew biostar84452
I got following errors:
> Task :biostar84452 FAILED

FAILURE: Build failed with an exception.

* Where:
Build file '/home/admin1/jvarkit/build.gradle' line: 673

* What went wrong:
Execution failed for biostar84452 com.github.lindenb.jvarkit.tools.biostar.Biostar84452.
> Error running javac compiler

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 710ms
1 actionable task: 1 executed

Although javac is located in my system:
admin1@waqasuddin:~/jvarkit$ javac
Usage: javac <options> <source files>
where possible options include:
  -g                         Generate all debugging info
  -g:none                    Generate no debugging info
  -g:{lines,vars,source}     Generate only some debugging info
  -nowarn                    Generate no warnings
  -verbose                   Output messages about what the compiler is doing
  -deprecation               Output source locations where deprecated APIs are used
  -classpath <path>          Specify where to find user class files and annotation processors
  -cp <path>                 Specify where to find user class files and annotation processors
  -sourcepath <path>         Specify where to find input source files
  -bootclasspath <path>      Override location of bootstrap class files
  -extdirs <dirs>            Override location of installed extensions
  -endorseddirs <dirs>       Override location of endorsed standards path
  -proc:{none,only}          Control whether annotation processing and/or compilation is done.
  -processor <class1>[,<class2>,<class3>...] Names of the annotation processors to run; bypasses default discovery process
  -processorpath <path>      Specify where to find annotation processors
  -parameters                Generate metadata for reflection on method parameters
  -d <directory>             Specify where to place generated class files
  -s <directory>             Specify where to place generated source files
  -h <directory>             Specify where to place generated native header files
  -implicit:{none,class}     Specify whether or not to generate class files for implicitly referenced files
  -encoding <encoding>       Specify character encoding used by source files
  -source <release>          Provide source compatibility with specified release
  -target <release>          Generate class files for specific VM version
  -profile <profile>         Check that API used is available in the specified profile
  -version                   Version information
  -help                      Print a synopsis of standard options
  -Akey[=value]              Options to pass to annotation processors
  -X                         Print a synopsis of nonstandard options
  -J<flag>                   Pass <flag> directly to the runtime system
  -Werror                    Terminate compilation if warnings occur
  @<filename>                Read options and filenames from file

at: /home/admin1/anaconda3/bin/javac
Version is: javac 1.8.0_152-release
OS:
LSB Version:    core-11.1.0ubuntu2-noarch:security-11.1.0ubuntu2-noarch
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.3 LTS
Release:        20.04
Codename:       focal

I tried with sudo as well, any help is highly appreciated.

Hi @waqasuddinkhan, since this is an issue purely related to installing jvarkit, I recommend posting it as an issue there.


Investigate increasing rate limit for excel api routes

We should investigate increasing our rate limiting for uploading excel documents. Currently if you upload about 3 in a row you will trigger the rate limit, causing the spinner to stall. We need to find a nice balance that won't overload or crash our pods but that can allow a user to submit multiple files in a row.

[x] monitor resources used while uploading excel in openshift
[x] adjust rate limit if our resources allow

Hey team! Please add your planning poker estimate with Zenhub @AntBush @marcellmueller @rafasdc

@MarsRomer Not sure if you wanted to review/test this you can now upload 3 (or more) excel files per minute in a row without the spinner timing out.

I tried, but the uploads are so quick I can't keep up! hahaha


fix(pure-cell): fix subcomponent addon and category

Пофиксила подкомпоненты  аддон и категория

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 81.906%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5784

Relevant Lines:
6502

💛  - Coveralls


Blocked connecting to sql from python azure functions

I am trying to host a timer trigger python based azure function that runs periodically. I am blocked connecting from python azure functions to sql. Same error has been reported in public thread that’s been closed but theres no clear resolution.
https://github.com/Azure/azure-functions-python-worker/issues/249
Since this is blocking my current work. would like to know if this is a supported scenario/has workaround/alternative drivers to connect to sql so I can look for alternative Azure services like Azure batch to host this python job if azure function will not work.
[Error] Executed 'Functions.TimerTrigger' (Failed, Id=45de9d66-99f0-43e8-9b5e-eda45c5768f5) Result: Failure Exception: Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib 'ODBC Driver 13 for SQL Server' : file not found (0) (SQLDriverConnect)") Stack: File "/home/site/wwwroot/.python_packages/lib/python3.6/site-packages/azure/functions_worker/dispatcher.py"
Detailed Error:
2019-05-23T00:16:08.922 [Information] Executing 'Functions.TimerTrigger' (Reason='This function was programmatically called via the host APIs.', Id=45de9d66-99f0-43e8-9b5e-eda45c5768f5)
2019-05-23T00:16:09.034 [Information] ['/azure-functions-host/workers/python', '/root/.pyenv/versions/3.6.8/lib/python3.6', '/root/.pyenv/versions/3.6.8/lib/python3.6/site-packages', '/home/site/wwwroot', '/home/site/wwwroot/.python_packages/lib/python3.6/site-packages', '/usr/local/lib/python36.zip', '/usr/local/lib/python3.6', '/usr/local/lib/python3.6/lib-dynload', '/usr/local/lib/python3.6/site-packages']
2019-05-23T00:16:09.043 [Information] Python timer trigger function ran at 2019-05-23T00:16:09.024444+00:00
2019-05-23T00:16:09.088 [Error] Executed 'Functions.TimerTrigger' (Failed, Id=45de9d66-99f0-43e8-9b5e-eda45c5768f5) Result: Failure Exception: Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib 'ODBC Driver 13 for SQL Server' : file not found (0) (SQLDriverConnect)") Stack: File "/home/site/wwwroot/.python_packages/lib/python3.6/site-packages/azure/functions_worker/dispatcher.py", line 288, in _handle__invocation_request self.__run_sync_func, invocation_id, fi.func, args) File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run result = self.fn(*self.args, **self.kwargs) File "/home/site/wwwroot/.python_packages/lib/python3.6/site-packages/azure/functions_worker/dispatcher.py", line 347, in __run_sync_func return func(**params) File "/home/site/wwwroot/TimerTrigger/init.py", line 126, in main conn = pyodbc.connect(dbAccessStr
Code used:
import datetime
import logging
import azure.functions as func
from sklearn.datasets import load_iris
import pyodbc
import os
import sys
def main(mytimer: func.TimerRequest) -> None:
logging.info(sys.path)
utc_timestamp = datetime.datetime.utcnow().replace(
tzinfo=datetime.timezone.utc).isoformat()
if mytimer.past_due:
    logging.info('The timer is past due!')

logging.info('Python timer trigger function ran at %s', utc_timestamp)

print("Testing...")
dbConnectionString = "<Connection string>"

def getConnectionStringAttribute(conn_string, name):
    return conn_string[conn_string.find(name) + len(name) + 1:conn_string.find(';', conn_string.find(name))]

dbDriver = '{ODBC Driver 13 for SQL Server}'
dbServer = getConnectionStringAttribute(dbConnectionString, "Server").split(',')[0]
dbPort = getConnectionStringAttribute(dbConnectionString, "Server").split(',')[1]
database = getConnectionStringAttribute(dbConnectionString, "Initial Catalog")
dbUsername = getConnectionStringAttribute(dbConnectionString, "User ID")
dbPassword = getConnectionStringAttribute(dbConnectionString, "Password")
dbAccessStr = 'DRIVER=' + dbDriver + ';SERVER=' + dbServer + ';PORT=' + dbPort + ';DATABASE=' + database + ';UID=' + dbUsername + ';PWD=' + dbPassword

conn = pyodbc.connect(dbAccessStr)
cursor = conn.cursor()
cursor.execute("SELECT TOP 10 * from dbo.MLAutoRetrainRunInfo_v3 ORDER BY Id DESC")
row = cursor.fetchone()
s = ""
while row != None:
    s = s + "%s\t%s\t%s\t%s\t%s\t%s\t%s\t%s\t%s\n" % (str(row[0]), str(row[1]), str(row[2]), str(row[3]), str(row[4]), str(row[5]), str(row[6]), str(row[7]), str(row[8]))
    row = cursor.fetchone()

print(s)

Resolving based on offline discussion. Solution was to use OBDC Driver 17 for SQL Server since that is what we currently install in the image.

The following code now work to connect from python azure functions on linux - the fix is to use
dbDriver = '{ODBC Driver 17 for SQL Server}'
Code used:
import datetime
import logging
import azure.functions as func
from sklearn.datasets import load_iris
import pyodbc
import os
import sys
def main(mytimer: func.TimerRequest) -> None:
logging.info(sys.path)
utc_timestamp = datetime.datetime.utcnow().replace(
tzinfo=datetime.timezone.utc).isoformat()
if mytimer.past_due:
logging.info('The timer is past due!')
logging.info('Python timer trigger function ran at %s', utc_timestamp)
print("Testing...")
dbConnectionString = ""
def getConnectionStringAttribute(conn_string, name):
return conn_string[conn_string.find(name) + len(name) + 1:conn_string.find(';', conn_string.find(name))]
dbDriver = '{ODBC Driver 17 for SQL Server}'
dbServer = getConnectionStringAttribute(dbConnectionString, "Server").split(',')[0]
dbPort = getConnectionStringAttribute(dbConnectionString, "Server").split(',')[1]
database = getConnectionStringAttribute(dbConnectionString, "Initial Catalog")
dbUsername = getConnectionStringAttribute(dbConnectionString, "User ID")
dbPassword = getConnectionStringAttribute(dbConnectionString, "Password")
dbAccessStr = 'DRIVER=' + dbDriver + ';SERVER=' + dbServer + ';PORT=' + dbPort + ';DATABASE=' + database + ';UID=' + dbUsername + ';PWD=' + dbPassword
conn = pyodbc.connect(dbAccessStr)
cursor = conn.cursor()
cursor.execute("SELECT TOP 10 * from dbo.MLAutoRetrainRunInfo_v3 ORDER BY Id DESC")
row = cursor.fetchone()
s = ""
while row != None:
s = s + "%s\t%s\t%s\t%s\t%s\t%s\t%s\t%s\t%s\n" % (str(row[0]), str(row[1]), str(row[2]), str(row[3]), str(row[4]), str(row[5]), str(row[6]), str(row[7]), str(row[8]))
row = cursor.fetchone()
print(s)


Ensure willDisappear closure is called before removing contentView

A very simple fix, solving issue #85.
Issue Link 🔗
Issue #85 - willDisappear closure not called in some cases
Implementation Details ✏

Added calls to lifecycleEvents.willDisappear() in the removePromptly(keepWindow:) and stretchOut(usingSwipe:, duration:) functions of EKContentView

Great, thanks!  :-)


[FLINK-24671][runtime] Return 0 buffers in use until subpartition vie…

…w initialization in order to avoid NPE

What is the purpose of the change
This PR fixes possible NPE in LocalInputChannel#getBuffersInUse
Brief change log

Return 0 buffers in use if subpartition view has not initialized yet

Verifying this change
This change added tests and can be verified as follows:

Added new test for local input channel

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

LGTM. Thanks @akalash!
BTW, could you double check/confirm that RemoteInputChannel does not have/does not need consider the same problem!

could you double check/confirm that RemoteInputChannel does not have/does not need to consider the same problem!

I also thought about that but this problem doesn't exist for RemoteInputChannel since getBuffersInUseCount is sum of receivedBuffers (which is always not null) and bufferManager#getNumberOfRequiredBuffers (which is also initialized in constructor)

Cool, thanks for confirming @akalash , do not forget to backport this to 1.14


Y-m-d Date Validation failed

Laravel Version: 8.74.0
PHP Version: 8.0.14
Database Driver & Version: 8.0.27-0ubuntu<IP_ADDRESS> for Linux on x86_64

Description:
Laravel date Validation failed to validate Mon Dec 08 1997 16:14:36 as Y-m-d format
Steps To Reproduce:

Set Request rule validation for birth_date with 'birth_date' => 'required|date:Y-m-d',
Send Postman POST Request with x-www-form-urlencoded and birth_date with value Mon Dec 08 1997 16:14:36,
PDOException thrown:

PDOException: SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect date value: 'Mon Dec 08 1997 16:14:36' for column
'birth_date' at row 1 in

Expected
"errors": {
   "birth_date": [
        "The birth date is not a valid date."
    ]
}

Use date_format:Y-m-d not date

@dinhquochan My bad. Thanks.


Only depend on aws-sdk-marketplacemetering and update omnibus gem

A new omnibus gem was recently released that fixes a bug in the omnibus buildkite pipelines which requires updating the omnibus gem in this repo's Gemfile.lock file. However, the current Gemfile has gem "aws-sdk", "~> 2" which conflicts with the omnibus' use of the newer aws-sdk version that breaks aws-sdk into individual gems to enable user's to avoid depending on hundreds of unnecessary aws-sdk gems. It looks like omnibus-marketplace reckoner only needs
the aws-sdk-marketplacemetering so this PR replaces the dependency on aws-sdk with aws-sdk-marketplacemetering.

More commits need to be added to this PR by an omnibus-marketplace developer to ensure that the project's code actually uses the aws-sdk-marketplacemetering properly. For example, the code in the following file will no longer work.
https://github.com/chef-partners/omnibus-marketplace/blob/master/files/chef-marketplace-gem/lib/marketplace/reckoner/aws-sdk/metering_service.rb


hoping this will curtail installing new dependencies.

in looking at AQCU-1009, the install of gsplot and repgen are also pulling in upstream dependencies when their versions are new. This basically leads to installing the fixed version of the dependency if it's not on the server from the fixed MRAN version, but then when gsplot or repgen are installed, they just go ahead and grab the latest and greatest. Wondering if this will prevent that from occurring.

Coverage remained the same at 99.04% when pulling 4c47be56f214c25b88cc57fa1e6cc1438faced5b on mhines-usgs:master into 367361839f381fb4d03e616465c68ffd3eddf8e4 on USGS-R:master.


chore: update @keywordnew to emeritus

Hello friends ❤ I feel this is now long overdue. I haven't been finding enough bandwidth to dedicate the level of time CommComm, Node.js, and OpenJS deserve. It's time for me to accept this reality.
Thus this PR, per CommComm's README.md: "Collaborators may request Emeritus status when they find themselves inactive."
I always welcome crossing paths with y'all in open source :-) This isn't the end. I can be reached via email (I read every email I get, eventually) and I'll be back when I'm ready to ramp up contributions.
My heartfelt gratitude goes out to everyone serving the Node.js project – past, present, and future. Thank you for the good times, the less good, and even the uncertain. It's been quite a ride! From developing CommComm to the metamorphosis into OpenJS, I'm grateful to have had the chance to contribute and learn with all of you 💖

No objections in 7 days, and this is a personal choice one (not something we can really deny). Going to merge.


Solution: building error C4577

Hey there!
First of all, thank you @ryanjon2040 for this amazing plug-in.
One problem I was facing when building UE4GameLiftClientSDK plug-in for UE 4.18.3 and VS 2017 is the error C4577 which I think it might be common in this environment.
For those facing the same problem, one easy solution--not sure about its dirtiness--is to add bEnableExceptions = true; to one line below line 9, on GameLiftClientSDK.Build.cs file. After doing this, probably you'll build your game successfully.

Ok, I've created a PR (https://github.com/YetiTech-Studios/UE4GameLiftClientSDK/pull/7) introducing this information in the README to enhance its visibility.
In case you, @ryanjon2040, think the PR doesn't make sense, I think leaving this issue open or at least with a descriptive label would come in handy for future adventurers.


Keyboard replay control

A bit of refactoring along with the introduction of keyboard control of replay (p to play/pause/resume and s to stop).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 65.04%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6690

Relevant Lines:
9643

💛  - Coveralls

@nhoening Great for demos...


Monaco Editor with Python Language Server

Summary
I've tried to use  PYLS on Monaco Editor using Monaco Language Client with the following configuration
this.editor.setModel(monaco.editor.createModel(exampleCode, 'python', monaco.Uri.parse('inmemory://dummy.py')););
    const services = createMonacoServices(this.editor);

And everything that this blog explained.
Everything works as expected but the auto complete. When I do ctrl + space on the editor, there's only "No Suggestions."  and I saw the PYLS logs error as follows
PYLS Server: 2017-12-08 09:56:21,029 UTC - ERROR - jsonrpc.manager - API Exception: {'message': "'Folder' object has no attribute 'read'", 'args': ("'Folder' object has no attribute 'read'",), 'type': 'AttributeError'}
Traceback (most recent call last):
  File "myVirtualEnv/lib/python2.7/site-packages/jsonrpc/manager.py", line 112, in _get_responses
    result = method(*request.args, **request.kwargs)
  File "myVirtualEnv/lib/python2.7/site-packages/pyls/language_server.py", line 64, in wrapped
    return func(*args, **kwargs)
  File "myVirtualEnv/lib/python2.7/site-packages/pyls/python_ls.py", line 140, in m_text_document__completion
    return self.completions(textDocument['uri'], position)
  File "myVirtualEnv/lib/python2.7/site-packages/pyls/python_ls.py", line 70, in completions
    position=position
  File "myVirtualEnv/lib/python2.7/site-packages/pyls/_utils.py", line 113, in race_hooks
    for impl, result in pool.imap_unordered(_apply, impls, chunksize=1):
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/multiprocessing/pool.py", line 668, in next
    raise value
AttributeError: 'Folder' object has no attribute 'read'

Could you help me? is it about the monaco URI that I pass ? inmemory://dummy.py
note: I've also asked this on Python Language Server repo since I don't know on which side is the issue happened...

It looks like a python error from pyls.

Ahh.. I see... thanks for previewing my issue @akosyakov
Besides this issue, is there any "special configurations" for connecting monaco with pyls? I've checked your PYLS fork and wondering is there any changes on it.

There is not, I think I once intended doing a PR but than didn't. Removed the fork.

@dirathea Have you solve the question now, can you give me a simple demo, thanks

@dirathea Actually, I don't know how to apply Python Language Server  to the Monaco Editor which is embeded in the online website. Can you give me the simple demo, thank you very much

@sally2015 https://github.com/TypeFox/monaco-languageclient/issues/75#issuecomment-385163286 and https://github.com/TypeFox/monaco-languageclient/issues/75#issuecomment-385229537 might help you get started a little.
The main thing you usually have to figure out is how to start the language server. If you're able to do that from the command line just use the same invocation in json-server-launcher.ts and you should be good to go for a start. There may of course be additional configuration options that are needed but it's a start.

were you able to achieve that @sally2015.
Does anyone have any suggestion on how to implement a Python Language Server to the Monaco Editor which is embeded in the online website.

@sally2015 I see you were able to figure it out https://github.com/Microsoft/monaco-editor/issues/833#issuecomment-402301159
If you think there is something can be done to help other, like add a comment to this line, you are welcome to open a PR :)

@akosyakov I was able to run the example provided with Monaco Language Client. The question of this thread, how to implement a Python Language Server for Monaco Editor on a online website?, is yet to be solved.

@anshullodha What do you mean? If you got the example to run then you have a working website. Did I misunderstand something?

I am also having trouble connecting Monaco language client to a "real" C++ language server in this case) other than the simple JSON language server in the example.
I guess adding an end-to-end example of Monaco editor, Monaco language client, and a "real" language server like C++ or Java working together would greatly make things a lot easier for me to follow, and maybe for other who are on this path as well.
Thanks!

@rcjsuen, have you been able to connect Monaco language client with any production language server other than JSON in the example?

@shoorlyne Those language servers are generally more complicated because they rely on having access to some (virtual?) files. This compounds the difficulty in a browser-based environment as you have to hook up the files yourself. This repository does not help you deal with that (as it's not really in the scope of the project). That being said I don't deny the usefulness of such an example, I'm just saying it's not super straightforward.
As to your actual question, yes, I have hooked this project with my own Dockerfile language server. But again, you can't really compare that with a C++ or Java language server as Dockerfiles are like JSON files and are completely standalone and it doesn't care about whether your files are on-disk or in the cloud.

@rcjsuen, since this repo already contains examples connecting with toy-ish language servers, I don't think it would be too off track to include a more complex/practical example though ;)
BTW, thanks for your Dockerfile language server, looks interesting!

Yup, I have tried that too. The UX is superb due to it being pure browse.

I have an example up here of hosting a stand-alone language server over a web socket. You then just need to hook up the Monaco-languageclient examples to point at it.
https://github.com/palantir/python-jsonrpc-server/blob/develop/examples/langserver_ext.py

On 28 Jul 2018, at 16:42, shoorlyne<EMAIL_ADDRESS>wrote:
Yup, I have tried that too. The UX is superb due to it being pure browse.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@rcjsuen Sir, could you please kindly tell me why is the "model.json" file?  and how does it being loaded into "memory".  Because I saw both the official example and your Dockerfile browser example contains this code :" var MODEL_URI = 'inmemory://model.json'; "

@lilifan It's "loaded into memory" because the content of the file is created completely from code.
https://github.com/TypeFox/monaco-languageclient/blob/602e7606c046036d3627b59064bbfb4b67e7d2e8/example/src/client.ts#L22-L32

@rcjsuen Thank you very much!  so "the content of the file"  is the value string contains of schema and line_endings?
By the way, I was curious about how did you know to use the "inmemory://model.json"?  because I didn't find it in any documents and any repositories.

It seems the example lacks syntax highlighting . Actually it has disabled the default syntax colorization Monaco editor provides.(https://microsoft.github.io/monaco-editor/)
Is there anyway we can get it back ?
@rcjsuen   @akosyakov

@lilifan It is mentioned in the VS Code repository.

@lilifan It is mentioned in the VS Code repository.

Thx.


tabsLocation

Version: 1.0.69
Platform: darwin 18.2.0
Plugins: none
appearance tabsLocation: left,  will make window crash

Only allowed values are top and bottom (as in the UI settings). Feel free to add a ticket for sideways tabs though.

left tab is very useful for me!!!  like this!


Plex crashes when trying to load from localhost

Same as the subject line. Telly is up and running correctly however when searching for channel guide it either freezes or doesn't load anything. Running an m3u file from PlayersKlub. No xml file available unless I'm not looking in the right spot.

Please share the log

I've found the issue. Known issue #108


Ubuntu-20.04 image will be included in Pouta

We are going to provide Ubuntu-20.04 image for our Pouta customers.
CCCP-3301

After this change, the following statement would be untrue:
Note that Ubuntu-18.04 is that one image that has ubuntu as the default username

After this change, the following statement would be untrue:
Note that Ubuntu-18.04 is that one image that has ubuntu as the default username
So it needs changing too.

This is not my edition but I would like to change it.

Couple of fixes plz

Yes I have fixed those.

After this change, the following statement would be untrue:
Note that Ubuntu-18.04 is that one image that has ubuntu as the default username
So it needs changing too.

Yes done.

One more thing - can you amend Ubuntu 20.04 to the table on the same page, too

One more thing - can you amend Ubuntu 20.04 to the table on the same page, too

Yes done.


Fix CI/CD

I tried adding the default iOS build/test Githuib Actions template, but there's some customization needed. Adding the bundle install/pod install commands as documented didn't quite cut it -- we're getting some dependency conflicts.
https://github.com/openscape-community/openscape/actions/workflows/ios.yml

This is passing now. I resolved the Bundler/CocoaPods issue, though I'm still not entirely sure why the additional step that installs a specific version of CocoaPods isn't redundant to the commands that were already there.
The Openscape app then builds in 12-15m.
I commented out the subsequent Test step, since we were getting the error "Scheme Openscape is not currently configured for the test-without-building action." I'm guessing the code did came with some automated tests to run, but we'll need to figure out how to properly invoke them.
So, I wouldn't consider this issue resolved. But we at least have a job now that validates on each commit that the app still compiles, which is worth something.

Hmm, looking through the code, I do see some references to (disabled) tests, e.g. in apps/ios/GuideDogs.xcodeproj/xcshareddata/xcschemes/Openscape - Dogfood.xcscheme, but I don't see these tests actually implemented anywhere.
It would definitely be a good idea to have tests, but writing them anew is beyond the scope of just fixing the CI configuration. Marking this as complete for now.


Automerge

Removes labelling.
Adds Scala Steward as GitHub Action.
Adds merge step to PR validation for PRs opened by moia-pipeline.

Merging seems to work, as long as the github-user for Scala Steward is actually moia-pipeline.

But if we use the GitHub action how will the user be moia-pipeline?

I guess moia-pipeline doesn't have an email so we might want to define author-email but I'm not sure to what 😁


Open ports on gateway device

This section only mentions AMQP, MQTT and HTTPS as being supported for inbound communication whereas the table also suggests MQTT + WS and AMQP + WS. Would be great to explicitly mention WebSockets as being supported as this leads to some confusion.

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

ID: 6f6b8e5f-7de7-b40b-ad37-dc9b5c8f7d63
Version Independent ID: 10a9b17d-cbf4-ec07-ae39-953d3d0fd411
Content: Create transparent gateway device - Azure IoT Edge
Content Source: articles/iot-edge/how-to-create-transparent-gateway.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@vslepakov Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@vslepakov I opened a pull request to add references to MQTT over WebSockets and AMQP over WebSockets inline. Since this is a straightforward update, I'll have GitHub resolve this issue once the pull request is merged. If you have any follow-up comments or concerns, feel free to post on this thread again and we can reopen the issue.
Thanks!


Build failing when working with strict mode

When tsconfig.json has strict mode on, when doing ng serve spring-flow dependencies throws error for code-mirror and Model

What can be done here.

I got a fix!!!!

Add lib-check-skip: true
"skipDefaultLibCheck": true,
"skipLibCheck": true

@JerinJ we've upgraded Flo to Angular 9 and used ng-packagr to build Flo with Ivy compiler. It's in branch angular9-plus. Wonder if you could give it a try and see if you get any problems (like the above reported) with that branch. Thanks in advance.


Added support for dot config (.typstfmt.toml)

I don't know whether we should create dot file by default (-C) or not (I think a lot of formatters do create dot file by default), but support for reading from dot file should exist. The order/priority of reading config files (from highest to lowest): typstfmt.toml → .typstfmt.toml.

remove DOT_CONFIG_FILE_NAME and just inline format(".{CONFIG_FILE_NAME}")

This is why I've done this the way I've done this:
cannot call non-const fn `typstfmt_lib::format` in constants
calls in constants are limited to constant functions, tuple structs and tuple variants (rustc E0015)

cannot call non-const fn `std::fmt::format` in constants
calls in constants are limited to constant functions, tuple structs and tuple variants (rustc E0015)

when both the CONFIG_FILE_NAME and DOT_CONFIG_FILE_NAME are present, a warning is printed

I know about makefile (from meme & comments), which has priority over Makefile, so I assumed that we can do the same (dot file is like `Makefile). But I also can see the problem from a user POV (which for some unknown reason have 2 config files in the same dir). I can do this, but are there any specific "requests" right away?

This is why I've done this the way I've done this:
cannot call non-const fn `typstfmt_lib::format` in constants
calls in constants are limited to constant functions, tuple structs and tuple variants (rustc E0015)

My bad, it's format!

I know about makefile (from meme & comments), which has priority over Makefile, so I assumed that we can do the same (dot file is like `Makefile). But I also can see the problem from a user POV (which for some unknown reason have 2 config files in the same dir). I can do this, but are there any specific "requests" right away?

What do you mean requests?
I just think it's an error if anyone had both .typstfmt.toml and typstfmt.toml in their directory structure.
They might try to modify the dot version and wonder why no change is visible

My bad, it's format!

I already tried it:

cannot call non-const fn `std::fmt::format` in constants
calls in constants are limited to constant functions, tuple structs and tuple variants (rustc E0015)

const DOT_CONFIG_FILE_NAME: String = format!(".{CONFIG_FILE_NAME}");
I already faced this problem. We either have to import a crate like lazy_static or duplicate the whole string slice.
This gives the same error: const CONFIG_FILE_NAME: &str = &DOT_CONFIG_FILE_NAME[1..];.
const CONFIG_FILE_NAME: &str = &DOT_CONFIG_FILE_NAME.get(1..).unwrap(); gives Option::<T>::unwrap is not yet stable as a const fn (rustc).

I just think it's an error if anyone had both .typstfmt.toml and typstfmt.toml in their directory structure.
They might try to modify the dot version and wonder why no change is visible

Ok, I'll show a warning in stderr.

I didn't mean DOT_CONFIG_FILE_NAME = "." + format!(...)
I meant, remove DOT_CONFIG_FILE_NAME and inline the above logic

Feel free to tell me when you're ready for another review!

All done.

Thanks for this contribution :)
It's good to go, I'm overreaching :p

It's good to go

Are you sure the global config feature on master would work with this? I need to check.

I read the code again and it would have a weird behavior indeed
I think it needs to be rewritten as
 if let Ok(mut f) = File::options().read(true).open(CONFIG_FILE_NAME)
else  if let Ok(mut f) = File::options().read(true).open(DOT_CONFIG_FILE_NAME)
else /* try global file */
else default

I read the code again and it would have a weird behavior indeed
I think it needs to be rewritten as (pseudocode)
if let Ok(mut f) = File::options().read(true).open(CONFIG_FILE_NAME)
else  if let Ok(mut f) = File::options().read(true).open(DOT_CONFIG_FILE_NAME)
else /* try global file */
else default

Nah, I also read the code and it looks fine. If global isn't loaded, then the program will panic. else default is redundant. Also, making a whole separate outer else if is too much. We have the same logic and same errors for both, so I only included a small if to get the correct file name to throw an error, if necessary.

Looks really nice! Good job!
I'm ready to merge whenever you are !

Thanks a ton !


Redraw at the bottom when bottom_toolbar is disabled

When I disable toolbar for prompt with Ctrl+B, it leaves an empty string. How to make prompt redraw itself at the bottom?
Start:

Ctrl+B - enabled toolbar:

Ctrl+B - toolbar disabled:

Expected:

@techtonik,
It is not possible to make the terminal scroll up again, so unfortunately nothing we can do about this.

Is it possible to upgrade prompt then to a full screen application? The full screen application can then occupy one or two lines. One line is only taken at start if no toolbar is enabled. Two lines if toolbar is present (prompt on the first line) or toolbar was switched off (prompt on the second line).


status "stopped" instead of "spawning", tick() method of LoadShape called only once

Describe the bug
This happens with high number of workers, for me it sometimes happens when I use ~80 workers, it does not happen each time though, I didn't find the root cause so far. It never happened for me when running standalone locust, with low number of workers it happened just once as far as I know.
Sometimes build starts with "stopped" status instead of "spawning". Number of users grows until it reaches expected value, then status changes to "running". The most problematic thing for me is that when it happens, locust stops calling tick() method of load shape, so shape doesn't really work and number of users get stuck on value returned in first call of tick() method.
Expected behavior
Status shown in GUI represents actual status and tick method is called every second until it returns None or the build is stopped manually
Actual behavior
GUI hows status "stopped" instead of "spawning", tick() method of LoadShape called only once
Steps to reproduce
set up master and around 80 workers (maybe it would happen more often with higher number of workers, I didn't have access to more machines to check it)
Environment

OS: centos 7
Python version: 3.8.2
Locust version: 1.4.3, 1.4.4, 1.5.2
Locust command line that you ran: locust --master --skip-log-setup --only-summary
Locust file contents (anonymized if necessary): this happens regardless of locust file, even with dummy locust file which doesn't do any requests, just task which creates random list

If you can, then have a look at fixing it your self @stanislawskwark . Or maybe @max-rocket-internet can have a look.

@cyberw I would be happy to fix it, I would need some guidance though. from what I checked until now - as I suspected shape failure is direct consequence of changes state - tick() is called only if state is STATE_INIT, STATE_SPAWNING or STATE_RUNNING.
from what I see first two calls are within milliseconds, and state is set to STOPPED, so my guess is that check_stopped is misbehaving on higher load and
       if (             not self.state == STATE_INIT             and not self.state == STATE_STOPPED             and all(map(lambda x: x.state != STATE_RUNNING and x.state != STATE_SPAWNING, self.clients.all))         )  evaluates to True because correct status was not set yet, could somebody confirm or give me some hint?

That code was written by @max-rocket-internet . Max, can you help out?

I was not able to reproduce it after applying  the change which I added in pull request - it seems to be solving the issue


Properties reserved JSON word?

From<EMAIL_ADDRESS>on April 30, 2013 16:38:21
Hi,
Execute the following code:
new JsonPath("[{"property":"test"}]").get("property");
Returns:
ArrayList with 1 value "test" = CORRECT
===============
Execute the following code:
new JsonPath("[{"properties":"test"}]").get("properties");
Returns:
LinkedHashMap with 2 values

"class" = "class java.util.ArrayList"
"empty" = "false"

NOT CORRECT
Is "properties" a reserved JSON, JsonPath,... word? I can't find any information on this.
Kind regards,
Geert
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=231

From<EMAIL_ADDRESS>on May 01, 2013 23:43:35
Hi,
This looks like a bug. "properties" is a reserved word in Groovy which is not handled correctly by REST Assured. For now please escape it manually, new JsonPath("[{"properties":"test"}]").get("'properties'");.
Thanks for reporting.
Status: Accepted

From<EMAIL_ADDRESS>on May 29, 2013 05:05:18
Ok so there seems to be two issues here. First off all "properties" is a reserved key word in Groovy and RA didn't escape it properly. How ever even though it's escaped properly it doesn't give the same result. Here's pure Groovy code:
def json = new groovy.json.JsonSlurper().parseText("[{"properties":"test"}]");
json.'properties'
Returns  [class:class java.util.ArrayList, empty:false]
How ever:
def json = new groovy.json.JsonSlurper().parseText("[{"property":"test"}]");
json.'property'
Returns [test]
So it seems to be an issue/feature in Groovy and I don't think it's worth (or even a good idea) to do anything about this.
How ever I will regard the issue as closed since "properties" is now automatically escaped in RA.
Status: Fixed


PR to address #98 and rework withParametersAssignable

Split withParametersAssignable into

withParameterAndSubclasses which finds methods with parameters matching a class and any of its subclasses (this replicates pre-v0.9.9 behaviour)
withParameterAndSuperclasses which locates superclasses (this replicates the post-v0.9.9 behaviour).

good idea, but the naming is not clear enough.

Do you have any suggestions on what they should be renamed to?

@ronmano any suggestions on method names for this?

I'm using 0.9.10 and also expected different behaviour:
Set<Constructor> constructorsAcceptingFruitTypes = getAllConstructors(
    classes,
    withParametersAssignableTo(Fruit.class));
// constructors accepting Fruit, Orange, Apple, Pear

how about adding withParametersAssignableFrom, e.g.:
Set<Constructor> constructorsAcceptingAnApple = getAllConstructors(
    classes,
    withParametersAssignableFrom(Apple.class));
// constructors accepting Apple, Fruit, Object

Now that I read back issue #23, I see this idea is exactly mentioned in https://github.com/ronmamo/reflections/issues/23#issuecomment-99437249:

Apple is assignable to Fruit: Fruit fruit = (Apple) apple;
Fruit is assignable from Apple: some use-case as above, but mentioned differently

PS: I am aware this PR is almost a year old, but nevertheless wanted to suggest this

@jamieps @fibbers sorry I was too lazy, and it seems outdated now... please resubmit if interested 🥇

@ronmamo I'm sorry but the PR that addresses this has the to/from logic backwards... @fibbers is totally right in his assessment of what "to" means and what "from" means. If there is not a PR open already to reverse this, I may open one myself.


Add support of grouped stacked bar charts

I plot the power consumption of my tracked devices daily, monthly, ...
I would love to add another bar next to the stacked bar with my total consumption so I can get an idea of how much energy usage I am not tracking.
Apex Charts should already support this with stacked groups.
A older issue also mentioned this #550.
Describe the solution you'd like
Support for grouped stacked columns would be splendid.
Describe alternatives you've considered
I tried two charts next to each other but its just not the same.
The y axis scale is not the same.

Not stale...

See this rejected pull request (I have the exact same scenario):
https://github.com/RomRider/apexcharts-card/pull/711
Meanwhile you can use a different Y axis ID for the total and you'll get what you want:
IMPORTANT: You must have a different chart style for total so that this works (e.g. use bars for current series and line or area for total)

Grouped stacked colums are not supported on time series in apexcharts.js.


Integrate with Spring Boot Admin

Make a visual link between the microservices and their Spring Boot Admin counterpart.
Optionally have both applications in the same Spring Boot app by using two annotations at the same time.

Moved because of https://github.com/ordina-jworks/microservices-dashboard-server/issues/82


Error during rancher deployment: failed to generate datastore_crypto_key (randBytes not defined)

I apologize that I didn't have the presence of mind to open this issue when I still had the error and stack trace in front of me, but I figured I'd at least open it in retrospect.
Environment<EMAIL_ADDRESS>Helm version: 3
Chart version: 0.80.0
Problem
I didn't define datastore_crypto_key in values.yaml so it fell back to conf/datastore_crypto_key.yaml. For some reason it failed to generate the key, error message randBytes is not defined.
Solution
I worked around it by generating it myself using openssl rand -hex 32 and defining it in datastore_crypto_key.
Hopefully this helps someone!

In order for the datastore_crypto_key generation to work, you need to use helm v3.5.0 (released January 2021) or newer.
The datastore_crypto_key generation was carefully crafted to match how StackStorm's st2-generate-symmetric-crypto-key script generates it.
If you can't use the helm-based generation, then please follow this section in the docs to generate a key:
https://docs.stackstorm.com/datastore.html#securing-secrets-admin-only
sudo st2-generate-symmetric-crypto-key --key-path datastore_key.json

And then put the contents of datastore_key.json in your values at st2.datastore_crypto_key.
NOTE: you should not need to run openssl rand ... to do this. openssl rand ... is incomplete and probably will not work in when StackStorm tries to load it to encrypt and decrypt keys.
You can do this on any ST2 installation, including in a vagrant box.

My notes from researching this:
Helm gets the randBytes function from the sprig library: https://masterminds.github.io/sprig/crypto.html#randbytes
Sprig got the randBytes function in v3.2.0 (from https://github.com/Masterminds/sprig/pull/253)
Helm v3.5.0 was the first version to use sprig v3.2.0: https://github.com/helm/helm/blob/v3.5.0/go.mod#L10
Helm v3.4.2 used sprig v3.1.0: https://github.com/helm/helm/blob/v3.4.2/go.mod#L10

The minimum version is documented in the readme:
https://github.com/StackStorm/stackstorm-ha/blob/e4b72dc2162824a64ec8150d4e8362c39e645e29/README.md#L12-L14

I'm closing this for now. If you were using v3.5 or greater, please reopen and let's diagnose further.


Add method for the components to skip constraints generation

For some cases, we only need the result of the component, but not its constraints or assignment rows.
We need to have a way to get the result of component's evaluation but without any constraints or rows apart from the rows for the output variables.
Suggested solution:

create an additional generate_assignments function (preferably - with the same name, as a template specialization of the original one), which has the same interface as the original one but does not generate any assignment row. It must only generate and seed the result variables with the correct value.
the same has to be done for generate_gates, generate_copy_constraints and any other interface-function. By interface functions, I assume functions, which exist for every component and are being used by the user. We do not need to do that work for internally-used functions.
patch the result_type structure so it would also depend on a (preferably - template) parameter. This structure currently returns a link to a cell in the table, and if we skip rows - this link becomes incorrect.

We can add skip parameter for current functions generate_assignments, generate_gates, generate_copy_constraints with default value FALSE.
generate_assignments(..., skip = false) {
    if (skip) {
        return empty_generate_assignments(...);
    }
    ...
}


Sui Wallet should export window.suiWallet object again

One month ago, our project AnimeSwap connects Sui Wallet vai window.suiWallet
The latest Sui Wallet only support SDK connection with React framework, its not developer friendly.
Not all builders building with React and want to install the SDK.
Martian Wallet can connect with window.martian.sui its nice for developers
We have no way to connect Sui Wallet any more
Why remove window.suiWallet, very bad for developers

We support the Wallet Standard for all wallet interactions. If you don't use React, you can use the Wallet Kit Core, or an ecosystem project that supports the wallet standard.


UDP support?

Was available in v14, removed in v15, will it come back?
(Using v14 right now for a torrent tracker, was about to refactor to v15)

You mean Unix sockets?

Nope, I mean adding UDP / Datagram sockets to the event loop so I get events from both TCP (HTTP) and UDP sockets from the same uWebSockets event loop.
Then I can read the actual data from the UDP socket myself using recvmsg.
Was possible like so in v14:
bool ret = h.listen<HandleUDP>(
			host,
			port,
			nullptr,
			uS::ListenOptions::REUSE_PORT | uS::ListenOptions::LISTEN_UDP,
			(uS::NodeData *)(uWS::Group<uWS::SERVER> *)hub,
			nullptr);

HandleUDP(uS::Socket *s;) {
    uv_os_sock_t fd = s->getFd();
    int recvd = recvmsg(fd, msg, flags);
}

I'm pretty sure this never was a thing, anyways..
You want to use us_poll from uSockets to just add your FD, or if that is not possible, just straight up use libuv they have UDP sockets.

Could be that I hacked support into uWebSockets at the time and forgot, thanks for the tip with us_poll will look into it.
I guess the issue is probably in the wrong repo anyways, if anything then uSockets should have support for opening (possibly reading/writing) UDP sockets.


archLifecycleVersion = '2.2.0-rc2' version need to be corrected

it should be '2.2.0-rc02' not 'rc2'. It's giving build error.

Not using RC anymore. Fixed meanwhile


https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2024/p3347r0.pdf

          https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2024/p3347r0.pdf

Originally posted by @wanghenshui in https://github.sheincorp.cn/wanghenshui/cppweeklynews/issues/121#issuecomment-2294820825

https://mit-public-courses-cn-translatio.gitbook.io/mit6-824/lecture-09-more-replication-craq/9.7-lian-fu-zhi-de-pei-zhi-guan-li-qi-configuration-manager


Internal Server Error when requesting joint WG session for IETF-114

Describe the issue
Requested a WG session for manet at IETF 114; I thought I had asked for this to be a joint session with babel and roll, but it was pointed out to me that this information was not showing up. I tried to edit the request by once more specifying that this was to be a joint session of three WGs as mentioned, but then when I hit the save button, I got an Internal Server Error message.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

Closing this as OBE. See also #6146


In ArcFace your classnum is 51332, but the advice data emore is 85742.

which data is 51332, or my understanding is wrong.

@eric-tc @foocker Hi, do you know the reason?

face_emore's classnum is 85742?


Dialog-flow-fulment agent inconsistency in sending Texts back to client

Been using the library for over 2 weeks, and it was working perfectly until today. It sometimes sends a response to messenger bot, sometimes it doesn't. I can not find the difference between the 2.
From the console, all the data its supposed to return, shows its coming back properly from CloudFirestore but sometimes, for some reason, the agent.add(new Text("")) is not invocked
Using firebase functions as webhook
Any help will be greatly appreciated

It appears its Dialogflow's 5 sec timeout, vs Firebase Functions cold start, trying to find ways around this by adding follow up intents


feat: update to latest key-protect module

Description
Upgrade the key-protect module to the latest version to enable metrics on the instance via the provider and prepare for the removal of the RestAPI provider
SKIP UPGRADE TEST due to removal of the RestAPI resource block, no actual impact from removal of this resource
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[x] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
use ibm provider to enable metrics instead of restapi provider. Also exposed new output key_protect_instance_policies

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@MatthewLemmond The upgrade test is being skipped. Can you create a new PR to allow upgrade test not to be skipped?

Okay, creating a new PR now.


Use workload cluster packages that avoid package context manipulation

This reduced the flakiness of the cluster creation and package approval process.

/hold
Wait until https://github.com/nephio-project/nephio-example-packages/pull/42 merges and we tag that package before merging this.

Other PR has merged and I tagged those packages.
/hold cancel

@vjayaramrh @electrocucaracha @henderiw I think this one is the last one that has to merge for the e2e README to work.

The e2e check is giving false positives, I'll try to test it locally until the fix is merged, but I think it is ok to approve it
/lgtm

@vjayaramrh @electrocucaracha @henderiw I think this one is the last one that has to merge for the e2e README to work.

Will take a look and try it out


Only use ModelViewSet ordering if it is explicitly set

Fixes #11165
This PR addresses renaming default_ordering to plain ordering and not passing "None" from the viewset to the index view.
I think most other ModelViewSet parameters behave sensibly with respect to overriding their counterparts set on the IndexView. If you see any others that should be addressed, how about creating a new issue and feel free to assign it to me.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[x] For new features: Has the documentation been updated accordingly?

@gasman OK I removed the commit that renamed default_ordering. This PR now only affects what happens if the ViewSet does not have an explicit ordering to send to the IndexView.

I marked this as 6.0 not because the change needs to get into the 6.0 release but because I want a resolution to the query args ordering test not passing.

Hi @cnk - just had a quick look, and it looks like:

test_custom_order_from_from_viewset is failing because ordering is being set on the IndexView here, and this should be default_ordering
test_custom_order_from_query_args is failing because the ordering of strid as passed in the URL is  being rejected as an invalid ordering by get_valid_orderings. By default, only fields that exist as columns in the listing are accepted as valid orderings, so adding strid as a column should hopefully fix this.


[Feature] Create a default shipment when creating SO

Related to #2970
Creates a default shipment with reference 1 when creating a new sales order.

@maksimstojkovic this should not be handled by the front-end. It should be performed by the database, when a new shipment is created

Amazing!

Let me know if you think True is a more intuitive default, or whether the feature should be opt-in.

I think false is a good default - it preserves the current behaviour
To finalize this, we'll need to add a unit test for the new functionality - in particular test that the setting changes what happens when a new SalesOrder is created.
https://github.com/inventree/InvenTree/blob/master/InvenTree/order/test_sales_order.py

Nice work :) please address PEP style checks and I'll merge in.
Note that you can run style checks locally with invoke style to save some time

Thanks for the contribution @maksimstojkovic

Congratulations on the successful front- and backend merge @maksimstojkovic !


Address watch descriptor reuse

From the inotify man page:

When a watch descriptor is removed by calling inotify_rm_watch(2) (or because a watch file is deleted or the filesystem that contains it is unmounted), any pending unread events for that watch descriptor remain available to read.  As watch descriptors are subsequently allocated with inotify_add_watch(2), the kernel cycles through the range of possible watch descriptors (0 to INT_MAX) incrementally. When allocating a free watch descriptor, no check is made to see whether that watch descriptor number has any pending unread events in the inotify queue.  Thus, it can happen that a watch descriptor is reallocated even when pending unread events exist for a previous incarnation of that watch descriptor number, with the result that the application might then read those events and interpret them as belonging to the file associated with the newly recycled watch descriptor.  In practice, the likelihood of hitting this bug may be extremely low, since it requires that an application cycle through INT_MAX watch descriptors, release a watch descriptor while leaving unread events for that watch descriptor in the queue, and then recycle that watch descriptor.  For this reason, and because there have been no reports of the bug occurring in real-world applications, as of Linux 3.15, no kernel changes have yet been made to eliminate this possible bug.

I think a solution to this problem would be to force the user to read the full event queue before adding another watch. In principle, it should be relatively straight-forward to enforce this at compile-time, but that would require us to rethink Inotify's API.

Thinking in terms of implementing futures (see #49), is it possible to continuously poll for watch events in a separate thread and return them via a future?

Certainly possible, but I don't think it would be an advantage. Even if you polled in short intervals, this issue would still be present (although less likely, maybe). You'd need some kind of synchronization between the main thread and the polling thread to fully solve the problem, and at that point this seems like way more complexity than is warranted. In addition, I don't think it's possible to prevent this issue at compile-time, if multiple threads are involved.
Generally speaking, I can't imagine circumstances where I would accept a solution to a problem that would require additional threads. That kind of thing would be way too heavyweight for what is supposed to be a simple wrapper.


object_name should be optional and random by default

When generating the file identification string we now allow to pass a unique sting to avoid collision when using multiple processes.
This is great but by default currently a constant string is being used ('object'). By default a random, unique string should be used, e.g. , <hash(\time)> etc.
This way the advanced user can still pass a meaningful name for debugging purposes. But the general user will never need to worry about this while still not having problems with name collision.

This issue of file naming is specific to the command line interface so I think we can close it.


Sorry, my bot code is having an issue.

I am currently getting this as a reply when testing my bot in the Emulator. Ive checked the bot app logs and there arent any errors. What kind of issue would this be?
Here is the part code that is causing this:
private async Task LogIn(IDialogContext context)
        {
            string token;
            if (!context.PrivateConversationData.TryGetValue(AuthTokenKey, out token))
            {
                context.PrivateConversationData.SetValue("persistedCookie", ResumptionCookie);

                Activity loginActivity = new Activity();
                loginActivity= loginActivity.CreateReply("Logging In");
                Trace.TraceInformation($"loginActivity Type : {loginActivity.Type}");
                Trace.TraceInformation($"loginActivity Action : {loginActivity.Action}");
                Trace.TraceInformation($"loginActivity Text : {loginActivity.Text}");

                await context.PostAsync(loginActivity);
                context.Wait(MessageReceivedAsync);
            }
            else
            {
                context.Done(token);
            }
        }

While this works :
private async Task LogIn(IDialogContext context)
        {
            string token;
            if (!context.PrivateConversationData.TryGetValue(AuthTokenKey, out token))
            {
                context.PrivateConversationData.SetValue("persistedCookie", ResumptionCookie);
                string fbLogin = $"Go to: {FacebookHelpers.GetFacebookLoginURL(ResumptionCookie, FacebookOauthCallback.ToString())}";

                await context.PostAsync(fbLogin);
                context.Wait(MessageReceivedAsync);
            }
            else
            {
                context.Done(token);
            }
        }

cant i pass an Activity type to PostAsync, isnt Activity type an IActivityMessage ?
im trying to use cards in dialog.

We added that error message so that it was obvious that the http request to post a user's message to the bot was failing with an unhandled exception:
https://github.com/Microsoft/BotBuilder/blob/master/CSharp/Library/Dialogs/DialogTask.cs#L443
even if all of the previous messages were successfully sent.  Do you see your reply message?

thanks got to see it with the debugger attached , thanks :)
"   at Microsoft.Bot.Connector.ConversationsExtensions.ReplyToActivityAsync(IConversations operations, Activity activity, CancellationToken cancellationToken)\r\n   at Microsoft.Bot.Builder.Dialogs.Internals.AlwaysSendDirect_BotToUser.d__4.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at Microsoft.Bot.Builder.Dialogs.Internals.DialogContext.d__10.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()\r\n   at StuckoBot_Application.SimpleFacebookAuthDialog.d__7.MoveNext() in F:\WORK\00 StuckoDev\StuckoBot Application\StuckoBot Application\StuckoBot Application\SimpleFacebookAuthDialog.cs:line 224"


Missing android createCalendar

Missmatch between flutter and android versions. Flutter has createCalendar method, while android version does not have it. Therefor it throws exception when creating a calendor

Create calendar is not supported in the current version. Not sure where you got the method from, but a pre-release has been published here (v3.1.0-dev.11079) and it supports creating a calendar.
You can have a look at the pre-release for now, a stable version will be released later.

v3.1.0 has been published, you should be able to create calendars in both platforms. Feel free to open a new issue ticket if you face any issues


remove hard coded deployment for RUN_JAR

@jlhughes this looks wrong, why is build.js hard coded specific deploymenets? Users should just use -J as needed

See https://github.com/kgrgreer/foam3/pull/3640


Redundant IsDevelopment check in the startup of the TodoTemplate project template

Currently, there is a redundant develop env check in the Middleware.cs file of the startup in the TodoTemplate:

the ideal change should be something like this:

the line is in the following link:
https://github.com/bitfoundation/bitplatform/blob/fcb83f2c6da2a1b42c792497b39f977426e96917/src/Templates/TodoTemplate/Bit.TodoTemplate/src/Server/Api/Startup/Middlewares.cs#L19

Hey,
I am Lexi and I am new to contributing to open source projects.
I would like to try and work on this issue if thats fine? :)

Hey, I am Lexi and I am new to contributing to open source projects. I would like to try and work on this issue if thats fine? :)

Sure, why not? FULL POWER AHEAD! (":

Perfect, thanks!
I will try my best :)

Morning,
I just fixed the code and would like to test it but I don't find any testing tutorial in the README's where they are said to be. Locigally since the IsDevelopment is already checked earlier the code should still work, but I don't know if thats the right way to think. What should I do?

Thanks for your efforts.
This change doesn't require much of a test. you can just try to build the project following this doc: https://github.com/bitfoundation/bitplatform/blob/develop/docs/how-to-build.md
Or just push the PR and our CI will build and run the tests for your changes.


Fix typo

Fixes #45451

@Potapy4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.


Simple browser debugger

I loved the embedded "Simple browser", but it is not useful for me because it doesn't allow me to debug my Javascript/Typescript code.
So I would like to be able to create a configuration in the launch.json file like this...
{
"name": "Run Simple Browser",
"type": "simple-browser",
"request": "launch",
"url": "http://localhost:3000",
"webRoot": "${workspaceFolder}"
}
run it and be able to do my development and debug my code without leaving vscode.

If you run the command to open DevTools for webviews does this give you access to the scripts?


[HW] RIYA TERI

Name: RIYA TERI
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/riyateri/
Attach the homework screenshots below for both step II and step III:

Looks great Riya, thank you.

Hey RIYA TERI, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/Peox87ZcStiA_lV0lu0wcw

Thank you @SonicDMG and @HadesArchitect


Fixes want_docker checks

With -n and -z checks want_docker is always True because in the string form,
any integer is a valid string and so the checks do not work as expected.
Making the condition more explicit should help.

@sayalilunkad I'm not sure it fixes the core problem: we do enter the code path for want_docker as expected (otherwise, there'd be no docker compute node). The problem is that in the past, the docker compute node was node2, and now it's node1, which leaves some room for a kvm compute node. And that's what is causing the tempest misconfiguration and then failure.

@vuntz @aplanas Indeed the problem is with two compute roles that are being assigned when it should strictly be one when using docker. The problem is with the knife search call here https://github.com/SUSE-Cloud/automation/blob/master/scripts/qa_crowbarsetup.sh#L557 that returns all the nodes running SLE12 which includes the dashboard node. We need another search specific for compute nodes.

Will close this PR. Please check this fix https://github.com/SUSE-Cloud/automation/pull/1333


Documentation in README emptied with Commit 0ba7470

Hi,
I just saw that with commit 0ba7470 the API documentation was removed from README.md. Most likely this is intentional since you are switching to Promises with Version 5.0.0? It would be awesome if you could re-add the documentation for developers who are using a version < 5.0.0.
Thanks.
Take care,
Simon

What do you want the documentation to keep from the outdated version? Why don't you use version 5 which is self-documenting with the types? I'm sorry, but I have to delete the deprecated code.
You can look at the documentation of version 4 in the commit that you tell me.

@botika - not everyone uses TypeScript or an IDE, and likely don't want to have to read through the code to figure out how it works before downloading the library.  Some basic examples in the README would probably help get people to use node-cache-manager.  The README, as it is now, tells users very little about what the library does.

I'm quite sure I'm not the only one currently using a < 5.0.0 Version, since migrating to a new API without proper documentation needs time and testing. I also agree with Bryan – not everyone is using TypeScript in their projects.
I understand that you don't want to maintain the deprecated examples in the Readme. How about adding a paragraph pointing users to the old version of the Readme? This will mitigate the time one is spending searching for the deprecated documentation.

https://github.com/node-cache-manager/node-cache-manager/commit/9f7c46ec6142e068ea6ffe790c2e6634fda9f557


Fixed some URLs in documents.

Change from http:// to https:// in some of the htmls to avoid incorrect link.

Wow, you went through so many URLs for this PR — thanks!


Add BasicTypes overload operators

Add overload operators for Vector3 and Vector4.

Includes +, -, *, /, < and <= operations.
Add ThrowLuaError in LuaVM to throw when dividing by zero.
Formats Lua usertypes in Scripting for easier maintenance.

I also think that the ovetloads should be added to RED4ext.SDK and used from there, wouldnt add more specializations into CET when the base is that, just makes things more cumbersome to maintain.

Looks good to me, I don't see how this could be made more performant, the compiler will do the SIMD optimization and calling the game function has a cost.

Unsure if this will get much SIMD optimization and unsure what is the cost of building the new object... Considering this was meant to shadow existing game functions anyway, I at least think it would be better to reuse them. I'll pull my review if you want to merge it but dont find this particularly good personally, reimplementing things...
Also think this should be part of RED4ext but that was another point, not performance-related.
But

Didnt mean to send it... At least I would change the comparison operators to take into account some form of epsilon if nothing else. Same for other comparisons, is not the best to compare floating points directly like this.

Nah, ended up editing my comments anyway to add few points... Too late to apologise for multiple messages...
Dismissed my review, do what you must based on my comments I guess.

About performance, I might write a tiny benchmark to compare time of execution between native methods and Lua implementation. See if there is a significant difference and, I guess, avoid any assumption on this matter. Maybe even detect different results...
About epsilon comparison, I was wondering if it should be used. Currently, it is not used with operator ==. But as you point out, it might be within native implementation. Hence, it might be better to rely on native implementation only.
About RED4ext.SDK implementation, if it can be directly implemented there and cascade to Lua without much effort, well yes. The though didn't cross my mind when talking about this missing "feature" in Lua. I'm wondering, currently files for Vector3 and Vector4 are generated. Would it be possible to write operators there without messing the code when generating them?

Here a tiny-benchmark.txt you can rename with .lua extension and try yourself.
First test is most likely too simple regarding floating point precision... However difference between overload operators and call to native functions speak for itself IMO (loop of 100 000 iterations). Of course it is even worse when you get and call global function without storing a reference before running computation. So I didn't bother to let it in attachment. Elapsed time here is in seconds:

Feel free to improve this benchmark if it is too simple to be representative or point out any mistakes.

Here a tiny-benchmark.txt you can rename with .lua extension and try yourself.
First test is most likely too simple regarding floating point precision... However difference between overload operators and call to native functions speak for itself IMO (loop of 100 000 iterations). Of course it is even worse when you get and call global function without storing a reference before running computation. So I didn't bother to let it in attachment. Elapsed time here is in seconds:

Feel free to improve this benchmark if it is too simple to be representative or point out any mistakes.

See discussion on RED4ext.SDK.

I'll update this PR after this is done https://github.com/WopsS/RED4ext.SDK/pull/140.


Remove logging configuration

in client.py logging.basicConfig is called on line 19, this configures the root logger with a default stderr stream handler.  By doing this, it means any other calls to basicConfig are ignored.  This leads to users having to call basicConfig before importing aonvif or other work arounds.  In my case, I'm not using basicConfig, so I end up with duplicate log entries in my terminal just from importing the library.   Please remove this or it might be better to move it to the CLI startup.  That way there still is default logging for using the CLI, but removes the logging side effects from importing.

@ottowayi thanks for the report. Colud your please create a PR with the changes?


btcposbal2csv.py doesn't extract segwit

btcposbal2csv.py doesn't extract segwit addresses, for example:
Public Address bc1 P2WPKH: bc1qetrqrygl7t0fn6hrdmuug3rrp8vjmum8fd8rsk
Public Address bc1 P2WSH: bc1q26swzzjmqgwvcjxplktpyj9xpkzpv8kgs7t545ku6s7v6fl0xt3slnx8k9

Yes, currently it does not support segwit address extraction. I'm not planning to add support at the moment. I would accept PR with the support.

Hi, I've added support for bech32 addresses in a repository located here: https://github.com/who-biz/chipsposbal2csv.
It has been modified to generate addresses with a different network byte, but the changes are easily portable if @Jolly-Pirate is still looking for a solution.

PR'ed changes for BTC network.


[web] A multiline textfield doesn't bring the cursor to the mouse click location. Need for TextArea widget

The following way of having a multiline textfield doesn't work well in a web app. it doesn't put the cursor at the clicked position and also doesn't have the resize option. I think there's a need for a TextArea widget.
TextField(
          keyboardType: TextInputType.multiline,
          maxLines: 10
);

❯ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v1.15.18-pre.1, on Mac OS X 10.15.2 19C57, locale en-GB)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[!] IntelliJ IDEA Community Edition (version 2019.3.1)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] VS Code (version 1.43.2)

Scanning for devices is taking a long time...⣯Error 1 retrieving device properties for ro.product.cpu.abi:
error: closed

[✓] Connected device (3 available)

! Doctor found issues in 2 categories.

Hi @nateshmbhat
see https://github.com/flutter/flutter/issues/51640#issuecomment-595414642 and follow up on https://github.com/flutter/flutter/issues/33523,
Closing as duplicate,
if you disagree please write in the comments and I will reopen it
Thank you


plugin.json: allow forwards-compatibility

Grafana Cloud is on 10.x

I also couldn't use Sneller in the cloud environment with the current plug-in:


[DS-6655][fix] remove zkmonitor

remove zkmonitor,resolve the conflict
this pr close #6655

Codecov Report

Merging #6788 (f139564) into dev (088a5a4) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #6788      +/-   ##
============================================
+ Coverage     41.82%   41.85%   +0.02%
  Complexity     3617     3617
============================================
  Files           641      641
  Lines         25897    25897
  Branches       2795     2795
============================================
+ Hits          10832    10838       +6
+ Misses        14083    14078       -5
+ Partials        982      981       -1

Impacted Files
Coverage Δ

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
83.05% <0.00%> (+1.69%)
:arrow_up:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
54.54% <0.00%> (+7.27%)
: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 088a5a4...f139564. Read the comment docs.


Issues building OpenCV 4.5.5 (OpenGL and Python)

Building OpenCV 4.5.5 (and 4.5.4 for that matter) with ./scripts/docker_build_opencv.sh seems to come with the following problems:
/usr/bin/ld: ../../lib/libopencv_highgui.so.4.5.5: undefined reference to `cvSetOpenGlDrawCallback'
/usr/bin/ld: ../../lib/libopencv_highgui.so.4.5.5: undefined reference to `cvSetOpenGlContext'
/usr/bin/ld: ../../lib/libopencv_highgui.so.4.5.5: undefined reference to `cvUpdateWindow'
collect2: error: ld returned 1 exit status
make[2]: *** [apps/annotation/CMakeFiles/opencv_annotation.dir/build.make:90: bin/opencv_annotation] Error 1
make[1]: *** [CMakeFiles/Makefile2:6923: apps/annotation/CMakeFiles/opencv_annotation.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

If I build with -D BUILD_opencv_apps=OFF  to just omit the offending opencv_annotation , the same error will happen during the process of building some ROS package that uses OpenCV.
It seems like the workaround that actually works is to disable OpenGL and enable GTK for highgui: -D WITH_OPENGL=OFF -D WITH_GTK=ON.
Another issue later occurs on the python check at https://github.com/dusty-nv/jetson-containers/blob/master/scripts/opencv_install.sh#L69 when building the target Docker container. OpenCV 4.5.5 has python install destination at site-packages and not dist-packages.
OpenCV 4.5.5:
--   Python 3:
--     Interpreter:                 /usr/bin/python3 (ver 3.8.10)
--     Libraries:                   /usr/lib/aarch64-linux-gnu/libpython3.8.so (ver 3.8.10)
--     numpy:                       /usr/lib/python3/dist-packages/numpy/core/include (ver 1.17.4)
--     install path:                lib/python3.8/site-packages/cv2/python-3.8

OpenCV 4.5.0:
--   Python 3:
--     Interpreter:                 /usr/bin/python3 (ver 3.8.10)
--     Libraries:                   /usr/lib/aarch64-linux-gnu/libpython3.8.so (ver 3.8.10)
--     numpy:                       /usr/lib/python3/dist-packages/numpy/core/include (ver 1.17.4)
--     install path:                lib/python3.8/dist-packages/cv2/python-3.8

This seems to be a common issue with a workaround: https://github.com/opencv/opencv/issues/21359#issuecomment-1003005474
I build using ./scripts/docker_build_ros.sh --distro humble after running ./scripts/docker_build_opencv.sh. The Dockerfiles and install scripts have been modified to use the local opencv tar and not to fetch the one from the nvidia cloud.
Maybe this information will be of help to people to fix their problems and maybe some of these fixes can be pushed to the repo.

have you solved this?

have you solved this?

Yes, I've linked towards the solutions in the post above.


Retraining Aizynthfinder

Hi! So I tired retraining aizynthfinder on reactions sources from Maestro. We created reaction smiles format, put it through rxnmapper(atom mapping program) and then through rdchiral for the retro templates. I put it through preprocess_rollout and training. But when I used, it showed errors and wasn't able to process many of the compounds I put through(only able to process 99 of 395 compounds). I have all the necessary columns for the training file, and followed the documentation for training. Any input on why this might be happening?

Thanks for your feedback.
It is however impossible for me to say anything about your issue unless you provide more information: can you paste in an error trace? can you provide examples of exact input that is causing your issue and the exact command you were trying to use?

The error I received when running the batch processing after using our newly trained model and unique_templates was ChemicalParserException : Number of reactants provided does not match number of reactant templates is this a issue with the model or with the unique_templates ? Thanks

With the limited information you are providing, I can at most give educated guesses.
First assumption I make is that you when say you are "running the batch processing", you mean that you are running the tree search using the aizynthcli tool.
The exception you are seeing ChemicalParserException comes to the best of my knowledge from the application of a template onto a product. Since we are doing the application in the retro order, what the exception thinks is a reactants is actual the product. So this exception is raised when the number of products and the number of templated molecules are mismatched. Since I don't see how the number of products can be anything but 1 in the tree search, it must mean that you have templates that doesn't have 1 product.
To dig further into this from my side, you need to

Show me exactly what line of code where the exception is coming from
Give me some input data to look at, such as a SMARTS/SMIRKS template string and a molecule SMILES

Hi all,
Samuel has essentially covered it. The error originates from rdkit and not the model or tree search. The templates should have only one 'reactant' and 'n products' as they are retro templates.
The error occurs if you have specified the templates incorrectly. If it was me I would first check all the templates to ensure only one reactant in the retro template. It could be as simple as reversing all the templates you have.
Second if there are multiple species on either side of the reaction, there is something wrong with the template extraction you have used or you have not filtered your reaction dataset for reactions that lead to only one product (which is more likely).
Hope it helps,
Amol

You can also check your templates reproduce the data they were generated from by taking each of the templates and applying them to their product from which they were extracted. If you regenerate the reactants in the dataset then you can verify the templates work.

Thank you! That was exactly the issue we used a specific unimolecular reaction that after it was put through atom mapping and rdchiral, the reactions came out with multiple reactants which corrupted our unique_template file. Thanks again for all the help!


Add brazilian portuguese translation

On the first glance it doesn't look like there are very many differences (any?) to the pt-PT translation. Is this worth adding as another language that needs maintenance?

It's up to you

Are there significant differences in the translation?
If not, would you be willing to update the missing ones in pt-PT instead?

some words have a different meaning
example

Fair enough :).


Inactive taxa get featured at https://www.inaturalist.org/taxa

From https://forum.inaturalist.org/t/inactive-taxa-shouldnt-be-featured-on-taxa/16023
We should probably only feature active taxa here.
eg

This is manually controlled, so if you want to unfeature it, edit the taxon and uncheck the box.

I looked at the time and the box wasn't checked.

See also Cassi's comment in the forum


Update Get-SPOPublicCdnOrigins.md

Added a code example based on the example overhere: https://dev.office.com/blogs/office-365-public-cdn-developer-preview-release

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOPublicCdnOrigins.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx Dennis, appreciated.


perf (Group.Opposite): direct inheritance for algebraic classes

We change the instance construction pattern from Function.Injective to more direct inheritance.

!bench


gaeinfo sample does not deploy (flexible, flexible/java-8)

To reproduce,
mvn clean package deploy:appengine
Results:
Unable to stage app: GAE Flex compat applications need to depend on the GAE API jar, but it was not found in the WEB-INF/lib directory.
Expected:
Deploy to GAE
Scope:

AppEngine flex samples in flexible and flexible/java-8 do not deploy.
Standard sample in appengine/java-8 was able to successfully deploy.

Note: There is an open issue for adding E2E tests #7733

We no longer support flex compat runtimes. we need to delete these samples

fixed via #8160; is there anything else that needs to be cleaned out?

I reviewed our sample set and this bug should be complete.


palazzetti adapter added to sources-dist-stable.json

RE-CHECK!

Automated adapter checker
ioBroker.palazzetti

[ ] :heavy_exclamation_mark: [E506] More non translated in german or russian words found in admin/words.js. You can use https://translator.iobroker.in/ for translations
[ ] :eyes: [W400] Cannot find "palazzetti" in latest repository

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.palazzetti

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "palazzetti" in latest repository

Add comment "RE-CHECK!" to start check anew

Needed:

Link to the manufacturer or device webpage
Automated testing on Travis-CI.org is not enabled
ConnectionType is missing
A lot of the Objects do have the role "state" please check if you can find better roles for it, if not make proposals for new ones
main.js: onObjectChange is not used so could be removed

Comments:

Objects handling: Objects seems not to be created dynamicly, so you can define them in io-package.json at instanceObjects
Objects handling: Object names should be human readable. Example "control.pwr" -> better  "Power" or "Turn Power on"
Objects handling: You have labels defined that are related to the states, for this purpose could be used the common.states: {1:"1";2:"ON"}.
libraries could be placed in folder "lib". Makes it a bit cleaner.

@inapsis Did you saw the comments?

@inapsis any update?

@inapsis do you have an update?

RE-CHECK!

Automated adapter checker
ioBroker.palazzetti

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "palazzetti" in latest repository

Add comment "RE-CHECK!" to start check anew

@inapsis do you have an update? How should we proceed?

Hi @Apollon77, I have tried to adapt all your comments and needs and posted already a new version some weeks ago.
I am not sure if something additional needed to request a review again.

Ahh ok, I waited for a info back when done :-)
I had a short check and found one last small thing:

please add js-controller >=2.0.0 as dep in io-package (because you use adapter-core 2.4.0)

Beside that I just saw that all the states are set in parallel ... you maybe want to use also Async methods and promises here
Thank you for a fast check.

RE-CHECK!

Automated adapter checker
ioBroker.palazzetti

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "palazzetti" in latest repository

Add comment "RE-CHECK!" to start check anew

Hi @Apollon77, thanks for your fast reply...
I adapted the js-controller dependency as requested.
For all set states I have already used adapter.setStateAsync,... nevertheless the promise result is now processed additionally.


Exclude fields like Status and creationTimestamp

Issue #, if available:
Description of changes:
https://github.com/aws/eks-anywhere/issues/191
Exclude fields from getting written to the github repo if flux is enabled.
This pr takes care of removing these fields from the cluster, gitops, and oidc configs.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/assign @terryhowe


Add timeout on ep.write

Hi Alex
In my application (R&S FSW over usbtmc), timeout expired during a write to the EP. That's why I added a timeout on each writing.

This looks good. I wonder if this would address issues that I've seen with the R+S NRP8S power meter... Sometimes we see a timeout expired after sending some commands.

Hi All,
Why is variable timeout on class Instrument fixed to 1sec. In my case I use oscilloscope and it doesn't really help because, instrument has to do auto scaling of the signal and then return amplitude. If possible please add timeout as an argument.I have done it locally.


[Investigation] Investigate null move pruning behaviour

E.g. test behaviour when depth_reduction_factor = 0

Thorough analysis! Much appreciated.