Feature Request: Overwrite coverage report comment when a test fails

Problem
When a commit causes a previously passing PR to fail a test, the old coverage comment remains. Since the default philosophy of the action seems to be to maintain and revise the coverage comment with the results of the latest run, it seems like it would make more sense to overwrite the comment with a message, something like:

Coverage report not available due to failing test(s).

Not a big deal, but I think this would be a better developer experience than leaving the stale comment.

Hello @blordpluto :wave:,
Could you please attach your action.yml file and screenshots of comments and action's console? Or a link to a PR? Looks like a bug to me, action should overwrite the report comment by default, if tests fail.

hello @ArtiomTr , not sure if I'm seeing exactly the same as described above, but I'm getting a new set of comments every time new commits are pushed to my PR. I've copied the job from our actions.yml as well as screenshots of the last 4 pushes that have each ended up with comments. Our actions run on push if the event matters.
The repo is private so I can't share it, but if you need more information please let me know.
This is the job in my actions.yml, we have a lerna monorepo and I have a matrix set up to run the tests in parallel in each package, it runs the tests and provides one comment for each package:
run_tests:
    name: Run Jest Tests
    runs-on: ubuntu-18.04
    strategy:
      fail-fast: false
      matrix:
        package-name: [package1, package2, package3]
    steps:
      - name: Checkout Repo
        uses: actions/checkout@v2
        with:
          fetch-depth: 0
      - name: Setup Dependencies. // this just sets up node, yarn, installs packages and caches node_modules
        uses: ./.github/actions/setup
        with:
          npm-auth-token: ${{ secrets.NPM_AUTH_TOKEN }}
      - name: Run Tests and Comment
        uses: ArtiomTr/jest-coverage-report-action@v2
        with:
          working-directory: packages/${{ matrix.package-name }}
          test-script: yarn run test:ci
          package-manager: yarn
          skip-step: install
          annotations: failed-tests

below are screenshots from my last 4 pushes for one of the packages, (but each package is generating its own comment which is causing the PR to get really spammy):

after looking at the source code I can see that since we are using the push event the action is not attempting to find the existing comments since it doesn't have access to the PR number to look them up with
would it be possible to allow passing a pr-number optional input that I could pass in after grabbing it from something like gh-find-current-pr? Then if that is populated for a non isInPR type it could still try to look up the comment and update instead of posting a new one in generateCommitReport

Hello @dalevfenton :wave:,
This action works with two event types: pull_request and push. pull_request is a standard way of handling things - this event is emitted each time when PR is opened or receives a new commit. push event is useful, when you don't have a PR and commit directly into the branch (hotfix for example). In that case, the comment is attached to the commit.
Could you please clarify, why you're using a push event when you have a PR? I don't have deep knowledge of GitHub APIs, but I have always been using a pull_request event, and everything was working as intended, without additional configurations.


Jw35 reimport remaining widgets

Update or create all remaining widgets from the development lobby_screens repository, adapting them for this framework as necessary. Includes a few fixes to the framework itself. All widgets are working, though in some cases with limitations that are mentioned in the commits and recorded as separate issues.
Note this PR is against the jw35-reimport-widgets branch, which is itself against dashboards. That's because jw35-reimport-widgets mainly contains changes to the framework, and jw35-reimport-remaining-widgets is mainly about widget code and I don't want to confuse the two. This does mean some care is needed to merge all the changes cleanly into dashboards.

conflicts need to be resolved after changes in the dashboards branch

Will do  - I was hoping I'd managed to avoid conflicts, but clearly not...

Conflicts fixed.


Adding CODE_OF_CONDUCT .md file

adding Guidelines for the contributors
add GSOC ext and hacktoberfest label to it

Okay @devxMani Assigning to you, have you completed the previously assigned issues?


⚠ Jenkins Plugins Artifactory has degraded performance

In ca37881, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1221 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 855e7fa after 25 minutes.


fixes for capi

These changes make the capi usable.  Since the interface directory is not available outside the project it can not be used.

Can you describe what is your change on the commit message?

Hi @jsorg71
Do you think it's ok to let dongping handle this patch and keep you in commit's signed-off like
Signed-off-by: Zhong Xiaoxia<EMAIL_ADDRESS>in this commit
https://github.com/01org/libyami/commit/f6f1483a62dfd1424483ecb24ec9305ea96c2093

@xuguangxin np, don't forget about the YamiVersion fix too.  Without that, I can't get the version number from C app.

ok, @jsorg71 thanks,

close this since #522 already have this commit
thanks @jsorg71


New GameService model

Back to XNA-like game components and services.

Multithreading (like GameServer)
Rendering (?)
Configuration
Initialization order
Nested services

Make public classes in namespace Fusion.Driver private.

Service model will be introduced later.
Now engine structure is static.
0fc683c2ed696435d919c3595f1afeb24debc9f4


Scripts installed only when an extra is installed

in [project.scripts]
is it possible to specify  to install a script only when an extra dependency is installed?
If so can that be included in the examples in the readme?

No, that's not supported, sorry.
Entry points (including scripts) tagged with extras are not recommended, according to the entry points spec. Part of the reason for this is that there's no separate record of whether an extra is installed or not - if you install all the dependencies for an extra separately, tools can consider that extra to be installed, but scripts conditional on that extra may not be installed.


Allow custom path for the buf binary

If the buf binary is vendored, it won't be available through PATH. Let us set its location manually in the tools preferences pane

This is coming in the next release of the plugin, but there is another fix we'd also like to include.

This feature is now available in v0.1.4 of the plugin.


Combobox : Adding sub menu header and option to set width to width of menu

Fixes issue #1267 @interactivellama
Hey guys! This is my first time contribution to DSR.
Additional description
In this PR:

