HIDP_CAPS and InputReportByteLength/OutputReportByteLength/FeatureReportByteLength

Seems report ID size (1 byte) is always prepended to the report data and InputReportByteLength/OutputReportByteLength/FeatureReportByteLength value contains it - even if report IDs are not used in TLC of HID Report Descriptor.
Please confirm this or refute and fix the page accordingly. Current description is super confusing.
InputReportByteLength

Specifies the maximum size, in bytes, of all the input reports (including the report ID, if report IDs are used, which is prepended to the report data).

OutputReportByteLength

Specifies the maximum size, in bytes, of all the output reports (including the report ID, if report IDs are used, which is prepended to the report data).

FeatureReportByteLength

Specifies the maximum length, in bytes, of all the feature reports (including the report ID, if report IDs are used, which is prepended to the report data).

Same applies to HidD_GetInputReport/HidD_GetFeature/HidD_SetFeature/HidD_SetOutputReport's ReportBufferLength and IOCTL_HID_GET_INPUT_REPORT/IOCTL_HID_GET_FEATURE/IOCTL_HID_SET_FEATURE/IOCTL_HID_SET_OUTPUT_REPORT
buffer length.
If the collection includes report IDs, the requester must set the first byte of the buffer to a nonzero report ID. Otherwise, the requester must set the first byte to zero. The output report -- excluding its report ID, if report IDs are used -- is located at ((PUCHAR)ReportBuffer + 1).
...
Input buffer length
The input buffer size, in bytes, must be large enough to hold the output report -- excluding its report ID, if report IDs are used -- plus one additional byte that specifies a nonzero report ID or zero.

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

ID: c659f9ab-41d9-1898-439e-7d4eae1a3cab
Version Independent ID: 5f131874-7fa6-f239-d726-3facd3f36dda
Content: _HIDP_CAPS (hidpi.h) - Windows drivers
Content Source: wdk-ddi-src/content/hidpi/ns-hidpi-_hidp_caps.md
Product: windows-hardware
Technology: wdk-api-reference
GitHub Login: @mhopkins-msft
Microsoft Alias: mhopkins

Hi @DJm00n
Thank you for this very thorough report. I am researching this and will update the topics when I have more information.
Thanks!
~ Mark

Hi @DJm00n,
To me, this is saying that the ID byte is always present. If it is not being used, the value of the ID byte will be zero. Therefore, I don't think this is incorrect, perhaps it could just be better written.
Is that consistent with your experience?
Thanks,
~ Mark

Hi @mhopkins-msft!
Yes, that is consistent. Current wording is somewhat confusing and misleading.
Adding @samtertzakian and @nefarius. Maybe they have some thoughts on this topic.

Hi @DJm00n,
Thank you for verifying. I've updated the topics to hopefully make them clearer. They should be republished soon.
As always, thank you for helping make our docs better!
~ Mark

Thank you @mhopkins-msft!
One more thing.
Can you please add something like This size is specified in InputReportByteLength/OutputReportByteLength/FeatureReportByteLength member of a HIDP_CAPS structure returned from HidP_GetCaps call. to the to HidD_GetInputReport/HidD_GetFeature/HidD_SetFeature/HidD_SetOutputReport's ReportBufferLength decription.


add no_main arg support in ockam::node macro

Current Behaviour

the current behaviour is documented in #2358 .
Proposed Changes

fixes #2358 .
Checks

[ ] All commits in this Pull Request are signed.
[ ] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor Licence Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the ockam-network/contributors repository.

well i guess this PR will be closed since i have ticked almost none of the checks...

Hi @pro465, thank you for sending this pull request ❤.
No, If you could accept our CLA, a merge should be pretty easy from there.

Read the CLA at: https://www.ockam.io/learn/how-to-guides/contributing/cla

To accept the CLA, please send a different pull request to our
contributors repository indicating that you accept
the CLA by adding your Git/Github details in a row at the end of the
CONTRIBUTORS.csv file.

thank you for the instructions!
unfortunately, it looks like one of the node tests failed. i will try to fix that after school tho.

Awesome, thank you.
When you're ready here's how to make the Commits check pass:
git reset develop
git add .
git commit -m "feat(rust): add `no_main` arg support in `ockam::node` macro"
git push --force

thank you again for the instructions!
unfortunately, i have little time remaining to go to school, so i gotta go...
see you later!

@mrinalwadhwa hello, im back!
TODO:

[ ] fix tests
[ ] improve wording in case of invalid syntax (i.e, #[ockam::node(time = 400)])
[ ] sign commit + pass commit checks

i think that the failed test was literally matching the error message char by char, and so when the reported error changed, it failed.
but i could not find where the tests are located. could you help me here?

@pro465 that's right, the macro tests use trybuild and are located here
https://github.com/ockam-network/ockam/tree/develop/implementations/rust/ockam/ockam/tests/node

We should also remove the no_main feature at the same time as this (since it should do nothing afterwards)

@thomcc well i guess that will be a breaking change, so that may require a major version bump.
let me add that to the TODO...

any suggestion for a better error message than just invalid syntax for this macro?
i could think of 'ockam::node' does not support this kind of syntax

'ockam::node' does not support this syntax.
Hmm 🤔

Bumping version to 0.4.0.-dev makes sense cc: @jared-s please note.
One think to note - we don't allow merge commits in the develop branch so please rebase to develop whenever you're ready for a merge.

@mrinalwadhwa ok got it, thank you

gotta go to school. since school time should be enough for anyone to suggest a better error message, we will see after school hours.
see ya later!

@mrinalwadhwa hello, im back!
so, looks like nobody suggested a better error message
so, since your suggestion has the highest "votes" in this PR, im gonna apply that, and also change the test error message to that.

seems like im almost done. just need to check if the tests succed, and then,

[ ] force push it to pass the commit convention test
[ ] sign the commit
[ ] rebase it onto develop

and we should be good to go & merge!

so i found that the error told me to update the version specified for ockam_macros in ockam/Cargo.toml
and there i noticed that ockam itself has a no_main feature, so i removed that and bumped ockam's minor version to indicate breaking change.
hopefully the version bump will not break ockam...

off to school, bye!

'ockam::node' does not support this syntax.

Hmm 🤔

@mrinalwadhwa
hey, how about unrecognized param passed?

and btw, i tested the branch locally and fixed the failing test (due to typos), and cargo check-ed the repo. and it passed without any compile errors, so i think this should also pass the tests, since i did not change much things outside the scope of this PR...
basically, this should finally pass the workflows (except the commit checks)

@mrinalwadhwa done!
could you please tell me the next steps (after force-pushing to pass the commit checks) ?

@pro465 that should be it, looks like the only remaining check to pass in the commit linter.
https://github.com/ockam-network/ockam/runs/4630472803?check_suite_focus=true#step:5:89
Maybe squash all commits down to a single commit with this message:
feat(rust): add no_main arg support to ockam::node macro

And sign it.

seems like i dont have the gpg command, so i cant it yet...

Here's a tutorial from Github that may help
https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

@mrinalwadhwa unfortunately, it didn't help :( i can't use or even install gpg through apt-get.
so i think i might need you to do that and open a PR to the forked repo; and, if necessary, transfer ownership of the forked repo to you, so that you can sign it and merge it yourself

@pro465 no problem, I can do the signing, could you squash all the changes down to a single commit please.
I'll sign and merge after that.

now we need to git rebase develop, right?

@pro465 I merged this in https://github.com/ockam-network/ockam/pull/2407
Congratulations and thank you for an awesome first contribution 🎉

@mrinalwadhwa thank you!

@mrinalwadhwa hey, we forgot to document about the possibility of no_main param in ockam::node!


BrowserModule should be removed

If I import the NgxSmartModal in my shared module i will get an error saying "BrowserModule has already been loaded. If you need access to common..."
To make it work I have to remove  the BrowserModule and BrowserAnimationModule from the NgxSmartModal decorators in ngx-smart-modal.js
NgxSmartModalModule.decorators = [
{ type: NgModule, args: [{
declarations: [NgxSmartModalComponent],
exports: [NgxSmartModalComponent],
imports: [CommonModule]
},] },
];

@lmauri Thank you for this report. I'll take a look at it asap. I'll firstly check if it won't break anything in project that currently use it.
Here is the concerned file.
I'll notice you when it's fixed.

Should be fixed by #38 .

PR #38 has been merged ! It should solve this issue. Many thanks to @thomascsd! 👍


Resque >1.18 and Resque.enqueue_at

Hi,
In the Readme you write that resque-remote is not necessary since it implements Resque.enqueue_at.
I've searched for this method in Resque 1.19 and couldn't find it or anything similar.
Would you have any explanation ?

It was a mistake that I made in assuming it was a part of the gem since it was in master. If you backtrack the commits on resque you can see that feature made it in around july of 2011, and the next release was 1.18, so I assumed it was in that release.
Later I realized that it is still not in any of the stable gem versions of resque and only to be found in master. Resque-remote (or similar) will obviously still be needed until that method is released as part of resque.
I'll make the change to the README.

I changed the wording in the README to reflect this in commit 9f88db2c.

Thanks for the clarification.


Comments are empty for non-canonical URLs

Comments show up at https://developers.google.com/web/updates/2017/02/media-session while they don't at https://developers.google.com/web/updates/2017/02/media-session#resources.
I guess we need to use the canonical URL when loading G+ comments.
R:@petele

The comment widget uses the following script block:
<script async defer src="https://apis.google.com/js/platform.js" onload="gapi.comments.render('gplus-comments', {href: window.location, width: document.getElementById('gplus-comment-container').offsetWidth, first_party_property: 'BLOGGER', view_type: 'FILTERED_POSTMOD'});"></script>

To fix it, we probably need to build the href param from window.location,  but without the hash.

I've sent you a patch to review. Let me know what you think.

Reviewed, I can't approve, but have tagged in someone who can.

Internal PR has been accepted and it is now live!


Rename (Minimum)EigensolverResult.aux_operator_eigenvalues from "eigenvalues" to just "values"

Summary
The eigensolver algorithms allow the evaluation of the expectation value of auxiliary operators at the final optimal point. These expectation values are generally not the eigenvalue of the auxiliary operators. However we currently store these expectations in an attribute called aux_operator_eigenvalues which is a bit misleading. This commit renames the attribute to just aux_operator_values to reflect that it's not an eigenvalue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

39 of 39 (100.0%)  changed or added relevant lines in 6 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.004%) to 84.381%

Files with Coverage Reduction
New Missed Lines
%

qiskit/utils/deprecation.py
4
85.19%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
54515

