Add Japanese translations for Node-RED v1.3.0

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
I added the additional Japanese translations for Node-RED v1.3.0.
English message, "successfully installed" in the externalModules.js was changed to "Installed module" without adding the new message into the catalog in this commit because I thought that these are for almost the same use.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 67.497% when pulling a6af0d3176ac4e80cf6eb305a8b2c873c6c1218a on kazuhitoyokoi:dev-jpn into 827f8d4d518f8fc16b84e6ab91d3ec71405eee88 on node-red:dev.

@dceejay Thank you!


Add solution for Day 3 in Swift language

@kristoft , this too failed. request to update this too.

also Day3 is assigned to @penelopearaujo

Closing since it assigned to another person


Node Functions container - install azurite globally

Why isn't azurite already in the container? I looked with npm list -g and it isn't there but I need azurite to run the container locally don't I?
Relates to: <Codespaces | Remote - Containers | Both>

cc @kulkarnisonia16 on expected contents for a Functions container/environment.


Decimal number accuracy isn't automatically provided

Description of issue:
When submitting a new Market Order. OANDA Rejects order due to incorrect decimal number accuracy.
"rejectReason":"TAKE_PROFIT_ON_FILL_PRICE_PRECISION_EXCEEDED"
Current behavior
async_v20 doesn't round numbers to the instruments accuracy. It currently rounds all decimal numbers to 5 decimal places regardless of what the instrument defines
Expected behavior
async_v20 should ensure that all decimal numbers sent to OANDA comply with the instruments accuracy

I have added a new helper function _in_context to format an OrderRequest per an instruments specification. And also added a corresponding argument to OandaClient to control the behavior of this formatting. See documentation
In order to support this. The initialization step now also retrieves all available instruments for an account. (More reason to to initialize first)
The improvement has made an additional class Unit that I defined for the purpose of managing decimal number precision redundant. And has been removed. Admittedly I previously failed to understand the relationship between Instruments and OrderRequests.
Although this addition doesn't strictly break the API. I feel there may be nuanced behavior implications I cannot predict. (Maybe someone was relying on the rounding of all decimal numbers to 5 places for other purposes). I believe i is also a massive improvement to the library. I will be bumping the major version from 2 to 3 for these reasons

Version 3.0.0b1 is on PyPI containing the above changes


Add missing migration for webhooks

Motivation and context

It seems that a migration was done in #4863 , but it was not added to the repository. This PR adds the missing migration, which is generated by python manage.py makemigrations.

Changed the webhook model status_code field type

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Added migration quite similar with this migration.
And the reason why the lists of available values differ is because of the version of Python being used. Because for different python version http.HTTPStatus can differ.
So I think we should make WebhookDelivery.status_code independent from Python version.
And for now I see only one way to do it: hardcode all possible status codes. @zhiltsov-max what do you think about it?

I think we can just store an integer without extra restrictions.

/check

/check


How do you get environments to work?

how do you get Environments to work when using renderMathInElement()?
Example if I write this:
\begin{Bmatrix}
   a & b \\
   c & d
\end{Bmatrix}

I just end up with this in the browser:
\begin{Bmatrix} a & b \\ c & d \end{Bmatrix}

That should work.  (It works on the demo page, for example.)  We'd need a more complete example of the HTML/JavaScript to help you debug.  I doubt it's related to environments specifically, but rather just not calling renderMathInElement() correctly.  (Did you check the console?)

@BenjaminHoegh , If you are not using the Auto-render extension, then the syntax you need is katex.render, not renderMathInElement
If you are using the Auto-render extension, then keep in mind that it always looks for math delimiters like $$...$$ or \[...\].  This is unlike LaTeX or MathJax, in which an environment is by itself enough to initiate math mode. The Auto-render extension needs the delimiters, even for environments.

@BenjaminHoegh I'm not sure we can do that, as \\ is valid LaTeX.  The underlying problem is that we don't yet support \\; see #171.  I'm reasonably hopeful that we'll be able to add support for \\, though, via #1287.


[Modules] Updated Service Fabric Clusters to new dependencies approach

Description
Updated Service Fabric Clusters to new dependencies approach
Close #2244
Pipeline references

Pipeline

Type of Change
Please delete options that are not relevant.

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update to documentation

Checklist

[ ] I'm sure there are no other open Pull Requests for the same update/change
[ ] My corresponding pipelines / checks run clean and green without any errors or warnings
[ ] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (readme)
[ ] I did format my code

Commited suggested changes, and updated readme. Ready for approval


upgrade '--all' and '--accept-package-agreements' flags should work together

Description of the new feature / enhancement
I tried to use the command winget upgrade --all command but after the package repos were successfully downloaded, it aborted because the package-agreements werent accepted. But I was not asked either if I tried to upgrade all. So I tried to append the --accept-package-agreements flag, but assumingly this won't work in combination with --all.
The annoying part is that it abborts the whole proccess of upgrading all packages and wont continue. So I have to manually upgrade all packages with included package-agreements in order to accept it, and then use --all to upgrade all others.
Without using both flags in combination it makes the whole purpurse of --all obsolete because it wont run no matter if you want or wont want to accept the package agreements
Proposed technical implementation details
I want the command winget upgrade --all --accept-package-agreements to work and upgrade all packages and automatically accept the agreemets, if nessecary.

Just had the same issue, one program wouldn't upgrade with "Package agreements were not agreed to. Operation cancelled."
"The arguments provided can only be used with a query." with "winget upgrade --accept-package-agreements --all"
Ended up just uninstalling the software that was causing issues and upgraded the rest.

Yeah, this is not good for automation. Also make sure that --accept-source-agreements is also compatible.

I have this problem when I install using Windows Store source, because I am prompted to accept the agreement upon install and every update.
winget --all will fail with:
Package agreements were not agreed to. Operation cancelled.

The workaround has been to first use winget or Store app to update the store sourced package(s), then winget --all will update the remaining winget sourced packages that I have.

Just adding that this definitely needs to be fixed:
winget upgrade --all --accept-package-agreements --accept-source-agreements
What's the point in having an upgrade --all flag if the flag won't even work?
If I can do an rpm -Uvh or apt-get -y upgrade on Linux, I should be able to do an winget upgrade --all --accept-package-agreements --accept-source-agreements as well.

I will bump this as well. This must be fixed ASAP.

This is a rather serious UX issue, with what appears to be an easy fix.
Would be great to get some ETA update/feedback from the MS Team.

as a workaround you're able to enter the Y automatically by piping it in powershell
ECHO Y | winget.exe" upgrade --all --silent

ECHO Y | winget upgrade --all --silent

Didn't work for me.
This should really be fixed. Either allow --all --accept-package-agreements --accept-source-agreements, or add a separate command like winget agreements --accept-all that flags any upgrades with "accepted" license terms so that the upgrade can proceed.

Supplying a dummy package search query works for me
winget upgrade --accept-package-agreements --all <any dummy string>
I've just upgraded all the packages with winget upgrade --accept-package-agreements --all 1

ECHO Y | winget upgrade --all --silent

Didn't work for me.
This should really be fixed. Either allow --all --accept-package-agreements --accept-source-agreements, or add a separate command like winget agreements --accept-all that flags any upgrades with "accepted" license terms so that the upgrade can proceed.

Looks like I missed a little off the command I used. It's actually
ECHO Y | cmd /c winget upgrade --all --silent --include-unknown

ECHO Y | winget upgrade --all --silent

Didn't work for me.
This should really be fixed. Either allow --all --accept-package-agreements --accept-source-agreements, or add a separate command like winget agreements --accept-all that flags any upgrades with "accepted" license terms so that the upgrade can proceed.

Yeah looks like I missed a couple a bit. In powershell this works for me
ECHO Y | cmd /c winget upgrade --all --silent

Does this merge mean it's been fixed?

Does this merge mean it's been fixed?

seems it has been released in preview version - https://github.com/microsoft/winget-cli/releases/tag/v1.5.101-preview
"Copy install behavior flags on upgrade --all by @florelis in https://github.com/microsoft/winget-cli/pull/2794"
waiting for release version. don't wanna mess with pre-release version personally

