Validation for ngMaxlength and ngMinlength is not working on TinyMCE input with SCE enabled

<textarea ng-model="myMCEContent" ui-tinymce ng-maxlength="180" name="body"></textarea>

I have created this <textarea> but ngMaxlength and ngMinlength are not working. For example when I typed just a single character the textarea became invalid. Then I've found that the directives work if I disable Strict Contextual Escaping. However, this is strongly discouraged.
Example in jsFiddle: https://jsfiddle.net/naXa/zsnLupvd/ (try to enter some text in textarea to see an error message, then enable $sceProvider, rerun, clear all text, and try typing again)
I use TinyMCE  and AngularUI wrapper for TinyMCE.
Why are ngMaxlength and ngMinlength not working in this case? And how to fix it?

Closing this as a duplicate of #145.
If you have any solutions please feel free to put out a PR. I will continue looking into it.


Base of project

Added several features:

a webpage to subscribe to the service (and unsubscribe)
a sample email the service should send

I think we can close this Pull Request because, as we discussed, it tries to address too many things at once. You should start with something more focused and adhere to the principle of single responsibility.


rustup; increase slack for timing tests

@bors r+

:pushpin: Commit ad07b2bd4479f75ae58b0f9aeaae1c951c3ecc92 has been approved by RalfJung

:hourglass: Testing commit ad07b2bd4479f75ae58b0f9aeaae1c951c3ecc92 with merge ce31a12f36e55922ab195822871a677b16dab333...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing ce31a12f36e55922ab195822871a677b16dab333 to master...


Change GTLTouchStaticLib target skip install setting to YES

What steps will reproduce the problem?
1.
Add GTL.xcodeproj to your project
2.
add GTLTouchStaticLib to your project's "target dependencies" build phase + add
libGTLTouchStaticLib.a to "link binary with libraries" build phase.
3.
archive your app for ad hoc testing.

What is the expected output? What do you see instead?

You get Generic Xcode Archive rather than the normal iOS App Archive. If you
change "skip install" setting on GTLTouchStaticLib target to YES, you get
proper iOS app archive.

What version of the product are you using? On what operating system?

6-Dec-2012, but this setting is wrong also on current version.

Please provide any additional information below.

You can see this issue in action with current ShareKit version
(https://github.com/sharekit/sharekit)

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 17 Apr 2013 at 12:29

No longer providing/suggesting a static lib.  Closing.


Add class to calculate next federal reserve effective date

Would this be useful to include in the ach gem? It adds a runtime dependency, but it's probably easy enough to make that optional. I'd also probably want to add some info about this to the README.
Background: The bank that handles our ACH files requires that the effective date of an ACH batch be the next business day for the Fed, or if today is not a business day, then the business day after that. They say this is common, but I don't really know how common it is. I was surprised there wasn't anything out there to do this, so I found myself rolling my own.
I also added Travis CI support, so you'd just have to flip it on at www.travis-ci.org.

I don't see any problem with adding the holidays gem as a dependency. I do think this is a useful addition to the gem. Thanks!


Module not found: Can't resolve '@emotion/react' in dual route (page,app)

Steps to reproduce
Link to live example: (required)
Steps:

make app router
2.make page router
3.set app CacheProvider

Current behavior
it doesn't work in mui when i set up page router and app router
i made a sample in my repo https://github.com/YoonIN/mui-next-route
what can i do for it?

Expected behavior
No response
Context
No response
Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: app router, page router

I clone your repo and the only thing missing is @emotion/server when I enter /test route.
Can you remove the folder node_modules, install @emotion/server and run dev again? It works on my end.

I clone your repo and the only thing missing is @emotion/server when I enter /test route.
Can you remove the folder node_modules, install @emotion/server and run dev again? It works on my end.

Oh! It's working now, thank you!

@siriwatknp
This works well in mutirepo, but does not work in monorepo configured with yarn workspace. Could you please take a look?
this is sample of yarn-workpsace sample (https://github.com/YoonIN/next-monorepo-mui)


ECONNRESET fix proposal for Darwin.

forc+exec is somewhat a bit buggy under Darwin, prefering posix_spawn
here instead.

Can you expand on what's happening and why this change fixes it? Makes me suspicious to see that fork/exec doesn't work.

I noted I got ECONNRESET errors with connections for read operations at least leading to abort. fork under Mac is known to be not fully working as expected/being buggy unless putting quite some workaround while posix_spawn arrange the situation in this area.

Almost all of the testapp.c tests are redundant, I'm pretty sure: https://github.com/memcached/memcached/issues/674 - might want to do this in favor of further hacking it up. Would be nice if testapp pivoted to actual unit tests and we kept the integration/end to end tests all in perl.

I was fiddling with a Darwin machine over the weekend and didn't get this to happen; is there anything I need to do to reproduce?

Do you get at least the abort cases ?

paste an example? What're your ./configure arguments? Test suite seems to pass without any errors for me.

This is on catalina and I haven't tried all of the configure args yet.

ah right because you re testing next branch maybe ? indeed it seems works better.

Ah yeah; I'm on next. so I guess that other fix fixes... this one too? Or I can't repro anymore.

likely yes :)

Closing this out. We can re-open and re-try if we have a .. re-production again :)
Thanks for the submission, as always!


Can't seem to figure out xml2lua usage

I'm attempting to parse an XML coming in via a POST call in nginx with.  I created a simple lua script that ngnix will use
local xml2lua = require("xml2lua")
local handler = require("xmlhandler.tree")

local function process()
  ngx.req.read_body()
  local req_body = ngx.req.get_body_data()
  ngx.log(ngx.STDERR, "Type of body " .. type(req_body))
  ngx.log(ngx.STDERR, "the body" .. req_body)

  local thehandler = handler:new()
  local theparser = xml2lua.parser(thehandler)
  theparser:parse(req_body)
  ngx.say(xml2lua.printable(thehandler.root))
  for i, p in pairs(thehandler.root.top) do
    -- print("table size " .. table.maxn(p))
    ngx.say(i .. " Test " .. p.inner)
  end
end

return process

Then I post some simple xml
<top> 	<inner stuff="val1">Data</inner> 	<inner stuff="val2">MoreData</inner> 	<inner stuff="val3">SomeMoreData</inner> </top>
I can print the xml body to the nginx logs but I get this error with regards to the lua table from the xml.
<IP_ADDRESS>, server: , request: "POST / HTTP/1.1", host: "<IP_ADDRESS>:8080" 2020/08/06 17:42:20 [] 8155#8155: *4 [lua] osphandler.lua:8: the body<top>         <inner stuff="val1">Data</inner>         <inner stuff="val2">MoreData</inner>         <inner stuff="val3">SomeMoreData</inner> </top>, client: <IP_ADDRESS>, server: , request: "POST / HTTP/1.1", host: "<IP_ADDRESS>:8080" 2020/08/06 17:42:20 [error] 8155#8155: *4 lua entry thread aborted: runtime error: /home/ec2-user/work//src/osphandler.lua:16: attempt to index field 'inner' (a nil value) stack traceback: coroutine 0:         /home/ec2-user/work//src/osphandler.lua: in function </home/ec2-user/work//src/osphandler.lua:4>
I'm new to lua so forgive any obvious issues as they aren't obvious to me.

It's probably an issue with your server response, not the library.
Firstly try to use the library with a local XML file.


Add polish localisation

Looks good for me. Let's wait for answers from @jakecraige about unused keys.

@tomkowz Thanks for reviewing this! In the future, can you make sure to make comments on the Files tab of the PR, as opposed to on the commit itself? That way it's easier to see when issues have been fixed, because the comment is collapsed once the diff changes for that line.

@gfontenot np. Yeah, thanks for advice, will remember. That was my first github review.

No worries, I appreciate you putting eyes on it.

I'll look into those unused keys and get back to you.

+1 confirmation on this being valid Polish.

@tomkowz Thanks for bringing up the unused keys and reviewing this.
Thanks @tailec for the translations :+1:

Merged in 8d4cf280c39fd3184f1ed0d22f232aa2587e3525


landcover.ipynb predictions misalignment?

It seems that there might be some misalignment of imagery within the landcover example? Looking at this output, the labels don't appear to be from the same portion of the imagery.

Indeed, I think you're right.
I confirmed that the coordinate labels match, so possibly some issue in panel or hvplot. I haven't had a chance to investigate yet.
I did notice that the two plots would pan in different directions. Panning the classification horizontally would move the imagery vertically.


Document how spec and code changes are released

Description of issue or feature request:
The ROADMAP doesn't adequately cover or explain how changes are introduced to the spec and codebase.  For instance, how are backwards-incompatible changes released?  Are there stable, alpha, beta releases?  Which branch contains the latest changes and design?

The release schedule can also be captured via TAPs, similar to Python's information PEPs (e.g., PEP 569 and 373).
Informational TAPs on releases was requested by @trishankatdatadog.  Thanks!

Thank you, sir!

There's a WIP TAP to address how spec changes are released https://github.com/theupdateframework/taps/pull/107

If this TAP theupdateframework/taps#107 is merged do we need that issue, yet?

I believe that TAP and the release process both document this issue. I will close this issue, feel free to reopen if this is still a concern.


How do i write a task for a remote host?

desc('Run a bash script');
task('test', function () {
run('sh test.sh');
});
If I run dep --hosts domain.com test
i get an error
[Symfony\Component\Console\Exception\CommandNotFoundException]
Command "domain.com" is not defined.
Although domain.com is defined and works if i run dep deploy staging

try dep test --hosts="domain.com"


Dropdown list issues

Selecting a dropdown entry above a keybind will prompt for a keypress
Dropdown entries scroll from the top instead of the bottom? It's weird.

https://github.com/Darktide-Mod-Framework/Darktide-Mod-Framework/commit/99799a4a10d9bf1e6656f991faf0db15f0dcda14

I found this in crosshair remap mod, guess it's an issue of dmf?
if the dropdown shows above the text part, options are reversed
says there are options ABCDE, normally it is like:
[ Option    v ]    [ Option    v ]
 A                  B
 B  --> scroll -->  C
 C                  D

but when the dropdown expands up:
 C                  D
 B  --> scroll -->  C
 A                  B
[ Option    v ]    [ Option    v ]

the options shown are still correct, but reversed, which is annoyed


config: Create 10090.erp.yml to add a collection for ERP software (Enterprise Resource Planing)

Add collection for ERP software

This collection is too small, I think it should be merged to https://ossinsight.io/collections/business-management


