fix build

@mattn thanks for contribution.
but your commit is not fix build. Could you tell me why you need to fix this?
/tmp/go-build513997367/github.com/lighttiger2505/lab/gitlab/_test/_obj_test/client.go:55:2: cannot use jobs (type []gitlab.Job) as type []*gitlab.Job in return argument

https://travis-ci.org/lighttiger2505/lab/builds/369107323?utm_source=github_status&utm_medium=notification

The result of building your branch on my machine is as follows
$ go build ./...
github.com/mattn/lab/gitlab
gitlab/client.go:47:2: cannot use jobs (type []gitlab.Job) as type []*gitlab.Job in return argument
github.com/mattn/lab/commands
commands/pipeline.go:126:45: cannot use jobs (type []"github.com/mattn/lab/vendor/github.com/xanzy/go-gitlab".Job) as type []*"github.com/mattn/lab/vendor/github.com/xanzy/go-gitlab".Job in argument to jobOutput

Ah, sorry. In my local repository, go-gitlab is master branch. And I built lab with go build. It seems that go-gitlab add breaking changes at https://github.com/xanzy/go-gitlab/pull/364 . So please close this PR.

@mattn Understood. I am happy to receive a pull request from a respecting person.

FYI, v0.9.0 is tagged in go-gitlab just now.

Thank you for teaching me.
Since I upgraded go-gitlab, I will merge your pull request.


[Identity] [External] MSAL’s confidential client credential does not silently authenticate

After an initial authentication with the confidential client credential, further authentications require new network requests. The authenticated account doesn’t get stored in any form of cache.
Besides that, if the cache plugins from the node extensions are used, no information is stored in the persistence cache.
This is inconsistent with MSAL for .NET, where client credential authentication can avoid further network requests once authenticated.

We’ve been investigating this, and we’ve narrowed it down, but we can’t commit to have this fixed by July.


Redis database id selection for CacheResponse and RequestCached

The enum Database is already defined and builder methods are already in place, the redis database needs to be selected before executing the cache_op_executor function.

As stated in the PR, using multiple databases in redis is an anti-pattern. If the need for such a solution is really required, we should look into having separate instance instead.
For more details, I found the document provided by @fmrsabino quite enlightening
https://redis.com/blog/7-redis-worst-practices/


fix: option namespacePrefixWarning should be optional

Coverage remained the same at 87.282% when pulling 9ad88a776dd8fa9fb90556a3310769c3b1c678a3 on DiamondYuan:master into 6a18a1033e00a0955c9ad5b0e57adfb1307e69ef on dvajs:master.


fix: stale personal details while creating task with new user

Details

When assigning task to accounts we haven't chat before, we generate optimistic personal details for them. Later the BE will respond with different account IDs and details but we do not clear the optimistic ones leading to stale data in Onyx. This PR fixes that.
Action Performed:

Assign task to an account you haven't chat before
Press the report header to open assignee account details
Wait till API succeeds

Expected Result: Account details show Hidden because the account ID in URL is stale
Actual Result: Account details show the correct assignee email but different avatar with report header
Screenshots

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/113963320/e52f9331-f372-47eb-a84d-faf4644bbd89

Fixed Issues

$ https://github.com/Expensify/App/issues/42027
PROPOSAL: https://github.com/Expensify/App/issues/42027#issuecomment-2107178694
Tests

Assign task to an account you haven't chat before
Press the report header to open assignee account details
Verify after a while, the account details show Hidden

[x] Verify that no errors appear in the JS console

Offline tests

NA
QA Steps

Assign task to an account you haven't chat before
Press the report header to open assignee account details
Verify after a while, the account details show Hidden

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/Expensify/App/assets/113963320/f34de8bb-c302-4810-95d9-96bd34abc4d1

iOS: Native

https://github.com/Expensify/App/assets/113963320/e6abcc40-05ea-4a8a-ab44-1bb5da3d9fa5

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/113963320/391439a8-c61b-42e4-b099-dfc568b78b09

MacOS: Desktop

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.81-11 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Website Loading Issue

Hello,
I have installed pH7CMS on my server. But it is not loading correctly. Tried it like 100 times. But, getting the same problem. My website link is https://ox69.net/hizuppo/ You can check it yourself how it looks like. I have contacted with the hosting provider and they said they have checked it and their server meets all the requirement for pH7CMS.
Can anyone please give me a solution.
Thank a lot.

num47;
I'm having exact same issue.  Just did an install on new server/host.  I used Softaculous (because I was lazy).  I'm going to re-install using manual setup of DB and dl from Sourceforge.
Seems that one of the CSS files is not loading. Admin area is fine.

Something to do with Cache settings under "tools"....?

@num47 You can first start by disabling the gzip compression in the options.
https://github.com/pH7Software/pH7-Social-Dating-CMS/wiki/[DRAFT]-Problem-Getting-Assets-(CSS-JS)-cURL
https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/106
That should give you some hint for investigation.
If not then
https://github.com/pH7Software/pH7-Social-Dating-CMS/wiki/How-to-ask-for-help

@num47 Also, please make sure you aren't running mod_security or modsecurity-crs on your server as it may cause unpredictable results.

@TGHepburn I actually have tried to install it both manually and using softaculous. None worked. You are a little better position than i am 'cos my admin panel is also loading the same way. And, what cache settings actually you were talking about? Is that a solution for this problem? Would you please explain a little if changing some settings would solve the problem? I have actually installed it on my own localhost server and it works perfectly fine there.
@polynamaude Thanks a lot for the suggestion. I will check it and see if things get fixed. Will update you.

@polynamaude added SetEnv no-gzip 1 to .htaccess to turn off the gzip. I am not sure if that's not a proper code to disable the gzip or it still not working after disabling gzip.

@num47 Did you take the time to read the three links I've sent ?
Look over if modsecurity-crs is enabled on the server ?
At last, there's a nice series of hint that tells you how to ask for help (if you do want to get helped). And now you're damn far from there.
https://github.com/pH7Software/pH7-Social-Dating-CMS/wiki/[DRAFT]-Problem-Getting-Assets-(CSS-JS)-cURL
I don't see nowhere something related to using SetEnv no-gzip 1 !!!!
If you don't want our answer then don't waste our time.

It worked finally! only disabling the gzip probably is enough. Wrote the following code on the .htaccess file and it laded nicely. The code is -
php_flag zlib.output_compression off
Though i first tried to use the following to disable the gzip and mod security. But those didn't work. And, apparently for me i do not need to use these following codes either.

@TGHepburn you can try this and see if this fixes your problem.

Last night I did a fresh install and things got worse somehow.  After re-install, behavior matched @num47 exactly: back-end and front-end had malformed pages.
@num47 Your suggestions did not work regarding modifications to the .htaccess file.  I was referring to the administration features which are shown in the attached GIF showing what happens when Gzip is disabled.  This has some effect on front end but does not help.
I have another shared host site which has installation of V15 working sweetly ( funio.com ).  The shared host with issues is hostupon.ca (one of the recommended hosting companies by pH7 ) .  Thus I am able to compare settings.

ModSecurity is OFF on both sites (though Cpanel )
Working has PHP 7.0; Site with issues has PHP 7.1 ( I will change this and report back time permitting)
PHP packages are the same (I'll add screenshot soon )
No changes have been made to the .htaccess file on working site.
both installed through Softaculous
both in root directory of domain/sub domain
working url : http://dating.whodidthis.ca
issues url: http://www.flirtbywire.net

I am going to hunt down any log files that may exist on this CMS as I have not read any mention of them yet.

@TGHepburn Either I'm missing something or didn't I gave a link called "how to get help" ?
Maybe with some logs we can try to see what's going on.
Just go into the administrative interface and disable all the cache type. This should help out.
There's also some other closed issue with the same cache problem. Look at them.

@TGHepburn You have to delete the cache after you use php_flag zlib.output_compression offin your .htaccess
You will find the cache in /_protected/data/cache/
And, the links @polynamaude shared actually will give you more insight about this problem.

@num47 Thanks for the tip.  I took it one step further and deleted everything.  The long weekend put a halt to my efforts but it was a good thing.  Turns out that the hosting company had not fully configured the server space.  The solution was to post ticket with hosting company.  They were quick to respond.  Now there are Apache errors showing up and the site behaves normally.  I'm sure it's due to my lack of knowledge.  I've read the links provided by @polynamaude  and will re-read again.  Thank-you for the help.

@TGHepburn Hi !
Great if it did help out.
Could you please go and give us a star on gitHub. This would be appreciated.
Also you can write a nice review on sourceforge.


Expose refund amount to flow.

This allows the flow builder to access the refund amount, e.g. for customer mails.
Example in a mail template:
{{ amount | currency(order.currency.isoCode) }}

nice one @kleinmann
thanks a lot :)


