Sync with SOLR doesn't work in large database

Hello! I have a serious problem with syncing solr + mongo. By an empirical way, I have found out that the problem is in mongo-connector.
Requirements:
CentOS: 6.2
MongoDB: 3.0.x
SOLR: 5.2.x
mongo-connector: 2.1

Briefly, I  have a mongo database (let's call it "db", the collections sum size is ~30GB). Several (six) collections have solr core and each core has mongo-connector. Solr cores and mongo-connectors are synchronized  in real-time.
(mongo-connector option --auto-commit-interval=0). On the host "host1.com" sync works well.
I did a dump of the "db" database  and moved it to "host2.com", where there was another database  - "db1" ("db1" also had a solr-process + mongo-connector for each core (same as "host1.com", that was synchronized in real-time). All worked).
I rolled a dump of "db". Now "host2.com" have 2 bases: "db" and "db1". I run  solr-process + mongo-connector for "db" - it didn't work (but bunch solr + mongo-connector for "db1" still was working).
I restarted  mongo-connector (of each core) for "db1" - synchronization stopped working, or worked for random cores with large intervals of time ( 10 minutes, and 30 minutes, and an hour, etc).
Restarting of mongo, solr-process and mongo-connector didn't help.
I tried to restart mongo-connector for one of cores in "host1.com" - sync for this core stopped working immediately.
Startup options for mongo:
mongod --storageEngine wiredTiger \
       --nojournal \
       --replSet rs0 \
       --dbpath / mongodb / data / \
       --fork \
       --logpath /mongodb/logs/mongod-tiger.log

Startup options for SOLR:
./solr/bin/solr -s /path/to/cores -p port

Startup options mongo-connector (a total of scripts - 6, but all scripts are similar. Difference in collection):
mongo-connector -m localhost: 27017 \
                -t http://host1.com:8983/solr/collection1 \
                -d solr_doc_manager \
                -n db.collection1 \
                -g collection1 \
                -o /logs/oplog_collection1.log \
                --logfile /logs/mc_collection1.log \
                --auto-commit-interval = 0

Thanks in advance. If you have any questions, please don't hesitate to ask me.

Hey @ilfroloff
When you say that mongo-connector stopped working, do you mean that it's running but not doing anything, or exiting immediately?
Can you please turn on verbose logging (-v) for mongo-connector and post the logs here (or link to them from your favorite text-paste site)?
Thanks

Hi @llvtt

When you say that mongo-connector stopped working, do you mean that it's running but not doing anything, or exiting immediately?

It's running but not doing anything.

Can you please turn on verbose logging (-v) for mongo-connector and post the logs here (or link to them from your favorite text-paste site)?

You can check logs here (for two different cores):
http://justpaste.it/mo8c (that part repeat any time): core not synchronized with collection
http://justpaste.it/mo8h (that part repeat any time): core synchronized once, but now isn't adding data to SOLR index
Thanks!

http://justpaste.it/mo8c (that part repeat any time): core doesn't synchronized with collection

This Exception message sticks out:
org.apache.solr.common.SolrException: java.io.FileNotFoundException: /srv/dev/cvision-dev/solr/cores/insights/insights/data/tlog/tlog.0000000000000000000 (No such file or directory)

I honestly don't know what this file is, but it might be complaining about a missing directory above it, i.e., /srv/dev/cvision-dev/solr/cores/insights/insights/data/tlog doesn't exist. This could also be a permissions issue.

http://justpaste.it/mo8h (that part repeat any time): core synchronized once, but now isn't adding data to SOLR index

These logs show that mongo-connector sent an "add" request to Solr with 1000 documents. It could be that these documents were successfully added, but that the Solr index hasn't been refreshed to make these additions visible. I see that you have --auto-commit-interval = 0 in the command line, but I think the syntax should be --auto-commit-interval 0. It's also a better practice to change the commit interval in solrconfig.xml anyway; see https://github.com/10gen-labs/mongo-connector/wiki/Usage-with-Solr#managing-commit-behavior

I honestly don't know what this file is, but it might be complaining about a missing directory above it, i.e., /srv/dev/cvision-dev/solr/cores/insights/insights/data/tlog doesn't exist. This could also be a permissions issue.

Yes, you are right the problem was in the index folder. Deleting of this folder, restarting and synchronization didn't help me.

These logs show that mongo-connector sent an "add" request to Solr with 1000 documents. It could be that these documents were successfully added, but that the Solr index hasn't been refreshed to make these additions visible.

May be you know why index wasn't updated?

I see that you have --auto-commit-interval = 0 in the command line, but I think the syntax should be --auto-commit-interval 0

Sorry, is was just wrongly copied :)
In my run script of mongo-connector, a command line in actually --auto-commit-interval=0 (is the same as --auto-commit-interval 0).
New details: I have tried to create two different mongo-procceses on the same physical server (ports 27017 and 27018 with different folders of data). For each mongo-proccess I have created a database:
db (an empty database, with all needed collections for solr-cores but without records) - for 27017,
db1 (a dump with capacity 30Gb) - for 27018.
I got the same result as that I have described here https://github.com/10gen-labs/mongo-connector/issues/298#issue-97928915 but after some unsuccessful tries to run mongo-connector for db1 (dump of 30Gb), synchronization started to work (please note, the whole collections was synchronized, index was updated immediately and after restarting of any of mongo-connectors everything continued to work).
To make synchronization work I added several records into 3 different collections.
For experiment, I have created totally the same environment (solr + mongo + mongo-connectors) with the separate mongo-proccess (port 27019, different folder), database db2 (initial dump with capacity 30Gb), on this physical server. I have run synchronization for each core - it run just for the biggest collections but while adding data into mongo, solr didn't reindex the changes. Restart of mongo-connector didn't help for mongo port 27018. The most interesting thing is that after restarting mongo-connectors for db1 they stopped to be synchronized in real-time (any changes in collections related to cores, didn't change the index of solr) but started to work in about hour. For db I didn't check (I would have a possibility on monday).
Could you please to answer my several questions:

How synchronizaiton solr + mongo works on mongo-connector level. Could you please describe in details the whole process of getting from mongo and adding changes in solr index, or may be you could advice where I can find documentation for mongo-connector and solr_doc_manager.
P.S. If I could see from --verbose logs it's all was caused by oplog that was initially empty for while rolling up the dump.
Is there are a full check of the initial state of the solr-core index and related mongo collection? If there is, how could I know the period of time during which this check is performed and the complit synchronization of collections and solr-cores?

These pages will provide the best information on the internals of Mongo
Connector:

https://github.com/10gen-labs/mongo-connector/wiki/System-Overview
https://github.com/10gen-labs/mongo-connector/wiki/Usage-with-Solr

https://github.com/10gen-labs/mongo-connector/wiki/Writing-Your-Own-DocManager

There is no check of the initial state of Solr cores. mongo-connector
assumes that the cores are either empty (and no oplog progress file, does
collection dump), or are filled with documents from MongoDB (and continue
replication from the oplog timestamp file).

On Fri, Jul 31, 2015 at 2:43 PM, ilfroloff<EMAIL_ADDRESS>wrote:

I honestly don't know what this file is, but it might be complaining about
a missing directory above it, i.e.,
/srv/dev/cvision-dev/solr/cores/insights/insights/data/tlog doesn't exist.
This could also be a permissions issue.
Yes, you are right the problem was in the index folder. Deleting of this
folder, restarting and synchronization didn't help me.
These logs show that mongo-connector sent an "add" request to Solr with
1000 documents. It could be that these documents were successfully added,
but that the Solr index hasn't been refreshed to make these additions
visible.
May be you know why index wasn't updated?
I see that you have --auto-commit-interval = 0 in the command line, but I
think the syntax should be --auto-commit-interval 0
Sorry, is was just wrongly copied :)
In my run script of mongo-connector, a command line in actually
--auto-commit-interval=0 (is the same as --auto-commit-interval 0).
New details: I have tried to create two different mongo-procceses on the
same physical server (ports 27017 and 27018 with different folders of
data). For each mongo-proccess I have created a database:
db (an empty database, with all needed collections for solr-cores but
without records) - for 27017,
db1 (a dump with capacity 30Gb) - for 27018.
I got the same result as that I have described here #298 (comment)
https://github.com/10gen-labs/mongo-connector/issues/298#issue-97928915
but after some unsuccessful tries to run mongo-connector for db1 (dump of
30Gb), synchronization started to work (please note, the whole collections
was synchronized, index was updated immediately and after restarting of any
of mongo-connectors everything continued to work).
To make synchronization work I added several records into 3 different
collections.
For experiment, I have created totally the same environment (solr + mongo

mongo-connectors) with the separate mongo-proccess (port 27019, different
folder), database db2 (initial dump with capacity 30Gb), on this physical
server. I have run synchronization for each core - it run just for the
biggest collections but while adding data into mongo, solr didn't reindex
the changes. Restart of mongo-connector didn't help for mongo port 27018.
The most interesting thing is that after restarting mongo-connectors for
db1 they stopped to be synchronized in real-time (any changes in
collections related to cores, didn't change the index of solr) but started
to work in about hour. For db I didn't check (I would have a possibility on
monday).

Could you please to answer my several questions:

How synchronizaiton solr + mongo works on mongo-connector level. Could
you please describe in details the whole process of getting from mongo and
adding changes in solr index, or may be you could advice where I can find
documentation for mongo-connector and solr_doc_manager.
P.S. If I could see from --verbose logs it's all was caused by oplog that
was initially empty for while rolling up the dump.
Is there are a full check of the initial state of the solr-core index
and related mongo collection? If there is, how could I know the period of
time during which this check is performed and the complit synchronization
of collections and solr-cores?

