Expose Hadoop metrics in Ganglia

Currently Ganglia shows only VM metrics, but we need expose at least Hadoop metrics as well.

Since very few people use Ganglia (afaik), I'm going to close this issue for now. Stackdriver is generally a better solution, because metrics persist even after a cluster is deleted.


Chapter 11 code?

There is a note in the 2e about the availability of code for the book.  Chapter 11 doesn't seem to be present.

Chapter 11, you are missed!

Also looking for Chapter 11 code.


BUG: renaming with onyo mv prints message calling it "moved"

❱ onyo mv accounting account
The following will be moved:
/Users/tkadelka/none-existing-folder/accounting -> account

The call does rename a folder, but the message "will be moved"

I have tested this: the commit message subject, contents, and operations log are all correct. It's just the user-facing message in the terminal that's incorrect.

I solved this a while ago and forgot to close the issue; here the same command that lead to opening the issue:
tkadelka in ~/onyo on git:master
❱ onyo mv accounting account
The following will be renamed:
/Users/tkadelka/onyo/accounting -> /Users/tkadelka/onyo/account
Save changes? No discards all changes. (y/n)

AFAICT addressed by PR #479


bosh create-env panic

$ bosh --version
version 2.0.25-687a73d-2017-06-07T23:31:58Z

Succeeded

$ bosh create-env ~/workspace/bosh-deployment/bosh.yml -o ~/workspace/bosh-deployment/aws/cpi.yml -o ~/workspace/bosh-deployment/bosh-lite.yml  -o ~/workspace/bosh-deployment/jumpbox-user.yml -o ops-bosh-lite.yml -l <(bbl bosh-deployment-vars) --vars-store ./creds.yml
Deployment manifest: '/Users/amitgupta/workspace/bosh-deployment/bosh.yml'
Deployment state: '/Users/amitgupta/workspace/bosh-deployment/bosh-state.json'

Started validating
  Downloading release 'bosh'... Skipped [Found in local cache] (00:00:00)
  Validating release 'bosh'... Finished (00:00:01)
  Downloading release 'bosh-aws-cpi'... Skipped [Found in local cache] (00:00:00)
  Validating release 'bosh-aws-cpi'... Finished (00:00:00)
  Downloading release 'bosh-warden-cpi'... Skipped [Found in local cache] (00:00:00)
  Validating release 'bosh-warden-cpi'... Finished (00:00:00)
  Downloading release 'garden-linux'... Skipped [Found in local cache] (00:00:00)
  Validating release 'garden-linux'... Finished (00:00:01)
  Downloading release 'os-conf'... Skipped [Found in local cache] (00:00:00)
  Validating release 'os-conf'... Finished (00:00:00)
  Validating cpi release... Finished (00:00:00)
  Validating deployment manifest... Finished (00:00:00)
  Downloading stemcell... Finished (00:00:01)
  Validating stemcell...

Compiling package 'golang_1.7/c82ff355bb4bd412a4397dba778682293cd4f392'...[CLI] 2017/06/08 10:58:10 ERROR - Panic: interface conversion: pkg.Compilable is *pkg.CompiledPackage, not *pkg.Package

we currently dont support compiled cpis.


Issue on page /notebooks/05-Python_I.html

Writing python programs
boilerplate.py
Alternatively, you can use ipython like you did above:
ipython boilerplate.py
should be
ipython3 boilerplate.py

fixed, thanks!


Error: Could not load settings for 'Northwind.WebUI'

