Deterministic intent parser should use the custom entity parser

Description:

Replace entity utterances with an entity placeholder in the DeterministicIntentParser patterns
Harmonize the output format of the CustomIntentParser to be consistant with the output of the BuiltinEntityParser

Codecov Report

Merging #672 into develop will decrease coverage by 0.12%.
The diff coverage is 87.5%.

@@             Coverage Diff             @@
##           develop     #672      +/-   ##
===========================================
- Coverage    88.95%   88.83%   -0.13%
===========================================
  Files           64       64
  Lines         3648     3635      -13
  Branches       672      667       -5
===========================================
- Hits          3245     3229      -16
- Misses         307      309       +2
- Partials        96       97       +1


Error: Column "id" is an identity column defined as GENERATED ALWAYS

Currently postgres jobs are failing on PHP 8.0 & 8.2 with below error:
...

171) CakephpFixtureFactories\Test\TestCase\Scenario\FixtureScenarioTest::testLoadScenario with data set #2 ('TenAustralianAuthors', 10)
CakephpFixtureFactories\Error\FixtureScenarioException: Error in Factory CakephpFixtureFactories\Test\Factory\AuthorFactory.
 Message: SQLSTATE[428C9]: <<Unknown error>>: 7 ERROR:  cannot insert a non-DEFAULT value into column "id"
DETAIL:  Column "id" is an identity column defined as GENERATED ALWAYS.
HINT:  Use OVERRIDING SYSTEM VALUE to override.

...

