Stress test: Misalignment when submitting transactions in quick succession

What happened?
While stress testing a new bridge dapp, I discovered what appears to be a Pali Wallet (or API) bug or limitation.  The behavior described below is not isolated to the new bridge dapp, it can be reproduced on the current dapp at bridge.syscoin.org, or likely on any dapp integrated with Pali.
I ran multiple distinct sessions (three) of a dapp all using the same wallet instance (e.g. have several tabs open of bridge.syscoin.org in the same browser). I used the dapp to submit the same type of transaction in Pali back-to-back on each session (e.g. submit multiple Mint SYSX transactions, each in quick succession, moving to each tab).  RESULT: the first dapp session appears to work fine, but the other sessions do not recognize that the transaction has been submitted at all, even though they have and they are onchain with confirmations. Of course when I attempt to re-mint (redo the transactions) through Pali that fails because a mint based on that burn proof has already been done onchain, so Pali appropriately refuses it. End result is the second and third dapp sessions are both at an impasse and cannot proceed to next steps.
What is interesting is this does not happen when submitting Metamask transactions similarly, only when using Pali Wallet.
There is more specific information available in a GitHub issue I originally posted on the repository of this new bridge dapp.  Reach out to me in Discord and I will share the link (that dapp is not public yet).
Version
1.0.28
What platform are you using?
Linux
What browser are you using?
Firefox
Is this a trezor issue?
No
Relevant log output
No response

From user osiastedian:
_"I was able to reproduce the issue as described.
I think this is a PaliWallet issue on how they handle simultaneous signing request on how it translates to dapps.
MetaMask once signed it immediate returns a transaction hash for it. Unlike Pali it still waits a little while then Transaction ID is given.
I've tried executing similar behavior on another Dapp, the old Bridge.
Video demo here:
https://drive.google.com/file/d/1ZGSbkJTKzg8F47h2LlIW5YB_bU9wE_0B/view?usp=sharing
And you'll notice PaliWallet behave the same way."_

hi @bstr156, we are looking into some bridge connection issues rn, as soon as we have a new version you can test before we submit it to the store, so i'll let you know, thank you 😄


mlterm: 3.8.0 -> 3.8.4

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlterm help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlfc help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlcc -h got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlcc --help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclient -h got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclient --help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclient help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclientx -h got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclientx --help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlclientx help got 0 exit code
ran /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin/mlterm-fb help got 0 exit code
found 3.8.4 with grep in /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4
found 3.8.4 in filename of file in /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4

cc "@vrthra @ramkromberg"

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘mlterm-3.8.4’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/applications/misc/mlterm/default.nix:9 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/lib/mlterm/libind_kannada.so
shrinking /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/lib/mlterm/libind_punjabi.so
shrinking /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/lib/libmlterm_coreotl.so
shrinking /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/lib/libmef.so.1.0.0
gzipping man pages under /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/lib  /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/libexec  /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4/bin
patching script interpreter paths in /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4
checking for references to /tmp/nix-build-mlterm-3.8.4.drv-0 in /nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4...
/nix/store/hl028yiknrqbkp4i12nn0nibr41rwgvn-mlterm-3.8.4

Success on aarch64-linux (full log)
Partial log (click to expand)
these paths will be fetched (2.72 MiB download, 17.50 MiB unpacked):
  /nix/store/kl1h5ljshkcniid52r1z00l55dbbvvz8-m17n-db-1.7.0
  /nix/store/lmc8fn84a29d2aaybn6gsna839bavhbr-vte-0.50.2
  /nix/store/nwgsdavg21qklm7qmbyavwqczmhw258r-mlterm-3.8.4
  /nix/store/wk2q3fhmjvwhr4pfpvrm71p14daxg5h9-m17n-lib-1.7.0
copying path '/nix/store/lmc8fn84a29d2aaybn6gsna839bavhbr-vte-0.50.2' from 'https://cache.nixos.org'...
copying path '/nix/store/kl1h5ljshkcniid52r1z00l55dbbvvz8-m17n-db-1.7.0' from 'https://cache.nixos.org'...
copying path '/nix/store/wk2q3fhmjvwhr4pfpvrm71p14daxg5h9-m17n-lib-1.7.0' from 'https://cache.nixos.org'...
copying path '/nix/store/nwgsdavg21qklm7qmbyavwqczmhw258r-mlterm-3.8.4' from 'https://cache.nixos.org'...
/nix/store/nwgsdavg21qklm7qmbyavwqczmhw258r-mlterm-3.8.4


Updates for Calico v3.8.3 and v3.9.2

[ ] Tests
[ ] Documentation
[ ] Release note

None required

Deploy preview for calico ready!
Built with commit f1e7e1301be81cfffd53503eb9ad15eebea9c08e
https://deploy-preview-2939--calico.netlify.com


Better handle spawn failures

From user log:
[2024-02-18 10:17:47.744] [error] Unexpected error occurred.
[2024-02-18 10:17:47.745] [error] {
  createIssue: '[function] function () { [native code] }',
  error: 'Error: spawn UNKNOWN\n' +
    '    at ChildProcess.spawn (node:internal/child_process:414:11)\n' +
    '    at spawn (node:child_process:784:9)\n' +
    '    at Launcher.launchSpring (C:\\Users\\user\\AppData\\Local\\Programs\\Beyond-All-Reason\\resources\\app.asar\\src\\engine_launcher.js:170:18)\n' +
    '    at Launcher.launch (C:\\Users\\user\\AppData\\Local\\Programs\\Beyond-All-Reason\\resources\\app.asar\\src\\engine_launcher.js:137:9)\n' +

We now have https://github.com/beyond-all-reason/spring-launcher/commit/729c4c1240da42d5f02a031eea12d9cb7b5f276a so meh


fix helm template -x pathing issues on Windows

tiller's rendering engine converts os filepath separators into unix-style filepath separators, so we need to split manifest names with a forward slash.
closes #4304

Pinging on this.

Pinging on this again - @bacongobbler @technosophos what more is needed to get this merged and released?  We had another bug report against the extension for it over the weekend.

Slating for 2.10; I'll ping someone to review tomorrow


Add SGLBO optimizer

add file sglbo.py => SGLBO optimizer
extended __init__.py with the SGLBO class
extended classes.rst with optimizers.SGLBO to build the docs for the optimizer
add tests for the SGLBO class

Overall looks good, we need an implementation for _update_lr though. I'd say just a simple pass will be okay in this case

Also a test case that utilizes the step function would be nice


update to pooling protocol

Pooling protocol details:
https://github.com/Chia-Network/pool-reference

closing duplicate of: https://github.com/madMAx43v3r/chia-plotter/pull/529


Add an update-able blocklist file for Via HTML

We probably need to replicate an approximation of the block list feature for Via HTML as we have for Via 1.
It's probably not going to be exactly the same, as video sites should no longer be a problem.
Some questions:

What is this list for?  Is it our place to choose what content people access through Via
Can we ever have any reasonable hope of keeping up with whatever we choose?

Notes
From a chat with Sean it sounds like the inclusion of a large list of porn sites etc might be as a part of loading a generic "bad sites" list which included lots of nefarious sites along with actively hostile ones. This likely served multiple purposes as porn sites are likely to have video content which caused us problems in the past.
Blocking based on content type seems close to an editorial decision we don't really want to make, or at least is likely a separate feature.
The aims of this tech are more likely:

Not to expose users to known hostile sites hosting exploits etc.
Giving us a mechanism to react to denial of service (intentional or otherwise) by proxying content we find burdensome

What is this list for?

Historically, there have been two major concerns:

Users using the proxy for something other than its intended purpose:

Degrading the service for legitimate users or
Costing us significant amounts of money or time that we'd like to be spending on other things

Publishers who have asked us not to allow their site to be proxied, and we have agreed (I think this only happened once or twice. The existing blocklist for Via in S3 has details)

Of the above, (1) was a far bigger issue. In particular users using Via to apparently circumvent firewalls to stream videos, play HTML games or, yes, stream porn. The existing blocklist in S3 was compiled from a combination of sites we ended up blocking manually via Cloudflare when they caused operational issues + taking a sample of web logs from Via and blocking the most frequently trafficked sites which were a) "not legitimate uses of an annotation proxy" and b)

e.g. Is it our place to choose what content people access through Via

Again, the current blocklist exists primarily for operational reasons, we're not worried about users annotating the naughty parts of Sons and Lovers.

Can we ever have any reasonable hope of keeping up with whatever we choose?

Since the blocker exists primarily to ensure uptime for legitimate users, rather than to prevent users accessing certain content for say, moral/censorship reasons, it is OK if we don't block all sites deemed "not legitimate use", as long as we can manage the traffic we have from an operational/cost perspective.

Just had a chat with @robertknight about the block list to cover off the motivation and maintenance. A quick summary of that chat:
Why we block
There are lots of legitimate reasons you might want a block list, but we only have one for a few:

We want a block list for:

Respecting publishers (or presumably another type of content producers) wishes about not allowing annotation
Preventing operational reasons caused by sites which cause us to proxy a lot of information

Good reasons to want a block-list that this isn't for:

Any kind of editorial stance
Preventing security problems for our users (serving malware, inadvertantly avoiding security checks etc)
Respecting institutional acceptable use (e.g. University of Blah doesn't allow site X on it's internal network, and would prefer we don't either)

The content we block
The content we block is therefore for a few different reasons

Publisher requested
Video content which generates high traffic
Image based content which generates high traffic
Game websites which make a lot of frequent ajax calls or something

We generally block sites when they cause us a problem not speculatively, and there aren't many publisher requests. This means the file is relatively static, whilst retaining the ability to change it quickly.
The sites we've blocked for causing trouble in the past have often been pornography etc, but the critical part here is that we blocked them based on an operation issue, not an editorial one. If someone genuinely wants to annotate that material, we don't actually have an issue with that, we just can't always afford the speed and bandwidth to proxy it.
This has a few implications:

If there is a way to make the site performant, it should come off the list for viahtml
When presenting users with pages about blocked content we give different feedback based on the block reason
For publisher requested sites we tell you the content isn't allowed
For other sites we say the content isn't available "Through the Hypothesis service"

Our wording here is a bit vague and could do with some tightening up I think. Basically I think the difference is "You can't annotate this" and "Annotate all you like using the browser extension". We actually suggest the extension for both, so I guess I still have some outstanding questions about the publisher one
Implementation ideas

It's handy to use the same list if possible
We hope viahtml can handle some sites that via cannot
To split the difference here we may want to qualify what types of content we are blocking more specifically
For example if sites are being blocked for being video sites, and that's not a problem, we can say so and let them through in viahtml only
Happily the block file contains a "reason" element and the only one that currently matters is "publisher-blocked", so we can change the rest to be more specific

The blocklist is served from here: https://hypothesis-via.s3-us-west-1.amazonaws.com/via-blocklist.txt
There are details about updating it here: https://stackoverflow.com/c/hypothesis/questions/102/250#250
Pie in the sky dreaming
It would be amazing if we kept stats about what sites cost us the most bandwidth so we can be a bit more evidence based. This would fit really neatly in a graphite/graphana style world, but we don't currently have something like that.
It might be interesting to see if @indigobravo has any ideas about a cheap and cheesy way to collect this info without re-inventing the wheel and creating our own crap version of a monitoring and metrics system.
There are other stats that would probably be nice to know too.

Done! But the blocklist needs to be edited to add the new categories


Implementing lama-refinement?

Hi,
I am trying to implement the lama refinement feature and hard time to figure it out.
Any idea where I start?

Lama Cleaner use the Torchscript model to do inference, as far as I know, lama-refinement requires backward process and need at least 24GB vRAM, so one should copy all training code(which is a lot) and integrate it into https://github.com/Sanster/lama-cleaner/blob/main/lama_cleaner/model/lama.py

Thanks will check it out


Low T-count for reflection

Improves T-count for ReflectAboutInteger and considers common case index == 0 as special case.

Somehow the license/cla check got stuck on this PR, and I'm not sure how to restart it...


Apps-generated tokens doesn't give full access to GraphQL queries/mutations

Hey everyone,
We are trying to use GraphQL mutations to get and add data to the database but we keep getting "You do not have permission to perform this action", even with a token generated from an app in the dashboard.
We use Saleor-platform v2.11, the tokens we use should have full access to the store so we don't know what to do now...
Is it a known issue?
How can we bypass this problem?
Thanks!

{
  "errors": [
    {
      "message": "You do not have permission to perform this action",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "customers"
      ],
      "extensions": {
        "exception": {
          "code": "PermissionDenied",
          "stacktrace": [
            "Traceback (most recent call last):",
            "  File \"/usr/local/lib/python3.8/site-packages/promise/promise.py\", line 489, in _resolve_from_executor",
            "    executor(resolve, reject)",
            "  File \"/usr/local/lib/python3.8/site-packages/promise/promise.py\", line 756, in executor",
            "    return resolve(f(*args, **kwargs))",
            "  File \"/usr/local/lib/python3.8/site-packages/graphql/execution/middleware.py\", line 75, in make_it_promise",
            "    return next(*args, **kwargs)",
            "  File \"/app/saleor/graphql/core/fields.py\", line 164, in connection_resolver",
            "    iterable = resolver(root, info, **args)",
            "  File \"/app/saleor/graphql/decorators.py\", line 15, in wrapper",
            "    return func(info.context, *args, **kwargs)",
            "  File \"/app/saleor/graphql/decorators.py\", line 31, in wrapper",
            "    raise PermissionDenied()",
            "saleor.core.exceptions.PermissionDenied: You do not have permission to perform this action"
          ]
        }
      }
    }
  ],
  "data": {
    "customers": null
  }
}

https://docs.saleor.io/docs/3.0/developer/users#authentication
Are you adding auth headers to the request in proper format? ie:
{
  "Authorization": "Bearer APPTOKEN"
}

Hello @krzysztofwolski,
Thanks for the reply.
Yes, this is how my headers look like in the GQL playground.
I tried with 'token', saleor says the user is anonymous when I use this one.
I get the permission error when I use 'crsfToken', so I guess this is the one I need to use.
I wonder if there's a problem in the way saleor checks the current user's permissions, but in the meantime everything seems to work when I log in with the same user in the dashboard...
I'm trying to see if there's a problem in saleor but it's a bit complicated with the dev skills I have as of today :)

Ok so in fact I didn't use a correct header...
It works better that way:
{
  "Authorization": "JWT <Token>"
}

I was using Bearer instead of JWT with the csrfToken but actually the right mix seems to be "JWT MyToken".