setAppendix deprecated, doc's suggested alternative is a getter

Expected Behavior
Use of the deprecated setAppendix in a build.gradle file should suggest using an alternative method (in IntelliJ IDEA and other IDEs), but more importantly it should suggest a setter.

Current Behavior
setAppendix is marked as deprecated without an alternative in the IDE. The documentation says to use a getter, getArchiveAppendix:
https://docs.gradle.org/current/javadoc/org/gradle/api/tasks/bundling/AbstractArchiveTask.html#setAppendix-java.lang.String-

Context

I was trying to set the appendix for an archive, so the output would have a particularly formatted name.
Steps to Reproduce

A full project is infeasible but you can reproduce this by adding this to your groovy build.gradle file:
jar {
  setAppendix("foo")
}

Your Environment
------------------------------------------------------------
Gradle 5.4.1
------------------------------------------------------------

Build time:   2019-04-26 08:14:42 UTC
Revision:     261d171646b36a6a28d5a19a69676cd098a4c19d

Kotlin:       1.3.21
Groovy:       2.5.4
Ant:          Apache Ant(TM) version 1.9.13 compiled on July 10 2018
JVM:          1.8.0_212 (AdoptOpenJDK 25.212-b03)
OS:           Mac OS X 10.14.5 x86_64

The recommendation is correct, Gradle is moving to using Property types in a lot of places to allow for more laziness when configuring / evaluating.
The method getArchiveAppendix() returns a Property<String> that can be set.
In Groovy, thanks to the dynamic aspect, you can do:
jar {
    archiveAppendix = "foo"
}

In Kotlin, you have to do:
jar {
    archiveAppendix.set("foo")
}


Country generation does not respect chosen locale

It appears that Address('en').country() does not respect the locale. Countries outside locale are still generated.
Looking at the code it appears there is no implementation for locale specific country, if so, please change issue to a feature request.

Can you explain, please, what do you exactly expect? .country() returns name of the country for the current locale.
For example:
address = Address('ru')

address.country()
'Китай'

address.country()
'Россия'

Do you expect something like this?
address = Address('en')

address.country()
'United States of America'

address.country()
'United States of America'

Yes my understanding was that it would return a country who belongs to the locale. Now I can see that it is not the case and it is intended to be that way. Thanks.
Nevertheless locale specific data would be a great feature

@andkii We will fix it ASAP. By default, it must return the name of the current country.
We can just add a parameter allow_random which will allow random values for it.
@ceccoemi If my memory doesn't let me down, you've already done something like this, haven't you?

Yes, we did the same thing in #664
I will work on it!

Great! Thanks!

@lk-geimfari what do you think about doing the same thing with the continent() function?

@ceccoemi I thinks it's can be useful.

@lk-geimfari no, maybe it's better to leave it as it is. Because some countries don't have a single specific continent, for example Russia and Turkey, they belong both to Europa and Asia. It could be ambiguous in some  situations.

@ceccoemi Agree. Let's don't change it.


[GHSA-h75f-hjcr-cvh8] Multiple cross-site request forgery (CSRF)...

Updates

Affected products
References
Source code location
Summary

Comments
add patch commit: https://github.com/moodle/moodle/commit/0cd720fe512d48c2af81fc054c042c9c63e8a234.
the commit msg has shown it's a fix for MDL-44606.
update vvr as well.

Sry, I have made a mistake: there is no fix in v2.5.5. What can I do if I want to correct it? Or can u help to correct it when processing this pr? Thx in advance.


Fix pylintrc

This is needed to change to our newer version of pylint. Why they just remove "s" from a bunch of stuff, I don't know.

Oh, hold of on this. I inadvertently snuck in a commit for another pull. I will fix it if needed. Or wait until that issue is fixed.

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/101/ for details

No pylint warnings for pull request 6046:

Unit test changes for pull request 6046:

WMCore_t.Database_t.RotatingDatabase_t.RotatingDatabaseTest:testArchive changed from failure to success
WMCore_t.WorkQueue_t.Policy_t.Start_t.Dataset_t.DatasetTestCase:testParentProcessing changed from error to failure

@ericvaandering, Do I still hold on to this or ready to be merged?

Still hold it.
Sent from a mobile device. Please excuse my brevity or transcription errors.

On Jul 7, 2015, at 10:33, ticoann<EMAIL_ADDRESS>wrote:
@ericvaandering, Do I still hold on to this or ready to be merged?
—
Reply to this email directly or view it on GitHub.

Please merge this now. I rebased to remove the extraneous commit


CoordinateTool widget DD to MGRS rounding issues?

When you input DD of "40 -121", MGRS converts to 10TFK7072529673.  Checking this with http://www.earthpoint.us/Convert.aspx gives you 10TFK7072529672.  Rounding issue?  Some documentation says to not round up, others do not.

@jmccausland If we are saying that this tool uses the WGS84 datum I think the tool rounding centimeters up is fine. The WGS84 datum to many is described as a reference frame more than a true datum and only accurate to ~1 to ~2 meters. Considering that the tool is still within the 1 meter tolerance do we need to make a change?

@jmccausland @migueldls @nfeuerstein Installed the NGA Geotrans 3.7 and tested the same coordinate...

