[2; .txt, .js] Outputs entirety of generated file despite there being only two changes; tries to allocate 16.3 TiB

Buy one issue, get another free!
(1) A description of the issue. A screenshot is often helpful too.
This occurred when running git diff on this before commit.

This continues for all 25,109 lines, excluding the empty final one.
It diffs data/data.txt fine, but then tries allocating ~18 trillion bytes for the diff of data/jbo.js, probably at least because it's 4.54 MiB squished onto one line. I admit I don't know what to do about this.

(2) A copy of what you're diffing. If you're diffing files, include
the before and after files. If you're using difftastic with a VCS
repository (e.g. git), include the URL and commit hash.
Link above; hash is 2190a95d50cfe15039df80f53a310f66a40008e3.
(3) The version of difftastic you're using (see difft --version) and
your operating system.
0.55, Windows 11

lojbo .ui
Thanks for the report.
I think the first issue is probably trying to diff lines ending \r\n against files ending with just \n and concluding that they're different.
The second issue is probably due to line diffing trying to highlight words within the same line.

I can repro the second issue, but not the first. Can you reproduce the first issue with local copies of the files? If so, could you attach them here?

OK, I can reproduce the first issue if I have two files that differ by line ending, but I'm not convinced that difftastic's behaviour is wrong here.
$ echo "one\ntwo" > a.txt
$ echo "one\nfoo\ntwo" > b.txt
$ unix2dos b.txt

$ difft a.txt b.txt
b.txt --- Text
1 one                             1 one
.                                 2 foo
2 two                             3 two

$ diff a.txt b.txt
1,2c1,3
< one
< two
---
> one
> foo
> two

Plain GNU diff also considers these files to be completely different. Do you have any special crlf settings in your Windows git setup?

I can repro the second issue, but not the first. Can you reproduce the first issue with local copies of the files? If so, could you attach them here?

Unfortunately I'll be away from my computer until next week, sorry. I don't think it's the line endings though? Pretty sure allwords.txt has been CRLF all along. When I get back I'll try adding a test entry to the database and regenerate allwords.txt and see if this still happens.
As for

Do you have any special crlf settings in your Windows git setup?

I don't remember ever messing with that- the one thing I know I've done to etc/config is using difftastic rather than the default diff lol
ni'o lu «lojbo .ui» li'u zo'u .ue xu do jbopre

The test entry has been made, it happens to mean 'line terminator' lol; I'll rerun the parsing stuff when I can

Reran the script with the new word.
allwords.txt still printed the entire file:

data.txt still didn't:

Replaced every \n with \r\n in the script, reran.
This fixed allwords.txt, but data.txt's changes were too big.

Reverted the changes to those two, manually replaced each LF with CRLF, committed, reran.

wheeeeeee

Updated to 0.56.1, turns out printing such a giant diff is REALLY bad for my terminal lol


Skip CPU-only tests on CircleCI machines with GPU

This PR makes the CircleCI machines with GPU skip the tests that don't actually need a GPU. These tests are already extensively tested in the other CPU-only workflows.