—
Reply to this email directly or view it on GitHub
https://github.com/10gen-labs/mongo-connector/issues/298#issuecomment-126819119
.

@ilfroloff Hello! Did this answer your question?

Hello! I'm going to close this issue due to inactivity but if the problem persists please feel free to reopen or open a new issue. Thank you!


Implement the Adventure API

Adventure is a server-side user interface library for Minecraft which is fast becoming the standard user interface system for many Minecraft related projects. It is currently implemented in Velocity, Sponge 8 and Paper.
Moving to use Adventure has the following benefits:

Brings Minestom into step with other server implementations.
Allows developers to move projects to Minestom more easily.
Reduces all user interface maintenance in Minestom.
Future-proofs Minestom developers against upcoming changes (for example, the upcoming Title changes).
Allows developers to use the advanced API that Adventure provides.

This has been discussed in Discord many times, and as far as I am aware there has been no change in stance on the subject.
I think it's likely that this is a waste of time at the moment. That being said, this may be a helpful reference.

I personally still do not get the advantage over a wrapper as the one @mworzala did.
But well I get the concern and the fact that it will help to reduce maintenance

Closed with #157


mgmt, prepare 2.14

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check for com.azure.resourcemanager:azure-resourcemanager
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager

API change check for com.azure.resourcemanager:azure-resourcemanager-appplatform
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-appplatform

API change check for com.azure.resourcemanager:azure-resourcemanager-appservice
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-appservice

API change check for com.azure.resourcemanager:azure-resourcemanager-authorization
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-authorization

API change check for com.azure.resourcemanager:azure-resourcemanager-cdn
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-cdn

API change check for com.azure.resourcemanager:azure-resourcemanager-compute
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-compute

API change check for com.azure.resourcemanager:azure-resourcemanager-containerinstance
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-containerinstance

API change check for com.azure.resourcemanager:azure-resourcemanager-containerregistry
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-containerregistry

API change check for com.azure.resourcemanager:azure-resourcemanager-containerservice
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-containerservice

API change check for com.azure.resourcemanager:azure-resourcemanager-cosmos
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-cosmos

API change check for com.azure.resourcemanager:azure-resourcemanager-dns
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-dns

API change check for com.azure.resourcemanager:azure-resourcemanager-eventhubs
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-eventhubs

API change check for com.azure.resourcemanager:azure-resourcemanager-keyvault
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-keyvault

API change check for com.azure.resourcemanager:azure-resourcemanager-monitor
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-monitor

API change check for com.azure.resourcemanager:azure-resourcemanager-msi
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-msi

API change check for com.azure.resourcemanager:azure-resourcemanager-network
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-network

API change check for com.azure.resourcemanager:azure-resourcemanager-privatedns
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-privatedns

API change check for com.azure.resourcemanager:azure-resourcemanager-redis
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-redis

API change check for com.azure.resourcemanager:azure-resourcemanager-resources
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-resources

API change check for com.azure.resourcemanager:azure-resourcemanager-samples
API review is created for com.azure.resourcemanager:azure-resourcemanager-samples. You can review APIs here.

API change check for com.azure.resourcemanager:azure-resourcemanager-search
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-search

API change check for com.azure.resourcemanager:azure-resourcemanager-servicebus
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-servicebus

API change check for com.azure.resourcemanager:azure-resourcemanager-sql
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-sql

API change check for com.azure.resourcemanager:azure-resourcemanager-storage
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-storage

API change check for com.azure.resourcemanager:azure-resourcemanager-trafficmanager
API changes are not detected in this pull request for com.azure.resourcemanager:azure-resourcemanager-trafficmanager


feat(ardrive_io) implement ardrive_io abstraction and an Android implementation PE-1863

Overview
[Important]: don't be afraid of review, most of the files are from the example project. The only file that matters on that example is:

example/lib/main.dart.

Abstraction overview
We are implementing an abstraction to handle I/O operations such as picking a file, picking folders, etc... our goal is to have an agnostic platform package. This PR aims to implement that abstraction and also implement an Android implementation.
Most of the code is related to the abstraction itself, only mobile_io.dart rely on the Android-specific implementation.
Entities
IOEntity
The base class for an agnostic file system entity.
IOFile
Base class for every file, where each platform can implement.
IOFolder
Base class for every folder, where each platform can implement.
IOException
Base class for exceptions handled in this library.
I/O Operations
The I/O operations on the file system will be handled at the ArDriveIO class. Its interface covers our needed methods:
Future<IOFile> pickFile({List<String>? allowedExtensions});
Future<List<IOFile>> pickFiles({List<String>? allowedExtensions});
Future<IOFolder> pickFolder();
Future<void> saveFile(IOFile file);

MobileIO
That class will implement the ArDriveIO for both platforms: Android and iOS. This PR will address an Android first implementation, but the same code can work for iOS, unless the method save().
Adapters
IOFileAdapter
IOFolderAdapter

Added a brief explanation of the package on README.


Add a utility that checks if SslProvider.JDK supports ALPN

Motivation:
We have a public utility OpenSsl.isAlpnSupported() that helps users to
check if ALPN is available for SslProvider.OPENSSL. However, we do not
provide a similar utility for SslProvider.JDK. Therefore, users who
configured ALPN with SslProvider.JDK will get a runtime exception at
the time when a new connection will be created.
Modifications:

Add public SslProvider.isAlpnSupported(SslProvider) utility method that
returns true if the SslProvider supports ALPN;
Deprecate OpenSsl.isAlpnSupported();

Result:
Users can verify if their environment supports ALPN with
SslProvider.JDK upfront (at bootstrap), instead of failing with runtime
exception when a new connection will be created.

@idelpivnitskiy after thinking a bit more about it I wonder if we not just want to add a method like this to SslContext and so make things easier for users to discover in the first place:
public abstract class SslContext {
    ....
    public static boolean isAlpnSupported(SslProvider provider) {
        ....
    }
}

@idelpivnitskiy let me know what you think...

@normanmaurer agreed, discoverability will be better if we add a static method to SslContext class. However, SslContext has a lot of deprecated static methods. Not sure if we want to add a new one to this abstract class.
Alternative could be to add a similar static (or non-static) method directly to the SslProvider enum. That will help to avoid forgetting to update this method in case we add a new SslProvider and also good for discoverability, because the check is related to SslProvider.
Additional questions to discuss in this case:

Should we deprecate OpenSsl.isAlpnSupported() and suggest to use a new method?
Should we also add SslProvider.default() (or SslProvider.defaultForClient()/SslProvider.defaultForServer()) and deprecate SslContext.defaultClientProvider()/SslProvider.defaultServerProvider()? That will help to remove all public static methods from SslContext in the 5.0 version.
WDYT?

Sounds like a plan 👌

Am 21.10.2019 um 19:44 schrieb Idel Pivnitskiy<EMAIL_ADDRESS>
@normanmaurer agreed, discoverability will be better if we add a static method to SslContext class. However, SslContext has a lot of deprecated static methods. Not sure if we want to add a new one to this abstract class.
Alternative could be to add a similar static (or non-static) method directly to the SslProvider enum. That will help to avoid forgetting to update this method in case we add a new SslProvider and also good for discoverability, because the check is related to SslProvider.
Additional questions to discuss in this case:
Should we deprecate OpenSsl.isAlpnSupported() and suggest to use a new method?
Should we also add SslProvider.default() (or SslProvider.defaultForClient()/SslProvider.defaultForServer()) and deprecate SslContext.defaultClientProvider()/SslProvider.defaultServerProvider()? That will help to remove all public static methods from SslContext in the 5.0 version.
WDYT?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@normanmaurer pushed the fix and updated commit message/PR description.
I will submit a follow up for SslProvider.default() (item 2 from my previous message) if that sounds good. Don't want to increase the scope of this PR.

@netty-bot test this please

@idelpivnitskiy thanks a lot!


rb_str_codepoints not available on all OSes

Windows doesn't have it and apparently some Mac OSes don't.

fixes #49


Fix minor toJSON sample usage code (it used JSON.stringify)

Nevermind, I should've looked more closely at the JSON api as it said.


Please update sample to show dotnet test integration

Would it be possible to update the code sample so that it works with dotnet test? Right now the sample only provides a standalone test execution experience, where most users will expect to be able to run tests via dotnet test.
Related: #3478

@MarcoRossignoli @drognanar

Answered here https://github.com/microsoft/testfx/issues/3478#issuecomment-2270577906


Windows: siganture errors installing driver