Geotrans also converted the coordinate 40 -121 to 10TFK7072529672, whereas our ArcGIS Pro addin, ArcMap addin and Web AppBuilder widget converts to 10TFK7072529673.
According to @kgonzago the Web AppBuilder widget is using the Geometry service to perform these conversions (http://tasks.arcgisonline.com/arcgis/rest/services/Geometry/GeometryServer).
Using the ToGeoCoordinateString endpoint I tested the conversion of this input coordinate. There is a parameter to set rounding.
With rounding disabled the service returns the same value as Geotrans...

and the online tool specified earlier (http://www.earthpoint.us/Convert.aspx)...

In the screenshot above, note the UTM centimeter value for northing (i.e. 4429672.97). In this particular case its more accurate to round.
With rounding enabled the service returns the same value as the widget, and ArcMap and ArcGIS Pro addins (and also the identify tool in ArcMap)...

@nfeuerstein My recommendation at this point is to leave the widgets and addins alone, i.e. continue to have them round since all three are producing consistent values and as @migueldls stated above you can't get sub-meter accuracy with WGS84. What we may want to do as a future enhancement is to have a configuration button/page for the addins/widgets that would the user to set rounding options, etc, which would be passed to the geometry service or projection engine to further refine how the conversion is made. Please advise how you want to proceed. Thank you.

@kgonzago @elinz can the geometry service be easily configured to point elsewhere?  If you use this widget on an air-gapped network (zero connection to the www) will it fail since its pointing to an AGO geometry service?

@nfeuerstein @kgonzago The web app containing the widget can be re-configured to point to an internal geometry service, which @Dbarnes1 tested the other week when testing the GRG widget.

In that case, I agree with your recommendation.  Leave it alone and we can entertain a future enhancement for configure rounding options.

@nfeuerstein @kgonzago FYI. Here is a blog, which explains how to configure a web appbuilder app in a disconnected enviroment... https://blogs.esri.com/esri/arcgis/2016/11/02/deploying-web-appbuilder-for-arcgis-developer-edition-in-a-disconnected-portal-for-arcgis-environment/

@nfeuerstein If it's ok with you, I'll go ahead and change the milestone to after next and change the label to an enhancement.

Sounds good @elinz

This is still an issue and needs to be moved to the new repo. Here is a screen capture of the issue using the updated widget:

This issue has been moved.


Default values for HAL settings

It seems like default values for HAL settings either are not set, or doesn't work for stm32cubef4.

I have some problems to repeat the condition that made the bug appear...


Add version to quant config

:version param: version of the compressors used to optimize the given model

Already done by #160. I did some scoping of this earlier and I decided not to add it directly to the config, although I forget the exact reason why


Don't how to setup the plugin with wsl (Neovim exited with code 127)

Hello guys,
I'm a really sorry to ask this question, I didn't find the answer and how to resolve it.
I don't know what do I have to put for the paths to use neovim with WSL, I tried it all and it doesn't work...

I have this error.
Thanks for your help

What nvim version do you have installed in WSL? (you need version 0.5+)
Are you setting vscode-neovim.useWS to true and vscode-neovim.neovimExecutablePaths.linux to the path of your nvim bin
(you can use which nvim to find the path)
What is the output of wslpath "c:\windows"?

I'm getting this same error when trying to use the plugin. useWSL is set to true. ExecutablePaths.linux is set to "/usr/bin/nvim" which is where the executable is located (appimage from releases page, v. 0.5) and wslpath output is "/mnt/c/windows". Any ideas what I could be doing wrong?

What nvim version do you have installed in WSL? (you need version 0.5+)
Are you setting vscode-neovim.useWS to true and vscode-neovim.neovimExecutablePaths.linux to the path of your nvim bin
(you can use which nvim to find the path)
What is the output of wslpath "c:\windows"?

Hey, first thank you for your reply ! I found the problem I installed on the wrong version of my wsl...
I just didn't know that Ubuntu wasn't my version per default, thanks for the help anyway !

I'm getting this same error when trying to use the plugin. useWSL is set to true. ExecutablePaths.linux is set to "/usr/bin/nvim" which is where the executable is located (appimage from releases page, v. 0.5) and wslpath output is "/mnt/c/windows". Any ideas what I could be doing wrong?
Edit: Path is set correctly too, running nvim from terminal works perfectly

Check your version per default on wsl mate, just did that and now it work perfectly !

I'm getting this same error when trying to use the plugin. useWSL is set to true. ExecutablePaths.linux is set to "/usr/bin/nvim" which is where the executable is located (appimage from releases page, v. 0.5) and wslpath output is "/mnt/c/windows". Any ideas what I could be doing wrong?
Edit: Path is set correctly too, running nvim from terminal works perfectly

Check your version per default on wsl mate, just did that and now it work perfectly !

Thanks so much buddy, that did the trick. I installed Docker before I installed my Ubuntu distro in WSL and never changed the default but now it's working perfectly.
A suggestion then to the repo maintainer(s), could be helpful to add a warning about this in the readme now that the cause is known. I'm sure lots of developers install Docker Desktop; and then later a personal use distro without thinking to change over the default. Or even better, a VS Code setting pair that lets you choose which distro has neo vim installed. Thanks for your time!


Can't install with symfony 4.1

Your requirements could not be resolved to an installable set of packages.
Problem 1
- dunglas/panthere dev-master requires symfony/browser-kit ^4.0 -> satisfiable by symfony/browser-kit[v4.1.0].
- dunglas/panthere dev-master conflicts with symfony/browser-kit[v4.1.0].
- dunglas/panthere dev-master conflicts with symfony/browser-kit[v4.1.0].
- dunglas/panthere dev-master conflicts with symfony/browser-kit[v4.1.0].
- Installation request for dunglas/panthere dev-master -> satisfiable by dunglas/panthere[dev-master].
Why exactly was the conflict with browser-kit 4.1 added?

This is known issue. Symfony 4.1.0 contains a BC break that impact Panthère. I've fixed it in 4.1, we have to wait for the release of 4.1.1. See symfony/symfony#27485.

Ah, thanks, fingers crossed it's soon. I saw there was a MR to add a 4.1 conflict, but there weren't any issues regarding the matter so I was confused!


Tommy.Extensions.Configuration

Add Tommy.Extensions.Configuration for easy use with microsoft extensions (for example in ASP.NET Core or generic host)
Simplify multi project setup using Directory.Build.props

Hi, js6pak!
Thank you for the pull request! I've looked at it and it appears good.
However, would you really prefer this being part of the Tommy repo instead of maintaining it yourself on your personal repository? I'll be more than happy to link to your repository in the README so that it can be more easily found.
In the case that you do decide to continue with this PR, how about adjusting the author for Tommy.Extensions.Configuration to include your name? At the very least I'd prefer having the proper attribution.
Thank you again for your work!

Hello, @dezhidki o/

However, would you really prefer this being part of the Tommy repo instead of maintaining it yourself on your personal repository?

I thought about it, but I can also maintain it myself on your repository :)
But if you really want me move out to personal repo I can always remake this into a Directory.Build.props PR

Thank you for the pull request!

Thank you for the amazing library!

I thought about it, but I can also maintain it myself on your repository

In that case, it is fine by me. I'll merge this request right now. Thank you again!


message filter dropping: frame "base_link" at time xx for reason 'unkonwn'

hello, thank your work so much. I run fast_lio2 for mapping, and change it to your mapping format.
then I run fastlio2 for odom, for example, it will publish tf: odom->base_link.
then I run your code, for global_frame_id :map, base_frame_id: odom, odom_frame_id:odom.
It seems that rviz show maybe ok? but the terminal always report:
message filter dropping: frame "base_link" at time xx for reason 'unkonwn'.
if I change the fixed frame from map to base_link, it won't report, but I think the fixed should be map?
can you give me some help?  Thank you! I don't know it is good to report such message filter .

Hi!
I do not know if this is the error, but base_frame_id must be set to the robot base frame. In your setup, it seems to be base_link. Fastlio2 is generates the tf odom->base_link, and DLL will generate the tf map->odom.
Try to set base_frame_id: base_link and check .
Fernando

Hi， I  am sorry， base_frame_id is base_link. When I run rqt_tf_tree, I can see map->odom->base_link (ICP method)。 I think maybe the warning means nothing? What more, it will first report ERROR: "odom" passed to lookuptransform argument target_frame dose not exits. But if when dll method, the node will die; when ICP method, the node is good, and I can use initial pose in rivz, then it can run for some time, but the cloud will get higher and higher, then total wrong.

Could you share with me the launch file you are using? Actually, if you pass me the rosbag and the launch file, I can try to reproduce the error.

Thank you. I don't solve this problem, I run ndt_omp for myself. Thank your help these days.


Replace 'slug' dependency

It's generating a security warning that had not been fixed for months.
It's requiring 'unicode' as dependency, leading to a huge footprint in client bundle

After the change.

Huge win for pages/collective.js 472KB instead of 2.1MB (non gzipped).

@clarete I added the tool produce that graph in the project this morning. https://github.com/opencollective/frontend/commit/632737507d702be5645492d0e0d299f5dec590af
From now on, run WEBPACK_BUNDLE_ANALYZER=1 npm run dev or WEBPACK_BUNDLE_ANALYZER=1 npm run build and it will be reported in the console.
I haven't collected exact data, but the website should be much faster now. Before, even if the javascript bundles were cached by the browser, they were taking up to 1 second to be actually loaded.

:tada: This issue has been resolved in version 2.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


JWT is not checking expiration time

It has JwtConstants.EXPIRATION_TIME, but seems it doesn't check for it.

@ruochao I just submitted this PR: https://github.com/pac4j/pac4j/pull/686 Is it ok for you?


ARROW-1246: [Format] Draft Flatbuffer metadata description for Map

Sounds good to me.  I was thinking about how to store Map data from Spark, which is just (key, value) pairs, in Arrow and assumed I'd have to make the types myself into something like List<entry: Struct<key: K, value: V>>.  So if I understand this change correctly, when reading back this type of data, I would know from the Map type that it was meant to be (key, value) pairs?

Thanks for the review, all. @jacques-n is there specialized code for handling maps in this form in the Arrow Java library, or Dremio?

No, we should open a jira to support

ARROW-1261


How to keep the original field information?

for example @timestamp    _type    ; and Field is at the root node

You get back the original SearchResponse from the underlying Java client so check their documentation. So you can either use the Hits they return or the mapped hits the kotlin Wrapper adds, or both. _type has been deprecated in ES 7. I don't think @timestamp is a top level field; it's just a normal field.

kibana index patterns _source.age ,in kotlin code Is there a convenient way to create an index for the dao attribute?

Schema generation is currently not part of the client. Elasticsearch has a schema less indexing that you can use that should work fine with whatever domain model you have. But at some point you'll want to narrow that down. A good starting point then is the autogenerated schema for ES.


Greenkeeper/proxyquire 2.1.3

Pull Request Test Coverage Report for Build 901

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

Totals

Change from base Build 271:
0.0%

Covered Lines:
628

Relevant Lines:
653

💛  - Coveralls


UPDATE readme.md

Hello there,
I would like to work on the readme.md file issue. Can you please assign it to me.

What enhancements you want to make and I will assign you this for it.

I have an idea to convert the standard jpg pic into gif.

Go, ahead

sir, if it doesnt work out , assign me thi sissue, i would like to work on it

sir, if it doesnt work out , assign me thi sissue, i would like to work on it


Confusion around output of normalize_user_addrs

I am confused by the addresses returned by Normalizer::normalize_user_addrs.
From the source documentation of Output, module, and actual computation, return addresses are definitely file offsets.
On the other hand, Normalizer source doc talks about "normalized" virtual addresses (ie. ELF virtual addresses) and the testuser_address_normalization_custom_so explicitly checks that output address is equal to ELF symbol address (ie. ELF virtual address).
The test succeeds because for libtest-so.so file offsets and ELF virtual addresses are equal (because file offset and virtual address of the executable segment are equal).
The reason I am pointing this confusion is that for the use case of remote symbolication, file offsets are not very useful because file offset alone is not sufficient to perform remote symbolization in the case of split debug info:

the first step of symbolization is to convert a file offset into an ELF virtual address, doing so requires knowing to which segment the file offset belongs to, and adjusting the file offset with the segment offset and segment virtual address
but in split debug info, segment file offsets are not equal to the ones from the original binary (because keeping only debug info made all executable and data segments almost empty), and therefore if only split debug info is available it is not possible to recover the ELF virtual address from the file offset.

Would it be possible to have ELF virtual addresses as output of Normalizer::normalize_user_addrs ?

Ah yes, parts of the documentation seem incorrect; as you said we always report file offsets from the normalization step. Will take another look at the tests as well. Thanks for pointing that out.
We used to report virtual addresses until 9f3c71344f9f6cd5d157aee13818390fc83541b8. Usage of file offsets allowed us to minimize the amount of work being performed on the "remote" end.
I think you should be able to post-process the output of the normalization to get virtual addresses as output instead. Basically, iterate over all the addresses, instantiate an ElfResolver on the corresponding path, and then use file_offset_to_virt_offset to retrieve the virtual offset. Can you check whether that's feasible?

The above should have addressed everything.

The above should have addressed everything.

The proposed solution works well for me, thanks for the quick help !


Remove ununsed import  error

I can't describe it well in title, so I just show the example.
try:
    import playhouse.postgres_ext as pw_pext
except ImportError:
    pass

in this case, pw_pext is unused import.
After autoflake --in-place --remove-unused-variables --remove-all-unused-imports $file
try:
    pass
except ImportError:
    pass

To be clear autoflake is replacing "unused imports" with pass  statements.

Running the autoflake command twice, the first with -remove-all-unused-imports flag and the second with --ignore-pass-statements, got the desired outcome (removed unused imports weren't replaced with "pass").


MiniPlayer album covers don't scale-to-fit properly

Images seem to scale to width, but the height should be fixed as well. Honestly, just make it a fixed square and call it good.

Fixed in working branch

Fixed in working branch


JDBC compatible URLS and cockroach convert-url

Closes: #11516, #11515

Added JDBC compatible urls to the standard output for cockroach start and cockroach start-single-node. It was already updated for cockroach demo.
Added cockroach convert url section to the Connection parameters page and a note linking to it on the cockroach sql page.

This change is

@ericharmeling any idea on that error?

any idea on that error?

https://cockroachlabs.slack.com/archives/G01Q35D80EM/p1636668441019300?thread_ts=1636665670.010600&cid=G01Q35D80EM
It might resolve if we just rerun the checks

It might resolve if we just rerun the checks

Actually, I think you might need to rebase on master and then force push


Bots section missing calling bot fields

The parameters necessary to enable a calling/meeting bot aren't documented here, but I'm fairly certain the should be. The two values needed are listed here: https://docs.microsoft.com/en-us/microsoftteams/platform/bots/calls-and-meetings/registering-calling-bot

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

ID: b449c9ba-a94c-4012-4721-cee5f87eeb8a
Version Independent ID: b2536bbc-eb27-547d-d7f5-bab3ded3bedf
Content: Developer Preview Manifest schema reference - Teams
Content Source: msteams-platform/resources/schema/manifest-schema-dev-preview.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@clearab - Thanks for reporting your issue. We will check this and update the document accordingly.


Performance improvements to "host" config items update: DB data migration is required

To track performance improvements for host config item update (triggered by dns/metadata changes)

DB data migration is required for service_index_id field (moving from Data to table field). Should be one time task

Tested upgrade from v0.69.1 -> v1.0.0-rc1.
After upgrade ,  instance table has  new column - "service_index_id" which is populated with service_index_id value which is from the "serviceIndexId" from Data field.


Inject tag or value to capture whether the build is the first or second build of the experiment.

When analyzing the results of an experiment, it can be easy to get lost in a build scan and not be sure whether it was the first or second build of the experiment. Often, the best way to check is by assuming from the performance of the build itself or navigating back to the summary view. It would be more concrete to capture this metadata as a tag or custom value so that it would be clear.

A note on the design while I'm thinking about it: Using a custom value will show up as a difference in the build scan comparison, but a tag will not.

A note on the design while I'm thinking about it: Using a custom value will show up as a difference in the build scan comparison, but a tag will not.

While that is true, I would find it easier to work with in the comparison if it's tag, since it shows up at the top with the rest of the build header in task inputs comparison for example. That way you're sure which build is which on that page without needing to jump to the custom values view.

I was also advocating for a tag when I wrote that. A custom value means there will always be a difference between the builds, which could be misleading.


Badge images are missing meaningful alternative text

Describe the bug
The badge images introduced in a recent build are missing meaningful alternative text. They have the default alternative text which WordPress supplies if none is entered, e.g.:
This image has an empty alt attribute; its file name is Badges_draft-2-07-300x300.jpg
To Reproduce
Steps to reproduce the behavior:

Go to https://dev--wecount.netlify.app/badges.
Inspect any badge image.
Observe that meaningful alternative text is not present.

Expected behavior
All images should include meaningful alternative text.
Additional context
A helpful tool for ensuring this is demonstrated in Pinecone, which blurs images which are missing alternative text: https://pinecone.platform.coop/components/detail/image--missing-alternative-text.html. The same principle could be used to apply a blur to images with alt text beginning with This image has an empty alt attribute. Blurred images would appear on the development site which would help prevent them from being published without meaningful alt text.

I'm noticing that this issue is not present on the production site.


no port setting :(

fixed in #14
please, use 0.3.2 version

thank you!

fixed in #14 please, use 0.3.2 version

tank you


update folder structure for llm in spin 3.0

@karthik2804 I know you've got other stuff in hand but it would be great to get this over the line - are you in a position to carve out a bit of time for it?  Thanks!

@itowlson my apologies for not getting this done sooner. I will update it today.


Prevent PIL from destroying exif metadata when we watermark images #238

We use PIL to produce watermarked images. Most PIL transformations discard EXIF metadata; to work around this, extract the EXIF before we do anything to the image and add it back after the watermark has been generated.

(FYI @brenoferreira & @kgodey)


Hide unnecessary info

Make it possible to hide the info at the bottom (filename, date, resolution) when viewing an image/video.

Do you mean to hide it altogether, or to pick and choose which info is
of interest to you?
On Mon, Jan 4, 2021 at 12:00 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

Make it possible to hide the info at the bottom (filename, date, resolution) when viewing an image/video.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Do you mean to hide it altogether, or to pick and choose which info is
of interest to you?
On Mon, Jan 4, 2021 at 12:00 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

Make it possible to hide the info at the bottom (filename, date, resolution) when viewing an image/video.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Why not both?
I personally prefer to have zero clutter. If I need more info, I click on the 3-dots menu and then on "info" :).

Why not both?
I personally prefer to have zero clutter. If I need more info, I click on the 3-dots menu and then on "info" :).

Ok, I'll add options for this.
On Mon, Jan 4, 2021 at 12:07 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

Why not both?
I personally prefer to have zero clutter. If I need more info, I click on
the 3-dots menu and then on "info" :).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/deckerst/aves/issues/24#issuecomment-753733415, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKBEXLNE4FQX7GTSNC4BQTSYEWIHANCNFSM4VSM3H7A
.