This seems to be working as expected, see e.g. the linux gpu unittest workfow where the test_classification_model[cpu... tests are skipped while the CUDA ones are run. test_generalizedrcnn_transform_repr is also properly skipped.
The tests aren't properly skipped on Windows because of https://github.com/pytorch/vision/issues/4006, but we can merge this anyway.
Note: There are still a lot of CPU-only tests that are run on the GPU CI. These are all the tests in tests files like test_transforms.py that don't include GPU tests at all. I will open another PR to deal with these.

@NicolasHug  On unittest_windows_gpu_py3.8, I see cuda tests being skipped. Example:
test/test_ops.py::TestDeformConv::test_forward[0-True-cpu] PASSED        [ 96%]
test/test_ops.py::TestDeformConv::test_forward[0-True-cuda] SKIPPED      [ 96%]
test/test_ops.py::TestDeformConv::test_forward[0-False-cpu] PASSED       [ 96%]
test/test_ops.py::TestDeformConv::test_forward[0-False-cuda] SKIPPED     [ 96%]
test/test_ops.py::TestDeformConv::test_forward[33-True-cpu] PASSED       [ 96%]
test/test_ops.py::TestDeformConv::test_forward[33-True-cuda] SKIPPED     [ 96%]
test/test_ops.py::TestDeformConv::test_forward[33-False-cpu] PASSED      [ 96%]
test/test_ops.py::TestDeformConv::test_forward[33-False-cuda] SKIPPED    [ 96%]

Is this related to the temporary CicleCI driver issue for win GPU?


Implement IBC version negotiations in IBC java

Add logic for version negotiation to IBC Java.
Take care to not break the connection to wasm

@sabinchitrakar @izyak
How do you feel about version negotiation?
Problem as raise in audit
Description
ICON's implementation of Inter-Blockchain Communication (IBC) currently adopts a simplistic approach to version
selection for counterparty chains, differing significantly from the methodology advised in the IBC specifications. In
the standard IBC-go implementation, the chosen version is an intersection of supported versions and counterparty
versions. However, in ICON's implementation, the version is directly taken from the message without considering
an intersection, as evidenced by the code comment // We currently support no official version
in the isSupportedVersion function. This approach overlooks the nuanced and critical process of version
negotiation outlined in the IBC specifications, particularly in ICS-003 connection semantics.
Problem Scenarios

Lack of Version Negotiation: ICON’s approach bypasses the negotiation process, potentially leading to
compatibility issues, especially as ICON aims to connect to IBC via ICS-08. Directly accepting versions from
messages without an intersection check may result in operational inefficiencies or mismatches in protocol
capabilities.
Siloed IBC Environment Risks: While ICON's current environment may be somewhat isolated, with only one
version in use, this design may not be sustainable in a broader IBC ecosystem. As ICON connects with more diverse
chains, the lack of a robust versioning strategy could pose integration and compatibility challenges.

https://github.com/cosmos/ibc/tree/main/spec/core/ics-003-connection-semantics#versioning
I have a feeling adding this might just cause us problems as we try to connect to more chains. Since we do not expect people to open random connection to ICON it might be a bad idea to add. What do you guys feel?

I think it will start to make sense once we have to create patches for different chain based on their version. But i dont think other chains will be making any patches on their end based on our versioning. The doc states Host state machine MUST utilise the version data to negotiate encodings, priorities, or connection-specific metadata related to custom logic on top of IBC. In my opinion for us its better to just reflect versionings of chain that we are connecting to since we would have made necessary changes based on their versioning.

Yeah but we should not need to create patches for the different chains(hopefully) And if we do, since we establish the connection we can still chose the correct version we want. It would instead be a very annoying scenario if we are prevented to connect to a chain only due to a missing string

Yeah, I agree.
Think for now, it should be alright, if we just check the implementation of the ibc-go before connecting to ICON. The cosmos implementation of ibc-go, as well as the composablefi fork, I believe only has one default version supported anyway.


Add funds to offer. Seller withdraw offer.

First pull request? Read our guide to contributing
Checklist:

[x] Code contains relevant tests for the problem you are solving
[x] Ensure all new and existing tests pass
[ ] Update any relevant READMEs and docs

Description:

Seller can now call withdrawOffer
New method addFunds for buyer to add funds to accepted offer
Arbitrator now no longer needs to be a contract
Arbitrator can call executeRuling with IPFS hash

I'll give @DanielVF a chance to review before I merge


feat(error): adding some more detailed error descriptions

What this PR does / why we need it:
Add your description
Which issue(s) this PR fixes (optional, close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @kdada
Release note:

NONE

/lgtm


Nohang: Package,  that prevents Linux from freezing when out of memory

Project description
Nohang package provides a highly configurable daemon for Linux which is able to correctly prevent out of memory (OOM) and keep system responsiveness in low memory conditions.
Metadata

homepage URL: https://github.com/hakavlad/nohang
source URL: https://github.com/hakavlad/nohang
license: mit
platforms:  linux

Anyone wants to pick this up? Nohang is like a thing on every other distro.


[razor] @addTagHelper should get syntax highlighting like @using

@addTagHelper should get syntax highlighting like @using

I assume this is with Razor.

It was with Razor - I was present at the workshop ;-)

This got fixed in the meantime. Verified in VSCode 1.10 insiders.


Fixes #1987, Improve the error handling when site extension installs fail via ARM

{
  "id": "/subscriptions/fb2c25dc-6bab-45c4-8cc9-cece7c42a95a/resourceGroups/Default-Web-EastUS/providers/Microsoft.Web/sites/kudutry/siteextensions/filecounter",
  "name": "kudutry/filecounter",
  "type": "Microsoft.Web/sites/siteextensions",
  "location": "East US",
  "properties": {
    "id": "filecounter",
    "title": null,
    "type": "Gallery",
    "summary": null,
    "description": null,
    "version": null,
    "extension_url": null,
    "project_url": null,
    "icon_url": null,
    "license_url": null,
    "feed_url": null,
    "authors": null,
    "published_date_time": null,
    "download_count": 0,
    "local_is_latest_version": null,
    "local_path": null,
    "installed_date_time": null,
    "provisioningState": "Failed",
    "comment": "System.IO.FileNotFoundException: Test FileNotFoundException\r\n   at Kudu.Core.SiteExtensions.SiteExtensionManager.<TryInstallExtension>d__27.MoveNext()"
  }
}

{
  "id": "/subscriptions/fb2c25dc-6bab-45c4-8cc9-cece7c42a95a/resourceGroups/Default-Web-EastUS/providers/Microsoft.Web/sites/kudutry/siteextensions/filecounter",
  "name": "kudutry/filecounter",
  "type": "Microsoft.Web/sites/siteextensions",
  "location": "East US",
  "properties": {
    "id": "filecounter",
    "title": null,
    "type": "Gallery",
    "summary": null,
    "description": null,
    "version": null,
    "extension_url": null,
    "project_url": null,
    "icon_url": null,
    "license_url": null,
    "feed_url": null,
    "authors": null,
    "published_date_time": null,
    "download_count": 0,
    "local_is_latest_version": null,
    "local_path": null,
    "installed_date_time": null,
    "provisioningState": "Failed",
    "comment": "System.Net.WebException: Test WebException\r\n   at Kudu.Core.SiteExtensions.SiteExtensionManager.<TryInstallExtension>d__27.MoveNext()"
  }
}

{
  "id": "/subscriptions/fb2c25dc-6bab-45c4-8cc9-cece7c42a95a/resourceGroups/Default-Web-EastUS/providers/Microsoft.Web/sites/kudutry/siteextensions/filecounter",
  "name": "kudutry/filecounter",
  "type": "Microsoft.Web/sites/siteextensions",
  "location": "East US",
  "properties": {
    "id": "filecounter",
    "title": null,
    "type": "Gallery",
    "summary": null,
    "description": null,
    "version": null,
    "extension_url": null,
    "project_url": null,
    "icon_url": null,
    "license_url": null,
    "feed_url": null,
    "authors": null,
    "published_date_time": null,
    "download_count": 0,
    "local_is_latest_version": null,
    "local_path": null,
    "installed_date_time": null,
    "provisioningState": "Failed",
    "comment": "Kudu.Contracts.SiteExtensions.InvalidEndpointException: Test InvalidEndpointException\r\n   at Kudu.Core.SiteExtensions.SiteExtensionManager.<TryInstallExtension>d__27.MoveNext()"
  }
}

{
  "id": "/subscriptions/fb2c25dc-6bab-45c4-8cc9-cece7c42a95a/resourceGroups/Default-Web-EastUS/providers/Microsoft.Web/sites/kudutry/siteextensions/filecounter",
  "name": "kudutry/filecounter",
  "type": "Microsoft.Web/sites/siteextensions",
  "location": "East US",
  "properties": {
    "id": "filecounter",
    "title": null,
    "type": "Gallery",
    "summary": null,
    "description": null,
    "version": null,
    "extension_url": null,
    "project_url": null,
    "icon_url": null,
    "license_url": null,
    "feed_url": null,
    "authors": null,
    "published_date_time": null,
    "download_count": 0,
    "local_is_latest_version": null,
    "local_path": null,
    "installed_date_time": null,
    "provisioningState": "Failed",
    "comment": "System.Exception: Test Exception\r\n   at Kudu.Core.SiteExtensions.SiteExtensionManager.<TryInstallExtension>d__27.MoveNext()"
  }
}

Looks good. Have you tried trigger an error with the new code to get a sense of what the error response will look like? If needed, just use a hacked Kudu build that throws in some random place just to trigger the exception handling.

@davidebbo  yes, just updated the PR to include sample output from my verification. what I did was hack kudu to throw error like this throw new Exception("Test Exception")

Looks good!
One minor issue (not really related): would be nice to populate the feed_url in the SiteExtensionInfo object you create. When I first looked at customer issue, I though they were not setting it because it was null, when in fact they were.

null feed_url input is a valid case, in this case Kudu will default to siteextensions.net
if we return feed_url, I think we should return the actual feed_url that kudu use. what do you think?

:smile:


Login file js edit and correction

#4063
"None"
Fixes:  #4063
Description
Edited the login and logout options and there were some corrections needed in the Login file (script) for issue: #4063

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

plz create a demo video to show what u have done . and explain properly

are u working on this issue? as gssoc ext is about to end.

Yes I am, will submit by the EOD.
On Tue, 29 Oct, 2024, 10:30 am Abhiruchi Sarswat, @.***>
wrote:

are u working on this issue? as gssoc ext is about to end.
—
Reply to this email directly, view it on GitHub
https://github.com/anuragverma108/SwapReads/pull/4079#issuecomment-2443216892,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXDSCHFNFXPX6MPXI727UNLZ54JAPAVCNFSM6AAAAABQVPYEQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBTGIYTMOBZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

work with updated branch and raise new PR @PriyanshiAgr


Feature request: Support .env*.local as overrides

To follow the convention set in create-react-app, I'm proposing the following additional environment files:

.env.local which overrides .env
env.development.local which overrides .env.development
.env.production.local which overrides .env.production.local

These would not be mutually exclusive so the following
# .env
A=hello world
B=1

# .env.local
B=2

Would result in the following environment variables:
A=hello world
B=2

Let me know if this feature would be accepted into the main branch and I'll make a PR. Guidance on how this would be implemented would also be appreciated.

From reading the source code, what happens when there are both .env and .env.development files present?
From my understanding, the development and production files are mutually exclusive. So that means that we would have to diverge slightly from the create-react-app's behaviour of the following:

Files on the left have more priority than files on the right:
[development]: .env.development.local, .env.development, .env.local, .env
[production]: .env.production.local, .env.production, .env.local, .env

Instead we would probably have to go with

Files on the left have more priority than files on the right:
[development]: .env.development.local, .env.development, .env.local, .env
[production]: .env.production.local, .env.production


Add missing 'anywhere'

eksctl generate hardware is missing the 'anywhere' subcommand.
Issue #, if available:
Description of changes:
I'm simply adding the missing 'anywhere' subcommand
Testing (if applicable):
Tested locally
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve
/lgtm

/cherry-pick release-0.10


Upgrade kustomize to v5.0.1

Add support for new Helm fields: additionalValuesFiles, skipTests, apiVersions and nameTemplate
Fixes: #223

Yes, I just bumped dependencies and after that fixed broken tests
In one case kustomize added creationTimestamp: null field
Other changes is because patches as I understand was for a long time an alias for patchesStrategicMerge
And in 5.0 version both patchesJson6902 and patchesStrategicMerge are deprecated in favour of newer patches:
https://github.com/kubernetes-sigs/kustomize/issues/5052#issuecomment-1470926760
So I converted patches (as alias to patchesStrategicMerge) to a newer patches
In one case I had to split files with patches into two separate files because newer patches doesn't support multiple patches in one file yet:
https://github.com/kubernetes-sigs/kustomize/issues/5049

Thanks for the background, I'm a bit concerned that this upstream change might break the kustomization_overlay's use of patches too. Although I would have expected a test to fail if it did. At the very least I'd have to document it in the release notes.
I will look into this as soon as I can. I am currently busy with a big release for the Kubestack framework. So it may take a few days until I find the time.


Coloring only the brackets around an expression

See the below code:
(La)TeX code:
The code of (La)TeX you tried to render:
\textcolor{red}{\left(}x\textcolor{red}{\right)}

Expected behavior:
I expected the output to look like: $\textcolor{red}{(}x\textcolor{red}{)}$
What works:
When I try to render the code: \textcolor{red}{(}x\textcolor{red}{)} (i.e. replacing \left( with ( and \right) with ). The code renders just fine.

This is how LaTeX works. \left and \right need to be balanced (they can't be inside separate braced scopes) so that it knows what to measure.
You have two alternatives:

Use \color, e.g., \color{red}\left(\color{black}x\color{red}\right)
Use manually sized brackets, like \big, \bigg, etc.  E.g. \textcolor{red}{\big(}x\textcolor{red}{\big)}


Eliminated a warning in WebSocketHandler

The waring says: "Variable 'frame' was never mutated; consider changing to 'let' constant".
Changed it from var to let.

This should be clear to get merged after https://github.com/vapor/websocket/pull/42 is shipped.

Hm, took a look at the circle.yml but I don't see what the linux-vapor test could be 🤔
@tanner0101 any thoughts on what that third test case is that's Waiting for status to be reported ? 🙏

Is it still possible to release a version 1.1.3 and get rid of the warnings in Vapor 3?
Thanks.


Multiple rate limiters using same key fail to behave as expected

Laravel Version: 8.83.26
PHP Version: 8.1.1
Database Driver & Version: MariaDB 10.4

Description:
Using multiple rate-limiters that use the same key does not work as expected. If the key is the same, only the first rate limiter is ever triggered.
Additionally, because both limiters use the same key, the count of remaining requests is decremented by the amount of times the same key is used.
Steps To Reproduce:
Define a simple rate limiter that returns an array of limits, using the same key.
RateLimiter::for('maps', function (Request $request) {
    return [
            Limit::perMinute(3)->by($request->ip()),
            Limit::perDay(30)->by($request->ip())
        ];
});

The above code will only apply the first limit, but the "Remaining" is decremented by two because both use the same key.
An alternative way that I was hoping would work but does not either:
return Limit::perMinute(3)->perDay(30)->by($request->ip());

In the above case, only the second limiting method applies (so perMinute is ignored, but perDay applies). Looking at the source code of the Limit class, it makes total sense and looks like it would require quite a few changes to make it work, so I think that its perhaps not as big an issue as the first case.

Note as a workaround, you can just define a different key for each limiter, e.g. by($request->ip().'-minute'). It doesn't feel very clean though :P

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so, please open up a new issue and we'll help you out.
Thanks!


Distribute package without default styles

Could you distribute a version of the package without the default styles?
We would like to customise a lot of the graph and would be good if we could exclude the default styles. Especially if using a css library such as tailwind where it is styled with css classes it becomes tricky to deal with the specificity of the default styles applying over our custom styles.

I think the next major version will not include the default styles anymore. Maybe we can already deliver a version without styles with an environment variable for example(without breaking changes)?

I think the next major version will not include the default styles anymore. Maybe we can already deliver a version without styles with an environment variable for example(without breaking changes)?

Should be okay to wait for the next major version if that is your plan already. I'm not familiar enough with the packaging but perhaps having a different build/import path for the styleless version could work too?

Should be okay to wait for the next major version if that is your plan already. I'm not familiar enough with the packaging but perhaps having a different build/import path for the styleless version could work too?

Thats also an idea, yep.

Thats also an idea, yep.

@moklick  how about  using tsdx as package builder ? ;)

@moklick  how about  using tsdx as package builder ? ;)

Thanks @ambroseus I will check if tsdx will work for us.

Thanks @ambroseus I will check if tsdx will work for us.

We checked tsdx but decided to keep our rollup-only packaging setup. We want to be able to use the latest version and be as flexible as possible.
I just pushed a new branch feat/nocss-entry. It works fine but it's ugly because you would need to do:
import ReactFlow from 'react-flow-renderer/dist/nocss/ReactFlow.esm.js'
Could anyone help me here on how to create another clean entry point? Do we need a nocss folder in the root folder with it's own package.json in order to have another main and module entry?
At the end I would like to do import ReactFlow from 'react-flow-renderer/nocss'.

We checked tsdx but decided to keep our rollup-only packaging setup. We want to be able to use the latest version and be as flexible as possible.
I just pushed a new branch feat/nocss-entry. It works fine but it's ugly because you would need to do:
import ReactFlow from 'react-flow-renderer/dist/nocss/ReactFlow.esm.js'
Could anyone help me here on how to create another clean entry point? Do we need a nocss folder in the root folder with it's own package.json in order to have another main and module entry?
At the end I would like to do import ReactFlow from 'react-flow-renderer/nocss'.

@moklick  plz review https://github.com/wbkd/react-flow/pull/846

@moklick  plz review https://github.com/wbkd/react-flow/pull/846

Thanks for your help here @ambroseus  :)
With v8.4.1 you can do:
import ReactFlow from 'react-flow-renderer/nocss';