Relevant Lines:
64606

💛  - Coveralls

This has been addressed by #8702 where the result has been renamed to aux_operators_evaluated 🙂


Docs: support IE11

Conditionally load a script with polyfills in IE 11 so the docs render correctly.
Before: 'Symbol' is undefined

After: Documentation renders in IE11

Confirmed the docs now work on IE11: https://deploy-preview-957--gestalt.netlify.app/


Store does not implement IUserLockoutStore.

I am getting the above error when trying to login on a standard MVC 5 project built following your instructions. This is the line of code where the exception happens:
var result = await SignInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, shouldLockout: false);

which belongs to the following method in the AccountController:
    //
    // POST: /Account/Login
    [HttpPost]
    [AllowAnonymous]
    [ValidateAntiForgeryToken]
    public async Task<ActionResult> Login(LoginViewModel model, string returnUrl)
    {
        if (!ModelState.IsValid)
        {
            return View(model);
        }

        // This doesn't count login failures towards account lockout
        // To enable password failures to trigger account lockout, change to shouldLockout: true
        var result = await SignInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, shouldLockout: false);
        switch (result)
        {
            case SignInStatus.Success:
                return RedirectToLocal(returnUrl);
            case SignInStatus.LockedOut:
                return View("Lockout");
            case SignInStatus.RequiresVerification:
                return RedirectToAction("SendCode", new { ReturnUrl = returnUrl, RememberMe = model.RememberMe });
            case SignInStatus.Failure:
            default:
                ModelState.AddModelError("", "Invalid login attempt.");
                return View(model);
        }
    }

Current Neo4jUserStore implementation does now implement the IUserLockoutStore<TUser, object> interface.


ci: fix issues after merges

Change .github/workflows/check_ucc_ui_files.sh to work with dynamic version after poetry build command.

Codecov Report

Merging #320 (67c0755) into main (274ea3f) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #320      +/-   ##
==========================================
- Coverage   77.01%   77.00%   -0.02%
==========================================
  Files          29       29
  Lines        1962     1961       -1
==========================================
- Hits         1511     1510       -1
  Misses        451      451

Impacted Files
Coverage Δ

splunk_add_on_ucc_framework/__init__.py
77.28% <100.00%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

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

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

v5.8.0
GitHub release

Your semantic-release bot :package::rocket:


asyncio in 3.5.1 breaks cython test-suite (or vice versa)

