Substantial revision

Great extension.  It's used by my team and I have suggestions for some significant revision.
I have an implementation of all the following items, but I'm cataloging them to find out the most approachable way to open PRs and request feedback.

[x] Add basic HTML tests, inspired by test_build_html.py from the Sphinx project.  My goal with this item is to prevent introducing regressions. PR: #33.
[ ] Implement the Domain API to enable the following improvements:
[ ] (Kind of a freebie from implementing the domain...) Create domain-specific xref targets like :command:fancytool install``.  These xref targets are valid within the project and can be referenced cross-project with intersphinx.
[ ] Add an optional command index of all commands and sub-commands in a docset.
[ ] Add an optional command-by-group index.  The proposal here is to add a :idxgroups: directive so that commands can be grouped in an index.  In the proposed tests, I used "ham on a stick" and "spam in a cone" as groups, but IRL, the groups are like "Core Services" and "Common Utilities."
[ ] Add a full_subcommand_name option so that when headings are printed for sub-commands, the fully-qualified command name is printed. My opinion is that this helps orient readers for projects that use deeply nested sub-commands.

I made a fake PR in my fork so you can skim the diffs.  If you are open to reviewing all the changes at once, that's fine by me, I just fear that could be unnecessarily challenging.  I can make one PR for the first issue.  I think the next four have to be one PR, though maybe I can make the domain and tests for xrefs into one PR and maybe break the two different indicies into separate PRs.
I'm not 100% sure, but I think that implementing the domain addresses https://github.com/ashb/sphinx-argparse/issues/11.  Refer to an example in a test file or some of the tests.
I hope the proposal is compelling.  Please let me know whether you are receptive to one "big bang" PR or which smaller ones you'd prefer. If I can clarify anything, please let me know.

@ashb , can you take a peek and let me know if my requests are compelling?  If you prefer step-by-step, please let me know what changes you'd like in the first PR.  Thanks very much.

Just seen this point, will take a look

@mikemckiernan That all looks amazing -- PR in what ever structure you think appropriate. Do we need to bump the minimum sphinx version at all?

@mikemckiernan That all looks amazing -- PR in what ever structure you think appropriate. Do we need to bump the minimum sphinx version at all?

The 4.3.2 version seems OK, and my org uses 4.5 at the moment.  I don't see a reason to bump the minimum Sphinx version at the moment.


Matrix bridge doesn't work

Please answer the following questions.
Which version of matterbridge are you using?
run matterbridge -version
version: 0.9.3 cd0a2be
version: 0.10.0-dev
Please describe the expected behavior.
It should login and work
Please describe the actual behavior.
Errors, different for release and git versions
Use logs from running matterbridge -debug if possible.
version: 0.9.3 cd0a2be:
INFO[2017-03-13T12:33:57+02:00] Enabling debug
INFO[2017-03-13T12:33:57+02:00] Running version 0.9.3 cd0a2be
INFO[2017-03-13T12:33:57+02:00] Starting gateway "mygateway"
INFO[2017-03-13T12:33:57+02:00] Starting bridge: matrix.bridge
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x91b71e]

goroutine 1 [running]:
github.com/42wim/matterbridge/gateway.(*Gateway).AddBridge(0xc4200a1090, 0xc420151d28, 0x3, 0x3)
        /go/src/github.com/42wim/matterbridge/gateway/gateway.go:47 +0x21e
github.com/42wim/matterbridge/gateway.(*Gateway).Start(0xc4200a1090, 0xc42008cd20, 0xc4200a1090)
        /go/src/github.com/42wim/matterbridge/gateway/gateway.go:68 +0x226
main.main()
        /go/src/github.com/42wim/matterbridge/matterbridge.go:55 +0x465

version: 0.10.0-dev:
INFO[2017-03-13T12:33:04+02:00] Enabling debug
INFO[2017-03-13T12:33:04+02:00] Running version 0.10.0-dev
INFO[2017-03-13T12:33:04+02:00] Starting gateway "mygateway"
INFO[2017-03-13T12:33:04+02:00] Starting bridge: matrix.bridge
INFO[2017-03-13T12:33:04+02:00] Connecting                                    module=matrix
DEBU[2017-03-13T12:33:04+02:00] &url.Error{Op:"Post", URL:"/_matrix/client/r0/login", Err:(*http.badStringError)(0xc420169b60)}  module=matrix
FATA[2017-03-13T12:33:04+02:00] Starting gateway failed &errors.errorString{s:"Bridge matrix.bridge failed to start: Post /_matrix/client/r0/login: unsupported protocol scheme \"\""}

Please add your configuration file
[telegram]
        [telegram.bridge]
        Token="mytoken"
        RemoteNickFormat="[{PROTOCOL}] <{NICK}> "

[matrix]
        [matrix.bridge]
        URL="https://matrix.org"
        Login="Bridge" #also tried @Bridge:matrix.org
        Password="mypass"

[discord]
        [discord.test]
        Token="mytoken"
        Server="[RU] Borderlands Community"
        RemoteNickFormat="**[{PROTOCOL}] <{NICK}>** "

[[gateway]]
name="mygateway"
enable=true
        [[gateway.inout]]
        account="matrix.bridge"
        channel="#that_is_linux:matrix.org"

        [[gateway.inout]]
        account="telegram.bridge"
        channel="-1001065669628"

        [[gateway.inout]]
        account="discord.test"
        channel="test_room_tlach"

matrix support only works in 0.10.0-dev
Seems like I made a mistake in my matrix sample config
URL should be Server
The following snippet should work
[matrix]
        [matrix.bridge]
        server="https://matrix.org"
        login="Bridge"
        password="mypass"

now it works, thanks


Sync git hooks when config file path changed

A patch to #17335.
Just like AppPath, Gitea writes its own CustomConf into git hook scripts too. If Gitea's CustomConf changes, then the git push may fail.

See:
https://github.com/go-gitea/gitea/blob/2cc7408b98a3f295f73a34e05a0a46b0c767c322/modules/repository/hooks.go#L78-L83


add parameter to Set api

We only have libcontainer.Container on Docker side, can't change config
in linuxContainer, pass config to libcontainer so we can change config of
container.
Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

Hi, Set command from Docker side is out, https://github.com/docker/docker/issues/11442
that PR needs this commit, can you help review? Thanks.

LGTM @crosbymichael @vmarmol ping

LGTM

LGTM


Error when Running node index.js

Hello,
I get this error whenever I run "node index.js".  Can you provide any insight?  And please let me know if you need more information.
PS D:\butler-cw-master> node index.js
2019-03-29T12:40:07.301Z info: Starting Qlik Sense cache warmer.
2019-03-29T12:40:10.337Z verbose: Starting loading of appid a099cb6b-53fb-4e70-b213-2832073b0c02
(node:7508) UnhandledPromiseRejectionWarning: TypeError: Converting circular structure to JSON
at JSON.stringify ()
at loadAppIntoCache (D:\butler-cw-master\index.js:211:57)
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:7508) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a
promise which was not handled with .catch(). (rejection id: 1)
(node:7508) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a
non-zero exit code.

I am getting the same error. Is there any way to resolve this?

I've tried,. but failed to reproduce this error.
I can only suggest you download the latest version from https://github.com/ptarmiganlabs/butler-cw/releases, a new version 2.2.0 was for example released today.
I however don't think that version includes anything that would fix the issue you are seeing.
I'd love to help, but you then have to provide as complete information as possible about your setup, versions of all software involved etc.
If something looks funky in your setup I'll of course let you know, but facts remain: The code works reliably here, and has been stable for at least a couple of years now. But there are of course a lot of variables that can and will affect things on your end.

Hi @mountaindude ,
Thanks for your response. I was able to resolve this error by passing the root certfificate as well for authentication while creating the engima session which seems to have been commented out in the code.
const configEnigma = {
        schema,
        url: SenseUtilities.buildUrl(urlConfig),
        createSocket: url =>
            new WebSocket(url, {
                ca: rootCert, // added root certificate here
                key: client_key,
                cert: client,
                headers: {
                    'X-Qlik-User': 'UserDirectory=Internal;UserId=sa_repository',
                },
                rejectUnauthorized: false,
            }),
    };

Very interesting!
I don't have a good explanation for this.
On my end things run just fine without referring to the root cert. Maybe I've imported into the keychain here on my development computer at some point... That could actually explain it.
Anyway - it's for sure a good idea to include the root cert, I'll update the code accordingly.
Cheers!

This issue was resolved in 2.2.1
https://github.com/ptarmiganlabs/butler-cw/releases/tag/v2.2.1


fn: enqueue

Cloud function triggered on new commits added to firestore to enqueue benchmark jobs on pubsub topic.

Not relevant any more. Tool a different approach.


result for each run

Can you please tell me how to get accuracy and other scores for each step? like for each parameter.
Thank you

I'm not sure I understand your question, could you clarify a bit in terms of what you want to achieve.

Closing here, feel free to reopen / create new, as needed.


Tab last/next in Mac terminals should switch tabs

tab next and tab last don't do anything in the Mac terminal.
For a quick look at the code, it looks like the tag is implemented, so I suspect it just needs some custom keybindings.
https://github.com/knausj85/knausj_talon/blob/master/apps/platforms/mac/terminal/terminal.talon#L8
Latest knausj master, August 2nd 2021
Talon 0.2.0-8
Terminal version 2.10 (433)

As I understood it comes from https://github.com/knausj85/knausj_talon/blob/12c2e6830a74a7e5cff6138969368642a0a567a6/code/platforms/mac/app.py#L15 and https://github.com/knausj85/knausj_talon/blob/12c2e6830a74a7e5cff6138969368642a0a567a6/code/platforms/mac/app.py#L19, but according to official doc it is different shortcuts for standard Mac terminal https://support.apple.com/guide/terminal/keyboard-shortcuts-trmlshtcts/2.11/mac/11.0

Yeah it's ctrl-tab when I do it manually like the Mac doc you found says 👍
Should be an easy-ish fix to overwrite those two commands in the terminal context for Mac.

@xavier630 I found there some discussion about refactoring of terminal in general https://github.com/knausj85/knausj_talon/issues/505
Maybe it makes sense to follow that issue as tabs implementation also was mentioned there.

As I understood it can be implemented as a part of #514.
At least according to the new "generic style".

PR #514 will only implemented kde-konsole according to the new "style". For tracking which terminals are in knausj but not yet migrated to the new style, #562 was created.
This branch contains an implementation in the new style for the apple terminal, however it is not tested since I do not have a mac.

