stacking cassette contexts ?

Hi,
I have been using vcrpy for tests for a little while, and recently I had an attempt at stacking cassette contexts. Something like :
import asyncio
import aiohttp
import vcr
import logging

logging.basicConfig()  # you need to initialize logging, otherwise you will not see anything from vcrpy
vcr_log = logging.getLogger("vcr")
vcr_log.setLevel(logging.INFO)

async def httpbin_get():
    async with aiohttp.ClientSession() as session:
        with vcr.use_cassette('get_outer.yml', record_mode='all'):  # want to record any traffic, for error debugging purposes...
            with vcr.use_cassette('get_inner.yml', record_mode='once'):  # recorded interraction once. replay during tests.
                async with session.get('http://httpbin.org/get') as resp:
                    print(resp.status)
                    print(await resp.text())

loop = asyncio.get_event_loop()
loop.run_until_complete(httpbin_get())

This gave me :
 python k7test.py
INFO:vcr.cassette:<function VCR._build_before_record_request.<locals>.before_record_request at 0x7fa666dc3d08>
INFO:vcr.cassette:<function VCR._build_before_record_request.<locals>.before_record_request at 0x7fa666ddfb70>
INFO:vcr.cassette:Appending request <Request (GET) http://httpbin.org/get> and response {'body': {'string': b'{\n  "args": {}, \n  "headers": {\n    "Accept": "*/*", \n    "Accept-Encoding": "gzip, deflate", \n    "Host": "httpbin.org", \n    "User-Agent": "Python/3.7 aiohttp/4.0.0a1"\n  }, \n  "origin": "<IP_ADDRESS>, <IP_ADDRESS>", \n  "url": "https://httpbin.org/get"\n}\n'}, 'headers': {'Access-Control-Allow-Credentials': 'true', 'Access-Control-Allow-Origin': '*', 'Connection': 'keep-alive', 'Content-Encoding': 'gzip', 'Content-Length': '185', 'Content-Type': 'application/json', 'Date': 'Thu, 31 Oct 2019 09:33:32 GMT', 'Referrer-Policy': 'no-referrer-when-downgrade', 'Server': 'nginx', 'X-Content-Type-Options': 'nosniff', 'X-Frame-Options': 'DENY', 'X-XSS-Protection': '1; mode=block'}, 'status': {'code': 200, 'message': 'OK'}, 'url': 'http://httpbin.org/get'}
200
{
  "args": {},
  "headers": {
    "Accept": "*/*",
    "Accept-Encoding": "gzip, deflate",
    "Host": "httpbin.org",
    "User-Agent": "Python/3.7 aiohttp/4.0.0a1"
  },
  "origin": "<IP_ADDRESS>, <IP_ADDRESS>",
  "url": "https://httpbin.org/get"
}

