Store conversion methods

As follow-up/an extension to discussion in issue ( https://github.com/alimanfoo/zarr/issues/129 ), it would be pretty handy to have some methods to convert back and forth between DirectoryStore and ZipStore. Given that Zip files really are best treated as a write once medium, this would allow one to write out to a DirectoryStore and then convert to a ZipStore. Similarly if editing needs to occur, one could extract the ZipStore and perform edits on the DirectoryStore and then archive it afterwards.

Musing on this a bit, maybe leveraging the update methods from the MutableMapping interface might be better. One would then provide it the store they would like to move their data to. For instance, converting to a DirectoryStore or a TempStore should be able to follow similar code paths, which should just work thanks to inheritance. This should also allow it to remain extensible in the future.

That is a very elegant idea!
On Wed, Mar 1, 2017 at 5:15 PM, jakirkham<EMAIL_ADDRESS>wrote:

Musing on this a bit, maybe leveraging the update methods from the
MutableMapping interface might be better. One would then provide it the
store they would like to move their data to. For instance, converting to a
DirectoryStore or a TempStore should be able to follow similar code
paths, which should just work thanks to inheritance. This should also allow
it to remain extensible in the future.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/137#issuecomment-283405709, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8QkveW6Cvgx6P7Kh97iYaKyYCEpwEks5rhaecgaJpZM4MP6u-
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

I think it would be worth adding an example to the tutorial to show how to copy data from one store to another.

Just to clarify, we are thinking of this as a docs/examples item? If so, I think I agree. The existing interface is already sufficient, but a couple line example would be good. Any thoughts on where this would documented?

I was actually thing that a convenience function like
zarr.copy_store(source, dest, source_path=None, dest_path=None) might be
useful. If source_path and dest_path were None this would be equivalent to
copying all key/value pairs from source to dest. If source_path were given
this would restrict copying to keys with this prefix, and then dest_path
would give a new prefix to prepend before storing.
Basically, it's a convenient way to copy data from one hierarchy directly
to another, without having to do any decompression/recompression, and
allowing for the path within source and dest to be provided so that parts
of a hierarchy can be selectively copied.
What do you think?
On Mon, Nov 20, 2017 at 2:59 PM, jakirkham<EMAIL_ADDRESS>wrote:

Just to clarify, we are thinking of this as a docs/examples item? If so, I
think I agree. The existing interface is already sufficient, but a couple
line example would be good. Any thoughts on where this would documented?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/137#issuecomment-345720693, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8QmRErRwZhzqrq0PYBMC2gYSxb3QXks5s4ZO8gaJpZM4MP6u-
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
Big Data Institute Building
Old Road Campus
Roosevelt Drive
Oxford
OX3 7LF
United Kingdom
Phone: +44 (0)1865 743596
Email<EMAIL_ADDRESS>Web: http://a http://purl.org/net/alimanlimanfoo.github.io/
Twitter: https://twitter.com/alimanfoo

No complaints from me. This is somewhat similar to how I imagined the Group copy method. Does that fit the use cases you are imagining here or do you see other benefits from handling this at the Store level instead of the Group level?

I was thinking there is potentially a use for both a low-level zarr.copy_store(source, dest, ...) where source and dest are store objects, and a higher-level zarr.copy(source, dest, ...) where source is array-like or group-like and dest is group-like.
The low-level copy_store() is for the case where you want to replicate data exactly, and so you just copy key/value pairs from one store to another, which is going to be faster because there is no need to decompress/recompress chunks, and because only initialized chunks will get copied.
The higher-level copy() would be for a more general case where you want to copy an entity (group or array), going via the create_group/create_dataset API. This could mean potentially that source and dest could be anything array- or group-like, e.g., either zarr or h5py, i.e., this would provide a way to migrate data between two zarr hierarchies, or zarr to/from h5py. This could also allow for copying but using different compression to store data in the destination than is used in the source.

I see. Sure that seems fine.
My initial thinking with copy was that it would special case Zarr objects and thus handle direct data replication with the performance of copy_store, but through the same easy high level interface.

I've had situations where I'd like to copy between two zarr objects, but
change the compressor, in which case it would not be possible to do a
direct store-to-store copy. So it would be nice to handle that use case, I
was thinking via copy().
On Monday, November 20, 2017, jakirkham<EMAIL_ADDRESS>wrote:

I see. Sure that seems fine.
My initial thinking with copy was that it would special case Zarr objects
and thus handle direct data replication with the performance of copy_store,
but through the same easy high level interface.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/137#issuecomment-345868495, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAq8QtFSlxpZWvnqycP77ThyoZgk5Qiiks5s4g6MgaJpZM4MP6u-
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
Big Data Institute Building
Old Road Campus
Roosevelt Drive
Oxford
OX3 7LF
United Kingdom
Phone: +44 (0)1865 743596
Email<EMAIL_ADDRESS>Web: http://a http://purl.org/net/alimanlimanfoo.github.io/
Twitter: https://twitter.com/alimanfoo


What's the recommended way of using material-components-web-react components?

At this point it seems documentations for this library is nonexistent. The general guidelines has a section about importing with ES5. Is that the recommended way of using this library?
It'd be nice to have an example of best practice in the documentations.

That guide is not specifically for React. When it comes to styles, most React libraries have an opinion about the styles approach. It is not clear to me which approach is preferred for this library.

Yes, that makes sense. But my point is something else. For example, create-react-app projects do not support importing sass files or some ES6 syntax by default, at least not until v2 is out.
There is a recent commit which transpiles ES6 to ES5 and sass to css, but it is now known to me how to customize the theme when importing css, as customizing by sass will be lost.
Ant Design React library ha s a detailed guide about how to use the library with create-react-app: https://ant.design/docs/react/use-with-create-react-app. I was hoping something similar could exist for this project to get a new user started without giving them even more decision fatigue in recent javascript world.

I was thinking that the #usage sections in each readme would be enough documentation for now: https://github.com/material-components/material-components-web-react/tree/master/packages/button#usage.
Customizing is similar to MDC-Web. Our intent was to reuse the Sass mixins we've built in MDC-Web documented here.
We've included the Sass within each package. However since we didn't compile the Sass into CSS, it might be easier to include the MDC-Web Sass/CSS files as necessary. So it looks like something like this:
// Buttons need ripple and button styles
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
...

@moog16 are you suggesting that the when using material-components-web-react, sass imports should come directly from material-components-web? Will it be guaranteed that in the future the sass files in material-components-web-react will not have extra styles?

@jimbojetlag what do you mean extra styles? Do you mean duplicate?

@moog16 I don't mean duplicate, I mean any styles in addition to material-components-web styles that might have to be necessary for the React wrapper element to function.

@jimbojetlag I cannot guarantee anything as it this project is in alpha similar to MDC Web. But from my standpoint it doesn't make sense that MDC React will have differing styles. The goal of this project is to share as much code (Foundation JS & CSS) from MDC Web, effectively making MDC React a mirror of MDC Web's components. We currently follow this pattern and have no extra styles. The exception being Material Icon, but that is not an MDC Web component.
Short answer: you can expect MDC React to NOT have extra styles in addition to MDC Web's styles.


jsdoc / typedoc

especially relevant for Registry.ts and objects/*
https://github.com/Microsoft/TypeScript/wiki/JSDoc-support-in-JavaScript
https://blog.cloudflare.com/generating-documentation-for-typescript-projects/
http://typedoc.org
https://github.com/Microsoft/web-build-tools/tree/master/apps/api-extractor

hmm, closing


sending unicode characters

one of those days where "not even thinking about it, assumed it'd be a no-op" turned into "where'd the day go :open_mouth:" 😅
curious how you'd go about sending unicode characters/emoji to the execute command?
thought it'd be as simple as jadbDevice.execute("input", "text", "😀")
then came across this stackoverflow and this repo
that seemed to work on the shell (with the exception of mixing in some special characters () ') but i ended up stumped on how to get it working from the jvm :sob:
any ideas?
val shades: String = """(•_•)
( •_•)>⌐■-■
(⌐■_■)
"""
val processArgs = Seq("adb", "shell", "am", "broadcast", "-a", "ADB_INPUT_TEXT", "--es", "msg", "'" ++ shades ++ "'")
val executeArgs = processArgs.drop(3) // just from `broadcast` through `shades`
val process = new ProcessBuilder(processArgs:_*)
process.start // works
jadbDevice.execute("am", executeArgs:_*) // /system/bin/sh: no closing quote

that was the gnarliest example (newlines, spaces, parens AND unicode chars) but just regular solo emoji also work from the process builder but fail from the execute method

This is not a problem with the jadb project, nor with the adb shell subcommand. But this is a problem with the input program on the android device. As you have noticed it cannot accept unicode. But the workaround with the keyboard sounds like a great work-around, no? Did you try that keyboard?
Also unrelated, I think you can call Java code from Scala, so no need to port this library, unless you enjoy it of course. :-)

yeah was just curious if you had any ideas/pointers
I did end up using that keyboard and just calling out to the ProcessBuilder, works well enough for me ¯_(ツ)_/¯
indeed you can!
mine is definitely not a port of this library! just basically a wrapper, calling out to it with a few parameters that were accumulating in my repl history/had to search online a lot for and wanted to commit somewhere 😅

Ah, I see. No, I don't have any good pointers really, I'm afraid. I would start by double-checking the source code of the input program to see if there isn't really any way to escape characters. I'm guessing there is not.... :-/


[Bug] Picking custom box art doesn't work correctly

When editing an added game to select a different boxart image, it is not being saved correctly.
Steps to reproduce the behavior:

Right Click on a game
Select 'Edit Game Details'
Click on the Boxart Image
Select a new image to be used & click open
Image stays as the previous boxart not the new one

Expected behavior
Boxart image should be updated correctly.
It appears that the imported image is being named incorrectly (missing ' Boxart' being appended to the game name) when saved to the icons folder; or the installedGames.json is not being updated with the new file name.
Screenshots/Video

Environment Information

OS: Windows 11
App Version: v1.15.0

Additional context
Deleting the original boxart.ico file & renaming the newly created file to have ' Boxart' appended fixes the issue.
Alternatively, you could edit the filename referenced in installedGames.json. This would keep the original ' Boxart' file & allow a rollback option to be added?

I've fixed this in the dev build. Give it a test and let me know if it works properly.

Just tested & it appears to be working fine now. Thanks.


txAdmin update

Hello,
txAdmin was awesome resource.
I for a while not update it and today after update of FX server, when I want to update txAdmin, I noticed, that txAdmin was integrated to Fx server it self...
Well I successfuly started txAdmin and it seems working via classic server IP.
Option for SSL setup was dropped (thats not good), so you created some odd way how to secure it with reverse proxy by cfx.re (I not really like it) - and here is problem, if I start txAdmin and access the secure (https) link by cfx.re there is blank page...
Other problem is, that after switching server on, then consol start spamming
[txAdminClient] LUA awaiting for environment setup...
Can anyone please help how to fix this problems?
Thank you for your answer and sorry for my bad english :/

Issues with the cfx.re proxy you should talk to FiveM, really doesn't depend on me.
The direct HTTPS option was dropped due to the complexity of integrating it after refactoring and integrating with the cfx.re proxy.
If you come from txAdmin v1, you need to go to your resources folder and delete the txAdminClient resource, as well as remove "start txAdminClient" from your server.cfg
I'm closing this issue, if you need anything else, come to our discord server (link in the readme file)


should not be possible to mount more than 2 QSFS in 1 VM

i tried to deploy 3 qsfs in same vm and worked fine
Moved from: https://circles.threefold.me/project/test-tfgrid3/issue/55

QSFS works in the same VM. not an issue


Icon Type

[ ] Folder
[X] File

Folder names
No response
File names
.nuxtrc
Graphic ideas
nuxt icon
Additional context
https://nuxt.com/docs/guide/directory-structure/nuxt-config#:~:text=.-,.nuxtrc,-ssr%3Dfalse
Code of Conduct

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

Thanks for the clear reference to this type of configuration

Thanks for the lighting fast release.


Google Keep sync, Getting Auth Token fails in Keep

I noticed that Keep does not sync.
Sadly I know very little about Googles auth system.
Logcat shows a related infos (I removed some sensitive bits):
10-29 14:49:33.097  2278  3233 W AppOps  : Bad call: specified package com.google.android.gms under uid 10107 but it is really 10081
10-29 14:49:33.099  2278  2996 I Xposed  : FakeGApps: faked signature
10-29 14:49:33.106  2278  2295 I Xposed  : FakeGApps: faked signature
10-29 14:49:33.115  3668  3668 D GmsSignInSvc: onBind: Intent { act=com.google.android.gms.signin.service.START pkg=com.google.android.gms }
10-29 14:49:33.125  3668  3668 D GmsPeopleSvc: onBind: Intent { act=com.google.android.gms.people.service.START pkg=com.google.android.gms }
10-29 14:49:33.129  3668  3668 D GmsDriveApiSvc: onBind: Intent { act=com.google.android.gms.drive.ApiService.START pkg=com.google.android.gms }
10-29 14:49:33.140  3668  3686 D GmsSignInSvc: bound by: GetServiceRequest{serviceId=SIGN_IN, gmsVersion=8083000, packageName='com.google.android.keep', scopes=[https://www.googleapis.com/auth/drive, https://www.googleapis.com/auth/drive.appdata], extras=Bundle[{com.google.android.gms.signin.internal.serverClientId=null, com.google.android.gms.signin.internal.usePromptModeForAuthCode=false, com.google.android.gms.signin.internal.offlineAccessRequested=false, com.google.android.gms.common.internal.ClientSettings.sessionId=71052304, com.google.android.gms.signin.internal.idTokenRequested=false, com.google.android.gms.signin.internal.forceCodeForRefreshToken=false}], account=Account<EMAIL_ADDRESS>type=com.google}}
10-29 14:49:33.140  3668  3686 D GmsSignInSvc: unimplemented Method: handleServiceRequest

10-29 14:49:38.198  2278  2956 I Xposed  : FakeGApps: faked signature
10-29 14:49:38.200  2278  2983 I Xposed  : FakeGApps: faked signature
10-29 14:49:38.207  3668  3685 D GmsAuthManagerSvc: getToken<EMAIL_ADDRESS>scope:oauth2:https://www.googleapis.com/auth/memento https://www.googleapis.com/auth/drive extras:Bundle[{sync_extras=Bundle[EMPTY_PARCEL], androidPackageName=com.google.android.keep, clientPackageName=com.google.android.keep, service_connection_start_time_millis=275809474, authority=com.google.android.keep, handle_notification=true}], notify: true
10-29 14:49:38.216  3668  3685 D GmsAuthManagerSvc: getToken: <<<<<129 chars of base64 token >>>>>
10-29 14:49:38.218 10454 12984 E KeepSync: IOException
10-29 14:49:38.218 10454 12984 E KeepSync: com.google.android.apiary.AuthenticationException: Could not get an auth token
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.apiary.GoogleRequestInitializer.getAuthToken(SourceFile:152)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.apiary.GoogleRequestInitializer.intercept(SourceFile:89)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.keep.util.KeepApiaryClient$KeepRequestInitializer.intercept(SourceFile:238)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.api.client.http.HttpRequest.execute(SourceFile:858)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(SourceFile:410)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(SourceFile:343)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.execute(SourceFile:460)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.keep.util.KeepApiaryClient.syncChanges(SourceFile:212)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.keep.syncadapter.KeepSyncAdapter.syncOnceToServer(SourceFile:317)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.keep.syncadapter.KeepSyncAdapter.onPerformSync(SourceFile:197)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at android.content.AbstractThreadedSyncAdapter$SyncThread.run(AbstractThreadedSyncAdapter.java:259)
10-29 14:49:38.218 10454 12984 E KeepSync: Caused by: com.google.android.gms.auth.GoogleAuthException: OK
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.gms.auth.GoogleAuthUtil.zza(Unknown)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.gms.auth.GoogleAuthUtil.zzc(Unknown)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.gms.auth.GoogleAuthUtil.getTokenWithNotification(Unknown)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.gms.auth.GoogleAuthUtil.getTokenWithNotification(Unknown)
10-29 14:49:38.218 10454 12984 E KeepSync: 	at com.google.android.apiary.GoogleRequestInitializer.getAuthToken(SourceFile:140)
10-29 14:49:38.218 10454 12984 E KeepSync: 	... 10 more
10-29 14:49:38.218 10454 12984 W KeepSync: Sync result 2
10-29 14:49:38.275  2278  2299 D SyncManager: failed sync operation<EMAIL_ADDRESS>u0 (com.google), com.google.android.keep, SERVER, currentRunTime 275804147, reason: AutoSync, SyncResult: stats [ numIoExceptions: 1]
```

Sadly I know very little about Googles auth system.

Probably not much less than the average Google employee. There is no public documentation of this part of the authentication API.
Seems like Google Keep requires the "Sign In API", which is a successor of the classic Android Auth API and not yet implemented in microG GmsCore. But as this API is rather new, you might be able to find an older version of Google Keep that does not make use of it yet.

latest Keep works fine for me. only trouble is i can't use the mike to take voice notes.


[HOLD for Payment 5/30][$250] Allow creating a Group Chat with a single member

Coming from this thread we have decided that there is no real reason why a user should not be able to start a Group Chat with only themselves (h/t @shawnborton who noticed that we cannot do this).
Deliverable:

Investigate the front end code and determine what changes need to be made to allow a user to take the following actions:

Add several people to a Group Chat via the Global Create menu
Advance to the "Confim" page with the "Start group" button.
Remove each member
"Start group" button should allowing creating a group with only yourself.

Current behavior we need to improve:

Verify if it appears that some backend changes need to be made (this is possible, I have not yet looked)

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016c3003963dd24c78
Upwork Job ID:<PHONE_NUMBER>262094336
Last Price Increase: 2024-04-15
Automatic offers:

abzokhattab | Contributor | 0

Proposal
Please re-state the problem that we are trying to solve in this issue.
Enable create group chat with only one user.
What is the root cause of that problem?
This is a new feature request.
What changes do you think we should make in order to solve the problem?

Since the current user option will always selected, we can remove the condition here and always show the confirm button

onConfirm={createGroup}

https://github.com/Expensify/App/blob/90412ec0c38cc1ccd0a1198cf1f4712474667906/src/pages/NewChatConfirmPage.tsx#L157
showConfirmButton

https://github.com/Expensify/App/blob/90412ec0c38cc1ccd0a1198cf1f4712474667906/src/pages/NewChatConfirmPage.tsx#L155

We should add an extra param in navigateToAndOpenReport like isGroupChat with default value is false, use this instead of here or we can update the isGroupChat condition by checking it includes the current user or not because for selfDM we always navigate to this report.
We should not use > 0 condition because it maybe a DM

And then pass isGroupChat as true here
Report.navigateToAndOpenReport(logins, true, newGroupDraft.reportName ?? '', newGroupDraft.avatarUri ?? '', fileRef.current, optimisticReportID.current, true);

https://github.com/Expensify/App/blob/90412ec0c38cc1ccd0a1198cf1f4712474667906/src/pages/NewChatConfirmPage.tsx#L104

Verify if it appears that some backend changes need to be made (this is possible, I have not yet looked)

I've tested and we don't need backend changes here.

Since we allow create group chat with single use now, we should separate the Create Chat and Next button. So we can go to the confirm group page without select any option

const footerContent = useMemo(() => {
    /**
     * Creates a new group chat with all the selected options and the current user,
     * or navigates to the existing chat if one with those participants already exists.
     */
    const createGroup = () => {
        if (!personalData || !personalData.login || !personalData.accountID) {
            return;
        }
        const selectedParticipants: SelectedParticipant[] = selectedOptions.map((option: OptionData) => ({login: option.login ?? '', accountID: option.accountID ?? -1}));
        const logins = [...selectedParticipants, {login: personalData.login, accountID: personalData.accountID}];
        Report.setGroupDraft({participants: logins});
        Navigation.navigate(ROUTES.NEW_CHAT_CONFIRM);
    };

    return (
        <>
            <ReferralProgramCTA
                    referralContentType={CONST.REFERRAL_PROGRAM.CONTENT_TYPES.START_CHAT}
                    style={styles.mb5}
                />

                {selectedOptions.length === 1 && (
                    <Button
                        success
                        text={translate('newChatPage.createChat')}
                        onPress={createChat}
                        pressOnEnter
                        style={styles.mb5}
                    />
                )}
                <Button
                    success
                    text={translate('common.next')}
                    onPress={createGroup}
                    pressOnEnter
                />
        </>
    );
}, [createChat, personalData, selectedOptions, styles.mb5, translate]);