Adding support for menu with subheaders.
(As in SLDS: https://lightningdesignsystem.com/components/menus/#With-Subheaders)

Currently the combobox only allows setting the width of the dialog to the width of the target. In Lightning Reports, we have a use case where we want the width of the dialog to be the width of the largest menu item, up to a maxWidth.

And here's how it would look if a menu item is smaller than the target element:

Pull Request Review checklist (do not remove)

[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at snapshot strings.
[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

At first glance, this is looking great! Could you add an example with a non-heading separator (line)? I didn't see one. It can be included in your existing example.

I added a new example for menu with separators (without subheaders)

@interactivellama This PR should be ready now - I fixed an NPE and some lint errors. Thanks!

Great first time contribution, btw!

Oh, and on more thing. I don't think there a snapshot test on the maxWidth prop nor the inheritWidthOf. The menu will need to be open.

@interactivellama I just pushed an update to address your feedback. Thanks for code reviewing it ! and let me know if we are good to merge.

@interactivellama Any chance we can merge this?


terraform: update the bucket policies

Formatted with jq
nix-releases inherits the policy from nix-cache
should fix #70

merging as it has been already applied

Shouldn't the bucket policies be removed from nixos-org/network.nix? Now they're managed in two places.

yes good point. https://github.com/NixOS/nixos-org-configurations/issues/73


Anonymous ip and connection type database added

I really dont know whats going on. I've signed the contributoer agreement. Besided that, the travis-ci seems it is not working well

Make sure the email address you gave when you signed the CLA matches the user.email your commits are signed with. That caught me up, once I amended the commits with the right email address the CLA check passed.
The Travis seems broken, I mentioned it two months ago on my PR but nobody has responded.

Make sure the email address you gave when you signed the CLA matches the user.email your commits are signed with. That caught me up, once I amended the commits with the right email address the CLA check passed.
The Travis seems broken, I mentioned it two months ago on my PR but nobody has responded.

Thank you @Sitwon . I have needed replace my last commits in order to change the user email.


Make istioctl pc route less verbose for regex match

Fixes #34344.
Makes it so that we don't print the regex engine type on isitoctl pc route. Does not implement the smarter prefix recognition.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@Monkeyanator: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
d50d76d6a247705adfb96da6b13cd58ed1a0ca47
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Discord restart messages in game doesn't work

I have the discord bot on and scheduled restarts on. When a restart has been scheduled, this was neatly placed in the in-game chat in the previous version. Now only "txAdmin" appears and if I do an Admin Broadcast, my username will be there.

Sorry, i'm not sure if I understood it correctly, what does this have to do with the discord bot?
Are you using the default chat?
i'm not seeing whats wrong, can you please elaborate or join our discord?

Hey Tabarra,
Thanks for the quick response!
I do indeed use the standard chat, but I made an adjustment that you have to type with a command. So /ooc /tweet /say /job and so on.
The previous txAdmin version had no problems placing the restart reminder in-game when the server started restarting (the 30, 15, 10, 5, 4, 3, 2, and 1 minute notifications).
Now that the update has been done from txAdmin, this is reflected in the console as
txaBroadcast "txAdmin" "Wensveen Roleplay is scheduled to restart in 15 minutes."
[txAdminClient] Admin Broadcast: txAdmin
In-game you will only see "txAdmin". This used to be "Wensveen Roleplay is scheduled to restart in 15 minutes."
If I want to do an admin broadcast via the dashboard with, for example: We're going to restart the game.
Then there will be in-game: AdminMike (my username).
No further message is displayed.
The only thing I have changed is the old version of txAdmin and the new version put back. So there has been an adjustment in txAdmin so that it no longer works for custom chat scripts.

Ohh okay I see whats happening.
I added a new parameter for the name of the admin doing the broadcast, and totally forgot to change the compat version limiter for the resource.
Just delete the txAdminClient resource and restart the server inside txAdmin and it will work fine (it will be reinjected).

Hello,
I did exactly what you asked, but still getting the same response.
Do you maybe have the old txAdminClient from the past version?

Did you managed to fix it?
I'm closing this issue, if you still need support please check our discord server (link in the README).


Introduction to PHP Operators

Proposed title of article
Introduction to PHP Operators
Introduction paragraph (2-3 paragraphs):
PHP Operator is a symbol i.e used to perform operations on operands. In simple words, operators are used to perform operations on variables or values. For example:
$num=10+20;//+ is the operator and 10,20 are operands
In PHP, we have different types of operators depending on the number of operands they take. There are some operators that take only one operand and perform an operation on it. This type of operators are called unary operators. Another type of operators are binary operators which take two operands.
Key takeaways:

Instaling PHP and Xampp in Linux
Define what is an operator and an operand
Learn the roles of different operators
Category of PHP Operators
Use Operators to perform mathmatical computation

References:
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@Serahmusoo Just wanted to make sure this topic wont overlap too much with this past article:
https://www.section.io/engineering-education/introduction-to-php/

@hectorkambow thanks for pinging me. This article is different from the published one. In the published article, the author did not discuss on operators. In this article, my attention is to equip the PHP beginners with knowledge on how to use different types of PHP Operators to accomplish their codding task. Anything similar(which I don't expect) will be referenced.
Please Ping me if there is anything else.  Thank you

👍  Understood, thank you for the clarification - approved


Automatic viewport following causing interaction issues during manipulation.

Interacting with the viewport while it is trying to follow the user results in jittery movement of the viewport.
The automatic viewport following option should be disabled while the user is actively manipulating the viewport object.

Made obsolete by transition to Unity XR Toolkit


{WIP} Rewrite isTuple to be more efficient and workaround back-end segfault

Targeting stable to be in the next point release.
This works around a compiler bug when compiling with -allinst

@NilsLankila See https://issues.dlang.org/show_bug.cgi?id=21017

A static assert within a of unit-test in std.algorithm.searching is failing.
I think the condition could have been negated by accident.
@nordlow Can you remember for what actual use-case you changed this back in 2015?

Started dusmite, will add a reduced test case once it finishes.

@UplinkCoder, change the commit message to "workaround issue ... - " so that it will not be auto closed, and assuming we will get a reduced case.

@NilsLankila reduced version:
https://issues.dlang.org/show_bug.cgi?id=21018

@MoonlightSentinel you can stop dustmiting.

thanks. I've reverted 21017 to a "phobos" issue. From the side of the issues everything is formally correct now.

@UplinkCoder Are you still working on this?

Nope closing.


Update events.rst

https://github.com/doctrine/doctrine2/pull/169

To give some context, it is indeed a lifecycle event: https://github.com/doctrine/doctrine2/blob/cd11723e6310b5973f50e7d9819c58916721d290/lib/Doctrine/ORM/UnitOfWork.php#L575-L579

@alexndlm thanks!


[chores] Centralized ProtectedApiMixin logic #337

Closes #337

Coverage increased (+0.003%) to 97.807% when pulling f2b2bc54a9443926dddb5e23ba8a77e37a8b4a97 on issue-337/centralize-protected-api-mixin into 339bb9f7688d857c141c023a14acc2f04a267a44 on master.


[GCP] Export a custom image to Cloud Storage

Description
Create an Issue for GCP to transform the image to Bucket
Currently, the packer script will create an image of the release. The goal is to add the missing part who is convert it to a bucket to allow users to use it.
See the GCP doc on it: export a custom image to Cloud Storage

I'm not sure why we need this @alallema, can you give me more info?

If you follow the documentation for deploying a Meilisearch image on GCP, you'll need the vmdk address.
In doing so, our GCP script only creates one image and does not go as far as the step that transforms this image into a public vmdk for contributors.

can confirm, the instructions in the gcp cookbook just don't work
https://www.meilisearch.com/docs/learn/cookbooks/gcp


Buffer Restore: Fix serializing >32KiB

I forgot to buffer.clear() after a write. Whoops.
This includes 2 additional, smaller improvements that I just happened
to notice: The GenRTF code calls to_string despite using fmt.

Yup exactly. That's how you end up with a 62GB buffer.txt. 😄


Add context creation to label

As discussed, adding different context creation then plain hall. Npw we have 6 context creation methods:

hall, same as before
last, we only take the last ref
max/mean/median, we take the max/mean/median count for each file across all refs
concat, we concatenate the delta docs

I had to modify a bit the data classes for the mapping. I also stumbled on some bugs in create_bow which I fixed - it took me a long time to find the origin, although there is not much chabged T.t

Okay so in the end I did not change the logic for converting diff to hall, then hall to context, however to avoid duplication and provide more readable code, I created multiple functions to encapsulate everything. Now:

Since both conversions are a corpus reduction and shared code up until when we process each files, I created a reduce_corpus function which takes as parameters the corpus to be reduced, the logger, the repo_mapping, refs dict and a FileReducer function, for chich I added typing.
I created a FileReducer function for each context, as well as for the conversion from diff to hall
I created an Enum object for context handling

This PR also contains an update to the README, the bugfix mentioned previously, and a commit on top of previous changes for label where I simply add error handling.
I tested the code, everything works, so I will merge this.


Fix JSON example on webhooks page

Proposed changes
Correct the JSON example on the Webhooks page. The comment_count parameter in the JSON body is a number, not a string.

@lilybguo can you confirm?

@joaofnfernandes I found this bug during one of the testing sessions where we used testkit clusters. I don't know how to load unmerged code onto a cluster and verify. can someone else verify it?

@lilybguo I believe this about hub webhook documentation. Did you misunderstand to be something else?

@manishtomar Ha I'm sorry, I was also looking at an orca bug, confused the two.
I don't have any context on this bug. The fix looks good, but I don't think I'm the right person to confirm

Thanks @manishtomar. Let's ship it!


[Fix,Conda] update conda download url

What: a small change to update conda download url.
Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

@tqchen Please take a look at it, thanks!

May I ask what the difference between the urls? Thanks!

May I ask what the difference is between the urls? Thanks!

The domain is different. Changing from repo.continuum.io to repo.anaconda.com.
Docker build fails with unable to download the miniconda installer with the current url https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh .
Changing to the conda's current url, https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh , succeeds.

Thanks @leandron @junrushao1994 @shibuiwilliam !

Thanks!


Indent Lua with 4 spaces, formatting XML files -- !!View Files Changed With "Hide Whitespace Changes" (in settings)!!

!! Do not complete this PR until the other PR (#78, documented in issue #77) is complete.  That PR is very simple - only a few characters !!
This updates all the indents to in lua 4 spaces and formats the XML files (removes the large horizontal spacing).
Make sure you select "ignore whitespace changes" when looking at the Files Changed section... or you'll have a lot of scrolling to do:

I believe this is all of the files in CQUI that are not pulled from other mods (integrations).  I did not format text XML documents.
Handy way I turned the 2-space formatting into 4-spaces, using find/replace with the Regex setting:
Find: ^(  ){1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}(  ){0,1}
Replace $1$1$2$2$3$3$4$4$5$5$6$6$7$7$8$8$9$9
Run it a single time on a file that has yet to be updated and it'll turn the 2 spaces into 4

Aren't you afraid that too many changes will cause eventually merge conflicts and you would need to resolve them manually, resulting in the double work?

If I keep pulling in the current changes from main branch it shouldn't.  This is also like tinkering, so I don't mind the work.
I will likely change this to a real PR - with whatever files I have changed so far - before the end of the weekend, once I make sure the issues with alignment are resolved, so the diff can be easily seen
I may just create a new branch and do PRs with max of a few files at a time if preferred.  The only diff will be the spacing

aaah!! there is an "ignore whitespace changes" - hiding under the diff settings.  So much better!

What kind of review do you expect us to do? 'Cos I don't think it makes sense to check every file and see if you really formatted it neatly. Unless someone wants to do so...
I am going to get this branch and load several games to see if all is working as expected. If nothing will break, then I can give it a go. If you want anything more or someone wants to do something more, then please say so.

If you set the ignore whitespace changes, then the site should just show a list of files that say their only changes were in whitespaces... save for the change I made to cqui_main.lua (version number) and the contributing markdown doc.  Basically verify what I see when I am looking at it.
Loading up a game or two and playing a bit is basically what I have been doing to test other changes.
If you complete the PR please do a squash merge since there's like 900 changes with these files (as part of the effort to ensure the PR would show as horizontal space changes only)

Merged the changes from master branch into here, just to verify that all files (save for the 2 mentioned above) now show like this when the ignore whitespace changes is set:

Tested. OK.

Thanks!


nvim_echo is nil value

I'm getting:
E5108: Error executing lua /home/mc/.vim/plugged/lightspeed.nvim/lua/lightspeed.lua:35: attempt to call field 'nvim_echo' (a nil value) when I try hitting s after installing via Plug.
I've pulled the very latest Neovim nightly (Ubuntu) on two different machines:
> apt policy neovim
neovim:
  Installed: 0.5.0+ubuntu2+git202106150022-b28d458f8-d569569c9-e70091b15~ubuntu18.04.1
  Candidate: 0.5.0+ubuntu2+git202106150022-b28d458f8-d569569c9-e70091b15~ubuntu18.04.1
  Version table:
 *** 0.5.0+ubuntu2+git202106150022-b28d458f8-d569569c9-e70091b15~ubuntu18.04.1 500
        500 http://ppa.launchpad.net/neovim-ppa/unstable/ubuntu bionic/main amd64 Packages
        100 /var/lib/dpkg/status
     0.4.4-1~ubuntu18.04.1~ppa1 500
        500 http://ppa.launchpad.net/neovim-ppa/stable/ubuntu bionic/main amd64 Packages
     0.2.2-3 500
        500 http://au.archive.ubuntu.com/ubuntu bionic/universe amd64 Packages

I've also tried a minimal vimrc file with only this Plug install and nothing else - same error.
This looks like a Neovim / Lua API problem - It was my understanding that Lua was now bundled with Neovim. Is there some way to check I have a compatible version of Lua compiled into Neovim correctly?

Update - this does seem to work with the nightly AppImage for some reason but not the nightly PPA.
Problem is probably on the NeoVim distribution rather than this plugin.

Alright, after more investigation it looks like the Ubuntu PPA is waaaay behind the actual source build.
Closing this.


[BugFix] Support storage_cooldown_ttl = ""

Fixes #29841
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 18 / 19 (94.74%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/TableProperty.java
3
4
75.00%
[515]

:large_blue_circle:
com/starrocks/alter/AlterJobMgr.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ast/SinglePartitionDesc.java
11
11
100.00%
[]

:large_blue_circle:
com/starrocks/common/util/PropertyAnalyzer.java
2
2
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


communityToolkit.MVVM does not generate source code for classes inheriting ObservableObject

Describe the bug
Can someone please have a look at this issue?
https://github.com/CommunityToolkit/dotnet/issues/623#issue-1602180394
Steps to reproduce the bug

Create a new Windows App SDK / WinUI 3 Packaged desktop app project
Install the ComunityToolkit.MVVM Nuget package, ver. 8.1.0
Create a class and inherit ObservableObject
   public partial class MyClassViewModel : ObservableObject     {         [ObservableProperty]         string customerName;     }
Build the project and go to Dependencies > Analyzers > CommunityToolkit.Mvvm.SourceGenerators > CommunityToolkit.Mvvm.SourceGenerators.ObservableObjectGenerator

Expected behavior
No response
Screenshots
No response
NuGet package version
Windows App SDK 1.2.4: 1.2.230217.4
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022-preview
Additional context
No response

Looking into the linked issue, it seems like this is the root issue causing this error. Closing as external: https://github.com/dotnet/roslyn/issues/67123


feat(viewport): add viewport name to life cycle arguments

Consider the following scenario:
The same view-model can be reused in different view ports (for example in a two-column layout). The rendering might slightly differ depending on which view port is it being rendered in. How should the view-model access this information?
Added viewport's name to config object being sent to activate / deactivate

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


Looks amazing

Hello David, i haven't started using your plugin yet but it looks fantastic. Do you have a live demo i can see in action?
Eventually i would love to use this to create a company structure that is dynamic based on people's job titles and hierarchy.

Hello @nicolasvahidzein , I will provide it.

Oh hello again :) Thank you buddy.

Just checking in to see if you had time for the demo link. Thank you David.

David?

Hello @nicolasvahidzein , I am currently dedicating my free time to the Neutronium project. I don't know yet exactly when I will have free time to work on Vue.D3.tree.

Hello david, i totally understand but there must have been a misunderstanding, i'm just talking about something simple like a jsfiddle. Not a website you need to setup or anything. Just a simple use case.

Added Live demo:
https://david-desmaisons.github.io/Vue.D3.tree/tree.html
https://david-desmaisons.github.io/Vue.D3.tree/hierarchicalEdgeBundling.html


Add Google Tag Manager

Add Google Tag Manager

@chrissnyder Yes, it's pure javascript, the DOM is somewhat irrelevant. it captures standard analytics page loads etc, and can be used as a platform to add custom JS events etc to track (as we have done with Seregenti).


Stop rewriting static libs

As per Ian's suggestion here, this PR makes it so -l:lib.+\.a$ are not rewritten, and instead passed as link-arg.
That seems to work in wasmtime-rb as per my rb-sys-dock experiments. I haven't tried the full rb-sys in CI yet with the crate changes.

Thanks for publishing a new version. wasmtime-rb's build gem step now passes 🎉 !


Update README.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 46565a826670ad3b30b879f9f234d536dab28bd6
https://deploy-preview-7576--docsdocker.netlify.com


Handle inconsistent data values

Description
As a developer, I want to maintain the data using  our controlled vocabulary.
Example: handle "not specified", "tumor types" etc.
Acceptance criteria

[x] check the templates with inconsistent vocabulary/typos
[x] update the templates

done


Proteogenomics: fix workflow folder names

workflow folder was missing an s at the end

Thanks @shiltemann for fixing this!


HHH-9802 : Allow running hibernate-infispan unit tests using Infinisp…

…an 7.2.1.Final as a run-time dependency
To run unit tests with Infinispan 7.2.1Final as run-time dependency with a 7.2 configuration:
cd hibernate-infinispan
../gradlew -PuseInfinispan7ForTesting=true -Dhibernate.cache.infinispan.cfg=src/test/resources/infinispan-7-configs.xml cleanTest test
Note that this currently causes several failure in InfinispanRegionFactoryTestCase due to NoSuchMethodError.

Pushed to 4.3 branch.


Add files via upload

Add Virginia's Community Colleges

@jackjc I am sorry, the domain vccs.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


DATAJDBC-373 - Integration Mybatis with spring-data-jdbc,make mybatis can mixed with…

Integration Mybatis with spring-data-jdbc,make mybatis can mixed with spring-data-jdbc in ONE repository by annotation with @MybatisQuery
And also Integration tests provided.
Please check out if this way is feasible ？
In my case we currently use mybatis for data access， and we are considering to migrate to spring-data-jdbc， but we have a lot of complex SQL which organized by Mybatis.
Considering the migration cost, how about use mybaitis and spring-data-jdbc together, so we can benefit form

the simplicity provided form spring-data-jdbc
another complex SQL solution provided by mybatis  (which now only provided by QueryDsl)

@neil4dong Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@neil4dong Thank you for signing the Contributor License Agreement!

Please create an issue at https://jira.spring.io/browse/DATAJDBC so we can discuss the goal of the PR first.

Issue was created. Adding here the link the ease tracking https://jira.spring.io/projects/DATAJDBC/issues/DATAJDBC-373

At a first very cursory glance this looks reasonable.
But I think we should treet MyBatis as a source for named queries.
I.e. by default we should try to find a MyBatis query based on the name of the method and alternatively offer the option to supply a name in the @Query annotation.
There is already work in progress for something similar based on property files (#180).
I'll will first try to get that one onto the main branch before giving a proper review to this issue with proper instructions how to proceed.
One minor thing until then: It seems like formatting got changed for JdbcQueryLookupStrategy.java. Please make sure you use the proper formatter. See https://github.com/spring-projects/spring-data-build/blob/master/CONTRIBUTING.adoc#quickstart

Since CrudRepository supports saveAll by default, findById I think spring data jdbc can provide functions like spring data jpa, single-table fast CRUD and so on, only support single table can, associated table does not process.

Hi, schauder , a year have passed.     I check the feture "propety file based NamedQuery", If i'm correct ,it just move sql to a property file.
If I need to build dynamic sql,  is there a way to do so without using JPA (only use spring-data-jdbc) In one repository ？
I also checked document https://docs.spring.io/spring-data/data-jdbc/docs/2.0.0.RELEASE/reference/html/#repositories.custom-implementations
It says I can mix springdatajdbc repository with customized repository，I tried to make a demo , but it doesn't work .  Is this feature implemented?
looking forward your apply ,thanks.

Custom methods should work, yes. If they don't please create a ticket.

@neil4dong Could you rebase this PR to resolve the merge conflicts?

@schauder   yes. the Custom methods is working in newer version.
sorry,  I have not check the github notifications due to I'm  very busy recently ,  I will rebase the PR when I'm free.

And when I use Customized Method , I feel that is also a good way to integrate Mybatis Into springdata. event better.
we can discuss here.
`
import xxxxxx.xxxxx.xxxxx.MybatisConfig;
import org.apache.ibatis.annotations.Mapper;
import org.springframework.beans.BeansException;
import org.springframework.context.ApplicationContext;
import org.springframework.context.ApplicationContextAware;
import org.springframework.context.annotation.Configuration;
import org.springframework.context.annotation.Import;
import org.springframework.data.jdbc.repository.config.AbstractJdbcConfiguration;
import org.springframework.data.jdbc.repository.config.EnableJdbcRepositories;
import org.springframework.data.jdbc.repository.support.JdbcRepositoryFactoryBean;
import org.springframework.data.repository.Repository;
import org.springframework.data.repository.core.support.RepositoryComposition;
import org.springframework.lang.NonNull;

import java.io.Serializable;
import java.lang.annotation.Annotation;
import java.lang.reflect.Type;

//Mybatis load before this
@Import(MybatisConfig.class)
@Configuration
@EnableJdbcRepositories(basePackages = "com.dmall.dfp.invoicing.dao.springdatajdbc", repositoryFactoryBeanClass = SpringDataJdbcConfig.MybatisWithJdbcRepository.class)
public class SpringDataJdbcConfig extends AbstractJdbcConfiguration {

    /**
     * 集成mybatis 和 spring-data-jdbc.    混合在一个类中使用.
     * 只需要SpringDataJdbc的Repository上implement Mybatis的接口就可以路由到mybatis
     *
     * 使用此类时，需要手动Import mybatis 的配置，使mybatis先于此类加载
     */
    public static class MybatisWithJdbcRepository<T extends Repository<S, ID>, S, ID extends Serializable> extends JdbcRepositoryFactoryBean<T, S, ID> implements ApplicationContextAware {
        private ApplicationContext applicationContext;

        public static Class<? extends Annotation> annotationClass = Mapper.class;

        /**
         * Creates a new {@link MybatisWithJdbcRepository} for the given repository interface.
         *
         * @param repositoryInterface must not be {@literal null}.
         */
        protected MybatisWithJdbcRepository(Class<? extends T> repositoryInterface) {
            super(repositoryInterface);
        }

        @Override
        public void afterPropertiesSet() {
            Class<? extends T> repositoryInterface = getObjectType();
            for (Type genericInterface : repositoryInterface.getGenericInterfaces()) {
                Class<?> clazz = genericInterface instanceof Class ? ((Class) genericInterface) : null;
                if (clazz != null && clazz.isAnnotationPresent(annotationClass)) {
                    Object bean = applicationContext.getBean(clazz);
                    setRepositoryFragments(RepositoryComposition.RepositoryFragments.just(bean));
                }
            }
            super.afterPropertiesSet();
        }

        @Override
        public void setApplicationContext(@NonNull ApplicationContext applicationContext) throws BeansException {
            this.applicationContext = applicationContext;
        }
    }
}

When we use the configuration above ,   We only need to make a mybatis interface and let mybatis mechanism work, then extends the interface on our Spring Data Jdbc Repository.

Let's move the discussion to https://jira.spring.io/projects/DATAJDBC/issues/DATAJDBC-373 since it seems we are discussing the merit of the feature itself and not just that of the current PR.

I implemented such functionality by custom NamedParameterTemplate

We won't merge the PR as it currently stands.
MyBatis could be integrated as a source for named queries, as stated before. And of course formatting should be honored.


Support a subsection of the touchpad for triggering teleport

Right now clicking anywhere on the touchpad triggers the teleport. It would be ideal if a user could limit it, to say, only the upper portion so that the remainder of the touchpad may be used for other purposes.

See PR https://github.com/Flafla2/Vive-Teleporter/pull/14


Adding support for Laravel 6

Hi, just curious to know if you will have an update for laravel 6? Thanks

I have added it, thanks for pointing out.


ROBOTISのe-Manualの3.5 Bringupのエラーについて

小松製作所の上原です。
下記のようなエラーが出ます。原因わかりますでしょうか。
ubuntu@ubuntu:~$ ros2 launch turtlebot3_bringup robot.launch.py
[INFO] [launch]: All log files can be found below /home/ubuntu/.ros/log/2024-09-18-04-51-33-846365-ubuntu-1216
[INFO] [launch]: Default logging verbosity is set to INFO
urdf_file_name : turtlebot3_burger.urdf
[INFO] [robot_state_publisher-1]: process started with pid [1217]
[INFO] [ld08_driver-2]: process started with pid [1219]
[INFO] [turtlebot3_ros-3]: process started with pid [1221]
[turtlebot3_ros-3] [INFO] [1726635094.950540588] [turtlebot3_node]: Init TurtleBot3 Node Main
[turtlebot3_ros-3] [INFO] [1726635094.952329244] [turtlebot3_node]: Init DynamixelSDKWrapper
[turtlebot3_ros-3] [INFO] [1726635094.960851490] [DynamixelSDKWrapper]: Succeeded to open the port(/dev/ttyACM0)!
[turtlebot3_ros-3] [INFO] [1726635094.968288672] [DynamixelSDKWrapper]: Succeeded to change the baudrate!
[turtlebot3_ros-3] [ERROR] [1726635095.002629602] [turtlebot3_node]: Failed connection with Devices
[turtlebot3_ros-3] [INFO] [1726635095.007564164] [turtlebot3_node]: Add Motors
[turtlebot3_ros-3] [INFO] [1726635095.007742437] [turtlebot3_node]: Add Wheels
[turtlebot3_ros-3] [INFO] [1726635095.007806312] [turtlebot3_node]: Add Sensors
[robot_state_publisher-1] [INFO] [1726635095.021790266] [robot_state_publisher]: got segment base_footprint
[robot_state_publisher-1] [INFO] [1726635095.022128292] [robot_state_publisher]: got segment base_link
[robot_state_publisher-1] [INFO] [1726635095.022185741] [robot_state_publisher]: got segment base_scan
[robot_state_publisher-1] [INFO] [1726635095.022217651] [robot_state_publisher]: got segment caster_back_link
[robot_state_publisher-1] [INFO] [1726635095.022245986] [robot_state_publisher]: got segment imu_link
[robot_state_publisher-1] [INFO] [1726635095.022272192] [robot_state_publisher]: got segment wheel_left_link
[robot_state_publisher-1] [INFO] [1726635095.022298416] [robot_state_publisher]: got segment wheel_right_link
[turtlebot3_ros-3] terminate called after throwing an instance of 'rclcpp::exceptions::RCLError'
[turtlebot3_ros-3]   what():  could not create publisher: rcl node's context is invalid, at ./src/rcl/node.c:428
[ERROR] [turtlebot3_ros-3]: process has died [pid 1221, exit code -6, cmd '/home/ubuntu/turtlebot3_ws/install/turtlebot3_node/lib/turtlebot3_node/turtlebot3_ros -i /dev/ttyACM0 --ros-args --params-file /home/ubuntu/turtlebot3_ws/install/turtlebot3_bringup/share/turtlebot3_bringup/param/burger.yaml'].
[ld08_driver-2] /dev/ttyUSB0    CP2102 USB to UART Bridge Controller
[ld08_driver-2] /dev/ttyACM0    OpenCR Virtual ComPort in FS Mode
[ld08_driver-2] FOUND LDS-02
[ld08_driver-2] LDS-02 started successfully
ls /dev
でデバイスが存在すること、権限も与えました。
sudo chmod a+rw /dev/ttyACM0
ubuntu@ubuntu:~$ sudo chmod 666 /dev/ttyACM0
また接続されている一覧を確認しました。
ubuntu@ubuntu:~$ lsusb
Bus 003 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 004: ID 0483:5740 STMicroelectronics Virtual COM Port
Bus 001 Device 003: ID 10c4:ea60 Silicon Labs CP210x UART Bridge
Bus 001 Device 002: ID 2109:3431 VIA Labs, Inc. Hub
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

解決しました。閉じます。


Improvement for gh-pages compatibility

Since 2nd November 2016, gh-pages with running jekyll blog are per default ignoring vendor and node_modules folders (see https://github.com/blog/2277-what-s-new-in-github-pages-with-jekyll-3-3 ). So it's good to update the gulp inject task and name the folder something like lib. I'll create a PR when I have some time.

Solved in #7

Merged.


_react2.default.Children.map(...).filter is not a function

I update my version of react-stockcharts from v5.0.0 to v6.1.0.
Then I  try to draw a simple chart, and it comes out an error.
Thank you for helping me fix it.
The version of packages list  as follows：
"react": "^15.0.0",
"react-dom": "^15.0.0",
"react-stockcharts": "^0.6.1",

This is part of my code
return <ChartCanvas ratio={ratio} width={width} height={height}
                    margin={{ left: 50, right: 50, top: 10, bottom: 30 }} type={type}
                    seriesName={seriesName}
                    data={data}
                    xAccessor={d => d.date} xScaleProvider={discontinuousTimeScaleProvider}>

                    <Chart id={1} yExtents={d => [d.high, d.low]}>
                        <XAxis axisAt="bottom" orient="bottom" ticks={6}/>
                        <YAxis axisAt="left" orient="left" ticks={5} />
                        <CandlestickSeries />
                    </Chart>
            </ChartCanvas>

and my data is ok

I dont see a problem it looks ok. Can you create a plunker?

?

I can't create a plunker, because there is no sense to do it.
It just  comes out an error in my project , and I  doubt that the dependency of other installation packages makes the mistake.  I'm looking for the reason and try to fix it .

I will not be able to help without a reproducible example or plunker. reopen when you have one


IntelliSense should auto-close at the end of the only-suggested-phrase

- VSCode Version: 0.10.11

OS Version: Fedora 23, 64bit

Steps to Reproduce:

For example I'm typing method, and IntelliSense suggests that only one phrase.
At the end of method| (pipe is the caret), IntelliSense doesn't auto-close. If I need to make new line, I have to type Enter twice: one to close IntelliSense, one to make new line.

I think it should auto close in such scenario.
Thanks, and have a nice weekend  :-)

I agree, that is annoying! An auto-close when there is only one matching string sounds like a good idea. Once you've typed out the complete suggestion, it should go away.

@jrieken not sure if this is you or @joao. I think @haibison is asking for auto accept if only one element is proposed and fully typed. I was able to reproduce this

To @joaomoreno for reference but I believe the box just closes when you continue typing one more character.

@jrieken I understand. But in my case, to make new line, I need to double Enter. I think it will help if the IntelliSense auto closes in such scenario.
P.S: I filed the issue while working with v0.10.11, but I just reproduced it in 1.0.0.

Fixed.


Add support to OPENSEARCH_CLIENT_APIVERSIONING

Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Description
To support backward compatability, will accept ELASTIC_CLIENT_APIVERSIONING,
but not both.
Issues Resolved
#53
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This change is not required since there is no plan to support this across clients


Add UserPass authentication method to hasicorp vault provider

Is your feature request related to a problem? Please describe.
The authentication method userPass is missing, sadly the SaaS provider which I use in my project only support this method.
Describe the solution you'd like
Adding the support for the userPass authentication method according to the already supported ones.
Describe alternatives you've considered
In the future the provider will add different authentication methods, nervertheless it would be good to support userPass here too
Additional context
https://developer.hashicorp.com/vault/docs/auth/userpass

hi! Can you give some more details how this saas provider work? Is it implementing open source vault and only allowing userpass as auth method?

Hi
yeah I would imagine they are using the open source variant and added a user backend because it is fully integrated in their portal.

@moolen thanks for merging! any chance that we can create anew release with this feature soon?

I can do a release but am on :palm_tree: after that. Got to look for someone to be available for providing hotfixes :sweat_smile:
I'll come back to you in a day or two :hourglass_flowing_sand:


Updating the skus names and clarified capacity description

This change include:

Aligning and adding new SKUs names. (We have backward compatibility to the old names).
Adding "Basic" tier.
Clarified capacity description.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@openapibot sdkautomation rebuild

@shahabhijeet Could you please approve this PR?


Llama3.1_8b Bringup

Model Card: 12942
Tensor Parallel Implementation of submodules:

MLP: 11803
RMS Norm: 11804
Attention: 11805
Decoder: 11806

PR: 12899
Cookbook: Llama31_8b_TensorParallel.md
Issues:

RMS Norm: 12427
Attention: 12428
Decoder: 12429

N300 supported now in main. Check PR https://github.com/tenstorrent/tt-metal/pull/13160 for more details


Global 'View' and Workspace 'Add': Flow To Load Check Data

As an ECMPS user, I want to view Flow To Load Check data in the Global-View and add new Flow To Load Check data in the Workspace so that I can look at real-time data or add test information as needed
Acceptance Criteria: (Global-view)
Given a user has selected Flow To Load Check from the Test Type Group dropdown
When the Flow To Load Check data is displayed
Then the user can see the data displayed as a separate table (see list below for columns and order)
Test Basis Code (Dropdown)
Bias Adjusted Indicator (Dropdown with values of 1, 2)
Average Absolute Percent Difference (Text)
Number of Hours (Text)
Number of Hours Excluded for Fuel (Text)
Number of Hours Excluded for Ramping (Text)
Number of Hours Excluded for Bypass (Text)
Number of Hours Excluded Pre RATA (Text)
Number of Hours Excluded Test (Text)
Number of Hours Excluded for Main and Bypass (Text)
Operating Level Code (Dropdown)
Given a user is viewing Flow To Load Check data
When the data is displayed in a table
Then the default sort order is alphanumeric descending by the contents in the 1st column
And the user will have the option to sort the columns in ascending or descending alphanumeric order

[x] The fields and their order specified

Pagination?

[ ] Yes
[x] No

Filter by keyword?

[ ] Yes
[x] No

Columns are sortable?

[x] Yes
[ ] No

Nested rows, expandable?

[ ] Yes
[x] No

Acceptance Criteria: (Workspace - Add)
Given the user has selected Flow To Load Check from the Test Type Group dropdown
When the user clicks on 'Add' for Flow To Load Check data
Then the system displays a modal
and the user can add new data for:
Test Basis Code (Dropdown)
Bias Adjusted Indicator (Dropdown with values of 1, 2 or Blank)
Average Absolute Percent Difference (Text)
Number of Hours (Text)
Number of Hours Excluded for Fuel (Text)
Number of Hours Excluded for Ramping (Text)
Number of Hours Excluded for Bypass (Text)
Number of Hours Excluded Pre RATA (Text)
Number of Hours Excluded Test (Text)
Number of Hours Excluded for Main and Bypass (Text)
Operating Level Code (Dropdown)
Given the user has added Flow To Load Check data
When the user chooses to save and close
Then the data is saved
And the modal closes
And the user is returned to the Landing screen
And the Flow To Load Check data record is reflected on the UI
Given the user has added new Flow To Load Check data
When the user chooses to Cancel
Then the user is returned to the Landing screen for Flow To Load Check data
And no updates are saved
Note: Flow To Load Check layout is similar to that of Fuel Flow to Load Data where it is nested under Test Data
Reference Invision URL:
Global-view: https://projects.invisionapp.com/freehand/document/areRdzxWH?zoomToItems=Y2w4M2oxbHg0MDBycjNiNnlkNWRrbHpzeA%3D%3D
Workspace: https://projects.invisionapp.com/freehand/document/areRdzxWH?zoomToItems=Y2w2YjNlbTFpMDFhMDNiNnljajh6cWM2dw%3D%3D
Testing
Test Case: TC1030
Tester: @JanellC

View Modal In Global


Use async write for tokio,async-std

Rather than passing std::io::Write to async methods, define separate tokio/async-std/std versions of the response_data_to_writer method on Request to accept the proper sync/async Write trait for the environment

Another nice one! Thank you


Error polyfill deletion

@MaximusHaximus we are having some troubles with this dependency, do we still need it in the wallet? Looks like it's not used in near-api-js internally.
Details here: https://github.com/near/near-api-js/issues/593

@volovyk-s thanks for the heads up -- I did some digging through git history and discovered that this polyfill exists due to usage of error.captureStackStrace() inside of the http-errors package, which is a package made explicitly for nodeJS  usage (thus the captureStackTrace() usage...), used for composing http errors (basically, errors with status/statusCode properties, but also some other API details around attaching custom properties).
Here's the problem in a nutshell:
https://github.com/jshttp/http-errors/issues/28
All that we currently use from http-errors directly in the wallet and near-api-js is the createError() method, which we could replace with our own implementation with guarding around the captureStackTrace() functionality for backwards compat for browsers.  We use the simplest form of createError(), so all we are really getting is the equivalent of creating an error and setting code and statusCode properties on it in one call.
There's even mention of doing this in this (sadly, closed) issue:
https://github.com/near/near-api-js/issues/119
it's worth nothing that both near-wallet and near-api-js also require http-errors dependency indirectly, by way of fetch-send-json package as well, which is a simple wrapper around fetch that we also own (https://github.com/near/send-json).
Since we want near-api-js to be browser-friendly, and it's clear from the issue I linked above that http-errors package does not intend to add compatibility for browsers (or react native -- see https://github.com/jshttp/http-errors/issues/74 also), I think it makes sense for us to have a browser-friendly version of http-errors that wraps the captureStackTrace() logic for older browsers.
However, error-polyfill does do some formatting/parsing of stack traces inside of its provided captureStackTrace() method, so removing this could negatively effect our stack trace content in older browsers if I am reading the code therein correctly.
Most importantly, by removing this polyfill, we probably broke old browsers being able to use the near-api-js since we still have the dependency on http-errors present.

@volovyk-s +1 to @MaximusHaximus comments, it's not about wallet, but about browser support
note that it doesn't seem hard to remove http-errors dependency from all of these packages (they are all ours) but it has to happen before removing polyfill


Combine some badge examples using drop-downs, tweak some titles

This updates several badges to take advantage of the change from #2882. These all seem improved by this change, though I'm happy to revisit if others disagree.
As discussed in that issue, there are several download badges which exist as separate services per time interval, which will need to be refactoried into a single service to work with this change.
I wonder if there are any common elements to these services which could be factored out at the same time. For example, we have a renderDownloadBadge function but it's not widely used.

Warnings

:warning:
This PR modified service code for bundlephobia but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for mozilla-observatory but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for packagist but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for sourceforge but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for visual-studio-marketplace but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against a95b9ea7e2ad79c6aa4a016f2cbd72e58d193d6a


Video Player 4K Error

Can not Play 4K in real Device. How to fix it ?. My Smart Monitor is 4K support

hi, please let me know your device info(Menu->Support-> About TV->Software version) and error log.

Hello sorry for late. My Smart monitor is m8 . When 4k video freeze and slow. Latest Tizen video player plugin is using


[5.8] Fix docblock

RequiredIf class allows both callback and bool in constructor:
    /**
     * Create a new required validation rule based on a condition.
     *
     * @param  callable|bool  $condition
     * @return void
     */
    public function __construct($condition)
    {
        $this->condition = $condition;
    }

so here it should be also bool included

Thanks.


Upgrade libp2p to at least v0.27.8

The current version of libp2p both in release v8.3.3 and on develop (and in v9 too, although that's not published in the chain registry yet) is set to v0.22.0.
The libp2p library is developed in lockstep with the Go compiler version and it only supports the last two major Go versions. (See notes here.)
As of August 8, Go 1.19 is end of life and unsupported. (See note here.)
Some risks this might pose:

Any developers keeping their Go version up-to-date will run into an error when trying to build the source code. (Mitigated by the README stating Go 1.19 is required.)
Any newly uncovered compiler bugs will not make it into Go 1.19, leaving binary builds vulnerable. (This is usually a risk about once every two to three years.)
The libp2p v0.22 library is fairly old. Looking at v0.23 I didn't find immediate security fixes that have to be applied but in the year's worth of work, something might have come up, that's worth looking at. (Admittedly, this is low priority but someone should go through the changelogs.)
The newer versions of Cosmos SDK will require Go 1.20 or above. (Also a low priority, until an upgrade.)

I understand that upgrading libp2p is not an easy undertaking, so I consider this a tracking issue (unless the team has some other ways of tracking it).

@greg-szabo  Thanks for the helpful suggestion and the pointers!
I think minimum we should move to go 1.20.
@lumtis you wanna take a stab at this? Not the highest priority right now but if it's relatively straightforward we should do it soon. cc @fadeev @kingpinXD @CharlieMc0


devnet has been deprecated. It needs to be removed from the code

I don't think devnet is specified anywhere in the code. cc @vgrichina

@kcole16 I think we have usage of "devnet" in README and copy:
README.md
1:# NEAR DevNet Wallet

src/translations/zh-hans.global.json
328:        "text": "这是开发者的预览版钱包，仅限用于 NEAR 协议 DevNet。了解更多："

src/translations/ru.global.json
302:        "text": "Это предварительный кошелек разработчиков. Он должен использоваться только для NEAR Protocol DevNet. Узнайте больше на сайте "

src/translations/en.global.json
333:        "text": "This is a developers' preview Wallet. It should be used for NEAR Protocol DevNet only. Learn more at "

that is not really referring to the devnet environment, but I think good idea to fix copy to make less confusing. May you help with that?

READMe will fix. It doesn't look like we're actually displaying the other copy anywhere (since refactoring the /create page).


Error while installing Terraform on on prem TFS 2017

Following errors were found in endpoint contributions: Authentication schemes ms.vss-endpoint.endpoint-auth-scheme-JWT, are not valid
I am trying to install Terraform on my on premise TFS 2017 update 2. but while installing I get the above error.

this is not supported with TFS 2017 U2


New set of columns added to table on every page action.

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior
I am getting a new set of duplicate columns added to the table every time I click on a new page in the table paging.
Expected behavior
Only have 1 set of the columns in the table.
Reproduction of the problem
What is the motivation / use case for changing the behavior?
Please tell us about your environment:
Windows 7 64 bit.

Table version: 0.7.x
0.9.3

Angular version: 2.0.x
2.1.2

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
Chrome

Language: [all | TypeScript X.X | ES6/7 | ES5]

Can you please provide a demo?

Closing due to demo not provided and age of request.


Add flutter_map to directly importable packages

It would be great if one could import 'package:flutter_map/flutter_map.dart';. Right now

Is there a map tile provider with CORS headers?
Context: https://flutter.dev/docs/development/platform-integration/web-images

This is now available in production DartPad


Upgrade Butter Knife 7.0.1 → 8.5.1

The 7.0.1 version was released mid-2015, it's probably about time to switch to the 8.x version of the library.

Thanks! I will merge that as soon as I'm done with my current work (I'm doing some internal refactoring and I think it will cause less merge conflicts if I merge your pull request afterwards)

Cool, just ping me & I'll rebase on top of your changes.


Feature/add generate one time password

Add generate one time password

Can we change the title and remove Feature/ as well as capitalize the title.


Duplicate search bar

Search bar is getting duplicated on Ubuntu. Initially happened while resizing window, But it persists on every launch after that regardless of the window size.
Screenshots
Resized

Full screen

Ao version 6.8.0
Operating System Ubuntu 18.04
Also hides item title on property panel, Making it impossible to edit item.

Same issue on macOS Mojave 10.14.3.

Apparently, it's a bug from Microsoft's side. Even if the duplicate navigation bar was to be removed, the searching functionality is completely down, since the search bar is unresponsive. It looks like, the only solution currently is to repeatedly use the View > Reload option, in order to reload the main app window, until the navigation bar appears normally. Verified the behavior on both the latest Chrome 72.0.3626.109 & Firefox 65.0.1.

Same issue on Windows10 1809. Fix it: restart system.

Apparently, it's a bug from Microsoft's side. Even if the duplicate navigation bar was to be removed, the searching functionality is completely down, since the search bar is unresponsive.

Could you please explain why you think this is MS issue? I've never seen this on thier web UI or any of the apps. Hmmm...

Verified the behavior on both the latest Chrome 72.0.3626.109 & Firefox 65.0.1.

i have it too online https://todo.microsoft.com/?app

This problem exists in macOS Mojave , according to my experience, this problem only occasionally occurs and after the View -> Reload the problem is resolved.

Is it still relevant? I get no duplication both online and in Ao.


Output directory

https://github.com/horrible-little-slime/distro/blob/de616e9427b19a1781794877abdea4b122c7f507/build.mjs#L18
If this is changed to something like outdir: "dist/scripts/distro/scripts/horrible-little-slime" it will be accessible to mafia for the set chatbotScript=tubeChatbot.js command. For now, I've copied it to my scripts folder manually.

Thanks king

Shot, works now :)


Tabs disable/enable swiping

Hi guys. I'm looking for a nice way to disable swiping to avoid users accidentally leaving a page when they don't want to. (camera for example) Can you point me in the right direction, I wouldn't mind implementing it.

Closing due to inactivity


Update InputBuilder.cs

Left-Alt should not be Extended.

This fix a really big issue in the code.  Thank You!
When simulating LeftAlt+Tab, the origin code will cause some software into a strange situation, just seems right Alt is pressing, can not respond to normal hotkeys, even ctrl+c / ctrl+v.


New package: axcore.tartube version 2.5.040

Pull request has been created with komac v2.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Verify:

This package's installer didn't seem to write any Registry entries.


Merge legacy branch Fix unit tests

Docker image: bucket-antivirus-function
Twistlock Image Scan
Vulnerabilities

CVE
Severity
Package
Version
Published since (days)
Has fix

ALAS-2019-1180
important
perl
1.04-294.43.amzn1
763
yes

ALAS-2013-177
medium
perl
1.04-294.43.amzn1
2940
yes

Total: 2
Critical: 0 High: 1 Medium: 1 Low: 0
Compliances
Private keys stored in image
Severity: high

Total: 1
Critical: 0 High: 1 Medium: 0 Low: 0


furnace: Add version 0.6.5

Add new app: the furnace chiptune tracker

Closes #11495

[x] I have read the Contributing Guide.

Added recommended changes

/verify


Learning to Cluster for Proposal-Free Instance Segmentation

Are you going to add the implementation for your instance segmentation approach to this repo, or can I find it somewhere else?
What kind of Upsampling have you used?

The instance segmentation is implemented during an internship. Due to company policy, it can't be public. I am happy to answer your implementation questions.
The upsampling function is torch.nn.functional.interpolate.


Fix type restriction logic for generic module instances

Revival of #5132. Fixes #4287. Fixes #4684. Does not affect #9660.
typeof(begin
  x = uninitialized Enumerable(Int64)
  x.is_a?(Indexable(Int64)) ? x : raise ""
end) # => (StaticArray(Int64, 3) | Tuple(Int64, Int64))

The union above is essentially all including types of Indexable(Int64) in the standard library. This is not ideal because if the other type implements the current type then we could simply return Indexable(Int64) here (see also https://github.com/crystal-lang/crystal/issues/2404#issuecomment-801069637), but this behaviour is consistent with non-generic module instances:
module A; end
module B; include A; end
class C; include B; end
class D; include B; end

typeof(begin
  x = uninitialized A
  x.is_a?(B) ? x : raise ""
end) # => (C | D)

The reason that previous PR fails seems to be that without the type restriction in Crystal::GenericModuleInstanceType#restrict it catches all AST node restrictions too, and leads to an infinite recursion inside GenericInstanceType#restrict(other : Generic, context). From my own experience it happened between Comparable and Pointer in the prelude:

Comparable(Array(T)) v.s. Pointer(T)
(Array(Array(Crystal::DWARF::LineNumbers::Row)) | Array(Array(PrettyPrint::Group)) | ...) v.s. Pointer(T)
Array(Array(Crystal::DWARF::LineNumbers::Row)) v.s. Pointer(T)
Comparable(Array(T)) v.s. Pointer(T)
...

I put a ❤  in the issue, but here are some more:
❤ ❤ ❤ ❤ ❤

The stack overflow is back if this is rebased on top of #10522. I'll need to rethink about this...


EHPR-49 | Review First 3 steps

https://freshworks.atlassian.net/browse/EHPR-49

@fwpushan @dbayly-freshworks
Rebased and updated with latest credential page changes
Also fixed EHPR-163 as it was blocking this ticket


pretrained weights

Hi,
Is there a option to download pretrained model?
Thanks.

@vLaznikova Hi, we are preparing the pre-trained weights and will release the models here ASAP.

@vLaznikova Hi, we are preparing the pre-trained weights and will release the models here ASAP.

Thanks!

are the pre-trained weights available now??


Can't build FortiClient on Manjaro

recipe.yml
app: FortiClient
binpatch: true
ingredients:
  dist: bionic
  sources:
    - deb http://us.archive.ubuntu.com/ubuntu/ bionic main restricted universe multiverse
    - deb http://us.archive.ubuntu.com/ubuntu/ bionic-updates main restricted universe multiverse
    - deb http://us.archive.ubuntu.com/ubuntu/ bionic-backports main restricted universe multiverse
    - deb http://security.ubuntu.com/ubuntu bionic-security main restricted universe multiverse
  debs:
    - xxxxxxxxxxx/forticlient_<IP_ADDRESS>7_amd64.deb
script:

Errors on:
glib-compile-schemas: symbol lookup error: /usr/lib/libgio-2.0.so.0: undefined symbol: g_module_open_full

Any ideas how to workaround that with recipe ?

Please ask the developers of FortiClient to provide an officially made and supported AppImage. Thanks!

is there a way around this error as it is not just FortiClient it is also X2go,  is it a version issue with libgio ?

Please ask the developers of FortiClient to provide an officially made and supported AppImage. Thanks!

If one could do that and expect it to happen we would not be packaging it our selves. But i like your optimism.


CVE in jupyter cells in vscode

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:1.63.2
OS Version:win10

Steps to Reproduce:
1.I open jupyter notebook in vscode；but i didn't get grammer height on my python code

2.i find that the right bottom button shows that "CVE" means select a language for current cell

3.so I clicked python in the list pulled down

4.the CVE button remains CVE nothing has changed,I'm upset now.
5.Please send some solutions thanks.

/duplicate of #139450


Android - Didn't find class "com.newrelic.agent.android.instrumentation.okhttp3.OkHttp3Instrumentation"

I have followed the installation instructions for Android to the letter and I receive the following crash report (ironically through New Relic) using the latest master branch code, react-native 0.42.0:
Didn't find class "com.newrelic.agent.android.instrumentation.okhttp3.OkHttp3Instrumentation" on path: DexPathList[[zip file "/data/app/com.spaces_whitelabel.mobile.peg-1/base.apk"],nativeLibraryDirectories=[/data/app/com.spaces_whitelabel.mobile.peg-1/lib/x86, /system/fake-libs, /data/app/com.spaces_whitelabel.mobile.peg-1/base.apk!/lib/x86, /system/lib, /vendor/lib]]
Do let me know if you need any more debug/project-specific information.
Many thanks

hi guys,did you solve the problem ,I meet the same problem. when I debug the project, this issue display. but <./gradlew clean installStagingRelease > can install the app and run successfully, it's so strange.

@yuppieboy Did you ever figure out a solution to this problem? I am having the same issue.

@nickbreaton sorry sir, I haven't solve the problem although I tried lot's of method，if you fix it，please ping me ,thanks a lot！

official newrelic react native agent is available https://docs.newrelic.com/docs/mobile-monitoring/new-relic-monitoring-react-native/monitor-your-react-native-application/


add a dseConfHome variable

Collector currently hard-codes to look for conf/dse files in /etc/default/dse, but this can vary by environment.

duplicate of https://github.com/datastax/diagnostic-collection/issues/102


PAYOUT_OUT_OF_SERVICE

Hi, the example works fine till the polling. During the polling i get the following message
{
  success: true,
  status: 'OK',
  command: 'POLL',
  info: [
    {
      code: 198,
      name: 'PAYOUT_OUT_OF_SERVICE',
      description: 'This event is given if the payout goes out of service during operation. If this event is detected after a poll, the host can send the ENABLE PAYOUT DEVICE command to determine if the payout unit comes back into service.'
    }
  ]
}

and the nv9usb+ is not active (do not accept bill).
I tried to send the command ENABLE_PAYOUT_DEVICE without any success

solved, missing command
eSSP.command('SET_CHANNEL_INHIBITS', {channels: [1,1,1,1,0,0,0,0] })


Where is System.Web.Http?

From @shaulbehr on March 6, 2018 10:19
Hi,
I'm trying to convert an ASP.NET project from .NET Framework 4.7.1 to .NET Core 2.1 (preview).
My methodology was to rename the folder from MyAspProj to MyAspProj.Old, then in the solution I created a new .NET Core ASP.NET project called MyAspProj, then copied over all the files from MyAspProj.Old. I added a reference to Microsoft.AspNetCore.All, which I hoped would bring through all the necessary types. but alas, everywhere that I have references to objects from the System.Web.Http namespace, my code is broken, with no obvious contenders to redirect the references.
For example, HttpConfiguration, which is used for Owin configuration.
And, while we're there, the Owin package itself is targeting .NET Framework. Is there a successor that targets .NET Core?
Copied from original issue: dotnet/corefx#27764

From @davidfowl on March 6, 2018 10:49
There's a doc that describes how to convert from WebAPI to ASP.NET Core https://docs.microsoft.com/en-us/aspnet/core/migration/webapi.

From @shaulbehr on March 6, 2018 12:16
@davidfowl Thanks for the link, that had quite a few useful tips.
Maybe you can point me further in the right direction. We are a company with a very successful product and a user base in the hundreds of thousands We're strongly considering migrating our whole codebase to .NET Core, mostly because we're really frustrated with SignalR not working as expected, and we hear that .NET Core has finally got SignalR fixed up. Plus of course all the other improvements, like way improved build times and EF being able to handle Group By queries.
BUT...
I've put a couple of days' work now into attempting to migrate our code, and I'm drowning. There are just so many things where the cheese got moved, and with such a new technology, there's almost no help on the Internet that deals with anything beyond the happy path, so every time I hit an edge case, I get frustrated.
I think our company could be an excellent test case for Microsoft to demonstrate how to migrate a real world project from .NET Framework to .NET Core. Are there any engineers on your team who could give us a few hours of time to coach us through migrating our code?
TIA,
Shaul

From @karelz on March 6, 2018 19:1
I think it would be best to move the discussion to ASP.NET Core repo https://github.com/aspnet as it is unrelated to CoreFX. @shaulbehr are you ok if I close it here?

@shaulbehr , it's important to think of ASP.NET (System.Web) as a different framework/library than ASP.NET Core, so certainly many things have changed.
One thing I'd recommend is to start out by creating a NEW ASP.NET Core project and then start bringing in controllers, views, NuGet package references, and other logic one by one (or in chunks), as that should make migration a lot easier.

@Eilon OK, I'll keep that in mind.
So let's say I start again with the basic infrastructure. In our current project we have a custom Startup class, which we declare in the appSettings section of web.config as
<add key="owin:AppStartup" value="FooLib.Startup, FooLib" />

I understand web.config is no longer operative, so then where do I declare this?

This is no longer configuration based. In your Startup.cs you specify the Startup class via the WebHostBuilder. If you make a new ASP.NET Core project you should see a call to UseStartup.

@davidfowl Ouch. We're using the "Onion Architecture", which, among other things, means our dependency injection layer sits above the API project, so that the project logic can be data-layer agnostic. The Startup.cs file is in the DI project. Using the web.config we could set the startup to point at the DI library, even though the API project doesn't have a reference to it; we just rely on our build & deployment scripts to ensure that the DLL is available.
How can we do this now?

we just rely on our build & deployment scripts to ensure that the DLL is available.

If you don't add a reference things won't work well.
There's an overload of UseStartup that takes a string that roughly maps to what owin:AppStartup provided. The problem is that the way assembly loading works on .NET Core currently, you can't just copy dlls to the output folder and have them be loadable. That's mostly because the dependencies need to be calculated at restore time and put into a .deps file (it's in your output folder).
It isn't impossible though, if you do want to support loading loose dlls in the output folder then you need to load them into the current context before executing the Startup code.

Are you using .NET Core or .NET Framework?

Right now: .NET Framework. But this is all in the context of migrating to .NET Core.

If you're running on .NET Framework then you don't really have to do anything. The assembly will Load just fine once it's in the output folder. On .NET Core you'll need to load it (and it's dependencies) into the default AssemblyLoadContext.
System.Runtime.Loader.AssemblyLoadContext.Default.LoadFromAssemblyPath

Cool, thanks.

There's an overload of UseStartup that takes a string that roughly maps to what owin:AppStartup provided.
Where do we invoke this overload?

@shaulbehr have you made a new ASP.NET Core project as yet?

Yes

You should be able to replace UseStartup with UseStartup("string")

Pardon my ignorance, but where is UseStartup supposed to be? I searched through the new .NET Core web project and I don't see that string anywhere...

Did you create a new ASP.NET Core 2.0 project? If yes, there should be a Program.cs file that has the call to UseStartup.

I chose Web API. This is what Program.cs looks like:

You can change that call to be something like:

Aha. I created the project, but I deleted the classes that were generated with it, thinking they were just like Class1.cs sample code, then copied over the files from my .NET Framework project. That was an important piece I was missing, it seems...

So, in your example, I presume MyMagicAssemblyName is the name of your referenced library. How, then, do we specify the class name? Does it have to be called "Startup"?

Another question: in our old implementation of OAuthAuthorizationServerProvider we had access to context.OwinContext.Environment, a dictionary of values stored in the Owin context.
If I've understood correctly, I need to convert our auth provider to inherit OpenIdConnectServerProvider, which seems to parallel most of the functionality, but I'm not sure what is analogous to the Environment mentioned above?

Ah, I think it's replaced by the HttpContext.Items collection. But that doesn't appear to be accessible within the OpenIdConnectServerProvider? How do I get to it?

You'll need to ask over at https://github.com/aspnet-contrib/ for OpenIdConnectServerProvider.

@davidfowl - regarding your code sample:
I presume MyMagicAssemblyName is the name of your referenced library. How, then, do we specify the class name? Does it have to be called "Startup"?

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Add Stylist Scroll bar

Fixes Issue🛠
#111
Closes #
Description👨💻
I have successfully added the stylist scroll bar
Type of change📄

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[] My code follows the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have added demonstration in the form of GIF/video file
[] I am an Open Source Contributor

Screenshots/GIF📷

@Rakesh9100  I have successfully added the stylist scroll bar . Please review my PR

also sorry @Rakesh9100 it was a bit late

Pls reduce the width of the scrollbar @divyansh2375
Why other file changes are visible, only style.css should be changed. Do fix.

Pls reduce the width of the scrollbar @divyansh2375
Why other file changes are visible, only style.css should be changed. Do fix.

ok i will gonna fix it

@Rakesh9100  i have lower the value of the scroll bar width

Pls reduce the width of the scrollbar @divyansh2375
Why other file changes are visible, only style.css should be changed. Do fix.

i changed only the styles.css but i don't know how to fix it ? can you tell me how can i solve it ?

I got it know someone has contributed to that loaders folder but i have not pulled it in my system

@Rakesh9100 you can review now i have successfully fixed it

Good, Merged!! ✨ @divyansh2375


fix(gatsby-ipfs-web-wallet): Bumping latest version

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


bs-react-native-jsx3-compat

This package is going to be a copy of bs-react-native but implementation of components (apis won't change) will use reason-react-native under the hood.
This will be a nice solution for people that will migrate to jsx3 with things like https://bloodyowl.github.io/blog/2019-04-19-an-alternative-migration-path-for-reason-react/
I am pushing the base (bs-react-native copy as bs-react-native-jsx3-compat) to master branch & will make PR so we can review diff.
I will also make several pr for missing props to new bindings because I found some while starting this compat layer.

I am around 50%. Should open a pr this week.

75%

98% :)

Available as pre-alpha PR https://github.com/reasonml-community/bs-react-native/pull/472


Add bundle logic and perf lab script

This adds a submodule dependency on coreclr's single-exe branch, and the logic to build with the bundler. The bundling logic doesn't use the PublishSingleFile option, so avoid passing that to the project. @rynowak PTAL
The defaults for the project will run the linker - I saw that you had been playing with defaults so this may be relevant.

@swaroop-sridhar

I left some feedback on a few things, but ultimately I don't want to block on any of that stuff. Let's get this in and see the automation light up.


Add support for spring-cloud Finchley and spring-boot 2.0.0

Add make target to test this Finchley with SB 2.0.0

I'd be happy to help if I can get some pointers on this.

Done in #167
I will cut 0.2.0 with this change


refactor: test module

Description
Use fixture to fix spawn too many server.
Related Issue
Related #3133

hey @casey I plan to use the fixture feat provided by rstest to fix flaky tests error. and refactor test file(test dir). now i only refactor wallet::send test file.
If you accept it, I will do the next .


Macro to create workspace for fit with 4 MVA bins, inclusive njet 14,…

… using RooParametricHist for tt, with JEC and JER systematics

First version of main fitting workspace creation maco.


can not build on ARM CPU

scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
gcc -o src/bloomd/art.o -c -std=c99 -D_GNU_SOURCE -Wall -Wextra -Werror -O2 -pthread -Isrc/bloomd/ -Ideps/inih/ -Ideps/libev/ -Isrc/libbloom/ src/bloomd/art.c
src/bloomd/art.c:5:23: fatal error: emmintrin.h: No such file or directory
#include <emmintrin.h>
^
compilation terminated.
scons: *** [src/bloomd/art.o] Error 1
scons: building terminated because of errors.
can somebody tell how to fix it?

@vlike The radix tree depends on a pretty specific x86 instruction, it could be ifdef and replaced with a linear scan on ARM, please open a PR if you get a chance!


Custom and dynamic HTTP auth info

We have implemented a custom API that forwards Druid queries to the Druid query API endpoint but we need to be able to provide an HTTP header with the JWT data because our custom API auth scheme uses  OIDC.
Apparently, Grafana Go SDK supports this scenario as explained here. You can see there is a Custom HTTP header section. However, we still need to make these headers values dynamic by providing the JWT data via dashboard variable.
Is there any chance you can support adding custom/dynamic HTTP headers?
Thanks

@jbguerraz any thoughts on this one?

Hello @jmichalak-fluxninja @layoaster
I'm back from holidays :)
We didn't use the grafana component for DataSourceHttpSettings since it offered useless settings. Nor we used the grafana http client on the golang side of it.
We would need to inject a grafana http client into go-druid (which is doable; see https://github.com/grafadruid/druid-grafana/blob/master/pkg/druid.go#L72 and https://github.com/grafadruid/go-druid/blob/master/druid.go#L317) and either use the big DataSourceHttpSettings or add headers feature to our custom one ( https://github.com/grafadruid/druid-grafana/blob/master/src/configuration/ConnectionSettings/DruidHttpSettings.tsx ).
Then we still need to manage the "dynamic" headers values. First we would need a way to alter a request (RequestMiddlewares ?) to alter its headers, prior to its execution, probably here: https://github.com/grafadruid/go-druid/blob/master/druid.go#L157
We also would need to pass dashboards variables within the DruidQuery (https://github.com/grafadruid/druid-grafana/blob/master/src/types.ts ) so the backend could use them for interpolation (so we write "${thevariable}" in the datasource settings and interpolate for each request since there is one instance of the druid client per data source and not per request).
It's not a trivial change but it's interesting! If you're willing to contribute it I'll be happy to actively support you along the way!

what's up with this @layoaster and @jmichalak-fluxninja  ? is it still needed ?


How To use with Nav Drawer Activity?

i just want to know that , it can implemented with Navdrawer Activity? How?

Hello, @yogesh2591. Try this


Return WalletAlreadyExistsException from createWallet

In this PR, we return WalletAlreadyExistsException in createWallet when createWallet is called and there's already a wallet created. With this change, it'll be easier for clients to detect that there's already a wallet created with the private key.

There were two vulnerabilities that dependabot found. Upgrading the dio resolves them as dependabot states. @LiorAgnin
https://github.com/fuseio/fuse_wallet_sdk/security/dependabot


Allow to secure the endpoints with the auth API

See https://docs.avax.network/apis/avalanchego/apis/auth

This is deprecated in Durango (see<EMAIL_ADDRESS>


.abap file have no code highlighting

Dear All,
ABAP codes has no highlighting.

Lepton uses highlight.js for syntax highlight.
highlightjs.org
If you find a language support missing, that 'll be a better place for a discussion 😃


Respect content type negotiation

Use the following rules to determine the response data type

Use options.dataType if present
Try to parse from the response content type header
Try to parse from the request accept header

Previously, only the first accept header MIME type was taken into account.
fixes #10

This looks great, thanks.

Can you look into the test failures and add some tests? This looks great, thanks.


Incorrect code produced for sign/zero extension from boolean to 64-bit integer

OpSelect with constants is used but the constants are always 32-bit integers.

Forgot to say in the original issue: I've got a simple patch for this one but I still need to spend time on writing more tests.


panos_security_policy Supposedly deleted all security policies not defined in terraform which is not true for panorama

Documentation link
https://registry.terraform.io/providers/PaloAltoNetworks/panos/latest/docs/resources/security_policy#panos_panorama_security_policy

Describe the problem
Warning
On the terrraform palo alto provider documentation is says the following:
: This resource will remove any security rule not defined in this resource. <--- This is not true for Panorama, tested on 10.1.7

Suggested fix

Clarify is this only for policies directly with Firewalls and not to Panorama Managed Security Policies

You're in the "changed functionality" zone of PAN-OS:
10.1.5 >= x > 10.2
Here's another issue about the same thing where this was investigated:  #350
In the short term, upgrading to 10.2 (and staying on 10.2) will resolve this issue.  The long term is still being figured out...


VmaAllocator_T::Map illegal use of [[fallthough]]

This is an illegal use of [[fallthrough]] that the compiler flags.  So just remove the VMA_FALLTHROUGH line.  Sorry if this is older code, and it's been fixed.
VkResult VmaAllocator_T::Map(VmaAllocation hAllocation, void** ppData)
{
    switch(hAllocation->GetType())
    {
    case VmaAllocation_T::ALLOCATION_TYPE_BLOCK:
        {
            VmaDeviceMemoryBlock* const pBlock = hAllocation->GetBlock();
            char *pBytes = VMA_NULL;
            VkResult res = pBlock->Map(this, 1, (void**)&pBytes);
            if(res == VK_SUCCESS)
            {
                *ppData = pBytes + (ptrdiff_t)hAllocation->GetOffset();
                hAllocation->BlockAllocMap();
            }
            return res;
        }
        VMA_FALLTHROUGH; // Fallthrough <- this doesn't need fallthrough, it has a return res above
    case VmaAllocation_T::ALLOCATION_TYPE_DEDICATED:
        return hAllocation->DedicatedAllocMap(this, ppData);
    default:
        VMA_ASSERT(0);
        return VK_ERROR_MEMORY_MAP_FAILED;
    }
}

You are right, it was invalid. I fixed it. Thank you for reporting this issue.


Integrate TSDF components into SLAM pipeline.

RFC=0019

@wally-the-cartographer merge

Merge requested by authorized user kdaun. Merge queue now has a length of 2.