Hi everybody,
Im new in Angular and fighting with the error:
Failed to load resource: the server responded with a status of 404 (Not Found) [http://localhost:4200/_configuration/Northwind.WebUI]
I would like to see that solution in action, just

cloned this git repo
dotnet build (WEBUI folder)
dotnet run (WEBU folder)
npm install (ClientApp folder)
ng serve (ClientApp folder)
open the chrome browser with http://localhost:4200/

Error 404 as mentioned above.

I saw a Pullrequest #224, merged them in the repo manually, but unfortuantelly didnt help.
Thanks,
Mehmet

Hi Mehmet. Thanks for checking out this repo.
When you launch it, you don't need to start the front end and back end separately. Just use dotnet run, and browse to https://localhost:44376. Please review the getting started steps here - https://github.com/jasontaylordev/NorthwindTraders#getting-started.
The project is an Angular + .NET Core SPA, and in development, the .NET Core back end proxies all requests to the front end. By launching 4200, you're only looking at the front end. Learn more here; https://docs.microsoft.com/en-us/aspnet/core/client-side/spa/angular?view=aspnetcore-3.1&tabs=visual-studio.
Also, recommend checking out the new Clean Architecture Solution Template; https://github.com/jasontaylordev/CleanArchitecture/. It's running the latest versions - .NET Core 3.1 and Angular 9. This is the getting started guide; https://jasontaylor.dev/clean-architecture-getting-started/.
Let me know if you run into any issues and I help out.
Happy Coding! 😃

Hi everybody,
Im new in Angular and fighting with the error:
Failed to load resource: the server responded with a status of 404 (Not Found) [http://localhost:4200/_configuration/Northwind.WebUI]
I would like to see that solution in action, just

cloned this git repo
dotnet build (WEBUI folder)
dotnet run (WEBU folder)
npm install (ClientApp folder)
ng serve (ClientApp folder)
open the chrome browser with http://localhost:4200/

Error 404 as mentioned above.

I saw a Pullrequest #224, merged them in the repo manually, but unfortuantelly didnt help.
Thanks,
Mehmet

Only run localhost5001 and not localhost:4200.

Recommend checking out https://github.com/jasontaylordev/CleanArchitecture/ instead.
It's more up to date and has better docs. Happy coding! :)


Incorrect explanation of lifecycle

The explanation and diagrams describing the lifecycle of Razor components is incorrect and misleading. Also see this closed issue: https://github.com/dotnet/AspNetCore.Docs/issues/22742.
I spend a few hours today finding out why I got a NullReferenceException while my OnInitializedAsync was not finished yet. Based on the diagrams and the documentation I thought the rendering would only take place after the OnInitializedAsync was completed (awaited). But looking at the source code, rendering is also executed before any awaiting is done.
So if you use OnInitializedAsync or OnParametersSetAsync, you have no guarantee that these methods are finished before the rendering is done.  Please update the docs to reflect this, or at least add a remark that the diagram is an oversimplification.

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

ID: a970fb72-3dd0-d173-585d-2ccdd7923f81
Version Independent ID: 4871f4f7-9cf6-ac7a-0f21-c65d34bd1119
Content: ASP.NET Core Razor component lifecycle
Content Source: aspnetcore/blazor/components/lifecycle.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @joostvz ... I'll ping Steve to ask about this. He said earlier that one might need to do exactly what you did ... consult the ref source ... because the diagrams and explanations are simplifications and that the logic in the framework is too complex to show/explain simply. However, your last line ...

you have no guarantee that these methods are finished before the rendering is done

... might be what those two sections can explain to cover it. I'll ping him shortly to see what he'd like to do.

@joostvz ... Can you clarify further what's "incorrect"? The section ...
https://docs.microsoft.com/aspnet/core/blazor/components/lifecycle#handle-incomplete-async-actions-at-render
... states ...

Asynchronous actions performed in lifecycle events might not have completed before the component is rendered. Objects might be null or incompletely populated with data while the lifecycle method is executing. Provide rendering logic to confirm that objects are initialized. Render placeholder UI elements (for example, a loading message) while objects are null.

@guardrex Good point, I didn't notice that section. I think I didn't notice it because the Lifecycle events section explicitly states:

Call OnInitialized{Async}. If an incomplete Task is returned, the Task is awaited and then the component is rerendered.

Also the diagram in this section confirms this statement, so I didn't have reasons to believe otherwise.
Indeed, I think a remark with a cross-link to the other section might help.


NetBSD - very slow network (IPv6 timeouts?)

I have tried to use cross-platform-actions to run NetBSD and I have noticed that when I use for example pkg_add or pkgin it takes virtually forever to complete.
After some googling I have found that there is some problem with IPv6, QEMU ticket here:

https://gitlab.com/qemu-project/qemu/-/issues/985

Is this something that cross-platform-actions could help with and give an option to workaround? I would not mind having ipv6 disabled in the VM if there was such option to do so. The problem is that it's almost unusable at the moment to even install a C++ compiler on such a runner.
I have also tried another project that provides NetBSD and it runs pretty well, although it uses a different virtualization software (virtualbox on macos only):

https://github.com/vmactions/netbsd-vm

Thanks a lot for these actions btw!

IPv6 should already be disabled. The NetBSD action is currently running on Linux which doesn't support hardware accelerated nested virtualization. I've recently added support for macOS runners which do support  hardware accelerated nested virtualization. I've not created a new release yet but you should be able to use the commit hash directly. Could you please try https://github.com/cross-platform-actions/action/commit/c6ffabb3c8bd9899a52924b7ecf314c60625dcc0?

I have actually tried macos-12 host on cross-platform-actions master branch (which already had that aforementioned commit merged) with NetBSD.
I'm using at the moment vmactions to test NetBSD because of this, but I can create a branch and run NetBSD with cross-platform-actions to double check the issue.

Ok, then I'll take a look. Do you have a link to a job that I can try?

I have created a branch in one of my projects, and an example run could be this one:

https://github.com/asmjit/asmjit/actions/runs/4251344675/jobs/7394864271

The netbsd-9.2 is the one that actually takes forever when run with cross-platform-actions. The other one (just called netbsd) uses vmactions/netbsd-vm - it fails, but that's because of scripting (when it succeeds it's like 10 minutes run time).
BTW: There is still a possibility I did something wrong in my scripts that prepare the environment as I'm really new to BSDs, but on the other hand this is something I can reproduce easily.
One more thing that is happening to the runners sometimes is that they are stuck after run finishes (even when everything is successful), basically until I kill the job (an example here: https://github.com/asmjit/asmjit/actions/runs/4250851364/jobs/7392460230) - but I can create another issue to track this problem, as I think this is not an issue in my scripts.

One more thing that is happening to the runners sometimes is that they are stuck after run finishes (even when everything is successful)

Might be https://github.com/cross-platform-actions/action/issues/21, but if you're already using master it might be something else. Could https://github.com/cross-platform-actions/action/issues/29 be related?

Definitely not related to #21 (I've got few of these EPIPE errors before, then switched to master and they are gone).
Likely to be #29 related yeah, so no need to open an additional issue.

BTW This is not crazily important for me to get netbsd working with cross-platform-actions as I will just use vmactions/netbsd-vm instead. What I found nice on cross-platform-actions is the ability to have all the OSes in a build matrix and just share all the boilerplate around it. With vmactions you basically are forced to use a separate action for each supported OS, which is not really ideal for me.

The question is how they install the dependencies. It almost feels like pkgin is the problem in my case. I can install it via pkg_add to install other dependencies, but when it's used it just takes enormous time to complete. In the vmactions VM I can actually use pkg_add instead to install python310 and the rest of the dependencies (totally avoiding pkgin), but I have no idea why the same approach doesn't work in cross-platform-actions VM.

I think this is working better: https://github.com/jacob-carlborg/asmjit/actions/runs/4256157138/jobs/7404681643. Installing Python was pretty quick but installing Cmake and Clang seems to take longer. Could you please give the netbsd-slow-network-46 branch a try.

It finished after 16m and 16s.

yeah that's pretty great - 16 minutes would work for me actually. Maybe if I can ditch pkgin it would be even better, but I have no idea why pkg_add doesn't find python

The fix is good enough, thanks! I think that if you merge the fix this could be closed. If there are different issues I would just open a new issue.


Calling close() on EventSource object on client side (browser) does not get handled as closing http connection on server side

It should lead to Cancelled event on publisher actor side. Trying to push events after EventSource.close() eventually lead to Broken pipe TcpClose event

Thanks for reporting, very interesting, as we are currently investigating another issue which might be related.
Could you please provide a little more context:

Where are you calling the close() method? On the JavaScript EventSource object?
Do you have a reproducible test case?

Yes it is on the JavaScript EventSource object.
If EventSource.close() is equivalent to closing the HTTP connection I should be able to provide a test.
Sent from my iPhone

On Mar 1, 2015, at 14:11, Heiko Seeberger<EMAIL_ADDRESS>wrote:
Thanks for reporting, very interesting, as we are currently investigating another issue which might be related.
Could you please provide a little more context:
Where are you calling the close() method? On the JavaScript EventSource object?
Do you have a reproducible test case?
—
Reply to this email directly or view it on GitHub.

#13 confirms that the issue may be higher up in the stack. I think that the issue should be closed here, but a new one is raised on akka-http with a reproducible test case.

@huntc, could you please provide the link to the respective issue on akka-http?

I've not raised one as yet - I wanted to discuss who might be able to chase it up later today.

Here's the related Akka HTTP issue: https://github.com/akka/akka/pull/16984

The actual root issue is here: https://github.com/akka/akka/issues/17122

@wkozaczuk, could you please retry? Akka HTTP has moved on and this might no longer be an issue.

Closing, because no more activity.


interrupt handler gives double fault interrupt a few seconds after boot

A few seconds after boot, the ISR calls a double fault interrupt, with no apparent cause or reason.

Guaranteed to be some sort of bug in the kernel, or in one of the drivers.

Try remapping IRQ 8?


Soundcloud Bridge returns error 401

Error message: `The requested resource cannot be found!
Please make sure your input parameters are correct!
cUrl error:  (0)`
Query string: `action=display&bridge=Soundcloud&u=radiojuicy1&format=Atom`
Version: `2018-12-11`

Is this only on my end or something that needs to be fixed regarding the bridge itself?

Had the same thing. Looks like the client ID/API key we use doesn't work anymore. Added a new one in the PR above which should fix this! (:


ButtonBar style doesn't fill entire width

I'm really confused by the documentation. I don't see any way without specifically setting a fixed value for self.settings.style.buttonBarItemLeftRightMargin to make the ButtonBar actually take up the full width like the examples in the README.
All four of the animated .gif files at the top have the bar straight across with the tabs justifying to fill the entire width.
Then the animated .gif for ButtonBar looks nothing like them and the one that does (Bar) has no text...
Really confused why there's isn't a way to get a combination of the two?

I'm not sure if I understood what you are trying to say.
Button Bar view which is the view that contains the tab bars is actually a UICollectionView. Tab Bar items are UICollectionViewCell's. The library calculates the width of each tab bar by adding text width and buttonBarItemLeftRightMargin * 2 so by setting up buttonBarItemLeftRightMargin you are increasing the tab bar size, ultimately it will depends on the title size but at least we can control the left, right marging of each tab.
In addition to that, we can also make the collection view tabs fill its collection view width by setting up buttonBarItemsShouldFillAvailableWidth before call viewDidLoad super implementation.
settings.style.buttonBarItemsShouldFillAvailableWidth = true

I'm gonna update the readme to point this out.
Let me know if this answer your question.

I had the same issue as @bdrelling.  settings.style.buttonBarItemsShouldFillAvailableWidth = true wouldn't fill the available width for me unless I specified settings.style.buttonBarItemLeftRightMargin = 0 as well.

@Raymond26 I am having the same issue, buttonBarItemsShouldFillAvailableWidthdoesn't work until buttonBarItemLeftRightMarginis 0

@Drusy or @Raymond26 Do you have this code for me? I have tried several things to use he whole width but it didnt work for me. My Code in the Parent is:
import UIKit
import XLPagerTabStrip
class SecurityParentView: ButtonBarPagerTabStripViewController {
let purpleInspireColor = UIColor(red:0.13, green:0.03, blue:0.25, alpha:1.0)

override func viewDidLoad() {

    settings.style.buttonBarItemLeftRightMargin = 0
    settings.style.buttonBarItemsShouldFillAvailiableWidth = true

    super.viewDidLoad()

    settings.style.buttonBarBackgroundColor = .white
    settings.style.buttonBarItemBackgroundColor = .white
    settings.style.selectedBarBackgroundColor = purpleInspireColor
    settings.style.buttonBarItemFont = .boldSystemFont(ofSize: 14)
    settings.style.selectedBarHeight = 1.0
    settings.style.buttonBarMinimumLineSpacing = 0
    settings.style.buttonBarItemTitleColor = .black
    settings.style.buttonBarLeftContentInset = 0
    settings.style.buttonBarRightContentInset = 0
    changeCurrentIndexProgressive = { [weak self] (oldCell: ButtonBarViewCell?, newCell: ButtonBarViewCell?, progressPercentage: CGFloat, changeCurrentIndex: Bool, animated: Bool) -> Void in
        guard changeCurrentIndex == true else { return }
        oldCell?.label.textColor = .black
        newCell?.label.textColor = self?.purpleInspireColor
    }
}

override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController] {
    let tabBasics = UIStoryboard(name: "Main", bundle: nil).instantiateViewController(withIdentifier: "sbBasics")
    let tabLaws = UIStoryboard(name: "Main", bundle: nil).instantiateViewController(withIdentifier: "sbLaws")
    let tabDefaults = UIStoryboard(name: "Main", bundle: nil).instantiateViewController(withIdentifier: "sbDefaults")
    return [tabBasics, tabLaws, tabDefaults]
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
}

/*
// MARK: - Navigation

// In a storyboard-based application, you will often want to do a little preparation before navigation
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    // Get the new view controller using segue.destinationViewController.
    // Pass the selected object to the new view controller.
}
*/

}

@david-schor Here is my setup but I don't think it will help, unfortunately, it depends too much on the number of element and the size of your text in the tabs ...
        settings.style.buttonBarBackgroundColor = R.color.light_gray
        settings.style.buttonBarItemBackgroundColor = R.color.light_gray
        settings.style.selectedBarBackgroundColor = R.color.ps_black
        settings.style.buttonBarItemFont = UIFont.exoExtraBoldFont(withSize: 14)
        settings.style.selectedBarHeight = 2
        settings.style.buttonBarMinimumLineSpacing = 20
        settings.style.buttonBarItemTitleColor = R.color.ps_black
        settings.style.buttonBarItemsShouldFillAvailableWidth = true

        if UIScreen.isIPhone5 {
            settings.style.buttonBarLeftContentInset = 20
            settings.style.buttonBarRightContentInset = 20
        } else {
            settings.style.buttonBarLeftContentInset = 40
            settings.style.buttonBarRightContentInset = 40
        }

@Drusy You are right but thank you anyway!

I found a solution. Try calling view.setNeedsLayout() and view.layoutIfNeeded() after configuring the settings. Like so,
override func viewDidLoad() {
    settings.style.buttonBarBackgroundColor = AppColors.blue
    settings.style.buttonBarItemBackgroundColor = AppColors.blue
    settings.style.selectedBarBackgroundColor = .white
    settings.style.buttonBarItemFont = .boldSystemFont(ofSize: 14)
    settings.style.selectedBarHeight = 2.0
    settings.style.buttonBarMinimumLineSpacing = 0
    settings.style.buttonBarItemTitleColor = .white
    settings.style.buttonBarItemsShouldFillAvailableWidth = true
    settings.style.buttonBarLeftContentInset = 0
    settings.style.buttonBarRightContentInset = 0

    super.viewDidLoad()

    view.setNeedsLayout()
    view.layoutIfNeeded()
}

This worked for me. I only encountered this issue in iOS 11. That too at random times. It's weird.

I found a solution. Try calling view.setNeedsLayout() and view.layoutIfNeeded() after configuring the settings. Like so,
override func viewDidLoad() {
    settings.style.buttonBarBackgroundColor = AppColors.blue
    settings.style.buttonBarItemBackgroundColor = AppColors.blue
    settings.style.selectedBarBackgroundColor = .white
    settings.style.buttonBarItemFont = .boldSystemFont(ofSize: 14)
    settings.style.selectedBarHeight = 2.0
    settings.style.buttonBarMinimumLineSpacing = 0
    settings.style.buttonBarItemTitleColor = .white
    settings.style.buttonBarItemsShouldFillAvailableWidth = true
    settings.style.buttonBarLeftContentInset = 0
    settings.style.buttonBarRightContentInset = 0

    super.viewDidLoad()

    view.setNeedsLayout()
    view.layoutIfNeeded()
}

This worked for me. I only encountered this issue in iOS 11. That too at random times. It's weird.

@Isuru-Nanayakkara , you are rockstar, you just saved my whole day, Thanks alot.
Please add this in document.

Welp, this issue is back. Apparently my earlier solution of calling view.setNeedsLayout() and view.layoutIfNeeded() doesn't do the trick either.

Demo project
I tested on an iPhone running iOS 13.4. I have the latest version 9.0.0 of the project.

@Isuru-Nanayakkara Have you got any solution bro? I have same issue

override func viewDidLoad() {
    configureBarStyle()
    super.viewDidLoad()

    view.setNeedsLayout()
    view.layoutIfNeeded()
}

Though the above-suggested solution fixed the issue by filling the entire width. There were some side effects.
In my case one of the child view controllers had UITableView in it and it was throwing a warning,

[TableView] Warning once only: UITableView was told to layout its visible cells and other contents without being in the view hierarchy (the table view or one of its superviews has not been added to a window). This may cause bugs by forcing views inside the table view to load and perform layout without accurate information (e.g. table view bounds, trait collection, layout margins, safe area insets, etc), and will also cause unnecessary performance overhead due to extra layout passes. Make a symbolic breakpoint at UITableViewAlertForLayoutOutsideViewHierarchy to catch this in the debugger and see what caused this to occur, so you can avoid this action altogether if possible, or defer it until the table view has been added to a window. Table view: <UITableView: 0x7f8366988000; frame = (0 0; 414 822); clipsToBounds = YES; autoresize = RM+BM; gestureRecognizers = <NSArray: 0x600002b2f780>; layer = <CALayer: 0x60000245df20>; contentOffset: {0, 0}; contentSize: {414, 438}; adjustedContentInset: {0, 0, 0, 0}; dataSource: (null)>

This was caused by the following code which asked the view to layout immediately in the func viewDidLoad()
    view.setNeedsLayout()
    view.layoutIfNeeded()

The actual fix that I came up with was somewhat similar. Instead of asking the whole view to layout itself, I connected an outlet for ButtonBarView and asked it to layout,
    buttonBarView.setNeedsLayout()
    buttonBarView.layoutIfNeeded()


Height of modal reduces after clicking in it and hovering on Close button

Untitled.mov.zip
Steps to reproduce this issue:

Visit https://picturepan2.github.io/spectre/components.html#example-modal-2
Click on Open small size Modal (it is important that this is the first modal you open)
Click inside the modal
Hover on Close button

Unable to reproduce this error if a bigger modal is opened first. However, once the problem appears, it affects all modals.
P.S.: Thanks for this great project!

@karmasakshi I couldn't reproduce it. Can you check if it is still there?

@karmasakshi I think it is a bug of Safari.


correct docs for t.teardown

Correction to README for the teardown method as it's on the t instance rather than test
See https://github.com/substack/tape/pull/546 for reference
Also open PR for updating type definition https://github.com/DefinitelyTyped/DefinitelyTyped/pull/54694

Codecov Report

Merging #558 (c4a4992) into master (a68277c) will decrease coverage by 0.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #558      +/-   ##
==========================================
- Coverage   96.70%   96.37%   -0.33%
==========================================
  Files           4        4
  Lines         607      607
  Branches      144      144
==========================================
- Hits          587      585       -2
- Misses         20       22       +2

Impacted Files
Coverage Δ

lib/results.js
99.31% <0.00%> (-0.69%)
:arrow_down:

lib/test.js
95.67% <0.00%> (-0.29%)
:arrow_down:

Continue to review full report at Codecov.

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


[bug ?] broken types for gocardless-nodejs/webhooks

Hi there. I'm trying to build integration with gocardless and validate a webhook signature with parse function form the import { parse } from 'gocardless-nodejs/webhooks'. But my linter doesn't see webhook.d.ts and fails with error File '/...path/node_modules/gocardless-nodejs/webhooks.d.ts' is not a module.
How can I include types from the webhook.d.ts ?
Stack:

serverless
typescript

Tsconfig
{
  "compilerOptions": {
    "sourceMap": true,
    "target": "es2020",
    "lib": ["es2020"],
    "strict": true,
    "noUnusedLocals": true,
    "moduleResolution": "node",
    "esModuleInterop": true,
    "typeRoots": ["./node_modules/@types", "./src/types"],
    "outDir": "dist"
  }
}

I think this is because module.exports = {} is used instead of export { ... } in the webhook.ts file.
A quick fix is to declare the module yourself to be able to import it.
declare module "gocardless-nodejs/webhooks" more on that here:  https://stackoverflow.com/questions/44058101/typescript-declare-third-party-modules

Sorry for the lack of activity, everyone. This should be fixed in #122


Compaction throughput is an upper limit, don't fail tests if throughp…

…ut is actually slower than the limit

+1. For reference, explanation for removal of assert is in here: https://issues.apache.org/jira/browse/CASSANDRA-10667


Add Documentation

Add Documentation
Purpose
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the changelog
[ ] Added tests
[x] Updated the spec
[ ] Checked native-image compatibility

@aashikam please review

@Sachindu-Nethmin Hey
Hiding sensitive information such as access tokens and client IDs in Git pull requests is critical to prevent unauthorized access and potential security breaches. Exposing such credentials can allow attackers to exploit your application, access restricted resources, or compromise user data, leading to serious damage.
7de1d1a1f1bdc853e71537ddeda32cb6f00551d2  This commit itself is a big security vulnerability. 😃
Even though you have removed it, by viewing the commit anyone still has access to the information.
To remove this from git history you will have to do a hard reset.
Perform the hard reset: Execute git reset --hard <commit-hash> to move the branch pointer to the specified commit and discard any changes in the working directory or staging area.
Verify the reset: Run git log again to confirm the branch now points to the desired commit.
Push the changes: Use git push --force to overwrite the remote branch

@Sachindu-Nethmin Hey
Documentation includes all these files:

I see that you have only update the README.md. It's pretty much the same content.
Here is a sample PR: https://github.com/ballerina-platform/module-ballerinax-dropbox/pull/5


chmod -x */.rs

Signed-off-by: Igor Gnatenko<EMAIL_ADDRESS>

Coverage remained the same at 0.0% when pulling f13f660f04bddb37ec94941aeb023025da79b9cf on ignatenkobrain:chmod into a995977ad35b3258c3bf890886bffcf7514cfc6c on gimli-rs:master.


Prepare for Uint8List SDK breaking change

A recent change to the Dart SDK updated HttpClientResponse
to implement Stream<Uint8List> rather than implementing
Stream<List<int>>.
This forwards-compatible change updates calls to
Stream.transform(StreamTransformer) to instead call the
functionally equivalent StreamTransformer.bind(Stream)
API, which puts the stream in a covariant position and
thus causes the SDK change to be non-breaking.
https://github.com/dart-lang/sdk/issues/36900

ping @Intelx

@jwashin are you the right person to review and merge this?


New method "asPrettyString()" for the ResponseBody

Pull Request for the issue: New method "asPrettyString()" for the ResponseBody

Thank you! :)


