Wrong mapping of $id field.

I think this is a wrong behaviour about mapping field in an embedded document.
I'm using version: 3.1.0
This is my example:
Main document
/**
 * @ODM\Document
 */
class Main
{
    /**
     * @ODM\Id
     */
    private $id;

    /**
     * @ODM\EmbedMany(targetDocument="Embedded")
     */
    protected $embedded;
}

Embedded document
/**
 * @ODM\EmbeddedDocument
 */
class Embedded
{
    /**
     * @ODM\Field(type="string")
     */
    protected $id;

    /**
     * @ODM\Field(type="boolean")
     */
    protected $active;
}

In the embedded document there is a field called $id that is a string type, not @ODM\Id.
The problem comes when i try to query that field.
$qb
      ->field('embedded')->elemMatch($qb->expr()
          ->field('id')->equals(1)
     )
;

$qb->getQuery()->execute();

This will result in this query:
"embedded":{"$elemMatch":{"_id":{"$id":"5a0d7aee26567e3c36166391"}}}
But i suppose to have:
"embedded":{"$elemMatch":{"id":1}}
So I investigate in the mapField function and i noticed that:
Doctrine\ODM\MongoDB\Mapping\ClassMetadataInfo line 1063: public function mapField(array $mapping)
  if (isset($mapping['id']) && $mapping['id'] === true) {
            $mapping['name'] = '_id';
            $this->identifier = $mapping['fieldName'];
            ...
 }

If I get the problem correctly, it stems from both embedded and parent document having id field. To pinpoint, thing is that you use $qb->expr()->field('id')->equals(1) and builder tries to help you by converting id to its database representation, and as far as builder knows, it is supposed to work with Main::class where id is @ODM\Id. I'm afraid there's nothing really we can do, I'd suggest using a different field name in the embedded document or diving a bit deeper into builder to see if the problem can be somehow worked-around.

@malarzm I'm not sure that's the issue. The query builder will look for mapping information in the metadata class for Main, not for Embedded. Renaming the field in the embedded document will result in no type conversion. Mapping would properly do type conversion if you called elemMatch('embedded.id'), but the resulting query would be invalid ().
@iskyd Assuming you have a document manager along your query builder, could you try this?
$embeddedExpr = new Doctrine\ODM\MongoDB\Query\Expr($documentManager);
$embeddedExpr->setClassMetadata($documentManager->getClassMetadata(Embedded::class));

$qb
    ->field('embedded')->elemMatch(
        $embeddedExpr->field('id')->equals(1)
    )
;

$qb->getQuery()->execute();

@alcaeus that give me the right mongo query.
"embedded":{"$elemMatch":{"id":1}}

Yeah, I expected it would. I’ll flag it as a bug and see if I can come up with something.

This is also true for non-embedded documents. Consider the following (bad) example:
{     "_id" : ObjectId("5a7b3a589e3af1e2df910e8f"),     "id" : 12345,     "name" : "Blah" }
Despite the bad naming (_id vs id), this is a valid Mongo document (resulting from a direct import from a different data source). A direct query on the collection will work as expected but this can't be mapped using the following:
/** @ODM\Id */
protected $id;

/** @ODM\Key(name="id", type="int") */
private $_externalId;

So that means we can't do $dm->findOneBy(['name' => 'Blah', 'id' => $id]) but $collection->findOne(['name' => 'Blah', 'id' => $id]) works.

I can confirm this behaviour described by @KarunaGovind!
I use XML mapping and have two fields "_id" (the MongoId Object") and an extra ID "id".
        <field fieldName="id" name="_id" id="true" type="string" />
        <field fieldName="employeeId" name="id" id="true" type="string" />

I explicitely set the names to "_id" and "id" but I always get the value of "_id". Maybe the name is normalized and has the "underscore" is removed which would result in "id" in both cases.
I think I once got a warning that "id" was marked as identifier and this should not be changed.
Would be great to have this fixed in the current version too.

@KarunaGovind, @webdevilopers
In ODM, both field name and name (aka database name) are immediately reserved and required to be unique. There may not be a field with a name equal to another fields fieldName attribute. The reason for this is because it's possible for you to create a query like this:
$builder->field('someReference')->references($someObject);

In this case, we wouldn't want you to have to know how exactly someReference is mapped (as ID, DBRef or ref). Also, you shouldn't have to know what identifier $someObject uses. So, we use mapping information as well as the someReference fieldName to get to this query object:
{
    'someReference.$id': ObjectId('123456...'),
}

This replicates behaviour from ORM, which is why it was built this way. Unfortunately, in your case this means that you'll have to find a different fieldName for your id field unless you can change the schema (which isn't always possible).
In the original example, this is a bit different: the query expression contains metadata for the original it was built for, which is why the id field is changed to _id. That's why changing the classMetadata instance in the expression object yields the correct result. One possible fix here is adding an optional $field argument to expr() which would allow you to get an expression based on the metadata of a nested field. I'll see if we can fix this in 1.2 or 1.3.

I'll see if we can fix this in 1.2 or 1.3.

TL;DR: no way for 1.x, no for 2.0. The version in the roadmap is still up-to-date.
My initial idea was to update the expr method in Query\Builder to take a field name and inject the target mapping for the field name into the newly created Expr object instead of the builder's metadata. While this can easily be done for straight-up field names, but when building queries for nested field names (think embedded1.anotherEmbedded.something), this gets complex quite quickly. The only place where we currently have this logic is DocumentPersister::prepareQueryElement, which I'd rather not touch at this time. In order to actually be able to take care of this in DocumentPersister, we'd also have to change the Expr class to not resolve nested expressions when they are used (which is currently done when using elemMatch, and, or, or any other operator that works on a nested query expression). This introduces a lot of complexity which I feel is not tested enough at this time. Instead, this should be a more in-depth refactoring of the persister logic along with a lot of testing to ensure we're not running into any regressions.
While I understand that it is only a workaround, I would still recommend to manually inject a separate ClassMetadata instance in any expo object until we've properly refactored this to take field names into account when preparing nested query expressions.

We currently switch to 2.0. If I see this correctly than it is still no possible to map the Mongo internal ObjectId Field "_id" to a different property on our model than $id? We would prefer e.g. $surrogateId and use $id for our real Domain Model ID instead of e.g. employeeId.

I will have to correct myself on the last comment. Indeed our solution on order to keep the domain model clean and stay with our internal "$id" property we can do this:
        <id field-name="surrogateId" />
        <field field-name="id" type="string" />

Just stumbled upon this issue again. Any updates?