PR Merged is just an indication that code has been merged in "main" branch (assuming that's the target).
Releases have links to included PRs (in this case it's in a preview release).
If the work has been merged for the scope indicated in the issue, then the issue can be closed.
A closed issue is not an indication that the "fix" or "feature" is in a release yet.
For this issue, additional work to be able to close it is in progress with:

https://github.com/microsoft/winget-cli/pull/2862

Once that work is complete and verified, this issue will be closed.
It will not be available (other than in a developer's individual build) until we cut our next "preview" release after it has been merged.
When we cut the next "stable" release after the work is in a "preview" release, then it will be GA (Generally Available).
This is being tracked in our 1.5 milestone.

I'm trying to use the command winget upgrade --all --accept-package-agreements --accept-source-agreements but  appear another error: The arguments provided can only be used with a query. How do I resolve this?

We've validated the referenced PR addresses these concerns.
This fix will be available in our next preview release after 1.5-preview(1).

winget upgrade -u -h -r --accept-source-agreements --accept-package-agreements
Is there something wrong with the above command? it gives me this error:
"The arguments provided can only be used with a query."
If I remove the last two agreement commands I get this error:
"Package agreements were not agreed to. Operation cancelled."

No package has been specified for the upgrade. If you want to upgrade all packages add "--all".

Can confirm that the latest preview release does honor the switches.  This was my Powershell:
Add-AppxPackage c:\temp\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle
winget upgrade -h --all --accept-package-agreements --accept-source-agreements
I do have a question though.  I thought WinGet was supposed to update all Store Apps with the -all switch?  When I open the Microsoft Store app on Windows 11, it shows more stuff.  Example is on a newly upgraded machine from Windows 10, the Snipping Tool has an updated version, but the winget command above does not find it and update it.  I have to do that manually via the store.  I've tried running the command with Administrator and User context.  Is there a reason for this?  Just some examples in my attached snip.

Try winget upgrade -uhr --accept-source-agreements --accept-package-agreements
Any Microsoft Store packages installed with WinGet will show version "Unknown", and Microsoft Store packages not installed with WinGet will not be included. We have future work to address this.
The source and package agreements are only necessary for the Microsoft Store source.

So how do we deal with this in the meantime?  What is the scripted way to update all Store apps on a computer?  We want to do this on a recurring basis with our monthly application patching process like we do with other apps.  All of the forums I read say "use winget for Windows 11" but this appears to not yet be supported.

And btw, this -uhr switch appears to update our Microsoft Office 365 Apps for Enterprise, which we DO NOT want.  I find this bizarre that winget would somehow manage this when we deploy it via ODT.  Do you also have a command line for exclusions in the works?

So how do we deal with this in the meantime? What is the scripted way to update all Store apps on a computer? We want to do this on a recurring basis with our monthly application patching process like we do with other apps. All of the forums I read say "use winget for Windows 11" but this appears to not yet be supported.

I was wondering about this, too. Has been an issue especially with per-user apps and not being able to upgrade the Store apps without having to open the Store manually. Hopefully this could be rolled up into Winget maybe so they can get upgraded at the same time - handy for auto-maintenance/scripting/new computer builds/imaging etc!

Bump. This needs to be fixed! A real package manager should require no user input when directed to do so.  There should be UAC bypasses for these updates as well as this is core security SOP..

I am not agreeing with the UAC bypass tough. I run my command prompt as admin if necessary. That will bypass it but you need to keep in mind that it will run every installer as admin.

That gets tricky... some installers change the location depending on how they are run (e.g. non-elevated goes to %localappdata% and elevated goes to %programfiles%). So if you elevate your terminal, you may see a difference in behavior (possibly unwanted), but if you don't elevate it, your command may not succeed.

Just to bring this topic back around.  The original request was for the cli to update all installed store apps when it currently does not.

@brinham I'd suggest creating a new issue, but I can share:
We need to get the version information from the Microsoft Store REST source so we know when there are available updates to those packages. Currently, we only get "Unknown" version for UWP and Win32 packages. Some work has been done to get version for Win32 packages, but not in all the endpoints, and sadly the one we need for version comparison still renders "Unknown". We're working with them to unblock the scenario.

Any updates? I still get that error.

still get this error, still seemingly no way to accept all the agreements.

For some reason, I still need to explicitly accept *.
If you are PowerShell, came up with this workaround,
Edit PowerShell profile
code $PROFILE
# or
notepad $PROFILE

Add the following function,
# Short command to upgrade all the installed packages
function Winget-Upgrade-All() {
    winget upgrade --all --accept-source-agreements --accept-package-agreements
}

Execute from PowerShell
Winget-Upgrade-All

winget upgrade --all --accept-source-agreements --accept-package-agreements   is not intuitive at all
Winget upgrade --all should at least upgrade the ones that can and just ignore the ones that can't rather than failing the entire task

This command work:
Winget upgrade --all --accept-package-agreements

--all should be the implication of acceptance   there should be a flag to reject agreements
99.99999% of people are going to accept due to the fact that they are upgrading software already on their machines the burden should be on the minority to flag not for the majority
it makes the software harder to use and isn't immediately obvious to non experts
additionally several tutorials already exist in the wild that give the --all command


Add is null support

All in good time! I'm mostly just trying to get https://github.com/brandsExclusive/svc-payment/pull/247 working.


Separate model_method.py into numerical_settings.py and model_method.py

Closes #19
@JFRudzinski @ndaelman-hu be aware of this. It will not affect you, but you will see that I separated the NumericalSettings classes into their own module.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
8

Relevant Lines:
12

💛  - Coveralls


theme: showVisitedLinks not working

First of all thank you @McShelby for sharing this theme with the world without asking for something in return. I am new to Hugo and I am not quit sure if it is a bug or a misconfiguration on my part.
As I saw the Hugo Learn theme does provide "showVisitedLinks" as follows:

This behavior seems not to be the case on Hugo Relearn:

If I understand the setup correct, then the code of the theme above is as follows:
[params]
  showVisitedLinks = true

See also https://github.com/McShelby/hugo-theme-relearn/blob/main/exampleSite/config.toml
Am I on the wrong track or should the tick appear when I have showVisitedLinks = true?

You're right, it should appear if you set showVisitedLinks = true. So  you've found a bug. ;-)

This happens at least if you set relativeURLs=true in your config.toml (but probably also in this case matcornic/hugo-theme-learn#468) . This is the case here with the theme's exampleSite.

Thank you for fixing this quickly. Now it works like a charm :checkered_flag: Very appreciated :pray:


How to run this image as NonRootUser in k8s ?

We want to use this image in our on-prem k8s deployment. In our on-prem k8s cluster we can't run pods with root privileges. Is there a way to make this docker image run as NonRootUser ?
@hopsoft

I had been trying this out lately and it is no executable as non root user. @pawanlucky1810 Did you get any insight yet , how to get it running as non root user?

@hopsoft Any insight on this would be really helpful


SPIKE: a simple setup for getting user info using an auth code

Closing, but this will still be here for reference


How to Enable GPU Computation in Python Wrapper

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)

Platform
Notebook (Lenovo Y740)

SDK Version
v2.25

Language
python

Segment
deploy in my own notebook

Issue Description
I mainly use your python wrapper for my own development. I installed sdk 2.0 here (https://www.intelrealsense.com/developers/).
I found that the current SDK viewer and the python wrapper only consume my CPU power, and I want to leverage my GPU computation. How can I enable GPU computation in python wrapper?

RealSense SDK 2.0 supports CUDA for graphics processing enhancement.  If your GPU supports CUDA, you can use CMake to build Librealsense with CUDA enabled by using the build customization flag BUILD_WITH_CUDA which is set to False by default.
https://github.com/IntelRealSense/librealsense/wiki/Build-Configuration#build-customization-flags

An alternative to CUDA is GLSL processing blocks, which were added in SDK version 2.22.0.
https://github.com/IntelRealSense/librealsense/pull/3654


chore(gha): comment on PR about reviews

Implements https://www.notion.so/sourcegraph/Set-up-a-restriction-on-PRs-to-require-approval-for-merging-89710798660744e8a84dab75a17deb85?pvs=4

@jhchabran what do you think of...

[!WARNING]
As we're migrating to Notion, we want to make sure that changes to the handbook are absolutely necessary. To ensure it is the case, it's now mandatory to get an approving review to merge a pull request.

Examples

Setting up a redirection to a Notion page: PR will be approved.
Correcting an important mistake on a page that doesn't have an equivalent yet: PR will be approved.
Updating content that isn't mission critical: PR will be rejected.

Sourcegraphers: Reach out to #wg-notion if you have any question.


Remove or resolve TODO 3.0 in OSSL_DECODER_from_bio.pod and OSSL_ENCODER_to_bio.pod

There is a TODO 3.0 in regards to missing examples in both manual pages.

The same is true for OSSL_DECODER.pod. I think it can be covered by this issue as well.


Setting publishDate = [":filename"] will cause hugo new content to fail with future dates

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.121.2-6d5b44305eaa9d0a157946492a6f319da38de154 darwin/arm64 BuildDate=2024-01-05T12:21:15Z VendorInfo=gohugoio

Does this issue reproduce with the latest release?
Yes
Setting the date to today is fine, setting the date to something in the future breaks.
Here's how to reproduce (terminal session in fish shell):
~/Downloads$tar xvfz hugo_0.121.2_darwin-universal.tar.gz
x hugo
x README.md
x LICENSE
~/Downloads$./hugo new site test
Congratulations! Your new Hugo site was created in /Users/justusperlwitz/Downloads/test.
[...]
~/Downloads$cd test
~/D/test$../hugo new content content/posts/2024-01-26-(random).md
Content "/Users/justusperlwitz/Downloads/test/content/posts/2024-01-26-10004.md" created
~/D/test$echo '[frontmatter]
         publishDate = [":filename"]
         ' > hugo.toml
~/D/test$../hugo new content content/posts/2024-01-26-(random).md
panic: [BUG] no Page found for "/Users/justusperlwitz/Downloads/test/content/posts/2024-01-26-7265.md"

goroutine 1 [running]:
github.com/gohugoio/hugo/create.(*contentBuilder).applyArcheType(0x140000f95c0, {0x140000545a0, 0x45}, {0x140009adad0, 0xa})
        /root/project/hugo/create/content.go:278 +0x238
github.com/gohugoio/hugo/create.(*contentBuilder).buildFile(0x140000f95c0)
        /root/project/hugo/create/content.go:246 +0x16c
github.com/gohugoio/hugo/create.NewContent.func1()
        /root/project/hugo/create/content.go:105 +0x24c
github.com/gohugoio/hugo/create.NewContent(0x14000993cc0, {0x0, 0x0}, {0x16b2e6984, 0x20}, 0x0)
        /root/project/hugo/create/content.go:109 +0x498
github.com/gohugoio/hugo/commands.newNewCommand.func1({0x0?, 0x0?}, 0x0?, 0x0?, {0x14000795c50, 0x0?, 0x0?})
        /root/project/hugo/commands/new.go:60 +0x160
github.com/gohugoio/hugo/commands.(*simpleCommand).Run(0x0?, {0x108232a18?, 0x109510780?}, 0x0?, {0x14000795c50?, 0x0?, 0x0?})
        /root/project/hugo/commands/commandeer.go:588 +0x48
github.com/bep/simplecobra.(*Commandeer).compile.func1(0x140007c4500?, {0x14000795c50?, 0x4?, 0x1065db95c?})
        /root/project/gomodcache/github.com/bep/simplecobra@v0.4.0/simplecobra.go:113 +0x54
github.com/spf13/cobra.(*Command).execute(0x140008cf800, {0x14000795c10, 0x1, 0x1})
        /root/project/gomodcache/github.com/spf13/cobra@v1.8.0/command.go:983 +0x840
github.com/spf13/cobra.(*Command).ExecuteC(0x14000023800)
        /root/project/gomodcache/github.com/spf13/cobra@v1.8.0/command.go:1115 +0x344
github.com/spf13/cobra.(*Command).ExecuteContextC(...)
        /root/project/gomodcache/github.com/spf13/cobra@v1.8.0/command.go:1048
github.com/bep/simplecobra.(*Exec).Execute(0x140005ced18, {0x108232a18?, 0x109510780?}, {0x1400004c0d0?, 0x1400008a648?, 0x104b202ec?})
        /root/project/gomodcache/github.com/bep/simplecobra@v0.4.0/simplecobra.go:155 +0xb4
github.com/gohugoio/hugo/commands.Execute({0x1400004c0d0, 0x3, 0x3})
        /root/project/hugo/commands/commandeer.go:65 +0x284
main.main()
        /root/project/hugo/main.go:25 +0x70
~/D/test$../hugo version
hugo v0.121.2-6d5b44305eaa9d0a157946492a6f319da38de154 darwin/arm64 BuildDate=2024-01-05T12:21:15Z VendorInfo=gohugoio

Here's a quick workaround:
hugo new content --clock 2030-12-31T15:00:00Z content/posts/2024-01-30-(random).md

This will take care of the issue until 2030 👍  adjust for 2040, 2050 and so on if you live further in the future (hi!!)


Consider bundle statinamic into a giant file when publishing

I would love to avoid this kind of situation in the future.
<EMAIL_ADDRESS>

One downside (not mentionned) are patches. Semver helps you to have patches and new features automatically, even for deps, or deps of deps.

Yup. But many people don't follow semver. Patches can break stuff (remember babel releases?  )

Obviously, it was unintentional bug that has been fixed in a couple of hours. How many bugs have been fixed without us noticing those bugs? We don't know.

This approach has it cons and pros. Instead of doing this,  I will try reduce the number of packages that statinamic depends on (like underscore.string,  only use 1 method of it).
When I'm looking for the posibility of implementing this. I wonder why did you encourage using modules in the first place ? (Like statinamic/lib/* ).

I will try reduce the number of packages

I am always trying to avoid a package when I can.

When I'm looking for the posibility of implementing this. I wonder why did you encourage using modules in the first place ? (Like statinamic/lib/* ).

Sorry I don't understand what you want to mean with this sentence.

I mean why not export everything in index.js instead of throwing an error ?

This is to avoid client side script to have all statinamic modules.
But sure I will revisit this thing when api become stable.


Fix bugs in slack computation of qubo_general01

I noticed that qubo_general01 raises ValueError: negative dimensions are not allowed  or returns incorrect results in several cases.

@msakai Thanks for this PR!
I'm investigating the reason of the traivs failure.

I guess that qbsolv failed to find the optimal solution by chance because of its non-deterministic nature.
I ran the job again and then it succeeded.
https://travis-ci.org/msakai/qubogen/builds/531338944

@msakai use random seed for qbsolv in test (9a458402106bfe6cf1ddd2f3d1354b9373c02b62).

@msakai Merged, thank you very much!


resize photos on mergin

has photos removed from ignore_mobile/photos and moved to ignore_mobile/photos_resized.  Not sure we need to store the photos here at all but at least they are smaller now.  Could probably have changed in place but not sure that the mergin server could handle manageing a change that large with all the same filenames so didn't risk a glitch

mergin version V60


cp: proxifier.template: No such file or directory

I am getting error
cp: proxifier.template: No such file or directory
After executing tethery.sh. I am using the trial version of Proxifier v2.17. The Proxifier app opens fine after executing the script but this guide is not working so I believe my issue is that proxifier.template is not being copied correctly in the script.
Has anyone else had this issue? I am on OS X El Capitan 10.11.6.
I will poke around in the tethery.sh file to see if I can fix my issue. I'll keep you posted. Please let me know if you need any more information regarding this issue.

I fixed the issue by changing the path from proxifier.template to ~/Downloads/tethery-master/proxifier.template

You should run the script from the directory it's in.  Sorry I didn't see this issue before, not sure why it didn't alert me.


createInvalidArgument function, other renames

In #3666 we're tossing an error because --compilers can no longer be used.  But we're using createInvalidArgumentValueError, which doesn't seem to be appropriate based on that function's documentation.
It looks like what we want is a createInvalidArgumentError, where the argument itself is disallowed (I'm going to create a PR for this imminently).
Somewhat related--this probably wants to be a separate PR--I'm thinking in these function names, "argument" should be replaced with "option", because what we're really dealing with are options.  They can be command-line arguments, but command-line arguments are just a means to configure options.  The same error should be thrown whether --compilers was used on the command-line, or .mocharc.json has a compilers option.
cc @vkarpov15

closed by 186ca36.

any other related refactors should be handled in a diff issue and PR


feat: add @rise-tools/kit-forms

Adds new Form utilities on top of Tamagui, suffix: "Field".
Also, moves SliderField, SwitchField and SelectField from "kit" to "kit-forms" (will be re-exported later anyways for clarity)
Groups all examples under single "Form"
Removes use of experimental local state as its no longer required to support forms in the app
Validation will be provided in a follow-up.

I have patched tamagui with patch-package and ignored one warning. Issues are either reported or pending PR merge/release in Tamagui. Created issue to track updates in the future and don't forget to get back to it!


Add content to Client's Quality section

Topics covered: testability, interoperability and efficiency
Bug: T274530

I find this section strange in general. Given that we already had trouble filling in the main quality goals in section 1 makes it hard to fill in this "other slightly less important quality goals" section here. What do you think about splitting it into "external quality" and "internal quality", and leaving external quality empty for now to be filled in with the help of a PM?

yep, splitting it like that makes sense. :+1:


vivisect 1.0.0 extracts less features than vivisect 0.1.0

971e599e6e707349eccea2fd4c8e5f67.exe_ and dc4dc746d8a14060fb5fc7edd4ef5282.exe_ extract less features in Python 3 (with vivisect 1.0.0 and 1.0.1) than with Python2 and vivisect 0.1.0 (not 0.2.0). From https://github.com/fireeye/capa/pull/421:
insn: py2(  841858) py3(  528575) 62.78% - 971e599e6e707349eccea2fd4c8e5f67.exe_
bb:   py2(   12149) py3(    7498) 61.71% - 971e599e6e707349eccea2fd4c8e5f67.exe_
insn: py2(    4694) py3(    3133) 66.74% - dc4dc746d8a14060fb5fc7edd4ef5282.exe_
bb:   py2(     461) py3(     368) 79.82% - dc4dc746d8a14060fb5fc7edd4ef5282.exe_

We should take a closer look at this, as it could be a bug introduced in vivisect 0.2.0 or 1.0.0.
Note that there are many other binaries for which vivisect 1.0.0 extracts more features than 0.1.0. Check https://github.com/fireeye/capa/pull/421#issuecomment-786111162 for details.

can we close this?

yes 😄


FileNotFoundError: [Errno 2] No such file or directory: '/app/xtts_api_server/models/v2.0.2/config.json'

So, I'm basically trying to deploy xtts but I get the error below:
FileNotFoundError: [Errno 2] No such file or directory: '/app/xtts_api_server/models/v2.0.2/config.json'
Here's my full log:
docker compose up
[+] Running 0/1
 ⠿ xttsapiserver Error                                                                                2.4s
[+] Building 751.2s (13/13) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.3s
 => => transferring dockerfile: 1.16kB                                                                                0.1s
 => [internal] load .dockerignore                                                                                0.3s
 => => transferring context: 2B                                                                                0.1s
 => [internal] load metadata for docker.io/nvidia/cuda:12.1.0-runtime-ubuntu22.04                                                                                1.8s
 => [stage-0 1/8] FROM docker.io/nvidia/cuda:12.1.0-runtime-ubuntu22.04@sha256:402700b179eb764da6d60d99fe106aa16c36874f7d7fb3e122251ff6aea8b2f7                                          0.0s
 => [internal] load build context                                                                                0.5s
 => => transferring context: 5.67MB                                                                                0.4s
 => CACHED [stage-0 2/8] RUN --mount=type=cache,target=/var/cache/apt,sharing=locked,rw apt-get update &&     apt-get install --no-install-recommends -y python3-dev portaudio19-dev li  0.0s
 => CACHED [stage-0 3/8] RUN --mount=type=cache,target=/root/.cache/pip,rw pip3 install virtualenv                                                                                0.0s
 => CACHED [stage-0 4/8] RUN --mount=type=cache,target=/root/.cache/pip,rw     pip3 install --upgrade pip setuptools wheel ninja &&     pip3 install torch torchvision torchaudio xform  0.0s
 => CACHED [stage-0 5/8] RUN mkdir /app                                                                                0.0s
 => CACHED [stage-0 6/8] WORKDIR /app                                                                                0.0s
 => [stage-0 7/8] ADD . /app                                                                                0.8s
 => [stage-0 8/8] RUN --mount=type=cache,target=/root/.cache/pip,rw     pip3 install /app                                                                                741.0s
 => exporting to image                                                                                6.7s
 => => exporting layers                                                                                6.7s
 => => writing image sha256:572578d8657a5e9f6eb1232f92dd7ebbf34f3a585cddc6db08cc6f0b909f6147                                                                                0.0s
 => => naming to docker.io/library/xttsapiserver                                                                                0.0s
[+] Running 2/2
 ⠿ Network docker_default            Created                                                                                0.2s
 ⠿ Container docker-xttsapiserver-1  Created                                                                                0.9s
Attaching to docker-xttsapiserver-1
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | ==========
docker-xttsapiserver-1  | == CUDA ==
docker-xttsapiserver-1  | ==========
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | CUDA Version 12.1.0
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | This container image and its contents are governed by the NVIDIA Deep Learning Container License.
docker-xttsapiserver-1  | By pulling and using the container, you accept the terms and conditions of this license:
docker-xttsapiserver-1  | https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | *************************
docker-xttsapiserver-1  | ** DEPRECATION NOTICE! **
docker-xttsapiserver-1  | *************************
docker-xttsapiserver-1  | THIS IMAGE IS DEPRECATED and is scheduled for DELETION.
docker-xttsapiserver-1  |     https://gitlab.com/nvidia/container-images/cuda/blob/master/doc/support-policy.md
docker-xttsapiserver-1  |
docker-xttsapiserver-1  | 2024-01-01 18:31:15.026 | INFO     | xtts_api_server.tts_funcs:create_directories:221 - Folder in the path /app/output has been created
docker-xttsapiserver-1  | 2024-01-01 18:31:15.026 | INFO     | xtts_api_server.tts_funcs:create_directories:221 - Folder in the path /app/models has been created
docker-xttsapiserver-1  | 2024-01-01 18:31:15.028 | INFO     | xtts_api_server.server:<module>:76 - Model: 'v2.0.2' starts to load,wait until it loads
docker-xttsapiserver-1  | [XTTS] Downloading config.json...
100%|██████████| 4.36k/4.36k [00:00<00:00, 4.71MiB/s]
docker-xttsapiserver-1  | [XTTS] Downloading model.pth...
100%|██████████| 1.86G/1.86G [16:11<00:00, 1.92MiB/s]
docker-xttsapiserver-1  | [XTTS] Downloading vocab.json...
100%|██████████| 335k/335k [00:00<00:00, 1.43MiB/s]
docker-xttsapiserver-1  | Traceback (most recent call last):
docker-xttsapiserver-1  |   File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
docker-xttsapiserver-1  |     return _run_code(code, main_globals, None,
docker-xttsapiserver-1  |   File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
docker-xttsapiserver-1  |     exec(code, run_globals)
docker-xttsapiserver-1  |   File "/app/xtts_api_server/__main__.py", line 46, in <module>
docker-xttsapiserver-1  |     from xtts_api_server.server import app
docker-xttsapiserver-1  |   File "/app/xtts_api_server/server.py", line 77, in <module>
docker-xttsapiserver-1  |     XTTS.load_model()
docker-xttsapiserver-1  |   File "/app/xtts_api_server/tts_funcs.py", line 146, in load_model
docker-xttsapiserver-1  |     self.model = TTS(model_path=checkpoint_dir,config_path=config_path).to(self.device)
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/TTS/api.py", line 70, in __init__
docker-xttsapiserver-1  |     self.config = load_config(config_path) if config_path else None
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/TTS/config/__init__.py", line 91, in load_config
docker-xttsapiserver-1  |     with fsspec.open(config_path, "r", encoding="utf-8") as f:
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/fsspec/core.py", line 103, in __enter__
docker-xttsapiserver-1  |     f = self.fs.open(self.path, mode=mode)
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/fsspec/spec.py", line 1295, in open
docker-xttsapiserver-1  |     f = self._open(
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/fsspec/implementations/local.py", line 180, in _open
docker-xttsapiserver-1  |     return LocalFileOpener(path, mode, fs=self, **kwargs)
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/fsspec/implementations/local.py", line 302, in __init__
docker-xttsapiserver-1  |     self._open()
docker-xttsapiserver-1  |   File "/usr/local/lib/python3.10/dist-packages/fsspec/implementations/local.py", line 307, in _open
docker-xttsapiserver-1  |     self.f = open(self.path, mode=self.mode)
docker-xttsapiserver-1  | FileNotFoundError: [Errno 2] No such file or directory: '/app/xtts_api_server/models/v2.0.2/config.json'
docker-xttsapiserver-1 exited with code 1

Hi, I know about the docker bug, I am planning to fix this problem soon

wait so the api work already?


cmd/go: go list -m all stuck

Go version
go version go1.20.7 linux/amd64
Output of go env in your module/workspace:
GOPROXY="https://proxy.golang.org,direct"

What did you do?
when I run go list -m all to detect version of a certain lib, it hangs for more than 5 minutes:
root:/work/project# go env | grep GOPROXY
GOPROXY='https://proxy.golang.org,direct'
root:/work/project# time go list -m all
//...
real	5m39.997s
user	1m39.085s

and I found this similar issue: https://github.com/golang/go/issues/60395
but I think the default GOPROXY already contains direct, also found it in the go.env here, not clear about the  switching to the default GOPROXY (https://proxy.golang.org) means.
but after I removed the default direct, it indeed takes less time:
root:/work/project# go env -w GOPROXY="https://proxy.golang.org"
root:/work/project# go clean -modcache
root:/work/project# time go list -m all
//...
real	0m30.754s
user	0m9.484s
sys	0m7.601s

What did you see happen?
N/A
What did you expect to see?
return ASAP

git or other operations are inherently slow.
this is just #60395


GetMessageDataTest

maybe someone could double-check for typos. My branch compiles tho so should be all good.

Typo in the branch name. Should be GetMessageDataTest.


[wip] trying ember 3.8

Some internal ember framework changes have resulted in ember model inheritance chains not working correctly. We don't use them in Intercom, and we're actively migrating our last handful of ember-model models to ember data so it's not something that we're planning to fix. I'm going to move this to our own repo
https://github.com/intercom/ember-model/pull/10


WIP Prevent multiple restarts when changing multiple attributes on a …

…composite operation.

Don't merge yet (although it might be pretty obvious that still need some refactoring)
This is something quickly that i though could prevent multiple restarts.
It does indeed prevent it. Tested on EAP7.1, single operations and composite operations modifying multiple attributes only trigger once restart (this time for sure).
WDYT @jmazzitelli ?
//cc @jshaughn

@josejulio  We should get some itests built so it always gets exercised and tested. @jshaughn already has tests that he's working on that exposed this problem, perhaps we just piggy back on what he's done for testing.

I have slightly modified it. (Removed a try-catch).
The problem with this approach is that I need to copy all the code from applyUpdateToRuntime, revertUpdateFromRuntime (still need to do that) and other private methods from the parent class.
I have thought on a second approach that should be cleaner and will send another PR with it. Following will be to add itests to check if the server is restarting more than once per operation (composite or not).

PR #292 supercedes this PR... closing.


Low performance of the SetStyling method on lines containing Unicode characters

I used VS 2017 debugger to monitor the performance of Scintilla and found that the performance of SetStyling was not so good.
It took about 1 second to style 700 tokens (700 calls to the SetStyling method), less than 5000 characters.
No time consuming operations were taken place between calling that method.
I examined the code and found that the SetStyling method called DirectMessage to SCI_LINELENGTH, SCI_POSITIONFROMLINE and SCI_POSITIONRELATIVE substantially, which might be the reason for the slow down.
Is there a way to skip some P/Invoke calls and make the SetStyling faster?

Perhaps... sounds like you might be in a good position to offer some suggestions. :)

I have little knowledge of Scintilla.
I saw you tried to determine whether whether the content contains Unicode characters.
Do you know how it store characters internally? If we know how characters were stored, we can use GetTextRange to extract the text to be styled, get byte counts on our own side and then call SCI_SETSTYLING with that byte count, which will reduce multiple P/invoke calls to one and hopefully can enhance the performance.

So Unicode is the real issue. Internally Scintilla stores text as UTF-8 bytes, so there is not a one-to-one mapping of characters to bytes and thus we need to make a few pinvoke calls to SCI_LINELENGTH, SCI_POSITIONFROMLINE and SCI_POSITIONRELATIVE (among others) and so some calculations. All of this translation happens in the LineCollection class and that is where you will find the helper methods you are looking for for converting byte to char positions and vice versa. There is already some significant optimizations going on in LineCollection (caching) which I won't try to explain and which I don't think you'll need to know to implement the approach you are suggesting.
In other words, I think you are correct to approach this as a problem of 'batching' multiple calls and/or 'caching' expensive position lookups/translations. In some combination of those hopefully you can minimize the number of pinvoke calls.

Thank you for your explanation.
Since UTF-8 was used internally by Scintilla, is it possible to use System.Text.Encoding.UTF8Encoding.GetByteCount to calculate the byte count instead of batches of P/Invoking calls? I'll give it a try if so.

I changed my code to style my tokens with the following line like:
scintilla.DirectMessage(STL_SETSTYLING, (IntPtr)token.ByteCount, (IntPtr)(int)token.SyntaxType);

The ByteCount was the UTF-8 byte count of the token.
The styling was still correct after the modification and the time consumption dropped from seconds to several milliseconds.
However, I found another problem. While I was creating tokens, I now had to call GetTextRange quite a few times, which had the same performance problems. I had to find workaround for it as well.

How are you arriving at the UTF-8 byte count?
The reason ScintillaNET does what it does is because a string like "Hèllo Wòrld" will have a length of 11 in C#/.NET, however, that same string will require 13 bytes to represent in UTF-8 (because of the grave marks). ScintillaNET APIs have been designed so that they work well with C#/.NET strings and developers don't need to make those kind of conversions manually. If you aren't accounting for the same, your approach will fail when you encounter some unicode characters.
If you are taking that into account, then ignore the caution.
As for GetTextRange, you can 'batch' this in the same way as I suggested for SetStyling. For example, instead of enumerating tokens and then calling GetTextRange for each, call GetTextRange for an entire line of text and then enumerate the tokens in that string. If that isn't enough, scale up to multiple lines (paragraphs), etc... This suggestion comes from the Custom Syntax Highlighting wiki recipie:

For performance or practical reasons you may prefer to get an entire line of text a time. It's entirely up to you how you go about writing your lexer.

Yep, I did the following calculation in my code like
var byteCount = System.Text.Encoding.UTF8.GetByteCount("Hèllo Wòrld");
// byteCount == 13

I modified my lexer, took your advice to get the entire line of text and calculated the byte count for each token. The performance became acceptable eventually.
BTW, is it possible to recompile Scintilla to use UTF-16 little endian internally instead of UTF-8? If so, we could save a very considerable amount of calculations on byte alignments.

UTF-8 is the only Unicode mode supported by Scintilla.

I checked out the source code of Scintilla and searched the web. I found that I had run into a ten-years-ago issue.
In year 2006, someone had already submitted a feature request to support UTF-16 in Scintilla for the sake of inter-operating to the huge amount of Windows Unicode applications.
https://sourceforge.net/p/scintilla/feature-requests/328/
The request is still open today but it appears that the author has paid no attention to it any more.
Finally I modified the source code of ScintillaNET to alleviate the Unicode performance issue. It is quite dirty still, but the performance is somewhat improved when a line is long and contains a lot of Unicode characters.
internal int CharToBytePosition(int pos)
{
    Debug.Assert(pos >= 0);
    Debug.Assert(pos <= TextLength);

    // Adjust to the nearest line start
    var line = LineFromCharPosition(pos);
    var bytePos = scintilla.DirectMessage(NativeMethods.SCI_POSITIONFROMLINE, new IntPtr(line)).ToInt32();
    pos -= CharPositionFromLine(line);

	// Optimization when the line contains NO multibyte characters
	if (!LineContainsMultibyteChar(line))
		return (bytePos + pos);

	if (pos < 10) {
		while (pos > 0) {
			// Move char-by-char
			bytePos = scintilla.DirectMessage(NativeMethods.SCI_POSITIONRELATIVE, new IntPtr(bytePos), new IntPtr(1)).ToInt32();
			pos--;
		}
		return bytePos;
	}
	var lineLength = scintilla.DirectMessage(NativeMethods.SCI_LINELENGTH, new IntPtr(line)).ToInt32();
	var ptr = scintilla.DirectMessage(NativeMethods.SCI_GETRANGEPOINTER, new IntPtr(bytePos), new IntPtr(lineLength));
	string s = Helpers.GetString(ptr, lineLength, scintilla.Encoding);
	unsafe
	{
		fixed (char* c = s) {
			return bytePos + scintilla.Encoding.GetByteCount(c, pos);
		}
	}
}

I applied the aforementioned patch to Scintilla, recompiled ScintillaNET, modified CharToBytePosition to return 2 * pos; and set the code page to 1200 when the Scintilla control was created.
The performance on processing lines with Unicode characters was dramatically improved. However, some method calls appeared to be broken. For instance the line number margin could not be set and the control appeared to be unstable.
Nevertheless, the patch had proven that changing the internal storage of Scintilla to UTF-16 Little Endian could obviously improve the performance for Unicode applications on Windows.

I'm not surprised that changing the codepage to UTF-16 made the control unstable, because UTF-16 is not supported by Scintilla. Quoting myself:

UTF-8 is the only Unicode mode supported by Scintilla.

The relevant blurb in the Scintilla documentation can be found here: http://www.scintilla.org/ScintillaDoc.html#SCI_SETCODEPAGE.
Not to be the pessimist, but if you search through our issues list you'll find plenty of people who have come before you and wondered the same things -- about how to improve Scintilla's performance and/or support alternate encodings. The short answer is that, yes, we might be able to squeeze some more performance out of ScintillaNET's UTF-8 (Scintilla's internal format) to UTF-16 (.NET's internal format) conversion, there is really no better option for Scintilla's encoding that UTF-8. That is the only form of Unicode Scintilla supports.
If you have suggestions though for improving the conversion that doesn't involve changing the codepage I'm all ears.

Thanks for your reply.
I knew that I was not the first one. The encoding issue was so common that almost each Windows programmer who worked with Unicode APIs would encounter.
I commented on the thread there: https://sourceforge.net/p/scintilla/feature-requests/1175/
And the author was reluctant to merge or work on the patch, for it was big, buggy and broken, unless someone else write some better code. He might also probably think that UTF-8 encoding in Scintilla was the best and the coding conversion was trivial stuff.
Recently I also invested some time to take a look at another powerful high-performance source code editor control, AkelEdit, the one hidden in the source code of the famous code editor AkelPad.
I did not take too much time to compare the features between AkelEdit and Scintilla. However, it does support Unicode and use UTF-16 as the internal storage, syntax highlighting, code folding, code completion (implemented via a plug-in) were all implemented, and it is programmed in C and it exposes Unicode interfaces. We might be also possible to write a wrapper around it and make another editor control.

Closing this as a known issue. We can re-open if necessary.


Missing guidance for the number format

[Enter feedback here]
We attempted to change the format from -#### to .####. This malfunctioned completely. It truncated the project number (-0001 became -1, and the updated project number became PRJ000-01.1  or some other garbage.
We need to know how to force the leading zeroes in the format--there is no guidance on the page. Should I just enter "."? or ".0000" or something else?

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

ID: 096c3daa-2e95-c034-6c2e-98b85116a987
Version Independent ID: e019bf0b-60d5-dc0d-5d59-d8d2b5aff937
Content: Make the chart of accounts delimiter unique - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/migration-upgrade/chart-accounts-delimiter-unique.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @RyanCCarlson2
Microsoft Alias: RCarlson

@Gadgetgeek2000 The #### mask for leading zeros in my sub-project setup works fine in my quick test on the latest builds.  Please contact support with the information if this does not work in a repeated trial for you.

@Gadgetgeek2000, thanks for contacting us. To contact Support, consider one of the options described here: Support contact options. In particular, you might want to go to the Dynamics 365 community page to get help with your questions from the Dynamics community.


Implement cache manager

Is your feature request related to a problem? Please describe.
Implement a cache manager like CRL system

@dariofurlan @albertorizzi I see you appreciated this feature, unfortunately I have no time to work on it and provide a realistic ETA! At the moment I'm working on other things related to DGC and I have to add exemption certificate validation to the core.
The idea behind cache manager was born after many users started complaining about the lack of a persistent file system support on some PaaS like Heroku, and I suggested to switch to another platform as quick fix. My idea is to specify as a strong requirement the support of persistent file system on the deployment environment. Do you think this is a bad idea? I don't want to overcomplicate the SDK.


[Project Addition] Arabic Date Species Classification using DL

Pull Request for DL-Simplified 💡
Issue Title : #733 Arabic Date Species Classification using DL

Info about the related issue (Aim of the project): Classifying Arabic Dates into 9 different species found in the arabic region.
Name: Arihant Bhandari
GitHub ID: https://github.com/Arihant-Bhandari
Email ID<EMAIL_ADDRESS>Idenitfy yourself: GSSoC 2024 Contributor

Closes: #733 Arabic Date Species Classification using DL
Describe the add-ons or changes you've made 📃
Wrote code for preprocessing and loading images from Dataset, turned labels into probability distributions, created models including custom CNN as well as pre-trained models, like ResNet-50, VGG16 and so on.
Type of change ☑
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested? ⚙

Models
Accuracy
Loss

ResNet-50
12.05%
14.1762

InceptionV3
90.36%
0.3815

CNN
83.13%
0.7560

VGG16
97.29%
0.1368

EfficientNetB7
18.37%
2.1760

DenseNet-121
96.99%
0.1515

Xception
94.58%
0.1908

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

hi @abhisheks008 this is my work on Arabic Dates Classification. if any changes i will be glad to do them over. thank you as always


Problem with training on Waymo

Dear @maudzung,
first, thank you for your work.
@lucabart97 and I, have spent the last week trying to train this network on the Waymo dataset.
We first used your other repo to convert Waymo to Kitti format.
Then we changed the train.txt and val.txt files and also kitti_config.txt.
However, we were not able to achieve any good results with the training.
Indeed, the train loss has a very weird behavior:

Can you give us some tips to make it work? Or point out what we are doing wrong?
Thank you very much in advance.
Best,
Micaela

Hi @mive93
Did you carefully visualize and check the converted dataset?

Yes, we did.
We visualized everything (cloud, images and labels) and the conversion was okay.

Because of my limited resource, I didn't try to train the network on the Waymo dataset. I thought I should be trained well on the Waymo dataset also. There are 5 components in the total loss, you can check each curve to check which components aren't changed over time.

Thank you for your answer.
I have checked the components, and the one causing problem is train_hm_cen_loss, that always has a fixed value.
Do you think it's related to the conversion?

Hi! I have noticed a similar issue while training on the Kitti Dataset but not always. During some runs, the loss starts reducing but after some epochs the center heatmap loss increases and stays constant while in some other cases it keeps increasing and decreasing. I have also noticed a similar case while changing resnet-18 to resnet-34 and I am not sure why.
Some failure cases:

Let me check again about the center heatmap generation to see what happended.

Thank you for your answer.
I have checked the components, and the one causing problem is train_hm_cen_loss, that always has a fixed value.
Do you think it's related to the conversion?

Hi, @mive93, @maudzung
I also tried to train the SFA3D in the Waymo dataset and meet the same loss problem when using resnet34 and resnet50. If I use loss = loss - (pos_loss/num_pos + neg_loss/num_neg) and set the LR rate to 0.0005, the loss problem will be solved but the prediction res will be worse. Have you solved this problem?
Zhimin Chen

I think the problem is related to the reported issue here. I'll try to solve the problem in the near future.

Hello @Zhimin-C , Can you tell me what is the values of the boundary min_Z and max_Z that you have used for Waymo please ? Because I think the range of Z-axis has an impact in the predictions according the dataset (the height of the lidar).
Thanks.

Hi,
https://arxiv.org/pdf/2006.15505.pdf
This HorizonLiDAR3D is the top method on the Waymo dataset 3D detection
challenge:
https://waymo.com/open/challenges/2020/3d-detection/
You can refer to their parameters in the experiment part. The height range
is -1 to 3.
Zhimin Chen
On Fri, Jun 4, 2021 at 9:30 AM SofianeB-03 @.***> wrote:

Hello @Zhimin-C https://github.com/Zhimin-C , Can you tell me what is
the values of the boundary min_Z and max_Z that you have used for Waymo
please ? Because I think the range of Z-axis has an impact in the
predictions according the dataset (the height of the lidar).
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maudzung/SFA3D/issues/15#issuecomment-854725460, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIPX4UJDM24BRHSJZPPMRJDTRDINRANCNFSM4S4HQ5VQ
.

@Zhimin-C Thank you very much for your quick response !


Link does not work: Resolve reserved resource name errors.

At the top of the page, it states:

For a list of the blocked words, see Resolve reserved resource name errors.

That link does not work (it targets the page itself).
Thanks,
--Neno

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

ID: 374ec8eb-64d4-3766-5c5e-39137a262ffd
Version Independent ID: 3abeeae9-6854-651a-483c-a1673eea9f9f
Content: Resource naming restrictions - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resource-name-rules.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@NenoLoje Thanks for the question! We are investigating and will update you shortly.

@NenoLoje - thanks for making us aware of this error. I will fix it.

The update has been merged and should get published today.
#please-close


Appears not to support streaming

e.g.:
https://github.com/tmattio/dream-encoding/blob/dde420426ba98da2bc7d26b0d67c3f0f2eb7b7ab/lib/dream_encoding.ml#L194-L195
This appears to read the entire body into memory, and then compress it. I assume (perhaps wrongly) that the underlying compression algorithms support streaming, and it seems like streaming would be more beneficial for precisely the kind of responses that you wouldn't want to load into memory an extra time (large ones).

assume (perhaps wrongly) that the underlying compression algorithms support streaming

It does! In fact, the code (which is almost identical to decompress's documentation example) creates a stream from the given string, so refactoring this to work on stream responses should be trivial 🤞
I'll get to it as soon as I have some spare time

I'll get to it as soon as I have some spare time

No rush! And I think you will likely need some improvements in Dream's streaming APIs to get it done, so do bug me :)


Fix #8 index error

Fix #8 index error

Thanks for the fix!

in util.py
80 if w not in dictionary:
81    dictionary[w] = len(dictionary)
facing this error:
if w not in dictionary:
TypeError: argument of type 'BabiConfigJoint' is not iterable
running on pyhton 3.7


Invalid credentials. Bad sign.

I got this error when trying to getInfo
fullcode:
const btcid = require('@slaveofcode/btcid')
const btcInst = btcid('apikey','secret')

btcInst.showInfo().then(res => {
  console.log(res);
})

sorry, typo issue, my bad :D


Azure App Configuration Tags not showing up in Cost Analysis

We have an Azure App Configuration deployed in our subscription and we have put 2 tags to every resource ('costType' and 'environment'). All resources are tagged and appearing when we pull the Cost Analysis report by Tags except the App Configuration. They show the Tags column as blank.
We would expect this would work like for every other resource (VMSS, storage accounts, API management, application insights, ...) on Azure.

The tags are supported now.


Enable parallel run of strands where global locks arent shared inside locks

Purpose

In the previous release the concurrent design was supported but the parallelism was reduced when locks were used. This has been fixed in this PR.

Fixes #21075
Approach

Collect global variables accessed within lock statement.
Analyse the lock statements and put them in to sets by the locks they use.
This will give us a set of disjoint sets.
Create JVM locks for each set.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #22320   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 8a3197b...4079f41. Read the comment docs.


Add gradle configs

As per comment from @srinivasankavitha, we have added the Gradle configs. Please let us know, if these changes are good.
Ref Issue #457
cc: @lakshmisunid @anshuls1208

@srinivasankavitha Please review these changes for the Gradle plugin.

Thanks for the updates and all the work going into this feature!


[crust-bump] pod update

Codecov Report

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


Use code-style for links to proto entities

In the generated documentation we should use backticks to format proto entities (messages, fields, services, etc.) as code.  Unfortunately the comments  extracted from proto files do not have backticks.
We could generate reference style links like so:
/// [`foo.bar.Baz`]: link-to-foo-bar-Baz
/// [foo.bar.Baz]: link-to-foo-bar-Baz

We would need to double the number of reference-style links.  We need a cleaner solution.

Should the type name be back-tick quoted?
Originally posted by @devbww in https://github.com/googleapis/google-cloud-cpp/pull/11677#discussion_r1200613769

No time for this at the moment. Closing.


Replace typo Set-SPOHubSite to Get-SPOHubSite

Also updated correct parameters for the Get-SPOHubSite Command.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOHubSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thx Michael, appreciated. Clear copy-paste issue and good to get it resolved.


Load the model from a specific check point which generated minimum loss

I am running the BERT model using tfhub similar to the example shared in the repository. I find that training loss is minimum during the 400th step and after that it is slightly higher when the  last check point is created. For the validation set the model is loaded from the last created check point i.e 636th step but i want to manually load the model from 400th step so that I can have better parameters of the model and enhance my accuracy.
In order to do so what are the changes i should make to the functions.
This is the loss during the last check point creation at the step 636

This is loss during the checkpoint creation at the step 400:

So while running the model on the validation set I want to load the model from the checkpoint 400th step and not from the 500th step.
Kindly help me out as this can enhance the performance of my model by atleast 3-5%.

were you able to solve this? if yes can you please tell me

prediction = estimator.predict(predict_input_fn, checkpoint_path='/content/drive/My Drive/BERT checkpoints/model.ckpt-7097')
You can manually give the checkpoint you want to load from in the estimator.predict. Hope this helps.

can you also please tell me is there a way to store the best model based on eval loss as this code is storing based on checkpoints

can you also please tell me is there a way to store the best model based on eval loss as this code is storing based on checkpoints

Even I was trying for that but could not find a solution as I am a begginer in TF, but on surfing over the repos on kaggle I found one implementation using Keras and might be useful for you.
[https://www.kaggle.com/xhlulu/disaster-nlp-keras-bert-using-tfhub](click here)
Since it uses Functional API you can add callbacks for saving the best model easily.


Startup Error

hi, I cloned This Project And checkout On Branch 'Master',
But I can't start it .
Give me a hint of something Exception, like [‘bash: ./gradlew: Permission denied’] ,
I think this is because there are not enough permissions to run Gradlew,
So my solution for now is #bash [chmod a+x gradlew].
Environment:  macOS Mojave 10.14.6  &&  JDK 1.8

Thanks for the solution!


Let's revive this library - add your feedback to the discussion!

https://github.com/TheWidlarzGroup/react-native-video-player/discussions/186

#186
for sure ready for it


⚠ Bofrost.fr has degraded performance

In 1b7b737, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1073 ms

Resolved: Bofrost.fr performance has improved in 261e7cf after 6 minutes.


Fix documentation : Broken links + Wrong formula

Hello,
As discussed on forum, find a PR that fixes broken documentation links
Regards
Philippe M.
UbikLoadPack Team

Thanks!


FE-233: bugfix - query spotlight feature broken

Scope & Purpose
Query spotlight feature on the UI was broken, and worked before #18524.
~We upgraded the jQuery version from 3.6.0 -> 3.6.4 in that PR, a minor upgrade which shouldn't have been a problem, but apparently broke the spotlight. (no typeahead elements were getting appended to the DOM)
For now, rolled back & pinned jQuery version to 3.6.0, which fixed this.
To be investigated further later.~
Turns out it wasn't 3.6.4 specifically, but that we had multiple jQuery versions. Merged them all to 3.6.4 and that works!

[x] :hankey: Bugfix

Checklist

[x] Tests

[x] Manually tested

[x] :book: CHANGELOG entry made

Related Information
(Please reference tickets / specification / other PRs etc)

[x] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/FE-233

[x] https://jenkins01.arangodb.biz/job/arangodb-ANY-jslint/28551/


Create a story list tile in the package

One of the generic widgets that the story_app package can use is a tile to display each story title in the story list page.

Here are some tasks we need to do to accomplish that:

[x] Create a StoryListTile widget in lib/src/story_list_tile.dart. For now it can be a simple wrapper around a ListTile. The only parameters that you need to expose are one for the story number and one for the story title.
[x] Export StoryListTile in lib/story_app.dart.
[x] Add the story_app package to the example app. (Use a relative reference to it in pubspec.yaml: path: ../)
[x] Replace ListTile in the example app with your new imported StoryListTile.

Put these changes in a few feature branch called i7_story_list_tile.

@JuniorArt225 Thanks for your work on this!
Here are a few things to do:

Whenever you take an issue, you can assign yourself. (in the top right of the window)

You have story_list_tile.dart located in example/lib/src/story_list_tile.dart. Instead, it should be located in lib/src/story_list_tile.dart (not in the example project). Actually, I see there is a widgets folder, so maybe lib/src/widgets/story_list_tile.dart is the best. (You can delete the empty.dart file.)
That was good you made a StoryListTile widget. However, this widget should return a ListTile (not StoryApp).
In example/pubspec.yaml you will need to import the library project:

dependencies:
  story_app:
    path: ../

In lib/story_app.dart you should export your new story_list_tile.dart file. (You can also delete that Calculator code.)
When you have the above changes working, you can open a PR. This should notify me and I can check the update before merging.

I haven't included all of the details, so if you have any questions feel free to ask. The first time making a package is a little confusing but it gets easier with practice.

@JuniorArt225 I've pushed some changes to main. You can check out what I did. Let me know if you have any questions.


LUTECE-2237 - Add forward URL for webapp's root path according to user session.

Thanks for style. We have preferred to duplicate the code because the actual default method will be probably deprecated soon.


operator installmodes - only support AllNamespaces

OLM is going to deprecate all install modes except "AllNamespaces" (at least, currently, they are telling operator authors to move away from the others now).

The reason for this change:
From: Daniel Messer
Date: Fri, Jul 9, 2021 at 10:35 AM
Subject: Re: specifying a dependency of an operator on another operator
Cc: Operator Framework<EMAIL_ADDRESS>
As for the general question about installmodes - this is something the we are moving
away from as part of the descoping exercise, in which all Operator just become global
singleton's and we have more precise RBAC controls that define where they are
available to be discovered by tenants, which tenants actually have usage rights and
inversely, in which namespaces the operator can become active.

Our recommendation today is to only support AllNamespace mode in your Operator.
That way dependencies will not run into the conflict you describe here.


Excellent doc page

Excellent doc page and video - that's all.
[shouldn't always come here to complain :)]

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

ID: 6758e597-965a-92c5-2fd3-5bcc5702deac
Version Independent ID: d0619375-0c8e-c2d4-f059-b1ae17fe7c52
Content: Deploy Docker containers to Azure App Service from Visual Studio Code
Content Source: articles/javascript/tutorial/tutorial-vscode-docker-node/tutorial-vscode-docker-node-01.md
Product: azure-nodejs
GitHub Login: @diberry
Microsoft Alias: diberry

@RussFerns thanks! CC @diberry


Support for ANY_USER_ACCOUNT in module vpc-sc egress rule.

This PR enables to support keyword ANY_USER_ACCOUNT when crafting VPC SC egress rule.
This keyword (identity_type) is currently not enabled by validation block in modules/vpc-sc/variable.tf.
Despite the fact that this option is specifically not enabled by fast vpc-sc module, it is indeed fully supported by underlying terraform resource google_access_context_manager_service_perimeters.
Desired state:

I applicable, I acknowledge that I have:
[ x] Read the contributing guide
[ x] Ran terraform fmt on all modified files
[ x] Regenerated the relevant README.md files using tools/tfdoc.py
[ x] Made sure all relevant tests pass

Nice catch. Thanks for the fix @xjantoth!
Can you sign the CLA so we can go ahead and merge it?


rename MemtableIndex -> TrieMemtableIndex to make merge to vsearch easier

@mike-tr-adamson, does this renaming make sense to you? Or you prefer a different name?

@jbellis @blambov I would prefer this was called ShardedMemtableIndex if it is going to go into ds-trunk. That is the distinguishing feature of this. The fact that we only have a trie implementation currently is secondary.

As the new guy to this code base, it's definitely a bit odd and confusing seeing classes named generic things when they are actually a single specific implementation, like InvertedIndexWriter (actually this is about Tries and has several components that don't make sense except for Tries) vs NumericIndexWriter (a name that makes more sense).
If we have a ShardedIndexWriter interface or abstract class to extract, let's do that, but let the implementing class name reflect what it actually does.

Closing this per Matt's request to not mess with ds-trunk for now.


Implement Tree Edit Distance for structural similarity

You can find the java implementation in DataRecordAligner project (see here ).
The distance was first proposed in Web Data Extraction Based on Partial Tree Alignment.
Tree Edit Distance is an evolution of Edit Distance and performs better for data record collections having optional items

Pull request with tests are welcome!

done


Nothing happens when reacted with emoji to message

Please provide more detail if possible including time (UTC) & frequency of this issue.

Seems to be fixed with the new Discord attachment method.


Figure out how to get rid of keyValue for logging

Describe the bug
Today we have keyValue scattered in different log statements, which are tied to the logging implementation
Expected behavior
We should not be tied to the logging implementation, hence we should remove every statement that couple our code to logstash logback json encoder.

/assign
I'm proceeding as following:

All the debug statements won't have structured fields anymore (since structured fields are useful only for production logging infra, we don't need structured fields for logging statements that won't be used in production, aka level <= debug)
For the remaining statements, I'm going to use the MDC https://github.com/logstash/logstash-logback-encoder#mdc-fields

All the debug statements won't have structured fields anymore (since structured fields are useful only for production logging infra, we don't need structured fields for logging statements that won't be used in production, aka level <= debug)

This is not true, they are useful in CI

Please, let's not do this, structured logging is important even in debug mode.

This is not true, they are useful in CI too

Do we actually have an infrastructure to read the json logs and search around it? And is it that fundamental? Do you have a solution that allows us to move away from our tight coupling to logstash json encoder?

tight coupling to logstash json encoder

Why this is a problem now?

We have zap assumption all over Knative.

The linked PR is clearly an example of weird logging code:
    MDC.put("id", egress.getUid());
    MDC.put("consumerGroup", egress.getConsumerGroup());
    MDC.put("destination", egress.getDestination());
    logger.error(msg, cause);
    MDC.clear();

Why this is a problem now?

Because I can't swap the slf4j implementation behind the hood, making slf4j useless (as you saw in the benchmark PR, I had to remove all the log lines using keyValue to remove the logging implementation). I wanted at some point to remove the json logging for e2e tests (like we did for eventing, json logs are unreadable for humans), using a simple ConsoleEncoder, but we can't do that with this tight coupling.

The linked PR is clearly an example of weird logging code:

It's super weird, but I can't see a better solution and it's still better than using slf4j with types specific to the logging impl. Do you have any better solution that doesn't tight couple us to a specific logger impl?

The logger doesn't have to be static, it can be injected like any other dependency.

The logger doesn't have to be static, it can be injected like any other dependency.

I don't see how this solves the problem, and even worse, it's not idiomatic for the usage of slf4j

From http://www.slf4j.org/manual.html
        int newT = 15;
        int oldT = 16;

        // using classical API
        logger.debug("oldT={} newT={} Temperature changed.", newT, oldT);

        // using fluent API
        logger.atDebug().addKeyValue("oldT", oldT).addKeyValue("newT", newT).log("Temperature changed.");

^ SLF4J API version 2.0.0

@pierDipi it sounds perfect! Will this work out with logstash json encoder? Any refs?

I don't know :man_shrugging:

SLF4J doesn't sound actively developed, nor logback...
2.0.0-alpha1/                                     2019-10-01 19:09         -

https://repo1.maven.org/maven2/org/slf4j/slf4j-api/
https://repo1.maven.org/maven2/ch/qos/logback/logback-core/

Ok I might have another solution, without scrambling too much our code

Log4J might be another valid option: https://logging.apache.org/log4j/2.x/log4j-api/apidocs/org/apache/logging/log4j/message/MapMessage.html#with-java.lang.String-boolean-

Re ^, an article https://www.elastic.co/guide/en/ecs-logging/java/current/_structured_logging_with_log4j2.html


minDate/maxDate not working when passed via datepicker-options

When I use uib-datepicker-popup and pass minDate/maxDate as properties via datepicker-options I get this error:
[$parse:lexerr] Lexer Error: Unexpected next character  at columns 46-46 [ä] in expression [Wed Jan 20 2016 19:02:20 GMT+0100 (Mitteleuropäische Zeit)].

The problem is caused in this loop:
  angular.forEach(['minDate', 'maxDate'], function(key) {
    if ($attrs[key]) {
      watchListeners.push($scope.$parent.$watch($attrs[key], function(value) {
...
      }));
    } else {
...
    }
  });

When using the min-date/max-date attributes, $attrs[key] resolves to watchData.min/maxDate. When using the way described above, $attrs[key] is the actual Date instance passed via the options.
Here's a fiddle:
https://jsfiddle.net/qz34osvr/

I will check.
To be real honest, they datepicker <-> popup relationship is a bit ugly. There are some decisions made in there that are a bit weird.
We plan in revamping the whole datepicker and popup API to make it simple to use and remove all this options vs attributes thing.
In the meantime, this is an issue for sure.

Scheduling this for 1.1.1 - this will be the last critical fix (unless something else big comes up) before that release, hopefully for Monday morning or around that timeframe.

Great, thanks!


Scheduled jobs not executed when using Redis on Heroku

It seems scheduled jobs are not run when using Redis on Heroku.
When creating the jobs using the admin panel I can see that they are created. From django I see output using:
        scheduler = django_rq.get_scheduler("default")
        scheduled_jobs = scheduler.get_jobs()
        print(scheduled_jobs)

but the nothing shows up in the redis logs (note I am running rqworker with the --with-scheduler parameter set.
Any clues?
Thanks in advance!

Hi,
This project is not about the scheduler packaged into https://github.com/rq/rq.
It's a scheduler that needs to be run on its own.
You can see here how to do so: https://github.com/rq/rq-scheduler#running-the-scheduler
Here's the django_rq doc about starting the rqscheduler: https://github.com/rq/django-rq#support-for-rq-scheduler
It seems that django_rq doesn't support the new RQ integrated scheduler so you need to use the docs above and remove the
--with-scheduler option.

Hi Graphman65,
Thank you a lot for these pointers, was able to make it work with:
rqscheduler --host localhost --port <port_nr>
also found out that I can run it with the manage.py script:
python3 manage.py rqscheduler
and then it takes the redis config as specified in settings.py.
Hope this can also help someone else :)


JSON API urls for collections

#61 add a pseudo-API rendering the info in JSON files, but just only for each one of the versions. Add new pages showing the full info for each tool, or maybe also one for ALL the tools at once.

upvote here !

Please see PR https://github.com/endoflife-date/endoflife.date/pull/260.


Laser drill and Laser Base

Industrial Foregoing Version:
TeslaCoreLib Version:
Crashlog (If Any):
The names on the laser drill and laser bases should really be switched if one doesn't do research into the items they are likely to end up making 4 laser bases and 1 laser drill. This means that they will have one item to supply power and 4 to produce ore. So those names really need to be switched.
Second is a suggestion more than an issue but I think the laser shooting down to the ground feature and the requirement of hitting bedrock in order for the laser to run should be re-added in just as it was in the 1.7.10. It also made a lot more sense when the lasers would damage players in 1.7.10.

First: Nah, you can clearly see in the block texture what is what.
Second: This is not a straight port of MFR stuff has changed.


Check flash in http requests.

Is it possible some how to check flash in http request?

If you have something that can parse flash responses in ruby then you can use the gem. The documentation will most likely say [binary data] for the response.

And how I can retrieve that flash response?

Via response_body, see the readme

Rails flash from responce_body?

Oh you're not talking about Adobe Flash. Can you explain how you're using the flash in an API?

For example I send post with some data to create an record. And I want to get some how session object to be able to retrieve flash messages.

@andriiNyzhnik this project is primarily focused on API testing, in which case you would not be using flash messages.  If you are using flash messages in your API you should be putting them in a header, or in the response body, where you can check their values.
If you are using this to test an html page, you can also check that the flash message is rendered in the response_body

@jakehow the problem is with flash that has not been rendered in html page: e.x. redirect.
So currently, for testing html it is not possible to retrieve flash outside html page, am I right?

I don't use flash in api. I make just simple http requests.

@andriiNyzhnik not clear to me what you are trying to do, but you should probably use response_body to check for the flash message.

Ok. You can close the issue.

@andriiNyzhnik closing, if you need to you can follow the redirect and make a second request.


Etesync dav bridge support

Hello,
I’m trying to add my local etesync dav bridge to sync my tasks on Errands, but when I click the Test button, nothing seems to be done.
Would you help to find what is not working?

Sorry, I was just missing to start my secret service daemon


 This also applies to non-SendContext-based headers.

I believe failing test has nothing to do with this PR. Maybe it's simply unstable?

Probably. I'm rewriting a ton of the underlying header code in response to this issue, so you might want to hold off on any future changes in the meantime. Should make it a lot cleaner and actually eliminate the need to filter headers at all (but it will still be possible, on all transports not just SQS).

That's great to hear. In the mean time is there a chance to include this mechanism as an mid-term solution? I think there's no way to hack MT in any way that would allow control over that behavior other than actually implementing that. Some good old Open–closed principle seems to be forgotten in this particular area.

@phatboyg Any update on this? ;-) It's actually a major blocker for SQS right now.

Nothing yet, but I'm working on it.

As this is gonna take at least few days any chance this PR could be merged? It basically replaces this "don't send headers" flag with a function filter that gives any control over what headers are being send.

I'm not a fan of throwing in a temporary solution, only to discard it the next day. I should have something pushed to develop today.


Broken link

Found a broken link while browsing https://100r.co/site/uxn_design.html - https://wiki.xxiivv.com/site/reverse_polish.html

Thanks :) I've fixed the broken link.


Magento installation failed: Autoload error: Module 'Magento_UrlRewrite' from '.../vagrant-magento/magento2ce/app/code/Magento/UrlRewrite' has been already defined in '.../vagrant-magento/magento2ce/app/code/Magento/UrlRewrite'.

[2017-10-09 19:18:26] STATUS: Installing/re-installing Magento [/vagrant/scripts/guest/m-reinstall]]
[2017-10-09 19:18:26] STATUS: >  Removing Magento configuration files (env.php and config.php) [/vagrant/scripts/guest/m-reinstall]]
[2017-10-09 19:18:26] STATUS: >  Clearing Magento cache [/vagrant/scripts/guest/m-clear-cache]]
[2017-10-09 19:18:26] STATUS:     >  Clearing directories containing temporary files [/vagrant/scripts/guest/m-clear-cache]]
[2017-10-09 19:18:27] STATUS: >  Dropping and creating 'magento' DB [/vagrant/scripts/guest/m-reinstall]]
[2017-10-09 19:18:27] STATUS: >  Dropping and creating 'magento_integration_tests' DB [/vagrant/scripts/guest/m-reinstall]]
[2017-10-09 19:18:27] STATUS: >  ./bin/magento setup:install     --db-host=localhost     --db-name=magento     --db-user=root     --backend-frontname=admin     --base-url=http://magento2.vagrant52/     --language=nl_BE     --timezone=Europe/Brussels     --currency=EUR     --admin-lastname=Admin     --admin-firstname=Admin<EMAIL_ADDRESS>   --admin-user=admin     --admin-password=123123q     --cleanup-database     --use-rewrites=1 [/vagrant/scripts/guest/m-reinstall]]
Autoload error: Module 'Magento_UrlRewrite' from '/Users/wouter/Business/Workspace/multimagento2/vagrant-magento/magento2ce/app/code/Magento/UrlRewrite' has been already defined in '/Users/wouter/Business/Workspace/multimagento2/vagrant-magento/magento2ce/app/code/Magento/UrlRewrite'.
[2017-10-09 19:18:27] ERROR: >  Magento installation failed. [/vagrant/scripts/guest/m-reinstall]]
[2017-10-09 21:18:27] INFO: >  See details in /Users/wouter/Business/Workspace/multimagento2/vagrant-magento/log/m-reinstall.log. For debug output set debug:vagrant_project to 1 in etc/config.yaml -reinstall]]

