Suppress ipdb output during debug

Is this the right place to make a feature request in spyder ide to suppress the source code echo while using ipdb (thank you for making spyder an awesome ide)?
when i am using spyder to debug the ipdb prompt echos the source code (and a few lines below and above) please let me add a  way to suppress this source code echo.
I am a python tutor and i really like to use sypder and ipdb to show new programmers how each line of code changes the variable values and the output of some simple loop or if statement.
Do i need to make a feature request for ipdb?
other people on stackoverflow have made this request and i agree with them:
https://stackoverflow.com/questions/44308115/suppressing-ipdb-output-in-spyder-ipython-interpreter
https://stackoverflow.com/questions/51703320/python-spyder-is-there-a-way-to-suppress-the-position-output-during-debugging?rq=1

Something like this in the appropriate spot might work:
ipdb.set_trace(context=0)

Nope that doesn't work.  IPython debugger doesn't allow zero context.
I can get it down to one line though by change a couple line in spydercustomize.py:
~line 387 change to:
@monkeypatch_method(pdb.Pdb, 'Pdb')
def __init__(self, completekey='tab', stdin=None, stdout=None,
             skip=None, nosigint=False, context=5):
    self._old_Pdb___init__(context=context)

~line 682 in the debugfile function change to:
debugger = pdb.Pdb(context=1)

thank you for the information about ipython debugger, should i make a feature request in ipython (or ipython debugger)?

This is the file that would need to be changed:
https://github.com/ipython/ipython/blob/master/IPython/core/debugger.py
I checked the line return logic and it returns an empty list with a context of 0, but that might break some things further down the line.
I don't think many people would want 0 context other than Spyder users.

@lazy2late, they will be removed in Spyder 4. For now, there's nothing we can do about it, sorry.

thank you very much

The echo is created in spydercustomize.py by self.print_stack_entry(self.stack[self.curindex]). I personally like it but there could be an option to remove the echo.

@impact27, thanks for finding the source of the problem. I think we should remove that line and let users press ll or list on the debugger to find the current location they are placed in.
@bcolsen?

I disagree, my eyes have to travel much more if I have to refer to the editor each time I make a step.

I don't use debug that much(But I might use it more with all these great improvements :+1: ) but I can see both sides.
I think we should definitely have them as an option. Maybe off by default though? The only thing I have against them is it just makes me have to go back further in the console to find the problem I was having.
Could we have the lines displayed if the commands are entered into the console and not if hotkeys or the buttons are used?

We can do hidden execution to send an enable / disable command with the comms.

How about: no option, echo if typed in the console, and don't echo if using the buttons?


Generate region per ocr bounding box overlap

Creates a region per overlapping ocr box:

this lives in https://github.com/mmarqz/label-studio-frontend/pull/5 now


Brother plugin not available

Please verify that this bug has NOT been raised before.

[x] I checked and didn't find a similar issue

Describe the bug*
Print labels on brother label printer is no longer available on the drop down list.
Plugin listed and loaded on administration, configuration (printer model, IP, etc.) is not changed.
I've noted some new options like linventreelabelmachine probably recently added.
Wat's I need to do to recover this feature?
Thanks for the support
Steps to Reproduce
Print Label from any part
Select sample part label (or others options available)
On printing plugin drop down list, brother printer is missing (previously available)
Expected behaviour
Print on brother label printer as before.
Deployment Method

[x] Docker
[ ] Package
[ ] Bare metal
[ ] Other - added info in Steps to Reproduce

Version Information
Inventree 1.16.8
Brother Plugin 1.0
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No relevant logs has been found

Please provide full server version information, this is not enough info to debug

Additionally:

Did you recently update either the plugin or your InvenTree instance?
What do you see in the "plugin error log" (visible in the plugin admin view)

The plugin was not updated, I've update it to V1.0 to try to solve the issue.
Here the status of my plugins, let me check in the log

Plugin error is clean

Not sure if there are correlation between Machine plugin and the Brother Plugin.

@wolflu05 does this sound like the issue you are still experiencing in https://github.com/inventree/InvenTree/issues/7709 ?

@wolflu05 does this sound like the issue you are still experiencing in #7709 ?