https://github.com/Expensify/App/blob/90412ec0c38cc1ccd0a1198cf1f4712474667906/src/pages/NewChatPage.tsx#L247
What alternative solutions did you explore? (Optional)
NA

Proposal
Please re-state the problem that we are trying to solve in this issue.
Allow creating a Group Chat with a single member.
What is the root cause of that problem?
new feature
What changes do you think we should make in order to solve the problem?
For that we need 3 steps (important step 3).

edit confirm button visibility here

// showConfirmButton={selectedOptions.length > 1}
showConfirmButton={selectedOptions.length > 0}

edit confirm button onConfirm here

// onConfirm={selectedOptions.length > 1 ? createGroup : undefined}
onConfirm={selectedOptions.length > 0 ? createGroup : undefined}

as we now allow creating a group chat with a single member, so we need to edit navigateToAndOpenReport here to differentiates between DM 1:1 and group chat with a single member, this function now is used in many places, so we need to confirm it will work fine in those cases (exist/not exist) for DM 1:1, group.

The only different now between DM 1:1 and group is participantAccountIDs.length > 1, and if we edit it to participantAccountIDs.length > 0 it will return true in both DM 1:1 and group with a single member, and if we pass isGroupChat as a parameter it will be hard code as we need to pass it in all cases (exist/not exist) for DM 1:1 and group.
So my fix for this is creating a new variable isSignalMemberGroupChat which differentiates between 1:1 DM and group with a single member, and the final isGroupChat definition will be  isSignalMemberGroupChat or participantAccountIDs.length > 1
// remove old isGroupChat
//  const isGroupChat = participantAccountIDs.length > 1;