Dear Coral Team,
I tried installing the Coral Windows driver from (https://dl.google.com/coral/edgetpu_api/edgetpu_runtime_20201105.zip).
I get these error messages when running the installer:
Adding driver package:  coral.inf
Driver package added successfully.
Published Name:         oem43.inf
Driver package installed.

Adding driver package:  Coral_USB_Accelerator.inf
Failed to add driver package: The hash for the file is not present in the specified catalog file. The file is likely corrupt or the victim of tampering.

Adding driver package:  Coral_USB_Accelerator_(DFU).inf
Failed to add driver package: The hash for the file is not present in the specified catalog file. The file is likely corrupt or the victim of tampering.

Total driver packages:  3
Added driver packages:  1
Installing performance counters

So two of the driver packages are not installed cause of signature errors. As a consequence the sample code does not run ("Failed to load delegate from edgetpu.dll").

@MSoegtropIMC Thanks for reporting, we are looking into this now.

For now, I suggest rolling back to the older driver:
https://dl.google.com/coral/edgetpu_api/edgetpu_runtime_20200728.zip
And you also need to remove your current tflite_runtime and downgrade it to an older version (make sure to change to the right python version):
pip3 install https://dl.google.com/coral/python/tflite_runtime-2.1.0.post1-cp37-cp37m-win_amd64.whl

Apologies, we are working to get this fixed ASAP

Thanks!
Btw.: it might make sense to change the installer batch file such that it produces a more prominent error message and pauses in case of an error. If you run the batch file with a click from explorer, you likely won't see the error message(s), because the console just closes at the end. This might result in all sorts of hard to spot errors down the road. I did run the batch file from a console window to see the errors.

@MSoegtropIMC thanks for the input, will also forward this to the team

Can confirm uninstalling the new version and using the older driver works. Also Windows SmartScreen by default blocks unknown/unsigned scripts downloaded from the Internet that ask for admin privileges and you have to manually tell it to go ahead. If the work done in the install.bat script can be moved to the signed installer then it would be a better user experience.

I'm using Python 3.8 and unfortunately I can't find tflite_runtime-2.1.0.post1-cp38-cp38m-win_amd64.whl anywhere (@Namburger the link for the Python 3.8 library leads to a 404 page). I'm forced to use tflite_runtime-2.5.0, and I'm forced to use the newer 20201105 driver, as the older 20200728 driver doesn't work with tflite 2.5.0.
It seems like the newer drivers aren't/are improperly signed. I can confirm that it does work, but I have to disable Driver Signature Enforcement as a workaround and deal with the warning below:

@pandalion98 ahh, that's true, I forgot that we did not have python38 supports in the previous release.
We're working on the fix for the new release, including the signed share lib now, please expect this to be done in a few weeks, apologies.

I had same problem on Windows 10 and downgraded runtime version to edgetpu_runtime_20200728.zip and tflite_runtime to 2.1.0. I was able to install the drivers after that, but trying to run the test script again was unsuccessful. What could be the problem here?
`C:\coral\tflite\python\examples\classification>python classify_image.py --model models/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite --labels models/inat_bird_labels.txt --input images/parrot.jpg
Traceback (most recent call last):
File "C:\Python\lib\site-packages\tflite_runtime\interpreter.py", line 162, in load_delegate
delegate = Delegate(library, options)
File "C:\Python\lib\site-packages\tflite_runtime\interpreter.py", line 121, in init
raise ValueError(capture.message)
ValueError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "classify_image.py", line 122, in
main()
File "classify_image.py", line 99, in main
interpreter = make_interpreter(args.model)
File "classify_image.py", line 73, in make_interpreter
{'device': device[0]} if device else {})
File "C:\Python\lib\site-packages\tflite_runtime\interpreter.py", line 165, in load_delegate
library, str(e)))
ValueError: Failed to load delegate from edgetpu.dll`

Hi, I have a LattePanda windows mini board CPU and got the same error with the latest  driver installer: download edgetpu_runtime_20201204.zip. I had miniconda python 3.8 installed and tried the 1105 install as @pandalion98 did after disabling driver signature enforcement, but that made the install wonky and I still got the same error.  So I created a python 3.7 environment and went with the 0728 and that worked.
Please let us know when a new fixed version is posted.
Thanks,
Jim

Hi, I have a LattePanda windows mini board CPU and got the same error with the latest  driver installer: download edgetpu_runtime_20201204.zip. I had miniconda python 3.8 installed and tried the 1105 install as @pandalion98 did after disabling driver signature enforcement, but that made the install wonky and I still got the same error.  So I created a python 3.7 environment and went with the 0728 and that worked.
Please let us know when a new fixed version is posted.
Thanks,
Jim

@MSoegtropIMC Is this issue still persistent ?

I didn't try any new version since then (I live with the rolled back version). If you did an updated which you expect to fix this, I will try that until next week.

Feel free to reopen if this issue still persists.


Broken link for android version

Hi there,
The link for the android version is directs back to objective-c project. Is that broken?

It has been fixed now.
Source Code⛓

Language Version
Project Name
Target Platform
Source Code Link

Swift
AAInfographics
iOS
https://github.com/AAChartModel/AAChartKit-Swift

Objective C
AAChartKit
iOS
https://github.com/AAChartModel/AAChartKit

Kotlin
AAInfographics
Android
https://github.com/AAChartModel/AAChartCore-Kotlin

Java
AAChartCore
Android
https://github.com/AAChartModel/AAChartCore


fix(international-phone-input): added missing flags

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.006%) to 84.512%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
9354

Relevant Lines:
10237

💛  - Coveralls


Revise older blog articles and mark some evergreen

After this change:

all release announcement articles are evergreen (no “outdated content” banner)

uh, except v1.0 which isn't yet imported into the blog

the formatting of older articles and author attributions is closer to how we do it today
some now-inaccurate TL;DR: sections are gone from the relevant articles

/kind cleanup

I agree with the changes here
/lgtm

zero objections or concerns; appreciate the clean up and evergreen marks :)

Thanks
Given that:
/hold cancel


Amend SE-0025: Add a section on the complications with private types.

See the new section for details. Discussion here: http://thread.gmane.org/gmane.comp.lang.swift.evolution/20718/
This is revision 2.1 of SE-0025, after further discussion on the lists and with the Core Team. You can see version 2.0 in #383.
cc @CodaFi

Looks great, thanks @jrose-apple


Revert "Draft: Add Err() function to Response for detailed errors (#2…