// you need these styles for React Flow to work properly
import 'react-flow-renderer/dist/style.css';

// additionally you can load the default theme
import 'react-flow-renderer/dist/theme-default.css';

https://reactflow.dev/docs/theming/

Thanks for your help here @ambroseus  :)
With v8.4.1 you can do:
import ReactFlow from 'react-flow-renderer/nocss';

// you need these styles for React Flow to work properly
import 'react-flow-renderer/dist/style.css';

// additionally you can load the default theme
import 'react-flow-renderer/dist/theme-default.css';

https://reactflow.dev/docs/theming/

@moklick  awesome! 🥇

@moklick  awesome! 🥇


add push-based source

add push-based source like twitter stream, where we need to store the source data into a storage. That will complete our stream replay impl.

Merge this into #1226


Disable beam VM distributed features

Description of changes
My proposal for #166229

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This should be partially merged with my proposal, I feel. There are cases when you would want to use BEAM distribution features when installing a BEAM app using Nix.
If a flag is added (enableDistribution?) that's set to false by default, you could then revert to deleting the cookie when it's enabled and just introduce a wrapper that warns and exits if RELEASE_COOKIE is unset, and leave setting RELEASE_COOKIE to the user, although perhaps it should include some information on what a release cookie is and how the user is advised to generate one.

I feel this should be partially merged with my proposal. There are cases when one might wish to use BEAM distribution features with NixOS services. It may be valuable to add a cookieFilePath flag to the services (null or string path) that is null by default but allows providing a path.
If I'm being honest, it may be more valuable to wait for my PR and simply have this behavior depend on that, rather than implementing this behavior into every BEAM service individually, which is always prone to errors with new contributions, although it means that I may want to have the mixRelease builder generate two derivations (one which is simply a wrapper) to avoid having to rebuild the entire app based on the flag.

I am no longer interested in working on this, but feel free to use the diff for a new PR or otherwise


UnhandledPromiseRejectionWarning: FetchError: request to https://fast-serve-track.azurewebsites.net/api/track failed, reason:  connect ECONNREFUSED <IP_ADDRESS>:443

Hi,
when i run npm run serve, I get following error, but the build seems to work...
what does this error mean and why is this request being made?
UnhandledPromiseRejectionWarning: FetchError: request to https://fast-serve-track.azurewebsites.net/api/track failed, reason:
connect ECONNREFUSED <IP_ADDRESS>:443
thanks!.

Hi, fast serve tracks the usage. Does https://fast-serve-track.azurewebsites.net/ url works for you?

Hi...
I guessed that it has to be the usage collection request...
well... this url works, but npm environment is behind a proxy... I guess it is being blocked by infrastructure...
anyway... in case this is the collection call, I guess i could ignore this error? at least, it seems so....

Yes, you can ignore it. Actually you shouldn't see it all, but somehow it appears. Will check why it happens.

btw... why are you collecting the anonymized data at all? Isnt the download count enough?...
Regards...

The download count is not informative, because you can install the tool globally once and use it for many months. I try to collect the real usage.

@Link631 which SPFx version do you use?

I fixed the problem in SPFx 1.13 & 1.14. Please upgrade helpers version - npm update spfx-fast-serve-helpers and see if the issue is gone.


Handle xdg-desktop-portal-wlr color-scheme error gracefully

Would it be possible to gracefully handle portal on wlroots compositors not finding color-scheme with a warning or info message instead of error? From what I read, this isn't planned (https://github.com/emersion/xdg-desktop-portal-wlr/issues/173) to be implemented in xdg-desktop-portal-wlr.

Sorry I don't full understand the request, you mean just lowering the log level for the message? Or is the library behaving in an unexpected way when it can't find the color-scheme value?

Sorry, I was testing latest wgpu and saw the error. Since it's only a logged (in config.rs) without changing behavior, lowering the level to info or debug would be fine. You may want to turn it into a warning, which I can understand. But at the moment it seems that wlroots compositors would always fail to respond here, and the log message would be visible in daily use, without a way to fix it. Therefore I would vote for info or debug level, if you're okay with that.

Sorry, I was testing latest wgpu and saw the error. Since it's only logged (in config.rs) without changing behavior, lowering the level to debug is sufficient. You may want to turn it into a warning or info, which I can understand. But at the moment wlroots compositors will always fail to respond here, and the log message will be visible in daily use, without a way to fix it. Therefore I vote for debug level, if you're okay with that.

without a way to fix it

Well, settings portals are compositor agnostic, you could run one even in raw TTY, so you can simply run one like GTK portal (do not confuse with gnome portal) alongside wlr one, and as far as I remember that's the recommended way to get settings portal as well as system file dialogs support in sway.
But that's unrelated to the issue, will probably lower the log level anyway.

You've likely haven't exported environment to the portals running. it's a bit racy on sway for example.

You've likely haven't exported environment to the portals running. it's a bit racy on sway for example.

I thought I was doing what's necessary by running the official sway package and making sure to source /etc/sway/config.d/* as the last line of sway's config. Anything I'm missing to get any sort of portal functionality?

You've likely haven't exported environment to the portals running. it's a bit racy on sway for example.

I thought I was doing what's necessary by running the official sway package and making sure to source /etc/sway/config.d/* as the last line of sway's config. Anything I'm missing?

You need this exec --no-startup-id dbus-update-activation-environment --all as the first line in the config iirc. it's in the sway wiki.