// we need to handle `isSignalMemberGroupChat` in (exist/not exist)
// for not exist group >>> participant === 1 and isDraftGroup called by cretae group(explained below).
// for exist group >>> participant === 1 and chatType is group.

// isDraftGroup definition
// as we pass `optimisticReportID` only when we create a new group, so if it not undeifined that mean this is draft group.
// we can also create new param isDraftGroup and pass it true when creating the group (only in this place)
const isDraftGroup = !!optimisticReportID;

const isSignalMemberGroupChat = participantAccountIDs.length === 1 &&
  (isDraftGroup || ReportUtils.getChatByParticipants(participantAccountIDs)?.chatType === CONST.REPORT.CHAT_TYPE.GROUP);

// the new `isGroupChat` will be
const isGroupChat = isSignalMemberGroupChat || participantAccountIDs.length > 1;

We don't need backend changes.
What alternative solutions did you explore? (Optional)

Thanks for the proposal everyone.
Proposal from @nkdengineer and proposal from @ahmedGaber93 both solves the issue.
In @ahmedGaber93's proposal ReportUtils.getChatByParticipants(participantAccountIDs)?.chatType === CONST.REPORT.CHAT_TYPE.GROUP); will probably always return false as we skip group chat while looking for the report in getChatByParticipants function.
As navigateToAndOpenReport  is used only in case of new group chat or general chats(existing and new), I believe we will be okay going with the @nkdengineer's proposal. I don't think we would want to go with the 3rd step though.
@marcaaron Do we want to show  two buttons for group chat and normal chat so that user can create group chat alone or even with one other user as suggested in 3rd step in this proposal?

Should we include two buttons—one for group chat and one for normal chat—allowing users to create group chats either alone or with just one other user

No, we should make the change described in the issue. That would be a decision for the Design team to consider, not us.

Thanks for the proposal everyone.
Update Proposal from @abzokhattab looks good.
🎀 👀 🎀 C+ reviewed

No, we should make the change described in the issue. That would be a decision for the Design team to consider, not us.

@marcaaron I think we should discuss about this because if we want to create a group chat with only one user, we need to select another to allow go to next and then de-select this one. I have another ideal for this action that we can do the dropdown button that will have two option one is create a group chat one is create a single chat.
https://github.com/Expensify/App/assets/161821005/05215872-1828-4c1c-8718-7c681192927b
What do you think about it? And should we discuss with design team about this here?

I don't think we want to do anything like that yet. If anything, we'll plan to pull out "Start a group chat" as it's own item or own tab, and then from there it makes sense to start the group without selecting anyone. But as it currently stands, I don't think we want to change the start a chat functionality like that. Curious what others think though.

@marcaaron Can you please check this comment whenever you are available. Thanks.

@abzokhattab Any update? When can we expect PR?

@abzokhattab Can we get please get an update on this one?

Apologies for the delay.. was pretty busy the last few days.
working on it now.

the PR is ready here

monitoring https://github.com/Expensify/App/pull/41315

monitoring https://github.com/Expensify/App/pull/41315

monitoring https://github.com/Expensify/App/pull/41315

monitoring https://github.com/Expensify/App/pull/41315

PR deployed to production 5 days ago so payment is coming up.
Payouts due:

[ ] Contributor: $250 @abzokhattab (offer here https://www.upwork.com/nx/wm/offer/101981803)
[ ] Contributor+: $250 @sobitneupane

Upwork job is here.
@sobitneupane are you paid in NewDot?
@sobitneupane or @abzokhattab I know this is a new feature but do we need a regression test for this?

@sobitneupane are you paid in NewDot?

@Christinadobrzyn Yup. I will request payment in NewDot.

Regression Test Proposal

Add several people to a Group Chat via the Global Create menu
Advance to the "Confirm" page with the "Start group" button.
Remove all members
Verify that the "Start group" button still shows up allowing you to create a group with only yourself.
Click on "Start group" button and verify that the group is created properly without any errors.

DO we agree 👍 or 👎

LGTM

Regression test - https://github.com/Expensify/Expensify/issues/400167
Payment planned for tomorrow

Ready for payment - no regression tests reported.
payment summary here - https://github.com/Expensify/App/issues/40258#issuecomment-2134553063
Closing this out as complete but let me know if we need to reopen for anything!

$250 approved for @sobitneupane


vmlatency, checkup, preflight: Verify NetworkAttachmentDefinition exists

Currently the NetworkAttachmentDefinition (NAD) namespace and name are passed
to the checkup, w/o verifying it is exists.
In case the NAD does not exists, the checkup will hang until the timeout, because Kubevirt does
not validate that the NAD exists ending up with VMs that wont schedule.
With this PR changes the checkup will verify that before starting the checkup setup,
if the NAD does not exist it will fail early and wont hang until the timeout.
The checkup ClusterRoles manifest is updated with the required permissions to fetch the NAD.
Note that go.mod file is updated, but no new dependencies were added.
Signed-off-by: Or Mergi<EMAIL_ADDRESS>

/cc @orelmisan

Thank you for the PR @ormergi.
As I see it, there are multiple changes made in a single commit:

Changing the ClusterRole name to kubevirt-vm-latency-checker.
Addition of a permission to get NetworkAttachmentDefinition objects.
Implementation of a Pre-flight check.

Can you please make these changes gradually over several commits?

Note that 1-3 are all part of the same commit because each change dependent on the other (i,e:  (3) requires (2), (1) wont make sense w/o (3)).

Thank you for the PR @ormergi.
As I see it, there are multiple changes made in a single commit:

Changing the ClusterRole name to kubevirt-vm-latency-checker.
Addition of a permission to get NetworkAttachmentDefinition objects.
Implementation of a Pre-flight check.

Can you please make these changes gradually over several commits?

Note that 1-3 are all part of the same commit because each change dependent on the other (i,e: (3) requires (2), (1) wont make sense w/o (3)).

I argue that the ClusterRole rename to kubevirt-vm-latency-checker does stand on its own.

I argue that the ClusterRole rename to kubevirt-vm-latency-checker does stand on its own.

Do you insist of extracting the rename changes to new commit?

I argue that the ClusterRole rename to kubevirt-vm-latency-checker does stand on its own.

Do you insist of moving the rename changes to new commit?

At least write the reason for this change it in the commit message if you don't want to split it.

@orelmisan , please check if all your concerns have been responded.


Is this in working order for the RSM2?

Does it first require that I add virtual devices via the MultiChannel app?

I believe this should still work, but the multichannel app now handles the RSM2 and there is really no need to use this custom code anymore.  If using this code, you would use both the app and the device type here, and wouldn't use the multichannel app (I doubt they work together).
You probably just want to use the new multichannel capability.  I have at least one of my RSM2's now using that and the end result is the same.  I forget if you need to add the virtual switches first and then associate them, or if it creates them automatically when defining the multichannel device... sorry it's been a while since I reconfigured to test the built-in multichannel functionality, but I know it works!


Additional fixes for BatchIndexConcurrencyTest (details below)

Default indexing thread for created test file no longer
interfering with the indexing threads under test

Fixed indexFresh check to account for the fact that
an index could have been created (!=null) but the
corresponding indexing job is still in progress in which
case we also don't want to start another indexing thread

These fixes are in addition to the earlier fix that made
indexIfFresh synchronized

@manstis please take a look next week.

+1 to merge.

@csadilek thanks for looking into this and fixing it!


Add get function to make verify email browsable

Verify email was not browsable. get function added in order to solve that.
Fix issue: https://github.com/Tivix/django-rest-auth/issues/558

Coverage decreased (-0.08%) to 95.96% when pulling 24675e77b107385650b1ce8bc70bcea4fbc2acab on brunomichetti:fix/make-verify-email-browsable into 624ad01afbc86fa15b4e652406f3bdcd01f36e00 on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


Update device schema and UI to include new fields

Background
ReportStream has requested that we add two new fields to our device schema - Equipment Model ID and Test Kit Name ID.
Action requested
Corresponding RS ticket: https://app.zenhub.com/workspaces/onboarding--operations-6166edbd409257001e09f1b2/issues/cdcgov/prime-reportstream/7557
Acceptance Criteria

Add columns to schema
Update UI (add/edit device screens) to include these fields

Additional context
Ticket order:

#4831 (Update device schema and UI to include new fields)
#4897 (Backfill existing devices with new fields)
#4898 (Include new fields in single entry result submission)
#4896 (Update fields required for bulk results uploader)
#4895 (Update device code lookup tool with new fields)

@zdeveloper can you confirm that this ticket order works?


fix broken tests on 4.4

Pull Request check-list
Please make sure to review and check all of these items:

[ ] Does $ tox pass with this change (including linting)?
[ ] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
[ ] Is there an example added to the examples folder (if applicable)?
[ ] Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
Please provide a description of the change here.

Codecov Report
Patch and project coverage have no change.

Comparison is base (e4408a9) 60.21% compared to head (e4408a9) 60.21%.

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

Additional details and impacted files
@@           Coverage Diff           @@
##              4.4      #14   +/-   ##
=======================================
  Coverage   60.21%   60.21%
=======================================
  Files         116      116
  Lines       30281    30281
=======================================
  Hits        18233    18233
  Misses      12048    12048

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


test 2 public

#sign-off

@dphansen: I'm sorry - only the author of this article, @bradben, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

test comment #sign-off

Only Microsoft employees can sign off on PRs. Your PR will be routed for review.


Require armel versioned ARMv6 libedgetpu

Currently ARMv6 libedgetpu is build with armhf which default hard float enable
Which will cause ABI problem on some old openwrt devices because it is
default setup to soft float point and will not dynamic link the libedgetpu
Do you consider release armel versioned library for old device s?

@kqalea
Interesting that you are asking for this!
Even with our armv6 release, we don't see many users who are using it. I'll bring this up with our team and will let you know.

FYI this request will be fulfilled with this repo :)
https://github.com/google-coral/libedgetpu