Thanks to @xavier630, who tested my apple_terminal implementation, I was able to implement "tab next" and "tab (last|previous)" the right way. I will open a (drafted) PR where you can comment/add changes you want to have.

I feel this issue is best resolved by changing the generic tab shortcut. It's not an issue unique to the Terminal. I submitted a PR that resolves it in Terminal, in addition to identical issues in Finder and Safari 😄


add Recognize anything test code. Image -> tag -> SAM

Running command:
python automatic_label_ram2tag_demo.py
--config GroundingDINO/groundingdino/config/GroundingDINO_SwinT_OGC.py
--ramtag_checkpoint ./Recognize_Anything_Tag2Text/pretrained/ram_swin_large_14m.pth
--grounded_checkpoint groundingdino_swint_ogc.pth
--sam_checkpoint sam_vit_h_4b8939.pth
--input_image assets/demo9.jpg
--output_dir "outputs"
--box_threshold 0.25
--text_threshold 0.2
--iou_threshold 0.5
--device "cpu"

RAM model has alreadly been added into Grounded-SAM, so we're going to close this PR~


Describe which permissions this action needs

Hi,
Could we please document which permissions that's needed to be able to use this action?
Docs for this is here:

https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs

For example, running linters such as checkov on workflows using what's suggested in this action might give warnings like:
github_actions scan results:

Passed checks: 0, Failed checks: 1, Skipped checks: 0

Check: CKV2_GHA_1: "Ensure top-level permissions are not set to write-all"
	FAILED for resource: on(Reviewdog suggester)
Error: 	File: /.github/workflows/reviewdog-suggester.yaml:0-1

Is this the way to go?
permissions:
  contents: write

Or could we do with even less permissions?

How do we verify though? Every repo may have different restrictions on permissions like this one prevents a write-all permission. I wonder if we can do anything about it

we don't need contents: write, contents: read is enough.
To write review comments and checks, we need some extra permissions:
permissions:
  contents: read
  checks: write
  issues: write
  pull-requests: write


imageannotation hyperlink added

@Malavi1 there is no existing issue that fixed by the PR!
You need to create the issue first > get assigned > raise PR.

@Malavi1 there is no existing issue that fixed by the PR!
You need to create the issue first > get assigned > raise PR.

But it fixes issue https://github.com/CircuitVerse/CircuitVerse/issues/3623
I forget the add the issue number it is going to fix.
This was raised by @Prerna-0202 and after I get assigned the issue I started working


TRY201 false positive

This code raises TRY201.
try:
    pass
except Exception as e:
    print(e)
    raise e

TRY201 wants us to simplify to:
try:
    pass
except Exception:
    print(e)
    raise

but we use print(e) so the rule should not be raised.
Ruff version: 0.0.231

I'll fix this now.

I could work on it.

Go for it!

Ok

I think it's correct to flag this, but the e shouldn't be removed... Since this is valid:
try:
    1/0
except Exception as e:
    print(e)
    raise

Oh wait, sorry, this might not be a bug? I thought you were saying that we were trying to autofix it to the broken version.

Yes, there is no bug.


Occurrence instance color

Let's say you have this code:
user.run()
user.click()
user.done()

you click somewhere on the first user, click CMD+D to select the first one, CMD+D and CMD+K to skip the second, then CMD+D to select the third. You end up with this:

As you can tell, other than the (blinking) cursor, there's no way to tell which are actually selected. It would be great if there was a setting to set the color for the other selected items.
Strangely enough, if the window is inactive, it does show correctly:

Maybe I'm missing something

Selection highlight overrides the selected text highlight in this case.


render(component:)

I personally would be happy with some render(ComponentClass) syntactic sugar, but it seems to me the most Railsy thing to do would be that the formal method be render(component: ComponentClass) in line with the :partial, :template, :inline, :file, :plain, :html, and :body options.

Good point! This is something we recently considered internally, but decided to wait on until we try to upstream. I wouldn't be surprised if this ended up being the syntax we go with 😄

Rails 6.1.0.alpha now supports passing an object as the first argument to render: https://github.com/rails/rails/pull/36388.
I've updated this repository and the gem to work with this syntax, passing an instance of the component in.
I actually think the component: syntax is a nicer way to go here, if not a bit verbose. I'm guessing this will come up on rails/rails as we move towards upstreaming the gem for 6.1.


Fade at the End

Is there a way to fade the music out at the end of the file like in Pygame Mixer. And if there's no way is it possible for you to code it? Otherwise i can try on my own, but i think it would be faster if you do it.
Btw i'm a big fan of this module, i searched for sth like this for like 2 days and then i found this.
Thanks in advance,
Johannes

I'm not a contributor, but the feature you requested is not built in, as far as I know. You could try something like this:
import threading
import time

from just_playback import Playback

class PlaybackFadeable(Playback):
    def _fade(self, start_fade, init_vol, dest_vol):
        self.set_volume(init_vol)
        while True:
            if not self.active:
                break

            secs_remaining = self.duration - self.curr_pos
            if secs_remaining < start_fade:
                self.set_volume(
                    (init_vol - dest_vol) * secs_remaining / start_fade
                    + dest_vol
                )
                time.sleep(0.1)  # might not need this, IDK
        self.set_volume(init_vol)

    def play(self, start_fade=2, init_vol=1, dest_vol=0):
        super().play()
        p = threading.Thread(
            target=self._fade,
            args=(start_fade, init_vol, dest_vol),
            daemon=True,
        )
        p.start()

This helps me a lot, thank you. I had the same idea but didn't deal with the module enough.

Hi everyone,
Support for fading in/out would be great to have in the next release.
Any ideas on how the API could look like ??

How about having them as (probably keyword-only) arguments in the load_file method? Another option would be to have two new methods, set_fade_in and set_fade_out ... but it would have to be made clear that calling either method would not effect already playing audio (i.e. previous calls to play).


Gunicorn multiprocessing - worker count

Great extension. Thanks!
I'm curious about number_of_gunicorn_workers():
Why is ([number of cores] * 2) + 1 used as default? Won't this create more than twice the number of loops that's actually needed?

Closing this due to inactivity :+1:


unc0ver 3.5.3 autoreboot

Describe the bug
autoreboot
To Reproduce
Steps to reproduce the behavior:

start jailbreak
2.autoreboot

Device (please complete the following information):

iOS Version: 12.4
iOS Device: iPhone 7 Plus
unc0ver Version: 3.5.3

use 3.5.2 version is ok

How do I get version 3.5.2 on my iphone 7? New to this

How do I get version 3.5.2 on my iphone 7? New to this

https://github.com/pwn20wndstuff/Undecimus/releases/download/v3.5.2/Undecimus-v3.5.2.ipa

Sorry but can you explain what to do after downloading the link because I'm not able to do anything

Sorry but can you explain what to do after downloading the link because I'm not able to do anything

use cydia impactor install this ipa to your phone

This issue is also happening on an iPad Air:
iOS Version: 12.4
iOS Device: iPad Air
unc0ver Version: 3.5.3

Hi, same with my iPad Air 12.4. I tried many times with 3.5.0, 3.5.2 & 3.5.3 and it failed (reboot while jailbreaking).


Annotation not set for Lambda (not using Express)

Hi.
In my lambda handler, I'm trying to set an annotation on a subsegment. I'm not using express.
const sub = AWSXRay.getSegment().addNewSubsegment('my-handler')

// Rest of my code...

sub.addAnnotation('response', JSON.stringify(response))

I can't see this set anywhere though. What am I missing?
Thanks in advance.

Hi @chearmstrong,
There are (at least) two possibilities here. The first is you are not closing your subsegment with sub.close(error) - which you must to guarantee all its attributes are included. The other is that this is a similar issue to #219 in which there are asynchronous functions in your code which cause the segment for the function to be ended prematurely, before your addAnnotation is reached. Please provide some more repro code to help determine which root cause is more likely.

Thanks for the reply @willarmiros - here's more code. I suspect it's the async thing. Is there a workaround for this atm?
const handler = async event => {
  return AWSXRay.captureAsyncFunc('my-handler', subSegment => {
    const timer = process.hrtime()

    subSegment.addAnnotation('body', JSON.stringify(event.body))

    return handleEvent(event)
      .then(response => {
        subSegment.addAnnotation('response', JSON.stringify(response))
        subSegment.close()

        return getSuccessResponse(response)
      })
      .catch(error => {
        subSegment.close(error, false)

        return getErrorResponse(error)
      })
  })
}

Hi @chearmstrong,
What's interesting is that I could not reproduce your issue with some generic handleEvent async function. Maybe it's because mine was a little too fast, but not sure exactly.
What happens in general in these types of situations is execution hits an async function like handleEvent and that function executes and returns quickly, as it should, because its callback (.then in this case) will eventually be added to the task queue to be executed later. However because handleEvent has finished executing, the top-level handler has reached the end of its execution and can now return its promise. The Lambda function correctly waits for this promise to be resolved before terminating, however I believe the X-Ray segment is sent before the promise is resolved, meaning the actions like adding that annotation are not reflected in the prematurely sent segment.
I hope this makes some sense. We are investigating this issue with Lambda currently. I will update this thread as we learn more.

Hi @chearmstrong,
Are you still experiencing this error? Here's the code I'm using to reproduce on Lambda, with node 12.x and X-Ray SDK 3.0.0:
const AWSXRay = require('aws-xray-sdk');

exports.handler = async event => {
  return AWSXRay.captureAsyncFunc('my-handler', subSegment => {
    subSegment.addAnnotation('body', JSON.stringify(event));

    return handleEvent(event)
      .then(response => {
        console.log('response=' + response);
        subSegment.addAnnotation('response', response);
        subSegment.close();

        return response;
      })
      .catch(error => {
        subSegment.close(error, false);

        return error;
      });
  });
};

async function handleEvent(event) {
  let promise = new Promise((resolve, reject) => {
    setTimeout(() => resolve("done!"), 1000);
  });

  let result = await promise;
  return result;
}

Perhaps the use of cls-hooked in 3.0.0 helped? I've consistently seen the correct annotation added when trying to reproduce the error, so please let me know if you can get the error reproduced more consistently.

Just been checking this one with v3 seems to be working as expected now (on our staging env at least). Will update prod with sees a lot of requests and monitor it - will come back if have an more issues.

