Bug fixed. Now the Sequence property is also Updated in the Update Cu…

Issue #187 fixed. Included a change that was already done in PR #186 and a new change in ObjectCustomActions.cs that makes the Sequence property being updated.
Please, let me know if I'm doing anything wrong with the PR.

Hi @luismanez, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

I have done the same pull request #186. However, I did not include the Provisioning part in it.
This one seems to be the preferred solution.

I saw that. You've fixed a bug and all the credit is for you about that bug. I'm fixing a different bug (related). I'm just including your change trying to don't cause any issue when the admin does the Merge.
Thanks for your fix!

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

Good fixes! Thanks! Regarding the merge issue you tried to solve by including the fix from @Trickass : it's actually better to -not- include changes that come from other PRs. Now his PR either would generate a merge conflict, or his PR would be listed as not relevant, as the change is already merged through this PR.
For contributions we keep track of the merged PRs and find the names of people submitting the PRs. Technically speaking @Trickass introduced a fix that will now be attributed to you through this PR. We will take a note however now to make sure that @Trickass will be attributed at the monthly master merge too.
A better solution would be: pull down the PR from @Trickass to your own branch in your own Repo: add your added code/changes to that branch, commit, and create a new PR from that branch. That way the commit from @Trickass will be retained and attributed to him. I will perform those tasks now so that both of you will be attributed.

Oh God, so sorry @erwinvanhunen and @Trickass I had no idea about how to do it right and I'm too new with Git. How can I pull down a specific PR from other user? If you want, just cancel my PR and I'll start again.
Thanks for your help and my apologies again.

No worries! :-) I went through the same thing when learning Git :-) It's all fixed now, so no need to create a new PR. To pull down a PR from someone else:
// Create a new branch
git checkout -b
git remote add
git fetch
git merge /
Then you publish the  branch to you github repo and make a new PR from that one. But as said, it's all done now :-)

There was a thing with the PR from @Trickass and that is that he was targeting the Master branch of Core. We do not accept PRs on that one so I manually retargeted his commit too while merging both PRs

Oh, my bad there then! :$
Any chance you could tell me how to do it better next time? How do I target another branch?

No worries about that :-) It's pretty simple to fix, but also easy to miss. I did it myself many times. When you create the PR, at the top of the page you will see your source branch and which branch you're targeting in the PnP-Core repo. You can switch from the master to the dev branch there through a drop-down.

Perfect! many thanks Erwin
@Trickass If it helps, I use to follow this article to do the PR: https://github.com/OfficeDev/PnP/wiki/Preparing-a-contribution-and-letting-the-PnP-core-team-know-via-a-pull-request
Cheers.


[Storage] Update StorageAccount.KeyCreationTime type with swagger change

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[x] Include updated management metadata.
[x] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

Hi, could you provide the link to the openapi spec review?

@JonathanCrd
The swagger API is in: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/storage/resource-manager/Microsoft.Storage/stable/2021-02-01/storage.json
The is the swagger PR for this change: https://github.com/Azure/azure-rest-api-specs/pull/13757

@JonathanCrd , @allenjzhang
Would you please help to merge it asap?
After it's mreged, we need to release the .net SDK, then raise Powershell PR which depends on .net SDK.


Add Personnel record

As an user I would like the ability to Add a new personnel record so that I can include additional personnel into the system

This is enabled but needs to be be cleaned up.


Add Platforms Tags and Page

TLDR:

replaces category tag with platform tag
page for each platform we support (JS, JVM, Native)

Preview
While open, this PR is available for preview here:
https://valencik.github.io/typelevel.github.com/platforms/
(note however that some links will be broken there)
I recommend checkout out that preview because these screenshots are probably already out of data (they are).

Platform Pages
I would like for these pages to:

talk a bit about the support and capabilities on each platform

e.g. the native page should link to Arman's blog post
e.g. the js page should briefly discuss the JS environment options: browser, severless, cli

list the projects that are supported by each platform

I want to be slightly more than just another project breakout list.
However, that might be achieved over multiple PRs so we can divide up the yak shaving.
Tags
This PR also removes the category tag in favour of the platforms tag.
I am hoping this isn't too controversial.
I do not think the category tag is useful in its current form, a possibly more useful tag might pull in the GitHub topics, but that is out of scope here. The category tag is also not universally used, and it almost requires a bit of moderation. Whereas the platform tag is a lot more factual.

Hey folks, I think this is finally ready for review.
You can preview the changes at: https://valencik.github.io/typelevel.github.com/platforms/js/

I don't think it's an either/or, at least in theory -- personally, I would leave category in place, but we should rethink what the breakdown is.
That said, I agree that platform is higher-priority.

I think the category tag could certainly come back in the future if someone gives it some love.
But as is I think it's incomplete, and perhaps noisy on the project cards.


fix(hubble): use strconv.Itoa for kafka.ErrorCode + add unit tests

Fixes: #35749
Use `strconv.Itoa` instead of `string()` for the correct behavior when converting `kafka.ErrorCode` from `int32` to `string`. Add relevant unit tests for Kafka plugin and handler.

/test

oops didn't realize clicking Update Branch on the Github UI will do that 🥲

/test

@joestringer looks like it is failing the conformance test for some reason. Also, I'm curious about the process of backport a bug fix to previous Cilium's versions. Do I need to open a different PR for that?

The failure looks unrelated, I filed https://github.com/cilium/cilium/issues/35872 for it and will re-run the test. For backport we can mark it for v1.16.


Fix GetMouseDelta() issue for Android

As mentioned in the discussion regarding the split of rcore.c by platform, here is the fix for the issue with Vector2 GetMouseDelta(void) on Android.
This now allows obtaining the delta for the first touch on Android. Tests were conducted with multitouch, and there were no problems or interference reported.

@Bigfoot71 Nice! Great improvement!


Search also elsewhere in the system sys.path

When I tried checking out this tool, nothing seemed to work on my system. I tried gi, gtk, PyQt5 on Python 2.7 and 3.4, but I always got the typical ImportError: No module named gi. After a lot of puzzling and digging around in the code, I found that vext is only searching in the system pip's install location (in my case /usr/lib/pythonX.Y/site-packages).
So why is this a problem? Because for some reason, all Python packages from my distro's (Fedora 23) repos happen to be installed in /usr/lib64/pythonX.Y/site-packages.
So I think vext should probably search everywhere in the system's sys.path that's not in the virtualenv's sys.path, not only in pip's install location.

Can you see what the output of import site; site.getsitepackages() is ?
Vext incorrectly assumes just one sitepackages dir:
from vext import env
env.getsyssitepackages()
Out[4]: '/usr/lib/python2.7/dist-packages'

Whereas on my system this isn't quite right:
In [1]: import site; site.getsitepackages()
Out[1]: ['/usr/local/lib/python2.7/dist-packages', '/usr/lib/python2.7/dist-packages']

I'll grab a vm of fedora and have a play.   Originally I did add everything from the sys.path, but now am trying to add just the minimal needed - it might make sense to revisit this, will have to have a think.
Python 3.4 needs more work to work generally, but py 2.7 should be working, so will try and fix.

>>> import site
>>> site.getsitepackages()
['/usr/lib64/python3.4/site-packages', '/usr/lib/python3.4/site-packages', '/usr/lib/site-python']

Same for 2.7.

K, I'll add in multiple site packages in the short term, and in the longer term move to vext adding all paths in systems sys.path, but gatekeeping them.

Hi,
I got this working - though you need to do pip install -U pip setuptools in the virtualenv first.    So far I've only updated vext.gi  but will upload the rest of the packages fairly soon.
I'll try and have a look into the detection for earlier setuptools.
S

OK, fixed the other problems so should install with pip install vext.pyqt4 etc.
If anything doesn't work, feel free to re-open this.


Internal network marked 'down' incorrectly

Error description:
I've got several services working great using this setup (thanks for your work!) However I've got one service not working properly, and I'm having a hard time diagnosing. I've got the following docker-compose file for the service gogs:
version: '3'

services:
  gogs:
    container_name: gogs
    image: gogs/gogs:latest
    restart: always
    ports:
      - "9022:22"
      - "9007:3000"
    networks:
      - reverse_proxy
    environment:
      VIRTUAL_HOST: my.subdomain.com
      LETSENCRYPT_HOST: my.subdomain.com
      LETSENCRYPT_EMAIL: my@email
      VIRTUAL_PORT: 9007
    volumes:
      - /data/services/gogs:/data

networks:
  reverse_proxy:
    external:
      name: webproxy

I'm able to connect at <IP_ADDRESS>:9007, but not from the given subdomain. When I look at the conf.d/default.conf output by nginx-gen, I see the corresponding entry:
# my.subdomain.com
upstream my.subdomain.com {
  ## Can be connected with "webproxy" network
  # gogs
  server <IP_ADDRESS> down;
}

Expected result
I would expect the upstream to be something like:
upstream my.subdomain.com {
  ## Can be connected with "webproxy" network
  # gogs
  server <IP_ADDRESS>:3000;
}

And if I change that configuration manually and restart nginx-web, the subdomain works just fine. Any thoughts on getting nginx-gen to recognize the upstream? Thanks in advance!

Try  to update your VIRTUAL_PORT to:
VIRTUAL_PORT: 3000
Or even remove this option that might work also.

Oh wow, I've been using this incorrectly apparently. :smile: Thanks for your help, that worked like a charm.


Prism proxy modifying files