…… (#322)

Revert "Draft: Add Err() function to Response for detailed errors (#246)"

This reverts commit 8d3cf4dd6e872f5c9fc50408ca74dc7c06e9b490.

Fixing PIT APIs introduced in PR #253 since they use response.Err()

Removing opensearchapi.Error usage from snapshot tests introduced in PR #237

Resolve Changelog conflicts

Description
I created a 2.x branch from main to track the 2.x line and this PR is to revert the breaking change on 2.x. Similar process was done when releasing version 2.3, 2.x has all latest changes from main and the breaking change is being reverted here.
Doing this so we can move to releasing a 3.0 version.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

But this PR is into main, so it's reverting a change that should stay in 3.0, no?

Oh my bad 🤦🏼♀ . I will raise against 2.x. Thank you for pointing out!


Grandma loses her colours when she spins

[ ] grandma keeps her color while spinning.

#115 grandma no longer losers her color #115


TypeError: undefined is not a constructor (evaluating 'new global.AudioContext()')

Hello everyone,
Currently I am not able to publish video or audio stream on iOS(v 8.2).
When I call the method session.publish it gives following error:
TypeError: undefined is not a constructor (evaluating 'new global.AudioContext()')
audioContext opentok.js:19676
Below is line 19676 :
"context = context || new global.AudioContext();"
So I investigate the opentok.js line:19676 and following "global.AudioContext" returns undefined:
Instead of global.AudioContext, global.webkitAudioContext is available in my project.
How can I solve the issue, do you have any idea ?

I also have this problem. it cannot publish my stream because of  1500 error and OT.exception :: title: Connection Failed (1013) msg: OT.Publisher PeerConnection Error: undefined is not a constructor (evaluating 'new AudioContext()')


Error running eval.py

I have trained the model with full set (bible) DB and I got model_epoch_40_gs_80000.* files in "logdir_s" directory.   When I tried with eval.py I got the following "Size mismatch between n_fft and window size" error.
Anybody experienced the similar problem?
$ python eval.py
...
2017-07-11 19:29:59.842658: W tensorflow/core/platform/cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
WARNING:tensorflow:Standard services need a 'logdir' passed to the SessionManager
Restored!
Traceback (most recent call last):
File "eval.py", line 69, in
eval()
File "eval.py", line 62, in eval
audio = spectrogram2wav(np.power(np.e, s)**hp.power)
File "/home/abc/Tacotron/tacotron/utils.py", line 93, in spectrogram2wav
X_t = invert_spectrogram(X_best)
File "/home/abc/Tacotron/tacotron/utils.py", line 105, in invert_spectrogram
return librosa.istft(spectrogram, hp.hop_length, win_length=hp.win_length, window="hann")
File "/usr/local/lib/python2.7/dist-packages/librosa/core/spectrum.py", line 294, in istft
raise ParameterError('Size mismatch between n_fft and window size')
librosa.util.exceptions.ParameterError: Size mismatch between n_fft and window size

Just noticed that this issue was addressed in the past, so I'll close it.


Allow 'dw' to apply to characters of strings

This is orthogonal to widechar support (#97); it's potentially useful even if charmaps remain 8-bit.
db "ABC", -1, "{dee}EF" with charmap "{dee}", $42 outputs 7 bytes, since each character post-charmap lookup gets handled by db.
dw "ABC", -1, "{dee}EF" could output 14 bytes, just as if it had been written dw "A", "B", "C", -1, "{dee}", "E", "F".
This can't be done with a macro, because even if the new lexer resolves the current issues around quotes and commas (consider mydwstring "AB, C{dee}", "EF, GH"), macro arguments would still see the individual characters { d e e }.
 (You'd see similar results from db using a parenthesized expression, I assume.)


Add file during chat

Implement https://github.com/langchain-ai/opengpts/issues/157

This PR enables file upload to thread based vector embeddings for RAG and Assistant with Retriever tool.
Co-created together with @bakar-io - Sema4.ai

Limited to only doc, docx related unstructured deps.
Checked that pdf, docx and html can be ingested.
=> build now succeeds.


changing license renewal link to point to new renewal webpage

Summary
Changing the license renewal link to point to the new renewal webpage. It currently points to an old wufoo form.
Ticket Link
https://mattermost.atlassian.net/browse/MM-26486
Related Pull Requests

Has server changes (https://github.com/mattermost/mattermost-server/pull/14907)

Screenshots
N/A

Similarly to the email in the other PR this link is appended with ?id=<license-id>&user_count=<user-count>. Shouldn't cause issues but just to confirm you're aware

Ah it looks like a test failed because the link no longer matches the snapshot's link (https://github.com/mattermost/mattermost-webapp/blob/master/components/announcement_bar/configuration_bar/snapshots/configuration_bar.test.jsx.snap). Is there a best practice here? I can go change the snapshot, just don't want to violate any testing norms.

Snapshots shouldn't be manually edited. I'll update it for you

@esadur sorry about the delay, had the update and forgot to push


Overlapping memslots can corrupt linked list

Describe the Bug
When I try to add a memblock that ends at the exact beginning of the next memblock via HAX_VM_IOCTL_SET_RAM, there seems to be a condition where the new block is being left as referencing itself via flink and blink causing an infinite loop in the driver.
See logfile below for details
Summary:
Host Environment

HAXM version: c072ad9
Host OS version: Windows 10 pro 1803
Host OS architecture: x86_64
Host CPU model: Xeon E31235
Host RAM size: 12 GB

To Reproduce
Connect memory pages in the layout described in the logfile.
Expected Behavior
No self-referencing pointer in the linked list.
Reproducibility
100%
Diagnostic Information
HAXM log:
haxm_info:memslot_set_mapping: start_gfn=0xa0, npages=0x10, uva=0x0, flags=0x80
haxm_debug:ramblock_ref: block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 2
haxm_debug:ramblock_deref: block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 1
haxm_info:ept_handle_mapping_removed: RAM=>MMIO: start_gfn=0xa0, npages=0x10, uva=0x570000
haxm_info:ept_handle_mapping_removed: Invalidated 0 PTEs
haxm_debug:ramblock_deref: Reset RAM block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 0
haxm_info:ramblock_free_chunks: Freeing <= 1 chunks, bitmap:
haxm_info:ramblock_free_chunks:   [0]=0x00
haxm_info:ramblock_free_chunks:   [1]=0x00
haxm_info:ramblock_free_chunks:   [2]=0x00
haxm_info:ramblock_free_chunks:   [3]=0x00
haxm_info:ramblock_free_chunks:   [4]=0x00
haxm_info:ramblock_free_chunks:   [5]=0x00
haxm_info:ramblock_free_chunks:   [6]=0x00
haxm_info:ramblock_free_chunks:   [7]=0x00
haxm_info:ramblock_free_chunks:   [8]=0x00
haxm_info:ramblock_free_chunks: All chunks freed: 128KB total, 0KB used
haxm_info:memslot dump begins:
haxm_info:memslot [0]: base_gfn = 0x0000000000000000, npages = 0xa0, uva = 0x0000000002200000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0x0)
haxm_info:memslot [1]: base_gfn = 0x00000000000000b0, npages = 0x8, uva = 0x00000000022b0000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0xb0000)
haxm_info:memslot [2]: base_gfn = 0x00000000000000bc, npages = 0x4, uva = 0x00000000022bc000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0xbc000)
haxm_info:memslot [3]: base_gfn = 0x00000000000000c0, npages = 0x40, uva = 0x00000000022c0000, flags = 0x01 (block_base_uva = 0x0000000002200000, offset_within_block = 0xc0000)
haxm_info:memslot [4]: base_gfn = 0x0000000000000100, npages = 0x10, uva = 0x0000000002300000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0x100000)
haxm_info:memslot dump ends!
haxm_info:memslot_set_mapping: start_gfn=0xb8, npages=0x8, uva=0x570000, flags=0x0
haxm_debug:ramblock_find: (FFFF9F882E1D9040): base_uva 0x570000, size 0x20000, ref_count 0
haxm_debug:ramblock_ref: block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 1
haxm_debug:ramblock_ref: block (FFFF9F882E1D9360): base_uva = 0x2200000, size = 0x110000, ref_count = 6
haxm_debug:ramblock_deref: block (FFFF9F882E1D9360): base_uva = 0x2200000, size = 0x110000, ref_count = 5
haxm_debug:ramblock_ref: block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 2
haxm_info:ept_handle_mapping_changed: RAM=>RAM: start_gfn=0xbc, npages=0x4, old_uva=0x22bc000, new_uva=0x574000
haxm_info:ept_handle_mapping_changed: Invalidated 0 PTEs
haxm_debug:ramblock_deref: block (FFFF9F882E1D9360): base_uva = 0x2200000, size = 0x110000, ref_count = 4
haxm_debug:ramblock_deref: block (FFFF9F882E1D9040): base_uva = 0x570000, size = 0x20000, ref_count = 1
haxm_info:memslot dump begins:
haxm_info:memslot [0]: base_gfn = 0x0000000000000000, npages = 0xa0, uva = 0x0000000002200000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0x0)
haxm_info:memslot [1]: base_gfn = 0x00000000000000b0, npages = 0x8, uva = 0x00000000022b0000, flags = 0x00 (block_base_uva = 0x0000000002200000, offset_within_block = 0xb0000)
haxm_info:memslot [2]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [3]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [4]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [5]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [6]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [7]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [8]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [9]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [10]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [11]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [12]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)
haxm_info:memslot [13]: base_gfn = 0x00000000000000b8, npages = 0x8, uva = 0x0000000000570000, flags = 0x00 (block_base_uva = 0x0000000000570000, offset_within_block = 0x0)

Any specific reason you are referencing commit https://github.com/intel/haxm/commit/c072ad9b68e1d558a9fb791511468d10a1a9b319?

Na,it's just that I have checked out this version and using it actively with my modifications, but there was no change in the way of the memslot implementation since then, so it shouldn't be a problem.
I suspect that
        // [3]
        if (*state & MEMSLOT_TO_INSERT) {
            memslot_move(src, dest);
        }

in memslot_process_end_diff_type is the reason for it,because it just replaces an item with a version that is self-referencing (see memslot_init) and the links aren't updated anywhere as far as I can see

This should fix it:
static inline void memslot_move(hax_memslot *dest, hax_memslot *src)
{
	hax_list_node entry;

    ramblock_deref(dest->block);
	entry = dest->entry;
    memslot_init(dest, src);
	dest->entry = entry;
}

Great thanks for your trouble shoot! I have added a comment in #199 for your reference.

Thanks for your quick response and commit in #237.


[colorManipulator] Prevent illegal color values

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Fixes one last niggle I had with colorManipulator:
The color can go out of range if an illegal coefficient is provided.

Replace the warning with a clamp() function.
Add jsdoc @return to comments.

I don't get it. What's wrong with a warning? Do we need to run clamp in production?

Toss of a coin on warn or limit - and limit won. :smile: (Both are computationally inexpensive.)
I can either make the warnings consistent or remove them altogether. Your call.

I don't think we should warn people, numbers can fall below or above the intended value from time to time. I mean it's floating point arithmetic, they are never accurate. It will be very painful for users to figure out why their code is generating a warning and they'll have to read all about caveats of binary representation of decimal numbers... :sleepy:

@alitaheri Alright, that sounds good to me.
What about using < and > instead of Math.min and Math.max?
That's less descriptive but perfom far better.

I wouldn't worry about micro optimizations like this, but I find
if (value < min) {
  return min;
}
if (value > max) {
  return max;
}
return value;

to be a lot easier to understand. than Math.min(Math.max(value, min), max); :see_no_evil: :see_no_evil:

Math.min(Math.max(value, min), max);

Does exactly what it says on the tin. Works and is tested, so will never need to be changed. And honestly isn't that hard to understand, especially since you already know what it does.
But if it makes y'all feel better, I can change it to freshman code. :laughing:

Updated.

Awesome!


docs: MLflow API and examples

Added docs and links to the API and examples provided by MLflow itself

cc @harupy

@baskaryan No feedback from author


[Android][Image] Loading images from SDCard

The use of GenericDraweeView makes it difficult to use images from the SDCard. It would be great to have a plugin architecture similar to iOS where we can add new resource handlers.

Can you share code? Are you using file:// URLs with <Image />?

We have a Camera Roll module coming that will let you load images from the gallery, see:
https://facebook.github.io/react-native/docs/known-issues.html

Hi @mkonicek, for our use case we'd like to be able to dynamically change the source image used during runtime. This is enabled on the iOS side via the pluggable "ImageLoader" system: https://github.com/facebook/react-native/commit/36444a65c700f4658d54a6be32bae700b228f6da

@arbesfeld Can you share some code to demo what you're doing in JS? That would help us understand this. I'm going on vacation tomorrow but will talk about it with the Fresco team next week.

Hi @mkonicek,
We are downloading new images to a device and saving them to the file:///data/data/com.*/ folder like so:
File file = new File(getFilesDir(), "images");
FileOutputStream fout = new FileOutputStream(file);
fout.write(...);

It (usually) works to access them from JavaScript like so:
<Image source="file:///data/data/com.apphub.io/files/images/foo.png" />

But hard-coding this path doesn't always work, as getFilesDir() behaves differently from device to device.
An ideal solution would allow us to handle "file" URLs with our own Android handler.
Alternatively, it would be nice if could write paths like so:
<Image source="~/images/foo.png" />

Where the ~ would resolve to getFilesDir().

This would indeed be very useful, esp for displaying things like user profile images, once the user has logged in.

