Unable to write cache, deploy falls back on previous version

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d1c9mkw1ler3mc
AWS Region
us-east-1
Amplify Hosting feature
Deployments
Frontend framework
Next.js
Next.js version
13.4.2
Next.js router
App Router
Describe the bug
During the Cloning Repository phase of my Amplify deployments (prior to running the build phase) I am seeing a "Unable to write cache" warning:
[INFO]: # Retrieving environment cache... [WARNING]: ! Unable to write cache: {"code":"ERR_BAD_REQUEST","message":"Request failed with status code 404"})} [INFO]: ---- Setting Up SSM Secrets ---- [INFO]: SSM params {"Path":"/amplify/d1c9mkw1ler3mc/main/","WithDecryption":true} [WARNING]: !Failed to set up process.env.secrets [INFO]: Setting NEXT_PRIVATE_STANDALONE=true to produce .next/standalone directory [INFO]: # Retrieving cache... [INFO]: # Extracting cache... [INFO]: # Extraction completed [INFO]: # Retrieved cache
During the Frontend phase of the deployment, the Build info suggests Amplify is building a previously deployed (cached?) version (not the current version) with build errors that have been fixed in the current version.
I checked that the commit hash in the Cloning Repository phase is correct, tried creating a new branch and deploying a new Amplify app, reconnected the repository, and added "Next.js version latest" as a live package update.
Nothing caused Amplify to get the current version of the connected repository branch to build in the deployment process.
Any help you can provide would be greatly appreciated.
Expected behavior
I expect the current version of my connected Next.js 13 repository branch to be checked out, cached, and built during the Amplify deployment process.
Reproduction steps

Redeploy recent failed deployment
Create new app with new branch and deploy
Update repository with new commit and deploy

Build Settings
Build appears to be working, but using a previous version of the code.  Problem occurs prior to build.

Log output

2023-07-23T12:07:28.576Z [INFO]: # Cloning repository: git@github.com:*******/******.git
2023-07-23T12:07:29.547Z [INFO]:
2023-07-23T12:07:29.550Z [INFO]: Cloning into '******'...
2023-07-23T12:07:29.550Z [INFO]: # Switching to commit: 0104f05f93e9d022d9f32e31523a2f254aab8803
2023-07-23T12:07:29.564Z [INFO]: Note: switching to '0104f05f93e9d022d9f32e31523a2f254aab8803'.
                                 You are in 'detached HEAD' state. You can look around, make experimental
                                 changes and commit them, and you can discard any commits you make in this
                                 state without impacting any branches by switching back to a branch.
                                 If you want to create a new branch to retain commits you create, you may
                                 do so (now or later) by using -c with the switch command. Example:
                                 git switch -c <new-branch-name>
                                 Or undo this operation with:
                                 git switch -
                                 Turn off this advice by setting config variable advice.detachedHead to false
                                 HEAD is now at 0104f05 Bump version
2023-07-23T12:07:29.602Z [INFO]: Successfully cleaned up Git credentials
2023-07-23T12:07:29.603Z [INFO]: # Checking for Git submodules at: /codebuild/output/src3541820501/src/*******/.gitmodules
2023-07-23T12:07:29.609Z [INFO]: # Retrieving environment cache...
2023-07-23T12:07:29.655Z [WARNING]: ! Unable to write cache: {"code":"ERR_BAD_REQUEST","message":"Request failed with status code 404"})}
2023-07-23T12:07:29.655Z [INFO]: ---- Setting Up SSM Secrets ----
2023-07-23T12:07:29.655Z [INFO]: SSM params {"Path":"/amplify/d1c9mkw1ler3mc/main/","WithDecryption":true}
2023-07-23T12:07:29.688Z [WARNING]: !Failed to set up process.env.secrets
2023-07-23T12:07:29.690Z [INFO]: Setting NEXT_PRIVATE_STANDALONE=true to produce .next/standalone directory
2023-07-23T12:07:29.692Z [INFO]: # Retrieving cache...
2023-07-23T12:07:36.238Z [INFO]: # Extracting cache...
2023-07-23T12:07:42.031Z [INFO]: # Extraction completed
2023-07-23T12:07:42.327Z [INFO]: # Retrieved cache

                                 # Starting phase: preBuild
                                 # Executing command: yarn install
2023-07-23T12:08:17.142Z [INFO]: yarn install v1.22.0
2023-07-23T12:08:17.198Z [INFO]: [1/4] Resolving packages...
2023-07-23T12:08:17.411Z [INFO]: success Already up-to-date.
2023-07-23T12:08:17.419Z [INFO]: Done in 0.28s.
2023-07-23T12:08:17.427Z [INFO]: # Completed phase: preBuild
                                 # Starting phase: build
2023-07-23T12:08:17.427Z [INFO]: # Executing command: echo "AUTH0_SECRET=$AUTH0_SECRET" >> .env.production
                                 # Executing command: echo "AUTH0_BASE_URL=$AUTH0_BASE_URL" >> .env.production
                                 # Executing command: echo "AUTH0_ISSUER_BASE_URL=$AUTH0_ISSUER_BASE_URL" >> .env.production
                                 # Executing command: echo "AUTH0_CLIENT_ID=$AUTH0_CLIENT_ID" >> .env.production
                                 # Executing command: echo "AUTH0_CLIENT_SECRET=$AUTH0_CLIENT_SECRET" >> .env.production
                                 # Executing command: echo "AUTH0_API_DOMAIN=$AUTH0_API_DOMAIN" >> .env.production
                                 # Executing command: echo "AUTH0_API_CLIENT_ID=$AUTH0_API_CLIENT_ID" >> .env.production
2023-07-23T12:08:17.427Z [INFO]: # Executing command: echo "AUTH0_API_CLIENT_SECRET=$AUTH0_API_CLIENT_SECRET" >> .env.production
                                 # Executing command: echo "JWT_SECRET=$JWT_SECRET" >> .env.production
                                 # Executing command: echo "NEXTAUTH_URL=$NEXTAUTH_URL" >> .env.production
                                 # Executing command: echo "NEXTAUTH_SECRET=$NEXTAUTH_SECRET" >> .env.production
                                 # Executing command: echo "GOOGLE_SEARCH_API_KEY=$GOOGLE_SEARCH_API_KEY" >> .env.production
2023-07-23T12:08:17.428Z [INFO]: # Executing command: echo "******_API_KEY=$******_API_KEY" >> .env.production
                                 # Executing command: yarn build
2023-07-23T12:08:17.549Z [INFO]: yarn run v1.22.0
2023-07-23T12:08:17.571Z [INFO]: $ next build
2023-07-23T12:08:17.842Z [INFO]: - info Loaded env from /codebuild/output/src3541820501/src/*******/.env.production
2023-07-23T12:08:18.046Z [INFO]: - warn No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
2023-07-23T12:08:18.057Z [INFO]: Attention: Next.js now collects completely anonymous telemetry regarding usage.
2023-07-23T12:08:18.057Z [INFO]: This information is used to shape Next.js' roadmap and prioritize features.
                                 You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
                                 https://nextjs.org/telemetry
2023-07-23T12:08:18.144Z [INFO]: - info Creating an optimized production build...
2023-07-23T12:08:37.899Z [WARNING]: Failed to compile.
2023-07-23T12:08:37.901Z [WARNING]: ./components/common/givingPageNav/index.tsx
                                    Module not found: Can't resolve '@/components/ui/container'
                                    https://nextjs.org/docs/messages/module-not-found
                                    ./components/givingPage/index.tsx
                                    Module not found: Can't resolve '@/components/ui/container'
                                    https://nextjs.org/docs/messages/module-not-found
                                    ./components/interest/index.tsx
                                    Module not found: Can't resolve '../interestsList'
                                    https://nextjs.org/docs/messages/module-not-found
                                    ./app/layout.tsx
                                    Module not found: Can't resolve '@components/provider'
                                    https://nextjs.org/docs/messages/module-not-found
                                    ./app/layout.tsx
                                    Module not found: Can't resolve '@/components/forbidden'
                                    https://nextjs.org/docs/messages/module-not-found
                                    > Build failed because of webpack errors
2023-07-23T12:08:37.940Z [WARNING]: error Command failed with exit code 1.
2023-07-23T12:08:37.940Z [INFO]: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
2023-07-23T12:08:37.946Z [ERROR]: !!! Build failed
2023-07-23T12:08:37.946Z [INFO]: Please check the supported SSR features to find if your build failure is related to an unsupported feature: https://docs.aws.amazon.com/amplify/latest/userguide/ssr-Amplify-support.html#supported-unsupported-features. You may also find this troubleshooting guide useful: https://docs.aws.amazon.com/amplify/latest/userguide/troubleshooting-ssr-deployment.html
2023-07-23T12:08:37.946Z [ERROR]: !!! Non-Zero Exit Code detected
2023-07-23T12:08:37.946Z [INFO]: # Starting environment caching...
2023-07-23T12:08:37.947Z [INFO]: # Environment caching completed

###
Additional information
From the logs above, the failure that starts with:
2023-07-23T12:08:37.899Z [WARNING]: Failed to compile.
2023-07-23T12:08:37.901Z [WARNING]: ./components/common/givingPageNav/index.tsx
                                    Module not found: Can't resolve '@/components/ui/container'
                                    https://nextjs.org/docs/messages/module-not-found

is from a previous version of the code.  It has been fixed in the commit 0104f05f93e9d022d9f32e31523a2f254aab8803 referenced in the Cloning Repository log output, but apparently not written to the cache that is used by the build process.

It appears there were issues with commits to Github that caused unchanged files with filename changes to not be included in the commit, which in turn caused the appearance of previous commits being cached in deployment.
The problem reported as above is not a deployment problem, and our issue has been resolved.


Fix CDT GCC parser to correctly parse C++ commands

Previously suggested parser command pattern:
xtensa-esp32-elf-(g?cc)|([gc]++)|(clang)
was incorrect for compilation of c++ files, as it is expanded by Eclipse into or between:
xtensa-esp32-elf-(g?cc)
([gc]++)
(clang)
Additional grouping around existing pattern solves that issue, and helps Eclipse to correctly find ESP-IDF includes for C++ files.

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