This is great!!! Thanks


Can you please help to solve this white lane crossed captcha?

new CBL({
  preprocess: function(img) { /* ... */ },
  blob_debug: 'debugSegmented',
  character_set: '0123456789abcdefghijklmnopqrstuvwxyz',
  blob_min_pixels: 40,
  blob_max_pixels: 100000,
  pattern_width: 24,
  pattern_height: 24,
  pattern_maintain_ratio: true,
  perceptive_colorspace: true,
});

Original

img.convolute([
  [1, 2, 1],
  [1, 3, 1],
  [1, 2, 1],
], 1/13);

img.binarize(200);

img.colorRegions(10, true, 1);

As you can see the solver recognized wrong segments cause of these white lane crossed characters.
Is there any documentation provided? Cause I found no one and just randomly substitute example options...

It's hard to make assumptions off one image, but this one looks like it'll be difficult to solve.
A couple of things to look out for that might help...

If all images have the same number of characters then there's a setting you can use to always return that many characters
If there are always 6 characters and they're always in the same place, then you can use the exact character splitting methods (the examples folder has a few examples)
If the distortion is always the same on each image (like a letter "p" always has the exact same shape) then that'll help too, however since your image has two 6's and two p's that aren't identical I doubt this is the case.
A possible strategy might be to either try to find the right combination of blur filters to try to get rid of the lines, or write some custom code to detect the lines (there is some line removal code in the library, but it currently isn't written to remove a line that's the same color as the background)

Good luck

Here is more images:

Thanks anyway.


Azure Batch supports Managed Identities

According to the Portal and other documenatation Batch supports system-assigned managed identities but isn't listed on this page

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

ID: 726adac9-1848-01a6-9586-842c81f820f7
Version Independent ID: 249db438-8c95-8e69-f69e-8e48fcd0b359
Content: Azure Services that support managed identities - Azure AD
Content Source: articles/active-directory/managed-identities-azure-resources/services-support-managed-identities.md
Service: active-directory
Sub-service: msi
GitHub Login: @barclayn
Microsoft Alias: barclayn

@MatthewSteeples
Thanks for your feedback! We will investigate and update as appropriate.

I may have jumped the gun with this actually, as I'm struggling to get it working. The UserVoice request still indicates that this is outstanding, so I'm not sure of the status of it anymore!

@MatthewSteeples Thank you for your feedback. We are working on updating the article to reflect the current state of support for managed identities by various resource providers.  #please-close


Optimizing isPrime in C++

Tried my first issue. If any problem, please tell.

Improve performance of the loop to at least square root of n. don't upload configuration file.

Hey, the performance of the loop is already square root of n.
Regarding configuration file, I will ensure it never happens again.

@all-contributors please add @gauravmadan583  for code


Add handling of boolean literals to condition evaluation

This PR always checks for booleans in condition_evaluation function. One thing I thought about: we could skip this function entirely if we don't have is (not) defined inside a condition. However, it'd require to either go through all items of the conditions to see if one of them is is (not) defined or to pass a &mut bool to check if the expression encounters one of them.
Another thing I thought: instead of handling the evaluation in the generator, we could potentially handle it in the parser directly (and therefore, not having the branches in the AST if the previous if excluded it). Not sure if it's a good idea or not though.
Anyway, this PR is much simpler than that since it only adds checks for boolean literals. :)

Good idea! Except for the missing newline, approved. :)

Woups. Fixed. :laughing:


ScrollView broken in 0.56.0 for a brownfield edge case in iOS

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
  React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.5
      CPU: x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
      Memory: 2.86 GB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
      Yarn: 1.7.0 - ~/.nvm/versions/node/v8.11.3/bin/yarn
      npm: 5.6.0 - ~/.nvm/versions/node/v8.11.3/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
    IDEs:
      Android Studio: 3.1 AI-173.4819257
      Xcode: 9.4/9F1027a - /usr/bin/xcodebuild
    npmPackages:
      react: 16.4.1 => 16.4.1
      react-native: 0.56.0 => 0.56.0
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
This commit broke ScrollView in certain cases. Details for it are copied from this comment.

In our brownfield project, we had a UIView category (UIView+Something.h) which implemented the protocol UITextViewDelegate which in turn extends UIScrollViewDelegate. Due to this, after upgrading to RN 0.56, our scroll views stopped working coz these lines are missing in the new else if block:
_contentView = view;
RCTApplyTransformationAccordingLayoutDirection(_contentView, self.reactLayoutDirection);

It took us a long time to investigate the issue, so I was wondering if you knew whether this usecase was missed out when this block was added? Is there is a better way to handle this? I know most greenfield apps won't run into this but you never know what legacy code brownfield apps have and making this better might avoid the pain we had to go through after the upgrade.

Reproducible Demo
Its hard to extract an example to share this but the description is self sufficient to identify the issue.
cc'ing @PeteTheHeat as he was planning on looking into this issue and coming up with an appropriate fix.

@karanjthakkar ^ should fix the issue

Thanks! 💯


Get value from viewChild searchbar is not getting

cause error of getValue is undefine
code is like
@ViewChild('searchbar') searchbar: any;
and calling function
console.log(this.searchbar.getValue());
in autocomplete.componenent.ts
public getValue() {
this.keyword
return
}
}
in <ion-autocomplete #searchbar></ same>

I am not sure I am following (the code is a little bit messy to understand :) )
Anyways, Iv'e checked it up and everything looks fine:
my home.ts:
import {Component, ViewChild} from '@angular/core';
import { NavController } from 'ionic-angular';
import {CompleteTestService} from "../../app/auto";
import {AutoCompleteComponent} from "ionic2-auto-complete/dist";

@Component({
  selector: 'page-home',
  templateUrl: 'home.html'
})
export class HomePage {

  @ViewChild('searchbar')
  searchbar: AutoCompleteComponent;

  constructor(public navCtrl: NavController, private completeTestService: CompleteTestService) {

  }

  clicked() {
    console.log(this.searchbar.getValue());
  }

}

And my home.html:
<ion-header>
  <ion-navbar>
    <ion-title>Home</ion-title>
  </ion-navbar>
</ion-header>

<ion-content padding>
  <ion-auto-complete [dataProvider]="completeTestService" #searchbar></ion-auto-complete>
  <button (click)="clicked()">click</button>
</ion-content>

<EMAIL_ADDRESS>

@rkmahale17 I believe that this is not the same issue we were talking about

No this is not that  issuse , after upgrating to<EMAIL_ADDRESS>attribute issue solved ..but this function calling is not happening......

@rkmahale17 please review my latest comment in #42

Yes.... it is working....
Thank You so much......


Missing class AbstractSymplifyConsoleApplication

Hello,
after latest composer update which updated following:
  - Upgrading symplify/autowire-array-parameter (v9.3.1 => v9.3.6)
  - Upgrading symplify/composer-json-manipulator (v9.3.1 => v9.3.6)
  - Upgrading symplify/console-color-diff (v9.3.1 => v9.3.6)
  - Upgrading symplify/console-package-builder (v9.3.1 => v9.3.6)
  - Upgrading symplify/easy-testing (v9.3.1 => v9.3.6)
  - Upgrading symplify/package-builder (v9.3.1 => v9.3.6)
  - Upgrading symplify/rule-doc-generator-contracts (v9.3.1 => v9.3.6)
  - Upgrading symplify/set-config-resolver (v9.3.1 => v9.3.6)
  - Upgrading symplify/skipper (v9.3.1 => v9.3.6)
  - Upgrading symplify/smart-file-system (v9.3.1 => v9.3.6)
  - Upgrading symplify/symplify-kernel (v9.3.1 => v9.3.6)

I get an error while running ecs:
 [ERROR] Class 'Symplify\SymplifyKernel\Console\AbstractSymplifyConsoleApplication' not found

Hi,
try bumping ecs to 9.3.6 as well. It's fixed there

There is no such a tug https://packagist.org/packages/symplify/easy-coding-standard :)

Good catch :+1: The release build was broken for ECS package.
The 9.3.10 should be fine now. How does it work for you? :)

Thanks it works now :)


Cancel previous ongoing runs in the PR/branch when you push new commits

Description (required)
Fixes #4781
What changes did you make?
Adding this to cancel any previous ongoing builds when you push new commits to a PR or branch. This will ensure that we don't run out of free build mins.
Note: Please ensure that you have read CONTRIBUTING.md if this is your first pull request.

@madhurgupta10 can you check now ?

@RishiKumarRay Now revert the extra changes, this will also test if ongoing builds are cancelled or not

@madhurgupta10 how to revert?

@madhurgupta10 how to revert?

You can push another commit by removing the unnecessary changes

@madhurgupta10 there was only some extra spaces i have removed that please check

@nicolas-raoul yeah sure i will remember this. Thanks for helping out and merging @madhurgupta10


Support 1.7 context package

#24

Mostly backwards compatible

LogHandler signature has changed for 1.7
Middleware and Afterware signature has changed for 1.7

Supports both x/net/context and context.
Middleware will modify *http.Request's context.

TODO

[x] Test on 1.6
[x] Attempt to reduce duplicated code

I'm going to publish this as 2.0.0. As much as I'd like to break more stuff I will leave it maximally compatible.
If this breaks your code you can:

make a few small changes in your LogHandler
or use Go 1.6
or use gopkg.in/guregu/kami.v1


Automated cherry pick of #2339: fix managered disk reset not update disk size

Cherry pick of #2339 on release/2.10.0.
#2339: fix managered disk reset not update disk size

Codecov Report

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

@@               Coverage Diff                @@
##             release/2.10.0   #2342   +/-   ##
================================================
  Coverage                  ?   7.55%
================================================
  Files                     ?     410
  Lines                     ?   63644
  Branches                  ?       0
================================================
  Hits                      ?    4811
  Misses                    ?   58422
  Partials                  ?     411

Flag
Coverage Δ

#aFlag
7.55% <ø> (?)

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 ed6c69b...5dcfa16. Read the comment docs.

/approve
/lgtm