Thanks for the explanation! "~/images/foo.png" is interesting. How does iOS do this? Sorry for the delay :(

This PR might be interesting: https://github.com/facebook/react-native/pull/4527

Fixed via https://github.com/facebook/react-native/commit/0fb86c2495157038ded99fc22bfa1e19652989d6


test: make test pass

This PR is to loosen timeout limit for some test case, to make them pass.

这两个 2.8 以后也经常挂


Add i18n tags

Closes #965

@brylie I added some of the tags. For autoforms, I am not sure how to add tags for the dropdown options, such as for the Send Feedback form.

@Alapan great work! To help us better track the translation process, please also reference the sub-tasks of #965 that this completes. You can see the sub-tasks that are linked to #965 by scrolling down in the comments section and looking for the 'open' label next to related issues.

We can merge this PR when all related issues have been linked. E.g. add multiple 'Closes #issue' to the PR description, each 'Closes' on its own line:
Closes #123
Closes #234
Closes #345
...

@Alapan Reviewing.. Code looks good. I was just wondering one thing why we have limited jQuery to 2.x ? Is there something breaking when using 3.x ? But it is not blocking merging of this, if not sure.
I just run this after lunch, and see that everything functions nice.

@Alapan If interested to translate autoforms, we have this package in project, https://github.com/gwendall/meteor-autoform-i18n
It might provide some help for what you were wondering.

Well done.


Mossy: Key table still appears in views like Gantt

When I first open Gantt, key table from 2D network view is not displayed, but if I shift to another tab, and come back to Gantt, the key table appears. We have seen this issue before.

Node and link key tables are no longer displayed when the gantt chart is re-opened. The phylogenetic tree does show the node key table, but the tree nodes are colored according to that chart, so it's appropriate to have it there, I think.


Refactor matrix views to move common code to meta-class

I've selected for refactoring 18 lines of code which are duplicated in 5 file(s) (1, 2, 3, 4, 5).

This is a duplicate of issue #201.


Task watchdog got triggered on CPU 0

Problem Description
While the basic BT receiving example works fine when playing music through my iPad Pro oder Huawei P30, the application crashes as soon as I start a couple of seconds on my Samsung S24+.

E (74637) task_wdt: Task watchdog got triggered. The following tasks/users did not reset the watchdog in time:
E (74637) task_wdt:  - IDLE0 (CPU 0)
E (74637) task_wdt: Tasks currently running:
E (74637) task_wdt: CPU 0: BTC_TASK
E (74637) task_wdt: CPU 1: loopTask
E (74637) task_wdt: Aborting.
E (74637) task_wdt: Print CPU 0 (current core) backtrace

Device Description
I'm using the DOIT ESP32 DEVKIT, but tried also different other WROM32 based boards. Always with the same result :-(
Downgrading the ESP-IDF to 2.0.17 did not change anything.
Sketch
#include "ESP_I2S.h"
#include "BluetoothA2DPSink.h"

const uint8_t I2S_SCK = 26;       /* Audio data bit clock */
const uint8_t I2S_WS = 22;       /* Audio data left and right clock */
const uint8_t I2S_SDOUT = 25;    /* ESP32 audio data output (to speakers) */
I2SClass i2s;

BluetoothA2DPSink a2dp_sink(i2s);

void setup() {
    i2s.setPins(I2S_SCK, I2S_WS, I2S_SDOUT);
    if (!i2s.begin(I2S_MODE_STD, 44100, I2S_DATA_BIT_WIDTH_16BIT, I2S_SLOT_MODE_STEREO, I2S_STD_SLOT_BOTH)) {
      Serial.println("Failed to initialize I2S!");
      while (1); // do nothing
    }

    a2dp_sink.start("BT-Box");
}

void loop() {
}

Other Steps to Reproduce
I found similar older issues here, but as in my case the same code works for my old phone but not for my new one, I believe there must be something about how the data is being send from the phone. Maybe there is some kind of too much data being send by newer devices?
Provide your Version of the EP32 Arduino Core (or the IDF Version)
3.0.4
I have checked existing issues, discussions and online documentation

[X] I confirm I have checked existing issues, discussions and online documentation

Your empty loop does never yields any control, so you are starving all other processes!
You have to add a delay, like in all my examples

Even with a delay(up to 1000) in the loop, a call to yield() it does not work.
I have also tried putting a higher value into the watchdog's timeout.
As said: The same code works when playing music on any of my devices, except for the S24+.

I don't think this is an issue with this library, but rather your S24+ or potentially the IDF.
I am using a Samsumg Galaxy A34 5G and don't have any issues...
You could try to activate the logging and compare the difference between a working Android and your non working Phone to get a hint where things start to go wrong...

I have the same problem. Esp arduino 3.0.4 , phone is  iphone 13.


feat(TDI-45202): Azure Storage Blob DLS Gen2 Input for TDP

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-45202
What is the new behavior?
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:


Dtreeviz recognize XGBoost classification model as regression model

(I got help from a google translator. If there is any expression that looks rude, I apologize in advance.)
First of all, thank you for making a great library!
I tried to visualize XGBoost classification model.
But dtreeviz made result about regression.
This is my code.
from sklearn.model_selection import train_test_split
from sklearn.datasets import load_iris

iris = load_iris()
X = iris.data
y = iris.target

X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.3, random_state=0)

import xgboost as xgb

dtrain = xgb.DMatrix(X_train, label=y_train, enable_categorical=True)
dtest = xgb.DMatrix(X_test, label=y_test, enable_categorical=True)

params = {
    'max_depth': 3,
    'eta': 0.3,
    'objective': 'multi:softprob',
    'eval_metric': 'mlogloss',
    'num_class': 3,
}
num_round = 100  # the number of training iterations

bst = xgb.train(params=params, dtrain=dtrain, num_boost_round=num_round)

from dtreeviz import trees

import graphviz

from dtreeviz.models.shadow_decision_tree import ShadowDecTree
from dtreeviz.models.xgb_decision_tree import ShadowXGBDTree

trees.dtreeviz(tree_model = bst,
               x_data = X_train,
               y_data = y_train,
               target_name = 'class',
               feature_names = ['f0','f1','f2','f3'],
               histtype = 'barstacked',
               tree_index=1,
               class_names = list(iris.target_names),
              )

I expected distribution graph and pie chart, but my code generated scatter chart.

Is there anything I've done wrong? Or is there a temporary solution?
[ Dependencies ]

Python 3.8.5
XGBoost 1.3.3
graphviz 2.46.1

hi @HyukdongKim, it's an issue from the library. thanks for creating an issue about it.
it's caused by how dtreeviz interprets the tree to be classifier or regressor. Fixed it, but it seems there are still few other issues.
I will come back when I will fix them.


8.1 Pascal Triangle - type 1 ques

My code is clear and well understanding
Please assign it to me

You can assign the Issue for yourself by using the '/assign' command. Make sure The algorithm/ data structure you are trying to add doesn't already exist. read the Doc before you proceed

Need to add a description and your approach. Thanks, Closing as of now.


Define anf_trans using anf_state to help typeclass resolution

I am trying to make coq-ext-lib classes stricter to resolve, to reduce the likelihood of infinite loops. This patch makes certicoq compatible with this coq-ext-lib PR: https://github.com/coq-community/coq-ext-lib/pull/126

@zoep can you merge this at your convenience?

Is there a schedule to release the changes since v0.9-beta on the Coq OPAM archive?
Or, maybe worth creating a coq-certicoq.dev version in the extra-dev repository?


fix off-by-one in coverage reporting

using getCommittedDocIdLimit() isn't really what we want
to do, it will (even in the simplest case) be 1 more than
the number of indexed documents.  Use getNumUsedLids(),
and do an extra check to avoid division by zero.

@toregge please review
@baldersheim @havardpe @geirst please also take a look.

I suggest sampling the committed docid limit just once, then add 1 to the docid space estimate before dividing.

@havardpe please take another look now

LGTM, but the comments are misleading

my bad, the first search range starts at docid 1, so you are right about document 0 being the one unaccounted for.

LGTM


axum_garde and State

To help demonstrate this issue I'll use the json example in this repository.
I've been successful in using garde and axum together to pass state and validate parameters, so thank you! Your project is quite helpful. The way I've been successful is to use axum's older Extension method for passing state, see here:
//! Simple usage of using `axum_garde` for a REST API
//!
//! Run the example using
//!
//! ```sh
//! cargo run --example json --features=json
//! ```
use axum::response::IntoResponse;
use axum::routing::post;
use axum::extract::Extension;
use axum::{Json, Router, Server};
use axum_garde::WithValidation;
use garde::Validate;
use serde::{Deserialize, Serialize};

// Define your valid scheme
#[derive(Debug, Serialize, Deserialize, Validate)]
struct Person {
    #[garde(ascii, length(min = 3, max = 25))]
    username: String,
    #[garde(length(min = 15))]
    password: String,
}

#[derive(Clone)]
struct AppState { }

async fn insert_valid_person(
    Extension(state): Extension<AppState>,
    // Perform validation on the request payload
    WithValidation(person): WithValidation<Json<Person>>,
) -> impl IntoResponse {
    println!("Inserted person on database: {person:?}");
    Json(person.into_inner())
}

#[tokio::main]
async fn main() {
    let state = AppState { };
    let app = Router::new()
        .route("/person", post(insert_valid_person))
        // Create the application state
        .layer(Extension(state));
    println!("See example: http://<IP_ADDRESS>:8080/person");
    Server::bind(&([127, 0, 0, 1], 8080).into())
        .serve(app.into_make_service())
        .await
        .unwrap();
}

That seems to work just fine. That said, there is a relatively new feature in axum called State - https://tokio.rs/blog/2022-08-whats-new-in-axum-0-6-0-rc1. This is now the preferred way to pass state as it is type safe, whereas Extension is not. The article also claims that State is faster than Extension.
When I rework the example above to use State rather than Extension I get an error:
//! Simple usage of using `axum_garde` for a REST API
//!
//! Run the example using
//!
//! ```sh
//! cargo run --example json --features=json
//! ```
use axum::response::IntoResponse;
use axum::routing::post;
use axum::extract::State;
use axum::{Json, Router, Server};
use axum_garde::WithValidation;
use garde::Validate;
use serde::{Deserialize, Serialize};