Cython test suite has tests specifically for asyncio 3.5 generators, as far as I understand.
Compiling cython 0.23.4 fails with python 3.5.1
(see https://launchpad.net/ubuntu/+source/cython/0.23.4-0ubuntu1 click on a desired arch, then click on buildlog, direct link for e.g. amd64 buildlog is https://launchpadlibrarian.net/233177552/buildlog_ubuntu-xenial-amd64.cython_0.23.4-0ubuntu1_BUILDING.txt.gz )
Reverting the changes to asyncio module introduced between 3.5.0 and 3.5.1 makes said test pass:
======================================================================
FAIL: runTest (__main__.EndToEndTest)
End-to-end asyncio_generators
----------------------------------------------------------------------
Traceback (most recent call last):
  File "runtests.py", line 1417, in runTest
    self.assertEqual(0, res, "non-zero exit status")
AssertionError: 0 != 1 : non-zero exit status

Now, this could be a bug in cython, cython test-suite, or the asyncio module... However, I cannot quite determine what's wrong is going on.
I guess I should also try running bisect with git checkouts of asyncio module... Would you be able to help with this? I'll report this bug to cython upstream too...

I recommend that you ask help from the Cython project first -- it's their
tests that are breaking.

Fixed in https://github.com/cython/cython/commit/7eed8d8ff9c872bc993a37438beca0f98e3aba38


Updated not working youtube title search

Since I know only basics of JS, I fixed it in simple way. Tested and works fine.
Noticed too late, but code isn't aligned properly for some reason :)

Hello, yes I might check it out later in free time. I was pretty sure there
would be some unexpected behaviour, but didn't have time to do proper
research. Thanks for info!
2018-03-09 10:48 GMT+01:00 Paco Hobi<EMAIL_ADDRESS>
@PacoHobi requested changes on this pull request.
Many thanks for your PR. I personally don't like to get info from the
title because the title sometimes contains info even if you are not playing
a video. For example for https://www.youtube.com/feed/trending the title
is "Trending - YouTube", and the PR changes would return "Trending" for
this case, which is not the expected behaviour.
I would recommend to try to get the info from the DOM, I had a quick look
and it seems that you can now get the video title with:
document.getElementById("video-title").firstChild.nodeValue
Are you up to trying this approach and opening a new PR or changing this
one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PacoHobi/CurrentSong/pull/37#pullrequestreview-102592379,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATg74eIoqH-NDimSDtucrbBGnpIpGkPlks5tclAJgaJpZM4SjNen
.


Task - Browse Collecting Event - Edit Collecting Event: Add/Current: no identifiers displayed

Feature or enhancement
In BCE, click Edit/pencil icon to edit CE, click Add/Current button.
After clicking ‘Add/Current’ the panel displaying the COs associated with that CE does not show the identifier/catalog number or determination (see screenshot/ignore red circle here). Fix: display the catalog number and determination (OTU?). The ID number is not useful to us.
@bpescador, @cgrinter, @rich-keller
Location
Task - Browse Collecting Event - Edit Collecting Event
Screenshot, napkin sketch of interface, or conceptual description

Your role
No response

Seconded, this is very annoying. Would love to see the catalog numbers/determinations I just created without a refresh.

Months ago, we made some changes on the server responses when we implemented extend parameter to optimize and reduce response time. Because of that, these attributes were missing there. Let us know if you see problems like this on other places to fix it.


What are the concerns?

Could you please elaborate a little about the concerns with enabling this option. I understand it might be IP/URL concerns to reach the external data sources, but do we also have security concerns like information from local sources might be visible in the queries for the external data sources, etc?

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

ID: 183a8242-3c94-8a80-6d36-85c608790a25
Version Independent ID: 2d8728c3-3662-27a3-6cd4-2c88b08f7847
Content: Merge or append on-premises and cloud data sources - Power BI
Content Source: powerbi-docs/service-gateway-mashup-on-premises-cloud.md
Service: powerbi
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Thanks for your comment, @erapade.  We’ve forwarded your feedback to the writer, who should respond shortly. Thanks!

@arthiriyer - can you answer the question in this issue? Thanks!

@erapade  This feature should be used along with the privacy settings for each data source to make sure the joins are handled in a secure manner.  However Privacy settings in Power BI desktop are not transferred to the service. You can set privacy settings for the On-prem data sources. For cloud data sources, it is set to “Private” by default in the service with an exception for the ones which are using anonymous authentication and these are set as “Public” by default.   We are however working on a feature which will help with setting these  explicitly for cloud data sources as well

Added a sentence to step 6 in “Merge or append on-premises and cloud data sources” (https://docs.microsoft.com/power-bi/service-gateway-mashup-on-premises-cloud) about making sure the privacy levels for both cloud and on-premises data sources are set appropriately.


Typo fix on  DropZonePage Component Documentation

Description

Possible typo fix if I understand the context correctly on the site on DropZone Component.

How Has This Been Tested?

Types of changes

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

Checklist:

[ X] The PR is submitted to the correct branch (dev).
[ ] My code follows the code style of this project.
[ ] I've added relevant tests.

The sentence could probably do with reworking but merging the obvious typo.  Thank you for taking the time to PR 🙏


Loading The Biases on To the FPGA !

Hii!! I wanted to ask about the part in "Using a Caffe Pretrained Network - CIFAR10.ipynb" where you've loaded the weights on the FPGA. After that, I thought that the biases will be loaded as well on the FPGA, but it wasn't. Is there a specific reason behind not loading the biases and using them ?

Hi, if I remember correctly my network was trained without biases. With small convolutional neural networks classifying small-scale datasets, biases are not that critical because we centralise the inputs anyway.


[step challenges] Skip button

In the Waypoints, there's only Go to next step option. There's no previous step to go back if I wanted to.
Browser's back button takes me to the start not the previous page.
Also, a skip button would be great. I had to click 'open link in new tab' every time to go to next step. What if I already had a github account. What if I already finished the chapter and I'm revisiting.
Back / Previous and Skip buttons would be of great help and make a better UX.

Certainly, that's a missing feature. I hope it will be added soon.
cc: @QuincyLarson

@sanspace Thanks for this suggestion. We are already working on a back button, and a skip button is a good idea.

 We just need a skip button now. Is anyone interested in helping with this?

I would like to work on this Skip Step button. Can I take this?

@gasperno of course.

I would like some suggestions regarding my idea behind Skip Button.
Instead of having an additional button, I would like to enable 'Go to next step' in two cases.

If the user profile is linked to Github or
If the user has already completed the challenge.

Is it the goal of forcing the user to 'open the new tab' is to link their Github profiles?
If a user is not linked to github and he/she completed the challenge, should such users be forced to open new tab or not?

@gasperno There's a couple step challenges that force you to open a link and not all of them are for Github, so that would not be a valid reason to let users skip. However, I really like the idea of not having a separate button, but just not disabling the button if they've already completed the challenge.

I noticed that there are the following types of links.

github links (login, profile picture, star fcc repo)
gitter link(login, download app, fcc/help)
freecodecamp/wiki
linkedin
commit to non-profit

@ltegman I think it is safe to force all users(logged via github or not) to click 'Open link in new tab' unless they completed the challenge. Even if they are logged in with github, 'Open in new tab' encourages them to set their profile picture,  go to fcc github repository. As you mentioned, all other non github links are definitely important to visit.
Besides, I think another button is not required as there would be 'Go to next step' button which is always disabled and Skip Step button which is always enabled-- allowing user to do the same function that Go to next step is preventing the user to do.


set numpy v1.19.1 in env3.6 install

Modification explanations:

The Contam Overlap tool crashes when trying to calculate Contamination & Visibility on the website. I recently found out our production server runs Python 3.6, while I originally wrote the tool updates and did all my test in Python 3.7. While attempting to run my tools in a fresh 3.6 environment, I received an error in sossContamFig telling me there was no where parameter in np.sum(). The where parameter was added in NumPy 1.17.1 (https://numpy.org/doc/stable/reference/generated/numpy.sum.html), but our 3.6 environment in production runs Numpy 1.15.

This seems to be the only snag I get from testing my 3.7 code in a 3.6 environment, so I've updated our 3.6 environment file to require 1.19.1 instead of 1.15.?

I also expanding the search radius for NIRCam and MIRI from 0.5" to 1.0" to query for more stars.

Hello @jaymedina, Thank you for submitting the Pull Request !

In the file exoctk/contam_visibility/sossFieldSim.py:

Line 161:24: W503 line break before binary operator
Line 163:24: W503 line break before binary operator
Line 168:75: W504 line break after binary operator
Line 178:37: W504 line break after binary operator
Line 266:5: E265 block comment should start with '# '
Line 285:5: E265 block comment should start with '# '
Line 315:5: E265 block comment should start with '# '
Line 339:5: E265 block comment should start with '# '
Line 376:5: E265 block comment should start with '# '
Line 387:5: E265 block comment should start with '# '
Line 407:39: E502 the backslash is redundant between brackets
Line 486:5: E265 block comment should start with '# '
Line 509:5: E265 block comment should start with '# '
Line 528:5: E265 block comment should start with '# '
Line 558:5: E265 block comment should start with '# '
Line 582:5: E265 block comment should start with '# '
Line 620:5: E265 block comment should start with '# '
Line 631:5: E265 block comment should start with '# '
Line 646:39: E502 the backslash is redundant between brackets
Line 759:5: E265 block comment should start with '# '

If you have not done so, please consult PEP8 and PEP257

@jaymedina Nevermind, I just took care of this since we want to get this in before the 1.1 release!

Thanks!


When the user has replied, the user is still prompted to reply. Why

Bug Report

I have read:

Usage information
Help information

I am using the latest version of the library.
Expected Behavior

Actual Behavior

Steps to reproduce the Behavior

Don't show reply input after use replied.

Try to move "removeReplyListener" to the first callback function, where your bot sends "You should add repo firstly!..."

https://github.com/alanhg/github-issue-bot/blob/c39269b8b8ba2d19d4bb31cc9754be8a598a4d99/app.js#L98-L98
I think you did not understand the problem, maybe my code block is not clear.

It would be nice to remove the listener once the user response is received.
// In function async...
let listenerReply;

let contentMessage = await bot.sendMessage(chat_id, text, {
    "reply_markup": {
        "force_reply": true
    }
});

listenerReply = (async (replyHandler) => {
    // More code...

    bot.removeReplyListener(listenerReply);

});

bot.onReplyToMessage(contentMessage.chat.id, contentMessage.message_id, listenerReply);

The next message sent by the bot should set property force_reply in reply_markup to false

Same here, removeReplyListener does not help!

this.bot.deleteMessage

this.bot.deleteMessage worked, thank you.

❤


Java app containerization and migration to Azure Container Apps

The Azure Migrate: App Containerization tool Java app containerization and migration Tool Today,  currently supports :

Containerizing Java Web Apps on Apache Tomcat (on Linux servers) and deploying them on Linux containers on AKS

Containerizing Java Web Apps on Apache Tomcat (on Linux servers) and deploying them on Linux containers on App Service

Containerizing ASP.NET apps and deploying them on Windows containers on AKS.

Containerizing ASP.NET apps and deploying them on Windows containers on Azure App Service.

RFE: Could you please intagrate this tool with  Azure Container Apps :

Containerizing Java Web Apps on Apache Tomcat (on Linux servers) and deploying them on Linux containers on Azure Container Apps
Containerizing .Net core apps and deploying them on on Linux containers on Azure Container Apps

@dariagrigoriu @kendallroden

@rahug1190

This should now be possible since we enabled this via Buildpacks some time back. Please get started here: https://learn.microsoft.com/en-us/azure/container-apps/java-overview


Changes to jit.rs broke compatibility with ARM64 Linux

https://github.com/LaurentMazare/tch-rs/commit/eae2b4ab9c4ce3fe6bda24f71fb06c1e68427a18 changed src/wrappers/jit.rs in a way that made it incompatible with aarch64 (ARM64 Linux) because, on that platform, c_char is unsigned.
As a result, attempting to compile the library on that platform causes this error:
error[E0277]: a value of type `Vec<u8>` cannot be built from an iterator over elements of type `i8`
#10 88.53    --> src/wrappers/jit.rs:270:44
#10 88.53     |
#10 88.53 270 |                 let v: Vec<libc::c_char> = v.iter().map(|&b| i8::from(b)).collect();
#10 88.53     |                                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ------- required by a bound introduced by this call
#10 88.53     |                                            |
#10 88.53     |                                            value of type `Vec<u8>` cannot be built from `std::iter::Iterator<Item=i8>`

Reverting the change allows the compilation to succeed.
I can submit a quick PR to revert this and restore portability.

Sorry about this and thanks for reporting, I've just pushed a fix that hopefully should work on both platform and not trigger the clippy lint that caused this code to be change.

Closing this as this is hopefully fixed, feel free to re-open if it's still an issue (I don't have an arm64 computer at hand to try it out).


Disabling/Skipping failing Windows tests

Desired behavior
Running the ign-gazebo buildfarm should be green as long as no new regressions are added to the code.
Alternatives considered

Using SKIP option with Cmake to specifically skip running some tests for Windows.
Using Labels to skip running some tests in our CI jobs, something like https://github.com/ros2/ros2/issues/900.
Remove the failing tests from the Windows CMakeLists.txt, so they don't compile at all for windows. (it would be good to avoid this one).

Implementation suggestion
I'm OK with either option 1 or 2, I don't know in advance which one would be preferred by the Ignition team or if there are other disabled tests in the stack so I could rely on infrastructure already in place. So I would like some feedback on it before starting.
Additional context
As of https://build.osrfoundation.org/job/ign_gazebo-ign-6-win/20/, the failing tests for ign-gazebo6 Windows are:
ign-gazebo6 tests failing, 60 tests failed out of 174
         7 - UNIT_Component_TEST (Failed)
         11 - UNIT_EntityComponentManager_TEST (Failed)
         21 - UNIT_SdfGenerator_TEST (Failed)
         25 - UNIT_Server_TEST (Failed)
         27 - UNIT_SimulationRunner_TEST (Failed)
         39 - UNIT_ign_TEST (Failed)
         51 - UNIT_EntityFeatureMap_TEST (Failed)
         53 - INTEGRATION_ackermann_steering_system (Exit code 0xc0000374)
         55 - INTEGRATION_air_pressure_system (Failed)
         57 - INTEGRATION_altimeter_system (Failed)
         59 - INTEGRATION_apply_joint_force_system (Failed)
         61 - INTEGRATION_battery_plugin (Failed)
         63 - INTEGRATION_breadcrumbs (Failed)
         65 - INTEGRATION_buoyancy (Failed)
         67 - INTEGRATION_buoyancy_engine (Failed)
         69 - INTEGRATION_collada_world_exporter (Failed)
         73 - INTEGRATION_contact_system (Failed)
         75 - INTEGRATION_detachable_joint (Failed)
         77 - INTEGRATION_diff_drive_system (Failed)
         79 - INTEGRATION_each_new_removed (Exit code 0xc0000374)
         81 - INTEGRATION_entity_erase (Failed)
         83 - INTEGRATION_events (Failed)
         85 - INTEGRATION_examples_build (Failed)
         87 - INTEGRATION_follow_actor_system (Failed)
         89 - INTEGRATION_force_torque_system (Failed)
         93 - INTEGRATION_halt_motion (Failed)
         95 - INTEGRATION_imu_system (Failed)
         97 - INTEGRATION_joint_controller_system (Failed)
         99 - INTEGRATION_joint_position_controller_system (Failed)
        101 - INTEGRATION_joint_state_publisher_system (Failed)
        103 - INTEGRATION_joint_trajectory_controller_system (Failed)
        105 - INTEGRATION_kinetic_energy_monitor_system (Failed)
        107 - INTEGRATION_lift_drag_system (Failed)
        109 - INTEGRATION_level_manager (Failed)
        111 - INTEGRATION_level_manager_runtime_performers (Failed)
        115 - INTEGRATION_logical_camera_system (Failed)
        117 - INTEGRATION_logical_audio_sensor_plugin (Failed)
        119 - INTEGRATION_magnetometer_system (Failed)
        123 - INTEGRATION_multicopter (Failed)
        125 - INTEGRATION_multiple_servers (SEGFAULT)
        127 - INTEGRATION_nested_model_physics (Failed)
        129 - INTEGRATION_network_handshake (Failed)
        131 - INTEGRATION_odometry_publisher (Failed)
        133 - INTEGRATION_particle_emitter (Failed)
        135 - INTEGRATION_particle_emitter2 (Failed)
        137 - INTEGRATION_performer_detector (Failed)
        139 - INTEGRATION_physics_system (Failed)
        143 - INTEGRATION_pose_publisher_system (Failed)
        145 - INTEGRATION_save_world (Failed)
        147 - INTEGRATION_scene_broadcaster_system (Failed)
        149 - INTEGRATION_sdf_frame_semantics (Failed)
        151 - INTEGRATION_sdf_include (Failed)
        153 - INTEGRATION_spherical_coordinates (Failed)
        155 - INTEGRATION_touch_plugin (Failed)
        157 - INTEGRATION_triggered_publisher (Failed)
        159 - INTEGRATION_user_commands (Failed)
        161 - INTEGRATION_velocity_control_system (Failed)
        163 - INTEGRATION_log_system (Failed)
        165 - INTEGRATION_wheel_slip (Failed)
        167 - INTEGRATION_wind_effects (Failed)

As of https://build.osrfoundation.org/job/ign_gazebo-ign-5-win/45/, the failing tests for ign-gazebo5 Windows are:
ign-gazebo5 tests failing, 59 tests failed out of 168
          3 - UNIT_Component_TEST (Failed)
          9 - UNIT_EntityComponentManager_TEST (Timeout)
         13 - UNIT_ign_TEST (Failed)
         19 - UNIT_ModelCommandAPI_TEST (Failed)
         23 - UNIT_SdfGenerator_TEST (Failed)
         25 - UNIT_Server_TEST (Failed)
         29 - UNIT_SimulationRunner_TEST (Failed)
         47 - UNIT_Gui_TEST (Timeout)
         51 - UNIT_EntityFeatureMap_TEST (Failed)
         53 - INTEGRATION_ackermann_steering_system (Exit code 0xc0000409)
         55 - INTEGRATION_air_pressure_system (Failed)
         57 - INTEGRATION_altimeter_system (Failed)
         59 - INTEGRATION_apply_joint_force_system (Failed)
         61 - INTEGRATION_battery_plugin (Failed)
         63 - INTEGRATION_breadcrumbs (Failed)
         65 - INTEGRATION_buoyancy (Failed)
         67 - INTEGRATION_collada_world_exporter (Failed)
         71 - INTEGRATION_contact_system (Failed)
         73 - INTEGRATION_detachable_joint (Failed)
         75 - INTEGRATION_diff_drive_system (Failed)
         77 - INTEGRATION_each_new_removed (Exit code 0xc0000374)
         79 - INTEGRATION_entity_erase (Failed)
         81 - INTEGRATION_events (Failed)
         83 - INTEGRATION_examples_build (Failed)
         85 - INTEGRATION_follow_actor_system (Failed)
         89 - INTEGRATION_halt_motion (Failed)
         91 - INTEGRATION_imu_system (Failed)
         93 - INTEGRATION_joint_controller_system (Failed)
         95 - INTEGRATION_joint_position_controller_system (Failed)
         97 - INTEGRATION_joint_state_publisher_system (Failed)
         99 - INTEGRATION_joint_trajectory_controller_system (Failed)
        101 - INTEGRATION_kinetic_energy_monitor_system (Failed)
        103 - INTEGRATION_lift_drag_system (Exit code 0xc0000374)
        105 - INTEGRATION_level_manager (Failed)
        107 - INTEGRATION_level_manager_runtime_performers (Failed)
        111 - INTEGRATION_logical_camera_system (Failed)
        113 - INTEGRATION_logical_audio_sensor_plugin (Failed)
        115 - INTEGRATION_magnetometer_system (Failed)
        119 - INTEGRATION_multicopter (Failed)
        123 - INTEGRATION_nested_model_physics (Failed)
        125 - INTEGRATION_network_handshake (Exit code 0xc0000374)
        127 - INTEGRATION_odometry_publisher (Failed)
        129 - INTEGRATION_particle_emitter (Failed)
        131 - INTEGRATION_particle_emitter2 (Failed)
        133 - INTEGRATION_performer_detector (Failed)
        135 - INTEGRATION_physics_system (Failed)
        139 - INTEGRATION_pose_publisher_system (Failed)
        141 - INTEGRATION_save_world (Failed)
        143 - INTEGRATION_scene_broadcaster_system (Failed)
        145 - INTEGRATION_sdf_frame_semantics (Failed)
        147 - INTEGRATION_sdf_include (Failed)
        149 - INTEGRATION_touch_plugin (Failed)
        151 - INTEGRATION_triggered_publisher (Failed)
        153 - INTEGRATION_user_commands (Failed)
        155 - INTEGRATION_velocity_control_system (Failed)
        157 - INTEGRATION_log_system (Failed)
        159 - INTEGRATION_wheel_slip (Failed)
        161 - INTEGRATION_wind_effects (Failed)
        167 - PERFORMANCE_level_manager (Failed)

Probably some of them overlap, so I'll start with the ign-gazebo5 tests and then forward port.

Thanks for ticketing the issue and looking into this, @Blast545 !
I recommend using the test macros provided by ign-cmake disable tests:
https://github.com/ignitionrobotics/ign-cmake/blob/ign-cmake2/include/ignition/utilities/ExtraTestMacros.hh
Here's an example:
https://github.com/ignitionrobotics/ign-gazebo/blob/c36d4aae1075aa7dce0c950a107ff05bf9ec7f54/test/integration/thermal_sensor_system.cc#L76-L77
This makes sure that the tests are still compiled on all platforms, they just don't run.

@chapulina I didn't know those existed, I'll use them. Thanks!

Mmm, is it important for this ign-gazebo5 test to create 100000 components? I also noted this test it not longer available in ign-gazebo6 :detective:
https://github.com/ignitionrobotics/ign-gazebo/blob/ba53127c5b1418c17ca0fdfdcdd04667c09b06ff/src/EntityComponentManager_TEST.cc#L184

Mmm, is it important for this ign-gazebo5 test to create 100000 components?

I guess we could reduce the number, is this causing issues on Windows?

I also noted this test it not longer available in ign-gazebo6 detective

That's because from Fortress the components aren't stored adjacently anymore

@chapulina Yeah, it's timing out on Windows. In particular, this check is failing:
https://github.com/ignitionrobotics/ign-gazebo/blob/ba53127c5b1418c17ca0fdfdcdd04667c09b06ff/src/EntityComponentManager_TEST.cc#L195-L200
There's a problem registering one of the components in the test in Windows so the test fails all of the times in windows, also printing thousands of lines to the output log.
I'm disabling the test for Windows so it won't be a problem there, just wanted to make sure that high number makes sense for that particular test.

Got it. It sounds like Windows has other problems besides the number of components. So I think for now it's enough to disable the test there, and we can revisit it later.


Using replace filter

Hello,
I see that twig.filter.replace can recieve 2 parameters: string and map,
I cannot figure out which format I need to pass the map.
Example:
{{ 'Number of reviews: ' | replace({'': '20'}) }}
Compiled into:
sb.append(twig.filter.replace("Number of reviews: ", {0: "", 1: "20"}));
And I expect that it will be compiled into:
{"": "20"}
I noticed that Twig_Node_Expression_Array use getKeyValuePairs in order to build hash array.
Thanks

same as for Twig itself (please see its documentation)

Thanks for your replay,
I re-checked again:
This is example fro Twig site:
{{ "I like %this% and %that%."|replace({'%this%': "foo", '%that%': "bar"}) }}
The output:
I like %this% and %that%.
Compiled string:
sb.append(twig.filter.replace("I like %this% and %that%.", {0: "%this%", 1: "foo", 2: "%that%", 3: "bar"}));
Expected compiled string:
sb.append(twig.filter.replace("I like %this% and %that%.", {"%this%": "foo", "%that%": "bar"}));
Possible patch: ArrayCompiler:38 replace with
$compiler->raw('{');
$first = true;
foreach ($node->getKeyValuePairs() as $pair) {
if (!$first) {
$compiler->raw(', ');
}
$first = false;
        $compiler
            ->subcompile($pair['key'])
            ->raw(': ')
            ->subcompile($pair['value'])
        ;
    }
    $compiler->raw('}');


Consider switching to fractional-ordering of all cards as the default sort

Instead of the error-prone method of having the entire array on a document. (Which is probably fine)
https://www.figma.com/blog/realtime-editing-of-ordered-sequences/#fractional-indexing/

Useful for #353.
Keeping track of order of sections is kind of a pain in general. There need to be expensive and error-prone transactions to splice sections as they're done. The only difference between sections and tags is that sections have an ordered list of cards, and sections are a kind of special tag that is the default collection the card shows up in. (Unless we do #259).
In general, what if sections didn't have any notion of order, but every single card was part of an ordering function intrinsically?
Then reading-list order would still have to exist (how specialized is this logic?)
Switching to this would allow "drag to reorder multiple cards" (covered in #174) and "drag to reorder in non-default sections" (#103) to work.

One thing to note: currently if you were to "import" a card from another card-web (e.g. #328) then you could define, by where you had the ID in the section ordering, where that imported card goes. But if we moved to each card having its own ordering based on a numeric field, then you'd need some other way to define where in the ordering the imported card goes.
... Or maybe you just don't let imported cards to go into the ordering--you can only link to them, and when you click to them you go to the other card-web. And if you want to "have" them in the main card-web you fork them, which then gives you a card object that can have its own ordering

Every card has a sort_fraction numeric field. The default set is all of the cards that have a non-orphan 'section', sorted in descending order by their sort_order field. There's a selectLowestSortFraction selector that returns the lowest sort order in the collection (note: this is another case where a user without access to all cards might not realize what the REAL lowest sort_fraction is, so new cards would inadvertently go somewhere in the middle.
Sections lose the notion of any particular order of their cards, it's just basically a tag that is exclusive, and the app configures which tags to order togeether into the default order.
When you add a new card, we do selectLowestSortFraction and subtract one and use that.
When you change card sort orders, we change the sort_fraction of the affected card (and only the affected card) to be halfway between the sort_fraction of the card immediately before it and the card immediately after it. If it's at the end, it gets a number halfway between the card immediately before/after it, and teh MAX_INT number.
Fractional indexing can get arbitrarily weird indexes the more reordering that happens. There needs to be a normalizeIndexing maintnence action that runs automatically, say, every 1000 reorders  or so (it doesn't really matter because it would take 100'0000 or more to start getting really weird.) normalizeIndexing goes through and distributes the indexes smoothly from - MAX_INT / 3 to positive MAX_INT / 3. That gives some space above and below to add new items.
This normalizeIndexing is a big deal transaction because it touches EVERY card, and nothing else should run during it. It also will always exceed the batch index size. Ideally we'd disable the cloud function that runs when cards change before running and then reenable. Maybe it's a gulp task that disables the updateLinks, runs the task as a one off, and then reenables? That's roughly the right cadence to do the fixup operation anyway.

How much specialized machinery for reading lists can be gotten rid of?
Also note: if you drag a card to the end of section, the card ID needs to be greater than that card, but lower than the next card that comes after (that's currently filtered out). In general, how should you handle reorders for cards that are notionally between the two cards, but filtered out? I guess assume that they go just after the first card, and before whatever card comes next that we know of (even if it's filtered)
And reordering should only be allowed for default sort order, disabled if it's any non-default sort. (Although I guess as a bonus we could allow reordering in default reversed order, just reversing the logic)

The approach of the renormalizing sort orders turning off updateInboundLinks and then re-enabling is really expensive, because the re-deploys of cloud functions are SLOOOOWWWWWWW.
It should be fine to run all of the functions on each card as it updates, as long as updateInboundLinks bails early if no links changed.
The updateFractionalSorts would be a cloud function that runs on a timer in th emiddle of the night, every 90 days or so. It's a simple matter of fetching all cards sorted by their current sort_fraction and then updating their sort_fraction to be smoothly distributed and then resaving, which will require a few batches of transactions but otherwise be OK.
MultiBatch is only available in the client, not on the server, but I guess I could copy it over...

We should spread the IDs not literally smoothly across the range, but smoothly across the range that will fit within a given transaction. That way even if one of the batches fails, the cards are still not incorrectly interleaved, because they were only smoothly spread out within their own batch.
... This might get a bit weird over time if the batches keep falling at roughly the same place. Maybe have the batch size be some random number between MAX_BATCH_SIZE and MAX_BATCH_SIZE / 2?

Note that things like createCard's rube goldberg style loading assumes that section also has to be reloaded to get the new set, but that won't be necessary with fractional ordering

[ ] Fix #504
[ ] Introduce a REORDERING_ENABLED bool that makes all reordering permissions pretend to be false. This will allow working on all of the machinery without making e.g. git bisect history allow doing reordering in an indeterminate state
[ ] Create the first fractional_sort maintenace task to give reasonable fractional sorts to all cards in dev
[ ] Keep on hacking to get all machinery working properly
[ ] Remove the REORDERING_ENABLED flag

All open work items are covered in sub-issues now


Add antenna height/gain changes with assert location feature

Can you add this as a separate transaction and only incur the Solana transaction fee?


is.edge() returns false in edge (tested on windows and mac os)

This is because the regex is:
var match = userAgent.match(/edge\/(\d+)/);
Whereas the user agent chosen by microsoft is edg. (see here https://docs.microsoft.com/en-us/microsoft-edge/web-platform/user-agent-string).
My user agent on macos is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.116 Safari/537.36 Edg/83.0.478.56
It is the same on windows too.
I propose changing the regex to: var match = userAgent.match(/edg\/(\d+)/);

This is because the newer versions of Edge are essentially Chromium now, and Microsoft changed the user agent to Edg to avoid inconsistencies with detecting the Spartan and Chromium user agents.
Given the reason why #214 was closed, I would wager that it would be acceptable that is.edge() returns false and is.chrome() returns true. I use is.ie() || is.edge() to detect cases where a user is using either IE or Edge Spartan, so if Edge Chromium were to be "fixed" I would personally prefer another function like is.edgeChromium() to detect newer versions of Edge instead.

is.edgeChromium()

I think this is a good proposal.

Any update on this? Would love to see edgeChromium.


Add SEI long term growth models to PyBaMM

Add the following SEI long term growth models:

[ ] Reaction limited
[ ] Solvent-diffusion limited
[ ] Electron-migration limited
[ ] Lithium-interstitial-migration limited

feel free to untangle the interface submodels as part of this 😅

@Scottmar93 I'll try to get this cleaned up and merged (I have merged in #933 )


Bug: U+2028 not escaped in strings

Description

Contrary to recommendations of the spec, the Unicode code points U+2028 and U+2029 are not escaped upon serialization. See Section 5.2 of the specification.
Deserialization fails when an input string contains Unicode code points U+2028 or U+2029.

Tested Versions
json5 = "0.2.7"
serde = { version = "1.0.106", features = ["derive"] }

Testcase
#[cfg(test)]
mod tests {
    use serde::{Serialize, Deserialize};
    use json5;

    #[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
    struct S {
        value: String
    }

    #[test]
    fn roundtrip_u2028() {
        let original = S {
            value: "\u{2028}".to_string()
        };

        let serialized = json5::to_string(&original).unwrap();
        let deserialized = json5::from_str::<S>(&serialized).unwrap();

        assert_eq!(deserialized, original);
    }
}

Any update on this one still failing? I just got bitten with parsing a remote JSON file which includes "\u{2028}". I've fixed it by replacing it before passing it to json5.


Restore the ability to disable ECH in Supermium

Since November 6, 2024, Roskomnadzor blocked Cloudflare ECH in Russia, as a result of which Supermium 126 stopped opening all sites hosted by Cloudflare, including the site with mods for the game The Sims 3 https://www.nraas.net. However, Google has removed from modern Google Chrome versions the ability to disable ECH by changing the #encrypted-client-hello flag to "Disabled" in chrome://flags, so this ability is not available in Supermium 126 either.
win32ss, can you restore to Supermium the ability to disable the #encrypted-client-hello flag in chrome://flags?

It's probably this commit: d871c7a7e27f8beb047bbc12111db6bde9833d3f :+1: ...

It actually isn't. There were some issues with that change, so it is being reverted and replaced with a simpler commit that is tied to the enterprise policy-set pref to disable it.

... FTR, Supermium-v121 is the last version where ECH can be controlled by the user via an exposed chrome://flags flag:


pip install error

c:\>pip install investpy Collecting investpy   Using cached https://files.pythonhosted.org/packages/cf/e8/1378dbf23b3390296fe5eff5155317eb782b59093a15714cfa77ceb4f42b/investpy-0.9.11.tar.gz     ERROR: Command errored out with exit status 1:      command: 'c:\users\xxx\appdata\local\programs\python\python38\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\xxx\\AppData\\Local\\Temp\\pip-install-o9ukmj96\\investpy\\setup.py'"'"'; __file__='"'"'C:\\Users\\xxx\\AppData\\Local\\Temp\\pip-install-o9ukmj96\\investpy\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\xxx\AppData\Local\Temp\pip-install-o9ukmj96\investpy\pip-egg-info'          cwd: C:\Users\xxx\AppData\Local\Temp\pip-install-o9ukmj96\investpy\     Complete output (7 lines):     Traceback (most recent call last):       File "<string>", line 1, in <module>       File "C:\Users\xxx\AppData\Local\Temp\pip-install-o9ukmj96\investpy\setup.py", line 35, in <module>         install_requires=parse_requirements(),       File "C:\Users\xxx\AppData\Local\Temp\pip-install-o9ukmj96\investpy\setup.py", line 17, in parse_requirements         with io.open('requirements.txt', encoding='utf-8') as f:     FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'     ---------------------------------------- ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Thank you for reporting this issue @kuhytitusdlz! I made a mistake on parse_requirements function on installation because I used a wrong version in the CI. Please stay tunned since I will be fixing this issue tomorrow and so on, a new investpy version will be released.

I had the same error when used "$ python -m pip install investpy". (having Python 2.7)
When tried to install using "$ python -m pip install https://github.com/alvarob96/investpy/archive/master.zip", it shows:
RuntimeError: Python version >= 3.5 required.
Seems installation from GitHub via PyPI is a better choice. :)

Still not working(
c:\temp>pip --no-cache-dir install investpy Collecting investpy   Downloading https://files.pythonhosted.org/packages/cf/e8/1378dbf23b3390296fe5eff5155317eb782b59093a15714cfa77ceb4f42b/investpy-0.9.11.tar.gz (4.3MB)      |████████████████████████████████| 4.3MB 819kB/s     ERROR: Command errored out with exit status 1:      command: 'c:\users\xxx\appdata\local\programs\python\python38\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\xxx\\AppData\\Local\\Temp\\pip-install-o8hgmgsc\\investpy\\setup.py'"'"'; __file__='"'"'C:\\Users\\xxx\\AppData\\Local\\Temp\\pip-install-o8hgmgsc\\investpy\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\xxx\AppData\Local\Temp\pip-install-o8hgmgsc\investpy\pip-egg-info'          cwd: C:\Users\xxx\AppData\Local\Temp\pip-install-o8hgmgsc\investpy\     Complete output (7 lines):     Traceback (most recent call last):       File "<string>", line 1, in <module>       File "C:\Users\xxx\AppData\Local\Temp\pip-install-o8hgmgsc\investpy\setup.py", line 35, in <module>         install_requires=parse_requirements(),       File "C:\Users\xxx\AppData\Local\Temp\pip-install-o8hgmgsc\investpy\setup.py", line 17, in parse_requirements         with io.open('requirements.txt', encoding='utf-8') as f:     FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'     ---------------------------------------- ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Installed with command
pip install https://github.com/alvarob96/investpy/archive/master.zip

Hi @kuhytitusdlz and @GMZ21931,
as you can see, investpy is just available on Python 3.5 or higher, so installing it with Python 2.7 or any other version rather than 3.5 or higher will error. Anyways I mentioned that 0.9.11 release installation does not work due to a setup.py configuration error which has been fixed in 0.9.12 release.
In order to install new investpy working version you can use:
python -m pip install investpy==0.9.12 --upgrade --user
Thank you!

by default tried install 0.9.11
Collecting investpy
  Using cached https://files.pythonhosted.org/packages/cf/e8/1378dbf23b3390296fe5eff5155317eb782b59093a15714cfa77ceb4f42b/investpy-0.9.11.tar.gz
    ERROR: Command errored out with exit status 1:
     command: 'c:\python\python37-32\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\admin\\AppData\\Local\\Temp\\pip-
install-2dllev50\\investpy\\setup.py'"'"'; __file__='"'"'C:\\Users\\admin\\AppData\\Local\\Temp\\pip-install-2dllev50\\investpy\\setup.py'"'"';f=getat
tr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'
))' egg_info --egg-base 'C:\Users\admin\AppData\Local\Temp\pip-install-2dllev50\investpy\pip-egg-info'
         cwd: C:\Users\admin\AppData\Local\Temp\pip-install-2dllev50\investpy\
    Complete output (7 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\admin\AppData\Local\Temp\pip-install-2dllev50\investpy\setup.py", line 35, in <module>
        install_requires=parse_requirements(),
      File "C:\Users\admin\AppData\Local\Temp\pip-install-2dllev50\investpy\setup.py", line 17, in parse_requirements
        with io.open('requirements.txt', encoding='utf-8') as f:
    FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

0.9.12 not abailable
c:\>python -m pip install investpy==0.9.12 --upgrade --user
ERROR: Could not find a version that satisfies the requirement investpy==0.9.12 (from versions: 0.3.6, 0.4, 0.5, 0.6, 0.7, 0.7.5, 0.7.6, 0.8, 0.8.1, 0
.8.2, 0.8.3, 0.8.4, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, 0.8.5, 0.8.6, 0.8.7, 0.8.8, 0.8.9, 0.9, 0.9.1, 0.
9.2, 0.9.3, 0.9.4, 0.9.5, 0.9.6, 0.9.7, 0.9.8, 0.9.9, 0.9.10, 0.9.11)
ERROR: No matching distribution found for investpy==0.9.12

Yes you are right I already checked it! I will update the new release to PyPI a few minutes. Sorry for that!

Hi @kuhytitusdlz!
0.9.12 investpy release is now available on PyPI at https://pypi.org/project/investpy/. So that now you can easily install it via: python -m pip install investpy==0.9.12
Thank you for reporting this issue! Make sure to star the repo if you found it useful. :star: :octocat:


Add --ErrorAction to standardize bypass flags

Another thought, what if we include a new parameter "--continueOnError" that when applied would continue the download of the main and dependencies even after a single dependency fails.

I'd propose a more generic -ErrorAction like PowerShell's implementation where WinGet would respect stop, continue, silentlyContinue, inquire, ignore and maybe even suspend. This would probably make the PowerShell cmdlet implementation and could also be useful in other areas too. It could replace the IgnoreUnavailable argument for the import flow, could be used to set the behavior of upgrade all, could be used to continue when depenency installation fails, and would be a great option as a generic parameter for scripting
Originally posted by @Trenly in https://github.com/microsoft/winget-cli/issues/2953#issuecomment-1593548891

I like this idea of having a parameter that can provide a deterministic experience for the user, that allows the user to determine what they want the experience to be.

Originally posted by @RDMacLachlan in https://github.com/microsoft/winget-cli/issues/2953#issuecomment-1593689824

If we are to replace existing parameters with new ErrorAction parameter, it will be a breaking change. And needs to go through all existing workflows to ensure the experience is accurate. As well as all Com and Powershell work. I would suggest making this a different work item not attached to this download work. For download, I think using --force or a specific flag is good for the purpose.
We also need to think about each different error case by case, some errors are hard stop (installer corrupt, etc), some errors (like dependency download failure for winget download) may let the workflow continue.

Originally posted by @yao-msft in https://github.com/microsoft/winget-cli/issues/2953#issuecomment-1593748567

My opinion would be that the --ErrorAction would only be for the soft-stop errors (ones which can be bypassed) and only in cases where continuing is non-destructive. Destructive errors that can be bypassed (e.g - overwriting pins, forcing an install without converting to upgrade) should still require --force

[Policy] Breaking-Change

I think this is a great to have feature, but should be moved out of the work for WinGet Download and into it's own feature recommendation.

My opinion would be that the --ErrorAction would only be for the soft-stop errors (ones which can be bypassed) and only in cases where continuing is non-destructive. Destructive errors that can be bypassed (e.g - overwriting pins, forcing an install without converting to upgrade) should still require --force

Agree with your thoughts on the --errorActions however the --force should enforce the actions to be performed.

This should also include work to have "settings" for the default behavior WinGet should use when a user has customized preferences.

My opinion would be that the --ErrorAction would only be for the soft-stop errors (ones which can be bypassed) and only in cases where continuing is non-destructive. Destructive errors that can be bypassed (e.g - overwriting pins, forcing an install without converting to upgrade) should still require --force

Agree with your thoughts on the --errorActions however the --force should enforce the actions to be performed.

Exactly what I was trying to say, but just didn’t word it right.


identityCenterPermissionSets - Add boundaryPolicy as a property

Is your feature request related to a problem? Please describe.
The LZA automation does not error when a boundaryPolicy property is specified for PermissionSets.  However, the boundary policy is not enabled on the roles created for the PermissionSet.
Describe the feature you'd like
Enable the boundaryPolicy property to apply to PermissionSets and be attached to roles supporting SSO.
Additional context
Specifying a boundary policy is a best practice for IAM and is supported by CloudFormation.
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-sso-permissionset.html

Hello, and thank you for filing an issue with the Landing Zone Accelerator team! I can confirm that this feature is on our release roadmap and will be available in our next minor release. This is a duplicate request, so I will be closing this out. You may follow #231 for updates.
Thanks for your support of the LZA solution!


Fix compiler warning

The recent nightly compiler is generating a warning about these parenthesis.
Those warnings cause build-failures when trying to use trybuild to do UI tests for the #[proptest] macro added in PR https://github.com/AltSysrq/proptest/pull/188.

@Centril any chance we can get this merged?


Bundler uses MRI even when started with JRuby (Windows)

On Windows, with this line my Gemfile:
ruby '1.9.3', :engine => 'jruby', :engine_version => '1.7.18'

The following occurs if an MRI ruby is before jruby on my path:
> ruby -v
ruby 2.1.5p273 (2014-11-13 revision 48405) [i386-mingw32]

> jruby -v
jruby 1.7.18 (1.9.3p551) 2014-12-22 625381c on Java HotSpot(TM) Client VM 1.8.0_45-b14 +jit [Windows 7-x86]

> jruby -S gem install bundler -v 1.9.4
...

> jruby -S bundle install
...

> jruby -S bundle exec rake db:create db:migrate
io/console not supported; tty will not be manipulated
Your Ruby engine is ruby, but your Gemfile specified jruby

If MRI is after JRuby on my path this problem does not occur (even though the JRuby installation does not have a ruby command).

It's up to you to have the correct Ruby version show up as the ruby executable -- the ruby directive in your Gemfile won't change the ruby version, but only validates it.

@segiddins why does it work when there is no ruby on my path at all?
Shouldn't the runtime that initialized the bundler process take precedence over the ruby on my path?

Ah I'm sorry, I misunderstood. Yes, the ruby that called the bundle command should be the one used. I'm guessing that might be a bug in bundle exec?

@segiddins cool. I'll submit a patch for this shortly. Thanks for taking a look!

It sounds like your rake command might have a shebang that points to MRI. bundle exec has no control whatsoever (and has no way to have control over) the ruby that will be run by the rake command. All it can do is set $RUBYOPT, and then run the rake command.

Oh I see, it's the rake on the path, not the ruby on the path. That makes it tricker.
So I've confirmed that I can fix this by running:
$ jruby -S bundle exec jruby -S rake db:create db:migrate

I will close this unless you have a suggestion on how to make this less verbose?

Run jruby -S bundle binstubs rake to create a bin/rake with a shebang that points to JRuby, and then just run bin/rake db:create.

@indirect @segiddins thanks!


fix: 8878 Google Sheet Data filter and Float issue

Google Sheet Data filter and Float data processing issue

Fix for Comma delimited Number with or without decimal places
Fix for Larger data introduced down the row grid
Fix for mixed Numeric DataType like Number and Float
Fix to ignore data of another DataType introduced down the line not Matching the Original DataType (first row datatype)

Fixes #8878 #7794 #9667
Type of change

Bug fix (non-breaking change which fixes an issue)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

Manually tested

Existing @Tests were checked to work fine

Added new @Tests to test new implementation

Test File : DataTypeStringUtilsTest.java all Tests

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

Ready for review, some issues in test cases - resolved now

@leotom2000 This seems to be the PR with the old implementation where the entire service as duplicated. Could you please verify and update the PR?

This issue is resolved by another PR
https://github.com/appsmithorg/appsmith/pull/11207
So closing this PR without any merges


Test autodiff with sensitivity maps

Test  autodiff  where the operators with sensitivity maps (different settings with multi-coils)

This is now fixed. Thank you @alineyyy for your updates.


Long loading time for export MP4 video on Mac

Hi,
Great tool and great features but after recording 1 minute video it takes a lot of time and resources to export it to MP4.

Thanks for taking a look!
Currently I'm working to replace MediaStream Recording API with native recording implementations depending on the OS to make mp4 the default format.

Sounds great, thank you for your support!


V1.2.4

Bug fixes to fundamental parameter calculations and inferences.

Will this release come out soon? @hover2pi


Update docker commit man page re: volumes

Update the man page for 'docker commit' to make explicit the fact that
'commit' does not save data in volumes.
Addresses comments in #7583
Signed-off-by: Lars Kellogg-Stedman<EMAIL_ADDRESS>

Thanks! I haven't checked yet if it's there already (reading on my phone), but could you update the online command line reference as well?
I'll have a proper look at the "Dockerfile recommendation" later, and check if the maintainers agree on adding that part (I think it's fine, but just to be sure)

That comment comes almost straight from the existing documentation.

That comment comes almost straight from the existing documentation (and is echoed in docs/reference/commandline/cli.md).

Thanks!
One small nit, apart from that LGTM!

@larsks :cry: this needs a rebase, could you rebase this PR? (the docs were moved from /docs/sources/.. to /docs/).
If you need help, please let us know!

I've rebased the change.  I note that due to the reorganization of docs/, you now have a dangling link in line 80 of docs/reference/commandline/run.md:
The `docker run` command can be used in combination with `docker commit` to
[*change the command that a container runs*](#commit-an-existing-container).

This should probably point to docs/reference/commandline/commit.md,
but I'm not sure exactly how that link should look (there didn't seem
to be any examples in the existing run.md document).

LGTM

@larsks for the contribution and the catch

LGTM
The failed build is not related and should be resolved by #14409. This only contains changes to the docs, so I'll merge

Thanks @larsks!

cherry-picked


⚠ SkipTheCommericals has degraded performance

In f006b0b, SkipTheCommericals ($STC) experienced degraded performance:

HTTP code: 200
Response time: 7561 ms

Resolved: SkipTheCommericals performance has improved in a57d51c.


How to extract ADF's origin(0,0,0) ?

How can I know the ADF's origin(0,0,0). I want to allign Unity's World origin with tango ADF's origin.
The metadata doesnot contain anything.

If you set the Delta Pose Controller or AR Pose Controller to Use the Area Description Pose, this will automatically happen. Note that since the Delta Pose Controller updates relative to where it starts, you'll want to make sure it is placed at 0,0,0 with no rotation.
Feel free to reply to this issue if you have other questions or close the issue if this works for you!

Seems like the issue is solved. Please reopen if this is still an issue for you.

Sorry for late reply. This did resolved the issue. Thanks a lot buddy.


No refresh notification pops up after promoting snapshot for one ADLS Gen2 blob

Storage Explorer Version: 1.14.1
Build: 20200710.5
Branch: hotfix/1.14.1
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ macOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Go to 'Settings' -> 'Data Explorers' -> Uncheck the setting 'Auto-refresh on change'.
Go to 'Settings' -> 'Services' -> 'Storage Accounts' -> Enable the setting 'Enable ADLS Gen2 snapshots (preview)' -> Restart Storage Explorer.
Expand one ADLS Gen2 storage account -> Blob Containers.
Create one blob container -> Upload one blob to it -> Create one snapshot for the blob.
Switch to snapshots view -> Click 'Promote Snapshot' -> Check the result.

Expect Experience:
Pop up a refresh notification.

Actual Experience:
No refresh notification pops up.
More Info:

It also needs to manually refresh under snapshots view after promoting snapshot with 'Auto-refresh on change' enabled.

For craig to investigate when he gets back. Make sure that things are consistent, or that at least gen2 is respecting the auto refresh setting if possible.

The problem is the promote job handler passes an IBlobContainerRef argument with the refresh event, but the handler is expecting an IBlobContainerConnection. So, the isMatchingContext function doesn't return true, and the editor refresh doesn't occur.

Hi @craxal Verified this issue on build  master/20200812.2. Reproduce. So we are going to re-open it.

@v-xianya The Cloud Explorer reference was not up to date for 20200812.2. Please verify again with a newer build.

Hi @craxal Thanks for the updates. Verified this issue on build master/20200813.4. Fixed.


iOS webview Unable to load

Every time the page is loaded, Xcode throws an exception and the page cannot load properly
Bug description:

log print
JSON value '{
    "_h" = 0;
    "_i" = 1;
    "_j" = 1;
    "_k" = "<null>";
    catch = "<null>";
    finally = "<null>";
    then = "<null>";
}' of type NSMutableDictionary cannot be converted to BOOL

Environment:

OS: macOS 15.0.1
OS version: macOS 15.0.1
react-native version: 0.72.5
react-native-webview version: 13.12.3

The only phenomenon is this log and exception, I don't know where the specific error occurred

After deleting this attribute, it can work
this function
```
  _onShouldStartLoadWithRequest = async nativeEvent => {
    if (!nativeEvent) {
      return false;
    }
    return true;
  };
```

delete async is work


Cannot scroll in channels nor PRO windows

See title.  There is no option to scroll when the content extends passed the visible window.  Specifically in the Pro and the Channels windows.
Running on Linux Mint 18 with cinnamon

@twmr1980
This issue has been fixed a while ago. If it should reoccur, please drop us a note.
Cheers, S


Table footer page number is not resetting after setting the offset

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
Table footer page number is not resetting after setting the offset
Expected behavior
Table footer page number should change after setting the offset
Reproduction of the problem
I have implemented the server side paging and sorting in my application.
following is the sample markup of my table
<ngx-datatable
#lntable
class="material striped"
[rows]="rows"
[columnMode]="'flex'"
[headerHeight]="40"
[footerHeight]="50"
[rowHeight]="'auto'"
[externalPaging]="true"
[externalSorting]="true"
[count]="page.totalElements"
[offset]="page.pageNumber"
[limit]="page.size"
(page)="setPage($event)"
(sort)="onSort($event)">
<ngx-datatable-column name="Mgr" prop="Mgr" [flexGrow]="1">
<ngx-datatable-column name="Client" prop="Client" [flexGrow]="1">
<ngx-datatable-column name="Suppress" prop="Suppress" headerClass="ngx-align-center" [cellClass]="getCellClass" [flexGrow]="1">
<ngx-datatable-column name="Remarks" prop="Remarks" [flexGrow]="2">

I have imported the datatable component in my component class as follows
import { DatatableComponent } from<EMAIL_ADDRESS>@ViewChild('lntable') lntable: DatatableComponent;
If the user try to go to another page by clicking on the page number in footer.
I am asking a confirmation in the setPage method as follows
setPage(pageInfo){

	let userChoice = confirm('Do you want to continue?')
	if(userChoiceSort === false){
		this.lntable.offset = this.page.pageNumber;
		return false;
	}

    this.page.pageNumber = pageInfo.offset;
    this.service.getData(this.page).subscribe(pagedData => {
    this.page = pagedData.page;
    this.rows = pagedData.data;
    });

}
if user cancel the confirmation, I am setting the offset but on the UI, footer page number shows the clicked page as highlighted.
What is the motivation / use case for changing the behavior?
Table footer page number should reset after setting the offset
Please tell us about your environment:
Windows

Table version: 9.3.x

Angular version: 4.x.x

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript 2.2.x | ES6/7 | ES5]

I believe to be having this problem now in Angular 10.


Add contributors.md

git log --pretty="%an%n%cn" | sort | uniq will give something to start from

After discussion we think this is unnecessary. It is easy to see a list of contributors via GitHub and we now have the codemeta file.

I disagree - contributors are not just the people who made commits in GH.
And CodeMeta is not easy to read.


Enhancement Voting

There are a lot of requested enhancements to the app and some of them are rather large.
Therefore, I request that everyone add their reaction on the first comment of any enhancement you want to see completed. Then I'll try to work on the most requested ones first.

I didn't see a open issue for this, but it is possible (now) to create a single certificate for both addresses (domain.com and www.domain.com)?
As far I remember, until a few versions ago it was not possible.
If it still is not possible, my vote is for this feature.

@navossoc That's SAN Support, #3. That was added in v1.9.0.

This is called a subject alternate name (SAN) and is included in the newest version 1.9


Allow for specifying a minimum width when opening a dialog using the DialogOptions class

Feature request type
Enhance component
Component name
MudDialog
Is your feature request related to a problem?
In the application I'm working on, some dialogs we open have a very minimum amount of content in them. Unless we open the dialog and specify the parameter that we want it to open full width, then the dialog opens and tries to remain as small as possible.
I'd like to be able to specify a minimum width for the dialog in the same way I can set the MaxWidth.
Describe the solution you'd like
Add a MinWidth property to DialogOptions that uses the same enumeration as the MaxWidth, and have the modal inspect this option when opening so that it is at least as wide as the minimum width specified if it has been provided.
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

So what if the window is smaller than the dialog's minimum width?

Now sure I follow what window you're asking about.
Do you mean the viewport? This shouldn't matter if I understand how the enum works in that it is responsive based upon the viewport's size.
If you are speaking about the contents of the modal dialog itself, even today there can be display issues when you pass in dialog options where the FullWidth property is set to false.
Either way, I believe it would be helpful to be able to specify a minimum width and the maximum width so that the dialog can at least have a minimum width rather than relying on it to automatically calculate how wide it needs to be at a minimum.


Try to remove the base directory first and avoid recursive operations

Fixes #11701
Status
Ready
Description
During the investigation of the slowness of MSUnmerged, we have noticed it looses some extra time in recursive operations while traversing the tree bellow the base directory of a given entry in the /store/unmerged/ area. With the following commit we are adding an extra step during which we try to delete the base directory first, and only if this operation is not successful due to NonEmpty error or similar, only then we enter the tree bellow and start purging recursively.
I have manually tested this and it works for some of the sites, so even if it is not about to work in all the cases it will definitely reduce the amount of communication with the servers.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
None
External dependencies / deployment changes
No

Thanks @amaltaro
I can merge this and cut the tag but before I close the issue I'd like to test one more thing - accessing at least one site's /store/unmerged/ area  through a protocol different  than webdav and check if there is a difference in the performance.

Also please  take a note about this comment. These two issues are closely related and the effort for them both goes in parallel.

Hi @amaltaro , my latest (forced) commit was just a fix for the cases with missing ctx objects when we emulate gfal2. This was not really related to the use case addressed here, but was  about a fixing a broken unit test. I did not split it in a separate Unit test commit, because I consider it as a fix of an overlooked situation when adding the ctx.free() step in the  MSUnmerged class itself.
I am merging this PR now and will soon create the new WMCore tag: <IP_ADDRESS> so I can deploy the new version of MSUnmerged tomorrow morning.

thanks @amaltaro merging now.


Use GitHub actions, test on python 3.7-3.10, remove python 3.6 support, remove Travis and AppVeyor

todo:

[ ] make sure all is green
[ ] remove travis
[ ] remove appveyor
[ ] maybe move tests into the package
[ ] maybe pin test and docs dependencies

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c8ae6b1). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #103   +/-   ##
========================================
  Coverage          ?   0.00%