Oops, thank you for this @dziekon .
Could you please click through to sign the CLA link above? Even though it's a small change we prefer if everyone signs the CLA. Then we can merge this fix ASAP.

Done, and thanks for merging it.

Cherry-picked as e822b37.


Upgrade to Spring Boot 3

Description
We are currently on Spring Boot 2.7.X, in order to get the latest features we should upgrade to Spring Boot 3.
Proposed solution

Upgrade to Spring Boot 3

Additional context

Follow the migration guide
Some of the dependencies may have to be updated in order to support spring boot 3

okta-spring-boot-starter
okta-sdk-java

once our Okta update goes in, this looks like it'll be blocked until https://github.com/okta/okta-sdk-java/issues/826 is resolved

Don't anticipate a ton of files changed, but if so we can do a team smoke test

This is waiting blocked on a couple of fixes from Hibernate (HHH-16593) and Spring Graphql bug with the operationName

Looked some more into this and it seems like it should be fine to upgrade to 3.1.x as long as we pin hibernate to 6.1.x since they don't fix the bug until 6.3


remove code for single edgewidth per line

I've raised compat to<EMAIL_ADDRESS>where this is implemented. Fixes #28. Also fixes the error from #23 (the self edge will be still invisible though).

Codecov Report

Merging #29 (b03e169) into master (9af5469) will decrease coverage by 1.13%.
The diff coverage is n/a.

:exclamation: Current head b03e169 differs from pull request most recent head c603a29. Consider uploading reports for the commit c603a29 to get more accurate results

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   39.02%   37.88%   -1.14%
==========================================
  Files           2        2
  Lines         164      161       -3
==========================================
- Hits           64       61       -3
  Misses        100      100

Impacted Files
Coverage Δ

src/recipes.jl
83.72% <ø> (-1.07%)
:arrow_down:

Continue to review full report at Codecov.

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

Eh?

I have no idea what happend there. Some GH action hickup? I manually triggered the pull_request actions again and it worked the second try....


Some issues about TurbSim

Dear,
(1) When using turbulent wind in FAST.Farm, high-resolution and low-resolution grids need to be generated separately with TurbSim?
(2) If the above is true, which InflowFiles should each of the two output files (.bts) be placed in? Low-resolution output file to InflowFile in .fstf file? High-resolution output file into the InflowFile in .fst file? Mod_AmbWind choose 2 or 3?
(3) If the high resolution and low resolution have the same dy and dz, do you still need to solve them separately with TurbSim? Do not select a InflowFile as shown below?

Dear @May0048,
Here are my answers to your questions:

Distinct TurbSim runs for the low- and high-resolution domains are used when Mod_AmbWind = 3 in FAST.Farm.
See the documentation found here regarding naming of InflowWind files: https://openfast.readthedocs.io/en/main/source/user/fast.farm/InputFiles.html#ambient-wind-with-inflowwind-module-input-files.
If you expect to use the same TurbSim discretization for the low- and high-resolution domains, Mod_AmbWind = 2 in FAST.Farm is what you should use.

Best regards,

Dear NREL Scientist：
In my recent work, I was looking to generate wind files for some EWM load case by Turbsim V2.0. The data I have is the data of typhoon conditions, Corresponds to IEC 61400-1 tropical load case, which means the mean wind speed will near 57 m/s. However, when I read Turbsim's manual, I found that Turbsim can not generate such kind of wind files. I don't know if my understanding is correct, and if so, I don't know if there is any way to simulate typhoon conditions.
Best regards!

Dear @xiaoxiao90vip2022,
NREL has not updated TurbSim (yet) to support the tropical cyclone (T) class, but you can always use TurbSim to generate wind based on user-specified turbulence conditions, e.g., as discussed in the following topics on our forum:
https://forums.nrel.gov/t/turbsim-ewm/419
https://forums.nrel.gov/t/turbsim-for-class-iv-from-iec-61400-1-2nd-edition/2047
That is, you can use TurbSim to generate winds that match EWM for T class by specifying:
IEC_WindType = “NTM”
URef = 57 m/s
PLExp = 0.11
IECturbc = [user-specified] %
But your question is a good reminder that we should upgrade TurbSim at some point to directly support Edition 4 of IEC 61400-1.
Best regards,


Address not active when restoring wallet.dat file from Windows 7 version 0.17.3

On a Windows 10 PC with version 0.17.6, restoring a wallet.dat file from Windows 7 version 0.17.3 does not make the address “active” (the wallet has no receiving addresses and a zero balance). Manually selecting Receive REQUEST PAYMENT will “activate” the address. Other combinations of restoring wallet.dat files (Windows 7 version 0.17.6 to Windows 10 version 0.17.6, Windows 10 version 0.17.3 to Windows 10 version 0.17.6) work correctly and give an “active” address. Wallets on Qtum-Qt were restored by copying the wallet.dat file into the Qtum directory and launching the wallet. Using File – Restore Wallet with the reindex option shows the same issue with the Windows 7 version 0.17.3 wallet.dat file (launches with no active address).
Wallet debug logs show the Windows 10 version 0.17.6 loading the wallet.dat file for both the working and failed restorations, but mapAddressBook.size() is different.
Win 10 wallet.dat file:
2019-07-25T13:47:07Z [default wallet] nFileVersion = 170600
2019-07-25T13:47:07Z [default wallet] Keys: 2002 plaintext, 0 encrypted, 2002 w/ metadata, 2002 total. Unknown wallet records: 1
2019-07-25T13:47:08Z [default wallet] Wallet completed loading in             324ms
2019-07-25T13:47:08Z init message: Rescanning...
2019-07-25T13:47:08Z [default wallet] Rescanning last 3 blocks (from block 415776)...
2019-07-25T13:47:08Z [default wallet] Rescan started from block 415776...
2019-07-25T13:47:08Z [default wallet] Rescan completed in               2ms
2019-07-25T13:47:08Z [default wallet] setKeyPool.size() = 2000
2019-07-25T13:47:08Z [default wallet] mapWallet.size() = 0
2019-07-25T13:47:08Z [default wallet] mapAddressBook.size() = 1
2019-07-25T13:47:08Z mapBlockIndex.size() = 415785
2019-07-25T13:47:08Z nBestHeight = 415779

Win 7 wallet.dat file
2019-07-25T13:41:59Z [default wallet] nFileVersion = 170300
2019-07-25T13:41:59Z [default wallet] Keys: 2001 plaintext, 0 encrypted, 2001 w/ metadata, 2001 total. Unknown wallet records: 1
2019-07-25T13:41:59Z [default wallet] Wallet completed loading in             432ms
2019-07-25T13:41:59Z init message: Rescanning...
2019-07-25T13:41:59Z [default wallet] Rescanning last 43110 blocks (from block 372669)...
2019-07-25T13:41:59Z [default wallet] Rescan started from block 372669...
2019-07-25T13:42:07Z [default wallet] Rescan completed in            8147ms
2019-07-25T13:42:07Z [default wallet] setKeyPool.size() = 2000
2019-07-25T13:42:07Z [default wallet] mapWallet.size() = 0
2019-07-25T13:42:07Z [default wallet] mapAddressBook.size() = 0
2019-07-25T13:42:07Z mapBlockIndex.size() = 415785

Expected behavior starting with bitcoin core v0.16.


Write a Python program to convert kilometers to miles

Description
Write a Python program to convert kilometers to miles
1 mile = 1.609 kilometers
Input  : 8.04672
Output : 5

How to contribute

Save the solution in program/convert-kilometers-to-miles.py file
Add convert-kilometers-to-miles.py file in convert-kilometers-to-miles folder

!assign


Failure: create-release workflow

hi

Another failure occurred: https://github.com/sophiewigmore/tiny-stack-release/actions/runs/2078710914

Another failure occurred: https://github.com/sophiewigmore/tiny-stack-release/actions/runs/2078710914


Support for other charsets than UTF-8

I have some issues crawling Japanese websites with SHIFT-JIS encoding.
Nokogiri is not doing any automatic charset conversion to UTF-8.
I fixed it by rewriting the Page#doc method and using kconv.
    require 'kconv'

    def doc
      return @doc if @doc

      noko_en_id = {
        Kconv::UTF8 => 'UTF-8',
        Kconv::EUC => 'EUC-JP',
        Kconv::SJIS => 'SHIFT-JIS',
        Kconv::ASCII => 'ASCII',
        Kconv::JIS => 'ISO-2022-JP'
      }[Kconv.guess(@body || '')]

      @doc = Nokogiri::HTML(@body, nil, noko_en_id) if @body && html? rescue nil
    end

@pcboy Thanks for reporting this.
Could you please open a pull request so that your patch can be evaluated and tested?
Thanks!


msedgedriver 116. Rejecting request with non-local host: <IP_ADDRESS>:58982

Starting with msedgedriver and edge 116 our automated tests fail and report an error as follows:
[0-0] 2023-09-07T19:37:08.536Z WARN edgedriver: [1694115428.536][SEVERE]: Rejecting request with non-local host: <IP_ADDRESS>:58982
[0-0] 2023-09-07T19:37:08.540Z WARN webdriver: Request failed with status 500 due to Host header or origin header is specified and is not whitelisted or localhost.
The test scripts and service being connected to are running on localhost so I'm unsure what this means.
Note there is no such problem with chromedriver 116, 117, or 118 running in the same environment.
Is there a simple work around or switch to disable this validation?
I have tried these options: ['--no-default-browser-check', '--no-first-run',
"--allowed-origins=edgedriver", '--remote-allow-origins=', 'whitelisted_ips=',
"--allowed-ips=*"],
Thanks

Is your request coming from a web page in a browser? This is not usually necessary when using Selenium or other non-browser client.
You might be able to get this working with --allowed-origins=localhost or --allowed-origins=*.


Update util-core to 20.8.0

Updates com.twitter:util-core from 20.7.0 to 20.8.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-minor

Codecov Report

Merging #855 into develop will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #855      +/-   ##
===========================================
- Coverage    89.34%   89.28%   -0.06%
===========================================
  Files          117      117
  Lines         9326     9326
  Branches       365      365