Have the same problem and maybe it is related to new magento2 developer version?
Tried to clear autoloader cache.

What surprises me is that this Vagrant box has all Magento core modules under app/code/Magento and not in /vendor. Maybe this has something to do with it?

But that should not be a problem for the autoloader.
I also tried with magento version 2.2.0 and there is the same problem.

@woutersamaey if you are on Sierra, please check out https://github.com/paliarush/magento2-vagrant-for-developers/issues/140 , if not - please provide more details about your environment.
Everything works for me on El Capitan with Virtual Box 5.1
BTW, core modules are in app/code when you install git-based version of Magento. They will be under vendor if you install composer-based version. Pay attention to this setting https://github.com/paliarush/magento2-vagrant-for-developers/blob/2.0/etc/config.yaml.dist#L2


Typo on Vowsjs.org reference

In the macros section of the Vowsjs.org reference, there's a snippet of code that defines respondsWith.
The following code:
var req    = this.context.name.split(/ +/), // ["POST", "/"]
    method = name[0].toLowerCase(),
    path   = name[1];

Shouldn't this be?
var req    = this.context.name.split(/ +/), // ["POST", "/"]
    method = req[0].toLowerCase(),
    path   = req[1];

This was fixed on the website and should be closed.


Add tests for the GitLab client in catalog-backend and refactor