Not really, I think this is more related to the brother plugin itself (I would transfer this issue to the plugin repo).
The brother plugin with 1.0.0 uses the machine interface instead of the old plugin interface. So @icvdok would just need to setup a machine use the brother driver now and when printing select machine printing > the previously created machine. This can be done in the Admin center of PUI > Machines (the django admin UI or CUI's admin functionality should not be used for this)
When the brother plugin is installed in v1.0.0 the brother plugin should show up to be selected as "driver" here:

If that does not work, you may need to restart the server and try a private tab / clear your browser cache.


Enable searching through all resources

According to the FHIR search spec, seaching all resources that match a provided parameter is possible:

All resource types: GET [base]?parameter(s) (parameters common to all types). If the _type parameter is included, all other search parameters SHALL be common to all provided types. If _type is not included, all parameters SHALL be common to all resource types.

https://www.hl7.org/fhir/search.html#Introduction
Currently, it doesn't seem that this is supported in this server implementation.
Alternatively, what other ways do I have currently to GET all resources specific to a Patient?

GoFHIR does support GET /Patient/id/$everything. It is implemented in EverythingHandler as Patient?_id=%s&_include=*&_revinclude=*

@eug48 thanks, $everything made the trick!


Add timer annotations in epoll1

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


[styles] Theme Provider is not working correctly when nested themes are used

Earlier themes were working fine before the 4.0 release but now the nested themes are overriding the themes which are outside its scope
for example in your documentation example https://codesandbox.io/s/material-demo-hdy1n
replace the demo.js code with the attached code in demo.txt file
demo.txt
The theme of components outside the nested themes are getting overridden by the nested one.
I am facing this challenge in my whole project as I have many nested themes and the ui has changed unexpectedly due to this,
earlier it was working fine with v3.9

@naman13924 You need a top level theme provider: https://codesandbox.io/s/material-demo-29fxp.

So basically multiple theme roots are not allowed on a page. There has to be a single provider root. Wouldn't that make multiple react trees impossible on a page?

@eps1lon I have demonstrated the simplest workaround in the codesandbox: one ThemeProvder at the top of the tree.
For the multiple React tree pattern, people have to use a different seed if they want to use different themes: https://material-ui.com/styles/api/#creategenerateclassname-options-class-name-generator.

@eps1lon I have demonstrated the simplest workaround in the codesandbox: one ThemeProvider at the top of the tree.

Pretty sure that's what I said

So basically multiple theme roots are not allowed on a page. There has to be a single provider root.

For the multiple React tree pattern, people have to use a different seed if they want to use different themes

I would guess this is a pretty basic use case when using multiple react roots. That shouldn't be hidden behind nested API docs.

@eps1lon Yes, I have repeated it for people that might have the problem and that skip read. It's a really important point. I do agree that there is close to no documentation it about and that we should address the problem. I wasn't sure it would impact many people. I suspect few people have this use case, but I don't have enough data to defend this claim. I will work on it as more people report it (hence gain priority over the other issues)

Am I right in thinking that this is what has broken support of our v0.X theme nesting?

Also wondering that @Ajaay. We have a v0 theme nested in a now v4 theme and are running into a ton of issues that seem to be theme related.

If it makes any difference I didn’t have problems with v4.0.0. Only when I updated to v4.0.1 did they start
Get Outlook for iOShttps://aka.ms/o0ukef

From: Olivier Tassinari<EMAIL_ADDRESS>Sent: Wednesday, May 29, 2019 2:44:15 PM
To: mui-org/material-ui
Cc: Ajaay; Mention
Subject: Re: [mui-org/material-ui] Theme Provider is not working correctly when nested themes are used (#15914)
@ztobenhttps://github.com/ztoben It's still supported. Do you have a minimale reproduction?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mui-org/material-ui/issues/15914?email_source=notifications&email_token=ABO3LPZTCPS6EHJ7KN52223PX2CC7A5CNFSM4HQCI2H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWPLYTI#issuecomment-496942157, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABO3LP66YW7LHLRA6I4BG3TPX2CC7ANCNFSM4HQCI2HQ.

With root I mean a provider which has no parent provider:
<React.Fragment>
	<ThemeProvider /> /* root, has siblings */
	<ThemeProvider /> /* root */
	<ThemeProvider /> /* child */
</React.Fragment>

<React.Fragment>
	<ThemeProvider> /* root */
		<ThemeProvider /> /* child */
		<ThemeProvider /> /* child */
	</ThemeProvider>
</React.Fragment>

This issue is driving me crazy. To be honnest all the explanations above do not sound clear to me and I'm not sure if I'm falling in a case cited above.
Whatever I have a nested theme that does not apply:
<ThemeProvider theme={darkTheme}>

            <div className={classes.navbar}>
              <MD.List>
                etc...
              </MD.List>
            </div>
</ThemeProvider>

Made some experiment and applying <MD.CssBaseline /> inside the nested theme will apply the dark theme, but globally. Even for the parents.

Just wanted to "upvote" this issue.
I think the current docs here: (Advanced / Theme Nesting) make it look like you should be able to call a theme within a theme and have it override all of the styles.
In my use case, it's overriding some of the styles, but not any of the type colors on <Typography> components (looks like those are by default inhereted from the body).
The fact that it seems partially implemented AND appears this way in the Docs makes it seem like a bug. I would suggest either making all theme styles work in the nested option, or revising the docs to make it more clear what nesting will achieve.
Thanks!

@alturnwall As we love to say: do have a reproduction (Codesandbox example preferably)

@mbrookes sure, I'm still pretty new at this, but here's a Codesandbox example.
There's a lot going on, but I'm mainly following up on this comment from @nareshbhatia:

However, I came up with a simple trick based on @oliviertassinari's hint that seems to work. I have introduced a <Paper> instance right below the nested theme. [...] This seems to work - the List background is dark and the text color is white. Here's the result. Please let me know if you find any holes with this approach.

Using <Paper> certainly does work—I'm just noting that (I think) the convention is confusing, and I didn't come across anything about that rule in the docs. The Theme Nesting docs seem to suggest that using the <ThemeProvider> will override anything that is a child of it—and I think that model makes a lot of sense, especially for typography.
For our actual app, we use <Typography> components outside of <Paper> all over the place, because <Paper> has a specific purpose in the design and I think it would be nice to be able to get the full power of a dark theme within any nested ThemeProvider sections.
Hope that makes sense, and thanks for the consideration.

@alturnwall Thanks for the reproduction, this is a different concern than the one we are discussing in this GitHub's issue. I've labeled the comments as off-topic, see: #21092 for a related concern. Basically, it's about CSS inheritance.

This nesting concept really makes working with micro frontends cumbersome since you should have different seeds or use disableGlobal. Because as you stated @oliviertassinari, it's only a workaround, the page will be torpedoed with style tags from different micro front pieces, and having too many of them in a document is just not good for page load and rendering performance.
You could checkout single-spa functionality and the way applications are put together to get a glimpse of the problem one might struggle with.

@pkhodaveissi I think that your concern is different and outside of the scope of this issue. We didn't optimize for micro-frontend, nor do we aim to, but are still interested in finding improvement opportunities.

@pkhodaveissi I think that your concern is different and outside of the scope of this issue. We didn't optimize for micro-frontend, nor do we aim to, but are still interested in finding improvement opportunities.

Sure I understand, but guess my question boils down to a solution to reduce the number of <style> tags introduced to the document while you're being forced to have multiple siblings of ThemeProvider element around(cuz of the architecture).

An update, this issue is being resolved in v5 thanks to #22342. So far, we have migrated the Slider in the lab, where this can be tested. It doesn't longer generate global class names, class names are hashed. However, the components have global class names that can be targeted.

I'm affected by this issue, we have multiple react roots on the page and because of this the classnames that are generated aren't deterministic, causing components to re-render unnecessarily.

@eps1lon I have demonstrated the simplest workaround in the codesandbox: one ThemeProvider at the top of the tree.
For the multiple React tree pattern, people have to use a different seed if they want to use different themes: https://material-ui.com/styles/api/#creategenerateclassname-options-class-name-generator.
Another workaround is to use the disableGlobal option.

Ditto on this issue - I'm working on a single-spa app as @pkhodaveissi mentioned, and having multiple React apps on the page that each use a MuiThemeProvider component with the same theme in their tree are causing conflicting stylesheets and unexpected behaviour in our components.
I confirmed that upgrading to MUI 5 would indeed resolve the issue for us, but seeing as MUI 5 isn't ready yet and we have quite a lot of migration tasks to do from 4 -> 5, I'm looking for an interim solution in the meantime.
@oliviertassinari what solution would you recommend in this case, something like the following in each app?
const generateClassName = createGenerateClassName({
  disableGlobal: true,
});
...
const App = () => {
...
return (
    <StylesProvider generateClassName={generateClassName}>
        <MuiThemeProvider theme={theme}>
        ....
        </MuiThemeProvider>
    </StylesProvider>
);
};

I couldn't quite figure out if StylesProvider is meant to replace or supplement MuiThemeProvider from the documentation.
Thanks!

@ahayes91 If you have multiple react roots, disableGlobal isn't enough. You need to have a uniq seed for each.
I'm closing as this is fixed in v5.

To be brutally honest, details about the ThemeProvider's technology should not even matter here, not even a little bit, or at least from the user's point of view.  Concerns about a feature's design are almost always of greater significance than the technology that enables said design.  And with this said, regarding the fact that multiple ThemeProviders must have a parent ThemeProvider element, there is an unasked yet obvious question that warrants asking: where is the user expected to learn about this somewhat esoteric yet potentially code-breaking ThemeProvider fact?  In the issues section of the mui Github?  No.  tldr: this should not be a design feature to begin with, but since it is and hasn't been changed yet, the docs should be extremely clear about it, which they aren't.

@Johnrobmiller It's a v4 only issue that we don't actively support anymore. This issue is gone in v5. We gave up on styling with global class names, it harms the DX of customizations but yields other interesting features that make it worth the change.

@oliviertassinari Sorry to bring this up again but we are in a middle of a migration from v4 to v5 and i would like to minimize the amount of research and surprises i'm going to face.
the seed solution for nested Themes in v4 wasn't 100% complete. The issue we had with this is that the seed wasn't chained to "nested MUI classes"
For example, given this style:
root: {
  '& .MuiBadge-badge': {
    // something
  }
}

The seed won't be chained to the MuiBadge-badge class name, therefore this selector will impact bades that are not part of this seed (scope).
We had to create a wrapper over makeStyles that goes through all keys (recursively) and chain the seed to all MUI-* classes.
The reason i'm bringing this is up, is to ask if the solution that was introduced in v5 bypass or address this issue in any way?
Otherwise, we will need to either fix this or at least have a way to workaround it in user-land (as we managed to do in v4).


Document TrustSDK specs

Should we add the ability to callback different url scheme? in callback, not always from the source app


[Bug] can't start betterdiscord installer appimage

I am getting the appimage from https://betterdiscord.app/ and clicking Download v1.5.3.
running in a terminal looks like this:
[samual@samual-pc Downloads]$ ./BetterDiscord-Linux.AppImage
Better Discord Installer 1.1.1
[6947:0518/111841.676023:FATAL:gpu_data_manager_impl_private.cc(439)] GPU process isn't usable. Goodbye.

a grey rectangle appears in the middle of the screen, then quickly disappears
I have tried deleting and redownloading over and over but nothing changes.
I'm running manjaro kde if that matters.

This isn't the right repo, and is a duplicate of https://github.com/BetterDiscord/Installer/issues/247

thank you


Introduces routing based on partition-host mapping

This PR introduces routing based on partition mapping, and thus resolves #35.
New / changed things:

PartitionRouter<K> that implements Router<K> - responsible for calling an end-point to fetch info about partition-host mapping and serving requests for a specific key (getHost(Key k)),
PartitionFinder and its implementation DefaultPartitionFinder that provides strategy for finding partitions,
For correct functioning of the router (key serialization), a specific Serde is needed. Thus we have to provide it from classes that use DefaultMirrorClient. This clashes a little bit with the changes introduces by #36, where in many cases Mirror works only with TypeResolver. Thats why there was a need to create a wrapper over TypeResolver and KeySerde to be able to pass them together: KeySerdeValResolverWrapper<K, V>

This is a draft as there are no tests yet.
The logic for dealing with a dynamic addition of replicas a la change listener will be handled in a separate issue.

I'm wondering if the router is not coupled too tight with the mirror. We already have a problem with a test in TopicDataClientTest because the router needs information from an endpoint before any mock values are provided.
   public DefaultMirrorClient(final String topicName, final MirrorHost mirrorHost,
                               final HttpClient client, final Serde<K> keySerde, TypeResolver<V> valueResolver) {
        this.client = client;
        this.parser = new MirrorValueParser<>(valueResolver, client.objectMapper());
        final StreamsStateHost streamsStateHost = StreamsStateHost.fromMirrorHost(mirrorHost);
        **this.router = new PartitionRouter<>(client, streamsStateHost, keySerde, topicName);**
        this.knownHosts = this.router.getAllHosts();
    }

For the merge conflicts, see https://bakdata.github.io/quick/0.6/developer/contributing/#dependent-pull-requests

I'm wondering if the router is not coupled too tight with the mirror. We already have a problem with a test in TopicDataClientTest because the router needs information from an endpoint before any mock values are provided.
   public DefaultMirrorClient(final String topicName, final MirrorHost mirrorHost,
                               final HttpClient client, final Serde<K> keySerde, TypeResolver<V> valueResolver) {
        this.client = client;
        this.parser = new MirrorValueParser<>(valueResolver, client.objectMapper());
        final StreamsStateHost streamsStateHost = StreamsStateHost.fromMirrorHost(mirrorHost);
        **this.router = new PartitionRouter<>(client, streamsStateHost, keySerde, topicName);**
        this.knownHosts = this.router.getAllHosts();
    }

You could use a supplier for providing the PartitionRouter


fix(FX-4319): commercial buttons section overflow

The type of this PR is: Fix

This PR solves FX-4319
Review app: commercial-buttons
Quick artwork example: Link
Problems

Commercial buttons starts overflowing the sidebar when screen width < 1075
iPhone SE (first generation) does the same when contact gallery is secondary

Solution
We discussed with Kseniya and decided that the buttons will be placed under each other by default (column view). When the screen width is 1192px or more (xl breakpoint according to the list of supported breakpoints), then place the buttons in a row
Demo
Problem №1 - when screen width < 1075

Before
After

Problem №2 - when the device has a small screen (e.g. iPhone SE first generation)

Before
After

(OH, I see it's a feature flag. What about just simply changing "Contact Gallery" to "Contact" at the smaller size with a simple Gallery

Also couldn't you just achieve this proposed fix with a responsive prop for flexDirection? (I'm just convinced we over rely Media)

Also couldn't you just achieve this proposed fix with a responsive prop for flexDirection? (I'm just convinced we over rely Media)

we can do this, but in this case it will look something like this in the code (it seemed to me that it was better to use Media and renderButtons pattern)

Responsive props are going to be much more light weight and are preferable. I only really reach for Media if it's like two totally different designs.

The use of Media for simply a copy change is kind of interesting, I guess. Though I don't see what benefit that has over the span

The use of Media for simply a copy change is kind of interesting, I guess. Though I don't see what benefit that has over the span

I mean, here we use react-i18next and it won't work so easy to use span with display={["none", "inline"]}

What about just simply changing "Contact Gallery" to "Contact" at the smaller size

We discussed this moment with Kseniya and decided to leave the current implementation


Brightness inversion

See #1.

This is a neat effect and I'm fine with including it but I'd rather have it as an option rather than replacing the existing effect. A preferences dialog should be added to the extension with a way to select between the two.

yeah, that would be a nice way to have multiple choices. that config ui could also be used for brightness settings (see #1) if that is ever implemented.
however, my gnome shell extension development skills are extremely limited. :)
that said, the effect i propose is actually the one that gnome itself also offers as a a11y feature, so one could argue (and in fact i do) that it should be the default.  colour inversion is probably also useful to some, but i gave both a fair try at night and can only see that brightness inversion is clearly the better option usability-wise.


artifact: Initialize if missing from db

Signed-off-by: Jason Lowe-Power<EMAIL_ADDRESS>

Testing is cool! The static analyzer caught one problem that was non-obvious (and another obvious problem that I was too lazy to test :/)

I wonder in what scenario we cannot retrieve the Artifact's, and whether the fake defaults will ever be written to the database.
My understanding is that, the only time that an Artifact being written to the database is when we call RegisterArtifact(). In that case, all information is not garbage, and other functions only read from the database. So, creating an Artifact using init won't result in fake defaults being written to the database.
If that is the case then I think this makes sense.
Other than that, I think we should output some warnings as well when we cannot retrieve the Artifact from the database. We don't want it to be silently failed.

Even though the underlying problem (someone deleting everything from the database) was different, I still think this change is useful. However, I can see how there may be unintended side effects.
@takekoputa, how do you suggest we have warnings?
I almost just feel like leaving this PR open and coming back to this if we find another use case (other than being ransomwared ;))