This is due to changes of phinx package in version 0.13.0 (PR: https://github.com/cakephp/phinx/pull/2085).

It is impossible with the actual phinx version to overwrite ID values.
See the issue here: https://github.com/cakephp/phinx/issues/2193
A solution would be to add a ->disablePrimaryKeyOffset() to by-pass the random setting of the primary key and let the SQL engine manage the assignment.


New comment.

Dear human,
Here's a new entry for your approval. :tada:
Merge the pull request to accept it, or close it to send it away.
:heart: Your friend Staticman :muscle:

Field
Content

message
Nice blog! Is your theme custom made or did you download it from somewhere? A theme like yours with a few simple tweeks would really make my blog stand out. Please let me know where you got your design. Bless you dadeefdcddfdbkfb

name
Smithc518

email
d121916794b2fb67a79810e031e2172a

url
http://dadeefdcddfdbkfb.blogspot.com/

hidden

date
2017-09-06T17:34:55.738Z

meh


Possibility to set Source name when using the built-in module loader

Is it possible to specify/change the Source name of an imported module? Currently when using the built-in module loader (using import) it uses the absolute file path, but I do not want to expose any absolute paths.
Also currently the two available module-loaders are hardcoded into the JSRealm, but it might be nice if we could write custom module loaders and bind them to a context.

Hi @rogro82,
Using a custom Truffle FS you can customize what gets loaded. In principle, you could use a custom FS to map arbitrary module names to your desired paths and sources. We have a small example here.
Providing a mechanism to customize the default module loader is an interesting idea, but might require some changes to the Context polyglot API, so this is something that might require some discussions.

Thanks! I was already started on a custom FileSystem and using virtual relative paths. Mainly because the remote chrome debugger would otherwise always show the actual file paths.
Because I had to do some additional work I decided to also implement a custom module loader which I now bind to the realm using reflection. It would be really appreciated if I could get a more stable solution.


Please, add my university, thank you

@cesarnandes file/lib/domains/br/edu/fef.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@cesarnandes Pull request merged. Thank you.


Add installation instructions for Python2

The main purpose to add this was to add python2-dev since without its installation make install doesn't work.

@palmergs can you please look into and merge this PR?

Closing this PR because of no response from the owner.


Created using Colaboratory

Realized that out of the 4 columns that could be used for predicting, that none of them would give me a valid prediction data since the majority class is greater than 70% (some case 100%).

Updated my dataset target column further. I cleaned my dataset going from ~1500 observations to just over 500.


⚠ Bofrost.de has degraded performance

In a519daf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1450 ms

Resolved: Bofrost.de performance has improved in 8bd28c8 after 9 minutes.


Data shuffling

Hi -
I get that one can define data splits in the CSV (e.g. what is used for training, validation and testing) or if no split column is provided the splits and their respective defined fractions are allocated randomly. If the latter is the case are these allocations shuffled after each epoch? E.g. is the training subset always the same the data after initial assignment? I cannot find a parameter to switch shuffling on or off.
Sorry for this somewhat naive question, this is probably visible somewhere inside the code but spotting a feature like that is beyond my skills :(

@tboo the split is done once before training, then data within the training split is shuffled each epoch, the subset is always the same, the order in which it is presented changes.
Would you like to turn shuffling off? In that case I may add a training parameter for that, like shuffle: True by default that you can set to false.

Thanks for the clarification. I am not intending to turn shuffling off. In fact the 'opposite' is the case. My question was if it is possible to shuffle the splits after each epoch, meaning that the validation and training data set is different at each epoch.
The reason I am asking is that my image classification reaches 90%+ accuracy on test and validation data during training but if I use the obtained model for prediction of previously unseen data my accuracy drops to 70%. I wonder if this is because the validation set is not representative enough and I thought selecting new validation data at each epoch would somehow compensate for that. But the key is likely to increase the sample size/split fraction of the validation data. I will play around with that.

@tboo I see. Leaking datapoints from the validation set or the test set into the training set is a bad practice, it will increase your numbers, but those numbers will be meaningless and will not improve your performance on additional held out data, it will probably decrease it because it would overfit.
The reason for your performance may have several explanations, among with a consistent different in the distribution between your original data and the held out one. If that's the case, probably spending some effort tagging more data from the same distribution of the held out would probably be a good idea.
Anyway, this is not an issue, but a discussion, I'm closing it and and encourage you to open a discussion on Gitq if you want to continue it further.


Fix UTF-8 Encoding.

Fix UTF-8 Encoding in CSV for Download and Email.

5f68f0239b3be8177daad113a22c50feffc51607 and<PHONE_NUMBER>33fb8144905c7bc696791abd3c47c1


Phobos 3 Setup

Copy over base packages that have no imports. Rewrite dub.sdl so that it can be used for Phobos3. Update .gitignore.
This PR lays down some of the basics of the Phobos 3 project. Moving forward, Phobos 3 will be developed and maintained side-by-side with the existing Phobos 2 code. This will simplify the process of maintaining both code-bases as we progress. Future PR's for Phobos will be required to submit fixes to both versions to ensure that both maintain parity.
@WalterBright @atilaneves Thought you two would want to know this was happening.
@thewilsonator Would you mind doing the review honors?

Okay, so we're not doing this in a separate repository then?
Does it not make sense to keep it separate since it is both organizationally and code-wise meant to be completely separate in management?

@LightBender is this meant to be compiled in or is it dub-only?

@ibuclaw The intent of V3 is to be a "source only" distribution, so no binary or compiled distribution. But it is still intended to be shipped with the compiler in source form, as V2 currently is.
The purpose of the DUB file is to make it easier to run tests and CI processes, while using a common interface that the community knows well. Precisely how that meshes with the release process isn't something that we've worked out yet. But if we need a binary distribution in the future, we should be able to work that in.

Source only? What?
This kills all value in adding a common event loop and many other libraries like that.

@ibuclaw The intent of V3 is to be a "source only" distribution, so no binary or compiled distribution. But it is still intended to be shipped with the compiler in source form, as V2 currently is.

OK, I will leave it out of downstream then.

@rikkimax Wouldn't the Event Loop need to be in DRT to work correctly across shared libs?

@LightBender Having Phobos v2 and v3 be developed side-by-side in the same branch of the same repo sounds reasonable to me, but I think it's better to use a separate branch (e.g. next or v3.x.x in order to:

ensure the current v2 releases remain unaffected
to allow more drastic changes to be done.

Can we please move this under a source directory ? DMD / druntime do it correctly, but we never corrected this error for Phobos. Now sounds like the right time.

both lib and sys needs to be in a directory that can independently be provided to dub

Why is that ? I'm not sure I understand the use case.

both lib and sys needs to be in a directory that can independently be provided to dub

Why is that ? I'm not sure I understand the use case.

It's a consequence of the import path switch on the compiler.
Having multiple binaries all share the same import path means that you can import any module from them, even if you only link in one of them.
As seen with arsd, people do this and do not have an understanding of why they get linker errors.

Hold on, why would you link manually ? We have Dub for this. If you're not into that, we have dmd -i and co.
And having separate directories for lib and sys does not prevent people for not passing in a module on the command line.

Yes but you are thinking about static libraries.
This thought process does not work when you realize that each sub root is meant to be its own shared library to ensure we never hit the 64k symbol limit on Windows.

This thought process does not work when you realize that each sub root is meant to be its own shared library to ensure we never hit the 64k symbol limit on Windows.

That's quite premature. If we reach that limit we can always find solutions to it. Some of those solutions do not require to change the code organization (e.g. per-package compilation). Also, that argument does not seem connected to your earlier argument:

Having multiple binaries all share the same import path means that you can import any module from them, even if you only link in one of them.

So I really don't see the necessity of multiplying entities beyond necessity.

@LightBender
I watched this repo for 2 days and im already annoied with the emails about d2 stuff
what happened to a draft repo? https://github.com/LightBender/PhobosV3-Design/discussions/16
I bit my tougue, and started to consider rewriting a small file but nessery to get before futher work: https://github.com/LightBender/PhobosV3-Design/discussions/24
Only for d2 stuff copied and pasted in 2 hour later with very legacy descriptions and ideas(modern c++ design isnt very modern) and an approval process that didnt react at all
Whats the plan? whats the process? Are we not trying to make a new high quality std or is this a minor version update where you will copy each d2 function 1 by 1?

@crazymonkyyy This is not intended as a complete ground-up redesign of Phobos, and nothing in the design docs or discussion has ever even considered that as a possibility. The primary reason we are doing it this way is that it is the only way to audit auto-decoding by default. Yes, that is giving us the opportunity to do some rethinks of stuff that is broken or has severe limitations that can't be overcome without a rebuild, but it is not all-encompassing. Furthermore the stuff that is being worked on right now is very low-level and used by everything else, so it's not expect to change much, but std.traits is up for a pruning so that is happening right now.
V3 will look noticeably different from V2, but it should still be familiar enough for existing V2 users to be able to port their code over to V3 without having to rebuild their codebases from the ground-up in the process. It is very much an incremental change that sets the stage for bigger things to come.

@Geod24 The question I have is, would this work with the DMD search paths for Phobos, or will we need to make some changes to the compiler?

We'd need to add one path to the configuration. Fairly trivial.


Make UiThreadRenderTimer to calculate the next tick time based on expected FPS

DispatcherTimer.Interval indicates the interval between the previous tick completion and the next one. Render timer, on the other hand, needs to have its interval to indicate time between tick starts.

UiThreadRenderTimer doesn't follow .NET or Avalonia naming conventions and should be fixed -- probably in 12.0.

A special case is made for two-letter acronyms in which both letters are capitalized, as shown in the following identifier:
IOStream

https://learn.microsoft.com/en-us/dotnet/standard/design-guidelines/capitalization-conventions#capitalization-rules-for-identifiers
Dispatcher.UIThread is the existing Avalonia convention as well.

UiThreadRenderTimer is a private api, which can't be used directly without enablding private api access.
We can rename it at any point.

@maxkatz6 OK, I did notice the PrivateApi attribute was removed here as well.

@robloo oh, you are right, it was. Still not a breaking change, but it was likely unintentional, cc @kekekeks.


'equals()' between objects of two inconvertible types

Summary
Presently, in the project, there is .equals() method between the target and arguments which are of incompatible types i.e... between String and CustomValidatingEditTextView. While such a call might theoretically be useful, most likely it represents a bug.
These types of inconvertible types between two objects must be resolved.

solved in a previous PR


[roadmap] Add brave search API for RAG

Why
Being able to search the internet would be a nice addition.
Description
Brave has also a free tier that allows you to search for free. It would be a fantastic update to be able to let the LLM analyze the question then formulate searches and RAG.
https://brave.com/search/api/
Requirements
Support of the brave endpoint, then add the ability to use search on the prompt page, so the user can choose to use web search or not.

Hi Gian Maria, contribs welcome for this :)


Issues with running API Migration Tool

We are migrating from on premises version to public cloud.  How to provide "to": version information for edge public cloud. This information need to be set into config.js
https://github.com/apigeecs/apigee-migrate-tool/blob/master/config.js

When running this tool I am getting below error. I am guessing it is due to cert issue. We are using our company signed ssl certificate for management server. Please let us know workaround for this.

Running tasks: exportAll
Running "exportAll" task
Running "exportProxies" task
========================= export Proxies ===========================
getting proxies...https://:/

Error: unable to verify the first certificate

Hi @kurtkanaskie , @adamjonescdk
I am trying to export data from 4.17.05 version of apigee. I used both http and https urls for management servers but it is failing with ">>null" as output.
++++++++++++++
Running tasks: exportProxies
Running "exportProxies" task
========================= export Proxies ===========================
getting proxies...http://xxxxxxx:8080

null
+++++++++++++

Can you please check if this version of Apigee supported by this tool?
Regards,
Satyendra

Hi @kurtkanaskie ,
Thanks for reply. Yes I can make management API call using curl. And Yes, We do curl -k for management API call.
We have another sandbox environment where management server is just http (not https) and it doesn't work with plain http.
Running tasks: exportProxies
Running "exportProxies" task
========================= export Proxies ===========================
getting proxies...http://xxxxxxx:8080/

null

===========
Regards,
Satyendra

I tried with just exportDevs only instead of exportAll it is not working. Here is the output
Running tasks: exportDevs
Running "exportDevs" task
========================= export Devs ===========================
getting developers... http://xxxxxx:8080
getting developers...http://xxxxxx:8080/v1/organizations/xxxxx/developers

Hi @kurtkanaskie
I resolved the certificate error issue. I am facing issue running importKeys. It imports credentials but developer apps have pending api product approvals. Out of 95 Apps 53 apps are having pending api product approval. Not sure if it is issue with migrate tool of something wrong in cloud management Api .
Regards,
Satyendra

Hi @kurtkanaskie
Thanks for reply. Looking into logs. It appears that tool is maping the approval call and getting back the correct response code 204 but when looking in apigee portal it is still showing Pending status.
Resp [204] for<EMAIL_ADDRESS>- app-web - OrderMgmtProduct - **key_removed** - https://api.enterprise.apigee.com/v1/organizations/org_name/developers/email_removed@brinker.com/apps/app-web/keys/**key_removed**/apiproducts/OrderMgmtProduct?action=approve ->
I tried the same api call using curl/postman got 204 response but this time app got approved successfully.
Is it something Apigee Management API blocking the approval status when we are running this tool in batch mode. When this tool runs it makes series of api calls to approve the api products. For example I have 10 API products for a given app. I see only one api product is in approved state for a given app rest 9 stay in pending state. Even though API returns success code 204  for all of them. In the next run another api product get approved. I tried to run multiple time an individual app like below
grunt importKeys -v --src=./data/apps/*/app-web
At this point I am not sure if it is the migration tool issue or Apigee Management API issue. But I am sure this was tested before. Tagging more folks to this. Guys Please help me.
@vinit4u16  @jerrylawson  @swilliams11  @giteshk  @stevetraut

Hi @kurtkanaskie ,
Regarding API Product approval, there's separate code to set status, there
may be an issue with that.
See:
https://github.com/apigeecs/apigee-migrate-tool/blob/99a6ac3fd37823a9da761f46d40ee1a7c3f5bfe7/tasks/app.js#L207
I worked with Apigee support on why management api for product approval giving successful response but it is not approving the all the products associated with a developer app. Apigee support said this API call is asynchronous so it returns a success response immediately and product approval happens in background. it has some traffic limit it seems.
They said if you make your client synchronous and introduce a delay (like 1/2 sec) then it will work. So is it possible for you to update your script to make product approval synchronous and introduce delay with some milliseconds. I am not a javascript expert so I am requesting you to send me updated scrip and i will test it in our environment and let you know. Appreciate your help in this matter. It will be helpful to other users who wants to use this in future.
Regards,
Satyendra

Hi @kurtkanaskie ,
Can you please help?

Hi @kurtkanaskie ,
Thanks for reply. Yes I can make management API call using curl. And Yes, We do curl -k for management API call.
We have another sandbox environment where management server is just http (not https) and it doesn't work with plain http.
============== Running tasks: exportProxies
Running "exportProxies" task ========================= export Proxies =========================== getting proxies...http://xxxxxxx:8080/ ">>" null
=========== Regards, Satyendra

Hi @satyendrasrivastava may I know what is your solution regarding error you encounter here when exporting?

@satyendrasrivastava
What have to done to resolve the cert issue?

@satyendrasrivastava @kurtkanaskie
We are on Apigee edge and we are trying the export the Products and ran into the same issue i.e
" Running "exportProducts" task
========================= export Products ===========================
getting products...https://api.enterprise.apigee.com

null"

Can you please help by providing what was done to fix the issue.
And we tried to export Developers and this time the tool is making a call to Apigee to get the developers but failed as
"Running tasks: exportDevs
Running "exportDevs" task
========================= export Devs ===========================
getting developers... https://api.enterprise.apigee.com
getting developers...https://api.enterprise.apigee.com/v1/organizations/dish-nonprod/developers

Note: Basic auth always returns 401 for user accounts that have MFA enabled.
https://docs.apigee.com/api-platform/system-administration/basic-auth
Or maybe you just mistyped your password.
To validate your password, login to the UI at https://login.apigee.com
If necessary, reset your password at https://login.apigee.com/reset"

Inputs are much Appreciated


Switch AsyncDisposableStack.prototype.disposeAsync to an async method

This updates the proposal to use the "async method" terminology from the Async Iterator Helpers proposal and Array.fromAsync, as defined by https://github.com/tc39/ecma262/pull/2942.
This is intended to be a purely editorial change, with no normative behavior changes, as the normative behavior is intended continue to match the intended behavior as described and agreed upon by the committee.

cc: @tc39/ecma262-editors

Closing for now. May revisit when https://github.com/tc39/ecma262/pull/2942 is merged.


pull request for e2e test at 01/26/2017 18:15:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added a new path [HTML ,CSS]

We'll have to provide a link to the front end development path at the end of the article, which is yet to be done

@aka8921 please proceed with that.


Updates docs to show the 'Result' type

Hopefully this resolves #385. @yinanfang Does this clarify things?

Cool, thanks! I think we need to apply this same fix in https://github.com/Moya/Moya/blob/master/docs/Providers.md and https://github.com/Moya/Moya/blob/master/docs/Examples.md too.

Ahh gotcha.  Will do.

:bow:

Current coverage is 91.47%

Merging #388 into master will not affect coverage as of 76de508

@@            master    #388   diff @@
======================================
  Files           13      13
  Stmts          446     446
  Branches         0       0
  Methods          0
======================================
  Hit            408     408
  Partial          0       0
  Missed          38      38

Review entire Coverage Diff as of 76de508

Uncovered Suggestions

+1.35% via ...ource/Response.swift#76...81
+1.13% via ...+ReactiveCocoa.swift#54...58
+1.13% via Source/Moya.swift#188...192
See 7 more...

Powered by Codecov. Updated on successful CI builds.

@ashfurrow phew, went a little nuts on Examples.md: ab85328

https://github.com/Moya/Moya/blob/ab853283d8463e53fe306ed2b52b623f9db88856/docs/Examples.md

Looks amazing! 😻

@colinta @ashfurrow Thank you so much for the update! I just came across another problem. I got kCFStreamErrorDomainSSL, -9843 error. I found that this happened because the Apple ATS which doesn't allows insecure HTTPS call. Alamofire suggested an solution here: https://github.com/Alamofire/Alamofire#app-transport-security & https://github.com/Alamofire/Alamofire/issues/977#issuecomment-167925391
Do you know how I can configure the network manager in Moya similarly? Should I open another issue for this?

Oops... I just found the documentation here: https://github.com/Moya/Moya/blob/master/docs/Providers.md#advanced-use Sorry about that. It's not a very smart question.


Please, create a step-by-step guide to set up an Azure OpenAI FREE account (for the API key)

Is your feature request related to a problem? Please describe.
In chapter 4, it seems to me that we need An Azure OpenAI API key. But setting up a FREE Azure OpenAI service is not straightforward. Also, it seems Azure needs a company email, not hotmail, gmail and the likes. What if I'm a freelancer?
Describe the solution you'd like
I'd like to see a step-by-step guide to set up a proper free Azure OpenAI account so that we can use the API key.
Describe alternatives you've considered
I've looked for the official Microsoft guide, but it's not satisfactory. It's confusing and cumbersome.
Additional context
Add any other context or screenshots about the feature request here.

Hey @bringonthestorm - there is no Azure OpenAI Free Account unfortnately. We have created instructions to use the OpenAI API (non-Azure). You do receive some free credits when creating an account there and do not require a busineess account. https://github.com/microsoft/generative-ai-for-beginners/tree/main/00-course-setup


[Linux] Updates of p2p and tray not working in About dialog

I think there might be permission issues here. @crioto I do not know if this is due to how the launcher is installing the tray now.
This is happening on most Linux systems tested. Last on was an Ubu 16.04.3 system.

Interesting steps:
(1) tried tray update, that worked
(2) after restart due to tray update the p2p update worked

I will review permissions

EZSSH does not seem to be working on the system.

I have check the permissions of installed files and they all belong to user test2. This excludes permission problems, so probably @Lezh1k can take a look at it and determine problem better.
Regarding p2p not being working - the service was down and this is a launcher's failure, so I will fix it

Need to change folder permissions in deb package.


[eas-cli] branch mapping utility functions

Why
This PR:

adds branch mapping utility functions and updates their definitions to be the same as www. The previous definitions were a rough approximation of what was declared in www and were not correct.
does not change the behaviour of our commands does not change. However, some assertions were added in order to get our cli code to play well with the new branch mapping definitions.
adds rollout utility functions for use later

Test Plan

[ ] new tests pass
[ ] current tests pass

/changelog-entry chore  added branch mapping utility functions


Update FlexSearch Dependency

This is a critical upgrade, mostly due to this issue. With the default configuration, if a Ghost post title includes the search query, then only those posts with the search query in the title will be returned, even if other posts have the query in other fields like string_tags.

Thank you for the feed back. I will do the upgrade but I want to fully re-test everything first before jumping from a 6.x to a 7.x version.


Fixed mesh renderers of the masked enemy not being updated

Fixed mesh renderer list of the masked enemy not being updated after applying/clearing cosmetics.

Does this fix anything?

The masked enemies are instance based, not pooled. So theres really no reason to add the cosmetics to the meshrenderers unless theres some sort of other issue pertaining to that

Does this fix anything?

Fixes compatibility with some other mods such as Masked Enemy Overhaul

Is there a repository link or something similar of a mod that has a bug with this?

https://i.gyazo.com/55916d58b95d7a6e8c3cb8b1d08710bb.png
https://i.gyazo.com/01afce1d6d77b976f4831f7e77e411d4.png
TLDR it not being updated can cause there to be null references when executing EnableEnemyMesh

I see, thank you for the fix. Will merge


Failed to open TUN device

OS: windows_10_pro_21H2_19044.1806
tap-windows: v9.9.2
nConnect: v1.0.8
1, When I use VPN mode

2, When I create a network called "nconnect tap0"

3,

Didn't I use it correctly?

Could you please undo the manual change you've done to the system (e.g. remove the network you created manually) and try this version #41 ? You need to build it yourself. I can provide a pre-built version if it's not convenient for you to build yourself.

I have tested the manual creation and installation of clients when adding network cards, but they are not allowed

I have installed nconnect client ,reported error too

So you got the same error as in your 2nd image?

yes!

It seems to be a problem in this library go-tun2socks and Ii haven't found a solution yet...


Support for unusual characters on a view's name

I have to support latin1 characters on some view's name, so I made a few fixes so that I could still use scenic.

@aokiji I deleted a few pieces of Hound feedback I'm comfortable ignoring here, but if you don't mind taking a look at the rest we can try and get this out in the upcoming release. It's mostly whitespace changes and splitting a few things over multiple lines.

If you rebase onto master, it may clear up some of the CI issues as well.

I rebased and acted on some of the feedback from Hound. Now all tests are passing. @calebthompson I would be very glad to see this change on the next release

Merged in #188


Error codes have to be changed accordingly in token endpoint response in device auth code grant flow

Describe the issue:
The error codes showing currently are not RFC matching in token endpoint response in Device auth code grant flow.

How to reproduce:
Can be changed accordingly to the RFC standard through the code level.

Expected behavior:
(According to the RFC)
https://tools.ietf.org/html/rfc8628

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.10.0
OS: Linux
Database: H2
Userstore: LDAP

Related issues:
https://github.com/wso2/product-is/issues/9060

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


Fix SAVED_LOCAL_CREDS message

[x] npm run test succeeds.
[x] npm run lint succeeds.
[x] Appropriate changes to README are included in PR.

It is different from the actual path and it was fixed.
Current:
Authorization successful.
Local credentials saved to: ./clasprc.json.

After:
Authorization successful.
Local credentials saved to: ./.clasprc.json.

LGTM. Thanks for the contribution @abetomo 🎉


Can't use when in full screen

Love the theme got a question, how do i make this theme work when in full screen mode? when i go into full screen mode the screen is not transparent

Don't you see your wallpaper, while in full screen? Probably its issue with electron, not with theme, but I will check. Please provide the screenshot and your config (part with apc or other extension configuration), maybe you can lower the value of window transparency to see the effect


robotsMap memory continues to grow

Hello, I am using colly to visit some websites and set c.IgnoreRobotsTxt = false.
As it runs, you will observe that the memory continues to grow over a relatively long period of time.
This growth rate is difficult to observe using pprof.
As a control experiment, I set up a set of colly with the same configuration, the only difference was setting c.IgnoreRobotsTxt = true.
After running for a period of time, the memory of the latter continued to stabilize at 1G, but the memory of the former continued to increase.
Apart from colly.NewCollector(), I can't find any other way to reset this robotsMap if c.IgnoreRobotsTxt = false is set

I can raise a PR to set the size or other limits of robotsMap to avoid continuous increase in memory.


[BUG]wrong again

Describe the bug | 描述错误 | BUGの説明
A clear and concise description of what the bug is.
To reproduce | 复现步骤 | 再現方法
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See error

Context | 运行环境 | バックグラウンド

OS: [win10]
Arch: [e.g. x64]

Error log | 报错日志 | ログ
0%
CPU: Core™ i7-4770
RealCUGAN-pro
2
0
Please Input
Please Input
C:\Users\Administrator\Downloads

Log
[{"label":"CPU: Core™ i7-4770","value":-1},{"label":"GPU 0: Intel(R) HD Graphics 4600","value":0},{"label":"GPU: Auto","value":114514}]
{"gpuid":-1,"inputpath":["C:\\Users\\Administrator\\Downloads\\微信图片_20230719074724.jpg"],"model":"RealCUGAN-pro","modelscale":2,"modelnoise":0,"outputpath":"C:\\Users\\Administrator\\Downloads","targetscale":null,"tta":false}
2023-07-19 07:48:41.720 | INFO     | __main__:main:58 - projectPATH: C:\Users\Administrator\AppData\Local\Programs\Final2x\resources\app\resources\Final2x-core
2023-07-19 07:48:41.720 | ERROR    | src.utils.getConfig:targetscale:125 - targetscale must be int or float
2023-07-19 07:48:41.721 | ERROR    | src.utils.getConfig:getConfigfromBase64toJson:61 - An error has been caught in function 'getConfigfromBase64toJson', process 'MainProcess' (7460), thread 'MainThread' (2240):
Traceback (most recent call last):
File "__main__.py", line 88, in <module>
File "__main__.py", line 63, in main
> File "src\utils\getConfig.py", line 61, in getConfigfromBase64toJson
File "src\utils\getConfig.py", line 73, in _setConfig
File "src\utils\getConfig.py", line 126, in targetscale
TypeError: targetscale must be int or float
2023-07-19 07:48:41.722 | ERROR    | __main__:main:63 - An error has been caught in function 'main', process 'MainProcess' (7460), thread 'MainThread' (2240):
Traceback (most recent call last):
File "__main__.py", line 88, in <module>
> File "__main__.py", line 63, in main
File "src\utils\getConfig.py", line 61, in getConfigfromBase64toJson
File "src\utils\getConfig.py", line 73, in _setConfig
File "src\utils\getConfig.py", line 126, in targetscale
TypeError: targetscale must be int or float
Traceback (most recent call last):
File "__main__.py", line 88, in <module>
File "__main__.py", line 63, in main
File "loguru\_logger.py", line 1251, in catch_wrapper
File "src\utils\getConfig.py", line 61, in getConfigfromBase64toJson
File "loguru\_logger.py", line 1251, in catch_wrapper
File "src\utils\getConfig.py", line 73, in _setConfig
File "src\utils\getConfig.py", line 126, in targetscale
TypeError: targetscale must be int or float
[7460] Failed to execute script '__main__' due to unhandled exception!
CLOSE CODE:1

Additional context | 附加内容 | 追加コンテキスト
Add any other context and screenshots to help explain your problem.

https://final2x.tohru.top/pages/guide/use.html#tooltip
Custom Scale should be a number so you should not set it to NULL. I will fix it.


[Beacon] Launch Planning

Overview
This pilot scope will be to use the Beacon MVP to 1)  run against DDS owned pages to refine the audits and the reports.  2)  engage a select list of adopters to run Beacon against their pages and provide feedback  3)  Plan for the Beacon launch to a broader group of adopters.
This epic will include linked issues to prepare for and launch Beacon to the ibm.com Carbon community.  (3 above)
Additional Information

This epic has been recreated in the Beacon repo.  epic 22

Closing this epic as the epic linked issue #1157 has been recreated in the Beacon repo.


ssh options

hi there, short question: How is it possible to define ssh options like ssh -o ServerAliveInterval=2 foo.bar.org in the configs? I've tried:
  foo:
    Hostname: foo.bar.org
    ClientAliveInterval: 2
    ClientAliveCountMax: 1

but those settings don't seem to be considered, thanks for feedback.

The Client* options are on server side, you want the one that start with Server*.

@ivomarino your of defining ssh options is correct, but as @ocean90 said, these options are not client-side ones
here is the full list of assh supported ssh options:
https://github.com/moul/advanced-ssh-config/blob/master/pkg/config/host.go#L17-L112

my issue, I wrote ServerAliveInterval but wanted to say ClientAliveInterval, fixed the text in the issue.


creating a slideshow

Is it possible to create a slideshow, something like these ones:
https://superuser.com/questions/833232/create-video-with-5-images-with-fadein-out-effect-in-ffmpeg/834035#834035
using this package?  I couldn't figure out how to get the --loop parameter.

I am also curious in this.  I've created a -filter_complex that would work in a command prompt, except I have so many pictures that the end command is too long.
The command I've worked out follows this basic format:
-loop 1 -t {duration} -i img_%03d.png
then once all the loops are specified each image appends this entry into a "- filter_complex" statement

" [{count+1}:v][{count}:v]blend=all_expr='A*(if(gte(T,{duration}),1,T/{duration}))+B*(1-(if(gte(T,{duration}),1,T/{duration})))'[b{count+1}v];
and finally to wrap up the -filter_complex each image also appends the following:
[b{count}v][{count}:v]

There's also a concat and some other things but I think I saw some explanations on how to pass those options through.
Is it possible to do such a complex command with ffmpeg-python?


Polyfilling Promise.allSettled does not work on Edge 18

Describe the bug
I am polyfilling Promise.allSettled via core-js and everything is working fine on Chrome and Firefox. However, when I open my webapp on Microsoft Edge, I get an error stating that allSettled is not defined:
Object doesn't support property or method 'allSettled'

Honestly I am not even sure if this should be filed here or under the core-js repository, in case it does not belong here I will move the issue there.
Did you try recovering your dependencies?
Yarn version:
➜ yarn --version
1.17.3

Which terms did you search for in User Guide?
I checked the guide for polyfills, and I am polyfilling like this in my app:
polyfills.js
import 'react-app-polyfill/ie11';
import 'react-app-polyfill/stable';
import 'core-js/features/promise/all-settled';

index.js
import './polyfills';
import React from 'react';
import ReactDOM from 'react-dom';
//...

Environment
➜ npx create-react-app --info
npx: installed 91 in 11.825s

Environment Info:

  System:
    OS: macOS High Sierra 10.13.6
    CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz
  Binaries:
    Node: 11.7.0 - ~/.nvm/versions/node/v11.7.0/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 6.5.0 - ~/.nvm/versions/node/v11.7.0/bin/npm
  Browsers:
    Chrome: 75.0.3770.142
    Firefox: Not Found
    Safari: 12.1.2
  npmPackages:
    react: ^16.8.6 => 16.8.6
    react-dom: ^16.8.6 => 16.8.6
    react-scripts: 3.0.1 => 3.0.1
  npmGlobalPackages:
    create-react-app: Not Found

core-js version:
➜ yarn list --pattern core-js
yarn list v1.17.3
warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version
<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>   └─<EMAIL_ADDRESS>
react-app-polyfill version:
➜ yarn list --pattern react-app-polyfill
yarn list v1.17.3
warning Resolution field<EMAIL_ADDRESS>is incompatible with requested version<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>✨  Done in 0.96s.

Steps to reproduce

Install the latest core-js
Polyfill Promise.allSettled from 'core-js/features/promise/all-settled';
Try to call Promise.allSettled in Edge

Expected behavior
Promise.allSettled should be polyfilled (output below from Chrome):
Object.getOwnPropertyNames(Promise)
(8) ["length", "name", "prototype", "all", "race", "resolve", "reject", "allSettled"]

Actual behavior
Promise.allSettled is not polyfilled (output below from Edge):
Object.getOwnPropertyNames(Promise)
(8) ["prototype", "name", "reject", "resolve", "all", "race", "length"]

Reproducible demo
I could provide a repro repository if needed.

try importing 'core-js/es/promise/all-settled' instead

Same issue here, but with IE11 (so I'm sure it's the same in Edge 18). Could this have something to do with the compat data? When I look in the debugger I don't even see the file been bundled in

Have you tried the fix suggested by @Roybie? I did not have the chance to try this yet.

@simonedavico I actually did that initially and it didn't work...ended up just polyfilling it myself

I have a similar problem. Perhaps someone can enlighten me, I'd expect Promise.allSettled to by polyfilled just by importing
import 'react-app-polyfill/stable';
since Promise.allSettled reached stable in core-js from v3.2.0 and I can see CRA is pulling in core-js v3.6.4 currently. Am I missing something?

If this is still an issue in the current version of Edge please open a new issue.


Document that "index" can not be used unquoted as a column name

For details, see discussion in https://github.com/cockroachdb/cockroach/issues/26993.

@rmloveland, we kind of cover this already here. But perhaps it's too subtle?

we kind of cover this already here. But perhaps it's too subtle?

I think so, yes.  I'm not sure if I would be able to understand it from that doc, at least not easily.  The crux of the issue seems to be that our behavior differs from Postgres, so I think the work item here is to document it explicitly with a mention of Postgres.


[clkmgr] Minor documentation updates

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

Thanks Tim, LGTM


added support for verge

Will Verge get added to the offline wallet? If yes, do you have any ETA?

add please

Add this features, please!

Any update on this? Want me to fix the conflict?

I guess they want money for that, even if the code is done:
https://trello.com/c/4Aa7Ug5a/20-altcoin-support-information

Add it please, it's booming

Please consider supporting it. Thx

add it please, it's the best addition to put on

Ledger is asking € 99000,- for the the app to be developed. Is it possible the community can pay for it?
It might be possible if enough hodlers are willing.

Could you please XVG to nano s. MUCH NEEDED.

The only thing this shitcoin is good for is a constant and never-ending stream of drama


Fix azure functions service bus binding maxAutoRenewDuration

Fixes #98589 #78261 #62110
Documentation incorrectly states that there's a maximum value for maxAutoRenewDuration on the Service Bus side

Docs Build status updates of commit 751b3f7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-functions/functions-bindings-service-bus-trigger.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@ggailey777,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

From what I can tell, this is a good update. Sorry for the delay.
#sign-off


Add the possibility to use BLF IK in walking module

This requires https://github.com/ami-iit/bipedal-locomotion-framework/pull/557

@S-Dafarra I think you can review the PR


[FLINK-27805][Connectors/ORC] bump orc version to 1.7.8

https://issues.apache.org/jira/browse/FLINK-27805
Apache ORC 1.5.x is EOL -- last HF release happened on Sep 2021 https://orc.apache.org/news/2021/09/15/ORC-1.5.13/
Need to bump to 1.7.x -- Release Notes
ORC now supports writers with FSDataOutputStream (instead of just paths previously) so cleaning NoHivePhysicalWriterImpl and PhysicalWriterImpl

@lirui-apache @JingsongLi Does one of you want to review this PR?

Aslo cc @pnowojski  / @akalash  that might be interested

Please add @liujiawinds as a co-author [1] to the commit.
[1] https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors

Thank you, @pgaref and @dmvk .
cc @williamhyun

@flinkbot run azure

BTW, @pgaref . As you know, @wgtmac started Apache ORC 1.7.9 RC1. Can we test RC1 in this PR?

https://github.com/apache/orc/issues/1437

If we need some patches in order to help this PR, Apache ORC community can cut RC2 accordingly to help Apache Flink community.

hey @dongjoon-hyun  -- thanks for keeping an eye!
Just triggered a run on 1.7.9-SNAPSHOT

Thanks @dongjoon-hyun and @pgaref. I will keep an eye on it and prepare RC2 if required.
BTW, it would be better to test 1.7.9-RC1 instead of 1.7.9-SNAPSHOT. But I think they have the same content except the version.

FYI @dongjoon-hyun  @wgtmac  we had a green run for 1.7.9.
Switching back to 1.7.8 to get this PR merged and I will create a new ticket for the 1.7.9 bump when its ready.

It's great! Thank you so much, @pgaref .

BTW, could you cast your +1 with that information on Apache ORC 1.7.9 vote thread?

BTW, could you cast your +1 with that information on Apache ORC 1.7.9 vote thread?

Sure, will do

Also, cc @williamhyun once more too..

@tzulitai / @dmvk  can you please take another look?


Add Status of Findings when Close Finding

This is a try to resolve the issue discussed in #7059
Result:

@alles-klar as far as I know you are an expert to the k8 deployment: Do you know why it fails?

I re-ran the failed ones and it seems to be working now.

@Maffooch you are right with the .get. Was a bit lazy from my side to do it in one file but not another. I implemented your suggestion and added a missing apostrophe.

I have 2 improvements that I would like to add in the future:

For the duplicates field there should be the possiblity to choose from findings which is the "original"
Remove the possibility to edit the status in the EditForm, s.t. nobody can just change the status without adding a note.

What do you think? Would this be also good PR?

For the duplicates field there should be the possiblity to choose from findings which is the "original"

This seems sensical to me when manually setting the flag.

Remove the possibility to edit the status in the EditForm, s.t. nobody can just change the status without adding a note.

I think this would be a little annoying tbh. Especially for smaller teams or folks just getting to know dojo. I think it would be best to leave that to the users

Remove the possibility to edit the status in the EditForm, s.t. nobody can just change the status without adding a note.

I think this would be a little annoying tbh. Especially for smaller teams or folks just getting to know dojo. I think it would be best to leave that to the users

@Maffooch could you explain me https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/finding/views.py#L744 then? It seems to me that somebody had the inital thought of checking whether the finding includes any notes? If so, then the contributor mad a mistake with closing_disabled!=0, which should be closing_disabled==0  as notes will be 0 when there are no notes and therefore closing_disabled will remain 0

Looks like something to do with note types from this #1539
From just a brief skim of the PR description, it looks like if you have a specific note type configured, it requires that a note of that type be present in the finding  before it is actually closed. I like this approach

@mtesauro could you please review the PR too or is something missing?

@italvi LGTM - approving and merging


WIP [spec2x] Don't teardown network in initramfs

This breaks bonding configuration via kernel args. This is not required for master branch, as FCOS uses NM configs instead of ifcfgs.
Previously admins were encouraged to setup ip= kernel arg and add ifcfg files in Ignition. Any difference in these configs could cause networking to be misconfigured, so in #78 initramfs networking was being torn down and new settings were applied.
This is no longer necessary, as ip and other network kernel args are being persisted, so custom Ignition modifications for static networking are no longer required. Also there should be no difference in initramfs networking and first boot networking setup by ifcfgs, as these use the same ip= kernel arguments.  Any difference between kargs config and persisted config would be dracut bug imo.
TODO:

[ ] Ensure this don't break RHCOS tests
[ ] Ensure this doesn't break openshift install
[ ] Ensure static IP can still be persisted via kargs

Worth noting this reverts https://github.com/coreos/ignition-dracut/pull/78

So, binding is about reliability, but at install and/or "Ignition boot" I think it'd generally be fine to use one interface.
So another approach is to just document that users who want to use bonding should use coreos.no_persist_ip with ip= for a single underlying physical interface, and specify the bonding configuration via Ignition, so it's used for all subsequent boots.

If we're reverting something, generally we should look at the rationale for the previous change, and describe why it no longer applies.
Unfortunately, the rationale in https://github.com/coreos/ignition-dracut/pull/78 isn't too detailed.  My recollection is that e.g. we wanted to support a case where a user has specified DHCP during the Ignition phase, but static IPs defined via Ignition thereafter?

My recollection is that e.g. we wanted to support a case where a user has specified DHCP during the Ignition phase, but static IPs defined via Ignition thereafter?

Updated the description. Not sure if "DHCP on Ignition phase and static IP later on" case would work - NetworkManager falls back to ifcfgs if no other config source (e.g. DHCP) has been found IIRC. I'll test that usecase as well with this PR

Context: https://bugzilla.redhat.com/show_bug.cgi?id=1693428

Less intrusive solution - https://github.com/coreos/ignition-dracut/pull/123


Sign in error using A/D Authentication

Able to do the activities (restart, shut down, start) when add the subscription using the publish setting file, but not able to do the activities while sign in the subscription.Sign in using A/D Authentication is not working fine.
PFB the screen shot.

This appears to be a bug in the Azure SDK for Java. I have submitted a new issue to their repo here.

This issue is still reproducible in the latest build.
PFB the screen shot.

The problem was that the automated build script still referred to the Azure SDK jars from Maven Central instead of the custom built ones in the repo. The script has been updated now and the next build should have the right jars.

This issue is fixed and working as expected in the latest build.


add ClusterRoleBinding as seen in CoreOS Tectonic

@fabxc @alexsomesan

This will be documented and handled by the prometheus-operator itself. Closing here.


[Question] appWithI18n should accept JSX.Element?

@EricKwan2014 How is your MyApp defined? It is likely that the types need to be improved. Feel free to do a PR yourself by adapting the types.

@aralroca
First of all, thanks for the effort!
I am using the similar coding with https://github.com/vercel/next.js/blob/canary/examples/with-next-seo/pages/_app.js
So that it default returned the JSX.Element haha
Maybe I can change the scalar type in my project
But I am not sure is it a good idea to provide the flexibility in your lib or let user to strict the type in _app level 😕

@EricKwan2014 You can define _app.js in all the possible ways that Next.js lets you do it. Perhaps it would be interesting to update the types to support more options.

@EricKwan2014 it's already solved in 1.0.1-canary.4 prerelease 😊
Can you confirm that it works for you? Thanks 🙏

Hi,
I am having the same issue using the version 1.0.7, can you help me with it ? I am not sure if it is a bug if I am doint it wrong, I am following the typings described in next documentation https://nextjs.org/docs/basic-features/typescript#custom-app.
This is my code :
import { AppProps } from 'next/app'
import appWithI18n from 'next-translate/appWithI18n'

function MyApp({ Component, pageProps }: AppProps) {/**/}