[ubuntu] provider-kairos has no image assets for 2.4.0

The release of 2.4.0 has no image assets.  Looking at what was uploaded to quay.io for Ubuntu (what I use) only shows RC1 for 2.4.0.  I haven't checked other OSes.
I'm not sure if the release was cut early on accident on GitHub, or if release automation failed to publish artifacts, but something seems off.

The provider-kairos release pipelines have been merged with the kairos ones. Now all artifacts are releases from the kairos repository, both core images and standard images (those with the provider). There will be new images as soon as we release 2.4.0 from kairos. From now on, the releases on this repo are just new releases of the provider binary.
Sorry for the confusion.


Cant compile pygame

I try to compile pygame2.0.0dev10 with nuitka but this give me a error:
Nuitka-Plugins:INFO: implicit-imports: Implicit module 'pygame.event.Event' suggested by 'pygame.mixer' not found.
Can i fix this?

It's not an error, it's an info, and one that will not be given anymore starting with 0.6.9, because we just cannot tell what version of package may or may not have an implicit dependency. So now we try it, and when it's there, we use it.


UI is totally broken during server/API initialisation because error responses and HTTP status are ignored

the client used to work as per #548, that is: when the API isn't yet ready, or in any other case of error response (marked by the HTTP status != 20x), the UI reported an error under the "Keyword Search" field.
Now it is completely broken during the startup stage, the specie selector is empty no specie is selected, no error is shown. Moreover, nothing is recovered once the server app is started and starts working normally, the only way is to reload the Knetminer page.
This can be fixed by adding an initial check on the main onReady() method, which should invoke some well-known API URL (eg, /dataset-info) and stop rendering anything if this returns an error, render an error message instead, plus a "Reload" button.
Assigning it to @manlikleo, though I'm not sure he will have time before leaving.

This should be given priority and hopefully not too hard to fix. @Arnedeklerk @manlikleo

@manlikleo I think it still doesn't work. I think the problem is that setupApiUrls() and/or its invoker (jquery's onLoad()) should also have a failure handler. This function tries to guess the API URL, from a couple of well-known URLs and eventually sets up related global variables. If it fails, everything else will fail after it.
From now on, we can go back to work on master.

@manlikleo when you're back, we need to move this ticket out of review and into In Progress. It's possible that some of your changes were lost (or are still on multispecies branch and should be on Master) once those are merged in, please also work on the setupApiUrls problem mentioned both above and on 668

@manlikleo I've changed this, the setTimeout() method was too convoluted and error-prone, please have a look, we might need something similar in future. The regular way to manage an error (connection unavailable or in timeout included) is to attach a failure handler to promises (.catch()) or to AJAX calls (like $.get(...).fail(...)).
Now it works pretty well during UI initialisation, the graphics is really nice.
However, it still doesn't work in other cases:

when typing search keywords: if the API is down, it doesn't show any count of associated genes, it used to show a red message under the text box. It still work like that when the API is available but returning errors like 'initialising'. At this point, I'd say this is a minor
if the API goes down after hitting 'Search', the DNA spinner spins forever and nothing happens, the UI seems stuck. Maybe it's a minor, presumably, the user will reload and discover the not-available page.
When clicking on the release notes icon, if the API is down or returning errors, nothing happens on the UI, the console is showing errors.

Comments on your list:

In my opinion the search box problem isn't really a problem so long as we catch the majority of cases where the API isn't initialised. I also think this is minor for now, unless I'm misunderstanding.
We could have a timer on the spinner which after 30 seconds prompts the user that "This search is taking particularly long, you might need to reload the page." - I've seen this on other sites before and it really just gives the power to the user to decide whether they want to wait and see, or not. Some users have reported waiting literal hours for searches, meanwhile we know that they literally will timeout and they are just wasting their time.
It is part of https://github.com/Rothamsted/knetminer/issues/679 (to remove the icon)