========================================
  Files             ?       8
  Lines             ?    1510
  Branches          ?       0
========================================
  Hits              ?       0
  Misses            ?    1510
  Partials          ?       0

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Throw an error if a template variable is missing

Current behavior is to simply write out the raw macro value:
vagrant@factotum:/vagrant$ target/debug/factotum ./vars.factotum
Task 'Say something' was started at 2016-06-13 16:05:51.901468438 UTC
Task 'Say something' stdout:
{{ message }}
Task 'Say something': succeeded after 0.0s
1/1 tasks run in 0.0s
vagrant@factotum:/vagrant$ target/debug/factotum ./vars.factotum -e '{ "message": "hello world" }'
Task 'Say something' was started at 2016-06-13 16:06:16.336410851 UTC
Task 'Say something' stdout:
hello world
Task 'Say something': succeeded after 0.0s
1/1 tasks run in 0.0s

I think failing would be a more obvious behavior, and we could document the Mustache function for converting a missing/undefined value in JSON to an empty string...

I looked into this, really I need to submit a PR for the rust-mustache crate. This is because the crate doesn't expose enough information for me to work out tokens that were not replaced, and my initial idea of just checking for {{.*}} in the resulting string won't work because mustache supports changing the delimiter (and things like how to escape {{ if you need it for some other reason).
It's for this reason that I'm going to move it back a release

Sounds fair...


Remove WMDataMining thread/code

Fixes #9701
Status
ready
Description
Remove the WMDataMining thread (and any reference to it), which hasn't been used for the last few years.
Is it backward compatible (if not, which system it affects?)
yes (thread has been disabled over the last few years)
Related PRs
none
External dependencies / deployment changes
Deployment changes: https://github.com/dmwm/deployment/pull/982

And we have been getting a harmless failure when pushing couchapps for central services deployment. It turns out I missed this fix: https://github.com/dmwm/deployment/pull/993
To be deployed tomorrow.


devtool/bin/spawn replacement in Node/DevTool? Looking for advice ...

I am working with this code that uses devtool/bin/spawn:
var headless = process.env.NODE_ENV === 'development' ? '' : '-hq';
var args = [ file, headless, '--bf', '-c', '-i', html, '--' ];
require('devtool/bin/spawn')(args.concat(process.argv.slice(2)));

Reading the README of this repo, it says:

This tool is mostly obsolete as much of the philosophy has been brought into Node/DevTool core, see here for details.

Just curious, is there something to replace devtool/bin/spawn in Node/DevTool? should I stop using this repo's bin/spawn?
I was even thinking about creating a module just for devtool/bin/spawn ... Then again, I don't know much about Node/DevTool ...
Mostly just looking for advice here. I really like how devtool spawns up Electron so I can run web and nodejs scripts via the command line. Optimally, I'd use similar code from Node/DevTool. I think I just need to be pointed in right direction by someone with more experience using these tools.
Thank you so much!

I actually see that devtool/bin/spawn links to server.js, so it's not a simple port of spawn to a new repo.
I looks like node inspect is where I should start looking.
Thanks for reading and sorry for the noise. :)


There is an error in the "__reformSequence" function

This function is used to convert a string into a SeqRecord object, not a Seq object. Because Seq objects do not have seq properties (in the getORFs function, you used forward = str(sequence.seq).upper()).

This code can solve the problem
def __reformSequence (sequence):

    if isinstance(sequence, str):

        sequence = SeqRecord(Seq(sequence), id="seqence")

    return sequence

Have you run any tests on the code? Could you send me the instances in which it crashes?


Missing Dependency: Microsoft.EntityFrameWorkCore (2.0.0)

Had to install it maually.

Hi @BrianGSC,
Normally I install something like Microsoft.EntityFrameworkCore.SqlServer, rather than Microsoft.EntityFrameWorkCore, as that is what I going to use. For instance the DataLayer just has Microsoft.EntityFrameworkCore.SqlServer installed. The database specific NuGet's then loads the other packages they need.
So, some projects might not have Microsoft.EntityFrameWorkCore installed but they still work. Does that make sense?

You are of course right. Maybe it is just a clean-up.
The Using Directive is referencing the Microsoft.EntityFrameWorkCore in severel clases.
Visual Studio registered it as an error -> build errors if not installed.
Sorry.
Regards Brian.

Hi @BrianGSC,
No problem at all. I hope you find the repo (and book) useful.


[Speech Recognition] Ensure it works when targeting API level 30

Requires <queries> tag in the manifest

Fixed - last week :-)


