Build is broken

Windows tests appear broken
https://travis-ci.com/quantumlib/Cirq/jobs/231923888
it looks like this broke it
https://github.com/quantumlib/Cirq/pull/2063

I wonder how the build passed with failing tests.

Error log:
$ powershell -command 'Set-MpPreference -DisableArchiveScanning $true'
before_install.2
0.97s$ powershell -command 'Set-MpPreference -DisableBehaviorMonitoring $true'
before_install.3
1.02s$ powershell -command 'Set-MpPreference -DisableRealtimeMonitoring $true'
before_install.4
40.39s$ choco install python --version 3.7.3
before_install.5
0.03s$ export PATH="/c/Python37:/c/Python37/Scripts:$PATH"
before_install.6
6.05s$ python -m pip install --upgrade pip wheel
install.1
86.62s$ python -m pip install -r requirements.txt
install.2
62.72s$ python -m pip install -r cirq/contrib/contrib-requirements.txt
install.3
53.92s$ python -m pip install -r dev_tools/conf/pip-list-dev-tools.txt
156.63s$ check/pytest --benchmark-skip --actually-quiet
........................................................................ [  0%]
........................................................................ [  1%]
........................................................................ [  2%]
.
.
.
........................................................................ [ 97%]
........................................................................ [ 98%]
.......................................................
The command "check/pytest --benchmark-skip --actually-quiet" exited with 3.

Notice that it stops before 99%.

@dstrain115 has also seen this failure, but on one of his PRs. Based on his description, it seems to randomly come and go. I suspect debugging it will require quite a lot of bisecting and repeating.

--actually-quiet is that burying the actual error messages?

I got a job to fail with the verbose flag: https://travis-ci.com/quantumlib/Cirq/jobs/231970083
It fails right after cirq/work/pauli_sum_collector_test.py::test_pauli_string_sample_collector_extra_qubit_x The command "check/pytest --benchmark-skip -v" exited with 3.

So helpful: "Exit code 3: | Internal error happened while executing tests"

The test after that is cirq/work/sampler_test.py::test_sampler_fail PASSED.

Async code.  Hahah yeah that wouldn't surprise me.

Now it crashed on test_pauli_string_sample_collector_extra_qubit_x.  https://travis-ci.com/quantumlib/Cirq/builds/126289206
cirq/work/pauli_sum_collector_test.py::test_pauli_string_sample_collector_identity PASSED [ 98%]
cirq/work/pauli_sum_collector_test.py::test_pauli_string_sample_collector_extra_qubit_z PASSED [ 98%]
The command "check/pytest --benchmark-skip -v" exited with 3.

Should we disable this test for Windows?

This could be pointing out an actual issue on Windows.  I wouldn't disable unless I know that this is caused by with async code on Windows only in pytest.

Does anyone have a windows device to easily debug this and not have to use travis to make it fail?

I can take a look

Stale


Make github issue template, separate out feature request & bug report

cref: https://github.com/cosmos/cosmos-sdk/tree/master/.github/ISSUE_TEMPLATE

I'm closing this since this is a copy of: https://github.com/osmosis-labs/osmosis/issues/1339


Update f/accelerator-admin-support with main

updating feature branch
Description

this is fixes tests and changes from the pf5 upgrade
How Has This Been Tested?

make sure tests pass and flows still work for accelerator admin page and image admin page
Test Impact

changed selectors
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

Seems the TolerationsModal test failed. @Gkrumbach07 when you get back Monday, please look into it.


Add a check before removing a child node when removing a preview

resolve one of the two parts of #77845

Closing as superseded by #77867.


Packaging track

On the packaging track:

createServer
Check if readme is ok
Move protocol.json to minecraft data
Remove unneeded dependencies from package.json (ie bedrock provider)
Check whatever else doesn't belong here and either put in a subfolder in example, either put in another prismarine repo
Put on npm
Use it in mineflayer

I will keep updating this as it get better
Final objective : PrismarineJs packages have support for bedrock as good as mcpc (without duplicating packages)

server and readme updated
Need to investigate moving protocol further, currently the js is pre-compiled instead of generated + evaled at runtime for easier debugging.  So moving the protocol things + generator tool would mean it’d have to interop with the protocol lib somehow
Moving chunk data to pchunk needs native support, so need to remove the mapping logic and add blocks and items data to mc-data. Interim solution could be to require bedrock-provider there for now
for Mineflayer I think the easiest approach is to have separate set of plugins but expose the same api. Only complex thing here is the inventories, there needs to be a higher level API than the current clicks (eg to move, swap, pickup, place, etc instead of clicking windows).

I'm going to close this (as initial objective is 90% done) in favor of #116


Ee 26876 no stop flag

There is no point in passing in a flag to this script just to tell it not to do anything.  Simply do not call the job.

Approval ignored by Github.  Naughty Github.


[#6577] Can you add a log line for this exception?

Fixes #6577
Description
This PR adds a log in the ProcessAsync method of the CloudAdapter class to show more information about the UnauthorizedAccessException exceptions.
Additionally, it updates the CloudAdapter unit tests.
Specific Changes

Added log to ProcessAsync method in CloudAdapter.cs.
Updated ExpiredTokenShouldThrowUnauthorizedAccessException unit test in CloudAdapterTests.cs.

Testing
The following image shows the error message with the new log.

The updated test passing.

Pull Request Test Coverage Report for Build 340162

0 of 0   changed or added relevant lines in 0 files are covered.
24 unchanged lines in 4 files lost coverage.
Overall coverage increased (+5.8%) to 79.114%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs
1
90.91%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/ServiceCollectionExtensions.cs
1
93.55%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/CloudAdapter.cs
21
76.87%

Totals

Change from base Build 337253:
5.8%

Covered Lines:
25841

Relevant Lines:
32663

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll


Authorization framework with Action Policy

🔗 Issue
This is the beginning of #450 , but it does not close it yet. My goal is to apply the framework to every controller action by the end of it. I will continue to work on the rest of the controllers in the meantime.
✍ Description
This PR is the beginning of the authorization framework with Action Policy. I have not applied the framework to the whole app, but I have applied it to 5 controllers which require both shared and unique configuration.
I wanted to show this PR so that reviewers can get eyes on the progress and confirm it's going in the right direction. If any patterns seem like they could be improved or should be abandoned, please let me know.
This PR will most likely be an intro to Action Policy to anyone who reviews this. I think you will find most of Action Policy intuitive and their docs are awesome. I have a quick summary of some of its features below if it will help you but please feel free to skip it unless you are confused.
Summary of Action Policy in pet-rescue
Anything I do not cover can most likely be found in the Action Policy docs.
Take a quick look at the current Organizations::PetsController on main. Now check out the new one in the PR. Hopefully, you see the improvement!
All of the authorization logic has been removed from the controller and instead exists in the Organizations::PetPolicy. This Policy is used via authorize! calls.
For actions involving existing records such as update, destroy, and show. We can call authorize on the record that is fetched in the set_ method. This will pass the record as a default context to the policy:
  def set_pet
    @pet = Pet.find(params[:id])

    authorize! @pet
  end

Action Policy finds the correct policy implicitly using the name of the model object passed to authorize!. If the controller is namespaced, it will look for the policy in the same namespace too. The lookup behavior is further explained in the docs.
However, some actions don't use an existing record, such as new. In this case, you can pass the class name to help Action Policy find the correct policy. In cases where there is no record, you may require more context than the defaults. The defaults are the current user and the record passed. In new here, I pass the Current.organization as an organization context so that in the Policy, we can make sure the current user's organization matches the current one.
Action Policy also support scoping. This is very useful for index actions. In this index, I call the scope using authorized_scope which returns relation.where(organization: user.organization) (relation being the @q.result ActiveRecord::Relation it was passed).
In Organizations::PetPolicy, you will see that you can alias rules so that rules that share common logic don't have to be rewritten.
If you are not familiar with Pundit, Action Policy looks up the rule for the action by looking in the policy for a rule that matches the method name plus ?. Action Policy also can have a default rule specified and out-of-the-box comes with manage? as a default rule, except toward index?, new?, and create?. So, in Organizations::PetPolicy, we just need to define what manage? checks which in this case is a call to permission? with :manage_pets. permission? is not a part of Action Policy. I will explain that below.
For logic that is shared between many rules, you can define pre-checks. In this case, I have a pre-check for both verify_organization! and verify_active_staff!. These are defined in ApplicationPolicy since they are shared.
For verify_organization!, I made some adjustments so that we can use manage? for both actions that have an existing record and those that don't. Recall that update uses authorize! @pet but new does not have an existing @pet to pass, so it is given the organization context directly. I have:
  def organization
    @organization || record.organization
  end

  def verify_organization!
    deny! unless user.organization_id == organization.id
  end

The organization getter will get the @organization context if it exists. If it does not, it assumes a record was passed and gets the organization from that. That way, organization is available for verify_organization! with the same logic for both cases.
I have tested authorization using the recommendations from Action Policy. palkan recommends that you unit test each Policy and then you simply need to test that the controller actions are using the proper authorization. No need to make multiple request tests for different user contexts. I really like this method and it works cleanly and is fast.
One note on tests though, is that I currently create the context objects in the policies unit tests. I want to refactor these into using build_stubbed instead but the one thing that I think may presently hinder that is rolify. However, I think fixing that will be easy.
permission? and authorizable.rb
In our policies, we could have rules like this instead:
def manage?
  user.has_role?(:staff) || user.has_role?(:admin)
end

We could DRY it for other tests like this:
def manage?
  staff? || admin?
end

private

def staff?
  user.has_role?(:staff)
end

def admin?
  user.has_role?(:admin)
end

What I do not like about that is that it only becomes clear what each role is permitted to do in the policy. These permissions can exist either in our heads or in the code. The Authorizable concern is an attempt to get this in the code. It also has central definitions for each role, so you can look in one space to find what is permitted for each role.
I am not completely happy with Authorizable as I have it written. For one, it is not open/closed. To make it so, I think we could easily create a permissions table and model, that can be associated with users. It would be useful for customizing individual permissions, if that is something clients may want. We could use roles simply for setting default permissions to users in this scenario.

Awesome @mononoken I will review this week! Looking forward to seeing what you have done.

@mononoken awesome work so far. Thank you for taking this on. I think Action Policy is similar enough to Pundit, and easy enough to follow, that it gets my vote for use here. To your points regarding permission? and authorizable.rb, I agree that it's better to explicitly name each policy based on what it is actually doing, and map those to roles like you have done. I am not opposed to how you have handled it in authorizable.rb. It very easy to modify, and also easy to read and understand what each role has authorization to do. Having it in a table would detract from this...but I am open to discussion on that!

@mononoken awesome work so far. Thank you for taking this on. I think Action Policy is similar enough to Pundit, and easy enough to follow, that it gets my vote for use here. To your points regarding permission? and authorizable.rb, I agree that it's better to explicitly name each policy based on what it is actually doing, and map those to roles like you have done. I am not opposed to how you have handled it in authorizable.rb. It very easy to modify, and also easy to read and understand what each role has authorization to do. Having it in a table would detract from this...but I am open to discussion on that!

Thank you for your time reviewing, @kasugaijin !

Having it in a table would detract from this...but I am open to discussion on that!

I think I agree I am not sure I am the biggest fan of the table idea. In thinking and researching other options that would make the permissions code more open to extension, I found this comment by palkan (author of Action Policy). He makes Permissions an attribute of users instead and stores them in JSONB field (or array). I think that's a better idea.
That said, I think authorizable.rb as it is currently is working fine and could be left as is until the need for permissions to become more extensible arises. Like, if client admins wanted the option to customize their individual staffs' permissions.

@mononoken

That said, I think authorizable.rb as it is currently is working fine and could be left as is until the need for permissions to become more extensible arises. Like, if client admins wanted the option to customize their individual staffs' permissions.

This is my sentinment, also. For MVP, what you have is a practical enough solution that is easy to understand, tells the story of what roles have what policies, and easy enough to update. If there comes such a time where we need more fine control of policies at the user level, we can explore (I think a JSON attribute would work fine).

Ah, just realized. @kasugaijin, I am not sure if you wanted to push the reviewed content that I had on this PR or wait until I have Action Policy implemented in the rest of the controllers. I just got back from a trip and started working again in the branch, but if you wanted to do the former I could fix any potential merge conflicts and start another branch. Let me know whatever you prefer.

Ah, just realized. @kasugaijin, I am not sure if you wanted to push the reviewed content that I had on this PR or wait until I have Action Policy implemented in the rest of the controllers. I just got back from a trip and started working again in the branch, but if you wanted to do the former I could fix any potential merge conflicts and start another branch. Let me know whatever you prefer.

Sorry...somehow I missed this question @mononoken. I think we can wait as it looks like you are making good progress, and would make sense to review all at once :)

