React / TS Feedback

[x] 1. The way the repo is structed, I guess we want to package all our controls in one pcf control solution? I think that’s alright for now, just wanted to confirm if that it is the intent.
[ ] 2. Add a quick read.me to build/run project and PCFs
[ ] 3. To reduce the amount of fetches, is it possible to load the "tag" data from getRelatedEntities?
[ ] 4. For loadData can we clarify in a comment what it loads? if it loads more than essential can we streamline.
[ ] 5. For associate or dissociate, have we tried one of them via context.webapi?
[ ] 6. With the queries it feels like the tagpickerbasecomponent is violating single responsibility principle by manual building those queries. If we can’t use webapi, maybe move those as an extension of WebAPI
[ ] 7. Localize component text properties
[ ] 8. Why is GetOuput in both index.ts. It seems it related to the proxy textbox field, can it be document through comments
[ ] 9. The two manifests are not including the same files, why is that the case?
[ ] 10. Variables section looks a bit busy with no comments in base component. It seems like all variables are required, can they be grouped and its usage commented on.

This repository would serve as a single package to share PCF components that aren't directly tied to the Compliance solution.

@pierluc-ssoucy 8. As for the GetOutput function, I was following the example provided by Microsoft. I'm not sure if it's actually required for a grid component.

@pierluc-ssoucy 9. The difference at the time was due to the CSS. Only the grid component made changes to the CSS. Now both components are updating it so they should be more or less inline.

@pierluc-ssoucy 3. It isn't possible to expand the properties from the getRelatedEntities request since the relationship entities only track the IDs and not the actual references.


Fix wid_edit's draw() - pass WIDGET as first argument

When attempting to edit a widget whose draw function takes the widget as an argument, we get an exception from that draw function. This fixes that by catering for both APIs.
The back button widget expects this, as does widsleepstatus, widlock and widlockunlock.
However, there are other widgets (widmessages, widbatpc and widalarmeta) which use something else for the first parameter. So I'm thinking we change the behaviour of the first three (to use this instead of an argument, and revert this wid_edit change), but we'd also need to change the back widget - but I'm not sure where this comes from.
Before I dig into that, does this sound like a good approach?

widbatpc and widalarmeta both can handle this as first argument because its type its checked. Afaik Bangle.drawWidgets() does also pass this to the draw function.

Ah I see what you mean, so perhaps this fix is ok as-is in that case

Thanks - yes, this fix is fine. Good spot about widmessages, widbatpc and widalarmeta - those really shouldn't rely on passing a different first argument (it's not like it's a big deal making that the second argument) - as @nxdefiant  pointed out, drawWidgets automatically passes the first argument in, and that is built into the firmware so I wouldn't change it.
Are you up for including changes to fix widmessages, widbatpc and widalarmeta as well?

Of course - for the moment I've chosen to pass in the widget as the second argument, but we could equally pass null there, since the same file defines that function and so can trust that it'll be unused - do you think that approach wins much in terms of performance/battery over the current one?

This looks great - thanks! Please can you bump the versions and I'll merge?

Ah yep - done

Thanks!


GH2424: Adds /sm argument to SignTool

@MihaMarkic your changes have been merged, thanks for your contribution 👍

Ah, right, sorry guys, I should have read the guidelines.


I can't rename my file types or create files

Type: Bug
So recently I haven't been able to rename new files. I can create them but whenever I try to rename it example: "index.html". I can't even type that in. I don't understand why it's not working now.

VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 5 7535HS with Radeon Graphics         (12 x 3294)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.21GB (6.04GB free)

Process Argv
--crash-reporter-id 24effb4a-a3ce-422e-ad80-bbc2ffed2bff

Screen Reader
yes

VM
0%

Extensions: none
A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
2e7ec940:30986170
pythontbext0:30879054
accentitlementsc:30887149
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
pydisww2:30991134
cppperfnew:30979542
d34g3935:30971562
fegfb526:30981948
bg6jg535:30979843
dsvsc020:30976470
jchc7451:30973076
7ghi1836:30988842

I've been trying to delete it and download it over again to try to get it to work but nothing seems to work. Even deleting everything that is vscode.

Hi @JordanCs24 are you still seeing this issue on VS Code 1.96 or later?

Yes I am
On Mon, Dec 16, 2024 at 3:00 PM Raymond Zhao @.***>
wrote:

Hi @JordanCs24 https://github.com/JordanCs24 are you still seeing this
issue on VS Code 1.96 or later?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/207951#issuecomment-2546610267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BE7QZPZKAQXMP2S6FUVXUSL2F4WO7AVCNFSM6AAAAABTW4RCX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBWGYYTAMRWG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add an Array Protocol & improve static typing support

Continuing from https://github.com/data-apis/array-api/issues/584, I've done a quick refactor of the array (and dtype) objects to make them typing Protocols.
@rgommers, I agree that making Array generic wrt the dtype is good for a followup.
It's not perfect, but mypy doesn't complain a whole lot, so I think this is a good start. I'm happy to make refinements.

We might want to put these in a follow-up PR if just to ship the concretely non-controversial stuff first—interested in what others think.

I was thinking the same thing last evening about moving this to a followup PR.

On namespace type hints (e.g. _CreatoinFuncs, ArrayAPINamespace): IMO I'm not sure on this yet, as they make contributing to the spec less ergonomic given we repeat the signatures, although they are pretty nifty to have. Is there a world where these could be dynamically construct them from the stubs instead?

Maybe? I haven't experimented yet but perhaps mypy will be satisfied with:
class ArrayAPINamespace(Protocol):

    zeros_like = staticmethod(creation_funcs.zeros_like)

@BvB93 if you have any feedback on this PR, that would be great to see.

@nstarman there are 3 errors left when running mypy array_api_stubs/_draft/array_object.py (from lots more on main). There are more errors in other files - but if we are going to maintain static typing support in this repo, adding a CI job running Mypy seems feasible. We could start with this file, and expand it to all files later. WDYT?

One thing that isn't nice is that Self now shows up instead of array in the signatures of the html docs.

If the array type is just text in the html docs and not 'clickable', then we can rename Self to array.

adding a CI job running Mypy seems feasible. We could start with this file, and expand it to all files later. WDYT?

That sounds like a good idea. I can try adding to the CI and create an ignore file that can later be whittled down as the package is made more type friendly.

If the array type is just text in the html docs and not 'clickable', then we can rename Self to array.

That works. It's not clickable - there is no array object in the standard on purpose, because in the end we don't care whether it's named ndarray, Tensor, or something else.

Sorry for the long interlude. I can't seem to resolve 3 linking errors in the docs  https://app.circleci.com/pipelines/github/data-apis/array-api/1088/workflows/ed2dfa75-c3c7-4845-8f28-e01fe35d0c8d/jobs/996?invite=true#step-103-1646

Sphinx is trying to create links for the type hints. I think to fix that you need to update the list(s) here https://github.com/data-apis/array-api/blob/main/src/_array_api_conf.py#L52-L77

I can squash commits if you don't use Squash & Merge (assuming this is approved, of course :) )

I think pre-commit is a good addition (would love to add hooks for other spec things myself), but probably not too useful until we get a workflow that runs it. That said, I'm not too sure if we'd want to do that just yet 🤔

pre-commit.ci can be turned on, sans additional GH workflows.

Looks good on my end!

Thanks @honno for the reviews and discussion.

Any reason to change array to Array? I think we might prefer to keep it as array like it is now.

The class would have to be called array, which is problematic re PEP-8 and also conflicts name wise with the array TypeVar. The latter can be changed, but for the former isn't it desirable to refer to array instances of some class Array?

The class name is specifically not part of the standard. I always felt that array made that a little clearer than Array, but maybe that's just me.
By the way, the protocol isn't going to require the class to be called Array is it? If it does, that's a problem.

By the way, the protocol isn't going to require the class to be called Array is it? If it does, that's a problem.

Not specifically. The protocol is a class and may be named anything. If it's named the same things as another variable, e.g. the array = TypeVar("array", bound=Array) then that can be problematic.
As a more zoomed out view. The goal of this and subsequent PRs is to enable the spec to also be a functioning & installable package that can be used for run-time checks of adherence to this spec.
>>> from data_apis.array import Array  # this spec
>>> from numpy import array, ndarray  # post v2.0

>>> issubclass(ndarray, Array)
True

>>> isinstance(array(1), Array)
True

To be clear, Array cannot be instantiated since it is a protocol without any implementation. Its use is entirely for structural subtyping (e.g. type hinting) and run-time checks.
At some level it's important that the spec adhere to Python standards, e.g. naming conventions.

The class name is specifically not part of the standard.

I'm not sure I understand then what's the problem with calling it Array?

LGTM, though it might be good to verify if the currently available downstream implementations properly satisfy subtype the protocol as this could potentially reveal some typing-related corner cases that we've missed thus far. Could be something for a follow up though.

It would be great to see this released. It looks very developed to me, and would be useful for us within xarray.

@rgommers. I'd be happy to finish / merge this in. I don't recall why this stalled out.

Rebased and applied fixes for things that had changed.

@nstarman Would you be interested in coming to the community meeting this week (Nov 28) to discuss static typing and next steps? It would be great to have some perspectives from static typing experts!

I would be very interested to come! It is Thanksgiving, but I should be able to join.


Bump copyright to the current year (2016)

Easy pick in favor of #7659

Can this be merged or closed?

@ # #

[ ]

[ ]

It's not even 2016 any more, so sorry closing this.


feat(Accessibility): Improve accessibility behaviors, make them reusable

Currently, accessibility behaviors bring few constraints on how they are implemented and used.
Main points:

reusing parts of behaviors: you want to implement ToggleButtonBehaviour attributes based on ButtonBehavior, without copy-pasting same code
behavior can be either an object or function, which brings additional complexity, as you always have to check it. If we gonna make behaviors more agnostic and distribute them for any kind of framework, I think we need to stick to one concrete structure of behaviors to avoid possible issues in future.
Already seen some complexities if to look at SelectableListItemBehavior
const SelectableListItemBehavior: (props: any) => IAccessibilityDefinition = (props: any) => ({
We define it with double function to pass correct props from BasicListBehavior

Solution is to change accessibility behavior definition to one solid structure
export type Accessibility = ((props: any) => IAccessibilityDefinition)

Reusability
Currently I made ButtonBehavior attributes reusable, by exporting it's attributes and using it in ToggleButtonBehavior as
export const ButtonAttributes = (props: any): AccessibilityAttributes => ({
  root: {
    role: props.as === 'button' ? undefined : 'button',
    'aria-disabled': !!props['disabled'],
  },
})

const ButtonBehavior: Accessibility = (props: any) => ({
  attributes: ButtonAttributes(props),
})

const ToggleButtonBehavior: Accessibility = (props: any) => ({
  attributes: {
    root: {
      ...ButtonAttributes(props).root,
      'aria-pressed': !!props['active'],
      'aria-disabled': !!props['disabled'],
    },
  },
})

Other behaviors can be changed like that in case we'll see any other meaningful opportunity to reuse behavior's parts (attributes, keyActions, etc).

Nice changes, @sophieH29 . Left some comments for consideration, and please fix the linting errors. After reasoning for/resolving the comments, should be good for merge.


Update Getting Started with F# with Cross Platform Tools to latest SDK

I made a few changes:

Updated links to point to new SDK
Cleaned up the text about VS Code as a potential text editor
Removed most steps, since they're not necessary now
Removed the redundant code snippets
Removed the test section, as it's still buggy with xUnit and doesn't reliably execute on OS X or *nix.

Open Publishing Build Service: The pull request content has been published and here are some changed files links of this time:

Docset 1 - core-docs
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.runtime.interopservices.iregistrationservices?branch=pr-en-us-1026
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.runtime.interopservices.itypelibconverter?branch=pr-en-us-1026
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.runtime.interopservices.itypelibexporternameprovider?branch=pr-en-us-1026
Please see 3429 more published files of this docset in report.

Status: SucceededWithWarning. Message: Warning occurred: Invalid file link:(~/docs/fsharp/tutorials/getting-started/dot.net/core). Referenced by file: docs/fsharp/tutorials/getting-started/getting-started-command-line.md at line: 0...
Open Publishing Report.

Open Publishing Build Service: The pull request content has been published and here are some changed files links of this time:

Docset 1 - core-docs
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.networkinformation.pingoptions?branch=pr-en-us-1026
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.networkinformation.pingreply?branch=pr-en-us-1026
https://stage.docs.microsoft.com/en-us/dotnet/core/api/system.net.networkinformation.prefixorigin?branch=pr-en-us-1026
Please see 3086 more published files of this docset in report.

Status: Succeeded.
Open Publishing Report.

LGTM. Never mind my question. Your topic suggests VS Code.


Add support for assembly target attributes

Would it be possible to add support for code generators defined with validOn = AttributeTargets.Assembly, meaning that they would not be attached to particular class?
For example:
[assembly: GenerateClass("Foo")]

@AArnott I think it's implemented by #67 already.


docs: correct typos in package name config

Fixes small typos in the config filenames in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
266

Relevant Lines:
266

💛  - Coveralls


Segfaults on PHP 5.3/5.4

I have a project, wsdl2phpgenerator, which uses php-vcr. First of all: Thanks a bunch for your work!
Second: Our test suite has been segfaulting on Travis when combining PHP 5.3/5.4 and the latest version of our dependencies. Lowest versions have been running fine.
I have tried to narrow down the problem and I believe it to be introduced in php-vcr 1.2.8. Setting 1.2.7 as the an upper bound to our dependency fixes our build. Checking the diff between 1.2.7 and 1.2.8 did not give me any clues regarding the cause of this.
Since PHP 5.3. and 5.4 are both EOL the quickest way to fix the situation would be to bump the required 5.* version to 5.5.

Hi @kasperg,
wsdl2phpgenerator is a great project, and is the tool we are using in what I describe in issue #153.
We were having the exact same problem as you mention, a build with tests that used php-vcr to provide fixtures for code generated with wsdl2phpgenerator was failing because php was segfaulting when running the tests.
After investigating the issue, I found an easy solution that seems to have fixed the problem on our side. It's in commit f8a40c3.
Could you try running your test suite with my fork and check if the segfaults go away?

@kasperg could you please test @buduchail solution?
@buduchail thanks for checking on that, I left some comments in your PR!

@renatomefidf I just tried it and can confirm that the change fixes this issue.

Thanks for everyone involved here.
This issue is from now fixed on php-vcr 1.3.1: https://github.com/php-vcr/php-vcr/releases/tag/1.3.1


gh-3403: Remove encodeURI call for redirect load property

Fixes #3403 -- developers can now return a redirect property from a module's load() without it be double-encoded during SSR.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[/] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

There seems to be a flaky test about a scroll bar.

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

thank you!


Could not compile dependency ssl_verify_hostname

Hi, i get this error installing phoenix_guardian

Can you help me solve it ?
i´m using ubuntu

Hey @ramstein74
I'm not sure what that error is about. I don't explicitly require anything for ssl_verify_hostname. Do you still get that error compiling if you remove Guardian?
What version of Elixir are you using?

elixir 1.2.0
I dont get the ssl error now but i need vc++ to compile comeonin
Oh my god...
2016-01-04 17:26 GMT+00:00 Daniel Neighman<EMAIL_ADDRESS>
Hey @ramstein74 https://github.com/ramstein74
I'm not sure what that error is about. I don't explicitly require anything
for ssl_verify_hostname. Do you still get that error compiling if you
remove Guardian?
What version of Elixir are you using?
—
Reply to this email directly or view it on GitHub
https://github.com/hassox/phoenix_guardian/issues/11#issuecomment-168743057
.

@ramstein74 ?
Is everything cool or is there still a problem?

i need more time to install a c++ compiler..
meanwhile thank you
2016-01-04 21:40 GMT+00:00 Daniel Neighman<EMAIL_ADDRESS>
@ramstein74 https://github.com/ramstein74 ?
Is everything cool or is there still a problem?
—
Reply to this email directly or view it on GitHub
https://github.com/hassox/phoenix_guardian/issues/11#issuecomment-168818104
.

:+1: let me know how you go :)


Create terms of use page

Add terms of use as a page inside the application on the SignUp flow. This page should be the first one on the Flow. The terms will be rendered inside the component. The content should be markdown or plain HTML, no react.
Design: https://www.figma.com/file/uEckQY4EaQ5rXwyISg6d1g/Fuel-Wallet?node-id=3127%3A16992&t=JIOcIEePnitWrYMS-4

Blocked because we're waiting for the final Terms of service document.


NSFW commands can be used in non-nsfw channels

The title is self-explicative. Some members were spamming gay hentai in general channel in my server, which didn't have any NSFW tag/option active.

Type .cm NSFW enable #yournsfwchannel (replace the channel with the one you want). Then it will only work in that channel. You can also type .listperms to see the hierarchy. All explained here 👍

Was there an update to the bot that made it so the commands were only
usable in NSFW flagged channels? If not you;ll have to set the permissions
up yourself. It doesn't do it on it's own and is clearly pointed out in the
permissions portion of the bot documentation.
On Tue, Aug 29, 2017 at 1:56 PM Jonas Berchol<EMAIL_ADDRESS>wrote:

The title is self-explicative. Some members were spamming gay hentai in
general channel in my server, which didn't have any NSFW tag/option active.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1544, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGad31ZwsJAsGawtBsuZX8jK04FJi4Kks5sdFDSgaJpZM4PGUrO
.

It doesn't matter which channels are marked as NSFW or not, you have to manually manage the commands using permissions module. Keep that in mind.

Someone must've enabled NSFW in that channel.
Take away their perms.
Next time, please ask in #NadekoLog before posting it as an issue.

Permission system allows you to disable nsfw on certain channels. It's independent of discord nsfw channel option.


If magnet scheme is not added destroys the visual result

Hi. I love so much your JS plugin Linkify.
It recognizes http, https, ftp and ftps (where's the complete list?!)
Anyway, this post it's about magnet.
If there's not magnet recognition a string like
text text text

magnet:?xt=urn:btih:5a7f5e0f3ce439e2f1a83e718a8405ec8809110b&dn=ernfkjenrkfk%5FSQ80%5FV%5Fv1.0.0%5Ferfnkerkf%5Ferfnkerkfefrfvegrteggt.net.rar&tr=udp%3A%2F%2Ftracker.leechers-paradise.org%3A6969&tr=udp%3A%2F%2Ftracker.coppersurfer.tk%3A6969&tr=udp%3A%2F%2Ftracker.opentrackr.org%3A1337&tr=udp%3A%2F%2Ftracker.zer0day.to%3A1337&tr=udp%3A%2F%2Ftracker.pirateparty.gr%3A6969&tr=udp%3A%2F%2Fp4p.arenabg.ch%3A1337&tr=udp%3A%2F%2Feddie4.nl%3A6969&tr=udp%3A%2F%2Ftracker.internetwarriors.net%3A1337&tr=http%3A%2F%2F<IP_ADDRESS>%3A2710%2Fannounce&tr=udp%3A%2F%2Ftracker.opentrackr.org%3A1337%2Fannounce&tr=udp%3A%2F%2Feddie4.nl%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.leechers-paradise.org%3A6969&tr=udp%3A%2F%2F9.rarbg.com%3A2710%2Fannounce&tr=udp%3A%2F%2F9.rarbg.me%3A2710%2Fannounce&tr=udp%3A%2F%2Ftracker.coppersurfer.tk%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.ilibr.org%3A80%2Fannounce&tr=udp%3A%2F%2Ftracker.ilibr.org%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.zer0day.to%3A1337%2Fannouncement

by using linkifyStr(str) gives the result
text text text

magnet:?xt=urn:btih:5a7f5e0f3ce439e2f1a83e718a8405ec8809110b&dn=ernfkjenrkfk%5FSQ80%5FV%5Fv1.0.0%5Ferfnkerkf%5Ferfnkerkfefrfvegrteggt.net.rar&tr=udp%3A%2F%2Ftracker.leechers-paradise.org%3A6969&tr=udp%3A%2F%2Ftracker.coppersurfer.tk%3A6969&tr=udp%3A%2F%2Ftracker.opentrackr.org%3A1337&tr=udp%3A%2F%2Ftracker.zer0day.to%3A1337&tr=udp%3A%2F%2Ftracker.pirateparty.gr%3A6969&tr=udp%3A%2F%2Fp4p.arenabg.ch%3A1337&tr=udp%3A%2F%2Feddie4.nl%3A6969&tr=udp%3A%2F%2Ftracker.internetwarriors.net%3A1337&tr=http%3A%2F%2F<IP_ADDRESS>%3A2710%2Fannounce&tr=udp%3A%2F%2Ftracker.opentrackr.org%3A1337%2Fannounce&tr=udp%3A%2F%2Feddie4.nl%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.leechers-paradise.org%3A6969&tr=udp%3A%2F%2F9.rarbg.com%3A2710%2Fannounce&tr=udp%3A%2F%2F9.rarbg.me%3A2710%2Fannounce&tr=udp%3A%2F%2Ftracker.coppersurfer.tk%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.ilibr.org%3A80%2Fannounce&tr=udp%3A%2F%2Ftracker.ilibr.org%3A6969%2Fannounce&tr=udp%3A%2F%2Ftracker.zer0day.to%3A1337%2Fannouncement

that there's not text and not link. It's a mix of both.
A fiddle to understand the result: fiddle

So. There's a way to disable the plugin for date pattern?
Or there's a way to enable others file's URI recognition like magnet, but also telnet, bitcoin, dns, facetime, finger, git, go, gopher, gtalk, irc, spotify, market, nfs, muble, ssh, tftp, vnc, network files and others of  this uri schemes?
P.S. Could you better explain what's the difference between plugin *.js,*.cjs.js, *.es.js?
Edit
Opening the main file I understood that network file scheme it's recognized.
	  fastts(Start, 'file', SCHEME, WORD, wordjr); // <<<<<<< HERE
	  fastts(Start, 'mailto', SCHEME, WORD, wordjr);
	  fastts(Start, 'http', SLASH_SCHEME, WORD, wordjr);
	  fastts(Start, 'https', SLASH_SCHEME, WORD, wordjr);
	  fastts(Start, 'ftp', SLASH_SCHEME, WORD, wordjr);
	  fastts(Start, 'ftps', SLASH_SCHEME, WORD, wordjr);

but what for the network file scheme that starts with //?
And reading better the documentation there's also the registerPlugin option, but the link See example plugin function implementations it's broken 😭 and the option registerCustomProtocol (that does not work with magnet😭😭).
Pls, could you give an example for magnet?
Tnx in advance

@gethminar thanks for reporting, the magnet link not working once registerCustomProtocol is called is definitely a bug, I'll fix this in an upcoming release.

So. There's a way to disable the plugin for date pattern?

Can you elaborate on this? I'm not sure what you mean.

but what for the network file scheme that starts with //?

This used to work but was removed following #206 because it's generally not expected. Can likely be added as a plugin using the Mention plugin as a reference

And reading better the documentation there's also the registerPlugin option, but the link See example plugin function implementations it's broken 😭 and the option registerCustomProtocol (that does not work with magnet😭😭).

Links have been fixed! Sorry about that 😅

@Nick Frasser

And reading better the documentation there's also the registerPlugin option, but the link See example plugin function implementations it's broken 😭 and the option registerCustomProtocol (that does not work with magnet😭😭).

Links have been fixed! Sorry about that 😅

It is like it was before 😅😅

So. There's a way to disable the plugin for date pattern?

Put the condition that there's not a magnet link discovery implementation.
The answer it's about the possibility of disabling the link adding for a date regex like this, so the link it's not added to this regex.

but what for the network file scheme that starts with //?

This used to work but was removed following https://github.com/Hypercontext/linkifyjs/issues/206 because it's generally not expected. Can likely be added as a plugin using the Mention plugin as a reference

Bad news. It would have been very useful. Please give some example, I'm not so able to do this. If there's something pre-made it's better (something like a file to add)

Hi @gethminar, quick update on this, there's a fix for this now available in the latest v4.2.0 release.
If you include
linkify.registerCustomProtocol('magnet', true)

Then magnet links should now be parsed correctly.


Add support for spectrogram transforms

E.g. transforms like this: https://github.com/zcaceres/spec_augment

Related question: we should probably include STFT in the deterministic transforms. Should we support torchaudio's and asteroid's?

Sounds good to me

Thinking out loud: would it make sense to make no API distinction between feature extraction and augmentation?
Spectrogram could be seen as an augmentation of audio.
It would allow an API like that:
augmenter = Chain(Spectrogram(), SpecAugment())
augmented_spectrogram = augmenter(audio)

That's what I have in mind at least.
Everything is a transform, and augmentations are just probabilistic ones.

Sounds good to me

Cool. So I'll think about outsourcing asteroid's filterbank API to another repo then.


MS-AMP needs a website

What would you like to be added:
A website for MS-AMP.
Why is this needed:
We want to provide our customers with more comprehensive information, including details about the design, user tutorials, and developer guides.
Without this feature, how does current msamp work：
Currently, msamp uses README.md to introduce itself.
Components that may involve changes:
Brief description of your proposal if any:
The website of superbench is a good reference to us.

#101


Migration of data alongside metadata

As the metadata model evolves, how to migrate the data from a previous version to a newer version?
In ENCODE-DCC, there was SnoVault (https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0175310)
What other tools support such migration?
Note to self: Can we define any translation/migration rules in linkml?

This can be done in Year 2 once we touch all of our other milestones. But it would be good to keep in mind how to define migration rules from schema version x to version y.


Error when opening quarto-nvim-kickstarter

Hi,
I am getting a similar error message to one that was reported earlier in this thread. After I cloned the repo and opened Neovim, I get 'Failed to run 'config' for image.nvim
nvim/lua/plugins/ui.lua:234: attempt to call field 'system' (a nil value)
stacktrace:
nvim/lua/plugins/ui.lua:234 in config
nvim/lua/config/lazy.lua:16
nvim/init.lua:2
I removed the files in .local /nvim share and state and cloned again, but the same message still came up.
I hope you can help.

what version of neovim are you on?

I tried  0.9.0 and  0.9.4 both gave the same response.

Ah, yes. Looks like, ironically, the function I'm using to check for dependencies for a new plugin via a shell command is only available in nvim 0.10, so I'm going to have to check for that as well.

Should be fixed now :)

Thanks, No errors now.


Unstable text area width

Describe the bug
I made a demo for a little application called LDAPWalker and the title is displayed using toilet (figlet-like). On my widescreen I can see the whole title on one line normally but half the time the text area seems reduced in width and the title is wrapped.
To Reproduce
Steps to reproduce the behavior:

open https://ldapwalker-demo.duckcorp.org/

Expected behavior
The text should use the available width on my widescreen with my browser on fullscreen.
Screenshots

Environment:

OS: Debian unstable
Browser:Firefox 95.0.1-1
ttyd: 1.6.3-3~bpo11+1

Additional context
When opening the application the overlay size displays 322*64 in both cases. It looks like a race condition.

I've tested it on https://ldapwalker-demo.duckcorp.org/ , it works fine.
The terminal width is correct. If you echo a very long line, it can use the full browser width.

@tsl0922 I'm sorry but it still can be reproduced. Just reload the page et as I said sometimes it's working and sometimes not.

I made some screenshots to show you.


Seq as alternative to Vec , Vec bugs

For things that

Don't need to be dynamically accessed
Support multiple bitwidths (for the purposes of compile-time generability, you want to be able to have some arbitrarily long list of signalName_index without having to specify unique names for each of the elements)

@jackkoenig said that Seq's of Chisel data types were supposed to give better names now, and that might be the solution to the Bundle of arbitrary length I wanted earlier for non-homogeneous Vecs.
A couple things I tried:
First, I wanted the equivalent of vec.asUInt (or .toBits).
It seems like the functionality exists here BUT it's private to Chisel...

How would you support the equivalent of cloneType?
Can we make it so that we can use := with Seq's? Rather than having to iterate through...

Bugs:
In an IO bundle, I have:
val test = Vec(Seq.fill(3)(Bits(OUTPUT, Width(3))))
In the body of my module, I have:
io.test(1) := Vec(Seq.fill(3)(state === UInt.Lit(0))).asUInt
This results in the following Chisel compile error
[error] (run-main-0) scala.MatchError: null
scala.MatchError: null
    at chisel3.core.Binding$.chisel3$core$Binding$$walkToBinding(Binding.scala:62)
    at chisel3.core.Binding$.bind(Binding.scala:88)
    at chisel3.core.Module.Port(Module.scala:64)
    at chisel3.core.Module.IO(Module.scala:89)
    at chisel3.core.Module.computePorts(Module.scala:138)
    at chisel3.core.Binding$.chisel3$core$Binding$$elementOfIO$1(Binding.scala:128)
    at chisel3.core.Binding$$anonfun$checkSynthesizable$1.apply(Binding.scala:172)
    at chisel3.core.Binding$$anonfun$checkSynthesizable$1.apply(Binding.scala:167)
    at chisel3.core.Binding$.chisel3$core$Binding$$walkToBinding(Binding.scala:63)
    at chisel3.core.Binding$.checkSynthesizable(Binding.scala:165)
    at chisel3.core.Data.bulkConnect(Data.scala:156)
    at chisel3.core.Vec$.chisel3$core$Vec$$doConnect$1(Aggregate.scala:68)
    at chisel3.core.Vec$$anonfun$do_apply$3.apply(Aggregate.scala:74)
    at chisel3.core.Vec$$anonfun$do_apply$3.apply(Aggregate.scala:73)
    at scala.collection.TraversableLike$WithFilter$$anonfun$foreach$1.apply(TraversableLike.scala:778)
    at scala.collection.Iterator$class.foreach(Iterator.scala:742)
    at scala.collection.AbstractIterator.foreach(Iterator.scala:1194)
    at scala.collection.IterableLike$class.foreach(IterableLike.scala:72)
    at scala.collection.AbstractIterable.foreach(Iterable.scala:54)
    at scala.collection.TraversableLike$WithFilter.foreach(TraversableLike.scala:777)
    at chisel3.core.Vec$.do_apply(Aggregate.scala:73)

If I replace the port in the IO bundle with:
val test = Vec(3,Bits(OUTPUT, Width(3)))
Stuff compiles...

The above is maybe a bad example b/c the elemets of the Vec are the same width, but assuming that they aren't, I'd definitely only want to allocate enough bits for each IO port... I tried to do the same with Seq inside the bundle with the hope that it'd do smart naming with reflection:
Inside the bundle:
val test = Seq.fill(3)(Bits(OUTPUT, Width(3)))
Inside the module body:
io.test(1) := UInt.Lit(0)
Firrtl crashes with:
[error] (run-main-0) firrtl.passes.CheckChirrtl$UndeclaredReferenceException:  @[SynchronousSARCtrl.scala 135:27]: [module SynchronousSARCtrl] Reference T_9 is not declared.
firrtl.passes.CheckChirrtl$UndeclaredReferenceException:  @[SynchronousSARCtrl.scala 135:27]: [module SynchronousSARCtrl] Reference T_9 is not declared.
    at firrtl.passes.CheckChirrtl$.firrtl$passes$CheckChirrtl$$checkChirrtlE$1(CheckChirrtl.scala:101)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlS$1$3.apply(CheckChirrtl.scala:130)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlS$1$3.apply(CheckChirrtl.scala:130)
    at firrtl.ir.Connect.mapExpr(IR.scala:242)
    at firrtl.Mappers$StmtMagnet$$anon$2.map(Mappers.scala:44)
    at firrtl.Mappers$StmtMap$.map$extension(Mappers.scala:55)
    at firrtl.passes.CheckChirrtl$.firrtl$passes$CheckChirrtl$$checkChirrtlS$1(CheckChirrtl.scala:130)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlS$1$5.apply(CheckChirrtl.scala:131)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlS$1$5.apply(CheckChirrtl.scala:131)
    at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:245)
    at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:245)
    at scala.collection.mutable.ResizableArray$class.foreach(ResizableArray.scala:59)
    at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:48)
    at scala.collection.TraversableLike$class.map(TraversableLike.scala:245)
    at scala.collection.AbstractTraversable.map(Traversable.scala:104)
    at firrtl.ir.Block.mapStmt(IR.scala:227)
    at firrtl.Mappers$StmtMagnet$$anon$1.map(Mappers.scala:41)
    at firrtl.Mappers$StmtMap$.map$extension(Mappers.scala:55)
    at firrtl.passes.CheckChirrtl$.firrtl$passes$CheckChirrtl$$checkChirrtlS$1(CheckChirrtl.scala:131)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlM$1$3.apply(CheckChirrtl.scala:144)
    at firrtl.passes.CheckChirrtl$$anonfun$firrtl$passes$CheckChirrtl$$checkChirrtlM$1$3.apply(CheckChirrtl.scala:144)
    at firrtl.ir.Module.mapStmt(IR.scala:443)
    at firrtl.Mappers$ModuleMagnet$$anon$11.map(Mappers.scala:115)
    at firrtl.Mappers$ModuleMap$.map$extension(Mappers.scala:125)
    at firrtl.passes.CheckChirrtl$.firrtl$passes$CheckChirrtl$$checkChirrtlM$1(CheckChirrtl.scala:144)
    at firrtl.passes.CheckChirrtl$$anonfun$run$1.apply(CheckChirrtl.scala:147)
    at firrtl.passes.CheckChirrtl$$anonfun$run$1.apply(CheckChirrtl.scala:147)
    at scala.collection.mutable.ResizableArray$class.foreach(ResizableArray.scala:59)
    at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:48)
    at firrtl.passes.CheckChirrtl$.run(CheckChirrtl.scala:147)
    at firrtl.SimpleRun$$anonfun$1$$anonfun$2.apply(LoweringCompilers.scala:47)
    at firrtl.SimpleRun$$anonfun$1$$anonfun$2.apply(LoweringCompilers.scala:47)
    at firrtl.Utils$.time(Utils.scala:56)
    at firrtl.SimpleRun$$anonfun$1.apply(LoweringCompilers.scala:47)
    at firrtl.SimpleRun$$anonfun$1.apply(LoweringCompilers.scala:45)
    at scala.collection.LinearSeqOptimized$class.foldLeft(LinearSeqOptimized.scala:124)
    at scala.collection.immutable.List.foldLeft(List.scala:84)
    at firrtl.SimpleRun$class.run(LoweringCompilers.scala:45)
    at firrtl.Chisel3ToHighFirrtl.run(LoweringCompilers.scala:61)
    at firrtl.Chisel3ToHighFirrtl.execute(LoweringCompilers.scala:68)
    at firrtl.Compiler$$anonfun$compile$1.apply(Compiler.scala:71)
    at firrtl.Compiler$$anonfun$compile$1.apply(Compiler.scala:70)
    at scala.collection.LinearSeqOptimized$class.foldLeft(LinearSeqOptimized.scala:124)
    at scala.collection.immutable.List.foldLeft(List.scala:84)
    at firrtl.Compiler$class.compile(Compiler.scala:70)
    at firrtl.VerilogCompiler.compile(LoweringCompilers.scala:201)