IKEA on/off plug should not be dimmable

My setup is a RaspBee board with the latest deConz software.
I have the new IKEA on/off socket plug. I can discover it in the Phoscon app, and it is visible in homebridge-hue as a light. However, it reports as being dimmable. It doesn't seem to have an effect, but it looks a bit strange.
I also have an OSRAM on/off socket plug and that one is not reported as dimmable.
Can I override this somewhere?
$ ph get /lights
{
  "etag": "xx",
  "hascolor": false,
  "manufacturername": "OSRAM",
  "modelid": "Plug 01",
  "name": "Schakel Osram",
  "state": {
    "alert": "none",
    "on": true,
    "reachable": true
  },
  "swversion": "V1.04.12",
  "type": "On/Off plug-in unit",
  "uniqueid": "xx"
},
{
  "etag": "xx",
  "hascolor": false,
  "manufacturername": "IKEA of Sweden",
  "modelid": "TRADFRI control outlet",
  "name": "Schakel Ikea",
  "state": {
    "alert": "none",
    "bri": 46,
    "on": true,
    "reachable": true
  },
  "swversion": "1.4.020",
  "type": "On/Off plug-in unit",
  "uniqueid": "xx"
}
...

deCONz shouldn’t expose state.bri if it’s not dimmable.  It probably does so, because the plug exposes a Level Control cluster (probably so it’ll react to Move to Level with On/Off sent by the Ikea switches).
If you expose it as an Outlet instead, it won’t be dimmable.  See
https://github.com/ebaauw/homebridge-hue/releases/tag/v0.10.0