[Bug Report] - Change Common JS Module Syntax into ES Module Syntax for "prism-react-renderer" in the "docusaurus.config.js" and resolve error when running "docusaurus start" command

Describe the bug.
Currently the import for prism-react-renderer is in the Common JS Module Style in the docusaurus.config.js file:
const { themes } = require('prism-react-renderer');

We should change this into into ES Module Syntax as mentioned in its documentation:
import { themes } from 'prism-react-renderer';

Currently following lines
const lightTheme = themes.github;
const darkTheme = themes.dracula;

are causing error when we run "yarn start" error:
$ yarn start
yarn run v1.22.19
$ docusaurus start
[INFO] Starting the development server...
[ERROR] TypeError: Cannot read properties of undefined (reading 'github')
    at Object.<anonymous> (/home/devilkillerag/Documents/OpenSource/uptane-website/docusaurus.config.js:5:27)
    at Module._compile (node:internal/modules/cjs/loader:1218:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
    at Module.load (node:internal/modules/cjs/loader:1081:32)
    at Module._load (node:internal/modules/cjs/loader:922:12)
    at Module.require (node:internal/modules/cjs/loader:1105:19)
    at module.exports (/home/devilkillerag/Documents/OpenSource/uptane-website/node_modules/import-fresh/index.js:32:59)
    at loadSiteConfig (/home/devilkillerag/Documents/OpenSource/uptane-website/node_modules/@docusaurus/core/lib/server/config.js:36:55)
    at async loadContext (/home/devilkillerag/Documents/OpenSource/uptane-website/node_modules/@docusaurus/core/lib/server/index.js:31:63)
    at async load (/home/devilkillerag/Documents/OpenSource/uptane-website/node_modules/@docusaurus/core/lib/server/index.js:74:21)
[INFO] Docusaurus version: 2.4.3
Node version: v18.13.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Describe the steps to reproduce the behavior.
No response
Expected behavior.
No response
Screenshots.
No response
Operating environment
No response
What browsers are you seeing the problem on?
No response
Additional context
No response

@devilkiller-ag  I believe it's okay based on the Docusaurus documentation found here: https://docusaurus.io/docs/migration/v3#prism-react-renderer-v20. However, it could potentially conflict with your local setup. Can you please verify whether you've updated your setup according to Docusaurus v3? We've recently migrated our site to v3.

[INFO] Docusaurus version: 2.4.3
Node version: v18.13.0
error Command failed with exit code 1.

as you can see it is still version 2.4.3 for your machine, you might need to run the latest pull

closed as resolved


Snapshot testing

Why
To easily track changes in conditionally rendered layouts (with a hard to track conditions that are going on), changes that involve variables and variable scoped pieces of layouts to render, etc.
This PR is a perfect example of a need of snapshot tests. With such functionality it would be much easier to check whether i18n provided via that pr works (translations are placed where they should be placed, there are no typos in translations and so).
What
Unit testing is already on this pr. As it uses Vitest, we should also use part of vitest that includes snapshot testing.
How
Following this guide.

very old, lets close


Fix components folder include for ecs

A beautiful 1 character pull request...
The @thi.ng/ecs package doesn't seem to publish the components folder:
Module not found: Error: Can't resolve './components/object-component'

Seems a typo was introduced around 0.3.34...  removing the s!

Oh wow! I Love it - 1 char PRs FTW! :) Will get that published asap... (also ignore the "all checks failed", having some temp repo-wide typing issues w/ snowpack...) Thanks!

Grand stuff.. cheers!
Great work on thi.ng btw... I love the no framework approach. Well needed in the javascript eco system!
Much appreciated!


[udemy] Unable to download webpage: HTTP Error 403: Unauthorized

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.10.05. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.10.05

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

DESCRIPTION
youtube-dl crash randomly with a 403 Unauthorized wile crawling into udemy course.
I have been able to download successly 35/150 videos about my course, but after a "first run", youtube-dl crash around 30, and on the second run, i cant even access to the homepage.
It look like udemy has implemented a system to detect the usage of this script, maybe a pause between 2 downloads could help to bypass this system;

LOG
youtube-dl -u ***USERNAME***  -p ***PASSWORD***  -v https://www.udemy.com/***COURSE***
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-u', u'PRIVATE', u'-p', u'PRIVATE', u'-v', u'https://www.udemy.com/***COURSE***/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.10.05
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.18.16-041816-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.4-0ubuntu<IP_ADDRESS>, ffprobe 3.4.4-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[udemy:course] Downloading login popup
ERROR: Unable to download webpage: HTTP Error 403: Unauthorized (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 604, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2211, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

...
<end of log>
...

Can you give me the bug witch is duplicated? I am working on a fix.

https://github.com/rg3/youtube-dl/issues/15839

Any update on this issue.
And is this being tracked in https://github.com/rg3/youtube-dl/issues/18126


[WebExt] WebRequest, Fx 77 tasks

Trello
https://trello.com/c/khmwRVQp/209-proxy-and-webrequest-api-related-tasks-fx-77
Acceptance criteria

[ ] filterResponseData from the WebRequest API returns script bytecode cache data

[x] Remove or revisit support for data:-URLs in the webRequest API (remove/replace WebRequestContent.js)

Completed "Remove or revisit support for data:-URLs in the webRequest API (remove/replace WebRequestContent.js)" as per developer feedback by updating webRequest.RequestFilter to clarify the pattern matched protocols as follows "Only requests made using HTTP or HTTPS will trigger events, other protocols (such as data: and file:) supported by pattern matching do not trigger events."
As this is a clarification of current functionality, nothing added to the release notes.

Completed filterResponseData from the WebRequest API returns script bytecode cache data  with a change to webRequest.filterResponseData() with confirmation from Shane, via a needinfo request on the bug, that the change is good.


feat: support for multiple original sites

Fix #834

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

GitHub release
v6.50.0

Your semantic-release bot :package::rocket:


chore(main): release 0.7.0

:robot: I have created a release beep boop
0.7.0 (2024-11-22)
Features

add limit exceeded message wrapper for resource exhausted msgs (#88) (e7c8762)

Miscellaneous

add agent and runtime-version metadata (26f2772)
empty commit to bump readme generator (290ec44)
empty commit to bump readme generator (de12218)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/momentohq/client-sdk-elixir/releases/tag/v0.7.0 :sunflower:


[textures] LoadTextureCubemap unloads image (there is no indication anywhere that it should)

[x] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] My code has no errors or misuse of raylib

Issue description
Since LoadTextureFromImage doesn't unload the image after creating the texture, I expected similar behavior from LoadTextureCubemap, because it wasn't specified otherwise anywhere. But for some reason my app was crashing when I tried to unload the image after calling LoadTextureCubemap. Removing UnloadImage call helped to fix this problem.
Environment
Windows 8.1 64x
ATI Technologies Inc.
AMD Radeon HD 8330
GLSL 4.60

@rmn20 Reviewed! In case of CUBEMAP_LAYOUT_LINE_VERTICAL the input image was directly assigned to internal image, now we are doing a copy, so original image.data is not unloaded by the function.


Slow bitwise operators

Expressions like
#pythran export foo(uint8[], uint8)
def foo(a, b):
    return a & b == a

are slow in pythran if a.dtype is np.uint8. It's no longer the case for uint64, so I guess we should introduce a representation change when all operations are bitwise.

arggg, I've do many bitwise operations on uint8.
Say me if there is something I can do to help.

On Sat, Jan 07, 2017 at 07:48:24AM -0800, Luis wrote:

Did you have a chance to improve speed for np.uint8 & np.uint8 ?

No, but thanks to your reminder, that will be my next task! Keep in
touch :-)

@LuisBL just to be sure : any reason why not nusing a uint64 ?

yes it's because I use numpy.unpackbits to store 8 0/1 in only one byte.
Cf. numpy.bitwise_and and numpy.unpackbits

I've got the fix but SIMD instructions are not used for comparison operators, I'll push it soon.


[NavigationMenu] add delay duration skip delay duration props

Closes  #1457

Not sure what's going on here, but it seems the CI checks haven't run and I don't have the usual button to allow them to run.
You might need to force push or add a new commit or something to trigger it again.
✌

Here's to hoping you can run the workflows now. Let me know if something is wrong.
I just want to take a moment to say that I love this project and would definitely try to contribute more if my college workload wasn't killer.
:heart:

I've addressed this feedback plus other issues I noticed via https://github.com/radix-ui/primitives/pull/1716 so will close this in favour.
Thanks again for your efforts and contributions to Radix @innub , much appreciated.


[HOPS-1156] Fix datanode not stale variable for Multi NN

[HOPS-1112] Fix firstBlockReport check in BlockManager for Multi NN
Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-1156
https://hopshadoop.atlassian.net/browse/HOPS-1112

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Can one of the admins verify this patch?

Jenkins test this please.

This seems to be breaking TestBlockManager
And it needs conflicts resolving.

