React Native Support

Has there been any consideration for supporting react native?

yes, the build system is already primed for multiple targets. but needs a pr to add src/native. i have no experience with RN but probably would be easy to do, i guess it also doesn't need all the hacks have have in the dom. you interested in filling in?

@drcmda I don't have personal time for it right now but I'm experimenting with above bounty :peace_symbol:

hi @drcmda @JackCA  I have this working in react native but it only returns the width, height, x, y bounds because they are the only ones officially supported by react-native
So is this ok? I can make a note of it in the documentation just to make it clear.

@mul1sh nice. I think you should probably follow the directory structure from react-three-fiber but maybe @drcmda has feedback about that.
The goal is to match the API of the web version so there's probably more to be done to match it there.

I'm also not sure the async change is going to work in practice

@JackCA thanks for the feedback, will do.

@JackCA  in order to follow the directory structure of react-three-fiber, I would have to add another dependency to this project i.e. react-reconciler is this ok?

@mul1sh are you sure? By that I just meant moving to src/native in this repo (since there is already src/web.

@JackCA Yes we need to add react-reconciler otherwise how will react know to switch to the react native implementation when running in react native? If anything thats how its been done in react-three-fiber so switching the directories is not enough 🙂

Any thoughts @JackCA  want to wrap this up by latest tomorrow

It's possible that using that as an example was misguided on my part, because that structure may only be that way because it uses react-reconciler (which is supposed to be for custom renderers and that's definitely not what this library is).
@drcmda can you speak to how @mul1sh should organize this? (and any other comments on code)

hey everyone, i wasn't online over the holidays. the async bit is a bit weird. hooks can't be awaited, that would only go for suspense. it should wait for the promise in there (.then(r => ....)) and then fill the data in.
also, using the navigator ... i think it would be best to fork index.ts, rename it index-native.ts, remove the web bits and just add the RN stuff in there. in package.json we can link this file using the "native" field.

Thanks @JackCA @drcmda for clarifying m questions, wrapping this up now 🙂

Doing my final testing now and then I submit in abit.

@JackCA  sorry but after many testing out many fixes, I don't think there is a way to get the left, right, top and bottom co-ords of a view in react native. The only way this seems possible is if the view is wrapped by a DOM Node and then using getBoundingClientRect() to get the co-ords something already supported by this lib.
So should I just submit it the way it is, where you will be able to get the x,y, width, pageX & pageY bounds of a view in react native as supported by the official api?

hey @mul1sh -- I'm a little confused because you mention DOM nodes which is on the browser side of things.
What is holding us back from using measure and/or measureInWindow in this implementation exactly?

@JackCA nothing is stopping me from using 'measure' or 'measureInWindow', its just that they won't return the left, right, top and bottom co-ords for react-native views like the current api does for html dom nodes via 'getBoundingClientRect()'
Sorry for the confusion this is what I meant, so should I just submit it like this or what else do you propose be done?

Sorry still debugging, sorting one final bug..

Sorry for the long delay but everything is fixed, doing 1 last test and then I submit in a few hours :)

Still testing this just a bit

Hey @JackCA  an update on this. I've been facing some issues with getting the RN tests to work well in iOS, they kept failing randomly but I think I finally see the issue. So kindly give me less than 3 days and I will have this fully working in both android & iOS plus the tests working successfully in these platforms too.

I think I've finally resolved all the issues with the tests. Doing 1 last manual test and then I submit my PR.

Almost done testing, will be submitting in a bit.

Finally got things to work well, so sorry this has taken so long :( submitting in a bit

Annd I got another bug, so still on it, sorry :(

Finally I think everything is working as it should, but still doing some final tests to make sure of this and then I submit :)

Almost done testing..

Finally done after a long time, preparing a PR to submit now

Sorry still polishing the PR abit and then I submit

Final update i think is good to go, will submit it in a bit :)

Still working on the PR..

Just testing now, everything looks great so far, i'm definitely submitting in a few :)

I got some more final bugs to deal with but I sorted them today, i'm confident i'll submit this by tomorrow latest, no more delays :)

Hey! Did that PR end up getting merged?

Is the PR up somewhere?


Handle pg_settings names with '.'

Fixes #64

Coverage remained the same at 51.815% when pulling<PHONE_NUMBER>61327c63f34760bb5a756af53e419e on iterion:topic/pg_setting_patch into 98ba5663221a514b0d334780eb3651b9838c42c7 on wrouesnel:master.

Superb - merged. I think with that I'll do another release.


Bumping datasets with version suffix

How are we expecting the bumping of datasets with version suffixes to work?
Currently we don't do what I thought (the suffix is essentially only used to look up the dataset).
Should "v1.1.1-test" patch bump to "v1.1.2-test", currently the suffix is lost after bumping

I don't think it makes sense to bump datasets with a suffix.     The suffix might be something like rc2, in which case "bump" could mean change to rc3, but we don't know that and we can't check for all plausible conventions.   The structure is totally up to the user.  I think we raise ValueError, just as is done for other bad input parameters.

Addressed in #97

Decided we do not want to deal with how do auto-bump a dataset that has a suffix.

Therefore if the user tries to auto-bump a dataset with a suffix an error is raised.
A user can work with version suffixes and bumping manually, when registering the updated dataset they specify the version and suffix manually rather than trying to auto bump.


Add imagelayers.io badges.

having issues with imagelayersio at this moment

Already reported here
https://github.com/CenturyLinkLabs/imagelayers/issues/29

Might be the new layer format.
Closing this one, we'll just do the micro badger thing.


Dynamic Group performance improvements for relationships in filtersets.

Related to: #3269
What's Changed
Some of my assertions may be incorrect. Please keep me honest.
This change is related to improving the way Dynamic Groups members are collected since they are built around filtersets. In doing some profiling it was determined that the table for relationships is hit twice, with one (1) query to the table for content types as well, resulting in three (3) queries each time a Dynamic Group is constructed from a filterset.
With these changes, this reduces the construction of a filterset that relationships down to a single query.

Changed RelationshipModelFilterSetMixin to perform a single OR query including select_related for source_type and destination_type vs. two single queries for each source/destination types.

Relationships lookup
Before
This query is performed once for each source/destination plus one for content type:
14:39:03.078 DEBUG   django.db.backends   utils.py                             debug_sql() :
  (0.008) SELECT "extras_relationship"."id", "extras_relationship"."created", "extras_relationship"."last_updated", "extras_relationship"."name", "extras_relationship"."slug", "extras_relationship"."description", "extras_relationship"."type", "extras_relationship"."required_on", "extras_relationship"."source_type_id", "extras_relationship"."source_label", "extras_relationship"."source_hidden", "extras_relationship"."source_filter", "extras_relationship"."destination_type_id", "extras_relationship"."destination_label", "extras_relationship"."destination_hidden", "extras_relationship"."destination_filter", "extras_relationship"."advanced_ui" FROM "extras_relationship" WHERE (NOT "extras_relationship"."source_hidden" AND "extras_relationship"."source_type_id" = 3) ORDER BY "extras_relationship"."name" ASC; args=(3,)
14:39:03.079 DEBUG   django.db.backends   utils.py                             debug_sql() :
  (0.000) SELECT "django_content_type"."id", "django_content_type"."app_label", "django_content_type"."model" FROM "django_content_type" WHERE "django_content_type"."id" = 11 LIMIT 21; args=(11,)
14:39:03.080 DEBUG   django.db.backends   utils.py                             debug_sql() :
  (0.000) SELECT "extras_relationship"."id", "extras_relationship"."created", "extras_relationship"."last_updated", "extras_relationship"."name", "extras_relationship"."slug", "extras_relationship"."description", "extras_relationship"."type", "extras_relationship"."required_on", "extras_relationship"."source_type_id", "extras_relationship"."source_label", "extras_relationship"."source_hidden", "extras_relationship"."source_filter", "extras_relationship"."destination_type_id", "extras_relationship"."destination_label", "extras_relationship"."destination_hidden", "extras_relationship"."destination_filter", "extras_relationship"."advanced_ui" FROM "extras_relationship" WHERE (NOT "extras_relationship"."destination_hidden" AND "extras_relationship"."destination_type_id" = 3) ORDER BY "extras_relationship"."name" ASC; args=(3,)

Analysis:

Source: https://explain.depesz.com/s/Enul
Destination: https://explain.depesz.com/s/Qqkj
Content type: https://explain.depesz.com/s/idys

After
This query is performed combining source/destination:
14:41:24.598 DEBUG   django.db.backends   utils.py                             debug_sql() :
  (0.014) DECLARE "_django_curs_8739029312_sync_1" NO SCROLL CURSOR WITH HOLD FOR SELECT "extras_relationship"."id", "extras_relationship"."created", "extras_relationship"."last_updated", "extras_relationship"."name", "extras_relationship"."slug", "extras_relationship"."description", "extras_relationship"."type", "extras_relationship"."required_on", "extras_relationship"."source_type_id", "extras_relationship"."source_label", "extras_relationship"."source_hidden", "extras_relationship"."source_filter", "extras_relationship"."destination_type_id", "extras_relationship"."destination_label", "extras_relationship"."destination_hidden", "extras_relationship"."destination_filter", "extras_relationship"."advanced_ui", "django_content_type"."id", "django_content_type"."app_label", "django_content_type"."model", T3."id", T3."app_label", T3."model" FROM "extras_relationship" INNER JOIN "django_content_type" ON ("extras_relationship"."source_type_id" = "django_content_type"."id") INNER JOIN "django_content_type" T3 ON ("extras_relationship"."destination_type_id" = T3."id") WHERE ((NOT "extras_relationship"."source_hidden" AND "extras_relationship"."source_type_id" = 3) OR (NOT "extras_relationship"."destination_hidden" AND "extras_relationship"."destination_type_id" = 3)) ORDER BY "extras_relationship"."name" ASC; args=(3, 3)

Analysis:
https://explain.depesz.com/s/xR8z
TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

How much do the query optimizations (single filter vs two filters) get us vs the index_together? Just curious to avoid a reordered migration in next.

On secondary review, I don't think the indexes have made any changes at all here. At least not on deeper inspection of the explain queries analysis.