I just noticed that deCONz also exposes sensors for those two plugs, and I did not yet have "sensors":true in config.json yet. I added that and now I see them both exposed as power outlets in homekit. Before they were exposed as lights.


Add FilesDumperPlugin

Introduce a new standard dumper which allows developers to conveniently
extract application internal files.  Sample usage transcript:
https://gist.github.com/8d3280e16acbf12a59a5

Review timeout, landing :)


Can I keep a client connection open beyond one function call?

WebSockets.open takes a function/do block and closes the connection automatically afterwards. Is there currently any way to open a connection, return the WebSocket for later manipulation, and have the ability to close the connection manually when we're done?
I know I could pass in a function which just loops forever but it's not a great solution for what I want to do.

Yes, there is. But the websocket can always be closed by the program on the
other side anyway.
The websocket closes when you exit that function. If you never exit it,
every call will fill up memory.
There is currently a possible problem in Julia 0.7 or 1.0 with responding
in a parallel worker process and passing the reference back. But Julia 0.6
benchmarks do that.
Den tor. 13. sep. 2018, 07:47 skrev Chris de Graaf<EMAIL_ADDRESS>
:

WebSockets.open takes a function/do block and closes the connection
automatically afterwards. Is there currently any way to open a connection,
return the WebSocket for later manipulation, and have the ability to
close the connection manually when we're done?
I know I could pass in a function which just loops forever but it's not a
great solution for what I want to do.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaWeb/WebSockets.jl/issues/113, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANTatP7ZZr6SvOq4Y8TOFdRuCi7ixaQSks5uafGAgaJpZM4WmnIs
.