Ok, I'll add options for this.
On Mon, Jan 4, 2021 at 12:07 PM Donkey-Doug<EMAIL_ADDRESS>wrote:

Why not both?
I personally prefer to have zero clutter. If I need more info, I click on
the 3-dots menu and then on "info" :).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/deckerst/aves/issues/24#issuecomment-753733415, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKBEXLNE4FQX7GTSNC4BQTSYEWIHANCNFSM4VSM3H7A
.


156 export model from path files

create ModelRegistry
load models into modelRegistry
pass the model into a request

To do:

[x] use default export (because I might use named exports later to provide models for specific paths)
[x] export a class and have Counterfact instantiate it (so that the class can also function as a type)
[x] generate base #model.ts file
[x] generate #model.ts files in subdirectories
[x] reference the type from #model.ts in files under path-types
[ ] the #model.ts file in a subdirectory should reference its parent

Almost there. Ran into a couple of issues.

I can't use the name #module.js because it confuses the heck out of Node and other tools. Changing it to $model.js.
If a directory does have its own model, at runtime it needs to import instance from '../$model.js; export default instance; at runtime. During static analysis, it needs export the type. Maybe using TypeScripts typeof operator will help.


Looping through object arrays

I have this list:
const list = [{"name":"Elon Musk","company":"Tesla"},{"name":"Bill Gate","company":"Microsoft"}]
I want to loop each of them and get their names and the company's value.
I have tried this, but it didn't work:
{{#each list}}     {{#each this}}
{{name}}

{{company}}
    {{/each}} {{/each}}

seems like a question for handlebars this project just adds handlebars to the express view engine.
 I have been on this issue for ages. Thanks in folds!
And sorry for this misconception, I thought the handlebars and express-handlers are the same.
I am really happy I got it resolved, thanks again!


[hip] Added gfx803,gfx1010,gfx1012 kernel binary hipModule test.

This is to address hipModule test seg fault in GFX1010.

@AlexXAmd Can't this be auto-generated now as discussed before in #1215 ?

Yes. This can be auto generated. But such mechanism does not exist yet.

Yes. This can be auto generated. This default file can still cause problem...

@AlexXAmd So can we switch the build scripts to auto generate this file and modify the PR to delete this file instead?

Yes. I was thinking about deleting the file. I am busy with some other stuff now. So I cannot do it.


Error Handling / Network traffic and communication monitor

I was thinking of introducing a new feature that will improve the network handler. Basicallly I will add a registerNetworkMonitor(FbBotMillNetworkMonitor) on the FbBotMillContext.  FbBotMillNetworkMonitor is an interface with two methods: onResponse(FbMessengerCallback) and onError(FacebookErrorMessage). Each of this methods will be used as a callback when a message is received from the Messenger platform, according to the kind of message. This will make the developers access data that are otherwise unavailable and may be useful in some situations. This will also be really good for unit testing.

Yes. We can then plug an analytics module so that we can track all of the information for a specific user.


0.24.0 generates wrong const-generic code

We have a register containing a field like this
            <field>
              <dim>8</dim>
              <dimIncrement>0x1</dimIncrement>
              <dimIndex>0-7</dimIndex>
              <name>CH%s_TX_THR_EVENT_INT_CLR</name>
              <description>Set this bit to clear the  rmt_ch%s_tx_thr_event_int_raw interrupt.</description>
              <bitOffset>24</bitOffset>
              <bitWidth>1</bitWidth>
              <access>write-only</access>
            </field>

When generating code with 0.23.0 this code produces the expected output:
    let mut data = 0u32;
    let data_ptr = &mut data as *mut _ as *mut u32;
    unsafe {
        &*(data_ptr as *mut generated23::generic::Reg<generated23::rmt::int_clr::INT_CLR_SPEC>)
    }
    .write(|w| unsafe { w.ch_tx_thr_event_int_clr(7).set_bit() });
    println!("With svd2rust 0.23.0: {:032b}", data);

The output is With svd2rust 0.23.0:<PHONE_NUMBER>0000000000000000000000
However, generating code from the same SVD with 0.24.0 the following code
    let mut data = 0u32;
    let data_ptr = &mut data as *mut _ as *mut u32;
    unsafe {
        &*(data_ptr as *mut generated24::generic::Reg<generated24::rmt::int_clr::INT_CLR_SPEC>)
    }
    .write(|w| unsafe { w.ch_tx_thr_event_int_clr::<7>().set_bit() });
    println!("With svd2rust 0.24.0: {:032b}", data);

outputs<PHONE_NUMBER>0000000000000010000000
Apparently, it's ignoring the bitOffset of the field.
I created a repo to reproduce this: https://github.com/bjoernQ/svd2rust_test
Maybe I'm doing something obvious wrong but it looks like a bug to me.

Yes. Those functions are not equivalent. And I'm really surprised someone already used one of them.
const-generic version just takes bitOffset, not field number.
In 0.23 those function were both present in generated code. Numeric version always, const-generic under feature.
I've deleted first one during refactoring.
If you need it I could try to return it back, but it will take time.

Thanks for the quick reply. I will look into our code and we can most probably work around it.
The behavior of the const-generic version is at least surprising and we started using them when we upgraded to 0.24.0 and found the numeric version removed - we just assumed it's equivalent
No idea if this will cause similar confusion for others and if it would be better to re-add the old functionality


Add endpoint for querying bot performance

Background
https://github.com/VolumeFi/paloma/issues/550
Done when
we have a new API endpoint that has the following information:

[x] total number of depositors across all bots as "users"
[x] total number of deposits + withdrawals across all bots as "volume"

@verabehr https://github.com/VolumeFi/bot-info-api/pull/7

follow up requests:

[ ] rename volume to bots
[ ] add one more endpoint called volume - this should have the sum of deposits and withdrawals in USD

https://github.com/VolumeFi/bot-info-api/pull/8


Port iomux to imxrt-ral

Note that each pad requires unsafe modify_reg() calls to convert to a
particular pads alternate usage.
This should be safe because each pad has its own register pair (mux/pad) for
setting things up. So long as the only place where these registers are
used is in the iomuxc instance in struct Peripherals, this is like just
fine.
I'm not excited to have to import the modify_reg!() macro in each gpio module, I didn't try using it using crate::ral::modify_reg!() which perhaps works already, will try later perhaps.

See my thoughts on the need to import modify_reg! throughout the gpio modules.
This all LGTM! Built with cargo build --features imxrt1062.
Mind running cargo fmt?

I'm going to skip the features for conditional compilation on this for now since the goal is to support the 1062 on the teensy and the 1060evk to start. I think we can revisit it in a nicer way perhaps once I do some work to support the 1010evk (imxrt1011). I don't have any other evks at the moment to test against so perhaps we can leave that up to others as they wish to contribute that work, with the way paved by the 3 boards and 2 chips being supported.

Thanks for the review @mciantyre


New sample script: Show how to perform operations if a command is not covered by the CLI

Adds new sample script to show how to perform operations if a command is not covered by the CLI for Microsoft 365.
Closes #2731

Thank you @sprider, we'll review it shortly 👍

@waldekmastykarz, Thank you for your feedback ! Could you please review the latest changes?

Will do! Thank you for such a quick turnaround?


Duplicate prompt

System Details

posh-git version/path: 0.7.1
PowerShell version:

Name                           Value
----                           -----
PSVersion                      6.0.0-beta
PSEdition                      Core
GitCommitId                    v6.0.0-beta.7
OS                             Darwin 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-37...
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Git version: git version 2.13.5 (Apple Git-94)
Operating system name and version: macOS Sierra 10.12.6 (16G29)

Issue Description
My prompt is duplicated:
10:35 $ powershell
PowerShell v6.0.0-beta.7
Copyright (C) Microsoft Corporation. All rights reserved.

/Users/felix/git/vscode [master ≡]> /Users/felix/git/vscode [master ≡]

further typing causes weird behavior:

Thanks for the bug report.  I can duplicate it on WSL/Bash.  I'll take a look at it.

Awesome, this is blocking me atm from using PowerShell as my primary terminal ❤

posh-git version/path: 0.7.1

How did you install posh-git? I would expect a different version number if you were using the develop branch, which has a few cross-platform fixes.

I followed the readme: PowerShellGet\Install-Module posh-git -Scope CurrentUser
I wanted to try the develop version but could not find any instructions on how to install that.

Unfortunately the PowerShell Gallery doesn't support prerelease versions yet.

I wanted to try the develop version but could not find any instructions on how to install that.

This should do it:
cd wherever/you/want/it/installed
git clone https://github.com/dahlbyk/posh-git.git
git checkout develop
.\install.ps1

That gives me an error:
PS /Users/felix/git/posh-git> ./install.ps1
Get-AuthenticodeSignature : The term 'Get-AuthenticodeSignature' is not recognized as the name of a cmdlet, function, s
cript file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is cor
rect and try again.
At /Users/felix/git/posh-git/src/Utils.ps1:163 char:16
+         $sig = Get-AuthenticodeSignature $profilePath
+                ~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Get-AuthenticodeSignature:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

The prompt is not duplicated afterwards anymore, but the input is still weird:

I also noticed the added import to my profile uses backslashes:
Import-Module '/Users/felix/git/posh-git/src\posh-git.psd1'

Yeah, we have definitely have more work to do to finish up the cross-platform / PowerShell Core support.

Happy to test any fixes. Really want to get back to Powershell 💪

The issue is with PSReadline see - https://github.com/lzybkr/PSReadLine/issues/468.  There was a partial fix documented here - https://github.com/PowerShell/PowerShell/pull/3867.
The workaround for now is to add this line to your profile after the import of posh-git (develop branch) e.g.:
Import-Module $Home/GitHub/dahlbyk/posh-git/src/posh-git.psd1

$global:GitPromptSettings.DefaultPromptSuffix = "`n> "

The two keys are use esc seqs instead of Write-Host and for now, you can't have an esc seq on the line with the > char because PSReadline rewrites that char in a different color (red) to indicate syntax errors on the command line.

Also, found and fixed the \ issue.  PR coming shortly.

That fixed it, thanks! Looking forward to a proper fix

The proper fix is sitting in this branch - it's at least preview quality, but I don't have custom key bindings working on Linux yet.

FWIW this workaround makes it tolerable on WSL/Bash on Windows 10.  There's still this weird cursor movement that happens as you type.  :-(

https://github.com/dahlbyk/posh-git/pull/487 has been merged into develop to fix Linux/macOS install issues.

Awesome to see the progress, keep it up :)

When can the fix for this become available in the release?  I have been hoping the fix would be released sooner with how it impacts usage. I continue to see the problem.
Using Mac OS 10.12.6 I just installed PowerShell 6.0.0-rc via brew.  Used install-module posh-git -scope CurrentUser.
Result is the double prompt and odd behavior on input.

The latest builds of PSReadLine address the double prompt and odd behavior on input, but installing requires replacing files in PowerShell 6.0.0-rc.
Hopefully that won't be necessary for the final release, but no guarantees, my free time doesn't necessarily line up with the PowerShell team's schedule.

@binaryWard If you clone this repo and checkout the develop branch and import the posh-git.psd1 file by its path, posh-git should work better on a Mac.  Also add this line to your profile after importing posh-git:
$global:GitPromptSettings.DefaultPromptSuffix = '`n$(''>'' * ($nestedPromptLevel + 1)) '

IIRC the current PSReadline impl behaves better if the prompt char > is on a line by itself.

I'm going to close this since the original issue is really a PSReadLine issue that has been fixed.  If you are running PS Core on Linux or macOS, you can install a beta of the 2.0.0 PSReadLine module with the following command:
Install-Module PSReadLine -RequiredVersion 2.0.0-Beta1 -AllowPrerelease -Scope CurrentUser

Getting this error when I'm trying to install it:
PS /Users/felix> Install-Module PSReadLine -RequiredVersion 2.0.0-Beta1 -AllowPrerelease -Scope CurrentUser
Install-Module : Cannot process argument transformation on parameter 'RequiredVersion'. Cannot convert value "2.0.0-Beta1" to type "System.Version". Error: "Input string was not in a correct format."
At line:1 char:44
+ Install-Module PSReadLine -RequiredVersion 2.0.0-Beta1 -AllowPrerelea ...
+                                            ~~~~~~~~~~~
+ CategoryInfo          : InvalidData: (:) [Install-Module], ParameterBindingArgumentTransformationException
+ FullyQualifiedErrorId : ParameterArgumentTransformationError,Install-Module

Sorry, we've not published Beta1 yet.  I was hoping you could try it from the GH source.  See the instructions I left in #501.

I don't see any instructions for PSReadline in #501?

Doh! Guess I should read these posts more thoroughly before replying.  You need to be on PS Core 6.0.0-rc2.  That is using a newer version of PowerShellGet (1.6.0) that understands the prerelease versioning.

You can also just update (install) the latest PowerShellGet if you're running Windows PowerShell, probably something like Install-Module -Force -AllowClobber -Scope CurrentUser PowerShellGet.


Add: Adrian Schönig's iOS development blog

Thanks for creating this directory. Would appreciate to have my blog included.

Thank you!


ML 7.10: Viterbi Algorithm (V)

Description
📌  Issues for Week 7
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
Viterbi Algorithm.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Bayesian_ML_&_Ensemble_based_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Video

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I am M. Ashish Reddy, I would like to work on this issue.
Batch Number: 2
Serial Number: 260
GitHub Profile Link: https://github.com/ashish-reddy-20-08
Approach: A detailed explanation with examples.

@raina-akshay @geeythree , Can you please assign me this.


Implement  with facebook/yoga flexbox

Hi, nice project. I'm thinking it would be great to use https://facebook.github.io/yoga/ to create  with the same support for flexbox as React Native, thoughts?

Interesting, but I don't know how well that would work with libui-node. It currently implements its own type of view, with limited features. There isn't a way to set a component position. I don't have the time to add this.

Yeah, I later saw that this is using libui bindings for node, I thought it was using its own renderer 👍 Maybe you should add to the readme that it is libui based?


[3.0] improve config bean initialization compatible with spring 3.x/4.1.x

What is the purpose of the change
improve config bean initialization compatible with spring 3.x/4.1.x.
Brief changelog
Verifying this change

Codecov Report

Merging #8407 (0416cb8) into 3.0 (bb94723) will decrease coverage by 0.09%.
The diff coverage is 77.77%.

@@             Coverage Diff              @@
##                3.0    #8407      +/-   ##
============================================
- Coverage     63.20%   63.11%   -0.10%
- Complexity      311      312       +1
============================================
  Files          1099     1099
  Lines         46057    46068      +11
  Branches       6955     6958       +3
============================================
- Hits          29112    29074      -38
- Misses        13693    13735      +42
- Partials       3252     3259       +7

Impacted Files
Coverage Δ

...notation/ReferenceAnnotationBeanPostProcessor.java
89.84% <50.00%> (-0.88%)
:arrow_down:

...ing/context/DubboBootstrapApplicationListener.java
84.78% <75.00%> (+3.70%)
:arrow_up:

...fig/spring/context/DubboConfigBeanInitializer.java
91.66% <100.00%> (+0.75%)
:arrow_up:

...g/context/DubboInfraBeanRegisterPostProcessor.java
100.00% <100.00%> (ø)

...ing/boot/autoconfigure/DubboAutoConfiguration.java
100.00% <100.00%> (+14.28%)
:arrow_up:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
43.47% <0.00%> (-15.22%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
61.01% <0.00%> (-6.78%)
:arrow_down:

...dubbo/remoting/exchange/support/DefaultFuture.java
87.71% <0.00%> (-6.15%)
:arrow_down:

... and 27 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb94723...0416cb8. Read the comment docs.


generic_const_impls

Firstly, please check this question.
I know that you are currently working on this, but how do you think it is possible to fix this issue without fixing this feature?

i think that the main issue in your example is https://github.com/rust-lang/rust/issues/92151 which I did not know about before.

I know that you are currently working on this, but how do you think it is possible to fix this issue without fixing this feature?

I would assume that this issue gets fixed before we stabilize generic_const_exprs, but it will only be relevant while using generic_const_exprs so i might be misunderstanding your question.
When encountering bugs or weird error messages in the future, please post them to https://github.com/rust-lang/rust/issues instead of here, as this repository is mostly used for design discussions and internal documentation


Doesn't work on Xubuntu 16.10

Script seems to be configured successfully, notification "bing wallpaper changed" appears, as well as the downloaded wallpaper in the directory, however the wallpaper isn't actually changed

As far as I'm concerned, this script only works on the Gnome desktop environment or closely related ones (such as Budgie), not XFCE.

Please can there be support for XFCE added in? I share the same problem as the OP.
Thanks :)

Closed issue due to inactivity.


New Youtube cookie consent screen causing errors

It seems that pretty recently (articles point to April 1 for when this started happening), Youtube started showing users a cookie consent page, which LavaPlayer does not know how to handle.
I was made aware of this from an issue in my project that depends on LavaPlayer. From what I could get out of the logs submitted, the error no PLAYER_JS_URL in youtube root was being printed. I copied the html printed in the logs into a file and opened it in my browser, and I got a page looking something like this:

Also in the logs was the URL consent.youtube.com, which the "OK" button redirects to, and was also mentioned in the article linked above.

Additional Info

This error occured with LavaPlayer version 1.3.59, but since 1) the error reportedly only started in the last few days, 2) articles point to this being a new occurrence, and 3) I did not see anything in commit history since 1.3.59 that would address this, I believe this would still happen on the latest version.
I tried to reproduce this error on my local machine, and everything worked fine. I think Youtube is slowly rolling this page out to select users, or possibly users in specific regions.