How much do the query optimizations (single filter vs two filters) get us vs the index_together? Just curious to avoid a reordered migration in next.

On secondary review, I don't think the indexes have made any changes at all here. At least not on deeper inspection of the explain queries analysis.

I ripped out the new indexes, as I feel my initial assertions about them working was incorrect.


Fix usedef issue with parfor loopnest variables.

Resolves #5597
The parfor callback for usedef analysis was slightly wrong when it came to handling variables used in the loopnests.  They were added to the enclosing use set regardless but this is wrong.  If those variables are present in the enclosing def set then this PR makes sure that they aren't added to the use set.  The bug manifested as one of the slice variables being detected as a race, when it isn't, and then the pre-existing parfor code to handle this situation kicked in which was not expecting this situation and so something asserted.

Ooops...seems like I merged in Siu's changes thinking it might be a phi issue. We'll need to undo that.

@DrTodd13, The revert commits will be undoing the my PR when merged. You interactive rebase your branch to remove the unwanted commits instead.

Squash merge?


DSC Resource for managing W2K12+ Distributed File System (DFS) Namespaces and Replication

I have created a DSC Resource Module for managing Distributed File System (DFS) Namespaces and Replication on Windows Server 2012+: https://github.com/PlagueHO/cDFS
It contains MOF based resources for the following:

cDFSRepGroup - This resource is used to create, edit or remove DFS Replication Groups. If used to create a Replcation Group it should be combined with the cDFSRepGroupMembership resources.

cDFSRepGroupConnection - This resource is used to create, edit and remove DFS Replication Group connections. This resource should ONLY be used if the Topology parameter in the Resource Group is set to Manual.

cDFSRepGroupFolder - This resource is used to configure DFS Replication Group folders. This is an optional resource, and only needs to be used if the folder Description, FilenameToExclude or DirectoryNameToExclude fields need to be set. In most cases just setting the Folders property in the cDFSRepGroup resource will be acceptable.

cDFSRepGroupMembership - This resource is used to configure Replication Group Folder Membership. It is usually used to set the ContentPath for each Replication Group folder on each Member computer. It can also be used to set additional properties of the Membership. This resource shouldn't be used for folders where the Content Path is set in the cDFSRepGroup.

cDFSNamespaceRoot - This resource is used to create, edit or remove standalone or domain based DFS namespaces.  When the server is the last server in the namespace, the namespace itself will be removed.

cDFSNamespaceFolder - This resource is used to create, edit or remove folders from DFS namespaces.  When a target is the last target in a namespace folder, the namespace folder itself will be removed.

cDFSNameSpace - This resource is now deprecated. Use cDFSNamespaceRoot and cDFSNamespaceFolder instead. It will be removed from future versions.

It contains Unit and Integration tests for all resources, using the DSCResource.Test\TestHelper.psm1 functions. However, the integration tests for the DFS Replication require an Active Director domain and at least two servers with DFS installed.
Note about tests: Tests require Windows Server 2012 (or above) to pass because required DFS Namespace and DFS Replication features to be installed. Tests pass on AppVeyor because AppVeyor.yml is configured to ensure the required features are installed.
Would there be interest in moving this module over to the community module library here?
If so, I'd need to convert the resources over from BMD_ to MSFT_ and other updates to the .PSD1,AppVeyor.yml etc before transferring the repo.
Thanks!

PlagueHO, I think this is a great idea. I use this resource on 27 servers in production. It would be a great addition to the community module library.

@Zuldan, thanks! That is excellent to know this is getting some use out there in the world :smile:
@KarolKaczmarek - if you are OK with adding this resource to the community ones, let me know and I'll make the necessary name changes (MSFT_/x/appVeyor.yml etc) and let you know once they are complete.
Then all someone (@tysonjhayes, @TravisEz13? :smile:) needs to do is a code review and give me the :+1: to transfer the repo.
Thanks!

Yes, it would definitely be good to have them here!

@KarolKaczmarek - Great! I'll convert the repo over to MSFT and x and then post a comment here when it is ready to be reviewed.

All changes done and ready to transfer over. Module is now currently available in this repo:
https://github.com/PlagueHO/xDFS
Once the review has been done, let me know and I'll transfer it over.
Thanks!

https://github.com/PlagueHO/xDFS/blob/dev/DSCResources/MSFT_xDFSNamespaceFolder/MSFT_xDFSNamespaceFolder.schema.mof , ensure should be a key

Awesome! Thanks @TravisEz13 for looking at this for me :+1:
Regarding making Ensure a key; wouldn't this then mean that a single namespace (or any of the other resources) could be made both Present and Absent in the same config? For example, it would allow a config like this:
        xDFSNamespaceFolder DFSNamespaceFolder_Domain_Finance_Present
        {
            Path                 = '\\contoso.com\departments\finance'
            TargetPath           = '\\fs_3\Finance'
            Ensure               = 'present'
            PsDscRunAsCredential = $Credential
        } # End of xDFSNamespaceFolder Resource
        xDFSNamespaceFolder DFSNamespaceFolder_Domain_Finance_Absent
        {
            Path                 = '\\contoso.com\departments\finance'
            TargetPath           = '\\fs_3\Finance'
            Ensure               = 'absent'
            PsDscRunAsCredential = $Credential
        } # End of xDFSNamespaceFolder Resource

This would then constantly add and remove this DFS folder over and over - which would be quite bad. I could be quite wrong though.
I'll get the other stuff done ASAP. I'll also change this to v<IP_ADDRESS> so that it is clear that there are some structural changes to the resources.

@PlagueHO No, you are correct, ensure should not be a key...

@TravisEz13 - I'll rename all "Rep" to "Replication". I think it'll be best if everything is the same (would be confusing if some things were Rep and others Replication).
I'll get this change made this week! Thanks again for reviewing this :+1:

Sounds good

@TravisEz13 - All changes finished now. Hopefully ready to go.
Cheers

There are some passwords hard coded in the samples otherwise look pretty good to me.

Good call - I'll change the passwords in the samples to come in via parameters.

@TravisEz13 - Done. All samples now use Read-Host and/or Get-Credential to read in the info. This eliminates the PSSA errors.

@KarolKaczmarek This look good to me

Cool. I can transfer this to PowerShell as soon as @KarolKaczmarek gives the OK. Thanks for reviewing this @TravisEz13

Thanks @PlagueHO
You can go ahead and transfer it.

@KarolKaczmarek  - thank you! @TravisEz13  - I've transferred it over to PowerShell now. Not sure if there is anything more I need to do here?
Once you've released a version of this as xDFS, I'll remove cDFS off PSGallery.

@KarolKaczmarek, @TravisEz13 - I thought I had transferred this, but I hadn't - because I don't have rights to transfer to the PowerShell org:

One solution is I could transfer the repo to one of you and you could then transfer it into the PowerShell org.
Here is a document covering transfers that I was using:
https://help.github.com/articles/transferring-a-repository/

Sure, feel free to transfer it to me and I'll move it to PS org

@KarolKaczmarek - cool. Done. Thank you!

Thanks, I've transferred it to our org, onboarded to our appveyor and waffle.io accounts and added as a submodule to DscResources. There were small updates needed to psd1 file, but otherwise it was ready to publish, so it's here: http://www.powershellgallery.com/packages/xdfs/<IP_ADDRESS>
Thanks for contributing and congrats on "surviving" our onboarding process :) I know it's not the easiest, but we try to keep the quality of contributions high, so once again thank you for doing this!
Let me know when you unpublish cDFS from PSGallery and we can close this issue.

Awesome! Thanks @KarolKaczmarek . It is a pleasure contributing and really wasn't a difficult process. This was the smallest of my modules I'm onboarding with the PowerShell team, so I feel sorry for your reviewers (thanks @TravisEz13 :smile: ). The cFSRM module has twice the number of resources as this one.
cDFS has been unlisted from the Gallery now so I'll close this.
Cheers to you and your team :+1:

@PlagueHO, well done mate. Impressive stuff.
Now onto to next resource! ;-)

Cheers @Zuldan :+1: cISCSI next perhaps?

@PlagueHO reading my mind :)


Fixing class.name issue

When initializing the same class multiple times - we were failed to put the .name on the new
function object. That is because we initialize the object by EnsureObjectReady before setting the homeobject.
Fixed that by setting the name after setting the homeobject.

@boingoing @pleath @LouisLaf please have look.

@zenparsing please take a look

@akroshg I'm curious: if this change takes care of subsequent class definition evaluations, where does the name property get set the first time the class definition is evaluated?

@zenparsing in the InitClass, that's where the Object is getting ready for the first time.

ping...

ping..


create our own vulnerability scan jobs

I would like to know if there is the possibility that I can create a scanning job (i.e scan-vulnerabilityreport-*) for a set images and then the operator does the rest of work like creating the custom report and also the Prometheus metrics. is that possible?

@RasheedJad the answer is yes, if you will create the scan job with the appropriate labels on trivy-operator namespace,  the operator will reconcile the scan job and create reports and metrics

@chen-keinan  any idea about the "appropriate labels" or where are they mentioned in docs or somewhere else so I use them?

@RasheedJad no docs for it ,you can check the code , however the simple way is run a scan and view it job description.
like this :  kubectl get <scan-job> -n trivy-system -o yaml

@RasheedJad could you please advice on use-case for creating own vulnerability scan jobs

I actually gave up on trivy for a tool called NeuVector and after trying the new tool, I am more satisfied with it.

I actually gave up on trivy for a tool called NeuVector and after trying the new tool, I am more satisfied with it.

thank you for the response . do you mind sharing what was not satisfying for you whilst using trivy tool ?

closing as not relevant.


bug: local mode restarts for no reason when wrangler starts up

It happens only on beta/main, logging this so I remember to investigate this asap.

Isolated to this change https://github.com/cloudflare/wrangler2/pull/1496


css assets use absolute path got error by required in local folder structure but not remain string

Version
5.0.1
Environment info
  System:
    OS: macOS 11.6.4
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
  Binaries:
    Node: 14.17.5 - ~/.nvm/versions/node/v14.17.5/bin/node
    Yarn: 1.22.11 - ~/.nvm/versions/node/v14.17.5/bin/yarn
    npm: 6.14.14 - ~/.nvm/versions/node/v14.17.5/bin/npm
  Browsers:
    Chrome: 98.0.4758.109
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.3
  npmGlobalPackages:
    @vue/cli: 5.0.1