export default appWithI18n(MyApp, {/**/})

And the typescript error is
Argument of type '({ Component, pageProps }: AppProps<{}>) => Element' is not assignable to parameter of type 'NextComponentType<PartialNextContext, {}, {}>'.
  Type '({ Component, pageProps }: AppProps<{}>) => Element' is not assignable to type 'FunctionComponent<{}> & { getInitialProps?(context: PartialNextContext): {} | Promise<{}>; }'.
    Type '({ Component, pageProps }: AppProps<{}>) => Element' is not assignable to type 'FunctionComponent<{}>'.
      Types of parameters '__0' and 'props' are incompatible.
        Type '{ children?: ReactNode; }' is not assignable to type 'AppProps<{}>'.
          Property 'pageProps' is missing in type '{ children?: ReactNode; }' but required in type 'AppInitialProps'


bank search command broken?

🐛 Bug Report
anytime i do for example =b --search=cape it says an unexpected error has occured?
can you fix this please it makes it harder to gear for masses and whatnot :c
Write a description of the bug here, give all the information you can about what it is, how it happened.

This was solved early today.


Getting ready for first release

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Update README.md

Update to use addons:create instead of :add and switched to rediscloud which offers more free space and is more consistent with Hubot main docs.

Merged the incorrect command in https://github.com/slackhq/hubot-slack/pull/237, and noted that we should update the account as well. Coming soon.