===========================================
- Hits          8332     8327       -5
- Misses         994      999       +5

Impacted Files
Coverage Δ

.../main/scala/com/twitter/algebird/Applicative.scala
54.83% <0.00%> (-3.23%)
:arrow_down:

...src/main/scala/com/twitter/algebird/Interval.scala
80.86% <0.00%> (-1.74%)
:arrow_down:

.../main/scala/com/twitter/algebird/BloomFilter.scala
93.80% <0.00%> (-0.89%)
:arrow_down:

Continue to review full report at Codecov.

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


[MRG] Optimize read_source_estimate function

I'm having a weird use case for which I need to load in 2413 source estimate objects. While waiting for it to load, I looked at the source code and found a simple way to make the loading roughly twice as fast. Reading an stc is probably almost never a bottleneck, but hey, it's a small change shrug.
Seeking to the end of the file can be a costly operation on some file systems, especially when reading from a network mount. Since all bytes of the file are read during this function anyway, it makes more sense to
just read everything into a buffer instead of doing file seeks.

Codecov Report

Merging #4107 into master will decrease coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4107      +/-   ##
==========================================
- Coverage   86.11%   86.01%   -0.11%
==========================================
  Files         350      350
  Lines       63561    63564       +3
  Branches     9691     9691
==========================================
- Hits        54737    54674      -63
- Misses       6138     6208      +70
+ Partials     2686     2682       -4

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 e945956...1a6bb64. Read the comment docs.

travis complains

thx @wmvanvliet


enable save button

Targets #1506
Enable the save button if an error occurred while project details editing

closeing this PR and some changes

closing this PR since it has a lot of unwanted changes.


Localize dependency on ouroboros-network and cardano-slotting to ouroboros-consensus

Ideally, we would import from ouroboros-network and/or cardano-slotting only in a few core ouroboros-consensus modules and nowhere else, and then other ouroboros-consensus-* packages should not need to depend on either ouroboros-network or cardano-slotting at all.

Closed in https://github.com/input-output-hk/ouroboros-network/pull/2327 .


re_org Ratelimit package

Coverage increased (+0.2%) to 62.566% when pulling f3d21c5e3b83457b8aabe111fef8900fa4804bfc on wangkirin:ratelimit into 604ad498b46f28ca416212143b29fddb68117b5d on ServiceComb:master.

@WillemJiang  I have already fix th CI , PTAL


Update inscriptions and logo

Fractal Block

New Logo
Add more inscriptions

the new icon is just a webpage. Please modify it. @IBW3B

The icon should be an image.


Orientation for Tiled Hexagon map off?

I created a Hexagon map in Tiled with a staggered odd index:

And when I convert the layout with the bevy_ecs_tilemap in helpers/tiled.rs (code) the map appears incorrectly if I define the HexCoordSystem as ColumnOdd.

It appears correctly if I define the HexCoordSystem as ColumnEven:

I think this behavior is somewhat "expected" because Bevy's y-axis is inverted relative to Tiled.
This was brought up previously in a stalled out PR that added support for StaggerAxis / StaggerIndex https://github.com/StarArawn/bevy_ecs_tilemap/pull/405#discussion_r1148564979 to that example, and the consensus seemed to be that it's correct to use the opposite index for column stagger when building the tilemap.

This appears to be an asset loading error with tiled. Please remember that the tiled example is only meant to be a minimal example of how to use tiled with bevy_ecs_tilemap as such we'll not be expanding features to support things like this.


we are going unsafe with this one 🗣 🔥 🔥

meteor calls the event system offthread, so not a good idea


Added family tree rebus

Associated Issue: #121
Summary of Changes

Added a rebus for "family tree"

Pull Request Test Coverage Report for Build 631

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

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


Spring Boot integration and binary rendering

Hello,
I was wondering why the spring boot module doesn't include an option to enable binary rendering. Is there a technical reason for not enabling it in this context ?

This feels to me as a rare edge case, typically for projects with a tight coupling with Spring MVC (like the one referenced in this commit).
Would it make sense to reinstate this ability through an auto-configuration option, disabled by default ?

I think the reasoning was that, the performance difference between binary/string templates would be neglectable compared to the overhead of Spring Boot. The way it is, it is less complicated for users and compatible with all interception mechanisms that spring offers.
@tschuehly what do you think?

Yes, that is a good explanation.
What do you want to achieve with binary rendering? @ltanguy

Nothing very specific actually, probably more the beauty of squeezing as much performance as possible, but I agree that the difference won't probably be significative in most cases.
I'm not sure to see where the use case of binary rendering fits, as almost every web application will rely on a framework of some sort (Spring, Quarkus, ...) that will have similar requirements.

Many applications make network calls to databases or other services. Once you are hitting the network for the data for your models, the CPU cost of binary vs text rendering is basically irrelevant.


Context menus not selectable or clickable through a Drawing Tablet

Hyprland Version
Hyprland, built from branch HEAD at commit 4986d74ef201171a930c312a8e3b72a22d9b84ee dirty (xwayland: fix use of xwayland coords in native spaces).
Bug or Regression?
Bug
Description
On many (but not all, for example Dolphin works) applications, the context menu does not work correctly in Hyperland while using a drawing tablet, with or without OpenTabletDriver.
I've tried on Sway and Plasma Wayland and it works as intended.
There's another bug I think it's related to this same issue, whenever there's something that triggers a cursor icon change (for example hovering into a resizable area border), if you go to another area where the cursor should be back to the default one, it just doesn't change, you can update the change by moving the mouse instead of the drawing tablet, video below.
How to reproduce
Open a context menu with a drawing tablet (for example on Xournal++), the items are not being selected when hovering over them and if you click on them the context menu just closes without having done any action.
Crash reports, logs, images, videos
First clip: opening a menu first with a mouse which is how the application should behave, then with a drawing tablet not being able to select anything and if I click with it it just removes the menu, then I do the same thing but switch to mouse to show that it works, then mouse into tablet and it doesn't work, then tablet into mouse into tablet again showing it doesn't work.
https://github.com/hyprwm/Hyprland/assets/10902117/1701cd74-2ddf-4562-b2f0-ea19140c226a
Second clip: Works as intended on sway
https://github.com/hyprwm/Hyprland/assets/10902117/deaa917f-46aa-4ba2-9851-c967a1d14588
Third clip: Cursor issue on Dolphin (also happening in other applications like Xournal, which is noticable at 00:17 on the first video), the cursor turns back to the default one when moving the mouse
https://github.com/hyprwm/Hyprland/assets/10902117/ffc36828-34b6-4311-adc0-817774db1178

I was thinking on opening a ticket about this it happens the same on blender when you try to hold and drag certain stuff like for example on the shading part if you want to adjust a slider you can't drag it

idk, does this happen on -git still?

Yes, it also happens on freshly installed git version (I used AUR package for it).
hyprctl version outputs that its build from commit bc15a8f60028539690cccdb78eed7fc9175fe792 so it's pretty much last version.

patch.txt
try this?

patch.txt try this?

Thank you but it does not seems to solve the problem. I also tried to change 2 more "refocus()" calls to "simulateMouseMovement()" but it doesn't solve the issue either.

no clue then why this doesn't work and me not having a tablet doesn't help either :P

I can try to check it later but I have little experience with C++ and no experience with wayland managers so I don't think I will be able to fix it by myself. However I did some testing and there are some info which can be helpful (I think):

The issue also appears in relative mode (changed in the configuration).
In sway I am able to click in the widgets in waybar using tablet's pen and they react on it (I can change workspaces, click wifi icon in tray and so on) - However I can't do this in Hyprland. It makes me think that maybe problem is not with how the cursor moves, but how the clicks processed?

patch.txt
can someone give me the log from moving their tablet pen around with this patch applied?

patch.txt
this could be a fix? idk test it I don't have a tablet

I'll try both of those, give me a little bit
Compiling from gentoo 0.37.1
https://gitweb.gentoo.org/repo/gentoo.git/tree/gui-wm/hyprland/hyprland-0.37.1.ebuild

likely cuz patch will fail because one hunk won't apply. just patch -p1 < /path/to/patch || echo ""

patch didn't fix cursor issues in osu
for xournal, i can select the options from the dropdown menus but i have to hold my pen down throughout the whole thing using artist mode
log if it's of any use:
hyprland.log
p.s. should i move my issue back to

[LOG] NOTE: further logs to stdout / logfile are disabled by default. Use debug:disable_logs and debug:enable_stdout_logs to override this.

someone hasn't read the log itself.

sorry, got back up to provide a proper log
and one quick thing, managed to get xournal menus working properly by using mouse button binding instead of linux artist mode binding for left click
all still under artist mode
hyprland.log

are you sture this is with the first patch?

this was with the second patch. i can compile with the first quickly, give me a second

was having issues uploading the file, here's the log with 1st patch only:
hyprland.log

thanks, will look into tomorrow

Not sure how much helpful it will be, but I tried to apply all three patches one by one, and third fails. I compiled and test, but this don't fix the problem. Please see actually applied changes and logs:
changes.txt
hyprlandd.log
Tablet testing starts at line 350 I reattached one and first tried to draw in xournalpp (It worked as expected even before the patches) and then I tried to click some menus and it is not possible to choose anything from menu.
Hope this helps.

patch.txt
try this. Apply over clean -git

patch.txt try this. Apply over clean -git

Thank you, I tried it on the clean git (also pulled from master just to be sure that it is the last version), and I can say it partially fixes the problem please see the video attached.
https://github.com/hyprwm/Hyprland/assets/40325905/271a34da-0f02-4e92-886c-1cd4d166bd0f

patch.txt try this. Apply over clean -git

xournal seems to work perfectly fine for me. osu still has the same issue, sometimes cannot even detect inputs now
p.s. is my osu issue still relevant in this thread?
https://github.com/hyprwm/Hyprland/assets/117125180/b04d2281-b7e6-4729-b3a3-7bf5f033568b

@qwertyuiopr I see that you using xournal and I am wondering then why xournalpp is not functioning properly for me. Maybe I am missing some dependencies on the arch or something, idk. I tried to check it later. However it is strange that everything works good for me in sway on the same distro and same machine.