Steps to reproduce

install latest v5.0.1 @vue/cli in global
copy src/assets to public/assets
edit App.vue as following

<template>

  <HelloWorld msg="Welcome to Your Vue.js App"/>
</template>

yarn serve

errors in terminal

Compiled with problems:X

ERROR in ./src/App.vue?vue&type=style&index=0&id=7ba5bd90&lang=css (./node_modules/css-loader/dist/cjs.js??clonedRuleSet-12.use[1]!./node_modules/vue-loader/dist/stylePostLoader.js!./node_modules/postcss-loader/dist/cjs.js??clonedRuleSet-12.use[2]!./node_modules/vue-loader/dist/index.js??ruleSet[0].use[0]!./src/App.vue?vue&type=style&index=0&id=7ba5bd90&lang=css) 5:36-80

Module not found: Error: Can't resolve '/assets/logo.png' in '/Users/wangj/Desktop/johnny/prac/absolute-path/src'

What is expected?
in v4.5.0 the absolute path in url will remain original string which is same with absolute path in
What is actually happening?
absolute path in url got mistakenly required in local env

I just found out that this error is due to css-loader's "url" options by default sets to "true", just sets the css options "url" to false as following solved this issue, webpack-style-loader-css-loader-url-path-resolution-not-working
const { defineConfig } = require('@vue/cli-service')