The code mentioned by @iskyd can be found in Doctrine\ODM\MongoDB\Mapping\ClassMetadata l. 1898:
        if (isset($mapping['id']) && $mapping['id'] === true) {
            $mapping['name']  = '_id';
            $this->identifier = $mapping['fieldName'];
            if (isset($mapping['strategy'])) {
                $this->generatorType = constant(self::class . '::GENERATOR_TYPE_' . strtoupper($mapping['strategy']));
            }

No updates to the issue. Having two fields share a name, even when one is the database name and the other is the property name in your domain model is not supported. The following mapping for example is invalid:
<id field-name="id" />
<field name="id" field-name="applicationId" type="string" />

Written as annotations, this would correspond to this:
/** @Id */
private $id;

/** @Field(name="id") */
private $applicationId;

The reason this conflicts is that the query builder no longer knows what to do with this builder expression:
$builder->field('id')->equals('value');

If the builder look at field names (meaning property names in your domain model), the resulting query would be:
{ _id: "value" }

However, the builder also supports passing database names, in which case the generated query would look like this:
{ id: "value" }

Since the query builder doesn't know what to do here, this kind of mapping should be avoided at all costs.
I'll note that the original issue wasn't about this, but rather that the query builder doesn't take field metadata into account when building expressions. In the original example, the id field in the elemMatch query operator uses the metadata from the parent document instead of from the embedded document, resulting in an invalid field name translation. That is a separate issue, but could be avoided by modifying the Query\Expr object after instantiation:
$elemMatchExpr = $qb->expr();
$elemMatchExpr->setClassMetadata($dm->getClassMetadata(Embedded::class); // Makes sure correct translation rules are used
$qb
    ->field('embedded')
    ->elemMatch(
        $elemMatchExpr
            ->field('id')
            ->equals(1)
    )
;

Not sure why I didn't think of this before, but that may solve the original problem @iskyd was reporting.
@webdevilopers if there is a different issue you're referring to, please create a new issue and I'll investigate it. Thanks!

Thanks for the instant feedback @alcaeus ! I get your point.
Our use case is the following:
We design persistence-ignorant Domain Models. We add our own IDs as value objects. The MongoDB required _id in our collections is auto-added by MongoDB, we ignore it in our application. This way we can easily switch to a different database without having to care about keys and their naming.
For instance:
/**
 * @MongoDB\Document(collection="account_details", readOnly=true)
 */
class Account
{
    /**
     * @MongoDB\Id()
     * @internal
     */
    protected $id;

    /**
     * @MongoDB\Field(type="string")
     */
    protected $accountId;
}

/**
 * @MongoDB\Document(collection="business_profile_details", readOnly=true)
 */
class BusinessProfile
{
    /**
     * @MongoDB\Id()
     * @internal
     */
    protected $id;

    /**
     * @MongoDB\Field(type="string")
     */
    protected $businessProfileId;

    /**
     * MongoDB\ReferenceOne(targetDocument=Account::class, storeAs="id", name="accountId")
     */
    protected $account;

The issue is that we cannot link these to documents via SonataAdminBundle since it only seems to allow references via properties marked with @Id. This is indeed not related to this bundle per se.

I understand, but not entirely. Here's what I'd do:
/**
 * @MongoDB\Document(collection="account_details", readOnly=true)
 */
class Account
{
    /**
     * @MongoDB\Id(type="string", strategy="none")
     */
    protected $accountId;
}

/**
 * @MongoDB\Document(collection="business_profile_details", readOnly=true)
 */
class BusinessProfile
{
    /**
     * @MongoDB\Id(type="string", strategy="none")
     */
    protected $businessProfileId;

    /**
     * MongoDB\ReferenceOne(targetDocument=Account::class, storeAs="id", name="accountId")
     */
    protected $account;
}

This will lead to these documents:
{
    _id: 'abc'
}

{
    _id: 'def',
    accountId: 'abc'
}

I'm not sure I understand what switching databases has to do with this. Here's the same document that doubles as an ORM entity for any SQL-based database:
/**
 * @MongoDB\Document(collection="account_details", readOnly=true)
 * @ORM\Entity(readOnly=true)
 * @ORM\Table(name="account_details")
 */
class Account
{
    /**
     * @MongoDB\Id(type="string", strategy="none")
     * @ORM\Id
     * @ORM\Column(type="string")
     */
    protected $accountId;

    /**
     * @MongoDB\ReferenceMany(targetDocument=BusinessProfile::class, mappedBy="account")
     * @MongoDB\ManyToOne(targetDocument=BusinessProfile::class, mappedBy="account")
     */
    protected $businessProfiles;
}

/**
 * @MongoDB\Document(collection="business_profile_details", readOnly=true)
 * @ORM\Entity(readOnly=true)
 * @ORM\Table(name="business_profile_details")
 */
class BusinessProfile
{
    /**
     * @MongoDB\Id(type="string", strategy="none")
     * @ORM\Id
     * @ORM\Column(type="string")
     */
    protected $businessProfileId;

    /**
     * @MongoDB\ReferenceOne(targetDocument=Account::class, storeAs="id", name="accountId")
     * @MongoDB\ManyToOne(targetDocument=Account::class, inversedBy="businessProfiles")
     */
    protected $account;
}

But what about queries? One identifier is called _id in the database, the other one account_id or accountId depending on naming schema? Query builders cover this:
$managerRegistry
    ->getManagerForClass(Account::class)
    ->createQueryBuilder(Account::class)
    ->field('accountId')
    ->equals('abc')
;

I'm curious which part of your use-case I missed, because I've seen this work for projects. Maybe its due to SonataAdminBundle making false assumptions, in which case I'd happily help fixing this.


Add option for Nodejs path like BUN_PATH

Describe the bug
Reflex tries to install the required node version by itself into data directory but this fails on systems like NixOS or Guix because its not allowed to run binaries directly
NixOS cannot run dynamically linked executables intended for generic
linux environments out of the box. For more information, see:
https://nix.dev/permalink/stub-ld

To Reproduce

reflex init and choose any template
reflex run

Expected behavior
Reflex runs without any errors just like in the documentation
Specifics (please complete the following information):

Python Version: Python 3.12.5
Reflex Version: 0.5.10
OS: NixOS 22.04

Additional context
Related issue: https://github.com/reflex-dev/reflex/issues/1470

Hmm... I checked and it seems to return a path that is OK
In [2]: get_node_path()
Out[2]: '/run/current-system/sw/bin/node'

In my case, The data directory  is ~/.local/share/reflex/, and if I were to delete this, then it should use the which("node") but that is not the case, it re-installs the node binary and then the function above returns something different
In [2]: get_node_path()
Out[2]: '/home/kraanzu/.local/share/reflex/fnm/node-versions/v18.17.0/installation/bin/node'

and hence the error. I'll do a bit more digging on this

I think this function might be the issue while deciding whether to install node or not:
def check_node_version() -> bool:
    """Check the version of Node.js.

    Returns:
        Whether the version of Node.js is valid.
    """
    current_version = get_node_version()
    if current_version:
        # Compare the version numbers
        return (
            current_version >= version.parse(constants.Node.MIN_VERSION)
            if constants.IS_WINDOWS
            else current_version == version.parse(constants.Node.VERSION)
        )
    return False

I'm currently using node version 18.20.4 which was the last version before they moved to 19.x
The function above checks for total equality which causes the problem here, even the patch version update will return a False

#3981 no longer solve this issue, but a follow up PR will.

I also saw another mentioned PR which changed how the nodejs binary is checked, is reflex now supporting latest nodejs ?

With PR #4006, we've added a CI check to run the integration test with latest node (22.9.0 at the time of writing this, but CI will always check the latest one.)
You can either try the PR or wait for 0.6.1 which should include this fix.
export REFLEX_USE_SYSTEM_NODE=1 will make sure that Reflex use the node found at which node instead of the fnm installed one.

Awesome. Thanks a lot


Update logo in README

Current State:
The Meshery logo used in the README is outdated.
Desired State:
Use the new logo as seen in https://github.com/layer5io/meshery-istio's README.

// @arturba

I'll take a look into in ;)

I'm not sure what do you want to achieve. Do you prefer this logo?

the repository you have linked has the same logo as this

Sorry for the confusion. You figured it out correctly. 😀thank you, @ArturBa.


silence preprocessor redefinition warnings

In file included from /home/vagrant/devel/vinecopulib/src/../include/bicop_class.hpp:38:0,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_parametric.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_archimedean.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/bicop_archimedean.cpp:20:
/home/vagrant/devel/vinecopulib/src/../include/c_tools.h:8:0: warning: "false" redefined
 #define false 0
 ^
In file included from /usr/include/c++/4.9/bits/atomic_base.h:36:0,
                 from /usr/include/c++/4.9/atomic:41,
                 from /usr/include/boost/math/special_functions/detail/bernoulli_details.hpp:19,
                 from /usr/include/boost/math/special_functions/bernoulli.hpp:16,
                 from /usr/include/boost/math/special_functions/gamma.hpp:35,
                 from /home/vagrant/devel/vinecopulib/src/../include/boost_tools.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_class.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_parametric.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_archimedean.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/bicop_archimedean.cpp:20:
/usr/lib/gcc/x86_64-linux-gnu/4.9/include/stdbool.h:42:0: note: this is the location of the previous definition
 #define false false
 ^
In file included from /home/vagrant/devel/vinecopulib/src/../include/bicop_class.hpp:38:0,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_parametric.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_archimedean.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/bicop_archimedean.cpp:20:
/home/vagrant/devel/vinecopulib/src/../include/c_tools.h:9:0: warning: "true" redefined
 #define true (!false)
 ^
In file included from /usr/include/c++/4.9/bits/atomic_base.h:36:0,
                 from /usr/include/c++/4.9/atomic:41,
                 from /usr/include/boost/math/special_functions/detail/bernoulli_details.hpp:19,
                 from /usr/include/boost/math/special_functions/bernoulli.hpp:16,
                 from /usr/include/boost/math/special_functions/gamma.hpp:35,
                 from /home/vagrant/devel/vinecopulib/src/../include/boost_tools.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_class.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_parametric.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/../include/bicop_archimedean.hpp:23,
                 from /home/vagrant/devel/vinecopulib/src/bicop_archimedean.cpp:20:
/usr/lib/gcc/x86_64-linux-gnu/4.9/include/stdbool.h:43:0: note: this is the location of the previous definition
 #define true true

Where are we on this?

do we actually need to define true/false there?

Probably not, we'll modify in the clean-up.

fixed by 146087fd7c7c76a55fc22e764e82537cb26a5e98


Chats (Hangouts and Talk) must be fetched with "format": "full"

Hangouts don't have a raw representation. But the text can still be fetched with "format": "full". This needs to be handled in a special way if you are defaulting to raw.

This also applies to older Google Talk messages. The raw representation is also truncated. So it seems you want to use full for all chat messages.

Backing up chats and restoring chats via GYB / Gmail API has never worked very well, they can't be restored back to a CHATS label and they usually end up unthreaded if they are restored. Changed GYB to use a default search of "-is:chat" for backup and to NOT backup any messages it sees that have a label of CHATS.


COMP: Libpng update

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

This change in clang: https://github.com/llvm/llvm-project/pull/74676
Breaks the build of ITK master on macOS.
We need this fix:
libpng: https://github.com/pnggroup/libpng/pull/529


Add a README in Home Assistant example

Hey,
I'm totally clueless on how to use the example about Home Assitant (https://github.com/rhasspy/rhasspy/tree/master/examples/homeassistant) . Sounds awesome to have a script to parse every entity on training... But I've no idea how to use it / where to put these files. A README and/or some entry in the doc would be much appreciated.
I'm ok to open a PR with some proper doc if anybody can quickly explain it to me here.
Cheers

A full tutorial on setup up Rhasspy with Home Assistant would be welcome.
So far I'm stuck at having a base station and satellite installed and managing to have the satellite "speak" via MQTT.
I sort of managed to the the "GetTime" intent working.
But I have no idea on how to capture the "HassTurnOn" intent that was created using the Home Assistant Example.
How can I capture the light/switch entity in the "HassTurnOn" intent and how do I link this back to a light/switch entity I can toggle or set the brightness of?

A full tutorial on setup up Rhasspy with Home Assistant would be welcome.
So far I'm stuck at having a base station and satellite installed and managing to have the satellite "speak" via MQTT.
I sort of managed to the the "GetTime" intent working.
But I have no idea on how to capture the "HassTurnOn" intent that was created using the Home Assistant Example.
How can I capture the light/switch entity in the "HassTurnOn" intent and how do I link this back to a light/switch entity I can toggle or set the brightness of?

Check out the new Wiki page:
https://github.com/rhasspy/rhasspy/wiki
You can always ask those questions on the community forum as well
https://community.rhasspy.org/


ci: remove if condition on pypi publish steps

Prior to this commit, there was an if statement on the PyPI publish steps in the publish job. However, the publish job is already being restricted to run only on releases. Therefore the if statement is not necessary. Morever, it looks like these PyPI publish steps were being skipped anyway; perhaps due to faulty match criteria.
In this commit, remove the if condition on each PyPI publish step, falling back to the publish job rules restricting publishing to tagged release events.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (5340768) to head (66ec8c4).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           68        68
=========================================
  Hits            68        68

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


tortoise: setup for model based tests

this pr implements initial infrastructure for writing model based tests, including one example.
the end goal is to implement sophisticated infra for writing long running tortoise simulations (including failure conditions), with sophisticated properties.
code organization heavily borrows from https://p-org.github.io/P/. every independent model (subsystem) is implemented as a separate object. core, beacon, hare are all independent models. they interact by exchanging messages. it enables simulation of failures for independent components. messages may get dropped or reordered. additionally every model may send events, unlike messages events are not dropped or reordered and guaranteed to be received by monitors.
TestBasicModel implements a simulation where part of smeshers are failing at every layer, but the tortoise is expected to continue verifying layers.

bors try

what do you think of adding anther MessageEpochStart to separate those per-epoch data and per-layer data?

actually even LayerEnd event needs to be removed. LayerStart is like a regular clock event, and layer end needs to be replaced. i don't see a good reason to add more events

what do you think of adding anther MessageEpochStart to separate those per-epoch data and per-layer data?

actually even LayerEnd event needs to be removed. LayerStart is like a regular clock event, and layer end needs to be replaced. i don't see a good reason to add more events

yeah. i think LayerEnd doesn't really make sense. for me MessageEpochStart/MessageLayerStart make more sense in the modeling.

i think the code will be clearer to have MessageEpochStart/MessageEpochEnd also.

i will experiment with them once i will get back to this

bors merge


Can't send HostConfig.* options to createContainer

(I'm not sure if this is really a dockerode issue, or a docker-modem issue, but hopefully this is an ok place to discuss).
Any nested config parameters, like the HostConfig section in the create container end point (https://docs.docker.com/reference/api/docker_remote_api_v1.18/#create-a-container) are not passed along to Docker when using dockerode.
Dockerode calls docker-modem with a ? at the end of the /container/create endpoint (https://github.com/apocas/dockerode/blob/8c1450bcb05f2e03b7dfc50afd8a9c9b40a153ba/lib/docker.js#L22), which causes docker-modem to send the options as a query string. This is needed because the name property must be set in the query string, not as JSON in the POST body.
However, all of the other options also get sent in the query string. Any nested config parameters, such as the HostConfig section in the options do not get serialised into the query string properly. Setting them as top level options, e.g. HostConfig.Dns gets them in the query string, but seems ignored by Docker.
I think the correct way to call this endpoint in the Docker API would be with the name option in the query string, and all the other options as JSON in the POST body. However, it seems that docker-modem takes an all-or-nothing approach, and it's not easy to separate one option key that should be sent in the query string.
I'm happy to submit a pull request for this, but just wanted to get your opinion first. Likely the approach I would take is to add something like a queryStringKeys option into the modem.dial method which would pull these out and send them in the query string, and then send the rest as JSON. The createContainer endpoint would then just need queryStringKeys: ["name"].
Thoughts?

Sorry, by bad. The Hostconfig options are still sent in the body, even when serialised in the query string. The problem was actually that the Dns entry should be specified on container start, not create (seems like the Docker docs are out of date).


Update AIKnowledgeRepresentation-section.adoc

improper use of where

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

+1 Thanks @cronos2546 !

ok to test


Feature Request: add charset/collate option during a mysql db resource creation

Hi, I would like to know if its possible to add the charset and collate options during the create mysql db resource creation as it defaults to the default variables within the mysql module
example switch to bin form general_ci
charset       = 'utf8'
collate        = 'utf8_bin'

Hi Ian,
At the moment I'm busy with some testing. If these succeed, I'll push them to github in the next few days (I'll update this ticket).
Hopefully I can create an release in the next week to, so it will also be available on the forge.
Kind regards,
Werner

It is in release 1.6.0 which was released few minutes ago. Can be downloaded from the Forge and github.


Flagged as Virus

Great tool!
Just an observation that any .exe created with this library seems to be flagged as viruses by Google (Drive) and Microsoft (Windows Defender).
For example, if you compile this to an exe and put it in Google Drive:
package main

import (
    "github.com/elazarl/goproxy"
    "log"
    "net/http"
)

func main() {
    proxy := goproxy.NewProxyHttpServer()
    proxy.Verbose = true
    log.Fatal(http.ListenAndServe(":8080", proxy))
}

When you try to download it, you get:

And furthermore, Google doesn't allow you to share it with others. So only the owner can download it.
Microsoft similarly instantly quarantines it once you try to run it.
It could just be the nature of the code, but just wanted to bring up this issue in case anyone had ideas for workarounds

Thanks,
Not sure how to resolve this issue.
On Wed, Jul 31, 2024, 20:55 Bryan Gillespie @.***>
wrote:

Great tool!
Just an observation that any .exe created with this library seems to be
flagged as viruses by Google (Drive) and Microsoft (Windows Defender).
For example, if you compile this to an exe and put it in Google Drive:
package main
import (
"github.com/elazarl/goproxy"
"log"
"net/http"
)
func main() {
proxy := goproxy.NewProxyHttpServer()
proxy.Verbose = true
log.Fatal(http.ListenAndServe(":8080", proxy))
}
When you try to download it, you get:
image.png (view on web)
https://github.com/user-attachments/assets/fd437df7-9b67-484d-812b-c398d4f8ecc2
And furthermore, Google doesn't allow you to share it with others. So only
the owner can download it.
Microsoft similarly instantly quarantines it once you try to run it.
It could just be the nature of the code, but just wanted to bring up this
issue in case anyone had ideas for workarounds
—
Reply to this email directly, view it on GitHub
https://github.com/elazarl/goproxy/issues/544, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7RIQHANLMTDR3EZ7A6A3ZPEQJJAVCNFSM6AAAAABLY664G2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DANJYGYZDSNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

We can close the issue, there's probably nothing you can do except advise people using the library to EV sign exes intended for distribution.


Question: Use action / hooks / filter?

Hey, is it posible to use action / hooks and filters? To, as example, fire some from the symfony part of the application the to the wordpress part?

Yes you can use all WordPress functions in Symfony as long as they are called after it's loaded. First you have to enable your theme to make it Symfony compatible, then you create WordPress-aware services. Those are able to call WordPress functions and hooks. Any service called by a WordpressService is able to make use of WordPress functions and hooks.

Symfony and WordPress are merged together, so yes you can call WordPress actions and filters from Symfony.

Is it possible to do this before Wordpress initialized (which initialize after Symfony) ?
I need to call some filters with apply_filters before initialize_services() is called in my child theme. I can't trigger Wordpress Hooks to achieve this because I need to call it when my controller is executed

What do you think of this solution (not tested), assuming you're using a custom Symfony controller that triggers the Wordpress loader:

create a Symfony custom event MyEvent extends Event
create a Wordpress service MyService extends AbstractWordpressService, and in the constructor your inject EventDispatcherInterface and immediately listen to MyEvent. In the listener you store the event data in a property of MyService
in your Symfony controller, you dispatch MyEvent with the data you need, before Wordpress initialization. At this point your data is stored in MyService but Wordpress isn't initalized yet
call apply_filters in MyService->initialize(), using the data you previously stored

Hi William.
It's a controller who renders a Twig, so controller is not triggering the Wordpress loader I think

Hi Loïc,
From what I understand, you want to make Wordpress render some html or some output in your controller using apply_filters, then print it in your Twig template.
Some possibilities:

create a Wordpress API endpoint, then call it with file_get_contents() before rendering the view
render nothing then call an ajax request to Wordpress that will return the rendered output

Not sure how to do it otherwise as Wordpress can't be unloaded... So you must find a way to isolate the rendering process.


Error : javax.net.ssl.SSLHandshakeException after adding amazon WAF(firewall)

In one of the services in my code, I am calling a 3rd party server to fetch the details (I am using Unirest to connect to the server and get the response). This process used to work fine till they added WAF(firewall) in the server. After enabling WAF in server, I am getting handshake failure exception error.
javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
I did some research and was able to fix the issue by adding CA certificate into truststore of server by using below command.
sudo keytool -importcert -keystore $JAVA_HOME/jre/lib/security/cacerts -storepass changeit -file  -alias “”
Just to verify , I asked them to remove the firewall and tested with same certificate it worked.
Question : Why did it fail when WAF(firewall) was added in server which was not failing before?
Note : Before and after adding WAF(firewall), certificates were same.
Unirest version : 1.4.9 ; Java version : 1.8
Thanks in advance.

It's possible that the firewall  has a intermediate cert that is sniffing packets for security reasons. Doesn't really have anything to do with unirest specifically. You would have had the same issue with literally any Java (and possibly other languages) network clients.


another caching bug

  ;; this is bugged. apparently results are cached.
  #_(rp/reg-query-sub
   ::entity-id-by-attribute-value
   '[:find ?e . :in $ ?a ?v :where [?e ?a ?v]])

  (rf/reg-sub
   ::entity-id-by-attribute-value
   (fn [_ [_ attr val]]
     (d/q '[:find ?e . :in $ ?a ?v :where [?e ?a ?v]] @connection attr val)))

First one does not return a different value if entities are retracted and re-added. Latter one does.

May be related to: #41

Hello @jleonard-r7 . Thank you for reporting this issue, I would really love to fix it, but just don't have enough time for that so far. If you like to figure out it by yourselves, there are some suggestions. re-posh is a simple adapter, which connects posh and re-frame framework. I think it's really unlikely that this caching bugs belong to re-posh and most likely they are somewhere inside posh. The author of posh is not maintained it anymore, so I decided to do it myself, but as I said recently didn't have much time.
I suggest you trying to reproduce these caching bug using just a pure posh (the changes should be minimal actually) and then dig into this library


Please update the Changelog.md with the latest v0.8.5

Thanks for reviving this crate! Please update the Changelog.md with the changes shipped in the latest v0.8.5.

Done


Use tube identifiers to plate samples

Note, this PR is currently incomplete and still needs a fair amount of work
(mainly surrounding the tests). I've opened this PR to run the full test suite.

This is now ready for review. I would appreciate a careful review, specifically of the sample plating interface since this is the main aspect that was changed. The main change is that when a study has a "specimen_id_column" set, then the user can plate using the values in that column. When this column is not set, the user has to plate the samples using the sample identifiers as defined by Qiita. See this example for the same study except one has a specimen_id_column and one doesn't:
Without specimen_id_column (users need to type the qiita id):

With a specimen_id_column (users need to type the specimen identifiers):

Here's a few notes:

I've updated the Travis build to use the current dev branch of Qiita. Since the master branch didn't have the most recent changes that are needed for this PR. This can be changed at a later time.
The PATCH request that dealt with the well updates will now handle using the specimen identifiers if such a column is present in the study's description.
Since I cannot modify the specimen_id_column attribute via the Python API (because labman doesn't have access to that attribute), I've decided to instead make direct queries against the DB.
Should we allow users to plate samples even if no study has been selected? I am not sure why this is possible, but it is. I would be happy to remove this altogether if we can all agree on that.

Thanks for the review @AmandaBirmingham!

I don't think I understand how the functionality works when samples from more than one study are being plated on the same plate. How does the user switch between studies to pick their preferred active one?

This functionality was already provided with the current interface, so it didn't require any additional changes. For example see in this animated gif how I have two studies to plate both of which have sample names with the letter s. However, depending on the study that is selected the suggestions and validations will pop up as expected. At the end of the animation, I add and select another study and type in the letter a which reveals only sample names for that third study.

Let me know if you would rather go through this in person, I can come find you by your office. In the meantime I'll finish addressing your review comments.

Awesome!

Agreed!  Thanks, @ElDeveloper!


Add check for bash --version

Add check for bash --version:
Example: https://github.com/adoptium/TKG/blob/6e73a9fa473738b6e92c7a65e59588b5b0f408a8/src/org/openj9/envInfo/MachineInfo.java#L194

I would like to work on this issue.

@mayTkhin24 are you still working on this? If you have any questions, please let us know. Thanks

Since this issue requires access to internal machines to check on plinux, zlinux, aix, zos, etc. Re-assign to @sophiaxu0424 then.

Thanks @LongyuZhang, will a take look at it sooner today/tmr

Closed by https://github.com/adoptium/TKG/pull/574. Thanks @sophiaxu0424.


A few minor issues in imageBlur

Thanks for the interesting sample!
I noticed these minor issues:

buffer0 and buffer1 in main.ts are associated with flip in the the shader, but where are these buffers being written to? I didn't see associated buffer write calls.
samp seems unused in the shader.
cubeTexture - naming is a little confusing.

buffer0 and buffer1 in main.ts are associated with flip in the the shader, but where are these buffers being written to? I didn't see associated buffer write calls.

They are set when they are created using mappedAtCreation, calling getMappedRange(), setting the value, and then unmaping.

buffer0 is just a buffer with 0 in it
buffer1 is just a buffer with 1 in it

One or the other is bound to flip. Bind buffer0 to not flip, bind buffer1 to flip. They don't need to be updated.

samp seems unused in the shader.

Yes. I'll remove it

cubeTexture - naming is a little confusing.

Agreed. Will rename

They are set when they are created using mappedAtCreation, calling getMappedRange(), setting the value, and then unmaping.

Sorry, missed that. Thanks for the quick response!


Change .sbtopts to .jvmopts because it works on Windows

Can some people try this out?  Namely, rename .sbtopts to .jvmopts on your Macintosh or Linux computer and see if it works just as well?  It appears to be one way to keep the source distribution compatible across operating systems, assuming it works.  Thanks for trying.

bsharp@Beckys-MacBook-Pro ~/github/eidos (bullets)]$ sbt test
Unrecognized option: -J-Xmx3g
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.
On Tue, Apr 10, 2018 at 12:53 PM, Keith Alcock<EMAIL_ADDRESS>wrote:

@kwalcock https://github.com/kwalcock requested your review on:
clulab/eidos#279 https://github.com/clulab/eidos/pull/279 Change
.sbtopts to .jvmopts because it works on Windows.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/279#event-1567043977, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFInid1qcJzosCW4bNO8IubEtrScThk2ks5tnQ2ngaJpZM4TO5YK
.

Sorry, just -Xmx6g.

I think it works -- set it to 1g and got a bunch of java heap space OOM
errors :)
not the most thorough diagnostic, but.....
On Tue, Apr 10, 2018 at 1:11 PM, Keith Alcock<EMAIL_ADDRESS>wrote:

Sorry, just -Xmx6g.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/279#issuecomment-380231172, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniZv2fPkcBGRSyqrztyDJ3xc-qHSEks5tnRH6gaJpZM4TO5YK
.

and back at 6G works swimmingly
On Tue, Apr 10, 2018 at 1:36 PM, Rebecca<EMAIL_ADDRESS>wrote:

I think it works -- set it to 1g and got a bunch of java heap space OOM
errors :)
not the most thorough diagnostic, but.....
On Tue, Apr 10, 2018 at 1:11 PM, Keith Alcock<EMAIL_ADDRESS>wrote:

Sorry, just -Xmx6g.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/clulab/eidos/pull/279#issuecomment-380231172, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniZv2fPkcBGRSyqrztyDJ3xc-qHSEks5tnRH6gaJpZM4TO5YK
.


SessionController worker dies during start, stops connecting shards

General Troubleshooting
Hey there! Before you report a bug or suggest a new feature,
please make sure to follow these steps first!

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version. (technically no, this happened on 332, but as far as I can see the relevant class did not receive any changes)
[x] I have checked the branches or the maintainers' PRs for upcoming features/bug fixes.

This is not the place to learn Java. Please refer to StackOverflow
for your general programming questions.

Issue
Issue Type

[x] Bug Report

For questions join the official JDA discord server: https://discord.gg/0hMr4ce0tIl3SLv5

Description
During startup this happened:
com.neovisionaries.ws.client.OpeningHandshakeException: The status code of the opening handshake response is not '101 Switching Protocols'. The status line is: HTTP/1.1 520 Origin Error
    at com.neovisionaries.ws.client.HandshakeReader.validateStatusLine(HandshakeReader.java:232)
    at com.neovisionaries.ws.client.HandshakeReader.readHandshake(HandshakeReader.java:54)
    at com.neovisionaries.ws.client.WebSocket.readHandshake(WebSocket.java:3231)
    at com.neovisionaries.ws.client.WebSocket.shakeHands(WebSocket.java:3110)
    at com.neovisionaries.ws.client.WebSocket.connect(WebSocket.java:2142)
    at net.dv8tion.jda.core.requests.WebSocketClient.connect(WebSocketClient.java:448)
    at net.dv8tion.jda.core.requests.WebSocketClient$StartingNode.run(WebSocketClient.java:1475)
    at net.dv8tion.jda.core.utils.SessionControllerAdapter$QueueWorker.run(SessionControllerAdapter.java:195)
java.lang.IllegalStateException: com.neovisionaries.ws.client.OpeningHandshakeException: The status code of the opening handshake response is not '101 Switching Protocols'. The status line is: HTTP/1.1 520 Origin Error
    at net.dv8tion.jda.core.requests.WebSocketClient.connect(WebSocketClient.java:453)
    at net.dv8tion.jda.core.requests.WebSocketClient$StartingNode.run(WebSocketClient.java:1475)
    at net.dv8tion.jda.core.utils.SessionControllerAdapter$QueueWorker.run(SessionControllerAdapter.java:195)

Shards stopped getting connected after that.

Fixed in release 3.5.1


Write a goal/outline for 1.0, plan milestone for v0.2

I'll be working on that today, before presenting the project at the Kyoto indies meetup tonight.

Done, see https://github.com/GDquest/godot-turn-based-rpg/milestones


[Bug] labels of glue/ax are all -1

ax = nlp.load_dataset('glue', 'ax')
for i in range(30): print(ax['test'][i]['label'], end=', ')

-1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,

This is the test set given by the Glue benchmark. The labels are not provided, and therefore set to -1.

Ah, yeah. Why it didn’t occur to me. 😂
Thank you for your comment.


Localize cardinal directions

Coordinates
Compass

#567 should resolve the compass part of this, the coordinates will need more changes in TS core for this to work.


build(deps): bump github.com/spf13/cobra from 0.0.7 to 1.0.0

Prerequisites
Put an x into the box(es) that apply:

[ ] This pull request fixes a bug.
[ ] This pull request adds a feature.
[ ] This pull request enhances existing functionality.
[ ] This pull request introduces breaking change.

For more information, see the Contributing Guide.
Description
Describe what this pull request achieves. Ensure you have read the Contributing Guidelines document before submitting.
Issues Resolved
List any existing issues this pull request resolves.
Checklist
Put an x into all boxes that apply:

[ ] I have read the Contributing Guidelines.

Tests

[ ] I have added tests to cover my changes.
[ ] All tests pass when I run make test.

Documentation

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Code Style

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

Codecov Report

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


WheelEvent constructor in lib.d.ts

The WheelEvent constructor should be able to take in an initialized dictionary as described here
https://developer.mozilla.org/en-US/docs/Web/API/WheelEvent/WheelEvent
However, the lib.d.ts file does not have this constructor defined

This part of lib.d.ts is automatically generated from IE's definitions and as you can see on that page IE doesn't support this, which is the cause here. We'll need to revisit this along with the other lib.d.ts bugs that are affected similarly.


[CPU] gravitee-am-api use 3 CPU to start

We deploy gravitee-am in kubernetes with a helm chart.
But when the resource limiting to a lower value (e.g. 500m, 1 or 2), management api starts slower and this slowness seems to lead to a race condition : /admin context returns 404

Hi @NAJWAJEMAA any news on your side ?


BetterCurses.curse(curseName) throwing TypeError

Most of the time the curse method throws a type error when trying to access user.targets.ids
Error message:
Uncaught (in promise) TypeError: user.targets.ids is not iterable     at Object.curse (curse.js:10)     at eval (eval at execute (foundry.js:32837), <anonymous>:4:14)     at Macro.execute (foundry.js:32837)     at Hotbar._onClickMacro (foundry.js:20566)     at HTMLLIElement.dispatch (jquery.min.js:2)     at HTMLLIElement.v.handle (jquery.min.js:2)
As far as I can see targets inside the user object is a Set so I don't understand why the module accesses an ids parameter?
Or could this be a problem with my vtt version (0.7.9) ?
Below is a screenshot when the error happens.

As you can see on the right of the screenshot a target token is indeed selected.

Thank you for reporting this. I can't replicate it, on my machine, also running 0.7.9, running user.targets.ids gives me an array of all the ids, but a fix should be relatively simple to execute. Expect it by the end of the week.

Great! Thanks for looking into this.
I somehow got the method working two times, but I couldn't reproduce it. When I use the debugger user.targets is always defined as a Set.


tests/lib/hardware: Fix unknown libcloud_conn var

Commit 617dc5a688dd did refactor some stuff and renamed libcloud_conn
to conn. One place was not renamed. Fixed now.
Signed-off-by: Thomas Bechtold<EMAIL_ADDRESS>

Ah, this looks like my mistake in merging my local branch. Sorry about that! Good catch :-)

Maybe you can also create PRs? That makes it more visible to others (eg. me:)) when you changed something.


flet: fix out of the box experience

Description of changes
Right now flet just tries to download the runtime bundle and fails, the objective of this PR is to fix that.
Basically I created this because flet changed a lot since I last tried in #251877
The plan here is to use their monorepo instead of the pypi packages, use a update script on flet-flutter-client and it should propagate to the python libraries so the core and the utilites (from the same repo) would always be synced.
Blockers:
[ ] Update to flutter 3.18>

Errors around language support with the http library
fl_chart errors: TextScaler not found

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

4 packages failed to build:

python311Packages.flet
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist

6 packages built:

flet-client-flutter
flet-client-flutter.pubcache
python311Packages.flet-core
python311Packages.flet-core.dist
python312Packages.flet-core
python312Packages.flet-core.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

2 packages failed to build:

python311Packages.flet
python311Packages.flet.dist

8 packages built:

flet-client-flutter
flet-client-flutter.pubcache
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python312Packages.flet-core
python312Packages.flet-core.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

10 packages built:

flet-client-flutter
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet-core
python312Packages.flet-core.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

10 packages built:

flet-client-flutter
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet-core
python312Packages.flet-core.dist

How can I fix this ofborg issue?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3288

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/generic-error-with-ofborg/38498/1

How can I fix this ofborg issue?

I wasn't able to spot the issue. Maybe people experienced with Flutter like @hacker1024 can help.

The third commit message should be python311Packages.flet: add lucasew as maintainer.

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

11 packages built:

flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet-core
python312Packages.flet-core.dist

How can I fix ofborg?
What is going on?

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

11 packages built:

flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet-core
python312Packages.flet-core.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

4 packages failed to build:

python311Packages.flet
python311Packages.flet.dist
python312Packages.flet
python312Packages.flet.dist

11 packages built:

flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python312Packages.flet-core
python312Packages.flet-core.dist
python312Packages.flet-runtime
python312Packages.flet-runtime.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

15 packages built:

flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet
python312Packages.flet-core
python312Packages.flet-core.dist
python312Packages.flet-runtime
python312Packages.flet-runtime.dist
python312Packages.flet.dist

Result of nixpkgs-review pr 279936 run on x86_64-linux 1

15 packages built:

flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
python311Packages.flet
python311Packages.flet-core
python311Packages.flet-core.dist
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet
python312Packages.flet-core
python312Packages.flet-core.dist
python312Packages.flet-runtime
python312Packages.flet-runtime.dist
python312Packages.flet.dist

@ofborg build python311Packages.flet

@wegank why is your name on the commits? What did you change?

I rebased this PR on staging-next because there are conflicts due to the mass Python update. My name isn't on the commits (check git log for that).


Annotations not detected for wrapped function

Not sure if this is a missing feature or if I'm doing something wrong, but I have some functions that I create using a higher order function. For example:
local function wrap(arg1, arg2)
  return function(...)
    -- Do work
  end
end

---@param foo string
---@param bar integer
M.exported_fn = wrap('a', 'b')

In this example, I can't get exported_fn to be picked up by lemmy-help. Is there something else I should be doing to get it to detect properly?

Got it. I'd like to leave this open as a feature request then

With the caveat that I haven't looked at any of the code, it seems like this should be discoverable by searching for ---@param and/or ---@return annotations, ignoring the code entirely. I can't think of a case where those would be present and the annotated object is not a function.


Hosting failed to start

I use .NET Aspire , macOS is okay ,but when I move to Windows 11 using Powershell and WSL , it always show me Hosting failed to start C:\Program Files\dotnet\packs\Aspire.Hosting.Orchestration.win-x64\8.0.2\tools\dcp.exe info returned non-zero exit code -1
I need to restart my win11 and run again

info: Aspire.Hosting.DistributedApplication[0]
      Aspire version: 8.0.2+3d0cf817f24298ba37bca94f15bd2f7061856e77
info: Aspire.Hosting.DistributedApplication[0]
      Distributed application starting.
info: Aspire.Hosting.DistributedApplication[0]
      Application host directory is: C:\Users\kinfeylo\Desktop\AOAI\SLM\phi3\dotNET\Phi3DotNETAspire\Phi3.Aspire.AppHost
fail: Microsoft.Extensions.Hosting.Internal.Host[11]
      Hosting failed to start
      System.InvalidOperationException: Command C:\Program Files\dotnet\packs\Aspire.Hosting.Orchestration.win-x64\8.0.2\tools\dcp.exe info returned non-zero exit code -1
         at Aspire.Hosting.Dcp.Process.ProcessUtil.ProcessDisposable.DisposeAsync() in /_/src/Aspire.Hosting/Dcp/Process/ProcessUtil.cs:line 155
         at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 115
         at Aspire.Hosting.Dcp.DcpHostService.StartAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpHostService.cs:line 67
         at Microsoft.Extensions.Hosting.Internal.Host.<StartAsync>b__15_1(IHostedService service, CancellationToken token)
         at Microsoft.Extensions.Hosting.Internal.Host.ForeachService[T](IEnumerable`1 services, CancellationToken token, Boolean concurrent, Boolean abortOnFirstException, List`1 exceptions, Func`3 operation)
Unhandled exception. System.AggregateException: One or more errors occurred. (Command C:\Program Files\dotnet\packs\Aspire.Hosting.Orchestration.win-x64\8.0.2\tools\dcp.exe info returned non-zero exit code -1)
---> System.InvalidOperationException: Command C:\Program Files\dotnet\packs\Aspire.Hosting.Orchestration.win-x64\8.0.2\tools\dcp.exe info returned non-zero exit code -1
   at Aspire.Hosting.Dcp.Process.ProcessUtil.ProcessDisposable.DisposeAsync() in /_/src/Aspire.Hosting/Dcp/Process/ProcessUtil.cs:line 155
   at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 115
   at Aspire.Hosting.Dcp.DcpHostService.StartAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpHostService.cs:line 67
   at Microsoft.Extensions.Hosting.Internal.Host.<StartAsync>b__15_1(IHostedService service, CancellationToken token)
   at Microsoft.Extensions.Hosting.Internal.Host.ForeachService[T](IEnumerable`1 services, CancellationToken token, Boolean concurrent, Boolean abortOnFirstException, List`1 exceptions, Func`3 operation)
   at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken)
   at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token)
   at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token)
   at Aspire.Hosting.DistributedApplication.RunAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/DistributedApplication.cs:line 310
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at System.Threading.Tasks.Task.Wait()
   at Aspire.Hosting.DistributedApplication.Run() in /_/src/Aspire.Hosting/DistributedApplication.cs:line 338
   at Program.<Main>$(String[] args) in C:\Users\kinfeylo\Desktop\AOAI\SLM\phi3\dotNET\Phi3DotNETAspire\Phi3.Aspire.AppHost\Program.cs:line 33

My .NET version 8.0.302
.NET Aspire - 8.0.2

In WSL , we have same error , you need to restart your labtop

fail: Microsoft.Extensions.Hosting.Internal.Host[11]
      Hosting failed to start
      Aspire.Hosting.DistributedApplicationException: Application orchestrator dependency check had an unexpected error System.TimeoutException: The operation has timed out.
         at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 78.
         at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 105
         at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 115
         at Aspire.Hosting.Dcp.DcpHostService.StartAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpHostService.cs:line 67
         at Microsoft.Extensions.Hosting.Internal.Host.<StartAsync>b__15_1(IHostedService service, CancellationToken token)
         at Microsoft.Extensions.Hosting.Internal.Host.ForeachService[T](IEnumerable`1 services, CancellationToken token, Boolean concurrent, Boolean abortOnFirstException, List`1 exceptions, Func`3 operation)
Unhandled exception. System.AggregateException: One or more errors occurred. (Application orchestrator dependency check had an unexpected error System.TimeoutException: The operation has timed out.
   at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 78.)
 ---> Aspire.Hosting.DistributedApplicationException: Application orchestrator dependency check had an unexpected error System.TimeoutException: The operation has timed out.
   at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 78.
   at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 105
   at Aspire.Hosting.Dcp.DcpDependencyCheck.GetDcpInfoAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpDependencyCheck.cs:line 115
   at Aspire.Hosting.Dcp.DcpHostService.StartAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/Dcp/DcpHostService.cs:line 67
   at Microsoft.Extensions.Hosting.Internal.Host.<StartAsync>b__15_1(IHostedService service, CancellationToken token)
   at Microsoft.Extensions.Hosting.Internal.Host.ForeachService[T](IEnumerable`1 services, CancellationToken token, Boolean concurrent, Boolean abortOnFirstException, List`1 exceptions, Func`3 operation)
   at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken)
   at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token)
   at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token)
   at Aspire.Hosting.DistributedApplication.RunAsync(CancellationToken cancellationToken) in /_/src/Aspire.Hosting/DistributedApplication.cs:line 310
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at System.Threading.Tasks.Task.Wait()
   at Aspire.Hosting.DistributedApplication.Run() in /_/src/Aspire.Hosting/DistributedApplication.cs:line 338
   at Program.<Main>$(String[] args) in /mnt/c/Users/kinfeylo/Desktop/AOAI/SLM/phi3/dotNET/Phi3DotNETAspire/Phi3.Aspire.AppHost/Program.cs:line 33