are you using the artist bindings on xournal? when i select the artist bindings in opentabletdriver, i have a similar issue where just tapping on the menus won't work, and i have to hold down my pen to access the menus. though using the mouse bindings work, they're only supposed to be there as a fallback option rather than what you use mainly.
https://github.com/hyprwm/Hyprland/assets/117125180/d0633ab6-a6ee-46cb-85e8-9475c8fc3359

are you using the artist bindings on xournal? when i select the artist bindings in opentabletdriver, i have a similar issue where just tapping on the menus won't work, and i have to hold down my pen to access the menus. though using the mouse bindings work, they're only supposed to be there as a fallback option rather than what you use mainly.
2024-03-18.21-03-10.mp4

Yeah, it can be this, but I just use standard wacom driver on arch, I didn't change any configuration on this etc, it just worked. Also, I am using xounrnalpp (fork of xournal I believe) and not just xournal. But I doubt that it can be a source of the issues here.

are you using the artist bindings on xournal? when i select the artist bindings in opentabletdriver, i have a similar issue where just tapping on the menus won't work, and i have to hold down my pen to access the menus. though using the mouse bindings work, they're only supposed to be there as a fallback option rather than what you use mainly.
2024-03-18.21-03-10.mp4

Yeah, it can be this, but I just use standard wacom driver on arch, I didn't change any configuration on this etc, it just worked. Also, I am using xounrnalpp (fork of xournal I believe) and not just xournal. But I doubt that it can be a source of the issues here.

uploaded video to discord because of 10mb file size limit on github
using xournalpp, i'm able to reproduce all of the exact same issues with artist bindings you are having. may have missed some from the video, but i don't doubt i can't recreate them either.
https://cdn.discordapp.com/attachments/1219382320377888900/1219394844024246332/2024-03-18_21-16-44.mp4?ex=660b24f2&is=65f8aff2&hm=70ce13959a9fe13247f6544f49999179b073a5996a841369f4cad8470434d821&

@vaxerski @qwertyuiopr I installed open tablet driver and it seems to fix the problem. I have no bindings in it, and I am able to normally draw and click on menus, so the problem is solved I think. It is strange for me that everything worked as intended with sway, but I am happy that it is fixed. Thank you :)

yea this can be closed I believe


Default configuration results in nans and infs from the first epoch.

Note:
Getting the provided code to run, required:

some small modifications to the environment (as specified in my comment here
And additionally it required me to specify the version base to hydra (to prevent a working directory change, resulting in the toy data not being found), such as @hydra.main(config_path="conf", config_name="config", version_base="1.2") in main.py.
Lastly some wandb paths needed to be respecified
If desired, I can provide a pull request for these changes.

Issue
Then when running the default configuration
 defaults:
  - data: eICU
  - model: tfm_sde

Resulted in bad training behaviour. This does not seem like the expected results at all.
Output:
 1 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/pytorch_lightning/loggers/wandb.py:396: UserWarning: There is a wandb run already in progress and newly created instances of `WandbLogger` will reuse this run. If this is not desired, call `wandb.finish()` before instantiating `WandbLogger`.
 2   rank_zero_warn(
 3 GPU available: False, used: False
 4 TPU available: False, using: 0 TPU cores
 5 IPU available: False, using: 0 IPUs
 6 HPU available: False, using: 0 HPUs
 7 `Trainer(limit_train_batches=1.0)` was configured so 100% of the batches per epoch will be used..
 8 [rank: 0] Global seed set to 42
 9 Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/1
10 ----------------------------------------------------------------------------------------------------
11 distributed_backend=gloo
12 All distributed processes registered. Starting with 1 processes
13 ----------------------------------------------------------------------------------------------------
14 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/pytorch_lightning/callbacks/model_checkpoint.py:604: UserWarning: Checkpoint directory ./outputs/checkpoints/Noise_MLP_Cond_memory_Module_SDE_Memory_3_eICU_DataModule_J30_v1/ exists and is not empty.
15   rank_zero_warn(f"Checkpoint directory {dirpath} exists and is not empty.")
16   | Name        | Type                             | Params
17 -----------------------------------------------------------------
18 0 | flow_model  | MLP_conditional_memory           | 140 K
19 1 | noise_model | MLP_conditional_memory_sde_noise | 139 K
20 -----------------------------------------------------------------
21 280 K     Trainable params
22 0         Non-trainable params
23 280 K     Total params
24 1.120     Total estimated model params size (MB)
25 [2024-12-17 11:30:01,859][torch.distributed.distributed_c10d][INFO] - Added key: store_based_barrier_key:1 to store for rank: 0
26 [2024-12-17 11:30:01,860][torch.distributed.distributed_c10d][INFO] - Rank 0: Completed store-based barrier for key:store_based_barrier_key:1 with 1 nodes.
27 configuring optimizers
28 Testing DataLoader 0: 100%|██████████████████████████████████████████████████████████| 10/10 [00:54<00:00,  5.46s/it]
29 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/pytorch_lightning/trainer/connectors/data_connector.py:224: PossibleUserWarning: The dataloader, val_dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the `num_workers` argument` (try 12 which is the number of cpus on this machine) in the `DataLoader` init to improve performance.
30   rank_zero_warn(
31 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:96: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
32   a = torch.tensor(pred).float().clone().detach()
33 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:100: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
34   b = torch.tensor(true).float().clone().detach()
35 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/ot/lp/__init__.py:571: UserWarning: Problem unbounded
36   check_result(result_code)
37 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:96: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
38   a = torch.tensor(pred).float().clone().detach()
39 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:100: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
40   b = torch.tensor(true).float().clone().detach()
41 Epoch 0:   0%|                                                                               | 0/231 [00:00<?, ?it/s]
42 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/pytorch_lightning/trainer/connectors/data_connector.py:224: PossibleUserWarning: The dataloader, train_dataloader, does not have many workers which may be a bottleneck. Consider increasing the value of the `num_workers` argument` (try 12 which is the number of cpus on this machine) in the `DataLoader` init to improve performance.
43   rank_zero_warn(
44 Validation: 0it [00:00, ?it/s]
45 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:96: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
46   a = torch.tensor(pred).float().clone().detach()
47 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:100: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
48   b = torch.tensor(true).float().clone().detach()
49 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/ot/lp/__init__.py:571: UserWarning: Problem unbounded
50   check_result(result_code)
51 Monitored metric val_loss = nan is not finite. Previous best value was inf. Signaling Trainer to stop.
52 Epoch 49, global step 23100: 'val_loss' reached inf (best inf), saving model to './outputs/checkpoints/Noise_MLP_Cond_memory_Module_SDE_Memory_3_eICU_DataModule_J30_v1/best_model-v1.ckpt' as top 1
53 [rank: 0] Global seed set to 42
54 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/pytorch_lightning/trainer/connectors/data_connector.py:224: PossibleUserWarning: The dataloader, test_dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the `num_workers` argument` (try 12 which is the number of cpus on this machine) in the `DataLoader` init to improve performance.
55   rank_zero_warn(
56 Testing: 0it [00:00, ?it/s]
57 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:96: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
58   a = torch.tensor(pred).float().clone().detach()
59 /Users/me/Code/fusion/TrajectoryFlowMatching/src/utils/metric_calc.py:100: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
60   b = torch.tensor(true).float().clone().detach()
61 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/ot/lp/__init__.py:571: UserWarning: Problem unbounded
62   check_result(result_code)
63 ┏━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
64 ┃        Test metric        ┃       DataLoader 0        ┃
65 ┡━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
66 │    1-Wasserstein_test     │            0.0            │
67 │    2-Wasserstein_test     │            0.0            │
68 │      Linear_MMD_test      │            nan            │
69 │       Mean_L1_test        │            nan            │
70 │       Mean_L2_test        │            nan            │
71 │       Mean_MSE_test       │            nan            │
72 │      Median_L1_test       │            nan            │
73 │      Median_L2_test       │            nan            │
74 │      Median_MSE_test      │            nan            │
75 │       Poly_MMD_test       │            nan            │
76 │       RBF_MMD_test        │            nan            │
77 │       l1_loss_test        │            nan            │
78 │       mse_loss_test       │            nan            │
79 │      noise_test_loss      │            nan            │
80 │         test_loss         │            nan            │
81 └───────────────────────────┴───────────────────────────┘

Am I missing something that is relatively easy to fix here?
I especially notice the line: 35 /usr/local/Caskroom/miniconda/base/envs/tfm/lib/python3.10/site-packages/ot/lp/__init__.py:571: UserWarning: Problem unbounded
But it might be something else.
Could you please give me some advice on how to debug and fix this? I am hoping to replicate your results and adapt them for my research.

Hi Trezorro, the dataset we have in the repo is a mock dataset since the real dataset needs users to undergo ethics training.
We have uploaded our processed dataset at PhysioNet under the name: Clinical Time Series Datasets for Trajectory Flow.


Change user approval expiry check

User expiry can be set with the env variable APPROVAL_EXPIRY = "30".
The value in days indicates when the account approval will expire.
This can not be done automatically but will only be checked once a user tries to logon.
Previously the check was against the time of approval. Hence it was not possible to extend this but only remove the expiry by removing the approval time stamp. This solution was done in order to prevent adding an additional field to ACL schema.
The approach was quite limited. With this PR a field expires_on:integer must be added to the ACL schema if the APPROVAL_EXPIRE environment variable is set.
A day integer will be set when a user registers. Resetting the password will reset the day integer.
A the expiry date will be checked against the current date on login. The approval will be removed if the account has expired. The login process will abort.
The admin view will show the date in red if today or in the past. An admin can clear the date or set a new expiry date.

The expiry_on date will not be set on password reset if already cleared.

There is an issue with the date input toggle. It does not seem possible to focus the editor input unless you click into a different row first. I opened a ticket with Tabulator in this regard. https://github.com/olifolkerd/tabulator/issues/4210

 User Table Admin - unable to set expiry


Mojs curve editor issue : hidden curve beyond boundaries

Hi,
I encounter an issue when editing the curve when it goes beyond the boundaries (below 0, and above 1 axis) as showed in the attached screen : it's hidden.
I imported it using the cdn method. Thanks if you have a workaround to deal with this.

Hi @lansolo99!
Thanks for the issue! This looks weird and should never happen so this is certainly a bug. Can you provide more info regarding OS/browser you use or any reproduction step to catch it locally? Is the bug consistent for you?
Thanks a lot!

Hi @lansolo99,
Can you provide browser/OS in order for us to investigate?
Otherwise this issue will be considered outdated and closed.
Thanks a lot!

Hi @legomushroom It's been ages I opened this issue 😀
I thought mos.js was not maintained anymore, it was probably using Chrome but you can consider closing this issue as I don't plan to follow up on this problem.

Hello @lansolo99,
Merci / Thanks for your feedback!
I will close this and reopen if someone face it again.

As you guessed, mojs was not maintained during a small period back in 2016/2017 because Oleg Solomka has moved to Microsoft, then Github core team, that's why. Since 2017 the project is still maintained, and the v1 should be released very soon :wink:


Embeded base64 image type detect problem

If html has embeded base64 image with type jpg won't show up after paste in wysiwyg mode
But if I change image type to jpeg then it works well

@rickywu I can't reproduce, would you upload the image file with the problem? I think you can upload both the jpg and jpeg used when pasting.

Copy this text file and paste in markdown mode, try to change jpg and jpeg to see right panel.
test.txt

@rickywu The mime type is wrong. Please refer to the links below.

https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/Data_URIs
https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types

@seonim-ryu But why examples support image/jpg MIME type?

@rickywu
I checked this problem. The behavior of version 1 and version 2 is different. I think this is a bug in version 2. And from next time, I want you to write down the issue according to the template. It is very difficult to identify the problem without the editor version or the browser, operating system or other information used.
Thanks for reporting the issue. :)


driver: esp32: I2C code refactoring

Use i2c_hal functions to enable support for
multiple SoCs.
Use DT compat to enable I2C with from device
tree configuration.
Added support for ESP32S2 and ESP32C3.
Signed-off-by: Sylvio Alves<EMAIL_ADDRESS>

@cfriedt @nashif, I can't figure out the reason for buildkite to be failing as it points to aruino_nano board failure. Is kconfig.esp32 change triggering that issue in the arduino project?

@cfriedt @nashif, I can't figure out the reason for buildkite to be failing as it points to aruino_nano board failure. Is kconfig.esp32 change triggering that issue in the arduino project?

I added a update intoarduino_nano_33_ble board to remove invalid I2C_X config.

Just rebased.

Rebased again.

@cfriedt @nashif PTAL as well. Buidkite has failed but error is not related to this PR. Can you trigger the build again?

@cfriedt @nashif Now it worked.


Extend the generator coroutine by the functionality to export additional information.

Each generator can now have a Details object, which can be read and written during generation. If no such details are needed, there is zero overhead compared to the previous implementation.


upgrade to capnp 0.20

They have recently updated Rust Cap'n proto to 0.20.x so this PR just updates the dependencies.
Also, thanks for the great work.

Merged, thank you!


Question about the implement  of the KLDiv Loss

As I can see in the file help_fun.py, you use SoftmaxFunction to normalize the teacher output but use logSoftmaxFunction to normalize the student output.
Why do you set this two different?

Mates have the same question please refer to the pytorch implement of the KLDivLoss


"Personalized Waiting" card breaks ordering

When using the personalized waiting card, when the customers change their order they don't choose a new item. They sit with the waiting for food (plate with fork and knife) timer ticking down but never order an item.

Tracking internally, patched in next update


fix pekko-sample-distributed-workers-scala

after https://github.com/apache/incubator-pekko-persistence-cassandra/issues/30 is done
imports non-existent pekko-persistence-cassandra jar
after it is fixed, this sample should be built as part of the GithubCI build to ensure it isn't broken again

PR #20 merged


Image sequence as input/output

Description of Problem & Solution
PySceneDetect can't import/export image sequences, first have to convert image sequences into video and then convert the resulting split videos into image sequences.
It's a convoluted process that wastes time, quality (when using lossy compression) or disk space (when using lossless compression).
Implementing that feature would solve all these 3 problems, actually 4 because the latest PySceneDetect still suffers from the extra frames problem ending up at the start/end of split videos, with an image sequence is much easier to delete them automatically/manually if you know the pattern.
Media Examples:
Splitting the example goldeneye.mp4 into scenes using PySceneDetect and ffmpeg version git-2020-01-21-fc6fde2 on Windows 10, I end up getting one extra frame, it would be much easier to fix that in an image sequence.
Proposed Implementation:
Running a command like:
scenedetect -img IMAGEFOLDER -o OUTPUT detect-content image-sequence
Would search the IMAGEFOLDER for an image sequence based on how the files were named (ie. frame0001.jpg to frame9000.jpg is a single video made of 9000 frames), then output the scenes in separate directories (ie. ../frame-Scene01/frame0001.jpg to ../frame-Scene01/frame0100.jpg is scene 1, ../frame-Scene02/frame0101.jpg to ../frame-Scene02/frame0200.jpg is scene 2, etc.)
If we end up getting an extra frame and we know the pattern (ie. in my case, the extra frame is at the end of the first scene, then the beginning of every other scene), we can systematically delete the interested frames (ie. frame 100.jpg, frame 101.jpg, frame201.jpg, and so on) automatically.
Alternative Solutions:
Or we could simply have a command where we tell PySceneDetect to delete X number of frames at the start and ending of each scenes to avoid the extra frame problem, imagine something like: scenedetect -img IMAGEFOLDER -o OUTPUT detect-content -r 1 image-sequence (would remove 1 frame at the beginning and end of every scene).
Even if I ended up erroneously deleting 1-2 frames that weren't an extra frame for every scene, that would still be okay, the tradeoff from having to manually fix the frames and having it done automatically even with that small error margin is totally worth it in my case scenario.

Hey @AndroYD84;
OpenCV supports image sequences out of the box.  You should be able to specify a sequence path as the input video, and PySceneDetect should handle it normally.  The only thing it cannot handle is using an image-sequence combined with the split-video command (as there is no video to split).
Is there a reason to actually specify an output folder for the case of an image sequence, though?  Once you have the frame numbers where the cuts are, you should be able to just use those images directly (or can write a script to put them into subdirectories).
Thoughts?

Is there a reason to actually specify an output folder for the case of an image sequence, though?

Yes, there's a very good reason, when a single folder contains a ginormous amount of pictures (ie. more than 100.000), which is likely to happen when splitting a full movie, any attempt to open it from a file manager (or anything that displays thumbnails) will instantly cripple the entire machine as it tries to generate thumbnails/text for every single of them, when they are split in folders this doesn't happen as the amount of files per folder is reduced significantly (ie. instead of 100.000 frames in 1 folder, you may get 1000 frames in 100 folders), video files already act like "folders" containing multiple frames inside, so they don't suffer from such problem, while image sequences will if they aren't organized properly.
Normally I'd want to check if pyscenedetect is detecting the scenes correctly and look at the output folder from a GUI interface, while the input folder content can be handled entirely from a command-line interface as the content result is usually well known already.

Gotcha, the easiest way I can think of to support this would be to add an argument to split-video that indicates if you are using an image sequence or not, for example:
scenedetect -i my_image_sequence%02d.jpg detect-content split-video --sequence

By default it would just copy each frame from the source directory, but could add an additional option to move them from their original location, e.g.:
scenedetect -i my_image_sequence%02d.jpg detect-content split-video --sequence --move

Thoughts on the proposed CLI changes?
There are some edge cases to keep in mind when implementing this (e.g. having frame numbers not starting from zero, gaps in the sequence, handling the case where creating a new directory fails, or moving an image fails) that I'll need to take into account.  Making good test cases and handling all the potential error cases will be crucial for implementing this feature, which means it'll be a bit more work than I'd like, but I understand your use case and agree it has value.  Will add this to the backlog of proposed features.
In the meantime, you can consider using the save-images command to output the first/middle/last frames that PySceneDetect is detecting for ensuring correct detection results (the command allows specifying an output directory).

Hi, thank you for awesome product. I have question about how to open images sequence with python API? I detect timecodes on video with pyscenedetect, after with openCV I parsed video on images (1 in 400ms for FPS=25), if FPS is different from 25, timecode from scenedetect and my parsing images differ. I declined from detect scene from video and want to detect on image sequence, I used example script from documentation, but VideoManager(cv2.Capture('path/to/seq')) except with ValueError: Unexpected element type in video_files list. How to hanlde this? Thank you for answer

Hi @milovan68;
I actually just found a bug in the VideoManager preventing image sequences from being loaded.  To answer your question, you should use VideoCapture(['path/to/seq%04d.jpg']) for example, however, currently this will result in an error ("Video file(s) not found.").
Re-marking this is a bug, and going to suggest that PySceneDetect should not be responsible for actually doing output of image sqeuences (the output provided by list-scenes is sufficient for doing that manually).

@AndroYD84, you should not be seeing any extra frames in the latest release (see #93 and #159).  I have re-marked this is as a bug, however, since image sequence input is currently not working.  I'm going to prefer to not have PySceneDetect handle manipulating or copying existing files, as one could write a script to do the same thing using the output of list-images.  Thanks!

Hi @AndroYD84;
Image sequences will be supported for input in the next release (v0.5.5).  There are still some non-trivial issues that exist with the save-images and split-video commands, so I'd prefer you create a new issue if you still require supporting copying the respective files to new folders.  I'll remove 'output' from this issue's title just to be clear.
In the meantime, you could write a Python script that uses the output from list-scenes to achieve the same thing (if you do, please share as it could be re-used for the feature).
Thanks!

Hi @AndroYD84;
Image sequences will be supported for input in the next release (v0.5.5).  There are still some non-trivial issues that exist with the save-images and split-video commands, so I'd prefer you create a new issue if you still require supporting copying the respective files to new folders.  I'll remove 'output' from this issue's title just to be clear.
In the meantime, you could write a Python script that uses the output from list-scenes to achieve the same thing (if you do, please share as it could be re-used for the feature).
Thanks!


#692/discriminate fixes

#692

@krzkaczor
Config

TypechainUserConfig is missing options

export default flag
interface does not need contractName and shouldn't have it.

ethers.interface is a representation of ABI standard and abstracts them

contract and contractFactory are sufficient for discriminate union

contracts contain ethers.interface as a prop


Create missing unit tests

Type of Change

[x] Bug fix (non-breaking change, fixes #3350 )
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Create a unit test for all cmdlets in the functions folder that does not currently have one
Each unit test checks for current parameters of cmdlets
Note: I know this adds no code coverage, but looking to add this after this PR. If this isn't the approach to these test let me know. I don't mind breaking it down into further PRs. 😄
Approach

Used a script to generate each test file.
Manually touched a few tests that did not pass the first time.
Commands to test

Execute Invoke-Pester .\Tests -Tag 'UnitTests'

Dude! Awesome, thank you! This was a big ol agenda item and I appreciate you addressing it 💯


Support for V3 Signatures

    Hi,

I'm trying to decrypt a file that was encrypted by SecureZIP and I'm getting the following error:
org.pgpainless.exception.SignatureValidationException: Signature is not valid: Malformed signature. Signature has no signature creation time subpacket in its hashed area.

I'm aware that RFC4880 requires the presence of the signature creation time as a hashed subpacket. It seems a little odd to me that a commercial product like SecureZIP would make such a mistake.
gpg doesn't really seem to care that there is no such subpacket, it decrypts the file just fine and states that the signature is correct (event with the --rfc4880 flag!).
I'm trying to determine what my options are. I think I could check if the OpenPgpMetadata contains an (invalid) signature made with the expected key, and simply ignore this specific kind of error.
On the other hand, getting SecureZIP to fix their bug (if it's really a bug on their side) seems like a really long shot.
What do you PGP experts think?
Originally posted by @bjansen in https://github.com/pgpainless/pgpainless/discussions/357

Comment by @bjansen:

Wait I think I got it. pgpainless throws an error on a signature packet in version 3, but apparently hashed creation times are only required in version 4.

I just released PGPainless 1.4.4 which contains the fix you proposed with #358

Thanks a lot!


Add count_distinct to valid denominator enum in schema. Other validation fixes.

Fixes: EPD-2217

lgtm
do you know why requirements.txt has so many additions?

I do not. Maybe something in my local environment was off. If I have time today I'll see if I can cut it down.


SemanticScene-compatible HM3D semantics annotations missing

Habitat-Sim version
v0.3.1

Are you using the bleeding edge version?

Yes.
❓ Questions and Help
I noticed that in the latest release v0.3.1, you have incorporated the new semantic annotations from PR #2307, which addressed issues mentioned in #1836 #2253 #2323.
However, with some efforts digging into the code, I found that the semantic annotations seemed to be loaded by a set of annotations as specified in config file

https://github.com/facebookresearch/habitat-sim/blob/5f2bc6a0bde1da77b334022ca28e8903952f4a24/data/test_assets/dataset_tests/dataset_0/test_dataset_0.scene_dataset_config.json

https://github.com/facebookresearch/habitat-sim/blob/5f2bc6a0bde1da77b334022ca28e8903952f4a24/data/test_assets/dataset_tests/dataset_0/semantics/dataset_test_scene.semantic_config.json

My questions is whether and when you are going to release these annotations for HM3D dataset including stages, regions, etc. If you are short-handed or this task is under-privileged, please give me some ideas about how to do this. I can create a PR for this later.
Context: I am working in my fork on a project about multi-agent system using environment 3D scene graph. Having GT semantic scene would help me a lot. FYR, https://github.com/SgtVincent/habitat-lab/tree/embodied_mas/habitat-mas/scene_graph

Thanks for your quick response! 🚀

You could, for example, annotate the existing scenes in blender and add the semantic regions configs to the dataset.

Going to look into this. I could also try some auto-processing scripts as you mentioned in other threads.

We may release some intermediate code and process documentation to make that process more transparent, but it hasn't been a priority recently.

I understand you have quite much stuff on your list. Look forward to your update.  :)

Hi @aclegg3 ,

we support 2D poly loops with floor base height and extruded ceiling height as an abstraction

How is this accessed?
I am geeting an empty list and 0.0 extrusion height when printing region.poly_loop_points and region.extrusion_height.


Linting with rubocop

Fixes issue with linting..

Looks like you copied .rubocop.yml of bundler :sweat_smile: I had only given the link as an example configuration. Can you please remove all the cop rules which are disabled (enabled: false)?

ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-darwin15]

I've made the changes as requested, bro.

Thanks @nkprince007 :heart:


HVPA feature promotion

How to categorize this issue?

/area auto-scaling usability
/kind enhancement
/priority normal
What would you like to be added:
Let's promote the HVPA feature gates in gardenlet to beta and enable them by default. After a couple of release we should then promote them to GA and eventually remove the feature gate.
Why is this needed:
Gardener is pretty much relying on HVPA for auto-scaling the etcds and kube-apiservers of shoot clusters. The past releases have shown that it's working quite well and stable enough to plan for its promotion, so that we can eventually get rid of the feature gate again.
/assign @ggaurav10 @amshuman-kr

/close due to the upcoming HVPA redesign


react-scripts not found

Hello @Mirage3098
Did you install the dependencies with npm install command?

I tried, but it came up with npm WARN and said something about the package.json being made with old version of npm or something.

I tried, but it came up with npm WARN and said something about the package.json being made with old version of npm or something.

Try using node v14 (use nvm for version manager)


Adding open_groups to BackendEntryPointEngine, NetCDF4BackendEntrypoint, and H5netcdfBackendEntrypoint

[x] Closes #9137 and in support of #8572
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Rough implementation! Still need to add tests and docs.
@TomNicholas, @shoyer, @owenlittlejohns, and @flamingbear

I know we touched on this a bit during our meeting on Tuesday but I think tests for open_groups in netCDF_.py and h5netcdf.py should probably go in test_backends_datatree.py. We mentioned we could create separate test files for it but it looks like test_backends_datatree.py, has an empty class for each of these backends and it looks like they would fit nicely in here.
test_backends_datatree.py
@requires_netCDF4
class TestNetCDF4DatatreeIO(DatatreeIOBase):
    engine: T_DataTreeNetcdfEngine | None = "netcdf4"

@requires_h5netcdf
class TestH5NetCDFDatatreeIO(DatatreeIOBase):
    engine: T_DataTreeNetcdfEngine | None = "h5netcdf"

Let me know if I have that right and what you all think.
@TomNicholas, @shoyer, @keewis, @flamingbear, @owenlittlejohns

Sounds good @eni-awowale
You may even be able to use basically the same groups test for multiple backends by adding it to DataTreeIOBase...

If it's okay with everyone I think we should also include a small netCDF file in test/data that has groups.

Having an included file to test against is a good idea, we just want it to be small.

example_1.nc and bears.nc don't have groups.

Where are these currently? We don't want to add files to the main repo - preferring to put them here https://github.com/pydata/xarray-data

You may even be able to use basically the same groups test for multiple backends by adding it to DataTreeIOBase...

So, instead of adding the same type of tests to TestNetCDF4DatatreeIO and TestH5NetCDFDatatreeIO I can add them directly to DataTreeIOBase, if the are not engine specific? Okay, that makes sense!
@TomNicholas example_1.nc and bears.nc are here test/data.

We don't want to add files to the main repo - preferring to put them here

Okay, great there might already be some some sample files in there with groups that I can use!

@TomNicholas example_1.nc and bears.nc are here test/data.

I didn't even know those existed... @max-sixty I know you have thought about example datasets for testing purposes - do you have an opinion on whether new files for testing should go in that directory or the separate repository?

@TomNicholas example_1.nc and bears.nc are here test/data.

I didn't even know those existed... @max-sixty I know you have thought about example datasets for testing purposes - do you have an opinion on whether new files for testing should go in that directory or the separate repository?

We want tests to be runable without a network connection. If we need new files for testing, which I agree would be a good idea in this case, please add them here. Just keep them as small as possible (the current test files are all under 10 KB in size, most under 1 KB).

Where are these currently? We don't want to add files to the main repo - preferring to put them here https://github.com/pydata/xarray-data

xarray-data is for sample/tutorial datasets, which can be much larger (enough data to make an interesting plot) than what we use for test data.

Hi folks! I think this PR is a couple commits away from merging but they are a couple things to sort out.
mypy checks failing
The open_groups function returns a dict[str, Dataset] type. However, the return type for DataTree.from_dict() is a MutableMapping[str, Dataset | DataArray | DataTree | None] and we call DataTree.from_dict() in our open_datatree() backend functions, so mypy flags this as having an incompatible type. See below.
"from_dict" of "DataTree" has incompatible type "dict[str, Dataset]"; expected "MutableMapping[str, Dataset | DataArray | DataTree[Any] | None]"  [arg-type]
I am not sure exactly how to proceed with this since updating the return type of DataTree.from_dict() causes other mypy checks to fail because the are expecting the MutuableMapping return type.
Some checks are failing but seem unrelated to the PR changes?
I noticed there are tests failing, but these seem to be consistent with the checks failing in main and are related to test_computations.py. The other test I see failing are the ubuntu 3.12 flaky tests. Are these failures we can ignore?
Updating _iter_nc_groups
I added the suggestion from @keewis  so _iter_nc_groups is returning the parent node and now we don't have to reopen the store twice 🎉
cc
@TomNicholas, @keewis, @shoyer, @flamingbear

Are these failures we can ignore?

flaky is something that may fail at random, mostly because of server or network issues. Usually it will pass again on the next run (I reran it for you, let's see what happens). The other test failures are indeed also on main, so we can ignore it here.
Edit: flaky passes now, so I believe you can ignore the remaining test failures

I am not sure exactly how to proceed with this since updating the return type of DataTree.from_dict() causes other mypy checks to fail because the are expecting the MutableMapping return type.

Maybe the typing crowd can help? cc @max-sixty, @Illviljan, @headtr1ck

mypy checks failing
The open_groups function returns a dict[str, Dataset] type. However, the return type for DataTree.from_dict() is a MutableMapping[str, Dataset | DataArray | DataTree | None] and we call DataTree.from_dict() in our open_datatree() backend functions, so mypy flags this as having an incompatible type. See below. "from_dict" of "DataTree" has incompatible type "dict[str, Dataset]"; expected "MutableMapping[str, Dataset | DataArray | DataTree[Any] | None]" [arg-type] I am not sure exactly how to proceed with this since updating the return type of DataTree.from_dict() causes other mypy checks to fail because the are expecting the MutuableMapping return type.

It's this type of problem: https://stackoverflow.com/questions/73603289/why-doesnt-parameter-type-dictstr-unionstr-int-accept-value-of-type-di
from_dict should probably switch from MutableMapping to Mapping instead.

@TomNicholas  and @keewis this issue we just moved over #9336 seems to be related to the latest batch of test failures.

Yay all checks are passing! Does someone want to give this a quick look before merging?
cc
@TomNicholas, @keewis, @owenlittlejohns, @shoyer and @flamingbear


signature of List() seems to have been changed and that breaks the controller code

The method signature of List is
List(ctx context.Context, list runtime.Object, opts ...ListOption) error

but in below line we are not passing correct type of parameters while calling the List method
https://github.com/xcoulon/podset-operator/blob/7b3e90b9eb7d683a4afc1fe1e20261827eb737f5/pkg/controller/podset/podset_controller.go#L110
does this code need a change or I am understanding something wrong.?

It seems there are changes in the newer version (that I was using) of the
controller-runtime/pkg/client version 2.0 but the code works fine on 0.1.4. Closing the issue.


Build error

Spec:
compileSdkVersion 28
minSdkVersion 19
targetSdkVersion 28
Manifest merger failed : Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91
is also present at [androidx.core:core:1.0.1] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory).
Suggestion: add 'tools:replace="android:appComponentFactory"' to  element at AndroidManifest.xml:5:5-20:19 to override.
After add 'tools:replace="android:appComponentFactory"' to manifest
Manifest merger failed with multiple errors, see logs

Thanks for posting the issue.
From a basic search, it seems to be because I use androidx rather than the support libraries. See here for more info: https://stackoverflow.com/questions/51742697/android-manifest-merger-fails-after-adding-dependency-com-google-android-materia
Are you able to upgrade to AndroidX? If not, what do you suggest I do to fix this problem in the library?

I just remembered that a similar library ButterKnife did something like this. Try adding android.useAndroidX=true in gradle.properties.
If that fixes your problem, then maybe I can include a note in the README.

thanks alot for you response.
Now, i still using kotlin without andoidx. i have not planning for upgrade to androidx.
And also have not suggestion about this issue.

No problem. Did you try adding android.useAndroidX=true to your gradle.properties? Give that a try and report back to me first
If that doesn't work, let me experiment with a solution this weekend and get back to you.
Also, if you like this library (and assuming we can get it working for you), don't forget to star the repository! The stars will help others find & use this library.

I've came up with a solution that will work for now.
I released a version of SegmentedButton that uses the Android support library rather than AndroidX. The version is 3.1.3-support. Change this in your Gradle file and it should work. I tested this on my machine using Android support library and Kotlin as well.
The README contains more information on this setup now as well. However, this approach is only temporary to give developers time to switch to AndroidX. With that in mind, I would recommend switching to AndroidX as soon as possible.
Closing this issue. Make a comment here if you're having issues and I'll re-evaluate.

thank very much for your solution and response.
Sorry, now i can't to contribution about this project. Hopefully in the future I can contribute
keep you the spirit


Support specifying docker image platforms to build for

The docker/build-push-action has a platforms arg (List of target platforms for build):
https://github.com/docker/build-push-action/blob/5cd11c3a4ced054e52742c5fd54dca954e0edd85/action.yml#L66-L68
If not specified, the default behavior is to build for the platform that the runner has.
But sometimes you might want to build for a different platform:
Some CapRover servers are e.g. Arm, but GitHub only supports Arm-based runners on the GitHub Team or GitHub Enterprise Cloud plan.
Could you please add a platforms arg to this action, and pass that on to the docker/build-push-action invocation here? :)
https://github.com/adamghill/build-docker-and-deploy-to-caprover/blob/4a1b7fd7ccd1ff1981632ff6ef3aea3405c3dad8/action.yml#L75-L84

Sure! That seems pretty straight-forward. If I make a PR, can you test it? I don't think I have an easy way to verify it's working as expected.

@adamghill Yes, I'll test it :)


Extend MCorr to use composite quadrature rule

In this pull request, the original term in the mass correction step

is replaced by
 so that the composite quadrature rule can play a role here.
Since this is an explicit term now, only the function getResidual is changed in the implementation.
Here are some details.

In order to compare to the original method, a new label mass_correction_reference is introduced.
Three new parameters are passed into the c code:   from VOF module;   created in postStep of the previous step; the velocity field  from VOF module.
Those new parameters in c code are called dt, q_H_old, q_v.
The first term of  is added into r in evaluateCoefficients_explicit_comp_quad; the 2nd term is introduced as f computed in the same function.

Some extension can be done for the next step.

Add boundary integral terms. Note that for Vortex2D problem, the boundary integral is zero.
Treat the 2nd term of    implicitly as . One has to compute  and update the calculateJacobian to include the term
More tests will be done since small difference is observed now.

Thanks for your comments. I'll make those changes now.

In this commit, the functions evaluateCoefficients_explicit_comp_quad, calculateResidual_explicit_comp_quad, and calculateElementResidual_explicit_comp_quad are deleted. Some indentions are corrected. Maybe it's still not good enough. I am trying to run some tests in order to get good convergence.

@cekees Thanks for your correction of some bugs. I noticed that in evaluateCoefficients  is used instead of . Is this right?  Using this commit, I can run the vortex2D problem to the tolerance 10e-12.

It turns out that the nonlinear solver failures on the first step were due to a bug with the new flux term. Specifically, the initial values weren't correct so the first corrected step had very large error.  It would be good to run this version with the new terms on and off for a range of hk's and meshes. You can compare those to the uncorrected level set and vof algorithms by setting applyCorrection=False and applyRedistancing=False (so that's 3 configurations by say 3 hk's by 3 meshes--see refStudy.sh script for example of how to script the runs). I think the next step after that is to extend to an implicit flux as @manuel-quezada did.  I suspect the next step after that should be to enable a bdf or crank-nicholson time discretization in the mass correction. You can also enable the composite quadrature for P2 elements by adding the required line to ModelFactory.h

Thanks. I will do such comparisons.

@wacyyang keep me on the loop on this so I can help whenever you have any problem.

@cekees @manuel-quezada In these commits,
I am trying to implement (mass_correction_reference=2). It means the term  is used in computing residual and Jacobian.  Can you help me to double check it? Thanks.

In this commit, I include theta-method (mass_correction_reference=4) for the time discretization  of mass correction problem. It means the term  is used to compute residual and Jacobian. I have compared (mass_correction_reference=4, theta=1) and the implicit metho**(mass_correction_reference=2)**. They produce same solutions. It seems that Crank-Nicolson method (mass_correction_reference=4, theta=0.5) gives a better result

That looks good. What was the target CFL?

It is runCFL=0.3. Compared to the original method (mass_correction_reference=0) which keeps the location or mass center but has many pieces, the Crank-Nicolson method (mass_correction_reference=4, theta=0.5) has one piece as whole but a little displacement of mass center. I am thinking about maybe combining them together like  will produce a good result.

@cekees I meet an error when trying to use P2 finite element pDegree_ls=2. After digging into the source code, I find it is related to https://github.com/erdc/proteus/blob/80b0db3a6ff0d05f437d43f1f8dcdc126315ea6f/proteus/mprans/MCorr.h#L2518 as indicated by its comment. Maybe we can use element diameter instead. What is your suggestion?

Yes, just use elementDiametersArray and switch the index to eN. In fact we don't even need this bit of code for the new method because we don't project H into the finite element space. Except in the last variant you added with the gamma, but I think we're going to loose accuracy whenever we project H. I'm guessing the center of mass problem is related to the space and time truncation error in NCLS and MCorr.

I will add elementDiameters into setMassQuadrature just as an extension and finish all tests first. I agree with you that adding gamma is not a good idea to solve the problem about the location of the mass center. Numerically, I can see the pollution of  on the computation and that is why the last picture of the result on gamma=0.95 still doesn't have a good improvement. For smaller gamma, it looks even worse.

Using P2-FE (pDegree_ls=2) and Crank-Nicolson method (mass_correction_reference=4, theta=0.5) I got this result. It shows big improvements including the location of the mass center :beers: More tests on using composite quadrature rules will be done.

I just pushed a commit where I am trying to implement backward Euler method in MCorr module. Unfortunately, it is still not convergent. I think I have some misunderstanding about this part (time integration, step controller ...)  Do you have any suggestions? Thanks for your help.

As you suggested, I have run the rotation2D problem using different methods. The results are shown here

White-BDF3, Blue-BDF2, Green-BDF1
Since the final time is T=1, the difference between BDF1, BDF2, BDF3 is small.
As for the comparison of 3 equivalent CN+BDF1 (original method mass_correction_reference=4, using BackwardEuler mass_correction_reference=5, simplified  version mass_correction_reference=6), all have exact same information on the final data as shown in paraview

However, for vortex2D problem, the final solutions are different although the difference on the computation of _r, dr_ at every quadrature point is <10e-10.


Basic Energy handling

Adds the {E} symbol used to represent energy counters in AllSets.json. This allows about 60 more cards to successfully parse on a run of encode.py.
~I don't know what's wrong with the remaining 19 cards that come up invalid, and~ I'm sure there's more to do to complete the handling of Energy counters, but this is a quick partial fix at least!
EDIT: Aha, it's Vehicles it doesn't like, vis the remaining 19 cards. New issue coming up.

Gonna change this to a different branch than master. Be back up momentarily.


Display 'nil' in lualine

I use Packer
After downloading wpm.nvim, it shows nil when used in lualine.
config:
  sections = {
	lualine_a = {
		"mode",
	},
	lualine_b = { "branch", "diff", "diagnostics" },
	-- show the current filename and session name,
    lualine_c = { "filename", require('auto-session-library').current_session_name },
	lualine_x = { "encoding", "fileformat", "filetype", require("wpm").wpm, require("wpm").historic_graph },
	lualine_y = { "progress" },
	lualine_z = { "location" },
     },

Like this:

What could be the reason?

That's strange. What does :lua vim.prettyprint(require("wpm").wpm show?

im.prettyprint(require("wpm").wpm

NO. I got this error

That's strange. What does :lua vim.prettyprint(require("wpm").wpm) show?

I migrated  from Packer.nvim to Lazy.nvim today, and the problem have solved...!
So I guess there might be something wrong with Packer.nvim
Anyway, thank you for your replay!

Hmmm, I wonder if I have the packer.nvim example wrong. Glad it worked out for you.

Closing in favor of #5.


Add Macports build with training tools for daily ci test

@stweil @Shreeshrii
Please pay attention to these new nightly builds and tests on those configs.
https://github.com/tesseract-ocr/tesseract/actions?query=workflow%3Adailyunittest
https://github.com/tesseract-ocr/tesseract/actions?query=workflow%3Adailycmake
@stweil
Also please check if workflows are correct and we can consider them like as you run tests, sanitizers on your machine.

@stweil @Shreeshrii
Please pay attention to these new nightly builds and tests on those configs.
https://github.com/tesseract-ocr/tesseract/actions?query=workflow%3Adailyunittest
https://github.com/tesseract-ocr/tesseract/actions?query=workflow%3Adailycmake
@stweil
Also please check if workflows are correct and we can consider them like as you run tests, sanitizers on your machine.

@egorpugin Thanks for merging these.  Yes, these need corrections and review.
I also need to check why the macos build failed here. It worked till make check on by test repo - see https://github.com/Shreeshrii/testactions/runs/1717479792?check_suite_focus=true

@egorpugin Thanks for merging these.  Yes, these need corrections and review.
I also need to check why the macos build failed here. It worked till make check on by test repo - see https://github.com/Shreeshrii/testactions/runs/1717479792?check_suite_focus=true

@egorpugin Please see what needs to be added to windows, I am getting error
  Could not find a package configuration file provided by "SW" with any of
  the following names:

    SWConfig.cmake
    sw-config.cmake

  Add the installation prefix of "SW" to CMAKE_PREFIX_PATH or set "SW_DIR" to
  a directory containing one of the above files.  If "SW" provides a separate
  development package or SDK, be sure it has been installed.

@egorpugin Please see what needs to be added to windows, I am getting error
  Could not find a package configuration file provided by "SW" with any of
  the following names:

    SWConfig.cmake
    sw-config.cmake

  Add the installation prefix of "SW" to CMAKE_PREFIX_PATH or set "SW_DIR" to
  a directory containing one of the above files.  If "SW" provides a separate
  development package or SDK, be sure it has been installed.

Windows cmake build requires sw at the moment.
If you want to build it without sw, it will require a lot of effort.

Windows cmake build requires sw at the moment.
If you want to build it without sw, it will require a lot of effort.


Unable to add Rule to multivalue control fields in Azure Devops

Hi @PreethaSarsan
I am using Azure DevOps. The extension seems to work for me with Rules too.
In below scenario I created a custom control of System Info and made it mandatory during bug creation.
I use custom Scrum template, I am not sure if that is impacting the outcome.

Hi, this should be resolved with the latest version. Closing the issue for now, feel free to report a new issue if you still have problems.

The multi value control field can be set mandatory but we can not add rule to the multivalue control field(eg: to make the field conditionally mandatory)


Container Apps Jobs Operator built-in role is not able to start Container App Jobs

Please provide us with the following information:

This issue is a: (mark with an x)

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

Issue description
A clear and concise description of the observed issue.
Steps to reproduce

Give the container job app "Container Apps Jobs Operator" role on container apps jobs level
Try to start the Container Apps Jobs. You will fail since the play button is grayed out.

Expected behavior [What you expected to happen.]
To be able to start the
Actual behavior [What actually happened.]
Screenshots
If applicable, add screenshots to help explain your problem.
I can not click on "Run now".

Additional context
Ex. Did this issue occur in the CLI or the Portal?
The "Container Apps Jobs Contributor" is working as expected.

I also mentioned this issue at https://github.com/microsoft/azure-container-apps/issues/1024

Thanks for raising this with us. We're looking into this.

We are investigating this issue. As a workaround, you can use CLI to Run the job


Backport: (PUP-914) Use resource ref when writing dot files

Use the ref attribute of a resource instead of to_s.
Internally, Puppet uses the ref attribute to identify and locate resources
within a catalog. This patch makes use of ref to identify nodes in graphviz
output as some Types, notably Whit, override to_s and either destroy or
obscure the identity of resource nodes.

@Sharpie could you add a test for this?

PR Triage Notes:

@kylog to review about including in 3.7.5. If so, @Sharpie we'll need a test for this.

Hi! Sorry, this one has been perpetually shoved down my TODO list by other things over the past few weeks.
I can look into overhauling the tests, but:

This basically swaps one input, to_s, for another: ref. Since it doesn't break existing tests, this sort of implementation change should be fine.

This has already been merged to master without any work on the tests.

PR Triage Notes:

@joshcooper to review and merge to stable


Reifying has_many through associations help

My associations aren't reverting. I must be doing something wrong.
Here's a similar setup to mine. I'm using the latest master branch.
class Ball < ActiveRecord::Base
  has_many :ball_colors
  has_many :colors, through: :ball_colors
  has_paper_trail
end

class BallColor < ActiveRecord::Base
  belongs_to :ball
  belongs_to :color
  has_paper_trail
end

class Color < ActiveRecord::Base
  has_paper_trail
  has_many :ball_colors
  has_many :balls, through: :ball_colors
end

This is what I'm doing.
ball = Ball.create()
ball.name = 'Before I add color'
ball.save

ball.colors << Color.create(name: 'blue')
ball.save #although this is unnecessary i think

ball.name = 'After adding color'
ball.save

b = ball.versions.last.reify(:has_many => true)
b.save

b.reload
b.name  #=> 'Before I add color'
b.colors #=> [Blue]

b.colors should be empty.
Also, ball.versions doesn't include the change when I added a color to the ball. There's only create,  update for 'Before I add color', update for 'After adding color'.
Could someone tell me what I'm doing wrong or point me to an example?
I've read the Associations section of the documentation but to no avail.

You added a color to the b object prior to making a new version, which explains why the b.colors is not empty when you reify the old version. If you had changed the order to be like this:
ball.name = 'After adding color'
ball.save

ball.colors.create!(name: 'blue')

Then I would expect b.colors to be empty when you reified the older version.
You shouldn't expect to see an additional version for when you added an association to the ball, because you didn't change the object attributes at all before calling save there.  Versions only get generated for events where save actually persists a change to the root object itself.  It seems to me as though everything is working as I would expect it is, what didn't work the way you thought it would?

Alright. I was expecting the wrong results.
After getting the proper version and associations back, I notice that when I save the reverted Ball object, the associations don't change in the database. I've tried adding (:mark_for_destruction => true) to the reify options but .save! still has no change to the associations. How can I make the reverted association I see in the object change in the database?
Also, if I wanted a change to a Color association to be versioned without changing the Ball object how should I go about it?
ball.touch_with_version
ball.colors << 'orange'

Would this be what is intended?
Thanks for your help.

It sounds like this is a dupe of #463.  @magister94 do you mind if we close this issue and continue the conversation in #463?  Thanks.

This is a dupe of #463 but #463 mention that #461 is a dupe of #643. Mind Blown !


Move custom-enable-credssp.ps1 to brooklyn-server

Now that https://github.com/apache/brooklyn-server/pull/75 is merged, I'll merge this.


mv service host

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

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

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Quite_Significant_Signed_NotApproved_12thJun15

Hi @brilliomsinterop, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, DNFBOT;

@brilliomsinterop, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@brilliomsinterop, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@brilliomsinterop, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;


Docs don't explain how to get the contents of a message.

I'm struggling to find the right API to use to retrieve the message body. If I take the raw data from a message part, I can convert it to a String but it has extraneous characters. Is there an API to use to get the contents of a message?

Hey @matadan,
Postal allows you to retrieve the mail contents as Parts, but there are many uses to it. We did not provide a specific way to use them. Maybe you could create some helpers that are specific to your domain.
If you want to access the textual part of the mail (if the mail has a text part), you can try to enumerate the body parts (with fetchResult.body?.allParts) and find one that has the 'text/plain' mime-type. You will then have access to the text content.

Thank you. I wrote this function:
func bodyText(body: MailPart) -> String? {
    for part in body.allParts {
        if part.mimeType.type == "text" && part.mimeType.subtype ==  "plain" {
            if let decodedData = part.data?.decodedData {
                let decodedString = NSString(data: decodedData, encoding: NSUTF8StringEncoding) as String!

                return decodedString
            }
        }
    }

    return nil
}

I added some improvements that could simplify your loop in #21
MimeType is now Hashable and Comparable
Some common types are provided for example: MimeType.textPlain
You can now type if part.mimeType == MimeType.textPlain {


JavaScript Function Redefinition Challenge

In the integration tests of #859 I wrote a script to redefine the javascript function for Date. Will it be useful to build a challenge around this which can be solved by modifying the functionality of a function in browser console which is expected to be used by the juice shop.
Example:

@bkimminich @J12934 @CaptainFreak
Please let me know of your views and suggestions.
Thank You!!

I don't see how this would be a security vuln in itself.
This would basically just be another case of putting too much trust in the client, like only validating inputs on the client.
The easier solution to abuse these vuln would be for an attacker to just send the tampered requests directly via curl than to mess with the client side javascript.

I don't see how this would be a security vuln in itself.
This would basically just be another case of putting too much trust in the client, like only validating inputs on the client.
The easier solution to abuse these vuln would be for an attacker to just send the tampered requests directly via curl than to mess with the client side javascript.

Yeah, I get it. I didn't think in this manner.
Now I think, I understand what kind of challenges would make sense.
Thank You!! for your feedback.
I will think more this time.😁😁😆😆