How can i wait for a asynchronus request between two calls??

I need wait for a response to use a response.value in next call that i make after first one automatically.
How i could make this?

@chuntin Looks like you're posting this issue on the wrong repo, which language are you having this problem with?

It's foro PHP.  I needed post a request wich returns a id Number and with that id i request another Api call asap i get it but the Api need 2 or 3 seconds between this calls to check first call(imagen are in)


Update readme.md to use Add-AzureRmEnvironment with latest updates

@viananth,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@vikasnav @bganapa please take a look and let me know if it needs update


Added Coursera Scraping

Description
Added a Feature of Coursera Scraping using selenium webdriver. The folder includes courses.py file which has a class based implementation of coursera webscraping. A sample test is included in tests/coursera_test.py.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[x] Does not break backward compatibility
[x] Does not produce any errors or warnings

Test Configuration:

Python Version : Python 3.8
Selenium 4.9.1
PyCharm 2022.2.2 (Community Edition)

Runtime version: 17.0.4+7-b469.53 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 1010M
Cores: 4

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] Any dependent changes have been merged and published in downstream modules

@nikhil25803 pls check this PR for Issue #188

Great work @THEGAMECHANGER416
Do not forget to

Give this repository a star ✨
Follow Clueless Community on GitHub ✅
Share your work on socials and tag us and Clueless 🫂