// Define your valid scheme
#[derive(Debug, Serialize, Deserialize, Validate)]
struct Person {
    #[garde(ascii, length(min = 3, max = 25))]
    username: String,
    #[garde(length(min = 15))]
    password: String,
}

#[derive(Clone)]
struct AppState { }

async fn insert_valid_person(
    State(state): State<AppState>,
    // Perform validation on the request payload
    WithValidation(person): WithValidation<Json<Person>>,
) -> impl IntoResponse {
    println!("Inserted person on database: {person:?}");
    Json(person.into_inner())
}

#[tokio::main]
async fn main() {
    let state = AppState { };
    let app = Router::new()
        .route("/person", post(insert_valid_person))
        // Create the application state
        .with_state(state);
    println!("See example: http://<IP_ADDRESS>:8080/person");
    Server::bind(&([127, 0, 0, 1], 8080).into())
        .serve(app.into_make_service())
        .await
        .unwrap();
}

$ cargo run --example json --features=json
warning: garde/garde/Cargo.toml: version requirement `0.3.2+8.13.9` for dependency `phonenumber` includes semver metadata which will be ignored, removing the metadata is recommended to avoid confusion
   Compiling axum_garde v0.11.2 (garde/integrations/axum_garde)
error[E0277]: the trait bound `(): FromRef<AppState>` is not satisfied
   --> integrations/axum_garde/examples/json.rs:41:32
    |
41  |         .route("/person", post(insert_valid_person))
    |                           ---- ^^^^^^^^^^^^^^^^^^^ the trait `FromRef<AppState>` is not implemented for `()`
    |                           |
    |                           required by a bound introduced by this call
    |
    = help: the following other types implement trait `Handler<T, S, B>`:
              <Layered<L, H, T, S, B, B2> as Handler<T, S, B2>>
              <MethodRouter<S, B> as Handler<(), S, B>>
    = note: required for `WithValidation<Json<Person>>` to implement `FromRequest<AppState, _>`
    = note: required for `fn(State<AppState>, WithValidation<Json<Person>>) -> impl Future<Output = impl IntoResponse> {insert_valid_person}` to implement `Handler<(axum_core::extract::private::ViaRequest, State<AppState>, WithValidation<Json<Person>>), AppState, _>`
note: required by a bound in `post`
   --> .cargo/registry/src/github.com-1ecc6299db9ec823/axum-0.6.18/src/routing/method_routing.rs:407:1
    |
407 | top_level_handler_fn!(post, POST);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `post`
    = note: this error originates in the macro `top_level_handler_fn` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0277`.
error: could not compile `axum_garde` due to previous error

This error message is currently beyond my ability to comprehend. If I remove the garde validation, the example works fine, so I guess there is some friction between garde and this newish feature in axum that is causing the issue. Any help would be appreciated.

garde supports custom validation with custom context types, and if one isn't provided via the #[garde(context(T))] attribute, then it defaults to (). What you're seeing is WithValidation attempting to extract () from the struct passed to with_state. axum supports substates, so the easiest way out is to implement FromRef<AppState> for ():
impl FromRef<AppState> for () {
    fn from_ref(input: &AppState) -> Self {
        todo!()
    }
}

Full example
//! Simple usage of using `axum_garde` for a REST API
//!
//! Run the example using
//!
//! ```sh
//! cargo run --example json --features=json
//! ```
use axum::extract::FromRef;
use axum::response::IntoResponse;
use axum::routing::post;
use axum::{Json, Router, Server};
use axum_garde::WithValidation;
use garde::Validate;
use serde::{Deserialize, Serialize};

// Define your valid scheme
#[derive(Debug, Serialize, Deserialize, Validate)]
struct Person {
    #[garde(ascii, length(min = 3, max = 25))]
    username: String,
    #[garde(length(min = 15))]
    password: String,
}

async fn insert_valid_person(
    // Perform validation on the request payload
    WithValidation(person): WithValidation<Json<Person>>,
) -> impl IntoResponse {
    println!("Inserted person on database: {person:?}");
    Json(person.into_inner())
}

#[derive(Clone)]
struct AppState {}

impl FromRef<AppState> for () {
    fn from_ref(input: &AppState) -> Self {
        todo!()
    }
}

#[tokio::main]
async fn main() {
    let app = Router::new()
        .route("/person", post(insert_valid_person))
        // Create the application state
        .with_state(AppState {});
    println!("See example: http://<IP_ADDRESS>:8080/person");
    Server::bind(&([127, 0, 0, 1], 8080).into())
        .serve(app.into_make_service())
        .await
        .unwrap();
}

Thank you @jprochazk. With this example I was able to get my application up and running w/ State and your garde.


Include the following in all files

/** @file file_name.extension
    @brief Brief file description. */
/*
  BSD 3-Clause License

  Copyright (c) 2024-present, Stronkkey and Contributors

  Redistribution and use in source and binary forms, with or without
  modification, are permitted provided that the following conditions are met:

  1. Redistributions of source code must retain the above copyright notice, this
      list of conditions and the following disclaimer.

  2. Redistributions in binary form must reproduce the above copyright notice,
      this list of conditions and the following disclaimer in the documentation
      and/or other materials provided with the distribution.

  3. Neither the name of the copyright holder nor the names of its
      contributors may be used to endorse or promote products derived from
      this software without specific prior written permission.

*/

For documentation purposes

You can exclude the @brief if you want.


exception creating properties on Links

1.7, If you create properties on an edge, you get the following exception. The Proerties on edge is being configured before any edges of this type are created.  Follow the script below to reproduce.
orientdb {touca}> create property speaks.out LINK;
Error: com.orientechnologies.orient.enterprise.channel.binary.OResponseProcessingException: Exception during response processing.
Error: com.orientechnologies.orient.core.exception.OSchemaException: Cannot change the schema while a transaction is active. Schema changes are not transactional
orientdb {touca}> create property speaks.in LINK;
Error: com.orientechnologies.orient.enterprise.channel.binary.OResponseProcessingException: Exception during response processing.
Error: com.orientechnologies.orient.core.exception.OSchemaException: Cannot change the schema while a transaction is active. Schema changes are not transactional
orientdb {touca}> CREATE INDEX unique_speaks ON speaks (in, out) UNIQUE;
Creating index...
Error: com.orientechnologies.orient.enterprise.channel.binary.OResponseProcessingException: Exception during response processing.
Error: com.orientechnologies.orient.core.exception.OCommandExecutionException: Error on execution of command: sql.create INDEX unique_speaks ON speaks (in, out) UNIQUE
Error: java.lang.IllegalStateException: Cannot create a new index inside a transaction
Script:
#drop database if already exists
drop database remote:locahost/touca root pass;

#create touca database
create database remote:localhost/touca root pass local;

#reconnect
connect remote:localhost/touca root pass;

#create classes/Vectors
create class User extends V;
create class Language extends V;
create class Label extends V;
create class Email extends V;
create class City extends V;

#create Links/Edges
create class has extends E;
create class speaks extends E;
create class is extends E;
create class translatesTo extends E;
create class livesIn extends E;

#create User Data
create vertex User set name = 'Luca';
create vertex User set name = 'Joe';

#create Language Data
create vertex Language set name = 'En-uk';
create vertex Language set name = 'En-us';
create vertex Language set name = 'Fr-fr';
create vertex Language set name = 'Ru-ru';
create vertex Language set name = 'Ar-sy';

#create Label Data
create vertex Label set name = 'Hello';
create vertex Label set name = 'Salut';
create vertex Label set name = 'Ciao';
create vertex Label set name = 'Good day';
create vertex Label set name = 'Bonjour';
create vertex Label set name = 'Hallo';
create vertex Label set name = 'Marhaba';
create vertex Label set name = 'Salut2';

#emails
create vertex Email set email =<EMAIL_ADDRESS>create vertex Email set email =<EMAIL_ADDRESS>
#edges for emails
create edge has from (select from User where name = 'Luca') to (select from Email where email = 'Luca@gmail.com');
create edge has from (select from User where name = 'Luca') to (select from Email where email = 'Luca@hotmail.com');

#index
create property speaks.out LINK;
create property speaks.in LINK;
CREATE INDEX unique_speaks ON speaks (in, out) UNIQUE;

Even though this error is closed, I keep getting this.
Error: java.lang.IllegalStateException: Cannot create a new index inside a transaction
What is meant by inside a transaction?

hi @nishantkumar1292,
the DDL operation (like create class/create property/create index) are not allowed when a transaction is running, so you cannot do
begin;
create  index ...;
commit;

with a snippet like this you will bet Error: java.lang.IllegalStateException: Cannot create a new index inside a transaction
Hope this help

I am doing this in a rails application and created a standalone function for this. But it is still giving me this error. I ran this function immediately after creating the database.
Did I miss something?

Are you using oCommandScript?
Sent from my iPhone

On 22 May 2017, at 07:35, Nishant Kumar<EMAIL_ADDRESS>wrote:
I am doing this in a rails application and created a standalone function for this. But it is still giving me this error. I ran this function immediately after creating the database.
Did I miss something?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Not using oCommandScript.
Using active-orient gem.
ORD.execute to run the query in case of complex once.


Add Type-Safety to RedisPipeline.enqueue

From the proposal discussion thread: https://forums.swift.org/t/discussion-nioredis-nio-based-redis-driver/22455/15

Is there are reason we shouldn't also have a declarative way for the command themselves? Something like