@Arnedeklerk Sounds good.
Re 2), the case where the API call isn't returning soon is different from the case it went down before hitting 'Search' or straight after. That ends up with a Js exception in both cases, and the event can be intercepted/manage either via some catch{} block, a Promise.catch() handler or a $.get().fail() handler. The current problem is we don't manage it at all and catching this case is the starting point to tell the user it took too long (telling it's taking too long without interrupting the wait is more complicated, and would require that interception anyway).
For instance, the follow is what I get when I load the UI with the API running, then I stop the API, then I hit 'Search'. As you can see, the event disrupts the UI and must be recovered as I said. Of the three, this the case that maybe is still a little concerning.
Moreover, @manlikleo let's have a chat on our next meeting on how to coordinate the application events. Throwing status information into UI elements (eg, $('.navbarselect-container').css("display") == 'none') is messy and should be avoided. Using things like async handlers, status objects or global status variables is preferrable (in this order of preference).


Tracking Other Replaceables

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
I would love to be able to track preventative maintenance tasks other than replacing batteries. For instance my furnace filter, water filters, or when my sceptic system was last pumped. Battery Notes already has most the underlying functionality already (date last changed, battery replaced button, battery type, etc.) and it seems like it could be easily extended to include other random trackables. The main change needed would be that there is not necessarily another existing entity to attach to like there is a battery (like my water filter has no related entity), so a new entity would have to be created vs extending an existing one.
And probably the next logical change would be a way to specify the expected life/change frequency of the trackable since there is no battery percentage (or filter age) being reported for my water filter.
Describe the solution you'd like
The ability to add "battery notes" not attached to an existing entity
A way to specify the trackable type (battery, filter, etc.)
A way to specify change frequency manually (ex. I want to replace my filter every 3 months). Along with this, the due date of the next change. Also, the templating function could be used (like the low battery status template) to track, for example, a water filter change after XXXX gallon/liters if water usage was available in HA.
Describe alternatives you've considered
I've looked at Grocy as an alternative, but setup is complex, clunky to use, not native HA UI (requires an addon) and I can't easily use NFC tags or QR codes to trigger replacement events. (With Battery Notes, you can scan a tag which would trigger the battery replaced button.)
Additional context
I realize the name of the integration would not be as applicable, so that is a consideration. Should this be a new integration or an extension of this one? It seems like much of the functionality is already there, hence why I am suggesting extending Battery Notes. Thank you for considering this. I have been looking for an HA solution for this for years and was very excited to discover Battery Notes being extended similarly to how you could track other replaceables.

I have thought about "Maintenance Notes" as a concept but it stretches what I want to do with this integration too far when you start to consider things like maintenance schedule calendars and brand new devices that don't exist in HA.
Battery Notes is a good starting point for anyone wanting to develop a new integration but at the moment I wouldn't have time to develop and maintain that.
I'll move this to ideas to keep it visible in case another integration dev would like to take up the challenge, it may also be worth raising it in the HA forums.


chore: use wurstmeister repository for kafka-docker

Remove fixme after https://github.com/wurstmeister/kafka-docker/pull/145 accept

Current coverage is 87.70% (diff: 100%)

Merging #23 into master will not change coverage

@@             master        #23   diff @@
==========================================
  Files             2          2
  Lines           122        122
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            107        107
  Misses           13         13
  Partials          2          2

Powered by Codecov. Last update c5b3ac7...220b49d


Add enforce.arrayOf() rule

This rule can be quite useful when a future schema validation rule will be added to enforce.
The general idea is quite similar to the enforce.any rule:
enforce(value).isArrayOf(
  enforce.isString(),
  enforce.isNumber(),
 enforce.isArrayOf(...) // Yeah, why not? A recursive call should be pretty simple here
)

The idea is that at least one needs to be truthy in order for the condition to be met - alternatively, an empty array should probably be valid as well unless explicitly mentioned:
enforce(value).isArrayOf(
  enforce.isString()
).isNotEmpty()

See Shape for reference.

#499 implement isArrayOf rule

Closed by #499


Dev editor

Summary by Sourcery
New Features:

Introduce new event listeners in the EditorExtension class, including onBeforeRender, onRender, onDidRender, onDidSelectionChange, and onDidContentChange.

@sourcery-ai review


Executing a search for Observation that returns no results produces a Bundle whose getTotal() method returns null, rather than usual int 0

Bundle results = client.search()
.forResource(Observation.class)
.where(Observation.PATIENT.hasId({some patient_id with no Observation resources}))
.returnBundle(Bundle.class)
.execute();
int resultSize = results.getTotal();
//resultSize is null, expected to be 0

In case this helps - I've also started to see Bundle.total intermittently missing from search results with HAPI 2.5 CLI, but haven't figured out what reproduces it yet. In my case the true number of resources to be found is approx 500, and except for the missing total the first page appears fine - the code errors out and doesn't attempt to get the next page.

I'm able to consistently reproduce this with JPA Server on v2.5


Class GetIt doesn't have a default constructor

Hi, I just wanted to play around, but I show an error that says GetIt doesn't have a constructor. Do you have any tips on how to solve this?

Hi, thx for using flutter_challenges repo, can you update your code base on this repo ? i fixed this issue

Thank you, Javier. I pulled and cleaned, but also had to restart Studio for some reason. In any case, thank you for fixing and I am now able to compile view in the simulator. I appreciate you sharing your knowledge for beginners like myself.


exception while executing task 99

Entire console being spammed with this across all servers on the network, restarts do not help.
Has this plugin and it's Github issues been abandoned? @LeeMcNeil @liam-wiltshire
[06:14:10] [Craft Scheduler Thread - 3/WARN]: [BuycraftX] Plugin BuycraftX v12.0.6 generated an exception while executing task 99
java.lang.IllegalArgumentException: Not a valid Mojang UUID.
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:122) ~[patched_1.12.2.jar:git-Paper-1618]
	at net.buycraft.plugin.UuidUtil.mojangUuidToJavaUuid(UuidUtil.java:21) ~[?:?]
	at net.buycraft.plugin.bukkit.BukkitBuycraftPlatformBase.getPlayer(BukkitBuycraftPlatformBase.java:67) ~[?:?]
	at net.buycraft.plugin.bukkit.BukkitBuycraftPlatformBase.isPlayerOnline(BukkitBuycraftPlatformBase.java:74) ~[?:?]
	at net.buycraft.plugin.execution.DuePlayerFetcher.processOnlinePlayers(DuePlayerFetcher.java:112) ~[?:?]
	at net.buycraft.plugin.execution.DuePlayerFetcher.run(DuePlayerFetcher.java:98) ~[?:?]
	at net.buycraft.plugin.bukkit.command.ForceCheckSubcommand.lambda$execute$0(ForceCheckSubcommand.java:31) ~[?:?]
	at org.bukkit.craftbukkit.v1_12_R1.scheduler.CraftTask.run(CraftTask.java:64) ~[patched_1.12.2.jar:git-Paper-1618]
	at org.bukkit.craftbukkit.v1_12_R1.scheduler.CraftAsyncTask.run(CraftAsyncTask.java:52) ~[patched_1.12.2.jar:git-Paper-1618]
	at com.destroystokyo.paper.ServerSchedulerReportingWrapper.run(ServerSchedulerReportingWrapper.java:22) ~[patched_1.12.2.jar:git-Paper-1618]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]

Hi @andrewkm
Just to confirm, this isn't a bug with the plugin - the bug is with the data being returned by the API - as the plugin is getting invalid data, it is then correctly throwing an exception. I appreciate from a user-side it's difficult to know when something is a problem with the plugin or when it's not.
We are working to rectify the root cause - in the meantime I have fixed the impacted commands.

Yes, it's clear the problem lies on the info the plugin is getting, however is there any way for a user to resolve these errors themselves via the Tebex panel?


"Land insights" should be LandInsight

Small typo, but LandInsight is the name of the company/product, not "Land insights".

Thanks for pointing that out @samdelacruz, fix now with 77de3a7


[Bug]: bal format command only works on non-default modules

Description
bal format <module_name> command seems to be working only for the non-default modules. when the default module is provided, the command ends up with the below error message.
error: couldn't find an existing module by the name: <module_name>
Steps to Reproduce

Create a new ballerina package with name foo
Execute bal format foo on the package root directory

Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Formatter
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

In such scenario, the entire project has to be formatted or just the default module?
.
├── Ballerina.toml
├── main.bal
└── modules
    └── util
        ├── Module.md
        ├── resources
        ├── tests
         │   └── lib_test.bal
        └── util.bal

Say this is a project structure, bal format would format all bal files in the project does bal format foo also should behave as the same or does it should format only the main.bal file?

After the discussion with @NipunaRanasinghe following decisions were made on how bal format should work
bal format [OPTIONS] [<package>|<source-file>]
OPTIONS
    --module <moduleName>
        Format only a specific module in the Package

     --file <fileName>
        Format only a specific file in the Package

Consider the below package structure,
.
├── Ballerina.toml
├── main.bal
└── modules
    └── util
        ├── Module.md
        ├── resources
        ├── tests
        │   └── lib_test.bal
        └── util.bal

Can format either an entire Ballerina Package or a standalone Ballerina file
bal format testPackage
bal format testFile.bal
If a specific module in a package need to be formatted,
bal format --module util
If a specific file in a package need to be formatted,
bal format --file main.bal
bal format --file modules/util/util.bal

I assume the bal format and bal format . commands format the current package.

Yes, Both commands would format the current package.

Ack. Looks good


The windows of download detail is too high.

The windows of download detail is too high. I think the windows should expand aspect ratio and make width become long.

Well this is a detail of a download page and should give users a brief description about a download! it's acceptable that this page has lots of elements. but you can close the part info section (which is optional) if it takes too much space in your monitor.


Idea: Monitor Challenge Progress

There are six challenges where each challenge needs around 1-3h to be completed. Each challenge is split into several tasks. People can catch up with progress branches if they spend too much time on a singlet task. There are also mandatory tasks so I can forward the project implementation.
This means: Everyone is working on a challenge and I have no clue where the people are working on.
Idea:

It would be amazing to have an overview where everybody is currently working on. This could be achieved by placing a button at the beginning of each task. When pressed it would signal to me (as the workshop host) that the given user (authenticated through github) just started.

In the end I could look at a progress overview that could look like this:
Challenge 05 - Testing (Progress)

Task 1: Stock Bug (Investigate) 🐛(▓▓▓ 100%)
Task 2: Stock Bug — Part 1(▓░░░ 25%)
Task 3: Stock Bug — Part 2(▓▓░░ 50%)
Task 4: Stock Bug — Part 3(░░░ 0%)
Task 5: Test RxJS with Oscar 🐙 — CPUs(░░░ 0%)
Task 6: Test RxJS with Oscar 🐙 — Humans(░░░ 0%)
Task 7: Can Oscar play multiple cards ?(░░░ 0%)

I could event locate people that are struggling if I include a timer.
That would be so hand, wouldn't it ? My plan is to work on a proof of concept 👌

Do you see this in some kind of small companion webservice or somewhere within the workshop itself?
Also, this should work even without authentication assuming people in the workshop actually want to use this as it's supposed to be used is fair.


Several pages don't have breadcrumb

Somehow I got to several rabbitmq docs pages (see below).
I want to look at the index page of these pages to scan for all available resources.
However, there is no breadcrumb or link to the parent page from the content.
Can anyone tell me if there are any sibling pages, or the parent page of the below pages?
https://www.rabbitmq.com/consumers.html
https://www.rabbitmq.com/passwords.html
https://www.rabbitmq.com/connections.html
https://www.rabbitmq.com/queues.html
By the way, thank you very much for the awesome document website.

All of those are pretty self-contained.

thanks. Is there a list of them?

There is no as how you group guides is a matter of opinion.


Problem with iPhone 7