feat(atomic,headless): quickview maximumDocumentSize parameter

https://coveord.atlassian.net/browse/KIT-673

With a requestedOutputSize of 0 bytes (meaning full document rendered):

With a requestedOutputSize of 1000 bytes:

Should we add this parameter to the atomic-result-quickview component? It will also make the manuel test easier

Should we add this parameter to the atomic-result-quickview component? It will also make the manuel test easier

I initially did that when I first put this parameter in the QuickViewState instead of QuickViewOptions. We then moved it into the options since we thought the users would adopt just one value to be used across all the results and therefore didn't need it in the state.

since we thought the users would adopt just one value to be used across all the results

How can I user set this value from inside atomic? Adding an option/prop to atomic-result-quickview that would then be passed to the quickview controller could be a way to set this value.

since we thought the users would adopt just one value to be used across all the results

How can I user set this value from inside atomic? Adding an option/prop to atomic-result-quickview that would then be passed to the quickview controller could be a way to set this value.

Then would we have to move the maximumPreviewSize parameter from the options to the state?

You won't need to add it inside the state, you can add an option to the atomic-result-quickview component and pass the value over to the buildQuickview method in the initialize method, we already pass the result as option.


DEV: fix linting action repo owner and isort command

Fixes a typo in the owner of github.repository, and the usage of the isort command

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 91.622%

Files with Coverage Reduction
New Missed Lines
%

src/cogent3/maths/scipy_optimize.py
1
85.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
29603

Relevant Lines:
32310

💛  - Coveralls


Fix Incorrect 'Sec-WebSocket-Accept' header value

For cherrypyserver on Python 3 and above (tested with Python 3.6)
Same logic applied as in wsgiutils.py
Screenshot of issue:

As you can see, the value of the 'Sec-Websocket-Accept' header is in the wrong format. It must be decoded when running Py3.

@Lawouach Can you review it, merge and release it if you agree?
This issue is blocking my websockets on py3 in combination with cherrypy.
Thanks!

I think this as a bug in CherryPy that has been fixed and released in 18.0.1
https://github.com/cherrypy/cherrypy/issues/1738#issuecomment-419641132
Mind trying updating CherryPy just to see if that helps before we decide to merge this one?
Thanks.

@Lawouach I cannot upgrade to CherryPy 18.0.0 or above because it's Python3 only.
I need support for both Python2 and Python3 and the latest version is 17.4.0.
I find it a bit strange that you did the same decoding for the wsgiutils.py, so I would assume that you do it also for cherrypyserver.py.
Please reconcider merging this one because versions below 18.0.0 of cherrypy should be able to work also with this fix. :wink:

@Lawouach, @jaraco, in which project do you think it should be fixed?
If needed, I can also make the PR for CherryPy with the changes as described in my previous comment.

I can merge it in this project but I'm away until next week. I'll try to cut a new release by Wednesday.
I don't know if the CherryPy project will backport though.

Happy to have bugfixes applied to 17.x. @h3llrais3r , would you make the PR? Just target the pr to the maint/17.x branch in CP and reference the bug/PR/commit that fixed it for 18.x? Thanks.

@Lawouach, @jaraco
I've provided the PR for both libraries...
For which option will we go? Or perhaps for both?