public enum RESPCommand {
    case get(RESPValueConvertible)
    case increment(RESPValueConvertible)
    case set(RESPValueConvertible, RESPValueConvertible)
    // ...
}

That way, batching of requests can become more natural and we don't need

public func enqueue<T>(operation: (RedisClient) -> EventLoopFuture<T>) -> RedisPipeline

where we have this weird operation: (RedisClient) -> EventLoopFuture<T> which I commented about before. Correct me if I'm wrong but I believe the T is just there because we need support all of the singular operations like $0.get(...) which already return an EventLoopFuture but of different types... If we created an enum RESPCommand we could have

connection.sendBatch([.get("foo"), .set("bar", 1), .increment("buz")])

As part of PR #36 RedisPipeline has been removed from NIORedis.
At a later point, a batching API could be explored again - but it is being left up to higher level libraries at this time.


feat: Add tabs for code snippets

This issue is a request to add tabs for code snippets, like this.
Requirements:

Add as many tabs as you want
Name tabs the way you want
Once a certain tab is chosen, the rest of the tabs on the page should switch to the same tab
While switching between tabs, the page shouldn't jump up or down
The tabs should support markdown
Ideally, the tabs should do code highlighting for specified languages```c
...

What I tried
Found two existing shortcodes

Code toggles - not exactly what we need
Tabs - exactly what we need

Tried to extract the code from the theme but failed due to lack of expertise in javascript, CSS, and HTML

Code toggles #
Code Toggle is implemented in the Hugo docs theme. This shortcode converts given code to YAML, JSON, and TOML formats, then creates a code block with respective tabs.

Hugo docs: Site configuration

Source code

Design system: Code Toggle

Tabs #
Tabs is implemented in the Learn theme. This is exactly what we need.

Basic example: tabs

Tried using it. The rendered result is 3 separate code snippets, no tabs.

Tabs implemented in the Learn theme

Source code
Tried to extract the code. Identified the files listed below and copied them. Apparently, it is not all the code that tabs has in the theme.

Some files containing the code for tabs are:

assets/css/tabs.css
assets/js/jquery-3.3.1.min.js
assets/js/tabs.js
layouts/shortcodes/tab.html
layouts/shortcodes/tabs.html

Other resources
Tried both below, but they seem to lack css and javascript files, because I get several code snippets rendered one after another and no tabs.

Bootstrap Tabs as Hugo Shortcodes
hugo-dynamic-tabs

@horw Feel free to take this issue.


Fix a Bug Serializing Rooms in W00S00 Quadrant

Unable to correctly serialize any rooms in the W00S00 quadrants as they would be deserialized as W00N00.

@jerroydmoore don't forget to upvote your own PR lol

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

dragoonreas
✔

semperrabbit
✔

PostCrafter
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Use meaningful name prefix forMinIO related resource

at least, add prefix minio-

Closed by https://github.com/goharbor/harbor-operator/pull/238/files.


feat: add support for nRF Connect SDK v2.8

See https://github.com/nrfconnect/sdk-nrf/tree/v2.8-branch

toolchain doesn't work on ubuntu24 for v2.8
Should be coming for 2.9.

Superseeded by #111


Double free on prev when running as root

Apparently this bug is not limited to root; it is reproducible with running ls in /home too.

If generate_prompt() sees there's nothing to expand/generate, it returns pwd unconditionally; so prompt ends up pointing to pwd too, thereby leading to a double free.


Not supported response of writable method in ScoreClient

There is an external function with a boolean return type , whenever I try to call that function through the ScoreClient ,an issue is raised that says "Not supported response of writable method in ScoreClient" . However when I call the same function by changing the return type to BigInteger, It did work smoothly.

The return value of the writable external method couldn't get via icx_getTransactionResult of JSON-RPC. that's why ScoreClient does not support it.

However when I call the same function by changing the return type to BigInteger, It did work smoothly.
What does it mean? 'call'?

@External
boolean distribute();

The interface method is defined in this way as it returns data type in the external method. Although the return value can't be obtained during icx_getTransactionResult, the return value can be obtained in inter-score calls. Thus the interface is defined with a return type to match the exact implementation in the contract.
It would be nice to have score-client generate the methods for the above interface method considering it as a void return type. This way the interface would remain the same as the implementation contract and the generated method would also work.


Clean up build scripts

Clean up build scripts

[x] Game Creator
[x] Pixel Creator
[x] Music Creator

This was fixed.


Detecting module when using AWS Codecommit repository as source

Installation details

[ ] IntelliJ IDEA 2018.1.5 (Community Edition)
Build #IC-181.5281.24, built on June 12, 2018
JRE: 1.8.0_152-release-1136-b39 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
[ ] intellij-hcl plugin version 0.6.12
[ ] Terraform version (v0.11.7`)

Terraform Configuration Files
module "lambda_app" {
  source                      = "git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/mytestrepo//src/modules/lambda?ref=v0.1.0"
  app_name                    = "xxxx"
  description                 = "xxxx"
  timeout                        = 10
}

Expected Behavior
The IDE should have detected the module from codecommit after running terraform get command
Actual Behavior
The IDE does not detected the module nor the module outputs
Steps to Reproduce

Create a aws codecommit repo  and added some terraform modules
Use the module from the codecommit repo in a terrraform file as show n above

Could you please add content of .terraform/modules/modules.json here? I'm interested in part which contains your source id.

@VladRassokhin  the module.json file looks as below
{
   "Modules":[
      {
         "Source":"git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/<reponame>//src/modules/cloudwatch_kinesis_monitoring?ref=v0.0.8",
         "Key":"1.kinesis_stream_monitoring;git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/<reponame>//src/modules/cloudwatch_kinesis_monitoring?ref=v0.0.8",
         "Version":"",
         "Dir":".terraform\\modules\\9d259ec562255e7d506220b13856f71e",
         "Root":"src\\modules\\cloudwatch_kinesis_monitoring"
      },
      {
         "Source":"git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/<reponame>//src/modules/lambda?ref=v0.0.8",
         "Key":"1.lambda_app;git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/<reponame>//src/modules/lambda?ref=v0.0.8",
         "Version":"",
         "Dir":".terraform\\modules\\7fadb607477b1833466779c976041ce7",
         "Root":"src\\modules\\lambda"
      },
      {
         "Source":"../cloudwatch_lambda_monitoring",
         "Key":"1.lambda_app;git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/<reponame>//src/modules/lambda?ref=v0.0.8|lambda_app_cloudwatch;../cloudwatch_lambda_monitoring",
         "Version":"",
         "Dir":".terraform\\modules\\bdf827d588683123b90eafddb4a39fb2",
         "Root":""
      }
   ]
}

AFAIU there's no module with source ref=v0.1.0 defined in modules.json but one with ref=v0.0.8. Please run terraform init and ensure there would be record in modules.json with required source

@VladRassokhin  I am indeed using the correct ref.. what i posted was a reference. I am able to download the module from codecommit properly. I ran deleted the local modules folder and rerean terraform init and terraform get commands.  The plugin doesn not recognize the module yet.
module "lambda_app" {
  source                      = "git::https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/cig-tf-modules//src/modules/lambda?ref=v0.0.8"
  app_name                    = "lambda"
  description                 = "lambda to ingest data "
  project                     = "${var.project_name}"
  environment                 = "${var.environment}"
  iam_policy_document         = "${data.aws_iam_policy_document.lambda_iam_role.json}"
  assume_role_policy_document = "${data.aws_iam_policy_document.lambda_assume_role.json}"

  environment_variables = {
    RAW_BUCKET = "${aws_s3_bucket.raw_bucket.id}"
  }

I believe this issue may be a more general issue with using https with git sources - I ran into the same issue with github:
When declaring a module with git::https://github.com.... as source the IDE highlights the module and says the source is missing. However, modules.json shows that the module is in fact installed.
See below for full details.
Terraform source
module "nomad_and_consul_servers" {
  source = "git::https://github.com/hashicorp/terraform-google-consul.git//modules/consul-cluster?ref=v0.2.0"