Sourcing /etc/sway/config.d/* as first line of sway's config works when using portal-gtk. Thanks for the directions. Haven't retested with portal-wlr yet. I would like to disable the document-portal and avoid the unused mount, but that's another issue.
This is what you can find in /etc/sway/config.d/50-systemd-user.conf:
# sway does not set DISPLAY/WAYLAND_DISPLAY in the systemd user environment
# See FS#63021
# Adapted from xorg's 50-systemd-user.sh, which achieves a similar goal.

exec systemctl --user import-environment DISPLAY WAYLAND_DISPLAY SWAYSOCK XDG_CURRENT_DESKTOP
exec hash dbus-update-activation-environment 2>/dev/null && \
     dbus-update-activation-environment --systemd DISPLAY WAYLAND_DISPLAY SWAYSOCK XDG_CURRENT_DESKTOP

What surprised me is that systemd seems to persist the environment even after I log out all sessions for the user, and old variables were still there and conflicting. Looks like more stuff about systemd I have to read. Oh well

Thanks for the directions, putting this at the start of sway's config works:
exec systemctl --user import-environment WAYLAND_DISPLAY SWAYSOCK XDG_CURRENT_DESKTOP QT_QPA_PLATFORMTHEME
exec dbus-update-activation-environment --systemd WAYLAND_DISPLAY SWAYSOCK XDG_CURRENT_DESKTOP QT_QPA_PLATFORMTHEME

Line 1 is probably unnecessary, but it's what the Arch package has in /etc/sway/config.d/50-systemd-user.conf.
xdg-desktop-portal-wlr didn't work on its own and gave me a resource-unavailable error. Is xdg-desktop-portal-gtk enough or should I complement it with xdg-desktop-portal-wlr?
Either way, the new sway config works for xdg-desktop-portal-gtk to properly answer dbus messages. What surprised me is that systemd seems to persist the environment even after I log out all sessions for the user, and old variables were still there and conflicting.
To solve that and modernize my environment, I migrated to greetd-tuigreet. That also works, but now I have to find the right way to start sway with all variables from ~/.config/environment.d visible. Before, I used to run sway from my login shell. For zsh, it's accomplished via export $(/usr/lib/systemd/user-environment-generators/30-systemd-environment-d-generator). At the moment, apps spawned via fuzzel as sway's launcher menu do not inherit environment variables such as QT_QPA_PLATFORMTHEME, despite explicitly listing it. Don't know how to launch sway the right way as the documented greetd recipe from Arch's wiki isn't enough. Maybe sway has to be run as a systemd unit.
This is getting off-topic and I shared the progress as documentation for anyone else coming here after searching the web. I'll stop after this message.

Yet, your issue has nothing to do with this issue. I'd suggest to continue it on #sway channel.


Support latest Emoji One version

Emoji One 2.2.6 already released, but emojionearea supports versions up to 2.1.4. Sad ( Thanks>

@mervick any idea on when this will come ? the current version is 2.2.7

Hi, I am looking for emojionearea update to support emojione v2.2.7.
I try to add emojione newest version to emojionearea but tired to do this!

Support for 2.2.7 was added in #89. This issue should be closed


[Bug]: failed to set 'sendConnectionAttributes' jdbc parameter

ODC version
odc422
OB version
not have
What happened?

What did you expect to happen?
Real IP displayed in session management
How can we reproduce it (as minimally and precisely as possible)?

Modifying Drive Properties
View Session Management

Anything else we need to know?
No response
Cloud
No response

pass


Iiwa boston01

Configuration files for the Kuka IIWA robot skin

@simeonedussoni @maggia80 what's the status of this PR?
Can it be merged safely?

@pattacini  this contains the configuration files for the kuka robotic arm we sensorized for IIWA. It can be merged safely as the files worked properly.

@simeonedussoni the branch iiwaBoston01 has been merged into master, hence it is gonna be removed.


Any easy way to compile this on windows?

I see this but it looks old:
https://github.com/kjk/the_silver_searcher
It would be great to be able to easily compile AG for windows.

Or a ready to use binary

Recently I setup a project for daily builds.
https://github.com/k-takata/the_silver_searcher-win32
(See also #1130)

In addition, ag is supplied as part of Cygwin project.

Actually, yes there's now an easy way to compile this for Windows:
https://github.com/JFLarvoire/the_silver_searcher
I've released there a few days ago 32 and 64-bits builds. See this link for details:
https://github.com/JFLarvoire/the_silver_searcher/releases


Conflict with Perfect Vision

I am getting incorrect cover values and the following error when running dnd5e-helpers, dnd5e-helpers-cover-expansion, and perfect-vision together.
VM145093:7 TypeError: Cannot set properties of undefined (setting 'cover')
[Detected 3 packages: dnd5e-helpers-cover-expansion, lib-wrapper, perfect-vision]
    at Function.userDefined (tokenCover.js:33)
    at tokenCover.js:66
    at Function._call (eval at <anonymous> (libWrapper-consts.js:9), <anonymous>:4:14)
    at Function.call (foundry.js:179)
    at Canvas.draw (foundry.js:17698)
    at async Canvas.<anonymous> (canvas.js:39)
    at async Scene.view (foundry.js:15619)
    at async Game.initializeCanvas (foundry.js:4514)
    at async Game.setupGame (foundry.js:4388)
    at async Game._initializeGameView (foundry.js:5213)
    at async Game.initialize (foundry.js:4286)
_call	@	VM145093:7
call	@	foundry.js:179
draw	@	foundry.js:17698
async function (async)
draw	@	foundry.js:17671
🎁call_wrapped	@	libWrapper-wrapper.js:467
wrapper	@	patch.js:17
🎁call_wrapper	@	libWrapper-wrapper.js:576
🎁Canvas.prototype.draw#0	@	libWrapper-wrapper.js:164
view	@	foundry.js:15619
Scene.view	@	monks-tokenbar.js:96
initializeCanvas	@	foundry.js:4514
async function (async)
initializeCanvas	@	foundry.js:4509
setupGame	@	foundry.js:4388
async function (async)
setupGame	@	foundry.js:4375
_initializeGameView	@	foundry.js:5213
_initializeView	@	foundry.js:5189
initialize	@	foundry.js:4286
async function (async)
initialize	@	foundry.js:4280
🎁call_wrapped	@	libWrapper-wrapper.js:467
🎁libWrapperInit	@	libWrapper-api.js:698
async function (async)
🎁libWrapperInit	@	libWrapper-api.js:688
🎁Game.prototype.initialize#0	@	libWrapper-wrapper.js:162
window.addEventListener.once	@	foundry.js:53613
async function (async)
window.addEventListener.once	@	foundry.js:53612

Sorry for the delayed response, I started using the Levels Cover system instead.


About the sub_project age estimation MAE (against apparent age)

Dear yu4u,
Thanks a lot for your wonderful work which is very neat, helpful and instructive.
My interest is the subproject-age estimation. I have gone through all the steps you have shown and got the similar results to yours. I also read your code and everything is very clear and understandable but about only one thing I'm not very clear, that is, the logic of combining APPA-REAL data and UTK data.
In generator.py, FaceGenerator function uses APPA-REAL apparent_age_avg and utk real age to generate training set, ValGenerator function only uses apparent_age_avg to test the model.
Question 1: the age estimation sub project aims at predicting apparent age not real age? So, the MAE is against apparent age? Any practical reason of predicting apparent age?
Question 2: If aim at predicting apparent age, if we use only apparent age(no real age) in train stage, the result will be better than combining apparent age(APPA-REAL data) and real age(UTK data)?
Question 3: the tag of age estimation shows 'improve pretrained accuracy' and you said 'Use APPA-REAL Dataset for finetuning CNNs for age estimation.' Here this improvement is based on comparison of the same metrics(MAE against apparent age) or comparison of MAE against apparent age and MAE against real age?
BTW, I am changing all apparent_age_avg to real age in the code, and will check the result MAE against real age.
Sorry for so many questions. Thanks in advance.
David

Hi, @DavidSUN2

Question 1

I did not focus on the target. The purpose is to use the appa-real dataset. It is easy to change train targets as you done.

Question 2

I have not done sufficient experiments for conclusion.
Instead,  please refer to the following useful slide:
https://pdfs.semanticscholar.org/0d48/ffcbfe76e224f0694ff4178f418f046efbd8.pdf

Question 3

I did not understand the question.
What is "this improvement" you are referring to?

Hi yu4u, thanks a lot for your reply. And sorry for my unclear questions.
About Question1, I am a little confused about the logic of the age estimation code. If the code trains all apparent-age images and then test apparent-age images, this makes sense to me. If the code trains all real-age images and then test real-age images, this also makes sense to me. But in your code, it trains the combination of apparent-age images(APPA-REAL) and real-age images(UTK) and then test apparent-age images(APPA-REAL), I cannot totally understand. Could you please give me some hint about this logic behind the code?
About Question2, thanks for your link. I read the pdf and Residual DEX makes sense to me: first train on apparent age, and then train on residual, and finally combine them together to predict real age. BTW, I have done some experiments and will show below.
About Question3,

This picture I screenshot shows that the age-estimation subproject improved the accuracy, I mean, compared with what results to say the accuracy is improved?

I did three experiments(all of them train 50 epochs):

train APPA-REAL real-age images + UTK real-age images, and then test APPA-REAL real-age images
train APPA-REAL apparent-age images + UTK real-age images, and then test APPA-REAL apparent-age images(the same as what you have done)
train APPA-REAL apparent-age images only, and then test APPA-REAL apparent-age images.
MAE, loss and accuracy curves all draw the same results: the second experiment works best, and the third experiment works worst.
About 'the second experiment works best', frankly speaking I'm not clear about the logic behind it which I have asked you in the previous questions. What I can guess is that the code is good at extracting features of apparent-age images and recognizing patterns of apparent-age images, just like us humans, therefore predict apparent age well, but what's the contribution of UTK real age images?
About 'the third experiment works worst', both train and test don't work well which surprise me because I assumed overfitting. I thought because of only using APPA-REAL dataset which is not large enough training will perform well(e.g. mae very low, even lower than training of experiment 2), but testing will perform bad(e.g. mae very high compared with training). However, the curves show that training and testing are very close to each other. What I can guess about the reason is that the APPA-REAL dataset are highly biased and not diverse enough about age distribution, therefore both training and testing don't work well. How do you think?

I cannot totally understand. Could you please give me some hint about this logic behind the code?

Sorry, there is no reason for this. I did not consider these consistencies.

This picture I screenshot shows that the age-estimation subproject improved the accuracy, I mean, compared with what results to say the accuracy is improved?

I simply mean by this that the accuracy of the trained model I uploaded improved compared with the previous one (the difference might be optimizer or learning rate scheduling... I have forgot it). There is no relationship with pretraining.

but what's the contribution of UTK real age images?

There is still correlation between real age and apparent age, thus increasing the number of training images might contribute to the accuracy.

What I can guess about the reason is that the APPA-REAL dataset are highly biased and not diverse enough about age distribution, therefore both training and testing don't work well. How do you think?

Training loss could be decreased in any case if training is successful. Please try changing optimizers and/or learning rate.

Hi yu4u, thanks a lot for your kind and patient reply. I have learned a lot from your code and replies.


typo and lang fix.

Thanks for the PR, "Pawtching" is an intentional misspelling, (See #3)
I think I will probably use color on the word to make it more clear that the misspell is intentional.
For .toLowerCase(Locale.ENGLISH), I think I know about which bug you are referencing.
I will use the same workaround as Recaf (see https://github.com/Col-E/Recaf/commit/7b991ef8f3ef7a87f3ed2df4f6a5943fc03f74ec ) as it will reduce the work to do to maintain the patcher.
Thank's for the help.
you can join our Discord if you wish to help or talk about KibblePatcher.

@Fox2Code I see, thanks.


Remove beta references

Removing beta references since the data connector is now GA.

@jdanielmyers @iamontheinet it looks like the press release for the Snowflake connector is going live today.  Can we merge this PR to remove the beta references for the Hasura data connector?

thanks for the pr - merged!


LeaderElectionManager invokes System.exit() directly

Bug Report
The LeaderElectionManager has a hardcoded invocation of System.exit(1) on stopLeading(), which is also a private method that cannot be overridden. This prevents me from managing the lifecycle of my own application, by installing a shutdown hook that invokes Operator.stop(), because a deadlock occurs when System.exit() is invoked by the LeaderElectionManager.
Even if I wasn't installing a shutdown hook, this would be undesirable behavior, because it prevents me from enabling leader election in unit tests.
What did you do?
I enabled leader election in my operator, where I also have a shutdown hook installed that invokes Operator.stop().
What did you expect to see?
I expected the Operator to shutdown cleanly.
What did you see instead? Under which circumstances?
The Operator hangs on shutdown.
Environment
Docker Desktop / Kubernetes 1.29.2
Java Operator SDK 4.9.2
$ java -version
openjdk version "17.0.10" 2024-01-16
IBM Semeru Runtime Open Edition <IP_ADDRESS> (build 17.0.10+7)
Eclipse OpenJ9 VM <IP_ADDRESS> (build openj9-0.43.0, JRE 17 Mac OS X amd64-64-Bit Compressed References 20240116_636 (JIT enabled, AOT enabled)
OpenJ9   - 2c3d78b48
OMR      - ea8124dbc
JCL      - 2aad089841f based on jdk-17.0.10+7)

$ kubectl version
Client Version: v1.29.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.2

Possible Solution
I worked around this by adding LeaderCallbacks to the leader election configuration that throw an exception before the System.exit() call can happen.
        var operator = new Operator(overrider -> {
            var callbacks = new LeaderCallbacks(() -> {
            }, () -> {
                // throw an exception to prevent hardcoded System.exit() invocation from
                // happening
                throw new RuntimeException(
                        "Preventing LeaderElectionManager.stopLeading() from invoking System.exit()");
            }, s -> {
            });
            var leaderElectionConfig = LeaderElectionConfigurationBuilder
                    .aLeaderElectionConfiguration(LEASE_NAME)
                    .withLeaderCallbacks(callbacks)
                    .build();
            overrider.withLeaderElectionConfiguration(leaderElectionConfig);
        });

This seems like a hack, but it works.

Hi @adriansuarez , it is true that this makes issues with test, the question is, what why would you like to stop it yourself explicitly? could you elaborate on what would you like to achieve?

@adriansuarez could you check if the PR helps your case?

@csviri Just approved it. Thanks for delivering a fix so quickly.

@adriansuarez thx, good to hear that it fixes it, note that this is currently for v5, that hopefully will be released next month. For this we could make a patch also for v 4.9.x if needed.

@csviri It's up to you. I can pick up the fix in v5 and use my workaround in the mean time.


what is the Swift 2.2 branch on Pod

Hey, you should be able to use 1.0.3 with Swift 2.2. Let me know if it works 👍

I added this line to my pod file:
pod 'PasswordTextField', '~> 1.0.3'
However it doesn't install this version, it installs 1.0.4. Am I missing something ?

Try
pod 'PasswordTextField', '= 1.0.3'

@bre7 , it works now. Thanks !! 👍

'~> 1.0.3' will use the most recent version before '1.1'

Thanks @bre7 👍


Enable Externalization in Swagger Springfox i.e. read @APIResponse messages from properties files

Externalization in Swagger Springfox i.e. read @APIResponse messages from properties files.
Problem statement.
@ApiOperation(value = "updateMe", notes = "Updates the retirement plan", responseContainer = "Object")
@ApiResponses(value = { @ApiResponse(code = 200, message = "Success"),
@ApiResponse(code = 400, message = "Bad Request", response = ReturnResponse.class),
@ApiResponse(code = 401, message = "Unauthorized", response = ReturnResponse.class),
@ApiResponse(code = 403, message = "Forbidden", response = ReturnResponse.class),
@ApiResponse(code = 404, message = "Not Found", response = ReturnResponse.class),
@ApiResponse(code = 500, message = "Internal server error occured", response = ReturnResponse.class) })
@RequestMapping(value = MoneyURIConstants.RETIREMENTPLAN_CONTRIBUTION, method = RequestMethod.PUT)
public ReturnResponse<?> updateMe(@PathVariable String employeeId, @PathVariable String company,
@RequestBody Employee, HttpServletRequest request);
Here I would like enable/add messages of api responses from property file.
like
@ApiResponses(value = { @ApiResponse(code = 200, message = "${get.success}"),

This is under implementation via this #1371

I have created custom plugins and using it from project. The properties file having key-values is created. but I don't know how to call it from Swagger Controller.

ControllerInterface- method
@ApiOperation(value = "get_contribution_api_name", notes = "get_contribution_api_notes", responseContainer = "List")
@DescriptorKey("get_contribution_api_notes")
@ApiResponses(value = { @ApiResponse(code = 200, message = "Success"),
@ApiResponse(code = 400, message = "Bad Request", response = Response.class),
@ApiResponse(code = 401, message = "Unauthorized", response = Response.class),
@ApiResponse(code = 403, message = "Forbidden", response = Response.class),
@ApiResponse(code = 404, message = "Not Found", response = Response.class),
@ApiResponse(code = 500, message = "Internal server error occured", response = Response.class) })
@RequestMapping(value = URIConstants.PLAN_CONTRIBUTION, method = RequestMethod.GET)
public Response<List> getContribution(@PathVariable String employeeId,
@PathVariable String company);

property reader class
@Service(value = "environmentConfig")
@Scope(value = "singleton")
@PropertySources
({
@PropertySource(value="classpath:/Project/swagger/api-plan-specs/src/resources/swagger-messages.properties",ignoreResourceNotFound=true)
})
public class EnvironmentConfig {
private static final Logger LOGGER = LoggerFactory.getLogger(EnvironmentConfig.class);
@Autowired
private Environment environment;
private static Environment env;
@PostConstruct
public void init() {
env = environment;
}
public static String getProperty(String key) {
LOGGER.debug("Getting property  :" + key);
return env.getProperty(key);
}
}

Properties file
swagger-messages.properties

#Swagger errors
get_contribution_api_name=getContribution
get_contribution_api_notes=Gets the list of plan

plugins

Validators

Please guide me here. I want to show property values from resource bundle.

@sac10nikam please don't cross post to unrelated issues.

Ok. Sorry
On Aug 20, 2016 12:42 AM, "Dilip Krishnan"<EMAIL_ADDRESS>wrote:

@sac10nikam https://github.com/sac10nikam please don't cross post to
unrelated issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/springfox/springfox/issues/1452#issuecomment-241109291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGOFOAvRpPDy8zC33twpogUqXZEkxr9Nks5qhgA5gaJpZM4JlRDl
.


[Event Request] Codeunit 1535 "Approvals Mgmt." - FindApprovedApprovalEntryForWorkflowUserGroup

Describe the request
Add OnBeforeFindApprovedApprovalEntryForWorkflowUserGroup event with IsHandled pattern to FindApprovedApprovalEntryForWorkflowUserGroup function in Codeunit 1535 "Approvals Mgmt.":
local procedure FindApprovedApprovalEntryForWorkflowUserGroup(var ApprovalEntry: Record "Approval Entry"; WorkflowStepInstance: Record "Workflow Step Instance"): Boolean
var
    WorkflowStepArgument: Record "Workflow Step Argument";
    WorkflowResponseHandling: Codeunit "Workflow Response Handling";
    WorkflowInstance: Query "Workflow Instance";
    IsHandled: Boolean;
    Result: Boolean;
begin
    OnBeforeFindApprovedApprovalEntryForWorkflowUserGroup(ApprovalEntry, WorkflowStepInstance, IsHandled, Result);
    if IsHandled then
        exit(Result);

    WorkflowStepInstance.SetLoadFields(Argument);
    WorkflowStepInstance.SetRange("Function Name", WorkflowResponseHandling.CreateApprovalRequestsCode());
    WorkflowStepInstance.SetRange("Record ID", WorkflowStepInstance."Record ID");
    WorkflowStepInstance.SetRange("Workflow Code", WorkflowStepInstance."Workflow Code");
    WorkflowStepInstance.SetRange(Type, WorkflowInstance.Type::Response);
    WorkflowStepInstance.SetRange(Status, WorkflowInstance.Status::Completed);
    if WorkflowStepInstance.FindSet() then
        repeat
            if WorkflowStepArgument.Get(WorkflowStepInstance.Argument) then
                if WorkflowStepArgument."Approver Type" = WorkflowStepArgument."Approver Type"::"Workflow User Group" then begin
                    ApprovalEntry.SetRange(Status, ApprovalEntry.Status::Approved);
                    exit(ApprovalEntry.FindLast());
                end;
        until WorkflowStepInstance.Next() = 0;
    exit(false);
end;

[IntegrationEvent(false, false)]
local procedure OnBeforeFindApprovedApprovalEntryForWorkflowUserGroup(var ApprovalEntry: Record "Approval Entry"; WorkflowStepInstance: Record "Workflow Step Instance"; var IsHandled: Boolean; var Result: Boolean)
begin
end;

Additional context
We need this to be able to implement additional "Approver Type" which has similar behaviour to Workflow User Group. Current events do not allow us to do it (missing required parameters + OnApproveApprovalRequest is local).
P.S. It seems that the usage of WorkflowInstance: Query "Workflow Instance"; is redundant, and it can be removed.
Internal work item: AB#506773

This event we wont add, as IsHandled is an anti-pattern.
Please change to event for verifying approver type instead of skipping logic, and create PR with code.


IOS14 cannot read exif data

It seems IOS14 cannot read the exif data of photo libary. It might cause by the new privacy system of IOS14. Does anybody have a solution ？
IOS14中的照片图库获取不到exif信息，可能是IOS14加强了隐私保护，有同行大佬有解决方案吗

You have to give more information. Are you getting no exif information at all? Or are specific fields empty?
Perhaps you should ask for the 'location' permission in your iOS app as well in order to get GPS lat and long. As well as permission for the photo's. I assume you have a hybrid app, using javascript since you're using exif-js.
You need to extend the info.plist file for this. Something like in below post dependening on if you're using phonegap or Ionic or the latter. Perhaps you need to remove platform ios and add it again in order to get this right. Since manually overwriting .plist file could be overwritten later.
https://stackoverflow.com/questions/1673579/location-permission-alert-on-iphone-with-phonegap

@weepy ###


Clean up globals logic to support CentOS 8 stream

This is a code simplification for distributions that don't change the
PostgreSQL version in a major release. As a bonus fact, it adds support for
CentOS 8 Stream which identifies itself without a minor version.
For completeness, the OS facts from CentOS 8 Stream:
# facter os
{
  architecture => "x86_64",
  family => "RedHat",
  hardware => "x86_64",
  name => "CentOS",
  release => {
    full => "8",
    major => "8"
  },
  selinux => {
    enabled => false
  }
}

This is an alternative to https://github.com/puppetlabs/puppetlabs-postgresql/pull/1222.

Codecov Report

Merging #1225 (3eb6299) into main (08e3a51) will increase coverage by 0.37%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##             main    #1225      +/-   ##
==========================================
+ Coverage   65.81%   66.18%   +0.37%
==========================================
  Files          14       14
  Lines         351      349       -2
==========================================
  Hits          231      231
+ Misses        120      118       -2

Impacted Files
Coverage Δ

...ns/postgresql/postgresql_acls_to_resources_hash.rb
0.00% <0.00%> (ø)

...b/puppet/functions/postgresql/postgresql_escape.rb
0.00% <0.00%> (ø)

lib/puppet/util/postgresql_validator.rb
90.90% <0.00%> (ø)

lib/puppet/type/postgresql_psql.rb
90.14% <66.66%> (ø)

...uppet/provider/postgresql_replication_slot/ruby.rb
75.86% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #1225 (3eb6299) into main (08e3a51) will increase coverage by 0.37%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##             main    #1225      +/-   ##
==========================================
+ Coverage   65.81%   66.18%   +0.37%
==========================================
  Files          14       14
  Lines         351      349       -2
==========================================
  Hits          231      231
+ Misses        120      118       -2

Impacted Files
Coverage Δ

...ns/postgresql/postgresql_acls_to_resources_hash.rb
0.00% <0.00%> (ø)

...b/puppet/functions/postgresql/postgresql_escape.rb
0.00% <0.00%> (ø)

lib/puppet/util/postgresql_validator.rb
90.90% <0.00%> (ø)

lib/puppet/type/postgresql_psql.rb
90.14% <66.66%> (ø)

...uppet/provider/postgresql_replication_slot/ruby.rb
75.86% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks @ekohl for your contribution as always.
Could you please take a look at unit tests which are failing with the second commit.

Thanks @ekohl for your contribution as always.
Could you please take a look at unit tests which are failing with the second commit.

Looks like I need to do a bit more.

Looks like I need to do a bit more.

I went through all files that used the full version. Also went through tests. It's confusing that this module places the files in spec/unit/defines rather than spec/defines as is the convention.

I went through all files that used the full version. Also went through tests. It's confusing that this module places the files in spec/unit/defines rather than spec/defines as is the convention.

:green_heart: now

:green_heart: now

Thank you @ekohl for the fixes.

Thank you @ekohl for the fixes.

@sheenaajay anything holding back a merge? I hope to see this in a released version soon.

@sheenaajay anything holding back a merge? I hope to see this in a released version soon.

@ekohl Sorry for the delay. Next one on the list :-) . We have a release coming soon for a bug fix and can get this also included. Thank you.

@ekohl Sorry for the delay. Next one on the list :-) . We have a release coming soon for a bug fix and can get this also included. Thank you.


Onedrive per machine installs

Would you please clarify if a machine is already on per user setup, does that version, even if it's already higher than suggested below have to be uninstalled first or can a new installation simply be run over the top using the /allusers switch.

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

ID: 087b8ed9-0b08-88a2-825f-727915801554
Version Independent ID: afd3bb47-f8a7-d1eb-a790-ff49c0794d2f
Content: Install the sync app per machine - OneDrive
Content Source: OneDrive/per-machine-installation.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@alanbmhs Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi Manuela,
I have received a reply from MS and they state that the Onedrive sync app can be run over the top of any existing per-user installation. I’ve now tested this and it does work. I now have another issue with the document in that it states in the first para that it can be installed in either Program File (x86) or Program Files depending on the architecture but then under the heading Deployment Instructions it states it will be installed in the x86 directory. So how do I install it in the Program Files directory on a 64 bit OS installation?
Alan.
From: Manuela Scanu<EMAIL_ADDRESS>Sent: Monday, 9 November 2020 17:54
To: MicrosoftDocs/OfficeDocs-SharePoint<EMAIL_ADDRESS>Cc: Alan Birch<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Onedrive per machine installs (#2425)
@alanbmhshttps://github.com/alanbmhs Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2425#issuecomment-723752747, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKXX3GQUHRUS556F5TLMS2DSO5YVXANCNFSM4TOUGV2Q.


API description

So as I saw there is an API from which this beautiful tool gets information. However, I was unable to find any info or source code for the API.
Is it private? Is it going to be open-sourced as well?
If it's private, then is it possible to have swagger documentation of endpoints?
I am asking due to having an idea of making an abstract class for API so that there could be other implementations in the project, apart from this private API!

So we had a Teams call, where @chilango74 described API being closed for a few reasons:

It uses secondary API, which could bring some legislation problems
It uses a few scrappers to get information about stocks

I was given access to the private repository where API lays, and it turns out that there is only integration with EOD which seems pretty legit to me. Indeed there are a few things to change still, such as secrets within the code, but I guess it would not be a problem and with the great cooperation of @chilango74 we would be able to open-source it!
So having that said, it's pretty obvious now that there could be different integrations. For instance, it could look like this:

API of @chilango74 which basically a proxy for EOD
EOD
Quandl

EOD allowed to use their data in an open-source API project. We will make okama-API code public after some refactoring.


Support magic arguments

Neither cell or line magics can accept parameters: they result in parse errors.

Is there a way to get my matplotlib inline whilst this issue is open?
%matplotlib inline
Using matplotlib backend: MacOSX
Magics with parameters are not supported

Not yet. The magics are not real python, so hy can't make ast for them. I
tried masking them in comments, but that didn't work either, as hy strips
comments.
The hy to ast to code approach is brittle as hell anyway. The real answer
will be to somehow wrap magics:
(%matplotlib inline)
As well as somehow handle cell magics that don't expect code. PRs or
suggestions welcome!
On 17:16, Sat, Jun 13, 2015 Steve Butcher<EMAIL_ADDRESS>wrote:

Is there a way to get my matplotlib inline whilst this issue is open?
%matplotlib inline
Using matplotlib backend: MacOSX
Magics with parameters are not supported
—
Reply to this email directly or view it on GitHub
https://github.com/bollwyvl/hy_kernel/issues/12#issuecomment-111753034.

Actually, just fixed this with #17, now on pypi as 0.3.0.
It breaks the code up around magics and compiles them as discrete chunks.
Still won't do inline magic!

sweet, just tried it out. Thanks!


Use smaller background image

Performance for an app this simple is very poor.  Partly down to the slow API you're using, but also the background image is huge - 3.4MB!

The issue is resolved with #6


feat(initializer): add userland templates to forge

Resolve local templates aswell as global templates, prioritizes global templates
Have you read the section in CONTRIBUTING.md about pull requests?
Yes
Are your changes appropriately documented?
Yes
Do your changes have sufficient test coverage?
Yes
Does the testsuite pass successfully on your local machine?
Yes
Summarize your changes:
Allow the init command to resolve templates from electron-forge and locally so we can package some templates with forge.

Welp this answers my question from the other PR. 🤷♂

Welp this answers my question from the other PR. :man_shrugging:

Quick reasoning for prepacking them

They are super lightweight (a few files with 0 npm dependencies)
It greatly reduces entry requirements (don't need to install multiple global modules just forge)

Wow cool. 😎


Update failing.

When running invoke update I get the error below. I'm not sure what additional arguments its looking for and I couldn't find anything in the docs for this.
https://inventree.readthedocs.io/en/latest/start/update/
root@server:/var/www/html/inventree# invoke update
'update' did not receive all required positional arguments!

That's a weird one. There should be no positional arguments required.
Are you up to date with the latest code?
Running invoke update works for me on latest master (checked just now)

All up to date. I think Il try a clean install and see what happens.

Installing a fresh copy didn't work on that server, it was running Ubuntu 18.04. I spun up a new Ubuntu 20.04 container copied over the original install and it worked fine there, so it was something with that container.

Might have been a weird version of invoke? Glad you got it working anyhow


update git ignore file

This PR will close #596 after accepting.

Coverage remained the same at 98.02% when pulling a881ccbf42d2623d85a60c1f7adbf84a6abac08b on amerllica:update-git-ignore into f70f4f6994381cc0234d1ad5c657b4973fbff17d on wellyshen:master.

@amerllica Looks great, thank you.


Conflicting import, imported name 'NavArgs' is ambiguous

Hi
after update version to 1.5.11-beta I got this error
e:****\ui\screen\NavArgsGetters.kt: (78, 100): Conflicting import, imported name 'NavArgs' is ambiguous
sample code :
@Composable
@Destination
fun T1(
    input : Array<P>
) {

}

import section in NavArgsGetter.kt file
import ***.ui.screen.destinations.T1Destination
import ***.ui.screen.destinations.T1Destination.NavArgs

On it, late night fixes are never a good idea 🤦♂
Expect a new version in the next few hours, I’ll update this issue.

New version available 1.5.12-beta!
Thank you so much @amirzeinaly for letting me know so quickly! 🙏

Thanks for your cooperation
All errors were fixed


Failing Markdown tests in build

Build that introduce this is IMO
https://monacotools.visualstudio.com/Monaco/_traceability/runview/changes?currentRunId=158523
With the problematic commits:
https://github.com/microsoft/vscode/commit/6d04ed55c7d5fde915210f8443da3d759e4add49
https://github.com/microsoft/vscode/commit/13bde28fe8d028b40f8b0c31079130ffe38c0180
The build got fixed here:
https://monacotools.visualstudio.com/Monaco/_build/results?buildId=158529&view=results
But from that on the markdown tests failed.

Made tests green again via https://github.com/microsoft/vscode/commit/30e1c0cfdaa3d1b3e7d4a0141bb8b0d3bcb51887
Not sure if that is the correct fix though.

@dbaeumer Thanks for jumping on this. There is definitely a change in my commit that wasn't intended - the escape regex should have been unchanged!


ROX-12750: Resource grouping - central part for ServiceIdentity, ScannerBundle and ScannerDefinitions

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #5075

PR #5091

PR #5092

PR #5093   👈

This comment was auto-generated by Graphite.


No parsing on Windows

So hallo erst mal... :smile:
No just joking, let's switch back to English: I've already replied in #6, but I think it's better to open a new issue about this so I can be sure everyone notices this.
In my case it still does not work in Atom, but if I call it manually...
See the gif for details:

Windows 7
2.3.0
Atom 1.0.6

@pfefferle Somebody there?
I've also another (possibly related problem). It seems there is a problem when loading libraries. It also shows a custom library I added and it shows me:
Warning: PHP Startup: Unable to load dynamic library 'D:\XAMPP\php\ext\php_<file>.dll' - Das angegebene Modul wurde nicht gefunden.

But in fact the DLL file exists and if I start php manually it does not show this message.
So I assume some environment variables are not set, because I get the same error if I execute php without the xampp_shell.bat.

Hmm, strange... I have tested it with Windows and XMPP and it worked... Will check my settings tomorrow and will post it here...

hey @rugk
I have tested it with several settings and it seems to work always:

with empty PHP executable path
using php.exe without path
using php.exe with path

You can try one more thing: Hit the right button on the .phar file and select "open with" and choose PHP.exe and try it once again.

...also try to use slashes / instead of backslashes \ in the paths.

Okay, now also my manual console mode does not work any more. So I ran it in the highest verbose mode and it showed me this output:
I
Legend: ?-unknown, I-invalid file syntax, file ignored, .-no changes, F-fixed, E
-error
Fixing time per file:
[0.064 s] D:\Foo.php

Note that I. As it states below the file was invalid. So that was my issue and I corrected it and then I could reproduce the issue again...
I tried both things you suggested but it did not work.

So I've modified the package to add -vvv for debugging (maybe with a (user-friendly) setting this would also be an idea to add to the package) and executed it like this.
And indeed it returned 'invalid' again:
I
Legend: ?-unknown, I-invalid file syntax, file ignored, .-no changes, F-fixed, E-error
Fixing time per file:
[0.020 s] D:\XAMPP\phplib\Foo.php

Fixed all files in 0.020 seconds, 4.000 MB memory used

But executing it on the command line works and just returns this:
F
Legend: ?-unknown, I-invalid file syntax, file ignored, .-no changes, F-fixed, E
-error
   1) D:\XAMPP\phplib\Foo.php (blankline_after_open_tag, single_line_after_impor
ts, operators_spaces, psr0, line_after_namespace, braces)
Fixing time per file:
[0.152 s] D:\XAMPP\phplib\Foo.php

Fixed all files in 0.152 seconds, 4.750 MB memory used

I've tried it several times now and the result is always the same. (Except that now Atom refuses to reload the changed file if I fix it with the command line)

Okay, I think I come closer: When I do not run the fix from the XAMPP shell, but use a normal CMD window I've get the same error.
So I tried to change the current directory to the PHP directory and executed the same command again. And... it worked. :smiley:
So I think that's what you have to do. (don't know why CS fixer otherwise thinks the file is invalid - possibly because PHP throws the error with my added library, which it tries to load but [somehow] cannot find if the dir is wrong)

Can you write that in a short chapter so I can add it to the readme? I think that also might help others ;)

I am no Windows user so I am not very helpful with this ;)

Well... I have not solved it. It was just an demonstration of the issue.
I mean you would have to add some code which changes the dir before the php cs fixer is executed.
I don't know if you can do this in JS in Atom so easily. Possibly you have to temporary change it and change it back after execution.
Or you could also try to add the PHP dir to the PATH environment variable (only modified inside of Atom) which would have the same effect.
Possibly I could of course also write a workaround to explain the user possibly has to add the php dir to PATH variable globally, so for all processes, but the other solutions would be better as they do not affect the whole system.

Hmm, I have tried it in a VirtualBox with Windows 8.1 and the latest XAMPP without any problems... Will try it again tomorrow to be sure though...

I already said that it may have to do with the PHP library (DLL) I added.
So unless you use this PHP may not throw an error always when cs-fixer tries to parse the scripts.

I'm encountering the same issue -- works fine via command line but no luck from Atom.

Okay, I added a small guide in #14.

@rugk I do have the php executable in my PATH - so I don't think that's my issue. Have a look at this screenshot from my dev tools console when I run the "fix" command:

I'm not entirely sure what the third log item is telling me. Any insights?

Can you try to add the full path to your php executable or use php.exe instead of only php?

@pfefferle I tried both, here is the console output:

Looks like the same result.

I do have the php executable in my PATH

You mean the dir of the PHP executable? As this was the thing which solved the problem in my case.

@rugk Yes, that's correct - I meant the path to the executable. I have C:\php (the location where php.exe lives) in the PATH.

@rjv
Try to change path to php-cs-fixer
Works:
C:\Users\Username\AppData\Roaming\Composer\vendor*fabpot\php-cs-fixe\php-cs-fixer*
Not working:
C:\Users\Username\AppData\Roaming\Composer\vendor*bin\php-cs-fixer*

@h7896v7347 Thank you for the recommendation. Switching the path to the fabpot directory did the trick. :+1:

@h7896v7347 @rjv
Maybe add this to the Readme?

I came here to comment what @h7896v7347 just did.
C:\Users\USERNAME\AppData\Roaming\Composer\vendor\ fabpot\php-cs-fixer\php-cs-fixer
makes this work on Windows. This is because "php-cs-fixer" in PATH from Composer points to "C:\Users\USERNAME\AppData\Roaming\Composer\vendor\ bin\php-cs-fixer.bat" which is a bat, and when php runs it just can't interprete it (even the php-cs-fixer exists in the same dir, but Windows reads the bat first).
C:\Users\USERNAME\AppData\Roaming\Composer\vendor\ fabpot\php-cs-fixer\php-cs-fixer is the phar itself and that's why it works

@pfefferle, I think you should add @h7896v7347 comment to the windows readme section. When you install with composer globally the PATH points to the bin folder which contains a bat file and not the phar which is where @h7896v7347 wrote.
Thanks!

will close this for now.


⚠ Bofrost.be has degraded performance

In 58c619d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1019 ms

Resolved: Bofrost.be performance has improved in 3dc2c08.


Can I extend the Tell Me page?

I would like to add functionality to the tell me page, example: search for a customer, a vendor, an invoice #...
Can AL Extension be created to add to the TELL ME page?

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

ID: 338187f3-3d4b-0e1e-4b32-809514eccf11
Version Independent ID: 37556d23-8b14-a8bf-ae3f-5ba85a725969
Content: Frequently Asked Questions about Tell Me - Business Central
Content Source: business-central/ui-search-faq.md
Service: dynamics365-business-central
GitHub Login: @bholtorf
Microsoft Alias: bholtorf

Hello @yannstlo
Thank you for your feedback. This is currently not possible, so you might want to suggest it or vote for an existing feature suggestion at https://aka.ms/bcideas.
Closing the issue as it's a feature request.


Unable to Access Intel-Optimized PyTorch Packages

We are having some trouble syncing changes and generating a PR in the notebooks repo. After investigating, it seems it's failing to install a requirement in notebooks repo.
Error:
ERROR: Could not install requirement intel-extension-for-pytorch==2.1.100 from [https://intel-extension-for-pytorch.s3.amazonaws.com/ipex_stable/cpu/./intel_extension_for_pytorch-2.1.100%2Bcpu-cp311-cp311-linux_x86_64.whl](https://intel-extension-for-pytorch.s3.amazonaws.com/ipex_stable/cpu/intel_extension_for_pytorch-2.1.100%2Bcpu-cp311-cp311-linux_x86_64.whl) (from -r /tmp/pipenv-ycsirlep-requirements/pipenv-8ax2iss2-constraints.txt (line 40)) because of HTTP error 403 Client Error: Forbidden for url: https://intel-extension-for-pytorch.s3.amazonaws.com/ipex_stable/cpu/intel_extension_for_pytorch-2.1.100%2Bcpu-cp311-cp311-linux_x86_64.whl for URL [https://intel-extension-for-pytorch.s3.amazonaws.com/ipex_stable/cpu/./intel_extension_for_pytorch-2.1.100%2Bcpu-cp311-cp311-linux_x86_64.whl](https://intel-extension-for-pytorch.s3.amazonaws.com/ipex_stable/cpu/intel_extension_for_pytorch-2.1.100%2Bcpu-cp311-cp311-linux_x86_64.whl) (from https://pytorch-extension.intel.com/release-whl/stable/cpu/us/intel-extension-for-pytorch/)
Known Issue: https://github.com/intel/intel-extension-for-pytorch/issues/745
Workaround suggested:
Use cn index url instead on all instances of this throughout the notebooks repo.

cc: @andyatmiami


add a pointer to community arch diag

Signed-off-by: Mike Ng<EMAIL_ADDRESS>This will point to the https://github.com/open-cluster-management/community/raw/main/assets/ocm-arch.png
which is being updated in https://github.com/open-cluster-management/community/pull/37 to include the newly presented policy.
FYI @juliana-hsu

/test build


Cap Lengths of description fields

Now we now what the design of the site will work, we should probably cap the lengths of some of the project fields so they fit within the page.
How longs should

description
title
workflow_description
introduction

be? @brian-c @DarrenMcRoy @heathv

This is a duplicate of https://github.com/zooniverse/Panoptes-Front-End/issues/758 -- close one or the other?

Design should fit the content, not the other way around. There are probably reasonable limits to put on these, but we shouldn't decide them based on what the site looks like right now.

agree we should have reasonable limits to stop essay's being written that the api has to serialize and the UI has to present somehow. Not sure what these are though as i'm sure edge cases will come along.

Based on data that exists in staging and production right now I'm going to do with the following limits:

Description: 300 Characters (longest is 220)
Title: 255 Characters (already the limit of the varchar field)
Workflow_description: 500 Characters (longest is 200)
Introduction: 1500 Characters (longest is 1200)

looks good to me.


ImportError: cannot import name 'APIClient' / running problem on python 3.6

Hi,
I was trying to use the test containers with django in order to set-up postgres db for tests. Unfortunately when I try to setup db according the documentation:
    postgres_container = PostgresContainer("postgres:9.5")
    with postgres_container as postgres:
        e = sqlalchemy.create_engine(postgres.get_connection_url())
        result = e.execute("select version()")

I receive the following error:
  File ".virtualenv/lib/python3.6/site-packages/django/conf/__init__.py", line 110, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/local/Cellar/python3/3.6.3/Frameworks/Python.framework/Versions/3.6/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "settings.py", line 14, in <module>
    from testcontainers import PostgresContainer
  File ".virtualenv/lib/python3.6/site-packages/testcontainers/__init__.py", line 1, in <module>
    from testcontainers.selenium import BrowserWebDriverContainer
  File ".virtualenv/lib/python3.6/site-packages/testcontainers/selenium.py", line 16, in <module>
    from testcontainers.core.container import DockerContainer
  File ".virtualenv/lib/python3.6/site-packages/testcontainers/core/container.py", line 3, in <module>
    from docker.models.containers import Container
  File ".virtualenv/lib/python3.6/site-packages/docker/models/containers.py", line 3, in <module>
    from ..api import APIClient
ImportError: cannot import name 'APIClient'

I'm using Python 3.6.3 and testcontainers 2.1.0. Do you know what could make a problem?
Thx

Hm, I really don't know. Could you try it with python 3.5? If it's the same i will try to dive into this problem

Hi, on python 3.5 look good (I've checked on python docker container so I had the problem with connection do unix socket but it seems good because there was a try to pull postgresql image).
The problem is that my application requires python 3.6 an higher (django).

Hm. need to look at it

@pnowy, you say I was trying to use the test containers with django in order to set-up postgres db for tests, however, your code example uses SQLAlchemy: sqlalchemy.create_engine(postgres.get_connection_url()).
Did you get testcontainters-python working with Django and can you describe how? Getting Django settings configured properly seems quite tricky to me.

@pnowy, you say I was trying to use the test containers with django in order to set-up postgres db for tests, however, your code example uses SQLAlchemy: sqlalchemy.create_engine(postgres.get_connection_url()).
Did you get testcontainters-python working with Django and can you describe how? Getting Django settings configured properly seems quite tricky to me.


Feature | Support for Send API V3.1

How's the progress coming along? If you need any help let me know!

How's the progress coming along? If you need any help let me know!

Hi @zmontgo!
I would really appreciate help and suggestions on this feature, I find the current architecture of this crate a bit difficult to scale and would prefer to re implement it.
The truth is that I created this crate when I was learning Rust and nowadays I would change many aspects from it.
I can open a Discussion provide input there and then move into actionable. I would close this PR and get started from a fresh state.
Let me know what do you think, thank you so much!

Hey!
I am learning Rust myself, so I look forward to learning from you!
I will say, if you feel the codebase isn't working well for you I'd opt to get started from a fresh state. I'd be happy to be engaged in that process.

Hey!
I am learning Rust myself, so I look forward to learning from you!
I will say, if you feel the codebase isn't working well for you I'd opt to get started from a fresh state. I'd be happy to be engaged in that process.

Im glad to have have your help in this crate. Of course, let me open a discussion so we can put our ideas together and then define tasks to achieve what we want to build.
I will ping you later today via repository discussions.
Thank you so much!

@EstebanBorai sounds great, thanks! I look forward to it and already have quite a few ideas as to what the client may look like. I think I've come up with an intuitive, elegant, and pragmatic solution.


hain-plugin-math: Rounding issue

Hello,
There seems to be a rounding issue with the built in calculator.
Steps to reproduce

Open Hain
Type =0.1+0.2 into the search bar

Expected behavior
I would expect for the result to be 0.3
Actual behavior
The result is showing up as 0.30000000000000004

Thanks!

Welcome to the strange world of floating point arithmetic on computers...
Not that we won't look into it though, that is an odd one.

Looks like the "fix" is to use toFixed. I'm not sure the best way for you to determine how many decimal places to show however.
http://stackoverflow.com/questions/10473994/javascript-adding-decimal-numbers-issue
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/toFixed


sched/tcb: add preserved tcb flag to support static tcb

Summary
sched/tcb: add preserved tcb flag to support static tcb
Add support for static tcb, applications in some special case can
initialize system resources in advance through static tcb.
|  static struct task_tcb_s g_tcb;
|
|  memset(&g_tcb, 0, sizeof(struct task_tcb_s));
|  g_tcb.cmn.flags = TCB_FLAG_TTYPE_KERNEL | TCB_FLAG_PRESERVED_TCB;
|  nxtask_init(&g_tcb, "PTCB", 101, NULL, 1024, ptcb_task, NULL, NULL, NULL);
|
|  ...
|  nxtask_activate(&g_tcb.cmn);

Signed-off-by: chao an<EMAIL_ADDRESS>Impact
N/A
Testing
sim/nsh

Let's ignore ci break which https://github.com/apache/nuttx/pull/11833


XLIFFs from txt2AL: Trans-Unit sections do not always align

In the generated XLIFFs by the txt2AL tool (here en-US and de-DE) there are a few places where the trans-unit sections to not align like they normally do, but are slightly moved with always one other trans-unit in between, so at first glance it looks as though there is a missing translation in the other xlf file. What is the reason for this? Bug or feature?

Duplicate of #3026

This is not a duplicate of #3026. Here one unit with same trans-unit-id is placed in a different position in the file compared to the other. In #3026 there are different trans-unit-ids for the same source in the app
It may have a common cause, but it is not the same behaviour.

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 25582.
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/