After I updated BarcodeScanning.Native.Maui from 1.1.1 to 1.4.0, the scanner no longer works on an old iPhone 7 with iOS 15.8.2.
The camera itself still works, but the UI is frozen and no barcodes are recognized. None of the buttons work anymore. An iPhone 14 Pro and an iPad do work.
There have of course also been other updates to .Net Maui and Visual Studio. The cause may also lie there.
What could be the cause of this?
Thanks in advance.

Hello.
Could you please roll back to 1.3.0/1.3.1 version and see if it works for you? @parigino1 reported in #37 that this version worked on iPhone 7.
1.4.0 didn't introduced any new functional changes so it shouldn't change iOS version requirements, but I did modify a lot of code so some unexpected bugs could be in there.

Hi, version 1.4.0 works for me on iPhone 7 (I have only this one), probably it doesn't work because of a wrong handle of the result?
As quoted by @afriscic
the "OnDetectionFinished" and "OnDetectionFinishedCommand" events now returns BarcodeResult[] instead of HashSet
Glad if I helped you!

Oh, yes, I forgot to mention the most important part...
But I assumed that the IDE would catch any type missmatch and not compile.

Works OK with version 1.3.1.  Thank you.

Hi,
Still the same problem with version 1.4.1.
The scan page is not on the mainpage in my app.

Hi, still the same problem with version 1.4.2.
I have crash reports from Sentry:

System.NullReferenceException: Object reference not set to an instance of an object.
App Hanging: App hanging for at least 2000 ms.

Hi, I have the same issue on iPhone 8 with iOS 16.7.5. The last version where it works is version 1.3.1.
I need to upgrade to version 1.4.2 to resolve the issue #37 with the camera focus and resolution on dual camera devices.

Hello.
As I don't have these older devices debugging is really impossible for me. I've gone over and over through code and found only one real difference between 1.3.1 and current version. I've created a new branch with potential fix. Please if you can do the following:

Clone the master branch and run Barcodescanning.Test on your device to see how it works,
Clone the issue-#43 branch and run Barcodescanning.Test on your device to see how it works.

Please report back afterwards.

Thanks for your reply. I was just testing with version 1.4.2, if I set CameraEnabled=true then the UI doesn't respond but it can scan, however if CameraEnabled is set to false the UI responds to touch.
I will test and report back.

Hi,
The master branch did not solve the problem.
For the issue-#43 branch I get the error: Git failed with a fatal error: repository not found.

BarcodeScanning.Native.Maui-issue-43.zip
Here is a zip of the branch

Hi,
The issue-43 is working on a iPhone 7.
Thank you very much.

Hi, just managed to test issue-#43 branch on iPhone 8 and it's still causing issues, the buttons don't respond and the device becomes hot.
Sorry for the bad news, is there anything else you can try?

I've pusshed some more minot tweaks today to issue-43 branch. See if it'll be any help.

I did a couple more tests (now with the correct branch) and it's working on iPhone 8 as well as on iPhone 13.
Thank your very much for your help fixing this issue, its very appreciated.

I'll mark it as fixed in 1.4.3
Thank you all for your feedback


Please don't work on the next version in the master branch

Is your feature request related to a problem? Please describe.
I read the API docs on the main page (README.md for master) of your Git repo, but they were for a version (0.8.0) that hasn't been released yet on NPM. That is **npm i broken-link-checker" gets version 7, not 8. Your example made sense, and seemed reasonable, so I couldn't figure out for quite a time why it didn't work. I eventually just guessed that your main README file must be the docs for a version that had not yet been released. So I looked for an older README and was up and running in about two minutes.
Describe the solution you'd like
Do your work for the next version in a branch called version_8, or the like, and merge it into master when it is ready to be released.
Describe alternatives you've considered
Alternatively, add text to the README.md for the master that says clearly where to look for the working docs.
Additional context
This is a great program and I appreciate your work. But the misdirection on how to use it cost me an hour or two.

Why not use NPM for documentation? It represents each release, with the latest being the default.

Closing for now.

I agree that NPM is a good way to go, but it is common to use GitHub as well. I don't want to want to add to your work, I just felt that this was a subject worth considering.


RHEL 8.5 Cannot be configured with firewalld

:speaking_head: Foreword
Thank for taking the time to fill this bug report fully. Without it we may not be able to fix the bug, and the issue may be closed without resolution.
:ghost: Brief Description
Standard Minimal Server ISO install of RHEL8.5 uses firewalld, enabled and running with zone public. By declaring this firewall cookbook to takeover, it always drops firewalld and installs a full set of IPTables, regardless of attributes given.
:pancakes: Cookbook version
Depends currently set to 4.0.0
:woman_cook:  Chef-Infra Version
17.8.25
:tophat: Platform details
Self hosted RHEL8.5 from default ISO with minimal server
Steps To Reproduce
Steps to reproduce the behavior:
Depend this cookbook.
Declare:
firewall 'any' do
anything
end
Will disable firewalld and install IPTables.
:police_car: Expected behavior
Firewalld to be installed/enabled (which it is by default in the OS) and then zones/rules set.
:heavy_plus_sign: Additional context
I think this code is related, lines 22..24 in libraries/provider_firewall_firewalld.rb:
provides :firewall, os: 'linux', platform_family: %w(rhel fedora amazon) do |node|    (node['platform_version'].to_i == 7 && !node['firewall']['redhat7_iptables']) || (amazon_linux? && !node['firewall']['redhat7_iptables'])  end

This is very likely fixed in https://github.com/sous-chefs/firewall/pull/247 but was never merged 14 months ago :(


Fix the label selector for Elasticsearch logging pods

Updated to use a constant for the es-key and es-value -- it is only used in one place.
Removed now redundant check for pod status.

My apologies - I misread the code.  I see it needs to be consistent with fluentd-elasticsearch/es-service.yaml and not with the rest of this go code.  LGTM.


Update transport.cc

fix #782 bug

I would say that this should fix my problem. Cannot verify it, since the problem happens rarely. But we are testing it.


Issue with pyright when running ./bootsrap

Hi, I tried to use the template today, but ran into an issue with pyright. When running ./bootstrap, I get the error message:
pyright..................................................................Failed

hook id: pyright
exit code: 250

WARNING: there is a new pyright version available (v1.1.309 -> v1.1.310).
Please install the new version or set PYRIGHT_PYTHON_FORCE_VERSION to latest
Assertion failed: ((static_cast<size_t>(off - 1)) <= (sizeof(arrow) - 1)), function AppendExceptionLine, file ../src/node.cc, line 1404.
Traceback (most recent call last):
File "./bootstrap", line 244, in
File "./bootstrap", line 235, in main
File "./bootstrap", line 54, in git
File "/opt/anaconda3/envs/wmtsa/lib/python3.8/subprocess.py", line 415, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/opt/anaconda3/envs/wmtsa/lib/python3.8/subprocess.py", line 516, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['git', 'commit', '-m', 'bootstrap from template']' returned non-zero exit status 1.
Do you have any idea what that is?
Thank you very much.

try running pre-commit autoupdate before running bootstrap. Basically the Pyright (python static analyzed, similar to mypy) pre-commit plugin notices that there's a newer version and says to update to it.
I also just ran pre-commit autoupdate in the repo, so if you clone again, you should be good.

I also just added pre-commit autoupdate to now be part of bootstrap, and to not run pre-commit on the first boostrapping commit, so this issue shouldn't come up again.

Thank you, it works now.


update.sh: Make user, repo, and branch easy to change

This makes it easy for me to try out e.g. https://github.com/rust-lang/rust/pull/99787 by temporarily doing the following change in the script and then run it:
index 5437494..ba0d8c6 100755
--- a/update.sh
+++ b/update.sh
@@ -4,9 +4,9 @@ pyjq() {
     python3 -c "import json, sys; print(json.load(sys.stdin)${1})"
 }

-user="rust-lang"
+user="aDotInTheVoid"
 repo="rust"
-branch="master"
+branch="rdj-dyn"

 curl -# https://raw.githubusercontent.com/${user}/${repo}/${branch}/src/rustdoc-json-types/lib.rs | sed 's/rustc_data_structures::fx::/std::collections::/g' | sed 's/FxHashMap/HashMap/g' > src/lib.rs

IMHO it would be overkill (and more complicated to use) if we made these changeable by script args ($2 $3 $4) instead, so I prefer this.

@aDotInTheVoid Any chance you could take a look at this please? I used it again for https://github.com/rust-lang/rust/pull/100762 and find it very useful.
The diff might look compilicated, but it's very simple actually. I just extract hardcoded strings into variables.
Let me know if you have any concerns and we'll sort them out somehow!

No concerns, just hadn't gotten around to looking over it, and it seemed non-urgent.


build the landing page wireframes

build the landing pages for the project
build the lo-fi
build the hi-fi

Done!


Append menu to more than one element

Hello,
Firstly great plugin.
I have a question, Is there anyway to append the same menu to more than one element?
Thanks

@BassemN, if you simply need the same menu in multiple places you can clone the first menu and then call a second slicknav on the clone. Because of the way the methods work you can't create 2 SlickNav instances from the same menu. http://codepen.io/ComputerWolf/pen/xGdaVJ

Thanks :)


Add finalization request

Coverage increased (+1.7%) to 78.44% when pulling<PHONE_NUMBER>3c152586ab28d88eaeaae9bd379546 on Fak3:finalize_req into 218d246193010510d19b8cd0a21bd9b15defcb7d on frictionlessdata:master.

@subhankarb can you take this on.


❎ Exclude files listed in project config from myst build and watch

Problems:

Without an explicit _toc.yml file, the project build pulls in all md/ipynb files from the project directory
During myst start all files within the directory are watched (potentially an arbitrarily large number of files... enough to break the watch function)

Current state:

Project config already has an exclude field for a list of files to "exclude" - however, nothing respects those excluded files...

What this pr does:

Files listed under exclude in the project config are treated as glob patterns. These come into play in two places:

During project build, any md/ipynb that matches an exclude pattern will not be added to the project
During myst start, everything matched by exclude is ignored (so you can ignore folders with lots of unrelated files)

To make this work, there were some changes to the watch function - primarily triggering a project reload on myst.yml changes

(there was also some refactoring to consolidate to a single watch "processorFn" rather than duplicating logic across the old "fileProcessor" and "siteProcessor")

Future steps

Currently you may only build your project with (1) explicit _toc.yml or (2) implicit directory contents with explicitly excluded files. It would be nice to also be able to have a simpler include or something in the project config to define the toc.
Further, if you exclude a file, it is not built at all; there is no way to build a file as part of the project but not include it in the project's toc (this is useful for notebook-generated figures - we want to keep the source notebook but not really have it as part of the project toc...)