https://bbc1.sics.se/jenkins/job/dist-test/33/
Flaky test  org.apache.hadoop.hdfs.server.datanode.TestNNHandlesBlockReportPerStorage.blockReport_09

Jenkins test this please.

Jenkins test this please.


Add an "Are you sure?" warning when flagging a user as spam

From https://forum.inaturalist.org/t/create-an-are-you-sure-popup-when-suspending-a-user-for-spam/45978
For curators, it's easy to accidentally select the "flag as spammer" option in the Admin Tools on a user's profile, so some user are getting flagged by mistake. Can we add a warning pop-up that says "Are you sure you want to flag this user as a spammer?" with buttons that say "Yes" and "No"?
I think the "Flag as spammer" option should be above the line as well, in the same section as "Suspend user".

Hello,
I made the changes and created a PR for this issue.
It will need translation for different languages.
Note: I didn't change the position of the option since it was your personal suggestion and not an issue talk on the initial topic. I can make the change if needed.


Pod Topology Spread Constraints - Mermaid Syntax error.

There seems to be a mermaid syntax error in the diagram on the page.
https://kubernetes.io/docs/concepts/scheduling-eviction/topology-spread-constraints/#example-one-topologyspreadconstraint

Seems this is a duplicat of  #37357
sorry for the noise.


IndexedDB?

I just found out this amazing project! Congratulations!
In my dreams I would like to replace @tanstack/query and urql entirely with nanostores/query. It would be a crazy joy! :smile:
The only thing that blocks me now is that I need to persist all the query data in the browser (like I do with those libraries).
I was wondering if you could create an adapter for IndexedDB other than that for localStorage which besides its already well known limitations also has the problem that it is sync and blocks the main thread.
Is there already an adapter for IndexedDB?
What do you think?
Once again: thank you for these magical projects!

Theory
localStorage and IndexedDB was created for very different use cases. For most cases it is a wrong idea to change adapter and do not change the logic.
@nanostores/persistent store was created to store small amount of data like user’s settings. It will be reasonless to put this small amount to IndexedDB.
IndexedDB should be used to work with a big amount of data (> 5 MB). You should use it only if you really will benefit from indexes and async API. The @nanostores/persistent is not benefit from index and async API because of internal design.
What do I suggest

The best option will be to write another store with design based on async API and indexes.
If you really know what are you doing you can write custom persistent engine with IndexedDB API, but I think it will be good option only to 5-10 projects in the world. We do not provide IndexedDB engine for this project, because it will force developers to make a mistakes (and these 5-10 projects can write engine in-house).

If you explain to me what data you plan to store in IndexedDB and how do you plan to use it, I can suggest the store design.

I recommend using Cache API for that https://developer.mozilla.org/en-US/docs/Web/API/Cache
Using IndexedDB will require a lot of extra work to keeping DB schema up to date.

Oh wow. This suprised me.
I've never seen Cache API used for anything other than file caching. But now that you mention it, it makes a lot of sense.
What do you think about adding this feature to https://github.com/nanostores/query?


760 Use TDP 1.0 releases files

Which issue(s) this PR fixes

Fixes #760
Additional comments
Still as draft due to spark not being release yet
Agreements

[x] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004.
[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

hbase_hbck2_release have the TDP SNAPSHOT version.


Set timeout to 5m and fixed running on Windows

Coverage decreased (-1.3%) to 94.712% when pulling 2c7bdc88dfbf483e0efe998d78e980dac0f80cca on kerray:master into 8a517053fcf882e728b9208f71e4d95081f98324 on handcraftedbits:master.


bug on change mode

Good evening,
I think I encountered a bug in nCAT
FLEX 6400M firmware version 3.7.4
Hamlib: last available today
command:
rigctl -m2 M USB 3000
output:
set_mode: error = read_string_generic(): RX 2 characters, direct=1
vfo_fixup:(from rig_set_mode:2496) vfo=VFOA, vfo_curr=VFOA, split=0
 rig.c(2520) trace
netrigctl_set_mode called, vfo=VFOA
netrigctl_vfostr: called vfo=VFOA
netrigctl_vfostr: vfo_opt=0
netrigctl_transaction: called len=11
network_flush called
write_block(): TX 11 bytes, method=2
read_string_generic called, rxmax=1024 direct=1, expected_len=1
read_string_generic(): RX 7 characters, direct=1
rig_set_mode: targetable retcode after set_mode(USB)=-8
Protocol error

 2:rig_set_mode: elapsed=79ms
 2:rig.c(2579):rig_set_mode returning(-8) Protocol error

rigctl_parse.c(2257):rigctl_set_mode returning2(-8) Protocol error

Protocol error

Actually this is a Flex bug (ticket SMART-10100). I think it will be fixed in 3.8.
But it's inconvenient so I will add a workaround in nCAT if I have the time. Thank you for the report.

Thank you for your quick support and great work!

By the way, the bug is only that the command reports failure -- the mode and bandwidth do actually change.


created new file  - 0000-01-02-kesavaraj-25.md

0000-01-02-kesavaraj-25.md file added for my slide show

successfully added new file. for My slide


Alternative Embedding Method

This is just some alternative method for embedding the videos.
Using https://embed.works/ before a BiliBili link allows the video to be embedded. (You can try this yourself, if it doesn't work wait a bit then remove or add a "/" at the end of the link)

It works by grabbing a direct link to the video to embed, if that doesn't work it downloads the video and then hosts it on a server.
Here's a link to the repository. https://github.com/jemand771/embed.works

Plausible

It would be funny if I rely on cobalt.tools for the video URL

Hmm, so temporary hosting seems to be the only solution then. Right?

Temporary hosting is probably the best but is out of scope for this project


Add a new type Tuple<3, T> = [T, T, T]

Suggestion

🔍 Search Terms

tupel
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

Add a Tuple type where we can dynamically create tuples:
type Tuple<L, T> = [T, T, T]; // L = Length of Tuple

📃 Motivating Example

This type would be in the same area as the Awaited type because it helps preventing complex type definitions.
Tupel-types currently have to be created manually each time. Especially for longer fix length arrays, this gets complicated really fast:
type FixLength6Array = [number, number, number, number, number, number] ;
const fixLength6Array: FixLength6Array = [1,2,3,4,5,6];

By using a Tuple type, we could fix this:
type FixLength6Array = Tuple<6, number>;

Also if we wanted to change the type of this tuple, this is done easily and has to be done in one place:
type FixLength6Array = Tuple<6, number | string>;

💻 Use Cases

What do I want to use this for?
One example would be an rgb-type which allows numbers and strings. It currently looks like this:
type NumberString = string | number;
type RgbTuple = [NumberString, NumberString, NumberString];

We could make this much simpler like this:
type RgbTuple = Tuple<3, NumberString>;

https://github.com/microsoft/TypeScript/issues/51841#issuecomment-1344953135

As for the utility type itself, apparently you can already implement it today: https://github.com/microsoft/TypeScript/issues/47874#issuecomment-1039157322

As far as I know, there's no way you can write Tuple<L, T> in your own code right now.

type TupleMap<T> = {
  0: [],
  1: [T],
  2: [T, T],
  3: [T, T, T]
  // go as far as you need to
}
type TupleN<N extends keyof TupleMap<unknown>, T> = TupleMap<T>[N];

type RGB = TupleN<3, number>;

I really recommend the lookup type method over tail-recursion wizardry in any case where you have a reasonable finite bound; it's generally more performant and easier to read.


ERROR on Install Python Extension

Hi,
When trying to run the release pipeline the following error occurs during Install Python Extension phase:
"##[error]Error: Failed to install site extension 'python 3.5.3 x86'. Error: Not Found (CODE: 404)"
Does anyone know what that means and how to resolve it?
To reproduce follow:
https://www.azuredevopslabs.com/labs/vstsextend/python/

I found the solution. I had to add the Python 3.5.3 x86 extension manually from within the azure portal. I didn't know about that and thought that the tutorial was just clone and go.
On the other hand using Python 3.6.4 x86 extension on my own project I did not have to configure it manually. It was added automatically.

Please how did you install it on the azure portal?

You can install extensions in the Azure portal by going to your AppService resource.
Then scroll down to the "Development Tools" section in the left-hand menu. Under the "Development Tools" section is the "Extensions" page.

On the "Extensions" page, click "+ Add" on top to search and add the extension you would like to add.


Fix/safari audio context

Redefined AudioContext using webkit so safari can use it.

Can Someone merge this and publish it to npm?

@LovaszNorbert I'll publish the new version to NPM soon.

Can someone please merge this and publish it to npm?
Thanks!

@buzinas can you publish a new version with this fix to npm?
Thanks!


What is the name of your plugin?
performance-analyzer-rca
What is the link to your GitHub repo?
https://github.com/opensearch-project/performance-analyzer-rca
Targeted release version
2.9.0
What type of the plugin are we on-boarding?
Observability
Did you read the on-boarding document
Yes
Does this plugin has all the necessary automated tests?
Yes
Have you completed the required reviews including Security reviews, UX reviews?
Yes
Have you on-boarded automated security scanning for the GitHub repo associated with this plugin?
Yes
Additional context

performance-analyzer-rca[RCA] is not a plugin but follows the same versioning as OpenSearch
RCA is an independent agent, running in a side-car JVM and doesn't interact directly with OpenSearch.

Currently, the plugin is packages with PA in the distribution build will continue to do so. We need assistance for publishing the RCA JARs to Maven Central and Snapshot repo, refer: https://github.com/opensearch-project/performance-analyzer-rca/issues/312

[Untriage]
@khushbr moving forward the performance-analyzer-rca[RCA] will be released as an isolated component?
or with PA part of the build manifest?
@gaiksaya @zelinh

Hey @prudhvigodithi , RCA will continue to be built with PA as part of the build manifest.

RCA is an independent agent, running in a side-car JVM and doesn't interact directly with OpenSearch.
In that case should RCA follow its own versioning?

I am going to close this issue for now.
Going forward, we want to converge the common functionality of RCA and Telemetry Collector  into a single Agent.  Ideally, this Agent should be version agnostic but given the history of RCA evolution with OpenSearch version and releases, it would be better to rehome the common components into Collector and not RCA.


Release .unitypackage file

hey, I just love this package. just what I wanted!
but I don't want to clone the repository everytime (install from git URL in the package manager). it'd be really handy if you have a unitypackage file in the releases of this repo so that we can download it and keep it for when we need to create a new project.
thanks.

Hey, sure, I mean you could already take the package files from Library/PackageCache and put it in a project to create a unitypackage but I've done it for you, you can find the unitypackage file in the releases. Cheers.
https://github.com/ssoher/com.sarpersoher.ide.sublimetext/releases/tag/v1.0

lovely! thank you man.


No headers when the request is invalid, always

Right now, if a request is made and the origin isn't in the whitelist of origins, As per the relevant section of the W3C CORS spec, if the request has something wrong (whether it's a non valid origin or request method or request headers and so on), no Access-Control-* headers should be returned. The browser will automatically take care of interpreting this as an error.
A bunch of advantages I thought of:

smaller response (I'm talking about the response size)
less information given on what the accepted origins are
simpler logic and options since we can nuke the :allow_origin option altogether

Hmm i spent a few hours today trying to make angular work with your project. It is hard for the user to figure out what is going on, or why some request doesn't return any Access-Control-* headers. Maybe it would be cool to be able to toggle some debug logging, so that it's easier to find out why no Access-Control-* headers are returned. This could be something like why some request is deemed invalid.
Appart from this i really like the work you have put into this project!

Hey @blacksails, thanks so much for the bug report (and the nice words :heart:)!
So, as I said in the issue, the reason behind not returning any Access-Control-* headers is compliance with the W3C CORS spec I linked above. That said, you're absolutely right here and this behaviour is probably very obscure for a user of this library; logging is a great idea, I'll add the functionality as soon as I can!
Sorry for the few hours you lost because of this :(
As an aside, may I ask you how you're using this library? Are you using Corsica as a plug (like plug Corsica, ... or as a router generator (use Corsica)? It would be nice to get some insight on how this library is used by actual users :). Thanks!

I used it as router generator. It seems cool to have control over how CORS should be handled in different parts of an API. One could imagine that some parts of an API would be used internally and other parts would maybe be public, with your router generator it is very easy to do something like that. :+1:

Hi @blacksails, I pushed a couple of new commits which include the logging functionality we discussed here. Would you be willing to try out the master branch and tell me what you think? If you think this solves the problem I can push a new version to hex.pm.
To use the master just list the :corsica dependency as
[...,
 {:corsica, github: "whatyouhide/corsica"}]

and to use logging add
config :corsica, log_level: :info

to your config/config.exs. Thanks for all the help! :)


fix issue #99 when jira does not support subtasks

Thanks for the PR @SteveDonie . Please feel free to raise more Pull requests or issues :).

fixes #99


Issues With CLI Examples

[Enter feedback here]
Are the CLI commands for bash?  If so the following is incorrect:
Incorrect path
Should the --template-file "c:\Templates\linkedTS\azuredeploy.json be --template-file /c/templates/linkedTS/azuredeploy.json?
Reference is:
az ts create \
  --name webSpec \
  --version "<IP_ADDRESS>" \
  --resource-group templateSpecRG \
  --location "westus2" \
  **--template-file "c:\Templates\linkedTS\azuredeploy.json"**

Syntax Error
Should az template-specs show be az ts show?

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

ID: 78acfe06-c626-fd75-d3a4-13f7149ca40a
Version Independent ID: 4d04da8a-df33-50ec-080b-8f983e38ecff
Content: Create a template spec with linked templates - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-specs-create-linked.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@NickBishop2112 Thanks for the question. We're investigating this and will get back to you shortly.

@NickBishop2112 Thanks for the question. We're investigating this and will get back to you shortly.

@NickBishop2112 Thanks for bringing this to our attention. I have submitted a PR to fix the issue that's in review. The changes should reflect post the merge.

@NickBishop2112 Thanks for bringing this to our attention. I have submitted a PR to fix the issue that's in review. The changes should reflect post the merge.


Simply tutor about build julia script into binary?

Does there simply tutor for building julia into binary? Something like:

f(x) = x^2+x
f(4)

and then compile it as a single executable file?
More complicated, compile as binary when use third party package?

Yes, check PackageCompiler.jl it provides a AOT compiler for Julia scripts. But notice we currently can not build a standalone binary (which means something completely without runtime) although in principle we can but it lacks of people working on it.

https://github.com/JuliaLang/PackageCompiler.jl
I'll add some more demo project to help people understand how to write a practical project with Julia later. But I don't have plans on building binaries, since just using it is simple, but explain how these things works would be more advanced.

And Julia is a JIT language, which means f(4) in your example compiles to binary instructions in the memory rather than a binary file. If you want to make real binary files, it's another story. The PackageCompiler README explains how already I think.

@Roger-luo  compiling Julia scripts help people distribute or deploy their projects. this is really dynamic languages lack of, especially python. (you may got a lots of dedencies trouble when deploying a python project on your server, but with a single executable file life goes much more easy.)

Just wonder how Julians think and do about this.

Yes, I understand a standalone binary is necessary for production. Julia can be compiled to one binary, if you don't care about the quality for now. It's just there are things with higher priority tasks (like improve JIT latency, compiler optimization) than making a nice standalone binary. Since most people use Julia on clusters instead of small devices, thus a large working binary should be sufficient for now.

And yes, since many dynamic language can not specified the generic code efficiently, the compilation sometimes is pretty bad. This not a problem for Julia, Julia can make shared libraries. The current problem is mainly about making small, standalone binaries, you won't want to distribute hello world with a blas compiled in it. But if you just want binary and do not care about the size, Julia can even be compiled to C callable shared libraries.
And I would suggest to join open source contribution, if there's some features you want and doable since the community is young and always lack of people working on things.

@Roger-luo thanks for your clarification. I will keep dig into Julia

这个issue应该已经解决了。


RegisterNativeAsync should be RegisterNative

Hi Team,
The sample code in RegisteredForRemoteNotifications () function,
Hub.RegisterNativeAsync(deviceToken, tags, (errorCallback) => {
            if (errorCallback != null)
                System.Diagnostics.Debug.WriteLine("RegisterNativeAsync error: " + errorCallback.ToString());
        });

Hub.RegisterNativeAsync should be Hub.RegisterNative

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

ID: 45afd17d-e909-ccd9-01c3-f746bdf406a2
Version Independent ID: d2c8cc69-5a37-4c73-d48b-9c058cf4595e
Content: Send push notifications to Xamarin using Azure Notification Hubs
Content Source: articles/notification-hubs/xamarin-notification-hubs-ios-push-notification-apns-get-started.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@jessejiang0214 Thanks for the feedback! We are currently investigating and will update you shortly.

@jessejiang0214 Could you please share more details on this to help you better on this. If you are facing any error?

Hi @SnehaAgrawal-MSFT,
The API of that nuget has been changed

There's no callback parameter for RegisterNativeAsync API

@jessejiang0214 Thanks for the valuable feedback! Its now updated.


chore(apps/prod/tekton/configs/triggers): limit harbor push event resource url

limit it to special server
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


Bug/UI/tra 4169 apply qury by enter

@mertyildiran As far as I know the code wasn't published yet

html element
On Mon, Jan 17, 2022 at 6:01 PM lirazyehezkel @.***>
wrote:

@.**** commented on this pull request.
In ui/src/hooks/useKeyPress.ts
https://github.com/up9inc/mizu/pull/660#discussion_r786135602:

@@ -0,0 +1,35 @@
+import { useCallback, useEffect, useLayoutEffect, useRef } from 'react';