along with a cassette yaml : get_inner.yml
What happened to the outer one ?
I would assume that, for subsequent run, the inner_get will get replayed, but I ll never get the record of it in the outer ?
Is that behaviour expected?
I am using vcr for tests (via https://github.com/kiwicom/pytest-recording), to record traffic and replay it as I am testing in CI.
And I am planning to use vcr as well, in code, to record any traffic from my HTTP client, in case an error happens, so that a user can easily access the data received just before the crash (a likely culprit for that crash).
Currently these two implementations currently interfere in my tests as my simple example above demonstrate.
But maybe I am holding this the wrong way, and there is another way to implement a "double recording" feature properly ?
Thanks for any hint.

@asmodehn thanks for the question.
I'll qualify that I've only recently started volunteering time on maintenance on this library since August 2019.
That behavior makes sense with what you are seeing and from what I understand of the code.
When you start a with vcr.use_cassette context manager vcr attempts to import the 6 supported  HTTP libraries and then patch them with the VCR implementations. Ending the context manager unpatches them.
This patching applies in a global sort of way so that it catches any deeply nested calls inside functions that are trying to hit the network so it won't let any requests pass through to the outer context since the inner patch is the most recent.
I see what you are trying to do and it is an interesting use case. Just like having separate loggers to separate out streams of log records, you are trying to separate streams of requests.
Have you seen this part of the documentation?
https://vcrpy.readthedocs.io/en/latest/advanced.html#ignore-requests
I'm not sure if it would work but it allows certain requests to ignore the patched version of aiohttp then fall through to the underlying aiohttp.
This might mean ignored inner requests fall through to the outer patched aiohttp instead.
VCR allows for custom patchers, matchers and persisters too. Where the documentation is lacking, sadly the next step is to read the source code. I'm happy to try and interpret parts of the code if you post links to any of it that seems obscure (and lot of it is).
I hope that helps. But I think you are running up against the bounds of the intended use cases of VCRpy and you might want to consider forking it as a starting point for patching and modifying it to better suit your needs.

Thanks a lot for the explanation @neozenith ... If the patches are global, then the behaviour I m experiencing makes total sense.
I guess I ll have to find out a way to implement both of the features I am interested in, in a way that work around that limitation... It seems I cannot have the double recording, but maybe I can change my requirements a little to have some kind of "selective" recording depending on the usecase...
I ll dive into the source code to see what I can find.
I'll post here if I manage to get something working...

Thanks. I'll close this issue for now as we have addressed the behaviour is as expected.
But please comment back here for follow up as a form of documentation.


Improve error feedback for build cache errors

Stage 4 build: https://builds.gradle.org/viewQueued.html?itemId=5422969

This PR should fix #2434.

Tests still failing:

CachedCustomTaskExecutionIntegrationTest.reports useful error when output dir is expected but file is produced
CachedCustomTaskExecutionIntegrationTest.reports useful error when output file is expected but dir is produce

Tasks declaring to create an @OutputDirectory should not be allowed to create an @OutputFile regardless of caching. The check should eventually be moved to task property validation, and fail when task output caching is disabled, too.
For 4.1, though, I think it's fine to not throw an error about this, and add a card to do proper validation later instead.

HttpBuildCacheServiceIntegrationTest.incorrect credentials cause build to fail

HTTP 401 is not a failure anymore, tests needs to be fixed/removed.

BuildCacheBuildOperationsIntegrationTest.records load failure[0]
BuildCacheBuildOperationsIntegrationTest.records load failure[2]
BuildCacheBuildOperationsIntegrationTest.records unpack failure

Test needs to be updated, as we are not failing the build anymore in these cases.
We seem to be missing a records pack failure case as well.

CacheTaskArchiveErrorIntegrationTest.corrupt archive loaded from remote cache is not copied into local cache

Looks like the .part file is not closed properly (Windows).

Stage 4 build: https://builds.gradle.org/viewQueued.html?itemId=5506237


Process PNG image data with sharp

I see that Sharp takes input as an actual file but I'm wondering if it can take for example PNG base64 image data as input? If so how would that be done? Or otherwise how can canvas.toDataURL be input into Sharp?
I did search but couldn't find this answer.
Thanks.

https://stackoverflow.com/questions/5867534/how-to-save-canvas-data-to-file/5971674#5971674
// strip off the data: url prefix to get just the base64-encoded bytes
var data = img.replace(/^data:image\/\w+;base64,/, "");

(GitHub's ability to handle timezones is totally broken right now - these comments are appearing out of chronological order.)

I tried this test = test.replace(/^data:image\/\w+;base64,/, ""); as you suggested and it works now...but it still throws the deprecated error?
DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.

Weird, your comment and my comment seem to have disappeared and all the time indexes on the previous comments are in the future..is it a problem my end or does github have a problem suddenly? I already replied to the comment that disappeared but my comment didn't appear either.

https://twitter.com/githubstatus/status/1126171459255185414

Weird so there really was a problem. Anyway it seems to be ok now. So the code works now but still throws the buffer deprecated error as in my comment above - any ideas?

sharp(Buffer.from(base64EncodedString, 'base64'))...

Ok that fixed it. That was what I tried previously from the thread you suggested at the beginning but didn't try it after removing the url prefix from the canvas data. Thanks for your help and btw Sharp is just awesome. :)


Resolve extra margin on code element (class: 'my-2')

Description
Issue number: #966

Resolves the extra margin issue on the "< code >" element with the class "my-2".

Checklist

[x] before https://files.slack.com/files-pri/T0239KTK4MP-F05DUG6PDFW/image.png
[x] After https://files.slack.com/files-pri/T0239KTK4MP-F05DKDC7ZQE/image.png

Hey @lorinaX, thanks for submitting this!
This looks correct, but could you attach a screenshot of the updated code component? That helps us review UI changes more quickly. Thank you!

would this do the trick?

Yep, that works!
I went ahead and ran the tests, if they pass we should be good to go

Yep, that works!
I went ahead and ran the tests, if they pass we should be good to go

Uh oh I think that when I ran pnpm install it modified the pnpm-lock file?


Fix taskwarrior completion

Fixes #3796

Actually as I said later in https://github.com/robbyrussell/oh-my-zsh/issues/3796#issuecomment-503275743, the compdef line doesn't hurt and what really fixed it was resetting the completion cache (~/.zcompdump*), see #7642.


edit pass: five Java on Azure articles

@Brendanmitchell @karlerickson  Please review the copy edits to these five articles, address the comments below, and let me know if I've changed the technical meaning anywhere. This is for task 1553556.
Thanks. — Jim
Comments:

/jdk/index.md

The corresponding TOC entry for this article should be sentence-cased as "Java long-term support"—could you fix this in a new PR after this one is merged?

/jdk/java-jdk-flight-recorder-and-mission-control

line 24, I added an H2 heading to give the on-page TOC at least two H2 headings, per guidelines.
line 111, for image azul-jfr-3.png, to align to style for images, could you remove the drop shadow?—can be fixed in a new PR after this one is merged.

/microprofile/deploy-microprofile-to-container-instances

line 146, the "Next steps" intro sentence promises links to more information about "various technologies," but the single link is "Sign in to Azure from the Azure CLI"—could you clarify what you'd like to link to?

/microprofile/configure-microprofile-with-keyvault

line 45, the sentence beginning "It doesn't demonstrate how to Dockerize ...." promises a section at the end of the tutorial with links to additional information, but there's no such section. Do you want to add links, or should we remove this sentence from the paragraph?
Throughout: I tried to distinguish the Azure Key Vault service from local key vault instances. Could you double-check to ensure that I've done this accurately?

Docs Build status updates of commit 7e7d3a4:
:clock10: Full build: the average full build time is 71 min(s) 32 sec(s), this is based on the last 281 build(s) for this repository.

Docs Build status updates of commit 7e7d3a4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

Docs Build status updates of commit 723df80:
:clock10: Full build: the average full build time is 71 min(s) 33 sec(s), this is based on the last 282 build(s) for this repository.

Docs Build status updates of commit 723df80:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

Docs Build status updates of commit a1200fc:
:clock10: Full build: the average full build time is 71 min(s) 35 sec(s), this is based on the last 283 build(s) for this repository.

Docs Build status updates of commit a1200fc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

@BrendanMitchell @KarlErickson  The review period for this edit pass is closing now. Please address my original comments (at top of PR), and provide feedback before the end of day Wednesday 6/26. Let me know if you still need more time.
Thanks!
— Jim

Hello,
I'm not sure how my name or GitHub account got attached to this but I'm am
not part of the dev team for Azure and have never worked on it.
Hope this helps,
Brendan
On Tue, Jun 25, 2019 at 12:53 PM Jim Wells<EMAIL_ADDRESS>wrote:

@BrendanMitchell https://github.com/BrendanMitchell @KarlErickson
https://github.com/KarlErickson The review period for this edit pass is
closing now. Please address my original comments (at top of PR), and
provide feedback before the end of day Wednesday 6/26. Let me know if you
still need more time.
Thanks!
— Jim
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-docs-sdk-java/pull/872?email_source=notifications&email_token=AF3VHK7NCE7PNCVJ352H3TLP4JZTXA5CNFSM4HY3EW72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYRM5AY#issuecomment-505597571,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF3VHKY5BDSTZYWXF2LVOLDP4JZTXANCNFSM4HY3EW7Q
.

Per @bmitchell287 in work item:
@Jim Wells @Karthik Sangiah @Doug Erickson @Karl Erickson - Hi Jim as far as the general edits go I've taken a pass and it looks like the majority of it was minor editorial, and I did not see any glaring issues. Appreciated. I've also made the PM @Asir Vedamuthu Selvasingh aware of this update and told him you would probably merge these changes this week. I would proceed with that.
Asir, Jim had a couple open questions to us per the pull request that I'll run by you in the next fundamentals meeting (sections that promise links to additional content but don't appear to). We'll review, and if any changes are needed, I'll submit the PR to fix.
Thank you Jim and team for the updates here.

@bmitchell287  Brendan, just to be clear, are you saying go ahead and merge this PR as is, and that your team will address my several original comments in a future PR? Thanks! —Jim

Docs Build status updates of commit 4922977:
:clock10: Full build: the average full build time is 71 min(s) 41 sec(s), this is based on the last 292 build(s) for this repository.

Several small tweaks for clarity, accuracy, and grammatical correctness.

Docs Build status updates of commit 4922977:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

Docs Build status updates of commit 13d9c67:
:clock10: Full build: the average full build time is 71 min(s) 42 sec(s), this is based on the last 294 build(s) for this repository.

Docs Build status updates of commit 13d9c67:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

@itechedit - Yes we'll evaluate your opening feedback, discuss where to go from there on those particular issues. Thanks.

#sign-off

Docs Build status updates of commit 1bb2bb1:
:clock10: Full build: the average full build time is 71 min(s) 43 sec(s), this is based on the last 295 build(s) for this repository.

#sign-off after minor fix

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

File
Status
Preview URL
Details

docs-ref-conceptual/jdk/index.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/jdk/java-jdk-flight-recorder-and-mission-control.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/configure-microprofile-with-keyvault.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-container-instances.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

docs-ref-conceptual/microprofile/deploy-microprofile-to-web-app-for-containers.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

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

@rloutlaw  Robert, I understand you're the person to contact to merge PRs in the azure-docs-sdk-java repo. Our team doesn't have perms. This PR is approved, signed off, and ready to go. Please merge it.
Thanks!
— Jim

@rloutlaw Appreciate it!
@ShawnJackson


“Missing” constants: electroncharge, electronmass, protonmass

[ priority=low ]
The manual says:

Many constants are available, including:

[⋮]
electroncharge, electronmass, protonmass, etc.

However, fend v1.4.7 knows none of those.
Fend does know electron_charge, but none of the two masses (with or without underscore).
If the two masses make it into fend, please also add neutron_mass.

Thanks for fend, it's very useful. It's like units(1) on steroids.

I've added electron mass, proton mass and neutron mass now. If there's any other units/constants you'd like feel free to let me know, or make a PR!

Whauw, that was fast. Colour me impressed. :+1:


[Nuxt / SSR Support] window is not defined

Describe the bug
Thank you for the amazing work. It is the best-looking and featured JS calendar I could find and very grateful for that!
I'm using Nuxt for front-end development and I can't find a solution to make it work properly. I used https://github.com/nhn/toast-ui.vue-calendar wrapper to try make it work.
However, since Nuxt uses SSR, it will always give window is not defined. I tried different solutions but couldn't find any. Probably, the only one to work would be to import it as a plugin, but doing that way it will bring 503.6K (gzipped: 102.7K) to the main bundle, and that it's a lot.
To Reproduce
Here is a minimal reproduction of Nuxt and tui.calendar
https://codesandbox.io/s/nuxt-tui-calendar-2b37r
Thank you very much! 🙂
Edit: SweetAlert2 solved this issue by doing the following https://github.com/sweetalert2/sweetalert2/pull/745/commits/652b6b742eb4c3cec10bda78fd91ac27ebf19442, https://github.com/sweetalert2/sweetalert2/commit/eb3806d0ded6957d625ab3afe44f6ff8da4c43c0

@Tragio
Hi. Lack of SSR support is not a bug, So I changed the label.
Next.js offers dynamic module loading which supports client-only modules.
But It looks like Nuxt doesn't support external modules like this.
We're going to support SSR in the next major version, However, It's going to be the next year. But Thanks for pointing this out.

@Tragio Hi. Lack of SSR support is not a bug, So I changed the label. Next.js offers dynamic module loading which supports client-only modules. But It looks like Nuxt doesn't support external modules like this.
We're going to support SSR in the next major version, However, It's going to be the next year. But Thanks for pointing this out.

Thank you very much. Do you have a more precise ETA for it?

I'll leave here the solution for anyone using Nuxt.
Create a file on plugins folder with .client.js (ex. tui-calendar.client.js) and write the following:
import Vue from 'vue'
import { Calendar } from '@toast-ui/vue-calendar'

Vue.component('Calendar', Calendar)

@Tragio
We have a goal to release the next major update by the end of next March.
But Things can be changed.

Thank you @Tragio I was looking for this and found this issue by stumbling upon your codesandbox instance while google search I think just by adding something like below in documentation or install instructions can work for now and can avoid confusion.
Nuxt
Create a file on plugins folder with .client.js (ex. tui-calendar.client.js) and write the following:
import Vue from 'vue'
import { Calendar } from '@toast-ui/vue-calendar'

Vue.component('Calendar', Calendar)

Thank you @Tragio I was looking for this and found this issue by stumbling upon your codesandbox instance while google search I think just by adding something like below in documentation or install instructions can work for now and can avoid confusion.
Nuxt
Create a file on plugins folder with .client.js (ex. tui-calendar.client.js) and write the following:

import Vue from 'vue'

import { Calendar } from '@toast-ui/vue-calendar'

Vue.component('Calendar', Calendar)

Hi @arafays 😄 yes you can do that way and it will work, but using that approach it will increase a lot the main bundle since you're importing to the whole app instead locally when you need it.


Update target user profile, value proposition, and user stories

Codecov Report

Merging #22 (2023041) into master (8c45ae0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #22   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 8c45ae0...2023041. Read the comment docs.


[ci skip] adding user @FaustinCarter

Hi! This is the friendly automated conda-forge-webservice.
I've added user @FaustinCarter as instructed in #16.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #16

@conda-forge/core Would someone mind merging this in? The previous maintainers appear to have given up. Happy to pick up where they left off.


Zoomit draw does not appear when share screen with MS Teams

HI, i hope s.o. can help me with this issue.
I installed zoomit on windows10 Lenovo Thinkpad T420. It worked fine until I started an MS Teams and I was told that draw does not show on the desktops of the participants. Zoom function worked, but not the draw. I later reconfirmed that testing internally.
I was sharing a chrome tab with youtube video.
Then, to test I installed zoomit on another PC and hosted a test MS Teams meeting, sharing screen with Thinkpad T420, which was this time a participant. This time zoomit draw appeared on the participant screen.
I was googling a lot find similar issue shared, but no success....

Hi, please direct your support query towards the Microsoft Q&A forum (the Sysinternals forums have recently transitioned there). It's the best source for feedback with regard to the tools or for community support.

HI, thank you. Just for reference, i opened the same on Microsoft Q&A and i got similar answer,


Ailispaw/barge update issue

I just updated the box to 2.5.4 and wocker seems to have stopped working. Vagrant is up, wp cli works, but opening wocker.dev (or pinging it) just loads perpetually and sometimes times out after a minute or two.
It's quite confusing. I downgraded back to 2.5.3 but it persists. I've cleared dns and disabled firewall so far without change.
I then made a fresh wocker install in a different directory and even that didn't work.
Is anyone else getting this, or has any idea what might be going on?

Hi @VesterDe ,
Thank you for letting us know the issue.
Could you tell me your OS, its version, and the version of Vagrant and VirtualBox you use?

Arch Linux, 4.11.9-1-ARCH kernel
Vagrant 1.9.7
VirtualBox 5.1.24r117012
It may just be something I did with my system in the last few days... I'll update this if I manage to fix it, any hints are appreciated.

What is the line of wocker.dev in the /etc/hosts during and after vagrant up?

<IP_ADDRESS>  wocker.dev  # VAGRANT:<PHONE_NUMBER>ddb7158ede8f207f9df701 (wocker) / 65f3a83e-a8ff-4804-b48e-baf75a180f58

Hmm. It looks good.
Can you ping to <IP_ADDRESS> and/or wocker.dev?

I agree yes.
No response from either the IP or the URL. However, if I ssh into vagrant, they do work.
wocker ~ $ ping wocker.dev PING wocker.dev (<IP_ADDRESS>): 56 data bytes 64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.283 ms

VirtualBox 5.1.24 released yesterday, I guess it may be the cause.
How is the output of ifconfig in the wocker VM?

Could you try to downgrade virtualbox and/or vagrant?

I'll try tomorrow (out of office in a bit), thanks for the help so far.

The same problem with me, after virtualbox update to version 5.1.24.
Update for Vagrant was done earlier.

Yep, can confirm, downgraded virtualbox to 5.1.24 and it fixed the problem.

Thank you for the confirmation, guys.
Also I found same issues on an other repo and its solution.
Vagrant up fails with VirtualBox 5.1.24 · Issue #8793 · mitchellh/vagrant
https://github.com/mitchellh/vagrant/issues/8793
[SOLVED] Can not ping virtualbox box after latest update (5.1.24-1) / Networking, Server, and Protection / Arch Linux Forums
https://bbs.archlinux.org/viewtopic.php?id=228365

FYI:
Mac OS X 10.10.5 and VirtualBox 5.1.24 work fine for me.
Hi @lakyller,
Is your host OS Arch Linux?

No my is: Ubuntu 16.04.2 LTS
But after I back to VirtualBox 5.1.22 everything works fine, again.

@lakyller Thanks.

This is presumably fixed in yesterdays release of 5.1.26, if someone upgrades and checks, we can close the issue.

Yeah, wocker works again with Virtuablx 5.1.26 for me. Closing this.

Changelog – Oracle VM VirtualBox
https://www.virtualbox.org/wiki/Changelog#v26

Linux hosts: properly bring up host-only network interfaces with iproute (5.1.24 regression; bug #16911)


WCL 6.0 with EWM 7.0.2 hanging to create new workitems

Hello,
I have a script to create or update workitems in EWM, which is using the WCL.
After the upgrade to EWM 7.0.2 SR1 I needed to update to WCL 6.0, which is working for update.
But when I try to create new workitems it is hanging forever.
(Before the upgrade the creation was also working fine)
Do you have any idea to solve the problem?

Version 6.0.1 works for me. With the information given, there is no way of telling what might be wrong.

Ok, thanks. The version 6.0.1 is the same as the "WCL-V6.0-20220913"?
The script was working fine with the previous version, for both Update and create workitems.
Do you know if there is any change of parameters for instance, previously I was using the wcl 4.9 if I am not wrong.
Apart from some differences in import file, which I didn't change, this is the command for the wcl.bat, for update and create/import:
cmd_arg = '-update '
'repository="' + config['RTC_Server'] + '/ccm" '
'user=' + usr_name + ' password="' + usr_pwd + '" '
'id=' + cr.pop('id') + ' '
run_command("wcl.bat " + cmd_arg, script_path + "/wcl")
cmd_arg = '-importworkitems delimiter="," encoding="UTF-8" timestampFormat="yyyy-MM-dd" '
'repository="' + config['RTC_Server'] + '/ccm" '
'user=' + usr_name + ' password="' + usr_pwd + '" '
'projectArea="' + config['Project_Area'] + '" '
'importFile="' + csv_file + '"'
run_command("wcl.bat " + cmd_arg, script_path + "/wcl")

I am not completely sure about what is in 6.0 compared to 6.0.1. The release is unfortunately a manual process.
I have done minimal fixes to the WCL that now allows to set the creation date when a work item is initially created.

Thanks, I updated to wcl 6.0.1.
Now the wcl finish and provide the following error:
(I downloaded the opencsv 5.7.1 but I will try with the one mentioned in readme 4.3.2)
Executing wcl.bat ...
Exception in thread "main" java.lang.NoSuchMethodError: com.opencsv.CSVReader.(Ljava/io/Reader;CC)V
at com.ibm.js.team.workitem.commandline.commands.ImportWorkItemsCommand.performImport(ImportWorkItemsCommand.java:389)
at com.ibm.js.team.workitem.commandline.commands.ImportWorkItemsCommand.importItems(ImportWorkItemsCommand.java:369)
at com.ibm.js.team.workitem.commandline.commands.ImportWorkItemsCommand.process(ImportWorkItemsCommand.java:319)
at com.ibm.js.team.workitem.commandline.framework.AbstractTeamRepositoryCommand.execute(AbstractTeamRepositoryCommand.java:89)
at com.ibm.js.team.workitem.commandline.WorkitemCommandLine.runCommands(WorkitemCommandLine.java:302)
at com.ibm.js.team.workitem.commandline.WorkitemCommandLine.run(WorkitemCommandLine.java:236)
at com.ibm.js.team.workitem.commandline.WorkitemCommandLine.main(WorkitemCommandLine.java:188)

With the opencsv 4.3.2 it worked. Thank you for you support.


EventBus interface question

I see EventBus has publish but not a subscribe?
Just curious how the events propagated to projects for querying?
I apologize if subscribe is defined somewhere else as I did not read the source code completely.

Hi, for now, the project ends on the publish of events, I think subscribe should be done very easy, for example, nats and rbmq has nice client libs, but definitely, to have a built-in interface for subscribing should be a nice feature

Would the Subscribe be something like below, your thoughts?
type EventBus interface {
Subscribe(eventTopic string) (Event, error)
}

I think it should be counterpart of publish interface
type Subscriber interface {
        Subscribe(bucket, subset string) (Event, error)
}

but I like the word topic probably instead of subset

Thanks for the response, makes sense to have it as as a separate interface.
Do you want me to pull and request for merge to include this interface and its implementation ?

How about something like this
type EventBus interface {
	Subscriber
	Publisher
}

type Subscriber interface {
        Subscribe(bucket, subset string) (Event, error)
}

type Publisher interface {
        Publish(event Event, bucket, subset string) error
}

I like it, yes for sure, help is welcome :D


PostData may need transformations

In the case of posting HLists, Options etc., the PostData is not simply the content wrapped with the ID
Instead it is part of the content, and the implicit postable.
The resulting PostData may have a different parameter P.

Done in d65b6df65c649 and 4426cf8877b23333a; untested


Reduce the number of OpenLineage JSON events emitted from Spark

Looking for feedback on the below Problem and Proposal.
We are using OpenLineage (io.openlineage:openlineage-spark_2.12:1.9.1) with our AWS EMR clusters to extract lineage and send it to a backend Marquez deployment (also in AWS). This is working fine and we are getting table and column level lineage.
Problem: Is we are seeing:

15+ OpenLineage events with multiple jobs being shows in Marquez for a single Spark job in EMR. This causes confusion because team members using Marquez are unsure which "job" in Marquez to look at.
The S3 locations are being populated in the namespace. We wanted to use namespace for teams. However, having S3 locations in the namespace in a way "pollutes" the list.

I understand the above are not issues/bugs. However, our users want us to "clean" up the Marquez UI.
Proposal: One idea was to have a Lambda intercept the 10-20 raw OpenLineage events from EMR and then process -> condense them down to a few events (to show the start, running and complete statuses) with one job ID and run, inputs, outputs details. Secondly, to swap out the namespace from S3 to actual team names via a lookup we would host ourselves.
While the above proposal technically could work we wanted to check with the team here if it makes sense, any caveats, alternatives others have used. Ideally, we don't want to own parsing OpenLineage events if there is an existing solution.

This is answered in https://github.com/OpenLineage/OpenLineage/discussions/597


Closing Apple Pay Sheet

Just the way you can use presentApplePay is there any way or method for closing it?

My apple pay sheet doesn't close by itself after payment method is created. It's stuck in Processing.


Add Contact us link to all docs pages

Description of changes:
Add a "Contact us" link to the top of all docs pages that points to https://aws.amazon.com/contact-us/sales-support-eks/

Should this also include a link to community support via slack or wherever we recommend people go?

/cherrypick release-0.11

Should this also include a link to community support via slack or wherever we recommend people go?

That's a good idea. I'll ask about that.

/lgtm


Regex Tester - escaped expressions

What's the Problem?
Let's say I have a string containing a regular expression in an escaped form.
Very typical situation when regexes are stored in configuration files.
In order to test such an expression, I will have to manually unescape/escape it, which is very error-prone.

The expression on the screenshot - a\\b can mean different things depending on where I've pasted it from. If it came from a string and not a regex literal then I will have to fix it manually before it matches the right thing (a\b - a followed by a word boundary, as in the first line).
Solution/Idea
Some way to escape/unescape the expression in place, so I don't have to do it manually.
It might be a pair of buttons near "Paste" button.
Or it might be a second input (edit one and the other is updated accordingly when valid). Although this might be complicated to do right.
Alternatives
I'm not aware of any tools that do this. It is a common problem.
A separate toy for regex escape/unescape would be too cumbersome to use when I need to test the regex. Although it might make sense in different scenarios.
Some way to stack toys into simple pipelines may be handy to have down the line.
Priorities
No response
DevToys Version
No response
Comments
Escaped expression may have to include quotes. Because it's important to know what quotes were used in order to unescape the expression.
Example: "\\\"" and '\\"'.

So just to clarify. For unescaping, you'd just like a button that collapses every \\ down to \? Or is it something more complex than that?

The more I think about it (without doing prototypes and extensive testing) - the more I am confused actually.
There are some things to take into account.

If we consider expressions copied from files in languages where strings use different quotes (", ' `) then it can potentially cause issues on some expressions, but I can't come up with a valid example.
Regardless of quotes, it seems fine to unescape both \\\" and \\" to \".
Just collapsing \\ will lead to different results.
So it's important to unescape all found escape sequences at once, not just \\.
Existing .NET specific methods: Regex.Escape/Regex.Unescape
Some JS implementation (should also work for JSON): https://github.com/nodef/extra-regexp/tree/master/src
There are be some differences like syntax for Unicode code points/surrogate pairs.

I would be really intimidated to ship this feature myself, honestly. Or spending more time on the research and testing to avoid any corner cases and accidentally broken expressions. And leaving a note on unavoidable limitations.

So it's important to unescape all found escape sequences at once, not just \\.

Mmm, I think this is the problem though. There's two different escapes being used: there's the regex escapes, and the escapes from whatever encoding you copied the regex pattern string from. The regex escaping shouldn't be changed, and deconstructing all the various escapes in all the various regex implementations is not trivial... Ref, ref, ref
But then, as you've pointed out, it's impossible to predict where any given string was copied from. In many languages, you could escape both \" and \' regardless of the enclosing quotes, or it may be optional. Quote escapes may be \' or '', and \\ may not be an escape at all. YAML, for example, escapes a double quote with \" and a single with '', but in a normal C/C++/C# string, '' isn't an escape at all... In fact in C#, prefixing a string literal with @ switches to an entirely different escaping scheme. And then, does " count as an escape or not...?
Honestly, I'm not sure that unescaping strings like this can realistically be automated in a reliable way. 🤔 Like we have right now, each of these escaping schemes would need a dedicated tool, I think. 😊

Yep.
So it must be a set of language-specific functions.
As it turns out to be quite complicated to be considered in the context of Regex Tester - I would prefer to see it as one separate toy - "Text - Escape/Unescape" with a dropdown to choose from supported languages.
Languages with multiple grammars can be represented with multiple entries like "C# regular" and "C# @-string".
Speaking of regex escapes vs string escapes: The goal is to remove the layer of string escapes without modifying the regex. So each implementation is only have to be aware of strings grammar of the language. Regex implementation doesn't seem to matter, except for whether it is compatible with the Regex Tester  toy after the unescape.
While there are languages with complicated grammars ("Yet Another Multi-Line"), this can be scaled down to one language and grow from there based on feedback.
Now this issue becomes very connected to

#160

What exactly #983 does about this request to grant the closure of this issue?
I can't see relevant changes from PR description and brief look through the diff.


authToken option?

I see the authToken option - could you inform how I obtain this so that I can set it? I tried getting something from the google oauth 2.0 playground, but it doesnt work (i.,e I get a RequestError in the response). Thanks!

You can use gp-get-token from the CLI tool https://github.com/dweinstein/node-google-play-cli

ok, thanks. Does this auth-token work? I used the get-token to get the auth token and passed it as an option i.,e authToken: "xxx", but i get the requesterror, so perhaps this doesnt work, or I am not using it right?
at Promise._fulfill (/Users/badri/Projects/GPAPI/node_modules/bluebird/js/release/promise.js:638:18)

message: '\n\nUnauthorized\n\n\nUnauthorized\nError 401\n\n\n',
name: 'RequestError',
statusCode: 401,


attach command

make "attach" a command, not a flag, so that users can call "scope attach [PID]"
make some of the help messages friendlier
add a debug command to the makefile

Where are the unit tests for the attach option?


docker not working

i did the following:
git clone https://github.com/scrapinghub/portia.git portia_docker
cd portia_docker

follow the installation instructions for docker at https://docs.docker.com/engine/installation/linux/ubuntulinux/ sections: perquisites and install.
docker build -t portia .

error:
Cannot connect to the Docker daemon. Is the docker daemon running on this host?

i typed
sudo service docker start

another time just to be sure i had the command right. My system is ubuntu 15.10

Hi @flip111 . Please try using  sudo docker build -t portia .
You want to run as a root user on ubuntu

close

Hi @devNo0b that helped :)
Now i get this error ... (i run docker hello-world first to show that docker works)
» sudo docker run hello-world

Hello from Docker.
This message shows that your installation appears to be working correctly.

To generate this message, Docker took the following steps:
 1. The Docker client contacted the Docker daemon.
 2. The Docker daemon pulled the "hello-world" image from the Docker Hub.
 3. The Docker daemon created a new container from that image which runs the
    executable that produces the output you are currently reading.
 4. The Docker daemon streamed that output to the Docker client, which sent it
    to your terminal.

To try something more ambitious, you can run an Ubuntu container with:
 $ docker run -it ubuntu bash

Share images, automate workflows, and more with a free Docker Hub account:
 https://hub.docker.com

For more examples and ideas, visit:
 https://docs.docker.com/engine/userguide/

» docker run -i -t --rm -v ./slybot/slybot/tests/data:/app/slyd/data:rw -p 9001:9001 --name portia portia
docker: Cannot connect to the Docker daemon. Is the docker daemon running on this host?.
See 'docker run --help'.

I'm doing this in directory /home/flip111/programs/portia


Extract helpers

WIP

Codecov Report

Merging #215 into master will decrease coverage by 0.07%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
- Coverage   92.63%   92.55%   -0.08%
==========================================
  Files          86       85       -1
  Lines        1072     1075       +3
  Branches      171      170       -1
==========================================
+ Hits          993      995       +2
- Misses         64       65       +1
  Partials       15       15

Impacted Files
Coverage Δ

...ges/warriorjs-helper-get-level-config/src/index.js
100% <100%> (ø)

packages/warriorjs-cli/src/Game.js
52.08% <50%> (-0.03%)
: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 e90795c...57f1d5a. Read the comment docs.


MSBot fails to Encrypt (missing --id parameter)

Tool
MSBot version 4.3.2-151
dotnet version 4.2.100
Describe the bug
At the end of a clone operation for a C# project, MSBot errors out with encryption failure(s).
NOTE: This secret is not recoverable and you should save it in a safe place according to best security practices.
      Copy this secret and use it to open the zbot.bot the first time.
Updating appsettings.json with botFilePath=zbot.bot
Saving the botFileSecret with dotnet user-secrets
dotnet user-secrets set botFileSecret [Kilroy's secret was here]
Command failed: dotnet user-secrets set botFileSecret [Kilroy's secret was here] Could not find the global property 'UserSecretsId' in MSBuild project 'C:\Users\chris\source\repos\ZBot\ZBot.csproj'. Ensure this property is set in the project or use the '--id' command line option.
Usage: msbot clone services [options]

The "--id" parameter should be used, but was not specified.
Ensure this property is set in the project or use the '--id' command line option.
The alternative, is adding this element to the .csproj file:
<PropertyGroup>
    <UserSecretsId>USER_SECRETS_GUID</UserSecretsId>
</PropertyGroup>

... but doing that would require creating a new GUID as well.
Better to let the tool do the work.

The code already detects this condition and adds the UserSecretsId.  I was not able to repo this


Update workflows

See https://github.com/senzing-garage/go-grpcing/issues/58

Pull requests

https://github.com/senzing-garage/demo-entity-search/pull/66


QA should get some devices that will support testing of haptics

There are two initiatives right now that are creating Android-based apps that make use of vibratory haptics.  The first is the "Haptics Playground", which is being developed in the quake repository.  The other is an app that will contain the Quadrilaterals simulation and will allow the use of haptics and speech-to-text capabilities, and for which there is not yet a repo.
The primary test device so far for the Haptics Playground is a Samsung Tab S7, model SM-T870.  QA should probably get at least one of these, and possibly one or more other Android devices that support vibration.  See the table in https://github.com/phetsims/quake/issues/4 for more information on what devices may work.
This should probably be funded under the a11y grants, but I'm not entirely sure.
Assigning to @oliver-phet and @KatieWoe to start the discussion of how these should be paid for, what we should get, and who should get them.  My estimate is that the development team will start to need some testing of these apps in about a month.
Tagging @jessegreenberg and @BLFiedler so that they are aware of this, since they are both working on the Quadrilaterals sim and the research that we hope to support with it.

@KatieWoe Does QA already have any supported device? If not, what Android device might be most useful for QA beyond  haptics testing?

I don't believe we do. We have an Android tablet, but I believe that it doesn't support haptics from when we last checked. I would need to do some research into Android devices to list a specific model, but a tablet that supports the latest available OS would be best. The current tablet only supports Android 8.1

My phone (samsung galaxy s21 ultra) seems to work with the patterns on http://www.hapticsjs.org/

@KatieWoe said:

I would need to do some research into Android devices to list a specific model...

Perhaps, but as I mentioned in the opening comment, there is a list of devices that should work list in https://github.com/phetsims/quake/issues/4.

Doing some issue hygiene - is there still an active need for haptics devices?

We've had to do testing for it once so far. If this is going to be a frequent feature that comes through testing I think QA should have something. But if it is going to remain rare I'm not sure. @jbphet what do you see going forward?

From the Inclusive Design team side - there are no current plans for haptics development that would call for a haptics-capable device in QA. That may change in the future with any changes in funding, of course. For the two cases this issue was opened for: Quadrilateral haptics was indefinitely postponed pending collaborator interest or future funding. Haptics Playground is stable enough as a tool for use prototyping with compatible devices. It also isn't seeing active development for the foreseeable future.

In that case, I don't think QA needs such a device at this time. Closing


Merge dev-v2.0.0 into staging

Description
This merge includes changes to add RFC2822 time utility methods.

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

By passing codecov as 0.01% decrease due to merge is small enough to ignore.


feat(credo): auto install latest version

Current dependencies on/for this PR:

main

PR #114   👈

This comment was auto-generated by Graphite.

Not sure if you're already aware but fyi: I'm getting Client 2 quit with exit code 1 and signal 0 when I use the latest version of elixir-tools and do not set a version (I don't see any credo output using OutputPanel)

Not sure if you're already aware but fyi: I'm getting Client 2 quit with exit code 1 and signal 0 when I use the latest version of elixir-tools and do not set a version (I don't see any credo output using OutputPanel)

@dvic Do you have curl installed?

Also make sure you're fully updated cuz there is a commit after this one that fixes a bug

Yup, now with the latest version it does work 👍


Testing with old and new generated metadata files

Description of issue or feature request:
We have generated a new set of metadata files with #1915.
Now we have two folders containing metadata files:

tests/repository_data
tests/generated_data

It makes sense to tests with both of them - repository_data was generated by the old client and we want to test that we are still compatible with it and the generated_data are folders containing metadata files generated with python-tuf version 1.0.0 and above.
It will be good to find a way to run part or all tests with metadata files from both folders.
Current behavior:
Testing only with tests/repository_data.
TO DO:

[ ] Decide which tests should use the old repository_data and which generated_data.
[ ] Change the tests according to the decision.

I think i can work upon it. some questions -

do i have to modify all the tests so that all tests can work on both the folders.?
all tests run in specific order or their running order is random ?
is this issue active ?
any suggestions or tips which can help me ?


Not compiling because of mm/hbmemory.o

I just checkout out latest devel and found out, that HermitCore unfortunately is not building correctly at the moment:
$ make veryclean
$ make
[...]
make[1]: Entering directory '/home/daniel/Entwicklung/HermitCore/hermit'
make[2]: Entering directory '/home/daniel/Entwicklung/HermitCore/hermit/usr'
make[2]: Nothing to be done for 'bootstrap'.
make[2]: Leaving directory '/home/daniel/Entwicklung/HermitCore/hermit/usr'
make[1]: *** No rule to make target 'mm/hbmemory.o', needed by 'libhermit.a'.  Stop.
make[1]: Leaving directory '/home/daniel/Entwicklung/HermitCore/hermit'
make: *** [Makefile:7: default] Error 2

Looks like you just forgot to commit mm/hbmemory.c

Ups, I just added the file.


grant RBAC permissions for provider roles to use PSP we have for crossplane

Done in https://github.com/giantswarm/management-clusters-fleet/pull/411


Duplicate identifier error

Hi,
I am learning angular2 and some associated graphing tools. I seem to be having a problem using angular2-highcharts. When I try to run using npm start, the following error occurs(for multiple lines):
typings/globals/highcharts/index.d.ts... getting error TS2300: Duplicate identifier.....
I went through a few solutions suggested in similar issues. I modified the tsconfig file to include an exclude portion to exclude node_modules, but that didn't work. Since am new to angular and typescript, I am not sure if I am missing something very basic here.

Seems like you installed highcharts typings.
https://github.com/gevgeny/angular2-highcharts/issues/33 this should help you. But basically just remove from your typings folder the highcharts typings files. Only thing you need is to install highcharts and angular2-highcharts and your good to go no need to install any typings (unless your using some kind of library's).

Thank you! That solved it.


Deployment Quiz 3/27

Deployment Quiz
1. Curl

2. Deployed Site

3. Security

Purpose Blog
Deployment helps us understand many things about an application and how the internet works. Reviews of this event should be compiled and contain the following.

Docker: Docker simplifies the deployment of backend applications by providing a uniform environment, separating them from other processes, and providing easy accesss across different systems. It allows to packaging of applications along with dependencies into containers, which makes sure that they run consistently. This makes Docker a valuable tool in terms of functionality and security of the deployed backend.

Nginx: Nginx functions as a web server, serving web content to users accessing applications over the internet. It also acts as a reverse proxy, which directs client requests to backend servers that are currently hosting the application, esuring load balance moreover Nginx enhances security by managing SSL termination decrypting HTTPS requests, and forwarding them to backend servers securely.

Route 53 (DNS): DNS (Domain Name System) translates domain names into IP addresses, allowing users to access websites and applications using readable names rather than numrical IP addresses. In the process of finding our backend application, DNS serves as a directory service, enabling users' devices to query DNS servers to resolve the domain name of the application to its corresponding IP address.

Certbot: When a user accesses a website secured with an SSL certificate managed by Certbot, their web browser makes a secure connection request to the server. Certbot automatically handles the SSL certificate issuance, renewal, and installation process, ensuring the connection between the user's browser and the website is encrypted and secure.


Maximum Credential Time Span of 2 Years

Attempting to run the below cmd-let:
New-MsolServicePrincipalCredential -tenantId "25fc..." -AppPrincipalId "697c..." -Type asymmetric -Value $cred -StartDate $cer.GetEffectiveDateString() -EndDate $cer.GetExpirationDateString() -Usage verify
fails with:
New-MsolServicePrincipalCredential : Invalid value for parameter.  Parameter
name: Credential.EndDate.
At line:1 char:1
I tried manually providing the date-time string as well (using the value returned by $cer.GetExpirationDateString()) but even that does not make it happy. I can feed a slightly earlier time to -EndDate and then it works. However, that does not eventually help because when the sample is run, signature validation fails. :-(
It seems like a timezone type issue...that too only for the EndDate parameter. These are the StartDate and EndDate values of the cert I am using: 6/15/2014 4:56:12 PM, 6/15/2016 4:56:11 PM (I am in IST).
I actually got this to work somehow but don't recollect how (darn, should have written down the final command that worked... :-().
(Edit: Figured it out. It was by not providing the StartDate, EndDate tuple at all. Then it seems to take them from the cert date. Duh! :-) However, there still seems to be a DST issue. My cert's IST expiration time is 4:56 PM, that should have been 12:26PM GMT as it is summer. However, it shows as 11:26AM GMT (which would be okay in winter) when I run Get-MsolServicePrincipalCredential.)
However, I also wonder why this cmd-let wants to get these parameters separately when it is also getting the entire raw certificate. Can't it pick these up form the cert data (and have a higher assurance of their authenticity as well)? Secondly, the fact that AAD uses these values subsequently for signature validation part is even more baffling/worrisome...as I believe the effective and expiration dates are supposed to be included in the authenticated blob inside a cert and any security validation should read them from there (and not from any other extraneous source). (Note that this is based on the observation that I could feed a slightly early-PM or AM expiration date and get the cmd-let to accept the certificate -- however the client_credential flow fails with invalid signature.)

An alternate fix is to ensure the "-End" date is not longer that 2 years out from the "-Start". There's seem to be a max time span.

Closing this issue as the example does no longer use New-MsOlServicePrincipalCredential


Run docker build on all PR builds

Similar to what builder does in its .travis.yml file, slugbuilder should run make ... docker-build on each component in this repo to ensure that the Dockerfiles are correct

This issue may not apply since I don't see any .travis.yml files in here. If not, please feel free to close


Bus error

Node version:
5.8.0
npm version:
3.7.3
Operating system:
Ubuntu 15.04
Command line used:
npm start
Steps to reproduce:
After test success, there is a Bus error without any other information.
Bus error (core dumped)
npm ERR! Linux 3.19.0-65-generic
npm ERR! argv "/usr/local/nodejs/bin/node" "/usr/local/nodejs/bin/npm" "run-script" "open:src"
npm ERR! node v5.8.0
npm ERR! npm  v3.7.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>open:src: babel-node tools/srcServer.js
npm ERR! Exit status 135
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>open:src script 'babel-node tools/srcServer.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-slingshot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     babel-node tools/srcServer.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-slingshot
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-slingshot
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/zzy/home/zzy/project/redux-demo/react-slingshot/npm-debug.log
ERROR: "open:src" exited with 1.
npm ERR! Linux 3.19.0-65-generic
npm ERR! argv "/usr/local/nodejs/bin/node" "/usr/local/nodejs/bin/npm" "start"
npm ERR! node v5.8.0
npm ERR! npm  v3.7.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: npm-run-all --parallel test:watch open:src lint:watch
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'npm-run-all --parallel test:watch open:src lint:watch'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-slingshot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm-run-all --parallel test:watch open:src lint:watch
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-slingshot
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-slingshot
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/zzy/home/zzy/project/redux-demo/react-slingshot/npm-debug.log

Bus error (core dumped)
ERROR: "open:src" exited with 1.


Some thoughts about use existing code from other projects

I tried Npcbot and I think  playerbot still the best idea of populate server with live bot and the ultimate fun of the way to go, However, the npc fighting and manage scripts are really well implemented, Is it possible to migrate that part of code to playerbot so that it could save so much time for development?

Probably not, given one uses creatures and the other uses characters as the vessel of bots.

I don't think so either. And I also used npc bot, but I think I prefer playerbot.

Guys, I've been testing some things and I even informed Zhang about it. I noticed that in the playerbot source there are many .csv extensions that are not being created automatically.
I even verified that there is already a download for some types of .csv. Here: https://github.com/celguar/mangosbot-bots/blob/878e02b2b612de8cf032fb69d433bbe2cff6b833/playerbot/aiplayerbot.conf.dist.in.wotlk#L520
AiPlayerbot.AllowedLogFiles = "bot_events.csv,bot_movement.csv,bot_pathfinding.csv,creatures.csv,crop.csv,Deaths.csv,gos.csv,load_map_grid.csv,pathfind_attempt.csv,pathfind_attempt_point.csv,pathfind_result.csv,player_location.csv,scan.csv,strategy.csv,travel_map.csv,travelNodes.csv,travelPaths.csv,unload_grid.csv,unload_obj.csv,vmangoslines.csv,zones.csv"
Another point that I noticed, especially the player_location.csv file, it made the bot much more responsive, to the point that I did quests in a party with them.

If you can test that too, that would be interesting.

@htc16 very interesting finding, I will try.

I already have the answer for these .csv files, I'll share it with you!

I just got the answers from the creator of these files I downloaded.

These .csv files just provide information about what is happening with the bots, how many levels the bots are gaining, how hard the server is working, how many bots are online, etc.

These .csv files just provide information about what is happening with the bots, how many levels the bots are gaining, how hard the server is working, how many bots are online, etc.

looks really cool

@rogerc66 please close this issue so we can clean up the high amount of old issues clogging this, I think Bench's answer still stands that they can't copy over the creature combat scripts for the playerbots. If something new comes up we can open up a new issue but for now this discussion is old and done.


Error when trying to extract images from a pdf

Hello everyone, I'm trying extract some images from a pdf.
I'm getting this error:

I think the error is explicit enough, is it possible to add those dependencies to the gradle file?
implementation 'com.github.jai-imageio:jai-imageio-core:1.4.0'
implementation 'com.github.jai-imageio:jai-imageio-jpeg2000:1.3.0'

Thank you !

Sorry having to remove this, the licensing for jpeg2000 is not usable for us as a opensource application

Might be possible now found a new jpeg200 which is apache license

Might be possible now found a new jpeg200 which is apache license

I see JPEG200 support was added in 0.28.0 but I still get the same error like in OP on the latest version (0.28.3).

Yeah it seems even that is dependant on the other license etc.. sadly still no fix!


Add Changelog.rst

Add a changelog file that we'll populate between each release. Can be empty for now.

Release-please will take care of that


livekit: wait for disconnect before callback

should fix a race condition where in node-sdks await room.disconnect() doesn't wait for the room status update to change.
draft because it doesn't work reliably yet: patched node-sdks still fails approximately one in fifty times.

node-sdks diff
diff --git a/packages/livekit-rtc/src/room.ts b/packages/livekit-rtc/src/room.ts
index 143774f..f9bebeb 100644
--- a/packages/livekit-rtc/src/room.ts
+++ b/packages/livekit-rtc/src/room.ts
@@ -16,6 +16,7 @@ import type {
   ConnectResponse,
   ConnectionQuality,
   DataPacketKind,
+  DisconnectCallback,
   DisconnectResponse,
   IceServer,
   RoomInfo,
@@ -148,7 +149,7 @@ export class Room extends (EventEmitter as new () => TypedEmitter<RoomCallbacks>
       return;
     }

-    FfiClient.instance.request<DisconnectResponse>({
+    const res = FfiClient.instance.request<DisconnectResponse>({
       message: {
         case: 'disconnect',
         value: {
@@ -157,6 +158,10 @@ export class Room extends (EventEmitter as new () => TypedEmitter<RoomCallbacks>
       },
     });

+    await FfiClient.instance.waitFor<DisconnectCallback>((ev: FfiEvent) => {
+      return ev.message.case == 'disconnect' && ev.message.value.asyncId == res.asyncId;
+    });
+
     FfiClient.instance.removeAllListeners();
     this.removeAllListeners();
   }

fixed on the node-sdks side.
merge this before merging livekit/node-sdks#223

I think just removing close_tx and waiting for engine_tx to ends should do the work

I think just removing close_tx and waiting for engine_tx to ends should do the work (we can close engine_tx on EngineInner)

i don't think this works, await room.disconnect() on Node doesn't unblock


Pattern slicing of CodableConcepts incorrectly includes display values

Section Number
Profiles throughout
Issue
In several profiles, pattern-type slicing discriminators of CodeableConcept (and possibly, Coding and code) elements incorrectly includes the display values in the slice definitions. This means that (a) a code display value is necessary to meet the slice definition, and (b) the specific display value used is necessary to meet the slice definition. This reduces profile usability for environments which may prefer to omit display values for resource size, do not need the clarity that a display value can provide, or which operate in other languages.
Proposed Change
Omit display values for codes used as slicing discriminators (and probably elsewhere in the profile definitions).
Priority:

Medium: Significant issue or clarification. Requires discussion, but should not lead to long debate.

@ElliotSilver , so are you saying that
* agent[source].type = DCM#110153 "Source Role ID"
* agent[source].who 1..1
* agent[source].network 1..1
* agent[recipient] obeys val-audit-source
* agent[recipient].type = DCM#110152 "Destination Role ID"
* agent[recipient].who 1..1
* agent[recipient].network 1..1

should be
* agent[source].type = DCM#110153
* agent[source].who 1..1
* agent[source].network 1..1
* agent[recipient] obeys val-audit-source
* agent[recipient].type = DCM#110152
* agent[recipient].who 1..1
* agent[recipient].network 1..1

Correct. See https://chat.fhir.org/#narrow/stream/179252-IG-creation/topic/Best.20practice.20for.20constraining.20CodeableConcepts.20and.20Coding.
Although, I'd leave the display values as comments.


Temporal table query via remote data source doesn't work

select * from DatabaseName.DataMart.DataSourceName.Students FOR SYSTEM_TIME AS OF '2021-08-01 T10:00:00'
Error message of
Temporal FOR SYSTEM_TIME clause cannot be used on remote data source 'DatabaseName.DataMart.DataSourceName.Students'.
Leaving us with the only option of querying the history table directly, this is silly :/
This should be adding as a limitation of Temporal tables
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 751e374d-cd64-1de5-c1d2-5c426c9534c9
Version Independent ID: 2c62a70f-cd2e-bf40-5974-dde1b5a10efe
Content: Querying Data in a System-Versioned Temporal Table - SQL Server
Content Source: docs/relational-databases/tables/querying-data-in-a-system-versioned-temporal-table.md
Product: sql
Technology: table-view-index
GitHub Login: @markingmyname
Microsoft Alias: maghan

@visch -- Derek, thank you for your feedback.
@markingmyname -- Mark, please look into this issue.

@visch did you review the limitations described here:  https://docs.microsoft.com/en-us/sql/relational-databases/tables/temporal-table-considerations-and-limitations?view=sql-server-ver15

Did you read the issue I posted?

@visch did you review the limitations described here: https://docs.microsoft.com/en-us/sql/relational-databases/tables/temporal-table-considerations-and-limitations?view=sql-server-ver15

Yes

@visch can you share what RDBMS you are using? Regarding this "Temporal FOR SYSTEM_TIME clause cannot be used on remote data source"

Microsoft SQL server

@visch I am trying to get some clarity regarding your statement "This should be adding as a limitation of Temporal tables".  This is not a question. Are you requested that we updated the guidance for limitations based on the error message you received when you ran the posted script?

@visch Yes I read your post. I am trying to get some clarity regarding your statement "This should be adding as a limitation of Temporal tables". This is not a question. Are you requesting that we updated the guidance for limitations based on the error message you received when you ran the posted script?

Got it.
Yes terrible grammar on my part. A better way to write this is -- Not being able to query a temporal table with "for system time" via a linked server should be listed as a limitation of temporal tables.

@visch I was able to confirm this internally. "Temporal syntax works only on tables or views that are stored locally in the database. If it is a remote object like a table on linked server or external table then you cannot use the FOR clause or PERIOD predicates directly in the query. " We will make this addition to the content https://docs.microsoft.com/en-us/sql/relational-databases/tables/temporal-table-considerations-and-limitations?view=sql-server-ver15  Please check back again to see this refreshed content.

Great, thank you very helpful to someone like me who tried every combination of things I could think of as I didn't' see it as a limit.
I'm guessing I need to submit a feature request to have a chance of this changing?

@visch not at all. I appreciate your input. I am updating the content as we speak and it should be reflected within the next docs refresh.

I understand the docs were updated, and I appreciate that, great step!
Now I want the ability to run a temporal query with a Linked Server, how do I add a feature request for that? Or is there a 0 percent chance of that happening? :D

@visch I'd like to look at things more positively. I will locate the appropriate channel and update this comment.


Rename ToDo templates to be more descriptive of local application stack

Rename AZD templates & update descriptions according to the following chart:

Current   Name
Proposed Name
Proposed Short   Description

ToDo application with C# API and Azure Cosmos DB
React   Web App with C# API and MongoDB on Azure
A   blueprint for getting a React web app with a C# API and a MongoDB database on   Azure. The blueprint includes sample application code (a ToDo web app) which   can be removed and replaced with your own application code. Add your own   source code and leverage the Infrastructure as Code assets (written in Bicep)   to get up and running quickly.

ToDo application with C# API and Azure SQL Database
React   Web App with C# API and SQL Database on Azure
A   blueprint for getting a React web app with a C# API and a SQL database on   Azure. The blueprint includes sample application code (a ToDo web app) which   can be removed and replaced with your own application code. Add your own   source code and leverage the Infrastructure as Code assets (written in Bicep)   to get up and running quickly.

ToDo C# Azure SQL + SWA + Functions
Static   React Web App + Functions with C# API and SQL Database on Azure
A   blueprint for getting a React web app with a C# API and a SQL database on   Azure. The blueprint includes sample application code (a ToDo web app) which   can be removed and replaced with your own application code. Add your own   source code and leverage the Infrastructure as Code assets (written in Bicep)   to get up and running quickly. This architecture is for hosting static web   apps with serverless logic and functionality.

ToDo Java Mongo
React   Web App with Java API and MongoDB on Azure
A   blueprint for getting a React.js web app with a Java API and a MongoDB   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Bicep) to get up and running quickly. This architecture is for hosting web   apps and APIs without worrying about the infrastructure.

ToDo Java Mongo ACA
Containerized   React Web App with Java API and MongoDB on Azure
A   blueprint for getting a React web app with a Java API and a MongoDB database   on Azure. The blueprint includes sample application code (a ToDo web app)   which can be removed and replaced with your own application code. Add your   own source code and leverage the Infrastructure as Code assets (written in   Bicep) to get up and running quickly. This architecture is for running   containerized apps or microservices on a serverless platform

ToDo NodeJs MongoDB
React   Web App with Node.js API and MongoDB on Azure
A   blueprint for getting a React web app with a Node.js API and a MongoDB   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Bicep) to get up and running quickly. This architecture is for hosting web   apps and APIs without worrying about the infrastructure.

ToDo NodeJs MongoDB (Terraform)
React   Web App with Node.js API and MongoDB (Terraform) on Azure
A   blueprint for getting a React web app with a Node.js API and a MongoDB   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Terraform) to get up and running quickly. This architecture is for hosting   web apps and APIs without worrying about the infrastructure.

ToDo NodeJs MongoDB ACA
Containerized   React Web App with Node.js API and MongoDB on Azure
A   blueprint for getting a React web app with a Node.js API and a MongoDB   database onto Azure. The blueprint includes sample application code (a ToDo   web app) which can be removed and replaced with your own application code.   Add your own source code and leverage the Infrastructure as Code assets   (written in Bicep) to get up and running quickly. This architecture is for   running containerized apps or microservices on a serverless   platform   This architecture is for running containerized microservices   without managing the servers.

ToDo NodeJs MongoDB SWA + Functions
Static   React Web App + Functions with Node.js API and MongoDB on Azure
A   blueprint for getting a React web app with a Node.js API and a MongoDB   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Bicep) to get up and running quickly. This architecture is for hosting   static web apps with serverless logic and functionality.

ToDo NodeJs MongoDB AKS
Kubernetes   React Web App with Node.js API and MongoDB on Azure
A   blueprint for getting a React.js web app with a Node.js API and a MongoDB   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Bicep) to get up and running quickly. This architecture is for running   Kubernetes clusters without setting up the control plane.

ToDo Python MongoDB
React   Web App with Python API and MongoDB on Azure
A   blueprint for getting a React.js web app with Python (FastAPI) API and a   MongoDB API in Cosmos database onto Azure. The blueprint includes sample   application code (a ToDo web app) which can be removed and replaced with your   own application code. Add your own source code and leverage the   Infrastructure as Code assets (written in Bicep) to get up and running   quickly. This architecture is for hosting web apps and APIs without worrying   about the infrastructure.

ToDo Python MongoDB (Terraform)
React Web App with Python API and MongoDB (Terraform) on Azure
A   blueprint for getting a React.js web app with Python (FastAPI) API and a   MongoDB API in Cosmos database onto Azure. The blueprint includes sample   application code (a ToDo web app) which can be removed and replaced with your   own application code. Add your own source code and leverage the   Infrastructure as Code assets (written in Terraform) to get up and running   quickly. This architecture is for hosting web apps and APIs without worrying   about the infrastructure.

ToDo Python MongoDB ACA
Containerized   React Web App with Python API and MongoDB on Azure
A   blueprint for getting a React.js web app with Python (FastAPI) API and a   MongoDB API in Cosmos database onto Azure. The frontend, currently a ToDo   application, is designed as a placeholder that can easily be removed and   replaced with your own frontend code. This architecture is for running   containerized apps or microservices on a serverless platform

ToDo Python MongoDB SWA + Functions
Static   React Web App + Functions with Python API and MongoDB on Azure
A   blueprint for getting a React.js web app with Python (FastAPI) API and a   MongoDB API in Cosmos database onto Azure. The frontend, currently a ToDo   application, is designed as a placeholder that can easily be removed and   replaced with your own frontend code. This architecture is for hosting static   web apps with serverless logic and functionality.

ToDo Application with a Java API and Azure Database for   PostgreSQL - Flexible Server on Azure App Service
React   Web App with Java API and PostgreSQL on Azure
A   blueprint for getting a React web app with a Java API and a PostgreSQL   database on Azure. The blueprint includes sample application code (a ToDo web   app) which can be removed and replaced with your own application code. Add   your own source code and leverage the Infrastructure as Code assets (written   in Bicep) to get up and running quickly.

@rajeshkamal5050 This should be done ahead of 5/9.
We just need to update the titles and paragraphs in our ToDo template READMEs + the repo side rail.
For example:
Title:

First paragraph:

Siderail:

Once this is done, we'll update awesome-azd

@hemarina thanks for pointing that out, yes we do!
Let's go with this:

Current Name
Proposed Name
Proposed Short Description

ToDo Application with a Java API and Azure Database for PostgreSQL - Flexible Server on Azure App Service
React Web App with Java API and PostgreSQL
A blueprint for getting a React web app with a Java API and a PostgreSQL database on Azure. The blueprint includes sample application code (a ToDo web app) which can be removed and replaced with your own application code. Add your own source code and leverage the Infrastructure as Code assets (written in Bicep) to get up and running quickly.

I recommend including Azure in the name because it will appear in Google search results and helps with SEO to have it there.
We'll also need a way to differentiate the samples.  I noticed above that you have the same name for multiple sames in the title.  For SEO reasons we shouldn't duplicate the title.  Imagine searching for "react web app azure" and seeing a number of results with the same title.  The choice on which one to click on should be apparent in the Title.
I recommend just removing "ToDo APplication" from existing titles, if your primary driver is length.

The primary driver isn't just length. It's about making it more approachable for application developers to identify that a template might be relevant to their local application stack, since they may not know about Azure offerings. These names were a joint decision with Starshot folks, LT, and azd. I can provide more details if you're interested via DM

Ultimately I recommend creating unique names for each template so they are distinguishable via title only.  I'd also include Azure in the title because they are Azure templates.

@jongio @savannahostrowski , I just updated the table at the top of this issue. if those changes look good (adding "on Azure
to the end of each & distinguishing the duplicates), I'll go ahead and open a PR to update them

LGTM - let's get them updated!

@weikanglim Can we update the names in the template list/init to reflect the unique name updates above? No need to add on Azure in the commands because that's sort of obvious based on context.

Yeah - we'll need to make another small PR to do a couple last minute amendments.

@savannahostrowski

Do we need "on Azure" when the user is initializing from a template? They're already in the context of azd.
The template duplication was fixed, we added - Terraform for the TF specific one herehttps://github.com/Azure/azure-dev/blob/2bab4461e3face1688ef5afc296d963108636715/cli/azd/resources/templates.json#L43

No need for "on Azure" outside of the template title on GitHub.


can not read properties of undefined (reading offsetHeight)

I'm using nuxtjs version 2.15.7 and get this error. ssr is set to false. the error shows up when the picker dialog is open and I scroll to see the time picker

what I should add is that this happens if the height of the picker is more than that of page. a little page scroll could reproduce this error. I test the application on chrome browser.

Hi @honarmanly
Please read this page https://alireza-ab.ir/datepicker/installation#nuxt and do it. And set SSR to true and test it.


Feat/return user info on invitation

What does this PR do?
Adds the functionality to return user information (id, email) on the invitation.
Which issue(s) does this PR fix?
https://github.com/authorizerdev/authorizer/issues/345
If this PR affects any API reference documentation, please share the updated endpoint references
https://github.com/authorizerdev/docs/compare/main...MussieT:docs:patch-1
Preview

@lakhansamani Thanks for the response. But one question, the written test for the invite_members implementation checks whether the response is not nil. Should I modify it to check if the response contains both the message and users?

Yes that would be better test earlier response just had message now its good idea to check if it has expected users or not

@lakhansamani updated the test.

Thanks @MussieT

Appreciated the response @lakhansamani , planned to have more contributions soon.


Bluetooth firmware loader

Looking into systemctl logs (sudo systemctl --failed), I realized a service called btfw_loader.service fails to load on my Coral Dev Board. btfw_loader.service executes ExecStart=/usr/bin/hciattach /dev/ttymxc1 qca 1000000 flow -b -t 120 while there is no hciattach in my system. Was wondering if this an issue with my mendel installation or the Bluetooth service is not implemented for Coral Dev Board yet.
Currently I do not need Bluetooth connection, just wanted to report it!
Kernel version: 4.14.98-imx

@vganjali I believe what's missing is that you'll need to enable this kernel option on the defconfig
CONFIG_BT_HCIUART_BCM

See here for how to do this: https://github.com/f0cal/google-coral/issues/56#issuecomment-632120325

Thanks for the report @vganjali . Note that we do test hciattach specifically and include a package for bluez tools (https://coral.googlesource.com/bluez-imx-debian/) - perhaps there is something missing in your install.

@vganjali I believe what's missing is that you'll need to enable this kernel option on the defconfig
CONFIG_BT_HCIUART_BCM=y

See here for how to do this: f0cal/google-coral#56 (comment)

@Namburger  thanks for the reply. I will check kernel options. It would be nicer to have a cmdline.txt to config Coral board during the boot time rather than compiling the kernel for each change (just like RPi).

@vganjali Were you able to fix this problem ?

Feel free to reopen if the issue still persists.


Add v0.2 of PR commenter task compatible with 0.2 test-metadata.

This is basically copy paste of v0.1 of this task - just instead of direct parameters, job-spec is being used.
Still WIP - need to test. I'll do this as part of my other efforts and I'll update this PR once the testing is done. Opening this PR for feedback

Ready for review. Tested in this PR: https://github.com/rhopp/rhtap-cli/pull/16#issuecomment-2491819513

For completeness - The change in test-metadata was done because $SNAPSHOT is itself a JSON which in turn constructed invalid job-spec json. I had two options - escape value of $SNAPSHOT and treat it as string and all the tasks that would use would have to unescape it (which might be relatively easy with jqs functions tojson and fromjson), or second option which I chose in the end is to embed the json object directly into job-spec. I think this makes everything easier.


Update README.md

README was made more user friendly and more themed around Allfeat

Keep in mind that this Repo contains the tools to build a website that hosts the documentation. It is not really meant to be used locally ; MkDocs will build the HTML+CSS for a nice looking documentation.
If the documentation is to be shared, it seems more appropriate to share a link to a server that hosts it, rather than sharing the repo that builds it, hence why editing README seems like an odd issue to have

Yes absolutely. Sorry if it was not clear enough, but the README should be here to help contributors to test their changes in a local developement environment, that's why it still important to have a README explaining how to setup it and how to contribute to the documentation.
Maybe you could remove all the parts explaining what is Allfeat in the README u made and instead having a more important part on how to contribute to the documentation (like explaining the good path -> create an issue for a missing doc/a bug/an incorrect text then create a PR...) maybe we should get inspiration on other documentation repos of other open-source projects

As contributing is already in the doc, yeah maybe is redundant to put it again the README my bad.

I've finalized this PR, so that the readme still at least looks nicer, and has some amount of introduction for contributors making pull requests


Explicitly register access thread to KVDK instance instead of implicitly

What is changed and how it works?
KVDK limits max access threads, in current impl, a thread will be implicitly registered to kvdk instance while calling any of read/write function. Meanwhile, it is required for access thread to manually release resource from the instance, which can be obscure.
In  this patch, we make thread register explicit. A thread must call function RegisterAccessThread() before accessing a kvdk instance, the occupied thread resource will be released  while calling ReleaseAccessThread() or terminating the thread.
Tests

Unit test
Stress test

Side effects
No

2 concerns:

This pull request may introduce backward incompatibilities to users.
Should we make another pull request to add these new APIs into Java module?


Cache Java/Maven dependencies

Java deps take a while to download and install. We should be caching them between builds.

One solution is to not assign cache key for java. BuildKit cache for java can then be shared between builds.

I'm not sure what you mean? The problem right now is that no directories are specified to cache during the build. We should do something like this https://github.com/railwayapp/nixpacks/blob/main/src/providers/node.rs#L71 to indicate which directory we want buildkit to cache.

I believe we are assigning buildkit cache id right? I'm saying for java usecase, we probably don't need that?.

I believe we are assigning buildkit cache id right?

Ya that is correct. I've just noticed that Java/Maven builds can take a long time sometimes so was thinking we could improve the by caching something. I haven't been in the Java ecosystem in a while though so not sure if there is a standard directory to cache.

Dependencies should end up in ~/.m2/repository, so that's probably what you want to cache.


Heath go-live 8-19-21 (design doc updates)

Docs Build status updates of commit 1ce9753:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/design/personal-apps.md
:warning:Warning
View
Details

msteams-platform/messaging-extensions/design/messaging-extension-design.md
:warning:Warning
View
Details

msteams-platform/tabs/design/tabs.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tabs/MobileLocationPermission.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tabs/MobilePermissions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/ui-templates/mobile-list.png
:white_check_mark:Succeeded
View

msteams-platform/bots/design/bots.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/deploy-and-publish/appsource/prepare/teams-store-validation-guidelines.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/activity-feed-notifications.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/app-structure.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/design-teams-app-advanced-ui-components.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/design-teams-app-ui-templates.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/enable-app-customization.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/understand-use-cases.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/device-capabilities/native-device-permissions.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/cards/design-effective-cards.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/task-modules/design-teams-task-modules.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/design/personal-apps.md

Line 38, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal tab's component anatomy.' is duplicated. Within an article, alt text must be unique.
Line 47, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal tab's structural anatomy.' is duplicated. Within an article, alt text must be unique.
Line 56, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal tab's component anatomy.' is duplicated. Within an article, alt text must be unique.
Line 65, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal tab's structural anatomy.' is duplicated. Within an article, alt text must be unique.
Line 91, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal bot component anatomy.' is duplicated. Within an article, alt text must be unique.
Line 102, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows personal bot component anatomy.' is duplicated. Within an article, alt text must be unique.

msteams-platform/messaging-extensions/design/messaging-extension-design.md

Line 115, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows how to open a messaging extension from a channel post on mobile.' is duplicated. Within an article, alt text must be unique.
Line 119, Column 1: [Warning-duplicate-alt-text] Alt text 'Example shows how to open a messaging extension from a channel post on mobile.' is duplicated. Within an article, alt text must be unique.

msteams-platform/tabs/design/tabs.md

Line 93, Column 1: [Warning-duplicate-alt-text] Alt text 'Illustration showing the UI anatomy of a tab.' is duplicated. Within an article, alt text must be unique.
Line 103, Column 1: [Warning-duplicate-alt-text] Alt text 'Illustration showing the UI anatomy of a tab.' is duplicated. Within an article, alt text must be unique.

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


Consider releasing an update to fix chrono issue

rust-tuf version 0.1.6 depends on chrono 0.3.1, which has been yanked due to wrong symver versioning.
This makes rust-tuf version 0.1.6 impossible to use:
$ cargo new --bin --vcs none tufexample
$ cd tufexample
$ echo 'tuf = "0.1.6"' >> Cargo.toml
$ cargo build
    Updating registry `https://github.com/rust-lang/crates.io-index`
error: no matching version `^0.3.1` found for package `chrono` (required by `tuf`)
location searched: registry https://github.com/rust-lang/crates.io-index
versions found: 0.4.0, 0.3.0, 0.2.25, ...

The only way for now is to use rust-tuf from git's branch develop.
$ sed -ibk "s|tuf = .*||g" Cargo.toml
$ echo 'tuf = {git = "https://github.com/heartsucker/rust-tuf.git", branch = "develop" }'
$ cargo build
    Updating git repository `https://github.com/heartsucker/rust-tuf.git`
    Updating registry `https://github.com/rust-lang/crates.io-index`
[...]
    Finished dev [unoptimized + debuginfo] target(s) in 36.1 secs

Could a new version of rust-tuf be released to crates.io?
Thanks!

Hey @nbigaouette-eai. Since the API has dramatically changed from 0.1.x to 0.2.0, and the 0.1.x series is unsupported, what you should do is switch to pointing to a pinned commit on the develop branch of this repo. I'm coming close to an alpha release of 0.2.0 which should have a relatively stable API, and you could easily change back over to using crates.io after that.

Yes, I'm already pointing to the develop branch.
The issue is that the version on crates.io is useless. This was an unfortunate screwup from the chrono crate where a version was released that did not respect symver; the increment was on the patch version (0.3.0 -> 0.3.1) but the API broke so it should have been released as a 0.4.0. Their 0.4.0 should have been a 0.5.0 instead... As such, the authors decided to yank 0.3.1 from crates.io.
And since chrono 0.3.1 is yanked from crates.io, rust-tuf 0.1.6 cannot be used anymore! I hoped a new version of rust-tuf could have been released, just to fix this issue.
So if chrono 0.3.1 and 0.4.0 have the same API, a hypothetical version bump from 0.1.6 to 0.1.7 (or 0.2?) could be made which includes just this fix and released to crates.io to solve this problem.
I understand rust-tuf is unsupported, but without a version bump on crates.io the package is useless... :(

Bumping rust-tuf's dependency to chrono 0.4.0 fails:
error[E0432]: unresolved import `chrono::UTC`
 --> src/metadata.rs:1:24
  |
1 | use chrono::{DateTime, UTC};
  |                        ^^^ no `UTC` in the root

error[E0432]: unresolved import `chrono::UTC`
 --> src/tuf.rs:1:5
  |
1 | use chrono::UTC;
  |     ^^^^^^^^^^^ no `UTC` in the root

error[E0432]: unresolved import `chrono::UTC`
 --> src/metadata.rs:1:24
  |
1 | use chrono::{DateTime, UTC};
  |                        ^^^ no `UTC` in the root

error[E0432]: unresolved import `chrono::UTC`
 --> src/tuf.rs:1:5
  |
1 | use chrono::UTC;
  |     ^^^^^^^^^^^ no `UTC` in the root

error: aborting due to 2 previous errors

error: Could not compile `tuf`.
Build failed, waiting for other jobs to finish...
error: aborting due to 2 previous errors

error: build failed

So their API changed... I'll see what I can do.

UTC --> Utc

I pushed a new version to crates.io.


Code block theme issue in homepage

while the system default theme is dark the code block's theme stays in light mode hence resulting in contrast issue causing the text to be unreadable

This problem goes away after toggling the theme from light to dark manually

hey can I work on this?

@dewanshDT ok

hey we can close this as this was the issue from docusarus and I've reported it there