Wow amazing @mononoken this is a lot of really good work. Thank you.
I do not have any issues with the 'how'. It all makes sense and will make managing user auth way more rigorous. There are a couple comments on the 'what' i.e., specific user authorizations, but that is easy stuff to fix. I am happy with the implementation. @nsiwnf over to you if you have any additional comments!

Wow amazing @mononoken this is a lot of really good work. Thank you. I do not have any issues with the 'how'. It all makes sense and will make managing user auth way more rigorous. There are a couple comments on the 'what' i.e., specific user authorizations, but that is easy stuff to fix. I am happy with the implementation. @nsiwnf over to you if you have any additional comments!

Thanks a bunch for taking the time to review @kasugaijin ! I know this was a lot to look over. Once I have an answer to this comment, I will get your comments fixed.

@kasugaijin I made the changes mentioned in your comments.

@kasugaijin I made the changes mentioned in your comments. Let me know if I missed any.

Looks good. Thank you!

This is looking great @mononoken!

Uff da! Going on over a month, but I think I am done implementing Action Policy to every controller!
However, there are two tests that were just added a few hours ago that are failing. I have comments on those here.
For @kasugaijin and @nsiwnf , I think not too much has changed since your last reviews as most of the remaining stuff was able to reuse logic and patterns. I think the biggest differences since the last review are:

I have verify_authorized in ApplicationController which means if any controller action is not authorized and called, it will raise an error. Actions that do not require authorization have added skip_verify_authorized to make it explicit that they are skipping authorization.
I added a rescue to ApplicationPolicy that will redirect users when performing an unauthorized action. I want to further customize this behavior to give more actionable feedback to users, but I think this is fine for now.

In regards to 2, ActionPolicy seems to have good support for streamlining providing failure reasons with i18n support. I may work on implementing that next in a separate issue.

lgtm 🎉 - sorry for all the one off comments btw

No problem at all! I appreciate your thoroughness a ton! I felt a little guilty making a big PR knowing y'all would have to review it.


Date format

Would it be possible to let the user format the data?
I prefer YYYY-mm-dd for instance

Closing due to age


where is the short term TDM part?

Hi, thank your for sharing codes.
I am reading your code and I only find the long term TDM as mentioned in the paper in base_module.py and the short term TDM is not found, so where is it?
Thank you very much.

Thanks for your comments, we implement S-TDM in ops/tdn_net.py  as mentioned in the README.
https://github.com/MCG-NJU/TDN/blob/170f380317d77f6f8a418153df43a02b58f88120/ops/tdn_net.py#L14

Thanks for your comments, we implement S-TDM in ops/tdn_net.py  as mentioned in the README.
https://github.com/MCG-NJU/TDN/blob/170f380317d77f6f8a418153df43a02b58f88120/ops/tdn_net.py#L14

Thanks for your comments, we implement S-TDM in ops/tdn_net.py as mentioned in the README.
https://github.com/MCG-NJU/TDN/blob/170f380317d77f6f8a418153df43a02b58f88120/ops/tdn_net.py#L14

ok, thank you

Thanks for your comments, we implement S-TDM in ops/tdn_net.py as mentioned in the README.
https://github.com/MCG-NJU/TDN/blob/170f380317d77f6f8a418153df43a02b58f88120/ops/tdn_net.py#L14

ok, thank you

Hi, thank your for sharing codes.
I am reading your code and I only find the long term TDM as mentioned in the paper in base_module.py and the short term TDM is not found, so where is it?
Thank you very much.

Did you find the code for long range TDM? Thanks

font{
    line-height: 1.6;
}
ul,ol{
    padding-left: 20px;
    list-style-position: inside;
}

    The short term TDN part is in the tdn_net.py.

                            ellery-holmes

<EMAIL_ADDRESS>

    签名由
    网易邮箱大师
    定制

On 6/9/2021<EMAIL_ADDRESS>wrote：

Hi, thank your for sharing codes.
I am reading your code and I only find the long term TDM as mentioned in the paper in base_module.py and the short term TDM is not found, so where is it?
Thank you very much.
Did you find the code for long range TDM? Thanks
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

font{ line-height: 1.6; } ul,ol{ padding-left: 20px; list-style-position: inside; } The short term TDN part is in the tdn_net.py. ellery-holmes @.*** 签名由 网易邮箱大师 定制 On 6/9/2021 @.***> wrote： Hi, thank your for sharing codes. I am reading your code and I only find the long term TDM as mentioned in the paper in base_module.py and the short term TDM is not found, so where is it? Thank you very much. Did you find the code for long range TDM? Thanks —You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks!


Respect CLI arguments and configuration when both provided

Closes https://github.com/graphql-hive/gateway/issues/310
User can provide the token in the CLI arguments, and have some registry configuration in `gateway.config`
Since token can be provided in the CLI arguments, it should not be required in the configuration.
[Extra but needed] : Fixes in Test suite
Besides the changes in the code, I also saw that `bin` serve runner was not there and the tests were never running with the binary.
So I fixed it by adding bin serve runner implementation in the test suite.
I also found that process.env is not passed to the Docker runners as we do for other runners.
But I couldn't figure out why args not passed to the Docker runners, I need @enisdenjo 's help there because we test args here for Docker as in the original issue.

CLI Arguments reference -> https://the-guild.dev/graphql/hive/docs/api-reference/gateway-cli
We also have CLI configuration reference but it doesn't even mention about reporting;
https://the-guild.dev/graphql/hive/docs/api-reference/gateway-config
But instead we have this page for the configuration API for reporting
https://the-guild.dev/graphql/hive/docs/gateway/usage-reporting
And it is not clear how to pass other options like selfHosting :/

CLI Arguments reference -> the-guild.dev/graphql/hive/docs/api-reference/gateway-cli
We also have CLI configuration reference but it doesn't even mention about reporting; the-guild.dev/graphql/hive/docs/api-reference/gateway-config
But instead we have this page for the configuration API for reporting the-guild.dev/graphql/hive/docs/gateway/usage-reporting
And it is not clear how to pass other options like selfHosting :/

Thanks!
Let's improve the docs around these as part of this PR (I understand its on a different repo...)

Actually this is a PR for fixing a bug more than changing it. But I added docs for Hive Client reference here to make it clear for users need to use more features from Hive Console.
This is the PR improving docs for that purpose.
@dotansimha
https://github.com/graphql-hive/console/pull/6145


Unauthorized to retrieve API key

Hello, im trying to retrieve an API key to use with homebridge. My integration in Homeassistant already works, it works with homebridge until a reboot of my server.
Im trying to access the key through IP:PORT/api and this is the response
[     {         "error": {             "address": "/",             "description": "unauthorized user",             "type": 1         }     } ]
It doesnt matter if I press the "Authenticate app" button in the advanced settings of the GUI. Any suggestions on what could be wrong?

This seems to not be related to Docker specifically... closing for now.


Use MiMalloc as global memory allocator

This should make allocation behavior more consistent across different builds and prevent memory fragmentation issues.

This change is

Doesn't fix our issues.


Issue on page /pairwise/needleman.html

Im taking the course CB2442 and I think there might be errors in the exercises 3.1 and 3.2 below 3.6.3. I think that the "Match" and "Gap penalty" values have been switched with each other. "Match" should be 3 and "Gap penalty" should be -2 but I might be incorrect.


rustup

https://github.com/rust-lang/rust/pull/95621 made channels strict provenance compliant. :)

@bors r+

:pushpin: Commit ebb70da4c66dba4a0e036377b3cc19cc41f92e9a has been approved by RalfJung

:hourglass: Testing commit ebb70da4c66dba4a0e036377b3cc19cc41f92e9a with merge 8a9ef3c2483b7ac7c554905b5109b7d8f8e82676...

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


Do I get anything for filing issue #1000?

Thanks! The amazing support you guys provide never ceases to amaze me! Keep it up!


update manual re. searching for standards

Not all standard fields have obstype STANDARD, so I updated the manual with a note explaining. Let me know if this is clear enough.

Looks good to me


Fail to build hlint-3.3.5 with ghc-lib-parser-ex-<IP_ADDRESS>

From https://github.com/shayne-fletcher/ghc-lib-parser-ex/blob/2167169d7b6e52c332e0e0b6147e5ec98d2039c0/ChangeLog.md#9005-released-2021-12-25

Rename isQuasiQuote to isQuasiQuoteExpr

It causes a compile error.
src/Hint/Match.hs:162:57: error:
Error:     • Variable not in scope: isQuasiQuote :: LHsExpr GhcPs -> Bool
    • Perhaps you meant one of these:
        data constructor ‘HsQuasiQuote’ (imported from GHC.Hs),
        ‘mkHsQuasiQuote’ (imported from GHC.Hs)
    |