The examples share server side sockets with other coroutines through mutable collections.
You can do the same thing with client side sockets. I can just not see the use for it. Can you give an example?

Come to think about it, I think it is very reasonable to include a function simply returning an open websocket or throw an error. This is analogous to the way you can open a file, and it makes throwing together a script step by step easier, allowing for inspection on the way. Thanks for the input!

As for my use case, I'm trying to wrap the Discord API. But I did find a middle ground with processing the connection in a coroutine, so this feature request is not so urgent anymore. I do still agree that it would be useful though :slightly_smiling_face:.

I wrote a module OpenTrick to make interactive playing with WebSocket server easier.
using OpenTrick
using WebSockets

io = opentrick(WebSockets.open, "ws://echo.websocket.org");
write(io, "Hello");
println(String(read(io)));

close(io)  # you can close io manually
wrapper = nothing; # or leave it to GC
unsafe_clear() # or you can clear all ios opened by opentrick manually

Awesome, I'll probably give it a try :slightly_smiling_face:

I am closing this issue, and would also recommend a look at this alternative approach:
inbox_outbox.jl


refactor: update to use Protobom Ent backend

Update to use Protobom Ent backend
Description
Update to use the protobom Ent storage backend for interacting with bomctl database.
Refactor patterns for reading/writing files