  gcp_region = "${var.gcp_region}"
  #...
}

module.json
{
"Modules":[
    {"Source":"github.com/hashicorp/terraform-google-consul.git//modules/consul-cluster?ref=v0.2.0",
     "Key":"1.nomad_and_consul_servers;github.com/hashicorp/terraform-google-consul.git//modules/consul-cluster?ref=v0.2.0",
     "Version":"",
     "Dir":".terraform/modules/22c6f959945f3bfa514d8e831b21c42f",
     "Root":"modules/consul-cluster"},
    {"Source":"modules/nomad-firewall-rules",
     "Key":"1.nomad_firewall_rules;modules/nomad-firewall-rules",
     "Version":"",
     "Dir":".terraform/modules/1f0fc681ce873fd5e37d2017d38fc953",
     "Root":""},
    {"Source":"modules/nomad-cluster",
     "Key":"1.nomad_clients;modules/nomad-cluster",
     "Version":"",
     "Dir":".terraform/modules/14324e2a9ad1b1540b52f70fce2a0025",
     "Root":""},
    {"Source":"../nomad-firewall-rules",
     "Key":"1.nomad_clients;modules/nomad-cluster|firewall_rules;../nomad-firewall-rules",
     "Version":"",
     "Dir":".terraform/modules/6d18ccfe735393c8f3df478b094b0255",
     "Root":""},
    {"Source":"git::https://github.com/hashicorp/terraform-google-consul.git//modules/consul-cluster?ref=v0.2.0",
     "Key":"1.nomad_and_consul_servers;git::https://github.com/hashicorp/terraform-google-consul.git//modules/consul-cluster?ref=v0.2.0",
     "Version":"",
     "Dir":".terraform/modules/dbbd79baae0b3bd83d8824315430dc09",
     "Root":"modules/consul-cluster"}
    ]
}

IDE info
PyCharm 2018.2.4 (Community Edition)
Build #PC-182.4505.26, built on September 19, 2018
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.17.14-202.fc28.x86_64

Plugin
version 0.6.12

Same issue here

Most probably it's fixed in 0.6.14, please try and report

I've tested 0.6.14 with module like
module "test-codecommit" {
  source = "git::ssh://git-codecommit.eu-west-1.amazonaws.com/v1/repos/test-terraform-module"
}

and it works fine.


Custom KServe model gives an error which is hard to figure out.

/kind bug
What steps did you take and what happened:
I used Python SDK with version kserve==0.8.0 and it works fine.
After I updated it to kserve==0.10.1, I got following error.
Traceback (most recent call last):
  File "/root/miniconda3/lib/python3.9/site-packages/uvicorn/protocols/http/h11_impl.py", line 373, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/root/miniconda3/lib/python3.9/site-packages/uvicorn/middleware/proxy_headers.py", line 75, in __call__
    return await self.app(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/fastapi/applications.py", line 270, in __call__
    await super().__call__(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/applications.py", line 124, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/middleware/errors.py", line 184, in __call__
    raise exc
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/root/miniconda3/lib/python3.9/site-packages/timing_asgi/middleware.py", line 68, in __call__
    await self.app(scope, receive, send_wrapper)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/root/miniconda3/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/root/miniconda3/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/routing.py", line 706, in __call__
    await route.handle(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/root/miniconda3/lib/python3.9/site-packages/starlette/routing.py", line 66, in app
    response = await func(request)
  File "/root/miniconda3/lib/python3.9/site-packages/fastapi/routing.py", line 235, in app
    raw_response = await run_endpoint_function(
  File "/root/miniconda3/lib/python3.9/site-packages/fastapi/routing.py", line 161, in run_endpoint_function
    return await dependant.call(**values)
  File "/root/miniconda3/lib/python3.9/site-packages/kserve/protocol/rest/v1_endpoints.py", line 69, in predict
    response, response_headers = await self.dataplane.infer(model_name=model_name, body=body, headers=headers)
  File "/root/miniconda3/lib/python3.9/site-packages/kserve/protocol/dataplane.py", line 276, in infer
    response = await model(body, headers=headers)
  File "/root/miniconda3/lib/python3.9/site-packages/kserve/model.py", line 118, in __call__
    else self.predict(payload, headers)
TypeError: predict() takes 2 positional arguments but 3 were given

What did you expect to happen:
It runs normally without errors.
What's the InferenceService yaml:
I used custom-model with KServe.Model and kserve.ModelServer().start([model]) (for test and developing purpose).
Anything else you would like to add:
I added a print line to know what 's going on. (print("DEBUG:", inspect.getfullargspec(self.predict)))
And I got DEBUG: FullArgSpec(args=['self', 'request'], varargs=None, varkw=None, defaults=None, kwonlyargs=[], kwonlydefaults=None, annotations={'return': typing.Dict, 'request': typing.Union[bytes, typing.Dict]})
The function arguments are different with current version... (very weird)
Environment:

Istio Version:
Knative Version:
KServe Version: 0.10.1
Kubeflow version:
Cloud Environment:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube/Kind version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release): Ubuntu 18.04.5

@iwyoo The method signature of predict method is changed in kserve 0.10. Now it has a additional parameter called headers.  Have you incorporate this change in your code ? Also you can refer the custom model example for more information.

@iwyoo The method signature of predict method is changed in kserve 0.10. Now it has a additional parameter called headers. Have you incorporate this change in your code ? Also you can refer the custom model example for more information.

Thank you for answer! The way the code should be different from 0.8.0. I confirmed that it works well in 0.10.1. I'll close the issue!


Replace yaml generation method

Purpose

Replace yaml generation method.
Resolves https://github.com/wso2-cellery/sdk/issues/607

Codecov Report

Merging #608 into master will decrease coverage by 0.18%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master     #608      +/-   ##
==========================================
- Coverage   64.63%   64.44%   -0.19%
==========================================
  Files          34       34
  Lines        1179     1170       -9
  Branches       98       99       +1
==========================================
- Hits          762      754       -8
- Misses        363      379      +16
+ Partials       54       37      -17

Impacted Files
Coverage Δ

...in/java/io/cellery/models/ServiceTemplateSpec.java
77.77% <ø> (-2.23%)
:arrow_down:

.../src/main/java/io/cellery/impl/CreateInstance.java
90.29% <100%> (+2.1%)
:arrow_up:

...src/main/java/io/cellery/impl/CreateCellImage.java
87.03% <100%> (ø)
:arrow_up:

...main/java/io/cellery/models/AutoScalingPolicy.java
90% <100%> (ø)
:arrow_up:

...ts/lang/src/main/java/io/cellery/CelleryUtils.java
53.5% <36.84%> (-1.05%)
:arrow_down:

...nts/lang/src/main/java/io/cellery/models/Cell.java
91.66% <0%> (ø)
:arrow_up:

...g/src/main/java/io/cellery/models/STSTemplate.java
50% <0%> (ø)
:arrow_up:

...ents/lang/src/main/java/io/cellery/models/API.java
75% <0%> (ø)
:arrow_up:

...ents/lang/src/main/java/io/cellery/models/TCP.java
66.66% <0%> (ø)
:arrow_up:

...nts/lang/src/main/java/io/cellery/models/GRPC.java
66.66% <0%> (ø)
:arrow_up:

... and 11 more

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 a89ba63...9c19843. Read the comment docs.


How to test child React component created in Render with React.createElement()

I am trying to test a custom react element built on top of some ad-hoc React elements Code:
return (
    React.createElement("div", {className: classes},
        React.createElement(TextField, {
            ref: "omniTextBox",
            className: "menu-textfield",
            onChange: this._onControlChange
        }),
    )
);

this._onControlChange manipulates state based on textfield value. It is what I want to test, with different input to the textfield triggering different state of component.
Jest code:
var customComponent = TestUtils.renderIntoDocument( <customField /> );
var customField = TestUtils.findRenderedComponentWithType(customComponent, customField);
TestUtils.Simulate.change(customField, {target: {value: 'Hello, world'}});

I tried to mock function for _onControlChange to see if it is called, it is called 0 times. I also tried to see state of the customField var before and after simulate() call, and nothing changes. I wonder if I am missing something to simulate typing in a text field

Please ask questions on Stackoverflow. There are now many React testing solutions available.


resolves #1346

Changelog entry
Add stripped version of yql/tools/mrrun to opensource repo
Changelog category

Not for changelog (changelog entry is not required)

There are failed tests in the ones I added, but they fail in the private repo as well. Maybe we need to just remove them idk


AppExtensionURLStringKey mentioned only once in README?

From the README:

An important thing to notice is the AppExtensionURLStringKey is set to the exact same value we used in the login scenario. This allows users to quickly find the login they saved for your app the next time they need to sign in.

That key is not used in any of the example code. Perhaps it was once used, but no longer? Maybe it should be added back or that paragraph changed to not refer to it anymore?

Good catch @pwightman :+1:
AppExtensionURLStringKey is a constant used internally in OnePasswordExtension. It represents the dictionary key for the URL string we use find and store logins. However, in this context (the README), it makes more sense to refer to the URLString instead.
I just PR'd the changes in #254 to address it :wink:
Cheers!

#1


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2365 ms

Resolved: Samsung performance has improved in 466a699.


[Grid] support customized breakpoints

[x] I have followed (at least) the PR section of the contributing guide.

Fix #26369.
This fixes the problem of passing custom breakpoints as props to the component Grid, the TypeScript throws an error when we pass these dynamic props and that happens because we are not supporting custom breakpoints in TypeScript.

Hi @mnajdova, thanks a lot for your time, I will check the changes!

Hi @mnajdova thanks a lot for taking the time to review the PR, could you please review it?

Hi @Mnajdova, thanks a lot for taking time to clarify things, I apprecicate your precious time. The comments are addressed!

Hi @Mnajdova, thanks a lot for taking time to clarify things, I apprecicate your precious time. The comments are addressed!

Hi @mnajdova, thanks a lot for taking the time to clarify things, I appreciate your precious time. The comments are addressed!

Hi @mnajdova, thanks a lot for your time. Thanks for removing the code!

This PR introduces a regression. It can be seen on https://mui.com/material-ui/react-grid/
<Grid key={12} />

which is because of
https://github.com/mui/material-ui/blob/4ab3a04c86aa5b7f4dfcc06fe2693cb9bd260c36/packages/mui-material/src/Grid/Grid.js#L479
that is because of the changes inside Grid.js


Add @experimental test decorator; honor test exclusion in import/export tests

Resolves #942
Resolves #894

Build failure tracked as #845.


feat: Table widget activation experiment

Description
Set up experiment, to check if users have a chance of activated if they see a Table widget already present on the canvas when they create a new app. This is behind a feature flag
PR fixes following issue(s)
Fixes #25630
Media
https://github.com/appsmithorg/appsmith/assets/12022471/74623e4e-782d-4223-9690-6f7b3eabacf4
Type of change

Experimental change ( behind feature flags)

Testing
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test


Contact details

Update the contact details
** Remove Bhatial's and PJain's details and replace them with the details of Aditya Bharti and Saumya Prabodh Srivastava.

Modifications to the contact form
** Update the email id to<EMAIL_ADDRESS>** Upon submission try to get the page not to redirect to another page altogether. If the from submission confirmation can be displayed as a popup then that would be better.
** What does the field 'Title' even mean? Perhaps, replace it with 'Name' (of sender) ??