lintian: command not found

Steps to Reproduce

Clone distro pelion-edge
Copy mbed_cloud_dev_credentials.c and update_default_resources.c
Run Command ./build-env/bin/build-all.sh --docker=bionic --deps --install --build --source --arch=amd64

Observation
all .deb and .lintian filesare generated under /build/deploy/deb/bionic/main/binary-amd64 but bit while verifying following command not found info message is provided
Update:- It doesn't cause any failure since error is being logged as info
Error
`>> verifying binary packages for amd64

verifying pe-nodejs binary package
INFO: Verifying Debian package!
/pelion-build/pe-nodejs/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
verifying devicedb binary package
INFO: Verifying Debian package!
/pelion-build/devicedb/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying edge-proxy binary package
INFO: Verifying Debian package!
/pelion-build/edge-proxy/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying global-node-modules binary package
INFO: Verifying Debian package!
/pelion-build/global-node-modules/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying kubelet binary package
buildtype=linux/amd64
INFO: Verifying Debian package!
/pelion-build/kubelet/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying maestro binary package
INFO: Verifying Debian package!
/pelion-build/maestro/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying maestro-shell binary package
INFO: Verifying Debian package!
/pelion-build/maestro-shell/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying mbed-edge-core binary package
INFO: Verifying Debian package!
/pelion-build/mbed-edge-core/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying mbed-edge-core-devmode binary package
INFO: Verifying Debian package!
/pelion-build/mbed-edge-core-devmode/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying golang-github-containernetworking-plugins binary package
INFO: Verifying Debian package!
/pelion-build/golang-github-containernetworking-plugins/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying mbed-edge-examples binary package
INFO: Verifying Debian package!
/pelion-build/mbed-edge-examples/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying mbed-fcc binary package
INFO: Verifying Debian package!
/pelion-build/mbed-fcc/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying pe-utils binary package
INFO: Verifying Debian package!
/pelion-build/pe-utils/deb/../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying metapackages using amd64
verifying pelion-edge metapackage
INFO: Verifying Debian metapackage!
/pelion-build/metapackages/pelion-edge/deb/../../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying pelion-edge-base metapackage
INFO: Verifying Debian metapackage!
/pelion-build/metapackages/pelion-edge-base/deb/../../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying pelion-edge-container-orchestration metapackage
INFO: Verifying Debian metapackage!
/pelion-build/metapackages/pelion-edge-container-orchestration/deb/../../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!
verifying pelion-edge-protocol-engine metapackage
INFO: Verifying Debian metapackage!
/pelion-build/metapackages/pelion-edge-protocol-engine/deb/../../../build-env/inc/build-common.sh: line 417: lintian: command not found
INFO: Done!

build finished <<<`

Currently verification does not stop build script even if it found a problem. Do we want to change it to stop the build on first failed verification or keep as it is @ryannowarm ?

@kbgl , I'm fine either way as long as the build script exits non-zero for a failed build.

Fix merged to dev branch.


[release/1.6] Prepare release notes for v1.6.8

See generated notes https://gist.github.com/dmcgowan/10fe7eb6d07d2c30cfc474f79e560f00
Closes #7255

opened https://github.com/containerd/containerd/pull/7268 - don't think it's urgent, but in case we want to include it (otherwise it can be in the next patch release)


removes removed env from workflows

Description
Quick PR to remove the remove the env that were removed recently
Notes
So we don't forget.

Thanks for this PR @Da-Colon , I forgot about this file!
To mention, I've just removed the secrets/envvars from Github and Netlify too.


Mediainfo with filebot

https://github.com/rix1337/docker-verarbeitung/blob/master/Dockerfile
I am pulling my hair out trying to get filebot to find mediainfo.
Any idea how to integrate mediainfo with filebot on alpine?

Same here, actually the only way is to use an OS based on Debian

Hi,
For that :

Compile libmediainfo, mediainfo, and libzen
Install (manually) fpcalc (chromaprint)
Link libmediainfo and libzen on filebot folder

I close this issue


Type invariant lost in double loop

When we have a double-nested loop, and only the inner loop updates an array, then the typing invariant for that array is added to the inner loop, but seems to be needed in the outer loop as well.
For example, this function (which looks artificial, but is a cut-down version of Conways game of life, where the double loop is necessary):
public function loop_bug(int width) -> bool[]
requires 0 <= width:
    bool[] ncells = [false; width]
    for x in 0..width:
        for y in 0..width:
            ncells[y] = true
    return ncells