[IOTDB-37]A WAL check tool script is desired

Add start-WalChecker.sh to start wal file check tool;
Add stop-WalChecker.sh to stop WalChecker server;

I will close this PR because something wrong on Win OS.


Shallow fusion with N-gram Language Model

Hi every one, do we have a shallow fusion implementation with N-gram Language Model (e.g Kenlm or Srilm) instead of Neural Language Model? If not, can you give me some instructions to do shallow fusion with N-gram and then maybe I can have a PR for this. Thanks in advance!

Hi, n-gram LM is usually much weaker than a neural network LM (such as RNN/Transformer). We only supported RNNLM rescoring, see #125.


Laravel 5 compatibility

We are using Laravel 5.8 in our project and we are not ready to switch to Laravel 6. Any chance you can make your package compatible with Laravel 5.8 and onwards?
 Problem 1     - Conclusion: remove laravel/framework v5.8.35     - Conclusion: don't install laravel/framework v5.8.35     - stancl/tenancy 2.x-dev requires illuminate/support ^6.0 -> satisfiable by illuminate/support[6.x-dev, v6.0.0, v6.0.1, v6.0.2, v6.0.3, v6.0.4, v6.1.0, v6.2.0, v6.3.0].     - stancl/tenancy v2.1.0 requires illuminate/support ^6.0 -> satisfiable by illuminate/support[6.x-dev, v6.0.0, v6.0.1, v6.0.2, v6.0.3, v6.0.4, v6.1.0, v6.2.0, v6.3.0].     - don't install illuminate/support 6.x-dev|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.0.0|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.0.1|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.0.2|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.0.3|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.0.4|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.1.0|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.2.0|don't install laravel/framework v5.8.35     - don't install illuminate/support v6.3.0|don't install laravel/framework v5.8.35     - Installation request for laravel/framework (locked at v5.8.35, required as 5.8.*) -> satisfiable by laravel/framework[v5.8.35].     - Installation request for stancl/tenancy ^2.1 -> satisfiable by stancl/tenancy[2.x-dev, v2.1.0].

I can't speak for @stancl but I notice in https://github.com/stancl/tenancy/issues/123 that it appears it was intentional to drop Laravel 5 support in the v2.0 branch while taking the opportunity to break backward compatibility in order to accomplish the goals of v2.
v1 will work with Laravel 6, but the configs and docs and internal methodology are different and it'll be harder to get support for older versions.

@drbyte is right:

it was intentional to drop Laravel 5 support in the v2.0 branch while taking the opportunity to break backward compatibility in order to accomplish the goals of v2.

I think that v2 doesn't depend on any 6.x-specific features, but I'm a bit hesitant to add 5.8 support.

I don't want to be stuck in 5.8 feature set. If I added 5.8 support, I'd reserve the right to remove it at any time without making a new major release and without porting future bugfixes to some branch for 5.8.
The CI takes like 5 minutes to run the tests already, not sure I want to increase this to 10 minutes.

@moazam1

we are not ready to switch to Laravel 6

Why exactly? Because of your dependencies? I suggest asking them to add L6 support instead of asking other packages to add support for a dead Laravel version.
L6 is an LTS version, so everyone should be motivated to upgrade. I'll think about if I want to add L5 support, but currently I'm leaning to the side of "no". It's just not worth the additional complexity for me, to support a few apps that haven't upgraded to L6 yet.

Yeah, I won't be adding this. L5.8 is unsupported for almost 2 months now and L6 is an LTS release.

Thanks @stancl we are dependent on other libraries. I decided to write code myself.


feat(api,robot-server): Robot server rpc http ff

Adds a feature flag (enableHttpProtocolSessions) that enables the creation of new HTTP protocol sessions, but disables creating RPC protocol sessions via SessionManager.
This is for a future beta release of experimental HTTP protocol sessions.
review requests
Please review the naming and the wording of the feature flag description.
testing
Tested with local robot-server and run-app

connect to robot
load a protocol
return to main page and enable enableHttpProtocolSessions
load a protocol and see this error:

risk
low.
closes #6305

Do we need this? I think the two workflows are pretty orthogonal in practice - people won't really be mixing them.

You're right that we don't strictly need it. A couple benefits I can think of:

it frees us from testing that you can use RPC and HTTP protocols interchangeably.
It advertises that the feature is there (not sure this makes sense)

Do we need this? I think the two workflows are pretty orthogonal in practice - people won't really be mixing them.

You're right that we don't strictly need it. A couple benefits I can think of:
* it frees us from testing that you can use RPC and HTTP protocols interchangeably.

That's a pretty good reason, but given that you can only execute calibration via RPC (at least right now) it kinda makes me wonder if it's not too early. Since we're in theory doing this so it can be used by hackers, to me it's better to put the burden of "please make sure you are careful about this" on them as opposed to the burden of frequently restarting the robot.
On the restarting robot front - to be sure that the two interfaces aren't being used interchangeably, I think we'd either have to

somehow detect when the feature flag is changed in the rpc/http session managers and delete owned sessions if it changes away from them
or
mark this feature flag as requires restart.

Otherwise, you could flip the flag with a session of the now-wrong type existing, and you're back in that scenario where we have to test interchangeability.

Do we need this? I think the two workflows are pretty orthogonal in practice - people won't really be mixing them.

You're right that we don't strictly need it. A couple benefits I can think of:
* it frees us from testing that you can use RPC and HTTP protocols interchangeably.

That's a pretty good reason, but given that you can only execute calibration via RPC (at least right now) it kinda makes me wonder if it's not too early. Since we're in theory doing this so it can be used by hackers, to me it's better to put the burden of "please make sure you are careful about this" on them as opposed to the burden of frequently restarting the robot.
On the restarting robot front - to be sure that the two interfaces aren't being used interchangeably, I think we'd either have to

somehow detect when the feature flag is changed in the rpc/http session managers and delete owned sessions if it changes away from them
or
mark this feature flag as requires restart.

Otherwise, you could flip the flag with a session of the now-wrong type existing, and you're back in that scenario where we have to test interchangeability.

I like your thinking. I'm happy to drop this FF.  @mcous @sanni-t ?

to me it's better to put the burden of "please make sure you are careful about this" on them as opposed to the burden of frequently restarting the robot.

IMO a feature flag is the best mechanism we have to say "please make sure you are careful about this". We're developing an experimental, subject-to-change, in some ways incomplete feature. I think putting it behind a feature flag (with restart required) is the responsible move here for our users

to me it's better to put the burden of "please make sure you are careful about this" on them as opposed to the burden of frequently restarting the robot.

IMO a feature flag is the best mechanism we have to say "please make sure you are careful about this". We're developing an experimental, subject-to-change, in some ways incomplete feature. I think putting it behind a feature flag (with restart required) is the responsible move here for our users

I'd agree with this if this had a UI associated with it. But if you enable this feature flag, use of the OT-2 through the runapp is completely unchanged, right? As long as that stays true, and as long as the only way to interact with this is to write your own client code, IMO we don't need the flag.

to me it's better to put the burden of "please make sure you are careful about this" on them as opposed to the burden of frequently restarting the robot.

IMO a feature flag is the best mechanism we have to say "please make sure you are careful about this". We're developing an experimental, subject-to-change, in some ways incomplete feature. I think putting it behind a feature flag (with restart required) is the responsible move here for our users

I'd agree with this if this had a UI associated with it. But if you enable this feature flag, use of the OT-2 through the runapp is completely unchanged, right? As long as that stays true, and as long as the only way to interact with this is to write your own client code, IMO we don't need the flag.

You both feel pretty strongly about this. I don't really care either way.  How about we reassess the decision about the feature flag until we get the HTTP PR merged? Let's get a feel for how things work in practice.


PR for Serialization with Rootname

Serialization with Rootname using XML and JSON

Hi @AnsaBaby,
I am going to close this PR. You will need to create a new one with only the code for the current Jira in it. Please refer to the author documentation regarding PR creation.
You need to create a new branch in your own forked repository for each Jira you are working on and then create the PR from that branch. There should not be code from other commits in the PR, only the code related to the Jira. Also the PR should be named with the Jira number.
All of this is listed in the document: "for authors - 3 - implementation code".
Cheers!


kernel: Add typedef forward declaration check for portability.

'k_thread_stack_t' and 'k_thread_entry_t' typedefs are forward-declared
in sys/arch_interface.h and, because this header is included by
arch/cpu.h, which is, in turn, included by kernel_includes.h, the
typedefs effectively get re-declared in kernel.h.
While this does not cause a compilation error when compiling with GCC
4.6 and above, GCC 4.5 and below strictly enforce the C99 standard and
do not allow redeclaration of the same typedef in the same scope.
This commit adds sys/arch_interface.h header inclusion checks in
kernel.h to skip declaration of the forward-declared typedefs.
Signed-off-by: Stephanos Ioannidis<EMAIL_ADDRESS>

Sample compilation error in GCC 4.5.1 due to redeclaration of the same typedefs:
[7/131] Building C object zephyr/CMakeFiles/offsets.dir/arch/tc32/core/offsets/offsets.c.obj
FAILED: zephyr/CMakeFiles/offsets.dir/arch/tc32/core/offsets/offsets.c.obj
ccache /opt/tc32/bin/tc32-elf-gcc -DBUILD_VERSION=v2.1.0-rc1-94-g1a5d7d13d8d8 -DCHIP_TYPE=7 -DKERNEL -D_FORTIFY_SOURCE=2 -D__ZEPHYR__=1 -I../../../../../kernel/include -I../../../../../arch/tc32/include -I../../../../../include -I../../../../../include/drivers -Izephyr/include/generated -I../../../../../soc/tc32/tlsr/tlsr826x -I../../../../../soc/tc32/tlsr/include -I../../../../../ext/hal/telink/.. -I../../../../../ext/hal/tmsis/include -I../../../../../subsys/testsuite/include -I../../../../../subsys/testsuite/ztest/include -isystem ../../../../../lib/libc/minimal/include -isystem /opt/tc32/lib/gcc/tc32-elf/4.5.1.tc32-elf-1.5/include -isystem /opt/tc32/lib/gcc/tc32-elf/4.5.1.tc32-elf-1.5/include-fixed -Os -imacros/home/stephanos/Dev/zephyr-stephanos/zephyr/tests/kernel/threads/thread_apis/build_tc32/zephyr/include/generated/autoconf.h -ffreestanding -fno-common -g -imacros/home/stephanos/Dev/zephyr-stephanos/zephyr/include/toolchain/zephyr_stdint.h -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-pointer-sign -Wpointer-arith -Werror=implicit-int -fno-jump-tables -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-strict-overflow -fno-reorder-functions -fno-defer-pop -ffunction-sections -fdata-sections -std=gnu99 -nostdinc -MD -MT zephyr/CMakeFiles/offsets.dir/arch/tc32/core/offsets/offsets.c.obj -MF zephyr/CMakeFiles/offsets.dir/arch/tc32/core/offsets/offsets.c.obj.d -o zephyr/CMakeFiles/offsets.dir/arch/tc32/core/offsets/offsets.c.obj   -c /home/stephanos/Dev/zephyr-stephanos/zephyr/arch/tc32/core/offsets/offsets.c
In file included from /home/stephanos/Dev/zephyr-stephanos/zephyr/arch/tc32/core/offsets/offsets.c:25:0:
../../../../../include/kernel.h:367:40: error: redefinition of typedef 'k_thread_stack_t'
../../../../../include/sys/arch_interface.h:44:40: note: previous declaration of 'k_thread_stack_t' was here
../../../../../include/kernel.h:387:16: error: redefinition of typedef 'k_thread_entry_t'
../../../../../include/sys/arch_interface.h:46:16: note: previous declaration of 'k_thread_entry_t' was here
ninja: build stopped: subcommand failed.