Thanks in advance!

#597 should be fixed in recent versions


Fixed typos

exmample -> example
intialize  -> initialize

@sgrepos : Thanks for your contribution! The author, @mikeblome, has been notified to review your proposed change.


python3Packages.gradio: init at 3.20.1

Description of changes
This PR add gradio by @gradio-app, a web server framework for making AI (anything really) demos.
I made this PR some time ago, made upstream fix some issues, then rebased it on master. The shap and markdown-it-py commits were initially update commits, but they were reduced to fixup commits during the rebase, as someone else upgraded them before me.
Depends on #187008
Close #235184

Things done

Built on platform(s)

[x] 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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Fun fact: the shap module has been updated three times while marked broken.

@OfBorg build python3Packages.gradio

I moved the python3Packages.shap over to my other PR #187008, is it helps with testing.

@ofborg build  python3Packages.gradio

This force-push changes the following:

python3Packages.hatch-requirements-txt: init at 0.2.0
python3Packages.gradio: 3.1.4 -> 3.11.0, migrate to hatchling, automate test selection

Gradio have now moved to pytest, which means i could steal the pytest hook i wrote for shap which automatically ignores failing network tests. I additionally expanded it to ignore tests that fail as a result of gradio upstream failing to upload certain test files to pypi.
Idea: Should i generalize turn this pytest xfail hook into a a checkInputs hook like pytestCheckHook? I believe other maintainers would love to have something like this.