162 |   guard $ not (any isLambda $ universe lhs) || not (any isQuasiQuote $ universe x)
    |                                                         ^^^^^^^^^^^^

The current HEAD on master should have no problem as it states this:
https://github.com/ndmitchell/hlint/blob/804b896d85fe75b5532a91c3e414f1d08c720868/hlint.cabal#L91

thanks. this has been mitigated (see https://github.com/commercialhaskell/stackage/issues/6369).

Thank you, I confirmed it.


Cpu usage

Quick question, sorry if it has been asked before:
If I set num_search_workers=16 I see that it only uses 8 cores to find the first solution and then launches the remaining ones for LNS after that.
Is this correct?

Actually only 5 until a solution is found.
We will improve that in the future, but this is not the highest priority.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 8 nov. 2019 à 15:23, Xiang Chen<EMAIL_ADDRESS>a écrit :

Quick question, sorry if it has been asked before:
If I set num_search_workers=16 I see that it only uses 8 cores to find the
first solution and then launches the remaining ones for LNS after that.
Is this correct?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1699?email_source=notifications&email_token=ACUPL3OMG3BAGFGKTEW7XLLQSVY5HA5CNFSM4JKX3XRKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HX7GXHQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3LTWAU7CFBXZU7HYR3QSVY5HANCNFSM4JKX3XRA
.

As always, thanks for your help!


Fix doc badge to point to PyData

Looks good to me.  Thanks for identifying the issue and taking the time to contribute a fix @jakirkham !

Of course. Glad to help. 😄


Interface stuck when use ptvs in vs2015

在VS2015上用PTVS时，打开python enviroments 再点击pip，准备导入pip包的时候，VS2015就无响应了，卡死了。每次必现啊

Could you open Tools, Python Tools, Diagnostic Info and either post the contents as a gist and link it here, or email as an attachment to<EMAIL_ADDRESS>

@danperor did you email us diagnostic info regarding this?


Odometry

Open Odometry module had to be modded to fit 38mm odometry wheel

Might need to print cross pieces with stronger material because the ends break easily


Create getting-started.md

#441

@ppk2533 please stop spamming random things if you don't want to be reported and blocked.

#475

They are blocked now.


Email Delivery issue

Hi Community,
I install postal on my dedicate server.
I've tried to send a bulk mail on this server.
some email ids are sent, but some mail id's mail store in the queue and held messages.
after this, all messages are stored in the queue and held message, also message rate showing 0.00/minutes
After I can flush the queue and held messages.
the mail will be perfectly delivered. and the message rate will be increased.
Guys, Please Help.
Thanks

Have you got a send limit in place? Have a look under Settings > Advanced Settings > Send limit

@willpower232 Thanks For Replay
I'm not set any limit on sending mail.
My issue is that,
for example, when I send 100 mail on the postal installed server.
the 80% mail is successfully delivered but the 20% mail is store in held and queue messages.
After than when I send more mail without a clear queue and held a message.
all 100% mail is store in the queue and held message not a single mail is sent.
also, I can check the message rate is zero.
Please check below screenshot,
https://prnt.sc/qz9q5v
after that when I clear all queue and held message.
After that send more mail, all 100% mail is delivered successfully.
and during this message rate will be increased.
Please suggest me that how can I send a with your postal application, without any interruption.

We're expecting for getting a reply soon.

That is a very large amount of email messages, can you share the output of postal status?

@willpower232 Thanks For Reply
quantity doesn't matter.
I'm sending 1k mail in the same server but the issue will be the same.
below are the postal status.
`root@ns101022:~# postal status
Procodile Version   1.0.22
Application Root    /opt/postal/app
Supervisor PID      5316
Started             2020-02-05 02:12:40 -0500
|| web
|| Quantity            1
|| Command             bundle exec puma -C config/puma.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => web.1            Running      05/02/2020   pid:5322     respawns:0      port:-        tag:-
|| worker
|| Quantity            30
|| Command             bundle exec ruby script/worker.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        start-term
|| Log path            none specified
|| Address/Port        none
|| => worker.1         Running      05/02/2020   pid:15292    respawns:0      port:-        tag:-
|| => worker.2         Running      05/02/2020   pid:12578    respawns:0      port:-        tag:-
|| => worker.3         Running      05/02/2020   pid:12580    respawns:0      port:-        tag:-
|| => worker.4         Running      05/02/2020   pid:15294    respawns:0      port:-        tag:-
|| => worker.5         Running      05/02/2020   pid:12584    respawns:0      port:-        tag:-
|| => worker.6         Running      05/02/2020   pid:14927    respawns:0      port:-        tag:-
|| => worker.7         Running      05/02/2020   pid:12588    respawns:0      port:-        tag:-
|| => worker.8         Running      05/02/2020   pid:15654    respawns:0      port:-        tag:-
|| => worker.9         Running      05/02/2020   pid:15656    respawns:0      port:-        tag:-
|| => worker.10        Running      05/02/2020   pid:15300    respawns:0      port:-        tag:-
|| => worker.11        Running      05/02/2020   pid:12605    respawns:0      port:-        tag:-
|| => worker.12        Running      05/02/2020   pid:15303    respawns:0      port:-        tag:-
|| => worker.13        Running      05/02/2020   pid:14935    respawns:0      port:-        tag:-
|| => worker.14        Running      05/02/2020   pid:12614    respawns:0      port:-        tag:-
|| => worker.15        Running      05/02/2020   pid:15659    respawns:0      port:-        tag:-
|| => worker.16        Running      05/02/2020   pid:15309    respawns:0      port:-        tag:-
|| => worker.17        Running      05/02/2020   pid:12625    respawns:0      port:-        tag:-
|| => worker.18        Running      05/02/2020   pid:15315    respawns:0      port:-        tag:-
|| => worker.19        Running      05/02/2020   pid:12632    respawns:0      port:-        tag:-
|| => worker.20        Running      05/02/2020   pid:15662    respawns:0      port:-        tag:-
|| => worker.21        Running      05/02/2020   pid:15665    respawns:0      port:-        tag:-
|| => worker.22        Running      05/02/2020   pid:12645    respawns:0      port:-        tag:-
|| => worker.23        Running      05/02/2020   pid:12648    respawns:0      port:-        tag:-
|| => worker.24        Running      05/02/2020   pid:15319    respawns:0      port:-        tag:-
|| => worker.25        Running      05/02/2020   pid:12656    respawns:0      port:-        tag:-
|| => worker.26        Running      05/02/2020   pid:15668    respawns:0      port:-        tag:-
|| => worker.27        Running      05/02/2020   pid:15324    respawns:0      port:-        tag:-
|| => worker.28        Running      05/02/2020   pid:12667    respawns:0      port:-        tag:-
|| => worker.29        Running      05/02/2020   pid:12670    respawns:0      port:-        tag:-
|| => worker.30        Running      05/02/2020   pid:12673    respawns:0      port:-        tag:-
|| cron
|| Quantity            1
|| Command             bundle exec rake postal:cron
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => cron.1           Running      05/02/2020   pid:5432     respawns:0      port:-        tag:-
|| smtp
|| Quantity            1
|| Command             bundle exec rake postal:smtp_server
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => smtp.1           Running      05/02/2020   pid:5435     respawns:0      port:-        tag:-
|| requeuer
|| Quantity            1
|| Command             bundle exec rake postal:requeuer
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => requeuer.1       Running      05/02/2020   pid:15330    respawns:0      port:-        tag:-`

@willpower232 We're expecting for getting a reply soon.

The held messages imply that the receipients are having problems receiving your email so those will have to be addressed directly.
The queued messages may well be locked by an older Postal process if it got restarted somehow. You could have a go at https://github.com/postalhq/postal/issues/717#issuecomment-451656179 to see if that gets the messages out of the queue.

@willpower232,
Thanks for providing me #717 (comment),
We're doing this before raised ticket.
But the issue are same
Please Help!!

Have you been able to clear the whole queue yet? If you clear the locking fields as suggested, does that help or is there anything else in that table which needs changing too?


php8.1-compatibility: Make package php8.1 compatible - do not pass nu…

…ll to trim().

Added this PR because we need php7.4 and php8.1 compatibility at the same time.

If you need PHP 7.4, fork the package.


Bad return status 400

Hello, currently still having the same bad return status of 400, any advice please?

It isn't possible to download watch firmware using Bifrost.


Can't find matching wheels with the --find-links flag

Overview
OS - Linux
uv version - 0.2.36
I run into this error while trying to install a package from a wheel from an external link with the --find-links flag while trying to install a specific version of iree-compiler with uv defined in my pyproject.toml as -
[project.optional-dependencies]
iree = [
    # must be pip installed with --find-links=https://iree.dev/pip-release-links.html
    "iree-compiler==20240507.886",  # see IREE compatibility notes in noxfile.py
]

Steps to reproduce
When I run uv pip install .[iree] --find-links https://iree.dev/pip-release-links.html --verbose, this is what I get -
DEBUG Using request timeout of 30s
DEBUG Found fresh response for: https://iree.dev/pip-release-links.html
WARN No packages found in `--find-links` entry: https://iree.dev/pip-release-links.html
DEBUG Found fresh response for: https://iree.dev/pip-release-links.html
WARN No packages found in `--find-links` entry: https://iree.dev/pip-release-links.html
  × No solution found when resolving dependencies:
  ╰─▶ Because there is no version of iree-compiler==20240507.886 and pybamm[iree]==24.5 depends on iree-compiler==20240507.886, we can conclude that
      pybamm[iree]==24.5 cannot be used.
      And because only pybamm[iree]==24.5 is available and you require pybamm[iree], we can conclude that the requirements are unsatisfiable.

The package wheels of the iree-compiler==20240507.886 are present in that link and work with pip, but not with uv.
When I run the same command with pip (pip install .[iree] --find-links https://iree.dev/pip-release-links.html), it successfully downloads and builds the wheels for my package.
Looking in links: https://iree.dev/pip-release-links.html
Processing /home
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting iree-compiler==20240507.886 (from pybamm==24.5)
  Downloading https://github.com/iree-org/iree/releases/download/candidate-20240507.886/iree_compiler-20240507.886-cp312-cp312-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl (64.5 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 64.5/64.5 MB 6.3 MB/s eta 0:00:00

I can take a look.

I've isolated this to a bug in https://github.com/y21/tl. I'll try to fix it there.

See: https://github.com/y21/tl/issues/68


Moved/Split platform-specific files into platform-specific directories

In preparation for Verilator/BP I've started to move platform-specific files into platform-specific directories.
The tl;dr of this is that each platform will have its own hardware.mk file that defines any platform-specific HDL and link.mk that defines how linking is done on the host.
Passes regression on timing_v0_8_4 and 4x4_fast_n_fake.

libbsg_manycore_runtime.so is now THE dependency for all linking
Moved aws.vcs.f (AWS VCS filelist) to platforms/aws-vcs
Moved gen_simlibs.tcl to platforms/aws-vcs
Moved contents of testbenches/link.mk to platforms/aws-vcs/link.mk
Moved msg_config.mk to platforms/aws-vcs
Modified dramsim3.mk, libdmamem.mk to add dependencies and append to LDFLAGS to libbsg_manycore_runtime instead of LDFLAGS (for application linking) and SIMLIBS
Included dramsim3.mk, libdmamem.mk, and libinfmem.mk in $(BSG_PLATFORM_PATH)/library.mk
Moved contents of simlibs.mk to platforms/aws-vcs/hardware.mk
hardware.mk now includes $(BSG_PLATFORM_PATH)/hardware.mk

Semi-related, my plan is to move libinfmem.mk, libdma.mk and libdramsim3.mk into feature-specific folders like in #584, but this PR seemed easier to do first.

on cosimulation. I'm bugswatting in the HDL by compiling the FPGA image
You'd be amazed how much VCS lest past it.

Yep, my experience is VCS > Verilator > Vivado in terms of permissiveness
On Wed, Jun 10, 2020 at 9:08 PM Dustin Richmond<EMAIL_ADDRESS>wrote:

on cosimulation. I'm bugswatting in the HDL by compiling the FPGA image
You'd be amazed how much VCS lest past it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/bespoke-silicon-group/bsg_replicant/pull/590#issuecomment-642345577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARW6WQBJRFSOTJTEAA3ZQDRWAVAZANCNFSM4N23BEPQ
.

Can VCS infer parameters? Cosimulation is passing, but a parameter wasn't defined - worse, the wrong name was used to pass it in, and it worked?

This PR compiled the FPGA bitstream, and passed regression on F1 (with the new libraries)
All tests except test_rom passed.


tests: setup CI for end to end tests

Get a Kubernetes cluster running on CI environment with service account token (needed by REANA-Job-Controller) and test that examples work using reana CLI (reana install-client, reana install-cluster, reana run-example...). Some known options:

Use GitLab Kubernetes integration, https://edenmal.moe/post/2018/GitLab-Kubernetes-Using-GitLab-CIs-Kubernetes-Cluster-feature/
PROs:
testing on the same environment as production (real Kubernetes cluster)
Minikube without VMs
PROs:
Well-known behavior
Supports StorageClass out of the box, i.e. can use Kubernetes' PVs
Mirantis kubeadm-dind-cluster
PROs:
As the Kubernetes cluster is created with multiple "nodes" (implemented as Docker containers in the running host), it's possible to stage affinity setups and run operations like node drain, cordon, etc

So far I could partially configure the first option with CERN Gitlab. I connected GitLab CI with a running magnum managed Kubernetes cluster but didn't use it in a pipeline yet.


Error while writing file

You have selected the wrong file... The file to patch is not python3.dll but StartAllBackX64.dll. As the default path to the file (C:\Program Files\StartAllBack\StartAllBackX64.dll) was not found you must define the correct path.


What is the list of all F# warnings (number and description)?

It would be helpful to me if there were a list of all possible F# warnings.  I have looked both within Microsoft's documentation as well as in general internet searches, but I have not found any such list.
The closest I have found to a list of #F warnings is the file FSComp.txt in the F# source code, but it doesn't seem consistent with the few passing references to warnings in compiler options documentation (against which I have created this issue).
For example, the documentation for the compiler option --warnon says (in part)...

...the 1182 (unused variables) warning is off by default.

...which seems consistent with FSComp.txt since the warning number 1182 appears on this line.
On the other hand, the documentation for the compiler option --warn says (in part)...

Level 5 warnings are: 21 (recursive use checked at runtime), 22 (let rec evaluated out of order), 45 (full abstraction), and 52 (defensive copy).

...which seems inconsistent with FSComp.txt since none of those warning numbers (21, 21, 45, and 52) appear on any line.
Is there documentation my Microsoft that lists all possible F# warnings?  If not, can such documentation be created?

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

ID: 3c40365d-5626-9a4f-4de3-15d14c40af2b
Version Independent ID: 8fecd942-bbd7-7e61-092d-084896250876
Content: Compiler Options - F#
Content Source: docs/fsharp/language-reference/compiler-options.md
Product: dotnet-fsharp
GitHub Login: @cartermp
Microsoft Alias: phcart

I seem to remember that these level 5 warnings are still in development. If so, the documentation is premature and should probably say so.


BUG: Ph Number validation should be added

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hey @Ishitamukherjee2004 can you please assign this issue to me

Assigned @Ishitamukherjee2004


No DNS configuration step.

Hi,
When I run the install I do not see any window about DNS entries.
I'm following this http://kamilslab.com/2017/01/22/how-to-turn-your-raspberry-pi-into-a-home-vpn-server-using-pivpn/
This was released in jan of this year but I have seen others since that have been written since then that show the same steps.
How do I fix this or is this feature change in the recent version?

I'm having the same issue. @rmccarthytt did you manage to figure it out?

I am having the same issue as well. I installed it on my RPi 3 a month ago with no problems, but I tried installing on a new RPi3 yesterday, and it will not work.

i am too

No, I haven't. Can we have a dev comment on this.

Maybe the script is exiting on confOVPN or above. Are you able to tell? Check if there are some error messages that could result in a fatal error.

I can try soon. But I did finish the install without errors., It just didn't work.

Old thread, lack of details.
closing.


Remove claim queue

[x] remove from client
[x] remove from server
[ ] remove from contracts
[x] cleanup on third party services (as i remember mautic may be used here so we could remove lists not needed anymore)

@sirpy It's partially done in your "wallet refactoring" PR
does it already removed from server / contracts ?
could you check please ?

@johnsmith-gooddollar yes it has been removed from server. still need to remove from schema and types. search for claimqueue

@johnsmith-gooddollar yes it has been removed from server. still need to remove from schema and types. search for claimqueue

thanks!

@sirpy and how about contracts ? could we close this task once refactoring PR will be merged

queue is not related to contracts

queue is not related to contracts
Hadar Rottenberg
Technical Lead, GoodDollar
+972-507 319 093  (telegram http://t.me/sirpy, whatsapp)  blog
https://medium.com/dablock gooddollar.org http://www.gooddollar.org/

https://www.facebook.com/TheGoodDollar
https://twitter.com/TheGoodDollar
http://instagram.com/gooddollarorg http://t.me/GoodDollarX
https://www.linkedin.com/company/gooddollar/
@.**>
http://youtube.com/channel/UC_8oVeHIUo9U8gAc2BwaviA
http://reddit.com/user/GoodDollar https://github.com/gooddollar
Join the OpenUBI https://t.me/openUBI movement!

On Mon, May 2, 2022 at 4:08 PM johnsmith-gooddollar <
@.***> wrote:

@sirpy https://github.com/sirpy and how about contracts ? could we
close this task once refactoring PR will be merged
—
Reply to this email directly, view it on GitHub
https://github.com/GoodDollar/GoodDAPP/issues/3569#issuecomment-1114835947,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFD254HC5HTDFH6VBQ4ELDVH7HVLANCNFSM5KD4UFRQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Validate for missing tokens

e.g.
path: /path/to/{this}/{that}
provides:
    that: [ foo, [] ]

We should throw an exception:
Found token `{that}` but there was no corresponding token provider

Fixed in #34


Expose liveness and readiness probe configuration in values settings

Different environments and different configuration may have different expectations for liveness and readiness.
We have experienced that the initial startup using declarative 'git' backend can take 30s or more on some occasions, which means that the default liveness probe would kill the container before it could complete.
By exposing these for users to override and configure, alternative initialDelay, periodSeconds, and thresholds are able to be tweaked and refined to support the user's specific expectations.
By default, the same probes that were previously hardcoded are configured in the defaulti values.yaml.

@GeorgeMac @markphelps - Please review when you have a chance and let me know if there are any questions or concerns. Thanks!


[Packaging] RPM update to support systemd-based OS

Hi,
I have updated some files to create a RPM package that deals with systemd:

conf-examples/carbon.conf: path updates for logs, data and schema
deploy/rpm: added logrotate + service file

In order to create a clean PR, I have some questions:

Do you want a spec file per distribution release?

go-carbon.spec.centos, go-carbon.spec.suse...

Do you want a generic logrotate file?

buildrpm.sh could create it on the fly depending on carbon.conf and the target OS

Mathieu

Hi,
I already have refactoring plan. All path will be changed in release 0.10 (not next 0.9 release).
0.10 roadmap:

Replace logging library from logrus to uber-go/zap. Remove reopen by HUP and inotify. Add periodic logfile changed check
Move /usr/local/bin/go-carbon to /usr/bin/go-carbon
Set config paths to /etc/go-carbon/go-carbon.conf, /etc/go-carbon/storage-schemas.conf, /etc/go-carbon/storage-aggregation.conf
Add systemd service file

I do not like to include log rotation logic into package. But I still think about it

I have pushed some stuff in my fork. I can create a PR if you want. Fell free to comment: https://github.com/mgrzybek/go-carbon/commit/d0641aeb5f367b8b3c4289c5a3f24cd6e877e025

I'll happily handle the systemd service integration in the debian packaging, but please merge the changes from @mgrzybek first to avoid double work.

Any progress on this?

systemd service has been added in 0.10.0 release


Send "Unlatched" status via MQTT

Hello,
I have the following "issue".
I'm using Nuki together with an automatic door opening mechanism which requires the Nuki locked to be UNLATCHED.
As of right now, I'm timing the opening of the door based on the state of the Nuki Lock.
If (Nuki Lock "Unlocked")
-Unlatch command
-Wait 3 seconds (waiting for unlatching)
-Open Door
If (Nuki Lock "Locked")
-Unlatch command
-Wait 8 seconds (waiting for unlocking and unlatching)
-Open Door
Problem is that this "time based" things in my experience are very very unreliable.
Any way to send the "unlatched" status to MQTT so I can use it Homeassistant instead of time based solutions.
br

@technyon  Thanks for the fast answer. I suspected some kind of behaviour like this.
In the MQTT I get only the:
nuki40/lock/binaryState (Locked/Unlocked)
nuki40/lock/action
Can't see, find any nuki40/lock/state topic.
Am I doing something wrong?

Yep, was doing something wrong. Just subscribed to the nuki40/lock/state.
So I can retrieve unlatched status.
When you open the lock from unlocked state it indeed won't show everytime the unlatched status.
But from LOCKED status it will always show UNLATCHED after using lock.open which for my usage scenario is exactly what I need.
Thanks for your awesome work.


Analyser not picking up changes after BuiltValue generation WebStorm/MacOSx

This issue started life in BuiltValue, making it was here because its beginning to seem a lot more analyser related. @davidmorgan
Formalities
VM
Dart VM version: 2.0.0 (Fri Aug 3 10:53:23 2018 +0200) on "macos_x64"
IDE
WebStorm 2018.2.3 EAP Build #WS-182.4323.31, built on August 30, 2018 JRE: 1.8.0_152-release-1248-b8 x86_64 JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o macOS 10.13.6

[x] Checked Stackoverflow
[x] Search Previously filled issues.
[x] Observable and repeatable symptoms.

Summary
After making changes to a BuiltValue abstract and successfully generating the implementation, Webstrom is unable to detect the generated file, causing the disliked red squiggly line to appear. By manually restarting the analyzer from with the IDE all is made good with the world.
The initial diagnostic and Q&A is here, would appreciate any help. This issue is not a show stopper, yet does interrupt the development flow.

We're also seeing this issue with over_react. It would be great to not have to restart the analysis server after every build.

@bwilkerson any updates on this?


AttributeError: 'ThreadRunCreated' object has no attribute 'type'

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
From the docs at https://github.com/openai/openai-python/blob/main/helpers.md:
from typing_extensions import override
from openai import AssistantEventHandler, OpenAI
from openai.types.beta.threads import Text, TextDelta
from openai.types.beta.threads.runs import ToolCall, ToolCallDelta

client = OpenAI()  # NOTE: this is an edit to the original code: `client = openai.OpenAI()`

class EventHandler(AssistantEventHandler):
  @override
  def on_text_created(self, text: Text) -> None:
    print(f"\nassistant > ", end="", flush=True)

  @override
  def on_text_delta(self, delta: TextDelta, snapshot: Text):
    print(delta.value, end="", flush=True)

  def on_tool_call_created(self, tool_call: ToolCall):
    print(f"\nassistant > {tool_call.type}\n", flush=True)

  def on_tool_call_delta(self, delta: ToolCallDelta, snapshot: ToolCall):
    if delta.type == 'code_interpreter':
      if delta.code_interpreter.input:
        print(delta.code_interpreter.input, end="", flush=True)
      if delta.code_interpreter.outputs:
        print(f"\n\noutput >", flush=True)
        for output in delta.code_interpreter.outputs:
          if output.type == "logs":
            print(f"\n{output.logs}", flush=True)

with client.beta.threads.runs.stream(
  thread_id=thread.id,            # using real thread ID
  assistant_id=assistant.id    # using real assistant ID
) as stream:
    for event in stream:
        if event.type == "thread.message.delta" and event.data.delta.content:
            print(event.data.delta.content[0].text)

throws the following error:
  File "/workspaces/ChatGPT-Arc/tmp/sandbox.py", line 108, in main
    stream_run(client, thread, assistant)
  File "/workspaces/ChatGPT-Arc/tmp/sandbox.py", line 80, in stream_run
    if event.type == "thread.message.delta" and event.data.delta.content:
  File "/usr/local/lib/python3.9/site-packages/pydantic/main.py", line 792, in __getattr__
    raise AttributeError(f'{type(self).__name__!r} object has no attribute {item!r}')
AttributeError: 'ThreadRunCreated' object has no attribute 'type'

It appears that the docs are out-of-date.
Using  event.data.object instead of event.type works for me:
        for event in stream:
            if event.data.object == "thread.run.step.delta" and event.data.delta.content:
                print(event.data.delta.content[0].text)

Note also that  client = openai.OpenAI() should be client = OpenAI() in the example code.
To Reproduce
See above
Code snippets
No response
OS
macOS
Python version
3.9.19
Library version
1.21.2

Thanks for reporting, this has been fixed :)

Thanks @RobertCraigie !
Do you know if the docs will soon be aligned in regards to vector store file batching:
The File Search docs show:
file_batch = client.beta.vector_stores.file_batches.upload_and_poll(
  vector_store_id=vector_store.id, files=file_streams
)

...but upload_and_poll is not described in the Vector Store File Batches docs.

cc @nknj re; adding upload_and_poll to the docs.

OpenAI is not adding higher-level helpers like upload_and_poll to the reference docs.

@rattrayalex I'm curious why OpenAI is not include some helpers in the docs? Why leave them undocumented? Will they soon be deprecated?

Thanks for the feedback. Longer term, we want to launch a SDK reference in our docs that documents these. The current reference is an API reference and doesn't include SDK specific helpers. These are documented in the helpers.md files in the SDKs and we link to them via our docs.

Will they soon be deprecated?
No, they wont!

This issue is not fixed. I just got the same error with the following:
with client.beta.threads.runs.stream(    thread_id = thread_id,    assistant_id = assistant_id ) as stream:    for event in stream:       if event.type == 'thread.message.delta' and event.delta.          text = print(event.data.delta.content[0].text)          print(f"{text}", end="", flush=True)          yield text
Error during streaming: 'ThreadRunCreated' object has no attribute 'type'

@GoldenJoe can you open a new issue with a full repro script & stack trace?


Roll Flutter Engine from fd41747c23a9 to 2ed05f0068e2 (2 revisions)

https://github.com/flutter/engine/compare/fd41747c23a9...2ed05f0068e2
2024-12-07<EMAIL_ADDRESS>Roll Skia from b37f1430f87a to d1e261ca7f6f (1 revision) (flutter/engine#57039)
2024-12-07<EMAIL_ADDRESS>Roll Dart SDK from 838ed940b056 to c7e47c6c5df6 (1 revision) (flutter/engine#57038)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


samples: mesh: nrf52: to solve issues regarding PTS which are pending after merging PR:9475

This PR will try to solve issues which are pending after merging
https://github.com/zephyrproject-rtos/zephyr/pull/9475

Separately defined publisher functions for Servers. This would help
to send extra status responses whenever required.

Signed-off-by: Vikrant More<EMAIL_ADDRESS>

Kindly don't merge as of now.

@carlescufi
could you helped me to resolved
https://app.shippable.com/github/zephyrproject-rtos/zephyr/runs/22312/7/tests
this issue ?

[ ] MMDL/SR/LLN/BV-06-C https://hastebin.com/mediwoduga.sql
[ ] MMDL/SR/LLN/BV-07-C https://hastebin.com/ajukokoqin.sql
[ ] MMDL/SR/LLN/BV-08-C https://hastebin.com/ocayekecul.sql
[ ] MMDL/SR/LLN/BV-09-C https://hastebin.com/nekedobewu.sql
[ ] MMDL/SR/LLN/BV-10-C https://hastebin.com/afomogiqox.sql
[ ] MMDL/SR/LLN/BV-11-C https://hastebin.com/uqewahunuk.sql
[ ] MMDL/SR/LLN/BV-12-C https://hastebin.com/adexujixak.sql
[ ] MMDL/SR/LLN/BV-13-C https://hastebin.com/nuxapejuqa.sql
[ ] MMDL/SR/LCTL/BV-04-C https://hastebin.com/ufagubazim.sql

As per <IP_ADDRESS>.3 of Mesh Model Specification,

When a Light Lightness Actual state change is completed as a result of processing an acknowledged
message or a transaction of messages other than a Light Lightness Set message, a corresponding
status message shall be sent to the originator to indicate the status change, and a separate Light
Lightness Status message shall be published. It is recommended to transmit a Light Lightness Status
message when the element has been physically turned on or off locally (as opposed to via the mesh
network).

& as per <IP_ADDRESS>.3

When a Light CTL state change is completed as a result of processing
an acknowledged message or a transaction of messages other than a Light CTL Set message, a
corresponding status acknowledge message shall be sent to the originator and a separate Light CTL
Status message shall be published.

2nd commit would take care of these two points.

Along with review it also needs feedback from PTS.
@MariuszSkamra if time allow you then please run PTS against it.

@jhedberg @carlescufi @MariuszSkamra @sjanc
As of now, I've not test performance of this App against Mobile App Seekbar/Scrollbar GUI ( .... while controlling GROUP of Nodes over single GATT Proxy connection).
As per my understanding current implementation could be processor hungry & it will take some time to process the data over mesh when user scroll Seekbar to vary Gen. Level or Light Lightness Actual or Light CTL states.
If user scroll Seekbar very fast then it could also hang Mesh n/w for while.

Solution is there should be only 10 virtual discrete points on Seekbar which are hidden for user.

I simply take reference from Bluetooth Mesh Study Guide to call Zephyr APIs
which are related to timer & workqueue Threads. So I need your help/suggestions to improve/optimize this App, to handle this situation so that whenever user vary Seekbar it should look like all nodes are in Sync.

Thank You !!

@Laczen Could you please review or help me to improve the 3rd commit ?

@jhedberg Please review 3rd commit now.
CC: @Laczen

@sjanc @carlescufi
Please review this PR. It will be great if you help me to optimize overall App to improve it performance.

Any update on PTS front ?

@MariuszSkamra @sjanc
Please execute PTS against this PR. This will help to stabilize the implementation or architecture.
Thanks in advance !!

I'm on sick leave now, coming back on wednesday, hopefully

@MariuszSkamra Please wait & don't execute PTS against this PR as of now. Thank You !!

As per current implementation, if client publishes Gen. Level Value as 100 %
& Transition Time as 60 Secs then in Gen. Level Status response Server provides correct present, target values as well as remaining time to achieve the target value.
But problem is, if in meantime (during 60 secs) client ask for Light CTL status or Light Lightness status, then it would only respond with present Light CTL / Light Lightness value. Here status would not contain target value & Remaining time. Currently I don't know how to crack it plus I don't know whether PTS is expecting this or not.
@MariuszSkamra So please go ahead as of now on PTS front.

In process of solving previous mentioned issue. Try to solve it today itself.

Mentioned issue has solved now completely !!
Please review someone.

@jhedberg Please review latest changes.
For previously mentioned issue (regarding details in status response even when bound state is in transition) I've to modify architecture aggressively (last commit).

@jhedberg
storage.txt
Kindly go thrugh with attached file. After these changes not able to save Values of SoC flash.

@jhedberg Now able to save data on SoC flash after making some additional changes.

@MariuszSkamra
Please execute PTS against this PR.
@jhedberg If other things are okay for you then please merged this PR. It is very heavy now.

@carlescufi
Not able to see progree on Shippable. Also tried to close & re-open this PR but issue hasn't resolved.

@jhedberg Kindly merge this PR. If anything is pending then I will take of it in next PR.
Hope someone will execute PTS against it before release of v1.14 :)

@jhedberg Thanks for merging.

@jhedberg Thanks for merging this PR.

@vikrant8051 you're welcome. I hope we can avoid your future PRs growing this big - this one really became too big and went on for too long time to do a proper review of it.


QuickReturnHeaderListFragment have a issues

header 覆盖住了上面的部分listview。我通过contentview getView 设置，但是无效，代码如下。
private static final int TYPE_HEADER = 0;
private static final int TYPE_ITEM = 1;

@Override
public int getItemViewType(int position) {
if (isPositionHeader(position))
return TYPE_HEADER;
    return TYPE_ITEM;
}

private boolean isPositionHeader(int position) {
    return position == 0;
}

@Override
public NewsDataItem getItem(int position) {
    return (NewsDataItem) mItemList.get(position - 1);
}

@Override
public int getCount() {
    return mItemList.size() + 1;
}

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    final ItemViewHolder itemviewHolder;

    if (TYPE_ITEM == getItemViewType(position))

convertView = null;

Hey @ayyb1988, can you be more specific?  What issues are you experiencing?

my means is     for    listview girdview 。the first item is coverd.

@ayyb1988 you should have read previous issue before creating a duplicate. It's 100% same as yourth.

@metalurgus I'am so sorry.  thx . have fix.


Treat Chinese ideograms as separate words

As requested on Community: The Search function does not support Chinese characters - General - SilverBullet Community, the discussion whether this makes sense or not continues there
Had to export the tokenizer to write tests in a separate file, is this how it's supposed to be done?

Marked as draft to wait for feedback from some Chinese speakers

It has to be a substring, random order is bad


double negative ish on health scrore

Problem with this page
https://plugins.jenkins.io/ldap/healthscore/
Expected behavior

No commit in the last 4 years. 0%

0 percent sounds bad. But it seems like from the above checks that there is comments newer than 4 years ago. So this feels like the check is inverted, if the only commits are more than 4 years ago, then negative points.
I think the check should be "had commits more recent than 4 years ago" so it sticks with convention
Or I'm wrong and it proves my point it's confusing

Actual behavior
Na
Possible solution
No response

thanks for reporting this @halkeye (and @zbynek and @smerle33 for transferring the issue here).

Or I'm wrong and it proves my point it's confusing

You are not wrong but still it is confusing. Once we see that the gap between the latest release and the latest commit is less than 6 months, we shouldn't try to see if there is less than a year between the two. And so on.
There is a large problem in the AdoptionScoring implementation, which I think I resolved in #392.
I'll merge this and release the project so that we can get rid of this invalid information.


Certora's formal verification code

Review Error for stevieraykatz @ 2024-03-15 20:21:48 UTC
User must have write permissions to review


From 3.x to 4.x, is there a new way to use request() ?

if I call my API in this way using Supertest 4.x:
return res.status(401).send()

and try to assert it in this way:
response = await request(app).post('/admin').send( { data_here } )
expect(response.status).toBe(401);

Supertest returns Unauthorized and not status code 401 as I'm expecting.
If I do the same thing using Supertest 3.x , it returns the status code I'm expecting to test it.
Is there a new way to do it on version 4.x ?

We used to set the expected status code, like 401, but that no longer returns the response, it throws the error instead.
Seems like a breaking change without documentation.

Same here, used this in 3.x:
  await request(app)
    .post('/system/invoice/generate')
    .set('content-type', 'application/json')
    .set('Authorization', 'invalidtoken')
    .send()
    .expect(httpStatus.UNAUTHORIZED)

Short migration guide would be nice

This may be a regression - see https://github.com/visionmedia/superagent/issues/1466

same and has been reported yesterday but still no fixes.

same and has been reported yesterday but still no fixes.

same and has been reported yesterday but still no fixes.

Guy, you don't really test? The exact same issue happened already with v3.4.0.
https://github.com/visionmedia/supertest/issues/534
Can you please create a meaningful test, so a new version wont break anything? Or mark this as a breaking change...

Don't you have any tests which cover these cases? I mean there was already something similar:
https://github.com/visionmedia/supertest/issues/534

Don't you have any tests which cover these cases? I mean there was already something similar:
#534

Feel free to open an PR and add those tests 👍

I’ve started adding a few tests, the problem is all the tests use callbacks only, so the regression was overlooked, again (sorry!).

Just to be clear, my comment was in @akirilyuk s direction.

@SMenigat should not the person who created the fix also add the tests which verifies the error wont happen again? Also should it not be the maintainers job to have meaningful tests for their own product?

+1 with @akirilyuk

#558 merged, I close this issue.
Thank you 🙏


Telegraf 1.9

Includes new release Telegraf 1.9 and 1.8 updates.

A few updates are still needed:

[ ] The new Telegraf version needs to be added in data/versions.toml.
[ ] The telegraf and telegraf_semver variables need to be updated in data/default_versions.toml.
[ ] The frontmatter title and name need to be updated on content/telegraf/v1.9/_index.md.
[ ] The frontmatter weight needs to be >1 on  content/telegraf/v1.8/_index.md to make sure the order of versions is correct in the sidebar.
[ ] Fix broken links (included below)

http://localhost:1313/telegraf/v1.9/about_the_project/release-notes-changelog/
- (619:4) 'HTTP Lis..' => http://localhost:1313/plugins/inputs/http_listener_v2/README.md (HTTP 404)
- (620:4) 'IPVS (ip..' => http://localhost:1313/plugins/inputs/ipvs/README.md (HTTP 404)
- (621:4) 'Jenkins ..' => http://localhost:1313/plugins/inputs/jenkins/README.md (HTTP 404)
- (622:4) 'NGINX Pl..' => http://localhost:1313/plugins/inputs/nginx_plus_api/README.md (HTTP 404)
- (623:4) 'NGINX VT..' => http://localhost:1313/plugins/inputs/nginx_vts/README.md (HTTP 404)
- (624:4) 'Wireless..' => http://localhost:1313/plugins/inputs/wireless/README.md (HTTP 404)
- (630:4) 'Stackdri..' => http://localhost:1313/plugins/outputs/stackdriver/README.md (HTTP 404)
- (853:4) 'wavefront' => http://localhost:1313/telegraf/v1.8/data_formats/input/wavefrount/ (HTTP 404)
- (619:4) 'HTTP Lis..' => http://localhost:1313/plugins/inputs/http_listener_v2/README.md (HTTP 404)
- (620:4) 'IPVS (ip..' => http://localhost:1313/plugins/inputs/ipvs/README.md (HTTP 404)
- (621:4) 'Jenkins ..' => http://localhost:1313/plugins/inputs/jenkins/README.md (HTTP 404)
- (622:4) 'NGINX Pl..' => http://localhost:1313/plugins/inputs/nginx_plus_api/README.md (HTTP 404)
- (623:4) 'NGINX VT..' => http://localhost:1313/plugins/inputs/nginx_vts/README.md (HTTP 404)
- (624:4) 'Wireless..' => http://localhost:1313/plugins/inputs/wireless/README.md (HTTP 404)
- (630:4) 'Stackdri..' => http://localhost:1313/plugins/outputs/stackdriver/README.md (HTTP 404)
- (853:4) 'wavefront' => http://localhost:1313/telegraf/v1.8/data_formats/input/wavefrount/ (HTTP 404)

http://localhost:1313/telegraf/v1.9/introduction/
- (594:70) 'Download..' => https://influxdata.com/downloads/#telegraf (HTTP 200 but missing anchor)
  - redirect path:
    - https://influxdata.com/downloads/ (301)
    - https://www.influxdata.com/downloads/ (301)
    - https://portal.influxdata.com/downloads (200)
- (596:13) 'InfluxDa..' => https://influxdata.com/downloads/#telegraf (HTTP 200 but missing anchor)
  - redirect path:
    - https://influxdata.com/downloads/ (301)
    - https://www.influxdata.com/downloads/ (301)
    - https://portal.influxdata.com/downloads (200)

http://localhost:1313/telegraf/v1.9/introduction/downloading/
- (590:50) 'InfluxDa..' => https://influxdata.com/downloads/#telegraf (HTTP 200 but missing anchor)
  - redirect path:
    - https://influxdata.com/downloads/ (301)
    - https://www.influxdata.com/downloads/ (301)
    - https://portal.influxdata.com/downloads (200)

http://localhost:1313/telegraf/v1.9/plugins/inputs/
- (1577:100) 'InfluxDB..' => http://localhost:1313/telegraf/v1.9/plugins/inputs/#influxdb-listener (HTTP 200 but missing anchor)

http://localhost:1313/telegraf/v1.9/plugins/outputs/
- (775:181) 'Stackdri..' => https://cloud.google.com/stackdriver/pricing#stackdriver_monitoring_services (HTTP 200 but missing anchor)


deps: Updated OTel to v0.83.0

Proposed Change

Updated OTel to v0.83.0

Checklist

[x] Changes are tested
[ ] CI has passed

Looks like they bumped the min go version to 1.20 for OTel, we'll have to update as well.

Looks like they bumped the min go version to 1.20 for OTel, we'll have to update as well.

I noticed that as well. I'll update that in another PR then rebase this on.

PR with go update in #1205


feat: webhooks

Fixes #63

Also, seems like tests are failing due to node-fetch. I'd recommend to downgrade the node-fetch version to the latest 2.x version. They made some breaking changes in v3 that we aren't ready to consume et

Codecov Report

Merging #92 (202daa6) into main (0533b9d) will decrease coverage by 1.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
- Coverage   86.27%   85.10%   -1.18%
==========================================
  Files          28        6      -22
  Lines         481       47     -434
  Branches       68        4      -64
==========================================
- Hits          415       40     -375
+ Misses         66        7      -59

Impacted Files
Coverage Δ

packages/rest/src/server.ts

...ackages/rest/src/schemas/CredentialOfferRequest.ts

...kages/rest/src/schemas/PresentationProofRequest.ts

packages/rest/src/schemas/ProofProposalRequest.ts

...est/src/schemas/AcceptCredentialProposalRequest.ts

...ges/rest/src/controllers/proofs/ProofController.ts

packages/rest/src/utils/ProofRequest.ts

...ges/rest/src/schemas/AcceptProofProposalRequest.ts

...kages/rest/src/schemas/ProofPresentationRequest.ts

...ckages/rest/src/schemas/CredentialOfferTemplate.ts

... and 12 more

Continue to review full report at Codecov.

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


Use gl::EnableClientState and gl::VertexPointer doesnt work

Im trying to use this:
gl::EnableClientState();

The compiler throws:

unresolved name gl::EnableClientState [E0425]

I was looking for this function in the bindings.rs and didn't find it. Will it be supported in the future?

That's a function from OpenGL 1 that has been deprecated and removed in OpenGL 3. It's not in the bindings on purpose.

Ok, i kind of suggested this. I've found this function in a lot of examples (like this one: http://stackoverflow.com/questions/6940708/what-is-the-purpose-of-enable-client-state) looking for a good way to load vertex and normal data into my vertex shader.
I already created a VBO containing both. The thing Im looking for right now is how to tell OpenGL to make my normal values get accesable by my vertex shader. Could you give me a hint: What is the up-to-date way of doing this?

A quick google for glEnableClientState gives: http://stackoverflow.com/a/11807263

Closing because we don't support the deprecated OpenGL APIs. Fell free to ask more questions though! :)


The battery icon shows an extra pixel while charging with USB

is this a bug ?

mmm looks like it, it doesn't happen for me, I have two radios here.
I'll let you know when I find it !

Great work 111!
Confirm this artifact also appears when the battery charge drops (or is transitioning) from 3 bars to 2. Same on both K5's - latest compilation.

Try it now, I've just uploaded a fix - hopefully.
Let me know if it still shows up.

Perfect thank you


Update ProGuard suggested config.

Closes #3582.

I thought it was the resource being stripped and not the class. The class is in the stacktrace.

Yeah, you're right.
This rule does work (had this issue on the app I work on), but I guess it keeps too much, then.
I'll go look at the ProGuard docs to see if I can find the right config.

ProGuard doesn't strip resources according to its docs.
On Tue, Oct 10, 2017 at 7:45 PM Eric Cochran<EMAIL_ADDRESS>wrote:

Yeah, you're right.
This rule does work (had this issue on the app I work on), but I guess it
keeps too much, then.
I'll go look at the ProGuard docs to see if I can find the right config.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/pull/3647#issuecomment-335639950, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEaxNNpOtSQGdP988acTbzKSj8jwoks5srAGhgaJpZM4P0tcR
.

What I think is going on, is that the stacktrace in the original issue #3582 may have been copied from Crashlytics (I don't remember, but probably), and therefore the class names were deobfuscated using the ProGuard mapping. That's why the class names show up in the stacktrace as though they were untouched by ProGuard--it's misleading.
Adding the -keep class ProGuard rule originally suggested by @csshuai fixes the problem, because the package name was actually being obfuscated, so the resource directory didn't match the class package name.

Isn't the resource looked up with an absolute path though?
And if not, we only need to prevent the class from being obfuscated.
Preventing it from being removed in shrinking is too strict.
On Wed, Oct 11, 2017, 12:41 AM Andrew Shu<EMAIL_ADDRESS>wrote:

What I think is going on, is that the stacktrace in the original issue
#3582 https://github.com/square/okhttp/issues/3582 may have been copied
from Crashlytics (I don't remember, but probably), and therefore the class
names were deobfuscated using the ProGuard mapping. That's why the class
names show up in the stacktrace as though they were untouched by
ProGuard--it's misleading.
Adding the -keep class ProGuard rule originally suggested by @csshuai
https://github.com/csshuai fixes the problem, because the package name
was actually being obfuscated, so the resource directory didn't match
the class package name.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/pull/3647#issuecomment-335680549, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEayuy2ngnxQmmP4CZoXz1VCxWqPUks5srEb3gaJpZM4P0tcR
.

Isn't the resource looked up with an absolute path though?

It had been, until OkHttp 3.9.0: 9bfe575960af8e7bb164642f2f513b5ccb39ff4f

And if not, we only need to prevent the class from being obfuscated. Preventing it from being removed in shrinking is too strict.

:+1: Then it should be:
 -keepnames class okhttp3.internal.publicsuffix.PublicSuffixDatabase

Short for -keep,allowshrinking https://www.guardsquare.com/en/proguard/manual/usage#keepoptions

Ah, yep. Totally forgot about that!
Let's do the -keepnames and then we should be good.
On Wed, Oct 11, 2017 at 1:17 AM Andrew Shu<EMAIL_ADDRESS>wrote:

Isn't the resource looked up with an absolute path though?
It had been, until OkHttp 3.9.0: 9bfe575
https://github.com/square/okhttp/commit/9bfe575960af8e7bb164642f2f513b5ccb39ff4f
And if not, we only need to prevent the class from being obfuscated.
Preventing it from being removed in shrinking is too strict.
👍 Then it should be:
-keepnames class okhttp3.internal.publicsuffix.PublicSuffixDatabase
Short for -keep,allowshrinking
https://www.guardsquare.com/en/proguard/manual/usage#keepoptions
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/pull/3647#issuecomment-335685724, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEEEcy06wT3qb6q5ZiIPm4SABC60LsJks5srE9ngaJpZM4P0tcR
.

Ah, thank you! works for me.


Update Installing Dependencies section in docs

Issue https://github.com/project-serum/anchor/issues/437

Before changing this, we should update the tutorials to use package.json. We can potentially consolidate each tutorial into one workspace as well.

Not relevant after https://github.com/project-serum/anchor/pull/990?

looks like the original issue was closed so Im closing this too. Will reopen if necessary


Add scalaModules for scalaVersion 0.x.

scala-xml + parser combinators should be optionally added
when scalaVersion >= 2.11 or 0.x (Dotty)
PS. The project does still not compile due a crash in the compiler
[trace] Stack trace suppressed: run last scalatest/compile:compileIncremental for the full output.
[error] (scalatest/compile:compileIncremental) java.lang.IllegalArgumentException: Could not find proxy for attempt: Int in List(val attempt, val tailLabel1701, method tryTryAgain, method futureValueImpl, class FutureConcept, class Futures, module class concurrent, module class scalatest, module class org, module class <root>), encl = method $anonfun, owners = method $anonfun, constructor $anon, anonymous class org.scalatest.exceptions.TestFailedException with
[error]   org.scalatest.exceptions.TimeoutField
[error] {...}, value tailLabel1701, method tryTryAgain, method futureValueImpl, trait FutureConcept, trait Futures, package concurrent, package scalatest, package org, package <root>; enclosures = method $anonfun, trait FutureConcept, trait Futures, package concurrent, package scalatest, package org, package <root>
[error] Total time: 37 s, completed Apr 28, 2017 4:20:34 PM

The compilation crash is expected and has been reported at https://github.com/lampepfl/dotty/issues/2233


Reporting Dashboard using Allure Reports

Why
To facilitate better test result management and enhance the debugging process by providing a visual and detailed report of test executions, which will include trends, graphs, and historical data.
What
To improve visibility and understanding of our automated test outcomes, we propose the integration of Allure Reports into our CI/CD pipeline. Allure Reports will provide a rich and insightful dashboard to display detailed test results, making it easier for developers and QA to analyze test executions and identify issues quickly.
How

[x] Install Allure Report generation tools in our test environment.
[x] Ensure compatibility and integration with our existing test frameworks (frontend/backend)
[x] Modify test scripts or CI/CD pipeline configurations to generate Allure result files during test runs.
[x] Set up a server to host Allure dashboards
[x] Customize the Allure dashboard to reflect our specific reporting needs, such as test severity, categories, and tags.
[x] Integrate Allure reporting into the CI/CD pipeline to automatically update reports with each build or on a scheduled basis.

I think we can probably close this one, the dashboard looks good.

@Ryun1 is this PR configured?

I have granted both @mesudip + @NabinKawan admin, please feel free to add the secrets

Reports accessible, under current break down. Works smooth.

Waiting to have all tests included in the dashboard, then we plan to close this ticket.

Here is the link for dashboard: allure-report

Can this be closed out @spannercode ?

@Ryun1 Yes, it can be closed.
Here is the final link for the dashboard which includes frontend, backend and web-app-boilerplate test reports.
https://intersectmbo.github.io/govtool-test-reports/


zencoder: percent-encode the destination and more improvements

Current coverage is 84.39% (diff: 76.00%)

Merging #123 into master will decrease coverage by 0.17%

@@             master       #123   diff @@
==========================================
  Files            28         28
  Lines          1860       1878    +18
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1573       1585    +12
- Misses          190        193     +3
- Partials         97        100     +3

Powered by Codecov. Last update 51369be...9170a19


[usbdev,sival] Enable silicon for usbdev_aon_wake_reset

First two commits are backports.

Should be ready for review now that the teacup backport has been done.

I have implemented usbdev_aon_wake_disconnect and added it to this PR


Allow atop to change chunk shape

Example
def double(a, axis=0):
    return np.concatenate([a, a], axis=axis)

y = atop(double, 'ij', x, 'ij', chunks={'i': lambda n: 2*n}, axis=0)

y = atop(double, 'ij', x, 'ij', chunks={'j': lambda n: 2*n}, axis=1)

cc @shoyer @jcrist

Seems fine to me. Only issue I see is that chunks is a common keyword in other functions, but means something else. May not be worthy of changing the keyword name though, I can't think of a better one.

I also like this functionality but think the chunks name is a little too generic. Based on other uses of chunks in dask, I would expect chunks to explicitly set chunks on the result.
Other options: chunks_map, chunking, rechunk, new_chunks...
I think rechunk is my favorite?

Changed chunks to rechunk

Actually, rechunk sort of implies adjusting chunk boundaries... maybe adjust_chunks?

Someone privately suggested that we also support the following inputs:

rechunk={'i': 10}: all chunks are of size 10
rechunk={'i': [10, 10, 20]}: explicitly list the chunk sizes (and check that there are three blocks in this dimension

Yes, those are similar to inputs I supported in my version.
On Thu, Oct 27, 2016 at 4:23 PM Matthew Rocklin<EMAIL_ADDRESS>wrote:

Someone privately suggested that we also support the following inputs:

rechunk={'i': 10}: all chunks are of size 10
rechunk={'i': [10, 10, 20]}: explicitly list the chunk sizes (and
check that there are three blocks in this dimension

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/1716#issuecomment-256797624, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABKS1unfLPnqbWWya4E2-bXl-JbcrKJYks5q4TKJgaJpZM4KiuEi
.

Implemented int and tuple forms of rechunk.
Waiting a bit on renaming to adjust_chunks in case any other names come up

Renamed to adjust_chunks


Make sure to schedule connect event on correct thread type.

Cannot blindly schedule on current thread.  It may not be the right type.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1397/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1291/ for details.

@zwoop Yes, this needs to get backported.  I ran into this issue testing the 7.1.x branch in production.


[LaTeX] Add *.lol to ignore list

The LaTeX listings package will generate *.lol files when building (list of listings). These should be added to the ignore list for LaTeX.
(I'd make a PR myself but am unfamiliar with node projects.)

Allow me to teach you how to fish -- No nodejs skills needed.
All of the data for this site is in the ./data directory.  This directory is broken down into 4 sections

classes - These are gitingore templates that cover a class of files like Images, Videos, Audio
custom - These are gitignore templates that aren't currently supported by GitHub
gitignore - This is a submodule to GitHub's master list of templates
patch - These patch files can be happened to any templates in the other directors to amend their configuration.  For example if you want to add a change to something on GitHub's master template list.

What you want to do is edit the ./data/custom/LaTeX.gitignore and add your suggestion to the end of the file.  If you submit a PR with that, I can merge it.

Ah so it's that simple! I'll make a PR with it later today.
On Mon, 23 May 2016, 17:53 Joe Blau<EMAIL_ADDRESS>wrote:

Allow me to teach you how to fish -- No nodejs skills needed.
All of the data for this site is in the ./data
https://github.com/joeblau/gitignore.io/tree/master/data directory.
This directory is broken down into 4 sections

classes - These are gitingore templates that cover a class of files
like Images, Videos, Audio
custom - These are gitignore templates that aren't currently
supported by GitHub
gitignore - This is a submodule to GitHub's master list of templates
patch - These patch files can be happened to any templates in the
other directors to amend their configuration. For example if you want to
add a change to something on GitHub's master template list.

What you want to do is edit the ./data/custom/LaTeX.gitignore and add
your suggestion to the end of the file. If you submit a PR with that, I can
merge it.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/joeblau/gitignore.io/issues/211#issuecomment-221016286


[Temporary] Swallow error uploading structured data

Description
Temporary swallow of "Error uploading structured data" for the day They add too much noise and make the eng runner job hell Those Errors will be properly handled by #5695 and this PR will be reverted by EOD
Of course, any real issue there would be caught by the activity monitor anyways
Risk

We are confident those errors do not directly affect production, so a temporary swallow is ok
This is temporary and will be reverted by EOD
I will take responsibility for it

PMRR


Implement Rng::fill

Fills a byte slice with random data.
It's too bad there's no generic gen function, but I think in most cases we want to fill byte slices.

Ah, yes indeed possible, however after trying it out, the benchmark doesn't read any performance difference, so I think the generated code is the same:
    /// Fill a byte slice with random data.
    #[inline]
    pub fn fill(&self, slice: &mut [u8]) {
        // Filling the buffer in chunks of 8 is much faster.
        let mut chunks = slice.chunks_exact_mut(8);
        for items in chunks.by_ref() {
            let r = self.gen_u64();
            let sl: &mut [u8; 8] = items.try_into().unwrap();

            *sl = r.to_ne_bytes();
        }

        for item in chunks.into_remainder() {
            *item = self.u8(..);
        }
    }

What do you think? The only nit with the above code is the unwrap()  I guess.

I'm not sure if the #[inline] tagging of the function is appropriate, given that it is relatively large. Can we get an LLVM IR of it?

I'm not sure if the #[inline] tagging of the function is appropriate, given that it is relatively large. Can we get an LLVM IR of it?

Yeah I agree, I had it there originally when the function was small and unoptimized, but I would remove it now. For the record, the benchmark isn't affected when #[inline] is removed.

#[inline] now removed.
Don't have time to link into the LLVM stuff, sorry.


RedisSessionStore + Rails 3.1 => Cannot logout

Hi, I'm using version redis-store-<IP_ADDRESS> to combine the RedisSessionStore and rails_warden in an rails 3.1 app. The problem is that I can not log out. When switching to the standard CookieStore everything works fine.
Perhaps it helps: I could track the problem down to somewhere in Rack::Session::Redis. set_session / merge_sessions does effectively not remove the key warden.user.default.key.
Any ideas? What helpful information can I provide you?

I'm experiencing the same problem here, pls fix this.

I will check soon.
Thanks

Hey, I just submitted a pull request that fixed this issue for me, though I didn't know how to write the test.  Sorry for double posting it here, I wasn't sure the best way to get the issue and pull request to reference each other?
https://github.com/jodosha/redis-store/pull/95

@faber, thank you I will check this weekend.

I'm getting a lot of "Bad old or new sessions provided." messages from the merge_sessions method too. It seems like new is set to 0 when I call request.reset_session. Any chance this could get merged?


Main Spec on Guildwindow not reflecting current spec

Hi,
Im guildmaster and asked guildies to install Guildbook, and we could see on the new bigger Guild Window the column "Main Spec" and even the professions.
Today a guildie told he respecdd from Restoration (shaman) into Dps (either Elemental or Enchancement), but Guild window column still showing "Restoration", instead of showing "Elemental" or "Enhancement".
Is the Main Spec supposed to be automatic refresh, do does it need some manual clicks ?
FYI: I tried opened Roster, clicked guildie chars and clicked Reset, but still mains spec remain old one. As side-effect 2 other guildies now  got their mains spec and professions cleared out.
I really would like to use this addon for guild management, but i keep bouncing into basic foundational problems that will make this addon unusable as will be hard to keep explaining workarounds to each new problem each new guildie encounters.
If addon not rock solid then guildies will not use it.. and im already started finding resistence.
So, look forward to understand the "Main Spec changed/refresh" issue solution.
Thanks.
~Budha~

RESPET BROADAST: Can addon detect talents change finished (number of free talents points to assign = zero) and then broadcast main spec to other guildies using guildbook ?
Whats best place for me to request this enhancement, as im having hardtime having guildies to adhere to using Guildbook, since it needs too much manual a actions for them to syn up thing, namely: recipes and mainspec.

The addon will check what points have been spent in which tree and where a guild member HASN'T set a spec, the addon will take a guess based on which tree has most points.
However, as with classic, TBC doesnt have a strict class>spec system (unlike retail). There are specs that dont fit points per tree and so character spec has been left as a player option. Personally opening your profile and selecting from a dropdown isnt a taxing requirement (as spec doesnt change often)
I could add a check box and option to ONLY use the spec which has the most points.
When/if we get a classic wrath then its much easier to say the tree with most points is going to be the characters spec.
Warcraft logs (iirc) has recently added new specs such as shaman tank (warden) and a couple warrior options i think.
Also, as a player levels up they may show with an incorrect spec if say a priest takes wand spec but then goes into shadow.
Its a tricky area tbh.

Recipes will sync over time as players open their profession windows, there is the option to share a 'snapshot' of your db via the import/export interface.

Let me be more clear:

Theres major problems with guildwindow/roster not showing to others main
spec and/or recipes.
we keep loosing hours in guild tyring to workaround why one player sets
mainspec on his profile, and all other guildies dont see mainspec on
guildwindo.

ITs getting really frustrating this.
Thanks//Kind Regards,
Emanuel Oliveira
On Wed, 8 Sept 2021 at 16:36, stpain @.***> wrote:

Recipes will sync over time as players open their profession windows,
there is the option to share a 'snapshot' of your db via the import/export
interface.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/53#issuecomment-915347081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDBGOH42FQG35EWYDMTUA57GZANCNFSM5CZWCQ2Q
.

does everyone have the latest version?
i am going to have to try and replicate this issue as its not something ive seen before.
have you tried to set/change specs with the debug window open to see if any comms happen?
i am working on adding an option to let the addon set your main spec, but if you're saying no comms come through then there is a different cause for your issues.

I never used debug option, but going try. As myself software engineer shall
not be hard enabling debug and try confirm why no comms.
Thanks and Kind regards,
Emanuel Oliveira
Senior Data Engineer
On Thu, Sep 9, 2021, 01:32 stpain @.***> wrote:

does everyone have the latest version?
i am going to have to try and replicate this issue as its not something
ive seen before.
have you tried to set/change specs with the debug window open to see if
any comms happen?
i am working on adding an option to let the addon set your main spec, but
if you're saying no comms come through then there is a different cause
for your issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/53#issuecomment-915665312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDAZVRBRNEYMXLXVFOLUA76CZANCNFSM5CZWCQ2Q
.

so you should see something like this, this is from the latest version on a character i ccreated earlier this evening.

Awesome, thanks, kook forward to test/debug.
btw:
How can i create an animated gif like that of yours?
And deems ots hosted on url Google content ? i wonder whats this
service/how to upload gif? i got google drive but unsure if related 😊
Thanks and Kind regards,
Emanuel Oliveira
Senior Data Engineer
On Thu, Sep 9, 2021, 01:59 stpain @.***> wrote:

[image: mainspec-comms]
https://user-images.githubusercontent.com/47498931/132604903-220e86b8-8cdf-4c22-b486-99a3b68ee985.gif
so you should see something like this, this is from the latest version on
a character i ccreated earlier this evening.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/53#issuecomment-915676809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDGR3QT7PTMT5XWY5QDUBABHPANCNFSM5CZWCQ2Q
.

its an app/program called ScreenToGif, then just drag and drop the image file here, probably has a size limit though

Thanks, I'll check it out, and create a similar gif showing 2 debugs:

me changing main spec and watching comms
will ask guildie to do same and see on my side if i see something coming.

Makes sense these 2 tests i guess.
Thanks and Kind regards,
Emanuel Oliveira
Senior Data Engineer
On Thu, Sep 9, 2021, 02:06 stpain @.***> wrote:

its an app/program called ScreenToGif, then just drag and drop the image
file here, probably has a size limit though
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/53#issuecomment-915679671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDHKC3ZJJFKTQ6HXFDLUBACBFANCNFSM5CZWCQ2Q
.


feat: clean up and update dependency

This PR is to standardize the package.json on each project to be the same / similar as Metamask's Snap
What did we updated:

update npmrc to use lts/* rather than hard code as 16
Not necessary to using patch version of yarn (yarn-berry), instead we can use yarn 3.2.1
remove metamask/rpc-method, as it has migrate into metamask/sdk
move allows-script cmd to root level for cleaning code, as it seem redundant
remove release.sh , as we cant release from local anyway
remove some un-used dependencies

Still need to check workflow with get-starknet.

@khanti42 verified the get-starknet is working too


show messages

Show Django messages in API browser.
If possible, vary style depending on severity.

Won't do.


created profile setup

I have created the routes component and also added styling to the visited navlinks. Please feel free to review and drop your feedbacks where necessary

[# STATUS: APPROVED :green_circle: :clap:
Hello  @AyoMoses1 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit: aa
Congratulations! 🎉
HIGHLIGHTS:

[x]  created profile setup

Now you can merge this branch and move on :1st_place_medal:
Happy Coding cheers 🥇 💯

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.]


SDK parameters should match API parameters

I lost an hour debugging an issue last night because I was passing the parameters from your api docs into the transmission::send() method.

Thanks for the feedback @andrew-pause - I'm going to link and close this out as a dupe of #67 - we'll be making some adjustments to the docs to make this more clear. We'll potentially look at renaming the params in a later release as well.