Whiteboard architecture

Concept up at https://miro.com/app/board/o9J_ku5CWJc=/ but won't be maintained, renaming accordingly.


bcbio-variation-recall square error

Hi!
I am running a freebayes joint variant call and have encountered an error on the bcbio-variation-recall square command. The full command was:
bcbio-variation-recall square -Xms250m -Xmx4g -XX:+UseSerialGC -c 6 -r 9:46566299-141213
431 --caller freebayes /scratch/PI/euan/projects/elite/bcbio/projects/project_n268_fb_joint/work/joint/freebayes-joint/elite
268/9/elite268-9_46566298_141213431.vcf.gz /share/PI/euan/apps/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa /scratch/PI/euan/
projects/elite/bcbio/projects/project_n268_fb_joint/work/joint/freebayes-joint/elite268/9/elite268-9_46566298_141213431.vcf-
inputs.txt
A couple bcbio-variation-recall square commands worked before this one encountered an issue, but I have no clue really what this function does and why it might be randomly failing.
The output generated in the bcbio-nextgen.log file:
Traceback (most recent call last):
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
_do_run(cmd, checks, log_stdout)
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'bcbio-variation-recall square -Xms250m -Xmx4g -XX:+UseSerialGC -c 6 -r 9:1-46565846 --caller freebayes/scratch/PI/euan/projects/elite/bcbio/projects/project_n268_fb_joint/work/joint/freebayes-joint/elite268/9/elite268-\9_0_46565846.vcf.gz /share/PI/euan/apps/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa /scratch/PI/euan/projects/elite/bcbio/projects/project_n268_fb_joint/work/joint/freebayes-joint/elite268/9/elite268-9_0_46565846.vcf-inputs.txt
AND
Traceback (most recent call last):
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/ipythontasks.py", line 50, in _setu
p_logging
yield config
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/ipythontasks.py", line 432, in square_batch_region
return ipython.zip_args(apply(joint.square_batch_region, *args))
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/joint.py", line 123, in square_batch_region
_square_batch_bcbio_variation(data, region, bam_files, vrn_files, out_file, "square")
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/variation/joint.py", line 173, in _square_batch_bcbio_variation
do.run(cmd, "%s in region: %s" % (cmd, bamprep.region_to_gatk(region)))
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
_do_run(cmd, checks, log_stdout)
File "/share/PI/euan/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'bcbio-variation-recall square -Xms250m -Xmx4g -XX:+UseSerialGC -c 6 -r 9:1-46565846 --caller freebayes /scratch/PI/euan/projects/elite/bcbio/projects/project_n268_fb_joint/work/joint/freebayes-joint/elite268/9/elite268-9_0_46565846.vcf.gz /share/PI/euan/apps/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa /scratch/PI/euan/projects/elite/bcbio/pr
ojects/project_n268_fb_joint/work/joint/freebayes-joint/elite268/9/elite268-9_0_46565846.vcf-inputs.txt
Any thoughts? :)
Thank you very much for all your help!
Luke