Idea: Should i generalize turn this pytest xfail hook into a a checkInputs hook like pytestCheckHook? I believe other maintainers would love to have something like this.

Lets move that into another PR to get this one merged faster.

I really hope it gets merged soon.

Rebased the pr, seems like the shap dependency broke again,

I'm tired of waiting for shap to fail building. Not fixing it anymore, i'll just disable the relevant test if it is broken.
Not bumping gradio version this time either, since they have done major changes which introduces a chicken-and-the-egg circular dependency problem. I'll solve that in a follow-up pr along with possibly mixing the github and pypi sources.
@ofborg build python3Packages.gradio

I would merge this but x86_64-darwin  fails, can you fix it or mark it as broken?

Seems like tensorflow timed out? Not sure what to do about that other than trying again
@ofborg build python3Packages.gradio

This time darwin succeeded while x86_64-linux instead failed, on the same package: scikit-image. Likely that package is flaky.
It's pretty safe to assume the problem is not introduced in this PR.
How do i rebase to staging?

How do i rebase to staging?

Isn't changing the target branch of the pr to staging enough?
https://nixos.org/manual/nixpkgs/stable/#id-<IP_ADDRESS>.3

something like
git fetch upstream staging:staging
git checkout staging
git checkout -b gradio_staging
git cherry-pick gradio-init~2 gradio-init~1 gradio-init~0

looking forward to this, feel free to post if you need anything.

The build gets stuck on linux. But here are some of the failures
  ├─ ⚠ pacup failed with exit code 1 after ⏱ 7s in installPhase
┃    ├─ ⚠ python3.10-returns-0.20.0 failed with exit code 1 after ⏱ 20m43s in pytestCheckPhase
┃    ├─ ⚠ matrix-synapse-1.86.0 failed with exit code 1 after ⏱ 12m25s in installCheckPhase
┃    ├─ ⚠ python3.10-qcodes-0.38.1 failed with exit code 1 after ⏱ 17m44s in pytestCheckPhase
┃    ├─ ⚠ python3.10-spyder-5.4.2 failed with exit code 1 after ⏱ 36s in installPhase
┃    ├─ ⚠ python3.10-numpyro-0.11.0 failed with exit code 139 after ⏱ 21m48s in pytestCheckPhase
┃    ├─ ⚠ python3.10-nbconflux-0.7.0 failed with exit code 1 after ⏱ 2s in buildPhase
┃    ├─ ⚠ python3.10-ocrmypdf-14.2.1 failed with exit code 1 after ⏱ 6m12s in pytestCheckPhase
┃    ├─ ⚠ python3.10-datasette-0.64.3 failed with exit code 1 after ⏱ 1m33s in pytestCheckPhase
┃    ├─ ⚠ conan-2.0.5 failed with exit code 1 after ⏱ 12m18s in pytestCheckPhase
┃    ├─ ⚠ python3.11-camelot-py-0.11.0 failed with exit code 1 after ⏱ 1s in buildPhase

the build is stuck for me on the following
 manim-0.16.0.post0 (pytestCheckPhase) ⏱ 3h27m40s
┃    ├─ ⏵ python3.10-tables-3.8.0 (installCheckPhase) ⏱ 3h48m14s
┃    ├─ ⏵ python3.10-stim-1.9.0 (buildPhase) ⏱ 4h2m42s
┃    ├─ ⏵ python3.10-pytorch-metric-learning-2.2.0 (pytestCheckPhase) ⏱ 4h5m52s
┃    ├─ ⏵ python3.10-botorch-0.8.5 (pytestCheckPhase) ⏱ 4h14m21s
┃    ├─ ⏵ miopen-5.4.2 (buildPhase) ⏱ 4h53m32s

I hoped there would be some bot to do it or something
@ofborg build python3Packages.gradio

It looks like you accidentally mass-pinged a bunch of people, which are now subscribed
and getting notifications for everything in this pull request. Unfortunately, they
cannot be automatically unsubscribed from the issue (removing review request does not
unsubscribe), therefore development cannot continue in this pull request anymore.
Please open a new pull request with your changes, link back to this one and ping the
people actually involved in here over there.
In order to avoid this in the future, there are instructions for how to properly
rebase between branches in our contribution guidelines.
Setting your pull request to draft prior to rebasing is strongly recommended.
In draft status, you can preview the list of people that are about to be requested
for review, which allows you to sidestep this issue.
This is not a bulletproof method though, as OfBorg still does review requests even on draft PRs.


Global 'click' event listener should account for port

My gridsome website is deployed on example.org. I have a link pointing to //examples.org:8080/foobar.
This global 'click' event should account for the port in its test, otherwise, it mistakenly thinks this is an internal link and overrides the domain.
N.B: honestly this whole event handler seems like a terrible idea... It's easy to circumvent by right clicking on an anchor and do "Open in a new..." (tab, window, whatever) and I can see a lot of edge cases arise. People should just use g-link. But that's just my two cents.

Thanks! The event is for capturing internal links inserted by for example v-html. It just does the same checks as router-link plus a few more. But we might remove it if we find a better solution sometime.


how to get the event after the dialogplus show?

I want to start a view animation after the dialogplus measure. But I can not get the event. I need to calculate the position and size before the animation start.

There is no listener for this event unfortunately. Could use a PR for this though.

PR?

pull request

Closing this issue due to long time period, can reopen if needed


Masthead:- Accessibility :- Close icon (issues related to hamburger menu once it is expanded)

Issue Description :- There is no label for the mentioned control.
Recommended Fix:- Give appropriate label.
URL : http://ibmdotcom-web-components-canary.mybluemix.net/?
Happening on :- Voiceover ,iPhone

@chetankan Is this when you tested the Web Components code or on React code too?  Please clarify when you open bug reports so we know where to address.

@ljcarot : In web component, added url now.

As the carbon-custom-elements  were updated, their close button now  have a proper aria-label


Update terriamap deploy docs

'basic here's how to terriamap with docker'

handbook-ify the cloudformation docs

If there's useful stuff for open source, retain that for the general docs

@peterhassall still something you want to do?

@peterhassall for when you get back, to answer above thanks

ping @peterhassall


Porting to other devices

Hey guys ,
I was just wondering if this code can be ported to any devices like the arduino or pi. Is there any other implementations done ?
Please do reply. Also excuse me if this question is not in the appropriate place .

Project moved to http://github.com/eclipse/wakaama


Update wasm-opt-for-rust proposal re fuzzing api

In the m1 deliverable I suggested not to expose wasm-opt's fuzzing features to the api, and this was accepted, with the suggestion we update the original proposal, re https://github.com/w3f/Grant-Milestone-Delivery/pull/552#issuecomment-1230409728

https://github.com/w3f/Grants-Program/pull/1195


ref(_tests): use gexec for executing commands

This PR represents a partial migration to use gexec, Gomega's built-in facilities for executing commands. The main benefits are:

Use built-in facilities for executing commands
Ability to capture stdout and stderr in a centralized location (should be done in a later PR)
Error reporting is standard and much clearer

Note that this PR does not completely migrate the test suite to use the new gexec-based code. Further migrations should be done incrementally and in future patches.
I've marked this a showstopper because the the current test case failure messages do not show the output of most failures, and the person running the test usually will need to insert their own debugging statements to proceed.
cc/ @gabrtv @michelleN

@bacongobbler made significant changes to this PR, which incidentally address both of your comments.

much better :)

:+1:


[Bug] Extreme latency when drawing with Ipad

Using excalidraw on my Ipad air 4 the drawing feature is extremely laggy. It will take half a second for the drawing to catch up with the pen which makes drawing much slower. If I'm writing at a normal speed the drawings with miss details such as bends in the line.

+1

+1

We're aware of this in the general sense and will be making optimizations. Does this happen when there are more shapes on the canvas (visible or not), or is it slow from the first pen stroke you're making on an empty canvas?

We're aware of this in the general sense and will be making optimizations. Does this happen when there are more shapes on the canvas (visible or not), or is it slow from the first pen stroke you're making on an empty canvas?

I believe this is possibly the same issue I'm having. This is a Samsung s6 lite which is a different device. However in having the same issue it is really laggy almost with a consistent amount of lag no matter the amount of objects on screen. Using android gpu debug mode I can see that it doesn't seem to directly be using the pen api(this is just an obsidian thing I think).
But one major problem that seems to be there is how the shapes are drawn it seems to be done as vectors or some sorts Becuase of the weird behavior this I'm not sure of.
I just started using this today but hope I can help since it has some of the best features for a drawing plugin.

@dwelle I am also seeing this issue. It is slow from the first stroke

We're aware of this in the general sense and will be making optimizations. Does this happen when there are more shapes on the canvas (visible or not), or is it slow from the first pen stroke you're making on an empty canvas?

Sometimes it's slow from the first pen stroke, and if not slow, it is uneven and skips. If there are embeddings or shapes in Excalidraw, it becomes even laggier than it already is.