adds the invariant to the inner loop:
invariant (forall i#1:int :: (((0 <= i#1) && (i#1 < Array#Length(ncells$85))) ==> Bool#is(ncells$85[i#1]))
&& (((i#1 < 0) || (Array#Length(ncells$85) <= i#1)) ==> (ncells$85[i#1] == Void)));
But Boogie cannot prove this invariant for each entry to the inner loop.
However, if you manually add that invariant to the outer loop as well, it is much happier.
It still does not quite verify, because it thinks the array length might have changed.
However, this version does verify with Boogie (if you manually copy the inner loop invariant to the outer loop).
public function loop_bug(int width) -> bool[]
requires 0 <= width:
    bool[] ncells = [false; width]
    for x in 0..width
    where |ncells| == width:
        for y in 0..width
        where |ncells| == width:
            ncells[y] = true
    return ncells

Perhaps the generated array-typing invariant could also say that the length of the array is unchanged?

It still does not quite verify, because it thinks the array length might have changed.

That is correct, and is the expected behaviour at this stage.  However, as you point out, it would be nice to avoid this!!  I have an open issue for it (see #68).

Right, now for the main issue related to the invariant for types on the outer loop.  Actually, the latest version of Whiley2Boogie the following passed for me straight away:
public function loop_bug(int width) -> (bool[] rs)
requires 0 <= width
ensures |rs| == width:
    bool[] ncells = [false; width]
    //
    for x in 0..width
    where |ncells| == width:
        for y in 0..width
        where |ncells| == width:
            ncells[y] = true
    //
    return ncells

Ok, I see what's happened.  I fixed this a couple of weeks ago.  It was actually a bug in WhileyCompiler (see here).  Therefore, you need to update your version of wyc to the latest devel snapshot.


how to output the summary information every step ?

in case that I want debug whether the loss is descending.

There are other ways to print tensors other than summaries: http://tensorpack.readthedocs.io/en/latest/tutorial/faq.html#how-to-print-dump-intermediate-results-in-training
Summaries are added to tensorboard in the callbacks:
http://tensorpack.readthedocs.io/en/latest/tutorial/summary.html#tensorflow-summaries .
And you can use a different period option: http://tensorpack.readthedocs.io/en/latest/modules/callbacks.html#tensorpack.callbacks.MergeAllSummaries


Fix Nav flow

Fixes #2071

Still would need to work out something for actual flow of the nav to a sidebar or something like that, for mobile or screens smaller than 1200px

Hmm, tests are failing because less version was updated, but not on vitreum.git

this is what the deployment looks like on my phone. It’s /new, and there is no title, so it seems to be shrinking that piece to nearly nothing

Seems fine to me. Merging!
Thanks @5e-Cleric ! 💯 ⭐ 🎉


Non Boop Steam games cannot boop custom icons.

I installed a new update which was putting non steam games being booped. Sadly all the games i tested won't let you change the game icons for them. It does nothing if you press boop on a custom icon if it is a non steam game. Same goes if you set custom artwork for yourself. It will not do anything if you boop them.

#7


Docs: Note that the radius arg of bd.Helix is the "minimum radius" when cone_angle>0

When cone_angle>0, the cone of the helix expands outwards (thus making the radiux arg the "minimum radius"). The docs don't currently mention this.

If the cone_angle<0 the helix gets smaller as it proceeds so the radius isn't the "minimum" but maybe the "base" radius.

Hmm good note; hadn't considered that. I don't think "base" radius is necessarily clear, as it's not clear what "base" means (I guess it's the radiux closest to the orgin?).
Perhaps:

When cone_angle != 0, the radius argument is the radius at the side of the cone closest to the origin. When cone_angle > 0, the radius expands outwards farther from the origin; when cone_angle < 0, the radius spirals inwards.


CAT-1918 : Adding check for version from metadata.json

Summary
This PR introduces a new check to ensure that only versions containing numerical values are considered for inclusion in the release process. The implementation involves retrieving the version information from the metadata.json file and validating it to contain only numeric characters. This enhancement aims to improve the reliability and consistency of the release workflow by preventing the inclusion of invalid or malformed version identifiers.
Additional Context
Add any additional context about the problem here.

[ ] Root cause and the steps to reproduce. (If applicable)
[ ] Thought process behind the implementation.

Related Issues (if any)
Mention any related issues or pull requests.
Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified.

I believe module version regex is defined https://github.com/voxpupuli/metadata-json-lint/blob/b5e68049c6be58aa63263357bb0dcad8635a6829/lib/metadata-json-lint/schema.rb#L141-L150

About above link yes we have already define versions for module but never know what might a malicious user will put in code...

Couple notes:

Does ubuntu-latest have a ruby or python interpreter in its default env? Computing the version with shell code is probably just going to be harder than it needs to be assuming it is easy to access a more advanced scripting language. I think you should consider using something like that if possible.

I'm curious about how using a high-level language would help in this case. Even if we use a high-level language like Ruby or Python, we would still end up executing it as a shell command (e.g., ruby ./<path to ruby/python script>). Or do you have a different approach in mind?
BTW the link which you shared above (https://github.com/voxpupuli/metadata-json-lint/blob/b5e68049c6be58aa63263357bb0dcad8635a6829/lib/metadata-json-lint/schema.rb#L141-L150) to fine the version which is considering the prerelease as well as build but we haven't come across such pattern for our modules as of now, so not sure if those expressions are the perfect for our use-case.

I'm fine with this being a separate step still, but i think it was only split out before for code reuse.

Hi @donoghuc, Thanks for your review. Wanted to explore your point 1 in details so have put my reply inline. If you can review and help me on the same to take it forward?

Also @donoghuc lets not over complicate the simple action most of the github actions are already used in such a way only, same thing is being done in gem release also
gem_release_prep

I believe module version regex is defined https://github.com/voxpupuli/metadata-json-lint/blob/b5e68049c6be58aa63263357bb0dcad8635a6829/lib/metadata-json-lint/schema.rb#L141-L150
About above link yes we have already define versions for module but never know what might a malicious user will put in code...

I pointed that out to say you should enforce that regex. The one you have here is insufficient for some of those valid formats.
I recommended using ruby or python for cleaning this input so you could just copy the logic from the schema over using jq and bash as I find those to be easier to read and maintain. I dont feel strongly about what you use to do the validation, just that the validation actually enforces the schema we expect.

Also @donoghuc lets not over complicate the simple action most of the github actions are already used in such a way only, same thing is being done in gem release also
gem_release_prep

We need to do this validation according to the module spec, the gem spec is different

@donoghuc As per the suggestion used ruby to resolve version tough I don't like such long like another way would be to add a script in repo and call it. But I believe this should suffice

@donoghuc accomodated your suggestions feel free to review and merge...

@malikparvez before this is merged the commits should be squashed into one with a detailed write up of what the change is and where the validation schema comes from in the commit message.

@malikparvez before this is merged the commits should be squashed into one with a detailed write up of what the change is and where the validation schema comes from in the commit message.

Already had updated the PR description, let me rebase it...

We need the information in the commit message so that it is part of the git history. That is how we can track changes to a project over time. The PR description is not in the git history, it is only on the github application.

We need the information in the commit message so that it is part of the git history. That is how we can track changes to a project over time. The PR description is not in the git history, it is only on the github application.

Well I usually like to keep the commit message short and crisp, further reference can always be traced wrt PR...

Vague commit messages are not acceptable. Looking at the history of this project it has been very difficult to understand the changes and how we got to this place. We need to be better at describing behavior changes and organizing those into logical commits so that when we audit issues like this we have justification and descriptions for critical changes.

@donoghuc also added the details in the commit message, lets merge this one and thanks for your guidance on this from the beginning...


[bulk] Validation: multiple issues

[bulk] Validation: multiple issues
This PR fixes issues in 4 files.

Docs Build status updates of commit f6674ce:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/preview/virtualmachines.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-quota-readme.md
:bulb:Suggestion
View (azure-java-preview)
Details

docs-ref-services/preview/virtualmachines.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 12, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'compute'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

docs-ref-services/preview/resourcemanager-quota-readme.md

Line 38, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8f91a68:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/preview/resourcemanager-recoveryservices-readme.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/virtualmachines.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-quota-readme.md
:bulb:Suggestion
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-redisenterprise-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-recoveryservices-readme.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.

docs-ref-services/preview/virtualmachines.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 12, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'compute'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

docs-ref-services/preview/resourcemanager-quota-readme.md

Line 38, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 46f79fe:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/preview/virtualmachines.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-quota-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-recoveryservices-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-redisenterprise-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/virtualmachines.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 12, Column 13: [Warning: ms-service-subservice-invalid - See documentation] Invalid value for 'ms.service': 'compute'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1ba7bef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/preview/resourcemanager-quota-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-recoveryservices-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/resourcemanager-redisenterprise-readme.md
:white_check_mark:Succeeded
View (azure-java-preview)

docs-ref-services/preview/virtualmachines.md
:white_check_mark:Succeeded
View (azure-java-preview)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@hallipr @dougeby
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!


Make spec tables sortable, e.g. shortcodes/gym-table

https://tabulator.info - i used this library for banknote-tabulator project, should be easy to integrate

Alternatively, use something lightweight that doens't break printability of the website, because #13


KCP - created with cookie

Describe the bug
Spam with "created with cookie" with different cookie numbers is coming up when client gets timeout and then returns to the game.
For example when user minimize his app on the android and then returns after timeout have occured, his connection does not drop instantly but instead spam with cookies goes on for a while before disconnect happens.
[IMPORTANT] How can we reproduce the issue, step by step:
Please tell us how to reproduce your issue, STEP BY STEP, with one of our built in examples.

Import latest Mirror version from git
Open scene MirrorBasic from Basic example and enable debug for KCP
Switch build target to Android and build it
Start in Unity editor as server only
Run your build as a client (I've used Bluestacks 5 emulator here)
Minimize your app on the device and wait 10s for timeout and server to disconnect the client
Now maximize back client's app and watch server's logs in Unity editor
There will be plenty of lines with "[KCP] kcp2k.KcpServerConnection: created with cookie=1884780640" (see screenshot bellow)

Expected behavior
"created with cookie" should not be spammed so many times
Desktop (please complete the following information):

OS: Windows 10
Build target: Android
Unity version: 2021.3.35 & 2022.3.21
Mirror branch: 89.6.0

To add that I've just found out that this seems to happen only when using emulator to run your app.
Tested now with 2 real different devices and it did not reproduce with these.

Assuming these are packets from the previous connection still arriving at the server, we could check if the cookie is 0 before creating the connection @miwarnec?
https://github.com/MirrorNetworking/Mirror/blob/9c9cc043b361891d3233d455e6c282931e0e2421/Assets/Mirror/Transports/KCP/kcp2k/highlevel/KcpServer.cs#L311-L347

I believe this is fixed in newer Mirror versions.

I believe this is fixed in newer Mirror versions.

Yes, this one is fixed now.
Just tested it with 89.9.0.


Removing/Changing background opacity?

I am really liking this setup, but I was curious if there was a way to change the opacity? I am hoping to create a slide-in menu from the bottom (thinking like the Twitter app) and for the overlay to be a bit darker at times.

It's slightly unclear how this applies to Angular. In Angular you would normally show the modal by:
    const options = {
      viewContainerRef: this.vcRef,
      fullscreen: false,
      context: {
        achievement
      },
      dimAmount: 0
    };

    this.modalService.showModal(ModalAchievementComponent, options);

So how does this.modalService.showModal apply to mainPage.showModal that's in your example.

I'd also like to know this. Had no success using Angular's ModalDialogService. It opens the default modal. Tried injecting a Page reference in Angular, but it won't accept the 'moduleName' param passed as the modal component.
this.page.showModal(ModalComponent, options)

@danielnitu please refer to the documentation to make sure you're overriding the modal funcion correctly: https://github.com/mukaschultze/nativescript-windowed-modal#typescriptangular
Also ensure you're using nativescript-angular >=7.2.4

@edusperoni Yes, it was my mistake after all.
The plugin works well, shoutout to all those involved in creating and maintaining it.


networktest: use StopDaemon RPC to stop lnd instead of SIGINT

This PR uses the StopDaemon RPC to stop lnd instead of SIGINT (or SIGKILL on Windows).
Currently, tests often fail with the reason:
lnd_test.go:67: Error outside of test: *errors.errorString lnd finished with error (stderr):
This is because lnd exits with error code 130 (stopped by SIGINT). When StopDaemon is used, the process exits without an error code. However, the code in this PR doesn't watch for an error returned from the RPC server as the connection is often dropped by lnd as it shuts down before the response can be sent/read.

Changes Unknown when pulling 0ae1f3765bc2b916c3d72fc1f990fdacccae7628 on aakselrod:lnd-finished-with-error into ** on lightningnetwork:master**.

Changes Unknown when pulling 0ae1f3765bc2b916c3d72fc1f990fdacccae7628 on aakselrod:lnd-finished-with-error into ** on lightningnetwork:master**.


Fixed the broken link in the doc (other-hadoop.html)

This fixes the link in http://druid.io/docs/latest/operations/other-hadoop.html
The fixed link :
http://druid.io/docs/latest/operations/use_sbt_to_build_fat_jar.md => http://druid.io/docs/latest/operations/use_sbt_to_build_fat_jar.html

:+1:

Thanks @jaebinyo ! Would you mind filling out http://druid.io/community/cla.html ?

I filled out the form and submitted.

Cool thanks! :+1:


Any way react-native-navbar will work with react-native 0.11.4?

I upgraded react-native to 0.12.0 to use react-native-navbar but now I am running into a lot of other issues. So, I was just wondering if there's any way to make this component work with 0.11.4

@vinu123 to be honest, I haven't tested it on 0.11.4, but I bet it will work with it without any problems. If can test it and confirm my thought, I would appreciate a PR to fix dependency version in package.json.

@Kureev Just tried. No luck.

@vinu123 you need to change dependencies hash in package.json first: >=0.12 should be replaced by >=0.11

@Kureev Works perfectly fine. Thanks so much.

@vinu123 Awesome, can you compose a PR with this change?

nvm, fixed in 1.0.3


[WIP]test CI

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/660/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/665/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/665/

please close this PR


Make throttle internal

Motivation
During the discussion in https://github.com/apple/swift-async-algorithms/issues/248 it became clear that the semantics of throttle are not 100% figured out yet.
Modification
This PR is making throttle internal to remove it from the upcoming 1.0.0 release. This gives us more time to investigate what exact semantics we want to have.

I think this should probably be public but underscored.

@swift-ci please test

@swift-ci please test


Only one gen_assets.go being ignored

There is a .gitignore file in cmd/query/app/ui/actual that ignores the gen_assets.go file, but there are three other files that are not ignored:
./cmd/query/app/ui/placeholder/gen_assets.go
./plugin/storage/es/mappings/gen_assets.go
./examples/hotrod/services/frontend/gen_assets.go

Should we just add a general entry in the top level .gitignore?

cc @pavolloffay  @jpkrohling

Closing after IRC discussion with @pavolloffay - this particular generated file is being ignored as iit is larger than the others.

We were checking all these static files before migration to esc (exception to UI bc that is large).
The build should regenerate them as part of specific build goals.  Another thing is that for example hotrod instructions from running from source allows direct go run ./main.go all - so no generation step needed.
Not checking them to codebase has one advantage - it assures that go run .. will always use the latest verison - but it requires to run generate step before. We will have to also regenerate as part of test goal


Compilation failed with webpack setup

I have clone the project with "dev-project-vue3" after that run the cmd 'npm i' getting package error. package alrady installed but still throwing error. please provide solutions

@manojditstek
The project "dev-project-vue3" cannot be cloned alone. It is the development and debugging environment of veaury. In the alias of webpack, the dependency of veaury will be pointed to the src subdirectory of the upper directory, unless you pass an environment variable BUILD_TYPE=remote , will point to the veaury installed by npm, which has been explained in the second screenshot you provided.

It is suggested that you can remove the point of veaury in alias.

@devilwjp Thanks for the explaination and suggestion.
If I configured with vite then project run and rendured react file but with webpack commented above line as you suggested, it's throwing error like this -

//Chat.jsx
export default function () {     return (
Hello
);   }

@manojditstek
This error comes from chat.jsx, because React is not imported in the file. Because your project is a vue project, some babel presets of react are not configured, so react will not be imported automatically, you need to import it manually in chat.jsx.
For example, add a line of code at the top of the chat.jsx file.
import React from 'react';

@devilwjp
Now it's working. Thanks for the help :)


Add table azure_log_analytics_cluster

References
https://docs.microsoft.com/en-us/rest/api/loganalytics/clusters/list
Note: This detail is required for NIST SP 800 53 Rev 5 controls.

There is no direct go API available, need to explore it more.


Market Research: Ask students the reason they are struggling with study

Create a Google Doc and question why students may struggle in school
Ways to do this

Talk with fellow students or teachers/peers
Interview other students
Read some student forums

Things to consider

Is note-taking really the reason students don't do well on tests?
How might their response relate to what we are building?
Is there something we might want to add to our app?

Format example

Student 1:  "I struggle with understanding my notes after I take them"
Student 2: "I struggle to concentrate while taking notes"

Sources
Student Doctor Forum
TheGradCafe
https://discuss.codecademy.com/t/question-of-the-week-note-taking-strategies/590373/3
Is note-taking really the reason students don't do well on tests?

Student: I use personal-made flashcards to avoid jotting too much notes from my lectures
Student: I summarize professor’s slides to avoid taking too much notes
Student: I take notes because it engages my brain. I don’t bother reviewing them
Student: I’ve seen many student taking pages of notes, but end up laughing about it later on
Student: “I’ve come to realize my obsessive note taking was a symptom of having too much free time”
Student: I use Anki, which is a flashcard app that helps me
Student: It’s nice to record and listen back through class sessions while studying
Student: "I much prefer taking notes by hand. It helps me really understand and work through what I’m learning"

Overall, it seems like the students who succeed are those that make work easiest for themselves. They take notes to stimulate their brains so the information becomes easier to recall.

It seems that more experienced students like to keep things simple and easy for themselves by taking less notes. And they achieve better results that way.
It also seems that less experienced students take more notes which makes things harder
It's likely that students want to increase how much information they call recall. So, it seems probable that they mostly use active recall to study

How might their response relate to what we are building?
I wonder if we could take student notes and find a way to challenge them in a fun and easy way. That would stimulate their brains without doing the work for them.

In what other ways could we process the notes we receive from students to make something that might stimulate their minds?
For example, could we use the notes we get from students and turn it into a simple game web game that would make things fun and increase active recall?

Market Research: https://docs.google.com/document/d/19tQY4co4WDu_I1dT_JPfN9ouIOEsou5r1G24OvczXQI/edit?usp=sharing

https://docs.google.com/document/d/1IUpH_y9m5wzxQHkJi_k5I-m_Yy9ewBr2a6xllPqRiLY/edit?usp=sharing

https://docs.google.com/document/d/1BzF-2aBvCqMoO2OwSopkBzaiQLyoukR_124gwRGZcto/edit?usp=sharing


Fixing Docker CI for security enabled tests

Description
Split our CI into 3 different workflows.

CI.yml -> This includes building AD, which includes UT and IT plus running multi node integration tests (linux, macos, windows). Windows is seperate here because I was not able to run it correctly with the -Dtest.logs=true option, only on linux and macos, I believe its okay for now as we only need that part for precision and recall and its enough to just see it on those two platforms.

Security-Test-CI.yml -> Assembles AD and sets up docker container to run AD integ tests with security plugin if its available. (Linux only cause no infra set up yet for windows and macos)

Bwc-test-CI.yml -> Runs BWC separately for AD (Linux only cause we don't have an older version with windows and macos)

Additionally it might be hard to notice cause of the new files but I changed some of the parsing logic in security-test-ci as it was failing to parse the correct docker version before.
Issues Resolved
resolves #275
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Security tests are failing here, I think this is a 3.0 specific problem as the same tests passed on this PR for 2.x: https://github.com/opensearch-project/anomaly-detection/pull/707
The errors are also related to http client which had some changes in 3.0 branch.
If rest of checks pass I believe we should still merge this in, backport the change and open an issue on security to further investigate errors.

Can we rename the workflows to follow a consistent convention? I think all underscore (verb-first) like mentioned here is a good way, or all--. I see our workflows are already a bit inconsistent

Codecov Report

Merging #710 (205ed14) into main (06e5edb) will decrease coverage by 0.31%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #710      +/-   ##
============================================
- Coverage     79.26%   78.94%   -0.32%
+ Complexity     4260     4234      -26
============================================
  Files           301      301
  Lines         17872    17871       -1
  Branches       1897     1897
============================================
- Hits          14166    14109      -57
- Misses         2810     2852      +42
- Partials        896      910      +14

Flag
Coverage Δ

plugin
78.94% <ø> (-0.32%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../java/org/opensearch/ad/AnomalyDetectorRunner.java
37.64% <0.00%> (-5.89%)
:arrow_down:

...ava/org/opensearch/ad/task/ADHCBatchTaskCache.java
88.88% <0.00%> (-3.71%)
:arrow_down:

...java/org/opensearch/ad/task/ADBatchTaskRunner.java
81.76% <0.00%> (-2.89%)
:arrow_down:

...arch/ad/transport/EntityResultTransportAction.java
86.61% <0.00%> (-2.37%)
:arrow_down:

...ain/java/org/opensearch/ad/task/ADTaskManager.java
76.52% <0.00%> (-1.44%)
:arrow_down:

...rch/ad/transport/AnomalyResultTransportAction.java
80.18% <0.00%> (-0.70%)
:arrow_down:

.../main/java/org/opensearch/ad/ml/CheckpointDao.java
69.74% <0.00%> (-0.64%)
:arrow_down:

.../main/java/org/opensearch/ad/cluster/HashRing.java
81.37% <0.00%> (-0.41%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
71.93% <0.00%> (-0.19%)
:arrow_down:

...ain/java/org/opensearch/ad/model/ModelProfile.java
70.90% <0.00%> (ø)

... and 1 more

More context on security failures here: https://github.com/opensearch-project/anomaly-detection/issues/712


Issue #257 : Update upload_collection method

Imported time and fixed the max_retries accordingly
This works
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Hi @retrogtx, thanks for your interest in qdrant-client
Unfortunately, these changes are not right from our point of view.
qdrant_client.py should not contain this kind of logic
Retrying the upload_collection operation is not correct, since it will retry to upload the entire collection from scratch.
Retry mechanism is inside upload_batch function in corresponding uploaders like https://github.com/qdrant/qdrant-client/blob/master/qdrant_client/uploader/rest_uploader.py and https://github.com/qdrant/qdrant-client/blob/master/qdrant_client/uploader/grpc_uploader.py

thanks for your response, I am actually a beginner to this open source stuff really
so what should be my next steps?
could the given logic work in the rest-uploader and grpc-uploader, so that I can code the same into them?

we've decided not to introduce this feature at the moment, thanks for you attempts!


DOM Exception: INDEX_SIZE_ERR (1)

i have this error:
Errore: DOM Exception: INDEX_SIZE_ERR (1)
if you have a big file (es 20MB) and you clic on the progress bar, when is loaded a different chunk, if you click again on a previus loaded chunk you see this error
cheers

Please provide a code sample to reproduce the issue. I will re-open this issue once the issue is reproducible.


Sorts credentials alphabetically before prompting the user during generation

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


Pushing image to desi registry times out when proxy is used

In /etc/default/docker ,
export HTTP_PROXY="corp proxy"
export NO_PROXY="deis-router-service-ip-address:80"
If HTTP_PROXY is not used, then, deis cannot pull the image from outside (docker hub)
If HTTP_PROXY is used, then, git push fails to push the image to the deis-registry (times out)
So added NO_PROXY for the deis-router-ip-address:80 . Still it is not able to push the image to deis-registry.
If HTTP_PROXY is removed, then, git push successfully pushes it to deis registry. But, should have the image ready by pulling it  with HTTP_PROXY on.
How to enable both?
Thanks

since the registry is internal it doesn't go through the router, it directly calls the registry service so you should set the NO_PROXY to "deis-registry-service-ip-address:80" if you are on v2.1 or before.
From the next release v2.2 which we will be releasing next week you should be able to just specify localhost as we drop a service on each node to proxy the requests

Hello,I have tried with NO_PROXY=deis-service-registry-ip-address:80. I always get this error. Not sure why it is trying to deploy to "localhost", instead of registry service address. This deis code is a week old code from github. Below is the log from deis-builder-instance2-
2016-07-13T15:52:49.858369201Z Traceback (most recent call last):2016-07-13T15:52:49.858458544Z   File "/deploy.py", line 120, in 2016-07-13T15:52:49.858325930Z Step 1 : FROM <IP_ADDRESS>:80/httpd:2.42016-07-13T15:52:49.858522365Z ---> d4ba297db48a2016-07-13T15:52:49.858531586Z Step 2 : COPY ./resolv.conf /etc/resolv.conf2016-07-13T15:52:49.858538559Z ---> Using cache2016-07-13T15:52:49.858545318Z ---> 8c32fba3f8ca2016-07-13T15:52:49.858553293Z Step 3 : COPY ./public-html/ /usr/local/apache2/htdocs/2016-07-13T15:52:49.858560356Z ---> Using cache2016-07-13T15:52:49.858567025Z ---> cee3071217da2016-07-13T15:52:49.858573774Z Successfully built cee3071217da2016-07-13T15:52:49.858580492Z Pushing to registry2016-07-13T15:52:49.858599704Z     log_output(stream, False)2016-07-13T15:52:49.858639743Z   File "/deploy.py", line 20, in log_output2016-07-13T15:52:49.858663361Z     for chunk in stream:2016-07-13T15:52:49.858678603Z   File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 217, in _stream_helper2016-07-13T15:52:49.870833599Z     data = reader.read(1)2016-07-13T15:52:49.870899076Z   File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/response.py", line 320, in read2016-07-13T15:52:49.882279299Z     flush_decoder = True2016-07-13T15:52:49.882384830Z   File "/usr/lib/python2.7/contextlib.py", line 35, in __exit__2016-07-13T15:52:49.901813750Z     self.gen.throw(type, value, traceback)2016-07-13T15:52:49.901897186Z   File "/usr/local/lib/python2.7/dist-packages/requests/packages/urllib3/response.py", line 233, in _error_catcher2016-07-13T15:52:49.901909829Z     raise ReadTimeoutError(self._pool, None, 'Read timed out.')2016-07-13T15:52:49.901917690Z requests.packages.urllib3.exceptions.ReadTimeoutError: UnixHTTPConnectionPool(host='localhost', port=None): Read timed out.Thanks!
From: Keerthan Reddy Mala<EMAIL_ADDRESS>To: deis/registry<EMAIL_ADDRESS>Cc: jolly2<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Wednesday, July 13, 2016 7:45 AM
Subject: Re: [deis/registry] Pushing image to desi registry times out when proxy is used (#61)
since the registry is internal it doesn't go through the router, it directly calls the registry service so you should set the NO_PROXY to "deis-registry-service-ip-address:80" if you are on v2.1 or before.
From the next release v2.2 which we will be releasing next week you should be able to just specify localhost as we drop a service on each node to proxy the requests—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

what is your docker version....this looks like similar to https://github.com/deis/registry/issues/60

Yes it is a similar problem . However, i see this problem when http_proxy and no_proxy are used together. If both are removed, deploy works fine. I would have to pull the images that i need from outside with http_proxy and then remove http_proxy to tag and push it to deis registry.
Docker version 1.11.2, build b9f10c9
Thanks

On Jul 13, 2016, at 10:19 AM, Keerthan Reddy Mala<EMAIL_ADDRESS>wrote:
what is your docker version....this looks like similar to #60
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Downgraded docker 1.10.3.  It is a different problem with deis-registry now. Still cannot use http_proxy and no_proxy settings together in docker daemon. I downloaded the images to a private repo on another machine and pulled and pushed into deis registry. Push seemed successful.  But, system became slow. gunicorn has hogged more than 100% of CPU for about 20 mins now. Deis is using all the 12 core cpu. Is there any config that I can tweak in to bypass this problem?

@jolly2 can you tell which process is hogging the cpu and get the logs of the corresponding deis component

ps -ef | grep 7494
106       7494  7372 55 08:53 ?        00:32:24 /usr/bin/python /usr/local/bin/gunicorn -c /app/deis/gunicorn/config.py api.wsgi
PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
7494 106       20   0  171272  47600  11296 R  99.6  0.2  33:59.03 gunicorn
System is extremely so to capture the logs..I will try. But, I had to hard boot the cluster everytime..
Thanks

@jolly2 just read through this issue. The reason the controller is deploying to localhost is because it is hitting the registry proxy. This is normal, and your proxy settings should reflect that.
As for the 12 core CPU issue, this has been resolved. There were known memory/CPU hogging issues in earlier versions of v2 that have long since been resolved. You should have a better experience if you're using v2.8.0.
If you have the time please feel free to try again on v2.8.0, but otherwise I'm going to close this as resolved for now unless someone else pipes up about proxy issues. Thanks!


Prefixes in sampler with runnig external code

Issue Description
What did you see?
When a code returns a csv (to an Optimiser in RAVEN) with multiple evaluations, RAVEN puts a prefix on to the evaluations. It looks like after that, the optimizer can not find the variables anymore to compute the gradient.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

Closure checklist passed...
Closure approved via #1163 (New optimization class)


Fix an error with an extraneous closing brace

The following closing braces did not correspond to the declaration of namespaces when C++20 is enabled.

C++20 is enabled:
https://github.com/LoliGothick/mitama-cpp-result/blob/fddf59f97930cc16a159e8b10b03198ea27d602a/include/mitama/thiserror/thiserror.hpp#L14

Lower version than C++20:
https://github.com/LoliGothick/mitama-cpp-result/blob/fddf59f97930cc16a159e8b10b03198ea27d602a/include/mitama/thiserror/thiserror.hpp#L18

https://github.com/LoliGothick/mitama-cpp-result/blob/fddf59f97930cc16a159e8b10b03198ea27d602a/include/mitama/thiserror/thiserror.hpp#L56

@LoliGothick こちらも、レビューする必要無いと思うので、マージしておきます！:raised_hands:


Implement DistributedLaunchMode.get_platform_information()

Imported from Bitbucket issue 21,  created by apdavison on 06-26-2010 at 21:24, last modified: 05-15-2013 at 09:55

Added a basic implementation of this in r131:72857ba8048d and r132:1a7fa8cfcffc, but it is very fragile and should not be relied on. This ticket really needs #20 to be resolved first, so I'm pushing this issue back to a later release.
Trac comment by apdavison on 07-01-2010 at 09:12


Messages vs MessageBase

The @aynik Messages class appears to implement an interface that's tightly coupled to the blockchain abstraction, which makes testing hard. Additionally, there's a number of odd object couplings inside the message class (see the #create's ChatMessage reference), and the class was not intended to be inherited from by messages, as per the ruby implementation. It would by my proposal that the Message class be migrated to the recently stubbed/created MessageBase implementation, and that the incumbent Blockchain class be renamed "BlockchainSpv" (or something similar)
@aynik - what are your thoughts here? If you look at the buyer class, this is how I believe these mesages should look whenever possible. Have a better idea?

the class was not intended to be inherited from by messages, as per the ruby implementation

I think I may have analysed this part incorrectly, what is the real purpose of it? I thought it was inheritance because I saw some polymorphism going on in other parts.

the incumbent Blockchain class be renamed "BlockchainSpv"

I agree with this in general terms, I think we should go forward and plugin-ize it, because I see alternative implementations using btcd (@krisives was interested in this part) or block explorers.
Let's first address the Messages issue and then we will follow with Blockchain, I will read now the code you submitted in depth.

I'm thinking that it may be a misunderstanding with the naming? I've checked the ruby code and Communication does inherit from MessageBase, well that should be the same as ChatMessage inheriting from Messages isn't it?

what is the real purpose of it?

I consider all non value-transfer transactions to be "Messages". These message types are outlined in the whitepaper, and this MessageBase class was modeled (roughly) after the ActiveRecord::Base class. Classes which inherit from MessageBase inherit the methods used to define the parameters of messages, as well as the methods needed to persist and load messages. (#find(), #save(), etc)

Communication does inherit from MessageBase, well that should be the same as ChatMessage inheriting from Message shouldn't it?

Indeed. Though it seems that ChatMessage has a bit too much logic in the class that should be moved into Message. I believe Message and MessageBase should be merged, I don't care what name we give that class (We can retain the name Message).
I believe the Messages class should be deprecated, and that find should move into the Message class itself. We may want to emulate the design of the orm2 library a bit more closely, and rather than return instances of (say) ChatMessage from finders, we return Hashes that are decorated with a mixin. Does this make sense? Thoughts?

Ok, more or less as I thought but I greatly appreciate the clarification.

it seems that ChatMessage has a bit too much logic in the class that should be moved into Message

You're right, I thought the time to do this will come soon after finishing the chat subcommand, at the time I wasn't sure about this because I did not inspect the other subcommands and message types, but I agree with this 100%.

I believe the Messages class should be deprecated, and that find should move into the Message class itself.

Yeah, at first I thought I would instantiate this to represent a collection of messages but as I completed chat it was more and more obvious messages don't need to be represented as a single object, hence the Messages class was left as a namespace just implementing find. This method can be renamed and relocated to the messages module itself, so using it is more intuitive (messages.findAll).
The orm2 collection and instance interface is not bad, but the library itself has given me some problems, I'll say let's use its method names for continuity but not it's ordering syntax and the other stuff.

we return Hashes that are decorated with a mixin

What do you mean with Hashes? TxIds?
Regarding caching right there, I still think is not the best. Drivers for bitcoind RPC or block explorers would not benefit from this as much as SPV, so I think most of the caching should be done there (and we'll have a much cleaner 'messages'). The decision to cache there was made to reuse dropzone_ruby's cache, so moving it from there will be soft-breaking (backwards compatible but transactions table will be no longer used) compatibility though.

Take a look at the newest commit and see what you think? Look at the implementation of the src/buyer.js and see if that wouldn't be a good model for the Communication class to follow. It's heavily inspired by the ruby implementation, but should allow for much easier testing and provides better interfaces for other connections down the line.
Regarding the hashes - I abandoned that suggestion. Ignore this.
If you're agreeable to the proposed interfaces, I'd like to rename MessageBase to Message, and ask that much of the existing Message code be inserted into a BitcoinSpv "connection". Would you like to do this? If so, I'll begin work on the other Message types, starting with their test code.

Referencing these here:
https://github.com/17Q4MX2hmktmpuUKHFuoRmS5MfB5XPbhod/node-dropzone/commit/66953e3224fea7e6bf0d0c1e95b262d8d580671f#commitcomment-13989425
https://github.com/17Q4MX2hmktmpuUKHFuoRmS5MfB5XPbhod/node-dropzone/commit/55f19ae7eda27d6acefc4d9dcbb92dc5ebed1557#commitcomment-13990592
https://github.com/17Q4MX2hmktmpuUKHFuoRmS5MfB5XPbhod/node-dropzone/commit/0c369a0fd239fb82000b9bbc90dc62d7793a0c53#commitcomment-13990676
https://github.com/17Q4MX2hmktmpuUKHFuoRmS5MfB5XPbhod/node-dropzone/commit/2a6ed4d1a5035eea3707f78f88dacb8f96759e51#commitcomment-13990720

See my response to those comments?


Update specs2-core to 4.12.5

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

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

labels: test-library-update, semver-minor

Superseded by #79.


How to set GOPATH correctly

I've correctly defined the GOPATH env variable and $GOPATH/bin is in the PATH, but I get the following error:
$ unik build --name ecspicsImage --path ./ --compiler rump-go-virtualbox --provider virtual box
INFO[0000] running unik build                            args=[] compiler=rump-go-virtualbox force=false host=localhost:3000 mountPoints=[] name=ecspicsImage path=./ provider=virtualbox
INFO[0001] App packaged as tarball: /Users/denisjannot/.unik/328098395
ERRO[0018] build failed: [cmd/build.go:95] building image failed: {[client/images.go:54] failed with status 500: [daemon/daemon.go:254] failed to compile raw image: {[rump/generic.go:116] Returned non zero status}}
And on the daemon side:
INFO[0352] Created container                             binds=[/Users/denisjannot/.unik/160396933:/opt/code] cmd=[] id=6ec8ef34dda379339dd16cf3e05ed54cd44e9a71952a796a4f1c102b3df15910
ERRO[0365] Container exit status non zero                status=1
ERRO[0365] 8+ cp /tmp/build/gomaincaller.go /tmp/build/mainstub.c .

[ -d Godeps ]
'+ GO15VENDOREXPERIMENT=1 godep restore
Tgodep: [WARNING]: godep should only be used inside a valid go package directory and
Xgodep: [WARNING]: may not function correctly. You are probably outside of your $GOPATH.
/godep: [WARNING]:	Current Directory: /opt/code
godep: [WARNING]:	$GOPATH: /go
?godep: WARNING: Go version (go1.5) & $GO15VENDOREXPERIMENT=1 wants to enable the vendor experiment, but disabling because a Godep workspace (Godeps/_workspace) exists
?+ CC=x86_64-rumprun-netbsd-gcc CGO_ENABLED=1 GOOS=rumprun /usr/local/go/bin/go build -buildmode=c-archive -v -a -x ecspics.go gomaincaller.go s3.go
WORK=/tmp/go-build677402343
Qecspics.go:17:3: cannot find package "github.com/codegangsta/negroni" in any of:
A	/usr/local/go/src/github.com/codegangsta/negroni (from $GOROOT)
7	/go/src/github.com/codegangsta/negroni (from $GOPATH)
Iecspics.go:18:3: cannot find package "github.com/gorilla/mux" in any of:
9	/usr/local/go/src/github.com/gorilla/mux (from $GOROOT)
/	/go/src/github.com/gorilla/mux (from $GOPATH)
Necspics.go:19:3: cannot find package "github.com/gorilla/sessions" in any of:

/usr/local/go/src/github.com/gorilla/sessions (from $GOROOT)
4	/go/src/github.com/gorilla/sessions (from $GOPATH)
Mecspics.go:20:3: cannot find package "github.com/unrolled/render" in any of:
=	/usr/local/go/src/github.com/unrolled/render (from $GOROOT)
3	/go/src/github.com/unrolled/render (from $GOPATH)

ERRO[0365] error handling request                        error=[daemon/daemon.go:254] failed to compile raw image: {[rump/generic.go:116] Returned non zero status}
Do I need to set the GOPATH inside the Unik Instance Listener VM ?

so, it looks like your error is related to the fact that you are using unrolled/render in your application.
It's mentioned in the documentation here that go apps should have their dependencies vendored in (that means there should be a folder called vendor in the root of your application source, which contains the sources for your app's dependencies. read more here.
This is because compilation is happening inside of an isolated environemnt (docker container) and is not using your local GOPATH. Therefore, you should install Godeps and run GO15VENDOREXPERIMENT=1 godep save ./... from the root of your application source (wherever you're pointing to with the --path flag).

I can move forward with GO15VENDOREXPERIMENT=1 godep save .
I have another error now, but not related to GOPATH anymore.
Thanks !

what's the error?

It was related to the cloud foundry module, but I didn't need it, so I've removed it and was able to build it.
The problem I have now is that I don't know how to include the subdirectories (where I have my html templates and other files)

currently, static files won't be included in go unikernels (they will for Node.js). we're working on adding this feature and it should be done soon.

for now, to package static files, i suggest using a project like Go-Bindata to compress static assets into Go source code so they'll be included with your unikernel

Thanks. I'll wait for the release that will include this feature


net: enable TCP keepalive on new connections from net.Dial

In our experience, missing network timeouts are one of the first sources of resiliency bugs in server applications. Unfortunately, these bugs are rare to experience during development and are not very easy to write tests for. As a result, we often see components hanging because somewhere in a dependency three layers down someone forgot to add an explicit timeout to a network connection/request/call. This is the result of many factors, one being that many languages (Go included) does not have default timeouts for network operations even though having a default timeout would align better with the common case (how many times do network operations require to not have timeouts?)
I realize this may be seen as an over-reaction to the problem, but I would argue that if the Go standard packages had default timeouts set on all network operations a whole class of resiliency problems would, for the majority of uses, disappear.
A couple of notes:

I'm not arguing it would solve all network-triggered resiliency problems in all cases, only in a good majority.
I'm also not arguing that users shouldn't be allowed to explicitly disable such timeouts, or to set them to something different. While I argue that the common case is that network operations should have a timeout, I agree that there may be a minority of cases where no timeouts make sense (the most common being when a different mechanism to detect timeouts is in use).

To summarize the proposal:

A suitable set of default timeouts should be identified (e.g. in https://github.com/golang/go/issues/22982 @bradfitz suggests 3 minutes as reasonable for http.DefaultClient)
The default timeouts identified above should be added to the Go standard packages

I'm unsure if this would break go1 compat, so feel free to classify as Go2 if required.

Let's enable TCP keepalive by default and then see what problems still remain to be solved. It seems like TCP keepalives would handle essentially all "accidental" (not malicious) long-lived connections.
Does anyone object to enabling TCP keepalive by default?

I have no strong opinion on enabling TCP keepalive by default. The following is just random thoughts.

TCP keepalive is a mechanism that tries to keep in-flight TCP connections alive by using zero-payload TCP segments, I mean, not a feature like application-level dead peer detection. If the peer application gets stuck for some reason and remains the established TCP connection alive, the mechanism cannot detect such condition.

The original issue sounds to me like there should be a better application-transport API surface including some help for implementing the dogma: computer networking? it's kinda illusion, huh? if you have any problem with it, don't tackle the problem directly, don't try to reclaim the lost stuff, do abort and retry. In addition, perhaps the net package might be not suitable for such purpose because the package intersects platform- and network-specific features and provides only read and write operations for byte sequences. I think it's better to have the abstraction, a bit more well-cooked network operation things such as Messages, Requests, Responses, and Calls, in packages other than the net package.

@rsc I don't have anything enabling TCP keepalives by default. I actually think they will solve part of the problem but, just to clarify, I don't think we can say that

TCP keepalives would handle essentially all "accidental" (not malicious) long-lived connections

TCP keepalives are normally handled by the TCP implementation in the kernel, so a "stuck" application would not be detected (until it's killed or crashes). Also TCP keepalives have the downside of being applicable only to TCP. :sweat_smile:
This is the reason I think starting with TCP keepalives is a good first step.

OK, let's do TCP keepalives on by default.
Also, Brad suggested making the http.Server configuration have zero duration timeouts mean "a good limit" instead of "unlimited". That sounds good too.

Great! Should I also open a followup go2 proposal to review the net APIs in light of what was discussed in this issue?

I was giving this a shot but I ran into something unexpected. Currently in net there is only support to set the keepalive interval/delay, but not the number of unacknowledged keepalives before killing the connection (TCP_KEEPCNT/tcp_keepalive_probes).
I suppose this means we are using the local kernel default for TCP_KEEPCNT, whatever that may be (e.g. on Linux this should be 9, on Windows 5, on Mac, FreeBSD and OpenBSD[1] 8).
Not being able to control this parameter means we can't precisely set the target total time before the keepalive mechanism considers the connection broken, since the total time is (initial delay + interval * count) and in go initial delay == interval.
What is the desired course of action here?

add a (private) method to set TCP_KEEPCNT on each platform; override the kernel defaults for our sockets and target the same total time on all platforms
assume that the local kernel defaults have not been changed; use different keepalive intervals (e.g. 18s on linux, 30s on windows, 20s on mac/bsd) depending on the platform so that in the "unchanged kernel defaults" case all platforms have the same total time (3m)
ignore the problem and use the same interval regardless of the platform; each platform will have a different total time (e.g. assuming an interval of 15s, on linux the total time would be 2m 30s, on windows 1m 30s, on mac/bsd 2m 15s)

[1] As a side note, it seems the openbsd support for keepalives states that openbsd doesn't support keepalives, but this does not seem to be the case.

@rsc if you have nothing against this, I will try to go with the third option ("use the same interval regardless of the platform")

Does using net.ipv4.tcp_keepalive_time as the default keepalives interval sound like a good idea to you?

@zhyon404 we don't do that (and this issue is already closed...)

@pam4 and I discovered that keepalive is disabled by writing to a dead link #31490


issue with git :cry:

when im inside a Git repo, git command act weird when i use tab to auto complete the command jump to middle of screen
EX :
$USER@HOST $ git p

after i click TAB git command jub
$USER@$HOST $ git                                    git pull

thats happen only inside Git repo
can any one help me please

With default configuration? More details?

is ok i found the problem and is done it was a zsh_prompt-git color problem
thnx for care no one those here oh-my-zsh help is f********

Good to hear that. You may want to close this issue if your issue is solved.


CORS issue - Potential resolved

In this PR I have added the cors module to support OPTIONS request and updated Github URL for koi-logs

@Salman-Arshad, there was a yarn.lock conflict so Im just adding it manually.

Sure
On Fri, 2 Jul 2021 at 1:00 AM, The Lone Rōnin @.***>
wrote:

@Salman-Arshad https://github.com/Salman-Arshad, there was a yarn.lock
conflict so Im just adding it manually.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ArweaveTeam/gateway/pull/84#issuecomment-872512529,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIQV5FJK3BK7V3ZO66CF2KLTVTCNDANCNFSM47IRLKSA
.


Unexpected behavior of getter methods for invalid dates obtained from parsing with patterns

When parsing invalid strings with patterns such as 'M', 'Y', L'(also when using strict mode), a moment.js object is returned that gives the current date when using getter methods such as month() or year() instead of an error.
format() still returns 'Invalid date' as expected.
var foo  = moment('xyz', 'L', true)
foo.month()
//=> 8
// expected: NaN
foo.year()
// => 2015
// expected: NaN
foo.format('l')
// => 'Invalid date'

Without parsing patterns, the following expected behavior is shown:
var bar  = moment('xyz')
bar.month()
//=> NaN
bar.year()
// => NaN
bar.format('l')
// => 'Invalid date'

I tested this with moment.js 2.10.6.

Without a pattern, it's falling back to Date object, which treats the string as invalid.
With a pattern, it's using moment's parser.  However, the true indicates strict mode is enabled, which clearly the string xyz does match the L pattern (which is MM/DD/YYYY in the default US English locale).

moment.js seems to be aware that the sting is not valid:
var foo  = moment('xyz', 'L', true)
foo.isValid()
// => false

Interesting!  Thanks.

Fixed in #2624.


FEATURE - i18n module

Description
Tools need to be able to read and write i18n entries for apps. There are various flavours of i18n files and we should have common way for handling all of them, that can be easily reused by tools.
UI5 resource modules (properties files)
UI5 manifest.json translation entries (properties files)
CAP CDS services (properties, CSV and JSON files)
Technical Design
What modules need to be created or enhanced?
New module @sap-ux/i18n
How would the functionality be exposed and how can it be consumed?
Acceptance Criteria
GivenWhenThen format (https://martinfowler.com/bliki/GivenWhenThen.html)
Given I have a ...
when I execute ...
then ... will happen.
Notes
Tasks

[ ] Parsers

[ ] properties
[ ] csv
[ ] json

[ ] CDS support
[ ] UI5 support
[ ] Consider mem-fs support?
[ ] Use common function to get i18n properties in https://github.com/SAP/open-ux-tools/pull/1475/files#diff-f2599904e616ac2446bef03eb16046f806b7c637da5b005a856be5abf0228a9aR97
[ ] Unit Tests added
[ ] Test Cases defined

Added in #1655


How do portal signs work?

I can't get them to work.

[Portal]
Warpname
And you go one block above the sign

Solved I hope

Yes, solved.


Change workflow with environmental variables and config file.

This PR stores all mongod default values in config file /etc/mongod.conf - to be able to read this file after docker run or docker exec and read there default or actual values. New values from OpenShift configuration options are updated in config file at the begging of run-mongod.
Enviromental variable are now declared in run-mongod to be declared in one place to improve maintainability (for example MONGODB_KEYFILE_PATH was declared twice). The values of these variables are obtained from /etc/mongod.conf.

This PR is part of #112

@omron93 to be honest I didn't understand very well the description :confused: . Could you please explain what is the purpose of this change?
The manipulation of the configuration file using grep and sed is much more fragile compared to the existing envsubst being replaced here.

 There is no MongoDB config file.
The purpose of the redesign of the scripts in #100 is to do entrypoint very simple and move a main script in CMD - normaly executable script which can be easily run inside container. To be able to make some changes after docker run -ti IMAGE bash and after that run the same script as during normal start.
With this change it should be possible to change some values in MongoDB config file and and the run the same script (run-mongod) and have mongod working.
Without this change I don't see any benefits of having config file (since it is not available before running run-mongod, and also some mongod options are passed in the command line - so there is not everything in config file and config file do this more complicated). So it would be better to pass all mongod options in command line.

[test]

@omron93 i would prefer envsubst here as well... sed/grep are not the greatest tools for doing templating of config files.

Without this change I don't see any benefits of having config file (since it is not available before running run-mongod, and also some mongod options are passed in the command line - so there is not everything in config file and config file do this more complicated). So it would be better to pass all mongod options in command line.

@mfojtik @rhcarvalho So what about removing config file completely?

@omron93 your idea of moving everything to the config file is good. It's much easier to look at the config file than having too many command line flags. I'd also suspect that not everything you can do in the config file can be done with flags (?).
I don't see a use case like you described, that a user runs the image with some alternative entrypoint/run bypassing the envsubst, and later wants to start mongo using the config file.
I think this alternate startup is unlikely, and even if a user does that, she can later use the provided image scripts to start mongo. Why would somebody want to do this? Perhaps there's some other improvement around that.
My proposal would be trying to get "best of both worlds", to use the config (template) for all we can (moving certain flags we pass today into the config template), and use envsubst to replace placeholders using environment variables. @mfojtik @omron93 agree with that?

I'd also suspect that not everything you can do in the config file can be done with flags (?).

Both methods are equivalent.

your idea of moving everything to the config file is good. It's much easier to look at the config file than having too many command line flags.

Yes, that's right. However in current situation from user POV configuration is not placed in standard location and also it contains content that have to precessed by envsubst so does not contain values and IMPOV it has no value for user.
From devel POV I think it is similar to situation where all config options for mongod is set on one place in the script. Also not all options are easily movable to config file - for example configs for replication (in that case, there should be two config files).

Why would somebody want to do this? Perhaps there's some other improvement around that.

For example to change some mongod configuration is same way with the same file as for other mongodb installations (from rpm,...). And then run image scripts.

However in current situation from user POV configuration is not placed in standard location and also it contains content that have to precessed by envsubst so does not contain values and IMPOV it has no value for user.

AFAICT, the current state reflects what worked for v2, and in that case the normal means of customizing configuration was via environment variables. I don't know if there was any way to provide your own configuration file. Please correct me if I'm wrong.
@omron93 as you pointed out, and I agree, there might be cases where a user wants to reuse an existing configuration file from other installations.
Would it be reasonable to support both scenarios?

If a configuration file is provided, use it as-is, try not to override anything with command line flags;
If no configuration file is provide, the current case today, allow customization via environment variables and use defaults where applicable.

Could that be done for both the standalone and replication cases?
How to provide a configuration file through OpenShift and plain Docker?
@omron93 @mfojtik @bparees what do you think about that?

Would it be reasonable to support both scenarios?

@rhcarvalho I like this idea :+1:

Could that be done for both the standalone and replication cases?

Replication adds two mongod options which affects standalone mongod server. So they can't be in standard config file. So it is possible to have two config file templates (one for standalone, second for replication) or add these two options in command line.

How to provide a configuration file through OpenShift and plain Docker?

In plain Docker it could be possible to mount config file (and if we support both cases, it should be normally detected). In OpenShift I don't know. However current method will still work.

@rhcarvalho So can I try to implement it?

@omron93 yes, sure and thank you! A combination of config templates for the common cases and ability to drop-in a custom config file may work fine for both OpenShift and pure Docker use cases.
@grdryn would it be useful for RHMAP to be able to provide a custom config file for mongo?

Closing in favor of #147


Commits in IntelliJ Idea (and other JetBrains IDEs)

Is there any way to use the hook functionality in IntelliJ Idea as well?
I have set the hook and it works. But not when I call or execute Commit within the IDE.
So far I have found the following:

That there used to be a plugin that could call this hook, but has not been developed for years and is therefore no longer compatible. Plugin Page

That there used to be a setting for hooks in 'Version Control' -> 'Commit' -> 'Before Commit'. intellij-support

i believe JetBrains should have some config for opening a file to add a commit message, like VSCode does. when file is opened, OC would generate a commit and paste it into a file

Thank you very much for the answer.
I really couldn't let go yesterday and tried everything to get it to work somehow. Without success.
Apparently, there is really no way to execute the prepare-commit-msg hook in IntelliJ Idea. (Without doing it manually)
Thanks a lot for your effort and this great tool. I always find it fascinating what AI can do today and how it makes so many things so much easier and more enjoyable. This is especially true for commit messages.
For the other IntelliJ IDE users:
Nevertheless, I found a slightly different solution, which I added my own touch:
The AI Commit Plugin adds a little colorful tick above the commit field in the UI, which fills it at the push of a button based on the selected changes (well, it also makes a request to the gpt-3.5-turbo-0301 API, which then fills it in). I wasn't quite happy with the UX of this plugin (and still not quite) and created a Fork with some changes, a diif. prompt and more visual feedback.
Maybe this is helpful for one or the other.

I wasn't quite happy with the UX of this plugin (and still not quite) and created a Fork with some changes, a diif. prompt and more visual feedback.

@Marc-R2 How to install your forked version to intellij idea?

@MutahharAllshore On the Release page of the repo, there is currently pre-release version 5.1.0, which you can download as a zip from the assets. You can manually install the zip in your IntelliJ IDE in the Plugin Manager via the Gear -> 'Install Plugin from Disk'.


Permission file for linux server

sorry, ia have problem to load css file in my admin template. do you have permission code to make it this template work ?

try: sudo chmod -R 0644  /var/www

Thanks @mitchellurgero - hopefully that solved the issue!


FIX: symbolic links on non NTFS partition

When analyzing data on FAT32, I encountered a non-explicit error with mne_watershed_bem as it attemps to create a symbolic link, which isn' possible on all types of partitions.

That sounds atrocious ... Harddrive mess
On Fri, 20 May 2016 at 21:15, Jean-Rémi KING<EMAIL_ADDRESS>wrote:

When analyzing data on FAT32, I encountered a non-explicit error with
mne_watershed_bem as it attemps to create a symbolic link, which isn'
possible on all types of partitions.
You can view, comment on, or merge this pull request online at:
https://github.com/mne-tools/mne-python/pull/3241
Commit Summary

FIX: handle impossible symbolic link
add warning

File Changes

M mne/bem.py
https://github.com/mne-tools/mne-python/pull/3241/files#diff-0 (13)

Patch Links:

https://github.com/mne-tools/mne-python/pull/3241.patch
https://github.com/mne-tools/mne-python/pull/3241.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/pull/3241

so you'd be against this kind of fix?

JR, no not directly. Can we add a force flag maybe to make this more obvious. People would then be explicitly prompted to this behavior, ideally by an error message telling them to put force to true in this case. Can we individuate this error? I mean can we know if it was this problem that prevented the linking from succeeding?

Can we add a force flag maybe to make this more obvious.

Sure, I'm happy with that. So I add a force param, and throw a guiding error in case symlink crashes?

Can we individuate this error? I mean can we know if it was this problem that prevented the linking from succeeding?

yes, symlinking on a FAT32 crashes

Sounds like a good approach.

yes, symlinking on a FAT32 crashes

But can the code know if it is a symlink-on-FAT32-crash? Maybe let's try to catch the error and show the error + append our hint to the force flag, saying that it does copies instead.

It's a permission error so it's not completely specific. I'll add a force option, and if sysmlink crashes with a perm error I'll suggest to check whether the partition is NTFS

Sounds good. You mean a prompt to NTFS or FAT32, these two are different.

@dengemann Shall I use force or force_copy as a param?

We have no precedent for this. If force is contextually sufficiently clear
prefer it, else the longer one
On Fri, 20 May 2016 at 23:22, Jean-Rémi KING<EMAIL_ADDRESS>wrote:

@dengemann https://github.com/dengemann Shall I use force or force_copy
as a param?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/pull/3241#issuecomment-220721615

Not sure how to deal with codecov error.
Also note that I don't think I can add a test, since we don't have Travis on FAT32...

@agramfort I didn't find any other occurrence of os.symlink, just some flash bem documentation asking the user to symlink their files.
I would feel safer to add a test, but I don't know how to do this for this kind of low level issues.

thanks @kingjr


TechInsights Plugin Enhancement to show related Facts

Hey, I just made a Pull Request!
Solves #19458
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)

[X] Added or updated documentation

[X] Screenshots attached (for UI changes)

[X] All your commits have a Signed-off-by line in the message. (more info)

@freben I would be very grateful for feedback and suggestions!


Rehost all used models on IPFS

@voodoohop I think this is a really cool initiative to ensure the integrity of the results. We just need to take into account how much space we need upfront on the IPFS server.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 196 ms

Resolved: ClamAV performance has improved in b865ec8.


please add Spanish podcasts to EBookFoundation Master List

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Improve Repo. I noticed that the Spanish podcast file does not contain many podcasts. I have added three useful data science related podcasts to the list.
For resources
Description
This PR features three data science podcasts in Spanish.
Why is this valuable (or not)?
No existing repertoire of data science podcasts in Spanish. These additions allow Spanish speaking technologists to explore the fundamentals of data science, including topics such as machine learning.
How do we know it's really free?
All podcasts are available through the site that was linked. No need to purchase or subscribe.
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

if spanish podcasts needs adding to Readme, please submit a PR


Remove testing dependency on requests

Datatypes's HTTPClient has most of the features, including making the basic request header. I think it's only missing the functionality to upload files found in endpoints.client.HTTPClient.post_file, if I add that to datatypes's client then I think I can remove the dependency on requests with very little work since Datatypes's client is designed to be a simple drop in replacement.

If I wanted to switchover to an async like requests library, here are some options:

niquests (pypi)


CNX-9214 Rhino - Send Geometry Conversions

https://spockle.atlassian.net/browse/CNX-9214
POC comment issues are collected into a parent story for now: https://spockle.atlassian.net/browse/CNX-9266

[x] Pending adding XML comments to all conversions.
[x] Pending PointCloud to Speckle conversion
[x] Ellipse conversion needs looking into as it is not being picked up.

Comment for posterity:
Ellipses were decided to be excluded from Send, as there are no ellipses ever in a rhino document: one can create a Nurbs curve from the mathematical data of an ellipse, and the ellipse can be retrieved in a non-deterministic way using Rhino's API.
Since this is not ideal, we're just sending them as NURBS, which simplifies the entire conversion logic significantly.


The Devil Inside (Videos not working anymore)

I need help! I did a clean install copied the files and the fixes! (Yours  and Audio) but videos are not working anymore...
Here is a Debug Log of The Devil Inside with Patch #1 (Plus DSOAL and ogg-winmm):
DDrawCompat-Devil.log.zip
I don't understand what changed! If this problem has nothing to do with your wrapper! I'm lost! xD
I'm using the Patch#1 EXE without any changes! :)
Your Wrapper fixes:

Most of the Graphics!
DDraw error!
Videos (Intro, etc)
Transparency issues (Helicopter looks beautiful!)
Random crashes! (Most of then except for my setup because secondary fixes)
Keyboard and Mouse works correctly!

Thanks for all that and much more! Your Wrapper is my favorite one!
Because it just works! The dgVoodoo2 is awesome too!
Originally posted by @Zerodarkage in https://github.com/narzoul/DDrawCompat/issues/352#issuecomment-2336752882
I have to close this one for now! I suspect something went wrong with my sound drivers!

Are you using a no-cd patch? If yes, don't forget to copy the Datas\Movies folder from the CD to the same directory in your install folder.

Are you using a no-cd patch? If yes, don't forget to copy the Datas\Movies folder from the CD to the same directory in your install folder.

I forgot about the NOCD...
Thanks!!! :D


[proposal] add "backgroundDecoration" to Scaffold

Use case
A common way decorate the background of a scaffold,
beyond the backgroundColor property (maybe an image or a gradient),
is to add a stack in the body.
By definition the scaffold body is The primary content of the scaffold.,
which is not the use case for a background decoration.
Proposal
Adding a Widget or BoxDecoration parameter backgroundDecoration
could make the usage of the scaffold cleaner
and better separate the content and "un-clutter" the body from the underlying decoration.
     Scaffold(
        backgroundDecoration: BoxDecoration(
          image:  DecorationImage(
            //...
          ),
        ),
        // body: Stack(
        body: ActualContent( //...

Hello @iapicca. Thank you for filing this request. Looks like a valid one. Keeping open for further insights from the team

There is a partial patch for this in https://github.com/flutter/flutter/pull/119007 that somebody interested in working on this could pick up and bring over the finish line.


turtlevcs: init at 0.6

Closes #252317
Adds a turtle package.
The nautilus plugin requires gnome.nautilus-python to be installed separately.
The package is a bit weird, since turtle_cli subcommands are implemented through subprocesses
of either $PYTHON or sys.executable. This doesn't work well with buildPythonApplications hooks
for setting up python dependencies.
I'm not very familiar with neither python packaging in general, nor nixpkgs' infrastructure for it,
so there might be better ways of achieving this.
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

The package needs a meta attribute.

Is there a reason you choose the by name category?

It is recommended in the documentation (since #237439 was merged).

oops, thanks for the heads up!

I removed the trailing comma and set $PYTHONPATH in the wrapper to get rid of the extra derivation.
Thanks for the suggetsion

Done

Wow, this is nice! Although it has been sitting for a while... Let me see if ofborg complains:
@ofborg eval

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1452


How do I have a bot respond to messages in a LiveChat channel?

I want my bot to be able to respond to a query in a live chat channel and extract data from the conversation, so I tried calling the bot by name in the live chat channel.  I can see in the console that the bot knows it is being called by a certain room, but it is unable to respond.  How can I get the bot into the live chat conversation?  It keeps getting stuck after the TryCache method in the rocketchat driver.

Same question

Add bot as the only agent
Send message via livechat once so the bot can enter the room
Send the following messages mentioning it. Ex: hubot ping

I have  the same problem.
I have set up hubot as the only livechat-agent
when I send the mesage from chat I have this in logs:

when I send the message from livechat only this line appears to log:

My hubot logic is on a catch all block:

Can you tell me if there are any 'tricks' in setting up hubot rocketchat adapter or rocketchat server?
Does anyone succeded on having hubot answering to the livechat?
I mention that I don't use the internal rocketchat hubot, but I use the hubot-rocketchat docker image

Hi again,
I managed to solve the issue.
I dig into your code and you have this:
RespondToLivechatMessage = (process.env.RESPOND_TO_LIVECHAT or "false").toLowerCase() is "true"
but RESPOND_TO_LIVECHAT is not documented on your page as being an environment variable.
Please update the documentation.
Regards,
Vali

Hi. I have set Rocket chat and Hubot, added hubot as only agent for Livechat, but Livechat doesn't see the agent online and doesn't allow user to start Livechcat conversation.
If I add a 'real' user to livechat agents, the Livechat works as expected, but without Hubot.
Hubot running as external service and has its own username in Rocket chat.
It does respond to direct messages (like 'help') and in #General channel (like 'test').
here are environment variables:
docker exec -it hubot-rocketchat printenv
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
HOSTNAME=cbad60eb2288
TERM=xterm
RESPOND_TO_DM=true
LISTEN_ON_ALL_PUBLIC=true
ROCKETCHAT_ROOM=''
EXTERNAL_SCRIPTS=""
ROCKETCHAT_URL=mychathost:3000
BOT_NAME=my.bot
RESPOND_TO_LIVECHAT=true
ROCKETCHAT_USER=my.bot
HUBOT_LOG_LEVEL=debug
ROCKETCHAT_PASSWORD=my.bot
NPM_CONFIG_LOGLEVEL=info
NODE_VERSION=4.8.3
YARN_VERSION=0.24.4
BOT_OWNER=No owner specified
BOT_DESC=Hubot with rocketbot adapter
HOME=/home/hubot

I can't log Livechat messages because the 'client' side of chat says it's offline, so I can't even send a livechat message from a website.
I've added Hubot both as livechat-manager and livechat-user, but it seems it doesn't work.
I think that Hubot doesn't register itself as livechat-agent.

Finally got Hubot responding, but with strange settings.

I've set both human and hubot both as livechat agents and managers
I've set Livechat Routing Method = Guest Pool

Then I've finally got Livechat working on client side and sent message. Hubot brain now receives livechat messages, but it responds only to standard messages like 'help', 'ping', etc.  But it doesn't respond to 'custom' messages like 'test' in Livechat.
It does not respond to these kind of messages in DM as well. I think it's the root cause of not responding in Livechat.
It DOES respont to message 'test' in #general (reply was: "Test? TESTING? WE DON'T NEED NO TEST, EVERYTHING WORKS!", as expected).
See logs.
Livechat test:
[Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)] INFO Message received with ID id
[Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)] INFO Message receive callback id byvgPzm3GW9zqtHie ts Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)
[Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)] INFO [Incoming] guest-3: test
[Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)] INFO Message sent to hubot brain.
[Fri Nov 09 2018 06:16:01 GMT+0000 (UTC)] DEBUG No listeners executed; falling back to catch-all

[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO Message received with ID id
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO Message receive callback id aJJKa5SSTG7wBE83A ts Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO [Incoming] guest-3: ping
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO Message sent to hubot brain.
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] DEBUG Message 'my.bot ping' matched regex //^\s*[@]?(?:my.\.bot[:,]?|![:,]?)\s*(?:PING$)/i/; listener.options = { id: null }
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] DEBUG Executing listener callback for Message 'my..bot ping'
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO Sending Message To Room: wEj4hKcP3BaAqJMK6
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] DEBUG Found cached Room ID for wEj4hKcP3BaAqJMK6: wEj4hKcP3BaAqJMK6
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] DEBUG Preparing message from string
[Fri Nov 09 2018 06:16:06 GMT+0000 (UTC)] INFO Message received with ID id

#general channel test log:
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Message received with ID id
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Message receive callback id rsB3P9W7RPwziiQfC ts Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO [Incoming] admin: test
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Looking up Room Name for: GENERAL
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO setting roomName: general
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Message sent to hubot brain.
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] DEBUG Message 'test' matched regex //^test$/i/; listener.options = { id: null }
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] DEBUG Executing listener callback for Message 'test'
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Sending Message To Room: general
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Looking up Room ID for: general
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] DEBUG Preparing message from string
[Fri Nov 09 2018 06:20:31 GMT+0000 (UTC)] INFO Message received with ID id

DM test log:
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Message received with ID id
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Message receive callback id 3xeDFvcaAxMnf2s6j ts Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO [Incoming] admin: ping
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Message sent to hubot brain.
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] DEBUG Message 'my.bot ping' matched regex //^\s*[@]?(?:my\.bot[:,]?|![:,]?)\s*(?:PING$)/i/; listener.options = { id: null }
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] DEBUG Executing listener callback for Message 'my.bot ping'
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Sending Message To Room: fEpvDSrCcpfZFMFDAuRKWDTjiY2FtcjZNe
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Looking up Room ID for: fEpvDSrCcpfZFMFDAuRKWDTjiY2FtcjZNe
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] DEBUG Preparing message from string
[Fri Nov 09 2018 06:28:36 GMT+0000 (UTC)] INFO Message received with ID id

[Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)] INFO Message received with ID id
[Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)] INFO Message receive callback id kAS3Twne7aoyAhmzJ ts Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)
[Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)] INFO [Incoming] admin: test
[Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)] INFO Message sent to hubot brain.
[Fri Nov 09 2018 06:28:40 GMT+0000 (UTC)] DEBUG No listeners executed; falling back to catch-all