== You can't have a Seq in a Bundle and therefore need some other mechanism to support heterogenous non-dynamically accessed Vec-like things?

I fiddled with this a bit.
val test = Vec(Seq.fill(3)(Bits(OUTPUT, Width(3))))

is wrong, particularly when used in a Bundle def, but should have been registering a better error message.  I just checked the code sample on a less complex variant of the chisel3 that does not try to do the compatibility layering and it returns the following error message:

Chisel.internal.Binding$BindingException: 'that' (Chisel.UInt@####): Not bound to synthesizable node, currently only Type description

The stack trace points to that line as the first user code point, but we can also see the problem is in the Vec.apply when it tries to do a connection. This actually points to a major UX problem with apply that I will now delve into....
First, we must ask ourself what the 'true type signature' of Vec.apply(in: Seq[T<:Data]) is. A key realization is that ALL Chisel.Data and subtypes are overloaded and conflate 2+ 'true-types' (partly due to the mutable state inside of them that toggles and then does something). A Data is either 'unbound' and thus is just representing a potential hardware type description that can be used to emit some register, wire, or port declaration. These are created with e.g. UInt(width=4) or Vec.apply(Int, T<:Data) and can only be used to create other unbound type descriptions or passed into a Reg, Wire, or IO. An interesting observation is that instances of these true-types are somewhat interchangeable. For example, it doesn't matter (except for performance) if you accidentally call chiselCloneType an extra time on a gen before using it anywhere. (This also excludes bad code relying on referential equality for unbound types.)
The other true types (depending on if you count each as a distinct true-type or not) is what you get from the result of Reg, Wire, and IO. Instances of these true-types, which I call 'synthesizable or bound' in error messages, can actually be used to describe hardware operations. You can (for different sets of them) assign them, add them, subtract them, do bit conversions between then, etc. Also, all those operations (that return something) return another bound true-type. Similarly, you generally shouldn't use these true-types in places where you could use an unbound type:
Reg(0.asUInt) is technically valid but equivalent to Reg(UInt()), not Reg(init=0.asUInt), as commonly expected. Note further that, since each instance of these true-types are bound to a specific hardware value, they are rarely 'interchangeable' without changing the underlying circuit.
Note, there is one particularly important case where you CAN'T use these values where an unbound-type is required: Bundle definitions. Bundle definitions are always assumed to be describing the unbound type. These definitions are then passed into and IO, Reg or Wire in order to materialize it in the graph. Chisel just doesn't know how to handle a 'partially-bound' type. For example, if you pass a Bundle that defines a field = Reg(...) into Wire, its implementation just has no way of coercing that field to now represent a Wire. (Also, since it WAS a Reg or Wire, what happens to already-emitted operations on that field? Should they point to the new rebound type? But what if that violates user expectations?). In a sense, Bundle definitions are like C-structs, they just represent the potential structure of data but you can't give them defaults.
The reason I elaborated that point so deeply is that understanding this deficiency in Chisel's use of the type system really helps drill down on what the error is. Also, this subtlety really outlines why Vec.apply can be so confusing, especially for new-comers. The different forms all seem to take and return the same type, but really do and return substantially different things.
Vec.apply(Int, T<:Data): Vec[T] both takes an unbound type of scala-type T and returns an unbound type of scala-type Vec[T] that can be used to instantiate Reg, Wire, etc. that is a hardware array of that value T. This function does not actually create hardware.
In contrast, Vec.apply(Seq[T<:Data]): Vec[T] both takes a BOUND type of scala-type Seq[T] and returns a bound Wire of scala-type Vec[T]. This function is creating hardware in the current Module: it creating a Wire(Vec(Seq.length, T)) and then connecting it to the input Data. Hence, why both the input and result must be bound to actual hardware.
Looking at your lines in the context of the prior discussion:
val test = Vec(Seq.fill(3)(Bits(OUTPUT, Width(3)))) // in a Bundle definition

The call to Vec.apply is the Seq overload and thus expects a bound type (since it is connecting it to something). However, Bits(OUTPUT, Width(3)) represents an unbound type (it isn't on the hardware graph) and thus Chisel complains at runtime since you can't emit operations referencing an unbound type.
Further, even if we fixed this so the call to Seq.fill was fixed to be given a bound type, the circuit should still fail to elaborate because the return of the Vec is bound. Thus, the IO call will complain that field test is already bound to hardware and thus can't be part of representing an IO.
Note that the null match error is due to a bug in the compatibility error triggered specifically because this Bundle is part of the io definition. The compatibility layer will try to simulate wrapping your io in IO for you (thus marking all the fields as being bound) by looking at the io of the currentModule. However, this issue is occuring due to the initialization of io, before it has been set to a value (and thus is null). This is partly the danger of introspecting on a class (e.g. passing a this point) in a class constructor. Clients of 'this' will expect a fully constructed class; however, the class has yet to be fully constructed....
val test = Vec(3,Bits(OUTPUT, Width(3))) // in Bundle definition

This works because that form of Vec.apply both takes and returns an unbound type. It is given an unbound type (good!) and returns an unbound type (good! because this line is used in a Bundle definition that is then passed into an IO call).
val test = Seq.fill(3)(Bits(OUTPUT, Width(3))) // in Bundle definition

This will compile but won't do elaborate because of how the Bundle reflection is specified. Bundle reflection is intentionally narrow and only looks at fields that are of type Data or Some[Data]. Thus, Chisel just doesn't know that you intend for test to actually be part of the unbound hardware type you are trying to define. I am unsure of your compatibility layer settings but this SHOULD have given you a frontend exception at the connection "io.test(1) := UInt.Lit(0)" that 'this' (io.test(0)) is not bound to a synthesizable type. That is Chisel just didn't know, when IO, Wire, or Reg was called, to actually bind the 'test' field to hardware.
See https://github.com/ucb-bar/chisel3/blob/master/chiselFrontend/src/main/scala/chisel3/core/Aggregate.scala#L354 and notice how only Data and Option are inspected. This could be amended to also pickup scala sequences of Data (although they would be emitted in an exploded form). Technically, this would be a backwards incompatible change because you can currently define Bundles that have no-argument defs that return a Seq[Data] without needing to supply a dummy arg/add them to the blacklist (e.g. flatten).
This desire is why I advocate (at least as a 'super-user' level feature) the ability to define a Bundle that doesn't use reflection to determine its elements, instead allowing the user to specify the namedElts mapping of String->Data. In your case you would have like:
override val namedElts = ListMap(
  (for((elem, idx) <- test.zipWithIndex) yield s"test_$idx" -> elem):_*
)

Sure, Seq might be popular enough to warrant inclusion in the actual reflection (excepting the compatibility concerns) just like Option was, but who knows what the next one will be. The problem is that the user is just stuck with the default reflection behavior with no way of fixing it.

Yeah, sorry, I didn't bother looking for the root causes of these issues. Thanks for spending time to delve into this.
I guess I don't necessarily care what the final solution is -- Vec is just currently really confusing to use [and I'm definitely not the only one with these problems], and I just really want some way to be able to programmatically add elements of varying widths (possibly types) based off of some generator parameter.
I agree that there should be a more general solution than special casing Seq and Option...
Also, there should be some better way to show noob chiselers what to do/not do as far as which apply methods to use [too many options = bad]. Having to sift through Chisel code is painful. .__.

On the specific matter of the apply methods (both for Vec and UInt), I am still insistent that they are a bad design in the API (regardless of their 'legacy status') since they exhibit such different behaviors even with similar names, argument, and return types.
The fact that you are not the first, nor will be the last, to have this issue reinforces my complaint.

There are a large number of outstanding issues around the Chisel3 Vec (e.g., #247, #260).  I strongly agree that some of the apply methods need to be deprecated, and we need to standardize around the "right" way to use Vec, with clear documentation of what it should and should not be used for.

Superseded by #445 which will have stricter type / hardware node checks.
Heterogeneous vecs are unsupported, recommended alternative is to use Record.


Mistake, please delete

@funerr Can you not close this yourself? It would be right next to the "Comment" button.

@impinball, Should I close it or not close it?

@funerr You should.
(It's generally best practice on any project in GitHub for these kinds of situations, as they can always be reopened.)


DEX-784 Password reset emails

Add tests and refactor auth Code model

Change Code.find to Code.find_all because it returns multiple
objects

Create new class function Code.valid_codes and include expires and
response in the query instead of calling is_expired and
is_resolved so we can make better use of sql instead of post
processing the result

Compress CODE_VALID_CHARACTERS to a string from list of characters
CODE_VALID_CHARACTERS is used in 2 ways:
random.choice(CODE_VALID_CHARACTERS)

and
if char in CODE_VALID_CHARACTERS

These work exactly the same way if CODE_VALID_CHARACTERS is a string
instead.

DEX-784 Password reset emails
Use this API to send the user a password reset email:
POST /api/v1/users/reset_password_email

{"email"<EMAIL_ADDRESS>
The email looks like:

Hello BetaBook user,
You can reset your password here: http://localhost:84/auth/code/FEB9F20A8C27C650E316A4B493EE544841FC24C89252661A4A36335E0D5ED600
The BetaBook team

^ Frontend needs to provide a page for resetting the password at /auth/code/<code_string>.
The API for resetting the password is:
POST /api/v1/auth/code/<code_string>

{"password": "new-password"}

/rebase


[BEAM-1334] Split UsesMetrics category and tests into UsesCommittedMetrics and UsesAttemptedMetrics

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @bjchambers
CC: @davorbonaci @amitsela

Changes Unknown when pulling c33fc4c1ea91557b845e13889c1791bb590e3f4c on aviemzur:split-metrics-test-category into ** on apache:master**.

Changes Unknown when pulling c33fc4c1ea91557b845e13889c1791bb590e3f4c on aviemzur:split-metrics-test-category into ** on apache:master**.

@bjchambers
Applied the renames you requested.

Coverage decreased (-0.09%) to 69.649% when pulling 772ec2cc90e8ec80c1c9d952e19afa640adb5351 on aviemzur:split-metrics-test-category into 34b4a6d9dc2cf5e8da43346077a36b460501afe2 on apache:master.


Enable CMake Unity build for Android

Can you maybe provide a bit of a description what is that and why is it useful? 😊

@ntadej Yeah I just opened it to see if it builds. 🙈


Updated documentation for modmail conversation (#1572)

(cherry picked from commit 742e61001e5eafbdd72eca5d8ace0f925c954533)
References

https://github.com/praw-dev/praw/pull/1572

Pull Request Test Coverage Report for Build 59234937c6c9fccc9e88b69bcdef9ba5b6440030-PR-41

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 34aa1e73399e358e0cfe207c5fcc1b12acfd2afe:
0.0%

Covered Lines:
3559

Relevant Lines:
3559

💛  - Coveralls

Pull Request Test Coverage Report for Build 59234937c6c9fccc9e88b69bcdef9ba5b6440030-PR-41

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 34aa1e73399e358e0cfe207c5fcc1b12acfd2afe:
0.0%

Covered Lines:
3559

Relevant Lines:
3559

💛  - Coveralls


can you tell me how to deploy  bch-insight  ?  I need it .

hi, all , can you tell me how to deploy  bch-insight  ?  I need it .
can you gave me a documents?

Same to you!

@kleetus

@alwaysAn0n can you help me ?

@linzhoushi2017 Sadly I cannot.  I saw your thread in Bitcoin-ABC though.  Since the issue doesn't really belong on that repo, I'll answer it here (it doesn't really belong here either though).
With regards to
[2018-06-08T08:50:00.319Z] error: uncaught exception: Error: Non-base58 character```

It looks like a character parsing issue in `insight-api` .  I would compare the `insight-api` version being used in `bitcore` and compare that against the most recent stable version of `insight-api`.  You might need to swap them.

Also, I think the best place for you to seek help about this is `bitpay/bitcore` or `bitpay/insight-api`.  Good luck

@alwaysAn0n  My  dependencies of =="insight-api" ==version is =="^0.4.3"==,  It is the latest release version. I think it was because of Bitcoin cash (BCH) address upgrade that caused all this error, but I didn't find a specific solution.I am looking for help everywhere.

@alwaysAn0n My dependencies of  "insight-api" version is "^0.4.3", It is the latest release version. I think it was because of Bitcoin cash (BCH) address upgrade that caused all this error, but I didn't find a specific solution.I am looking for help everywhere.

You can just trick it if you set usecashaddr=0 in the bitcoin.conf this basically changes the node to still use legacy address format


Specifing different object directory other than the source directory for the object file and .mod file

I'm using fortran-language-server with vscode. It creates .mod files in the sources directory for each modules. Usually, when building with makefile I create a different obj directory where I put all the .mod and .o files. can this be done with the fortls so that the main sources directory stays clean?

Hi there, so fortls does not provide linting in VSCode, that is the job of the underlying Fortran extension, see extension requirements.
If you are using Modern Fortran you can do what mentioned in this issue https://github.com/krvajal/vscode-fortran-support/issues/175

ok. thanks,


FIX: uploaded column check

If one of the required columns was the last one the if clause in import_data fails, because the newline character from the input file is still attached. This was fixed by adding a strip() before the split().

Thanks a lot for pointing this out!
With the nbdev setup, we can unfortunately not make direct changes to the .py files, but need to edit and then export the notebooks in the nbs folder. Just telling you in case you want to make any edits in the future. I will incorporate the strip() in the notebook and export the py file again.


I don't understand why you do open-source this to put all this junk inside your soft.
Wanted to try something else than https://sindresorhus.com/caprine/, but I guess that's definitely the best solution.

I didn't have the above "junk" on my mac and windows during installation and after installation.
But yea, now I've been using caprine, seems fine since this one hasn't received updates for over a year now

A bad actor got ahold of the .com website and started spreading malware on it. The real website for this app is messengerfordesktop.org

Damn this is still an issue. I'm the owner of the .org website. Since this is still not resolved after a year I suppose I will have to fork again and fix this once and for all.

As the dude in #1334 pointed out, the fact that @aluxian hasn't immediately attempted to distance his own name/software/GitHub profile from the malware, while not exactly damning evidence, is still pretty suspicious.
No matter what kind of monetization scheme he planned for the app, whether it be in-app ads or whatever, I'm pretty sure malware is the opposite of the right way to go.

Windows PUA blocked the app, both downloaded installer from the main page and the portable app from the release page on Github. This is a serious issue.
https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?name=PUA%3AWin32%2FInstallCore&threatid=213927
Also VirusTotal report on the portable zip file
https://www.virustotal.com/#/file/ca05a30b24ab283676b9ba0104ce9344ab2bd2a95eed284c30702cc07f18cc52/detection

The source code (package.json) even has this:
"homepage": "https://messengerfordesktop.com"
Seems it's deliberate?

I feel like @aluxian made himself quite clear that he put ads in the installer to get  a bit of revenue from this side project of his. It all started from this. It's then that the problems started getting worse and worse with no words from him.
@Sytten I think this is the right course of action. If you accept I might even come help, that'll give myself to take a peek into Electron app development.

IMO this repository should be reported and closed.

That's sad, really. This app was good.
I see Caprine took up a lot of the feature I missed when I tried it some time ago. Oh well...

Yeah, i've been using caprine for a while. Its stable for me.

So I got got by the .com version. I've removed MacKeeper and the Safari extension, but how do I get rid of the rest of it?

i just did a quick sudo dpkg
--remove messengerfordesktop
is there any reason this isn't enough?

AFAIK The Linux packages don't feature the malware as they don't include
the installer, since the package is installed by your system's package
manager.
On Wed, Apr 11, 2018, 00:49 Squilliam<EMAIL_ADDRESS>wrote:

i just did a quick sudo dpkg
--remove messengerfordesktop
is there any reason this isn't enough?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aluxian/Messenger-for-Desktop/issues/1343#issuecomment-380272035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEGocxc5neGAsIajbP2Qgq-TUt8Ymoiaks5tnTb6gaJpZM4SKow2
.

--
Nathan Graule
+33 6 79 58 29 83


add name option for route building

This may also solve https://github.com/channelcat/sanic/issues/895

no, it can not. the later name will be ignored as sanic use url as key that caused only first name will be the winner.
app = Sanic('test_dynamic_route')

@app.route('/overload', methods=['GET'], name='route_first')
async def handler1(request):
    return text('OK1')

@app.route('/overload', methods=['POST', 'PUT'], name='name_will_be_ignored')
async def handler2(request):
    return text('OK2')

Not sure if this makes sense.
# sanic/router.py

class RouteNameExists(Exception):
    pass

class Router:
    # other codes
    def __init__(self):
        # other codes
        self.routes_names = {}

    def _add(self, uri, methods, handler, host=None, name=None):
        # other codes
        if handler_name in self.routes_names:
            raise RouteNameExists(
                "Route name already exists: {}".format(handler_name))

        self.routes_names[handler_name] = (uri, route)

    def remove(self, uri, clean_cache=True, host=None):
        if host is not None:
            uri = host + uri
        try:
            route = self.routes_all.pop(uri)
            for handler_name, pairs in self.routes_names.items():
                if pairs[0] == uri:
                    self.routes_names.pop(handler_name)
                    break

        except KeyError:
            raise RouteDoesNotExist("Route was not registered: {}".format(uri))

        # other codes

    @lru_cache(maxsize=ROUTER_CACHE_SIZE)
    def find_route_by_view_name(self, view_name):
        # ...
        return self.routes_names.get(view_name, (None, None))

I tried, seems work.

@lixxu do you mean that it should be ok to close #895 ?

yes, it's ok to close. but seems there's a bug that when specify the name that same as the handler name for first function (don't know why). like this:
# below will not work
@bp.get('/<post_id>', name='show')
async def show(request, post_id):
    pass

# below is ok
@bp.get('/<post_id>', name='show')
async def get(request, post_id):
    pass


CUDA error: operation not supported when calling cusparseCreate(handle) on 4090 GPU

I run python exps/det/CRN_r18_256x704_128x128_4key.py --amp_backend native -b 1 --gpus 1 and have below errors
/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/connectors/data_connector.py:240: PossibleUserWarning: The dataloader, train_dataloader, does not have many workers which may be a bottleneck. Consider increasing the value of the num_workers argument(try 40 which is the number of cpus on this machine) in theDataLoaderinit to improve performance.   rank_zero_warn( Epoch 0:   0%|                                                                                | 0/29634 [00:00<?, ?it/s]Traceback (most recent call last):   File "exps/det/CRN_r18_256x704_128x128_4key.py", line 359, in <module>     run_cli(CRNLightningModel,   File "/work/data06/git_/CRN/exps/base_cli.py", line 85, in run_cli     trainer.fit(model)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 771, in fit     self._call_and_handle_interrupt(   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 722, in _call_and_handle_interrupt     return self.strategy.launcher.launch(trainer_fn, *args, trainer=self, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/strategies/launchers/subprocess_script.py", line 93, in launch     return function(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 812, in _fit_impl     results = self._run(model, ckpt_path=self.ckpt_path)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1237, in _run     results = self._run_stage()   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1324, in _run_stage     return self._run_train()   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1354, in _run_train     self.fit_loop.run()   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 204, in run     self.advance(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/fit_loop.py", line 269, in advance     self._outputs = self.epoch_loop.run(self._data_fetcher)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 204, in run     self.advance(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/epoch/training_epoch_loop.py", line 208, in advance     batch_output = self.batch_loop.run(batch, batch_idx)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 204, in run     self.advance(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/batch/training_batch_loop.py", line 88, in advance     outputs = self.optimizer_loop.run(split_batch, optimizers, batch_idx)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/base.py", line 204, in run     self.advance(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 203, in advance     result = self._run_optimization(   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 256, in _run_optimization     self._optimizer_step(optimizer, opt_idx, batch_idx, closure)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 369, in _optimizer_step     self.trainer._call_lightning_module_hook(   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1596, in _call_lightning_module_hook     output = fn(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/core/lightning.py", line 1625, in optimizer_step     optimizer.step(closure=optimizer_closure)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/core/optimizer.py", line 168, in step     step_output = self._strategy.optimizer_step(self._optimizer, self._optimizer_idx, closure, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/strategies/ddp.py", line 278, in optimizer_step     optimizer_output = super().optimizer_step(optimizer, opt_idx, closure, model, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/strategies/strategy.py", line 193, in optimizer_step     return self.precision_plugin.optimizer_step(model, optimizer, opt_idx, closure, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/plugins/precision/native_amp.py", line 85, in optimizer_step     closure_result = closure()   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 148, in __call__     self._result = self.closure(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 134, in closure     step_output = self._step_fn()   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/loops/optimization/optimizer_loop.py", line 427, in _training_step     training_step_output = self.trainer._call_strategy_hook("training_step", *step_kwargs.values())   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 1766, in _call_strategy_hook     output = fn(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/strategies/ddp.py", line 344, in training_step     return self.model(*args, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/parallel/distributed.py", line 799, in forward     output = self.module(*inputs[0], **kwargs[0])   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/pytorch_lightning/overrides/base.py", line 82, in forward     output = self.module.training_step(*inputs, **kwargs)   File "exps/det/CRN_r18_256x704_128x128_4key.py", line 298, in training_step     preds, depth_preds = self(sweep_imgs, mats,   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "exps/det/CRN_r18_256x704_128x128_4key.py", line 281, in forward     return self.model(sweep_imgs, mats, sweep_ptss=inputs['pts_pv'], is_train=is_train)   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "/work/data06/git_/CRN/models/camera_radar_net_det.py", line 90, in forward     feats, depth, _ = self.backbone_img(sweep_imgs,   File "/home/iot/miniconda3/envs/CRN/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl     return forward_call(*input, **kwargs)   File "/work/data06/git_/CRN/layers/backbones/rvt_lss_fpn.py", line 397, in forward     key_frame_res = self._forward_single_sweep(   File "/work/data06/git_/CRN/layers/backbones/rvt_lss_fpn.py", line 304, in _forward_single_sweep     geom_xyz, geom_xyz_valid = self.get_geometry_collapsed(   File "/work/data06/git_/CRN/layers/backbones/rvt_lss_fpn.py", line 195, in get_geometry_collapsed     points = ida_mat.inverse().matmul(points.unsqueeze(-1)).double() RuntimeError: CUDA error: operation not supported when callingcusparseCreate(handle)`
Epoch 0:   0%|          | 0/29634 [00:11<?, ?it/s]

Hello, have you solved the problem yet?

Try change these codes in base_lss_fpn.py and rvt_lss_fpn.py .
points = ida_mat.inverse().matmul(points.unsqueeze(-1)).double()
points = (ida_mat.cpu().inverse().cuda()).matmul(points.unsqueeze(-1)).double()
combine = sensor2ego_mat.matmul(torch.inverse(intrin_mat)).double()
combine = sensor2ego_mat.matmul(torch.inverse(intrin_mat.cpu()).cuda()).double()

I didn't face the same error when I used 3090 GPU, so I am not sure, but I think that error is related to the data type (half, float, double).


Reconnect w 2FA Not Working on Desktop

Bug description
When SSH connection is lost I cannot enter the verification code upon entering R. Seems like the focus is on the terminal and – even when clicking on the pop-up window I cannot enter the 2FA code.
MacOS 14.5 MacBook Air
Screenshot attached
Steps to reproduce

Login with 2FA e.g. Google Authenticator
Wait until server cuts connection
Enter "R" to reconnect: verification code pop-up shows up
I cannot enter the code in the pop-up since the "focus" is not on the pop-up. Even clicking on it does not help.

Hello @karero 👋
Many thanks for your report 🙏
I'm planning it for the upcoming minor release.

Hi @isontheline That's cool. 😎 Thank you. 🙏
Please check the iPhone app as well. I think the same is happening there too.

Hi @karero 👋
Could you please try WebSSH 27.2.1377 ?
https://testflight.apple.com/join/QSrBK59z

Tested on MAC and iPhone, both work fine now.

Thanks for your feedback @karero 🙏


feat: third party app support in EVADB

This PR introduces a generic interface to support 3rd party apps in EVADB. As an example. the template for integrating slack has been added. In a subsequent PR the integration with slack will be completed.

I'd like to understand the context of this PR better. We discussed the Slack chatbot use case yesterday, and I believe that it can be a standalone app without changing EvaDB internally or adding third-party integrations.
Is this PR going to facilitate some other new use cases? If so, could you give some detailed examples here and also explain a bit about how it will interact with EvaDB?
Thank you!

It seems that only the name is different. One is called APPLICATION, and the other is called DATABASE. We can support these two syntax in the evadb.lark, and share all other components. So we don't need to involve the app_type, which also simplifies the code base.

It seems that only the name is different. One is called APPLICATION, and the other is called DATABASE. We can support these two syntax in the evadb.lark, and share all other components. So we don't need to involve the app_type, which also simplifies the code base.

Are you suggesting to use the same interface as DATABASE? Otherwise, we will need to somehow differentiate between applications and databases.

It seems that only the name is different. One is called APPLICATION, and the other is called DATABASE. We can support these two syntax in the evadb.lark, and share all other components. So we don't need to involve the app_type, which also simplifies the code base.

I think it is good to have some differentiator. I would prefer to keep app_type and remove every other difference beyond the lark parser?

It seems that only the name is different. One is called APPLICATION, and the other is called DATABASE. We can support these two syntax in the evadb.lark, and share all other components. So we don't need to involve the app_type, which also simplifies the code base.

I think it is good to have some differentiator. I would prefer to keep app_type and remove every other difference beyond the lark parser?

Thanks Kaushik for all the hard work. Sorry for the back and forth on the design. This sounds good to me. Let's do it.

It seems that only the name is different. One is called APPLICATION, and the other is called DATABASE. We can support these two syntax in the evadb.lark, and share all other components. So we don't need to involve the app_type, which also simplifies the code base.

I think it is good to have some differentiator. I would prefer to keep app_type and remove every other difference beyond the lark parser?

Thanks Kaushik for all the hard work. Sorry for the back and forth on the design. This sounds good to me. Let's do it.

Np, thanks a lot for your comments!

@kaushikravichandran @aryan-rajoria What is the status of this PR?

That's a big PR, @kaushikravichandran! Nice job, let's wrap it up.

If it is good to merge, let's do that. Students working on application integrations will require the handler inferface.

I will take care of the PR.

I will take care of the PR.

Thanks. I have resolved the latest set of conflicts now.

@kaushikravichandran Please add docs for the app handler interface soon as students will need it.

Any reason the unfortunately repeated conflict merging on this PR. @xzdandy @kaushikravichandran

@kaushikravichandran is reducing the size of this PR.


request: ability to format date/time format

Is your feature request related to a problem? Please describe.
Not a problem, visual enhancement
Describe the solution you'd like
Simple provide the ability to enable 12hr/24hr/ AM / PM - fully descriptive version of the date etc. etc.
Describe alternatives you've considered
I have checked all documentation on HACS and can't see any way to alter date and time formats.
Additional context
N/A

See #368

For context, and people that find this in future.
Locale settings in the Profile page enable you to use HA’s pre-structured options, and, whilst it does work the list is limited.
equally, date filter options are also very limited and don’t offer full day name options, meaning you’d either have to manually modify this GIT js or wait for improved functionality here or in locale settings for the user.


[WIP] Add more examples and some helper functions

Hello @holgern! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file examples/exploration-vs-exploitation.py:

Line 33:1: E402 module level import not at top of file
Line 35:1: E402 module level import not at top of file
Line 46:1: E402 module level import not at top of file
Line 47:1: E402 module level import not at top of file
Line 53:1: E302 expected 2 blank lines, found 1
Line 59:1: E305 expected 2 blank lines after class or function definition, found 1
Line 59:54: E251 unexpected spaces around keyword / parameter equals
Line 59:56: E251 unexpected spaces around keyword / parameter equals
Line 69:1: E402 module level import not at top of file
Line 70:1: E302 expected 2 blank lines, found 0
Line 106:1: E305 expected 2 blank lines after class or function definition, found 1

In the file examples/optimizer-with-different-base-estimator.py:

Line 12:80: E501 line too long (80 > 79 characters)
Line 18:1: E402 module level import not at top of file
Line 20:1: E402 module level import not at top of file
Line 34:1: E302 expected 2 blank lines, found 1
Line 40:1: E305 expected 2 blank lines after class or function definition, found 1
Line 40:1: E402 module level import not at top of file
Line 42:17: E128 continuation line under-indented for visual indent
Line 51:1: E402 module level import not at top of file
Line 53:1: E302 expected 2 blank lines, found 1
Line 117:1: E305 expected 2 blank lines after class or function definition, found 1
Line 132:1: E402 module level import not at top of file
Line 133:1: E402 module level import not at top of file
Line 136:1: E402 module level import not at top of file


Stop building performance artifacts for Android

Our full performance benchmark has been broken since Jan 9th. This is the link to the first failure: https://fusion.corp.google.com/runanalysis/test/prod%3Agrpc%2Fcore%2Fmaster%2Flinux%2Fgrpc_full_performance_master/prod%3Agrpc%2Fcore%2Fmaster%2Flinux%2Fgrpc_full_performance_master/KOKORO/22b4aa53-f5a6-49df-b746-cd2b537a8db9/1578618684324/build %232950/Targets?target=grpc%2Fcore%2Fmaster%2Flinux%2Fgrpc_full_performance_master&drilldownTab=logs
The possible culprit is the Android build failure:
+ ./gradlew -PskipCodegen=true :grpc-benchmarks:installDist
Starting a Gradle Daemon (subsequent builds will be faster)
*** Skipping the build of codegen and compilation of proto files because skipCodegen=true
*** Android SDK is required. To avoid building Android projects, set -PskipAndroid=true

FAILURE: Build failed with an exception.

* Where:
Build file '/home/kbuilder/performance_workspace/grpc-java/cronet/build.gradle' line: 49

* What went wrong:
A problem occurred evaluating project ':grpc-cronet'.
> SDK location not found. Define location with sdk.dir in the local.properties file or with an ANDROID_HOME environment variable.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 9s

Let's see if disabling it fix the performance benchmark or not.

There's a parallel effort from @codeblooded to properly fix these tests.

@nicolasnoble  Oh... I didn't know that, thanks for reminding. If the proper fix takes non-trivial effort, this PR can be a quick fix.
(Don't know if this PR fixed the problem or not yet.)
Fusion BUILD

Hey @lidizheng, looks like we were both proactive here. :smile:
I made a similar change in #21879.  Looks like two minds think alike.   I confirmed it's building there.  I can close mine and we can merge your branch, no biggie.  I'll make myself a reviewer and suggest some other things so it's the same.

Some references for a later date:

PR #21628 for a similar change and description (fixing interop tests)
PR #21848 for related documentation changes.

@codebender Thanks for providing the context 👍  The fusion build has been up for more than 1 hour which suggests it has passed the build phase.


feat: Uglify-es support for quantum

I asked at gitter for the best way to design an API for it, but got no reply.  So here is the simplest implementation possible, though it probably isn't very straightforward to use.
It picks up uglify's options from the UglifyESPlugin or could be triggered by quantum option uglify: { es6: true }

Well, I hope it's comprehensible.  :)


Allocated time exceeded

When running the action, this error occurs
Runtime error encountered: Waiting for DevTools protocol response has exceeded the allotted time. (Method: Runtime.evaluate)
LHError: PROTOCOL_TIMEOUT
    at Timeout._onTimeout (/usr/local/lib/node_modules/lighthouse/lighthouse-core/gather/driver.js:413:21)
    at listOnTimeout (internal/timers.js:549:17)
    at processTimers (internal/timers.js:492:7)

I do experience the same issue here.
There is a lighthouse issue (https://github.com/GoogleChrome/lighthouse/issues/10996) which points to https://github.com/GoogleChrome/lighthouse/issues/6512.
This might be helpful to figure out the cause for this issue.


Testing

Todo:

[ ] Add CI

What do when dependency chain breaks because Travis uses Node server < 4.0 in 2016?


ElasticSearch timeout should be handled differently not to push additional traffic

The property 'sleepOverClientException' is needed, by default, 1 minute. For whatever exception thrown while communicating with elasticsearch server, it will sleep for sleepOverClientException.

#177


Stuck on startup when upgrading from 0.9 to 0.10

Emerald Wallet gets stuck here:

The temporary fix is to click to the right of the monitor icon and select a node.

can you clear your settings by following this: https://github.com/ETCDEVTeam/emerald-wallet#troubleshooting
let me know if a fresh start fixes it

Yup, that also fixes it


Collapse ConfigFile into Config, use pyserde to serialize

This should be equivalent to the old implementation, with the exception of flag changes:

Remove --install-dir and --beta
Add --config-file

It's a pretty invasive refactor, so I intend to persist tracker settings in a later change.

Good call.  Turned out what I thought was a test error was a bug: forgetting to set config_path before calling save()


basic chat system fixes, add a "any message" function

fixed every bugs on DirectMessage(), also fixed OnIngameGlobalMessage() send a team chat packet (1 line fix lol), added more log for every chat type
should fixed half of the https://github.com/L-Leite/cso2-master-server/issues/23 (still not working now: clan, party, channel, megaphone(need item to use it, item id: 2010-2012. global is not available in client))
note: clan and party (include the ingame party) can be simple fixed by copy and paste the other codes, but still need the original clan and party function so we can use it
now if user is requesting login and get the 401 code, server will send a message tell them "Wrong account / password"

about "any message":
it is a simple function to send "any" message to the client, include 4 different types
(note: type "Congratulate" has a timer on the client, if the timer didn't run out, send more "Congratulate" type's message will be useless, it will not draw at the client. server can not stop the timer)
it will be need on most function has "return false", like user is joining a full room, you can send a "room is full" message to the client now. (#CSO2_POPUP_ROOM_JOIN_FAILED_FULL)
should help you with https://github.com/L-Leite/cso2-master-services/projects/1#card-21401708
IMPORTANT: send a message with DialogBox type will stop the "gear loading" immediately

my testing sample:

the code i'm using for test:
// chat "any message" testing start (TypeScript)
// tested in usermanager onLoginPacket() and working
// NEED TO CHANGE "connection"!!!
// ||||||||||||||||||||||||||||||||||||||||||||||||||
// import things
// import { ChatHandler } from 'handlers/chathandler'
// import { ChatMessageType } from 'packets/definitions'
// import { ChatService } from 'services/chatservice'

const chatHandler: ChatHandler = new ChatHandler(new ChatService('https://implement.me.invalid'))

// sample of using wrong type
chatHandler.OnAnyMessage(connection, 'I\'m the wrong type message', ChatMessageType.Room)
// sample of using null message
chatHandler.OnAnyMessage(connection, '', ChatMessageType.System)

// normal message
chatHandler.OnAnyMessage(connection, 'Hello World!', ChatMessageType.Congratulate)
// normal message with non-ascii text (file must be saved as UTF-8)
chatHandler.OnAnyMessage(connection, '你好世界！', ChatMessageType.System)
// #text (game translation) supported
chatHandler.OnAnyMessage(connection, '#CSO2_ITEM_NAME_m99railgunplus', ChatMessageType.System)
// simple html code (<color>) and "\n" supported
chatHandler.OnAnyMessage(connection,
    'CLICK THE BUTTON BELOW\nTHEN YOU CAN CLOSE ME!\n\nGood\n:)',
    ChatMessageType.DialogBox)
// chat "any message" testing end

Thanks for your work


Issue 689: Add type/type_id to endpoint object

Related Issue: 689
Description of changes:

Moved device type enums to endpoint object
Updates device/network_endpoint objects to have specific type requirements/descriptions

Hi @Aniak5 , - Along the topic of Endpoint - saw a minor grammatical error in the description of the Endpoint observable in the observable object
The endpoint object describes an entity that has name, uid, and IP address.
should be
The endpoint object describes an entity that has a name, uid, and IP address.
Is this something you could add to this PR?

Hi @Aniak5 , - Along the topic of Endpoint - saw a minor grammatical error in the description of the Endpoint observable in the observable object
The endpoint object describes an entity that has name, uid, and IP address. should be The endpoint object describes an entity that has a name, uid, and IP address.
Is this something you could add to this PR?

Yup definitely, good catch @mr-splunk!


usage: updateNewMaxId after sanity check

Description
This PR fixes an issue with usage sanity check.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[x] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@nvazquez
this is a known issue we have discussed before.

@blueorangutan package

@weizhouapache a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1648

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2501)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31146 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5646-t2501-kvm-centos7.zip
Smoke tests completed. 91 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


Audio now playing screen shows #FFFFFF

Search the code for #ffffff, try commenting some out until you find the offending one.

Haw.. ok...DONE!


invite me to organization

Contact Details
<EMAIL_ADDRESS>Discord Username
No response
Code of Conduct

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

thanks.


What is the format I should use for the config file?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I need some help setting up a config file. While, yes, I can get things to work by typing in "--u username --p password," I would like to tuck this all into a file. Unfortunately, I've no idea what format to follow in terms of entering my username/password. Also is there a way to set a default directory since that would save a little extra times in pointing it to a list of files to download.
Finally, on a different matter, because I don't want to download 10+GB for an hour and something video (yes...seriously...that's what I'm facing right now), I was wondering if there was a better way to enter something besides the following:
yt-dlp -f "((bv*[fps>30]/bv*)[height<=720]/(wv*[fps>30]/wv*)) + ba / (b[fps>30]/b)[height<=720]/(w[fps>30]/
to get everything to download in 720p or lower if not available...ideally in a config file, but I understand if this is something I'm going to have to type out manually.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vu', '-a', 'files.txt', '-p', 'PRIVATE', '--username', 'PRIVATE', '--password', 'PRIVATE']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b634ba742] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 2023-05-18-git-01d9a84ef5-full_build-www.gyan.dev (setts), ffprobe 2023-05-18-git-01d9a84ef5-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, sqlite3-3.35.5, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1890 extractors
[generic] Extracting URL: files.txt
ERROR: [generic] 'files.txt' is not a valid URL. Set --default-search "ytsearch" (or run  yt-dlp "ytsearch:files.txt" ) to search YouTube
  File "yt_dlp\extractor\common.py", line 715, in extract
  File "yt_dlp\extractor\generic.py", line 2410, in _real_extract

For storing and automating credentials, use a .netrc file with the --netrc option. See https://github.com/yt-dlp/yt-dlp#authentication-with-netrc

This section of the readme covers all you need to know about config files: https://github.com/yt-dlp/yt-dlp#configuration

the options are case-sensitive:

-vu is not the same as -vU
-vu is equivalent to --verbose --username, while -vU is equivalent to --verbose --update
this is why yt-dlp doesn't recognize files.txt as an argument to -a- in your log; it is parsing -a as the argument to the --username option (-vu -a files.txt)


fix handle elasticsearch search result

fix https://github.com/sethuiyer/Image-to-Image-Search/issues/14

Thanks for your contribution!

Thanks for your contribution!


Add support for disabling specific steps

For my local mesos setup, I only want docker to be set up - I don't care about etcd, fleet, etc.
It'd be good to be able to explicitly switch off certain steps, so they don't run at all.

Today I learn about the 'steps' flag - thanks @xh3b4sd


Give a CSS style to "OK" link on focus for accessibility

Right now there's only a CSS style for the a:active selector.
Accessing the cookiconsent section using the keyboard and focusing on the accept button will not change the button color or highlight the status in any way.

@piiupilt is this an issue with Cookie Consent 3?

Fixed in #166.
Included in release 3.0.2


Device not supported on mac

Hello,
Using a Mac (running macOS 10.13.4) I set-up both my Mac and my Ledger Nano S according to the OpenPGP Card Application User Guide
I followed these steps

[x] Disable SIP
[x] Add the info provided in section 3.2.2. to /usr/libexec/SmartCardServices/drivers/ifd- ccid.bundle/Contents/Info.plist
[x] Reboot
[x] Install GPG stable 2.2.6 through brew (command brew install gpg)
[x] Set a key template for slot 1 as described in section 4.4.1
[x] Add configuration in ~/.gnupg/scdaemon.conf as described in section 5.1.1

It still seems that my ledger is not supported:
$ gpg --card-status
gpg: selecting openpgp failed: Operation not supported by device
gpg: la carte OpenPGP n'est pas disponible : Operation not supported by device

Note: It also seems that version of macOS does not sport pcsc_scan out of the box
What more is needed to make the OpenGPG app usable on this system?

This issue is duplicate of issue #18
I close this issue in favor of issue #18 but the problem is not solved


Fix doc with attachment lifecycle - updating a doc with attachments that was fetched with {attachments: false} is throwing error

Fix doc with attachment lifecycle - updating a doc with attachments that was fetched with {attachments: false} is throwing error

Users can db.get(id, { attachments: false }).then(doc => db.put(doc)) without error
Added tests that prove failure before this change and success after this change

thanks for contribution!
just published a new version on npm
Greez
Christoph


Add InstanceOperations.setProperties() method

Fixes #2244
These changes add a setProperties method to InstanceOperations. There are many places in the code that set multiple properties one after another each with its own call to setProperty. This change may make the code cleaner by allowing multiple properties to be set with a single call using a map of properties.
Not sure if this is a change we want to make, just thought I'd give it a shot. If this is merged, I'll create a follow on ticket to find and convert places in the code that may benefit from this.

the other changes likely will be superseded

In that case, should I change this PR to only include the doc updates or keep everything?

the other changes likely will be superseded

In that case, should I change this PR to only include the doc updates or keep everything?

You can either change this PR to only include the doc updates, and we can hold off on the other changes until later... or you can submit a new PR with only the doc updates, and when that is merged, you can rebase this PR and mark it "WIP" for the rest of the stuff, until we're ready for it.

I went ahead and changed this PR to only include the doc improvements. Another PR can be made down the road if we choose to revisit #2244


Initialization of lists and a scalar on the same line fails

Hello team, my apologies if it's not a bug, but procedure
def test(a,b):
    x,y,data,loss=[],[],[],0.0
    for i in range(len(a)):
        x.append(a[i])
        y.append(b[i])
        loss=loss+abs(a[i]-b[i])
    return x,y,loss
test(np.asarray([1,2,3]),np.asarray([4,5,6]))

gives

([1, 2, 3], [4, 5, 6], 9.0),

and its njitted version
njit(test)(np.asarray([1,2,3]),np.asarray([4,5,6]))

fails in Numba 0.43.0 with

TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Internal error at <numba.typeinfer.StaticGetItemConstraint object at 0x000002835D8BEB70>:
--%<----------------------------------------------------------------------------
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\errors.py", line 627, in new_error_context
yield
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\typeinfer.py", line 348, in call
assert itemty.is_precise()
AssertionError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\typeinfer.py", line 144, in propagate
constraint(typeinfer)
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\typeinfer.py", line 351, in call
self.fallback(typeinfer)
File "C:\ProgramData\Anaconda3\lib\contextlib.py", line 130, in exit
self.gen.throw(type, value, traceback)
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\errors.py", line 635, in new_error_context
six.reraise(type(newerr), newerr, tb)
File "C:\ProgramData\Anaconda3\lib\site-packages\numba\six.py", line 659, in reraise
raise value
numba.errors.InternalError:
[1] During: typing of static-get-item at  (2)
--%<----------------------------------------------------------------------------
File "", line 2:
def test(a,b):
x,y,data,loss=[],[],[],0.0

I know it looks ugly to init variables like this, but since it's syntactically valid in vanilla python, maybe it can also be expected to be compilable by Numba?

I'm curious how multiple assignment shows up in the bytecode.  Does Python actually build and immediately unpack a tuple?  (If it does, that would explain why Numba has a problem.)

It looks like:
  5           0 BUILD_LIST               0
              2 BUILD_LIST               0
              4 BUILD_LIST               0
              6 LOAD_CONST               1 (0.0)
              8 BUILD_TUPLE              4
             10 UNPACK_SEQUENCE          4
             12 STORE_FAST               2 (x)
             14 STORE_FAST               3 (y)
             16 STORE_FAST               4 (data)
             18 STORE_FAST               5 (loss)

the issue is that there's list(undefined) being fetched from a static getitem from the unpack tuple. I think it might be ok to remove that assert now as the undefined list would get caught in the legalization pass and a more suitable complaint issued.

If we could rewrite that assignment into:
x = []
y = []
data = []
loss= 0.0

Then I think type inference would be able to figure out what the list element types are based on the loop body.

I'm not sure that's needed? I think it's that the list data is undefined (it's dead) which is the problem, and because the multiple assignment is handled via a static getitem it hits a code path that isn't travelled often which has that assert on it. Were that assert removed the legalization pass would pick up that data is dead and complain about it being an undefined list type?
This seems ok:
from numba import njit
import numpy as np

def test(a,b):
    x, y, loss = [], [], 0.0
    for i in range(len(a)):
        x.append(a[i])
        y.append(b[i])
        loss=loss+abs(a[i]-b[i])
    return x,y,loss

test(np.asarray([1,2,3]),np.asarray([4,5,6]))
njit(test)(np.asarray([1,2,3]), np.asarray([4,5,6]))

Oh, I see.  I missed that data was unused.  Makes sense!


Updated subresource for Data Factory.

Updated subresource for Data Factory to correct value 'dataFactory'.

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

#label:"aq-pr-triaged"

@asudbring For review
Thanks.

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


Make crossword works with stateVisible soft keyboard

It can help to exclude moving of keyboard at start of crossword loading

I found that we had called showSoftInput even in stateVisible mode so I removed this additional call. Maybe it can help in straggle with delation in rendering,

In reality softKeybaord is got from a manifest so stateVisible from activity doesn't work


Add deeptools to monitored repositories

We need to observe this, deeptools tools are not on a usual location.