Inb4 thank you for great tool!
Issue Description:
When using the Prism proxy to proxy requests to another backend, it has been observed that the library is modifying files during the proxy process.
This behavior is unexpected and can lead to validation errors.
Steps to Reproduce:
Choose a Prism version v5.2.0.
Set up a proxy configuration to forward requests to a target backend server (e.g., http://backend.example.com).
Initiate a request to the proxy server, providing a URL that should be forwarded to the backend.
Monitor the content of the files that are being proxied and compare them to the original content.

Expected Behavior:
The library should act as a transparent proxy, forwarding requests to the target backend without altering the content of the files being proxied.
The files received from the backend should remain unchanged and identical to the original files.
Actual Behavior:
During the proxy process, the library is modifying the content of the files being proxied.
File comparison functions (utilizing PHP's built-in functions) indicate disparate hashes between files. Moreover, methods for retrieving Mime-Type yield distinct outcomes. For instance, the original file, characterized by the Mime-Type 'application/vnd.ms-excel' (denoting an .xls file), undergoes a transformation during proxying, resulting in a Mime-Type alteration to 'application/octet-stream'.
Additionally, there is a discrepancy in file sizes. In the original context, the file has a size of 7680 bytes, whereas after passing through the Prism proxy, its size increases to 10942 bytes.
Impact:
Validation problems: File not pass validation rules based on mime-type.
Data Corruption: The modifications introduced by the library during proxying can lead to data corruption, making the proxied content unusable or inconsistent with the original data.

Additional Information:
Library Version: v5.2.0

Workaround:
At this time, there is no known workaround to prevent the library from modifying files during the proxy process.
Users are advised to avoid using the proxy functionality of the affected library until the issue is resolved.
Proposed Solution:
Investigate the codebase of the library to identify the root cause of the file modifications during the proxy process. Implement a fix that ensures the content of proxied files remains unchanged when passed through the proxy.
Environment:
OS Ubuntu 20.04
PHP programming language to validate hash values and ascertain Mime-Types.

@Freeshnik , thanks for your report.  At this time, Prism doesn't support binary data in either mock or proxy mode.  However, we agree that it should and welcome PRs, especially for the easier proxy case.

Hi @EdVinyard  is there any progress on the above issue.

Hi people
I do face the same issue. So in our setup we have prism validation proxy running to validate our traffic against the Open API contract. However there is one endpoint for downloading pdf files. Unfortunately the pdf file is being modified when passing through the proxy. This is a huge drawback since it makes it unusable for us. I would expect the proxy to pass binary data or even the whole request and response the way it originally was (except custom validation headers). This does not seem to be the case.
I also removed the endpoint definition from my open api contract to avoid validation in assumption that the file is not being modified. However this did not fix it so it must be an issue with the proxying mechanism.
Is there a current work around for this problem? It would be massively helpful to fix this issue.

Hi @adrianburkard I had the same issue a few months back and we tried Schemathesis Hooks and it's working as expected. If you are testing your Open Api specifications against your API Endpoint Schemathesis could be an alternative tool.
Check the below link whether it works for your requirements.
https://schemathesis.readthedocs.io/en/stable/extending.html#generating-payloads-for-unknown-media-types


Adding category "Network" for each network provider policy

Adding the metadata object with Category = Network for each Azure Policy.
Fixed identation for each policy to be 4 spaces instead of 2 spaces to provide a consistency across the json files.
Some policies has more specific category set, such as "Azure DNS", these have not had their category updated.
I've only updated tags on policies without the metadata category attribute.

Same as other submissions please fix properties, name on all of them
Plus unresolved Git conflicts

@SebastianClaesson  https://github.com/Azure/Community-Policy/blob/master/CONTRIBUTING.md#contribution-guide

@SebastianClaesson https://github.com/Azure/Community-Policy/blob/master/CONTRIBUTING.md#contribution-guide

Hi @techlake , I'll go through all of these. Seems they all might have been created since before the contribution guide. I'll submit a new PR once it's done.

Thank you.


Store name and surname separately

The program will ask the user to input the following:
First name:
Last name:

Fixed in cb5af37cb9a2e73ffdfeda326914940cb6929cb6


CMake 3.XX rewrite

This is my third pull request on this repository.
Goals
Let's face it, the CMake system is old. While it works, it's not very usable, and was designed on old CMake principles which are now outdated, and in some areas, considered bad practice. This PR is intended to updated CMake to use today's target/property based standards.
Priority Changes

[ ] Removing/Updated all CMake scripts to use targets/properties over directory based linking
[ ] Remove all unnecessary compiler definitions, and replace them with proper compiler supported macros over CMakeLists defined compiler defs.
[ ] Add, within CMake, the ability to use include-what-you-use, link-what-you-use (CMake built-in), and another static analysis tool.
[ ] Switch away from git submodule whether it be git subtree or some other method, sub-module likely will not stay.
[ ] Rewrite Appveyor and Travis CI scripts to work with the updated build system.

Low-Priority Changes

[ ] Create a linux binary.
[ ] Ninja build system on the CI systems.

Note
Many of the commits this branch receives will not compile. This is to be expected when modifying the build system in such a transformative manner.
Please let me know of any ideas, changes, comments, or criticisms on the information I have written above.

This PR is being closed on account of too many changes being required all in a single PR. A new PR will be opened with a more minor change compared to the end goals listed in this PR.


net: openthread: Frame Pending Bit improvements

A couple of improvements in Frame Pending Bit handling:

Allow specifying Frame Pending Bit mode (Thread, Ziggbe handle it differently),
Forward the information, whether the radio driver acknowledged the frame with FPB bit set.

The latter fixes Thread Sleepy End Devices, which are currently broken in Zephyr (it stopped working due to upstream OpenThread changes introduced in recent OT update).

@jukkar Thanks, fixed.

I've rebased and resolved the conflict.


Further improve LG's performance

Three operations to improve LG's performance

Cache Antlr parsing result of TemplateBody  to reduce the repeated parsing
Remove unnecessary visit operation to reduce the visiting time
[UpdateTemplate/AddTemplate/DeleteTemplate Method] Separately parse the template that needs to be updated, without parse the whole LG file.

2, 3 looks perfect sense to me.
For 1, adding cache to template body, we should be very carefully choosing which kind of cache we want to use and how to expire it.  If 3 can solve the issue in Composer's scenario very well, we should evaluate whether 1 (caching body) is necessary.
For 3, we need to tests more on different cases, like keep updating\adding\deleteing templates.

You are up early...
Get Outlook for Androidhttps://aka.ms/ghei36

From: Tom Laird-McConnell<EMAIL_ADDRESS>Sent: Saturday, June 27, 2020 6:34:32 AM
To: microsoft/botbuilder-dotnet<EMAIL_ADDRESS>Cc: Chris McConnell<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [microsoft/botbuilder-dotnet] Further improve LG's performance (#4083)
@tomlm approved this pull request.
[:shipit:]
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fbotbuilder-dotnet%2Fpull%2F4083%23pullrequestreview-438698809&data=02|01|chrimc%40microsoft.com|828545c1b95047fcb55408d81a9ed3c5|72f988bf86f141af91ab2d7cd011db47|1|0|637288616749436531&sdata=8a7GbtG2cRBQi8axECbJC6Tr5si3SAoTuesw85s9rXc%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEGSCMLKGJ2YDPOBAIQUDQLRYXYORANCNFSM4OALOKYA&data=02|01|chrimc%40microsoft.com|828545c1b95047fcb55408d81a9ed3c5|72f988bf86f141af91ab2d7cd011db47|1|0|637288616749446527&sdata=8nQGgiWP%2BwPLNSN%2BNuUZxQDe6pClf6fHTOlWw92wCOs%3D&reserved=0.


bug: menu bar of docs website is misleading when narrow

Describe the bug
When viewing https://docs.kcp.io/kcp/main/ in a browser window of the width I typically use, the menu bar is incomplete and the user gets no clue that items are omitted.  The attached screenshot shows an example.

Steps To Reproduce
Visit that web site and play around with the browser window width.
Expected Behaviour
When items are omitted I expect a concise way to reach them (e.g., hamburger menu or "..."), or at least a visual clue that there is more to see.
Additional Context
No response

Closing as we've switched to mkdocs with a new theme and this should be resolved. @MikeSpreitzer please let me know if you see otherwise.


[cpackget] Installed pack is not purgeable

Describe the bug
  cpackget rm --purge ARM.CMSIS.6.0.0-dev53+g825ef36.pack

Fails to remove the pack version from .Download folder.
To Reproduce
Steps to reproduce the behavior:

cpackget add https://github.com/ARM-software/CMSIS_6/releases/download/dev%2Fv6.0.0-dev53/ARM.CMSIS.6.0.0-dev53+g825ef36.pack
cpackget rm --purge<EMAIL_ADDRESS>See error:

I: Using pack root: "C:\Users\jkrech\AppData\Local\Arm\Packs" (default mode - no specific CMSIS_PACK_ROOT chosen)
I: Removing<EMAIL_ADDRESS>E: pack not purgeable

In verbose mode you can see the file name pattern match:
D: Listing files and directories in "C:\Users\jkrech\AppData\Local\Arm\Packs.Download" that match "ARM.CMSIS.6.0.0-dev53+g825ef36.(?:pack|zip|pdsc)"
Probably the . - + characters confuse the regular expression search.
I don't know why the tool does search for an exact filename?
Expected behavior
The pack version file is located in the .Download folder and must be removed:
Environment (please complete the following information):

OS: Windows
Version 10

Additional context
N/A

Semantic Versioning specifies that two versions that only differ after "+" (build meta information) are considered "identical".
This means the build meta information should be ignored.
When the user refers to a pack version with "+" is should only be considered if the user is specifying a file (filesystem or url).
It could make sense, to rename the pack version and strip the version after "+" in .Download. The pack version directory must not include "+1234".

usgin cpackget version 2.1.3
cpackget add https://github.com/ARM-software/CMSIS_6/releases/download/dev%2Fv6.0.0-dev84/ARM.CMSIS.6.0.0-dev84+gc53c0f5.pack

stores downloaded pack in $CMSIS_PACK_ROOT/.Download/ARM.CMSIS.6.0.0-dev84.pack
installs into $CMSIS_PACK_ROOT/ARM/CMSIS/6.0.0-dev84
cpackget rm --purge<EMAIL_ADDRESS>and
cpackget rm --purge<EMAIL_ADDRESS>
both remove the installation and the packfile correctly.
Therefore this issue can be closed.


Tommy - Sprint 6: Portfolio + Presentations + TeleOp coding

By 24/12/20, I will:
Begin and finish programming and testing the intake for the yellow sample Autonomous route and specimen alliance specific Autonomous route
Finish writing the autonomous and business sections of my presentation script and the autonomous section of the Engineering Portfolio

12/9
Started the autonomous section of the Engineering Portfolio
Had our 15-minute Stand Up Meeting to discuss what was worked on, what will be worked on, and obstacles

12/11
Filed intake
Had our 15 minute Stand Up Meeting to discuss what was worked on, what will be worked on, and obstacles

12/12
Shaved the 3D-printed version of intake

12/13
discussed options about hang and game strategy
Organized new carrying case for game pads


Autobuild docs on PRs

From https://docs.readthedocs.io/en/stable/guides/autobuild-docs-for-pull-requests.html.
This would be nice for contributions.

Works now!


Why this project needs go get github.com/tools/godep

Why don't you use godep save -r  to rewrite source code?

The import path looks very ugly and I don't like it.

Cool reply.

:-)


Fix build on aarch64

Scope & Purpose
Fix the places where x86 dependencies had been introduced unconditionally which prevented builds on AARCH64 from completing. With this PR, builds can once again be performed on AARCH64.

[X] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Backports:

[X] No backports required
[ ] Backports required for: (Please specify versions and link PRs)

Testing & Verification
(Please pick either of the following options)

[ ] This change is a trivial rework / code cleanup without any test coverage.
[X] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

Hi,
this probably at least partially duplicates the effort in this branch: https://github.com/arangodb/arangodb/pull/13391

Yes, @dothebart there is certainly a lot of overlap. My code simply enables AARCH64 to build again so may be worth commiting in the short term unless you pretty much have the #13391 ready to merge soon.
At least the fix of USE_SIMDCOMP is possibly still worthwhile.

Hi,
meanwhile the devel branch comes with ARM support. If you feel that any of your sugestions are still missing, please open a new PR.
Thanks for your contribution.


Use msg_witnessblock instead of msg_block in all functional tests

In https://github.com/dtr-org/unit-e/pull/798 only the minimally necessary adjustments were done to functional tests to make them pass. This leads to confusion sometimes when working on them as blocks seemingly arbitrary fail to validate when introducing new features.

Bitcoin PR https://github.com/bitcoin/bitcoin/pull/15664

I think we better remove msg_witness_block. Because why make a distinction? It is a block. And we don't have non-witness ones

We can do that. For the sake of clarity when reviewing I would prefer to do it in two steps:

Replace all msg_block with witness block (makes it clear which places had to be touched and what was affected)
Rename msg_witness_block to msg_block (this latter step should be trivial then)

I did go with making it msg_block in one shot... #1029


Multiprocessing local game

Migrated a tool from the development of the strangefish bot into the reconchess package. MultiprocessingLocalGame and play_multiprocessing_local_game allow users to test games without server connections that mimic the behavior of server-based games more closely.

In particular, I would like feedback in two places:

My attempt to follow the format and organization of reconchess with the names and placement of the new classes and functions.

My switch-case style implementation of _respond_to_requests using a dictionary of lambda functions. To accomplish the same task without lambda functions requires 16 more function definitions, which seems to be less readable than lambdas in this specific case.

I am also concerned about the addition of import multiprocessing as mp into reconchess/play.py. If importing the multiprocessing package each time only play_local_game or play_remote_game is needed is enough of a drawback, play_multiprocessing_local_game and its sub-functions could be moved to a different file that must be imported directly (reconchess/mp_play.py).


<IP_ADDRESS> doesn't count as a local host

Hello,
I'm setting up our rails app in Semaphore CI for the first time today and the default settings out of the box for a Rails workflow includes a postgres install and DATABASE_URL where the host is "<IP_ADDRESS>"
This fails Database Cleaner's safeguards with:
DatabaseCleaner::Safeguard::Error::RemoteDatabaseUrl:
ENV['DATABASE_URL'] is set to a remote URL. Please refer to https://github.com/DatabaseCleaner/database_cleaner#safeguards

Where the docs say that the host has to be one of: "localhost, .local or <IP_ADDRESS>" - any reason why <IP_ADDRESS> isn't included?
For now I can change the semaphore ENV var to use "localhost" but it would have been nice for their default <IP_ADDRESS> to have been okay with this gem, if there is no particular reason not to include it.
However, I'm happy to find out if there is something I am not understanding, please let me know, and thank you for this wonderful gem that has served me for nearly 15 years!

@joemsak Hi, thanks for submitting this issue! I think this was not considered because most of the people using this gem would probably use localhost or <IP_ADDRESS> so this sounds like an edge case.
I see that sometimes <IP_ADDRESS> is used to refer to "this host":

https://www.rfc-editor.org/rfc/rfc5735#section-3
https://unix.stackexchange.com/questions/419880/connecting-to-ip-0-0-0-0-succeeds-how-why

So I would be open to a PR that adds a new value to this constant: https://github.com/DatabaseCleaner/database_cleaner/blob/main/lib/database_cleaner/safeguard.rb#L42 👍🏼

Thank you so much @etagwerker - I've got advice from a coworker, someone with pretty deep technical knowledge, he says using "<IP_ADDRESS>" is "iffy" and "platform dependent" and he "hates" when it's used. So I think for now the recommended approach is to change OOB defaults, such as this one from Semaphore CI, to match what is already accepted by the gem.
Hopefully this PR at least helps in a search for the next person (-:


Specify content-type in uploading images

Currently when the user uploads a file from the compose box, we always set the content-type to application/octet-stream.
Instead, we should set the content-type to an appropriate specific value, based on what we know about the file.
Before Zulip Server 9.0 (and before Zulip Server in main until last week), this didn't matter because the server would look at the extension in the filename to decide how to handle the file. As part of the thumbnailing work, though, the server now relies on the content-type set by the client. The symptom is that images don't get displayed in the message at all.
Reported by @alexmv. Chat thread here.

Implementation notes:
It looks like adding the content-type in the HTTP request will be straightforward. We'll need our ApiConnection.postFileFromStream to add a parameter for it, as well as its caller the uploadFile endpoint binding.
Then it's a matter of finding the right content-type to set. If the platform gives us a content-type, we should use that. If it doesn't, we can look at the filename's extension ourselves and make an inference; for comparison, see getMimeTypeFromFileExtension in src/utils/url.js in zulip-mobile. I'd guess that when getting an image or video from the camera, the platform tells us the content-type, but that when picking an existing file it may not — after all, it doesn't really know any more than the file's name and contents either.


The -optimize flag does not always give optimal solution

The following command can give the optimal convergence layers as 11, 10, or 6. Obviously it should only give 6.
protocon -def M 2 -x ../examplespec/TokenRing.prot -def N 2 -param N 3 -style '((future & shadow) % puppet)' -optimize

Adding actions can introduce a cycle, which adds more fixpoint states and can decrease the worst-case number of steps to reach a fixpoint. This is what's happening for the token ring, but optimization should still work for silent protocols.
We can't use a metric for "number of steps" that can decrease when actions are added. As long as the over-approximation of legitimate states does not grow, we could just use number of convergence steps. However, as the code is currently written, the over-approximation can grow if we add an action that is not a candidate for that system, which can happen if multiple systems are being considered simultaneously.
I'm investigating possibilities in Issue #4. It is a recurring concern worth documenting.

Fixed in commit ae8ec74cefe998511290f950852db89db62fb2a8 by keeping the candidate lists consistent across systems during synthesis.


Build a search state to enable sharable URLs

Beefy would like to allow for sharable links to Vault search parameters. For instance, a quick-access URL that shows the Vault list pre-filtered for Stablecoin vaults on the BSC network might look like https://app.beefy.com/?type=stable&chain=bsc
All filter types from our Filters should be supported, although for readability, should not simply follow the keys defined here, but should have their own mapping so that we may tweak them as needed.

Values for chain should use our canonical names rather than the ID.
Values of "all" should simply be default/no value.

On the frontend, as users further refine their filter options, the state should be updated to reflect the current parameters so that the page can be shared as-is.

Good one, I am going to start working on it

I made my first attempt at that! #1211


Update cpanfile

The explicit dependency on bioperl in the cpanfile was missing. This adds it in.

DO NOT MERGE!!!
@EbiArnie  - I was too optimistic ... for reasons cpanm installs BioPerl 1.7
See excerpt from TravisCI log below.
==> Found dependencies: Bio::Seq, Bio::DB::RefSeq, Bio::Tools::Run::RemoteBlast, Bio::SeqIO, Bio::DB::GenBank, Bio::Root::Version, Bio::DB::GenPept, Bio::DB::SwissProt, Bio::DB::EMBL
--> Working on Bio::Seq
Fetching http://www.cpan.org/authors/id/C/CJ/CJFIELDS/BioPerl-1.7.8.tar.gz ... OK

Guess it's safe to close this as it looks like no changes will be applied anyway


ux(gui-client): Buttons in the webview look like they're never clicked

Describe the bug
The buttons don't change color when the mouse button is down, so they only appear pressed for 1 or 2 frames
To Reproduce

Open the Settings windows
Go to either tab
Click any button
It looks as if the button was not clicked

The button does twitch because the text changes, at least on the settings Apply button.
Expected behavior
Expect a normal cycle of the button reacting to mouse hover, mouse down, mouse up
Screenshots / Logs
N/A
Platform (please complete the following information)

Component (i.e. macOS client / Linux client / Gateway / Admin portal): Tauri Client
Firezone Version (e.g. 1.0.0 or N/A): 1.1.2 (latest)
OS and version: (e.g. Ubuntu 22.04 or N/A): Windows and Ubuntu
Deployment method: (e.g. Docker / Systemd / App Store or N/A): Any

Additional context
N/A

Fix here is to add a depressed state to the buttons. Possibly want to style them more like the native OS and not like our website too.


add custom state check

related issue: https://github.com/kyma-project/lifecycle-manager/issues/196

/test pull-lifecycle-mgr-tests


Release Google.Cloud.SecurityCenter.V1 version 3.7.0

Changes in this release:
New features

Add user_name field to the finding access (commit 8f4176b)
Add kernel_rootkit field to finding's list of attributes (commit d98be1c)

Documentation improvements

Miscellaneous style improvements (commit d98be1c)

Created release for Google.Cloud.SecurityCenter.V1-3.7.0


[Android] Screenshare is crashing a test app of other android users

Environment:
SDK Version: 3.6.1 - https://github.com/DolbyIO/comms-sdk-flutter/releases/tag/3.6.1:
Client Platform: Flutter test app.
Description/Steps-to-reproduce:
-Install the QA test app
-Connect to the conference 1st Android device
-Connect to the conference 2nd Android device (you can connect as many devices as you want)
-Start screenshare as 1st Android device
Expected result:
Screenshare is working properly.
Actual result:
Screenshare IS NOT working (bug: https://jira.dolby.net/jira/browse/CCS-4029) but that is not the case here. After clicking "Start screenshare" every participant which is on android system is kicked out from the conference and test app crashes from it's side. This is reproducing when user is trying to reconnect to the conference. iOS users test app is working fine and they are not kicked from the conference.
Devices:
DV-WRO-SAM96 Samsung Galaxy TAB S8 + Android 12
DV-WRO-SAM98 Samsung Galaxy Tab S8 Ultra + Android 12
DV-WRO-SAM90 Samsung S22 Plus + Android 12
Private Phone Poco X3 NFC + Android 12

Fixed by @azduda

Reproduced on 3.7.0 beta.1 - https://github.com/DolbyIO/comms-sdk-flutter/releases/tag/3.7.0-beta.1
This time iOS device also can screenshare and then random Android device is disconnected. Reconnecting to the conference by crashed user is not possible until the screenshare is stopped.

Reproduced on develop branch https://github.com/DolbyIO/comms-sdk-flutter/pull/253 using(these devices crashed):
DV-WRO-SAM90 - samsung s22 Plus + Android 12
DV-WRO-SAM96 - Samsung Tab S8 + Android 12

This will be fixed by a new version of the Android native SDK.


[HUDI-1728] Fix MethodNotFound for HiveMetastore Locks

Brief change log
Fixed Method Not Found for HiveMetastoreBasedLockProvider
Renamed configs to match with other configs
Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

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

Codecov Report

Merging #2731 (a3fc957) into master (0e6909d) will decrease coverage by 37.03%.
The diff coverage is n/a.

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

@@             Coverage Diff              @@
##             master   #2731       +/-   ##
============================================
- Coverage     46.43%   9.40%   -37.04%
+ Complexity     3278      48     -3230
============================================
  Files           476      54      -422
  Lines         22583    1989    -20594
  Branches       2408     236     -2172
============================================
- Hits          10487     187    -10300
+ Misses        11196    1789     -9407
+ Partials        900      13      -887

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
9.40% <ø> (ø)
0.00 <ø> (ø)

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

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/hudi/common/metrics/LocalRegistry.java

...nal/HoodieBulkInsertDataInternalWriterFactory.java

...common/model/HoodieFailedWritesCleaningPolicy.java

...a/org/apache/hudi/common/model/HoodieBaseFile.java

...pache/hudi/common/config/HoodieMetadataConfig.java

...org/apache/hudi/hadoop/realtime/RealtimeSplit.java

.../org/apache/hudi/cli/commands/TempViewCommand.java

...rg/apache/hudi/common/fs/inline/InLineFSUtils.java

...pache/hudi/hadoop/HoodieColumnProjectionUtils.java

...sioning/clean/CleanMetadataV1MigrationHandler.java

... and 411 more


Initial version of PyCNN network visualizer

Import pycnn_viz instead of pycnn and call PrintGraphviz().
Key features:

See parameter names
Optionally show the dimensions of each node
Distinguish explicitly-provided dimensions ({{}}) from inferred
dimension labels ({})
Dimensions are computed and checked, and any errors are highlighted in
the graph
Verbose and compact modes
Optionally see names of lookups instead of indices (like seeing the
actual word of a word embedding)
Collapse RNN states into single boxes
Optionally detect and collapse BiRNNs
Optionally assign labels to individual expressions

Some example network graphs displaying a variety of features can be seen here:

@yoavg

Looking at the second graph, shouldn't the dim from pickneglogsoftmax be 1?
On Fri, Jun 24, 2016, 00:38 Dan Garrette<EMAIL_ADDRESS>wrote:

Import pycnn_viz instead of pycnn and call PrintGraphviz().
Key features:

See parameter names
Optionally show the dimensions of each node
Distinguish explicitly-provided dimensions ({{}}) from inferred
dimension labels ({})
Dimensions are computed and checked, and any errors are highlighted
in the graph
Verbose and compact modes
Optionally see names of lookups instead of indices (like seeing the
actual word of a word embedding)
Collapse RNN states into single boxes
Optionally detect and collapse BiRNNs
Optionally assign labels to individual expressions

Some example network graphs displaying a variety of features can be seen
here:
[image: pycnn_viz_example1]
https://cloud.githubusercontent.com/assets/484190/16321063/d6133924-3992-11e6-919c-5d80c42bd08e.png
[image: pycnn_viz_example2]
https://cloud.githubusercontent.com/assets/484190/16321064/d618572e-3992-11e6-91ea-1f8959f22972.png
@yoavg https://github.com/yoavg
You can view, comment on, or merge this pull request online at:
https://github.com/clab/cnn/pull/62
Commit Summary

Initial version of PyCNN network visualizer

File Changes

A pycnn/pycnn_viz.py
https://github.com/clab/cnn/pull/62/files#diff-0 (1104)

Patch Links:

https://github.com/clab/cnn/pull/62.patch
https://github.com/clab/cnn/pull/62.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clab/cnn/pull/62, or mute the thread
https://github.com/notifications/unsubscribe/AAcVtHGPKkd6k3FgERzKq66azFCQ2ugnks5qOvzmgaJpZM4I9RBT
.

@yoavg Yep, good catch. Fixed.

before I do the merge, can can you try adding the following as the last lines of pycnn.pyx, and see if it works:
if '--pycnn-viz-mode' in sys.argv:     print >> sys.stderr, "PyCNN loaded in visualization mode."     from pycnn_viz import *
?
the effect should be that adding the commandline switch --pycnn-viz-mode will import your expressions instead of the pycnn ones -- it works for non-cython modules, but I don't know if it holds also for cython ones. If it works, I think it is worth including.

If you add those lines, it compiles fine, but when you import pycnn it gives the following, presumably since the classes with those names are already loaded:
PyCNN loaded in visualization mode.
Traceback (most recent call last):
  File "bilstmtagger3.py", line 1, in <module>
    from pycnn import *
  File "build/bdist.macosx-10.11-intel/egg/pycnn.py", line 7, in <module>
  File "build/bdist.macosx-10.11-intel/egg/pycnn.py", line 6, in __bootstrap__
  File "pycnn.pyx", line 1168, in init pycnn (pycnn.cpp:38137)
TypeError: Cannot overwrite C type AdamTrainer

I also tried putting the full contents of pycnn.pyx under the else side of the statement, but apparently that's not valid:
Error compiling Cython file:
------------------------------------------------------------
...

  from pycnn cimport *
  cimport pycnn

  cdef init(random_seed=None):
      ^

Thoughts?
Also, maybe it should just be called --vis-mode since the "pycnn" part is implied?

(Actually, --pycnn-viz-mode makes more sense and is far less likely to conflict with any option defined by the user.)

hmm. annoying. oh well.
I will have some time to play with this tomorrow or on Monday if you rather wait and not do it yourself.
But I believe the following should work:
Rename the pycnn.pxd, pycnn.pyx files to cpycnn.{pxd,pyx} (and update the makefile and setup.py accordingly).
Add a pycnn.py file with:
import sys
if '--pycnn-viz-mode' in sys.argv:
   from pycnn_viz import *
elif '--pycnn-gpu' in sys.argv:
   from gpycnn import *
else:
   from cpycnn import *


chore(deps): unpin sharp plugin version

Reverts #528
The issue from https://github.com/gatsbyjs/gatsby/issues/33557 was solved in<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Name in Ember.Resource clashes with properties: ['name']

When i have a name property like this :
App.Contact  = Ember.Resource.extend({
  url:        '/contacts',
  name:       'contact',
  properties: ['name'],
 ....

The name property is being used in the json as :
{"value of property name"=>{"name"=>"value of property name"}

when i post a form using the edit template
{{#with contact}}
{{view Ember.TextField valueBinding="name" placeholder="name"}}
{{#if id}}
{{submitButton "Update"}}
{{else}} {{submitButton "Create"}}
{{/if}}
{{/with}}

thanks a lot for all the help

Thanks for reporting this. I believe I've fixed these naming collisions, but please reopen this if the problem continues.
FYI: I'll add some proper tests to this lib now that it's seeing a fair bit of use.

@dgeb thanks for fixing this so quickly.
There is still one problem with this bug though.
When i click to show the new text field like this

{{#with company}}
  {{view Ember.TextField valueBinding="name" placeholder="Name"}}
  {{view Ember.TextField valueBinding="description" placeholder="Description"}}
  {{#if id}}
    {{submitButton "Update"}}
  {{else}}
    {{submitButton "Create"}}
  {{/if}}
  Cancel
{{/with}}

It display the company in the value attribute like this :
<input id="ember793" class="ember-view ember-text-field" type="text" value="company" placeholder="Name">

Makes sense. I'll have to handle initialization of these props too. For now, when you create a new object, initialize its properties like this:
App.Contact.create({name: ''})

I'll follow up when I get a better fix in place.

thanks again that fixed it!

I've made some changes to prevent these and other namespace collisions. Please see the CHANGELOG for details:
https://github.com/cerebris/ember-rest/blob/master/CHANGELOG
I've also updated the example app and blog posts to reflect these changes.

thanks again!


Minor improvements

These changes allow faster camera switch and improve robustness when enabling disabling rotation multiple times

Awesome contribution !!! thank you !


cp: i18n support one config file per scenario

What type of this PR
/kind feature
What this PR does / why we need it:
cp: i18n support one config file per scenario
Which issue(s) this PR fixes:
Related Erda Issue
Specified Reivewers:
/assign @Effet

tested dop in local:


Dota 2 map is not correctly rendered

SW: VRF
Build version: <IP_ADDRESS>4
Action:

Load any dota 2 map package (e.g. : dota_722.vpk)
Open file world.vwrld_c

Expected: Map should be rendered successfully (with texture)
Observed:  Map is not rendered as expected.

Add the dota pak01 in settings.


BUG REPORT: docker logs follow gets stuck when the json-file log driver rotates files

Output of docker version:
Docker version 1.11.2, build b9f10c9

Output of docker info:
Containers: 5
 Running: 4
 Paused: 0
 Stopped: 1
Images: 8
Server Version: 1.11.2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 63
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.4.0-24-generic
Operating System: Ubuntu 16.04 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 7.795 GiB
Name: ip-10-47-10-249
ID: VVQ3:QEHD:WM5H:SY6Q:Y43P:MFQ6:BZ5Z:GO2M:YZ5Y:VQ5P:FSXW:JFDE
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support

Additional environment details (AWS, VirtualBox, physical, etc.):
Reproduced on Ubuntu 15.10, 16.04 EC2 instances. Reproduced on Ubuntu 14.04 docker-machine VM on my laptop. All tests were performed at a terminal in an SSH session connecting to a local docker daemon.
Steps to reproduce the issue:

To demonstrate the issue, Configure the Docker daemon to use the json-file log driver with a max-size of 10mb.
Mine looks like:

$ cat /etc/systemd/system/docker.service.d/docker-opts.conf
[Service]
ExecStart=
ExecStart=/usr/bin/docker daemon -H fd:// -s aufs --bip=<IP_ADDRESS>/24 --log-driver="json-file" --log-opt max-size=10m --log-opt max-file=5

Attempt to print the numbers 1 to 200000 to the stdout of a container and use docker logs to follow the output by running the following command:

$ docker run -d --name loggen ubuntu:16.04 /bin/bash -c 'for ((i=1; i<=200000; ++i)); do echo $i; done' && docker logs --timestamps --follow loggen > boom.txt
71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a

Describe the results you received:
I would expect to see the numbers 1 to 200000 along with the time each number was output by the loggen container in boom.txt
What I actually see:
$ tail boom.txt
2016-06-17T06:39:55.123832147Z 131491
2016-06-17T06:39:55.123837123Z 131492
2016-06-17T06:39:55.123842067Z 131493
2016-06-17T06:39:55.123846929Z 131494
2016-06-17T06:39:55.123851785Z 131495
2016-06-17T06:39:55.123856759Z 131496
2016-06-17T06:39:55.123861681Z 131497
2016-06-17T06:39:55.123866606Z 131498
2016-06-17T06:39:55.123871552Z 131499
2016-06-17T06:39:55.123876490Z 131500

Looking at the files generated by the json-file log driver for this container shows that things have gone wrong when the docker log driver rotates the json files.
$ sudo -s
$ ls /var/lib/docker/containers/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a/*.log*
/var/lib/docker/containers/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a-json.log
/var/lib/docker/containers/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a-json.log.1
$ tail /var/lib/docker/containers/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a-json.log.1
{"log":"131491\n","stream":"stdout","time":"2016-06-17T06:39:55.123832147Z"}
{"log":"131492\n","stream":"stdout","time":"2016-06-17T06:39:55.123837123Z"}
{"log":"131493\n","stream":"stdout","time":"2016-06-17T06:39:55.123842067Z"}
{"log":"131494\n","stream":"stdout","time":"2016-06-17T06:39:55.123846929Z"}
{"log":"131495\n","stream":"stdout","time":"2016-06-17T06:39:55.123851785Z"}
{"log":"131496\n","stream":"stdout","time":"2016-06-17T06:39:55.123856759Z"}
{"log":"131497\n","stream":"stdout","time":"2016-06-17T06:39:55.123861681Z"}
{"log":"131498\n","stream":"stdout","time":"2016-06-17T06:39:55.123866606Z"}
{"log":"131499\n","stream":"stdout","time":"2016-06-17T06:39:55.123871552Z"}
{"log":"131500\n","stream":"stdout","time":"2016-06-17T06:39:55.12387649Z"}
$ head -n 10 /var/lib/docker/containers/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a/71943ba916fb365fb3178ab9dabd37158d690b49aeaa595420fde4f92b62256a-json.log
{"log":"131501\n","stream":"stdout","time":"2016-06-17T06:39:55.123881404Z"}
{"log":"131502\n","stream":"stdout","time":"2016-06-17T06:39:55.123994259Z"}
{"log":"131503\n","stream":"stdout","time":"2016-06-17T06:39:55.124001593Z"}
{"log":"131504\n","stream":"stdout","time":"2016-06-17T06:39:55.124006555Z"}
{"log":"131505\n","stream":"stdout","time":"2016-06-17T06:39:55.124011382Z"}
{"log":"131506\n","stream":"stdout","time":"2016-06-17T06:39:55.124016163Z"}
{"log":"131507\n","stream":"stdout","time":"2016-06-17T06:39:55.124020854Z"}
{"log":"131508\n","stream":"stdout","time":"2016-06-17T06:39:55.12402562Z"}
{"log":"131509\n","stream":"stdout","time":"2016-06-17T06:39:55.124030396Z"}
{"log":"131510\n","stream":"stdout","time":"2016-06-17T06:39:55.12403511Z"}

Describe the results you expected:
I would expect to see the numbers 1 to 200000 along with the time each number was output by the loggen container in boom.txt
Additional information you deem important (e.g. issue happens only occasionally):
I did a little more digging and found an upper bound on the --log-opt max-size where the docker logs --follow was not broken by rotating the json log files.
When max-size is <= 215k, docker logs --follow behaves as expected.
When max-size is >= 216k, docker logs --follow gets stuck.

Are you saying that the docker log follower didn't jump to the next log file?

Yes. It stops following when logs start being sent to a second file.

I'm seeing the same behavior with:
docker version:
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 12:59:02 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 12:59:02 UTC 2015
 OS/Arch:      linux/amd64

docker info:
Containers: 17
Images: 249
Server Version: 1.9.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 283
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.16.0-4-amd64
Operating System: Debian GNU/Linux 8 (jessie)
CPUs: 2
Total Memory: 7.815 GiB
Name: XXXXXXXXXXXXXXXXX
ID: 5MWW:M4LM:3UIM:5UOC:7AQD:DSWA:MIAA:WBFF:EMHR:2DH6:ITQC:QBVW
WARNING: No swap limit support

@cpuguy83 could this be a duplicate of https://github.com/docker/docker/issues/23913? It's also using rotation

@markine There's been a number of fixes since 1.9.1, please test on 1.11.2 and let us know if you still have a problem.

@thaJeztah Not sure, that's this issue number :)

@lukeck Can you provide daemon logs when this occurs?

@cpuguy83 hmf, wrong copy/paste again; meant this one: https://github.com/docker/docker/issues/21220

Funny you guys would be looking at this exactly when I run into the problem :)

Could you please test with #24514? Thanks!

@cpuguy83 Thank you, unfortunately I don't think I have the luxury of bumping the docker version arbitrarily in my Kubernetes environment. @lukeck would love to hear whether #24514 resolves the issue for you.

Nice one @cpuguy83! I should be able to take #24514 for a spin in the next few hours.

@cpuguy83 Confirming, fixes issues on ARM64.

@cpuguy83 Not sure if it is related to upgrading to rc4 or otherwise, but after applying your patch I am getting a bug.
docker exec -it mycontainer sh
$ exit
.... hang

Any ideas?

@cpuguy83 Never mind, the error I reported here before is unrelated, I think +1 for merging this

We hit this a well. It looks like this is available on v1.13.x tags.


taskdef class com.intellij.ant.InstrumentIdeaExtensions cannot be found

This and #1246 are blocking the Flutter plugin for IntelliJ from being released for 2023.1 EAP. It is possible #1246 is fixed although still open. I was hoping to test that.
I'm using Gradle 7.5.
I updated the IntelliJ plugin for Gradle from 1.9.0 to 1.12.0 and got this error.

What went wrong:
Execution failed for task ':flutter-idea:instrumentCode'.
taskdef class com.intellij.ant.InstrumentIdeaExtensions cannot be found
using the classloader AntClassLoader[]

Fixed in 1.13.1-SNAPSHOT

@caiqichang Please migrate to IntelliJ Platform Gradle Plugin 2.x in the first place.


Issue with ION ordering in code generation

Consider below example from reduced_dentate model (#107),
NEURON {
    SUFFIX ccanl
        USEION nca READ ncai, inca, enca WRITE enca, ncai VALENCE 2
        USEION lca READ lcai, ilca, elca WRITE elca, lcai VALENCE 2
        USEION tca READ tcai, itca, etca WRITE etca, tcai VALENCE 2
        RANGE caiinf, catau, cai, ncai, lcai,tcai, eca, elca, enca, etca
}

UNITS {
        (mV) = (millivolt)
    (molar) = (1/liter)
    (mM) = (milli/liter)
    (mA) = (milliamp)
    FARADAY = 96520 (coul)
    R = 8.3134  (joule/degC)
}

:INDEPENDENT {t FROM 0 TO 100 WITH 100 (ms)}

PARAMETER {
    depth = 200 (nm)    : assume volume = area*depth
    catau = 9 (ms)
    caiinf = 50.e-6 (mM)
    cao = 2 (mM)
}

ASSIGNED {
        celsius (degC)
    ica (mA/cm2)
    inca (mA/cm2)
    ilca (mA/cm2)
    itca (mA/cm2)
    cai (mM)

    enca (mV)
    elca (mV)
    etca (mV)
    eca (mV)
}

STATE {
    ncai (mM)
    lcai (mM)
    tcai (mM)
}

Here is comparison of data structures order from mod2c and nmodl :

Issues :

order of ion variables is wrong
same ion variable can be used as read as well as write : in this case the variable shouldn't be declared as const

This issue was also seen while testing #148.

@iomaganaris : see for long I have been postponing this! :disappointed:

We figured out that there is an issue with the ordering of the ion variable indexes on https://github.com/pramodk/reduced_dentate/blob/master/mechanisms/LcaMig.mod. Especially the issue is with cao and cai variables indexes. To fix this issue we changed a bit the mod file to: https://github.com/pramodk/reduced_dentate/blob/master/mechanisms/LcaMig.mod

Let's look at the examples of TLcaMig.mod in the reduced_dentate model:
$ cat mechanisms/LcaMig.mod
TITLE l-calcium channel
: l-type calcium channel

UNITS {
	(mA) = (milliamp)
	(mV) = (millivolt)
	(molar) = (1/liter)
	(mM) = (millimolar)
	FARADAY = 96520 (coul)
	R = 8.3134 (joule/degC)
	KTOMV = .0853 (mV/degC)
}

PARAMETER {
	glcabar		 (mho/cm2)
	ki=.001 (mM)
	cao (mM)
        tfa=1
}

NEURON {
	SUFFIX lca
	USEION lca READ elca WRITE ilca VALENCE 2
	USEION ca READ cai, cao VALENCE 2
        RANGE glcabar, cai, ilca, elca
        :GLOBAL minf,matu
	RANGE minf,matu
}

In the above example, cao is defined first in the PARAMETER block and hence appear ahead of cai in the NMODL symbol table.
In case of NMODL, when C++ code is printed, its getting read ion variable based on symbol table: https://github.com/BlueBrain/nmodl/blob/189e35c81201b49d5cfc99256a19b7e78364a3e8/src/codegen/codegen_helper_visitor.cpp#L69
    auto ion_vars = psymtab->get_variables_with_properties(NmodlType::useion);
    /// read variables from all ions
    auto read_ion_vars = psymtab->get_variables_with_properties(NmodlType::read_ion_var);
    /// write variables from all ions
    auto write_ion_vars = psymtab->get_variables_with_properties(NmodlType::write_ion_var);

    /**
     * Check if given variable belongs to given ion.
     * For example, eca belongs to ca ion, nai belongs to na ion.
     * We just check if we exclude first/last char, if that is ion name.
     */
    auto ion_variable = [](const std::string& var, const std::string& ion) -> bool {
        auto len = var.size() - 1;
        return (var.substr(1, len) == ion || var.substr(0, len) == ion);
    };

    /// iterate over all ion types and construct the Ion objects
    for (auto& ion_var: ion_vars) {
        auto ion_name = ion_var->get_name();
        Ion ion(ion_name);
        for (auto& read_var: read_ion_vars) {
            auto var = read_var->get_name();
            if (ion_variable(var, ion_name)) {
                ion.reads.push_back(var);
            }

In case of MOD2C, the order of read ion variables is based on the order of USEION statements in the MOD file (i.e. PARAMETER block doesn't influence anything!):
https://github.com/BlueBrain/mod2c/blob/42bb3c3d6fa70e8f4ad89ab448910c006ad30359/src/mod2c_core/nocpout.c#L1998
https://github.com/BlueBrain/mod2c/blob/42bb3c3d6fa70e8f4ad89ab448910c006ad30359/src/mod2c_core/nocpout.c#L2363
And hence the issue!
So, I think the solution is, NMODL should give up the ordering based on symbol table and use USEION statements from the AST to get variable order. This will resolve the issue for LcaMig.mod. The issue mentioned for ccanl.mod is already resolved.
I will work on this tomorrow.

NOTE: simple solution of moving PARAMETER block after NEURON block will solve the issue. Or, doing other way around will introduce bug in any model!


HTTP 404 when running app/target/app.jar as in the tutorial

At this page in the tutorial the problem below occurs:
https://enroute.osgi.org/tutorial/020-tutorial_qs.html
I'm using ubuntu 18.10, oracle java 10.0.1 and mvn 3.5.3.
I cloned the osgi.enroute repo today, it is at master 2497da546.
The mvn verify step went well, it took just over 2 minutes here.
Then  java -jar app/target/app.jar also appeared to go well,
but the http://localhost:8080/quickstart/index.html
page showed an HTTP 404 when running the target app.
So I restarted to get a log:
java -jar app/target/app.jar >& log.txt
which gave this log.txt when the 404 happens:
log.txt
Early in the log there is this, it could be the root cause, but I'm only starting with OSGi:
! Failed to start bundle org.apache.aries.javax.jax.rs-api-<IP_ADDRESS>806180958, exception Unable to resolve org.apache.aries.javax.jax.rs-
api [4](R 4.0): missing requirement [org.apache.aries.javax.jax.rs-api [4](R 4.0)] osgi.wiring.package; (osgi.wiring.package=javax.xml.bind.annotation) Unresolved requirements: [[org.apache.aries.javax.jax.rs-api [4](R 4.0)] osgi.wiring.package; (osgi.wiring.package=javax.xml.bind.annotation)]
Minor point:
I installed bndtools 4.0.0.REL in eclipse 4.7.3a from the eclipse marketplace,
but I don't think I have used them yet for the tutorial.
The tutorial here https://enroute.osgi.org/tutorial/015-Prerequisite.html
says that bndtools is not yet available in the eclipse marketplace, this has probably changed recently.

The enRoute examples target Java 8 (the current long term support release of Java). You can see a discussion of why you get this failure in this issue
If you use Java 8 it will be fine, otherwise you need to start Java 10 including the JAXB annotations on the root classpath, or you need to re-resolve with Java 10 as the target platform with a JAXB API jar in your repository.
They really did a number on API backward compatibility in Java 9...

Closed as a duplicate of #47


run sync tests on all devops snapshots

run db-sync sync tests on the snapshots provided by devops for Mainnet & Testnet

IOHK Snapshot restoration  GitHub Actions flow supports any url user will provide for snapshot field.
Additionally there is a latest keyword that can be used in snapshot url field - it will pick the latest available IOHK snapshot for particular environment that supports snapshots.


The memory issue persists even after running twice the command. (minor issue: no logs)

What are you experiencing an issue with?

[X] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[X] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
it shouldn't eat up my RAM, slowing my computer down a lot. It should also create logs in the designated folders.
Actual behavior:
It eats up my RAM, slowing my computer to the point that it takes a whole minute to load up the task manager, and after running twice the command lodctr /R, in a command prompt run as administrator, it works fine until i restart my pc.
It doesn't create logs in the designated folder, as it wasn't present. All I did was manually create it, hoping that in someway it would. It does not.
How often does this problem occur?
This problem occurs everytime I restart my computer.
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:
Boot the computer
What is the version of Aurora you are experiencing this issue with?
0.6.1. -dev3
Does the previous version of Aurora have this issue?
Never had this problem before, but I jumped straight forward from v 0.6.0. to -dev3.
Please include any logs, screenshots, or files that are related to this issue:
I have no logs, and I am currently trying to fix that by reinstalling the program from scratch.
https://imgur.com/bB7Yn5m

Duplicate of #758


handle client being killed

Change-type: patch
Signed-off-by: Ryan Cooke<EMAIL_ADDRESS>

Will also add handling on the client side to handle testbot disconnects

@balena-ci rebase

@balena-ci rebase

Can one of the admins verify this patch?


Add Travis build status to front page.

This makes it easier to view the current build status and to find the Travis CI
page for the project.

Awesome!


Tests fail to open the file examples/binary_classification/binary.test

Description
Many tests succeed, but then they fail to find the file examples/binary_classification/binary.test:
[==========] Running 30 tests from 8 test suites.
[----------] Global test environment set-up.
[----------] 4 tests from Partition
[ RUN      ] Partition.JustWorks
[       OK ] Partition.JustWorks (0 ms)
[ RUN      ] Partition.PartitionOneElement
[       OK ] Partition.PartitionOneElement (0 ms)
[ RUN      ] Partition.Empty
[       OK ] Partition.Empty (0 ms)
[ RUN      ] Partition.AllEqual
[       OK ] Partition.AllEqual (0 ms)
[----------] 4 tests from Partition (0 ms total)

[----------] 5 tests from ArrowChunkedArrayTest
[ RUN      ] ArrowChunkedArrayTest.GetLength
[       OK ] ArrowChunkedArrayTest.GetLength (0 ms)
[ RUN      ] ArrowChunkedArrayTest.GetColumns
[       OK ] ArrowChunkedArrayTest.GetColumns (0 ms)
[ RUN      ] ArrowChunkedArrayTest.IteratorArithmetic
[       OK ] ArrowChunkedArrayTest.IteratorArithmetic (0 ms)
[ RUN      ] ArrowChunkedArrayTest.BooleanIterator
[       OK ] ArrowChunkedArrayTest.BooleanIterator (0 ms)
[ RUN      ] ArrowChunkedArrayTest.OffsetAndValidity
[       OK ] ArrowChunkedArrayTest.OffsetAndValidity (0 ms)
[----------] 5 tests from ArrowChunkedArrayTest (0 ms total)

[----------] 1 test from ByteBuffer
[ RUN      ] ByteBuffer.JustWorks
[       OK ] ByteBuffer.JustWorks (0 ms)
[----------] 1 test from ByteBuffer (0 ms total)

[----------] 11 tests from ChunkedArrayTest
[ RUN      ] ChunkedArrayTest.constructorWithChunkSize0Throws
[LightGBM] [Fatal] ChunkedArray chunk size must be larger than 0!
[       OK ] ChunkedArrayTest.constructorWithChunkSize0Throws (0 ms)
[ RUN      ] ChunkedArrayTest.constructorWithChunkSize
[       OK ] ChunkedArrayTest.constructorWithChunkSize (0 ms)
[ RUN      ] ChunkedArrayTest.getChunkSizeIsConstant
[       OK ] ChunkedArrayTest.getChunkSizeIsConstant (0 ms)
[ RUN      ] ChunkedArrayTest.getChunksCount
[       OK ] ChunkedArrayTest.getChunksCount (0 ms)
[ RUN      ] ChunkedArrayTest.getAddCount
[       OK ] ChunkedArrayTest.getAddCount (0 ms)
[ RUN      ] ChunkedArrayTest.coalesceTo
[       OK ] ChunkedArrayTest.coalesceTo (0 ms)
[ RUN      ] ChunkedArrayTest.clear
[       OK ] ChunkedArrayTest.clear (0 ms)
[ RUN      ] ChunkedArrayTest.doubleFreeSafe
[       OK ] ChunkedArrayTest.doubleFreeSafe (0 ms)
[ RUN      ] ChunkedArrayTest.totalArraySizeMatchesLastChunkAddCount
[       OK ] ChunkedArrayTest.totalArraySizeMatchesLastChunkAddCount (0 ms)
[ RUN      ] ChunkedArrayTest.dataLayoutTestThroughGetitem
[       OK ] ChunkedArrayTest.dataLayoutTestThroughGetitem (0 ms)
[ RUN      ] ChunkedArrayTest.testDataLayoutWithAdvancedInsertionAPI
[       OK ] ChunkedArrayTest.testDataLayoutWithAdvancedInsertionAPI (0 ms)
[----------] 11 tests from ChunkedArrayTest (0 ms total)

[----------] 5 tests from AtofPreciseTest
[ RUN      ] AtofPreciseTest.Basic
[       OK ] AtofPreciseTest.Basic (0 ms)
[ RUN      ] AtofPreciseTest.CornerCases
[LightGBM] [Warning] convert to double got underflow or overflow:<PHONE_NUMBER>86231580793728971405303415079934132710037826936173778980444968292764750946649017977587207096330286416692887910946555547851940402630657488671505820681908902000708383676273854845817711531764475730270069855571366959622842914819860834936475292719074168444365510704342711559699508093042880177904174497792
[       OK ] AtofPreciseTest.CornerCases (0 ms)
[ RUN      ] AtofPreciseTest.ErrorInput
[LightGBM] [Fatal] no conversion to double for: x1
[       OK ] AtofPreciseTest.ErrorInput (0 ms)
[ RUN      ] AtofPreciseTest.NaN
[       OK ] AtofPreciseTest.NaN (0 ms)
[ RUN      ] AtofPreciseTest.Inf
[       OK ] AtofPreciseTest.Inf (0 ms)
[----------] 5 tests from AtofPreciseTest (0 ms total)

[----------] 1 test from Serialization
[ RUN      ] Serialization.JustWorks
[LightGBM] [Info] Debug sample data path: examples/binary_classification/binary.test
[LightGBM] [Fatal] Cannot open data file examples/binary_classification/binary.test
/usr/ports/misc/lightgbm/work/LightGBM-4.4.0/tests/cpp_tests/test_serialize.cpp:25: Failure
Expected equality of these values:
  0
  result
    Which is: -1
LoadDatasetFromExamples result code: -1

[LightGBM] [Info] Saving data reference to binary buffer
/usr/ports/misc/lightgbm/work/LightGBM-4.4.0/tests/cpp_tests/test_serialize.cpp:37: Failure
Expected equality of these values:
  0
  result
    Which is: -1
LGBM_DatasetSerializeReferenceToBinary result code: -1

[LightGBM] [Fatal] Check failed: (num_data) > (0) at /usr/ports/misc/lightgbm/work/LightGBM-4.4.0/src/io/dataset.cpp, line 39 .

/usr/ports/misc/lightgbm/work/LightGBM-4.4.0/tests/cpp_tests/test_serialize.cpp:52: Failure
Expected equality of these values:
  0
  result
    Which is: -1
LGBM_DatasetCreateFromSerializedReference result code: -1

*** Signal 11

Stop.
make: stopped in /usr/ports/misc/lightgbm

Reproducible example

Environment info
LightGBM version or commit hash:
Command(s) you used to install LightGBM

Version: 4.4.0
FreeBSD 14.1

Thanks for using LightGBM and for your work repackaging it for FreeBSD.
To run the C++ tests for this project, run the following from the root of the repo:
cmake -B build -S . -DBUILD_CPP_TEST=ON
cmake --build build --target testlightgbm -j4
./testlightgbm

I suspect you're running them from a different working directory, as LightGBM's CI jobs did prior to #6368.
As I said the last time you opened a report like this, it's really difficult to help with only a bit of logs and a version number (https://github.com/microsoft/LightGBM/issues/5976#issuecomment-1636077782). The next time you open a report like this here, please provide enough information for us to reproduce the issue (like the exact commands you ran).


Investigate x64 RHEL build queues

[Triage] RHEL build queues are no longer used.  Closing as obsolete.


Allow special-form "export" with empty operand

A plan to have an effect of prohibiting new define for the environment by exercising an empty export procedure.

Discard this issue as it no longer seems useful.


three node cluster , restart crash and configure lost

three node clusterd :
-name<EMAIL_ADDRESS>-name<EMAIL_ADDRESS>-name<EMAIL_ADDRESS>on 18.10 run  ./bin/emqttd_ctl cluster<EMAIL_ADDRESS>on 18.11 run ./bin/emqttd_ctl cluster<EMAIL_ADDRESS>then, restart 18.9, 18.10 , 18.11 , run './bin/emqttd_ctl cluster' found cluster configure is lost  。
try to run ./bin/emqttd_ctl cluster<EMAIL_ADDRESS>on 18.11  will carsh:
RPC to<EMAIL_ADDRESS>failed: {'EXIT',
{{case_clause,
{error,
{merge_schema_failed,
"Incompatible schema cookies. Please, restart from old<EMAIL_ADDRESS>= [{name,schema},{type,set},{ram_copies,['emqttd@<IP_ADDRESS>']},{disc_copies,['emqttd@<IP_ADDRESS>']},{disc_only_copies,[]},{load_order,0},{access_mode,read_write},{majority,false},{index,[]},{snmp,[]},{local_content,false},{record_name,schema},{attributes,[table,cstruct]},{user_properties,[]},{frag_properties,[]},{storage_properties,[]},{cookie,{{1436,444971,772460},'emqttd@<IP_ADDRESS>'}},{version,{{4,0},{'emqttd@<IP_ADDRESS>',{1436,446112,654991}}}}]<EMAIL_ADDRESS>= [{name,schema},{type,set},{ram_copies,['emqttd@<IP_ADDRESS>','emqttd@<IP_ADDRESS>']},{disc_copies,['emqttd@<IP_ADDRESS>']},{disc_only_copies,[]},{load_order,0},{access_mode,read_write},{majority,false},{index,[]},{snmp,[]},{local_content,false},{record_name,schema},{attributes,[table,cstruct]},{user_properties,[]},{frag_properties,[]},{storage_properties,[]},{cookie,{{1436,430392,499098},'emqttd@<IP_ADDRESS>'}},{version,{{5,0},{'emqttd@<IP_ADDRESS>',{1436,446165,408554}}}}]\n"}}},
[{emqttd_mnesia,cluster,1,
[{file,"src/emqttd_mnesia.erl"},
{line,144}]},
{emqttd_ctl,cluster,1,
[{file,"src/emqttd_ctl.erl"},
{line,85}]},
{rpc,'-handle_call_call/6-fun-0-',5,
[{file,"rpc.erl"},{line,205}]}]}}

anthother question restart node , the auth user and password is lost

The same problem. After any server restart it looses the user.


Remove trailing slash requirement for build cache URL

Context
There does not seem to be a reason to require a trailing / when creating an instance of HttpBuildCacheService. This sometimes causes us to create workarounds.
To me, removing the trailing / is the most logical permanent fix for this.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@octylFractal, do you think it is necessary to remove the trailing / from the snippets?
For example: https://github.com/gradle/gradle/blob/31bff5917cb0573b30258e963cdad58e1d8b128b/subprojects/docs/src/snippets/buildCache/configure-built-in-caches/groovy/settings.gradle#L15

I think it is better to leave it in place, since the docs may be referenced by people who use older versions of Gradle and it would be better if it worked for them too.

@octylFractal I'm not sure if you noticed, but I made it so that all tests run without a trailing slash. Therefore, I added test a test to make sure url can be specified with a trailing slash.
Do you think I should do the opposite and run all tests with a trailing slash (like before) and add a test testing it works with no slash?

No, I think it's good. Thank you :)

@bot-gradle test and merge please.

OK, I've already triggered a build for you.


Vorkath helper

This feature would:

Count Vorkath hits (predict occurrence of poison/scorpion phase)
Show the hit radius of a fireball

AoE overlays have already been removed by request of Jagex.

Duplicate of #2447


run a task at a specific time of day

-can be used in conjunction with the other frequency methods to easily set the time.

This needs the CI\CD updates too. But also please add tests for the new method, it should be a fairly easy copy-paste from a few existing tests.

@colethorsen Now that #43 is merged can you please rebase this?

@MGatner is this good now?

Same - probably some cleanup but let's get the content in place.


CAMEL-10296: waitForConfirmsOrDie does not return if no timeout is set

for publisher acks and guaranteed delivery

The boolean should likely be volatile, or an AtomicBoolean.
Also mind about checkstyle. You can run a build with
mvn compile -Psourcecheck

Yes, I was looking at the PR and I noticed a bit of sourcecheck errors

Thanks, I'm reviewing and I'll merge soon

Already saw the checkstyle violations this morning and fixed them right now.
As each message publish is handled by a new RabbitMQMessagePublisher there should be no concurrent access on the local variables. But to be on the save side I set the variable to volatile to make sure we do not run into any concurrency issues here.

I merged the PR, but I amended the last commit message to refer to CAMEL-10296 and not CAMEL-10239
Thanks a lot @gessnerfl
Can you close the PR?

Thanks for the PR it has been merged. Do you mind closing this?


Manual Refresh Uninstaller or automatic Refresh after Uninstall causes BCUninstaller to shut down

Carrying out an uninstall or dong a manual Refresh of uninstallers causes BCUninstaller to shut down on completion of refresh at least on my pc both with Version 3.7 and 3.71. I have not seen this before. By the way there is no error message coming up during this process just an application shutdown.
Thanks for a great app. It is a must have in my arsenal of PC Tools. Better than paid for uninstallers.

Can you check %LOCALAPPDATA%\Microsoft\Windows\WER\ReportArchive for any files starting with AppCrash_BCUninstaller*? If there are some, delete them all, crash BCU again, and upload here the new appcrash that appears.

The is no ReportArchive Folder in WER folder. There is only an ERC Folder which contains a file called viewedids.resp.

I found the right location at "C:\ProgramData\Microsoft\Windows\WER\ReportArchive" There are no files starting with AppCrash_BCUninstaller.

I can't reproduce the issue. Can you try downloading a fresh version and run it from a different folder? Maybe something got corrupted.

I found the right location at "C:\ProgramData\Microsoft\Windows\WER\ReportArchive" There are no files starting with AppCrash_BCUninstaller.

I tried as you suggested in your email. I re-downloaded the Version 3.6.1 Portable version and extracted it in another directory. I set it up with settings as I have before and after the initial loading of uninstallers, I then refreshed the uninstallers and again just as the progress reached the end the application disappeared with no message or dialogue. Maybe it is one of the settings I use that triggers this behaviour. I attach my BCUninstaller.settings file in the hope that you find something there or at least replicate this unusual behaviour.
Good Luck.
BCUninstaller.settings.zip

Can you try disabling user ratings and usage statistics under Settings>Misc?

Hi, I did as you suggested and disabled those settings. Closed BCU before starting it again and Refreshing and again it just disappeared as it got to the the end. But once before I got this Error which I pasted and copied. But did not get it on the last one with the settings above disabled.  Sorry for this trouble. I will try turning off some of the additional settings I set one by one to see if any of these trigger the BCU shutdown.
Error.txt

Actually, I created a new folder and extracted BCU into that and ran it. I accepted every default setting and then refreshed uninstallers and again when it reached the end it closed itself. So it must be something strange about my system and not my settings.

Can you try creating a new user account, and running BCU on it? Maybe some other application is interfering, or some .net settings are bad.

Actually, I created a new folder and extracted BCU into that and ran it. I accepted every default setting and then refreshed uninstallers and again when it reached the end it closed itself. So it must be something strange about my system and not my settings.

Sorry for the delay in replying. I used another user account on same pc with both Version 3.7.1 as well as the new Version 3.7.2 and both exhibited the same behaviour as before.

Can you replace the executable of 3.7.2 with this, and see if refreshing the list still closes BCU? (don't worry about the blank list)

Hi Marcin,
I replaced the executable with the one in the link. I got the attached errors on the first scan and the exact same ones on the renew. And again on the end of the renew the app closed itself.
Cheers. The Fly.
Thanks in advance for all your hard work.
On Thu, 16/2/17, Marcin Szeniak<EMAIL_ADDRESS>wrote:
Subject: Re: [Klocman/Bulk-Crap-Uninstaller] Manual Refresh Uninstaller or automatic Refresh after Uninstall causes BCUninstaller to shut down (#58)
To: "Klocman/Bulk-Crap-Uninstaller"<EMAIL_ADDRESS>Cc: "TheFly4242"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Received: Thursday, 16 February, 2017, 5:14 AM
Can you replace the executable of
3.7.2 with this,
and see if refreshing the list still closes BCU? (don't
worry about the blank list)
—
You are receiving this because you authored the thread.
Reply to this email directly, view
it on GitHub, or mute
the thread.
BCUninstaller, Version=<IP_ADDRESS>525, Culture=neutral, PublicKeyToken=null | MSIL | Microsoft Windows NT 6.2.9200.0 | 64bit
System.ComponentModel.Win32Exception
Access is denied
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
at UninstallTools.Factory.SteamFactory.StartProcessAndReadOutput(String filename, String args)
at UninstallTools.Factory.SteamFactory.GetSteamInfo()
at UninstallTools.Factory.SteamFactory.get_SteamLocation()
at UninstallTools.Factory.PredefinedFactory.GetSteamUninstallerEntry()
at UninstallTools.Factory.PredefinedFactory.GetUninstallerEntries()
at UninstallTools.Factory.ApplicationUninstallerFactory.GetMiscUninstallerEntries()
BCUninstaller, Version=<IP_ADDRESS>525, Culture=neutral, PublicKeyToken=null | MSIL | Microsoft Windows NT 6.2.9200.0 | 64bit
System.ComponentModel.Win32Exception
Access is denied
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
at UninstallTools.Factory.SteamFactory.StartProcessAndReadOutput(String filename, String args)
at UninstallTools.Factory.StoreAppFactory.d__2.MoveNext()
at System.Collections.Generic.List1.InsertRange(Int32 index, IEnumerable1 collection)
at System.Collections.Generic.List1.AddRange(IEnumerable1 collection)
at UninstallTools.Factory.ApplicationUninstallerFactory.GetMiscUninstallerEntries()

For your information the application disappears just after it displays the dialog "Populating installer list".

I'm still having trouble replicating the problem, or finding anything that could cause it.
Do you happen to run any antiviruses or automation programs (like AutoIt)? If yes, what happens when you disable them?

Hi,
I don't have any Automation Software installed. As for Anti-Virus, I have Comodo Firewall, Panda Free Antivirus, Malwarebytes Free, SUPERAntiSpyware Free Edition and SpyBot and Destroy. I disabled all of these and BC Uninstaller Ver 3.7.2 still does the same thing. When the dialog that says "Populating installer list" reaches 100% the application just disappears.
Cheers.
TheFly4242
On Tue, 7/3/17, Marcin Szeniak<EMAIL_ADDRESS>wrote:
Subject: Re: [Klocman/Bulk-Crap-Uninstaller] Manual Refresh Uninstaller or automatic Refresh after Uninstall causes BCUninstaller to shut down (#58)
To: "Klocman/Bulk-Crap-Uninstaller"<EMAIL_ADDRESS>Cc: "TheFly4242"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Received: Tuesday, 7 March, 2017, 3:50 AM
I'm still having trouble
replicating the problem, or finding anything that could
cause it.
Do you happen to run any antiviruses or automation
programs (like AutoIt)? If yes, what happens when you
disable them?
—
You are receiving this because you authored the thread.
Reply to this email directly, view
it on GitHub, or mute
the thread.

May I add that this issue has stopped occurring since Version 3.12. Also BC Uninstaller loads much quicker from starting application.

This is weird, nothing that I changed should have affected this. My best guess is that your compiled assembly store got corrupted or there was a bug in NET that got fixed in an update.


Scrolling menu

If there are too many entries to fit on screen the menu repositions itself so that the bottom options are not cut off from the view.
I'm currently using 4.1.0 which I know is very old but I'm curious if there is anything in that or a newer build to handle a large menu on limited space such as with a scroll bar?
If not can we get it added or is it an insane idea and I should look to make more use of sub menus?

Hi, I tested this with the latest demo by adding more menu items. The behaviour is the same with the latest version.
Since you described a small map area in your application my feeling is that you need to tidy up your entries. In my opinion adding scrollbars makes it more difficult to use. As you said maybe the use of submenus make sense.

Just an example I found on my NAS to show it's not always a terrible thing to add scrolling


Microsoft.Bot.Connector package dependency restrictions on Microsoft.Rest.ClientRuntime

Hi,
Microsoft.Bot.Connection has a NuGet dependency for Microsoft.Rest.ClientRuntime set to (>= 1.8.2 && < 2.0.0).
This clashes with Microsoft.Bot.Builder 3.3.3 which installs (>= 2.3.2)

Well, I just realized that We get both by just installing the Microsoft.Bot.Builder 3.3.3 package. Nevermind.


Deprecation Warning when accessing headers

bug: (node:52579) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated is shown when res.request parameters are accessed
test case:
$ node
Welcome to Node.js v12.6.0.
Type ".help" for more information.
> const request = require('superagent')
undefined
> (async () => { const res = await request.agent().get('https://google.com'); console.log(res.request.url) })()
Promise { <pending> }
> (node:52579) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated
https://www.google.com/

possible fix
https://github.com/BrowserSync/browser-sync/pull/1698/files
extra info
https://github.com/villadora/express-http-proxy/issues/413

In my own debugging the warning is caused by Request#_redirect which calls cleanHeader with a reference to req._headers.
It looks like this method and the helper function grab a reference to the internal collection of headers and delete some of them, then use this object to update the internal header values in Request. Node's own Request / OutgoingMessage doesn't have a bulk update method so normally this wouldn't work but I guess the Superagent mock version can still do it, so even though getHeaders gives you a shallow copy, it should be OK. Maybe?  You'd have to grab the copy once (as in the PR from @ad-m ) and pass it to cleanHeader instead of referring to _headers again, and I'm not familiar enough with the codebase to say whether there are side effects since the old code deletes the headers in the actual Request while the new method will only delete from the shallow copy.

Any update on this?

v5.1.2 is released to npm and everything should be fine wit latest versions of Node now 🎉
If you find any further deprecation warnings in the future please let me know!
npm install superagent@latest

or with yarn:
yarn add superagent@latest

https://github.com/visionmedia/superagent/releases/tag/v5.1.2

thanks @niftylettuce @ad-m and everyone who helped out along the way 👏🙏👍


"Hide Advanced Ports on Nodes" impacts node disambiguation

When connecting collapsed nodes, if the "Hide Advanced Ports on Nodes" is deselected, the disambiguation shows extra options:

It should probably work the same wether this is selected or deselected.

I think filtering based on type will help here, but if there's a valid port to connect from/to we should probably always show it? Wdyt, @dglazkov?

Filter by type will help a lot, but in this case where there are multiple types on each side, it wouldn't resolve the problem.  It seems like Nodes default ports should be honored wether advanced ports are hidden or not.


Use community templates

The README deployment/testing steps were updated to match the KinD tests in the Github Actions.

Good ideas. Updated.


Fix required Node.js version

The CommonJS built depends on node:process that was introduced only in v16 is missing in v15:

And it breaks on Node.js v15:

...and also with Jest (26.6.3) running on v16:

I see that 5.0.0 does not have this problem. It seems that it was introduced in this commit: https://github.com/mikaelbr/marked-terminal/commit/06a320c25bc305b463a7c2437ee10b9f98937e4c

After further investigation, I found that v14.19.3 indeed supports node:process while v15.14.0 is not. I'm not sure how and why, but it doesn't change anything, the version range is still incorrect:

This looks odd. So might be that there's an issue with node non-lts? Also kind of unsure why where the process import comes from, but probably something to do with rollup. Not sure if bumping engine is the right approach or to find out how to work around the issue.

Ok, so might need to bump to at least 14.18

Also kind of unsure why where the process import comes from, but probably something to do with rollup

I'm almost sure it's Rollup.

Not sure if bumping engine is the right approach or to find out how to work around the issue.

I'm here with you. I opened this PR to start a conversation. I would suggest not to include node:process, as it doesn't present in the ESM version and works (from what I can see) well. If there's a difference between using process with/without node:process, it might lead to inconsistent behavior across versions.
firebase-tools (that uses marked-terminal) also have this problem, but they exclude 15 with this query ^14.18.0 || >=16.4.0. I doubt that's the best for marked-terminal, but it's sensible for them because they don't support v15 on Firebase Functions

Now requires 16 as a minimum (could also drop that as it has end-of-life, but trying to be backwards compatible).


Calendar() method returns Error 401 with the valid api key

I generate an api key from the website and used the code snippet s there to test the api. Although everything was fine with get() method, but the calendar() method returns the Error:401 with the same api key. here is the output:
Error: 401
{'message': 'You have exceeded the rate limit for this API key. Please try again at 12:54 Eastern Standard Time.'}
The message is shown even at the first call.

Hey Sara,
Its because you're using the get method followed by the calendar method
If you just intend on using the Calendar, then use the Calendar method only
Let me know if this helps,
JBlanked

Yep, My bad ;) You're right. Thanks for the great job.

Thanks friend!! I'm glad it's working for you. Have a wonderful day.


python3Packages.mat2: 0.12.2 -> 0.12.3

Motivation for this change
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)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 155170 run on x86_64-linux 1

3 packages built:

mat2 (python39Packages.mat2)
metadata-cleaner
python310Packages.mat2

I manually cherry-picked the mat2 bump to https://github.com/NixOS/nixpkgs/pull/157323.

I manually cherry-picked the mat2 bump to https://github.com/NixOS/nixpkgs/pull/157323.


Extract pingometer connection into before filter

Would de-duplicate pingometer connection/raise code.

I’m not sure that makes the most sense. Didn’t you say we should be caching all the data locally? (#17) If that’s the case, the only time we should be hitting Pingometer is in the event hook and in tasks. Not sure a before filter is worthwhile when only one route needs it.

Ah, ok in #17. I was just seeing copy-pasted code in both routes and wanted to clean it up:

https://github.com/codeforamerica/snap-it-up/blob/master/app.rb#L52-L58
https://github.com/codeforamerica/snap-it-up/blob/master/app.rb#L115-L121

It is copy-pasted, but the thing is it shouldn’t really be there at all in one of those ;)
If you want to clean it up, I think the right clean up is to finish the local caching work, not de-dupe code that shouldn’t be there in the first place.


Fix: render the sun

As per #3800.
In revision 1063c8356b7ba96b6f5e0c95e233587f123e5023, the first parameter of SceneTransforms.clipToDrawingBufferCoordinates changed. The sun renders properly with this fix

Nice find!

@lilleyse please update CHANGES.md in master.


FIX-#2798: Fix number of partitions for dataframe on a cluster

Signed-off-by: Igoshev, Yaroslav<EMAIL_ADDRESS>
What do these changes do?

 They seem unrelated to the changes.

Yes, it looks more reliable (albeit bloated a little, I think). Fixed.

I don't think it is critical issue. I suggest leaving as it is.

@vnlitvinov For testing and debugging purposes I often set and reset the number of partitions. Part of the appeal of Modin's architecture is the flexibility in partitioning so making it read-only would be counter to this.
I am imagining at some point in the future that the system can change the global number of partitions on the fly, like when new compute nodes become available or nodes die/go away. Would we still have this flexibility with your approach?

@devin-petersohn , we can set and reset the number of partitions as it was before. The PR is adding a separate class method is_put, which is read-only, to handle initialization of the number of partitions correctly.

@devin-petersohn , yes, you are right, though. It won't work for the following case:
import modin.pandas as pd
from modin.config import NPartitions
import numpy as np
NPartitions.put(10)
data = np.random.randint(0, 100, size=(2 ** 10, 2 ** 10))
df1 = pd.DataFrame(data)
NPartitions.put(11)
df2 = pd.DataFrame(data) # the number of partitions will be 10

@vnlitvinov For testing and debugging purposes I often set and reset the number of partitions. Part of the appeal of Modin's architecture is the flexibility in partitioning so making it read-only would be counter to this.

I never suggested to make the variable read-only, only the field is_put to highlight that it isn't expected to be modified outside of NPartitions class.

I am imagining at some point in the future that the system can change the global number of partitions on the fly, like when new compute nodes become available or nodes die/go away. Would we still have this flexibility with your approach?

However, if a user doesn't touch NPartitions in any way and new compute nodes become available or nodes die/go away in the middle of a workflow, the number of partitions will be equal to its initial value. An example:
# Let's say that we have 3 nodes with 16 cores on each one
import modin.pandas as pd
df = pd.DataFrame(frame_data) # NPartitions is equal to 48
...
# one node dies unexpectedly
df2 = pd.DataFrame(frame_data2) # NPartitions is equal to 48, not 32. A user should set NPartitions himself

Something went wrong with TeamCity tests.

Rebased on the latest master to restart the tests.


repo DonPAPI changed location of DonPAPI.py

chmod: cannot access '/opt/lwp-scripts/DonPAPI-main/DonPAPI.py': No such file or directory

Thanks! Fixed in 0.8.3
Now the ./install.sh script installs DonPAPI using pipx instead of downloading the repo.


Update azure-stack-security-foundations.md

Looks like the link for Microsoft Service Trust Portal is incorrect. Changed it on this fork.
https://servicetrust.microsoft.com/ViewPage/Blueprint --> This is where it points to
https://servicetrust.microsoft.com/  --> This is where it should probably go to

@rammalkani : Thanks for your contribution! The author(s) have been notified to review your proposed change. @JustinHall