Nice, works well!
I think it would be good to have a pass in the future where files that are included are added to the watch. For example, I am working with a lot of tex docs right now, and I would love an auto-generated TOC but also want to see the changes live when we update an included document like a figure or table.


Mouse scroll not working properly after disabling.

When disabling the plugin and scrolling with the mousewheel over the MCS element, scrolling down doesn't work properly, but scrolling up is ok.

Disable plugin.
Scroll down using mousewheel on MCS element.
Scroll doesn't work.

MacBook Air
Mac 10.11
Firefox 40.0

Hi @samerxx,
Did you get any solution for it?
I'm also having similar issue
I have redactor editor, which uses HTML default scroll-bar and for outer section of it I have mCustomScrollbar,
I'm disabling the MCS when in my redactor text editor I have large content with it's scroll-bar
then if I move mouse out of the editor I enable the MSC again to scroll outer section
The issue I'm facing is, if I move mouse out of the editor, and bring it again inside editor (without scrolling the outer section), I'm able to scroll the editor content up but can't scroll it down
Please look into this issue @malihu, and suggest what can be done in this scenario

I cannot reproduce this in either OSX (10.10.5) or Windows (7) using standard mouse and magic trackpad...
When scrollbar is disabled (via disable method`) the content should not scroll. Have I undestood correctly?
@samerxx Can you create a JSFiddle or send me a test page?

Hi @malihu, Thanks for reverting back
I'm using Ubuntu 14.04, and when I use $(selector).mCustomScrollbar("disable"), it works fine, it disables the scroll-bar, but the other scroll bar on the page, which is a default HTML scroll-bar, gets affected.
The issue comes up on Chrome, Firefox (at least I've tested these 2 browsers)


command line help is cryptic

Hi,
I have used bam2fastq on a bam file containing ONLY unaligned reads (bit flags are either 77 or 141) with following command:
bam2fastq -o C14IAWNRP1#.fastq C14IAWNRP1.trimmed.noplant.bam
It has output all the reads in the bam file to fastq (which is exactly what I wanted but hoping to understand the params better)
--aligned
--no-aligned
Reads in the BAM that are aligned will (will not) be extracted.
[Default: extract aligned reads]
--unaligned
--no-unaligned
Reads in the BAM that are not aligned will (will not) be extracted.
the "will (will not)" descriptions are pretty confusing and there seems to be contradicting default behaviors described.
Otherwise works fine :)

@jts
Agree！！！
I have tried to extracted reads from a bam file
I FOUND  when using --aligned option  actually un-mapped reads was extracted
when I was   using  --no-unaligned ,this will work .


Skip verify to master

https://github.com/open-cluster-management/multicloud-operators-subscription/pull/247 is difficult to merge.
Instead, I ported the logic to the master branch directly.

Got the  error in the travis log:
+make lint
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                             Dload  Upload   Total   Spent    Left  Speed

100    93  100    93    0     0    478      0 --:--:-- --:--:-- --:--:--   479
.build-harness-bootstrap:1: *** missing separator.  Stop.

Hi @frankwg
Yup that issue on travis is still due to your travis settings, there isn't that much we can change at the moment.
Anyway, I want to merge your commit as soon as possible though, do you have time for a meeting we can go through some possible solutions for your commits.
Please do let me know your time perference.

I‘m available any time except 5pm ~ 9pm Beijing time everyday.

@frankwg can we do it today 9.40am Beijing Time?

sure, but how?  webex?

https://bluejeans.com/146574401

PR is in the ibm branch, close for now


Pip10 update

Update pip-tools for pip10 compatibility (and backwards compatibility with pip9)
Contributor checklist

[x] Provided the tests for the changes
[x] Requested (or received) a review from another contributor
[x] Gave a clear one-line description in the PR (that the maintainers can add to CHANGELOG.md afterwards).

/cc @vphilippon
I didn't add tests, since I just rebuilt existing functionality using pip10 as well.  This should work with both versions of pip and is all green locally

I think this is a good change. IMHO being compatible with Pip 10 is better than vendoring old Pip, since vendoring always has its problems. (E.g. we would have to backport all security and bug fixes from upstream Pip, etc. The PyPI URL change being a good example.)
It seems that the build matrix doesn't include Pip 9 anymore though. Can you add it back with Pip 8 and 10 so we could see if all tests pass on other Pip version than 10 too?

Thanks for putting this together @techalchemy !
As @suutari said, if we're un-vendoring, we'll want to put back the pip "dimension" in the test matrix.
Here's the commit that remove it: https://github.com/jazzband/pip-tools/commit/61163e043fbedb6c3fefb406399321e7b2221ec6
Ideally, we'd keep pip 8 to 10 compatibility, but if this gets messy, we can look to drop pip 8. I'll try to find some usage stats on pip 8 first though.

OK I hit most of the major items -- I have no idea if this will work with the matrix in question becuase of the way it grabs the index url -- we may need to keep this pointed at the current pypi.

Alrighty so I was wrong, but now this is mostly working except a tiny pip8 error and I would appreciate any guidance at all on what to do with that one

scratch that, it was just a new style marker test

@techalchemy
Ok first off, once again, thanks a million time for your work.
Second, sorry for the delays in reply, its hard to get the time to give a whole proper check/reply at this.
By unvendoring, we fully match the pip the end-user chooses to use, default PyPI URL and everything else. I agree with @suutari and you on this.
But by unvendoring, we also put ourselves (and all pip-tools users) back at risk of breakage on any pip update. The stuff we use was explicitly refactored in a package named _internal for crying out loud.

The least-horrible thing I can think of is to go ahead and unvendor pip, and if we get to regret this, just vendor it back.
Thoughts?

While not ideal, would a better solution to be to vendor the most recent version of pip? This would require a bit of upkeep, but would avoid the cost of maintaining compatibility across three major versions of pip.

I don't have particularly strong feelings except to say that we vendor pip over in pipenv (and have been doing so) in order to maintain compatibility with piptools specifically and that also can cause issues related to certificates, but gives us some flexibility to make changes to the dependency resolver directly.  In all likelihood we will end up vendoring pip 10 anyway so I can see an argument for that, but if you're not modifying the resolver it may not make sense.
Overall they moved some APIs around primarily to drive some discussion about whether the internal apis really ought to be available externally, and we are in touch about that (I believe pipenv is probably the worst offender but also is part of the PyPA and works collaboratively with the pip team). Overall I don't see much risk of any sweeping API changes, but I guess you never know. Either approach is going to come with maintenance overhead. I think that's just the cost of maintainership.
In the end no decision is going to protect pip-tools from having to handle adjusting the codebase in the future, I'm kind of split on this one.  IMO unvendoring is easiest

Also sincere apologies I know this is nearly impossible to review properly, here is a near-complete set of the changes (sans the first commit) without the removal of pip 9:
https://github.com/jazzband/pip-tools/pull/657/files/4c7fce7897e4dbbba524b59e0a52ee38e37a7aa6..3ba2956232fad316b8c07daf65c7a23d4aa96033

@vphilippon I see you as the lead maintainer of pip-tools currently, so IMHO it's your call if the Pip should be vendored or not. I'd vote for not vendoring, if anyone cares.

I vote for un-vendoring pip, along with using stricter version requirements in our install_requires.
The release of pip-tools that un-vendors pip should depend on pip>=8.0,<10.1.  Patch releases of pip should be allowed.  When a new major/minor version of pip is released, nothing will break.  People can submit issues/pull requests to enable new versions of pip when they become available.

@suutari no problem I agree with that rebase / squash. @tysonclugg I think this approach makes the most sense for now as well except that I believe pip is switching to calver which will complicate this.

@techalchemy You're right that CalVer will complicate things, especially given the new pip deprecation policy.  But I'm not aware of a better alternative, aside from helping the pypa team to make the necessary pip internal APIs public if at all possible.
The pip release cadence docs state that master should always be in a releasable state.  We should include a pipmaster factor in our tox.ini in order to catch issues before a breaking version of pip is released.

we should include a pipmaster factor

Best suggestion yet

Thanks for everyone input.
Lets go with un-vendoring then, I think it makes sense.
@tysonclugg's suggestion to put pip in the install_requires worries me for Windows user though. Doing pip install pip-tools will cause issue as pip will to uninstall/modify itself if it doesn't match, leaving ugly errors like:
Exception:
Traceback (most recent call last):
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\basecommand.py", line 215, in main
    status = self.run(options, args)
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\commands\install.py", line 342, in run
    'Successfully downloaded %s', downloaded
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\req\req_set.py", line 795, in install
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\req\req_install.py", line 767, in commit_uninstall
    response = ask_path_exists(
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\req\req_uninstall.py", line 142, in commit
    rmtree(self.save_dir)
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\_vendor\retrying.py", line 49, in wrapped_f
    return Retrying(*dargs, **dkw).call(f, *args, **kw)
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\_vendor\retrying.py", line 212, in call
    raise attempt.get()
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\_vendor\retrying.py", line 247, in get
    six.reraise(self.value[0], self.value[1], self.value[2])
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\_vendor\retrying.py", line 200, in call
    attempt = Attempt(fn(*args, **kwargs), attempt_number, False)
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\utils\__init__.py", line 102, in rmtree
    onerror=rmtree_errorhandler)
  File "c:\python278\Lib\shutil.py", line 247, in rmtree
    rmtree(fullname, ignore_errors, onerror)
  File "c:\python278\Lib\shutil.py", line 247, in rmtree
    rmtree(fullname, ignore_errors, onerror)
  File "c:\python278\Lib\shutil.py", line 247, in rmtree
    rmtree(fullname, ignore_errors, onerror)
  File "c:\python278\Lib\shutil.py", line 252, in rmtree
    onerror(os.remove, fullname, sys.exc_info())
  File "f:\virtualenvs\tmp-abfbe3982928ec9c\lib\site-packages\pip\utils\__init__.py", line 114, in rmtree_errorhandler
    func(path)
WindowsError: [Error 5] Access is denied: 'f:\\tmp\\pip-o9ongj-uninstall\\virtualenvs\\tmp-abfbe3982928ec9c\\scripts\\pip.exe'

which will end up as recurring Github a few minutes later.
This would force Windows users to use python -m pip install pip-tools, and that's going to break stuff for a lot of users and automated CI system right now. As a dev working on Windows... Please don't 😄 .
We can add a check early in pip-compile/pip-sync to check pip version and act on that. The check will occurs on pip-tools's use rather than install, but I think it's the best we can do right now. IIRC there was a check like that prior to vendoring pip.

I honestly think building against master to stay ahead of issues is the best bet. But I don’t think there is much risk of huge sweeping changes in the near future

@vphilippon You're right that having an install_requires line for pip is a bad idea.  Having the pipmaster factor in CI should be enough, without the install_requires changes I had proposed.

@vphilippon Would you like me to pick this up and add the pipmaster CI factor?

Sorry for the delay on this I’ve been trying to manage a pipenv release. Feel free to push a pipmaster factor here to speed things up, no issues with that from my end

@techalchemy We're not using appveyor.yml, and I don't have access to change settings on the pip-tools Appveyor project.  Either I create an appveyor.yml, somebody gives me access, or someone else adds the pipmaster factor there.  Which will it be?

@tysonclugg: It's .appveyor.yml here: https://github.com/jazzband/pip-tools/blob/master/.appveyor.yml

@techalchemy I'm trying to understand why we should use an environment variable to determine the pip version rather than inspecting the pip.__version__ attribute in the test suite.
Can you please enlighten me?

@tysonclugg this is because we are configuring the environment to tell the tests what we are expecting to see, i.e. in this case we expect pip 9 and above to pass the test and we indicate which pip version we expect via the environment.  What we actually get in the code may be different, so to avoid having to build sanity checks everywhere it's just easier to check the thing we are setting everything based off of.

@techalchemy May I change the decorator to wrap unittest.expectedFailure instead of unittest.skipIf, so that we can see an unexpected pass/fail when the environment doesn't match reality, rather than skipping the test?

@tysonclugg sure but it's a bit different for pytest, just a sec and I'll add it

@techalchemy My mistake, carry on!

OK all, squashed down to 2 commits, building against pip@master, hopefully this is good to go

Small aside: I understand the desire to squash changes down to the minimal required, but I'm a little dismayed that contributions are incorrectly attributed to the person who performed the squash.  Obviously this is more of a social issue than a technical one.
It bummed me out when I finally had a change accepted into @django, only to have a core member gain attribution (in terms of git stats) for the work I did.  Not so much the case here, but you get the gist of what I'm saying.
Don't worry about undoing the squash in this particular instance - just be mindful of the social implications when squashing and/or rebasing commits.
I wonder if git could be improved to maintain attribution for squashed commits?

You should consider the reason for your offer here— are you here to help out the project, or are you merely interest in attribution?

I already said not to worry about undoing the squash - I'm here to help.  My offer to help isn't dependent upon attribution.  I'm sorry I mentioned it, I'm ready to move on.

@techalchemy I've just become aware that GitHub recently added support for “Co-authored-by” trailers on commit messages.  This answers my question regarding if git could be improved to maintain attribution for squashed commits.
Could you please amend the commit message(s) to include the relevant "Co-authored-by" trailer lines to include myself and the others who have contributed in various forms (ie: code, ideas, etc) to this PR?

Please stop derailing this PR over a line of testing instructions that is already attributed to you.

All set on the changes

Thanks everyone!

Hi all, a Jazzband roadie here.
@tysonclugg has asked me to weigh in here with regard to a missing attribution of contribution for the comment above and for a commit @tysonclugg has allegedly contributed as well (in his own words: "to add the pipmaster factor to tox.ini and travis.yml"). I can't verify the latter as it no longer is part of the pull request or the set of commits that landed in master due to the way the commits of this pull request were squashed.
It's a particular unfortunate combination of circumstances that requires reminding everyone of the existence of the Jazzband code of conduct which governs the conduct in Jazzband projects. One aspect of it is the ethical and professional manner in which we must collaborate, including respecting the right for attribution of contributions by fellow members.
Reading through the comments of this PR I see there were multiple opportunities to acknowledge @tysonclugg's contribution (e.g. keep the commit intact, adding a “Co-authored-by” line to the commit message, updating an AUTHORS file, leaving a code comment etc). As none of these steps were taken and multiple requests for attribution (e.g. https://github.com/jazzband/pip-tools/pull/657#issuecomment-387286401) have been ignored this is a violation of the Jazzband Code of Conduct.
Additionally I've found the response to the requests for attribution lacking the appropriate respect towards @tysonclugg as it specifically boasted to take great care to attributing contributions (and failing to do so) and called @tysonclugg's motivation to contribute into question at the same time. I want to be clear that this is not acceptable behavior towards fellow Jazzband members as it does not cater to a welcoming community.
Here are the actions to follow:

@techalchemy Please add an attribution for @tysonclugg's contribution somewhere along the lines of other contributions to this project.

@techalchemy Consider offering an apology to @tysonclugg for the misconduct in professional and ethical aspects of this collaboration.

@tysonclugg Please accept my apology for the experience you've had and my thanks for working on resolving it by reaching out to me.

This pull request will be locked as it's not an appropriate avenue for feedback. Please write to<EMAIL_ADDRESS>instead.

Update on the actions taken:

@vphilippon agreed to add an attribution of @tysonclugg's contribution to the CHANGELOG upon the next release.

@techalchemy has been freed from the action to add an attribution of @tysonclugg's contribution as stated in the previous comment.

I consider the matter resolved now.


fix(hotkeys): preserve hotkeys if changed, and reduce re-rendering

Context
fixes #3598
Hotkeys were not getting restored after refresh.
There were too many re-renderings happening because of modals and useTranslation
Changes & Results
Fixes the re-renderings as shown below by refactoring and not changing language if the same
Current behaviour
https://github.com/OHIF/Viewers/assets/7490180/864562db-997d-4178-9a08-dd657a976edd
After this PR

No re-rendering -> No search for studies

Testing

Changed hotkeys in the user preferences should be preserved after refresh
the app should feel faster after save preferences (since there is no re-renderings)

Checklist
PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] OS:
[] Node version:
[] Browser:

Looks great. Great catch on the rerendering!


Random.stateless_binomial is not available on GPU

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.10
Custom Code
Yes
OS Platform and Distribution
Linux
Mobile device
debian
Python version
3.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
cuda11/cudnn8
GPU model and memory
Tesla V100-PCIE-16GB
Current Behaviour?
In distributed environnement  on  2 GPU
python    -u mainTest2.py --n_gpu=2
generates a bug generating binomial random sample (works well with normal random)

Standalone code to reproduce the issue
import numpy as np
import tensorflow as tf
import time
import sys, os
import argparse
import matplotlib

parser = argparse.ArgumentParser()
parser.add_argument('--n_gpus', type=int, default=1)
args = parser.parse_args()
n_gpus = args.n_gpus

device_type = "GPU"
devices = tf.config.experimental.list_physical_devices(
          device_type)
devices_names = [d.name.split('e:')[1] for d in devices]
print("DEVICES ", devices_names)
print("NGPU USED" , n_gpus)
strategy = tf.distribute.MirroredStrategy( devices=devices_names[:n_gpus])

with strategy.scope():
    optimizerControl= tf.keras.optimizers.Adam(learning_rate = 1e-3)
    modelControl = tf.keras.Sequential([tf.keras.layers.Dense(8, activation = tf.nn.relu),
                                        tf.keras.layers.Dense(1 )])
    # CREATE the random generator under the strategy scope : then using distribute different streams are used
    gen = tf.random.Generator.from_seed(1)

@tf.function
def cal():
    #x= gen.normal( [])
    x= gen.binomial(shape=[1], counts= 100. , probs=0.2)
    return x

x= strategy.run(cal)
print(" x val ", x)

Relevant log output
Traceback (most recent call last):
  File "/gpfsgaia/home/C97830/MASTEREQUATION1D/PYTHON/mainTest2.py", line 36, in <module>
    x= strategy.run(cal)
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/distribute/distribute_lib.py", line 1315,
in run
    return self._extended.call_for_each_replica(fn, args=args, kwargs=kwargs)
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/distribute/distribute_lib.py", line 2891,
in call_for_each_replica
    return self._call_for_each_replica(fn, args, kwargs)
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/distribute/mirrored_strategy.py", line 676
, in _call_for_each_replica
    return mirrored_run.call_for_each_replica(
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/distribute/mirrored_run.py", line 83, in c
all_for_each_replica
    return wrapped(args, kwargs)
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/util/traceback_utils.py", line 153, in err
or_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/C97830/ANACONDA_TF271/envs/TFNEWGPU/lib/python3.10/site-packages/tensorflow/python/eager/execute.py", line 54, in quick_execu
te
    tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
tensorflow.python.framework.errors_impl.InvalidArgumentError: Cannot assign a device for operation binomial: Could not satisfy explicit dev
ice specification '/job:localhost/replica:0/task:0/device:GPU:0' because no supported kernel for GPU devices is available.
Colocation Debug Info:
Colocation group had the following types and supported devices:
Root Member(assigned_device_name_index_=1 requested_device_name_='/job:localhost/replica:0/task:0/device:GPU:0' assigned_device_name_='/job
:localhost/replica:0/task:0/device:GPU:0' resource_device_name_='/job:localhost/replica:0/task:0/device:GPU:0' supported_device_types_=[CPU
] possible_devices_=[]
StatefulRandomBinomial: CPU
_Arg: GPU CPU

Colocation members, user-requested devices, and framework assigned devices, if any:
  binomial_resource (_Arg)  framework assigned device=/job:localhost/replica:0/task:0/device:GPU:0
  binomial (StatefulRandomBinomial) /job:localhost/replica:0/task:0/device:GPU:0

Op: StatefulRandomBinomial
Node attrs: S=DT_INT32, T=DT_FLOAT, dtype=DT_INT32
Registered kernels:
  device='CPU'; dtype in [DT_INT64]; T in [DT_DOUBLE]
  device='CPU'; dtype in [DT_INT64]; T in [DT_FLOAT]
  device='CPU'; dtype in [DT_INT64]; T in [DT_HALF]
  device='CPU'; dtype in [DT_INT32]; T in [DT_DOUBLE]
  device='CPU'; dtype in [DT_INT32]; T in [DT_FLOAT]
  device='CPU'; dtype in [DT_INT32]; T in [DT_HALF]
  device='CPU'; dtype in [DT_DOUBLE]; T in [DT_DOUBLE]
  device='CPU'; dtype in [DT_DOUBLE]; T in [DT_FLOAT]
  device='CPU'; dtype in [DT_DOUBLE]; T in [DT_HALF]
  device='CPU'; dtype in [DT_FLOAT]; T in [DT_DOUBLE]
  device='CPU'; dtype in [DT_FLOAT]; T in [DT_FLOAT]
  device='CPU'; dtype in [DT_FLOAT]; T in [DT_HALF]
  device='CPU'; dtype in [DT_HALF]; T in [DT_DOUBLE]
  device='CPU'; dtype in [DT_HALF]; T in [DT_FLOAT]
  device='CPU'; dtype in [DT_HALF]; T in [DT_HALF]

	 [[{{node binomial}}]] [Op:__inference_cal_68]

'Because no supported kernel for GPU devices is available.'ì

It is not related to the distributed scope is that this kernel is registered available only on CPU device:
https://github.com/tensorflow/tensorflow/commit/1dc93053290e0c42f2d5f5515ba6a3577b74dd3e#diff-f1dd8832f1da21ec2f59283757f802d8db2d3951062a12281e5019f24bba016bR528
import tensorflow as tf
tf.config.set_soft_device_placement(False)

counts = [10., 20.]
# Probability of success.
probs = [0.8]
with tf.device("CPU"):
  binomial_samples = tf.random.stateless_binomial(
    shape=[2], seed=[123, 456], counts=counts, probs=probs)
  print("CPU ", binomial_samples)
with tf.device("GPU"):
  binomial_samples = tf.random.stateless_binomial(
    shape=[2], seed=[123, 456], counts=counts, probs=probs)
  print("GPU ", binomial_samples)

P.s. This is a feature request.

/cc @brianwa84

Hi @sachinprasadhs , I was looking into this bug and I would love to correct it but the "tensorflow/python/ops/gen" file which actually contains the code of stateless_random_binomial is git ignored, so if I can help and contribute to it in any way do let me know!
Also if tensorflow is having any feature requests or bug fixation to be done, kindly let me know I would love to contribute!

Hi @Sar-thak-3 , Thanks for showing interest to help Tensorflow community.
gen files are  autogenerated files during build, original source code for these will be in .td files.
You are always welcome to choose any issues which doesn't have any PR linked to create a new one.
Ideal ones for contributions is to look for "stat:contribution welcome" label.
Here is a link for all such issues. Thanks!

Hi @sachinprasadhs , thanks a lot for your prompt response. It would be great if you can share the contributing guidelines which include the libraries and things that need to be installed priorly. As I was trying to make changes in C++ files, but while running simply as C++ file it is showing a lot of errors. I believe there is a specific procedure to first set up which I can't find and I need your help in the same.

Please refer: https://github.com/tensorflow/tensorflow/blob/master/CONTRIBUTING.md

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Question sentinel slave

Hello,
I read that when sentinel is used and we put role: 'slave', ioredis will try to connect to the slave available but what happen if no slave is available and only the master is, will ioredis use the master to make its read or will it wait until a slave comes back online? In the case where there is one slave for the master waiting for the slave will stop the ioredis instance for reading anything right?

If there's no slave available and the role option is set to the 'slave', then ioredis would retry to reconnect to the sentinel and fetch the slaves info again or just reject with an error according to the value of the sentinelRetryStrategy option(Check out README.md for more information). ioredis would never connect to the master when role is slave.

Would it be bad you think to have an option that would let it log to a master while no slave is available but still try to relog to a slave using the sentinelRetryStrategy to connect to the slave once its available?
Because in case the slave are unavailable application that were just reading from slave won't be able to work anymore. Maybe have an option like 'enforceRole = false'?

I prefer to implementing this logic on the client side since it's not hard.

I already implementing it, see https://github.com/tangxinfa/ioredis_sentinel_connector
ioredis_sentinel_connector
Best efforts on connect slave with ioredis's SentinelConnector.
WARNING: This module will change the prototype of ioredis's SentinelConnector.
There is no difference when role is "master", but for "slave" following the order:

Local slave.

Master if no slaves.

Local master.

Remote slave.


Upgrade git-commit-id-maven-plugin 9.0.0 -> 9.0.1

This PR contains the following updates:

Package
Type
Update
Change

git-commit-id-maven-plugin
build
patch
9.0.0 -> 9.0.1

Release Notes

git-commit-id/git-commit-id-maven-plugin (git-commit-id-maven-plugin)
v9.0.1: Version 9.0.1
Compare Source
Version 9.0.1 is finally there and includes various bug-fixes and improvements :-)
New Features / Bug-Fixes:
The main key-aspects that have been improved or being worked on are the following:

#754 / #760: Fix an issue where empty git.properties had been generated in submodules when injectAllReactorProjects=true

bump several maven plugins

bump org.apache.maven.plugins:maven-checkstyle-plugin  from 3.3.1 to 3.4.0
bump org.apache.maven.plugins:maven-release-plugin from 3.0.1 to 3.1.0
bump org.apache.maven.plugins:maven-surefire-plugin from 3.2.5 to 3.3.0
bump org.apache.maven.plugins:maven-clean-plugin from 3.3.2 to 3.4.0
bump org.apache.maven.plugins:maven-dependency-plugin from 3.6.1 to 3.7.1

Getting the latest release
The plugin is available from Maven Central (see here), so you don't have to configure any additional repositories to use this plugin. All you need to do is to configure it inside your project as dependency:
<dependency>
    <groupId>io.github.git-commit-id</groupId>
    <artifactId>git-commit-id-maven-plugin</artifactId>
    <version>9.0.1</version>
</dependency>

Getting the latest snapshot (build automatically)
If you can't wait for the next release, you can also get the latest snapshot version from sonatype, that is being deployed automatically by github actions:
<pluginRepositories>
    <pluginRepository>
        <id>sonatype-snapshots</id>
        <name>Sonatype Snapshots</name>
         <url>https://s01.oss.sonatype.org/content/repositories/snapshots/</url>
    </pluginRepository>
</pluginRepositories>

Even though the github actions will only deploy a new snapshot once all tests have finished, it is recommended to rely on the released and more stable version.
Known Issues / Limitations:

This plugin is unfortunately not working with Heroku which is due to the fact how Heroku works. In summary Heroku does not copy over the .git-repository but in order to determine the git properties this plugin relies on the fact that it has access to the git-repository. A somewhat workaround to get some information is outlined in https://github.com/ktoso/maven-git-commit-id-plugin/issues/279#issuecomment-301284022
Using maven's plugin prefix resolution (e.g. mvn com.test.plugins:myPlugin:myMojo) might result in unresolved properties even with <injectAllReactorProjects>true</injectAllReactorProjects>. Please refer to https://github.com/git-commit-id/maven-git-commit-id-plugin/issues/287 or https://github.com/git-commit-id/maven-git-commit-id-plugin/issues/413#issuecomment-487952607 for details and potential workarounds

Reporting Problems
If you find any problem with this plugin, feel free to report it here

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

Suggested commit message:
Upgrade git-commit-id-maven-plugin 9.0.0 -> 9.0.1

See:
- https://github.com/git-commit-id/git-commit-id-maven-plugin/releases/tag/v9.0.1
- https://github.com/git-commit-id/git-commit-id-maven-plugin/compare/v9.0.0...v9.0.1


ARM fix

I recently compiled nix for my beagleboard black - it went smoothly, but I got two compile errors, both in uio.rs.
The parameter offset to ffi::pread and ffi::pwrite are i32 on my arm. Maybe they should be isize? I am not sure how to express that in rust in a portable manner.

I think using off_t here is the right thing to do but off_t does not have the right value for ARM in your environment.  Reading the libc lib.rs, it would appear that off_t should be an i32: https://github.com/rust-lang/rust/blob/8b7c17db2235a2a3f2c71242b11fc429a8d05a90/src/liblibc/lib.rs#L454

I am sorry, my original bug description was not super great. I was back at my machine today and conjectured a fix real quick in attached pull request.

Fixed by d9808975608e7818be2d564357469dab0a31e1d2

thanks!


Parse VS 2017 projects also when namespace is present

Test uses a sample from 4.6.1 VS2017 project format, with a namespace and slightly different version storage (node instead of attribute).

There seem to be 2 things there -
An xml namespace  at the root node, and that the package's Version can be an element not an attr?

Turns out the updater needs to be changed slightly as well, otherwise NuKeeper will put two versions of the dependency in.

Relevant https://github.com/NuGet/Home/issues/4358 https://github.com/dotnet/cli/issues/5425

Workaround used for dotnet add package not removing old version every time: run dotnet remove package first.


Ollama model rename feature

is it possible to rename the ollama models so i can give users a more friendly model description name and they can choose models more clear.

Closing this in favour of #1041, let's continue our discussion there!

I cannot figure out after several hours how to simply rename the model I'm using:
mannix/llama3.1-8b-abliterated:latest

Can't figure it out, UI if it supports this is very confusing :-)


Bad option: "extends"

Hi,
I'm trying to use the extends feature of jshint to specify a particular configuration for my tests folder.
I use the jshintrc option specifying the jshintrc in this way:
jshint: {
    dev: {
        options: {
            jshintrc: '.jshintrc',
            debug: true,
            noarg: false
        },
        src: [
            'app/**/*.js'

        ]
    },
    tests: {
        options: {
            jshintrc: 'tests/.jshintrc'
        },
        src: [
            'tests/*.js',
        ]
    }
},

The first task works correctly and it validates everything, when, instead, I run the second task, I get:
Running "jshint:tests" (jshint) task
Linting tests/something.js ...ERROR
>> Bad option: 'extends'.

Is this grunt script not supporting the extends option or am I doing something wrong?

ah nevermind... I was on an outdated version. Sorry!


ispyb omega_start value should be read from hardware instead of supplied by params

from PR review comment by @DominicOram in #1279
"Should: We should be able to read the omega from the smargon when we do the pin tip detection, can we make an issue for that?"

Remember that directionality on smargon vs what we want to put in ispyb is flipped


[ENHANCEMENT] Allow saving duration from JSON editor

Now that a JSON editor was added in #1012, this PR allows saving a new duration from the editor.
This is just the first PR of multiple that will allow users to pick if they want to save the selected time range as default and/or the selected variables as default.
Screen recording
https://user-images.githubusercontent.com/9369625/236255218-cfdd270f-4a0a-4ff2-beac-55b055abd169.mp4
Checklist

[x] Pull request has a descriptive title and context useful to a reviewer.
[x] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, IGNORE.
[x] All commits have DCO signoffs.

UI Changes

[x] Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
[x] Code follows the UI guidelines.
[x] Visual tests are stable and unlikely to be flaky. See Storybook and e2e docs for more details. Common issues include:

Is the data inconsistent? You need to mock API requests.
Does the time change? You need to use consistent time values or mock time utilities.
Does it have loading states? You need to wait for loading to complete.

Other question: do you think it's worth adding a test for this?

Yeah I'm definitely planning to add an e2e, I've created a few tests with Playwright in previous PRs but I'm not super fast with it yet. Since this is going to be followed quickly by a few related features (saving defaults when currently selected change), I'd like to create the tests all at once.
Also, since the JSON edit flow isn't tested yet, I have some questions about sharing constants. For instance, I'll need to update the code icon button so it can be clicked and am planning to import this 'Edit JSON' constant but since it lives in @perses-dev/dashboards, how can I use it in @perses-dev/e2e? Should I move it to @perses-dev/core, but I noticed that e2e doesn't import from core yet, is this okay to add as a dependency?

Going ahead and merging since I want to include this in a snapshot release I'm about to do that will test out a few of my open PRs like #1138 and tooltip highlighting / echarts-nightly branch.