Closing this PR as #21018 removed the duplicate definitions.


Added email notification on Yarn application shutdown

Also added an option to control the maximum allowed number consecutive number of failures to get the ApplicationReport in case the Yarn cluster is down, e.g., fir maintenance.
Signed-off-by: Yinan Li<EMAIL_ADDRESS>

@sahilTakiar @zliu41 can you review?

LGTM

@zliu41 updated 958b45f.

LGTM

@zliu41 actually I just realized using a plain integer has one problem: the getApplicationReportFailureCount is shared and updated in both event handlers handleApplicationReportArrivalEvent and handleGetApplicationReportFailureEvent, which may be called by different threads. So for any change to getApplicationReportFailureCount in any of the two event handlers to be visible to the other, it at least needs to be volatile. So I still think AtomicInteger is a good choice here.


Fix stripe test

The two test cases, testRecurring1, and testStoreWithoutBillingAddress, are not working. This is a minor fix to the two tests.
In addition to the fix, since I use PyCharm IDE and I believe many others do as well, I suppose it wouldn't hurt to ignore the aux files folder in .gitignore.

There seem to be other failing tests as well which should be fixed while we're at it.

@tyc85 Thanks for the fixes to the tests. Merged them.
@tuxcanfly Beanstream, Paylane, Braintree and Eway have closed down our sandbox accounts and hence tests fail. We will have to go through the process or reopening accounts on them.


Maven Build lombok Failure