module.exports = defineConfig({
  transpileDependencies: true,
  css: {
    loaderOptions: {
      css: {
        url: false,
      }
    }
  },
}

https://cli.vuejs.org/guide/html-and-static-assets.html#url-transform-rules

https://cli.vuejs.org/guide/html-and-static-assets.html#url-transform-rules

From docs: - If the URL is an absolute path (e.g. /images/foo.png), it will be preserved as-is.
No notes about special behavior on url in css.
In this issue URL is an absolute path (/assets/logo.png), but is not preserved as-is and loaded as a module. In v4 it works as expected.

Bump, exactly!
Should be reopened imo, have exactly the same problem 👍

Bump, exactly! Should be reopened imo, have exactly the same problem
+1

Bump, exactly!
Should be reopened imo, have exactly the same problem 👍

we agree!

Spent hours trying to figure this out, so I agree, this should definitely be looked at again and can be very easily be reproduced on a clean vue-cli 5 projec. Everything is exactly as described here:

https://cli.vuejs.org/guide/html-and-static-assets.html#url-transform-rules

From docs: - If the URL is an absolute path (e.g. /images/foo.png), it will be preserved as-is. No notes about special behavior on url in css.
In this issue URL is an absolute path (/assets/logo.png), but is not preserved as-is and loaded as a module. In v4 it works as expected.

We used an absolute path, but webpack was still trying to look for the asset inside /src not from /public.
Setting url to false fixed the issue for us as well.

I used an absolute path ('/') to import a file into main.js from public dir and had the same problem.

Okay, but i just found out that setting url option to false, breaks relative file paths in SFC style block. So for example, now
/fonts/icomoon.eot
as a absolute path works and compiles, but using relative path in SFC style block is not working, with 404 for the asset being referenced:
background-image: url('../assets/checkbox-check.svg');
checkbox-check.svg:1          GET http://localhost:8080/assets/checkbox-check.svg 404 (Not Found)

You can pass url: false to css-loader to prevent this.  Finally, I'm glad I found it here.
The project has been running well until today when I upgraded @ vue/cli 5.0.8. Damn it


Add note to Electron 2.0 release notes about dropping mips64el support

This should also be added to https://github.com/electron/electron/releases/tag/v2.0.0

Thanks 👍


provide a way to easily extend the classpath of the broker

To use jdbc persistence, currently requires an init container. broker properties can now replace the xml config changes, however getting the jdbc client jars on the classpath is problematic without an init container.
One thought of using a config map with -cp postfix is not a flyer b/c jars will typically be too large for etcd. ( > 1M)
However having something similar for an extra mount may make sense. An extra mount with some postfix or annotation can be added to the classpath of the broker or the lib directory or lib extra directory.
There is one caveat, the broker run command creates an isolation classloader for the broker, which means that any jar on the classapth won't have access to artemis interfaces, so a plugin jar would need to be in the lib directory.
An agent jar would need to be on the classpath and some way to disable the isolation classloader would work. (The jolokia jvm agent needing access to the Artemis JAAS callback handler is a concrete use case).
A jdbc jar would work on the classpath or in the lib dir as it has no need to callback to any artemis interface.
We may want to fully manage the classpath of the broker, ie: avoid the run command, or have some Artemis option to disable the isolation class loader.

Allow broker classpath extension using custom paths
https://issues.apache.org/jira/browse/ARTEMIS-4444

The combination of the resource templates allowing the provision of a custom init container that can copy resources and the ARTEMIS-4444 classpath extension will suffice for this use case.


Oracles data could not be loaded

Description
https://explorer.vega.xyz/oracles
Steps to Reproduce

Go to https://explorer.vega.xyz/oracles

Expected behavior
Oracles are listed.  For now we need to work around it.

https://github.com/vegaprotocol/vega/issues/9207

Unblocked now that the bug is fixed in dev and testnet, but in mainnet no deploy has happened. The REST response is equally useless, so the fix here is to do a better job of handling the error. Last resort: turn off the oracle pages.

Fixed on mainnet


Added ifacconf-typst to conferences

Repo URL: https://github.com/avonmoll/ifacconf-typst

Thanks for the awesome contribution, @avonmoll !


kotlin language version not updated to 1.4 with gradle 6.8-M1

Gradle 6.8-M1 uses kotlin 1.4.10, but does not allow to use new kotlin 1.4 features (like trailing commas). This is probably due to src/kotlin-dsl/org/gradle/kotlin/dsl/KotlinSettingsScript.kt defining "-language-version", "1.3", "-api-version", "1.3". Please update to "1.4".

This was done on purpose because it is a breaking change for existing builds.
Scripts will move to language and api 1.4 on the next major version, 7.0.
See https://github.com/gradle/gradle/issues/14888
Also see the discussion in #12660 for some more context


write more thorough documentation

There's a cluster howto right now, but we'll need some more detailed documentation before people go and try to run this in production.
For example, growing the number of serving nodes, and how you'll have a delay before they serve after you bring them up, as they'll have to sync their local database...

Would be helpful to have a list of Ubuntu/Debian packages that are necessary to build. I believe the list of dependencies in the README is complete, but a specific list ready to pass on to apt-get install would be very convenient and avoid many questions.
Here's a starting point:

protobuf-compiler
libprotobuf-java
libprotobuf-dev
python-dev
libjson-c-dev
libgoogle-glog-dev
libgflags-dev
libldns-dev
google-mock
libgtest-dev
libsqlite3-dev
cmake
ant
libevent-dev


chore: update .gitattributes

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Covariant Return Type Issue

When Class B extends Class A, my understanding is that type safety is preserved when the methods of Class B have a return type that is covariant with respect to the return types of the same methods on Class A (if present). In the example flow try link below, this is verified in the Dog extends Animal check I provide. However, when extending the Array class with a custom List class and implementing map, flow detects an error when returning List<U> from the method, even though it extends Array<U> and is therefore a subtype. Is there something I am missing or is this a bug?
Offending Code:
class List<T> extends Array<T> {
  map<U>(callbackfn: (value: T, index: number, array: Array<T>) => U, thisArg?: any): List<U> {
    return (null: any);
  }
}

Link:
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoAxjAhgZx2AQQDsBLAWyxgB4CA+MAb1TDAoAcqAhWgCgEoAXIVIVq3RmABOAUwAuAV0lEwPIvJgwhWIgE8+YAL6ojGbHjAAROAHMa9aQA9Z0ogBN8xcpTuNmrLBzc-EJWtuIMUnKKyqrqmmDaeobGqKiObHCSsmCu0lBY6tmYuPgAMiQ4slQAKvZOLu6EkpJYOjX0TCzsVACqvOiUMABGWOgA1lBEQjwAbpTy0kLVADRgJG6OQmpkQ9KSq1jNrUIER221+gC89D2rsgAWFafWAPxauoJg5ZW9HX4sMgUShUag07z0AG4-EYTKggA

Seems to work now


Added Kappa Measure

I am new to contributing to open source repositories. Please let me know if there is something else that I need to do or if there is scope for improvement. Thanks.

Thanks @Rahulub3r. Thanks for the contribution. Could you please add some tests.

Codecov Report

Merging #738 (0c1abdb) into dev (6914722) will decrease coverage by 0.14%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##              dev     #738      +/-   ##
==========================================
- Coverage   85.09%   84.95%   -0.15%
==========================================
  Files          36       36
  Lines        3395     3402       +7
==========================================
+ Hits         2889     2890       +1
- Misses        506      512       +6

Impacted Files
Coverage Δ

src/measures/finite.jl
96.14% <14.28%> (-2.07%)
:arrow_down:

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 6914722...0c1abdb. Read the comment docs.

@Rahulub3r The tests live in the test/ folder in a file of the same name as you edited in src/

@Rahulub3r The tests live in the test/ folder in a file of the same name as you edited in src/

@ablaom @OkonSamuel Added the test cases. Let me know if I need to do anything else.

Thanks @Rahulub3r

@Rahulub3r Just wanted to thank you for this work. I appreciate it takes some work to wrap your head around the interface and if you are also new to git workflow, that only makes the path to a successful PR steeper.
Great to have your contribution.
I you are interested and available to help out further, there is a lot to be done in the measures area, in particular just adding missing measures.  See this filtered project list and in particular this list (which needs updating).

@ablaom You're welcome. I will look into the list and see how I can contribute.


libminisat.so.2 not found when running scripts/symbiotic

When running Symbiotic from the scripts directory, it crashes because the libminisat.so.2 file is not found.

Should be fixed by bd0ed911f1


Constants SQL_TYPE_UTCDATETIME and SQL_TYPE_UTCTIME aren't declared anywhere

The constants SQL_TYPE_UTCDATETIME and SQL_TYPE_UTCTIME are currently cited here:
SQL Data Types
https://docs.microsoft.com/en-us/sql/odbc/reference/appendixes/sql-data-types?view=sql-server-2017
But they're not declared in any header I can find, nor can I find any relevant info on them after extensive searching. Why are they documented?

@rloutlaw Can you please take a look at this issue?

@ryanmajidi This needs to be assigned to the SQL docs team, who manages their reference docs.

Moving this to the sql docs repo issue 2248


raw2probabilityInPlace

https://github.com/Azure/mmlspark/blob/9ce617b5fd11654ae124018a905f5df1cf50077c/src/lightgbm/src/main/scala/LightGBMClassifier.scala#L91-L101
I have found the probability been calculate by  1 / ( 1 + e^(-2 * x)),
normally it should be  1 / ( 1 + e^-x) ,  could you please tell why the x should be multiplied by 2 ? Thanks for advance.

@tanjiaxin please see page 8 of Friedman (1999) "Greedy Function Approximation? A Gradient Boosting Machine".  They use -2 there for the loss function, and from there they use -2 in the probability estimates.  This is also what spark ML uses for their GBTClassifier, please see the reference PR here:
https://github.com/apache/spark/pull/16441
I will try to find the loss function formula in lightgbm and validate whether the 2 is needed here.  It makes a slight difference in the distribution of the probability values but it shouldn't make a difference in terms of the predicted label, right?  As long as X = 0 corresponds to the hyperplane and positive values are on one side vs negative on the other the predicted labels should be the same.

I think you are right, it looks like lightgbm's GetGradients, which is used in training, doesn't have the -2:
https://github.com/Microsoft/LightGBM/blob/c920e6345bcb41fc1ec6ac338f5437034b9f0d38/src/objective/binary_objective.hpp#L102
And the convert output which is used in prediction also doesn't seem to have it:
https://github.com/Microsoft/LightGBM/blob/c920e6345bcb41fc1ec6ac338f5437034b9f0d38/src/objective/binary_objective.hpp#L157
I'll update it to remove the -2.

Thanks for your explaination.


Can't add new tag from mobile device

Hello,
Tags editor is working fine on desktop but can't add new tags from mobile device.
After writing tag name on input, focusing out of the tag input field, isn't adding a new tag like it did for desktop.
Is there any solution for mobile device?
Thanks

Any one has any idea?

I recommend you check the more advanced options in the MDB UI Kit:
https://mdbootstrap.com/docs/standard/components/chips/

@filipkappa Thanks for the suggestion but i am looking to get a solution for bootstrap tags input.
Any one has any clue to fix the mobile issue for tags editor?


Dictionaries are not serialised correctly when the service protocol is query

When serialising dictionaries in queries they are currently being serialised as
<dictionary name>.<entry key>=<entry value>

The output should be of the form
<dictionary name>.entry.N.key=<entry key>
&<dictionary name>.entry.N.value=<entry value>

where N is the index of the key, value pair.
An example of this is SNS.createTopic() set one of the attributes and you get a MalformedInput error returned back from AWS.
I have had a look at how to resolve this. The best I've come up with so far is https://github.com/adam-fowler/aws-sdk-swift-core/blob/serialize-dictionary/Sources/AWSSDKSwiftCore/AWSShapes/Dictionary.swift. I have hand replaced some of the dictionaries with this DictionaryShape class in the SNS Shape classes and it fixes the issue.

@adam-fowler it appears we are missing support for serializing complex structures to query parameters - here is the ruby sdk test that demonstrates some of the other variants.
I assume you looked at JSONSerializer.serializeToFlatDictionary in the core lib? This is where the <dictionary name>.<entry key>=<entry value> comes from.
Did you try fixing it there? you might try something like.
for (i, kvp) in dictionary.enumerated() {
    let key = kvp.key2
    let value = kvp.value
...

flatted["\(key).entry.\(i).key"] = key2
flatted["\(key).entry.\(i).value"] = value2

Let me know I can make a branch for you to try it. I do worry considering how many variants there are it might break something else. It seems pretty clear that a [String:String] dict should be serialized this way, so it may make sense to limit the change to just this type of dict.

Hi,
I attempted to do the change at the serializeToFlatDictionary() level but realised this wouldn’t be possible. The Json for a dictionary looks pretty much the same as the json for a structure so there is no way to tell them apart when flattening the dictionary. That is why I tried to do this at a point where we knew for sure we were dealing with a dictionary. It’s not the cleanest of changes but it only affects dictionaries used within queries so cannot break anything else.
I have changes to the code generator that output this new format dictionary for queries. The only problem is when I run the code generator is reorders the parameters for loads of functions.

Found a way to fix this. Added a new encodeToQueryDictionary function that creates the query flat dictionary directly from the AWSShape instead of it going via a json structure. The code is in https://github.com/swift-aws/aws-sdk-swift-core/pull/53 along with a bunch of other fixes.

Fixed with https://github.com/swift-aws/aws-sdk-swift-core/pull/53


Add generic motor control interface

Tiet has requested an interface that will let us send motor commands to specific motors via driverstation. We can probably use LiveWindow?

am I understanding correctly, Mr Tiet wants to be able to send speed and directional information to specific motor like the flywheel through driver station?

yes


PanelAttention: Fix delay

Problem:
If you move your mouse from another panel quickly, it means that the mouseover will be executed before the debounce mousemove on the previous panel. This causes inconsistency.
Solution:
Debouncing onMouseEnter too prevents this, but if you press e within 100ms of entering the new panel, it will always enter the old panel.
If this approach isn't good enough, we could try to decrease the debounce time. If there is any other approach that I could consider, I'm all ears!
Closes https://github.com/grafana/grafana/issues/90020

📦 Published PR as canary version: 5.9.1--canary.867.10450028526.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>yarn add<EMAIL_ADDRESS>

@tskarhed see the comment from @ashharrison90

@torkelo Yes, we tried to fix it, but we just couldn't get it running when linking Scenes. I could make the code change, but I have no way of testing it. Since I get no error, I can only assume it has to do with some incompatible version of Grafana UI, although updating the dependency locally did not do the trick.
I would really appreciate some help with testing it, I had similar issues the last couple of times working with Scenes.

@tskarhed it looks like we still have the problem that when yarn linking scene into main repo the grafana/ui dependency is not the one from runtime (core) but the scenes get's' it's own copy bundled in (using the version specified in this repos package json)
I recall that we had fix this

@tskarhed so the reason it's not working now when testing this is that PanelChrome used is an old version that does not have onMouseEnter onMouseLeave etc,
added a release label to this PR, we can test with the npm canary release that will be generated

i've hacked it locally (just updated scenes-app to use canary versions of all the @grafana/ packages)...
and the leading option works, but you have to specify both leading and trailing, like:
  const debouncedMouseMove = useMemo(() => debounce(setPanelAttention, 100, {
    leading: true,
    trailing: false,
  }), [setPanelAttention]);

I still can't get this to run. I changed to "@grafana/ui": "11.3.0-192050" in Scenes' package.json. If someone else could verify that this works, it would be great.

i can verify this works. you probably don't want the changes in scenes/package.json and yarn.lock tho

Oops 😢

i installed Grafana v11.3.0-75433 (cdf035f813)  to address a different (un-related) bug,  and wanted to make a note here that this is still occurring (ie there is still a large lag/delay on pressing e to edit a panel, so you frequently end up editing the wrong panel accidentally).
apologies if this is planned to be fixed in a different version, but i wanted to bring this up (As from the best I can tell, this merge should be in the nightly that im currently running - ie  v11.3.0-75433 ).
happy to provide any further data/assistance
thank you

Confirming. Still an issue on v11.3.0-196835 (d1ffcc22d9)

@bob454522 @frifox Can you reproduce this on https://play.grafana.org/ ? It is running Grafana v11.3.0-75826

Seems to be fixed.
Don't know how play.grafana.org version tags correlate to docker images, but:

Upgraded my local docker instance to grafana/grafana-oss-dev:11.3.0-198777 = it's fixed.
Downgraded to older grafana/grafana-oss-dev:11.3.0-196835 image = bug is back.

I am not exactly sure when the Scenes version bump happened in core Grafana, but I bet that's when it was fixed. As for the version - 11.3 hasn't been released yet, so it should only be solved in either dev versions or Grafana Cloud until then.


Check page titles

Check if page titles are correct for every shown component.

[x] static pages
[x] event tabs
[x] other pages

Implemented by #98


Bug with non-breaking spaces in 0.3.0?

Hi,
I've found a very weird case where html2text returned broken output after upgrading to 0.3.0, and I was able to narrow it down to this line:
$html = str_replace("\xa0", " ", $html);
I examined the contents of $html in a hex editor immediately before and after the line and found this diff. The original source snippet reads "für Ihre ", with the spaces being nbsp's, and is UTF-8 encoded.
Before: 66 c3 bc 72 c2 a0 49 68 72 65 c2 a0
After: 66 c3 83 c2 bc 72 c3 82 20 49 68 72 65 c3 82 20
So apparently the nbsp's (c2 a0) have been transformed into c3 82 20, which looks like a regular space (20) but with some gibberish in front of it. Also, the multi-byte character 'ü' (c3 bc) is now c3 83 c2 bc, which is also nonsensical.
I've downgraded to 0.2.3 and all is fine now, but I'd like to let you know in case you'd like to look into this.

This definitely sounds like sadness, some weird interaction between UTF-8 and str_replace. Maybe there needs to be some utf8_encode()/utf8_decode(), maybe there needs to be some iconv/mbstring, maybe html2text simply can't support this... What was the hex output of html2text 0.2.3 on the same input?
Can you create a pull request with a new test (in tests/), both with the before and expected after text encoded as UTF-8, and we can go from there?

Sure thing :)

I have the same problem. All the content that was converted correctly before the upgrade from 0.2.3 to 0.3.0 now brings question marks characters, not recognized as UTF8, breaking the output.
Removing the line
$html = str_replace("\xa0", " ", $html);
from "Html2Text" class resolves the problem

Can you all check out the new non-breaking-space branch and see if this resolves the issue for your tests? (PHP + character encodings = much sadness!)
If so, I'll merge into master and release 0.3.1 right away. If not, can you please contribute more pull requests off this branch, with additional tests?
Thank you @ulrichsg for the test contribution!

The new commit in the non-breaking-space branch solves the problem for my use cases

Resolved with PR #32 and released as 0.3.1


KAFKA-4272: Add missing 'connect' Windows batch scripts

cc @ewencp for review. Thanks.
I was able to run the quickstart scenario with the standalone script successfully. And the distributed script is almost the same.

LGTM. Thanks for the patch.


fix: pylint simplifications

Pylint updated and is now failing. These simplifications fix the lint errors.

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


Add support for COSE for VCs.

This PR adds basic support for COSE_Sign1 and COSE keys using coset and ciborium and implements the COSE part of the JOSE-COSE spec.
Since I needed the base64 library I also upgraded it to the latest version.

I've added more doc comments, a couple of examples, and more COSE-key encoding/decoding tests (which is the real addition from coset). I'll try to add the vc-jose-cose-test-suite to didkit-http before the PR is merged, but I'm not against a new review round.

Very happy to see this landed. Could you cut a new release with this? (I did my own version, but it was very hacky.)


SignUp page to Home Page Routing

I found that Whenever Sign Up to Home Page Routing is not proper. It redirects to Error 404 page.
Please assign this issue under GSSoC'23 Contributor.
https://github.com/MonalikaPatnaik/UMatter/assets/91741581/63210e1a-754f-44fe-b04d-707aad4e6bbe

@MonalikaPatnaik @Mansi168 Please assign this issue

Hey..navbar will be added to the sign up page..and it is already assigned to someone else!


Fix Response Value for Atomic Locks

Unfortunately, the return values of set are different than setnx, which means that previous changes to set locks atomically are completely broken. The return value for set is OK upon success instead of 1. This PR fixes this issue.

Thank @jpulec! Are you able to add a test so in the future an API change like this would be caught?

@evantahler As it turns out, this change did break existing tests. I must have just not taken the time to run them locally after making my previous changes (:disappointed:). It's probably worth setting up a CI solution like Circle or Travis for this repository, though, so that tests get run whenever a PR is opened.

This project used to be testing against Travis... but apparently that’s broken not.
@bleonard would you mind granting me access back to the TR Travis account, or, I could move this project to the ActionHero team, and use the team Circle account we’ve got there...

Closing due to https://github.com/taskrabbit/node-resque/commit/1c5273cbc2abafac0db334ea2e3bf752e5b290f8


DWEB-9

What does this PR do?
Added test case to verify right amount is tracked with order completed event.
Are there breaking changes in this PR?
No.
Any background context you want to provide?
NA
Is there parity with the server-side/android/iOS integration components (if applicable)?
NA
Does this require a new integration setting? If so, please explain how the new setting works
NA
Links to helpful docs and other external resources
NA

Closing this PR as raised PR to upstream.


See the installation section in the README for the right one.

oh, you're right.
sorry.


Added a login page under the name "login-page-with-social-media"

I have a added a login page:
"login-page-with-social-media"
Please kindly review my PR and merge this under "hacktoberfest-accepted" label.

https://github.com/jmrchelani/awesome-login-pages/tree/main/login-page-with-social-media

PR merged now
Thanks for you contribution, PR counted for hacktoberfest.
Don't forget to star the repo
I would like to offer some swags from Loginradius ,to avail those please fill this form https://docs.google.com/forms/d/e/1FAIpQLSfnwWUWlRRwZpgmOX7X8lGMKotLbVe1JXCjPFSahLxLWugwig/viewform


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add custom creator support for ReusableManager

coverage: 84.271% (-0.1%) from 84.366%
when pulling 38dcc55a9fe95762ee37a102e0b9c24938c9b39e on 34-add-custom-creator-support-for-reusablemanager
into e852451e468776d251cb336cd2582fb19184b18f on main.

coverage: 84.275% (-0.09%) from 84.366%
when pulling 3672c56e92c9e63f1d7534bfbbc150e7089b179f on 34-add-custom-creator-support-for-reusablemanager
into e852451e468776d251cb336cd2582fb19184b18f on main.


Error on Mac

Error #15: Initializing libiomp5.dylib, but found libiomp5.dylib already initialized
This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results.
How could I ensure that only a single OpenMP runtime is linked into the process?

no idea, cannot help with that - the issue is more general and not related to the code per se.


Update tutors on change

I know I'm making a lot of issues and I'll stop if you want me to.
This one is more of an opinion than a problem. I believe that when the user changes the subject or the number, the board should be updated without the click of a button. I personally find this more user friendly. However, this is just an opinion so feel free to ignore.

Duplicate of #8; I love the suggestions, but don't forget to check if your issue has already been submitted!


Create field args mapper and cache args resolution

Small enhancement/performance optimization + new feature.
We already have input parse value which turns array into something user defined, but same thing for field args' is missing.
And because this could be something expensive (for example - user might validate if args are within defined boundaries) - it doesn't make sense to call that mapper (or even the code that collects the args) more than once for same input. (For example in sub fields of iterable - args won't change in between each iterable).
Did brief test on performance optimization - it was about 15ms on ~800 calls - not something big, but I believe its good to have it - it will be beneficial even to users that doesn't use that args mapper.