rename fetch.Exec() to fetch.Fetch()
update Fetcher interface Fetch method definition to return []byte instead of a parsed document; parsing the bytes is done afterwards from within the main fetch.Fetch function
pass file arguments as stream readers/writers instead of strings

Miscellaneous

add go test workflow
reorganize Makefile, add test-unit target
small fix for pkg/fetch/http.ParseURL() not returning nil when it should

Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules
[X] I have checked my code and corrected any misspellings

Going to break this into follow-on issues and separate PRs to keep these changes small.

Putting back into draft until @puerco is able to merge https://github.com/protobom/storage/pull/12 and cut a release so that this repo's go.mod can be updated


When confirming, return the delivery tag from Publish.

With multiple goroutines publishing messages simultaneously, being able to identify which Publish resulted in which delivery tag is important for processing notifications later. If there's another way to do this, I wasn't able to find it.

:+1:
I also had the same problem, so I had to do my own locking around the channel, but the thing I disliked most was that I had to mimic exactly the same "+1" logic that that already exists in the library. If, for some reason, that changed, or if something didn't get incremented the way I try to mimic, then all will start exploding.
The only I problem I see with this commit is that it will break semver compatibility with existing applications using this package, but I am totally positive about the need for this.

Watch out for basic.return, which doesn't have a delivery tag, so you can't line them up with ack or publish messages by anything other than channel and receive order as far as I can tell.

@foolswood regarding basic.return, I am setting the mandatory flag on the message, and adding a delivery_tag with the corresponding value to the headers of the message. Then, the only thing I am serializing, is the publishing of the message. When I receive a basic.return, I check the delivery tag in the headers, and notify the sender about it. Similarly, when the routing succeeds and I get a confirmation, I also notify the sender. Confirmation and returns are handled in parallel. I think this is safe, but it would be better if we could be told by the package what delivery tag is going to be used before it is used. It would be even better if we could control the delivery tags. That way, we could just increment the delivery tag ourselves in a synchronous way, and then handle all publishing, confirmation and returns in parallel.

@imkira I can't see a delivery tag in the amqp frame headers, or the arguments for a return. I've tried reading the spec and looking at the packet raw in wireshark.
My understanding of what a return frame contains is below, hopefully you can show me what I've missed:
Headers:
       +-------+---------+--------+
Octet: |   0   |   1:3   |   3:7  |
Field: | Type  | Channel | Length |
Type:  | octet |  short  |  long  |
       +-------+---------+--------+

Payload:
       +-------+--------+------------+------------+----------+-------------+
Field: | Class | Method | Reply-Code | Reply-Text | Exchange | Routing-Key |
Type:  | short | short  |   short    |  shortstr  | shortstr |  shortstr   |
       +-------+--------+------------+------------+----------+-------------+
         (60)     (50)

Frame end (octet)

Thanks.

@foolswood It is not. I deliberately put the delivery tag (by serializing all publishings to guess the delivery tag), and then building a message with a custom header.
something like
  if msg.Headers == nil {
    msg.Headers = amqp.Table{}
  }
  msg.Headers["delivery_tag"] = int64(deliveryTag)

Like I said, since streadway/amqp does not allow you to get a delivery tag before publishing anything, I am serializing all accesses to this channel, so that I can predict the next delivery tag before calling channel.Publish. Then I just put that custom header in the message. When a message gets returned, I know which delivery tag it was when because the whole message is sent back. Because delivery tags are reset to 0 when the channel is closed, I am forcefully timing out all "pending" deliveries so the caller knows something failed.

Fair, I had seen the assumption being made in other libraries and was trying to make sure that nobody else fell into the trap.

I'm afraid this is a no-go, at least at the moment: it assumes that publisher confirms are enabled and breaks the API.


navigating to link is broken

https://github.com/excalidraw/excalidraw/assets/14358394/2c3cadd2-cb01-4109-ab40-93ba23bf7a2c

Looks like this has recently regressed in https://github.com/excalidraw/excalidraw/pull/7804/files
cc @ryan-di @dwelle

2024-04-06.09-26-20.mp4

I also noticed that if I right click to change the link, it opens the new tab with "about:blank" and I can't change it.

I think it's fixed now


adding correct policy relation to role so the role can grab secrets described in the policy

Hey, I was having issues getting secrets from roles I was creating and fixed it by doing this. Please let me know if there is anything that I can do to help with this!

@taylorKonigsmark Thanks for your PR!
This is a pretty good idea! Can you also update the README.md in the quick start (I would suggest another bullet point here: https://github.com/Boostport/kubernetes-vault/tree/master/deployments/quick-start#4-confirm-that-each-pod-of-the-sample-app-received-a-vault-token) to suggest the user to use the token to try to retrieve some secrets?

Thanks!


finalisers -> finalizers

What this PR does / why we need it: corrects misspelling in RBAC rules
Which issue this PR fixes: fixes #4346
Release note:
NONE

/kind bug

/cherry-pick release-1.5

/lgtm


warp-ctc supports multiple GPU?

Does warp-ctc support multiple GPU?
I have built a model with 8 towers each resides in a different GPU using Tensorflow.
Each tower calculates CTC loss for 1/8 split of the input batch.
But the model doesn't work. It complains there is not a kernel for CTCLoss on GPU1, but doesn't complain on GPU0.
Does it mean that CTCLoss of Tensorflow doesn't support multiple GPU?
And does warp-ctc support multiple GPU?

Cannot assign a device to node 'ctc/CTCLoss': Could not satisfy explicit device specification '/device:GPU:0' because no supported kernel for GPU devices is available.
It seems Tensorflow CTCLoss doesn't support GPU at all.