Hi,
I tried to create the msal4j jar file with Maven using the instructions listed in https://github.com/AzureAD/microsoft-authentication-library-for-java/wiki/Maven.
mvn clean results in a BUILD SUCCESS
But mvn package gives me the below error. How can I resolve this? Thanks!
[INFO] Scanning for projects...                                                                                [INFO]                                                                                [INFO] ---------------------< com.microsoft.azure:msal4j >---------------------                                                                                [INFO] Building msal4j 1.4.0                                                                                [INFO] --------------------------------[ jar ]---------------------------------                                                                                [INFO]                                                                                [INFO] --- lombok-maven-plugin:<IP_ADDRESS>:delombok (default) @ msal4j ---                                                                                [WARNING] Unable to detect tools.jar; java.home is C:\Program Files\Java\jdk-13.0.2                                                                                WARNING: An illegal reflective access operation has occurred                                                                                WARNING: Illegal reflective access by lombok.javac.JavacTreeMaker$TypeTag to method com.sun.tools.javac.code.Type.getTag()                                                 WARNING: Please consider reporting this to the maintainers of lombok.javac.JavacTreeMaker$TypeTag                                                                          WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations                                                                      WARNING: All illegal access operations will be denied in a future release                                                                                [INFO] ------------------------------------------------------------------------                                                                                [INFO] BUILD FAILURE                                                                                [INFO] ------------------------------------------------------------------------                                                                                [INFO] Total time:  10.007 s                                                                                [INFO] Finished at: 2020-02-24T15:15:38-05:00                                                                                [INFO] ------------------------------------------------------------------------                                                                                [ERROR] Failed to execute goal org.projectlombok:lombok-maven-plugin:<IP_ADDRESS>:delombok (default) on project msal4j: Unable to delombok: InvocationTargetException: pat -> [Help 1]                                                                                [ERROR]                                                                                [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.                                                                                [ERROR] Re-run Maven using the -X switch to enable full debug logging.                                                                                [ERROR]                                                                                [ERROR] For more information about the errors and possible solutions, please read the following articles:                                                                  [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

@brendangl What JDK version are you using?

@sangonzal I was using jdk-13.0.2. I switched to jdk1.8.0_241 and then it worked. So there's a compatibility issue with the newer JDK version, but for my purposes, it's fine to use JDK 1.8.

@brendangl Yes there seems to be a problem with org.projectlombok:lombok-maven-plugin when using JDK12 and up - need to look in it.
You can run mvn install -DskipITs. I've updated the wiki with this info.

msal4j jar can always be downloaded from https://mvnrepository.com/artifact/com.microsoft.azure/msal4j,
currently we support minimal JDK 8+,
so have corresponding maven-compiler-plugin and lombok plugins configuration


[Scheduled Maintenance] Site down upgrade to v0.18.5

I'm upgrading lemmy to the latest release (v0.18.5), so it will be down, butit shouldn't take more then a few minutes!

Upgrade complete.


added link to api docs

@edenhill

LGTM
Also include the docs link in the github project description, it seems to be common practice.
Example:
https://github.com/confluentinc/confluent-kafka-python


[Snyk] Upgrade html-snapshots from 2.10.3 to 2.10.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.10.3 to 2.10.4.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 days ago, on 2023-09-29.

Release notes

Package name: html-snapshots

2.10.4 -<EMAIL_ADDRESS>

2.10.3 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

5b2aa50 Merge pull request #509 from localnerve/rc-2.10.4
f8c0f0d @ 2.10.4 -<EMAIL_ADDRESS>393d685 Merge pull request #508 from localnerve/snyk-upgrade-08f0725f98142f438ee76346d4da160c
ed310cb fix: upgrade html-snapshots from 2.10.2 to 2.10.3
a634a78 Merge pull request #507 from localnerve/snyk-upgrade-eb7b508dc7633a42ce562d2428f19122
901cf9d fix: upgrade html-snapshots from 2.10.1 to 2.10.3
842882e Merge pull request #506 from localnerve/snyk-upgrade-326fc65fb9a49e33605e7ffc552a4caa
c4d8f69 fix: upgrade html-snapshots from 2.10.2 to 2.10.3
b77083c Merge pull request #505 from localnerve/snyk-upgrade-1dc3d1faadc83252155740ce1421c775
1e61fa9 fix: upgrade html-snapshots from 2.10.2 to 2.10.3
8f4c990 Merge pull request #504 from localnerve/snyk-upgrade-512b199fa2febd4169ea7ce6b7f6f9db
322000d fix: upgrade html-snapshots from 2.10.1 to 2.10.3
f2ba4c9 Merge pull request #503 from localnerve/snyk-upgrade-dbd02b09938f941c5eeea28af563d153
8c5bb4a fix: upgrade html-snapshots from 2.10.1 to 2.10.2
d5bf6eb Merge pull request #502 from localnerve/snyk-upgrade-ff002ce6d9dc36ce5a586e9d0a599ee8
174ac65 fix: upgrade html-snapshots from 2.10.1 to 2.10.2

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same when pulling 7fbab2fb1b8d155b6879a83bac757353f3d3d5ff on snyk-upgrade-0b4776082dbf9812fc4799dc620c545f into ad62f07be0142dddaf1931b9a25c292086af6b3e on master.


Npm install failure on Kubuntu

Hello, here the error I keep receiving:

npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm ERR! Linux 4.4.0-31-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "jshint" "gulp-util" "gulp-sass" "gulp-cssnano" "gulp-autoprefixer" "gulp-sourcemaps" "gulp-jshint" "jshint-stylish" "gulp-uglify" "gulp-concat" "gulp-rename" "gulp-plumber" "gulp-bower" "gulp-babel" "browser-sync"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm

ERR<EMAIL_ADDRESS>install: node scripts/install.js
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node scripts/install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs node-sass
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.

I can see thath dependency check failed and my system architecture is the source fo the problem. Try to google it but found only few old posts, confirmed not be helpful anymore.
That's why I am contacting you as stated in the terminal output.

I tried npm install --no-optional with similar effect

I've found the solution here: https://github.com/sass/node-sass/issues/1601
On ubuntu 16.04  you have to create symlink, but I dont like doing something when I am not sure what is it for so if you can explain it to me I will much appreciate it.
ln -s /usr/bin/nodejs /usr/bin/node

http://stackoverflow.com/questions/26320901/cannot-install-nodejs-usr-bin-env-node-no-such-file-or-directory/26320915
The node package was registered a long time ago for something else, so it conflicts unless you use the link workaround. The old wiki page from the nodejs wiki seems to have been eaten from their repo reorg, but it explained this.

We used to have a work around for this which was removed in 3.5.0. I think
it'll be worth resurrecting. This has come up a couple times a this month.
On 25 Jul 2016 3:22 PM, "Nick Schonning"<EMAIL_ADDRESS>wrote:

http://stackoverflow.com/questions/26320901/cannot-install-nodejs-usr-bin-env-node-no-such-file-or-directory/26320915
The node package was registered a long time ago for something else, so it
conflicts unless you use the link workaround. The old wiki page from the
nodejs wiki seems to have been eaten from their repo reorg, but it
explained this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/1641#issuecomment-234843200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWPS1DhucJfgwLo0X0YWYSokxvfdIks5qZEgIgaJpZM4JToQ5
.


Use correct label for extra relays when editing user

The existing code copies the useracctype element to useracctypeN after the label has been changed to that of the first relay. This means that the user edit dialogue will show the same label for all relays, rather than the correct label for the relay in question. This sets the label after the element has been copied so they will all be correct.

Thank you so much for your contributions @mcnewton !
I'm going to review everything soon, sorry but in these last few months I've been busy with other stuff. I'll try everything in the next few days!

@matjack1 No worries, I know what it's like finding time to review PRs! Thanks for the cool project :)

Nice, thanks!


Conda channel installation painfully slow

conda install -c esmvalgroup esmvaltool -c conda-forge -c birdhouse takes forever and half to complete ie Solving environment takes some 20min and I got pissed off and killed it - running it again now to see if it actually creates the env and installs the tool correctly so I'll update this issue as I have more info. But as it stands now, this needs serious mending :hammer_and_pick:

This is a problem with Conda, not really an esmvaltool issue. Try updating or re-installing conda, that helps sometimes.
By the way, things get even worse if you try to install all the R dependencies too.

yep, but why is it so slow from the conda side? I updated to conda-4.6.7_py37 and it's still solving the environment since 10min ago - I will investigate a bit more, but conda installations in general are not so slow (or they shouldn't be, even for packages with such large dep trees like ours). Anyways, I'll dig more today and let you guys know


OnScrolled Event not firing on iOS 17.3.1 device when I scroll the VLV content

Hello, I've added an OnScrolled event handler to my VLV declaration in XAML, like so:
        <vlv:VirtualListView
                           Adapter="{Binding SettingsAdapter}"
                           OnScrolled="OnVirtualListViewScrolled"
                           BackgroundColor="Transparent"
                           Margin="8, 0"
                           HorizontalOptions="FillAndExpand"
                           VerticalOptions="FillAndExpand" >

I added the method to the page code-behind like so:
private void OnVirtualListViewScrolled(object? sender, ScrolledEventArgs e)
{
    Console.WriteLine($"VLV Scroll Position: ScrollX = {e.ScrollX}, ScrollY = {e.ScrollY}");
}

I set a breakpoint on the Console.WriteLine statement, then built/deployed to an iPhone SE running iOS 17.3.1, navigated to the page containing the VLV component and scrolled the content up and down. The breakpoint never gets hit.
Have I misunderstood the use of this event? Thanks!

Same here!


ENH: add next and prev document metadata to notebook meta

Implements #317
This PR will add next and previous document metadata to the generated notebooks.
Such as:
 "metadata": {
  "date":<PHONE_NUMBER>.837961,
  "filename": "index.rst",
  "kernelspec": {
   "display_name": "Python",
   "language": "python3",
   "name": "python3"
  },
  "next_doc": {
   "link": "index_toc",
   "title": "Table of Contents"
  },
  "title": "Python Programming for Economics and Finance"
 },

If metadata is valid:

next is referenced by nb.metadata.next_doc
previous is referenced by nb.metadata.prev_doc

[ ] fix travis

@DrDrij this implements the changes we discussed. Could you setup PR's for the theme's. If you would like to debug let's iterate on lecture-python-programming initially as it has lowest build times.

@DrDrij once this passes travis I will merge and issue a new release of sphinxcontrib-jupyter

Perfect @mmcky  I will update the themes over the weekend!


Phase lock

=================
Benchmark results
=================
bench                          | fail |   n |       perf |   sem% |   std% | peak_memory |      score | weight
brax                           |    0 |   1 |  724034.46 |   0.5% |   4.2% |        6448 |  724034.46 |   1.00
dlrm                           |    0 |   1 |  389259.69 |   0.7% |   5.3% |        7624 |  389259.69 |   1.00
bf16                           |    0 |   8 |     293.38 |   0.3% |   7.2% |        5688 |    2362.93 |   0.00
fp16                           |    0 |   8 |     290.09 |   0.2% |   4.3% |        5688 |    2330.33 |   0.00
fp32                           |    0 |   8 |      19.13 |   0.0% |   0.8% |        6066 |     153.18 |   0.00
tf32                           |    0 |   8 |     146.85 |   0.2% |   4.5% |        6066 |    1179.77 |   0.00
bert-fp16                      |    0 |   8 |     161.63 |   0.9% |  13.5% |         nan |    1322.86 |   0.00
bert-fp32                      |    0 |   8 |      40.11 |   0.5% |   8.2% |       35464 |     325.34 |   0.00
bert-tf32                      |    0 |   8 |     130.62 |   0.9% |  13.5% |        1504 |    1068.94 |   0.00
bert-tf32-fp16                 |    0 |   8 |     161.43 |   0.9% |  14.4% |         nan |    1323.11 |   3.00
reformer                       |    0 |   8 |      62.26 |   0.3% |   5.9% |       29304 |     501.64 |   1.00
t5                             |    0 |   8 |      51.27 |   0.4% |   9.8% |        5462 |     414.95 |   2.00
whisper                        |    0 |   8 |     162.55 |   2.3% |  51.4% |       36724 |    1305.80 |   1.00
llama                          |    0 |   8 |     471.59 |   4.5% |  80.5% |       32326 |    3580.15 |   1.00
opt-1_3b                       |    0 |   1 |      30.24 |   1.3% |   6.7% |       41854 |      30.24 |   5.00
opt-6_7b                       |    0 |   1 |      13.64 |   0.0% |   0.2% |       55202 |      13.64 |   5.00
stargan                        |    0 |   8 |      53.89 |   2.4% |  52.8% |       40710 |     432.30 |   1.00
super-slomo                    |    0 |   8 |      31.33 |   2.6% |  56.3% |       69972 |     250.82 |   1.00
davit_large                    |    0 |   8 |     322.84 |   0.6% |  13.4% |       34118 |    2614.22 |   1.00
davit_large-multi              |    0 |   1 |    2187.11 |   2.1% |  16.7% |         nan |    2187.11 |   5.00
focalnet                       |    0 |   8 |     376.81 |   0.7% |  15.5% |       27464 |    3052.06 |   2.00
resnet152                      |    0 |   8 |     519.42 |   0.9% |  21.6% |       31216 |    4196.06 |   1.00
resnet152-multi                |    0 |   1 |    3490.39 |   3.5% |  27.7% |         nan |    3490.39 |   5.00
convnext_large-fp16            |    0 |   8 |     306.49 |   2.1% |  32.5% |       31292 |    2503.00 |   0.00
convnext_large-fp32            |    0 |   8 |      60.60 |   0.8% |  12.6% |       51200 |     495.10 |   0.00
convnext_large-tf32            |    0 |   8 |     158.14 |   0.9% |  14.6% |       53566 |    1296.65 |   0.00
convnext_large-tf32-fp16       |    0 |   8 |     315.07 |   1.9% |  29.4% |       31292 |    2574.94 |   3.00
regnet_y_128gf                 |    0 |   8 |     115.83 |   0.5% |  10.1% |       35514 |     937.53 |   2.00
resnet152-ddp                  |    0 |   1 |    4305.36 |   4.3% |  32.4% |       31740 |    4305.36 |   0.00
resnet50                       |    0 |   8 |     906.21 |   2.4% |  52.1% |       14686 |    7311.99 |   1.00
resnet50-noio                  |    0 |   8 |    1161.23 |   0.3% |   7.3% |       31350 |    9370.68 |   0.00

Scores
------
Failure rate:       0.00% (PASS)
Score:             802.90
```


Addition of CLI flags for all configuration parameters.

The replicator agent now supports passing all configuration params
as CLI flags. The CLI flags can be used in conjunction with config
files, CLI flags taking precedent.
Coses #60

This looks lovely, nice work @jrasell this will add much-needed flexibility!


Better travis CI

Compile for different modes.
Add support windows CI.

Signed-off-by: Alexander Kurbatov<EMAIL_ADDRESS>

Recreated for master to avoid mess with history. Seems that the Dev branch is not in sync with master.

Example CI:
https://github.com/Cryptyc/Sc2LadderServer/pull/138/checks?check_run_id=94916826

Forgot to patch install.sh.
Fixed.


mp: autophagic vacuole formation

Hello,
I think the autophagic vacuole is an organelle,
therefore "autophagic vacuole formation" (GO:45) should
have as a parent "organelle organization and
biogenesis" (GO:6996).
Thanks
Pascale
Reported by: pgaudet
Original Ticket: "geneontology/ontology-requests/3194":https://sourceforge.net/p/geneontology/ontology-requests/3194

thanks www.movied.org


Deploy:fetch errored

Hi, I need some help debugging the following stack trace. Seems to be a local problem and it does not show a lot of info.
'deploy:fetch' errored after 4.07 s
Error: Command failed: /bin/sh -c git fetch --depth=1 shipit --prune && git fetch --depth=1 shipit --prune "refs/tags/*:refs/tags/*"
Warning: Permanently added the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts.
From github.com:jsdario/netbeast-api
 * [new branch]      master     -> shipit/master
 * [new branch]      production -> shipit/production
 * [new branch]      stage      -> shipit/stage

    at ChildProcess.exithandler (child_process.js:203:12)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:172:7)
    at maybeClose (internal/child_process.js:818:16)
    at Socket.<anonymous> (internal/child_process.js:319:11)
    at emitOne (events.js:77:13)
    at Socket.emit (events.js:169:7)
    at Pipe._onclose (net.js:469:12)

Hi, I have the same problem
I have difficulty working through the SSH, while working through HTTPS so good
my config
module.exports = function (shipit) {
  require('shipit-deploy')(shipit);
  shipit.initConfig({

    default: {
      workspace: '/tmp/github-monitor-2',
      deployTo: '/tmp/deploy_to',
      repositoryUrl: 'git@bitbucket.org:blackflagsoft/der.git', // ssh or https
      ignores: ['.git', 'node_modules'],
      keepReleases: 2,
      deleteOnRollback: false,
      key: '/home/biperch/.ssh',
      shallowClone: true
    },

    staging: {
      servers: 'biperch@server.ru:2222'
    }
  });
};

Log
biperch@biperch-desktop:~/project/test_iskyweb$ shipit staging  deploy

Running 'deploy:init' task...
Finished 'deploy:init' after 4.67 ms

Running 'deploy:fetch' task...
Deleting existing workspace "/tmp/github-monitor-2"
Running "rm -rf /tmp/github-monitor-2" on local.
Create workspace "/tmp/github-monitor-2"
Workspace created.
Initialize local repository in "/tmp/github-monitor-2"
Running "git init" on local.
@ Инициализирован пустой репозиторий Git в /tmp/github-monitor-2/.git/
Repository initialized.
List local remotes.
Running "git remote" on local.
Update remote "git@bitbucket.org:blackflagsoft/der.git" to local repository "/tmp/github-monitor-2"
Running "git remote add shipit git@bitbucket.org:blackflagsoft/der.git" on local.
Remote updated.
Fetching repository "git@bitbucket.org:blackflagsoft/der.git"
Running "git fetch --depth=1 shipit --prune && git fetch --depth=1 shipit --prune "refs/tags/*:refs/tags/*"" on local.
@ Из bitbucket.org:blackflagsoft/der
@  * [новая ветка]     biperch-dev2 -> shipit/biperch-dev2
@  * [новая ветка]     biperch-dev3 -> shipit/biperch-dev3
@  * [новая ветка]     biperch-dev5 -> shipit/biperch-dev5
@  * [новая ветка]     develop    -> shipit/develop
@  * [новая ветка]     kirill2kostia -> shipit/kirill2kostia
@  * [новая ветка]     kirill_3   -> shipit/kirill_3
@  * [новая ветка]     kirillos   -> shipit/kirillos
@  * [новая ветка]     kirillos2  -> shipit/kirillos2
@  * [новая ветка]     kot-work1  -> shipit/kot-work1
@  * [новая ветка]     kot-works  -> shipit/kot-works
@  * [новая ветка]     kot_work_home -> shipit/kot_work_home
@  * [новая ветка]     master     -> shipit/master
'deploy:fetch' errored after 16 s
Error: Command failed: git fetch --depth=1 shipit --prune && git fetch --depth=1 shipit --prune "refs/tags/*:refs/tags/*"
Из bitbucket.org:blackflagsoft/der
 * [новая ветка]     biperch-dev2 -> shipit/biperch-dev2
 * [новая ветка]     biperch-dev3 -> shipit/biperch-dev3
 * [новая ветка]     biperch-dev5 -> shipit/biperch-dev5
 * [новая ветка]     develop    -> shipit/develop
 * [новая ветка]     kirill2kostia -> shipit/kirill2kostia
 * [новая ветка]     kirill_3   -> shipit/kirill_3
 * [новая ветка]     kirillos   -> shipit/kirillos
 * [новая ветка]     kirillos2  -> shipit/kirillos2
 * [новая ветка]     kot-work1  -> shipit/kot-work1
 * [новая ветка]     kot-works  -> shipit/kot-works
 * [новая ветка]     kot_work_home -> shipit/kot_work_home
 * [новая ветка]     master     -> shipit/master

    at ChildProcess.exithandler (child_process.js:202:12)
    at emitTwo (events.js:100:13)
    at ChildProcess.emit (events.js:185:7)
    at maybeClose (internal/child_process.js:850:16)
    at Socket.<anonymous> (internal/child_process.js:323:11)
    at emitOne (events.js:90:13)
    at Socket.emit (events.js:182:7)
    at Pipe._onclose (net.js:475:12)

Please remove "shallowClone: true" and run again

@prasanth-cubettech Thanks


interrupting debugging causes unexpected data to serial port

Steps to reproduce

run pyocd-gdbserver with K64F (dappling v0243)
(gdb) continue
open serial port connetion
Now serial communication seems to work as expected
send SIGINT to interrupt debugging:(gdb) <ctr+c>
Program received signal SIGINT, Interrupt.
0x20000014 in ?? ()

strange data appears to serial port:$T0207:00000000;0d:281e0020;0e:11890000;0f:22890000;#1a$c04080c0c000000000008
00000000000c04080c0e0010100800b002000000000140000206c0c0020dc0c002000000000af680000281e0020118900002289000000000041d0ff022028
1e002000000000020000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000#6f$ff#cc$ff#cc$0000000090eb0000b9eb000055eb0000abeb0000ff00000066eb00001
feb000032eb00007eeb000076eb0000b2eb000086eb0000a1eb000044eb000001000000#66$l<?xml version="1.0"?><!DOCTYPE feature SYSTEM "th
reads.dtd"><threads><thread core="0" id="1">Thread mode</thread></threads>#aa

It looks like some buffer overflows etc..
Might also related to https://github.com/mbedmicro/pyOCD/issues/309.

This might be actually invalid defect. I'll try with different way and close if I cannot reproduce.

The data you show is the gdb server protocol stream that is being sent to/from pyocd-gdbserver and gdb.

Yep, this was my mistake - test environment I did use leaks data from gdb channel to serial port.


[GearVRf] Sample application: how to handle input from a gamepad (related to #74)

A new sample application to demonstrate how to handle input from a gamepad. Fire keys change background colors. Left motion key translates the scene.

Thanks Dotan, let me review your gamepad sample shortly. I’ve got a device now^^

One concern I found which is not related to input event(or maybe any kind of event not from GL Thread) is like, it can't handle change which directly calls GL commands. So you can't do something like Texture.update() or GVRConsole.writeLine() with these kinds of events.
Codes look good to me. However I may need to find a gamepad to do the verification.

OK, please let me know if you have any concern regarding this pull request or regarding the performance of this sample. Thanks

Sdk settings in the manifest, need to be aligned with:
<uses-sdk
android:minSdkVersion="16"
android:targetSdkVersion="21"
Handlers never return false, either consumed or not: dispatchKeyEvent() and dispatchGenericMotionEvent()
Need to decide to catch on which key event, UP or DOWN, but should not be both: e.g.  public boolean dispatchKeyEvent(android.view.KeyEvent event) {
if (event.getAction()== KeyEvent.ACTION_UP) {. . .
In code files,
Can remove unused lines of  “import . . .” in the .java files
Avoid unnecessary blank lines, especially at the beginning and end of each method. I assume this is your personal coding style, but...

Thanks for your comments. I will take care of it when I'm back from PTO.

Small corrections to the code:

dispatchGenericMotionEvent and dispatchKeyEvent call super if the event is no$
in dispatchKeyEvent only ACTION_DOWN is handled
unused imports and empty lines were removed
Android minSdkVersion and targetSdkVersion were changed

It looks good to me.

looks good to me as well.

i'll probably add some additional code this weekend so people can see how to track the other axes and buttons.

Other buttons can be added, but I implemented only a very basic sample assuming we just want to demonstrate how to handle key events from the gamepad.

I think adding additional features to this basic sample would be a good idea. GearVRf users also will do that based on this sample.
I saw Android documents specific to this topic, including code examples for various controller actions


chore(readme): updated the semantic tags in readme

PR summary
updated the semantic tags in readme
Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Controller 'ngModel', required by directive 'debounce', can't be found!

This is my code
<input ng-model="chunk.userInput" debounce="500">

and I get the following error:
Controller 'ngModel', required by directive 'debounce', can't be found!

Apart from the debounce directive my input field works just fine. My model is properly updated. But it just doesn't debounce!
I'm running angular v. 1.2.25

+1

+1

+1. This error even occurs without using the directive anywhere.

This is likely related to https://github.com/angular-ui/bootstrap/issues/6239
Name collision when ui-bootstrap typeahead directive generates a debounce on the tag.

+1

+1


[RESOLVED] S100-Santiago has been restarted (uptime < 10m)

Problem started at 21:00:56 on 2023.04.23
Problem name: S100-Santiago has been restarted (uptime < 10m)
Host: S100-Santiago
Severity: Warning
Operational data: 00:00:12
Original problem ID: 19883
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23522&eventid=19883

Problem has been resolved in 10m 6s at 21:11:02 on 2023.04.23
Problem name: S100-Santiago has been restarted (uptime < 10m)
Host: S100-Santiago
Severity: Warning
Original problem ID: 19883


Error message when switching buffers

Describe the bug
Occasionally when switching buffers I get the following error message:
Error detected while processing FileType Autocommands for "*":
Error executing lua callback: ...ter-refactor/lua/nvim-treesitter-refactor/navigation.lua:186: E31: No such mapping
stack traceback:
	[C]: in function 'nvim_buf_del_keymap'
	...ter-refactor/lua/nvim-treesitter-refactor/navigation.lua:186: in function 'detach'
	...vim/lazy/nvim-treesitter/lua/nvim-treesitter/configs.lua:518: in function 'detach_module'
	...vim/lazy/nvim-treesitter/lua/nvim-treesitter/configs.lua:527: in function 'reattach_module'
	...vim/lazy/nvim-treesitter/lua/nvim-treesitter/configs.lua:131: in function <...vim/lazy/nvim-treesitter/lua/nvim-treesitter/configs.lua:130>

This doesn't seem to actually affect any functionality: I can still use smart-rename after the error occurs (which makes sense, perhaps, as it appears to be related to removing a mapping, but I don't know why that would be).
To Reproduce
This happens specifically when I'm switching from one buffer to another (e.g. with ]b), but not every time I do so. It doesn't seem to occur with the same buffer twice, so if I go A->B->A->B and it appears the first time I switch to B, it won't appear the second time I do.
Expected behavior
No error message.
Output of :checkhealth nvim-treesitter

nvim-treesitter: require("nvim-treesitter.health").check()
========================================================================
## Installation
  - OK: `tree-sitter` found 0.20.7 (parser generator, only needed for :TSInstallFromGrammar)
  - OK: `node` found v16.18.0 (only needed for :TSInstallFromGrammar)
  - OK: `git` executable found.
  - OK: `cc` executable found. Selected from { vim.NIL, "cc", "gcc", "clang", "cl", "zig" }
    Version: Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
  - OK: Neovim was compiled with tree-sitter runtime ABI version 14 (required >=13). Parsers must be compatible with runtime ABI.

Legend: H[ighlight], L[ocals], F[olds], I[ndents], In[j]ections
+) multiple parsers found, only one will be used
x) errors found in the query, try to run :TSUpdate {lang}
The following errors have been detected:

ERROR: julia(highlights): ...im/0.8.1/share/nvim/runtime/lua/vim/treesitter/query.lua:219: query: invalid node type at position 3592 for language julia
julia(highlights) is concatenated from the following files:
| [ERROR]:"/Users/ben/Library/nvim/lazy/nvim-treesitter/queries/julia/highlights.scm", failed to load: ...im/0.8.1/share/nvim/runtime/lua/vim/treesitter/query.lua:219: query: invalid node type at position 3592 for language julia

Output of nvim --version
NVIM v0.8.1
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.8.1/share/nvim"

Run :checkhealth for more info

Additional context
Add any other context about the problem here.

I can also reproduce this when i try to jump to next / prev usage. Which it errors out and no jump is performed.

Also running into this annoying stacktrace. Looking at the source (below) its assuming that if a mapping is present in the config.keymaps table that the mapping must still exist for buffer. Any number of things could have taken place to invalidate this assumption before this detach function is called.
Trying to remove a mapping that no longer exists isn't really a error day-to-day users should care about. The call to nvim_buf_del_keymap could instead be wrapped in a pcall with the error written out to an appropriate log file or discarded entirely.
function M.detach(bufnr)
  local config = configs.get_module "refactor.navigation"

  for _, mapping in pairs(config.keymaps) do
    if mapping then
      api.nvim_buf_del_keymap(bufnr, "n", mapping)   # <-- Wrap this in a `pcall`
    end
  end
end


Create binary-core-api module

Need to expose some things w/o exposing binary-core's API such as File system exceptions.

Adding the blocked label until #137 is completed.