any schedule on the fix for this? the lag is extreme even on the very first stroke, im talking relative to goodnotes' latency of strokes


Run tests via GitHub Actions on pull requests

xcbeautify is already included in the runner image macOS-14.
Once other platforms are introduced, we could use a matrix similar to ChimeKit.
If I am not mistaken, the GitHub Actions job will first be executed when this branch is merged into main.
Closes #3.

Thank you! Let's see how this runs


Association without foreign key got ERROR Foreign key is missing

Issue
Versions

sequelize: 6.28.0
sequelize-typescript: 2.1.5
typescript: 4.9.3

Issue type

[ ] bug report
[x] feature request

Actual behavior

Ref. Issue: #355
I'm trying to define a pair of @HasMany / @BelongsTo association without foreign key, for use with custom include / join on conditions.
But I still got the error: Foreign key for {Model1} is missing on {Model2}, even though I used the option: constraints: false.
Not sure if this is supported with Sequelize and sequelize-typescript ?
Expected behavior

Should be able to define associations without foreign keys for use with custom include/joins
Steps to reproduce

Define two models
Add @HasMany / @BelongsTo pairing without foreign keys
Try to call Model.findAll() with include: { on: ... } option

Related code

AnonymousQuestionModel.ts
// EXAMPLE CODE

export interface AnonymousQuestionAttributes {
    // ...
}

@Table
export class AnonymousQuestionModel
  extends Model<AnonymousQuestionAttributes>
  implements AnonymousQuestionAttributes {
    @AutoIncrement
    @PrimaryKey
    @Column({ type: DataType.BIGINT })
    id!: number

   @Column
   email!: string

   @Column
   subject!: string

    @Column
    description!: string

    @BelongsTo(() => QuestionModel, { constraints: false, foreignKeyConstraint: false })
    maybeUserQuestion?: QuestionAttributes | null
}

export interface QuestionAttributes extends Omit<AnonymousQuestionAttributes, 'email'> {
    user: UserAttributes
}

QuestionModel.ts
@Table
export class QuestionModel extends Model<QuestionAttributes> implements QuestionAttributes {
    @AutoIncrement
    @PrimaryKey
    @Column({ type: DataType.BIGINT })
    id!: number

   @Column
   subject!: string

    @Column
    description!: string

    @ForeignKey(() => UserModel)
    userId!: number

    @BelongsTo(() => UserModel)
    user!: UserAttributes

    @HasMany(() => AnonymousQuestionModel, { constraints: false, foreignKeyConstraint: false })
    maybeDuplicateQuestions: AnonymousQuestionAttributes[] = []
}

QuestionsService.ts
export class QuestionService {
  constructor(protected readonly questionModel: ModelStatic<QuestionModel>) {
  }

  findAllWithDuplicates(opts: FindOptions<QuestionAttributes>) {
    return this.questionModel.findAll({
        ...opts,
        include: [
            {
                association: 'user',
            },
            {
                association: 'maybeDuplicateQuestions',
                on: {
                   [Op.and]: [
                        Sequelize.where(
                           Sequelize.col('QuestionModel.user.email'),
                           Op.eq,
                           Sequelize.col('maybeDuplicateQuestions.email')
                    ),
                    Sequelize.where(
                        Sequelize.col('QuestionModel.subject'),
                        Op.eq,
                        Sequelize.col('maybeDuplicateQuestions.subject')
                    ),
                }
            },
        ],
        order: [
            [Sequelize.col('QuestionModel.id'), 'desc'],
            [Sequelize.col('maybeDuplicateQuestions.id'), 'desc'],
        ],
    })
  }
}

@quaos Do you found any solutions?

@quaos Did you found any solutions?

No luck. The latest workaround I'm using is adding a "dummy FK" for the relationship.


The Monte Carlo algorithm, as presented, is not self-evident

Here:
https://github.com/javad-doliskani/divpoly_pit/blob/98734cc4aff5acd423f0d10277c6f9d0162e0eb5/divpoly_pit.tex#L166-L167
it would be clearer if you wrote E(𝔽_q) ≅(ℤ/(p±1)ℤ)² instead of the weaker statement on #E(𝔽_q).

Done.


Pangram Grep hole needs more randomness

It can still be passed without properly considering non-letter characters (see my python solution). It can also be solved by a sort of "brute force" method which doesn't consider the case of the letters (see my javascript solution).
Maybe use more tests with much more randomness, choosing from all visible ascii or even unicode, as well as more mixed case letters.

@prplz & @romancortes How about the new approach? Still cheat-able? :-P

I accidentally cheated a tiny bit in python and I'm not 100% sure how

Maybe there should be some sort of protection against solutions that "randomly" pass sometimes.

The shortest Perl solution is currently flapping - it wasn't previously. It fails whenever a test string contains a hyphen. Two of the new test cases do, but because they are chosen at random, it's possible to have a set of test cases without either.
I disagree with @prplz that a valid solution should correctly handle all printable ascii, but i do agree that it should at least be able to handle all common punctuation marks:
.!?,;:-'"

I think all of these are covered in the test cases. One way to prevent flapping solutions that fail whenever one or more of these characters is present, is to select a manditory subset of test cases which contain all of the punctuation marks to be tested, and randomly select from the remaining test cases.


Future request: Add HasFlag method

Please, add HasFlag method, to avoid boxing/unboxing performance issue:
public static bool HasFlag(this Color value, Color flag) => (value & flag) == flag;

This method is already added for enums that have the [Flags] attribute 🙂


WIP: fix the test for API propose_update

sadly the test keeps failing in CI :/ (master, pypi)

this fix also issue in https://github.com/packit-service/packit/issues/468


clr-dg-pagination should not set clrDgPage on destruction

Select one ...  (check one with "x")
[x ] bug
[ ] feature request
[ ] enhancement

Expected behavior

I should be able to bind to clrDgPage and only have it change when user clicks something.
Specifically, I am binding it to a global data service, so having it stay on the last selected value, allows user to return to a view without having to page forward again.
Actual behavior

During destruction, the pageNumber binding gets one last update, to return to page 1.
Reproduction of behavior

Include a link to the reproduction scenario you created by forking one of the Clarity Plunker Templates:

Globally provided data service, with pageNumber attribute
Create two routes
On route one put a data grid, and bind it using

      <clr-dg-pagination #pagination [(clrDgPage)]="pageNumber" [clrDgPageSize]="service.pageSize" [clrDgTotalItems]="service.totalItems"></clr-dg-pagination>

Put this in your component
  set pageNumber(value: number) {
    console.log('setting ', value);
    this.service.pageNumber = value;
  }

  get pageNumber(): number {
    return this.service.pageNumber;
  }

Navigate to page two of the data grid. See that console says 'setting 2'
Navigate to the second route,
See that console says 'setting 1', when you leave the first route.
Environment details

Angular version: 5

Clarity version: V11?

OS and version: Mac OS X

Browser: [all]

This is a duplicate of #2016 and #1099.
As mentioned in these issues, we are currently blocked because of an Angular bug and there is no workaround we can implement until it is fixed. We can't even provide you with a time estimate, it's fully out of our hands.


Implement all the Commands of the form "/session/:sessionid/localstorage/CMD"

Would this cause localStorage not to work when using selenium for testing?


Openbot hardware

Hi, We have used Arduino nano in our project with the old boot loader we cannot able to make it work in sense we are not able to replicate some of the functionalities of the openbot but we can make it work with Arduino uno.

We have used the Arduino Nano with old Bootloader for all experiments. What does not work for you?

The nano with old bootloader is not responding with the phone. can you help us through this?

Sorry, For now we are going ahead with the uno board which is working for us


React Virtualized Window Scroller not rendering new rows

I'm trying to follow the example for the window scroller in react virualized but can't seem to get any new rows to render (See example).
Here's the code I wrote:
const Row = ({ index, isScrolling, isVisible, key, style }) => {
    console.log(`Rendering row ${index}`);
    return tracks[index] ? (
      <TrackItem
        key={key}
        id={tracks[index].id}
        title={tracks[index].title}
        artist={tracks[index].artist}
        album={tracks[index].album}
        explicit={tracks[index].explicit}
        albumArtKey={tracks[index].albumArtUrl}
        trackKey={tracks[index].trackUrl}
        style={style}
      />
    ) : (
      <></>
    );
  };