After spending some time with @kinfey, the OnnxRuntime GenAI scenario in Aspire Kinfey is building worked on my machine.

Some details:
Aspire Version
aspire                     8.0.1/8.0.100
Model
Phi-3-mini-4k-instruct-onnx (cpu_and_mobile/cpu-int4-rtn-block-32-acc-level-4)
Hardware
CPU

Memory
64GB
Next steps
Since Kinfey's PC is a newer AI PC, we'd want to investigate / validate what the potential issues are that prevent the sample from working on his PC:

Memory availability
There are issue running Aspire on AI PC hardware

cc: @natke @Vivihung

@kinfey @luisquintanilla so are you good/unblocked for now?
We made some fixes that will ship in the next Aspire release. These fixes should give you a better, more informative error message than just dcp info exited with code -1.

@karolz-ms I'm okay with closing. If further investigations reveal the issues are with Aspire, we can open a new issue.
@kinfey are you okay with that?

Closing for now--we should be good in 8.1. Re-open or open a new issue please if problems persist in 8.1 release.


[BUG] Memory Leak

Describe the bug
When starting a job with a wordlist containing high number of data for a long period of time causes the pc to stuck.
At first - starting a job with high number of data will not have any issues, only later when it has progressed more it gets slower and slower until finally unresponsive at all.
Can't open task manager to show more details, because the entire PC is not responding, so I can't tell if it's the CPU or Memory or even Disk fault.
It doesn't matter if the browser is closed or open on the job details, result is the same.
(My guess - Probably because the program uses more and more memory without cleaning it every once in a while?)
Don't know if it's related - The only thing shown is: "fail: Microsoft.EntityFrameworkCore.Query[10100].
To Reproduce
Steps to reproduce the behavior:

Create a wordlist containing high number of data (for example: 4,000,000).
Start a job with the selected wordlist for a long period of time (for example 6 hours).
See error - PC become slower and slower untill unresponsive. If browser is open - Job details page of the UI is stuck on the reloading animation (white background with blue circle reloading).

Expected behavior
PC should be responsive all the time.
Desktop (please complete the following information):

OS: Windows 10 - 64bit
Browser Chrome
Version 0.1.7

Please answer the following questions:

Are you getting a huge amount of hits?
Do you have bot logs enabled? If you do, can you turn them off?
How much RAM do you have available on your PC?
Which hits output are you using? If you're using database can you please switch to FileSystem and try again?

Please answer the following questions:

Are you getting a huge amount of hits?
Do you have bot logs enabled? If you do, can you turn them off?
How much RAM do you have available on your PC?
Which hits output are you using? If you're using database can you please switch to FileSystem and try again?

No. Normal amount of hits.
No. Bot logs was and still is disabled.
4GB.
HTTP Request on the config. Not using the database.

@ObsessedHacker regarding number 4 I am referring to this section in the job options

@ObsessedHacker regarding number 4 I am referring to this section in the job options

I am aware.
I clicked on "X" when creating the job.

@ObsessedHacker regarding number 4 I am referring to this section in the job options