Add Tests and Refactor GitLab Client
This PR adds tests for the GitLab ingestion client under plugin/catalog-backend in order to add coverage and increase confidence. The primary purpose is to increase confidence in its implementation and prepare it for reuse when ingesting groups and users (increase utility of client). The PR makes the following changes:

Add test coverage for GitLabClient and paginated to capture externally facing behaviour.
Improve TypeScript inference by allowing developers to pass in a type into paginated and pagedRequest as generics instead of forcing any (but defaulting to any to maintain compatibility).
Refactor and open up pagedRequest so that it can be used to make other calls to the GitLab API with a given client instance. This is preparation for a contribution which will bring org and user parity for gitlab to match github.
Add convenience method to determine whether the client is configured against a SaaS or Self Managed instance.
Add documentation.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] ~Screenshots attached (for UI changes)~
[x] All your commits have a Signed-off-by line in the message. (more info)

Nice work! 🙌  This needs a changeset


Error "CRITICAL - Database data_collection not found in Athena catalog AwsDataCatalog"

Following CID Framework (https://catalog.workshops.aws/awscid/en-US/dashboards/advanced/cost-anomaly/deployment), I'm trying to deploy Cost Anomaly Dashboard using cid-tools.
# deployed to data_collection.
#
$ cid-cmd deploy --dashboard-id aws-cost-anomalies --athena-database data_collection

CLOUD INTELLIGENCE DASHBOARDS (CID) CLI 0.2.47 Beta

Loading plugins...
	Core loaded

Checking AWS environment...
	profile name: default
	accountId: xxxxxxxxxxxxxxx
	AWS userId<EMAIL_ADDRESS>	Region: us-east-1

Discovering deployed dashboards...  [------------------------------------]    0%  None

Required datasets:
 - ca_summary_view

Looking by DataSetId defined in template...complete

There are still 1 datasets missing: ca_summary_view
Creating dataset: ca_summary_view
CRITICAL - Database data_collection not found in Athena catalog AwsDataCatalog

$

I've deployed "Data Collection Lab" using a different database name "data_collection".
Are there any options to deploy anomaly dashboard?
Best

athena-database parameter must be adjusted in this case. Please let us know if still an issue


Layout cfg

I refactored a little and added layoutCfg to be passed as prop
  layout: {
    type: 'dagre',
    rankdir: 'LR', // The center of the graph by default
    align: 'DL',
    nodesep: 20,
    ranksep: 50,
    controlPoints: true,
  }

Thanks for the response.
I'll mark this branch as a draft and prepare a separate branch with the specific changes to the config.
Regarding script before template, I thought, from the documentation, that this was the convention
https://vuejs.org/api/sfc-script-setup.html

Following conventions are good and I welcome such change. Just let's keep them separate :)