Any idea why phpstan could be falling? I can't run it locally to test, because for some reason it requires php8.2 and I have php8.3 locally.

Any idea why phpstan could be falling? I can't run it locally to test, because for some reason it requires php8.2 and I have php8.3 locally.

Can you not see the output in https://github.com/webonyx/graphql-php/actions/runs/10038021756/job/27739070572?pr=1587?

Error: Method GraphQL\Tests\Executor\ResolveTest::buildSchema() has parameter $testField with no value type specified in iterable type array<string, mixed>.
Error: PHPDoc tag @param for parameter $testField contains unresolvable type.

GitHub also inlines the errors for me:

Locally:
 php -dmemory_limit=-1 ./vendor/bin/phpstan
Note: Using configuration file /home/bozhidar_hristov/projects/graphql-php/phpstan.neon.dist.
 414/414 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 [OK] No errors

:shrug:

Sorry to bother you, can we merge this? :)

Ping

@Warxcell Please merge the latest changes from master, let's see if tests and static analysis pass.

Done. No idea why phpstan is failing:
Locally I have this error (which again, no idea why happens)
------ -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  Line   src/Utils/SchemaExtender.php
 ------ -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  424    Method GraphQL\Utils\SchemaExtender::extendImplementedInterfaces() should return array<int, GraphQL\Type\Definition\InterfaceType> but returns array<int, GraphQL\Type\Definition\InterfaceType|(GraphQL\Type\Definition\NamedType&T of GraphQL\Type\Definition\Type)>.
 ------ -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

 [ERROR] Found 1 error

Making draft, as I might seen a bug.

Please add docs at https://webonyx.github.io/graphql-php/type-definitions/object-types/#field-configuration-options

I'm thinking something - maybe its better to use both $fieldDef and $fieldNode as cache key. So if
field {
   field2 <- if this is from interface, and it returns two versions of that interface - it would be called twice.
}

Thank your @Warxcell for your hard work and persistence on this!

Released with https://github.com/webonyx/graphql-php/releases/tag/v15.16.0.


Add Java cross version testing

To prevent problems like the one fixed in #89 in the future.

TLDR the underlying issue is that org.apache.maven.plugin-tools:maven-plugin-tools relies on ASM which needs to be updated to support target JVM bytecode, e.g.  maven-plugin-tools v3.6.1 uses asm v9.1 which supports Java 17.
When doing the update in #89 I tried updating to the latest maven-plugin-tools version (currently v.3.7.1 which supports up to Java 20) but all versions >3.6.1 where throwing failures with missing noaop jar
Execution failed for task ':plugin:compileKotlin'.
> Error while evaluating property 'filteredArgumentsMap' of task ':plugin:compileKotlin'
   > Could not resolve all files for configuration ':plugin:compileClasspath'.
      > Could not find sisu-guice-3.2.3-noaop.jar (org.sonatype.sisu:sisu-guice:3.2.3).
        Searched in the following locations:
            https://repo.maven.apache.org/maven2/org/sonatype/sisu/sisu-guice/3.2.3/sisu-guice-3.2.3-noaop.jar


Status email

Success/Failure emails?

For a "hello world" project that will not see real use, this is unnecessary.


Window.title cannot be changed during runtime

Can someone please help me out with this? I cannot programmatically change the title of the windows at runtime while the theme is applied.
Disabling the theme allows the window title to change dynamically so I know it isn't my code.

I'm using <IP_ADDRESS> and WindowTitle only has a get method, no way to set it.


MS Edge 17 WebM support update

MS Edge 17 in Windows 10 version 1803 supports WebM with both VP8 and VP9 video, and both Vorbis and Opus audio codecs, in progressive sources. (The current notes indicate partial support for the older MSE-only implementation.)
This is supplied by the 'Web Media Extensions' pack which is now system-installed in 1803. Unfortunately I can find very little documentation on this; https://blogs.windows.com/msedgedev/2017/12/05/introducing-web-media-extension-package-ogg-vorbis-theora-support/ mentions introduction of the extension package which could be manually installed on Windows 10 1709 / Edge 16 but concentrates on the Ogg-based codecs (the pack includes the Vorbis audio codec, but Opus and VP8/VP9 are built into the Win10 system already).
Resources:

https://developer.microsoft.com/en-us/microsoft-edge/platform/status/vorbisaudiocodec/
https://blogs.windows.com/msedgedev/2017/12/05/introducing-web-media-extension-package-ogg-vorbis-theora-support/

The "Web Media Extensions" package from https://www.microsoft.com/store/productId/9N5TDP8VCMHS Vorbis and Theora in an OGG container:

Included Technologies:
•	OGG Container Parser
•	Vorbis Decoder
•	Theora Decoder


Real account cannot grab account details (paper works)