+const useKeyPress = (eventConfigs, callback, node = null) => {
what is this node param?
—
Reply to this email directly, view it on GitHub
https://github.com/up9inc/mizu/pull/660#pullrequestreview-854586093, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXITUH4XOWTHFPSXWRB6KETUWQ4OPANCNFSM5MEZQEWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Sure i will find them and refactor to use it?
On Mon, Jan 17, 2022 at 6:03 PM lirazyehezkel @.***>
wrote:

@.**** commented on this pull request.
looks like a nice hook.
we have some more places in the code with enter key press, I would
refactor those parts as well to use this hook. wdyt?
—
Reply to this email directly, view it on GitHub
https://github.com/up9inc/mizu/pull/660#pullrequestreview-854587431, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXITUHY4QHWRU7BHOBE4YHDUWQ4TJANCNFSM5MEZQEWQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

go for it


Allow binary messages in ockam message send

Current behavior
Only text messages can be sent via ockam message send command.
Occasionally, it's useful to be able to send binary messages, for example testing a decryptor.
Desired behavior
Allow to send binary messages specifying the --hex flag.
When this flag is used, both message and response are hex-encoded.
ockam message send --from node1 --to /service/echo --hex 00aabbff
00aabbff

I'm going to give this a try.


Ability to Group By Defined Ranges

This feature is fantastic but doesn't quite offer the flexibility that I've seen elsewhere.  For example, in your example above, you allow to Bin by a range such as months.  Where there isn't a logical data set like dates, such as a simple Whole Number field, it would be great if you could group by set ranges.  I suppose it would be similar to the rules in conditional formatting.
In my instance, it would allow us to create a hierarchy of projects codes. Eg 10000-15000 = Portfolio A, 15001 - 20000 = Portfolio B.
Often people are using DAX to calculate bands for ranking their data. This suggestion could also be used to make such groupings easier to Manage. For example, applying the banding to a % Profit field you could group 0 - 30% = Low Profit Customers, 31%- 60% = Mid Profit Customers etc.

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

ID: 7a1e0b0a-ff95-b2f7-44db-5c07ca876ab4
Version Independent ID: 1c975595-bf44-5f66-3e08-02c3e8e63797
Content: Use grouping and binning in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-grouping-and-binning.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

@kcinmelbourne Thanks for your comment. We’ve forwarded your feedback to the writer, who should respond to this thread. Thanks!

Thanks for posting – note that we can only answer feedback related to the documentation, and can’t provide direct support or respond feature requests.
Your best venue for getting an answer or feedback on your question is from the community (https://community.powerbi.com) or from support at https://support.powerbi.com. You can also provide feedback using the Give Product Feedback button at the bottom of the article page.


Display the field(s) referenced by the metadata in the sample's JSON

Splitting out from open-contracting/pelican#1. @pindec wrote:
+1 for display the field(s) referenced by the metadata in the sample's JSON.
I interpret this to mean, where "Metadata" is given for failed paths, those paths should be clickable to auto-expand the relevant JSON and jump to it in "Data Preview".
It's frustrating to know that particular fields have been flagged as failing, but having to click through large JSON blocks to get to them.
The issue is also present for "Sample of successes", except no path is given in the metadata.
Originally posted by @pindec in https://github.com/open-contracting/data-quality-tool/issues/1#issuecomment-566092961

#49 and #50 are alternative solutions to this, and require even less effort (no need to click on a sample and then scroll through the JSON).
That said, I'll keep this open, as it might be as simple as setting selectedValue.sync on the vue-json-pretty component.


Splitting HLS stream into multiple MediaItems each with start and end positions

I want to split an HLS audio stream into chapters and add them as a list of MediaItems to the Media3 player using MediaSessionService. I have this following code in MediaController.
From the composable screen:
fun getMediaItems(uri: String, chapters: List<Chapter>, token: String): List<MediaItem> {
    val requestMetaData = MediaItem.RequestMetadata.Builder()
        .setExtras(bundleOf( "TOKEN" to token,)) .build()

    var offsetInMs = 0L

    return chapters.map { chapter ->
        val mediaMetaData = MediaMetadata.Builder()
            .setTitle(chapter.title)
            .build()
        val clippingConfiguration = ClippingConfiguration.Builder()
            .setStartPositionMs(offsetInMs)
            .setEndPositionMs(offsetInMs + chapter.duration)
            .setRelativeToDefaultPosition(true)
            .build()
        offsetInMs += chapter.runtime

        MediaItem.Builder()
            .setMediaId(chapter.id)
            .setUri(uri)
            .setMimeType(MimeTypes.APPLICATION_M3U8)
            .setClippingConfiguration(clippingConfiguration)
            .setRequestMetadata(requestMetaData)
            .setMediaMetadata(mediaMetaData)
            .build()
    }
}

From ViewModel:
var mediaControllerFuture = MutableStateFlow<ListenableFuture<MediaController>?>(null)

fun prepare(context: Context, mediaItems: List<MediaItem>) {
    val sessionToken = SessionToken(
        context,
        ComponentName(context, PlaybackService::class.java),
    )
    MediaController.Builder(context, sessionToken)
        .buildAsync()
        .also {
            val listener = Runnable {
                mediaControllerFuture.value?.get()?.let { controller ->
                    controller.addListener(listener)
                    controller.addMediaItems(mediaItems)
                    controller.prepare()
                    controller.play()
                }
            }
            mediaControllerFuture.value = it
            mediaControllerFuture.value?.addListener(listener, MoreExecutors.directExecutor())
        }
}

Service:
class PlaybackService : MediaSessionService() {
    private var mediaSession: MediaSession? = null

    override fun onCreate() {
        super.onCreate()
        val customFactory = object : MediaSource.Factory {
            override fun createMediaSource(mediaItem: MediaItem): MediaSource {
                val token = mediaItem.requestMetadata.extras?.getString("TOKEN") ?: ""
                val dataSourceFactory = DataSource.Factory {
                    val dataSource = DefaultHttpDataSource.Factory().createDataSource()
                    dataSource.setRequestProperty("Authorization", token)
                    dataSource
                }
                return HlsMediaSource.Factory(dataSourceFactory)
                    .createMediaSource(mediaItem)
            }
        }
        val player = ExoPlayer.Builder(this)
            .setMediaSourceFactory(customFactory)
            .build()
            .apply {
                playWhenReady = true
            }

        mediaSession = MediaSession.Builder(this, player)
            .build()
    }

    override fun onTaskRemoved(rootIntent: Intent?) { ... }

    override fun onDestroy() { ... }

    override fun onGetSession(controllerInfo: MediaSession.ControllerInfo): MediaSession? {
        return mediaSession
    }
}

However, the player is still playing the entire stream (ignoring start/end positions) and doesn't look like it is moving to the next MediaItem. Or, if it is indeed advancing to subsequent MediaItems how can I find when the player actually advances (so that I can update the player UI to indicate the current chapter being played)? Are there any listener events I can track?
Update: Notification UI showing the entire audio length and not segmented as I would have expected.
I tried this callback, but never saw it being invoked except once at the very beginning.
override fun onMediaItemTransition(mediaItem: MediaItem?, reason: Int)

This response helped https://github.com/androidx/media/issues/185#issuecomment-2015557497!

Based on the example code here, it doesn't look like you need to manually be creating an HlsMediaSource(.Factory) and could instead just rely on DefaultMediaSourceFactory (which would then do the clipping for you).
Concretely, this would just be in your code above, instead of:

return HlsMediaSource.Factory(dataSourceFactory)
  .createMediaSource(mediaItem)

You should instead be able to do:
return DefaultMediaSourceFactory(dataSourceFactory)
  .createMediaSource(mediaItem)

Based on the example code here, it doesn't look like you need to manually be creating an HlsMediaSource(.Factory) and could instead just rely on DefaultMediaSourceFactory (which would then do the clipping for you).
Concretely, this would just be in your code above, instead of:

return HlsMediaSource.Factory(dataSourceFactory)
  .createMediaSource(mediaItem)

You should instead be able to do:
return DefaultMediaSourceFactory(dataSourceFactory)
  .createMediaSource(mediaItem)

Thanks. Curious, if I use DefaultMediaSourceFactory instead would HLS streams still work?

If you have the HlsMediaSource.Factory on your runtime classpath and the URL ends in .m3u8 or you set the MIME type on the MediaItem manually, then yes - see class level javadoc here: https://developer.android.com/reference/androidx/media3/exoplayer/source/DefaultMediaSourceFactory

Thanks @icbaker that helps. Will be easy to support other formats rather than hard coding HLS here.

This case can be closed, got resolution, thanks!


Introducing ColumnCollection's to define columns to keep.

This PR introduces ColumnCollection's which are flags to denote certain types of - as the names suggests - collections of columns.
Background: we got a couple questions recently on why certain columns were not existing in processed arrays at e.g. ProduceColumns or CreateHistograms level. The reason always was that the columns to keep were not properly defined in the cfg.x.keep_columns map. However, we already discussed that this is not very transparent anyway.
I think it's correct that these kind of "columns to keep" need to be defined somewhere, but in most use cases it would be sufficient to have a placeholder in the keep_columns map that means something like "just keep all columns produced by the calibrators / selector / producers / ...". ColumnCollections are exactly that.
So far, there are just a handful of flags (e.g. ColumnCollection.ALL_FROM_SELECTOR) but this could even be extended in the future.
For the particular use case of determining the columns to keep at various places where some columns are dropped (ReduceEvents, UniteColumns, MergeSelectionMasks), I added find_keep_columns to our mixin classes which makes a lot of sense inheritance-wise (e.g. all tasks having the SelectorMixin should be able to access columns produced by a selector and thus, should be able to resolve the ColumnCollection.ALL_FROM_SELECTOR flag).

@mafrahm I want to add another commit soon to correctly combine sets with those found in a super function.


TypeError: preproc() takes from 2 to 3 positional arguments but 4 were given

之前跑通过，最近跑demo都会报这个错误
Traceback (most recent call last):
File "tools/demo.py", line 296, in
main(exp, args)
File "tools/demo.py", line 287, in main
image_demo(predictor, vis_folder, args.path, current_time, args.save_result)
File "tools/demo.py", line 180, in image_demo
outputs, img_info = predictor.inference(image_name)
File "tools/demo.py", line 136, in inference
img, ratio = preproc(img, self.test_size, self.rgb_means, self.std)
TypeError: preproc() takes from 2 to 3 positional arguments but 4 were given

did you find solution I got the same problem


User management method does not work.

User management method does not work.
These are the instructions below.
https://docs.mongodb.com/manual/reference/method/js-user-management/
thanks.
Name | Description
db.auth() | Authenticates a user to a database.
db.changeUserPassword() | Changes an existing user’s password.
db.createUser() | Creates a new user.
db.dropUser() | Removes a single user.
db.dropAllUsers() | Deletes all users associated with a database.
db.getUser() | Returns information about the specified user.
db.getUsers() | Returns information about all users associated with a database.
db.grantRolesToUser() | Grants a role and its privileges to a user.
db.removeUser() | Deprecated. Removes a user from a database.
db.revokeRolesFromUser() | Removes a role from a user.
db.updateUser() | Updates user data.
passwordPrompt() | Prompts for the password as an alternative to specifying passwords directly in various mongo shell user authentication/management methods.

Hello @hangum,
This is a know issue
See https://youtrack.jetbrains.com/issue/DBE-9645
But I started to implement mongodb-js/mongosh for Java and soon this problem will be resolved

Current state:
db.auth() is not implemented
other methods seem to be implemented but I didn't check all of them

@kornilova203  Thanks a lot.


Mutual kNN graph

It's probably not a concern of this repository, but I was wondering if there is a convenient way to get mutual kNN graph through pynndescent.

I believe the following would work:
from pynndescent import NNDescent
import scipy.sparse
import networkx

index = NNDescent(data)
adj_mat = scipy.sparse.lil_matrix((data.shape[0], data.shape[0]), dtype=np.int16)
adj_mat.rows = index._neighbor_graph[0]
adj_mat.data = np.ones_like(index._neighbor_graph[0], dtype=np.int16)
adj_mat = adj_mat.multiply(adj_mat.transpose())
mutual_knn_graph = networkx.from_scipy_sparse_matrix(adj_mat)

With potentially a little finessing of the sparse matrix (some conversions between sparse matrix formats may be required for example).

Thanks!
In pynndescent, after constructing this lil_matrix here:
https://github.com/lmcinnes/pynndescent/blob/master/pynndescent/pynndescent_.py#L1044
I can also do:
result.multiply(result.T).sqrt()
Right?

Yes, actually that should work too. Much easier. Thanks.

This is old now, and I think the link above is not a permalink.
Anyways for anyone else doing the same thing you can build the adjacency graph like this:
import sparse
indices = list()
indptr = list()
count = 0
indptr.append(count)
for row in index._neighbor_graph[0]:

    for item in row:
        indices.append(item)
        count+=1
    indptr.append(count)

adj = sparse.csr_matrix( ( np.ones(len(indices)), indices, indptr))

#assert all the indices in the adjacency are really the ones given in the neighbor graph:
for _ in range(50):
    idx = np.random.choice(adj.shape[0])
    assert np.all(np.isclose(index._neighbor_graph[0][idx], adj[idx].indices))


How to unit test expected failures?

When I'm writing code, I often want to be able to test that it returns a the right error in a particular situation. With hand-implemented errors, I would use a match statement for this, since errors can't be expected to implement PartialEq. Normally I would make this more informative and concise by using assert_matches from the matches crate. For example:
use std::error::Error;
use std::fmt;
use std::fmt::{Display, Formatter};

#[derive(Debug)]
pub enum MyError {
    BadThingHappened {
        badness_level: usize,
    },
}

impl Display for MyError {
    fn fmt(&self, formatter: &mut Formatter) -> fmt::Result {
        unimplemented!()  // I have omitted this
    }
}

impl Error for MyError {
}

fn my_fn() -> Result<(), MyError> {
    Err(MyError::BadThingHappened {
        badness_level: 9
    })
}

fn main() {
    match my_fn() {
        Err(MyError::BadThingHappened { badness_level: 9 }) => (),
        _ => panic!("the wrong error was returned")
    }
}

How could I test this behavior with a failure error? For example:
#[macro_use] extern crate failure;

use failure::Error;

#[derive(Debug, Fail)]
pub enum MyError {
    #[fail(display = "a bad thing of level {} happened", badness_level)]
    BadThingHappened {
        badness_level: usize,
    },
}

fn my_fn() -> Result<(), Error> {
    Err(Error::from(MyError::BadThingHappened {
        badness_level: 9
    }))
}

fn main() {
    // What would I put here to test that my_fn has returned the correct error?
}

Would using downcast_ref be a good way to do this? It appears to work for the use case above.
fn main() {
    match my_fn()
        .expect_err("my_fn should return an error")
        .downcast_ref()
        .expect("the wrong type of error was returned")
    {
        MyError::BadThingHappened { badness_level: 9 } => (),
        _ => panic!("the wrong error was returned"),
    }
}


RTL gen pass fail to catch all System Verilog keyword

VerilogTranslationPass fail with the following example.
def mk_aType(a: int = 4, b: int = 6):

    return mk_bitstruct(
        "aType",
        {
            "a": mk_bits(a),
            "b": mk_bits(b),
            "const": mk_bits(32),
        },
        namespace={},
    )

class test(Component):
    def construct(s, t):
        s.in_ = InPort(t)

if __name__ == "__main__":
    m = test(mk_aType())
    m.elaborate()
    m.set_metadata(VerilogTranslationPass.enable, True)
    m.apply(VerilogTranslationPass())

When running verilator --lint-only ./test__t_aType__a_4__b_6__const_32__pickled.v will give the following error.
%Error: test__t_aType__a_4__b_6__const_32__pickled.v:10:16: syntax error, unexpected const
   10 |   logic [31:0] const;
      |                ^~~~~
%Error: Exiting due to 1 error(s)

Fixing is easy as I can just not name things with the System Verilog key. But I will still report it as you have checks in place to prevent this, and the check fails to detect it.

Hmmm ... "const" is on the keyword list here:

https://github.com/pymtl/pymtl3/blob/master/pymtl3/passes/backends/verilog/util/utility.py#L112

So I think we would catch if you name a signal const but maybe we don't check field names?

This is likely the case. My guess is the bitstruct field is not checked.

Would make a great fix for a pull request if you are interested!

It is intended that we do not check for Verilog keywords when making a new bitstruct class, since it still works for Python simulation. We only check if the bitstruct fields are Python keywords or valid Python identifiers.
https://github.com/pymtl/pymtl3/blob/4b3bc183b14ffacd069f48dc5997b00045aea53f/pymtl3/datatypes/bitstructs.py#L744-L747
I think it would be more appropriate to check if the field names are Verilog keywords in the translation pass.

Right ... I think our point was we currently have a list of verilog keywords that get checked at translation although currently it looks like we don't check if bitstruct fields are verilog keywords at translation ...


Adding mesh object to empty viewer

🐛 Bug
Adding a 2D mesh object (like are found in shapes) to an empty viewer triggers an error on the camera.set_range() call due to not all the bounds being set in 3D. This seems to be more of a vispy error to me, but worth flagging here. (Note here the vectors layer has been modified to use a mesh object and not a lines object).
  File "examples/add_vectors_coords.py", line 35, in <module>
    layer = viewer.add_vectors(pos)
  File "<string>", line 3, in add_vectors
  File "/Users/nicholassofroniew/Github/napari/napari/components/_viewer/model.py", line 177, in add_layer
    self.reset_view()
  File "/Users/nicholassofroniew/Github/napari/napari/components/_viewer/model.py", line 140, in reset_view
    self.camera.set_range()
  File "/Users/nicholassofroniew/anaconda3/lib/python3.6/site-packages/vispy/scene/cameras/base_camera.py", line 273, in set_range
    bounds = self._viewbox.get_scene_bounds()
  File "/Users/nicholassofroniew/anaconda3/lib/python3.6/site-packages/vispy/scene/widgets/viewbox.py", line 151, in get_scene_bounds
    b = ob.bounds(axis)
  File "/Users/nicholassofroniew/anaconda3/lib/python3.6/site-packages/vispy/visuals/visual.py", line 251, in bounds
    self._vshare.bounds[axis] = self._compute_bounds(axis, view)
  File "/Users/nicholassofroniew/anaconda3/lib/python3.6/site-packages/vispy/visuals/mesh.py", line 535, in _compute_bounds
    return self._bounds[axis]
IndexError: list index out of range

empty layers can now be added for the viewer for all layers, which is captured in the tests introduced by #401


Support for custom open/closed block statements

Silverstripe allows registering additional block statements, e.g. the cacheinclude module provides statements like:
<% cache_include 'App/Includes/SomeInclude', 'IncludeCacheName' %>

<% cache 'CacheName' %>

        <% loop $SomeExpensiveLookup %>

{$Title}

        <% end_loop %>

<% end_cache %>

The plugin should support these statements, based on whatever the PEG parser allows

0.3.0 includes this functionality


Add the 'coin' to the 'payment' notification so that the coins icon can be displayed.

Otherwise, it's just a generic icon, which just looks weird:

done


Customizing the Photo button area

Forgive me, I couldn't find anything in the docs.
Is there a way of hiding area below the photo /video preview (the white area with the capture button) in the config?
Thanks!

hmmm, found the set color customization, however it possible to set the camera preview height to the full size of the phone (minus the nav bars?)

How did you find the color customization for the photo button?

There is 2022 year outside🎉 Old outdated issues are closing. Please search if there are already existing issues with the same problem before opening new ☮


Add host target triple for s390x

@swift-ci please smoke test

Looks good, @ddunbar can you double check?

Thanks for the fast turnaround!

Will this be merged to swift-4.1-branch as well?


needs lineage clarification.  Customers think Synapse scan should has lineage.  Please provide a link for customer to get lineage.

[This page needs lineage clarification.  Customers think Synapse scan should have lineage information after completing this QuickStart.  Please provide a link for customer to get lineage.]

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

ID: 89adddc6-0852-4d53-0764-df3a9aa3a7cd
Version Independent ID: 4125350c-be6f-62d7-5d8c-7881a4e39fab
Content: Connect an Azure Purview Account  - Azure Synapse Analytics
Content Source: articles/synapse-analytics/catalog-and-governance/quickstart-connect-azure-purview.md
Service: synapse-analytics
Sub-service: purview
GitHub Login: @julieMSFT
Microsoft Alias: jrasnick

@tifischer
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Hi @julieMSFT - Could you please review the below feedback:
Below is the additional information shared by @tifischer.
SUMMARY
Problem Statement
The concern is that Synapse Lineage does not appear.  The documentation provided is here:

Cause
Lineage would appear in the case that you use Azure Data Factory, Azure Data Share or PowerBi. One of the platform features of Azure Purview is the ability to show the lineage between datasets created by data processes. Systems like Data Factory, Data Share, and Power BI capture the lineage of data as it moves. Custom lineage reporting is also supported via Atlas hooks and REST API.
Resolution
I have provided a support guide with more details regarding capturing lineage in Purview: https://docs.microsoft.com/en-us/azure/purview/catalog-lineage-user-guide
Following URL covers topics related to lineage:

https://docs.microsoft.com/en-us/azure/purview/catalog-lineage-user-guide
https://docs.microsoft.com/en-us/azure/purview/how-to-link-azure-data-factory
https://docs.microsoft.com/en-us/azure/purview/how-to-link-azure-data-share
https://docs.microsoft.com/en-us/azure/purview/how-to-lineage-powerbi

Feature Request
A Feature request for Data Lineage from Synapse Pipelines is available in the feedback forum. Please consider voting for the feature :https://feedback.azure.com/forums/932437-azure-purview/suggestions/42721016-data-lineage-from-synapse-pipelines

Fixed in this PR #162424


All tests fail due to package management

Prerequisites

[ X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in QA
Followed all applicable steps in the TROUBLESHOOTING guide: TODO
Checked for relevant discussions on Discord: https://discord.gg/pPERUuv
Checked that your issue isn't already filed: https://github.com/COVID-19-electronic-health-system/Corona-tracker/issues

Description
All tests are currently failing
Steps to Reproduce

Clone the repo
cd client && yarn install && yarn run test a

Expected behavior:
Tests pass
Actual behavior:
Tests fail
Reproduces how often:
100%

This is currently the pinned issue on create-react-app - going to need a temporary workaround


Public Portal: Menu items overlap container and extend beyond region on page refresh

When the page refreshes, the menu items pull more than what the horizontal container displays and overlap beyond the region. After the refresh, everything goes back to the original view.
Steps to reproduce the behavior:

Go to 'https://dsd-npo-publicportal-test.azurewebsites.net/settings/theme'
Click on 'color picker'
From the icon picker, click on the color adjuster and see how the horizontal menu responds
Actual Results: The system pulls all the menu items which goes beyond the horizontal menu region

Expected behavior
Upon page refresh, the horizontal menu should keep the original state
This is not limited to the theme page
screen-capture (60).webm

Related Item
#1174


Bind function is not working

What OS are you using (uname -a, or Windows version)?

Windows 10, version 1903
amd64
Edge Chromium Beta 84.0.522.28
What programming language are you using (C/C++/Go/Rust)?

go version go1.14.4 windows/amd64
What did you expect to see and what you saw instead?

I tried to run the example code. In Devtools I found that the functions noop and add do exist, but no results were given when the functions are called.
package main

import (
	"log"

	"github.com/zserge/webview"
)

func main() {
	w := webview.New(true)
	defer w.Destroy()
	w.SetTitle("Hello")
	w.Bind("noop", func() string {
		log.Println("hello")
		return "hello"
	})
	w.Bind("add", func(a, b int) int {
		return a + b
	})
	w.Bind("quit", func() {
		w.Terminate()
	})
	w.Navigate(`data:text/html,
		<!doctype html>
		<html>
			<body>hello</body>
			<script>
				window.onload = function() {
					document.body.innerText = ` + "`hello, ${navigator.userAgent}`" + `;
					noop().then(function(res) {
						console.log('noop res', res);
						add(1, 2).then(function(res) {
							console.log('add res', res);
							quit();
						});
					});
				};
			</script>
		</html>
	)`)
	w.Run()
}

Using the new DLLs from https://github.com/geaz/sharpWebview/tree/master/libs solves the issue.


Add wilk to Contributors list

Test

@wilkinv, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I accepted it?


User building extension from source experiences several errors

Extension install in Brave throws an error: "Unrecognized manifest key 'browser_specific_settings'."
More specifically the code string throwing the error :
"browser_specific_settings": {     "gecko": {       "id": "{e22ae397-03d7-4622-bd8f-ecaca8c9b277}"     }   },

Additionally

From the reporter:
1. I clicked the purple button to DL source.
2. Saved zip folder to DL folder
3. Unzipped to User->Appdata->local
4. Opened Brave Extensions (my Dev mode is always on)
5 Clicked "Load Unpacked"
6. Selected the unzipped hiro folder from dir
7. In Brave tab the auto opened, initiated new wallet setup.
8. Recorded Seed
9. Created/confirmed password
10. Clicked "receive Bitcoin"
11. Sent in some sats from external self custody wallet
12. Once tx confirmed, attempted to connect and swap on Inswap
13. Clicked connect at app.LNswap.org
14. Selected BTC->STX
15. Clicked Start

@pseudozach

seems like step 14 is the key

Indeed sponsored transaction is only supported with LNBTC <--> STX
You will get an additional lightning invoice to pay the fee with Lightning. The dapp will pay the actual mining fee in STX for you "sponsored transaction".
I am closing this issue, I assume the errors we saw are unrelated and are likely unrelated build errors. Like the code identified for Firefox and related errors can be ignored when building for Brave.


Documentation site is broken

The Documentation link located at the bottom of https://github.com/tiscordlib/tiscord/blob/main/packages/tiscord/README.md redirects to tiscord.me, which shows a GitHub Pages page saying "There isn't site here"

fixed, was an issue with github pages disabling itself because i recreated the stable branch recently


The vertical scrollbar of ImSequencer is not displayed

When drag the mouse on the right edge, the items can be scrolled, but scrollbar is not shown.
please refer to the video attached below:
https://user-images.githubusercontent.com/10323618/138114221-1e14e0e5-cf3e-4ab6-82d1-7b47363d0395.mov

Hello @qoyooo ,
Sorry to invade your topic... but,
Do you have some code to help a bit on ImSequencer?
I am super stuck and I don't know where to start from to make something reusable…

@moebiussurfing
please refer to https://github.com/qoyooo/gltfViewer.

Thanks @qoyooo ,
I compiled your code and it works.
But I dont know how to see the curves or to read the code to learn how to read the frame values for each param...
I can edit e.g the position in a frame, but can't see the curves or keyframes..
It's maybe here where you read the current frame values for each parameter?
      if (selectedEntry >= 0 && selectedEntry < mySequence.GetItemCount()) {
           uint32_t index = mySequence.animation->channels[selectedEntry].samplerIndex;
           // switch (mCurrentGizmoOperation) {
           // case ImGuizmo::OPERATION::TRANSLATE:
               ImGui::SameLine();
               ImGui::InputFloat3(ICON_MDI_ARROW_ALL "Location", glm::value_ptr(mySequence.animation->samplers[index].outputsVec4[currentFrame]));
           //     break;
           // case ImGuizmo::OPERATION::ROTATE:
               ImGui::SameLine();
               ImGui::InputFloat4(ICON_MDI_ROTATE_ORBIT "Rotation", glm::value_ptr(mySequence.animation->samplers[index].outputsVec4[currentFrame]));
           //     break;
           // case ImGuizmo::OPERATION::SCALE:
               ImGui::SameLine();
               ImGui::InputFloat3(ICON_MDI_ARROW_EXPAND_ALL "Scale", glm::value_ptr(mySequence.animation->samplers[index].outputsVec4[currentFrame]));
           //     break;
           // default:
           //     break;
           // }
       }

BTW, could be nice to make a pull to the main repo example, no?
To move/rotate the cube. On the @CedricGuillemet examples,
curves are not linked to move the scene.


[WIP] Move cli resources to sensu::cli class

Pull Request Checklist

Description

Move cli resources to sensu::cli class
Manage sensu user/group so to support sensu::cli on host not running agent or backend
Motivation and Context

This will allow non-backends to have sensuctl setup which is supported by Sensu Go. This also would make it easier to have a Bolt task that can use sensuctl to populate Bolt Inventory v2.
The management of sensu user/group was necessary to support using sensu::ctl class on a host that doesn't include sensu::agent or sensu::backend as the agent and backend packages had previously been what adds the user/group.

Marked WIP as needs support for Windows and possibly addressing #1163 as it was the acceptance tests for sensu::cli where I ran into #1163.

Removed WIP. I will fix #1163 in a different pull request that will have to incorporate this PR. The changes necessary to fix #1163 are rather involved and potentially not backwards compatible as a parameter will have to get removed.

I am struggling to get simple "Does file exist?" to work in appveyor. I got 2 appveyor instances stuck on RDP but didn't accept my password so have to wait till they time out. I'll cleanup windows testing once I'm able to debug why checking file existence is not working.

Fixed Windows tests and restored previous test matrix. This PR should be good to go once Travis tests complete.

Closed #1159 as this had similar changes that made that PR not necessary.

I've rebased with master and added Bolt v2 inventory task that requires the CLI changes in order to run inventory on non-backend node.

Added commit to make managing agent passwords much more consistent. Updated PR description as moved agent_password and agent_old_password from sensu::backend class to sensu class. This way both backends and agents could use same value and ensure that value is propagated to sensu:agent user and also updated in agent.yml. Previous behavior was flawed and made changing the password very difficult.

Added commit to consolidate sensu_config which was just a way to set a few sensuctl settings and moved those settings into sensuctl_configure and exposed them via sensu::cli class. The previous sensu_config would allow invalid settings and had no validation and didn't really do much that couldn't be handled by the type managing sensuctl configuration.


feat(tasks/queue-size): Implement queue_size

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Implement queue size to limit low priority tasks from overwhelming the queue
Plan and SQL:
explain analyze SELECT COUNT(*) AS "__count" FROM "task_processor_task" WHERE (NOT "task_processor_task"."completed" AND "task_processor_task"."num_failures" < 3 AND "task_processor_task"."task_identifier" = 'edge_request_forwarder.forward_identity_request');
                                                                       QUERY PLAN
---------------------------------------------------------------------------------------------------------------------------------------------------------
 Aggregate  (cost=69541.34..69541.35 rows=1 width=8) (actual time=5.017..5.018 rows=1 loops=1)
   ->  Index Scan using incomplete_tasks_idx on task_processor_task  (cost=0.38..68634.33 rows=362804 width=0) (actual time=0.018..5.013 rows=4 loops=1)
         Filter: ((task_identifier)::text = 'edge_request_forwarder.forward_identity_request'::text)
         Rows Removed by Filter: 1
 Planning Time: 0.305 ms
 Execution Time: 5.037 ms
(6 rows)

How did you test this code?

Adds unit test cases

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (929afeb) 95.53% compared to head (9267a92) 95.53%.
 Consider uploading reports for the commit 1d15e17 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2826   +/-   ##
=======================================
  Coverage   95.53%   95.53%
=======================================
  Files         994      994
  Lines       28072    28095   +23
=======================================
+ Hits        26818    26840   +22
- Misses       1254     1255    +1

Files
Coverage Δ

api/edge_api/identities/edge_request_forwarder.py
100.00% <100.00%> (ø)

api/task_processor/exceptions.py
100.00% <100.00%> (ø)

api/task_processor/models.py
95.41% <100.00%> (+2.96%)
:arrow_up:

.../task_processor/test_unit_task_processor_models.py
100.00% <100.00%> (ø)

api/task_processor/decorators.py
88.40% <28.57%> (-5.45%)
:arrow_down:

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


Support v2/lookup/bvn mono api

Please add methods to send requests to api v2

Hi @yarscript apologies, just picking this up now. I'll look into this and revert.

Hi @timoladoyinbo
Thanks for reply, It's ok :)
I was need in few requests so I implement them in my project so I'd like to create PR with it stuff

Hi, @yarscript sounds good. Please feel free to make the PR