@willarmiros I think I'm seeing this too (not using express, but rather, using lambda-api to handle request routing. I'm just trying to add a top level annotation as follows, but nothing is being recorded:
import AWSXRay from 'aws-xray-sdk';

// lambda-api route handler
api.get('/some_route', async () => {
    const segment = AWSXRay.getSegment();
    segment.addAnnotation('myAnnotation', 'some_value');

    return someAsyncFunctionCallThatHitsOurDB();
});

Ah, is this the whole 'top level segment in lambda is immutable' issue? If so, would be nice if there was an easy way to generate a custom subsegment for the lambda too. (In this case I might be able to do it with lambda-api middleware...


ip_service_tag

New security standard will require all IPs to be tagged.
Please note this IP tagging system is different from general resource tagging.

@LiliDeng LGTM


Farsight swapping between "Standby" and "Sync" every ~second

I am having Problems with Farsight, it flashes between Standby and Sync, everything else is working fine. Even when its not connected to LCU & Server it is still doing the same thing. It starts before LiveEventsor Ingame is synced. After quitting pectate I get an Error Message

I accepted the Pop-up with the Memory warning, did I need to setup anything else for Farsight to work?

Have the same issue.
is this also the reason the new overlay's timer and stat updates don't work? I guess they are based on the farsight info?

i'm having the same issue, is there a fix for this overlay's timer and stat not updating problem ?

Same issue here. Tried a full clean install of the Toolkit and the Observer Tool to be sure.

The native farsight modulel had an update recently that should fix it, i guess that requires a new release for the Observer Tool.

Farsight swapping between "Standby" and "Sync" was/is due to how farsight works and means it is not working correctly, after the newest update to the farsight module and therefore also an update to the observer tool this issue should be fixed. Please feel free to test it and come back to us.

Works as intended now!
Thanks for the release!


Extend CycloneDX presenter with dependency graph

#67 brings about the Cyclone DX presenter. We should extend out select catalogers to capture inter-dependencies for each package discovered. This data can better be leveraged to add the Dependency Graph Extension to the Cyclone DX presenter (https://cyclonedx.org/ext/dependency-graph)

Fixed by #706

Should we close this now?

I believe so, merged in https://github.com/anchore/syft/pull/768


Examples from the Bible stories reference links don't load anything.

v1.1.0 build 2747aff QA
In tW article under Examples from the Bible stories - when clicking on any reference links for a split second it shows loading but never loads anything.

@kintsoogi thanks, I'll retest and provide examples.

@kintsoogi I've retested with the https://release-v2-1-0--gateway-edit.netlify.app/ link v2.1.0 build f43bad3 QA and here’s the steps I took.

Opened  Matthew  1:1
Went to translationWords Article and scrolled down to Examples from the Bible stories:
when clicking on any reference links for a split second it shows loading but never loads anything.

https://github.com/unfoldingWord/gateway-edit/assets/82067076/8998de7b-5bb2-490b-af71-7ace6db94bbf

I also discovered that above Examples from the Bible stories: the links under Bible References: gives 404 error.

Need to work on translation-help-rcl
find the URL/Links path is wrong (When we click on the links it directs us to
GET | https://git.door43.org/api/v1/repos/unfoldingWord/en_obs-tn/contents/content/46/06.md?ref=master.
there is no directory in - door43

Connected RCL's -

translation-help-rcl
(for card content)
resource-workspace-rcl
(for card styles, dragable cards etc )
single-scripture-rcl
(for scripture card)

v2.1.0 build ad1df2f QA Examples from the Bible stories reference links are loading content from OBS. Looks good.

Links are loading OBS content. Looks good to me and tested with v2.1.0 build ad1df2f QA.
Private Zenhub Image


memberofteam

Is there a way that we can check if a user is a member of any team?
i have a dropdown menu that users can switch teams from, the problem i have is that if the user is not a member of a team it just displays a laravel error.
I created the following but it still errors or doesnt show the correct information:
@if(is_null(Auth::user()->teams))
                            Select Site
                        @else
    			            @if(Auth::user()->currentTeam->name)
                                <i class="fa fa-users"></i> {{ Auth::user()->currentTeam->name }}
                            @else
                                Select Site
                            @endif
                        @endif

If you have a teams relationship defined on your user model, you could do something like this
Auth::user()->teams->isNotEmpty()

I'll close this since its an old issue. Feel free to open another if you still need help.


Content Publishing - Guidelines (02 IBM.com Style Model overview page MVP version)

User Story

As an IBM.com Library pattern adopter,

I need a way to learn more about the (Name of the Pattern here) pattern

so that I can utilize the pattern to my day to day work.

Additional information
Based on the IBM.com Style Model page content and IA discussion and work session done in #37, this task is to publish the MVP version of the content agreed on with Wonil.
https://github.com/carbon-design-system/digital-design-ideation/issues/37
Acceptance criteria

[ ] Content crafting is done.
[ ] Asset image creation is done.
[ ] Page design is done.
[ ] Publishing with markdowns is done.
[ ] QA'd by Jenny
[ ] Web page preview is approved by Linda, Jeff and Wonil
[ ] Web page is published

@JennySanchez Can I close this issue or should I move it to sprint 20-17?  Let me know.  Thanks.

The content is here: https://ibm.box.com/s/hno90qbj7x0kkhrla07328naa7sbbd64
And the design/prototype is here: https://ibm.invisionapp.com/share/EJO0GLC65MU#/319702416_IBM-Dot-Com--style-Models--home
And the Sketch file for the design: https://ibm.box.com/s/38d5p6bhz5lq9r73uh9gh8qr2qe7a8wy

Preview link: https://ibmdotcom-library.s3.us-south.cloud-object-storage.appdomain.cloud/deploy-previews/650/index.html
PR: https://github.com/carbon-design-system/ibm-dotcom-library-website/pull/650

Feedbacks

Case: Style model
What are the next steps?
Proper apostrophe
Contents -> content
Show numbers in the principles
Text size and spacing heading size too small
Contribution should be feedback?
List intro is weak
Line spacing in the diagrams
Main contents -> main content

@JennySanchez Hi Jenny, It looks like the PR merge is blocked.  Can you take a look at it?  I am moving this issue to the next sprint so that I can close out 20-19.
cc;  @wonilsuhibm

@RobertaJHahn The merge is blocked bc there are two change requests and needs to be reviewed by four reviewers.
@wonilsuhibm can you take a look at the two change requests? Is it intentional to have four reviewers for this PR? If it is, once the updates are done, you might wanna ping all reviewers to review it asap so we can get the content live asap.


ZEPPELIN-335. Pig Interpreter

What is this PR for?
Based on #338 , I refactor most of pig interpreter. As I don't think the approach in #338 is the best approach. In #338, we use script bin/pig to launch pig script, it is different to control that job (hard to kill and get progress and stats info).   Besides that I implement another interpreter type %pig.query to leverage the display system of zeppelin. For the details you can check pig.md
What type of PR is it?
[Feature]
Todos

Syntax Highlight
new interpreter type %pig.udf, so that user can write pig udf in zeppelin directly and don't need to build udf jar manually.

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-335

How should this be tested?
Unit test is added and also manual test is done
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

ping @Leemoonsoo @bzz @felixcheung @abajwa-hw @vgmartinez @maparco @smartinsightsfromdata please help review.

Is it possible have a non-Tez version of Pig and would it work with this interpreter?

Yes, it supports local, mapreduce and tez engine.

Could you add documentation?

@felixcheung please check pig.md

@zjffdu Thanks for your great work! 👍
While I was quickly looking through pig.md, found some need-to be-changed(?) parts. So I've just pushed my commits to your Zeppelin repo instead of leaving multiple review comments one by one in here. Could you check https://github.com/zjffdu/zeppelin/pull/2 out? :)

Thanks @AhyoungRyu for the help, merged.

how about LICENSE file?

@felixcheung Thanks for the review, license is added and comments are addressed.

Thanks, comment replied, in particular, this

Do you intent to include the TODO in this PR, or as a follow up?
Otherwise I think this is good to go - you can remove the WIP in the title and I'll merge if there is no more comment.

Thanks @felixcheung , Do you know how to run SELENIUM test ? I follow the instruction on READM.MD, but fails
Zeppelin comes with a set of end-to-end acceptance tests driving headless selenium browser

```sh
# assumes zeppelin-server running on localhost:8080 (use -Durl=.. to override)
mvn verify

# or take care of starting/stoping zeppelin-server from packaged zeppelin-distribuion/target
mvn verify -P using-packaged-distr

Try the commandline from Travis?
https://github.com/apache/zeppelin/blob/master/.travis.yml#L75

I did it, but fails with the following error:
*** RUN ABORTED ***
  java.lang.RuntimeException: Unable to load a Suite class that was discovered in the runpath: org.apache.zeppelin.AbstractFunctionalSuite
  at org.scalatest.tools.DiscoverySuite$.getSuiteInstance(DiscoverySuite.scala:84)
  at org.scalatest.tools.DiscoverySuite$$anonfun$1.apply(DiscoverySuite.scala:38)
  at org.scalatest.tools.DiscoverySuite$$anonfun$1.apply(DiscoverySuite.scala:37)
  at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
  at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:244)
  at scala.collection.Iterator$class.foreach(Iterator.scala:727)
  at scala.collection.AbstractIterator.foreach(Iterator.scala:1157)
  at scala.collection.IterableLike$class.foreach(IterableLike.scala:72)
  at scala.collection.AbstractIterable.foreach(Iterable.scala:54)
  at scala.collection.TraversableLike$class.map(TraversableLike.scala:244)
  ...
  Cause: java.lang.RuntimeException: Could not initialize any driver
  at org.apache.zeppelin.AbstractFunctionalSuite.getDriver(AbstractFunctionalSuite.scala:64)
  at org.apache.zeppelin.AbstractFunctionalSuite.<init>(AbstractFunctionalSuite.scala:39)
  at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
  at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
  at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
  at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
  at java.lang.Class.newInstance(Class.java:442)
  at org.scalatest.tools.DiscoverySuite$.getSuiteInstance(DiscoverySuite.scala:69)
  at org.scalatest.tools.DiscoverySuite$$anonfun$1.apply(DiscoverySuite.scala:38)
  at org.scalatest.tools.DiscoverySuite$$anonfun$1.apply(DiscoverySuite.scala:37)

@Leemoonsoo @bzz Do you know how to run SELENIUM test locally ? Thanks

@felixcheung Test is passed, I suspect the selenium test is flaky so failed last time.

@zjffdu thanks for working on this interpreter! Looks great

I think @AhyoungRyu has a comment?

@felixcheung @AhyoungRyu 's comment is addressed.

merging if no more comment


chore: Introduce prettier-eslint and upgrade eslint related packages

Enhanced from #875.

Introduced prettier-eslint.
Upgraded to eslint@5 and related packages. Notice eslint-import-resolver-babel-module requires babel@7, so it and its peer dependency babel-plugin-module-resolver are kept the same.
In order to not change too many codes, prettier is locked too. And some eslint rules are updated or turned off temporarily.
.eslintrc is replaced by .eslintrc.js as it is recommended to deprecate the non-extension configuration file.

Tips,

Double asterisks are only useful in Bash >=4 with globstar option on.

Coverage remained the same at 48.688% when pulling b370d1432fd1186ab5ffcc92b530d2bf7ddeb44a on chinesedfan:prettier-eslint into a2aa221cf78f5e371bd4c4c4012d48d83b75fc36 on gitpoint:master.


Toggl lagging on MAC - long sync time - Toggl unusable during sync

I am experiencing long sync times when updating task info or starting new task.
An umbrella appears and I cannot interact with the app. It takes about 30 seconds (sometimes more) and it is really annoying. It started couple months ago when app got "locked" from time to time for couple of seconds but as time went it got much worse. Now it appears many times a day for longer times.
I am not sure if it is because I have more tasks logged or if it is something on my Mac. It doesn't depend on network connection quality - I experienced this on both slow and fast connections.
I am running OSX 10.11.6 (15G31).

Please try to clear the local cache. https://support.toggl.com/clearing-the-local-data-from-toggl-desktop/#mac
With the next release there will be automatic cache cleaner that will avoid these kinds of slowdowns in the future.

Thanks! That worked. The only small disadvantage is that you get logged off during the cache clearing.
And thanks that you are working on making this automated.


Infinite tunnel when obstructed by block while turtle joining the queue

I have found out that if the turtle encounters a block while trying to get to the end of the staging area queue it will mine the block obstructing, but then continuously try to tunnel in a straight line. I believe I had to reboot and reset the affected turtle in order to get it to return home.

Huh, I'll take a look at that. It was fine after a reboot?

Huh, I'll take a look at that. It was fine after a reboot?

I'm not sure if I am having the same problem or no but when I am adding a turtle I face it towards the disk drive run the code and link it but once I add the one piece of coal and refuel it will move turn to the left and tunnel for ever.

I'm not sure if I am having the same problem or no but when I am adding a turtle I face it towards the disk drive run the code and link it but once I add the one piece of coal and refuel it will move turn to the left and tunnel for ever.

I'm not sure if I am having the same problem or no but when I am adding a turtle I face it towards the disk drive run the code and link it but once I add the one piece of coal and refuel it will move turn to the left and tunnel for ever.

Are you sure that you changed where the base is?

I'm not sure if I am having the same problem or no, but when I am adding a turtle, I face it towards the disk drive, run the code and link it, but once I add the one piece of coal and refuel it will move turn to the left and tunnel forever.

Are you sure that you changed where the base is?

I'm not sure what you mean.

@ramonGonzEdu Yes I did change it Can you think of anything else.
Thank you

Did you possibly mess up when you made the gps system, giving incorrect coordinates?


A

!merge

!squash

!check

I thought @deiwin went mad when read the email notification

I was trying to show off the bot ... And then it failed.

2017/03/23 00:29:11 Checking for fixup commits for PR salemove/github-review-helper#32.
2017/03/23 00:29:11 Getting commits for PR salemove/github-review-helper#32 failed. Received an unexpected head with SHA of b1d3358030c47735c252ad91bf036a46d9ab5fb3.. Trying again.
2017/03/23 00:29:11 Getting commits for PR salemove/github-review-helper#32 failed. Received an unexpected head with SHA of b1d3358030c47735c252ad91bf036a46d9ab5fb3.. Trying again.
2017/03/23 00:29:11 Getting commits for PR salemove/github-review-helper#32 failed. Received an unexpected head with SHA of b1d3358030c47735c252ad91bf036a46d9ab5fb3.. Trying again.
2017/03/23 00:29:11 Responding to the HTTP request with:
2017/03/23 00:29:11 Error: Getting commits for PR salemove/github-review-helper#32 failed. Received an unexpected head with SHA of b1d3358030c47735c252ad91bf036a46d9ab5fb3.

So the same old race caused it to fail. I'll have to add a delay between the retries.


Add AKS as a meta issuer

Summary
Add AKS as a meta issuer, now that AKS OIDC Issuer feature is available.
This was tested using an AKS cluster with the feature EnableOIDCIssuerPreview following instruction here.
Signed-off-by: Nghia Tran<EMAIL_ADDRESS>Ticket Link

Fixes https://github.com/sigstore/fulcio/issues/231
Release Note

Config change to support AKS per-cluster issuer URLs.

FYI, we validated this in our staging environment earlier this week.


Initial value assigned to r.

In capsnet_em.py em_routing():
r = tf.constant(np.ones([batch_size, caps_num_i, caps_num_c], dtype=np.float32) / 32)

should this be:
r = tf.constant(np.ones([batch_size, caps_num_i, caps_num_c], dtype=np.float32) / caps_num_c)

as in paper:
∀i, c: Ric ← 1/size(L + 1)

Hi, @gyang274
Thanks for spotting a mistake! The code is fixed as you suggested. @www0wwwjs1
Regards!


data format

In train directory
train/
img/
0/
gt/
0/
in 0/ need I to put one image or batch?
In my case, I have 6 classes. How to look groundtruth mask image?  0,1,2,3,4,5 in the last channel, in all channels, certain colors like in palette_refer.png

Got the same problem!

What is the correct folder structure for the data necessary to run this model?


Amplify invoke function

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

** Which Category is your question related to? **
function
** What AWS Services are you utilizing? **
none
** Provide additional details e.g. code snippets **
What does 'amplify function invoke FUNCTION_NAME' do other than running node script? Do I need it when testing function?
My current usage:

Run script that invokes all the functions in the backend with custom events
Inspect returned values from the handlers
Check which executions failed

Now, I just want to know if I could just execute handlers directly without running them through 'amplify function invoke'. It would make it much simpler to inspect the returned values and I could also avoid hacking optional questions about handler function file and handler function to invoke.
Are there any plans on the roadmap to upgrade testing of functions so that it is not limited to a single function with a single event.json? It would be extremely helpful especially since the @function directive was added.

@sceptyk Yes, you could drectly call the handler function. The CLI just wraps around the event object from the specified file - event-xyz.json and then passes it to the handler to test the function.
We do support multiple event files. Please take a look at - https://github.com/aws-amplify/amplify-cli/issues/1187#issuecomment-512544471 for more details around it.

@kaustavghosh06 Thanks for the reply, it answers my question. I was actually still on the older version of cli and didn't notice you can use multiple event files. Thanks for pointing this out.


Social Media Cards

https://github.com/delucis/astro-og-canvas#readme
https://github.com/shepherdjerred/astro-opengraph-images

Both plugins fail due to an incompatibility regarding SSR:

https://github.com/delucis/astro-og-canvas#readme

01:12:20 [ERROR] [vite] x Build failed in 159ms
[commonjs--resolver] [plugin vite:resolve] Cannot bundle Node.js built-in "node:fs/promises" imported from "node_modules/astro-og-canvas/dist/generateOpenGraphImage.js". Consider disabling ssr.noExternal or remove
 the built-in dependency.

https://github.com/shepherdjerred/astro-opengraph-images

24:58:49 [ERROR] [vite] x Build failed in 1.23s
[commonjs--resolver] [plugin vite:resolve] Cannot bundle Node.js built-in "fs" imported from "node_modules/astro-opengraph-images/dist/util.js". Consider disabling ssr.noExternal or remove the built-in dependency.

Searching for a fix / alternative later :)

I will try to contribute the static render capable og images

This can now be solved rather trivially by using the new image-service.
PR should include OpenGraph tags that reference a dynamic URL.
This will required #187 to be closed first.


fix: extend max retries for submitting package

:tada: This PR is included in version 3.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


QRScanDialog$WebcamDependencyException: Missing Dependency Error Doesn't Display Link

On Windows 11 Home v23H2, this error references a link, but the link is not displayed.

sparrow.log:
2024-09-29 12:14:38,299 ERROR [JavaFX Application Thread] c.s.s.c.FileImportPane [null:-1] Error importing QR
com.sparrowwallet.sparrow.control.QRScanDialog$WebcamDependencyException: Your system is missing a dependency required for the webcam. Follow the link below for more details.

[https://sparrowwallet.com/docs/faq.html#your-system-is-missing-a-dependency-for-the-webcam]

source:
https://github.com/sparrowwallet/sparrow/blob/1bbc586cd616e46c6ebd9615fb0e616ae44fb98c/src/main/java/com/sparrowwallet/sparrow/control/QRScanDialog.java#L143
I was able to fix the problem by installing the security package linked from that FAQ.
I think this is worth investigating, because some users may be comfortable with downloading and installing a Microsoft security patch (thus working through this independently), but not tracking down the link in source code.
I'd be interested in working on a PR, but not sure what the contribution policy is. If that's agreeable, just let me know :)

Thanks for the report. This is an issue which unfortunately only presents itself occasionally (ie on certain systems). The right place to look would be here.
I'm happy to accept small PRs, or even just a description of what's causing the issue.

I proposed an update in [#1512]. Two main issues I learned about:

The calculation for width wasn't handling newlines and URLs. In making changes for the second item, I found that the inherent javafx wrapping seems to be appropriate, but I don't have the (practical) ability to test this on Mac or Linux.
The referenced URL wasn't rendered as a link, because JFX Label doesn't do that automatically. So, the proposed change is to find URLs and create JFX Hyperlink objects from them, then join with surrounding Label objects for regular text.

This seems to work within the modal in question, including for details sections without links.


How could I set a specify cell's color in a worksheet ??

having same question. only found this fork: xlsx-style.
https://www.npmjs.com/package/xlsx-style
so, if the feature is not supported yet, should it better be documented?

@hieuhtse05008 @kl23
Actually it is possible. Let's assume you have a cell A1. In order to fill with with some color you can use the following line:
A1.s = {fill:{fgColor: {rgb:"86BC25"}}};
Of course you can define your own color between the quotes.

Checkout this package - has styling and all formatting features
https://github.com/natergj/excel4node


Improve docs and fix heroku logo

Link for the logo was wrong. It wasn't pointing to the real png.
I also saw some parts of the docs that could be better written.
Please review:

[ ] @albertein

Looks great :)


need help with eraser tool

hello, can someone help me to add border to eraser tool, so i want to see border of brush when i am erasing

In the latest version(2.9.0), There’re methods called ‘renderBegan(from:on:)’, ‘renderMoved(to:on:)’ and ‘renderEnded(at:on:)’ in class Brush. These methods are called when touches ware began, moved or ended. You can add an image with frame of your eraser and update its position in these methods.


chore(view): migrate seed browser to plugin views

This PR aims to migrate Seed Browser to plugin views from dendron-next-server. Came across it while deprecating next server.
Dendron Extended PR Checklist
Code
Basics

[ ] code should follow Code Conventions
[ ] circular dependency check: make sure your code is not introducing new circular dependencies in plugin-core.  See Avoiding Circular Dependencies.
[ ] sticking to existing conventions instead of creating new ones

eg: if configuration for utilities are already in one module or package, add future utilities there as well

Extended

General

[x] check whether code be simplified
[x] check if similar function already exist in the codebase. if so, can it be re-used?
[x] check if this change adversely impact performance

Operations

[x] when shipping this change, will it just work or will it introduce additional operational overhead due to complicated interface or known bugs?

Architecture

[~] check if code is introducing changes on a foundational class or interface. if so, call for design review if needed

eg: making changes to DNode

Instrumentation
Basics

[~] if you are adding analytics related changes, make sure the Telemetry docs are updated

Extended

[~] can we track the performance of this change to know if it is successful?

eg: see usage for export pod

Tests
Basics

[ ] Write Tests
[ ] Confirm existing tests pass
[x] Confirm manual testing
[ ] Common cases tested
[ ] 1-2 Edge cases tested
[ ] If your tests changes an existing snapshot, snapshots have been updated

Extended
NA
Docs
NA
Close the Loop
Extended
NA

can we not load the images until the page is ready? (low priority since the seed browser will need some re-work now that we have self contained vaults). if you can do it in <30min, feel free, otherwise ignore it

gave it a quick try by enclosing it in useEffect and adding document.readyState as dependency, didn't work out. Will try again in a follow-up PR

gave it a quick try

Let's defer it - usage of this view is very low; let's just try to get across the line of deprecating next-server


Improve searching with Lucene

Improve speed of the searching with Lucene.
Note that it is slow to search when the first time searching or after the repository was modified because the processing contains indexing.
The index data will store in $GITBUCKET_HOME/tmp/_index directory.

:+1: Searching is pretty terrible with a repo with lots of files.
Could it perhaps update the index on a push?

How can I fix this failed?
Only push with any changes?

We are planning embedded Elasticsearch integration for full-text search.

@takezoe
I am looking forward to the full-text search!


added testing for endpoints

Description
PR for the accidental push to master!
Added endpoint test code for author content, content library, and user.
Type of change
Please delete options that are not relevant.

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

Change Status

[ ] Complete, tested, ready to review and merge

How Has This Been Tested?

[ ] Test A

Checklist

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

Nice we have testing !


Enhance request of transactions in JSON with date ranges

Contact Details
<EMAIL_ADDRESS>Issue category
Enhancement
Issue description
Currently, the API requires separate requests for each day of transaction data. This approach leads to excessive API calls and increased server load. In order to improve the API, the requests should be enabled for a specific date range and supported by an appropriate pagination method to limit processing time.
Although, the query parameter and pagination of the transaction endpoint have been already adjusted, the exact behavior should be defined and open questions clarified.
Issue reasoning
This Requirement was brought up within the bLink Multibanking initiative and by several existing and new participants to improve efficiency and performance. Using date-range requests reduces the number of API calls, making interactions more efficient and reducing server load.
Request
Date Parameter

The query parameters are already adjusted to "dateFrom" and "dateTo". However, the usage of the parameter and sorting of data should be more concise. Therefore, either the description of the parameter or the example value should be adjusted.

Pagination
The current pagination used is a combination of cursor-based and offset-limit pagination with links in the response that require the server to process all data.
Cursor-based pagination would be appropriate for the API and the following adjustments could be agreed on:

Query Parameter "cursor": Define in the description, which attribute should be used as cursor
Query Parameter "limit": Define a maximum and default number of entries to be returned
"linksPagination" in Response: Remove the Pagination-Links in the response and add a link to the next page as Response-Header

Open Questions

Currently, transaction list of the current day is requested, by sending the request without a date. Should intraday data be included, when querying date ranges or clearly excluded?
Status Code 204 would not be required anymore for the transaction endpoint as a list of transactions should always be returned?
Status Code 202 would still be used, in case data is requested for a booking day, that is not completed yet?

Proposed change
Date Parameters
- Change "dateFrom" example value from 2018-04-13 to 01.01.2023
- Change "dateTo" example value from 2018-04-13 to 31.12.2023

Pagination
- Change "cursor" description from "Optional parameter for pagination. An opaque string value used for pagination"
to
"Optional parameter for pagination. An opaque string value used for pagination.
Use EntryId as the cursor reference"
- Change "limit" value to max. 50 and default 20
- Add Response-Header "X-Next-Cursor" with description "An opaque string value, or an empty string if there are no more results."

Affected API's
AIS
Code of Conduct

[X] I agree to follow this project's Contribution Guidelines

Meeting Notes 15.8.24
@simonmeyerm elaborates the change request. It was stated that the parameters "date form" and "date to" are in the current specs. But the usage is for single request by setting identical date for both parameters.
Following details were agreed:

Only closed days can be requested with date range function. Therefore the transaction with booking status "pending" will not be included. The request for intraday transaction applies analog to the present: no date will be filled in.
Pagination:

The community agreed that there needs to be a consent for equal usage of pagination through all APIs.
The pagination shall be curser based (instead of offset paging).
The links shall be placed in the header.
Action Point: @dkoeni will check the issue and provide feedback.
Action Point: @simonmeyerm will then prepare the Pull Request for discussion on next meeting.

The error codes 202 and 204 may still be accurate and therefore they will be kept as at the present.

Meeting Notes 26.9.24
The description has been extended and adjusted and cursor setup for pagination is included. The Pull Request will be created after the dependency of #125.

The pull request was merged into the V5 branch. This issue can therefore be closed.


Trg 7.01 legal documentation, issue #78 fix

Legal documentation update
What:
SECURITY.md file update
Why:
to be compliant with Trg 7.01 and fix issue #78

Please first merge PR #74

just changes to be compliant with Trg 7.01, looks correct from my side

@SebastianBezold please approve and merge


Network usage

Hi thank Harvesters team for this great library .
Why network's usage is always 100% but with my camera's application It's about 5..6% ?

Hi, could you tell me the GenTL Producer that you're using now?

One more question: Do you mean the usage reaches 100% when you call the ImageAcquirer.start method? Would that turn quite once the image acquisition stops by the stop method call?

Closing. Please feel free to reopen if needed.


Launching unit test

Unit test can be executed by netcoredbg using the following command.
netcoredbg --interpreter=cli -- dotnet test Some.Tests.dll

The problem is to find out to fit the arguments to arguments of adapter (see https://github.com/alexhokl/nvim-dap-dotnet/blob/main/lua/dap-dotnet.lua#L115).
{
	type = "coreclr",
	name = "[dap-dotnet] Launch unit test",
	request = "launch",
	-- preLaunchTask = "build",
	program = "test",
	args = {"Some.Tests.dll"},
	cwd = "${workspaceFolder}",
	stopAtEntry = false,
	env = {
		ASPNETCORE_ENVIRONMENT = "Development"
	}
}

I the next few days I'll try to take a crack at this.
I'll try to apply the same idea as nvim-dap-go by discovering the test I want to debug using treesitter.
Debugging unit tests is my main workflow when I work on my projects. I rarely debug the application itself.
It would be really great if I can make it work without having to do the dance of starting the test, make that wait for the debugger to attach and only then continue debugging it. At least I don't want to do that manually.
If I have any success I'll propose a PR.

I didn't really have time to work on this trough December. I hope to get cracking this month. Now I need to really study how to make use of treesitter to be able to query about the function we are in and if it is a test or not. It's an area of programming totally new for me so I expect the on boarding to be a bit bumpy, but as I work daily with C# I really want to make this to work.
Of course if you guys have any input about how to better approach this let me know.


Egress SNAT for egress traffic from workload based on an annotation on the pod

Description
As I mentioned in the issue 3275, this proposal provides a consistent egress IP for outbound connections from the pod without the need for an edge router.
A proposal to apply SNAT on egress traffic from workload towards external network (IP address that doesn't fall in any of the calico IP pools). Some issues that this addresses:
https://github.com/projectcalico/calico/issues/3275
https://github.com/projectcalico/calico/issues/5549
I tested this with Felix FV with the following tests:

ETCD datastore with linux dataplane
ETCD datastore with eBPF dataplane
Kubernetes datastore with linux dataplane (Fails, and not sure why)
Kubernetes datastore with eBPF dataplane

Related issues/PRs
https://github.com/projectcalico/calico/issues/3275
https://github.com/projectcalico/calico/issues/5549
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

Just a note that this solution has outstanding issues to be fixed, more here: 3275

@sriramy thanks for the PR!
We're discussing this one a bit within the maintainers team at the moment. We need to work through a few scenarios to make sure this solution is comprehensive enough and is something we are able to merge. We'll report back here with the outcome of that discussion.
I also saw your comment here https://github.com/projectcalico/calico/issues/3275#issuecomment-1288922328 - we will think about that. The multi-node scenario is one of the key limitations that we need to understand and that we were discussing earlier as well.

@sriramy thanks for the PR!
We're discussing this one a bit within the maintainers team at the moment. We need to work through a few scenarios to make sure this solution is comprehensive enough and is something we are able to merge. We'll report back here with the outcome of that discussion.
I also saw your comment here #3275 (comment) - we will think about that. The multi-node scenario is one of the key limitations that we need to understand and that we were discussing earlier as well.

Hi, wondering if there was some conclusion on this topic? :)

@sriramy sorry for the long delay, we've been a bit slow over the holidays. I don't think there's been much traction on this one I'm afraid.

Hey @sriramy, sorry for the delay still, I think we are still missing some of the discussion needed for this so I'll re-ping the team to get the conversation rolling.

No worries, we are still interested in the feature though. I would like to contribute if some way forward is suggested. :) Do you think something like Cilium Egress Gateway[1] or Antrea Egress[2] can be a better idea than what I have suggested above?
[1]: https://docs.cilium.io/en/v1.10/gettingstarted/egress-gateway/
[2]: https://antrea.io/docs/v1.3.0/docs/egress/

I would like to see this scenario as well. We use Calico BGP to advertise an external IP for a service. It works great for ingress, the Pod is reachable by the external IP. For  egress it gets SNAT'd using the nodes IP. Would like for it to go out with the same external IP used for BGP advertisements and ingress.
For our usecase these are actually kubevirt VMs so there is only 1 Pod endpoint
There are some egress gateway solutions(Calico Enterprise has one) but thats not what I was looking for. That means all pods go to one egress vrouter and get NATd out using the same IP.
In our case each Pod (VM) has its own external IP that is externally routaBle, It may not always be the case theres an edge gateway for the nodes (so no Internet access, at least via the Calico interface). Our nodes only have Internet via a separate mgmt NIC. This is why node SNAT may not work.
As an example w/ openstack neutron, when a VM has a floating IP, it ingresses and egresses with the floating IP. The router does NAT both directions. It isnt: "ingress on floating, but egress using node or some other  egress gateway" Only VMs without a floating IP share a common snat egress gateway
In this K8s case the "floating IP" is our external IP on the Service(or external Loadbalancer IP)

Hey @sriramy, sorry for the delay again, I'll definitely keep you updated as we progress on this. The members of the team that would have the best insight into this have been focused on some other feature work so I'll try to get their attention on this. Thanks for your patience and we're looking forward to working with you on this.
@xagent003 could I ask you to clarify this portion of your use case?

In our case each Pod (VM) has its own external IP that is externally routaBle, It may not always be the case theres an external gateway for the nodes (so no Internet access, at least via the Calico interface). Our nodes only have Internet via a separate mgmt NIC. This is why node SNAT may not work for VMs/Pods needing Internet.
My understanding is that if everything has to go through the separate NIC on your nodes, traffic would leave with that IP which seems pretty similar to the case where we're using the node's IP. Is the issue that those IPs differ and you're looking to choose one over the other?

My understanding is that if everything has to go through the separate NIC on your nodes, traffic would leave with that IP which seems pretty similar to the case where we're using the node's IP. Is the issue that those IPs differ and you're looking to choose one over the other?

So, this is a multi-tenant multi-cluster cloud so all the nodes are locked down in different VLANs and VRFs at the TORs to provide isolation between different clusters. I am told right now there's an edge gateway device per, but, they want to get rid of it as it won't scale. On the other hand, another "mgmt interface" does exist, but its connected via an entirely different means, no K8s traffic should use it. I'm not a data center expert, so hope that made sense... But for now ignore it, imagine it doesn't exist :)
Now, suppose bond1 has an IP of <IP_ADDRESS>. This is the interface we use for Calico. Traffic outbound gets SNAT'd using <IP_ADDRESS>. Except there is no public gateway for this (it does not route to the Internet). Now we do have public IP space. We set up Calico BGP to peer with the TORs then assign an explicit public IP to the service like such:
apiVersion: v1
kind: Service
metadata:
  name: netdebug4
  annotations:
    metallb.universe.tf/loadBalancerIPs: <IP_ADDRESS>
spec:
  selector:
    ngpc.pf9.io/vmService: vmcalicobridge2
  ports:
    - protocol: TCP
      port: 22
  externalTrafficPolicy: Local
  type: LoadBalancer
  #externalIPs:
    #- <IP_ADDRESS>

The BGP works great, using either the externalIP on the Service, or using MetalLB (we still use Calico for the BGP, metalLB is only so we can set type: LoadBalancer and externalTrafficPolicy: Local). Either way above Service gets a public IP of <IP_ADDRESS>. Inbound traffic is routed down to bond1 for the public IP, <IP_ADDRESS> in above example. And then kube-proxy NATs it to the Pod (Kubevirt VM).
On egress I wanted to also use <IP_ADDRESS> as source IP. Not any node IP.
The containers CIDR is not routable. The Services CIDR is not routable. The node IPs potentially not either. Which means egress(public, to the Internet) won't work unless it goes out using the public IP

hey @sriramy and @xagent003 , I touched base with the team and while there was a lot of discussion, I believe that one of the major hold ups is that we're not exactly sure how the changes in this PR ensure that reply traffic gets routed back to the pod. For floating IPs, Felix programs a local device route for each floating IP and BIRD propagates them in the same way it propagates pod IPs. To help us understand how this is supposed to work for these changes, maybe we need to back up and ask:

What is the pattern of SNAT IPs that are going to be assigned to pods when using this feature?
How do we arrange that traffic to one of those SNAT IPs (coming from outside the cluster) will be routed to the correct node?

hey @sriramy and @xagent003 , I touched base with the team and while there was a lot of discussion, I believe that one of the major hold ups is that we're not exactly sure how the changes in this PR ensure that reply traffic gets routed back to the pod. For floating IPs, Felix programs a local device route for each floating IP and BIRD propagates them in the same way it propagates pod IPs. To help us understand how this is supposed to work for these changes, maybe we need to back up and ask:

What is the pattern of SNAT IPs that are going to be assigned to pods when using this feature?
How do we arrange that traffic to one of those SNAT IPs (coming from outside the cluster) will be routed to the correct node?

Hi @mgleung, This PR was just an RFC patch really.
For a full solution, I was hoping to create a IPPool object for IP addreses that can be used for SNAT so the routes are exported via BGP. We could add a new allowedUse type called "SNAT" or something similar.

Hi matt, not sure I understand your questions...
I dont understand #1 what you mean by pattern? As I mentioned, we want it to be SNAT'd using the same externalIP assigned to the service. So packets will ingress and egress using the same IP. We take a public IPv4 address and add it to a Service as the externalIP
For #2, doesn't BGP take care of it? We advertise the external IP associated with a Service. The packet ingress's to that node. thereafter, kube-proxy "loadbalances" it to the Pod. Which in our case is a singular Kubevirt VM. Very likely on the same node because we want to use externalTrafficPolicy: local.
On egress, it would take the exact same path outbound... except it gets SNAT'd using the Service's external IP rather than the node IP on the NIC.

@mgleung @sriramy OVN-Kubernetes seems ot have something like this: https://github.com/ovn-org/ovn-kubernetes/blob/master/docs/egress-service.md
The Egress Service feature enables the egress traffic of pods backing a LoadBalancer service to exit the cluster using its ingress IP. This is useful for external systems that communicate with applications running on the Kubernetes cluster through a LoadBalancer service and expect that the source IP of egress traffic originating from the pods backing the service is identical to the destination IP they use to reach them - i.e the LoadBalancer's ingress IP.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sriram Yagnaraman seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you everyone for the interest and discussion above, and apologies this hasn't come sooner. I think for now I am going to have to close this PR. There are a few reasons at play here:

Our enterprise product has committed to a different approach to solving this problem via the use of egress gateways.
We aren't fully sure this approach is a complete solution. Specifically the concerns around multi-node clusters and reliance on host IP addresses for SNAT require additional work to make viable.
This type of feature is - for now - out of scope for the Calico opens-source project, so our ability to invest time in fully building out this feature set is too limited.

Again, thanks all for your input and contributions above.


use log.Fatal() instead of panic() in example client and server

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2290   +/-   ##
=======================================
  Coverage   85.93%   85.93%
=======================================
  Files         108      108
  Lines        8089     8089
=======================================
  Hits         6951     6951
  Misses        857      857
  Partials      281      281

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 59c2c34...2ba4617. Read the comment docs.


prefixing: what about Samsung SmartTV browsers?

Hello,
as I red on the docs, "your function may vary", depending on the browser version.
I've got a couple different Samsung SmartTV, on which I'd like to have fullscreen for videos: I've got mixed results and found little docs about these browsers, so I'm sharing my findings.
With the script below, I was able to make it work on the older model (F5500), while it doesn't on the newer (ES6710), while the sample page of fscreen couldn't get none of the two on fullscreen.
I'm attaching relevant screenshots, any plan to support such browsers?
`
function setupPlayer(fullScrBtn) {
var isFullscreenAvailable = document.fullscreenEnabled
|| document.mozFullScreenEnabled || document.webkitFullscreenEnabled
|| document.msFullscreenEnabled || false;

if (isFullscreenAvailable) {

	var myEvent = function(e) {
		var fullscreenElement = document.fullscreenElement
				|| document.mozFullScreenElement
				|| document.webkitFullscreenElement
				|| document.msFullscreenElement;
		if (fullscreenElement) {
			// we are now on fullscreen
		} else {
			// we have exit fullscreen mode
		}
	};

	// add event listeners

	// W3C standard
	document.addEventListener('fullscreenchange', myEvent, false);
	// Firefox 10+, Firefox for Android
	document.addEventListener('mozfullscreenchange', myEvent, false);
	// Chrome 15+, Safari 5.1+, Opera 15+, Chrome for Android, Opera Mobile
	// 16+
	document.addEventListener('webkitfullscreenchange', myEvent, false);
	// IE 11+
	document.addEventListener('MSFullscreenChange', myEvent, false);

	// enable button
	// alert(fullScrBtn.disabled);
	fullScrBtn.disabled = false;
	// alert(fullScrBtn.disabled);
}

}
function requestFullScreen(element) {
var el = element || document.documentElement;

var fullscreenElement = document.fullscreenElement
		|| document.mozFullScreenElement
		|| document.webkitFullscreenElement || document.msFullscreenElement;

if (fullscreenElement) {
	if (document.exitFullscreen) {
		// W3C standard
		document.exitFullscreen();
	} else if (document.mozCancelFullScreen) {
		// Firefox 10+, Firefox for Android
		document.mozCancelFullScreen();
	} else if (document.webkitExitFullscreen) {
		// Chrome 20+, Safari 6+, Opera 15+, Chrome for Android, Opera
		// Mobile 16+
		document.webkitExitFullscreen();
	} else if (document.webkitCancelFullScreen) {
		// Chrome 15+, Safari 5.1+
		document.webkitCancelFullScreen();
	} else if (document.msExitFullscreen) {
		// IE 11+
		document.msExitFullscreen();
	}

} else {

	if (el.requestFullscreen) {
		// W3C standard
		el.requestFullscreen();
	} else if (el.mozRequestFullScreen) {
		// Firefox 10+, Firefox for Android
		el.mozRequestFullScreen();
	} else if (el.msRequestFullscreen) {
		// IE 11+
		el.msRequestFullscreen();
	} else if (el.webkitRequestFullscreen) {
		if (navigator.userAgent.indexOf('Safari') != -1
				&& navigator.userAgent.indexOf('Chrome') == -1) {
			// Safari 6+
			el.webkitRequestFullscreen();
		} else {
			// Chrome 20+, Opera 15+, Chrome for Android, Opera Mobile 16+
			el.webkitRequestFullscreen(Element.ALLOW_KEYBOARD_INPUT);
		}
	} else if (el.webkitRequestFullScreen) {
		if (navigator.userAgent.indexOf('Safari') != -1
				&& navigator.userAgent.indexOf('Chrome') == -1) {
			// Safari 5.1+
			el.webkitRequestFullScreen();
		} else {
			// Chrome 15+
			el.webkitRequestFullScreen(Element.ALLOW_KEYBOARD_INPUT);
		}
	}
}

}
...
`
ue46es6710 - 1 - tv tag

ue46es6710 - 2 - browser name

ue46es6710 - 3 - support prefixed

ue46es6710 - 4 - fscreen failing

ue46es6710 - 5 - fullscreen disabled

ue46f5500 - 1- tv tag

ue46f5500 - 2 - browser name

ue46f5500 - 3 - support prefixed

ue46f5500 - 4 - fscreen failing

ue46f5500 - 5 - fullscreen enabled

ue46f5500 - 6 - fullscreen working

forgot to mention: on a regular browser Fscreen works fine, see shots

Hi, I don't have have time to support Sumsung SmartTVs in this library, however, I'd be happy to accept a pull request.
With regards to the code, it looks like it does almost the exact same thing as fscreen. One main difference is that it uses document.documentElement if no element is provided. The fscreen example site uses the root div as the fullscreen element, but fscreen will work with any element you give it. Also, in your pictured example, the element you're making fullscreen is a video.
Have you tried using fscreen with a video element, or with document.documentElement on your SmartTV? It wouldn't surprise me if the Samsung browser restricts which elements can go fullscreen.

Hi rafrex
thx for the reply, my intention was to help spread support to those devices.
looks like the browser (when used standalone) has limitations, even web-apps (samsung tv apps, web based) are restricted to 960x540 (I tried larger and had to scroll):
http://developer.samsung.com/tv/develop/legacy-platform-library/tut00055/index#Creating-the-Basic-Application
in addition, on the "full-screen-working" TV, there was no way to get back to normal view (only the "enter" event was available!), so I switched to a non-web TV app, see shot

cheers
.k.


Hardware acceleration

What's the value for disabling hardware graphics acceleration under General?

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

ID: 6159d774-f257-769d-27ca-cc7b7bdae54a
Version Independent ID: c88c5ada-49c7-7333-cd36-234ddb2a9cb0
Content: Options Page, Environment Node Properties - Visual Studio
Content Source: docs/ide/reference/options-page-environment-node-properties.md
Product: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

Hi @r3m0t! Can you clarify what you mean?


v0.11.0 release

[Bug Fix]  In BPTSparse when combination of bucket and key is repeated (#207) @ShiMaRing
[Bug Fix]  MInInt function compatible with 32-bit operating systems (#208) @xujiajun
[Bug Fix]  index EOF issue#213 (#214) @xujiajun
[Perf] optimize concurrent read performance (#205) @elliotchenzichang
[Perf] Use biobuf optimaze startspeed (#212) @elliotchenzichang
[New Feature] Support reverse iterator (EntryIdxMode: HintKeyAndRAMIdxMode and HintKeyValAndRAMIdxMode) (#202) @zeina1i
[New Feature] Add support for IterateBuckets regularized matching (#198) @Nailcui
[New Feature]  list all key of bucket in memory mode (#206) @Nailcui
[New Feature]  feat: add PrefixScan in memory mode  (#211) @Nailcui
[Refactor]  make default options to be created in a factory method (#196) @elliotchenzichang
[Refactor]  use size constant value (#204) @elliotchenzichang
[Chore] add iterator example (#209) @xujiajun
[Chore] remove option StartFileLoadingMode (#218) @xujiajun

Release history:
https://github.com/nutsdb/nutsdb/issues/193


[lit-next] Distinguish property and internalProperty better

Description
Other popular frameworks have a concept of props which are passed in from the outside and should be treated as immutable and state which is set internally and is mutable.
In Lit, we have property which is passed in and internalProperty which is managed internally. It probably makes sense to change internalProperty to state to leverage user knowledge of other frameworks.
When marked as a property a value may be both a "prop" and "state." This should be clearly indicated in some way.
Users have also reported issues with setting up default values that mutate inside the element. In DOM there are analogs with input.value which is set via an attribute (which does not change) and a property which does.
Acceptance criteria
Decide on the right design, implement, and document changes.

Hello,
I reccomend you to check out the code more better. This code isn't to our standards. Please include code. We are not responsible for this. You can increase the input value working range by improving your code
Kind regards,
Sebastiaan

Some possible ideas in addition to renaming @internalProperty to @state, we may also want to consider the following. Most of this would be for documentation and linting behavior only.

In addition to @state, adding state: true|false to property options for JS users. (This sets attribute to false)
Adding input: true|false and @input to indicate a property that should only set by users of the element and not mutated by it. LitPlugin would warn if you set this property in your element.
Adding output: true|false and @output which creates a readonly property only with a getter and no setter (and attribute to false) and generates a set<name> function which (optionally?) generates an event with the property's changed value. Alternatively, readonly and notifier could be explicitly separate options perhaps in addition to the output shorthand.

Fixed via https://github.com/Polymer/lit-html/pull/1485.


[Question] : Folder structure for tables

do we need to have the tables in the same sequence as they have dependencies?

This is a good question. I would try to put dependent SQL in the same file, since ordering of statements will be obvious. Otherwise, I'm thinking we will order alphabetically by file path.

BLUF: filePaths.sort().join("\n").forEach(runSQL) should run successfully on a fresh database.

We in SQL server format doesn't follow any sequence for the tables which are dependent was trying to replicate the same and that didn't work with Postgres. When I appended script000 and sequence in the order they are dependent then that worked. We would like to keep the tables in their own file so that it easy to represent schema for each. Is there any way we can automatically figure out the dependencies and order them accordingly?

Like using annotations like below

--> depends: a.sql, b.sql
CREATE TABLE c (
    -- ...
);

Thoughts on that ? @Navbryce

Closing. The PR above details the canonical ordering.


Here are the highlights of this release:
 - Kotlin DSL: new reference documentation, assignment syntax by default
 - Kotlin DSL is now the default with Gradle init
 - Improved suggestions to resolve errors in console output

For more details see https://docs.gradle.org/8.2/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.
> Could not determine the dependencies of null.
   > SDK location not found. Define a valid SDK location with an ANDROID_HOME environment variable or by setting the sdk.dir path in your project's local properties file at '/Users/michiel/gh/pondersource/Android-Solid-Services/local.properties'.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

BUILD FAILED in 1m 21s

Should I download https://developer.android.com/studio ?

Ah, https://stackoverflow.com/a/19986294/680454 was useful. I'm now going through the Android Studio setup wizard, and then I'll try:
export ANDROID_HOME=/Users/{YOUR_USER_NAME}/Library/Android/sdk

OK, problem fixed!


nothing happens on clicking the marker icon present on the map

Steps to reproduce:

go to any map shown on the contact page
hover over marker icon
3)click

Actual result: pointer turns into hand icon but nothing happens on clicking it
Expected result: user should be redirected properly or it should not change into hand icon on hovering over it.

Reported by: Vinod<EMAIL_ADDRESS>Source URL: https://kensway.webflow.io/contact
Issue details: Open in Marker.io
Device typedesktopBrowserChrome <IP_ADDRESS>Screen Size1366 x 768OSWindows 10 (10.0.0)Viewport Size1366 x 641Pixel Ratio@1x

keeping the pointer cursor;
and redirecting to the Google map link upon user-click (on a new window)

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


use open/close

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

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

Testing Guidelines

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

/azp run java - servicebus

API change check
API changes are not detected in this pull request.

/azp run java - servicebus

/azp run java - servicebus

/azp run java - servicebus

/azp run java - servicebus

/azp run java - servicebus

/azp run java - servicebus


[BUG]: Does COE Apps dashboard report shows only canvas apps or model driven apps as well?

Describe the bug
It seems that COE Apps dashboard  doesn't show all apps across the tenant, same as "Get-AdminPowerApp" whether shared with MPP Admin/GA Admin OR not - it seems this only shows all canvas apps and some model driven apps.
Can you please clarify which APPs will be visible in COE Apps dashboard if run by MPP/Global Admin?
Component (please tell us which flow or app you are experiencing issues with):

Core Componenet
Version 1.44
Power BI Report

To Reproduce
Steps to reproduce the behavior:

Go to Power BI Dashboard
Click on 'Apps'
Locate to 'Apps Creation Trend' tabular report
See that some apps are not visilbe, while it's visible in UI

Expected behavior
All apps types across the tenant should be visible in PowerBI dashboard for Power Platform Admin whether shared/not shared

Hello,
you're using quite an old version of the CoE Starter Kit, so would upgrading to the latest version so to make sure you have the latest bug fixes and features.
The CoE Starter Kit does show model driven apps, however the admin would need to be a System Administrator in the environments with model driven apps. Ensure your role has the correct level of access. Additionally, only published model driven apps (not those in draft status) are shown.
Hope that helps,
Manuela

Hello,
Thank you for the response with more information, MPP Admin have "System Administrator" role in the environment of MDA Apps which is not being visible in COE App.I have also published it's customization.
However, When I try to change the OrgURL in latest PBI file, I am facing following errors:

Can you please share if I am missing anything here?
Thanks,

Are you logged into Power BI with the admin account that has access to the entities?
In the pop-up "Apply your changes" currently I can only see "failures with other queries". If you scroll down, you will find the query that is failing (& causing other queries to fail) - could you comment back with that specific failure.

I am logged into Power BI as "MPP Admin" that has access to all entities in the environment, however when I check the "admin_app" entity, I see that couple of MDA apps are not listed in the table, even after publishing the customization related to that - I assume in this case, I will not see the app in PBI Report - is that correct? does this require any specific flow to run?
While regarding pop-up errors, when I scroll down I see following errors:

Do I require to update anythin anywhere else?
Thanks again for your time and help!
Regards,
Dipti

Hello,
when you navigate to make.powerapps.com > Solutions > Center of Excellence Core Components and search for PVA do you see the three PVA entities and one flow

As MPP Admin,I do not see any of these components available under make.powerapps.com > Solutions > Center of Excellence Core Components.This pops up following questions:

Are these components compulsory to run "Power BI Dashboard Report" ?
Do I require to remove old "Core" components and add new core components from latest COE Kit?
Does these components would need "Global Admin" rights to configure?

Thanks and Regards,
Dipti

Thanks for the suggestions!
I have updated the COE core components accordingly and now I can see PVA customizations in the solutions which I have upgraded>published on the existing core components.
Then, I published latest PBI report and I (MPP Admin) see the current org's apps count, however I yet do not see a specific model driven app which is shared with me with owner rights and that says probably the count of all apps can not be "True"
Further, under "Apps" report when I click "App Type" filter, It does not show "Model Driven Apps" filter as indicated in the following screen:

Questions:

What could possibly missing at my end? (I confirm that the count of Apps in PBI Report and "admin_App" entity is same.)
On the other hand, "Get-AdminPowerApp" has different results in terms of count and it's apps, can you please share how PBI report gathers Apps details from the different environments with it's limitations?

Regards,
Dipti

Hello,
the Power BI dashboard is based on the Common Data Service entities, such as admin_App, admin_Flow that are part of the CoE Starter Kit. Those entities get populated by the sync Flows - Admin | Sync Template v2 (Apps, Flows, Connectors,...). Those flows use the admin connectors - Power Platform for Admins, Power Apps for Admins, Power Automate Management  to gather information about resources in your tenants. So it uses "Get Apps for Admins" per environment to gather information about canvas apps.
Some resources, such as model driven apps and chatbots do not have a management connector but their information is stored in underlying system entities - for model driven apps, that entity is called appmodule and for chatbots it is called bot. Those entities exist in each environment (and are system generated) and what the CoE Kit does here is gather information from those entities.
Because this information is stored in CDS entities, it adheres to CDS security roles and the user running the Sync Template v2 (model driven app) flow will have to have System Administrator access to each environment in order to view the data here. The other limitation is that for environments of type Developer (those created through a community plan) this information is not available, as only the user with the community plan has access to the environment and access can't be granted to admins.
You could look at the Admin | Sync Template v2 (model driven apps) to see if there's any errors in getting information from the environments.
Hope that helps,
Manuela

Hello,
I confirm that MPP Admin has "System Administrator" role in the environment of MDAs, however following connectors are not granted by GA yet, therefore those flows are not currently functions:
Power Platform for Admins,
Power Apps for Admins,
Power Automate Management
apart from access on above connectors, does this report also require "Dynamic 365 Role" in addition to "MPP Admin Role" ?
Thanks and Regards,
Dipti

No, the Power Platform Service role is sufficient. You can see a list of all the connectors used as part of the CoE Kit here in the 4th bullet point https://docs.microsoft.com/en-us/power-platform/guidance/coe/setup#prerequisites

Closing out issues without action items for us. Let us know if you need more help after getting access to those connectors. And thanks for using COE


Support printing strings

It works in CPU and CUDA modes. Only constant-inferrable strings are allowed. Full string support will require native strings.
Based on PR #1959.

[Current coverage][cc-pull] is 87.17%

Merging [#1961][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master      #1961   diff @@
==========================================
  Files           299        300     +1
  Lines         53452      53595   +143
  Methods           0          0
  Messages          0          0
  Branches       5526       5550    +24
==========================================
+ Hits          46596      46723   +127
- Misses         5932       5941     +9
- Partials        924        931     +7

Powered by Codecov. Last updated by [9b26e83...1891fc0][cc-compare]
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/numba/numba/compare/9b26e838eb605f141b2df7c55803aac63b467907...1891fc0e049e5ea10fd3e463d911c9297496ffae
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1961?src=pr

The PR looks good but what happened on AppVeyor!?

The PR looks good but what happened on AppVeyor!?

It happens from time to time. I'm not sure what it is, my only explanation is that the VM may crash or be terminated for a reason. I'll launch a new build.

Appveyor seems happy now.  Merging.


Unexpected behavior calling tf.keras.Model.call() with named parameters

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04.3 LTS
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.2
Python version: Python 3.7.4 Anaconda
CUDA/cuDNN version: ROCM:

Package: rocm-libs
Version: 3.0.6
Priority: optional
Section: devel
Maintainer: Advanced Micro Devices Inc.
Installed-Size: 13.3 kB
Depends: rocfft, rocrand, rocblas, hipblas, rocsparse, hipsparse, rocalution, rocprim, rocthrust, hipcub
Homepage: https://github.com/RadeonOpenCompute/ROCm
Download-Size: 802 B
APT-Manual-Installed: yes
APT-Sources: http://repo.radeon.com/rocm/apt/debian xenial/main amd64 Packages
Description: Radeon Open Compute (ROCm) Runtime software stack

GPU model and memory: AMD Radeon VII

The full environment script does not work for my machine, but:
`python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"`

>>> v2.0.0-rocm-3-g0826c3a 2.0.2

Describe the current behavior
I am getting odd behavior when calling a tf.keras.Model's call method when using the names of the method's parameters. The method works as expected when using position only arguments, but breaks when using the names. However, when I call my model_instance.call() with the names of the method parameters, things work as expected. It is making me wonder which __call__ method I am calling when simply running model_instance().
Describe the expected behavior
Using the names of the parameters in a tf.keras.Model's call method should not be raising an error.
Code to reproduce the issue
First, a little bit of setup showing that calling a tf.keras.layers.Attention instance from a function works with and without using the names of the positional arguments in a user defined function, call:
import tensorflow as tf

def call(q, v, k, mask_q=None, mask_v=None):
    """ Call attention instance """
    return attn(inputs=[q, v, k], mask=[mask_q, mask_v])

x = tf.random.uniform((1, 2, 2))
attn = tf.keras.layers.Attention(use_scale=True)

# position arguments work well
call(x, x, x)

>>> <tf.Tensor: id=89, shape=(1, 2, 2), dtype=float32, numpy=
array([[[0.62968266, 0.6612503 ],
        [0.6235384 , 0.73767066]]], dtype=float32)>

# naming the parameters also fine here
call(q=x, v=x, k=x)

>>> <tf.Tensor: id=89, shape=(1, 2, 2), dtype=float32, numpy=
array([[[0.62968266, 0.6612503 ],
        [0.6235384 , 0.73767066]]], dtype=float32)>

Things start getting weird when doing something similar within a tf.keras.Model:
class MyAttention(tf.keras.Model):

    def __init__(self):
        super(MyAttention, self).__init__()
        self.attention = tf.keras.layers.Attention(use_scale=True)

    def call(self, q, v, k, mask_q=None, mask_v=None):
        return self.attention(inputs=[q, v, k], mask=[mask_q, mask_v])

my_attention = MyAttention()

# Still works with positional arguments
my_attention(x, x, x)

>>> <tf.Tensor: id=106, shape=(1, 2, 2), dtype=float32, numpy=
array([[[0.62968266, 0.6612503 ],
        [0.6235384 , 0.73767066]]], dtype=float32)>

# Breaks when naming the arguments in my_attention:
my_attention(q=x, v=x, k=x)

>>>
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-15-5fa3b47998d9> in <module>
----> 1 needs_attention(q=x, v=x, k=x)

TypeError: __call__() missing 1 required positional argument: 'inputs'

Finally, if I explicitly call my_attention.call():
my_attention.call(q=x, v=x, k=x)

>>> <tf.Tensor: id=106, shape=(1, 2, 2), dtype=float32, numpy=
array([[[0.62968266, 0.6612503 ],
        [0.6235384 , 0.73767066]]], dtype=float32)>

Other info / logs
Here is a gist to show this behavior:
https://gist.github.com/yngtodd/f3bda25503a9611765ab33c1178db48c

Can't reproduce this error with latest tf-nightly 2.2.0.dev20200203. Probably the issue is fixed by the recent changes.


Make emails deleteable after a week

This is so we don't end up with lingering emails and subscription
contents in the system which will never be deleted or recovered.
Trello:
https://trello.com/c/tfu9zXuy/534-delete-unused-data-after-a-year

Thanks for the review @benthorner!

Emails can be archived immediately. This makes more sense now, because we no longer send any sending-specific stats along with the email.

Do you mean archiving the emails as soon as they've been sent as part of the DeliveryRequestWorker? Currently the archiving worker runs every 30 mins which is pretty soon after an email has been sent.

Emails can be deleted once they're archived (after a week).

So this sounds like there needs to be some sort of logic where we delete pending emails regardless after a week but don't for emails that could be archived (not pending) but haven't been... yet. Does that capture what you mean?

Sorry @1pretz1, I fear I've made this more confusing than it is. Does this make more sense?

We don't need a scope for archiving emails; they can be archived at any time.
Emails can be deleted when archived_at < 1.week.ago.

So we always archive, but we can only delete once we've archived (which should always be true).

Sorry if I'm being dense @benthorner (entirely possible).

they can be archived at any time

By 'any time' do you mean 'any status' @benthorner ?
If so, my original thinking was that we don't want to archive pending emails as it may be confusing that we have emails in our archives that we've never attempted to send and our only there due to a system error but now as I'm writing this I'm thinking that this may actually be good as it gives us some visibility of an issue that may vanish if these emails just disappear after a week without being pushed to s3.

Emails can be deleted when archived_at < 1.week.ago.

This makes sense in the context of pending emails being archived otherwise archived_at would never be populated for these and so the emails would never be deleted.

@1pretz1 that's OK, it's good to clarify.

By 'any time' do you mean 'with any status' @benthorner ?

Yes, I do. The archiving has never made much sense to me, and we plan to get rid of it. Until then, though, we should try to keep it working as best we can. Archive doesn't have to mean "old" or "finished" - it's just a historical record - so it's consistent to put any and all emails in it, irrespective of what happens to them (noting that we don't archive any status information anymore).

Sure, thanks for explaining @benthorner !
Have updated accordingly.

Sure, I've added that to the commit message!