I am aware.
I clicked on "X" when creating the job.

Ok perfect it was just because you mentioned Http Request so I wasn't sure we were on the same page ^_^ Alright I will look into this and see if I can replicate it.

@ObsessedHacker regarding number 4 I am referring to this section in the job options

I am aware.
I clicked on "X" when creating the job.

Ok perfect it was just because you mentioned Http Request so I wasn't sure we were on the same page ^_^ Alright I will look into this and see if I can replicate it.

Don't know if it's related - Now when it crashed again, the only shown text is "warn: Microsoft.AspNetCore.Server.Kestrel[22]".

Also - I noticed now when the progress is 99.86% (4301545/4308524) it doesn't finish it, it doesn't reach 100%.

Also - I noticed now when the progress is 99.86% (4301545/4308524) it doesn't finish it, it doesn't reach 100%.

That is a separate issue, I am aware of that but never found the cause as it's hard to debug (you need to run it for a very long time)

Edit - And now after I closed the program - on task manager I see 100% Disk usage from "Service Host: SysMain"

Are you using blocks like File Write or File Append in your config? They are known to have problems and hang the thread.

Edit - And now after I closed the program - on task manager I see 100% Disk usage from "Service Host: SysMain"

Are you using blocks like File Write or File Append in your config? They are known to have problems and hang the thread.

