Browserslistc and Android

As I discovered in #30986, if I remove Android from .browserslistrc tests fail on Legacy Edge. For example: https://github.com/twbs/bootstrap/runs/788301337?check_suite_focus=true
This might be an issue with Babel, opening an issue to track it.
/CC @hzoo

@hzoo we dropped Legacy Edge support in our main branch, but not when I remove Android from browserslist then Safari 10.0 fails: https://github.com/twbs/bootstrap/runs/1399985635
Any hints how to proceed with this? I'd like to get any browserslist-related changes in before we release v5.0.0-beta1 :)

I noticed that 'let' and 'class' are still both bable'd from the js source code
npx browserslist shows the target browser list

opera mini (and quite a few others) are target browsers
> 0.5% means the target browsers list might change during the life of v5 as the browser db is updated

Maybe mention in the doco

that babeljs config (.babelrc) also uses .browserslistrc
.browserlistrc contains a > 0.5% line (i.e. target browsers with > 0.5% of world traffic)
can do (e.g.) > 0.5 in GB, putting (say) China or Africa there will make a difference

The babel and browserlist config is worth point out so people can update their own build chains (I was unaware of a separate .browserslistrc, so didn't understand how the bablejs config worked)

I noticed that 'let' and 'class' are still both bable'd from the js source code
npx browserslist shows the target browser list

opera mini (and quite a few others) are target browsers
> 0.5% means the target browsers list might change during the life of v5 as the browser db is updated

Maybe mention in the doco

that babeljs config (.babelrc) also uses .browserslistrc
.browserlistrc contains a > 0.5% line (i.e. target browsers with > 0.5% of world traffic)
can do (e.g.) > 0.5 in GB, putting (say) China or Africa there will make a difference

The babel and browserlist config is worth point out so people can update their own build chains (I was unaware of a separate .browserslistrc, so didn't understand how the bablejs config worked)

@XhmikosR Anything still to do here since we're not supporting Legacy Edge?

@XhmikosR Anything still to do here since we're not supporting Legacy Edge?

Yeah, the issue is still present and we still have Android >= 6.

Yeah, the issue is still present and we still have Android >= 6.

The debug:true option in .babelrc shows which browser version cause the use of which babel plugin/polyfill.

The debug:true option in .babelrc shows which browser version cause the use of which babel plugin/polyfill.

The problem is two-fold AFAICT:

should our dist files be ES5 or ES6? We are already using some new features already, but using less transpiled code will result in smaller dist files
there's still the failures on Safari which I don't get why they happen

So, it seems the error we are hitting is Safari 10 throws an "Undefined is not a function" error in places like:
[].concat(...foo)

@XhmikosR hard to validate as I can't get Safari 10 to run on my macbook but [].concat(...document.body.children) throws indeed some error in Browserstack's Safari 10.1. Array.from(document.body.children) would work in that case.

@alpadev thanks for testing it out :) That being said, I think the issue is that this should have been handled by Babel, shouldn't it?
Also, check out my comment above; I'm not sure if there's a convention about the dist files syntax. Traditionally, we were using ES5 or even ES3. Now, on main we do use some modern APIs already.

Here's a temp branch to see the size diff:

https://github.com/twbs/bootstrap/compare/main-xmr-browserslist
https://ja2r7.app.goo.gl/nJogapDpNshsgcUV6

The difference is quite big... Putting aside that it's probably a breaking change to land it now, though.

😲 nice improvement. I wish reducing payload size was always that easy (too bad we can't get foresee the impact).
I agree with you that ideally Babel should take care to be compliant with our targeted browsers.
Maybe related: https://github.com/babel/babel/issues/11886 and https://github.com/babel/babel/issues/8298 (Last comment here mentions this https://github.com/babel/rfcs/pull/5)

TBH, there shouldn't be any real impact because our supported browsers support these features. Hence why my initial plan was to move ahead with this change, but I kept hitting the test failures.
Now, I have no idea if it's something in our Babel config, a bug upstream or something else. So, I kept this change back. And technically, I'm not sure if we can land the change in beta3, even if we sorted out the test failures.

To my understanding using Babel in "loose-mode" might result in non spec compliant code when it comes to using the spread operator with certain things.

"Well, this is a cost of using loose mode. It's not always spec-compliant and such cases might bite you."
"Loose mode is meant to work only in a specific subset of all the possible cases"

About those tests in Safari 10.. Safari 11 was released 31.10.2017.. Considering macs usually get around 7 years of OS updates that means one needs to use a mac that is like 10+ years old to be incompatible. According to https://caniuse.com/usage-table the percentage is quite low too.
Hard to say what would be the best thing to do =/ That's like the fun we frontend guys always had with the rapid development the last decade..

We could replace [].concat(...element.children) with Array.from(...element.children). The support is quite good https://caniuse.com/?search=array.from

@XhmikosR So I investigated this topic a little further. Also I have to say thank you for giving me a reason to do this.
Babel always been kinda blackbox for me TBH. You set it up. Maybe got to fiddle with the configuration because of outdated informations. Eventually you're happy with the result but after a while someone comes up with a problem that is related to this (usually some old IE in my case).
So that we're on the same boat - Babel should transform our code so it's fully compliant with our targeted browsers. Although with our current configuration Babel isn't fully doing this.
One reason for this is that we're using "loose mode" (source). In loose mode, the transformations do not fully conform to the ecma spec and usually result in a much simpler implementation. The advantage is that the resulting code is less complicated and significantly smaller (~15kb for our bundle).
To pick up your example from above.
Source:
children(element, selector) {
  return [].concat(...element.children)
    .filter(child => child.matches(selector))
}

loose: true
children: function children(element, selector) {
  var _ref2;

  return (_ref2 = []).concat.apply(_ref2, element.children).filter(function (child) {
    return child.matches(selector);
  });
}

loose: false
children: function children(element, selector) {
  var _ref2;

  return (_ref2 = []).concat.apply(_ref2, _toConsumableArray(element.children)).filter(function (child) {
    return child.matches(selector);
  });
}

function _toConsumableArray(arr) {
  return _arrayWithoutHoles(arr) || _iterableToArray(arr) || _unsupportedIterableToArray(arr) || _nonIterableSpread();
}

// ...

Another reason. For browsers that don't support some JS features, Babel can be configured to automatically add polyfills but in our case this isn't happening as well. (https://babeljs.io/docs/en/babel-preset-env#usebuiltins)

About the Browserslist configuration. Babel is resolving targeted browsers with the option mobileToDesktop: true (source).
From the Browserslist docs:

mobileToDesktop: Use desktop browsers if Can I Use doesn’t have data about this mobile version. For instance, Browserslist will return chrome 20 on and_chr 20 query (Can I Use has only data only about latest versions of mobile browsers). Default is false.

In our case with the Android >= 6 Browserslist returns every Android WebView/Chrome version starting from 37 since thats what they use as the lowest version. And it looks like that Babel handles those versions equally (source). I could be wrong but it looks like we're effectively targeting chrome starting from 37 not 60.

If you set the debug option in .babelrc, it shows which babel plugin is being activated by which browser versions, e.g.
proposal-numeric-separator { "ios":"10.3" }
If a plugin is being activated by just 1 line of code, it can be easier to just rewrite it.
Unless you have useBuiltins set (which .babelrc does not), babel only does transpiling (e.g. let => var) and not polyfills. Setting useBuiltins and debug shows which polyfill plugins are activated.
In practise, I've found it easier to avoid polyfills, e.g. in for loops, not using the  array.entries() iterator.
Aside: in .browserlistrc, setting > 0.5% means the babeljs plugins activated will change over time. See npx browserslist@latest --update-db

@alpadev so this means it's expected that we get this result since we are using loose: true, right?

I could be wrong but it looks like we're effectively targeting Chrome starting from 37 not 60.

That is correct, it's a side effect of using Android >= 6 since Chrome is used there too.
Not sure how to proceed at this point. I mean, I definitely don't want us to disable loose, nor use polyfills. If we could work around these, that would be the best. That being said, I'm not sure if using modern stuff in our dist code will cause any issues for consumers. Technically, we don't support browsers which don't support these features, but you never know...
My initial plan was to make our dist files as modern as possible for v5, but since I had trouble for months, so, I slated the change for later.

Alright, after discussing this with @mdo, we agreed to proceed with dropping Safari < 12, thus we should be able to get dist files with modern code. I'll prepare a PR tomorrow.

@aavmurphy you're correct. Adding debug: true definitely helps with finding those spots but that's rather some manual process and ideally it should be some set and forget thing IMO. I'm also not a huge fan of polyfills but I guess it depends on what is required/expected and in some cases they may be mandatory.
@XhmikosR Yeah loose: true could have been the culprit in case of Safari 10. In general even for valid code the loose transformations could have a different output because the implementation may be too simple and/or not getting the intention right. (e.g. here). Not saying we should disable loose mode as there is usally just a little change required to make it work and the fully spec compliant transformations are likely to bloat our code when it isn't necessary. (Some article in case you're still unsure what loose does.)
To further optimize Browserslist to our needs, we could utilize the supports query feature

supports es6-module: browsers with support for specific features. es6-module here is the feat parameter at the URL of the Can I Use page. A list of all available features can be found at caniuse-lite/data/features.

Also we could utilize https://github.com/browserslist/browserslist#configuring-for-different-environments and create a modern and compatiblity bundle just in case you have concerns about the support.
In the end Babel uses the informations provided by Browserslist to decide what transformations are necessary so the better it matches our requirements the lesser transformations would happen.
Hope that helps.

I tried loose: false, and it still fails on Safari 10...
I think we can afford just targeting Safari > 12, see #33399. I mean, that was the initial idea for v5. What's I'm not sure about is if everything else is OK/following the best practices. Like our dist files, rollup config etc.
I mean, from the looks of it, we could even drop babel. I'd say that we try to make one change at a time. We can keep babel of course, since this will allow us to use features that aren't shipped to browsers yet, just pointing it out what I notice so far.

It might be that the ( ) => { } syntax ("arrow functions") being replaced causing the problem.

that's not ios & safari <10, not opera mini, and maybe not (some of the chinese ones)

npx browserslist@latest --update-db updates
npx browserslist
shows which browsers are being supported.

We don't support those browsers and the issue we are having is specific to
spread and Safari.
On Sat, Mar 20, 2021, 15:59 Andrew Murphy @.***> wrote:

It might be that the ( ) => { } syntax ("arrow functions") being replaced
causing the problem.

that's not ios & safari <10, not opera mini, and maybe not (some of
the chinese ones)

npx @.*** --update-db updates
npx browserslist
shows which browsers are being supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/31153#issuecomment-803341239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVLNPS3NGN3TG66MKVPSTTESS3JANCNFSM4OFUDHNQ
.

I tried loose: false, and it still fails on Safari 10...

Yeah sorry, from looking at the changes in the code I've been guessing that loose: true could have been the problem but I wasn't really able to validate that. Also I just realized the differences in the code I mentioned here were actually caused by Android >= 6.
I just managed to install MacOS Yosemite on a VM and was able to install Safari 10.1.2 there. I can confirm that with loose: false Safari is still throwing the error from the screenshot I posted here. But after enabling Babel to add polyfills as well, I got Bootstrap to run in Safari 10.1.2 without errors (both, with loose: true and loose: false).

I think we can afford just targeting Safari > 12, see #33399. I mean, that was the initial idea for v5.

You're likely right about this. Safari 12 is able to be installed on MacOS Sierra (released in September 2016) up to Mojave (September 2018). Looks like Apple isn't really into supporting older versions of their browser but rather keep them in line with their OS major releases once a year.

What's I'm not sure about is if everything else is OK/following the best practices. Like our dist files, rollup config etc.

TBH I'm not very experienced with rollup because I've been using webpack most of the time. When I fiddled around to enable polyfills in Babel I ran into a problem tho, where I had to enable the commonjs plugin for the bundle too. While reading the documentations I realized that in their examples they have the node-resolve plugin always going first but in our configuration it comes after the babel plugin. Since everything been working so far I'm not sure if it really matters tho.

I mean, from the looks of it, we could even drop babel. I'd say that we try to make one change at a time. We can keep babel of course, since this will allow us to use features that aren't shipped to browsers yet, just pointing it out what I notice so far.

I really have no opinion about Babel. I usually added it to my build stack because it's mentioned in a lot of articles and for the lack of native support for ES6 in the past but I also had my moments when it wouldn't do what I expected.  One thing to mention about features that are partly shipped yet. There is another configuration flag https://babeljs.io/docs/en/babel-preset-env#shippedproposals. In general I'm with you - if we consider dropping it we should do this step by step and the debug option can help with narrowing down what browser support we're giving up. If you find it useful I can also post a list with all Babel transformations and their browser versions.


Piping ov to ov breaks commands and terminal

Repro
> ov file | ov

Then type /private.
Expected behaviour: private is searched. First ov prints output to stdout and doesn't affect the next call.
Actual behaviour: Mode selection menu appears when p is pressed. Unable to exit with q, terminal session is corrupted. It seems both processes are reading commands and interfering with each other.
Without piping it works fine.
Why I need to pass ov to ov?
I have some tools which are configured to use ov by default, e.g. I want this: delta --help | bat -l help.
less handles such case fine.
Info
> ov --version
ov version 0.36.0 rev:d671c94

Platforms: MacOS, Linux

Thank you for Issue.
This is certainly a problem.
I want to redirect the ov while displaying it on the screen, but if it starts twice, there will be problems.

Changed so that the screen is not displayed when output is redirected.
The --force-screen option is now required to display the screen.

Thanks! I built it from sources and this case with piping works now.


Don't display contributors that have no lists or no active lists in search dropdown

Overview
Don't display contributors that have no lists or no active lists in search dropdown
Describe the solution you'd like
Update the contributors endpoint to prevent returning contributors who have either

no lists at all
no active lists
no public lists

Also ensure that users who are logged in and click on the "My Facilities" Link in various places in the app end up in a proper state: essentially the ID will "miss" the set of options for collating it on in the contributors response, so figure out how to handle that.

I think we may also want to filter out contributors who have active/public lists if those lists do not have any facility matches. Otherwise the menu will display contributors whose lists are still in progress but as yet have no matches to display.

Also ensure that users who are logged in and click on the "My Facilities" Link in various places in the app end up in a proper state: essentially the ID will "miss" the set of options for collating it on in the contributors response, so figure out how to handle that.

This is already working in a correct way without changes. Essentially If I click "My Facilities" without having any lists, my contributor ID will not match any of the contributors sent back, so the search will return no facilities and show a message that no facilities were returned:

This is already working in a correct way without changes.

I am not 100% happy with the way this is working as we are adding an invalid item to the contributor list and the fact that it is an integer looks broken. I can imagine a contributor thinking "My company is not named '1'. Why did it search for that."
We have enough information to show a more helpful message to the contributor.

We know they are logged in
We know they are searching for themselves
We know that they have not active facility lists

With this information we could show a "You have no active facility lists. Contribue your facilities to see them on the map" message with a helpful link
This is a bit challenging and, if we agree to implement it this way, it may make sense to create a separate issue for it.

I am not 100% happy with the way this is working as we are adding an invalid item to the contributor list and the fact that it is an integer looks broken. I can imagine a contributor thinking "My company is not named '1'. Why did it search for that."

This makes sense. One thing I considered and can do here is to add a has_active_lists property to the user serializer which we can check on in the navbar to determine whether to show the "My Facilities" link in the dropdown. If has_active_lists is false, we just don't show the link.
This would unfortunately not immediately make the link re-appear after the user has uploaded a list during the same session. We currently do not show the "My Facilities" link on the list page if there are no lists, but it does automatically show once a list has been uploaded.

I would prefer to guide them toward contributing rather than hide links.

Made #431 for implementing the My Facilities behavior change.


.MathJax_Menu does not exceed a left property of >627

Test case: http://dizzyzane.web.fc2.com/markdown/tests/tests.html & http://dizzyzane.web.fc2.com/markdown/tests/tests.html#top (same page, different colouration)
The menu will not go into the right half of the column on a 1280px wide monitor.
Chrome version: Version 51.0.2704.0 canary (64-bit)
Firefox: 45.0.1

MathJax tries to keep the menu within the space allocated to the document.body so that the menu is shown within the space of the document when possible (e.g., if the document is in an iframe, it needs to stay within that iframe's space).
Your document.body has CSS that sets max-width: 768px, so when MathJax asks for the document.body.offsetWidth, it gets 768 and thinks that is the size of the space it has to work with.  unfortunately, with an auto margin, the document.body.offsetLeft is still 0, not the size of the margin, and so MathJax thinks that the document body is flush left.
It may be possible to use getBoundingClientRect() to get the correct offset of the body.  The computation of the correct position is pretty fragile, and there are a number of situations like this where it fails.  It may be possible to resolve it in a future release.
@Peter, this might also help with the incorrect positioning of the menu in Readium.

@dpvc Readium turned off the menu I think. But it might be useful for the next iteration of the menu.

The issue1434 branch of my fork of MathJax includes code to better handle the situation when the document body has margins or is scrolled.

==> Merged

==> In testsuite
UI/math-menu/issue1434.html


gcfflasher: 4.4.0 -> 4.5.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dresden-elektronik/gcfflasher/releases.
meta.description for gcfflasher is: CFFlasher is the tool to program the firmware of dresden elektronik's Zigbee products
meta.homepage for gcfflasher is: https://github.com/dresden-elektronik/gcfflasher
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.5.2 with grep in /nix/store/753n9gyp0rna4ypzqgkn9x5j70l0xg1a-gcfflasher-4.5.2
found 4.5.2 in filename of file in /nix/store/753n9gyp0rna4ypzqgkn9x5j70l0xg1a-gcfflasher-4.5.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

gcfflasher

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/753n9gyp0rna4ypzqgkn9x5j70l0xg1a-gcfflasher-4.5.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gcfflasher https://github.com/r-ryantm/nixpkgs/archive/c8b7043ed3d8e009ab949007cc381a8bb0ba2835.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c8b7043ed3d8e009ab949007cc381a8bb0ba2835#gcfflasher

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/753n9gyp0rna4ypzqgkn9x5j70l0xg1a-gcfflasher-4.5.2
ls -la /nix/store/753n9gyp0rna4ypzqgkn9x5j70l0xg1a-gcfflasher-4.5.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

gcfflasher

Maintainer pings
cc @fleaz for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Build on my machine and appears to be working
@NixOS/nixpkgs-merge-bot merge

Build on my machine and appears to be working
@NixOS/nixpkgs-merge-bot merge


Always console.warn on "Strong mode is-check failure"

A lot of user code catches errors – since this is a dev tool, it'd be great if this was echoed before throwing
CC @vsmenon

going to close this as obsolete: we made this a debugger; statement, and also ti's going away soon


fix(Template) Draft with template variable create a codeblock for lists

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Coverage remained the same at 95.535% when pulling 6b28a12519003e2ad9e09f34a522fc959161b77d on jeromesimeon:js-wrapped-list-variable into 807cd3e9f5371c8a44e87bb84970726f78aa7d6d on accordproject:master.


Check that the given product exist at server

Check that the given product exist after getting a product at the server.

@gyorb See the usage of __check_prod_db function:
https://github.com/Ericsson/codechecker/blob/16f75a580ddb16b8902f4f371921a239b84e633c/libcodechecker/server/server.py#L356-L357
We pass a Product object to the __check_prod_db function and check that the parameter exist.
https://github.com/Ericsson/codechecker/blob/16f75a580ddb16b8902f4f371921a239b84e633c/libcodechecker/server/server.py#L269-L272
If the product object (the parameter) is None we write the following log message to the output:
[WARNING 2018-10-26 11:01] - The product None does not exists
Traceback (most recent call last):
  File "/home/username/codechecker/build/CodeChecker/lib/python2.7/libcodechecker/server/server.py", line 357, in do_POST
    self.__check_prod_db(product)
  File "/home/username/codechecker/build/CodeChecker/lib/python2.7/libcodechecker/server/server.py", line 272, in __check_prod_db
    raise ValueError(error_msg)
ValueError: The product None does not exists

I refactored the code and rebased my commit. Now we should pass a product endpoint to the  __check_prod_db function and it will check the existence of the product.

Please wait for travis to finish before merge.


Generated SectionList Grid, last row shifed

Hi,
I have a question about sectionlist generated by ignite2.
I'm new to React Native and creating my first app in RN, so pointing me out if I get something wrong.

The last item in the second section is shifted because section footer, "END SECTION!!!", and list has a style justifyContent: 'space-around'.
So I remove the line
renderSectionFooter={this.renderSectionFooter}

So why this happens? And idea?
How to reproduce?
ignite g list
change width and height from 100 to
width: 150,
height: 120,

Thanks

What version did you use to generate?  This yellow box is due to changes in React Native that haven't been settled just yet.

Hi @GantMan
I'm using ignite 2.0.0

@Derek-X-Wang It appears your issue is with flexbox styling, not anything specific to Ignite. If you're still having trouble with this I'd suggest posting to our community Slack or StackOverflow, or do some flexbox tutorials to learn more about some of it's oddities.


Fix and Update Data Sources

A few supported_ta fields were poorly
formatted in the new data_source objects.
Fix the formatting of those fields and
update those objects to use the latest
relevant TAs available on Splunkbase

Didn't realize this work was already done: https://github.com/splunk/security_content/pull/3072
Closing out this PR


Filter line and ticket list enhancement

Instead of the tags-field I think we need a status and a type dropdown.

There is 2 seperate issues here split them into 2 issues.

Aye.

Great. :)
Thanks and closed.


Added basic types to app config.

Currently not used. Just a push to fix #71. Real fix inside guh-libjs->modelsHelper.js

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://guh.guru:8080/job/guh-webinterface-pull-request/51/
Test PASSed.


Webview doesn't download

Steps to Reproduce
To recreate
Run this code and try going to any stocks page and clicking on the export to excel button, it should show the download tab or download it, instead it does nothing in the flutter webview app
Expected results:
Show download tab or download the file
Actual results:
Nothing, it goes to the link and refreshes the page, no download popup or file getting download

Code sample

              body: WebView(
                initialUrl: "https://www.screener.in/",
                javascriptMode: JavascriptMode.unrestricted,
                userAgent: _proxyUserAgent,
                onWebViewCreated: (controller) {
                  this.controller = controller;
                },
                zoomEnabled: false,
                navigationDelegate: (NavigationRequest request) {
                  if (request.url.startsWith("https://www.screener.in/")) {
                    return NavigationDecision.navigate;
                  } else if (request.url.contains("google")) {
                    return NavigationDecision.navigate;
                  } else {
                    _launchURL(request.url);
                    return NavigationDecision.prevent;
                  }
                },
              ),

Logs

Flutter run --verbose
![image](https://user-images.githubusercontent.com/30836518/153265415-1fbc3b09-35d3-4b67-bd64-e8c4442cdafb.png)

Flutter analyze
![image](https://user-images.githubusercontent.com/30836518/153265221-1cff9295-d23e-4811-af93-df6c03cbd6b3.png)

flutter doctor -v
![image](https://user-images.githubusercontent.com/30836518/153265659-a4626e94-a98a-494c-aa39-2393a30adc55.png)

![image](https://user-images.githubusercontent.com/30836518/153263373-b32af5a6-a5ad-4c4b-82aa-71a44652e4ce.png)
https://user-images.githubusercontent.com/30836518/153264657-250ad7d2-5000-4bc8-849f-b4464e2aa4a0.mp4

My bad, I forgot that xml and ESC files are not located in the same identity matrix)':

Hey @THELEMITED didn't get you, I'm new to flutter
And I tried the inapp plugin as well but that didn't work either
So I guess I might be missing something out
Any idea on how to get it to work?

Hi @KaviHarjani, this is due to the fact that the webview_flutter plugin doesn't support downloads at the moment.
There's an issue tracking this in https://github.com/flutter/flutter/issues/39321. Kindly follow up there for more information.
Closing from here as duplicate.
Thank you

Hey thanks @danagbemava-nc


completed ci build

Missing workflow file

Need to create a new pull request to get workflow to run?


Rename the repository to eodiro-server

Since the eodiro-api2 does more than just serving APIs, it is now more adequate to use the name eodiro-server.

This issue only happens on Apple Safari (both macOS and iOS/iPad OS).


Calculations not rounded

What happened?
After update to v2023.8.0-beta25 bucket and duration calculation are not rounded with local "," decimal pointer.
Decimal separator ',' is only in GUI and it is not an issue.
How to reproduce
After update I called Calculate All Zones.
Relevant log output
No response
Which version are you running?
v2023.8.0-beta25
Diagnostics file

[X] I have attached a diagnostics file

Additional information
config_entry-smart_irrigation-6e18a9ad38f8ac00ef4285672ad17839.json-2.txt

Did you read the notes when you installed it? Please go back and read it.


Documentatino for major version locking

We're trying to figure out a way to lock packages using major/minor versions.
I've tried multiple formats, but the documentation doesn't really go into detail on how to lock versions, and more specifically lock major versions of a package. It would be awesome to see a topic on this in the documentation.

Pipenv uses the same specifier format as pip; it is outlined in PEP 440. Contribution on bridging the gap between Pipenv’s documentation and this standard document is very much welcomed.

Hi @uranusjr I just created a PR on this, let me know if you find this suitable

Thanks @trallard, sorry I should have specified better:
We're looking to version-lock existing packages, which means that we're trying to figure out the toml format and the available options. Essentially we're looking for the easiest way to change a package from '=*' to a locked major-version.

@trondhindenes I think if you use pipenv update <package> should allow you to update a package version
Similarly --selective-upgrade should do and then pin the version on both your Pipfile and Profile.lock (I might need to do some digging around to provide more accurate feedback)
On the other hand the PR tackles the point raised by @uranusjr of bridging the gap in the docs (perhaps I should have opened a new issue for this)

hey @trondhindenes I just checked this over,  even if you already have a given package installed you can use pipenv install "package~=x.y.*  or (== although this will definitely prevent you from getting any updates)
This will then install the specified version, lock it and modify your Pipfile. Hope this helps

Hi @trallard, @uranusjr and @trondhindenes I made some changes on this issue, but for some reason I can not see the update on GitHub. Well, I take another approach for this issue, It is possible to merge both solutions?

@GilbertoCS Can you submit a pull request with your changes on this issue so I can compare them with @trallard’s side by side and try to merge them? (Apologies if you already did; please mention the PR #.)

Hi @uranusjr as you request I create an PR #3046.


use to testpr

retest

retest

retest

retest

retest

retest

retest

ok to test

retest

ok to test

retest

ok to test

retest

retest


Unable to scrape Istio Pilot additional metrics

Hi team,
We are in process of adding some more metrics for Istio Pilot, additional to what the default metrics offer. We want to expose pilot_debounce_time , for that have added the following to istio.yaml in datadog-confd:
instances:
      - use_openmetrics: true
        istiod_endpoint: "http://%%host%%:15014/metrics"
        histogram_buckets_as_distributions: true
        collect_counters_with_distributions: true
        tag_by_endpoint: false
        include:
          - pilot*
        extra_metrics:
          - pilot_debounce_time: 'pilot.debounce'
          - pilot_debounce*

But the configuration doesn't seem to pick and when running the agent in DEBUG mode, it prints:
Skipping metric `pilot_debounce_time` as it is not defined in `metrics

Have tried different combination like, only use extra_metrics or the include , but no luck.
Output of the info page
Agent 7.43.0 - Commit: e3c1ac4 - Serialization version: v5.0.67 - Go version: go1.19.5

Additional environment details (Operating System, Cloud provider, etc):
Steps to reproduce the issue:

Use above mentioned configuration to scrape debounce metrics from Istio Pilot
Run agent with DD_LOG_LEVEL : DEBUG
It prints that the metric is skipped.

Describe the results you received:
The metric is not scraped and is unavailable in DD.
Describe the results you expected:
pilot_debounce_time should be scraped and available in DD.
Additional information you deem important (e.g. issue happens only occasionally):

Hi @ra-grover , could you submit a support case so we can get more information about your setup? I am trying to replicate the issue (with another metric since our test env doesn't emit pilot_debounce_time).
For additional context,
Skipping metric `pilot_debounce_time` as it is not defined in `metrics

means that it detected pilot_debounce_time in your OpenMetrics output, but it skipped collecting it since it's not declared in metrics.py or in extra_metrics. This shouldn't be happening (and when I try this with another similar metric, I can't replicate this issue). If you can submit a support case, we will be able to help you more in-depth.

Closing this due to lack of activity.


Multi-layered render buffers

Applications using geometry shaders to duplicate their geometry often render to different layers of the render buffer. While it is quite simple to just copy each layer to a separate texture and present those to the OSVR-RenderManager, it would be nice to be able to present this render buffer without any extra work.
While I do use this technique in the Dolphin Emulator, I do not require this feature as our post-processing system will separate the layers anyway.

I'm trying to decide if this should be merged with issue 11 (https://github.com/sensics/OSVR-RenderManager/issues/11) and I feel like it should.  Have a look at that issue and add any comments to it that are needed to cover your case.  We're planning to discuss this issue at our retreat early in March and I'd like to make sure we cover all the bases.
Please enumerate the things you see being in the different layers: two eyes with different projection matrices, environment map/background and foreground, static and moving objects, etc.

@russell-taylor This is not the same as the layering in the compositor as is being talked about in issue #11 . I'm talking about a render target that has a texture with multiple layers. This is known in OpenGL as layered rendering, you can find out more about it here: https://www.opengl.org/wiki/Geometry_Shader#Layered_rendering


maas-dhcpd does not start/work when installing 3.3 on 22.04 from deb

After a lot of testing, i am now pretty convinced that there is something broken/or missing in the MaaS 3.3 installation when choosing to install from "deb" instead of "snap" (while using the playbooks from the current main branch). The playbook finishes successfully and initially all looks good, but i was not able to make the maas-dhcpd actually start.  I am currently not sure what the reason behind this is, but when choosing "snap" as installation method, everything seems to work just fine and exactly as expected (meaning dhcpd starts immediately after enabling DHCP for at least one subnet/vlan).
While debugging, i also found this related ticket:
https://discourse.maas.io/t/maas-3-3-dhcdp-process-not-running/6544

Facing the same problem, done multiple deployments with this ansible playbooks and maas-dhcpd does not start. Tried with deb deployment of version 3.4 and 3.3 with no luck.
Restarting the maas-dhcpd service and then running service maas-dhcpd status displays that the dhcpd-interfaces file is missing
○ maas-dhcpd.service - MAAS instance of ISC DHCP server for IPv4
     Loaded: loaded (/lib/systemd/system/maas-dhcpd.service; enabled; vendor preset: enabled)
     Active: inactive (dead)
  Condition: start condition failed at Thu 2024-01-25 08:44:57 UTC; 1s ago
             └─ ConditionPathExists=/var/lib/maas/dhcpd-interfaces was not met
       Docs: man:dhcpd(8)

If i create that file, then it complains that dhcpd.conf is missing. I do not want to manually write this configuration file so my troubleshooting stops there.
○ maas-dhcpd.service - MAAS instance of ISC DHCP server for IPv4
     Loaded: loaded (/lib/systemd/system/maas-dhcpd.service; enabled; vendor preset: enabled)
     Active: inactive (dead)
  Condition: start condition failed at Thu 2024-01-25 08:47:27 UTC; 1s ago
             └─ ConditionPathExists=/var/lib/maas/dhcpd.conf was not met
       Docs: man:dhcpd(8)`

Any idea where to start solving this in scope of these playbooks?

This is a potentially hacky solution, but the maas snap creates both of these files If you're willing to try copying those into the deb environment?
(I can't guarantee this will work, I've not tried it)


bytes: Examples missing for some functions

The bytes package has examples in most functions, but some are missing.
I would like to contribute with examples for the bytes package for my first contribution and get some experience.

Please go ahead, no need to ask for permission. Fine to mention this issue in the commits.

Thank you!! I thought about just going ahead, but I think it is wise to always at least talk to someone first.
I am on it.

Added tests at:
https://go-review.googlesource.com/c/go/+/58030


Requesting a new PyLadies Chapter website: Hamburg

This template should be used for those requiring a new PyLadies website.
What is the name of the PyLadies chapter requesting the website?
Hamburg
What is the custom domain desired for the PyLadies chapter website?
hamburg.pyladies.com
What are the GitHub handles of the PyLadies chapter organizers?
terezaif, neinkeinkaffee, chalendony,

Name
GitHub Handle
Slack Handle
Team Role

Tereza Iofciu
terezaif
Tereza
co-organiser

Gesa Stupperich
neinkeinkaffee
Gesa
co-organiser

Avaré Stewart
chalendony
Avaré Stewart
co-organiser

Annemarie Paul
AnMaPa
Annemarie Paul
co-organiser

Do you want your website to use one of the following templates or do you prefer it to be blank?

[x] PyLadies Netlify HTML Basic Website, example: chicago.pyladies.com
[ ] PyLadies Website served via GitHub Pages with my own design
[ ] Fork our existing PyLadies repo
Insert repo name: [Add repo name here]
[ ] I'll use my own template

Where will you host the website?

[x] GitHub Pages
[ ] Heroku
[ ] Other: [Please specify as this is needed for @pyladies/tech-and-infra-admins to setup the DNS record accordingly]

Issue checklist
For the author to complete:

[x] Completed the 5 questions above
[x] Did you notify the PyLadies Slack channel #project-tech-infra?
[x] Have you added your PyLadies chapter website as a submodule to this repository? Here's an example pull request adding Chicago to the repo.
Insert pull request: [https://github.com/pyladies/pyladies-hamburg-website/pull/1]
[x] If applicable, have you removed your old PyLadies website from main PyLadies repo?
Insert pull request: [https://github.com/pyladies/pyladies/pull/532]

For the @pyladies/tech-and-infra-admins to complete:

[x] Insert link to repo: https://github.com/pyladies/pyladies-hamburg-website
[x] Is the custom domain created?
[x] Are the organizers added to a PyLadies team with proper permissions for the repo? Insert name of the team: @pyladies/hamburg
[x] Is the GitHub team added to the repo with maintain permissions?

/cc @pyladies/tech-and-infra-admins

@terezaif I added in the option for the issue where you can specify the repo you'd like us to clone, would you mind updating? This will help others provide examples if they have the same situation 😄

@terezaif confirmed in Slack their chapter wants to use the Netlify template

Created CNAME and CODEOWNERS.  Netlify preview can be seen hamburg.pyladies.com

@terezaif to close this out can you please open a PR to add your repo as a submodule to this repo as well as link the PR that removed your old project code from the pyladies monolith? Thanks!

Can you pls add AnMaPa (Annemarie's handle) as codeowner?

@terezaif  -- Apologies for the delay.  I sent an invite to the Hamburg Organizers team to @AnMaPa -- that should give codeowner access.  Please let me know if it doesn't.

@terezaif  -- Noticed today that you haven't added your repo as a submodule to this one yet, so just a friendly reminder.  😄

@BethanyG I am really confused :smile: regarding the website.. i made a pr to pyladies/pyladies-hamburg-website .. this I have to add as a submodule? ( I'll .. google that :D )

@BethanyG done now!!

@lorenanicole @terezaif -- Just merged the Submodule PR (https://github.com/pyladies/chapter-websites/pull/19).
Unless there is anything else I've forgotten, I think we're good to close this!  I will leave it open for another day or so, and then close if nothing else comes up.

@lorenanicole  -- Forgot to merge https://github.com/pyladies/pyladies-hamburg-website/pull/1 until just now.  When you get a chance, can you walk me (or point me to instructions) through pulling in an update to the Hamburg submodule?  Should I clone down this repo and do it locally and re-push, or is there an easier GitHub only way of doing it??

Yah the update needs to be done locally.


Native Github Issue Integration

   when I am in the workbench
      I can see a "issues" slideout
        when I fill it out
            then a (labelled) issue is automatically logged to github

Issues:

authentication (need a TW GH bot?)
See implementation on GBIF

The instance of ARCTOS at Harvard implemented Bugzilla (instead of GitHub). It has a flavor very much like the way GBIF implemented it -- making it both easy to use (not intimidating), but still useful for the programmers.

What ever is chosen, it should be easy to use, and preferably from inside TW, not leaving to visit another site no matter if it's an feature request, or bug.
(Not sure if this is useful idea or not, but it's very easy to set up a google form that feeds GitHub. The Google Form could be linked inside the TW instance).

It is possible to customize the resulting labels on submission of a ticket using such a method. We did this with the Darwin Core Hour form.

Not sure if Bugzilla is even an option, but I found it much easier (intuitive) to teach people to use, to produce useful tickets, than GitHub. It looks like GBIF might have gleaned their categories from this software.

Won't fix, the link out is good enough.


Android scenario app tests with platform views + rotations sometimes display different "gray" color

For example:

I was hoping this would be resolved by https://github.com/flutter/engine/pull/51169, but it was not.
/cc @dnfield @jonahwilliams for any other ideas. A specific color to be set instead of relying on the default?

@gmackall I wonder if this is something you could potentially help with?
Other ideas welcome, but otherwise I might suggest just trying to set a specific color instead of using the default?

If this is due to an animation we can't control, like rotation, we could try forcing it to speed up with something like: https://www.androidpolice.com/2016/10/22/guide-set-custom-animation-speed-android-works-stock-devices/

Gray I'm not sure how you prefer to be tagged, I've assigned you but if it's not something you can take on reasonably soon please assign it back. Thanks!

Thanks for the tag, I should be able to take a look tomorrow!

Bumping to P1, as @jason-simmons pointed out:

There has been a recent increase in Skia Gold comparison failures in the linux_android_emulator_skia_tests builder.  6 out of the past 13 runs failed (https://ci.chromium.org/p/flutter/g/engine/console)

AFAICT this is the only test experiencing this problem (at least at this scale), and it seems entirely due to the "grey-ness" factor changing.

Capturing discussion from @matanlurey offline that neither of us is sure where the grey box comes from in the first place, so that is the first thing I will try to figure out

Capturing discussion from @matanlurey offline that neither of us is sure where the grey box comes from in the first place, so that is the first thing I will try to figure out

Doesn't directly answer this, but there is a color change on the transition between tests that seems very likely to be the culprit here. Need to figure out if/why the screenshot could be happening before we finish the transition in our out of each case.
device-2024-03-07-152047.webm

Still trying to recreate this, ran the test 100 times with seq 100 | xargs -I{} dart ./testing/scenario_app/bin/run_android_tests.dart --out-dir=../out/android_debug_unopt_arm64 --smoke-test=dev.flutter.scenariosui.PlatformTextureUiTests --logs-dir=/Users/mackall/Desktop/MultipleCases/logs_dir/run{}
and compared the output images with imagemagick after combining the logs
seq 100 | xargs -I{} magick compare -verbose -metric mae combined_logs/copy1.png combined_logs/copy{}.png diffs/diff{}rnd1.png
and didn't find any differences (ex:)
Image: combined_logs/copy5png
  Channel distortion: MAE
    red: 0 (0)
    green: 0 (0)
    blue: 0 (0)
    all: 0 (0)

I wasn't using the emulator pulled in by gclient though, I'm going to try the above again with the exact same emulator in case that makes a difference here

Looks like there is a --no-window-animation flag we can pass, which gets rid of the opening and closing animations. But I still see a moment of different color at the end of tests.

Can we set all of the animation scale values from https://github.com/flutter/flutter/issues/144657#issuecomment-1980008823 down to zero in the run script?

Ok I believe I've fixed this by setting the splash screen color to be transparent, see:
Uploading translucentsplash.webm…

Woot!

Ok I believe I've fixed this by setting the splash screen color to be transparent, see: translucentsplash.webm

YESSSSSSSSSSS.


Docs Migration redirect URLs

Overview
Added redirect URLs for popular docs pages.

General landing pages link to the new "Welcome to MRTK" page.
Popular conceptual docs link to their corresponding URL.
URLs won't work until new docs pages go live

Changes

Fixes: # .

/azp run mrtk_docs

Should be merged when we publish MRTK 2.6

@keveleigh, is it possible for you to cherry pick this for the dev branch?

@polar-kev Definitely! Let me know wherever it needs to go


Add ctrl+s shortcut to save a paste

This only eats the ctrl+s input if the paste submit button is present in the DOM.
We also ensure that we are not capturing ctrl+alt+s inputs to avoid catching the Polish S. See https://medium.engineering/the-curious-case-of-disappearing-polish-s-fa398313d4df
Closes #148

Some further testing reveals that detecting the alt-key to not eat that input doesn't work on firefox if privacy.resistFingerprinting is enabled in about:settings. This is due to firefox not setting keydown.altKey with this privacy flag enabled.

Is that setting default-true. I saw @Kwpolska mentioned that it might block some compose-keys?

It defaults to false, and if this is how it behaves, that seems like a Firefox bug.

Perfect, then this goes in. Thank you :)


Several example appear to have incorrect Scanamo exec syntax

Several examples on the Scanamo site use the following syntax, but it doesn't compile.
Scanamo.exec(client)(operations)

On version 1.0.0-M10 of Scanamo, it appears that the correct syntax is:
Scanamo(client).exec(operations)

Thanks for this! Closing in favour of #492 , please feel free to add items if you find anything.


uninitialized constant Grape::Formatter::Base (NameError)

Running into the following error:
/opt/berkshelf-api # berks-api /opt/berkshelf-api/config.json
/usr/lib/ruby/gems/2.2.0/gems/grape-msgpack-0.1.2/lib/grape/msgpack.rb:39:in `<top (required)>': uninitialized constant Grape::Formatter::Base (NameError)
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/gems/2.2.0/gems/grape-msgpack-0.1.2/lib/grape-msgpack.rb:1:in `<top (required)>'
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api/endpoint.rb:2:in `<top (required)>'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api.rb:23:in `require_relative'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api.rb:23:in `<module:API>'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api.rb:10:in `<module:Berkshelf>'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api.rb:9:in `<top (required)>'
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api/srv_ctl.rb:72:in `start'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/lib/berkshelf/api/srv_ctl.rb:58:in `run'
	from /usr/lib/ruby/gems/2.2.0/gems/berkshelf-api-2.1.3/bin/berks-api:5:in `<top (required)>'
	from /usr/bin/berks-api:23:in `load'
	from /usr/bin/berks-api:23:in `<main>'

Gem list:
/opt/berkshelf-api # gem list

*** LOCAL GEMS ***

activesupport (4.2.6)
addressable (2.4.0)
archive (0.0.6)
axiom-types (0.1.1)
berkshelf-api (2.1.3)
bigdecimal (1.2.7)
buff-config (1.0.1)
buff-extensions (1.0.0)
buff-ignore (1.1.1)
buff-ruby_engine (0.1.0)
buff-shell_out (0.2.0)
builder (3.2.2)
celluloid (0.16.0)
celluloid-io (0.16.2)
chef-config (12.8.1)
coercible (1.0.0)
descendants_tracker (0.0.4)
diff-lcs (1.2.5)
domain_name (0.5.20160309)
equalizer (0.0.11)
erubis (2.7.0)
faraday (0.9.2)
ffi (1.9.10)
grape (0.15.0)
grape-msgpack (0.1.2)
hashie (3.4.3)
hitimes (1.2.3)
http (1.0.3)
http-cookie (1.0.2)
http-form_data (1.0.1)
http_parser.rb (0.6.0)
httpclient (2.7.1)
i18n (0.7.0)
ice_nine (0.11.2)
io-console (0.4.3)
json (1.8.3)
minitest (5.4.3)
mixlib-authentication (1.4.0)
mixlib-config (2.2.1)
mixlib-log (1.6.0)
mixlib-shellout (2.2.6)
msgpack (0.5.12)
multi_json (1.11.2)
multi_xml (0.5.5)
multipart-post (2.0.0)
nio4r (1.2.1)
octokit (4.3.0)
power_assert (0.2.2)
psych (2.0.8)
rack (1.6.4)
rack-accept (0.4.5)
rack-mount (0.8.3)
reel (0.6.1)
retryable (2.0.3)
ridley (4.5.0)
rspec-core (3.4.4)
rspec-expectations (3.4.0)
rspec-mocks (3.4.1)
rspec-support (3.4.1)
sawyer (0.7.0)
semverse (1.2.1)
test-unit (3.0.8)
thread_safe (0.3.5)
timers (4.0.4)
tzinfo (1.2.2)
unf (0.1.4)
unf_ext (<IP_ADDRESS>)
varia_model (0.4.1)
virtus (1.0.5)
websocket-driver (0.6.3)
websocket-extensions (0.1.2)

This is in Docker, so fresh gems etc:
RUN gem install -N berkshelf-api bigdecimal

I'm actually going to close here as using grape 0.14.0 fixes the issue and redirect this to grape project.


Indentation applying wrong for PSR2

I am using php7.
php-cs-fixer is PHP CS Fixer 2.3.1 by Fabien Potencier and Dariusz Ruminski
This is before format code
 public function findByTagAndBlog(Tag $tag, Blog $blog)
    {
        $query = $this->createQuery();

        return $query->matching($query->logicalAnd($query->equals('blog', $blog), $query->contains('tags',
            $tag)))->setOrderings(array('date' => QueryInterface::ORDER_DESCENDING))->execute();
    }

After I use php-cs-fixer by this command php-cs-fixer fix /path/to/project --rules=@PSR2
 public function findByTagAndBlog(Tag $tag, Blog $blog)
    {
        $query = $this->createQuery();
        return $query->matching(
                $query->logicalAnd(
                    $query->equals('blog', $blog),
                    $query->contains('tags', $tag)
                )
            )
            ->setOrderings(array('date' => QueryInterface::ORDER_DESCENDING))
            ->execute();
    }

actually, after reformat code It is indentation 8 spaces not for 4 spaces

body method was indented for {. probably we don't have feature to normalize {

@keradus not for {
it has problem as code below
 $query = $this->createQuery();
        return $query->matching(
                $query->logicalAnd(
                    $query->equals('blog', $blog),
                    $query->contains('tags', $tag)
                )
            )
            ->setOrderings(array('date' => QueryInterface::ORDER_DESCENDING))
            ->execute();

now, you have provided 3rd version of code... no idea now what is the outcome you got...

I think my code should display like this after fix from php-cs-fixer
 public function findByCategoryAndBlog(Category $category, Blog $blog)
    {
        $query = $this->createQuery();
        return $query->matching(
            $query->logicalAnd(
                $query->equals('blog', $blog),
                $query->equals('category', $category)
            )
        )
        ->setOrderings(array('date' => QueryInterface::ORDER_DESCENDING))
        ->execute();
    }

I couldn't reproduce this, I assume it has been fixed since its report so I'm closing. If it still happens, please submit a new issue.


The ClickHouse Data Processor’s write position does not revert to its original state after an exception is triggered by the addBatch() method.

Error: The ClickHouseDataProcessor doesn't reset the write position after encountering exceptions during addBatch(). This leads to subsequent batches using the wrong position and failing with serialization errors.
Scenario:

Calling addBatch() with invalid data (null for non-nullable or wrong data type) throws a SqlException.
The write position remains at the previous attempt (not reset to 0).
Adding the next batch with the incorrect position causes a serializer exception due to mismatched data and serialization method.

Question:

How can we reset the write position of ClickHouseDataProcessor after handling addBatch() exceptions?
Alternatively, is there a way to add subsequent batches without relying on the potentially incorrect write position?

I was debugging and saw the wrong value here:

I tried using the clearParameters() method in my catch block but it hasn't got any effect.

The same issue has been resolved #1373 .


the tracker don´t work

Hello, I hope someone can help me, for some reason the program no longer sends information to the page and no data is updated, can someone help me?
The last day it worked and I have some record was July 30

Hi! Please, check the latest update. The version of tracker should be 1.83. Also you can use full reset button, input your token again and start tracking.

It is something quite strange what happens, it does not register absolutely anything apart from the rank, I have already tried everything else, some bug?

So, do you still have problems? Please, add screenshots that will image your issue (especially important screenshot of your open tracker)

since jul 30, noting its updated

but all its runing

Thank you for your screen. Did you try full reset button? When you did that, what happened?

Also you might need to set up date format in app settings, because your date format seems to be non-standard.

well with the full reset nothing happens, just enter the token again and do not register anything, although with date format I have not tried anything

although previously I did not make any movement and it worked correctly

although previously I did not make any movement and it worked correctly

So, you can try to set up data settings byself.  In tracker settings you should choose Data Format and according to instruction (https://mtgarena.pro/date-setting-manual/) add correct date settings byself.

Bingo, thx


Create directories on install

$(DESTDIR)$(BINDIR) may not exist.
I ran into this while making an ebuild for Gentoo. The package is first installed into a clean DESTDIR before it is copied to the system.

@tastytea That looks good! I unfortunately am not able to test the code because currently do not have access to a linux machine, but from looking at the changes you have made, it seems like it works fine :)
Will merge right away!


I don't want to change the original src for srcset. Is better way to deal with it?

Well,
I'm using picturefill, but I would like to keep the original src attribute to have a semantic html. If i juste make something like this before the polyfill starts(I just made with jQuery for example purposes, we could easily do with querySelector too):
$('img').each(function(){
var source = $(this).attr('src');
$(this).attr('srcset', source);
$(this).removeAttr('src');
});
This way it would fire double download?

Hey there @alexanmtz,
Thanks for getting in touch! This probably isn't the best way to approach things, though. If semantic purity is your main concern, this document will become invalid once that JS has got a hold of it. As well, we recommend loading Picturefill asynchronously as soon as possible. If you do this, there's a chance of Picturefill finishing execution before the document tree is ready to be parsed (document.ready()), and since your proposed script here will execute before Picturefill, there's a chance your document tree won't be ready, which means it'll apply this to maybe some images, but not all.
So all things considered, this is brittle. There're a couple of solid ways you can solve this, though, so fear ye not! Something to note beforehand: srcset support is actually getting pretty solid. (85% support in Brazil right now 💪🏼) — although, of course, your actual target market may vary. But it's a good baseline!
If semantic purity and strict adherence to the spec is your main concern, these are your options:
You can either provide a normal src with a valid image source in addition to the srcset, and risk a double download in the 15% of browsers lacking native srcset support
Alternatively, my favourite solution is to provide an empty placeholder gif inline in your src attribute:
<img src="data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw==" srcset="rad-wolf.jpg 200w, uber-rad-wolf-hd-retina.jpg 400w" alt="A rad wolf">
This is valid HTML and won't fail any validation. Unlike empty sources (src="")or invalid inline Base64s (src=data:image/gif;base64,WHARRGARBL, this also won't throw any weird browser errors, as the previous inline base64 gif is a very small transparent image.
This works for:

srcset-supporting browsers (85%)
non-srcset supporting browsers whose JS-execution succeeded (14.835% (~98.9% of 15% (source)))

The remaining 0.165% of your visits will be stuck with the alt text.
Hope that helps, my dude, but if you have any further questions please do reply back! Have a rad day. 💯

Thanks for all the clear information @jegtnes. The jQuery code it was just a illustration of a more accurate and general solution to accomplish a Picturefill Polyfill "pure". The main message it was: If I just change the src attribute to sourceset via js it could avoid the double download and make a more unobstrusive picturefill? With a positive answer of this, I would thing a better way to integrate this inside de picturefill code. But I imagine that this couldn't avoid the double download, because the image source would be downloaded as the document loads and before change the attribute(that's what I suppose to).
Keep the source and sourceset in image attribute it would be a really great solution. And I appreciate and thank you for this advice. Thats the one that I will use.
The semantic is really a concern to me and this feature (use of image tag with source, as it is) would make picturefill a better polyfill. Use a empty gif I don't think a good solution because the original image is a empty gif and the original image it should be the one.
I'm not a fascist semantic guy :p . It's just that I think there's benefits to make the HTML "original", that JS is a layer of enhancement. For example, in a project that I was working of price comparison, I should scrap to find prices in other sites, and one of them return instead of the original images a loading replacement because they use the loading image as the source. I think the page have to keep the original structure and not to be so JS dependent. But for this is depending on the case.

Hey again!
I totally understand the point of using JS and polyfills entirely as a progressively enhanced layer. Generally, it's more reliable and robust. That being said, I'm not 100% sure about the technical feasibility of doing this kind of source swapping, though, I haven't worked extensively on Picturefill core myself (but a lot of very smart people have).
I'm not sure I understand your use case, though, so let me try to get to the bottom of this.
There are three distinct and different use cases for responsive images, but let's look at the two relevant to us here:

To serve the browser an identical-looking image at various different resolutions so you don't need to download an image much bigger or much smaller than you need. This is the equivalent of having smaller meal portions for children at a restaurant (except instead of asking you what you want, the water will automatically assume your children want the smaller meals by default (this is possibly a terrible analogy)). For this, you use the srcset and sizes attributes on an img, and the browser does some guess work to assume which version is right. No picture necessary! :bomb:
To serve the browser a different version of the image entirely at different viewport widths. These could be different dimensions, different sizes, or possibly different images entirely. This is called art direction. For this, you use the picture element, combined with one to several source elements with the srcset attribute (if you try to apply src to a source inside a picture, it will be ignored) and one img tag. The reason we apply the srcset to the img tag is as a fallback so that browsers that don't understand source elements but understand srcset (like Android 2.3) can still benefit from some responsiveness. Progressive enhancement! ✨

More information here: http://alistapart.com/article/responsive-images-in-practice
In either case, if you are using picture and want to replace the src with a srcset with JS as you described earlier: If there's only one image source in srcset (as there would be if you renamed src to srcset), I don't think you need to use srcset at all, as good old src will serve the same purpose and be universally understood. If there are multiple images in srcset alongside the src, these will be overridden by the src -> srcset replacement you suggested, so you may be doing more harm than good.
As I said, I'm not 100% sure about your use case, so if I got something wrong or this doesn't clear things up, let me know!

Hi @jegtnes,
Thanks again for all clarification. I guess now I have a overall understanding of all use cases. I'm now using picture, with a image src attribute to works clean and keep the semantic and the srcset attribute in the same img and I'm getting great results. I will try the solution 1! (And the analogy it was not so terrible, but even fun :-)

Hey dude! No worries :) Glad you managed to figure things out. We're here if you need anything else. ⚡


Support Hangul syllables when searching some text (bug 1771477)

it aims to fix https://bugzilla.mozilla.org/show_bug.cgi?id=1771477;
hangul contains some syllables which are decomposed when using NFD, hence
the text must be correctly shifted in case it contains some of them.

/botio unittest

/botio integrationtest

/botio unittest

/botio integrationtest


Wildcards not supported?

Several search engine bots support wildcards.
I've checked this for the following robots.txt files, and noticed that robots-txt-parser seems like to ignore this for the following robots txt files:
https://www.zalando.nl/robots.txt
https://www.bcc.nl/robots.txt
But supports it for the following robots txt file:
https://www.karwei.nl/robots.txt
Hope you could do a check to see what is causing this behaviour.

Thanks for bringing this to my attention, I'm looking into it now and I should hopefully have a fix out over the coming days.

Hey keyndark, I managed to get tests running that confirmed the issue but I haven't managed to fix one edge case. I'm aiming to have a fix done for Friday this week but that timeline may change.


given labeling matrix + ground truth, helper function to compute accuracies + coverage

So that these steps don't need to be done by hand, especially when comparing to the generative model stats, which can be generated via gen_model.learned_lf_stats()

L_train.lf_stats(session) can generate a table of statistics about the label matrix (function here) with example in intro tutorial, cell 16


mmdet bbox_nms nonzero warning

/opt/model_factory/mmdetection/mmdet/core/post_processing/bbox_nms.py:54: UserWarning: This overload of nonzero is deprecated:
nonzero()
Consider using one of the following signatures instead:
nonzero(*, bool as_tuple) (Triggered internally at  /tmp/pytorch/torch/csrc/utils/python_arg_parser.cpp:766.)
labels = valid_mask.nonzero()[:, 1]

Duplicate https://github.com/open-mmlab/mmdetection/issues/3467
labels = valid_mask.nonzero(as_tuple=False)[:, 1]


hwmv2: Port NXP RT Family

@nordicjm @tejlmand
I did consider and try using variants but found revisions was easier to implement and in my opinion made more sense. The only issue is that there is no such thing as multidimensional revions. Such as, revision A and B, and revision hyperflash and qspi.
My response to jamie above:

Not only are revisions the most convenient way to handle this in the board definition, but also, in my view, these really are different board revisions, because switching between hyperflash and qspi requires reworking the board hardware with a soldering iron. And hyperflash and qspi are not really variants of the SOC at all, they are still using the same interface to the SOC, they are just two different flash chips. The fact they are just on the same board is just because it's an EVK... these are really two different board versions that happen to exist on the same PCB.

My response to Jamie above:

Not only are revisions the most convenient way to handle this in the board definition, but also, in my view, these really are different board revisions, because switching between hyperflash and qspi requires reworking the board hardware with a soldering iron.

I can accept this as a valid reason for choosing board revision over variants.
A bit of advice to avoid future questions / wondering related to this, then I propose to add this reasoning in the revision.cmake.
But you still need to address comments like these:
https://github.com/zephyrproject-rtos/zephyr/pull/69264#discussion_r1497000683
https://github.com/zephyrproject-rtos/zephyr/pull/69264#discussion_r1497002866
I suggest you try it once more, cause the proposed pattern has worked for any other board.
Alternative push a fixup commit to this PR with the changes you did which was not working, perhaps @nordicjm or I can see what is wrong.

@tejlmand  @nordicjm I am trying to do the pattern like suggested here https://github.com/zephyrproject-rtos/zephyr/pull/69264#discussion_r1497002866 but it simply does not build for me, and I have no idea why
is this the only blocking request?

@decsny it looks like some of the RT1170 EVKB board files were missed in the rename:
$ find tests -name "mimxrt1170_evkb_cm7*"
tests/drivers/i2c/i2c_target_api/boards/mimxrt1170_evkb_cm7.overlay
tests/drivers/dma/loop_transfer/boards/mimxrt1170_evkb_cm7.overlay
tests/drivers/dma/loop_transfer/boards/mimxrt1170_evkb_cm7.conf
tests/drivers/mbox/mbox_data/boards/mimxrt1170_evkb_cm7.overlay
tests/drivers/mbox/mbox_data/boards/mimxrt1170_evkb_cm7.conf
tests/drivers/adc/adc_api/boards/mimxrt1170_evkb_cm7.overlay


#989/selection day timetable

bit of a smelly function to match candidates with selection day panels and assign them slots.
Returns a timetable and a list of the unassignable candidates.
see the tests too, think ive probably missed something but hopefully the function covers whatever you can throw at it

@tomlovesgithub Can this be closed or is it pending checking once the frontend is up and running?

@tomlovesgithub Can this be closed or is it pending checking once the frontend is up and running?

Lets get this closed. Any refinements can be done when the front end is up and running


python310Packages.google-cloud-bigquery-logging: 1.2.0 -> 1.2.1

Changelog: https://github.com/googleapis/python-bigquery-logging/blob/v1.2.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 224510 run on x86_64-linux 1

4 packages built:

python310Packages.google-cloud-bigquery-logging
python310Packages.google-cloud-bigquery-logging.dist
python311Packages.google-cloud-bigquery-logging
python311Packages.google-cloud-bigquery-logging.dist


feat: add support the valgrind include by triple

Add IAI_CALLGRIND_<triple>_VALGRIND_INCLUDE as an alternative to IAI_CALLGRIND_VALGRIND_INCLUDE
Partial closes #101

I'm going to finish this pr now.


Add Hawk.buildRx()

StrictMode showed that .build() was doing a lot of work on the main thread, even with the callback setup. I tried to wrap the callback with Rx, but I was having issue with Schedulers, Looper, Handler and so on.
I have just created a new method that wraps the synchronous .startbuild(), and exposes an Observable version of build().

This is a great addition, apart from the comment, could you also add test and use 2 space indentation please? Then we can merge it.

Sure! I'll add what you asked asap.

Hi Orhan
I added a few more commits. Any feedback, please?
Ivan

@hamen great! I've just added one comment, and could you also squash the commits please?

@orhanobut How about that? :)

@hamen almost there! but still need rebase for commits, 6 commits so far, can you squash them by rebasing? :)

I guess we can close this one. The new one looks cleaner :)


#581 may be a regression

@per-mathisen-arm notes in #581

First, it requires an environment variable to be present or a command line option to be used in order to use the tool, so it makes usage harder. Second, it requires the .so file and the .json file to be in the same directory, but the default 'make install' for gfxreconstruct puts them in separate directories.

So #581 may need to be reverted or patched to fix behavior

Also the .so file's name will be "libVkLayer_gfxreconstruct.so" on Linux, while the script is looking for "VkLayer_gfxreconstruct.so" and will abort if it isn't found.


add new audio transcription and translation endpoints (API version 1.2.0)

https://platform.openai.com/docs/api-reference/audio

This seems to be fun 🙈
The request has a parameter response_format which can be set to one of the following: json, text, srt, verbose_json, or vtt.
As the parameter name suggests, this results in very different return messages. Here are example messages of the various formats:
text
Hello, how are you? Fine and you? What is your name? My name is Tom.\n
json
{"text":"Hello, how are you? Fine and you? What is your name? My name is Tom."}
verbose_json
{
  "task": "transcribe",
  "language": "english",
  "duration": 11.83,
  "segments": [
    {
      "id": 0,
      "seek": 0,
      "start": 0.0,
      "end": 12.32,
      "text": " Hello, how are you? Fine and you? What is your name? My name is Tom.",
      "tokens": [
        50364,
        ...,
        50980
      ],
      "temperature": 0.0,
      "avg_logprob": -0.3095785223919412,
      "compression_ratio": 1.0625,
      "no_speech_prob": 0.031427089124917984,
      "transient": false
    }
  ],
  "text": "Hello, how are you? Fine and you? What is your name? My name is Tom."
}

vtt
WEBVTT\n
\n
00:00:00.000 --> 00:00:12.320\n
Hello, how are you? Fine and you? What is your name? My name is Tom.\n
\n

srt
1\n
00:00:00,000 --> 00:00:12,320\n
Hello, how are you? Fine and you? What is your name? My name is Tom.\n
\n
\n

I am not an audio expert but I guess there is a good reason to have different out formats. But in a REST way this is quiet weird, as it breaks with every REST standard.
Right now I am not sure how to handle this properly. Currently I have the following ideas:

Always force verbose_json and return a corresponding response which has more or less the same format as all the other responses.

Let the API user control the response_format parameter and return a response corresponding to the format the user has chosen. text=> TranscriptionTextResponse; vtt=> TranscriptionVttResponse etc.

Always force verbose_json in the background (similar to idea 1) but provide on the response object methods for all the available formats (ex. $response->json() or $response->vtt()) which will return the response in the correct format. This would probably be the most convenient option for the package users, but we have to rebuild the format conversion in our package and can not make use of the API functionality which is some kind of an overhead and maybe lead to more errors.

@nunomaduro What do you think which option to prefer? Or do you have another idea?
Spontaneously I would go for option 2 even if it's a bit weird to have multiple possible response classes.

@gehrisandro any ETA when it will done?

@NumairAwan No ETA. Keep in mind, we work on this in our free time.

IMHO 2 seems the way to go. I can see a lot of valid reasons to use the other response formats, so enforcing JSON doesn't seem good to me.

While implementing I took a little different approach:
$client->audio()->transcribe([...]) will always return a TranscriptionResponse regardless which response_format was chosen.
The returned value from the API is then always stored within the text property. All the other properties task, language, etc. are only filled if response_type = verbose_json. Otherwise they are all null.
@mpociot @nunomaduro What do you think?

By the way translate() is an exact copy of transcribe() because both endpoints currently behave in the exact same way but I guess sooner or later these two endpoints will divert and therefore I created two different response classes even if it is not necessary right now.

@gehrisandro i was also implementing audio endpoint but faced the same issue, i was needed to integrate whisper in project but i guess i need to wait for the implementation.

released: https://twitter.com/gehrisandro/status/1631554420558233601


raftstore: A new format for data to replace RaftCmdRequest

What problem does this PR solve?
Issue Number: close https://github.com/tikv/tikv/issues/7994
Problem Summary:
What is changed and how it works?
Proposal: #7994
The cost of decode and encode for protobuf is too much so I hope a better one format for RaftCmdRequest so that we can save more CPU of apply, and decode apply cmd faster. See more details in #7994
Related changes
Do not merge this PR before https://github.com/tikv/rust-rocksdb/pull/511
Check List
Tests

Unit test
Integration test

Release note

A faster format for Entry, and this feature will disable when upgrade for compatibility

/bench

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@BusyJay Any comment?

@BusyJay Any comment?


How Can I Select MFA Method?

Can't figure out how to act at this point:

How do I select the option, or what do I specify here?

When I specify e.g. 0, it just fails


Expose IPAM state via REST

Expose IPAM state:
{
  nodeId,
  nodeIP,
  nodeName,
  podNetwork,
  vppHostNetwork
}

via HTPP REST interface (endpoint is GET /ipam).
Example output:
$ http <IP_ADDRESS>:9999/ipam
HTTP/1.1 200 OK
Content-Length: 158
Content-Type: application/json; charset=UTF-8
Date: Fri, 17 Aug 2018 11:07:53 GMT

{
    "nodeIP": "<IP_ADDRESS>",
    "nodeId": 1,
    "nodeName": "vagrant-arch.vagrantup.com",
    "podNetwork": "<IP_ADDRESS>/24",
    "vppHostNetwork": "<IP_ADDRESS>/24"
}

Signed-off-by: Tomas Slusny<EMAIL_ADDRESS>

Coverage decreased (-0.05%) to 76.214% when pulling a61104d556fb6b66505c65d498014d91b4e3aff2 on deathbeam:add-dump-of-ipam into 68ebcacb4e5228a85b6c1e96c6eefd57aec6c01c on contiv:master.

This is a good start, but the URL should be versioned, i.e. not just ipam, but something like contiv/v1/ipam. We have those kinds of URLs for vpp dumps, we should keep the same structure. Please talk to Ondrej Fabry about what exactly it should be.


Query input source for reading local trees. Ref T53270.

This implementation can be used to use a query as an input source when the tree is within the same job. It is intended for pipeline jobs.

I think I've incorporated all the feedback from the first round but I haven't yet addressed the query engine single source. I created the abstract class AbstractStreamDataSource that is now shared by AbstractStreamFileDataSource and DataSourceQuery.  Make comments on the new version and I'll include those changes in the next commit and a fix for query engine single source.

Closing in favor of more general query input source pull request.


(Android): App Refresh rate keeps lowering to 24

Approximately once every few hours when I open Bluebubbles, the in-app refresh rate lowers itself to 24 and is extremely choppy.
I have to open appearance settings, close it, and then re-open it in order to view the Refresh rate settings and set it back to Auto/120
FYI I am using a Galaxy S22 Ultra and the newest version of the BlueBubbles app.

This will be fixed in the rewrite which we are close to completing. We will be releasing an alpha in the Discord soonish. Probably in the next couple weeks. It should greatly increase performance and frames

Good to know, absolutely love this whole project so thank you for all of your work lol.


Allow creating Send and Call mediators with endpoint specified as None

It should be able to create Call and Send mediators with endpoint being None as per the documentation.
This is not supported in the VsCode now.

The latest release v1.0.5 has resolved the issue. Please reopen if the problem persists.


can't download gdcvault video

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 that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.04.11. 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 2017.04.11

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

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

youtube-dl.exe -v http://www.gdcvault.com/play/1435/An-American-engine-in-Tokyo
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://www.gdcvault.com/play/1435/An-American-engine-in-Tokyo']
[debug] Encodings: locale cp936, fs mbcs, out cp936, pref cp936
[debug] youtube-dl version 2017.04.11
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: none
[debug] Proxy map: {}
[GDCVault] An-American-engine-in-Tokyo: Downloading webpage
WARNING: [GDCVault] It looks like http://www.gdcvault.com/play/1435 requires a login. Try specifying a username and password and try again.
WARNING: [GDCVault] Could not login.
ERROR: no suitable InfoExtractor for URL None/xml/9350_1238021887562UHXB.xml
File "main.py", line 19, in
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl_init_.py", line 464, in main
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl_init_.py", line 454, in _real_main
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 1890, in download
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 772, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 838, in process_ie_result
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 795, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 589, in report_error
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpsqdjcx2e\build\youtube_dl\YoutubeDL.py", line 551, in trouble

Description of your issue, suggested solution and other information
this video can be viewed in chrom without login.

Thanks for the report. This will be fixed in the next version.


Incorrect Warning Pylance (VSCode)

Description
Basically VSCode is telling me that an specific line has an error, but then I run the program and it does exactly what I wanted it to do.
Code example to reproduce the issue:
# This appbar is going to be used in multiple pages, so I can just declare it once
appbar = ft.AppBar(title=ft.Text(value="A very cool program thanks to Flet!"), bgcolor="blue")

appbar.title.value = "Such a wonderful GUI!"

Describe the results you received:
Warning from VSCode (Sorry for any possible typos, I have my editor in Spanish and I translated the message manually):
Cannot assign the member "value" for the type "Control"
  The member "value" is unknown Pylance[reportGeneralTypeIssues](https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportGeneralTypeIssues)

Describe the results you expected:
No warning I guess, since the program perfectly works
Additional information you deem important (e.g. issue happens only occasionally):
I think it could not be an error itself from Flet, but something very specific than occurs when you do something strange like acceding a sub-sub-attribute. I wanted to report it anyway, since even on that case, I'm pretty sure that I will receive a very good explanation.
Flet version (pip show flet):
Name: flet
Version: 0.11.0
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: /home/samuel/Workspace/flet-windows-optimizer/.venv/lib/python3.10/site-packages
Requires: copier, flet-runtime, packaging, qrcode, watchdog, websocket-client, websockets
Required-by:

Operating system:
Pop!_OS (Linux distro based on Ubuntu) 22.04 LTS.
Additional environment details:
I have typing enabled, and I can bypass that warning by writing an in-line comment like # type: ignore.
Have a nice day!

So, appbar.value could be either Control or None and Control could be any control inherited from it, Text in your case. I guess Pylance doesn't handle generic collections/properties well.
However, you can "help" Pylance with an assertion:
appbar = ft.AppBar(title=ft.Text(value="A very cool program thanks to Flet!"), bgcolor="blue")

assert isinstance(appbar.title, ft.Text)
appbar.title.value = "Such a wonderful GUI!"

and warning goes away! :)
If someone knows a better way to deal with that please share your ideas!


New version: XWolfOverride.DuckDNS version 2.2b

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Verify:

Hi @KaranKad,
Does this publisher frequently mix letters and numbers in the version number?

2.2b is the only version where this publisher has mixed letters and numbers.


Trying to Install Embark Addon v1.7.0 to Blender 2.90.1

Describe the bug
A clear and concise description of what the bug is.
After installing the addon, when I try to click and Enable v1.7.0 of the addon, I receive the following error:
Traceback (most recent call last):
File "C:\Program Files\Blender Foundation\Blender 2.90\2.90\scripts\modules\addon_utils.py", line 382, in enable
mod.register()
File "C:\Users\evans\AppData\Roaming\Blender Foundation\Blender\2.90\scripts\addons\blender-tools-main_init_.py", line 37, in register
register_recursive(REGISTER_CLASSES)
File "C:\Users\evans\AppData\Roaming\Blender Foundation\Blender\2.90\scripts\addons\blender-tools-main\utils_init_.py", line 60, in register_recursive
register_recursive(obj.REGISTER_CLASSES)
File "C:\Users\evans\AppData\Roaming\Blender Foundation\Blender\2.90\scripts\addons\blender-tools-main\utils_init_.py", line 56, in register_recursive
bpy.utils.register_class(obj)
ValueError: register_class(...): already registered as a subclass 'EmbarkAddonPreferences'
To Reproduce
Steps to reproduce the behavior:

Go to Addons
Click Install, and select from disk the latest 1.7.0 zip file of the Embark Addon
Click Enable checkbox for addon
See error

Expected behavior
Addon to install with no error message.
Screenshots

Device:

Windows 10
Blender Version 2.90.1

Additional context
n/a

@firebert85 Hmm, did you already have a version of the plugin installed before trying to install this one?
It looks like it's trying to register the same thing twice. can you check your %APPDATA%\Blender Foundation\Blender\2.90\scripts\addons folder (paste that path into Windows Explorer) and make sure you don't already have an old version of the plugin installed there?
If there is already a embark_blender_tools folder in your addons folder, you'll need to delete that before installing the new one.

@moppius
Trying this again on the new 2.91 release, and still get the attached error message every time I try to click the Activate check box
This happens no matter how many times I remove the addon from within Blender and make sure the scripts/addons folder in 2.91 folder is deleted.

This is a new one--I  just got this error message after another complete clean remove/delete from the //scripts/addons folder

Weirdly, I did manage to reproduce this once the other day - I think there is a race condition in registering modules and the update operator, I have a potential fix for it that I'm testing locally.
Thanks for following up!

Closing this for now, will see if anyone else hits it after the speculative fix in release 1.7.1.


add chunksize to vchunk interface

when implementing fixed pg size, we need to know the exact size of a real chunk. add this interface to support this function.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 66.92%. Comparing base (1a0cef8) to head (0bbf523).
Report is 79 commits behind head on master.

Files with missing lines
Patch %
Lines

src/lib/device/vchunk.cpp
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #572       +/-   ##
===========================================
+ Coverage   56.51%   66.92%   +10.41%
===========================================
  Files         108      109        +1
  Lines       10300    10591      +291
  Branches     1402     1432       +30
===========================================
+ Hits         5821     7088     +1267
+ Misses       3894     2823     -1071
- Partials      585      680       +95

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

can we targeting all fix_pg_size change to a branch and tested ?

can we targeting all fix_pg_size change to a branch and tested ?

ATM, actually, I am not quite sure how many changes will we exactly need for fix pg size. so I add this small step here, so that it will not block the initial development of fixed pg size feature.


findDOMNode is deprecated in StrictMode

The Table component from the react-virtualized package is throwing a warning:
Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of Grid which is inside StrictMode. Instead, add a ref directly to the element you want to reference.
I'm also not sure if this warning is preventing the content of my table from rendering or not as I haven't figured that out yet.

I am also running into this issue and I'm not sure why. It doesn't seem related to the documentation I've seen revolving passing registerChild as a ref to my div that's the child of CellMeasurer. My table code looks like the following:

type LogsTableViewProps = {
  data: LogsData[]
}

export default function LogsTableView({ data }: LogsTableViewProps): ReactNode {
  const [cache] = useState(
    new CellMeasurerCache({
      fixedWidth: true,
      minHeight: 35,
    })
  )

  const rowGetter = ({ index }: { index: number }): LogsData => data[index]
  const columnCellRenderer = (cellData: TableCellProps) => {
    const { dataKey, parent, columnIndex, rowIndex } = cellData

    const log = data[rowIndex]
    const content = log[dataKey as keyof typeof log]

    return (
      <CellMeasurer
        cache={cache}
        columnIndex={columnIndex}
        key={dataKey}
        parent={parent}
        rowIndex={rowIndex}
      >

{content}

      </CellMeasurer>
    )
  }

  return (
    <AutoSizer>
      {({ width, height }: { width: number; height: number }) => (

          <Column
            label="Date/Time"
            dataKey="time"
            width={250}
            cellRenderer={columnCellRenderer}
          />
          <Column
            label="Message"
            dataKey="message"
            width={1000}
            cellRenderer={columnCellRenderer}
          />

      )}
    </AutoSizer>
  )
}

+1, having the same error in my console. Everything seems to work though.

+1, having the same error in my console. Everything seems to work though.

Is there any way to fix this ?

Is there any way to fix this ?

Beste bvaughn/react-virtualized,
Wij hebben je e-mail ontvangen en geregistreerd onder ticket nummer #101878765.
Binnen 1 werkdag ontvang je een reactie.
Voor een snellere reactie kan je ons ook bereiken via WhatsApp op nummer +31 85 001 3385.Heb je een urgent probleem, dan zijn wij tijdens kantooruren telefonisch bereikbaar op +31 85 001 3385.
Met vriendelijke groet,
Team VIVITION

Pascalstraat 18H
6716AZ Ede
The Netherlands
T: +31(0) 85 001 3385
Website: www.VIVITION.com
Email<EMAIL_ADDRESS>

Beste bvaughn/react-virtualized,
Wij hebben je e-mail ontvangen en geregistreerd onder ticket nummer #101878765.
Binnen 1 werkdag ontvang je een reactie.
Voor een snellere reactie kan je ons ook bereiken via WhatsApp op nummer +31 85 001 3385.Heb je een urgent probleem, dan zijn wij tijdens kantooruren telefonisch bereikbaar op +31 85 001 3385.
Met vriendelijke groet,
Team VIVITION

Pascalstraat 18H
6716AZ Ede
The Netherlands
T: +31(0) 85 001 3385
Website: www.VIVITION.com
Email<EMAIL_ADDRESS>

+1. Having the same problem when use Table with AutoSizer

Still having the problem when using WindowScroller in 9.22.3

related to issues/1353

Any updates on this issue?

The warning will cause header and body async in table resizing.
It sync  by scrolling table.
https://user-images.githubusercontent.com/8370453/121707310-638d0180-cb11-11eb-9ad5-29b04a032033.mov

Hey guys, having the same problem using 9.22.3. Do we have any update of this issue?

As for
 — it is not related to AutoSizer, the problem is actually here — https://github.com/bvaughn/react-virtualized/blob/e360d958b99fa965fd3f1caed4506403ad652cd3/source/Table/Table.js#L342
Waiting contributors to fix this - #1353

A simple workaround would be const Grid = this.Grid._scrollingContainer;

Has there been any updates on this? I see both the console error and the issue mentioned above where elements don't render. See https://codesandbox.io/s/focused-hugle-mhegpz?file=/src/App.js for an example (there's 100 list items, but only 16 are getting rendered)

I have the same issue. Any updates?

I solved the error showed in console, though I had no issues with displaying the table even with it.
I was using the CellMeasurer along with the List component.
I found the way to fix it by going through the source code, cellMeasurer uses findDomNode() if no child is registered, so we just have to pass the registerChild prop as ref to the child.
eg.
<CellMeasurer
  cache={cache}
  columnIndex={columnIndex}
  key={dataKey}
  parent={parent}
  rowIndex={rowIndex}
>
  {({ registerChild }) => (

{content}

  ))}
</CellMeasurer>

I solved the error showed in console, though I had no issues with displaying the table even with it. I was using the CellMeasurer along with the List component.
I found the way to fix it by going through the source code, cellMeasurer uses findDomNode() if no child is registered, so we just have to pass the registerChild prop as ref to the child.
eg.
<CellMeasurer
  cache={cache}
  columnIndex={columnIndex}
  key={dataKey}
  parent={parent}
  rowIndex={rowIndex}
>
  {({ registerChild }) => (

{content}

  ))}
</CellMeasurer>

The types for this are incompatilbe when you are using typescript however, which leads me to believe something is wrong?

@PerfectionVR it's not wrong, the library is justing using really old react dependencies IMO.

I opened #1787 to fix this in WindowScroller and Table but I believe the author no longer maintains this lib or monitors PRs.

I solved the error showed in console, though I had no issues with displaying the table even with it. I was using the CellMeasurer along with the List component.
I found the way to fix it by going through the source code, cellMeasurer uses findDomNode() if no child is registered, so we just have to pass the registerChild prop as ref to the child.
eg.
<CellMeasurer
  cache={cache}
  columnIndex={columnIndex}
  key={dataKey}
  parent={parent}
  rowIndex={rowIndex}
>
  {({ registerChild }) => (

{content}

  ))}
</CellMeasurer>

I tried this - the warning is gone but the list render is super laggy now...

You can try this, worked for me
<CellMeasurer cache={cache} columnIndex={0} key={key} rowIndex={index} parent={parent}>
        {({ registerChild }) => (

 {
              if (element && registerChild) {
                registerChild(element);
              }
            }}
          >
            // Your content here ...

        )}
      </CellMeasurer>