My hubot coffee script is pretty simple:
module.exports = (robot) ->
  robot.hear /^test$/i, (res) ->
   res.send "Test? TESTING? WE DON'T NEED NO TEST, EVERYTHING WORKS!"

  robot.respond /^test$/i, (res) ->
   res.send "Test? TESTING? WE DON'T NEED NO TEST, EVERYTHING WORKS!"

It seems I have two issues:

Hubot doesn't register itself as online agent when It's the only Livechatagent
Hubot doesn't respond to custom direct messages (maybe script problem?)

UPDATE. This coffeescripts works in DM and Livechats.
Somehow Hubot doesn't want to use regexp wildcards for direct messages
module.exports = (robot) ->
  robot.hear /^test$/i, (res) ->
   res.send "Test? TESTING? WE DON'T NEED NO TEST, EVERYTHING WORKS!"

  robot.respond /test/i, (res) ->
   res.send "Test? TESTING? WE DON'T NEED NO TEST, EVERYTHING WORKS!"

So far left the only issue:

Hubot doesn't register itself as online agent when It's the only Livechatagent

O meu problema está que em uma das interações, quando ele o usuario, manda o email, ele retorna com o nome do bot, causando a interrupção da interatividade. Será que falta alguma configuração? Pois dentro da sala fazendo testes, a interação acontece normalmente, e externamente não.
'red<EMAIL_ADDRESS>Aonde red é o nome do meu bot.

there's now a "assign bot's to livechat" setting in rocketchat

This does not work for RASA bot.
Followed documentation on rasa bot and it answer if I post in #genaral channel, but if I post to livechat a queue appers but the webhook created does not call the bot