No. Not using those blocks.

Edit - And now after I closed the program - on task manager I see 100% Disk usage from "Service Host: SysMain"

Are you using blocks like File Write or File Append in your config? They are known to have problems and hang the thread.

No. Not using those blocks.

Alright then this clearly looks like a memory leak somewhere. I will let you know if I find out more.

IDK why the ram keeps increasing even without any usasge. When the ram get's to its limit it crashes.
Please fix this issue ASAP.
HOPING FOR POSTIVIE RESPONSE

Please follow this guide and send me the .gcdump file, otherwise it's really hard to debug this issue to see where the memleak is coming from.
https://devblogs.microsoft.com/dotnet/collecting-and-analyzing-memory-dumps/

collecting-and-analyzing-memory-dumps/
iam trying to but idk
why it says
root@98bbf7f42051:/home/coder/project# dotnet tool install --global dotnet-gcdump
Tool 'dotnet-gcdump' is already installed.
root@98bbf7f42051:/home/coder/project# pidof dotnet
13758
`root@98bbf7f42051:/home/coder/project# dotnet gcdump collect -p 13758
Could not execute because the specified command or file was not found.
Possible reasons for this include:

You misspelled a built-in dotnet command.
You intended to execute a .NET program, but dotnet-gcdump does not exist.
You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.
root@98bbf7f42051:/home/coder/project# `

@sudo216 if you have the chance to do this on windows, please try, as we also tried on linux but it was giving errors (we got past this part though, it had to do with the fact that dotnet-gcdump is not added to the path).

the tool is in your homedir in .dotnet/tools
so best is to add
this to your ~/.bash_profile if your system is a non-sudo system.
export PATH="$PATH:/root/.dotnet/tools"
if you have used sudo, then it is
export PATH="$PATH:/home/yourusername/.dotnet/tools"
(maybe check before, which one is the right one)
and then log out and log in again.
if you get some error about some .so file missing you also need to set the DOTNET_ROOT environment variable. so also add
export DOTNET_ROOT=$(dirname $(realpath $(which dotnet)))

the tool is in your homedir in .dotnet/tools
so best is to add
this to your ~/.bash_profile if your system is a non-sudo system.
export PATH="$PATH:/root/.dotnet/tools"
if you have used sudo, then it is
export PATH="$PATH:/home/yourusername/.dotnet/tools"
(ofc replace yourusername ;) - maybe check before, which one is the right one)
to add it to your Path and then log out and log in again.
if you get some error about some .so file missing you also need to set the DOTNET_ROOT environment variable. so also add
export DOTNET_ROOT=$(dirname $(realpath $(which dotnet)))