Luke;
Sorry about the issues and for the detailed report. Unfortunately I don't see see any informative errors in what you pasted. The messages say that the bcbio-variation-recall command, which is what runs the joint calling of the initial calls, failed but don't give a clue as to why.
Practically, the FreeBayes joint calling implementation is pretty disk intensive and can cause issues. How many samples are you running with joint calling? If it's a smaller number, than using batch calling (calling everything together) is probably a better option. If it's a larger number, is using gatk-haplotype joint calling an option? That will scale much better than our current FreeBayes implementation.
If you're stuck at FreeBayes joint calling we can diagnose more with more of the traceback that you could post at https://gist.github.com/ and link to here. Hope this helps.

I am joint calling 268 samples. I would definitely switch to gatk-haplotype joint calling instead if it scales better! Thanks for that advice! Yeah I am sorry I couldn't provide a better debug log, but that is pretty much all it said... Hopefully it will be fine after I switch to gatk.
Thanks!

Hi Brad,
Here is the gist: https://gist.github.com/lsturm1/a2a1ab80b464ee141fc552a6205e356b
I didn't put the whole bcbio-nextgen-debug.log because it gets very big quick with joint calling.
It seems as though this is the main error: vcffixup: could not open stdin
This error does not crash the whole job though.
Would love your thoughts. Thanks for the help!

Luke;
Thanks for the details, this helps a lot. I believe we're running into a bug with the current FreeBayes (https://github.com/ekg/freebayes/issues/310) here, causing FreeBayes to segfault. I pushed a workaround for this to bcbio-variation-recall, so if you do:
bcbio_nextgen.py upgrade --tools

and then re-run your analysis in place it should hopefully work cleanly. Hope this fixes it for you.


create score data table

make a table for scores that display date, map name, mods,  accuracy, combo, and pp.

done


TensorBoard Torch branding

Hi I'm the TensorBoard lead developer.
Thank you for using the tensorboardX name. Please note we recommend the X as a placeholder in custom builds. The X can be whatever you want. We have a way to customize this in the Polymer code.
Would it be possible to have the GUI display the "TensorBoard-X" or "TensorBoard/Torch" or "TensorBoard/PyTorch" in the top left of the GUI?

Looking through the codebase, is it more accurate to say that this converts the PyTorch data to a the events format TensorBoard can read? If that's the case, maybe this is using the canonical TensorBoard release, and this project is more like an adapter. In that case, apologies for the confusion.


Self query retriever with Vector Store type <class 'langchain_community.vectorstores.faiss.FAISS'> not supported.

System Info
I've been trying to create a self query retriever so that I can look at metadata field info. This issue comes up. Should I be using another vector store to make this work? I can only really work with FAISS. I cannot use ChromaDB since my Python environment is limited to a previous version.
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[X] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction

Create a FAISS Vectorstore DB
Create a metadata_field_info object and pass it to a SelfQuery object
Create LLM with this retriever

embedding_function = OpenAIEmbeddings(openai_api_key=OPENAI_API_KEY, model="text-embedding-ada-002")

    db = FAISS.load_local(input_dir + "/" + "storage/deploy/faiss-db", embedding_function)#, distance_strategy="COSINE", normalize_L2 = True)
    # retriever = KNNRetriever(vectorstore=db)

    llm = ChatOpenAI(
        temperature=0,
        verbose=False,
        openai_api_key=key,
        model_name="gpt-3.5-turbo"
    )

    metadata_field_info = [
        AttributeInfo(
            name="source",
            description="The document this chunk is from.",
            type="string",
        ),
        AttributeInfo(
            name="origin",
            description="The origin the document came from. Comes from either scraped websites like TheKinection.org, Kinecta.org or database files like Bancworks. Bancworks is the higher priority.",
            type="string",
        ),
        AttributeInfo(
            name="date_day",
            description="The day the document was uploaded.",
            type="integer",
        ),
        AttributeInfo(
            name="date_month",
            description="The month the document was uploaded.",
            type="integer",
        ),
        AttributeInfo(
            name="date_year",
            description="The year the document was uploaded.",
            type="integer",
        ),
    ]

    # retriever = db.as_retriever(search_type="similarity", search_kwargs={'k': 6}, metadata_field_info=metadata_field_info)

    retriever = SelfQueryRetriever.from_llm(
        llm, db, "Information about where documents originated from and when they were published.", metadata_field_info, verbose=True
    )

Expected behavior
Successfully create a SelfQuery retriever with FAISS vector store.

@XariZaru Unforunately FAISS  is currently not supported in SelfQueryRetriever but I am going to create a PR to integrate FAISS but for that you will have to wait.

Update: After Researching on the internet I realized FAISS does not have a internal query language like Chroma or Vectara therefore SelfQueryRetriever does not support FAISS. Remeber the flow of SelfQueryRetriever is as follow:
User_Question ---> LLM_Call --> Query_Language for Vector DB ---> Query_Executed ---> Document Returned.
Since the third part is missing, using FAISS with SelfQueryRetriever does not make any sense.

@XariZaru  I have created a PR for it's support.

I swapped to ChromaDB and Pinecone.

@keenborder786 Same Issue for Milvus
ValueError: Self query retriever with Vector Store type <class 'langchain_milvus.vectorstores.milvus.Milvus'> not supported.


Short solution needed: "chrome headless keep running" (chrome-headless)

Please help us write most modern and shortest code solution for this issue:
chrome headless keep running (technology: chrome-headless)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to keep running Chrome in headless mode


wget command makes the VM freak out

just stops everything

I am unable to reproduce this with the currently live WebVM. Please provide a screenshot of the broken state, with devtools console open, with the full command you ran visible.

I'm glad to hear that you got it to work. It's possible that what you're seeing is just a delay. I'm closing this issue.


Update AWS instructions

http://docs.pachyderm.io/en/latest/development/deploying_on_the_cloud.html#amazon-web-services-aws
We need to update our '3 line hack' to the k8s start script.
Basically just added that first line to kubernetes/cluster/common.sh:
  KUBE_MANIFESTS_TAR_URL="${SERVER_BINARY_TAR_URL/server-linux-amd64/manifests}"

If you leave the other 2 lines, you get 'unbound variable' errors and the script exits before completing (but thinks that it has worked ... and so errs silently). Specifically, it thinks that it has spun up the k8s service (and it may well have), but it didn't update the config / credentials for kubectl, so I have no way of knowing if it has, since I cannot connect.

Since Matt and I wrote and debugged this the first time, I'll try to take a crack at it today if @msteffen has some time to help

Ran through the guide and turns out we dont need to add anything to get this to work. They seem to ahve fixed the k8s start script. Updating our guide to not include those 3 lines and closing until someone runs into a problem.