Since the change was made in the master branch of CherryPy (https://github.com/cherrypy/cherrypy/issues/1738#issuecomment-419641132) I feel there would be symmetry in using your PR you submitted over there.

@Lawouach, I see that @jaraco has just merged it in the cherrypy:maint/17.x branch, so it should be fixed. 😉


ENH: Adding in hierarchy type

This pull request is for introducing another tree data type Hierarchy.
The main motivation behind this is there are other instances of trees other than Phylogeny such as  hierarchical clusterings.  And using Phylogeny to represent these data types are incredibly misleading.  This is critical for reducing confusion for performing analyses with Gneiss, which can operate directly on Phylogeny and Hierarchy types.

Is there anything else that is required here?  This would be really nice for clarifying the semantics for tree types other than phylogenies in the future.

I'm fine with this as written, but we'll need some transformers for it (it looks like your commit history may have gotten backwards, I see the AlphaDiversity stuff).
Anyone else have thoughts here?

Just a note that this resolves #99.

Sorry we don't need transformers, I completely forgot how this worked when I wrote that comment.

Thanks!
On Fri, May 12, 2017 at 11:35 AM, Evan Bolyen<EMAIL_ADDRESS>wrote:

Merged #112 https://github.com/qiime2/q2-types/pull/112.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/qiime2/q2-types/pull/112#event-1080460469, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD_a3Ruo-E7FjbCCCsooQZnScFLCxK0Mks5r5KaEgaJpZM4M3u6X
.


1.5.07 BALL_COUNT should be 0 at first

The extension work has already been done in the base project meaning that the program renders 10000 balls right away. Needs to be set to 0 to just render a single ball for the initialize challenge, or rewritten to have no handling for multiple balls to leave it for students to figure out.

Whoops. Fixed the ball counts to 1 for the exercise, and 10000 for the solution. As for the loop, you're talking about this one, right?
    for (int i = 0; i < BALL_COUNT; i++) {
        balls.add(new BouncingBall(viewport));
    }

I think this one behaves as you want it to, @xyclos, where BALL_COUNT is indeed the literal number. That look right to you? If not, please reopen. Thanks!


Add tests

🚀
Closes #186

so cool. we'll get to 100% test coverage eventually.

@joshreisner  You're dang right we will! 🔥

@myarete can you please reach out to Renae on C4R Slack?


Fix black borders on FlxCamera.shake()

:3

ok no more commits my bad

i give up maybe someone else can fix this


Reordering Requests

I thought I used to be able to drag and drop requests in the FlashPost pane to reorder them or to sort them into folders.  If so, it no longer seems to work.  If I try to drag one, I get a ghost of the item and a blue line appears to make it look like things are working, but moving and dropping have no effect.
My main use case for this is when I want to make a similar request to one that exists. I duplicate the existing one, make my changes, and then want to sort it into the correct folder, but cannot currently.
Any thoughts as to what's up?
Thanks!

It works fine for when i try now. There is no issue. Please update your vscode to latest version and update Flashpost to latest version.

SO, I tried a fresh install of both vscode and FlashPost, and the reordering works, though there is still something odd about it.  When I drag an item to a new spot, it moves and the enter new description dialog drops down. Is that expected behavior?
Thanks!


Ollama Chat connection not possible

I cannot connect the "Ollama Chat" model to the "Conversational Retrieval QA Chain".


Request: Add example for conferences

There are currently examples at https://docs.nexmo.com/voice/voice-api/conversation in python / ruby / php, but I could personally use a node example.
If anyone has something that's not polished and ready just laying around, I'd totally love to see it in a gist or whatever.
Otherwise I may just go through the python example piece by piece and try to replicate. If I do that successfully maybe I'll make a pull request. Thanks!

@joshkpeterson - sorry for the delay. https://github.com/nexmo-community/nexmo-node-quickstart/pull/11 provides an example.

Example already added, closing this issue


geometryc soft lock with large mesh

When using this mesh, I'm getting a soft lock. Git wouldn't let me upload a .obj so I just changed the extension to .txt, so just change it back to .obj before trying it.
bigObject.txt

cc @attilaz

@horsesNhalo I will check it, but in the meantime have you checked it where is it stuck?
@bkaradzic
I had a similar problem. It was stuck here:
https://github.com/bkaradzic/bgfx/blob/master/examples/common/bounds.cpp#L327
This line not always make maxDistSq larger.
For example if maxDistSq=148846.094, distSq=148846.750, _step = 0.01
lerp(maxDistSq, distSq, _step) will be 148846.094 again. So it seems to related
be float inaccuracy.
It should check the "no change" case and handle it somehow.
I guess with smaller _step value it would occur on more models.

It should check the "no change" case and handle it somehow.

Ah ok, probably better to scale model into some sane range first...

I have checked the attached file. It has the same problem. For example maxDistSq is 900354112.0 and distSq is 900356032.0, and the new maxDistSq is the same as before.
@bkaradzic This model has even bigger boundary. Where should this scale happen? As calcMinBoundingSphere can have an infinite loop it should happen inside it, but it shouldn't allocate extra memory I guess.

Basically we should calculate AABB of object once file is parsed. And we can:

error out if object is outside of reasonable bounds (lets say 4096.0f ^3), inform user and force them to scale manually object
scale whole object into 4096.0f ^3 bounds automatically


transformers already installed: MissingDependencyException: Not all required importable packages are installed to use the configured NLU pipeline. To use this pipeline, you need to install the missing modules: - transformers (needed for LanguageModelFeaturizer)

Rasa Open Source version
2.8.31
Rasa SDK version
2.8.8
Python version
3.7
What operating system are you using?
Linux
What happened?
I've installed package transformers (version: 3.5.1), and use " LanguageModelFeaturizer" in pipeline.  But exception message shows no package transformers found. I dont kown why, any ideas?
Command / Request
cmd that led to the bug:
rasa train -c config/config_pretrained_embeddings_bert_zh.yml --data data/medical/M3-training_dataset_1564317234.md data/medical/stories.md data/medical/Diseases_converted.yml data/medical/Drugs_converted.yml data/medical/Departments_converted.yml data/medical/Foods_converted.yml data/medical/Symptoms_converted.yml --out models/medicalRasa2 --domain config/domains2.yml --num-threads 5 --augmentation 100 -vv

Relevant log output
error logs:
2022-08-16 16:57:35 DEBUG    urllib3.connectionpool  - Starting new HTTPS connection (1): api.segment.io:443
2022-08-16 16:57:36 DEBUG    urllib3.connectionpool  - https://api.segment.io:443 "POST /v1/track HTTP/1.1" 200 21
2022-08-16 16:57:37 DEBUG    rasa.__main__  - Failed to run CLI command due to an exception.
Traceback (most recent call last):
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/__main__.py", line 118, in main
    cmdline_arguments.func(cmdline_arguments)
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/cli/train.py", line 59, in <lambda>
    train_parser.set_defaults(func=lambda args: run_training(args, can_exit=True))
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/cli/train.py", line 103, in run_training
    finetuning_epoch_fraction=args.epoch_fraction,
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/api.py", line 124, in train
    loop,
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/utils/common.py", line 296, in run_in_loop
    result = loop.run_until_complete(f)
  File "uvloop/loop.pyx", line 1456, in uvloop.loop.Loop.run_until_complete
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/model_training.py", line 104, in train_async
    file_importer, output, fixed_model_name
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/model_training.py", line 130, in handle_domain_if_not_exists
    file_importer, output=output_path, fixed_model_name=fixed_model_name
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/model_training.py", line 765, in _train_nlu_with_validated_data
    **additional_arguments,
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/nlu/train.py", line 97, in train
    nlu_config, component_builder, model_to_finetune=model_to_finetune
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/nlu/model.py", line 164, in __init__
    components.validate_requirements(cfg.component_names)
  File "/home/hcl/miniconda3/envs/rasa2/lib/python3.7/site-packages/rasa/nlu/components.py", line 88, in validate_requirements
    f"Not all required importable packages are installed to use "
rasa.exceptions.MissingDependencyException: Not all required importable packages are installed to use the configured NLU pipeline. To use this pipeline, you need to install the missing modules:
  - transformers (needed for LanguageModelFeaturizer)
Please install the packages that contain the missing modules.
MissingDependencyException: Not all required importable packages are installed to use the configured NLU pipeline. To use this pipeline, you need to install the missing modules:
  - transformers (needed for LanguageModelFeaturizer)
Please install the packages that contain the missing modules.

Rasa 2.8.31 uses a different version of transformers 2.11.0 (see here). I would recommend creating a fresh venv and only installing rasa, this will ensure you will get all rasa's dependencies in the versions that the 2.8.x version is compatible with.


[DNM] 2.5 Compatible finetune changes

Changes to module/lm and to modules/trainable_anyscale to be removed/moved to clients/anyscale

This PR needs to refresh


Fix master branch (and switch to pytest)

Coverage decreased (-1.3%) to 98.718% when pulling a9e409d264ac1a30270580613db23a8e29cb6d48 on ticosax:fix-tests into c60f7327c41e6827ddb95ed243c5d949fde69193 on jettify:master.

Coverage decreased (-1.3%) to 98.718% when pulling a9e409d264ac1a30270580613db23a8e29cb6d48 on ticosax:fix-tests into c60f7327c41e6827ddb95ed243c5d949fde69193 on jettify:master.

@ticosax I probably move aiohttp_sse to https://github.com/aio-libs, since most of my projects live there, jazzband looks more Django oriented. So if you still have interest, I think we can figure out something there
(cc @asvetlov )

@jettify transferring the project to aio-libs is fine with me.
FYI Jazzband is focused on python, not only django, so aiohttp_sse could be accepted.
Note also that publishing of releases to pypi is done automatically with jazzband.
That's something that is worth considering, otherwise you will end up doing administrative tasks to grant access to pypi for maintainers.


Improve documentation on publishing distributions

Remove 'simply' as it's only simple for the author but not necessarily
for the reader.
Adapt Kotlin snippet to artifact method being able to process
TaskProviders now.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@ljacomet should this go to master or some other branch in order to be part of 7.0 RC1?

@britter master was the right target. Thanks for the fix.


Migrate tf.contrib.seq2seq to addons.

Only the V2 implementation is copied from tf.contrib.seq2seq, and
exported.

@qlzh727 can I include some part of seq2seq as a part of my gsoc proposal :)

@tabshaikh, can u give more details about the proposal? Are u trying to use seq2seq or trying to update it?

@qlzh727 I was thinking maybe help in moving it from contrib ?

Hi @tabshaikh, the migration work is pretty much done on my side, so probably consider some new project? :)

Now all the tests are passing and c_ops issue has been fixed. Please take a look.
There are few open questions so far:

I reworked all the existing class to be keras layers and work with 2.0 API. Since the new implementation has different base class comparing to existing class, I have to create new class and then give them _v2 suffix in the class name. Ideally I would prefer to not have v2 in the class name since the v1 implementation is not exposed at all. What do u think?

The v1 implementation is not exported since it does not work with 2.0 API, eg using variable scope etc. Existing user who use tf.contrib.seq2seq will have to update their code to use new addon implementation, (the API has slightly changed to be more object-oriented and fit keras convention).

The README doc is current a big TODO, which will update to include example and migration guide from tf.contrib.seq2seq. This work can probably go parallel with the code review.

Great contribution!
For 1) and 2), I think it's time to clean up v1 implementation totally for addons, so _v2 suffix seems unnecessary. @seanpmorgan What do you think, Sean?
I'm fine with 3), Qianli. Thank you for the great work.

Haven't had time to do a full review but some thoughts:
re: 1 and 2) above: This is interesting because as shown in #86 we have to consider backwards compatibility for saved graphs. Now because the old version is breaking in 2.0 we might be able to get away with removing V2, but I'm not sure if thats a good idea seeing as for other sub-packages we'll need to keep it. Its ugly I agree but might just simplify things.
re 3) above: Sounds good thanks for bearing with us as we review this large contribution and simultaneously define our repo structure.
@karmel @martinwicke @facaiy Another question I had was how do we want to implement a sub-package like this. There are several pieces which already match an existing API we have defined support for (e.g.  SequenceLoss is a subclass of Keras Loss). This is our first contribution that has that problem so worth discussing.
I think we could either:

Just allow everything to exist under seq2seq package. This is slightly confusing as I would hope that all losses are available in losses directory.
House the loss and anything else that can co-exist in the pre-existing subpackages and then import them within seq2seq
Open to other ideas as neither of these sound great....

@seanpmorgan, for case like SequenceLoss, I think the judgement should be based on how generic it could be. So far, I think it is a quite specific loss function that is useful in encoding-decoding situation. Having said that, if you feel this can be a generic loss for RNN, then probably it should be moved to Loss package and imported here.

Done. Removed v2 suffix and added README.md for general guild line and code example. PTAL.

I think the judgement should be based on how generic it could be.

+1, make sense for me

I think the judgement should be based on how generic it could be.

+1, make sense for me. I haven't seen the codes, so we can make SequenceLoss private?

Not quite sure about that, currently user will have to use it by tfa.seq2seq.SequenceLoss. If we don't expose it, then user will have no way to access it.

Thank @qlzh727 for the great work. I have a glance at the changes, I'm kind of worried that the PR is so large that review process might take too much time.
I think the feature is quite useful. Moreover, the changes are moved from tf.contrib, and @qlzh727 comes from keras team. Hence, I suggest to merge the PR at first, and then to create an issue in which we discuss what to do next.
These are what I can think of:

code style problem: PEP8
use public tf API.
fix bug:

Current the beam_search_decoder is not broken due to unknown reason from
c kernel, which need more time to debug.

move c++ codes to custom_ops?
might move some class to the corresponding loss/layer module?

What do you think? @seanpmorgan @karmel @qlzh727

@qlzh727 Qianli, could you merge the latest master branch, and run make code-format in a docker container ? See Development Environment

Thoughts:
Firstly I agree that the code is super useful, and as I understand it Qianli you or someone else in your team will maintain the package going forward correct? Because of that I agree this can be an expedited merge, but still needs some conforming changes. I was going to try to get a full review in on Saturday.
Regarding points:

PEP8 should be relatively easy to fix prior to merge
Public tf.API is also rather easy. I can take a pass on it, as I'm familiar with a lot after our re-org. Anything that isn't available we can leave the private APIs and add it to the list we're about to submit to TF core
I'm alright with this being future work as long as its understood who's maintaining this subpackage
Yes, losses etc. can all live under seq2seq as discussed, but we really should keep all our custom ops together (see image and text after PR #91 goes through)
Think I'm in agreement that these can live under seq2seq. Its not perfect but a repo like this will never be.

Ideally we can merge this by early next week and include in 0.2.0

Thoughts:
Firstly I agree that the code is super useful, and as I understand it Qianli you or someone else in your team will maintain the package going forward correct? Because of that I agree this can be an expedited merge, but still needs some conforming changes. I was going to try to get a full review in on Saturday.

Thanks. Yes, I will probably be the owner for this, and can fallback to @ebrevdo who is the original author if needed. The code has been under CI test within Google as well, so if anything breaks, we will probably detect that within Google too.

Regarding points:

PEP8 should be relatively easy to fix prior to merge
For PEP8, I guess you are referring to 4 space indent vs 2 space indent we use within google. As the code in tensorflow/tensorflow is using 2 space indent by default, should we also be consistent with that code style? Personal wise, I also prefer 2 space indent which makes code more compact and can fit more information in one line.
Public tf.API is also rather easy. I can take a pass on it, as I'm familiar with a lot after our re-org. Anything that isn't available we can leave the private APIs and add it to the list we're about to submit to TF core
Ack, I can probably do it in follow up changes, since this one is already too large. When I made the original change within Google, it was split up to small change which is one per module. In tfa's case, I somehow have to do it in a bulk change, otherwise the library could be left in a unusable state.
I'm alright with this being future work as long as its understood who's maintaining this subpackage
Ack
Yes, losses etc. can all live under seq2seq as discussed, but we really should keep all our custom ops together (see image and text after PR #91 goes through)
Ack, I will leave the current one as is since PR #91 has not been submitted yet. Will do the clean up once that is submitted.
Think I'm in agreement that these can live under seq2seq. Its not perfect but a repo like this will never be.
Ack.

Ideally we can merge this by early next week and include in 0.2.0

For PEP8, I guess you are referring to 4 space indent vs 2 space indent we use within google. As the code in tensorflow/tensorflow is using 2 space indent by default, should we also be consistent with that code style? Personal wise, I also prefer 2 space indent which makes code more compact and can fit more information in one line.

Understand the personal preference, but we made the decision to stick with PEP8 formatting a little while ago, and I think a change is a tough sell. To make it easy if this code also exists internally,  I believe you can just run make code-format in the tensorflow/tensorlow:nightly- custom-op docker image.

Ack, I can probably do it in follow up changes, since this one is already too large. When I made the original change within Google, it was split up to small change which is one per module. In tfa's case, I somehow have to do it in a bulk change, otherwise the library could be left in a unusable state.

Sounds good. We have some existing private API usage already.

Ack, I will leave the current one as is since PR #91 has not been submitted yet. Will do the clean up once that is submitted.

Since merged. Apologies for the rather large restructure during the review. If you could also add an entry in the central README as a subpackage that'd be great.

No, we absolutely should not.

So currently we use a few private APIs for say testing. We discussed this in https://github.com/tensorflow/addons/pull/55 and are planning on submitting the ones we felt useful but not public as an issue to tf core.
I hope we can have fully public api usage ASAP, but some things may need APIs created or files copied like tensor_util

TensorFlow will have some testing utilities, but I'm wary about publishing
things that are not obviously universally useful and very related to
TensorFlow itself.
Some of tensor_utils falls into that category, but probably not everything.

@martinwicke Makes sense the review process for public API should be pretty limiting and we'll see which of the ones we propose will make the cut. In the mean time would you recommend we lean toward doing accessing private APIs or making a copy of the code in addons?

For small utilities, I'd make a copy.

We should probably export tf.contrib.framework.with_shape and make it eager
friendly.
On Tue, Mar 19, 2019, 7:11 AM Martin Wicke<EMAIL_ADDRESS>wrote:

For small utilities, I'd make a copy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/72#issuecomment-474387728, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtim6bYqbdLnhpzKuVQOJP5dcYB8xtlks5vYPAXgaJpZM4bfmPt
.

How useful is with_shape in eager though? It's mostly input validation, some special-casing for (since removed) dimension 1 scalars, and a lot of graph mode code.
I think in eager mode, where shapes are available at point of calling, this function has very little value, but maybe I'm missing something very subtle it's doing?

For library authors, who cannot assume that a computation is happening
inside a tf.function or not, it is necessary.  In addition to checking
what's known of the static shape, it also performs an assertion check at
runtime in case the shape inference is malforned (so the tf.function
building code passed fine but actual tensor shapes are wrong), and to fill
in the gaps when shape data is fully available.
On Tue, Mar 19, 2019 at 8:20 AM Martin Wicke<EMAIL_ADDRESS>wrote:

How useful is with_shape in eager though? It's mostly input validation,
some special-casing for (since removed) dimension 1 scalars, and a lot of
graph mode code.
I think in eager mode, where shapes are available at point of calling,
this function has very little value, but maybe I'm missing something very
subtle it's doing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/72#issuecomment-474419760, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtim93QTvdkHjzcO1ZQR2K4jJhVundFks5vYQA_gaJpZM4bfmPt
.

I agree with Martin that the shape check is much easier since there isn't any unknown information at that stage. I can probably update that in a follow up PR.

That's not the case.  Users inside a tf.function (especially for an RNN)
will have missing shape information.
On Tue, Mar 19, 2019 at 10:14 AM Qianli Scott Zhu<EMAIL_ADDRESS>wrote:

I agree with Martin that the shape check is much easier since there isn't
any unknown information at that stage. I can probably update that in a
follow up PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/72#issuecomment-474476079, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtim5UszEktTxRexPHVYeCtKVocbStqks5vYRsAgaJpZM4bfmPt
.

Note also that there's tf.ensure_shape, not sure that's exactly what we want here.

That's not the case. Users inside a tf.function (especially for an RNN) will have missing shape information.
…
On Tue, Mar 19, 2019 at 10:14 AM Qianli Scott Zhu @.***> wrote: I agree with Martin that the shape check is much easier since there isn't any unknown information at that stage. I can probably update that in a follow up PR. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#72 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/ABtim5UszEktTxRexPHVYeCtKVocbStqks5vYRsAgaJpZM4bfmPt .

If the code is called within tf.function, then it be under graph mode and the existing code should be able to cover it.

@martinwicke, I check the code of tf.ensure_shape, which does a static shape check (eg, undefined dimension is just treat as wildcard). The tensor_utils.with_same_shape does extra check for those undefined dims at runtime.

Right -- can you call this tf.function in a way which would create undefined shapes?

tf.function can be called in a way that creates undefined shapes; it's
especially an optimization with RNNs:
@tf.function()
def my_rnn(x):
...
my_rnn(ones(3, 10, 5))  # uses fixed shape [3, 10, 5]
my_rnn(ones(3, 15, 5)) # second cache miss, starts tracking input shapes
my_rnn(ones(3, 20, 5)) # builds a tf.function with input shape (3, None, 5)
On Tue, Mar 19, 2019 at 11:14 AM Martin Wicke<EMAIL_ADDRESS>wrote:

Right -- can you call this tf.function in a way which would create
undefined shapes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/72#issuecomment-474507537, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtim7V_ZPluk7db4lNb1pYa88fqxel9ks5vYSj8gaJpZM4bfmPt
.

tf.ensure_shape documentation says:
  NOTE: This differs from `Tensor.set_shape` in that it sets the static shape
  of the resulting tensor and enforces it at runtime, raising an error if the
  tensor's runtime shape is incompatible with the specified shape.
  `Tensor.set_shape` sets the static shape of the tensor without enforcing it
  at runtime, which may result in inconsistencies between the statically-known
  shape of tensors and the runtime value of tensors.

Unless the documentation is not correct, or I am reading it wrong, it appears to me that this should do what you want, no?

tf.ensure_shape documentation says:
  NOTE: This differs from `Tensor.set_shape` in that it sets the static shape
  of the resulting tensor and enforces it at runtime, raising an error if the
  tensor's runtime shape is incompatible with the specified shape.
  `Tensor.set_shape` sets the static shape of the tensor without enforcing it
  at runtime, which may result in inconsistencies between the statically-known
  shape of tensors and the runtime value of tensors.

Unless the documentation is not correct, or I am reading it wrong, it appears to me that this should do what you want, no?

I did some quick test with tf.ensure_shape, and the following code didn't throw any error, which it should (2 and 4 are different). tf.ensure_shape take the shape information as TensorShape and I cannot give it array_ops.shape().
`
import tensorflow as tf
import numpy as np
def ensure_shape(a, b):
return tf.ensure_shape(a, b.shape)
p1 = tf.placeholder(tf.int32, shape=(3, None))
p2 = tf.placeholder(tf.int32, shape=(3, None))
p3 = ensure_shape(p1, p2)
with tf.Session() as sess:
Expect error here
sess.run(p3, feed_dict={p1: np.ones((3, 2)), p2: np.ones((3, 4))})

`

@tensorflow/api-owners should we fix tf.ensure_shape to allow you to pass the result of tf.shape?

@martinwicke ensure_shape is supposed to be about dynamically checking that the static shape holds; if you make the shape dynamic you get nothing; you might as well do tf.Assert(tf.equal(tf.shape(x), tf.shape(y)) or something of the sort.

That would work -- the question is: If I understand Eugene's comment correctly, the signature of a tf.function makes it somewhat hard to verify a shape, is that correct?

Using tf.function repeatedly with fixed structures but different shapes can lead to tracing a graph with partially unknown shapes but this has nothing to do with the behavior of ensure_shape AFAICT.
I'm confused.

I want to avoid adding more shape verification functions to TF. The assert expression you wrote (tf.Assert(tf.equal(tf.shape(x), tf.shape(y))) looks fine to me. @qlzh727 would that work? Or if now. can you repost the actual use you were looking at with_shape for?

Done. Changed to use control deps with tf.assert_equal, and also remove the tensor_util which was only used for shape check.
The shape check was used when we want to clone a nested tuple which holds the attention states. We allow user to overwrite some fields during clone, and we have to make sure the value user provided are with same shape as the existing value. Since some shape might be partially defined (eg timestep, batch), and the tf.with_shape only does the static shape check.

Yes, the sanity check somehow is failing. I was trying to fix it by "make code-format", but the auto formatted code was failing at lint check. Also some py3 lint check "object not callable" is not correct. The object is inheriting a keras layer, which is a callable.
I think the sanity test might need some update, and we can ignore the the warning so far.

@seanpmorgan, I am going to squash and merge this PR now since you have LGTMed. Feel free to reopen it or create follow up issues if you see any problems. Thanks.

Thanks. Just for reference the sanity check is in the process of being setup and is failing at the moment / can be updated.


Remove version: qBittorrent.qBittorrent version 4.5.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/qbittorrent/files/qbittorrent-win32/qbittorrent-4.5.0/qbittorrent_4.5.0_lt20_qt6_x64_setup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


drivers: tcpc: add FUSB302B TCPC driver

This adds a driver for the onsemi FUSB302B USB Type-C controller IC, implementing a TCPC and VBUS-sensor.
This IC is present on the otterpill board introduced in #62830.
The driver is pretty minimal, intended to only implement the minimal subset of TCPC functionality required for requesting a higher voltage than 5V from a source.
For testing simplicity, this PR currently contains the otterpill board definition from #62830 as well.

curious if anyone is using this?

To update the current status on this: The driver does work in that it can successfully request a specific voltage from a PD PSU, i didnt test any functionality beyond that.
There are some problems with reliability though, IIRC the continuous measurement of CC states seemed to interrupt communication with the PSU at times (since it meant connecting the measurement block in the chip to each CC one after another). It might be worth using the interrupt functionality of the FUSB302 and just storing the current state in the driver, instead of querying the chip often. (the interrupt functionality is not used at all in this driver currently)
That said, I was just working on that out of personal curiosity, and having the proper devboard laying around. I'm not planning on working on this more in the short term. The latest version of this is found in an OOT module, which i made for easier experimenting: https://github.com/ottojo/zephyr_fusb302b
If anyone wants to make a proper FUSB302 zephyr driver, feel free to use this as a starting point, but also know that there is a linux driver out there for example, which surely is a lot more mature and actually battle tested.


Null-reference exception when response code is not defined

If server returns a code that is not defined in openapi spec, generated modules will not only throw an UndeclaredResponseException but also a null-reference exception.

Cause:
https://github.com/Azure/azure-powershell/blob/master/src/MySql/generated/runtime/UndeclaredResponseException.cs#L46
- json = json.Property("error") ?? json;
+ json = json?.Property("error") ?? json;

This code is not generated and shared across all the generated module. And you should find it in  powershell/resources/runtime/csharp/client/UndeclaredResponseException.cs.


largest-series-product corner cases

As part of the work to have consistency between tracks as in https://github.com/exercism/todo/issues/104 and https://github.com/exercism/todo/issues/13. Comments are in individual commit messages.

@petertseng I merged your JS changes, but on that track failed to request that you squash the commits for the PR in to one. Would you mind doing that here please?  If you need me to do it just ask.

Ah yes, no problem. The commits are hereby squashed.

👍Thanks!


Add temporary file IoHandle.md for purposes of PR discussion

Description: Refactoring the FD- Take 2

@mattklein123 Please review.

@sbelair2 thx for the super detailed writeup. I will look in the next few days. @envoyproxy/maintainers  also if you have some time?

not stale

@sbelair2 I'm going to close this out. We can refer back to comments in here if needed.


Fix more clippy::doc_lazy_continuation lints

This fixes all remaining instances of clippy::doc_lazy_continuation except for in some generated code.

I found them because I'd added it to the top level Cargo to enable it. :)
I'd wanted to enable it in the commit, but ran into an issue with a couple of remaining ones in generated code and didn't quite have the time at the moment to fix them, but also didn't want merge conflicts on these if I let it all sit until I had the time.

Oops I meant to flag this for merge yesterday. Doing so now.
Also makes sense! And yeah feel free to enable them at the top-level too. It's ok to add #[allow] to generated code as we've done that for ISLE code before (we can perhaps change that to allow(clippy::all) too or something like that)


Config classes can not be extended inside of a form or single field

Reproduction
https://formkit.link/e11450953673688c79860ae3af353e21
Describe the bug
At the moment the global created classes inside of the config, can not be extended with the config prop of a form or single field.
I would expect that the classes can/will be extended and not completely overridden.
Environment
• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

They are being overwritten by your configuration, I think this is expected, if you want to extend you should use the other forms of extending styles like using classes prop or section-key.

I think we can work around this in our project
I added this report because in discord it was described that it should work like this. So that you can extend the classes on a form level without overwriting the complete "default" classes config from the initialization of the formkit plugin.
In the end, I would expect it like this because otherwise, you need always to add all classes from the initialization also to such a form usage.

What use case do you believe this would solve? do you think this could be a improvement, and how do you think it would work for overwritting instead of extending?

The config object is not deeply extended, nor will it be. The config object performs a shallow merge ontop of the global properties and allows you to reset any config properties, including classes. However this is why we created the classes prop — specifically so you can achieve a bulk deep extension.
I dont think the expectation that config would deep merge is incorrect, but unfortunately its would be an architectural change that would add too much weight to the package.


Adds measure & transmission of back-pressure from Job queue to Receivers

The logging statement is in the next to last commit, suitable for test-bed purposes (but not yet for merging).

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/5/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-0e5ebbf2)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of cf0c2588a90c602643cf078d48347ad4c8d11850 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/5/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/6/
Build Log
last 10 lines
[...truncated 13 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f b540fcc3dc8768f596148dec121e3f7c6b1c6f47
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 2319088fa0c69f45b2f9ab146e8eb48b8f4c7243 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/6/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/7/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-958e0945)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 9a0354898ef175788a14fed5a850e6978cc9d48e to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/7/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/8/
Build Log
last 10 lines
[...truncated 18 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 1799606c6d94f836a79af8fc3d11298405e270f8
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 7e9a0b3480f3b9aadc5235d9e9b4c5e2640009d4 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/8/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/9/
Build Log
last 10 lines
[...truncated 13 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 1799606c6d94f836a79af8fc3d11298405e270f8
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 6859f932919c09e79c15b0ecbbb94845bbb270be to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/9/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/10/
Build Log
last 10 lines
[...truncated 13 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 1799606c6d94f836a79af8fc3d11298405e270f8
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 00ffb5b5a7397925337fe45f8b1f4aac59f40838 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/10/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/11/
Build Log
last 10 lines
[...truncated 13 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 1799606c6d94f836a79af8fc3d11298405e270f8
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of fa398da9850b3dd317c7dc4165b5f7f0d0d5fd9f to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/11/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/12/
Build Log
last 10 lines
[...truncated 13 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 1799606c6d94f836a79af8fc3d11298405e270f8
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of c4b1ef277ce5fa8f19f3fe8b2fdfe71203da6996 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/12/ and message: Merged build finished.

Test FAILed.

Note to self : the behavior of batch processing speed logging is not correct in case no elements are processed during this batch.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/13/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-760486a6)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of f7bbf86e8511b6a0c61c4d9eb66ed1a1bbba75f3 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/13/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/14/
Build Log
last 10 lines
[...truncated 18 lines...]
> git config core.sparsecheckout # timeout=10
> git checkout -f 0302a0f5e889a4d3a44c4d40b1738bd2d7182dc5
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 38f0f9cbb865a5ab9a625d37b14bf6d3b562a7fb to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/14/ and message: Merged build finished.

Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/15/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-61048db1)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of c2daea426dc0f9bc448a45d9a758cb11339ccb23 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/15/ and message: Merged build finished.

Test FAILed.

Still missing explicit types :)
Scalastyle checks failed at following occurrences:
[error] /home/ubuntu/workspace/ghprb-spark-multi-conf/label/Spark-Ora-JDK7-PV/scala_version/2.10/streaming/src/main/scala/org/apache/spark/streaming/receiver/CongestionStrategyImpl.scala:31:15: Public method must have explicit type
[error] (streaming/compile:scalastyle) errors exist
[error] Total time: 13 s, completed May 23, 2015 2:23:45 AM
[error] /home/ubuntu/workspace/ghprb-spark-multi-conf/label/Spark-Ora-JDK7-PV/scala_version/2.10/streaming/src/main/scala/org/apache/spark/streaming/receiver/CongestionStrategyImpl.scala:31:15: Public method must have explicit type
[error] (streaming/compile:scalastyle) errors exist

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/19/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-a4a0e474)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of b75807c17304aa318a4ac48e670b669388d49467 to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/19/ and message: Merged build finished.

Test FAILed.

Re-opening against a Streaming-specific branch. See:
https://github.com/typesafehub/spark/pull/10

Refer to this link for build results (access rights to CI server needed):
https://ci.typesafe.com/job/ghprb-spark-multi-conf/20/
Build Log
last 10 lines
[...truncated 20 lines...]
First time build. Skipping changelog.
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10
Triggering ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark JDK-7 PV (i-03c381d3)
Configuration ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 is still in the queue: Waiting for next available executor on Spark-Ora-JDK7-PV
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.10 completed with result FAILURE
ghprb-spark-multi-conf » Spark-Ora-JDK7-PV,2.11 completed with result FAILURE
Notifying upstream projects of job completion
Setting status of 4190fbe6f155ab5dee10b6262b57a216959d98fa to FAILURE with url http://ci.typesafe.com/job/ghprb-spark-multi-conf/20/ and message: Merged build finished.

Test FAILed.


Show Navbar on small displays too.

This will show the Navbar on devices with smaller screens too.

@trick77 nice, thanks, merged


.ExecAlways flag added to BuildStepExecutable step (Exec)

FastBuild's Exec step has .ExecAlways parameter that was not exposed to the developers that use SharpMake.
Its valuable for code generation tasks that should be run every time.
More details here:
https://www.fastbuild.org/docs/functions/exec.html

Merged, thanks!


Gitter fix for our guidelines

This change is

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Upgrade/improve CI/CD dependencies

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Update 0.6.x docs to describe setting command options via .ember-cli file

(As implemented in #331 by @kpfefferle)

Added in #335


⚠ Bofrost.be has degraded performance

In de01f63, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1666 ms

Resolved: Bofrost.be performance has improved in 8bc4a6e.


The project has no license

Hi @deniskrut, @andrewpaster, @baumanab
Notice that our project doesn't have license, that's a little unprofessional for a open source project, especially there has much extends adoption and development in an online education project. "No license terms" means users don't know it can be used freely. Another reference is Can I call my program "Open Source" even if I don't use an approved license?.
So here I suggest the project add license, recommend we choose one of the below license, firstly because they're all Permissive Licenses, but they have different redistribution disclaimer:

MIT
Apache-2.0
BSD-3-Clause

Regards,
Brandon

Added!


decode adds -v4 to res\drawable-*dpi folders

Information

Apktool Version (apktool -version) - 2.1.1
Operating System (Mac, Linux, Windows) - Windows 7
APK From? (Playstore, ROM, Other) - any

Steps to Reproduce

Open original apk (as zip), which have dirs res\drawable-hdpi, res\drawable-mdpi, etc.
apktool  d PackageInstaller.apk.
Open decoded res folder, it will contain drawable-hdpi-v4, drawable-mdpi-v4, etc. There were no -v4 in original res

APK
Any. Checked on standard android like PackageInstaller, APKs from playstore or self created.
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? Do not installing
Are you using the latest apktool version? Yes

Duplicate of #1212


Create dataset iitb_english_hindi_corpus

uid: iitb_english_hindi_corpus
type: processed
description:

name: IITB English-Hindi Corpus
description: The IIT Bombay English-Hindi corpus contains parallel corpus for English-Hindi as well as monolingual Hindi corpus collected from a variety of existing sources and corpora developed at the Center for Indian Language Technology, IIT Bombay over the years. This corpus has been used at the Workshop on Asian Language Translation Shared Task since 2016 the Hindi-to-English and English-to-Hindi languages pairs and as a pivot language pair for the Hindi-to-Japanese and Japanese-to-Hindi language pairs.
homepage: https://www.cfilt.iitb.ac.in/iitb_parallel/
validated: True

languages:

language_names:

Indic
Hindi
English

language_comments:
language_locations:

Southern Asia
India
Pakistan

validated: False

custodian:

name: IIT Bombay
in_catalogue:
type: A university or research institution
location: India
contact_name: Anoop Kunchukuttan; Pratik Mehta; Pushpak Bhattacharyya
contact_email<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>contact_submitter: True
additional: https://www.iitb.ac.in/
validated: False

availability:

procurement:

for_download: Yes - after signing a user agreement
download_url: https://www.cfilt.iitb.ac.in/iitb_parallel/dataset.html
download_email:

licensing:

has_licenses: Yes
license_text:
license_properties:

non-commercial use

license_list:

cc-by-nc-nd-4.0: Creative Commons Attribution Non Commercial No Derivatives 4.0 International

pii:

has_pii: Yes
generic_pii_likely: somewhat likely
generic_pii_list:

names

numeric_pii_likely: somewhat likely
numeric_pii_list:

fax numbers

sensitive_pii_likely: unlikely
sensitive_pii_list:
no_pii_justification_class:
no_pii_justification_text:

validated: False

processed_from_primary:

from_primary: Taken from primary source
primary_availability: Yes - their documentation/homepage/description is available
primary_license: Unclear / I don't know
primary_types:

web | wiki
web | news or magazine website
web | official website
news articles

validated: False
from_primary_entries:

media:

category:

text

text_format:
audiovisual_format:
image_format:
database_format:

.TGZ

text_is_transcribed: No
instance_type: The corpus contains 1.49 million parallel sentences.
instance_count: 1M<n<1B
instance_size: 10<n<100
validated: False

fname: iitb_english_hindi_corpus.json

This dataset contains:

[ ] Parallel corpus English-Hindi

Train: 95.6 MB
Dev and test: 318 KB

[ ] Hindi monolingual: 2.3 GB

DONE for Hindi monolingual: https://huggingface.co/datasets/bigscience-catalogue-data/iitb_english_hindi_corpus
Sample:
{'text': 'भारतीय राजनीति में पिछड़ेपन के विचार का सूत्रीकरण उसे खत्म करने के मकसद से किया गया था'}

#self-assign

Done! LM repo for Hindi monolingual: https://huggingface.co/datasets/bigscience-catalogue-lm-data/lm_indic-hi_iitb_english_hindi_corpus

Thanks @mariosasko


Alarm triggered when switching modes during "leave delay"

Describe the bug
When switching from one mode to another, the HA integration receives a "triggered" event during a "leaving delay".
To make it more precise: I have an "Away" mode which has a 30 second time to leave the house. All possible triggers are disabled during that time. When I let that mode enter it's state (so after 30 seconds) and then switch back, no "triggered" state is received. But when I switch to another mode during this 30 second delay (for instance, I forgot something and need to go back inside and switch to another mode) the HA integration receives a "triggered" message.
To Reproduce
Steps to reproduce the behavior:

Setup a mode with a 30 second "leave delay".
Switch to that mode using the passcode on the keypad or via the app.
While the countdown timer sounds/shows switch back to another (lower/less secure) mode. E.g. "off" or "home".
See "triggered" status appear in Home Assistant.

Expected behavior
I expected that the modes would switch but that the integration would not receive/send/process a "triggered" message.
Screenshots
Node-Red debug message of received state.

Logging from Eufy security add-on:
2022-09-03 09:34:39.066  INFO  Alarm mode for station T8010N23214816C9 changed to: undefined
2022-09-03 09:34:40.310  INFO  Received push notification for changing guard mode
{
  guard_mode: 6,
  current_mode: 6,
  stationSN: 'T8010N23214816C9'
}

**Hardware Information about Home Assistant
RaspberryPi 4

alpha version should fix this (v5.x.x)


AttributeError: 'Block' object has no attribute 'drop_path1'

Dear authors,
Thank you for your code release. I encountered an issue when running the program. The error message is as follows:
AttributeError: 'Block' object has no attribute 'drop_path1'
The error occurs at line 155 of the 'model/base/ViT_Network.py' file:
latent_feat = latent_feat +block.drop_path1(block.ls1(attention)).
Here is the detailed stack trace and the model architecture for reference:
VisionTransformer(   (patch_embed): PatchEmbed(     (proj): Conv2d(3, 768, kernel_size=(16, 16), stride=(16, 16))     (norm): Identity()   )   (pos_drop): Dropout(p=0.0, inplace=False)   (blocks): Sequential(     (0): Block(       (norm1): LayerNorm((768,), eps=1e-06, elementwise_affine=True)       (attn): Attention(         (qkv): Linear(in_features=768, out_features=2304, bias=True)         (attn_drop): Dropout(p=0.0, inplace=False)         (proj): Linear(in_features=768, out_features=768, bias=True)         (proj_drop): Dropout(p=0.0, inplace=False)       )       (drop_path): Identity()       (norm2): LayerNorm((768,), eps=1e-06, elementwise_affine=True)       (mlp): Mlp(         (fc1): Linear(in_features=768, out_features=3072, bias=True)         (act): GELU(approximate='none')         (fc2): Linear(in_features=3072, out_features=768, bias=True)         (drop): Dropout(p=0.0, inplace=False)       )     )     # Other blocks...   )   (norm): LayerNorm((768,), eps=1e-06, elementwise_affine=True)   (pre_logits): Identity()   (head): Linear(in_features=768, out_features=200, bias=True) )
It seems that the Block object does not have an attribute drop_path1, which is causing this error. Could you please provide guidance on how to resolve this issue?
Thank you!

No worries. Problem solved. Appears that the timm version should be timm=0.6.7.


feat(s3store): pass the contentType metadata field to S3 if supplied

After this PR, if the client passes a contentType value in the metadata object, the resulting file on S3 will be created with the value passed.
This change causes tusd for Go to match the NodeJS version's behavior, discussed here: https://stackoverflow.com/questions/74148196/how-to-resolve-application-octet-stream-in-s3-using-tus-node-tusd-uppy-or-net.

Thank you for the PR! tusd currently uses the filetype metadata field to set the Content-Type and Content-Disposition headers in GET responses: https://github.com/tus/tusd/blob/9d85248e11a4e3127321c7d1ff562ada66c9f6f0/pkg/handler/unrouted_handler.go#L1107-L1139
The contentType field is not yet used in tusd. Although it would be handy to be compatible with tus-node-server here, I think it's more important to be consistent in tusd itself. Would you mind using the filetype metadata field for setting the Content-Type on S3 as well?

@Acconut absolutely!! I'll add handling for the filetype field later today. Thank you for reviewing!

@Acconut I have updated this PR with handling for the filetype metadata field and added another test. Thanks for the suggestion.

Thanks for the update! Apologies if my comment was not clear enough but I think we should only use filetype for obtaining a media type and not use contentType at all. Compatibility with tus-node-server would be nice, but then tusd uses different metadata fields for different purposes, which is not good.

Ok @Acconut, while I personally would prefer that publicly posted solutions for one Tus implementation would work against them all, my preference to get something merged that handles my current need to overwrite the file in S3 post-upload is greater. I will make that adjustment so that contentType is not used.

Great, thank you so much for the prompt reviews!


Configurable fzf keybindings

Provides one new global variable.
g:fzf_preview_default_key_bindings
This command determines keyboard shortcuts during an interactive FZF
session. Options are a string passed directly to fzf's "--bind" option.
Default value is 'ctrl-d:preview-page-down,ctrl-u:preview-page-up,?:toggle-preview'

I opted for 1 global variable mapping directly to the command line options because:

FZF's key syntax is a bit different than Vim's. Additional abstraction (eg, providing separate variables for each option) would likely be confusing if we just required the key.
For users who understand fzf's cli, this solution is simpler
Changes very little existing code. It just makes what already exists configurable.

Finally, I added a space after the prompt. Most shells I've worked with have a space after the prompt, and the lack of a space bothered me.

Thank you!


Add Then (>>) to monad spec

For example:
Id.prototype.then = function (m) {
    return m;
}

Promise.prototype.then = function (m) {
    var promise = this;
    return new Promise (function (resolve) {
        return promise.fork (function (dontCare) {
            return m.fork (resolve);
        });
    });
}

Then you could have code like this:
somethingToDo (x) // Returns Future
    .chain (thisAlsoReturnsAFuture)
    .then  (justDoFinalFuture)

instead of:
somethingToDo (x) // Returns Future
    .chain (thisAlsoReturnsAFuture)
    .chain (konst (justDoFinalFuture))

Thoughts?

If then is the same as >>, then:
m.then(k) === m.chain(function () { return k; });

It may be worth mentioning that this is the case and that then can be useful, but I don't know if then should need to be implemented outright or appear as a requirement for any of the structures.
If you can think of a way to incorporate then into fantasy-land, go ahead and submit a PR! I'd love to see how it comes out on paper and would be happy to see mention of >> in the chain specification.

then has a very specific meaning in promise libraries. Please avoid name clashes with those.


canvas DOM element is appending on every toCanvas call

Your .toCanvas do parentEl.appendChild(canvas); on every call.
https://github.com/jibrelnetwork/ethereum-qr-code/blob/master/src/ethereumQrPlugin.js#L44
I think it is not cool idea.
Because some UI can allows users call this method many times.
For example:

user entered amount/address
UI draws canvas
user re-entered amount/address
UI re-draws canvas
Istead of it, now it will draw canvas on each user input.

Plugin goal is just generating the address and QR code. The way parent application uses it is up to developer.
If you mean to have smth like throttle / debounce logic - I suggest that it should be in the parent application, it I the plugin.

I understand, but in this case user of the library must not only use API, but manage DOM.
I recommend you to implement some logic of remove/removeChild/replaceChild.


Update fs2-core to 1.1.0-M2

Updates co.fs2:fs2-core from 1.1.0-M1 to 1.1.0-M2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-core" } ]

labels: semver-pre-release

Codecov Report

Merging #320 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #320   +/-   ##
=======================================
  Coverage   97.34%   97.34%
=======================================
  Files          24       24
  Lines        1319     1319
  Branches       39       39
=======================================
  Hits         1284     1284
  Misses         35       35

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 9cc96f1...20df8cc. Read the comment docs.


Not able to run

Getting this error while importing

This version of the Android Support plugin for IntelliJ IDEA (or Android Studio) cannot open this project, please retry with version 4.1 or newer.

My studio version specification

Android Studio 4.0.1
Build #AI-193.69<IP_ADDRESS>26763, built on June 25, 2020
Runtime version: 1.8.0_242-release-1644-b01 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
GC: ParNew, ConcurrentMarkSweep
Memory: 1246M
Cores: 4

Can you please suggest me how can i resolves this error?

you need to update your android version studio to the beta or canary versions, please refer to this link
https://developer.android.com/studio/preview


Add the newsletter vol. 1 / Update the article list

Add the newsletter vol. 1
Update the article list

Add @ikhoon's Reactive Streams articles
Add @carlsagan21's LINE Bot API slides

Miscellaneous:

Update the article list stylesheet so that the table does not
overflow in a wide screen.

Thanks for reviewing :-)


Contact form not functional,Need to implement Nodemailer for email message delivery (frontend + backend)

Describe the bug
The contact form on the website is not functioning properly, and submitted messages are not being sent to the specified email address. Users are unable to contact the site owners through the form.
Expected behavior
The contact form should send user-submitted messages via email using Nodemailer. Upon submission, the form should validate inputs, and if successful, trigger an email to the designated address with the user's message.
Additional context
Implementing Nodemailer will ensure reliable email delivery from the contact form

Kindly assign me this issue to resolve. @Itachii27 @muruga21 @aswathcm29 @jahnvisahni31 @Vaibhav-Kumar-K-R

go ahead @Hemant9808


Allow customizing badge color

Currently, Stackage badges show up as green. But it's common for badges representing published versions of software to be blue (some examples: ex 1, ex 2, ex 3). It would be useful to be able to customize the badge color with a query string. shields.io does this for Hackage.

Hey @pgujjula, can you point me to an example of a current Stackage badge?

I'm using the current Stackage badge in my project here: https://github.com/pgujjula/apply-merge

Sure, here's some examples: Shelly, bitvec, world-peace

Sounds good, thanks!

Sounds good, thanks for the consideration!

Would you want to contribute a patch? I myself won't have time for this right now. :)

Yeah I can work on it sometime this week :)

Hey @chreekat, I went through the README to set up the project. I was able to successfully run the dev server with yesod devel, but it seems like there's no packages, so I can't test badges. In order to run the postgresql-based server, I tried running the cron job to set up the database, but I get an error:
$ export PGSTRING=postgresql://postgres:postgres@localhost:5432/stackage
$ stack exec stackage-server-cron
*** Exception: CredentialChainExhausted

CredentialChainExhausted is an amazonka error, do I need access to AWS?

Or perhaps there is a simpler way for me to get test data for development?

Oof, I forgot about that. I don't know how to get around the problem of missing packages. (I'm still new to maintaining this project, so maybe there's a way I just don't know.)
I think creating test data for local development is worth its own issue. But it doesn't have to block the badge color project.
For badge color, go ahead and put up a PR when you think it seems good enough. We can fall back and rely on code review and testing it live :upside_down_face:

I opened a PR here: https://github.com/commercialhaskell/stackage-server/pull/331 :)


Make sure plugin rootfs is unmounted on upgrade

In some cases, if a user specifies -f when disabling a plugin mounts
can still exist on the plugin rootfs.
This can cause problems during upgrade where the rootfs is removed and
may cause data loss.
To resolve this, ensure the rootfs is unmounted
before performing an upgrade.
Fixes #32515

@cpuguy83 : That is no cute animal.

That is no cute animal.

Ok, let's close

Updated

DockerSwarmSuite.TestAPISwarmLeaderProxy is failing on powerpc. Stacktrace at https://gist.github.com/anusha-ragunathan/81d151b187e1ec4266e08b8a699ed4ed

LGTM (CI failures are unrelated.)

@cpuguy83 @anusha-ragunathan should this be cherry-picked into 17.05?

@thaJeztah : That would be nice.

ping @vieux FYI ^^


Update eniconfig.tf

Adding Prefix Delegation
This is a best practices to use on EKS
Here is the doc
https://aws.github.io/aws-eks-best-practices/networking/prefix-mode/index_linux/

Including best practices adding prefix delegation

that was a great read. thanks!


CS Group - Missing Targetare link

ENVIRONMENT: production
Browser: Chrome
Version: 120
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
Open the url
Hover over the company's logo.
EXPECTED RESULTS:
There is a 'Targetare' button (with link) just under the 'Careers' one. Link: https://targetare.ro/2316981/cs-group-romania-sa
ACTUAL RESULTS:
The 'Targetare' button is missing.
Attachments:

Retested ✔


Accessibility: enableing/disabling analyzers with a keyboard nonfunctional

Describe the bug
Using a screen reader, it's super unclear how you toggle the state of an analyzer. The standard keys to check/uncheck a checkbox don't work in the analyzers list. The way I currently have to do it is by focusing the cell in the table next to each analyzer that says true/false, route my mouse with NVDA+Shift+M, then left click with NVDA+Left bracket.
To Reproduce

Start NVDA and Ghidra.
Open the analyzers window.
Attempt to check/uncheck analyzers using standard keyboard navigation commands.

Expected behavior
In a perfect world, this list would function like a standard list of checkboxes that I can up and down arrow between, and check/uncheck with the space bar. In addition, there wouldn't be an extra cell to the left of each analyzer that says the state, it would be reported directly in the list.
Environment (please complete the following information):

OS: Windows 10 22H2
Java Version: openjdk <IP_ADDRESS> 2022-08-12 LTS
Ghidra Version: 11.0.1-20240130
Ghidra Origin: Scoop for Windows

Thanks for the info.  Fixing this widget is one of the changes we have on the horizon.