  return (

      <WindowScroller ref={windowScrollerRef}>
        {({ height, isScrolling, registerChild, onChildScroll, scrollTop }) => (

            <AutoSizer disableHeight>
              {({ width }) => (

                  <List
                    className="outline-none"
                    ref={listRef}
                    autoHeight
                    height={height}
                    isScrolling={isScrolling}
                    onScroll={onChildScroll}
                    overscanRowCount={5}
                    rowCount={tracks.length}
                    rowHeight={48}
                    rowRenderer={Row}
                    scrollToIndex={scrollToIndex}
                    scrollTop={scrollTop}
                    width={width}
                  />

              )}
            </AutoSizer>

        )}
      </WindowScroller>

  );

For whatever reason the example doesn't seem to work in my app. It does however work when I use an AutoSizer without the WindowScroller attached. Struggling to see the issue here.

@kierancrown same problem but with just List component. Rows just rendered above outside of List box area.

I'm also having the same issue when trying to combine the WindowScroller, AutoSizer and List.
It might be worth noting that I get this error in the console:
(index.js:1 Warning: findDOMNode is deprecated in StrictMode. findDOMNode was passed an instance of WindowScroller which is inside StrictMode. Instead, add a ref directly to the element you want to reference. Learn more about using refs safely here: https://reactjs.org/link/strict-mode-find-node)
I tried wrapping the AutoSizer, then the List component with  - this removed the error but didn't fix the scroll problem.

I was struggling with the same issue a few days earlier. I also patterned my component to the WindowScroller, AutoSizer, and List example. What fixed it for me was assigning a ref to the parent container of the WindowScroller and using it as the scrollElement like so:
const scrollElementRef = useRef()

return (

    <WindowScroller scrollElement={scrollElementRef.current}>
      {() => { ... }}
    </WindowScroller>

)

scrollElement={containerRef.current}

What versions are you using? I still can't get this to work using your example.

I'm using react: 17.0.1, react-virtualized: 9.22.3

I'm using react: 17.0.1, react-virtualized: 9.22.3


Clean up GraphQL schema code

No functional changes, but a complete reorg of the schema code, that should give us a much cleaner base to work on: clear and consistent naming conventions, factor common code (e.g. auth that was duplicated everywhere), remove unnecessary complexity, fix a gazillion compiler warnings.
The diff is maybe not the best way to review the changes, because a lot of code has moved: all schema code is now under io.stargate.graphql.schema, with SchemaFactory being the entry point.

"DDL" denotes everything related to the /graphql-schema URI: DdlSchemaBuilder (formerly KeyspaceManagementSchema) and fetchers.ddl.* (formerly io.stargate.graphql.fetchers).
"DML" denotes everything related to the /graphql/{ksName} URIs: DmlSchemaBuilder (formerly GqlKeyspaceSchema) and fetchers.dml.* (formerly in DataFetchers.java, now split into individual files).

I also added CreateTableFetcherTest to demonstrate that the GraphQL logic can be unit-tested in isolation, without any backend or web server. It is straightforward for DDL operations. DML operations will require a bit more work because we'll need to mock DataStore queries and responses, but it can be done.
I would have done the same for the servlet code, but it's harder because our servlets access package-private methods in graphql-java-servlet types. I'm getting more and more convinced that we should drop that dependency.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 18

Complexity increasing per file
==============================
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/UpdateMutationFetcher.java  14
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/QueryFetcher.java  6
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/InsertMutationFetcher.java  12
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/CassandraFetcher.java  1
- graphqlapi/src/test/java/io/stargate/graphql/schema/fetchers/ddl/CreateTableFetcherTest.java  1
- graphqlapi/src/main/java/io/stargate/graphql/schema/NameMapping.java  2
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/MutationFetcher.java  1
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/DropTableFetcher.java  3
- graphqlapi/src/test/java/io/stargate/graphql/schema/GraphQlTestBase.java  2
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/CreateTableFetcher.java  15
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/DmlFetcher.java  11
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/SingleKeyspaceFetcher.java  2
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/AllKeyspacesFetcher.java  1
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/dml/DeleteMutationFetcher.java  8
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/DdlQueryFetcher.java  28
- graphqlapi/src/main/java/io/stargate/graphql/util/CaseUtil.java  13
- graphqlapi/src/main/java/io/stargate/graphql/schema/DdlSchemaBuilder.java  4
- graphqlapi/src/main/java/io/stargate/graphql/schema/DmlSchemaBuilder.java  27
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/AlterTableAddFetcher.java  5
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/CreateKeyspaceFetcher.java  9
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/KeyspaceFormatter.java  5
- graphqlapi/src/main/java/io/stargate/graphql/schema/fetchers/ddl/AlterTableDropFetcher.java  5
- graphqlapi/src/main/java/io/stargate/graphql/schema/SchemaFactory.java  1
- graphqlapi/src/main/java/io/stargate/graphql/schema/CustomScalar.java  10
- graphqlapi/src/test/java/io/stargate/graphql/schema/DdlTestBase.java  1

Clones added
============
- graphqlapi/src/main/java/io/stargate/graphql/schema/DmlSchemaBuilder.java  4

See the complete overview on Codacy


Could not register tab error

Describe the bug
Upon starting the app just after updating to the latest version, I got an error message just after the main form showed. The error message says:

Could not register tab. Error -2147024809.

To Reproduce
Steps to reproduce the behavior:

Install latest version 0.3.0
At end of installer, choose to run the app
Main screen shows, and error appears.

Screenshots

Desktop (please complete the following information):

OS: Windows 10
Version: 0.3.0

The app still seems to work just fine after this error. I have no idea where it may be coming from, but the only "tabs" involved are in the setup screen, and those show just fine too.
If I open the app again, I don't see this error message.

Looks related to the TTaskbar component:
https://stackoverflow.com/questions/34427903/what-is-the-proper-way-to-create-ttaskbar-instance-at-runtime

Manually implemented taskbar progress directly via Windows API rather than TTaskbar. No telling if this will fix this particular bug, but based on what I've seen, it should. Again, this error only appeared once in the entire time I've been working with this app.


Instead of only error messages being read, extra information is being read which is not required and can be confusing to user.

Is it an issue related to Adaptive Cards?
No.
What is the PWD impact?
Screen reader users will be confused if extra information is being read which is not required.
What browsers and screen readers do this issue affect?
Windows: Edge with Windows Narrator
Are there any code-based customization done to Web Chat?
No, I am using Web Chat without any customizations except "styleOptions".
What version of Web Chat are you using?
Latest production
Which area does this issue affect?
Others or unrelated
What is the public URL for the website?
https://servicesuat.microsoft.com/#/opdashboard/
How to reproduce the issue?

Hit the URL https://servicesuat.microsoft.com/#/opdashboard/
Tab till iris bot icon in the header section and press enter.
Iris bit welcome screen will be displayed
Tab till type your message textbox and type Sap access request.
Select self and press enter.
Start Narrator and tab till submit button and press enter.

What do you expect?
Screen reader should read only the displayed error messages.
What actually happened?
Observe that two error messages are being displayed when user is not filling in the required fields but the name for all the checboxes are being read along with error message which is confusing for the user.
Do you have any screenshots or recordings to repro the issue?
11059109A11y_Intelligent Support_Iris bot_SAPaccessrequest_ScreenReader.webm
Did you find any DOM elements that might have caused the issue?
No response
MAS reference
MAS 1.3.1 – Info and Relationships
WCAG reference
No response
WAI-ARIA reference
No response
Adaptive Card JSON
No response
Additional context
No response

#A11yMAS;#A11ySev2;#MAS1.3.1;#HCL;#Accessibility;#FeatureBotframeworkwebchat-Apr23;

This is an issue with Adaptive Cards. I tried the JSON in AC Designer and repro.
{
  "type": "AdaptiveCard",
  "version": "1.3",
  "body": [
    {
      "type": "ColumnSet",
      "columns": [
        {
          "type": "Column",
          "width": "2",
          "items": [
            {
              "type": "TextBlock",
              "weight": "bolder",
              "text": "Please enter the below details to create SAP user account request",
              "wrap": true
            },
            {
              "type": "Input.ChoiceSet",
              "id": "Roles",
              "value": "",
              "style": "expanded",
              "isMultiSelect": true,
              "choices": [
                {
                  "title": "ES Project Manager",
                  "value": "YR:ES_PROJECT_MGR_0000"
                },
                {
                  "title": "ES Relationship Manager",
                  "value": "YR:ES_RELATIONSHIP_MGR_0000"
                },
                {
                  "title": "ES Operations Specialist - Non-Confidential",
                  "value": "YR:ES_OPS_SPECIALIST_0000"
                },
                {
                  "title": "ES Operations Specialist - Confidential",
                  "value": "YR:ES_OPS_SPECIALIST_CNFD_0000"
                },
                {
                  "title": "ES Operations Specialist - Confidential(USPS)",
                  "value": "YR:ES_OPS_SPC_USPS_CNFD_0000"
                },
                {
                  "title": "ES Operations Pricing Maintenance (Business Add On)",
                  "value": "YRESOP0000"
                },
                {
                  "title": "ES Business Desk Support Read Only - Confidential",
                  "value": "YR:ES_BUS_SUPPT_DISP_CNFD_0000"
                },
                {
                  "title": "ES Business Desk Support Read Only",
                  "value": "YR:ES_BUS_SUPPT_DISPLAY_0000"
                },
                {
                  "title": "ES Operations Read Only - Non-Confidential",
                  "value": "YR:ES_OPS_DISPLAY_0000"
                },
                {
                  "title": "ES Operations Read Only - Confidential",
                  "value": "YR:ES_OPS_DISPLAY_CNFD_0000"
                },
                {
                  "title": "ES Operations Read Only - Confidential(USPS)",
                  "value": "YR:ES_OPS_DIS_USPS_CNFD_0000"
                },
                {
                  "title": "ES Project Controller - Non-Confidential",
                  "value": "YR:ES_PROJECT_CONTROLLER_0000"
                },
                {
                  "title": "ES Project Controller - Confidential",
                  "value": "YR:ES_PROJECT_CTRLR_CNFD_0000"
                },
                {
                  "title": "ES Compliance - Non-Confidential",
                  "value": "YR:ES_COMPLIANCE_0000"
                },
                {
                  "title": "ES Compliance - Confidential",
                  "value": "YR:ES_COMPLIANCE_CNFD_0000"
                },
                {
                  "title": "Enterprise Services Authorization Role (Business Add On)",
                  "value": "YRESAD0000"
                },
                {
                  "title": "ES Business Read Only (Plan and Forecast Access)",
                  "value": "YR:ES_BUS_RO_PLFC_0000"
                }
              ],
              "wrap": true,
              "isRequired": true,
              "label": "Role",
              "errorMessage": "Please select the role"
            },
            {
              "type": "Input.Text",
              "id": "Justification",
              "value": "",
              "isMultiline": true,
              "isRequired": true,
              "label": "Business Justification",
              "errorMessage": "Please provide the business justification",
              "wrap": true
            }
          ]
        }
      ]
    }
  ],
  "actions": [
    {
      "type": "Action.Submit",
      "data": {
        "id": "SAPUserRequest_Submit"
      },
      "title": "Submit"
    }
  ]
}

Please file the issue to Adaptive Cards team.


Taint analyzer change

Purpose
Resolve https://github.com/ballerina-platform/ballerina-lang/issues/15242
Also fixes https://github.com/ballerina-platform/ballerina-lang/issues/16175
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #16200 into jballerina will decrease coverage by 0.09%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##             jballerina   #16200     +/-   ##
===============================================
- Coverage         11.94%   11.85%   -0.1%
+ Complexity         2736     2729      -7
===============================================
  Files              1496     1496
  Lines             72148    72377    +229
  Branches           8633     8691     +58
===============================================
- Hits               8620     8582     -38
- Misses            62719    62990    +271
+ Partials            809      805      -4

Impacted Files
Coverage Δ
Complexity Δ

...g/compiler/semantics/model/symbols/BVarSymbol.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...alang/compiler/semantics/analyzer/TypeChecker.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lerinalang/compiler/desugar/AnnotationDesugar.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ler/semantics/model/symbols/BAnnotationSymbol.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...iler/tree/expressions/BLangTypeConversionExpr.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ang/compiler/semantics/analyzer/TaintAnalyzer.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../compiler/semantics/analyzer/SemanticAnalyzer.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../compiler/semantics/model/symbols/TaintRecord.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../ballerinalang/util/diagnostic/DiagnosticCode.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...erinalang/compiler/parser/BLangPackageBuilder.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1478b0b...aa57fc5. Read the comment docs.