Improve Themes section (#816)

Fixes: #816

In Vaadin docs, there is mentioning of Dependencies.java with @NpmPackage annotations. Clarify this.

I avoided this intentionally, since it looks like details interesting only to experts.

Rewrite the text and clarify that it is needed when you create a custom add-on

The first paragraph starts with: An add-on may contain a file with custom styles. Isn't it says that these styles are meant for add-ons?)

The first paragraph starts with: An add-on may contain a file with custom styles. Isn't it says that these styles are meant for add-ons?)

No. When people see "add-on" they will think about standard add-ons from the Marketplace.


Enable assertions in test_wamr.sh script

There already has been a -g build iwasm with debug version


Schlage BE469 support for alarm_level, access_control and alarm_type

Which device is missing?
Schlage BE469NXCEN Touchscreen DeadBolt
What are the IDs?
Manufacturer ID: 0x003b
Product type: 0x6341
Product ID: 0x5044
from a zwcfg_...xml:
Did you check if we really don't have it?
There is a  BE469.json file but the manufacturer is Allegion?  There is a similiar file under Schlage but the manufacturers' id is 0x0003?

[ ] It is not in the configuration DB
[ ] It was not merged recently or has a pending PR
[ ] It is not in the TODO list

Do you have a link to the PDF manual?
Manual here.
If so, it helps in creating the config file.
I tried integration with Home Assistant.   I can control lock and see battery level.  I am missing sensors seen in Open Zwave 1.4 for alarm_level, access_control and alarm_type.  Not sure where these sensors came from?
I got this in the Zwave-JS log which does not look good:
```16:31:34.698 CNTRLR [Node 004] ConfigurationCC: skipping interview because CC version is < 3 and there is no config file``

There is a BE469.json file but the manufacturer is Allegion? There is a similiar file under Schlage but the manufacturers' id is 0x0003?

@blhoward2 any idea?

I am missing sensors seen in Open Zwave 1.4 for alarm_level, access_control and alarm_type. Not sure where these sensors came from?

alarmLevel and alarmType are to be found under Notification CC. I don't know where they appear in HomeAssistant.
For the future, please use the correct issue template. The device is identified, so it is not a missing file:

This ensures that you are asked for the relevant information (especially a log) so we can narrow down the problem.

Heh, I just saw @blhoward2 asked you to open this. Anyways, this

From Google search it seems that Allegion now owns the Schlage electronic lock product line.  Hopefully, only the Manufacturers ID is important and not the assigned name.  I thought that the configuration was missing because of this log file message:
16:31:34.698 CNTRLR [Node 004] ConfigurationCC: skipping interview because CC version is < 3 and there is no config file

We only match on the manufacturer ID.

Tried again with latest versions and got this in the log file:
16:05:19.153 CNTRLR   [Node 004] ConfigurationCC: skipping interview because CC version is < 3 and t
                      here is no config file
16:05:19.161 CNTRLR   [Node 004] NotificationCC: doing a complete interview...
16:05:19.163 CNTRLR   [Node 004] Interview stage completed: CommandClasses
16:05:19.165 CNTRLR   [Node 004] Interview stage completed: OverwriteConfig
16:05:19.166 CNTRLR » [Node 004] requesting node neighbors...
16:05:19.198 CNTRLR « [Node 004]   node neighbors received: 1, 7, 8, 13
16:05:19.199 CNTRLR   [Node 004] Interview stage completed: Neighbors
16:05:19.200 CNTRLR   [Node 004] Interview completed
16:05:19.202 CNTRLR   [Node 004] The node is ready to be used

Now seems to complete the interview despite saying it would skip it?
I now get sensor.none_alarmlevel and sensor.none_alarmtype in Home Assistant although they are both unknown.
Probably, there is no problem with the config file so this issue can be closed.

ConfigurationCC: skipping interview because CC version is < 3 and there is no config file

Means that the ConfigurationCC interview is skipped, not the entire interview. Prior to V3 we'd have to query 255 possible parameters just to check if they exist and risk waiting for a timeout for 10 seconds each. Hence the necessity for configuration files.

I got the BE469 working with zwave_js  in Home Assistant.  I used zwave_js_event to trigger an automation that set an input_text helper to display the reason for the locks' last action.


iiif sandbox rfc v1

See https://github.com/stephenwf/iiif-preview for Cloudflare Worker + KV implementation
From Slack:
https://iiif-preview.stephen.wf/p3/1e4670aedf229497b12a7d29dfe385ec/manifest.json
https://uv-v4.netlify.app/#?manifest=https://iiif-preview.stephen.wf/p3/1e4670aedf229497b12a7d29dfe385ec/manifest.json
API matches mostly what’s in the RFC:
POST https://iiif-preview.stephen.wf/store
{ ... manifest }

Returns
{
  "location": "https://iiif-preview.stephen.wf/p3/8142e470b7eb6edb1e5434e8b3eb50ca",
  "updateLocation": "https://iiif-preview.stephen.wf/update/8142e470b7eb6edb1e5434e8b3eb50ca/ee02f8e59abbb58d8b9931c399547dd414f1fecdfac2752c2a257031e86e5013",
  "deleteLocation": "https://iiif-preview.stephen.wf/delete/8142e470b7eb6edb1e5434e8b3eb50ca/de2cb145ed0930cc6a391b91ac02de11b19138abd88b58eebb793cd166f9fa0e",
  "expirationTtl": 172800
}

Location - This is the URL to your manifest
Update location - PUT to this with new Body, you get back the same as above with a new Update location (this rotates, acts like an E-Tag)
Delete location - This will be stable, unlike the update location which could change. You can DELETE to either the latest update endpoint or this delete location
Expiration TTL - This is a TTL (currently 48-hours)

@stephenwf Looking at the above source code suggests an enhancement that the ME could use.
instead of /store there are two locations you can POST to

/store
/preview

The latter of these has the current behaviour, with an expirationTtl (could be less than 48hrs even)
The /store path does not set an expiration; it will be there as long as the service is up.
The ME can interact with the service in two places -

on the preview menu, where it POSTs to /preview, these manifests expire
on the save menu, where it POSTs to /store. The option will be called something like "permalink" or "share" TBC.

These options are always available in all builds of the ME, with other integrations following on later.
In some scenarios an adopter (deployer) could disable them if they have alternative preview mechanisms or don't want their manifests stored in the service at all.

Also looking at that - the KV impl is nice, I agree! But also notice that if for whatever reason the KV backing approach proves too costly (I don't think it will) you could still implement that with a worker/lambda + R2/S3 + 2 secrets. You wouldn't get the expiry so easily though. So if the biggest use of this service is previewing while editing, the storage won't just grow and grow because old previews will get kicked out.
Then the challenge is in the UI - make sure that "preview" is obviously ephemeral; if you want a permanent URL, use "Save".

@tomcrane I think you can set expiry at the S3 bucket level, so a store/preview could be 2 buckets.


If statements inside an element attribute

Data elements render inside an attribute of an element in a template, but the conditional if statement does not seem to work (see title attribute of div element below):

When it renders I see:

The injected annotations seem to be the problem.  Any way to stop those?
Thanks,
Joseph Conder

Using {{if}} or {{each}} within attribute markup does not currently work by default, if JsViews is loaded, because JsViews sets a flag for JsRender: $.views.activeViews = true, which tells JsRender to insert comment-based annotations for data binding. You can temporarily set $.views.activeViews = false before rendering, and then switch back after rendering, and things should work fine.
Otherwise you can do:
{{#if preferred_first != first}}

{{else}}

{{/if}}

Or again, you can use data binding, along the lines of:


Purge redis vizjson and embeds #4651

Somebody from @CartoDB/backend please review

Frontend tests were OK :+1: (details)

:+1:


Support AG listener wildcard hostnames (currently in preview)

Application Gateway support for listener hostname wildcard configurations is currently in preview.
(reference: https://docs.microsoft.com/en-us/azure/application-gateway/multiple-site-overview#wildcard-host-names-in-listener-preview)
Supporting this feature though ingress configuration would be extremely helpful when hosting different multi-tenant sites.
Currently, when trying to apply a host rule including a wildcard configuration, for example:
spec:
  rules:
  - host: '*foo.bar.com'
    http:
      paths:
      - path: /
          serviceName: backend
          servicePort: 80

Results in an error (appears to be core Kubernetes Ingress validation so the solution may have to be done with annotations somehow or convince the K8 team there are certain areas of the spec that should leave validation up to the implementing system?):
The Ingress "foobar-ingress" is invalid: spec.rules[0].host: Invalid value: "foo.bar.com": a wildcard DNS-1123 subdomain must start with '.', followed by a valid DNS subdomain, which must consist of lower case alphanumeric characters, '-' or '.' and end with an alphanumeric character (e.g. '.example.com', regex used for validation is '*.a-z0-9?(.a-z0-9?)')
Thank you in advance for your consideration of this feature as well as your time, and effort supporting this project.

Use appgw.ingress.kubernetes.io/hostname-extension annotation to provide a wildcard hostname. This is supported on AGIC but not yet documented.


Add typedoc docs

This PR adds typedoc support for generating API documentation.
Example:

You can build the html docs from this branch as:
    yarn install
    yarn run docs

@rochdev all the comments are addressed, PTAL once you get time. Thanks


[All mobile devices] On Leaderboard page Popup for Contributors info should be aligned vertically center

Can I take this?

@VishakhaAwasthi01 We have to send this through design first. They may want to keep it this way.

@buckyroberts Based on my understanding from the design itself, the modal wanted to be like a mobile 'native'-esque modal which it will come up from the bottom. Is

@buckyroberts Based on my understanding from the design, the modal should come out from the bottom of the screen like a mobile native-esque experience. Is that right @kristykjlee ?

@buckyroberts @kristykjlee @thesanjeevsharma

will this work? If yes please assign it to me

I like it the way it is currently. It gives a more native-app-like experience. Also good for one-hand use I guess. What do you say @mofodox ?

I like it the way it is currently. Its design is also similar to the current design of Navigation Link Modal when the user clicks the Hamburger button in the right top corner plus gives a more native-app-like experience. Also good for one-hand use I guess. What do you say @mofodox ?

Yes I guess that's how the design is designed for. It makes the design more "native-esque".

Yeah please close this issue then.


Upd (UI/UX)  : Improvement of Website

As the website needs a single scrolling page
There is a need to improve the current website with the following features  :

The website lacks information therefore is a need to add infographics and content to the website.
Addition of features, for example, what the product does, how does it help the users, etc.
Website also lacks responsiveness.
Addition of footers, creation of query forms, etc is appreciated.

Reset Password Page: https://iwasat.events/reset.html UI has to be improved.

The page also lacks sufficient text.
For inspiration, we can look into these designs

Reset Password Page: https://iwasat.events/reset.html UI has to be improved.

The page also lacks sufficient text.
For inspiration, we can look into these designs

Point to be noted:
Kindly make Pull Request in DEVELOP Branch

Point to be noted:
Kindly make Pull Request in DEVELOP Branch


Bump Dependencies #6744982019

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/6744982019

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


The player falls through the moving platform when the platform goes up

I'm submitting a...

[X] Bug report.
[ ] Feature request.

Bug report
What is the current behavior?
When the player is on  the moving platform and the platform goes up, the player fall through
What is the expected behavior?
The player should stay on the platform as this moves upwards or downwards
Tell us the steps to reproduce the bug, and if possible share a minimal demo of the problem.

Jump in the moving platform when its reachable, wait till it starts going upwards, the player should fall through
The other non-moving oneway platforms dont suffer from this
This used to happen when Godot 3.0 was released but got fixed... maybe some bug crawled back again.
This was on Windows 10 64bits and Godot 3.2 stable

Thanks, I'll check it now

It seems increasing the safe margin did the trick. I couldn't reproduce the bug with that margin. But please reopen if it still happens after that fix.


Local compression in CI should be done in parallel

I noticed that creating the archive of benchmark suites is currently taking almost 4 minutes: https://github.com/iree-org/iree/actions/runs/3577745816/jobs/6017397077. It's doing gzip compression locally which is done in a single thread. These should be doing something that's parallel. I did extensive experiments when optimizing the full build directory archive upload time and concluded that it was best to use zstd. You can see an example usage at https://github.com/iree-org/iree/blob/b79ca72c9b61ab87cf9a9405891c68cd3fe049bc/.github/workflows/ci.yml#L143
See discussion in https://github.com/iree-org/iree/issues/9881

Nice. Looks like the tar -czf I was originally referring to was removed in https://github.com/iree-org/iree/pull/11380 as well


tests: fix key manager flaky tests

Signed-off-by: Chester Liu<EMAIL_ADDRESS>
What problem does this PR solve?
Problem Summary:
Some of the key_manager tests are flaky on my system:
thread 'manager::tests::test_key_manager_rotate_master_key' panicked at 'assertion failed: `(left == right)`
  left: `2`,
 right: `1`', components/encryption/src/manager/mod.rs:889:9

...

failures:
    manager::tests::test_key_manager_rotate_master_key
    manager::tests::test_key_manager_rotate_master_key_rewrite_failure
    manager::tests::test_master_key_failure_and_succeed

What is changed and how it works?
What's Changed:
I suspect the failure is caused by locking error so I added related locking code.
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note

Thanks for contributing.
The LOCK_FOR_GAUGE is required not only when tests read gauge value, but also when tests could potentially mutate the gauge. @hunterlxt Mind help triage if any other tests need fixing?

Thanks for contributing.
The LOCK_FOR_GAUGE is required not only when tests read gauge value, but also when tests could potentially mutate the gauge. @hunterlxt Mind help triage if any other tests need fixing?

It would be better to add the LOCK_FOR_GAUGE into all unit tests because Dict::open() could modified the var and the behavior  happens in almost all tests. @skyline75489

add the LOCK_FOR_GAUGE into all unit tests because Dict::open() could modified the var and the behavior  happens in almost all tests

This is also my initial thought. Thanks for confirming this. I've added the locking code.
By the way do I need to manually squash the commits into one?
Have a nice weekend 😄

add the LOCK_FOR_GAUGE into all unit tests because Dict::open() could modified the var and the behavior  happens in almost all tests

This is also my initial thought. Thanks for confirming this. I've added the locking code.
By the way do I need to manually squash the commits into one?
Have a nice weekend 😄

No need, the bot will squash them for you.

/merge


Logs enable/disable

Hello,
Would it be possible to add a switch for enabling/disabling logging? Currently, both cameras generate two log entries per second as per the following:
[11/4/2023, 10:54:07 AM] [homebridge-tapo-camera] [IPC01] Motion detected false
[11/4/2023, 10:54:11 AM] [homebridge-tapo-camera] [IPC01] Motion detected true
[11/4/2023, 10:54:12 AM] [homebridge-tapo-camera] [IPC01] Motion detected false
[11/4/2023, 10:54:16 AM] [homebridge-tapo-camera] [IPC01] Motion detected true
[11/4/2023, 10:54:17 AM] [homebridge-tapo-camera] [IPC01] Motion detected false
[11/4/2023, 10:54:38 AM] [homebridge-tapo-camera] [IPC01] Motion detected true
[11/4/2023, 10:54:40 AM] [homebridge-tapo-camera] [IPC01] Motion detected false
[11/4/2023, 10:55:06 AM] [homebridge-tapo-camera] [IPC02] Motion detected true
[11/4/2023, 10:55:07 AM] [homebridge-tapo-camera] [IPC01] Motion detected true
[11/4/2023, 10:55:07 AM] [homebridge-tapo-camera] [IPC02] Motion detected false
I believe that the velocity and volume of data is impacting the SD Card's life (in my case at least where Homebridge is installed on an RPi 4.
In advance, thank you very much for your time and support.
Best Regards,
Panos

Very good report, I'll reduce the verbosity of these


Solaredge new sensors

Description:
Adding additional monitored_conditions to the SolarEdge platform.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8213
Example entry for configuration.yaml (if applicable):
sensor:
 - platform: solaredge
    api_key: YOUR_API_KEY
    site_id: YOUR_SITE_ID
    monitored_conditions:
      - current_power
      - last_day_data
      - last_month_data
      - last_year_data
      - life_time_data
      # new conditions
      - site_details
      - inventory_inverters
      - inventory_meters
      - inventory_sensors
      - inventory_gateways
      - inventory_batteries

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

Alright, that didn't go as planned. Will close this one and open a new pull request.


Revert "Check split_out to decide on sorted groupby (#802)"

With https://github.com/rapidsai/cudf/pull/11755 merged in and reflected in our gpuCI images, we should no longer need #802 in for gpuCI to pass.

Codecov Report

Merging #807 (ebf793f) into main (06fb48f) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #807      +/-   ##
==========================================
- Coverage   74.88%   74.77%   -0.12%
==========================================
  Files          71       71
  Lines        3588     3587       -1
  Branches      748      748
==========================================
- Hits         2687     2682       -5
+ Misses        771      770       -1
- Partials      130      135       +5

Impacted Files
Coverage Δ

dask_sql/physical/rel/logical/aggregate.py
89.94% <100.00%> (-0.06%)
:arrow_down:

dask_sql/physical/rel/logical/window.py
95.37% <0.00%> (-1.74%)
:arrow_down:

dask_sql/_version.py
32.27% <0.00%> (-0.29%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

rerun tests


url in hyperlink

Could we please have an example doc. with a link that doesn't get found w/o this?


Add Rows in DataGridView Class returns an error

I am attempting to add rows to a datagrid and I receive the error below. It appears a different methodology for adding rows is needed since the data binding is done when the grid is initialized with the Easy Grid Setup VI. FYI, I am attempting to add rows where one column is combo boxes, but each combo box has different parameters to select from.
Error 1172 occurred at Invoke Node Error calling method System.Windows.Forms.DataGridViewRowCollection.Add, (System.InvalidOperationException: Rows cannot be programmatically added to the DataGridView's rows collection when the control is data-bound.)
Possible reason(s):
LabVIEW: (Hex 0x494) A .NET exception occurred in an external assembly. For information about correcting this error, copy the following exception (in bold), and search the Microsoft Developer Network (MSDN) Web site or the Web for a possible explanation.
System.InvalidOperationException in DataGrid.lvlib:DataGridView.lvclass:Add Rows.vi->Untitled 1.vi

If you are using the databound table, then use the method for the table, addrows, not the add rows to the datagrid, as you need to add to the data source itself once it is bound.

Unfortunately, this option does not allow me to define the parameters for a combo box. I'm looking to add a row, but each row may have different options in a combo box that a user could then select from. I will try to create a datagrid without using the Easy Grid Setup VI which does the databinding during the setup.


Merge eBPF program information types with the types in ebpf-verifer

Merge eBPF program information types with the types in ebpf-verifer.
Currently ebpf-verifier uses C++ types that the MIDL marshaller can't handle. If we get the ebpf-verifier folks to switch to using C style types, switch to using them.

Requires changes in the PREVAIL verifier project.

Need to file an issue in the vbpf/ebpf-verifier project

ebpf_program_types.h duplicates types in PREVAIL's helpers.hpp header for example


Revert "Return content-length in 'range' requests"

Reverts kennethreitz/httpbin#314
@kennethreitz @sigmavirus24 @Lukasa If any of you have a moment, could I get confirmation I'm not completely off base here. #259 made the decision not to include data length related headers in httpbin and allows the server to specify. Requiring a Content-Length header for the streamed response seems fundamentally incorrect and has broken portions of the /range endpoint.

This escalated from #344 - data was not being streamed from /range, but returned all in one go at the end. Looking from API standpoint response headers could contain Content-Length and still return data incrementally - there should be no standard violation here. Is server implementation preventing that somehow?

I defer to @nateprewitt and @Lukasa here.

@Lukasa I definitely agree that Content-Length isn't forbidden. In fact, it's even used in the RFC spec examples for Range. What I'm trying to rectify here is the fact httpbin is now unconditionally forcing a Content-Length header. On top of that, the value isn't correct for any request using a range header other than Range: 0- because the content length is always the full length passed to /range/<num bytes>.
It seems the requirements that prompted this PR in the first place may have not been fully fleshed out, making this change unnecessary. Since it doesn't seem to be providing significant value and is actively affecting the endpoint integrity, I'm suggesting we revert it.
@lietusme My local tests yesterday seemed to suggest the Content-Length was affecting the streaming. I'm getting mixed results locally now when watching things with tcpdump. It's very possible there's an issue in the new deployment on Heroku. Either with gunicorn or heroku?

I have no objection to removing it, but the other option would surely be to fix it so it returns the correct length, yes?

@Lukasa, if we want to adopt the forced use, yeah. I figured this was the least intrusive to how things worked prior to #314. If there's a good reason to adopt Content-Length on every range request then I agree we should just fix it.

I'm kind of inclined to say that we should. Not putting it there is a bit gross, really.

I aggree that Content-Lenght makes sense in /range responses:

Length is known by service - thats by standard to return if known
Range API purpose is ranged download, not streaming service

That said, /range should still return chunked response incrementally.
@nateprewitt - no idea if its server/deployment issue. I currently use https://httpbin.org, planning on deploying locally in future.

That said, /range should still return chunked response incrementally.

Hang on, we need to clarify something.
If range is returning chunked, it must not set Content-Length. RFC 7230 forbids it. By exclusion, then, if it's setting Content-Length, it must not return chunked. You can have either: not both.

Sorry, I did not mean Chunked Transfer Encoding, but the fact that /range could still trickle transfer incrementally for given duration in given chunk sizes - simulating slow connection. This does not mean that content should be undefined length, but just that its being transferred continuously. This is my primary use of /range API - simulating slow connection. Maybe my workflow needs different API, but usecase seemed appropriate for what /range does.

Yup, so that can definitely work.
The general rule for WSGI is that when an application returns a Content-Length header the WSGI server will not chunk the data: it'll just enforce the content length. When the WSGI app does not return Content-Length, the server may choose to either set Connection: close or Transfer-Encoding: chunked, depending on its own preferences.
In this case, we know the Content-Length, so we should just set it and be done.

Ok, I'm on board. I've added #346 to supersede this.


Error displays on "Property" page

VS Version: 16.1.0 Preview 4.0 [28904.22.d16.1stg]
Python Tool Version: 16.1.19121.1
Repro step:

Create a python application
Click on the project, click "Property Pages" button on "Properties" panel

Actual Result:
Error displays on "Property" page

Expect Result:
No error display.

I was able to reproduce. I'll file an internal bug.
Error text:
Could not find any resources appropriate for the specified culture or the neutral culture.  Make sure "Microsoft.VisualStudio.Editors.PropPageDesigner.PropPageDesignerView.resources" was correctly embedded or linked into assembly "Microsoft.VisualStudio.AppDesigner" at compile time, or that all the satellite assemblies required are loadable and fully signed.

Internal bug 888081

This was fixed yesterday in 16.1 dogfood builds.