Port changes of [#17164] to branch-2.8

What changes are proposed in this pull request?

when s3 write type is CACHE_THRU, initiateMultipartUpload creates MultipartMetaFile without closing the outstream, causing leak in BlockWorkerClient resource.
createFilePOption in CopyObject didn't set any write type (which should respect alluxio.proxy.s3.writetype), causing all objects copied are in the MUST_CACHE write type.

Why are the changes needed?
To fix the above 2 problems.
Does this PR introduce any user facing changes?
No.
[This is an manually-generated PR to cherry-pick committed PR https://github.com/Alluxio/alluxio/pull/17164 into target branch branch-2.8]

alluxio-bot, merge this please


Created README

With couple of useful links to that wanderers know where to start

@bonustrack does a readme seem useful to you?

Closing this PR for now, feel free to open an issue to add a readme.


[!] Refactor architecture

During a sabbatical from phpDocumentor I got thinking that phpDocumentor 2 was too change resistant was because it was architected for too much flexibility and compartmentalised into illogical divisions.

An example of the latter was the way the transformer worked; you need to be in the phpDocumentor\Transformer, phpDocumentor\Compiler and parts of the phpDocumentor\Plugin\Core namespaces. Anyone with knowledge of the architecture would figure this out but it makes it harder for those that don't: the most people

An additional issue was that the UBIQUITOUS LANGUAGE was lacking in clarity and was also not properly expressed in the domain of phpDocumentor.
As a thought experiment I did a partial re-implementation of phpDocumentor if I would have done a complete rewrite. Although part of me would want to finish the greenfield implementation and replace the current version of phpDocumentor with it this would not be convenient as a lot of business rules would be lost and a long time would pass until phpDocumentor 3 is available.
What this basically comes down to is that we apply some concepts from DDD and clearly define 3 aggregates:
Documentation

Definition

And undiagrammed: the Template aggregate.
In addition we want to make use of a more clear UBIQUITOUS LANGUAGE, including the use of DDD concepts such as REPOSITORIES and FACTORIES.
We identify the following concepts at the moment:
Definition

Documentation
    Project
        A Project is a series of Documentation spanning multiple versions where each version refers to the same undertaking.
        Examples of a project can be a Software Project, such as phpDocumentor_, or the authoring of a Book or standalone
    reference, such as PSR-5_.

        phpDocumentor does not actively track a Project but instead assumes that all Versions that are defined in the configuration belong to the same project.

    Version
        A Version of a Project represents a unique series of files that can be transformed into a set of Documentation. It
    is not necessary for the locations of Document Groups to be on the same Data Source as sometimes manuals and source
    files are located on different locations.

    > For example: the source files used to generate an Api Reference may be in a different VCS repository than the
    > source files for the User Manual for the same Version

    Documentation

    Document Group
        A set of Documentation can include several multiple Guides, Api References and similar objects. Such an object is
    called a Document Group as it combines into a logical grouping and offers the option to add new Document Groups
    in the future should the need arise.

Guides
    Guide
    Guide Language
    Document
    Translation
    Chapter
API Documentation
    Api Reference
    Api Language
    Fqsen
    Element
    PHP
        Visibility
        Namespace
        Constant
        Function
        Class
        Interface
        Trait
        Class constant
        Property
        Method
        DocBlock
        Summary
        Description
        Tag
Rendering
    View
    Renderer
    RenderAction
    Template

not all terms are provided with more information; this may be added upon later if needed

This is a significant effort that will allow phpDocumentor to grow and also to help people get acquinted with phpDocumentor. I have identified the following areas that need to be addressed but I am confident more will show during the refactoring process and these should be added to the list below:

[x] Add the Version Entity and VersionNumber Value Object
[x] Add the Documentation Entity and DocumentGroup Value Object
[x] Route file access via FlySystem
[x] Add the FlyFinder plugin so that we have access to the new Finder
[x] Add all specifications that are in place now (not all are implemented yet)
[x] Add the Definition Aggregate and move as much of the configuration options there
[x] Replace the Transformer with the Renderer and Writers with RenderActions

This looks great :+1:

Thanks @TomasVotruba!

Closing this issue as most of the moving parts have been implemented or are being implemented; I will create new issues to specifically address points that still need to be done


Goodsync - wrong checksum

Description of issue
Goodsync fails to install due to a checksum mismatch.
Output:
melon@thor:~$ brew cask install goodsync
==> Satisfying dependencies
==> Downloading https://www.goodsync.com/download/goodsync-v10-mac.dmg
######################################################################## 100.0%
==> Verifying checksum for Cask goodsync
==> Note: running "brew update" may fix sha256 checksum errors
Error: Checksum for Cask 'goodsync' does not match.

Expected: 40eb93ddd65fb153b2c6fc332326381a198c9ad3c80de223a7102e840ef4b068
Actual:   ee94b9715c6f7b8785f8c24c0da8f08426c1824b27eb6a140fb81c771414d050
File:     /Users/melon/Library/Caches/Homebrew/Cask/goodsync--10.5.1.dmg

To retry an incomplete download, remove the file above.
Error: Install incomplete.

@MelonSmasher this is due outdated cask. I have submitted PR to update this cask.

https://github.com/caskroom/homebrew-cask/issues/36344


Add remote dispatcher

Closing this because N300 FD was redesigned and L chip dispatcher is the same as R chip dispatcher


ColoredConsoleTarget - Added DetectOutputRedirected to skip coloring on redirect

Notice that target-branch is master

Codecov Report

Merging #3523 into master will increase coverage by <1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #3523    +/-   ##
=======================================
+ Coverage      80%     80%   +<1%
=======================================
  Files         361     361
  Lines       28702   28702
  Branches     3831    3831
=======================================
+ Hits        22956   22972    +16
+ Misses       4644    4626    -18
- Partials     1102    1104     +2

(restarted build for unstable test)

yet another flaky test...

Updated wiki: https://github.com/NLog/NLog/wiki/ColoredConsole-target


RuntimeError: unreachable executed

models.zip
Models from here, via the Sample STEP files link.

Foxtrot is a proof-of-concept demo, not an industrial-strength CAD kernel. It may not work for your models! Even in the screenshots above, there are a handful of surfaces that it fails to triangulate

https://github.com/Formlabs/foxtrot#disclaimer

Foxtrot is a proof-of-concept demo, not an industrial-strength CAD kernel. It may not work for your models! Even in the screenshots above, there are a handful of surfaces that it fails to triangulate

Could it ignore failing surfaces instead of stop the whole process?

@kzhsw there is a free online alternative option to view STEP format files (as well as IGES and BREP formats): https://3dviewer.net/

@kzhsw there is a free online alternative option to view STEP format files (as well as IGES and BREP formats): https://3dviewer.net/

Thank you for referencing this. This is based on occt-import-js, which is much slower than foxtrot on handling large models.


just to test CI

just to test the CI. I think Swift 5.3 regressed allocations.

@swift-server-bot test this please


Refactor OperationalCredentialsDelegate.h so that GetIntermediateCACertificate and GetRootCACertificate are asynchronous

Problem
Currently, in OperationalCredentialsDelegate.h, GetIntermediateCACertificate and GetRootCACertificate are blocking call. It'll be beneficial if both call can be made asynchronous with callbacks. Some use cases, obtaining root certificate and intermediate certificate requires a network call.
Proposed Solution
Refactor GetIntermediateCACertificate and GetRootCACertificate in OperationalCredentialsDelegate.h so that it's asynchronous.

Fixed by https://github.com/project-chip/connectedhomeip/pull/8550


Please clarify migration from active workspace to newly created cluster

Two points of confusion to consider clarification:
When the cluster is provisioned, the commitment tier is set, will the customer be immediately billed for the cluster or is that after a workspace is linked? If the workspace is already running on a commitment tier, how does that effect pricing? Lets say they pay for a workspace 500MB tier on July 1st. They create a cluster on July 10th and the link the workspace on July 15th. Will they be charged twice for the first month? The doc should clarify.
2nd request for clarification:
"When a Log Analytics workspace is linked to a dedicated cluster, new data ingested to the workspace, is routed to the cluster while existing data remains in the existing Log Analytics cluster."
What is the user experience here? Will they visually see 2 workspaces (one on the cluster and one where previously ingested data resides until archival. Do you need to do anything to manage or cleanup after?
How does this impact archival?
Will this break data export or ADX integration?

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

ID: 336076fa-b185-2d98-7b5c-75b6540c99e7
Version Independent ID: b1728f53-57c5-273b-8d75-29cd781f1a56
Content: Azure Monitor Logs Dedicated Clusters - Azure Monitor
Content Source: articles/azure-monitor/logs/logs-dedicated-clusters.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@AndrewBlumhardt
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for reaching out Avatar.
Billing can occur only once, either on workspace, or per cluster's plan after the link. If cluster's billingType is set to "Cluster", the workspace billing would be attributed to the cluster, but if billingType is set to "Workspaces", ingestion billing is distributed between linked workspaces proportionally to their ingestion.  In your example, workspace is billed until the 10th, and then billed per the cluster plan.
Regarding the 2nd question, previously ingested data remains and isn't moved. The service handles the cross query, retention, and any other aspect that provides seamless experience to customers without any action on their part.

Hey @AndrewBlumhardt! Thanks for the suggestions 👍
We've updated the article based on your feedback. Appreciate your help keeping our docs clear and accurate.
Thanks!
#please-close


Logseq converter

Ok, this is my attempt to update the PR originally made by @tekacs.
The only addition on my part of far is renaming 3 functions as discussed in the previous PR.

@eirikhm who are the active reviewers on this repo?


Official Implementation

Hello,
I would like to know if this is the official implementation of the paper "Incorporating External Knowledge through Pre-training
for Natural Language to Code Generation", if not how does it relate to the official implementation ?

Hi,
No, this is a clone of the original one done to update and upgrade it.
We did not improve the results a lot, though.
You can find the original repository by going to the root of this project and following the link after "cloned from".

Thank you for the fast response. I found out about this project by researching a Java ASDL grammar text file. I found a simplified one in this project, could you by any chance point me to a complete Java ASDL grammar text file ?

I cannot point you to another grammar, but even if it is called simplified, this one was able to parse completely a large corpus of Java code.

I cannot point you to another grammar, but even if it is called simplified, this one was able to parse completely a large corpus of Java code.

I have a couple of questions (I promise they might be my last ):

Citation: I’m currently drafting a research article and would like to reference this grammar and testing approach accurately. Could you advise on your preferred citation format?

Test File Selection: I’m interested in understanding the methodology used to choose the test files for validating the ASDL grammar. Could you provide insight into how these files were selected? For example, were they based on common Java constructs, edge cases, or specific language features?

Thank you in advance!


docker event polling support

This PR is to address issue#208
Summary of the changes:
A separate docker event monitor process created during each polling iteration. Since docker monitor is a blocking, it needs to be created as a separate process. Further, it shares a queue with crawler and puts container creation event(deletion event is commented out for now) into queue, which is then immediately crawled. If there are no new container creation events, then periodic crawling gets exercised. Ideally, there should be a single monitor process running all the time, but then this process needs to be started with worker (which would break the abstraction).
If there are any container creation events while the monitor process is not ready (while crawling all containers for their periodic routines or crawling recently notified container event) then those events will be missed and those containers will be crawled in periodic crawl.

Current coverage is 88.60% (diff: 62.12%)

Merging #215 into master will increase coverage by 0.01%

@@             master       #215   diff @@
==========================================
  Files            86         88     +2
  Lines          3162       3220    +58
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2801       2853    +52
- Misses          361        367     +6
  Partials          0          0

Powered by Codecov. Last update 8ed157e...96bdc4e

Looks good!

What's the main motivation behind polling- missing crawling a container that comes and goes in a single crawler frequency cycle?
What's the ideal scenario again where a worker breaks the abstraction?
Can missing events be avoided in the ideal scenario?
The test case description is very helpful!

@sahilsuneja1
0. Main motivation behind polling:
Yes, for short-lived containers (think of serverless platform), we want to be able to crawl them.
1. Worker abstraction:
Polling is a blocking pattern i.e. the polling process has to be blocked for an event. So it needs to be a separate process. Now, ideally, we should have a single instance of this process which then need to be started with workers. But in current work, to keep it optional and abstracted thing for docker only, it is spawned in dockercontainers.
2. handling missing events:
I think it should be fine, since we can not crawl container which has lifetime shorted than time it takes to crawl them. (i.e. crawl-time > container-lifetime)

Instead of killing that polling process, can it be kept alive, dumping all start events to the same queue across all worker iterations? This way you wouldn't miss an event?

Yes thats the ideal solution we want. For that we need to create and maintain reference to the queue from worker class and create polling process along with worker.

Can you test the following two things please: 1. check for memory and file descriptor leaks after letting it run for an hour or two with a bunch of containers being created and deleted; 2. can you double check that if lots of containers are created (>1000s) in a burst, things don't get hairwire (like if the time spent crawling from events is larger than the sleep interval).

@ricarkol thanks for pointer on timeout wrapper. I have made change accordingly.
Besides, I ran some performance tests to measure container create events successfully captured by crawler. Experiment and observations noted below.

Start crawler with interval frequency 120 sec, save o/p to local file
Create 100 new redis container in loop. Measure the time taken by each docker run.
On average a new container is created every 200 ms, causing a potentially container create event.

Time taken by each container:
['0.135', '0.162', '0.163', '0.156', '0.169', '0.172', '0.141', '0.209', '0.181', '0.171', '0.176', '0.164', '0.18', '0.163', '0.183', '0.185', '0.177', '0.164', '0.18', '0.155', '0.182', '0.163', '0.176', '0.156', '0.179', '0.205', '0.18', '0.171', '0.182', '0.175', '0.182', '0.177', '0.182', '0.173', '0.228', '0.182', '0.177', '0.186', '0.174', '0.186', '0.182', '0.183', '0.172', '0.194', '0.18', '0.221', '0.176', '0.208', '0.209', '0.166', '0.186', '0.173', '0.197', '0.181', '0.22', '0.242', '0.218', '0.196', '0.222', '0.213', '0.189', '0.184', '0.196', '0.186', '0.23', '0.534', '0.275', '0.194', '0.185', '0.205', '0.176', '0.19', '0.186', '0.216', '0.196', '0.548', '0.654', '0.223', '0.249', '0.212', '0.203', '0.191', '0.199', '0.197', '0.238', '0.995', '0.251', '0.217', '0.27', '0.495', '0.594', '0.559', '0.637', '0.841', '0.813', '0.917', '0.947', '1.045', '1.322', '1.501']

~40% container create events are missed by crawler, for its crawl time is greater than create time interval

Containers successfully crawled
['1', '3', '5', '7', '8', '10', '12', '14', '16', '18', '20', '22', '24', '26', '28', '30', '32', '34', '35', '37', '39', '41', '43', '45', '47', '48', '50', '52', '54', '55', '56', '58', '59', '60', '62', '64', '66', '67', '69', '71', '73', '74', '76', '77', '78', '79', '80', '82', '84', '86', '87', '88', '90', '91', '92', '93', '94', '95', '96', '97', '98', '99', '100']

Computed stats of all o/p files and measured their relative mtimes. e.g. order all files by their mtimes and then compute diff between consecutive items. Although this is not clearly a throughput of crawler, but also contains poll sleep between docker events.

Still good to understand these numbers. New o/p file is created every ~350 ms.
['0.328', '0.304', '0.328', '0.232', '0.340', '0.364', '0.336', '0.364', '0.328', '0.348', '0.336', '0.360', '0.360', '0.368', '0.348', '0.380', '0.324', '0.252', '0.372', '0.356', '0.368', '0.360', '0.408', '0.324', '0.248', '0.372', '0.376', '0.360', '0.224', '0.264', '0.376', '0.208', '0.264', '0.384', '0.368', '0.744', '0.292', '0.384', '0.376', '0.372', '0.232', '0.900', '0.492', '0.204', '0.256', '0.244', '0.392', '0.404', '1.204', '0.248', '0.316', '0.740', '0.596', '0.556', '0.720', '0.848', '0.908', '0.904', '0.924', '1.256', '1.396', '1.044']

After 120 secs, when poll interval expires and crawler enters periodic polling mode, it crawls all 100 containers. Measured the o/p file creation interval again. (this is probably better representation of crawler throughput)

['0.032', '0.036', '0.036', '0.040', '0.036', '0.036', '0.036', '0.036', '0.036', '0.044', '0.136', '0.024', '0.032', '0.024', '0.136', '0.028', '0.128', '0.036', '0.124', '0.032', '0.032', '0.036', '0.032', '0.028', '0.132', '0.032', '0.028', '0.132', '0.028', '0.132', '0.028', '0.132', '0.036', '0.040', '0.132', '0.028', '0.132', '0.028', '0.128', '0.028', '0.028', '0.032', '0.128', '0.036', '0.028', '0.040', '0.136', '0.028', '0.132', '0.028', '0.128', '0.032', '0.028', '0.132', '0.032', '0.128', '0.024', '0.140', '0.028', '0.140', '0.028', '0.136', '0.028', '0.132', '0.032', '0.028', '0.136', '0.032', '0.144', '0.028', '0.132', '0.032', '0.132', '0.036', '0.132', '0.028', '0.132', '0.032', '0.128', '0.028', '0.136', '0.028', '0.136', '0.028', '0.128', '0.028', '0.136', '0.028', '0.128', '0.032', '0.148', '0.032', '0.044', '0.128', '0.032', '0.152', '0.032', '0.132', '0.028']

@nadgowdas: Why does test 448.1 fail?

@sahilsuneja1 I am not sure why 448.1 failed. I believe its travis error for resource allocation. Can you restart travis tests?

@nadgowdas: restarted but failed again!

Few updates from offline discussion with @ricarkol and @sahilsuneja1

Current crawler implementation supports numprocess=1 and with this constraint we can work with this PR. (although loosing some events for short-lived containers).
Need to gauge requirement and urgency for multi-threaded workers numprocess > 1. In this implementation, the event design need to change, since it would cause all worker threads to register for events and process same events.
there we would need to have a single event manager (as a separate process) for all workers  which would manage container creation and deletion events and also scheduling these events on right worker thread.

@nadgowdas:Thanks for collecting these points here. Still need to address the test that fails even after restart

@sahilsuneja1 I am not able to see the travis errors in Details. Are you able to view it ?

Yes, https://travis-ci.org/cloudviz/agentless-system-crawler/jobs/186859983

For some reasons this opens me blank page.

How about https://s3.amazonaws.com/archive.travis-ci.org/jobs/186859983/log.txt

thanks. This worked for me

I've added check in the test-case to check for docker version >=1.8.1 for docker event support.

Thanks; merging; enhancements later.


ISIS-2052: Remove unused private constuctors

Please reject the PR, it breaks the test. Sorry -j

closed without merging, as requested.


ESConfiguration Custom Resource Issue

Creation of stack failing and Rollback initiated because the following issue:
"CloudFormation did not receive a response from your Custom Resource. Please check your logs for requestId [a910c57e-19a1-421f-9f6a-fa922b54181e]. If you are using the Python cfn-response module, you may need to update your Lambda function code so that CloudFormation can attach the updated version."

Researching about the issue, I've found this re:Post Knowledge Article that can help, maybe...
How do I update the AWS CloudFormation cfn-response module for AWS Lambda functions running on Python 2.7/3.6/3.7?

Hi Bruno, thanks for reporting the issue.
Those pointers are to improve response from Lambda failures to cloudFormation, but don't address the issue on Lambda function that was failing.
Either way I did a quick fix in the failing lambda function by specifying a version for the request module.
Can you please try to deploy it again and let me know if it works for you?
Thanks!
Pablo


chore(master): release 1.1.224

:robot: I have created a release beep boop
1.1.224 (2023-10-24)
Miscellaneous

deps: update dependency tailwindcss to v3.3.4 (092b6c1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.224 :sunflower:


Investigate the number of directly adjacent targets introduced by target padding.

Issue moved to broadinstitute/gatk #2880 via ZenHub


⚠ ClamAV has degraded performance

In 2407124, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 77 ms

Resolved: ClamAV performance has improved in 9db4743.


fix(chat): remove extra symbols from auth method

The page at https://dev.mixer.com/reference/chat/methods/auth has some extra symbols (quotes, commas) that probably shouldn't be there.
Additionally, the link to the "Connection" section is fixed to point to the correct page.

It's unrelated to the content of this PR, but maybe a link to the commit message guidelines could be added to the contribution guidelines? :)


A path in an 'extends' option must be relative or rooted, but 'tslint:all' is not.

Bug Report

TSLint version: 5.9.1
TypeScript version: 2.6.2
Running TSLint via: (pick one) CLI

with tslint.json configuration:
i had this until 5.9.1 (with 5.8.0 it was no problem)
"extends": [     "tslint:latest"   ],
this will throw now this error:
error TS5024: Compiler option 'extends' requires a value of type string.
tried this:
"extends": "tslint:latest",
and got this error:
error TS18001: A path in an 'extends' option must be relative or rooted, but 'tslint:latest' is not.

Hello Google user! If you're NOT using tslint, you're in the wrong place, you're looking for:
https://github.com/Microsoft/TypeScript/issues/18865
You may be running into this issue because TypeScript can't find what you're extending.
If you're trying to extend a module, e.g. "extends": "@sindresorhus/tsconfig", you have to make sure it's installed where you're calling tsc

if you're calling tsc locally, what you're extending needs to be installed locally (npm install @sindresorhus/tsconfig)

Or just make sure you're calling the local tsc


Optional chaining in scoped-elements Cache.js breaks webpack 4

Expected behavior
Recently a minor update (1.2.4 -> 1.3.0) of @open-wc/scoped-elements introduced optional chaining implementation that breaks all webpack 4 builds.
Imo, this kind of thing should have been a major bump because lots of builds are going to pull in 1.3.0 updates that are still on webpack 4 (vue cli, create-react-app, storybookjs all are still on webpack 4) automatically and then have broken builds.
Webpack 4's version of acorn isn't updated to the latest and won't be, but in lots of places upgrading to webpack 5 is a non-starter. Therefore, webpack cannot support optional chaining, so the current solution has been to back down the version of scoped-elements to 1.2.4, the last working version
IMO, this line:
https://github.com/open-wc/open-wc/blob/master/packages/scoped-elements/src/Cache.js#L22
should be reverted back to non-optional-chained impl and a major bump release instead to re-include it.

It's strange that webpack doesn't support standard javascript syntax, but we had this issue with import.meta for a long time as well. In this case I think it's fine to rewrite the code to remove the usage, the optional chaining doesn't add that much here.

Is the version of acorn that webpack uses. Is an older one, webpack 5 I think has an updated version, but it’s acorn that didn’t like the optionals I believe

Webpack issue about this is here: https://github.com/webpack/webpack/issues/10227
The issue is closed because they have fixed it in Webpack 5, they will not backport it to webpack 4.
Webpack 5 support in progress:

React https://github.com/facebook/create-react-app/pull/7929
Vue https://github.com/vuejs/vue-cli/pull/5611
Angular https://github.com/angular/angular-cli/pull/18873
Storybook https://github.com/storybookjs/storybook/issues/9216

For storybook, here is what you can do, .storybook/main.js
module.exports = {
  // ...
  webpackFinal: async config => {
    return {
      ...config,
      module: {
        ...config.module,
        rules: [
          ...config.module.rules,
          {
            test: [
              new RegExp(`node_modules(\\/|\\\\)@open-wc(.*)\\.js$`),
            ],
            use: {
              loader: require.resolve('babel-loader'),
              options: {
                plugins: [
                  require.resolve('@babel/plugin-proposal-optional-chaining'),
                ],
                presets: [
                  [
                    require.resolve('@babel/preset-env'),
                    {
                      useBuiltIns: 'entry',
                      corejs: 3,
                    },
                  ],
                ],
                babelrc: false,
              },
            },
          },
        ],
      },
    };
  },
};

for vue-cli, vue.config.js:
module.exports = {
  transpileDependencies: [
    // can be string or regex
<EMAIL_ADDRESS>  ]
}


⚠ Bofrost.es has degraded performance

In d682138, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1501 ms

Resolved: Bofrost.es performance has improved in bfae17e after 1 hour, 7 minutes.


[CT-858] [Enhancement] Connection is always closed after each query

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Every time a query is executed, it is then closed. This occurs with 1->XX threads, tested up to 16.
When using dbt-snowflake this causes you to have to re-login every time you issue a query, which if you have hundreds of models this can cause a massive slowdown as authentication to Snowflake is slow, especially when you are a long distance away from the server (Perth, AU -> US East 1 is 250ms for example, so having to reconnect every query when you have hundreds of models is unpleasant).
I have logged an issue on Snowflake here - https://github.com/dbt-labs/dbt-snowflake/issues/201 , but I believe this is on the dbt-core level.
Expected Behavior
A single connection is made. It would be even better if a login request for SF was made in a single thread, then that's reused for all. That would also fix MFA as well, I think? But that is out of scope
Steps To Reproduce

Run dbt-snowflake
Set the threads to say 2
Have 4 queries, execute them all
Run this query:

select *
from table(information_schema.login_history_by_user())
order by event_timestamp desc;

You can see it in the logs as well:
10:24:59.898232 [debug] [ThreadPool]: Opening a new connection, currently in state closed

Relevant log output
10:24:59.898232 [debug] [ThreadPool]: Opening a new connection, currently in state closed

### Environment

```markdown
- OS: Linux, Mac
- Python: 3.10.4/3.9
- dbt: 1.1.1

What database are you using dbt with?
snowflake
Additional Context
No response

Thanks for opening here as well @joshuataylor!
I think you're right, the behavior going on here is defined in dbt-core, and could be relevant to other adapters as well. It does seem to yield the trickiest complications on Snowflake.
Today, during a dbt run, dbt opens separate connections for:

the caching (metadata) queries it runs at the start of dbt run (connection named 'master')
each model it compiles/runs

It then closes each connection as it completes.
Risks to be wary of here:

I do remember, a long time ago, that we ran into issues around leaving Snowflake connections open, such that the running thread would never yield: https://github.com/dbt-labs/dbt-core/issues/2645, https://github.com/dbt-labs/dbt-core/pull/2650
If a dbt run completes, and leaves a connection open, on data warehouses that support "auto-resume" compute, it can have real $$ implications: https://github.com/dbt-labs/dbt-spark/issues/280, https://github.com/dbt-labs/dbt-spark/pull/285

So the goal here would be to reuse / recycle more connections while dbt is running, while still guaranteeing that at the end of a run, dbt always closes all its connections. (In an ideal case, we'd also handle authentication in a single thread, and be able to reuse that auth across multiple threads — but that feels like it might be out of scope for this effort.)
At its very simplest, the idea would be:

Don't close connections once they complete, but mark them as "done"
Instead of creating new connections, set_connection_name should try to grab a "done" connection from the pool, rename it, and use it

I think this is likely to be a big lift, requiring a deep dive into some internals of dbt's adapter and execution classes that we haven't touched in some time. I'm not sure when we'll be able to prioritize it. I agree that it feels important.
Relevant code
The context managers. The default behavior is to release a connection as soon as it's done being used.
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/adapters/base/impl.py#L209-L226
Called once for each node that compiles/runs:
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/task/base.py#L312
"Releasing" a connection, which actually means closing it:
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/adapters/base/impl.py#L182-L189
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/adapters/base/connections.py#L182-L195
For a connection with conn_name, check to see if any existing connections by that name, otherwise open a new one:
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/adapters/base/connections.py#L123-L160
Cleanup that happens at the very end of runnable tasks:
https://github.com/dbt-labs/dbt-core/blob/e03d35a9fc606f5b1fc7e94c0bc04dbe88f558cf/core/dbt/task/runnable.py#L438-L439

I'll have a dig through and see what if I can find an elegant solution that hopefully (🤞) won't impact connections such as Spark.
As an alternative, in dbt-snowflake we could also check if the connection is closed, the token is valid then reuse the connection, as it should still be set on the handle.
As another alternative, if we could solve this on the dbt-snowflake level in the interim this would be a big speed win. We could add the token when logged in to the connection, then add it to connection. This would involve updating the connection contract though, maybe adding a metadata or other key that connectors could use?

@joshuataylor Ah - so, rather than reusing connections, just reusing the result of authentication?
I don't have a good sense of whether there are any potential security risks with taking that approach. If it works, though, and substantially speeds up the process of opening new connections, that our current dbt-core approach (treating connections as a commodity) might hold muster for the foreseeable future.
I'll reopen the dbt-snowflake issue with that scope in mind, since the changes would be specific to that codebase.

We are running into this issue with our dbt on redshift, and it is making building models that are separated out for modularity annoying as each model needs to create a connection to redshift when it is run. If I have 5 separate models that each need a connection, this adds a significant time cost as compared to a single model that only needs one.
I'm wondering if there are any plans to prioritize this?

Could https://github.com/dbt-labs/dbt-snowflake/pull/428 be used? This has been working out great
I also have a local dev version that does a few tricks to cache etc, but my hacks should only be used in development :).

Adding a "We're still interested" note to this item.
Right now we are more or less sidelined by the inability to reuse MFA results across multiple queries for dbt.


Feature/animal module - Ectothermic metabolism

Description
This adds a temperature sensitive ecothermic metabolism and a temperature insensitive endothermic metabolism to the functional groups. This also works as trial approach to having temperature sensitive functions for ectothermic animals broadly.  Let me know if you think there is a better way to be implementing the idea of temperature input as I intend to reuse the structure.
The temperature variable I am currently using will be replaced with a reading of the actual temperature data from the grid square and vertical layer the animal occupies.
I have split the ever-growing dummy_animal_module file into separate files focusing on the 'AnimalCohort' class, the 'AnimalCommunity' class, and the group of dummy plant and soil classes.
I have also added new input validity checks and revised the testing for 'AnimalCohort'.
Fixes # (issue)
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Codecov Report

Merging #230 (f5a8c83) into develop (299e39b) will decrease coverage by 0.31%.
The diff coverage is 89.32%.

@@             Coverage Diff             @@
##           develop     #230      +/-   ##
===========================================
- Coverage    95.23%   94.93%   -0.31%
===========================================
  Files           36       38       +2
  Lines         1553     1598      +45
===========================================
+ Hits          1479     1517      +38
- Misses          74       81       +7

Impacted Files
Coverage Δ

...tual_rainforest/models/animals/functional_group.py
86.84% <76.19%> (-13.16%)
:arrow_down:

...al_rainforest/models/animals/animal_communities.py
90.47% <90.47%> (ø)

...ual_rainforest/models/animals/scaling_functions.py
95.23% <90.90%> (-4.77%)
:arrow_down:

...rainforest/models/animals/dummy_plants_and_soil.py
92.30% <92.30%> (ø)

...irtual_rainforest/models/animals/animal_cohorts.py
98.07% <94.44%> (ø)

virtual_rainforest/models/animals/animal_model.py
100.00% <100.00%> (ø)

virtual_rainforest/models/animals/constants.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hi Taran,
I saw earlier in this thread that Diego had mentioned setting up universal constants in core. Same issue applies to the scaling factor (-0.65) that’s in this equation – we want to avoid hard-coding that.
Regards,
Rob
From: Taran @.>
Sent: Wednesday, June 7, 2023 2:46 PM
To: ImperialCollegeLondon/virtual_rainforest @.>
Cc: Ewers, Robert M @.>; Review requested @.>
Subject: Re: [ImperialCollegeLondon/virtual_rainforest] Feature/animal module - Ectothermic metabolism (PR #230)
This email from<EMAIL_ADDRESS>originates from outside Imperial. Do not click on links and attachments unless you recognise the sender. If you trust the sender, add them to your safe senders listhttps://spam.ic.ac.uk/SpamConsole/Senders.aspx to disable email stamping for this address.
@TaranRallings commented on this pull request.

In virtual_rainforest/models/animals/scaling_functions.pyhttps://github.com/ImperialCollegeLondon/virtual_rainforest/pull/230#discussion_r1221630739:

if metabolic_type == MetabolicType.ENDOTHERMIC:

   return terms[1] * mass_g ** terms[0]

elif metabolic_type == MetabolicType.ECTOTHERMIC:

   b0, exponent = terms

   return (

       b0 * mass_g**exponent * exp(-0.65 / (BOLTZMANN_CONSTANT * temperature_k))

   )

else:

   raise ValueError("Invalid metabolic type: {metabolic_type}")

Alright, thanks folks!
—
Reply to this email directly, view it on GitHubhttps://github.com/ImperialCollegeLondon/virtual_rainforest/pull/230#discussion_r1221630739, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOG5KF7HD7FHMZHGURNQVPTXKCAYNANCNFSM6AAAAAAYVVQPK4.
You are receiving this because your review was requested.Message ID<EMAIL_ADDRESS>


Question about Test establishment

Hi,
I'm a student learning 802.11 fuzzing.
From my understanding. To try the script from you, I need 3 devices: Target AP, Target STA, and attack laptop(with 2 WNICs, one for monitor, one for attack). Is my understanding correct?
In addtion, I quickly check the codes, before the fuzzing, the script uses 'monitor WNIC' to scan the AP network to find the 'target STA'?
So the 'attack WNIC' should be 'monitor mode', the 'Monitor WNIC' should be 'Normal Mode'?
In the readme.md, to switch 'attack WNIC' to monitor mode, 'sudo airmon-ng check kill' command is used to kill process, after kill these process, how can I set the connection between 'Monitor WNIC' and 'Target AP'?
Many thanks

Hello,
Yes, you will need an AP and a STA, along with a WNIC device, to use this tool. To fuzz SAE frames, you will need two different WNICs, one for attacking and one for monitoring. For the remaining frames (Management, Control, Data), you can use only one WNIC, which the tool will monitor and attack with the same WNIC.
Any WNIC you are going to use must be in monitor mode. This has to be done for the tool to be able to use this WNIC. Usually, to convert a WNIC into monitor mode, you will have to execute the following commands, depending on the WNIC driver:
sudo airmon-ng check kill
sudo airmon-ng start wlan0

or
sudo airmon-ng check kill
sudo ip link set wlan0 down
sudo iw dev wlan0 set type monitor
sudo ip link set wlan0 up

Now, for the attacker WNIC to be able to ping the targeted STA, both devices must be connected to the same network. To achieve this, the attacker's VM must be connected to this AP with an Ethernet cable. Otherwise, any VM in NAT/Bridged mode would also work. For instance, in our case, we used a laptop, an Ubuntu VM as the attacker, a smartphone for the STA and a USB WNIC. The laptop and the smartphone were connected to the same AP, which the fuzzer was going to target. A USB WNIC was directly connected to the Ubuntu VM, to inject frames to this AP, with the assist of this fuzzer.
Although, the ping functionality is needed to identify a case in which the STA becomes unresponsive, i.e., remains connected to that AP but without any internet access, the tool can be used without this functionality, by commenting out any relevant lines in the code. After that, there is no need for the attacker to be connected to this AP, and the tool will only monitor for Deauth/Disass frames.
If you have issues with the ping requests, you can restart the machine you are using the fuzzer and instead of executing this command, which disables all network relevant services:
sudo airmon-ng check kill

disable only the wifi service, with this command:
sudo service wpa_supplicant stop

Hi,
Thanks for your explaination!
I successfully build the test environment.
During some testing, I receive 'STA is unresponsive'.
And I think the hexdump below is the frame which maybe cause STA offline etc.

Is that means some vul happened?

Hi,
It could possibly be vuln, but then again you have to make sure that this is not a false positive.
You can do it directly through the fifth mode of the WPAxFuzz tool called ''DoS attack module'', where you constantly transmit the above frame and wait to see how the connection between the AP and the STA will react.

Closing due to inactivity. Feel free to re-open it again, if needed.


FR: Set a (soft) limit

A lot of people give their users a soft limit on disk usage as hard limits can get things to corrupt. It would be great if you could set a soft disk usage limit, so the user can see how much they 'have left' or how much they are 'over their limit'.
As of now, there's no way to check this easily in Craft..

Version 1.2.0 and 2.2.0 are out with that change.


It is not working anymore. Nov 09 2023

Hi there,
Since today morning the ads screen blocking is not working anymore.
What happening is, when I click on any youtube, the screen flashes quickly, I see the ads screen, then the screen goes like Fullscreen then get closed and it return to the youtub video, however the video is black and nothing is playing.
More info:
I am using Chrome, it got updated yesterdays.

My aplogies, it worked. I don't know why the adsblock got activated.
I disabled the AdBlock and returned normal, the ads screen is not showing anymore.
Thank you!.

You can close the issue then if its solved

Yes sir, I'll close it, You may mentioned that the Adblock sometimes reactivated after browser or extension update.


Create loggers property for Trainer and LightningModule

What does this PR do?
Implement a loggers property that returns a list of loggers. This is meant to replace LoggerCollection, which will then be deprecated.

Part of #11232
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Setters for trainer.logger and trainer.loggers have been implemented.
Discussion can be found here https://github.com/PyTorchLightning/pytorch-lightning/issues/11232#issuecomment-1027467784

We are going to handle the special case for the loggers setter where someone assigns trainer.loggers = [LoggerCollection]
This will make it easier to refactor the codebase such that anywhere the user passes in a logger, we can easily put it in a list and assign it to trainer.loggers.
If we try to manually check and sanitize this everywhere it occurs in the codebase, it will be harder to deprecate LoggerCollection and trainer.logger. Sanitizing the input to trainer.loggers would also remove information about whether the user provided a LoggerCollection, which would lead to incorrect behavior if the LoggerCollection is of size 1. There are multiple solutions to this problem but, in my opinion, they are all messier than adding a special case to the setter.

Could someone please rerun the CI and enable auto-merge?

Can we update this PR to include the proposal in https://github.com/PyTorchLightning/pytorch-lightning/issues/11232#issuecomment-1032146770?


chore(main): release 1.5.2

:robot: I have created a release beep boop
1.5.2 (2023-06-19)
Bug Fixes

MiniProgram: fix bug in sendRequestPost method for UrlLink class (e6f48d0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/deloz/wechat-bridge/releases/tag/v1.5.2 :sunflower:


Adding translation file for Persian (Ir-Fa) Language

@kingofnull thank you, great job, added in 1.0.18


Clarify every and NULL

More details: https://github.com/prisma/prisma/issues/11819#issuecomment-1258160223

Just encountered the same issue as in the comment, and I think it's a particular gotcha for an m-to-n relationship, where some of the records had no related entries in the other side.
I've been thinking how to express the issue so it's easy to understand by readers with varying degrees of experience in the area of SQL, programming and boolean logic.
Taking inspiration from the discussion in the thread, I propose this note of caution:

[!CAUTION]
Using every when querying a model with a reverse relationship, of which there are related records that do not relate to any records of the queried model, can lead to surprising results.
For example, consider a query for categories, where every post matches a given postId. When there are any categories without assignments (CategoriesOnPosts), the following query would also return those categorties in addition to categories that are related to the requested Post. I.e:
await prisma.category.findMany({
  where: {
    posts: {
      every: { postId },
    },
  },
});

This behaviour is intentional, and is designed to be naturally aligned with the Javascript Array.every() function. For instance, consider filtering in-memory objects in a JS runtime such as Node.js or a browser, where an array of categories:
// GIVEN category.posts is either an empty array [] or is an array with some Post ids
categories = [
  {name: 'Category A', posts: []},
  {name: 'Category B', posts: [1, 2]}
];
categories.every(category => category.posts.includes(postId)).length
// outputs 2

I reckon we should add this note (or a similar one to it) in one, or in possibly more than one, of these sections:

https://www.prisma.io/docs/orm/prisma-schema/data-model/relations/many-to-many-relations#explicit-many-to-many-relations
https://www.prisma.io/docs/orm/reference/prisma-client-reference#every
https://www.prisma.io/docs/orm/prisma-client/queries/relation-queries#filter-on--to-many-relations

@janpio @meletj what are your thoughts? Is the wording good? Is the proposed section to add it to good?

We could also add a recommendation to achieve the desired outcome with. I can think of a number of options, each generating a slightly different query:

Query the categories of a given post - useful when you only want to get the post and the category
Query the relation table directly - useful when you need info on the assignment
Query the categories as before, but add the some: {} relation filter alongside the every filter - this filters out any categories with no posts.

I think either of these may be apropriate in different situations, so I think we should add all of them. If you know of any other options, which are more recommended, we can add them

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


training taking too long on local GPU

I am training a quadruped to walk similar to deepmind mujoco tutorials. I am running the training on an Nvidia L4 GPU
Colab Script - https://colab.research.google.com/drive/1uL1TKBYXOYpeajOiaF1whhsB0SSKPEe3?usp=sharing
the following cell takes a very long time and never finishes execution. I reduced the timesteps to just 10
train_fn = functools.partial(
      ppo.train, num_timesteps=10, num_evals=10,
      reward_scaling=1, episode_length=1000, normalize_observations=True,
      action_repeat=1, unroll_length=5, num_minibatches=32,
      num_updates_per_batch=4, discounting=0.97, learning_rate=3.0e-4,
      entropy_cost=1e-2, num_envs=64, batch_size=64,
      network_factory=make_networks_factory,
      randomization_fn=domain_randomize,
      policy_params_fn=policy_params_fn,
      seed=0)

env = envs.get_environment(env_name)
eval_env = envs.get_environment(env_name)
make_inference_fn, params, _= train_fn(environment=env,
                                progress_fn=progress,
                                eval_env=eval_env)

on reducing the timesteps, i thought the training would complete faster. I made sure that jax is utilizing GPU. in addition my GPU shows 100% usage in nvidia-smi
Any help would be appreciated!

I was using a non mjx compatible robot xml which threw no errors but slowed everything down.

@GandharvMahajan How did you convert the robot xml to be compatible with MJX? I am also noticing slow down with a custom environment.

@uvikas you can see some examples here and here of models that were modified to work well with MJX/brax


Server performance metric: completed vs. scheduled samples per second

For the Server scenario, we have observed a slight difference between the current performance metric ("Scheduled samples per second") and the actual performance metric ("Completed samples per second" reported under "Additional Stats") e.g.:
================================================
MLPerf Results Summary
================================================
SUT name : SUT
Scenario : Server
Mode     : Performance
Scheduled samples per second : 1497.65
Result is : VALID
  Performance constraints satisfied : Yes
  Min duration satisfied : Yes
  Min queries satisfied : Yes

================================================
Additional Stats
================================================
Completed samples per second    : 1497.60

The difference seems to become more pronounced on systems with higher target QPS.
This issue also affects compliance testing (which is how we discovered it in the first place).
I would suggest to swap the two metrics.

@georgelyuan  please take a look.

WG: the proposal is to use "Completed samples per second" for reporting the server performance, rather than the "Scheduled samples per second". Anton will provide more data points.

WG: we will keep the  "Scheduled samples per second" as the official metric. It is by design that "Completed samples per second" will be slightly different.


Feature/remove admin settings

Removed references to old admin settings table.
Using Dotenv for these values.

You'll also want to also drop setting_fields from the schema, right?

@stackus - I believe you are correct, will update this PR before merging. Thanks.

Closing this because it broke rake sample:demo

sorry for the noise, recreated this branch from scratch here


Stuck With Leshy Bug

Hi! I've had a problem where I can't select anything after I enter in. Leshy says welcome to the card creator and then I'm locked, I can't do anything except exit the game. I've gotten it to kind of work one time except there were no cards to put my stats on, but that was when I first downloaded it. I've tried putting it into the games files manually and using the Thunderstore mod manager. I also checked to see if I have the required files, and I do. Thank you!

Please include the mod list and log/error when this issue occurs.

I have the same problem. the log say this:
[Error  : Unity Log] MissingMethodException: void SaveFile.NewPart1Run()
Stack trace:
UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) (at <3f8c35799b544274b189c23afcaaf82e>:0)

My mod are:
API_dev-API
IngoH-Badly_Translated_Inscryption
IngoH-InGame_Card_Creator
IngoH-Language_Loader
IngoH-Modded_Save_Fixer
Kopie-Smalltweak_Hammer
MADH95Mods-JSONCardLoader
SpecialAPI-BoonAPI
SpecialAPI-NodeAPI

I have the same problem. the log say this:
[Error : Unity Log] MissingMethodException: void SaveFile.NewPart1Run() Stack trace: UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) (at <3f8c35799b544274b189c23afcaaf82e>:0)

That means your Inscryption is outdated. Updating to V1.08 should fix it.

Ok, l update it and now work thank you.

Close this?


feat: added .github/workflows/close-issue-with-banned-phrases.yml

What is the purpose of this pull request?

[ ] New Icon
[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other: Spam blocker

Description

Auto close brand requests.
Example
https://github.com/jguddas/lucide/issues/15
Before Submitting

[x] I've read the Contribution Guidelines.
[x] I've checked if there was an existing PR that solves the same issue.

Now I'm kinda hopein someone creates spam, well it won't take long if history is any indication.


Disconnect client

I have a problem with discoverability of the device and the reconnection after i disconnect from a client.
The only solution was to reboot the rpi and i be able to re-discover and re-connect from my client app.
If you have some advice for resolving this issue.
EDIT: I have an Error 133 (0x85) when i tried to reconnect.

I faced the same issue.


Use Angular 5 and typescript

Much rather prefer Vue, since it takes a day to learn and doesnt force us to use a full SPA backend, but every module can have its own page and its own Vue instance without any hassle.
If people wanna use plain JS to build their modules thats fine, and if they wanna use typescript that works too.

not in the next 2 - 3 years, after that we might re-implement the back-office


Refuse to run dbgeng in 32 bit python on 64 bit windows

What actually happens when you do run dbgeng in 32-bit python on 64-bit windows? I seem to remember some headache when I was investigating the potential dbgeng solutions.

Never mind, I caught up in Slack. LGTM.

Attaching the debugger to a 64 bit process croaks with HRESULT=0xd0000055, which I interpret as "Yeah don't do that".


Filament panels aren't registered when configured inside a module

I created a module to store all Filament-related files, but I noticed that the panels aren't registered when I load them inside a module (adding them to the module.json providers array). Everything works perfectly if I load the ServiceProvider inside the module directly through the bootstrap/providers.php file.
use Filament\PanelRegistry;

//...

public function register(): void
{
  $this->app->resolving(PanelRegistry::class, function () {
      dd('test!!!');
  });
}

Any idea why the app->resolving callback isn't being called when the ServiceProvider is loaded through the module.json? Thank you!

you may need to run php artisan module:clear-compiled to update the cache.


fix(TypeScript): correct typing on Session

I was trying out the library in a TypeScript Next.js app and I thought it was weird that Session.set didn't return the value that was passed in. I realised that it actually does after digging in a little bit, so this PR is primarily to fix that.
Additionally, I thought I'd check the other types on the Session and also found that Session.save returns the cookie header value as a string.

Perfect :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No tagging of repository versions

Actual
This repository does not use tagging.
This makes any cross referencing of the source with version history, and understanding detailed changes between versions hard.
Expected
For every release there should be a corresponding repository tag with the source for that version.

Please tag releases.

We are now using releases, thanks for the suggestion!


Community Operator: ODH Release 1.0

Signed-off-by: Landon LaSmith<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@mvalarh This PR converts to the bundle format and we removed our old legacy channel which pointed to operator versions <=0.5.2 that had no upgrade path to the current operator. The CI tests are failing because we removed the old versions and allow/operator-recreate is not enabled. Is this something I can enable myself for this PR CI tests?

@LaVLaS since you are deleting old existing versions, we need you to confirm that operator will be recreated. You can confirm this action by clicking Ready to review button.

@mvalarh Do you need any additional confirmation from me to enable allow/operator-recreate?

/ok-to-test

@LaVLaS package name and directory name differs
Package name from '/tmp/community-operators-for-catalog/community-operators/opendatahub-operator/0.6.0/metadata/annotations.yaml' is 'opendatahub' and it is not same as operator dir 'opendatahub-operator'


New namespace reduce added with function toValue()

Hi @Smoren,
I've added a new namespace reduce with function toValue() and totally covered it by tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
172

Relevant Lines:
172

💛  - Coveralls

Hi @anastansa,
Thank you for your pull request!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
172

Relevant Lines:
172

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
172

Relevant Lines:
172

💛  - Coveralls


doc: add notes on server urls with path

Description
Currently paths in server url is silently ignored, and it makes confusion (see #561 for example). Update the doc to address the limitation.

@ldez updated the target branch to v1.4.


HTTP/2 limit header accumulated size

Motivation:
The Http2FrameListener requires that the Http2FrameReader accumulate ByteBuf objects. There is currently no way to limit the amount of bytes that is accumulated.
Modifications:

DefaultHttp2FrameReader supports maxHeaderSize which will fail fast as soon as the maximum size is exceeded.
DefaultHttp2HeadersDecoder will respect the return value of endHeaderBlock() and fail if the max size is exceeded.

Result:
Frames which carry header data now respect a maximum number of bytes that can be accumulated.

@nmittler - PTAL

@Scottmitch looks good ... just a few comments.

LGTM

Cherry-picked 4.1 (ecacd11) master (a9e57f1).
Thanks for review!


Remove System.Collections.Immutable dependency

Fixed #1040

Thanks. Will merge when green.

@jskeet it looks like there's some error diffing files. Is this a known issue or something my change caused?

Ah yes, bug in your change - dependencies are in apis/apis.json. CI noticed that regenerating the project files would have put the dependency back in. You need to remove it from apis/apis.json. (Another option is to remove it from there to start with and run generateprojects.sh.) Sorry that it's a bit convoluted, but it saves a lot of time in other ways :)

No that's ok, I just haven't been following too closely recently so I wasn't aware of that file. Let's see if this get's things working.


Examples are outdated because SchemeObject fields are now protected

SchemeObject.x, SchemeObject.y, SchemeObject.width, SchemeObject.height now can be obtained with SchemeObject.getBoundingRect(), but there is not way to access SchemeObject.params.
params can hold user's data, and an access must be provided somehow.
Example widely uses SchemeObject fields as public ones.

added getters for x, y, width, height, params https://github.com/NikitchenkoSergey/scheme-designer/commit/9a360b9265d45bd3554de1ce6b62eae56e407ace


[jsonConfig]: wrong warninmg for password, if use a tableItem

No existing issues.

[X] There is no existing issue for my request.

Problem location
ioBroker Log
Warning message
[
{
"instancePath": "/items/_calendars/items/calendars/items/5",
"schemaPath": "#/definitions/passwordProps/additionalProperties",
"keyword": "additionalProperties",
"params": { "additionalProperty": "attr" },
"message": "must NOT have additional properties"
},
{
"instancePath": "/items/_calendars/items/calendars",
"schemaPath": "#/patternProperties/%5E.%2B/allOf/23/if",
"keyword": "if",
"params": { "failingKeyword": "then" },
"message": "must match "then" schema"
},
{
"instancePath": "/items/_calendars",
"schemaPath": "#/properties/items/patternProperties/%5E.%2B/allOf/8/if",
"keyword": "if",
"params": { "failingKeyword": "then" },
"message": "must match "then" schema"
}
]
Source link

Additional information
The UI works as expected, but Schema validation is wrong here.
I get the warning also in IDE:

here is the json file:
jsonConfig.json
for "sendTo" I get also warning for "width" and "title"

the json config seems to be valid in the meantime


Generic simulation method for AgentType

(related to #495 )
AgentType with a:

single source of truth for the transition functions
usable by a generic simulation function

depends on #761

This is the FrameAgentType
https://github.com/econ-ark/HARK/pull/865

closing as duplicate

closing as duplicate


feat: fix layout

Codecov Report

Merging #3 (2bf3e29) into master (0d0ae41) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2bf3e29 differs from pull request most recent head 21ff0d8. Consider uploading reports for the commit 21ff0d8 to get more accurate results

@@          Coverage Diff           @@
##           master      #3   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           9       9
  Lines          68      68
  Branches        4       4
======================================
  Misses         68      68

Impacted Files
Coverage Δ

src/component-panel/ComponentTree.tsx
0.00% <ø> (ø)

src/component-panel/index.tsx
0.00% <ø> (ø)

src/previewer.tsx
0.00% <ø> (ø)

Continue to review full report at Codecov.

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


LookupError: gradient registry has no entry for: ClipByValue

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
binary
Tensorflow Version
2.13.0-dev20230210
Custom Code
Yes
OS Platform and Distribution
Google Colab
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Error occurred when mixed precision policy and `tf.saturate_cast` used together.
No error without mixed precision policy.

Standalone code to reproduce the issue
https://colab.research.google.com/drive/1hWTWBG7S8h6q4DK1qBNl2id0aK7h-3n4?usp=sharing

Relevant log output
---------------------------------------------------------------------------
StagingError                              Traceback (most recent call last)
<ipython-input-6-1edf848c14d2> in <module>
     12 model = models.Model(inputs=inputs, outputs=outputs)
     13 model.compile(optimizer='adam', loss='mse', run_eagerly=False, jit_compile=True)
---> 14 model.fit(images, labels, epochs=1, batch_size=2)

1 frames
/usr/local/lib/python3.8/dist-packages/keras/utils/traceback_utils.py in error_handler(*args, **kwargs)
     68             # To get the full stack trace, call:
     69             # `tf.debugging.disable_traceback_filtering()`
---> 70             raise e.with_traceback(filtered_tb) from None
     71         finally:
     72             del filtered_tb

/usr/local/lib/python3.8/dist-packages/tensorflow/python/framework/func_graph.py in autograph_handler(*args, **kwargs)
   1169           except Exception as e:  # pylint:disable=broad-except
   1170             if hasattr(e, "ag_error_metadata"):
-> 1171               raise e.ag_error_metadata.to_exception(e)
   1172             else:
   1173               raise

StagingError: in user code:

    File "/usr/local/lib/python3.8/dist-packages/keras/engine/training.py", line 1284, in train_function  *
        return step_function(self, iterator)
    File "/usr/local/lib/python3.8/dist-packages/keras/engine/training.py", line 1249, in run_step  *
        outputs = model.train_step(data)
    File "/usr/local/lib/python3.8/dist-packages/keras/engine/training.py", line 1054, in train_step  **
        self.optimizer.minimize(loss, self.trainable_variables, tape=tape)
    File "/usr/local/lib/python3.8/dist-packages/keras/optimizers/optimizer.py", line 542, in minimize
        grads_and_vars = self.compute_gradients(loss, var_list, tape)
    File "/usr/local/lib/python3.8/dist-packages/keras/mixed_precision/loss_scale_optimizer.py", line 1249, in compute_gradients
        grads_and_vars = self._optimizer.compute_gradients(
    File "/usr/local/lib/python3.8/dist-packages/keras/optimizers/optimizer.py", line 275, in compute_gradients
        grads = tape.gradient(loss, var_list)

    LookupError: gradient registry has no entry for: ClipByValue

Hi, @shkarupa-alex
Aplogize for the delay and I was able to replicate the same issue with tf-nightly on Google Colab and also on Ubuntu 20.04 , for your reference I have added gist-file-tf-nightly and screenshot below and it's giving me same error as you mentioned above in the error log output LookupError: gradient registry has no entry for: ClipByValue
Here is screenshot for code execution on Ubuntu 20.04 with tf-nightly:

I tried the same code on latest stable version of Tensorflow==2.11 and it's working fine as expected for your reference I have added gist file. Thank you!

Hi, @shkarupa-alex
I apologize for this issue got closed after stale from Bot and I have tested the same code with Tf-nightly and our tensorflow pre-release tensorflow==2.12.0rc0, I'm getting the same error, for your reference I have added gist-file and we'll have to dig more into this issue and we'll update you soon. Thank you!

Hi, @sachinprasadhs
Could you please look into this issue ? Thank you!

@qlzh727, the ClipByValue gradient is commented out and there is a TODO assigned to you from 2018: https://cs.opensource.google/tensorflow/tensorflow/+/master:tensorflow/python/ops/clip_ops.py;l=126;drc=ec69e86e01436b58d53f49b12782a0452bd26c8c. Do you remember why it's commented out?

@qlzh727, the ClipByValue gradient is commented out and there is a TODO assigned to you from 2018: https://cs.opensource.google/tensorflow/tensorflow/+/master:tensorflow/python/ops/clip_ops.py;l=126;drc=ec69e86e01436b58d53f49b12782a0452bd26c8c. Do you remember why it's commented out?

I think I did this when I was on sync rotation for TF, and it was failing for the presubmit when I try to merge the change. I don't remember for the context of what's the failure now. Maybe we can try to re-enable it again?

@qlzh727 can you do this? If you don't have time I can try.

@qlzh727 can you do this? If you don't have time I can try.

I am actually quite booked for other tasks at the moment. Thanks for the help.


ePromos to confirm if they want recommendation widget on PDP

Yes, we want this widget. Are further details necessary at this point?


Use host shell in terminal when running inside Flatpak

This improves the usability of Lapce significantly for people unaware of the $ flatpak-spawn --host bash trick. With this, the Lapce Flatpak's default terminal runs on the host outside of the container and has access to all your regular programs. Without this patch, what you get is a sandboxed shell that only has access to whatever is available inside of the Flatpak container, which can be unexpected and confusing.

Can I somehow re-run the CI pipeline? I'm not familiar with GitHub actions.

Push new commits to branch or force-push branch

Tested both as flatpak and native - seems to work as intended.

@Newbytee Since users can deny access to org.freedesktop.Flatpak (via Flatseal for example) can you extend the flatpak detection to handle the following case?

Lapce runs in flatpak container
flatpak-spawn --host is not available

You can test this by running:
flatpak install --user https://dl.flathub.org/build-repo/107147/dev.lapce.lapce.flatpakref
flatpak override --user --no-talk-name=org.freedesktop.Flatpak dev.lapce.lapce
flatpak run --user dev.lapce.lapce

In this case the terminal doesn't start at all. There should be a fallback to the default (in-container) terminal.
After testing, run flatpak override --user --reset dev.lapce.lapce

@Newbytee Since users can deny access to org.freedesktop.Flatpak (via Flatseal for example) can you extend the flatpak detection to handle the following case?
* Lapce runs in flatpak container

* `flatpak-spawn --host` is not available

You can test this by running:
flatpak install --user https://dl.flathub.org/build-repo/107147/dev.lapce.lapce.flatpakref
flatpak override --user --no-talk-name=org.freedesktop.Flatpak dev.lapce.lapce
flatpak run --user dev.lapce.lapce

In this case the terminal doesn't start at all. There should be a fallback to the default (in-container) terminal.
After testing, run flatpak override --user --reset dev.lapce.lapce

Thanks, that's a good idea. I didn't think about this possibility. I'll fix it as soon as possible. Maybe you could make an issue about this?


Installer error - unable to open registry key

Win 7 x64
Using Elm-Platform-0.15.exe from the link on the Elm Atchitecture tutorial page.

Tested also on Windows 8.1 x32 with no error.

My guess is that you did not have admin privileges when you tried it the first time, so it wasn't letting you edit a certain environment variable. It looks like you got it working though. If things are still messed up, please open an issue on the elm-platform repo describing more of your situation!


⚠ Bofrost.at has degraded performance

In cc80595, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1116 ms

Resolved: Bofrost.at performance has improved in a232866.


Project descriptions are displayed in the "Name" column [New UI]

Description of the bug
The projects listed show the description in place of the name, and thus no names of the projects are seen. Search terms for something in the name still works.
To Reproduce
Steps to reproduce the behaviour:

Go to Private or Public cloud products
Observe that project descriptions are in the "Name" section

Expected behaviour
The names of the products should be displayed in the "Name" column
Screenshots
Private Zenhub Image
Private Zenhub Image
Definition of done:

[ ] Project names are correctly displayed in the "Name" column

We have removed the description form the table


Error  Traceback (most recent call last): line 133, in      import eddlib.experiment

I would greatly appreciate any help on this error. I had re-installed statsmodel 0.9.0 thinking this might be the issue, but didnt seem to change anything. Any help would be greatly appreciated.
edd -g 5 --write-bin-score hg19.chromsizes hg19_unalignable_regions.bed BL2_CT_R1_rd.bam BL2_CTR2_rd.bam chipseq
Traceback (most recent call last):
File "/data/home/hfx541/.local/bin/edd", line 133, in
import eddlib.experiment
File "/data/home/hfx541/.local/lib/python2.7/site-packages/eddlib/experiment.py", line 19, in
import estimate
File "/data/home/hfx541/.local/lib/python2.7/site-packages/eddlib/estimate.py", line 1, in
from algorithm.max_segments import GenomeBins, IntervalTest
File "/data/home/hfx541/.local/lib/python2.7/site-packages/eddlib/algorithm/max_segments.py", line 7, in
import unalignable_regions
File "/data/home/hfx541/.local/lib/python2.7/site-packages/eddlib/algorithm/unalignable_regions.py", line 5, in
import pybedtools
File "/data/home/hfx541/.local/lib/python2.7/site-packages/pybedtools/init.py", line 16, in
from . import contrib
File "/data/home/hfx541/.local/lib/python2.7/site-packages/pybedtools/contrib/init.py", line 4, in
from . import long_range_interaction
File "/data/home/hfx541/.local/lib/python2.7/site-packages/pybedtools/contrib/long_range_interaction.py", line 357
print("%d (%.1f%%)\r" % (c, c / float(n) * 100), end="")
^

Aha! I just ran into this issue too.  pybedtools installation seems to be python3 (hence the print statement with brackets) whilst edd runs in python 2, so we need to downgrade pybedtools to an earlier version.
For me that meant reinstalling pybedtools with conda, making sure it didn't update pysam or statsmodel in the process.
I hope this helps!

Same issue here. They don't explicitly say it in their changelog, but I believe pybedtools removed Python 2 compatibility starting in version 0.9. pip install pybedtools==0.8.2 did the trick for me.

Thank you both for helping. I will try it this way

installing pybedtools==0.8.2 has solved the issue.
Many thanks


No longer works

I think the devs slightly changed the yes button on the are you still here popup. Either that or something else is causing issues. whatever the reason this script no longer works and I am not smart enough to fix it myself.

You should be able to just replace the images named "_afkΘ.png" with the ones you want to click, naming them the same. I'll re-download Roblox and try to fix it. You need to be in 1920x1080 resolution, and (I think) fullscreen should be off—though it might actually need to be on.

Ok does the picture have to be the exact same size as the old one?

You should be able to use any resolution; just make sure there are no moving elements (like water or characters that change every frame). The code takes a screenshot, checks if any target images are on the screen, and clicks if it finds a match.

Ok I will try this out when I get the chance.

Hi, I am sorry about the issues.
Even tho I am the maintainer, I have hundreds of projects in the works and can't dedicate any time to this anymore.
You're welcome to view my other guides, projects, mods, tools, and more.
Or if you'd like, you're more than welcome to create a PR and continue as I normally do push and merge them within 24hrs

I've never done that before but I can try.


Suspicios-looking code in Instant

Sorry if this is a false alarm, but is it intentional that https://github.com/ThreeTen/threetenbp/blob/master/src/main/java/org/threeten/bp/Instant.java#L492 never returns any value, and always throws an exception, either a DateTimeException or an UnsupportedTemporalTypeException?
If this is intended, maybe it would help to add a short code comment?

The bug here is that it should always throw UnsupportedTemporalTypeException. Not a big bug though.


I can't run Meteor on Windows 7 or 10

After install i am able to run meteor creat fro the appdata/local/.meteor directory.  When I cd to simple-todo i see ".meteor, js, html,css" directory and files.  When I open a browser and type "http://localhost:3000"  i receive "This webpage is not available"...Please let me know if you are able to assist.

Duplicate from #55 and not related to this plugin.

I insist. If you can run Meteor, there's nothing that this plugin can do. You have to found out why you can't run meteor like any regular command. This plugin just does that.

I apologize PEM, I was attempting to post on Meteor and responding to email directly.  Thank you for your help.