i tried but dk why
the error comes
coder@f110c9f609d8:~/project/dump$ pidof dotnet
7915
coder@f110c9f609d8:~/project/dump$ dotnet gcdump collect -p 7915
Writing gcdump to '/home/coder/project/dump/20210417_102226_7915.gcdump'...
Failed to collect gcdump. Try running with '-v' for more information.
coder@f110c9f609d8:~/project/dump$ dotnet gcdump collect -v -p 7915
Writing gcdump to '/home/coder/project/dump/20210417_102239_7915.gcdump'...
0.0s: Creating type table flushing task
0.0s: [Error] Exception during gcdump: System.Net.Internals.SocketExceptionFactory+ExtendedSocketException (13): Permission denied /tmp/dotnet-diagnostic-7915-304489597-socket
at System.Net.Sockets.Socket.DoConnect(EndPoint endPointSnapshot, SocketAddress socketAddress)
at System.Net.Sockets.Socket.Connect(EndPoint remoteEP)
at System.Net.Sockets.Socket.UnsafeBeginConnect(EndPoint remoteEP, AsyncCallback callback, Object state, Boolean flowContext)
at System.Net.Sockets.Socket.BeginConnect(EndPoint remoteEP, AsyncCallback callback, Object state)
at Microsoft.Diagnostics.NETCore.Client.UnixDomainSocket.Connect(String path, TimeSpan timeout) in //src/Microsoft.Diagnostics.NETCore.Client/DiagnosticsIpc/UnixDomainSocket.cs:line 57
at Microsoft.Diagnostics.NETCore.Client.PidIpcEndpoint.Connect(TimeSpan timeout) in //src/Microsoft.Diagnostics.NETCore.Client/DiagnosticsIpc/IpcTransport.cs:line 138
at Microsoft.Diagnostics.NETCore.Client.IpcClient.SendMessage(IpcEndpoint endpoint, IpcMessage message, IpcMessage& response) in //src/Microsoft.Diagnostics.NETCore.Client/DiagnosticsIpc/IpcClient.cs:line 42
at Microsoft.Diagnostics.NETCore.Client.EventPipeSession..ctor(IpcEndpoint endpoint, IEnumerable1 providers, Boolean requestRundown, Int32 circularBufferMB) in /_/src/Microsoft.Diagnostics.NETCore.Client/DiagnosticsClient/EventPipeSession.cs:line 30    at Microsoft.Diagnostics.NETCore.Client.DiagnosticsClient.StartEventPipeSession(IEnumerable1 providers, Boolean requestRundown, Int32 circularBufferMB) in //src/Microsoft.Diagnostics.NETCore.Client/DiagnosticsClient/DiagnosticsClient.cs:line 66
at Microsoft.Diagnostics.Tools.GCDump.EventPipeSessionController..ctor(Int32 pid, List`1 providers, Boolean requestRundown) in //src/Tools/dotnet-gcdump/DotNetHeapDump/EventPipeDotNetHeapDumper.cs:line 235
at Microsoft.Diagnostics.Tools.GCDump.EventPipeDotNetHeapDumper.DumpFromEventPipe(CancellationToken ct, Int32 processID, MemoryGraph memoryGraph, TextWriter log, Int32 timeout, DotNetHeapInfo dotNetInfo) in //src/Tools/dotnet-gcdump/DotNetHeapDump/EventPipeDotNetHeapDumper.cs:line 52
[  0.1s: Done Dumping .NET heap success=False]
Failed to collect gcdump. Try running with '-v' for more information.
coder@f110c9f609d8:~/project/dump$ ls
coder@f110c9f609d8:~/project/dump$

i dumped but idk if i did correct or not. Although
here(https://drive.google.com/file/d/1OhgfZxCx8Q_ZkxahFtkgiMMyR9tg7YTC/view?usp=sharing)

Please can anyone confirm if the memory leaks are still present in 0.1.12? I fixed the big one, so maybe now it's acceptable? Please let me know.

Please can anyone confirm if the memory leaks are still present in 0.1.12? I fixed the big one, so maybe now it's acceptable? Please let me know.

It's hard to replicate the issue since the job has to run for a long period of time, so I guess for now it's fixed?


[Feature] Git Commad line prompt

Add Git command line prompt with autocomplete like this mock

Very interesting feature. But there are serveral questions that stop me to introduce this feature into this app.

Which git commands to be supported in the prompt? Should we introduce the document for commands?
How do we display the output of commands since it will be very different even the same command, for example: git log vs git log --pretty=format:%H
Since this app provides a toolbar button that supports external terminal, which will be more powerfull with auto-complete feature enabled, for example: https://www.git-scm.com/book/en/v2/Appendix-A%3A-Git-in-Other-Environments-Git-in-Zsh, I do NOT think this app will do better than that.

* Which git commands to be supported in the prompt? Should we introduce the document for commands?

Initially, you may want to support all the commands in the SourceGit.Commands namespace, they almost all have a parser for the output. We may want to try generating the intellisese using Source Generator.

* How do we display the output of commands since it will be very different even for the same command, for example: `git log` vs `git log --pretty=format:%H`

as  @aikawayataro tell, we can o handle output and output history.

* Since this app provides a toolbar button that supports external terminal, which will be more powerfull with auto-complete feature enabled, for example: https://www.git-scm.com/book/en/v2/Appendix-A%3A-Git-in-Other-Environments-Git-in-Zsh, I do NOT think this app will do better than that.

I know these features and that's where I got my inspiration from. The Git prompt should not completely replicate the terminal but should allow you to use only git commands.

I think it's better to just integrate an in-app terminal. Not sure if there's a free space where we can stick it, making GUI more complex is a bad thing by me. Implementing terminal emulator for dotnet sounds very interesting though.

I agree

The original intention of this project is to provide a Free & Open-source & Cross-platform Git GUI Client. There are currently no plans to add this feature.


[IS Start-up] Errors logged when starting IS pack, MyAccount and Console could not be accessed

Describe the issue:
Several errors were logged in the terminal when starting the IS alpha pack (see details in below in comments).
Even though the pack started and the Management Console can be accessed, MyAccount and Console cannot be accessed - getting 404 when trying to access the URLs.
How to reproduce:
Test platform: Windows 10, JDK8, Firefox 97
Download and start the IS 5.12 alpha pack (checked with alpha 12 and 13)
Expected behavior:

No errors are expected at startup
MyAccount and Console should be available to access

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.12 alpha 12 and 13
OS: Windows 10
Database: H2 and MySQL
Userstore: JDBC

Optional Fields
Related issues:

Suggested labels:

Screenshots

@deshalw shall we remove the doc link and add the relevant details in the git issue shortly? Because the doc is not publicly available

Error references:
ERROR {org.wso2.carbon.webapp.mgt.loader.SharedClassLoaderFactory} - Error while creating class loader for CXF3 java.io.IOException: The filename, directory name, or volume label syntax is incorrect
ERROR {org.apache.catalina.core.ContainerBase.[Catalina].[localhost].[/accountrecoveryendpoint]} - Exception sending context initialized event to listener instance of class [org.wso2.carbon.identity.mgt.endpoint.util.listener.IdentityManagementEndpointContextListener] java.lang.NoClassDefFoundError: org/apache/cxf/jaxrs/provider/json/JSONProvider
ERROR {org.wso2.carbon.tomcat.internal.CarbonTomcat} - Cannot stop context org.apache.catalina.LifecycleException: An invalid Lifecycle transition was attempted ([before_stop]) for component [StandardEngine[Catalina].StandardHost[localhost].StandardContext[/accountrecoveryendpoint]] in state [DESTROYED]

Workaround: https://github.com/wso2/product-is/issues/12767#issuecomment-982338215

Duplicates https://github.com/wso2/product-is/issues/12767


[MSSQL] Refreshing triggers causes hang

I am working with triggers on Microsoft SQL Server 2008, and after creating a trigger in the SQL Editor, if I go to the DB navigator -> Table -> Triggers view (which I already had open) and press F5 to refresh, DBeaver hangs and then eventually displays an error like the attached image.
Note: I am using DBeaver CE 4.0.5, and I have the same result on multiple different computers connecting to multiple databases.

Can't reproduce. Do you see this problem with all tables?
Note: when you refresh triggers view all table properties are refreshed. Maybe some queries are conflicting/lead to dead-lock or something.
Could you check Query Manager view (with metadata queries enabled) to see what is DBeaver actually tries to do?


Visual Studio 2017 RC C# Notepad Sample Does Not Work for FindElementByAccessibilityId

I tried out the latest build of WinAppDriver and the Notepad Sample and couldn't get the test to run successfully due the file path edit box not working.
https://github.com/Microsoft/WinAppDriver/blob/master/Samples/C%23/NotepadTest/Scenario.cs#L63
https://github.com/Microsoft/WinAppDriver/tree/master/Samples/C%23/NotepadTest
Here is the video of my repo:
https://www.youtube.com/watch?v=itbGurSPLiA
WinVer: Windows 10 Build 1511 (OS Build 10586.633)
WinAppDriver - 0.6.1610.28001
Microsoft Visual Studio Enterprise 2017 RC
Version 15.0.25914.0 D15REL

Hi Ian,
VS 2017 doesn't appear to be the issue, and WAD 0.6 seems ok as well.
The issue I'm seeing looks related to behavior on that version of Windows.  For me the failure is on line 109.  To get it passing I removed the file extension from the TestFileName global so it's just "NotepadTestOutputFile".  Now the test passes but the file explorer window isn't getting closed.
What I saw in your video was different with the text getting entered into the filename box, and I haven't been able to reproduce this.  Is the file name being typed into Notepad a consistent repro?  Is it possible something caused focus to move back to the notepad window during the test?
Thanks.

please open new issue if there's still an issue with the notepad test.  Thank you.

Hi @ianceicys,
We have just updated our C# samples in the repository. The NotepadTest you mentioned above should no longer have the issue. They are tested on Windows 10 PC with the latest Windows 10 version (Version 1607 or later). Please give it a try!


Notes transporter with nodemailer

Hi Andris,
I'm a new in nodejs and looking for a module which can send emails from Node.js Application.
However the problem is that i want to use NOTES routing instead of using SMTP or POP3 or any other protocol. so I wonder if there is a way to send emails using nodemailer and NOTES routing. and if not can you guide me creating a NOTES transporter for nodemailer.
Thank you

You mean like Lotus Notes NRPC protocol? I think it is proprietary, I have no idea how it works and I can't test it as I do not have access to any Lotus software.
If you are willing to implement such tranport yourself then look into nodemailer-pickup for an example, this tranport stores message to a file instead of sending it to anywhere, so it gives a quick overview how transports work for Nodemailer https://github.com/andris9/nodemailer-pickup-transport


KAFKA-3486: fix autocommit when partitions assigned manually

The fix here keeps the autocommit task running regardless of the subscription/assignment state. Before we were trying to start it and stop it depending on whether the group had been joined, but that clearly doesn't work when partitions are assigned manually.

@hachikuji One edge case issue, but otherwise LGTM

LGTM

Nice job simplifying the code @hachikuji


PxrLMLayer specular and clearcoat bump inputs are missing

There is a field for the Specular Bump in the PxrLMLayer node, but it can't be connected to anything. Same problem with the ClearCoat Bump.

The python exporter still consider those fields, overriding the diffuse bump, making impossible to add a bump other than on diffuse.

For some reason the node editor doesn't display all the parameters correctly compared to the material tab.  The material tab gives you access to the clear coat bump pattern input but the input for specular bump is missing there too.

Hmmm...  it seems the sockets inside the advanced panels are not getting made.

Fixed.


Trying to use for throws an error.

A for of the form:
(for [p start-low]
  ...)

Throws the following error:
WARN unsopported form => [p start_low]

I am using doseq instead.

Hi,
The reason for this error is for is not yet implemented.
Best,


Adding Prettier and Husky for pre-commit hooks.

Fixes #13

Thanks! Been wanting to try out Husky on a project for a while 👍


Call sink.eof before returning the sink in MDAO's select and remove a…

…n eof call from TreeNode that should not longer be needed.

Note travis breakage is unrelated to this. Potential fix is here https://github.com/foam-framework/foam2/pull/2358/files but I want Adam to help.


reading transaction closing balance (record :62F:)

Is it possible to access the transaction closing balance (record :62F:)? I don't see this in transaction.data? I've read through the docs but I'm still not sure how to get this piece of information. I did find mt940.models.Balance for parsing this information but I'm not sure how to use this to get the information in record :62F:. Thanks for your help.

I'd also like to read record :60F: from a transaction. Thanks.

60F should be supported: https://github.com/WoLpH/mt940/blob/develop/mt940/tags.py#L145-L146
And 62F as well: https://github.com/WoLpH/mt940/blob/develop/mt940/tags.py#L197-L198
If it's somehow not working for you than you might have a different unrecognized format. We even have working tests for both: https://github.com/WoLpH/mt940/blob/5d277eb8db56ff958a0645a90f0e26f12ef44766/tests/jejik/generic.sta
Resulting in: https://github.com/WoLpH/mt940/blob/5d277eb8db56ff958a0645a90f0e26f12ef44766/tests/jejik/generic.yml#L4-L19

I think the format of the records in the MT940 file I'm using are OK. I can see the final_opening_balance and final_closing_balance for the last record / transaction in the parsed MT940 file but I want to get this information for each record / transaction.
For instance, if I parse the MT940 file like this:
transactions = mt940.parse('path/to/abnamro.sta')
transaction = transactions[0]

What Python code would get me the values of the :60F: and :62F: fields for this specific transaction?

Well, it's scoped to Transactions by default but that's easily fixable with a little monkeypatch. I believe this should do the trick:
from mt940 import tags
from mt940 import models

tags.BalanceBase.scope = models.Transaction

Thanks, I'll try it out tomorrow. Is there documentation the I missed about this?

I honestly wasn't expecting this to be needed so no, it's not documented.
It's more of a view the source thing in this case.
Perhaps I should change the default scope for the balance or add it to both
scopes by default. I'll have to think about it for a bit.

That didn't work. Here's what I get:
>>> import mt940
>>> from mt940 import tags
>>> from mt940 import models
>>> tags.BalanceBase.scope = models.Transaction
>>> transactions = mt940.parse('/home/ben/abnamro.sta')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ben/Development/project/env/lib/python2.7/site-packages/mt940/parser.py", line 45, in parse
    transactions.parse(data)
  File "/home/ben/Development/project/env/lib/python2.7/site-packages/mt940/models.py", line 232, in parse
    transaction.data[k] += '\n%s' % v.strip()
AttributeError: 'Balance' object has no attribute 'strip'

Any ideas? Thanks again for your help on this.

I've just pushed a new release to the dev which fixes this issue. The proper fix is going to take a bit more time but perhaps this already does the trick for you :)

Thanks, this should be fine for my needs.
From a high level, it makes sense to me that the balance information is available by default on each Transaction but it's also reasonable to just add documentation on how to do this with a monkey patch. Let me know if you prefer to monkey patch solution and I'll make a PR to add this to the docs.
I'm working on some other stuff now but I'll get back to using this soon. Thanks again for your help.

Ultimately it would be nice to do both, but since that will take a bit more work I think docs would be preferable for the time being.
Thanks for the help!

I'm just getting back to this now. It seems like record :60F: comes before record :61:. With the monkey patch applied, the opening and closing balance info isn't correct for the transaction because the transaction is create when a :61: is found.
I've made a PR just for discussion but it's not correct yet. I'll make an update in a few minutes with an explanation.

Fixed via https://github.com/WoLpH/mt940/commit/472f987426ff95d811a28d00eed322f34a61a094.


MONDO:0004620 and MONDO:0020330 [Merge?]

Mondo term (ID and Label)
MONDO:0020330 classic Hodgkin lymphoma, lymphocyte-depleted type
Reason for deprecation
Seems to be redundant with second concept; based on Orpha:98846 mapping to UMLS and where UMLS maps the SCTID 118610003 .  But the NCIT C9125 is specific for the adult-onset form... so maybe this isn't a merge but rather a clean up of the names/concepts (Make Hodgkin's lymphoma, lymphocytic depletion clearly the "adult" onset form, set it as a child of the above term?)
Term to be merged with
MONDO:0004620  Hodgkin's lymphoma, lymphocytic depletion
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Hello @kanems
I'm Sabrina, a new curator who recently joined the Mondo team.
We will address your issue as soon as possible. Thank you for your patience.
If this is a high priority, please let me know and we can prioritize it. Thank you!


Fix: Clean Excluder interface specification

Codecov Report

Merging #176 into master will increase coverage by 0.16%.
The diff coverage is 13.33%.

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
+ Coverage   52.18%   52.35%   +0.16%
==========================================
  Files          33       33
  Lines        4179     4179
==========================================
+ Hits         2181     2188       +7
+ Misses       1845     1835      -10
- Partials      153      156       +3

Impacted Files
Coverage Δ

supervisor/supervisormock.go
0% <0%> (ø)
:white_check_mark:

cmd/remoteenforcer/remoteenforcer_linux.go
0% <0%> (ø)
:white_check_mark:

trireme.go
53.17% <100%> (ø)
:white_check_mark:

supervisor/supervisor.go
79.12% <16.66%> (ø)
:white_check_mark:

monitor/linuxmonitor/linuxMonitor.go
68% <0%> (+7%)
:white_check_mark:

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 f1a3b94...bc9b374. Read the comment docs.


OTEL_TRACES_EXPORTER default value incompatible with @opentelemetry/sdk-trace-base

Discussed in https://github.com/open-telemetry/opentelemetry-js/discussions/3415

Originally posted by fischerman November 16, 2022
What's the meaning of this log message?
Exporter "otlp" requested through environment variable is unavailable.

Why am I seeing it? I'm providing the exporter in code (not via env variables). OTEL_TRACES_EXPORTER is undefined.
const exporter = new OTLPTraceExporter()
const sdk = new NodeSDK({
        resource: new Resource({
            [SemanticResourceAttributes.SERVICE_NAME]: '...',
        }),
        traceExporter: exporter,
        sampler: new AlwaysOnSampler(),
        instrumentations: [getNodeAutoInstrumentations({
            '@opentelemetry/instrumentation-graphql': {
                depth: 1000,
            }
        })]
    });

$ npm ls | grep @opentelemetry
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>

The problem
The default value for OTEL_TRACES_EXPORTER is "otlp", but BasicTracerProvider doesn't register any exporters so this would lead to a diag.error log by default.

Hey, I am running into this bug and I think it's causing my traces to not be exported.
This line https://github.com/open-telemetry/opentelemetry-js/blob/ef6fcdcd1a7e5234758e3bcc9ff6c29db5ff4d23/packages/opentelemetry-sdk-trace-base/src/BasicTracerProvider.ts#L89 is being run because the _buildExporterFromEnv method is returning undefined.
I am also providing the exporter in code
const api = require("@opentelemetry/api");
const {
  ConsoleSpanExporter,
  BatchSpanProcessor,
  SimpleSpanProcessor,
} = require("@opentelemetry/sdk-trace-base");
const {
  OTLPTraceExporter,
} = require("@opentelemetry/exporter-trace-otlp-http");
const { NodeTracerProvider } = require("@opentelemetry/sdk-trace-node");
const { registerInstrumentations } = require("@opentelemetry/instrumentation");
const {
  AwsLambdaInstrumentation,
} = require("@opentelemetry/instrumentation-aws-lambda");

api.diag.setLogger(new api.DiagConsoleLogger(), api.DiagLogLevel.ALL);
const provider = new NodeTracerProvider();

const spanProcessor = new SimpleSpanProcessor(
  new OTLPTraceExporter({
    url: "https://api.honeycomb.io",
    headers: {
      "x-honeycomb-team": process.env.LIBHONEY_API_KEY,
      "x-honeycomb-dataset": process.env.LIBHONEY_DATASET,
    },
  })
);

provider.addSpanProcessor(spanProcessor);
provider.addSpanProcessor(new BatchSpanProcessor(new ConsoleSpanExporter()));
provider.register();

registerInstrumentations({
  instrumentations: [
    new AwsLambdaInstrumentation({
      requestHook: (span, { event, context }) => {
        span.setAttribute("faas.name", context.functionName);
        span.setAttribute("faas.event", event);
      },
      responseHook: (span, { err, res }) => {
        if (err instanceof Error) span.setAttribute("faas.error", err.message);
        if (res) span.setAttribute("faas.res", res);
      },
    }),
  ],
});

Is there a work around?

I had code just like the author's but I had also exported OTEL_TRACES_EXPORTER: "otlp". Once I removed that export, it worked fine (due to #3492 fix)

Try that, it should work, then I think we can close this issue.

@legendecas
last release includes #5138. In this new version the BasicTraceProvider does not try to get the exporter from env if a processor is passed in the constructor.
With the following setup the log does not appear.
package.json
{
  "name": "otel-js-3422",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@opentelemetry/api": "^1.9.0",
    "@opentelemetry/auto-instrumentations-node": "^0.53.0",
    "@opentelemetry/exporter-trace-otlp-http": "^0.55.0",
    "@opentelemetry/sdk-node": "^0.55.0",
    "@opentelemetry/semantic-conventions": "^1.28.0"
  }
}

index.js
const { Resource } = require('@opentelemetry/resources');
const { NodeSDK } = require('@opentelemetry/sdk-node');
const { AlwaysOnSampler } = require('@opentelemetry/sdk-trace-base');
const { getNodeAutoInstrumentations } = require('@opentelemetry/auto-instrumentations-node');
const { ATTR_SERVICE_NAME } = require('@opentelemetry/semantic-conventions');
const { OTLPTraceExporter } = require('@opentelemetry/exporter-trace-otlp-http');

const exporter = new OTLPTraceExporter()
const sdk = new NodeSDK({
    resource: new Resource({
        [ATTR_SERVICE_NAME]: '...',
    }),
    traceExporter: exporter,
    sampler: new AlwaysOnSampler(),
    instrumentations: [getNodeAutoInstrumentations({
        '@opentelemetry/instrumentation-graphql': {
            depth: 1000,
        }
    })]
});

I think we can close it :)


Updating mbed-os to mbed-os-5.3.5

Please test/merge this PR and then tag Master with mbed-os-5.3.5

LGTM


Adding NodeTypes allows to select Node Type Implementations instead (intended?) of Types

When I try to add a new Node Type, the dialog presents me with a selection of the type. T
Tbh I have no idea what that type indicates in the dialog, should this be a type to inherit from?
Here a screen:

If I fill out the form of the dialog and press the button I get 500's:

Checked with latest version, seems to be gone :)


Example not working.

I have absolutely no idea why exactly, but the example as is doesn't work for me. I have to change the notification id by any other name for it to work.

Hi,
which sample specifically doesn't work for you?

This one: https://ampbyexample.com/playground/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-user-notification%2Fsource%2F

My bad I just understood what I did wrong. It was simply saved in the local storage several days ago and I forgot about it.
Not enough sleep :(


Problem when execute command

Hi everyone.
It is my first time using pwnedornot.
I installed in termux - noon rooted Android and when i use
python3 pwnedornot.py
I get the following message
Traceback (most recent call last):                     File "/data/data/com.termux/files/home/pwnedOrNot/pwnedornot.py", line 55, in <module>
    from html2text import html2text                  ModuleNotFoundError: No module named 'html2text'

How can I fix it?
Thank you

@LFreitas88 try : pip3 install html2text

@thewhiteh4t Confirming this works. However you should include this as part of the Dockerfile as well.