Hi I recently created the account, and realized endpoint url needs update. Somehow I see for people, old account works still fine (mysterious). Here is what I did
changed self.base_trade_url = 'https://tradeapi.webullbroker.com/api/trade' to self.base_trade_url = 'https://ustrade.webullfinance.com/api/trade' and
to v5 for
def account(self, account_id):
return f'{self.base_trade_url}/v5/home/{account_id}',
but now I get following error : {'msg': 'Your session has expired, please login again.', 'traceId': 'XXX', 'code': 'trade.token.expire', 'data': {}, 'success': False} .
Any idea how to fix this now?
Appreciate any help!

If you are having trouble for login, you can directly get the accesstoken and refreshtoken without login.
https://github.com/tedchou12/webull/wiki/Workaround-for-Login

If you are having trouble for login, you can directly get the accesstoken and refreshtoken without login.
https://github.com/tedchou12/webull/wiki/Workaround-for-Login

There is no issue with login. Modules are executing properly where account details are not needed (example account activities). But get_account is not working.
I saw people saying new accounts have these issues while old ones are still working...any ideas?

I tried directly using the token from website and work without logging in. Again, paper account worked (it could fetch account details), but real account gave same error:
{'msg': 'Your session has expired, please login again.', 'traceId': 'XXX', 'code': 'trade.token.expire', 'data': {}, 'success': False} .
Stuck here :(

@tedchou12 It works after changing
def account(self, account_id):
return f'{self.base_trade_url}/v3/home/{account_id}'

@tedchou12 Is there a way to get the endpoints for all the actions? Like place order, check orders, oco etc...

@tedchou12 Is there a way to get the endpoints for all the actions? Like place order, check orders, oco etc...

@Avinashkn
What do you mean? Can you elaborate?

This can be closed, it is resolved with #185  and expanded with full api V5 update in pull #186

solved after @avinaskn fix, already pulled in. Thx!


Optimum-neuron 0.0.19: Error when exporting mistral model to inf2

System Info
optimum-neuron version 0.0.19
Amazon Linux 2023
EC2 inf2.xl instance

Platform:

- Platform: Linux-6.1.75-99.163.amzn2023.x86_64-x86_64-with-glibc2.34
- Python version: 3.9.16

Python packages:

- `optimum-neuron` version: 0.0.19
- `neuron-sdk` version: 2.17.0
- `optimum` version: 1.17.1
- `transformers` version: 4.36.2
- `huggingface_hub` version: 0.20.3
- `torch` version: 1.13.1+cu117
- `aws-neuronx-runtime-discovery` version: 2.9
- `libneuronxla` version: 0.5.538
- `neuronx-cc` version: <IP_ADDRESS>+c5231f848
- `neuronx-distributed` version: 0.5.0
- `neuronx-hwm` version: <IP_ADDRESS>+e34678757
- `torch-neuronx` version: <IP_ADDRESS>.12.0
- `torch-xla` version: 1.13.1+torchneuronc
- `transformers-neuronx` version: 0.8.268

Neuron Driver:

Traceback (most recent call last):
  File "/home/ec2-user/aws_neuron_venv_pytorch/bin/optimum-cli", line 8, in <module>
    sys.exit(main())
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/commands/optimum_cli.py", line 163, in main
    service.run()
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/commands/env.py", line 100, in run
    self.print_apt_pkgs()
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/commands/env.py", line 50, in print_apt_pkgs
    apt = subprocess.Popen(["apt", "list", "--installed"], stdout=subprocess.PIPE)
  File "/usr/lib64/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib64/python3.9/subprocess.py", line 1821, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'apt'

Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[X] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
Error when compiling mistral-based model to inf2
optimum-cli export neuron --model ./pytorch_model/ --task text-generation --sequence_length 4096 --batch_size 1 pytorch_model_neuron/
Traceback (most recent call last):
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/transformers/utils/import_utils.py", line 1382, in _get_module
    return importlib.import_module("." + module_name, self.__name__)
  File "/usr/lib64/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/neuron/modeling.py", line 47, in <module>
    from .modeling_decoder import NeuronDecoderModel
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/neuron/modeling_decoder.py", line 38, in <module>
    from transformers_neuronx.config import ContinuousBatchingConfig, NeuronConfig
ImportError: cannot import name 'ContinuousBatchingConfig' from 'transformers_neuronx.config' (/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/transformers_neuronx/config.py)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib64/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib64/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/exporters/neuron/__main__.py", line 27, in <module>
    from ...neuron import NeuronModelForCausalLM
  File "<frozen importlib._bootstrap>", line 1055, in _handle_fromlist
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/transformers/utils/import_utils.py", line 1372, in __getattr__
    module = self._get_module(self._class_to_module[name])
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/transformers/utils/import_utils.py", line 1384, in _get_module
    raise RuntimeError(
RuntimeError: Failed to import optimum.neuron.modeling because of the following error (look up to see its traceback):
cannot import name 'ContinuousBatchingConfig' from 'transformers_neuronx.config' (/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/transformers_neuronx/config.py)
Traceback (most recent call last):
  File "/home/ec2-user/aws_neuron_venv_pytorch/bin/optimum-cli", line 8, in <module>
    sys.exit(main())
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/commands/optimum_cli.py", line 163, in main
    service.run()
  File "/home/ec2-user/aws_neuron_venv_pytorch/lib64/python3.9/site-packages/optimum/commands/export/neuronx.py", line 226, in run
    subprocess.run(full_command, shell=True, check=True)
  File "/usr/lib64/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'python3 -m optimum.exporters.neuron --model ./pytorch_model/ --task text-generation --sequence_length 4096 --batch_size 1 pytorch_model_neuron/' returned non-zero exit status 1.

Expected behavior
Model should compile successfully

Hi @thesyntaxinator, could you upgrade your transformers_neuronx version, just tested with transformers_neuronx-0.9.474, the import should be ok.

That worked, thanks!


Fixed item links in report

This should do the trick. #9

Tested and working perfectly. Thanks :)

You're welcome.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/client-to-site-vpn/ibm (source)
module
patch
1.7.10 -> 1.7.11

terraform-ibm-modules/landing-zone/ibm (source)
module
minor
5.23.0 -> 5.24.0

Release Notes

terraform-ibm-modules/terraform-ibm-client-to-site-vpn (terraform-ibm-modules/client-to-site-vpn/ibm)
v1.7.11
Compare Source
Bug Fixes

deps: update required_provider to latest for the landing zone da extension (#252) (4c24487)

terraform-ibm-modules/terraform-ibm-landing-zone (terraform-ibm-modules/landing-zone/ibm)
v5.24.0
Compare Source
Features

expose DNS options in the vpc variable of the root level module (#795) (18af306)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@toddgiguere any idea ?

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v5.2.0

Your semantic-release bot :package::rocket:


Improve layout, readability, and formatting of API Reference docs

Updates #355

Put all links on main API Reference page into a single, less cluttered toctree
Add simplified package and package doc templates, and make headings to be more readable
Add sphinx-automodapi extension to add summaries at the top of module docs
Some misc docstring edits
Rename DbDict to SQLiteDict for consistency with other backends (with aliases for backwards-compatibility)

Coverage remained the same at 99.225% when pulling 9c15d7271aafd0c881be37cb9af5c38088be65b9 on JWCook:docs into 90fced70a9f0d2e1e498bcb51a54aef74b973029 on reclosedev:master.


[Doc] Incorrect configuration file for configuring My Account Application

The documentation page [1] needs to be updated in all related WSO2 Identity Server versions, including the latest ones, to reflect the deployment.toml configurations that need to be added instead of the configurations in the deployment.config.json file as any changes made to this file are overwritten at every server restart. An example deployment.toml configuration is shown below.
[myaccount]
personal_info.enabled = false
overview.disabled_features = ["overview.accountActivity","overview.consents"]
security.disabled_features = ["security.accountRecovery","security.mfa", "security.activeSessions", "security.manageConsents"]

[1] - https://is.docs.wso2.com/en/5.11.0/develop/configure-my-account/

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Flaky test: InvalidFilePathForLogs_ServerStillRuns - Shutdown Timeout

This test fails occasionally with the following error:
System.TimeoutException : IIS failed to stop site HttpTestSite20180722211208.
   at Microsoft.AspNetCore.Server.IntegrationTesting.IISApplication.WaitUntilSiteStopped()
   at Microsoft.AspNetCore.Server.IntegrationTesting.IISApplication.StopAndDeleteAppPool() in /_/src/Microsoft.AspNetCore.Server.IntegrationTesting.IIS/IISApplication.cs:line 141
   at Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer.Dispose() in /_/src/Microsoft.AspNetCore.Server.IntegrationTesting.IIS/IISDeployer.cs:line 33
   at Microsoft.AspNetCore.Server.IntegrationTesting.FunctionalTestsBase.StopServer() in /_/test/Common.FunctionalTests/Utilities/FunctionalTestsBase.cs:line 48
   at ReflectionAbstractionExtensions.DisposeTestClass(ITest test, Object testClass, IMessageBus messageBus, ExecutionTimer timer, CancellationTokenSource cancellationTokenSource) in C:\Dev\xunit\xunit\src\xunit.execution\Extensions\ReflectionAbstractionExtensions.cs:line 76
------- Stdout: -------
| [0.001s] TestLifetime Information: Starting test InvalidFilePathForLogs_ServerStillRuns at 2018-07-23T04:12:08
| [0.005s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Deploying [Variation] :: ServerType=IIS, Runtime=CoreClr, Arch=x64, BaseUrlHint=, Publish=True
| [0.005s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Removing environment variable ASPNETCORE_ENVIRONMENT
| [0.005s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Executing command dotnet publish  --output "C:\BuildAgent\temp\buildTmp\f2b22182-fd9b-45da-ad23-5eab21e51edb" --framework netcoreapp2.2 --configuration Release --no-restore -p:VerifyMatchingImplicitPackageVersion=false
| [0.225s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout: Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
| [0.225s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout: Copyright (C) Microsoft Corporation. All rights reserved.
| [1.121s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout:   TestTasks -> C:\BuildAgent\work\33bdfc1cae7b2a38\modules\IISIntegration\test\TestTasks\bin\Release\netcoreapp2.2\TestTasks.dll
| [1.198s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout:   Microsoft.AspNetCore.Server.IIS -> C:\BuildAgent\work\33bdfc1cae7b2a38\modules\IISIntegration\src\Microsoft.AspNetCore.Server.IIS\bin\Release\netstandard2.0\Microsoft.AspNetCore.Server.IIS.dll
| [1.538s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout:   InProcessWebSite -> C:\BuildAgent\work\33bdfc1cae7b2a38\modules\IISIntegration\test\WebSites\InProcessWebSite\bin\Release\netcoreapp2.2\InProcessWebSite.dll
| [2.226s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet-publish stdout:   InProcessWebSite -> C:\BuildAgent\temp\buildTmp\f2b22182-fd9b-45da-ad23-5eab21e51edb\
| [2.269s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: dotnet publish finished with exit code : 0
| [2.271s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Backed up C:\Windows\system32\inetsrv\config\applicationhost.config to C:\Windows\system32\inetsrv\config\applicationhost.config.HttpTestSite20180722211208backup
| [2.278s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Copied contents of IIS.config to C:\Windows\system32\inetsrv\config\applicationhost.config
| [2.285s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Could not add environment variables to worker process: Filename:
|                                                                                Error: Unrecognized element 'environmentVariables'
| [2.286s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Configured AppPool HttpTestSite20180722211208Pool
| [2.286s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Configured Site HttpTestSite20180722211208 with AppPool HttpTestSite20180722211208Pool
| [2.286s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Could not find AspNetCoreModuleV2 section in global modules; creating section.
| [2.286s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Could not find AspNetCoreModuleV2 section in modules; creating section.
| [2.339s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Site HttpTestSite20180722211208 has started.
| [2.339s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISDeployer Information: Successfully finished IIS application directory setup.
| [2.341s] Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.LoggingTests Debug: Method: GET, RequestUri: 'http://localhost:54208/HelloWorld', Version: 2.0, Content: <null>, Headers:
|                                                                                {
|                                                                                }
| [3.435s] Microsoft.AspNetCore.Server.IISIntegration.FunctionalTests.LoggingTests Debug: StatusCode: 200, ReasonPhrase: 'OK', Version: 1.1, Content: System.Net.Http.HttpConnection+HttpConnectionResponseContent, Headers:
|                                                                                {
|                                                                                Transfer-Encoding: chunked
|                                                                                Server: Microsoft-IIS/7.5
|                                                                                Date: Mon, 23 Jul 2018 04:12:12 GMT
|
...

@jkotalik / @mikeharder - can one of you take a look at this flaky test?

Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

@jkotalik - please use the workflow above to process this issue.

https://github.com/aspnet/IISIntegration/issues/1098 for the first issue, https://github.com/aspnet/IISIntegration/pull/1096 for the second.

Gets500_30_ErrorPage failed again.
This comment was made automatically. If there is a problem contact @ryanbrandenburg.

This might be a product issue in ANCM blocking app shutdown, or it could be IIS just failing to shutdown for reasons unrelated to ANCM.  We should investigate adding more logging to IISDeployer.Stop() and/or IISDeployer.WaitUntilSiteStopped().

AppOfflineDroppedWhileSiteFailedToStartInShim_AppOfflineServed is returning 0x80070002, which is file not found. May be related to app_offline being missing/present.

Moving to preview2 since preview1 has already shipped.

InvalidFilePathForLogs_ServerStillRuns failed with about the same error on 2.2.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

InvalidFilePathForLogs_ServerStillRuns failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

InvalidFilePathForLogs_ServerStillRuns failed with about the same error on 2.2.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

cc @pakrym

First occurances of this issue happened before IISDeployer rewrite that fixed a lot of deployment related failured and later ones are dotnet publish related mass failures that were fixed. Closing this issue.


docs(useQuery): simplify disabling queries example

Commit Message
You may copy this for squash and merge:
docs(useQuery): simplify disabling queries example

Change conditional rendering to `if` - `else` instead of nested ternary

PR Notes
Hi, proposing a stylistic change which makes the example easier to understand (subjectively). Please feel free to close the PR if you'd prefer the existing convention :)

I realized the rest of the examples follow the same convention of nested ternaries.


mpsc Channel and custom retry logic make for never-closing channel

I am writing a AWS Kinesis client using the Rusoto AWS SDK and I have hit an issue where I cannot retry requests without cloning the futures::sync::mpsc::channel Sender and still have the right kind of drop semantics for closing the Receiver.
The code spawns a thread to handle receiving records, it batches them up, and will create inflight number of concurrent requests. This is great for throughput as Kinesis has relatively high latency (50ms) and small request size (500 records). These requests can fail for a number of reasons: the TCP connection sees an error, the kinesis stream is under-provisioned for the current message rate (bursts, etc), Kinesis has errors, etc. I need to simply retry requests when they fail.
My first stab at retrying requests has me cloning the Sender and moving that handle in to the Receiver thread. Now the issue is that when the original Sender is dropped the receiver does not automatically close -- my Receiver thread is now open forever and I am leaking threads in my code.
So my issue, is there a way to create a Sender which does NOT count for the life of the channel? Or is there a way to close() a Sender which includes closing all the other Senders? Either one of these solutions may yield data loss for requests which need to be retried. But perhaps it would be useful? Or, perhaps there is a better way to structure this retry logic? Could I do it in the AndThen to keep that initial request going?
Here is code which spawns the Receiver thread and attempts retries:
pub fn kinesis_tx(stream_name: String, chan_buf: usize, inflight: usize) -> RecordsChannel {
    info!("CREATING kinesis channel");
    use futures::sync::mpsc::channel;
    use futures::{Future, Sink, Stream};

    let (tx, rx) = channel(chan_buf);
    let client = Arc::new(KinesisClient::simple(Region::UsWest2));

    let mut retry_tx = tx.clone().wait();
    std::thread::spawn(move || {
        info!("STARTING kinesis batch put thread");
        let puts = rx.chunks(500)
            .map(|batch: Vec<PutRecordsRequestEntry>| {
                let input = PutRecordsInput {
                    records: batch,
                    stream_name: stream_name.clone(),
                };
                let chain = client.put_records(&input).then(|put_res| {
                    let retval: Result<
                        Result<PutRecordsOutput, (PutRecordsError, PutRecordsInput)>,
                        (),
                    > = match put_res {
                        Ok(res) => {
                            trace!("match put_res: it worked");
                            Ok(Ok(res))
                        }
                        Err(err) => {
                            trace!("match put_res: failed");
                            Ok(Err((err, input)))
                        }
                    };
                    retval
                });
                chain
            })
            .buffer_unordered(inflight);

        for put_res in puts.wait() {
            match put_res {
                Ok(Ok(put)) => {
                    if let Some(failed) = put.failed_record_count {
                        if failed > 0 {
                            error!("{} record(s) failed to commit to kinesis", failed);
                            let put: PutRecordsOutput = put;
                            for rec in put.records {
                                if rec.error_code.is_some() {
                                    error!("failed record: {:?}", rec);
                                }
                            }
                        }
                    }
                }
                Ok(Err((put_records_err, put_records_input))) => match put_records_err {
                    PutRecordsError::HttpDispatch(dispatch_err) => {
                        error!(
                            "http dispatch error: {:?}. retrying records...",
                            dispatch_err
                        );
                        for record in put_records_input.records {
                            match retry_tx.send(record) {
                                Ok(()) => debug!("Wait#send succeeded"),
                                Err(e) => error!("Wait#send error {:?}", e),
                            }
                        }
                    }
                    PutRecordsError::Unknown(raw_message) => {
                        error!("unknown error: '{:?}', retrying records...", raw_message);
                        for record in put_records_input.records {
                            match retry_tx.send(record) {
                                Ok(()) => debug!("Wait#send succeeded"),
                                Err(e) => error!("Wait#send error {:?}", e),
                            }
                        }
                    }
                    other => {
                        error!("unhandled kinesis error: {:?}", other);
                    }
                },
                other => {
                    error!("puts.wait() fallthrough: {:?}", other);
                }
            }
        }
        info!("STOPPING kinesis batch put thread");
    });

    return tx;
}

I have also encountered this situation, and think the there should be a close method on senders that closes the channel for all, no matter how many clones there may be.
Since this does not exist, I've resorted to including an additional cancelable token future that can be closed from any sender, and a wrapped receiver checks this token first on polls.

@seanmonstar can you point to an example of how you do the cancelable token future?

@xrl it's slightly more than  a channel, since I conflated a callback concept directly into it, but here: https://github.com/hyperium/hyper/blob/v0.11.18/src/client/dispatch.rs
The cancelable token is in the same folder, cancel.rs.
However, the downside to this is that all the information is actually already inside the mpsc channel. I have to allocate a secondary slot to hold this canceled state, and do additional atomic operations on each poll to check it. It'd be more efficient to just allow the mpsc channel to do this directly.


Jesse/fix unit test

@gatsbyz consider returning back Slack notifications for Nightly builds through this PR, because those were deleted accidentally I suppose through #1430 (https://github.com/0xPolygon/polygon-edge/commit/024d01104a311386ad7d23ed03f361ba987029de#diff-0d5658b415099a82c11c03a06ca4ec765b4003a1f4b2f3f1943980a882cf8aa6L46-L49).

@Stefan-Ethernal that was a duplicated part of the code so it is right to remove on those lines as the notification block starts  https://github.com/0xPolygon/polygon-edge/commit/024d01104a311386ad7d23ed03f361ba987029de#diff-0d5658b415099a82c11c03a06ca4ec765b4003a1f4b2f3f1943980a882cf8aa6L67-L71 here

@Stefan-Ethernal that was a duplicated part of the code. i think it was a rebase error on my part. so the notification block actually starts 024d011#diff-0d5658b415099a82c11c03a06ca4ec765b4003a1f4b2f3f1943980a882cf8aa6L67-L71 here

Ok, but strange there weren't Slack notifications for yesterday's NB. :thinking:


Allow taps to generate catalog without auth, and/or without JSON schema validation on settings

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/235
Originally created by @aaronsteers on 2021-10-07 22:31:35

Currently, some taps require credentials in order to perform discovery and at least in theory, config settings are allowed to alter the output of the catalog (flattening and stream maps, for instance).
Given that we want discovery to work even without auth (at least for sources which can support it), we need to modify tap and target behaviors to be able to generate a catalog, even if the tap config validation otherwise would not pass due to missing credentials.
Any of these could work in theory:

Discovery mode skips JSON schema setting validation (not ideal perhaps because of the risk of KeyError exceptions that may be hard for the user to debug).
Discovery mode does perform validation, but rather than producing a hard failure, it prints a non-fatal warning message before continuing the discovery process.
Some other tap-level setting where a developer informs the SDK whether and how the catalog can be generated when creds are not provided.

I'm leaning towards option 2 because it is the least invasive, it's relatively easy for users to debug, and it does not require the developer to modify their existing code. (Executions that require creds will naturally fail, and executions that don't require creds will naturally succeed.) Since we're not technically ignoring or disabling config, this also means that stream maps and flattening behaviors will happen as usual, along the normal code path and without need to alter or make exception.

View 12 previous comments from the original issue on GitLab


vueRouter import in a story produce an error in storyshot

I have a story with a vueRouter mocked with custom routes like this:
const customRoutes = [
  {
    path: '/campaign/<id>',
    name: 'Campaign',
    params: {
      campaignId: '<id>'
    },
    component: MetricContent
  }
];

And I declare my template like this:
export const MetricContentView = defaultTemplate.bind({});
MetricContentView.args = defaultProps;
MetricContentView.decorators = [vueRouter(customRoutes)];

When I start storybook it works well as expected but I need to create some storyshots I have the following:
 Unexpected error while loading ./index.stories.js: Cannot use import statement outside a module
     Jest encountered an unexpected token

    Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.

    Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.

    By default "node_modules" folder is ignored by transformers.

    Here's what you can do:
     • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.
     • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/configuration
    For information about custom transformations, see:
    https://jestjs.io/docs/code-transformation

    Details:

    /Users/xxx/node_modules/storybook-vue3-router/dist/esm/index.js:5
    import vueRouter from './withVueRouter';
    ^^^^^^

    SyntaxError: Cannot use import statement outside a module
        at new Script (node:vm:100:7)
        at Runtime.createScriptFromCode (/Users/xxx/node_modules/jest-runtime/build/index.js:1728:14)
        at Runtime._execModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1596:25)
        at Runtime._loadModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1185:12)
        at Runtime.requireModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1009:12)
        at Runtime.requireModuleOrMock (/Users/xxx/node_modules/jest-runtime/build/index.js:1210:21)
        at Object.<anonymous> (/Users/xxx/src/sections/CampaignDetailed/HeaderBlock/components/MetricContent/index.stories.js:2:1)
        at Runtime._execModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1646:24)
        at Runtime._loadModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1185:12)
        at Runtime.requireModule (/Users/xxx/node_modules/jest-runtime/build/index.js:1009:12)

      at Object.error (node_modules/@storybook/client-logger/dist/cjs/index.js:74:67)
      at node_modules/@storybook/core-client/dist/cjs/preview/executeLoadable.js:84:32
          at Array.forEach (<anonymous>)
      at node_modules/@storybook/core-client/dist/cjs/preview/executeLoadable.js:77:18
          at Array.forEach (<anonymous>)
      at executeLoadable (node_modules/@storybook/core-client/dist/cjs/preview/executeLoadable.js:76:10)
      at executeLoadableForChanges (node_modules/@storybook/core-client/dist/cjs/preview/executeLoadable.js:127:20)

Did I miss something in the configuration ?

Or do you know how can I mock this module with Jest ? I've try something like:
jest.mock('storybook-vue3-router');
import vueRouter from 'storybook-vue3-router';
const mockedDependency = vueRouter as jest.Mocked<typeof vueRouter>;
export default mockedDependency;

and reference it in jest.config but with no result.
Do you have an idea @NickMcBurney ?

For the initial you mentioned, you could try using require rather than import.
I'm not sure why you would want to use this package within a jest test, it's specifically built for Storybook not jest and won't work outside of Storybook - you probably ne next importing full vue-router for this.

@budet-b, would it be possible for you to provide a small reproduction repo? If you can I'd be happy to investigate for you.

Closing as stale.


feat: add host support for packed multiexponentiations (PROOF-877)

Host-side version of #140

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


Can you provide code template customization?

🚀 Feature Proposal
for example:
#
# @lc app=leetcode.cn id=5 lang=python3

This code vscode can't autocomplete. It need set interpreter.
#!/usr/bin/python3
# -*- coding: utf-8 -*-
#
# @lc app=leetcode.cn id=5 lang=python3

Motivation
File Header Setting
Contribution

Dup with #469


Default config.toml is not working

Hello. I'm getting the errors listed below when trying to use the default config.toml in the installation guide.
thelevic@Levis:~/blog$ hugo server
Start building sites …
hugo v0.92.2+extended linux/amd64 BuildDate=2023-01-31T11:11:57Z VendorInfo=ubuntu:0.92.2-1ubuntu0.1
ERROR 2024/03/27 15:28:52 render of "section" failed: "/home/thelevic/blog/themes/tailwind/layouts/_default/baseof.html:2:18": execute of template failed: template: search/search.html:2:18: executing "search/search.html" at <site>: can't evaluate field LanguageCode in type *langs.Language
ERROR 2024/03/27 15:28:52 render of "taxonomy" failed: "/home/thelevic/blog/themes/tailwind/layouts/_default/baseof.html:2:18": execute of template failed: template: _default/terms.html:2:18: executing "_default/terms.html" at <site>: can't evaluate field LanguageCode in type *langs.Language
ERROR 2024/03/27 15:28:52 render of "taxonomy" failed: "/home/thelevic/blog/themes/tailwind/layouts/_default/baseof.html:2:18": execute of template failed: template: _default/terms.html:2:18: executing "_default/terms.html" at <site>: can't evaluate field LanguageCode in type *langs.Language
ERROR 2024/03/27 15:28:52 render of "taxonomy" failed: "/home/thelevic/blog/themes/tailwind/layouts/_default/baseof.html:2:18": execute of template failed: template: _default/terms.html:2:18: executing "_default/terms.html" at <site>: can't evaluate field LanguageCode in type *langs.Language
Error: Error building site: failed to render pages: render of "home" failed: "/home/thelevic/blog/themes/tailwind/layouts/_default/baseof.html:2:18": execute of template failed: template: _default/list.html:2:18: executing "_default/list.html" at <site>: can't evaluate field LanguageCode in type *langs.Language
Built in 4 ms

It seems that there is an issue with search and languages? Thanks.

Seems that you used the old version of hugo. Please upgrade to latest or at least v0.120


feat: add space between modal buttons

Right now, modal buttons are glued together:
https://mll-lab.github.io/react-components/?path=/story/modal--open-by-button
This PR fixes the spacing between them by applying the styles like antd manually. Does not work out of the box because the specific antd classes with our prefix are not generated corectly.

@mic-web ready to go when conflicts are resolved.

 Better clear it next time.

 Better clear it next time.

You're right, I will.

Ich bin vom 28.09.2022  bis 04.10.2022  nicht erreichbar. Ihre E-Mail wird nicht weitergeleitet. In dringenden Fällen wenden Sie sich bitte an Herrn Igor Podolskiy @.***  @.***, Tel. +49-89-99017503).
Mit freundlichen Grüßen
Michael Adlfinger
I am out of office from 28.09.2022  till 04.10.2022 . Your email will not be forwarded. For urgent matters please contact Mr. Igor Podolskiy @.***, phone +49-89-99017503).
Best regards
Michael Adlfinger
@spawnia approved this pull request.
--
Reply to this email directly or view it on GitHub:
https://github.com/mll-lab/react-components/pull/186#pullrequestreview-1129674454
You are receiving this because you authored the thread.
Message ID: @.***>
To: "mll-lab/react-components" @.***>
Cc: "mic-web" @.***>
"Author"<EMAIL_ADDRESS>
Reply-To: "mll-lab/react-components" @.***>


Index defining documentation

Hi!
Is there any documentation on index definition building using Riddle?
I see quite large library, but there are no even comments :)

Many months too late, but I've just gone through and rewritten the README to be far more useful: https://github.com/pat/riddle/blob/develop/README.markdown - very sorry it took so long!


Extracting bold text and non bold text from pdf

Is there a way to extract bold text and non bold text from pdf ?

I think actually no.

You can convert it to HTML with high_level.extract_text_to_fp(fin, output, laparams=LAParams(), output_type='html', codec=None) and filter it with soup or lxml.

Hi @lkmh, thanks for your question! There is no such thing as bold or italic text in a PDF. However, most PDF's use multiple variants of the same font-family to get bold (and italic) text. E.g. a specific font for bold, one for italic and one for regular. The names of these fonts often reflect this.
You could use extract_pages to get the pages of a pdf, and iterate over the elements in it. For every character you should check the font-name and see if it contains "bold".
See the docs and e.g. this comment on extracting font names.

I did something like that using the help from the website:
from pdfminer.high_level import extract_pages
from pdfminer.layout import LTTextContainer, LTChar
for page_layout in extract_pages('PDF_FILE.pdf'):
for element in page_layout:
if isinstance(element, LTTextContainer):
for text_line in element:
# print(text_line)
            for character in text_line:
                if isinstance(character, LTChar):
                    if 'Bold' in character.fontname:
                        print(text_line)

I've found that using the pdfplumber library (based on pdfminer) for something like this is slightly easier.
See jsvine/pdfplumber#89 (comment) - you'll just need to add an additional condition like "Bold" in obj["fontname"] as harozudu did above.
with pdfplumber.open(file_to_parse) as pdf:
    text = pdf.pages[0]
    clean_text = text.filter(lambda obj: not (obj["object_type"] == "char" and "Bold" in obj["fontname"]))
    print(clean_text.extract_text())

No need for nested loops :)

I was looking for almost the same solution, thx :)


improve time travel handling

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

41 of 41 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 99.65%

Files with Coverage Reduction
New Missed Lines
%

src/modules/olMap/handler/measure/OlMeasurementHandler.js
1
96.96%

src/modules/olMap/handler/draw/OlDrawHandler.js
1
98.59%

Totals

Change from base Build<PHONE_NUMBER>3:
-0.03%

Covered Lines:
12974

Relevant Lines:
12977

💛  - Coveralls