5min test

Hello. I see you closed this, but thanks for opening it.
Just FYI I implemented a number of the same improvements while traveling today:

I fixed the tera issue. I was encountering that in live too and was just editing the number back locally.
Added prolonged power
Switched to minutes because that makes more sense for humans.

I don't plan to refresh every 5 minutes or bind to 80:

5 minutes is too quickly for how much data I download on my slow connections. Although I haven't watched how often the urls update, with 5 threads it takes me more than 5 minutes to refresh all new data already so it would effectively be a loop {}.
I don't want to bind to 80 because I feel the best  deployment for rust webapps right now is running as a non-root user behind a nginx reverse proxy.

Anyway, neat of you to open this. Tell me if you fork it into something cool.

Hey!
I actually opened this accidentally when I was trying to do a merge into the master branch of my repo, but good to know you appreciated it rather than were annoyed by it.
Yea I made the 5 minute change after a request from my guild members, although soon after discovered that it seems as though Blizzards API caches AH data for an hour anyway.
I am currently running it on a VPS on VPSDime, and it pulls down the data for all servers in about 4 seconds, but I can imagine on a slower connection that it'd have a bit more trouble.
Thanks for the tip on not binding to port 80, modifying your webapp was the first time I'd touched anything rust, so I'm not that familiar on the do's and the don'ts, but making something that has been tried and tested public facing does make a lot more sense in reflection.

No annoyance at all about the pull request. That's what github is for. :) Worse case scenario, someone politely declines the change and you move ahead with it in your own fork. Much better than the old drama of the Sourceforge days.
As for the 5 minute change: You've probably already seen this but if you check the 'AUCTION DATA STATUS' call with your api-key on https://dev.battle.net/io-docs you'll see it contains a lastModified field. blood-money is using that to determine whether it's worth redownloading data for the realm. Like you say, they cache it for around an hour. This is probably why TUJ only bothers updating once an hour.
4 seconds is really fast! I bet your VPS's connection is much faster than mine. Although I wonder if what's going on is it takes a a bit longer the first time, and then it's very fast for most of the subsequent calls because lastModified hasn't changed? (i.e. so it isn't bothering to download the auction house data again).
Regarding port 80: Not a rust thing really. It's just that I don't want the code to use one of the privileged root-only ports (i.e. < 1024) by default. Instead hardcodes the port numbers now (3000 for us and 3001 for eu) which isn't that great either (at minimum it should accept a commandline flag). There's nothing wrong with using port 80 in your own deployment.
I like to run webapps behind nginx in general:

It means I only need to set up https certificates once, which is especially nice because I don't like using them with python or rust (go has some pretty easy interfaces to using certificates, though).
nginx is pretty resilient to things like slow-connection DoS whereas Iron itself (the web framework I'm using) isn't quite there yet.
Neither of these are an issue for just running it for your own guild though.

I suspect that's just confirmation of things you already knew or suspected. Thanks for poking at things. :)


This corrects the suffix placement for several images.

We use the pattern ${{package.name}}-foo so the suffix comes after the parent package name, but prior to the subcomponent.

Blah, I missed one place in the rekor config :-/


Update vsphere_overview.json

What does this PR do?

The out of the box dashboard does not show any event when template variable  $vcenter_server and $vcenter_datacenter is used
Motivation

https://datadog.zendesk.com/agent/tickets/1862372
The code suggest $vcenter_server.value and $vcenter_datacenter.value is being used currently in for vSphere events widget defination. However, when we select a value for template variable $vcenter_server and $vcenter_datacenter, the widget shows no result. This is because the widget query is then scoped to source:vsphere $vcenter_server.value $vcenter_datacenter.value. This works when the query is changed to  "source:vsphere $vcenter_server $vsphere_datacenter". The events that are ingested by Vsphere integration has a no tag named vcenter_datacenter but has vsphere_datacenter.
Additional Notes

We can reproduce the error in demo org here https://app.datadoghq.com/screen/integration/249/vmware-vsphere---overview?_ga=2.36746720.1513437160.1727045382-1943259521.1724729273&_gl=1*18s4xm3*_gcl_au*ODM5MTQzNTQ1LjE3MjUyMzYxMjk.*_ga*MTk0MzI1OTUyMS4xNzI0NzI5Mjcz*_ga_KN80RDFSQK*MTcyNzM5Nzk4Ni4xMTguMS4xNzI3NDAyOTA1LjAuMC4yNzA1OTkyNTQ.*_fplc*UHZGTTJDR21kSGhLTGI5S0VWWm5qS3hXYk1PUEhtZFQ5cGRmaGhtQjA5TFNHVGk0MzlOZnE4VHlnVTU0bVJQb3IlMkZ2R3hqZ0JDbmNsOHRPOUp1MzNGS0tET2I3T1VyOEcwQXNvRld1cElpaDNWRE5qSlhHNkFqNVRHS204ZlElM0QlM0Q.&fromUser=false&refresh_mode=sliding&from_ts=1727399315230&to_ts=1727402915230&live=true
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Here is the support notebook that we may use to verify https://app.datadoghq.com/notebook/9920192/sayed-sep-27-2024-13-13?range=172800000&tpl_var_vcenter_server[0]=vcsa-nyc-01&tpl_var_vsphere_datacenter[0]=us-nyc-prod-01&start=1727234080463&live=true

Thank you everyone :)


Work with IPv6 in the new tracker.

add sort by task.

cc @rongou

ugr .. where's this cancelled error coming from ...


[Chrome] bootstrap.js as a chrome extension content script

Mac OS X El Capitan 10.11.6
Chrome Version 54.0.2840.98 (64-bit)
In the context of chrome extensions, if the bootstrap javascript is executed as a content script on a page that already uses bootstrap javascript, these plugins will no longer work.
To reproduce:

Download this repo: https://github.com/dgbrokaw/bootstrap-extension.
Load the repo folder as an unpacked extension in chrome: chrome://extensions/.
Open the page http://getbootstrap.com/javascript/#dropdowns.
Click on the dropdown buttons.  They will not open.

This seems like you're using bootstrap for what it's not intended, and noting that this then conflicts on sites that use it as intended? I would think the onus is on you there to debug what you're doing at your end in the extension?

Before executing, you could check if there's already an identical named function.
  if ($.fn.bootstrapFunction) {
    // bootstrap.js already included on the site
  } else {
    // execute
  }


The --remote argument is not considered in umpf merge.

I tried to merge a remote branch with umpf merge --remote=downstream_repo feature/fancy, but the branch feature/fancy was not found. I could resolve this problem by either checking the branch out locally or by using umpf merge downstream_repo/feature/fancy. Is the --remote argument supposed to work as I expected?

No, the argument is always the full branch name. Exactly what git merge would expect as well. The remote argument is mostly there for historic reasons: It can specify the prefix that should removed from the branch when creating the text for the subject and commit message. Now umpf has been improved to detect that automatically so it is no longer needed in most cases.


Only one dynamic mesh works in the scene if it’s on an imported FBX

Using Unity 2020.3.14 & Steam Audio v.4.0.2
If I import geometry from an external 3D package such as blender or Maya, I am unable to use more than one dynamic mesh in the scene. Multiple meshes will be exported as dynamic (with the Steam Audio Geometry & Dynamic Object component) but only one will occlude audio and influence the reverb. Generally this is the last one exported as dynamic.  It doesn’t matter if the FBX has multiple meshes or just one.  It’s the same every time
Using the default Unity cubes as dynamic objects works correctly.  I can export any number of Dynamic objects without any problems.  There is no shortage of the number of dynamic meshes that I can make with the built in primitives.
If I combine imported meshes with unity cubes, I can use as many cubes as I want but I'm still limited to only one working imported mesh/game object as a dynamic object.

Thank you so much for taking a look into this strange bug!  Beyond this bug, I have been really enjoying Steam Audio!

Hey @Julianx9000 - Sorry for late response. I tried to reproduce the issue with FBX meshes but they behave as expected when marked as dynamic objects. Make sure to export the scene again after marking your FBX meshes as dynamic objects, they might still be part of static mesh if you did not export the scene again. I assumed you are using default ray tracer and followed the steps here - https://valvesoftware.github.io/steam-audio/doc/unity/dynamic-object.html Also, see the Note in the documentation above.
If that does not help, can you share a small standalone project where we can repro this issue or may be FBX model you used. Thanks.

Closing this issue since we haven't been able to reproduce it.


Typo "commmand" > "command"

@TheQwerty : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @TheQwerty! We appreciate your help on fixing our docs.
#sign-off


NullReferenceException on iOS

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
StackTrace: at Plugin.SharedTransitions.Platforms.iOS.Renderes.SharedTransitionPageRenderer.Dispose
Version Number of Plugin: latest
Device Tested On: iPhone 6s and iPhone 8
Version of VS: latest
Version of Xamarin: latest stable
This is just happening when performing some normal navigation that works fine on Android.

Will disable the library for now while I do other testing.
If it's an error with the library can we have a fix ASAP please.
Sorry, bit stressed atm.

i'm planning an update for this weekend, i think they changed something in the latest versions i have some problems with frames too.
i already fixed this and pushed to the master branch, i need to make some test and maybe i can pubblish it tomorrow

new version should be available today


feat: Add the ability to change the nonce

Try out this version of the Hiro Wallet - download extension builds.

In the preview transaction we can now update the nonce before sending.

cc @jasperjansz for design guidance

@kyranjamie Yes indeed, it might be better to add it to the form,  but we have a problem for the signing popup : it does not have any form, just the read only view, maybe it's ok to use this pattern there ?

we have a problem for the signing popup : it does not have any form

Ah, yes. I guess we have to do it in the drawer there, then.

we have a problem for the signing popup : it does not have any form

Ah, yes. I guess we have to do it in the drawer there, then.

There is no drawer for the signing confirmation, but we can add a second at the bottom for advanced. Previously users were able to easily change account and network during the confirmation screen, this is no different.

All in all I would recommend a different approach to this type of feature from what you've implemented. It seems like you're duplicating a lot of the logic around how we generate the transaction itself, where you only really need to inject a different value into the nonce state, and everything else will automatically update. This is what I'd recommend if we want to do this feature:

create a component that contains a number input
create a new atom called something like customNonceState which only takes input from this component
add customNonceState to this area, with a condition of if(customNonce) return customNonce

the complexity will come from if we want to reset this per transaction, or persist it going forward.

@aulneau Got it ! You approach might be simpler. Basically use the state dependencies to reinitiate the transaction itself via useMakeTransferEffect. I will try that thanks !

Just running this branch, isn't it a bit strange not to have some kind of label here for the dropdown? @jasperjansz

Yep agreed. Let's getting some guidance from @jasperjansz on the above.

Just as a note, this is meant only for advanced users, hence the lack of label. But maybe we can do something better.

I already added it to the feat/rbf branch, closing this in favour of https://github.com/blockstack/stacks-wallet-web/pull/1639


checkbox: How to display on the specified node ?tks

Use CSS selectors and set display:none to the checkboxes that you do not need. I can't help you with a selector as that is specific to your structure.
Best regards,
Ivan


Implements Practice Test Taking

Implements the Functionality for Taking Practice Tests in the FrontEnd

Task linked: CU-3cjweqx Create Test Display Inner View


Make ALVR client detect changes of audio devices

The changes updates the logic of play_audio_loop and record_audio_loop, to support detection of changing audio device. The idea is using the native API android.media.AudioDeviceCallback to listen the change of audio device, and pass the ID and the properties back to the client core lib.
In the logic of play_audio_loop and record_audio_loop, the audio streaming thread will be changed to a loop, and keep openning the audio stream as long as the loop is still running. The stream will be closed once it detects the audio device is added or removed, and letting the thread re-open an updated stream.
There is an issue at the moment when I test it. The input stream sometimes stuck at open_stream() without returning, which can only be resolve by pause-resume. I am not sure why it happens or how to solve it in code.

nOt StAlE

Whops, deleting the zarik-dev branch closed this PR. It can be reopened against the master branch if necessary


/** convertin to ###*

While technically correct, no one wants that extra * there.

No, they do.

Really? It's just from comments like
/**
 * blah
 **/

Is there something I'm missing? Are they for a purpose?

Yes, because it allows you to make jsdoc comments in coffeescript.
###*
# gets the circumference of a circle
# @param r radius
###
getCircumference: (r) ->
  Math.PI * r * 2

I didn't realize this was the syntax for coffescript docs.


Added in Readme the optional functions

Added in Readme file the optional functions

Ok, I will review this errors.


ObjectID support

This is the error I'm getting in the console: Meteor does not currently support objects other than ObjectID as ids.
Im using winstondb for logging to a collection.  This way when I log it can go to the console and also be persisted.  It seems that when it creates a document it makes an ObjectID('something') as the _id field (like most other mongodb databases do) instead of just the random string that Meteor does.
I was under the impression that meteor could still function with this type of object Id. If so it would be great if you could support it in your frontend.
Meanwhile I'll see if I can tell winston to just use strings instead of object ids.
Thanks!

Trying to find the problem, but success so far. I have created a temporary solution, which transforms the IDs into strings and shows them, however, this will cause problems updating documents.
This is only in developement branch for now.

Cool I will give it a try. Thanks!

Seems to be working for me.  Thanks.

ObjectID support is implemented, check it out! :)


kubectl-ko: add new command ovn-trace for tracing ovn lflows only

[x] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Features
Bug fixes
Docs
Tests

Which issue(s) this PR fixes:
Fixes #(issue-number)
WHAT
copilot:summary
copilot:poem
HOW
copilot:walkthrough

Need to update the doc


Add anchor links to docs headers

Headers in our docs don't contain anchor links to enable us to link to a specific section of a specific page. For example it's currently impossible to link directly to Step 1: Setup a Hosted Database on this page.
We should be able to link to that section with a URL like https://ethyca.github.io/fidesops/deployment/#step-1-setup-a-hosted-database, and similarly for all significant headers.

Headers tags should already work as anchor links by default (https://ethyca.github.io/fidesops/deployment/#step-1-setup-hosted-database) for at least through h3 - is there something else needed?


Can't install spaceship using npm

I am getting this error, please help me.

Environment
Zsh version: zsh 5.2 (x86_64-ubuntu-linux-gnu)
Operating system: Linux - Ubuntu 20.04

The error is pretty self-describing: you don't have permissions to install spaceship /root folder. Please, make sure you're not running trying to install it under root user — this might be harmful to your system.


#14 Updated Cake.Core reference

Updated Cake.Core reference as described in #14

I've accepted the previous PR for this.


Markdown preview fails to load required CSS and JS on VSCode startup

VSCode Version: 1.46.0
OS Version: Windows 10 / Windows_NT x64 10.0.18363

Steps to Reproduce:

Open any .md / markdown file
Open the preview to the side
close the VSCode window
re-open VSCode

this should automatically re-open the same previously open workspace/folder
the Markdown file and its preview tab should re-open automatically

Expected outcome:
Markdown preview is rendered according to current theme - or to what's setup in markdown.style / markdown.previewStyle.
Actual outcome:
Only inline styles and default browser styles are rendered, leaving the preview looking distorted.
looking at the webview console, I can see 7 errors prefixed with Failed to load resource: net::ERR_UNKNOWN_URL_SCHEME for:

pre.js
markdown.css
highlight.css
checkbox.css
katex.min.css
index.js

and this is how my preview then looks like:

Workaround:

Close preview
re-open preview

Does this issue occur when all extensions are disabled?: Could not test.

Maybe related to #100727

hi @gjsjohnmurray - yes, i can confirm that i saw the same behavior with markdown.previewStyles and markdown.styles

@mjbvz I saw this happened with the insider version as early as May 21 and mysteriously back to normal May 25. I don't think the commit 61f796e is the real cause, though the problem may disappear after you revert it back.

@mjbvz I've had reports that this was fixed in the Insiders release 9b620b2, however it is broken again for every user I've checked with in:
Version: 1.47.0-insider (user setup)
Commit: 6c235c723cb2d8defb840f732eac620f54aa8b9d
Date: 2020-06-26T06:20:06.931Z
Electron: 8.3.3
Chrome: 80.0.3987.165
Node.js: 12.13.0
V8: 8.0.426.27-electron.0
OS: Windows_NT x64 10.0.18362
How could this occur?


Number is reset to initial value on pressing the "Done" button

Similarly to the closed issue, I am having a problem when clicking on the "Done" button. The displayed value is reset to the initial value (0.00 in most cases) when using the MoneyMaskedTextController. I can reproduce this on iPhone 5 and Android simulated and on a Samsung s10. I've stripped all my logic out of the code and still the issue happens. Should you need more information, please feel free to check in.
This is my implementation of the issue (I have stripped out unnecessary containers, etc. but I have tested it and it does the same):
Widget _createCurrencyField(Field field) {
    final _currencyController = MoneyMaskedTextController(
        initialValue: 0.0,
        decimalSeparator: ',',
        thousandSeparator: '.',
        rightSymbol: '',
        leftSymbol: '€');

     return TextField(
            controller: _currencyController,
            keyboardType: TextInputType.number,
          );
  }

Storing the Controller in a global variable worked for me.
class _LoginPageState extends State<LoginPage> {
  MaskedTextController controller = MaskedTextController(mask: '000.000.000-00');

@override
  Widget build(BuildContext context) {
  ...
  }

Widget _buildCampoCPF() {
    return Container(
      margin: EdgeInsets.only(top: 20.0),
      child: TextFormField(
        controller: controller,
        ...
      ),
    );
  }

Storing the controller in a variable did not work for me


Add css to the application

Modify the UI by adding CSS to the components

hey @warrenferns , i would like to work on this issue. would you please assign this to me? also can you please host this site online on platform like netlify or vercel?

I have some CSS changes ready, you can assign this to me


Consider returning standard output, and grouping later (e.g. in email output), e.g. with a "GroupOn" field

basic testing done, but some rework required. Benefits to be confirmed

This would require returning pscustomobjects all around. It's do-able, but I'm waiting for a clear reason why  it would be an improvement


Update dependencies

Closes #25

[x] update typescript
[x] Figure out and update other dependencies
[x] BREAKING: Remove node<10 support

Depends on https://github.com/danger/danger-js/pull/996

For the first 5m of my review I was convinced this was on the danger-js repo and I couldn't understand the changes at all. Ha.
This looks good, dropping 10 is fine for new plugins 👍
Thanks, this is good grunt work ❤

@orta Could you take a look at the travis-ci env vars when you get the time? I think the token there no longer works: https://travis-ci.org/danger/generator-danger-plugin/jobs/651034472?


Application signature and Date unseen in new application review page

I submitted an application and signed it with a false name and chose a date that was in the future.   The submission was accepted.  When reviewing the new application, I was unable to locate a place to see the signature or the date signed to verify the false date or name.

I added this in the recent commit and it'll appear at the bottom as:

Closing issue, but this can be reopened or a new one created if further changes are needed. This was a quick and simple fix.


Create count_inversions.cpp

Counts the number of inversions in a list using merge sort. The number of Inversions in a list is the measure of the list's proximity to being sorted in ascending/increasing order.
Description of Change

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

I had previously closed this PR. @kvedala had asked to implement using templates which I did not know then. I have, since then, learned about STL library and templates from YT and formatted the code to the best of my ability. It has passed all tests. Let me know if there are any more changes you'd like to see.

Please enable GitHub Actions in your repository of this fork in this link: https://github.com/rakshitraj/C-Plus-Plus/actions

Done. Please mark this. It is being counted as pending action.

@kvedala @ayaankhan98 please review. Thanks. 🙂

Will merge this PR. If there are any errors, please make a new PR to propose a fix. Thank you. 🙂


Mob 3604 devs want to implement business logic for the engagement launcher approach

Jira issue:
https://glia.atlassian.net/browse/MOB-3604
What was solved?

Remove all the configurations in favor of ConfigurationManager
Get rid of configurations passed through Activity extras
Move extra keys to internal class
Make activities internal
Minor refactoring

Release notes:

[ ] Feature
[x] Ignore
[ ] Release notes (Is it clear from the description here?)
[ ] Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

[ ] Is the feature sufficiently tested? All tests fixed? Necessary unit, acceptance, snapshots added? Check that at least new public classes & methods are covered with unit tests
[ ] Did you add logging beneficial for troubleshooting of customer issues?
[ ] Did you add new logging? We would like the logging between platforms to be similar. Refer to Logging from Android SDKs → Things to consider for newly added logs in Confluence for more information.

Screenshots:

@DavDo You've done a good deal of work! 💪  Agree with @gugalo, it would be much easier to review and spot issues if a huge PR is split into smaller parts.

Thank you guys for your review.
I will try to address comments and split this PR into separate smaller parts, but since there are a lot of connections, I'm afraid that there will still be one that is comparatively big.

Taking into account the work done already, it makes sense to proceed with this PR, isn't it? Otherwise we would have to review everything once again. Not sure how to do better.

Taking into account the work done already, it makes sense to proceed with this PR, isn't it? Otherwise we would have to review everything once again. Not sure how to do better.

I'd also prefer to proceed with this PR, but I'm also ready to split it if you really want it.


perf(Slider): avoid excessive re-renders on thumb position update

Description

The purpose of this PR is to enhances the performance of slider component by optimizing the state update logic. Previously, the minThumbPosition and maxThumbPosition properties were being updated during the componentDidLoad() lifecycle hook, which could trigger unnecessary re-renders and impact performance.
To address this case, the state update logic has been improved to utilize requestAnimationFrame(). This ensures that updates to minThumbPosition and maxThumbPosition are deferred until the next frame render, reducing the potential for excessive re-renders and enhancing overall performance.
Related Issue

Fixes #ISSUE_NUMBER
Documentation

Screenshots (if applicable)

Checklist

[x] I have read the CONTRIBUTING document.
[x] I have read the CODE OF CONDUCT document.
[x] I have reviewed my own code.
[x] I have tested the changes locally.
[x] I have updated the documentation (if applicable).
[x] I have added unit tests and e2e tests (if applicable).
[x] I have requested reviews from relevant team members.

Additional Notes

Hmmm, is it just me or do I see that the tooltip remains a bit behind when dragging the slider thumb(s) 🤔 ?

It doesn't happen here for example, and I guess that since the requestAnimationFrame defers the position update, hint the small delay.

Putting this on hold momentarily. We're not getting 100% of the smoothness we would like.


dask Array blocks not actually chunks

Describe the issue:
Hello, I am probably doing something very wrong. I am trying to iterate over the chunks of a dask array as such:
import dask.array as da
import numpy as np

def unexpected_behavior():
    chunk_size = (25, 25, 25)
    a: da.Array = da.ones(shape=(100, 100, 100), chunks=chunk_size)
    assert a.chunksize == chunk_size

    for block in a.blocks:
        assert block.shape != chunk_size  # unexpected! :(
        print(block.shape)  # (25, 100, 100)

def desired_behavior():
    chunk_size = (25, 25, 25)
    a: da.Array = da.ones(shape=(100, 100, 100), chunks=chunk_size)
    assert a.chunksize == chunk_size

    for i in np.ndindex(a.numblocks):
        block = a.blocks[i]
        assert block.shape == chunk_size

if __name__ == "__main__":
    unexpected_behavior()
    desired_behavior()

It looks like the blocking is only applied along the zeroth dimension.
Environment:

Dask version: 2023.4.1
Python version: 3.10.6
Operating System: ubuntu 22.04.2 LTS
Install method (conda, pip, source): poetry

Oh interesting, using ravel gives me the expected behavior:
import dask.array as da

def expected_behavior():
    chunk_size = (25, 25, 25)
    a: da.Array = da.ones(shape=(100, 100, 100), chunks=chunk_size)
    assert a.chunksize == chunk_size

    for block in a.blocks.ravel():
        assert block.shape == chunk_size

if __name__ == "__main__":
    expected_behavior()

Thanks for the issue @ahnsws. .blocks returns an array-like object where the items in the array are the blocks that comprise the larger Dask array. Hopefully some of the docs here https://docs.dask.org/en/stable/generated/dask.array.Array.blocks.html#dask.array.Array.blocks make things a little more clear.
It sounds like ravel() meets your needs. FWIW you can also index directly into blocks -- here's an example with the array in your original snippet:
In [16]: chunk_size = (25, 25, 25)
    ...: a = da.ones(shape=(100, 100, 100), chunks=chunk_size)

In [17]: a
Out[17]: dask.array<ones_like, shape=(100, 100, 100), dtype=float64, chunksize=(25, 25, 25), chunktype=numpy.ndarray>

In [18]: a.blocks
Out[18]: <dask.array.core.BlockView at 0x1223b8220>

In [19]: a.blocks.shape
Out[19]: (4, 4, 4)

In [20]: a.blocks[0, 3, 1]
Out[20]: dask.array<blocks, shape=(25, 25, 25), dtype=float64, chunksize=(25, 25, 25), chunktype=numpy.ndarray>

I'm going to close this issue as it sounds like you have what you need with .blocks.ravel()


feat: Add support for array buffers and auto-selecting file format

This PR adds:

Support for array buffers, that are needed to send binary formatted multipart data.
Support for auto-selecting the output format of the files that are loaded using the loadFiles function.
Support for Uint8Array for the files upload procedure when upserting a file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

19 of 29 (65.52%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.6%) to 83.114%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/files/files.ts
9
10
90.0%

src/content/content.ts
4
6
66.67%

src/client/BuilderClient.ts
6
13
46.15%

Totals

Change from base Build<PHONE_NUMBER>:
-3.6%

Covered Lines:
299

Relevant Lines:
343

💛  - Coveralls


Error on Install

I am trying to install this, I am on Arch and using NodeJS 5.9.1, I installed Browserify 13, and I tried doing the bash build.bash script but i got this back:
$ bash build.bash
Error: Cannot find module 'bs58' from '/home/jza/Downloads/etherid-js'
    at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
    at process (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
    at ondir (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
    at load (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
    at onex (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
    at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:82:15)
build.bash: line 6: uglifyjs: command not found
Error: Cannot find module 'bs58' from '/home/jza/Downloads/etherid-js'
    at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
    at process (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
    at ondir (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
    at load (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
    at onex (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
    at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:82:15)

npm install bs58
Alexandre Naverniouk
On Mon, Mar 28, 2016 at 2:39 AM, JZA<EMAIL_ADDRESS>wrote:

I am trying to install this, I am on Arch and using NodeJS 5.9.1, I
installed Browserify 13, and I tried doing the bash build.bash script but i
got this back:
$ bash build.bash
Error: Cannot find module 'bs58' from '/home/jza/Downloads/etherid-js'
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
at process (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
at ondir (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
at load (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
at onex (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
at FSReqWrap.oncomplete (fs.js:82:15)
build.bash: line 6: uglifyjs: command not found
Error: Cannot find module 'bs58' from '/home/jza/Downloads/etherid-js'
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:46:17
at process (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:173:43)
at ondir (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:188:17)
at load (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:69:43)
at onex (/usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:92:31)
at /usr/lib/node_modules/browserify/node_modules/resolve/lib/async.js:22:47
at FSReqWrap.oncomplete (fs.js:82:15)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/lexansoft/etherid-js/issues/1


Move println to io

Most languages don't have Println builtin (with the notable exception being rust). The rest have them in an Io / Console or other package. I think we should follow this.

I think we should keep builtin as small as possible and packages more grouped. We should decide what package is for what and break off pieces into those packages.

I agree that some stuff could (and maybe should) be moved from builtin somewhere else, but I'm not sure about Println.
builtin package currently serves 2 purposes:

compiler is aware of that package and uses knowledge about its structure in analyzer and desugarer
things that are used frequently (I personally think that printing comes to this section)

I would say to

I agree that some stuff could (and maybe should) be moved from builtin somewhere else, but I'm not sure about Println.
builtin package currently serves 2 purposes:

compiler is aware of that package and uses knowledge about its structure in analyzer and desugarer
things that are used frequently (I personally think that printing comes to this section)

Also, if there would be strong arguments for moving println out of builtin, then the question arise - should it be io?  I believe there are other languages with this structure (e.g. Odin)

builtin.go actually has a println also. These builtins are meant for simple printing/debugging and the others under fmt have error handling and more sophisticated things happening.
// implementation-specific way and writes the result to standard error.
// Print is useful for bootstrapping and debugging; it is not guaranteed
// to stay in the language.
func print(args ...Type)

// The println built-in function formats its arguments in an
// implementation-specific way and writes the result to standard error.
// Spaces are always added between arguments and a newline is appended.
// Println is useful for bootstrapping and debugging; it is not guaranteed
// to stay in the language.
func println(args ...Type)```

Maybe this would be a good approach?

@swork1 I find it confusing. I saw many posts on the internet people asking why Go have 2 println functions. I don't think we have that problem they have, that they solved by having 2 println functions. We should have only one.
Moving to fmt however might be a good point. Not sure

I'm going to close this one for now. We can reopen if needed


How to visualize the detection results?

Thanks for your great work!I successfully installed the environment and trained the model according to the Readme.md.And I want to visualize the detection results of the test. Could you provide support or could you provide a way that I can visualize the results?Thank you!

Visualization can be enabled by setting --debug 4 in the train or test scripts. Please refer to the opts.py file for more details.


Companion is killed, while setting up the SC Ui Mixer Instance

Hi there,
in my Setup, a UI16 is part of a network with fixed IPs. It has <IP_ADDRESS>, while the PC has <IP_ADDRESS>, Subnet Mask is <IP_ADDRESS>
When I try to setup the Soundcraft Ui Instance in the GUI of Companion, just like I did it before with my Analog Way Pulse 2h, my Hyperdeck, my Atem Mini and my Panasonic AW-HE40s, the Companion-GUI tab says "Houston we have a problem", because the companion app quits, right after setting up the target IP to <IP_ADDRESS>.
What am I doing wrong?
Any workaround on that issue?

Thanx,
I can confirm it is #42. After I have manually installed the plugin it worked.
So cool
Von: Ferdinand Malcher @.>
Gesendet: Freitag, 8. Oktober 2021 16:27
An: bitfocus/companion-module-soundcraft-ui @.>
Cc: Bonn, Mathias GIZ @.>; Author @.>
Betreff: Re: [bitfocus/companion-module-soundcraft-ui] Companion is killed, while setting up the SC Ui Mixer Instance (#44)
Thank you for the report. This looks quite similar to the bug described in #29https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbitfocus%2Fcompanion-module-soundcraft-ui%2Fissues%2F29&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968641375|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=y7cTEuOAlywHv9jdpXeHyAZCjVn6bpS2N6yxtMZgUTo%3D&reserved=0 and #42https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbitfocus%2Fcompanion-module-soundcraft-ui%2Fissues%2F42&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968651381|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WY5Zh9%2FqdnlyqMX1IdLvfkL7IhvNiVvKt%2BhEfOJ1S08%3D&reserved=0. Can you confirm?
You can work around this by disabling and enabling the instance.
The bug is already fixed but not released to the stable companion builds yet. You can upgrade the module manually or try one of the beta builds of Companion.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbitfocus%2Fcompanion-module-soundcraft-ui%2Fissues%2F44%23issuecomment-938686645&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968651381|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lpK%2BCvawmtxXWWSjVKcfHR14exiPUbga6Wt6YpOqVOA%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVSRQYSA4WVISSZYPDJUKG3UF35RVANCNFSM5FTW4JIA&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968661370|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bWrZTm8wavcRCCdneE0HLx%2BjXnlhoG%2FBD%2FmZIjwaKQg%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968661370|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=m8b%2B1ft9V2EHjI3iFWRJTHejogq1CbBLP8KsJpBlkPo%3D&reserved=0 or Androidhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|mathias.bonn%40giz.de|80998a2282f54329254108d98a67a21b|5bbab28cdef3460488225e707da8dba8|0|0|637692999968671359|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=5Cbck7FKqfm8XFdR4Tx5EvQmycI3XOhUJQKVqxSh2o0%3D&reserved=0.

Deutsche Gesellschaft fuer Internationale Zusammenarbeit (GIZ) GmbH;
Sitz der Gesellschaft Bonn und Eschborn/Registered offices Bonn and Eschborn, Germany;
Registergericht/Registered at Amtsgericht Bonn, Germany; Eintragungs-Nr./Registration no. HRB 18384 und/and Amtsgericht Frankfurt am Main, Germany; Eintragungs-Nr./Registration no. HRB 12394;
USt-IdNr./VAT ID no. DE 113891176;
Vorsitzender des Aufsichtsrats/Chairman of the Supervisory Board: Martin Jaeger, Staatssekretaer/State Secretary;
Vorstand/Management Board: Tanja Goenner (Vorstandssprecherin/Chair of the Management Board), Ingrid-Gabriela Hoven, Thorsten Schaefer-Guembel


Threads

Hello. Is it possible to read thread of replied messages?

Duplicate #174. It is a good feature I would like to implement after the v1.0.0.


PyRuler update code.py dependencies: add usb_hid

I updated the PyRuler example to work with CP 6.1.0. import usb_hid was needed.

Ok, everything should be where it's supposed to go now.


Fixes envoy filter to work with istio 1.7.4

Fixes issue #122. Seems like the typed config makes the filter work again with istio 1.7.x. I've used the example here: https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/ext_authz_filter

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: stefanhenseler
To complete the pull request process, please assign brenodemedeiros after the PR has been reviewed.
You can assign the PR to them by writing /assign @brenodemedeiros in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@googlebot I signed it!

@googlebot I signed it!

I think the EnvoyFilter should be gradually deprecated, because the CUSTOM action in AuthorizationPolicy seems to be a less fragile way of doing this: https://istio.io/latest/docs/tasks/security/authorization/authz-custom/
Have you tried it out with AuthService?

This is done. Please check the book-info/readme.md now. thanks!


Update collection.js

Very solid catch! thanks

Just checked, the 3 tests above it have the same typo. Want to fix that too @elgubenis?

sure


Crash when logging into another tenant

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.6.0-beta.2 (commit c4a201e55a13eba77f9037b296a90e0f81b00dbe)
Output from az version
{
  "azure-cli": "2.44.1",
  "azure-cli-core": "2.44.1",
  "azure-cli-telemetry": "1.0.8",
  "extensions": {
    "containerapp": "0.3.20"
  }
}

Describe the bug
Crash after logging into another tenant, I can type the command, it pops a browser, I can login in the browser successfully, and it briefly displays "Retrieving subscriptions" before crashing with the below stack trace:
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x594292]

goroutine 1 [running]:
github.com/azure/azure-dev/cli/azd/pkg/tools/azcli.(*SubscriptionsService).ListTenants.func1(0xc00071ee70?, 0x9?)
        D:/a/_work/1/s/cli/azd/pkg/tools/azcli/subscriptions.go:174 +0x32
sort.partition_func({0xc00071f020?, 0xc00057e300?}, 0x0, 0xe, 0xbdd28?)
        C:/hostedtoolcache/windows/go/1.19.0/x64/src/sort/zsortfunc.go:142 +0xaf
sort.pdqsort_func({0xc00071f020?, 0xc00057e300?}, 0xc0005c84e0?, 0x0?, 0x300?)
        C:/hostedtoolcache/windows/go/1.19.0/x64/src/sort/zsortfunc.go:114 +0x254
sort.Slice({0x923d80, 0xc0005c84e0}, 0x10?)
        C:/hostedtoolcache/windows/go/1.19.0/x64/src/sort/slice.go:23 +0x97
github.com/azure/azure-dev/cli/azd/pkg/tools/azcli.(*SubscriptionsService).ListTenants(0xc000492d00?, {0xc4e4b8, 0xc0004a1530})
        D:/a/_work/1/s/cli/azd/pkg/tools/azcli/subscriptions.go:173 +0x1d5
github.com/azure/azure-dev/cli/azd/pkg/account.(*SubscriptionsManager).ListSubscriptions(0xc000326c80, {0xc4e4b8, 0xc0004a1530})
        D:/a/_work/1/s/cli/azd/pkg/account/subscriptions_manager.go:171 +0x1c8
github.com/azure/azure-dev/cli/azd/pkg/account.(*SubscriptionsManager).RefreshSubscriptions(0xc000326c80, {0xc4e4b8?, 0xc0004a1530?})
        D:/a/_work/1/s/cli/azd/pkg/account/subscriptions_manager.go:74 +0x2c
github.com/azure/azure-dev/cli/azd/cmd.(*loginAction).Run(0xc000093d60, {0xc4e4b8, 0xc0004a1530})
        D:/a/_work/1/s/cli/azd/cmd/login.go:201 +0x2b2
github.com/azure/azure-dev/cli/azd/cmd/middleware.(*MiddlewareRunner).RunAction.func1({0xc4e4b8?, 0xc0004a1530?})

        C:/Users/cloudtest/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:902
github.com/spf13/cobra.(*Command).ExecuteContext(...)
        C:/Users/cloudtest/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:895
main.main()
        D:/a/_work/1/s/cli/azd/main.go:61 +0x24d

To Reproduce
azd login --tenant-id <anotherTenantHostname>
Expected behavior
Not crash?
Additional context
Note that we are  CSP Indirect Provider. Access is being provided via Delegated Admin Permissions.

Hello @ctolkien
While we investigate this error, can you try running this: azd config set auth.useAzCliAuth "true"
This will make azd to use the Azure cli for authentication (as it used to be before version 0.5.0).
I assume you don't have this issue when working with the Azure cli (i.e. az login --tenant-id XXXXX) , right?
Some other questions..  can you tell if the name of the Tenant you are trying to authenticate contains some special characters ?

@weikanglim @wbreza , the signal error: signal 0xc0000005 means Access violation.
Since the error is coming from:
sort.Slice(tenants, func(i, j int) bool {
		return *tenants[i].DisplayName < *tenants[j].DisplayName
	})

I assume that the tenants client is not setting the DisplayName for one of the subs.  (Might be related to the Delegated Admin Permissions).

While we investigate this error, can you try running this: azd config set auth.useAzCliAuth "true"

OK I think the plot is a little bit thicker (I actually had that config set from 0.5 to allow us to deploy to other tenants, but had removed it).

With useAzCliAuth set to true, no crash.
If we az login --tenant anothertenant and then attempt to azd provision resources, we are hitting auth errors during deployment which clearly indicate that we are not using the login credentials from the other tenant, but from my home tenant.

(I am a contributor role from my own tenant, which cannot alter permissions, but admin when logged in directly to the tenant, we can deploy any templates which alter permissions, they all fail with it being "unauthorized")

If I logout of azd via azd logout and then try to provision again, it crashes as per my OP.
Logging back into az does not fix the above crash, I also need to login again to azd.

OK, been trawling through some go code, which admittedly I struggle with, and yes - when we login to another tenant we have an object in the response which does not match what you expect.
Sample response when I az login --tenant foo.onmicrosoft.com:
[
  {
    "cloudName": "AzureCloud",
    "homeTenantId": "tenantId of the tenant I have logged into",
    "id": "id of the subscription listed below",
    "isDefault": true,
    "managedByTenants": [
      {
        "tenantId": "my home tenant id"
      }
    ],
    "name": "SubscriptionName",
    "state": "Enabled",
    "tenantId": "tenantId of the tenant I have logged into",
    "user": {
      "name": "my email",
      "type": "user"
    }
  }
]

Notably, there is no name property on the first object which I think you map to DisplayName, and I imagine also crucially, there is no id on the details of the subscription either. Seems the information from both objects needs to be combined to provide the full details.
From subscriptions.go:


scripts: west: boards: support listing out-of-tree boards

So far only upstream boards were listed. Add public
zephyr_module.parse_modules() function to get information about
out-of-tree board roots. Append them to user provided args.board_roots,
so out-of-tree boards from west modules are listed as well.

@mbolivar-nordic ping


travis: PHP 7.0 nightly added + few improvements

--dev is by default for almost a year

Thanks for the PR!

:


data race in conn.go

hi guys
data race is detected when we run go-zookeeper on : go build -race
WARNING: DATA RACE
Write at :
github.com/samuel/go-zookeeper/zk.(*Conn).recvLoop()
github.com/samuel/go-zookeeper/zk/conn.go:820
Previous read at :
github.com/samuel/go-zookeeper/zk.(*Conn).sendSetWatches()
github.com/samuel/go-zookeeper/zk/conn.go:567
maybe Conn.lastZxid needs to be lock protected?

I can't reproduce anymore. Close?
$ go version
go version go1.11 linux/amd64

$ git log -n1
commit c4fab1ac1bec58281ad0667dc3f0907a9476ac47
Author: Samuel Stauffer<EMAIL_ADDRESS>Date:   Tue Jan 30 11:37:22 2018 -0800

    Fix race conditions in t test

$ go build -race .
$

Reproduce this case when test my app with go race. The log:
2021/06/21 15:21:58 recv loop terminated: err=failed to read from connection: read tcp <IP_ADDRESS>:56952-><IP_ADDRESS>:2181: i/o timeout
2021/06/21 15:21:58 recv loop terminated: err=failed to read from connection: read tcp <IP_ADDRESS>:56976-><IP_ADDRESS>:2181: i/o timeout
==================
WARNING: DATA RACE
Write at 0x00c0001ee000 by goroutine 77:
  github.com/samuel/go-zookeeper/zk.(*Conn).recvLoop()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:910 +0xc4c
  github.com/samuel/go-zookeeper/zk.(*Conn).loop.func2()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:521 +0x2f1

Previous read at 0x00c0001ee000 by goroutine 8:
  github.com/samuel/go-zookeeper/zk.(*Conn).sendSetWatches()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:637 +0x472
  github.com/samuel/go-zookeeper/zk.(*Conn).loop()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:535 +0x5a4
  github.com/samuel/go-zookeeper/zk.Connect.func1()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:223 +0x3c

Goroutine 77 (running) created at:
  github.com/samuel/go-zookeeper/zk.(*Conn).loop()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:516 +0x576
  github.com/samuel/go-zookeeper/zk.Connect.func1()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:223 +0x3c

Goroutine 8 (running) created at:
  github.com/samuel/go-zookeeper/zk.Connect()
      /gopath/src/github.com/samuel/go-zookeeper/zk/conn.go:222 +0x8c6

     // ... more application call stack
     // call like this
     // zkConn, chanConnect, err := zk.Connect([]string{zkAddr}, time.Second*30, zk.WithLogInfo(false))
==================

centos 7
go 1.16.2
zookeeper 3.6.2


Revert "(PUP-8483) Require Ruby 2.3.0 or up"

Our CI system is not yet running against a new enough puppetserver,
since we are still in the process of getting back on to the nightlies
train. Until we can get CI reconfigured to use the latest builds of
puppetserver, we will have to keep 1.9.3 around.
This reverts commit 4be4bc27011ad35201ad10ca71902a1ba2f77dba.

😢

We can put it back soon 😃

We'll have it back up as soon as we start testing against nightlies

CLA signed by all contributors.

Now that we've bumped to newer puppetserver versions in https://github.com/puppetlabs/ci-job-configs/pull/4227, can we close this PR?

Appears to no longer be necessary.


NaN  probabilities  for step_sample

When I run experiment on my dataset, sometimes I got the following error .
  File "train.py", line 136, in <module>
    main()
  File "train.py", line 113, in main
    TrainLoop(
  File "/mnt/dolphinfs/ssd_pool/docker/user/hadoop-mtcv/xiaotingxuan/DiffuProj/text-DiffuSeq/train_util.py", line 183, in run_loop
    self.run_step(batch, cond)
  File "/mnt/dolphinfs/ssd_pool/docker/user/hadoop-mtcv/xiaotingxuan/DiffuProj/text-DiffuSeq/train_util.py", line 202, in run_step
    self.forward_backward(batch, cond)
  File "/mnt/dolphinfs/ssd_pool/docker/user/hadoop-mtcv/xiaotingxuan/DiffuProj/text-DiffuSeq/train_util.py", line 249, in forward_backward
    t, weights = self.schedule_sampler.sample(micro.shape[0], dist_util.dev())
  File "/mnt/dolphinfs/ssd_pool/docker/user/hadoop-mtcv/xiaotingxuan/DiffuProj/text-DiffuSeq/diffuseq/step_sample.py", line 56, in sample
    indices_np = np.random.choice(len(p), size=(batch_size,), p=p)
  File "mtrand.pyx", line 935, in numpy.random.mtrand.RandomState.choice
ValueError: probabilities contain NaN

the  original code is
def sample(self, batch_size, device):
        w = self.weights()
        p = w / np.sum(w)
        indices_np = np.random.choice(len(p), size=(batch_size,), p=p)

I add some code to avoid  NaN error, but I am not sure if it‘s ok to do that. Has anyone also  met this problem, I am really confused, Hope someone can give me some advice
        p = [0 if np.isnan(i) else i for i in p]

Usually the code won't get NaN, but it's ok to add this line.

Hello！I met the same error, and I use your method. But It makes the sum of p is not 1,  there is another error,  because of np.random.choice . so I change the code to:
        p = w / np.sum(w)
        if np.sum(np.isnan(p)) > 0:
            p[np.isnan(p)] = 0
            p = p / np.sum(p)
        indices_np = np.random.choice(len(p), size=(batch_size,), p=p)
        indices = th.from_numpy(indices_np).long().to(device)

But I got the NAN error again.

This is my train.sh, I just use my dataset and change the dim

Hope you can give me some advice

Hello, I also  meet Nan error when using my own dataset.
One advice is to set you batch size smaller, I set --bsz 64.(It works for me)
Another advice is to change the code,
        if hasNan:
            print("has Nan prob p=",p)
            size = len(p)
            p = np.ones(p.shape) * (1/size)
            print("new p =",p)

Ok, I'll try your method, thank you!!!!

commented

Hello, I was wondering how you eventually resolved the issue. I have tried modifying the code as suggested above, but the training still results in NaN errors, which has been troubling me for a long time. If you have any advice, I would be very grateful！

Hello, I was wondering how you eventually resolved the issue. I have tried modifying the code as suggested above, but the training still results in NaN errors, which has been troubling me for a long time. If you have any advice, I would be very grateful！

Actually, when I change the batch size from 128 to 64, the NaN errors disappear. I’m not sure if it will work for you. I think the code suggested above can avoid NaN errors in the sample function. Perhaps you need to post a new issue to describe your problem in more detail, so other people can help you. Good luck!


spring boot 2 support

support for Spring Boot 2

@bliessens thank you very much for your contribution! I don't use this ATM and therefore I can't test your changes. Did you tested it?

Yes we tested with Spring Boot 2.1.10.


Fix build_version.rs

Part of #167 (comment)
Apply of gtk-rs/gir#917
cc @sdroege , @GuillaumeGomez , @kinnison

Now we in deadlock with https://github.com/gtk-rs/examples/pull/310

@GuillaumeGomez please merge to unblock part of the breakage at least :)

CI passed

Thanks!


display multiple attribute values

includes handling fix for the comparison table edge case

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


server function calls not working on Cloudflare

Which project does this relate to?
Start
Describe the bug
Server functions don't work on Cloudflare deployments (via cloudflare-pages preset), wrangler dev, and apparently also on Stackblitz.
[wrangler:inf] GET / 200 OK (56ms)
✘ [ERROR] Error: Context is not available

      at Object.use
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:1:103891)
      at V
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:96187)
      at _
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:108302)
      at null.<anonymous>
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:107762)
      at Object.assign._ [as url]
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:108277)
      at Object.assign.url
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:106760)
      at Object.beforeLoad
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:455:108503)
      at null.<anonymous>
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:1:148928)
      at null.<anonymous>
  (file:///home/nize/dev/web/tss-cloudflare-server-fn/.wrangler/tmp/pages-liyiqW/chunks/runtime.mjs:1:152252)
      at new Promise (<anonymous>) {
    routerCode: 'BEFORE_LOAD'
  }

Your Example Website or App
https://github.com/dotnize/tss-cloudflare-server-fn
Steps to Reproduce the Bug or Issue
or stackblitz: https://stackblitz.com/~/github.com/dotnize/tss-cloudflare-server-fn

pnpm install
pnpm build
pnpm preview via wrangler

The app doesn't work at all due to a server fn call in __root.tsx's beforeLoad. Server fn calls from other routes/loaders (like in index.tsx) also don't work.
Expected behavior
Server functions should work as expected, as it already works on local vinxi dev, node-server builds, or Vercel deployments
repo deployment on vercel, working as expected:
https://tss-cloudflare-server-fn.vercel.app/
Screenshots or Videos
No response
Platform

OS: Linux
Browser: Chrome
Version: 130

Additional context
No response

Having the same issue - one thing (possibly related) I have noted is that even the samples I checked on the website are broken with the same error, for example https://tanstack.com/router/latest/docs/framework/react/examples/start-basic-react-query

i had the same error but in a different use.
i was calling a server function on form submit. while everything work locally, i had the same Context is not available once deployed on cloudflare but using json actually fix it for me in this specific use-case
import { createServerFn, json } from "@tanstack/start";

const action = createServerFn("POST", async () => {
  // this doesn't work on cloudflare but work locally
  // return { msg: 'hello' };
  // this works on cloudflare
  return json({ msg: 'hello' });
});

// ...

function Page() {
  return (
    <form
      onSubmit={async (event) => {
        event.preventDefault();
        event.stopPropagation();
        await action();
      }}
    >
       {/* ... */}
    </form>
  )
}

no clue if this is related in some way

after some digging, seems like the issue might come from nitro
related issue: https://github.com/unjs/nitro/issues/1943

I got it working, this is my config
import type { App } from "vinxi";

import { defineConfig } from "@tanstack/start/config";

const tanstackApp = defineConfig({
  react: {
    babel: {
      plugins: [["babel-plugin-react-compiler", { target: "18" }]],
    },
  },
  server: {
    preset: "cloudflare-pages",
    rollupConfig: {
      external: ["node:async_hooks"],
    },
  },
});

const routers = tanstackApp.config.routers.map((r) => {
  return {
    ...r,
    middleware: r.target === "server" ? "./app/middleware.tsx" : undefined,
  };
});

const app: App = {
  ...tanstackApp,
  config: {
    ...tanstackApp.config,
    routers: routers,
  },
};

export default app;

The rollupConfig.external addition posted by @Talent30 seems to work for me too. I did have to create a wrangler.toml to set compatibility_flags = [ "nodejs_compat" ].

The rollupConfig.external addition posted by @Talent30 seems to work for me too. I did have to create a wrangler.toml to set compatibility_flags = [ "nodejs_compat" ] to get it to build.

Yes you need that. Nitro needs async context

Just so I'm up to date on this thread, we are waiting on https://github.com/unjs/nitro/issues/1943 to be resolved, so that it may be updated in Vinxi, correct?

Just so I'm up to date on this thread, we are waiting on unjs/nitro#1943 to be resolved, so that it may be updated in Vinxi, correct?

Hi, @SeanCassiere.
I don't think this will be fixed by Nitro in the near future since no one has had looked at the issue for a year now.
For now, we have to set compatibility_flags = [ "nodejs_compat" ] and rollupConfig: {       external: ["node:async_hooks"],     }  as the configuration I posted previously.
Perhaps we should update the documentation to tell people to enable compatibility_flags = [ "nodejs_compat" ] and update the cloudflare-pages default to include rollupConfig.external.
What are your thoughts?

@SeanCassiere @Talent30
I'm interested in deploying to Cloudflare, definitely would want this in the docs.

just create PRs for the docs then?

just create PRs for the docs then?

Hi there,
Would you like me to put the rollupConfig.external setting in for cloudflare-pages preset as well?

it might be a skill issue on my end, but adding the rollupConfig make my build crash with the following error:
[9:57:06 AM]  ERROR  ../../node_modules/vinxi/runtime/http.js (71:9): "AsyncLocalStorage" is not exported by "__vite-browser-external", imported by "../../node_modules/vinxi/runtime/http.js".
file: /Users/xstevenyung/Code/gieok/node_modules/vinxi/runtime/http.js:71:9

69: import { getContext as gContext } from "unctx";
70:
71: import { AsyncLocalStorage } from "node:async_hooks";
             ^
72:
73: /**

i am the only one with this issue ?

@SeanCassiere I've encountered the exact same error with the firebase deployment but i was unable to fix it or find a workaround. Should this issue be reopened?
example of my app.config.ts
export default defineConfig({
  server: {
    preset: 'firebase',
    firebase: {
      serverFunctionName: 'serverBriskfitStatic',
      gen: 2,
      nodeVersion: '22',
      httpsOptions: {
        region: 'europe-west3',
        maxInstances: 5,
      },
    },
    alias: {
      '/fonts': '/public/fonts',
    },
    esbuild: {
      options: {
        sourceMap: true,
        target: 'es2022',
      },
    },
  },
  vite: {
    resolve: {
      alias: {
        '/fonts': '/public/fonts',
      },
    },
    plugins: [
      tsConfigPaths({
        projects: ['./tsconfig.json'],
      }),
      // visualizer({ open: process.env.NODE_ENV === "production" }),
    ],
    build: {
      minify: true,
      rollupOptions: {
        treeshake: true,
      },
    },
  },
});

and this is the example of my serverFn:
export const setLoginCookie = createServerFn({
  method: 'POST',
})
  .validator((data: { token: string }) => data)
  .handler(({ data }) => {
    const event = getEvent();

    const { token } = data;
    setCookie(event, USER_AUTH_KEY, token, {
      httpOnly: true,
      sameSite: 'strict',
      secure: true,
      expires: new Date(Date.now() + 7 * 24 * 60 * 60 * 1000), // 7 days
    });
  });

Should this issue be reopened?

SInce it's for a different deployment target, it'd be best if it has its own separate issue.


Updates

tries to do basic updates:

[x] move to Project.toml and add a description for how to build docs locally
[x] update how/where to report issues and communicate with us
[x] remove no-longer used appveyor

I have a bunch of improvement ideas of our docs other than this, but let me do them in another PR. (as discussed in https://github.com/JunoLab/Juno.jl/issues/419)

@pfitzseb Are b136f4c and c11244a okay for you ?


database config from environment variable

instead of reading db.conf from orm.py, instead the contents of that file should ideally be passed via an environment variable

Fixed in b26a645. However, orm.py is not really used anywhere in the app and this was for an obsolete functionality. So it is fine running the app without this variable.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:18: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:40: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:40: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:40: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_f15ij1iu0w/clone/recipe/meta.yaml:40: missing_description: The recipe is missing a description
Errors were found


jsx-max-depth: Maximum call stack size exceeded

Hello, jsx-max-depth rule seems to crash in certain cases. This issue was spotted by automated CI run - it is not blocking my development or anything. https://github.com/AriPerkkio/eslint-remote-tester/actions/runs/428814308
I checked those other RangerError issues and this doesn't look like a duplicate.
"eslint-plugin-react": "^7.21.5",

{
  "root": true,
  "parserOptions": {
    "ecmaVersion": 2020,
    "sourceType": "module",
    "ecmaFeatures": {
      "jsx": true
    }
  },
  "settings": {
    "react": {
      "version": "16.13.1"
    }
  },
  "plugins": ["react"],
  "rules": {
    "react/jsx-max-depth": "error"
  }
}

Minimal repro:
This is really as far minimal as I could narrow it down.
import React, { Component } from 'react'

const MyComponent = () =>
;

var A = { D: '' };

class B extends Component {
    C() {
        A = A;
    }

    render() {
      const { D } = A;
      return <MyComponent  D={D} />;
    }
}

RangeError: Maximum call stack size exceeded
Occurred while linting <text>:81
    at findJSXElementOrFragment (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:1:1)

Crash reports from real projects
Rule: jsx-max-depth

Message: Maximum call stack size exceeded Occurred while linting <text>:81
Path: neurahealth/neuracovid/src/components/Login/index.js
Link

        const { t, i18n } = this.props;
        const { selectedOption } = lang
        if (localStorage.getItem(firebaseAuthKey) === '1')
        return <Splashscreen t={t} i18n={i18n}/>;
        return <LoginPage handleGoogleLogin={this.handleGoogleLogin} t={t} i18n={i18n} options={options} handleChange={this.handleChange} selectedOption={selectedOption} selectedOptionState={this.state.selectedOption}/>;
    }
}

const LoginPage = ({ handleGoogleLogin, t, i18n, handleChange, selectedOption, selectedOptionState }) => (

RangeError: Maximum call stack size exceeded
Occurred while linting <text>:81
    at findJSXElementOrFragment (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:1:1)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)

The A = A part seems to be what triggers it; basically we don't have code to account for someone reassigning the same value to a variable. I can't figure out what triggers it in the real-world example, though.
Not sure how to fix this one :-/

basically we don't have code to account for someone reassigning the same value to a variable

This can also reproduced with following:
import React, { Component } from "react";

const MyComponent = () =>
;
let ref = "";

class MyClassComponent extends Component {
  updateRef(valueForRef) {
    ref = valueForRef;
  }

  render() {
    // This part is required for the crash even though reassigned value is not used
    const valueForRef = ref;

    return <MyComponent valueForRef={ref} />;
  }
}

This week's scheduled scan caught a case with bit more sense. Just adding this here if it helps in future:

Rule: jsx-max-depth

Message: Maximum call stack size exceeded Occurred while linting <text>:22
Path: Devorein/Qocean/client/src/components/Modal/WarnModal.jsx
Link

				<Modal className={classes.modal} open={isOpen} closeAfterTransition>
					<Fade in={isOpen}>
						<DialogContent>
							<Icon icon="cancel" onClick={onClose} popoverText="Close Modal" />

{children}

								<Button className="modal_buttons--cancel" onClick={onCancel}>
									{cancelMsg ? cancelMsg : 'Cancel'}
								</Button>

RangeError: Maximum call stack size exceeded
Occurred while linting <text>:22
    at Reference.hasOwnProperty (<anonymous>)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:87:15)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)
    at findJSXElementOrFragment(/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:101:49)
    at find (/home/runner/work/eslint-remote-tester/eslint-remote-tester/ci/node_modules/eslint-plugin-react/lib/rules/jsx-max-depth.js:93:18)

@AriPerkkio your case seems to already be fixed. I'll add it just in case.

Looks good. Let's close this issue and I'll open new one if this reappears in future scans. Weekly scheduled smoke test: eslint-plugin-react #29

Looks like this is still valid issue. As this can be reproduced with https://github.com/yannickcr/eslint-plugin-react/issues/2880#issuecomment-748452788 I'll just reopen this issue.
Added failing test case with minimal setup: https://github.com/AriPerkkio/eslint-plugin-react/commit/c10a2ac172dca3d4526e40075b3c4e2592dff8ed
I'll take a look at the rule implementation next week.


choose a better name for the method 'connectOrConstruct'

Alternatively: replace all three methods with a single connect/join/whatever method that takes an optional flag for callers that actually care about whether the store should exist or not.

StorageNG will obsolete this.


Allow empty metadata values

Allow empty metadata values to be stored and returned by tusd.
Fixes #397

Thank you very much! I just added an additional test and merged this commit into master: https://github.com/tus/tusd/commit/87045a8fbf9fcf6f287a095d7bcf97eec2f6e983 and https://github.com/tus/tusd/commit/26b84bcb1c818f95ea236fbfe1c43e47c32574ab


PWA-697

Pull Request Test Coverage Report for Build 1075

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.008%

Totals

Change from base Build 1072:
0.0%

Covered Lines:
1616

Relevant Lines:
3900

💛  - Coveralls


goreleaser version conflicts with templates

What happened?
The release pipeline run https://github.com/SchwarzIT/go-template/actions/runs/3394226442 fails because of some templating issue.
How can we reproduce this?
See pipeline
gt version
-

Search

[X] I did search for other open and closed issues before opening this.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
No response

Created an issue at goreleaser to tackle this: https://github.com/goreleaser/goreleaser/issues/3573


Allow syncing data to gov.uk dev vm

As we move to gov.uk infrastructure, we will need to use the gov.uk dev vm

allow this by passing govuk_dev as the destination host argument. Also
update the readme to be a little more verbose and include this argument.

:+1: not tested, but seems sound.


Having in a unionAll creates a misplaced having clause

Example code
var foo = knex.select('id').from("mytable").having('val', '>', '0');
var bar = foo.clone();
bar.unionAll(foo.clone());
bar.unionAll(foo.clone());
console.log(bar.toString());

Output:
select `id` from `mytable`
union all
select `id` from `mytable` having `val` > '0'
union all
select `id` from `mytable` having `val` > '0' having `val` > '0'

It appears that the union all from the first subquery is applied at the very end.
Tested on version 0.11.10

I've been looking at this a little bit. Seems related to https://github.com/tgriesser/knex/issues/913#issuecomment-240234262

Looking at both now, I concur. Guess that helps narrow the source of the problem down to the union logic!

Released in 1.0.5


[feature] - routing: add additional validation to MPP related send payment params

Is your feature request related to a problem? Please describe.
Today, for the SendPaymentV2 call we have a series of mpp parameters users can define:

max_parts
max_shard_size_msat

If users aren't careful, then it's possible that the combination of: max parts, max shard size, and the payment amount are incompatible.
Here's an example of incompatible params:

max_parts = 2
max_shard_size_msat = 10_000
amt_msat = 100_000

In this case, we can't satisfy the payment as with only two max parts, we can only send 20_000 msat.
Describe the solution you'd like
Add validation, reject if params don't allow the full value to be sent.
Describe alternatives you've considered
Silently modify the params to make them compatible.
Additional context
Right now incorrect values just result in a path finding loop that'll go until the timeout, as the payment flow can't actually be satisfied.

Picking this up

Picking this up

Are you still on it?

Thanks for the reminder @Abdulkbk .


Clarifying Deployment Instructions

While chatting I have heard that our instructions for deploying could be clearer. Specifically, it seems it is unclear what needs to be done while re-deploying. We should review the documentation and consider if there are ways to clarify things.

It's mostly documented here. It'd be helpful to have more actionable feedback (where are people stuck, what errors are they getting, etc).

@sethvargo 💯
I plan to walk through them with open eyes and see if there aren't places we could provide clarity
@sherifkozman can you provide specific ways in which you (and folks you work with) may have gotten lead astray or stuck?

@sethvargo @crwilcox we managed to get things running quickly however working with a few HAs over the GCP deployment we found a few issues :

Using terraform to provision initial infrastructure doesnt enable the required permissions for Cloud build to complete some of the required tasks.

I believe there was a case where resources manager API needs to be enabled for new projects so deployment can be complete.

Other HAs have mentioned they are unable to find enough on deployment eslewhere on other providers or what is required to manage the update/deployment equivilent of the /scripts folder on other environments.

I am happy to join on a call anytime to dicuss any of the topics

Hi @sherifkozman
Thanks for the feedback.

Using terraform to provision initial infrastructure doesnt enable the required permissions for Cloud build to complete some of the required tasks.

Do you know which permissions are missing? I just tried in a clean project yesterday and it ran successfully to completion (took 13min).

I believe there was a case where resources manager API needs to be enabled for new projects so deployment can be complete.

Terraform enables the resource manager API. We do require that billing is configured on the project.

Other HAs have mentioned they are unable to find enough on deployment eslewhere on other providers or what is required to manage the update/deployment equivilent of the /scripts folder on other environments.

We're currently focused on GCP because that's where our expertise are. Other cloud providers can be found here: https://www.terraform.io/docs/providers/index.html.

Hey @sethvargo A new GCP project would have these permissions disabled by default
https://console.cloud.google.com/cloud-build/settings/service-account
I believe a deployment will need Cloud Run, KMS , Service account enabled for Cloud build. I believe they would be impacted by the Resource Manager API being disabled by default as the deployment fails as being unable to "setIamPolicy" for services such as KMS Keys.
I have seen this behaviour in a couple of projects where terraform doesnt enable the resources manager API so things fall off from there.

Here's the list of APIs we enable - we could add cloudresourcemanager.googleapis.com to that list. Cloud Run and Cloud KMS are already in that list.

Cloud Build is already given permission to manage Cloud Run services (including deploy)

Cloud Build does not need permission to use or access the KMS keys. That is and should continue to be restricted to the specific services.

Are these hypothetical issues or are your Terraform runs failing? If they are failing, can you please share the terraform apply output?

Hi all, I'm new to GCP and Terraform and while trying to get this working I consistently ran into errors when following the Terraform doc when I would use the optional settings like "create_generate_service = true" and these settings, which get put into a terraform.tfvars file. When I ran terraform apply I would get some variation of this error:
google_sql_database_instance.db-inst: Creating...

Error: Error, failed to create instance en-koqlo: googleapi: Error 400: Value requested is not valid., invalidFlagValue

  on database.tf line 22, in resource "google_sql_database_instance" "db-inst":
  22: resource "google_sql_database_instance" "db-inst" {

which is less than helpful. From what I can tell, it seems like whenever there were variables in terraform.tfvars that overwrote variables in vars.tf or database.tf, I got a invalidFlagValue error and Terraform failed. I eventually got around it by just changing the values in database.tf but I consistently got that error when I followed the docs.
The contents of terraform.tfvars that would cause the error looked like this:
project               = "project-name"
cloudsql_tier         = "db-custom-1-3840"
cloudsql_disk_size_gb = "16"
create_generate_service = true

and I got it to work after I gave up on the cloudsql_tier and create_generate_service options and changed the cloudsql_disk_size_gb option in database.tf

Hi @sraybuck
Thank you for the feedback. The way to enable the generate service is a bit out of date. I updated it in GH-572.
I believe the issue is with the max_connections flag. We bumped this value to 100,000, but that might exceed the possible value for a smaller db instance.

@sethvargo bumping the max_connections = 10000 when setting db_tier to db-custom-1-3840 causing terraform to crash resulting in :
Error, failed to update instance settings for : googleapi: Error 400: Value requested is not valid., invalidFlagValue

GH-576

Can you try removing lines 33-41 in database.tf and seeing what you get? That error is coming from the Cloud SQL API, and I agree it's not very informative.

According to the CloudSQL documentation this flag is still in beta and by default Cloud SQL sets the max connections depending on the machine type.
I fixed it by setting a lower value
https://cloud.google.com/sql/docs/postgres/quotas


[dif/sysrst_ctrl] Add remaining DIFs to header.

This completes the System Reset Controller DIF library API by adding the following features to the header:

configuration, getter, and setter DIFs for pin inversion, override, and read features, and
configuration, getter, and setter DIFs for the Ultra-Low-Power (ULP) wakeup feature.

Note: this depends on #12303.

PR dependency removed.


Issues when running the dream.py script on M1 mac: AttributeError: module 'torch._C' has no attribute '_cuda_resetPeakMemoryStats'

Hello!
First of all, thanks for all the hard work!
I'm seeing the following error when trying to use the dream.py script:
$ python scripts/dream.py

* Initializing, be patient...

Loading model from models/ldm/stable-diffusion-v1/model.ckpt
LatentDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.52 M params.
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 32, 32) = 4096 dimensions.
making attention of type 'vanilla' with 512 in_channels
Using half precision math. Call with --full_precision to use more accurate but VRAM-intensive full precision.
>> Setting Sampler to k_lms
model loaded in 5.91s

* Initialization done! Awaiting your command (-h for help, 'q' to quit)
dream> Lolcat
Traceback (most recent call last):
  File "/Users/antonin.lenfant/PERSONNEL/stable-diffusion/scripts/dream.py", line 518, in <module>
    main()
  File "/Users/antonin.lenfant/PERSONNEL/stable-diffusion/scripts/dream.py", line 99, in main
    main_loop(t2i, opt.outdir, opt.prompt_as_dir, cmd_parser, infile)
  File "/Users/antonin.lenfant/PERSONNEL/stable-diffusion/scripts/dream.py", line 208, in main_loop
    image_list  = t2i.prompt2image(image_callback=callback, **vars(opt))
  File "/Users/antonin.lenfant/PERSONNEL/stable-diffusion/ldm/simplet2i.py", line 283, in prompt2image
    torch.cuda.torch.cuda.reset_peak_memory_stats()
  File "/opt/homebrew/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/cuda/memory.py", line 260, in reset_peak_memory_stats
    return torch._C._cuda_resetPeakMemoryStats(device)
AttributeError: module 'torch._C' has no attribute '_cuda_resetPeakMemoryStats'

The weird thing is that it doesn't appear when calling the original txt2img.py script like this
python scripts/orig_scripts/txt2img.py --prompt "lolcat"

It works fine and I get the expected output.
My guess is that we're calling a CUDA-specific method (cuda.reset_peak_memory_stats()) on a non-CUDA device but I'm not very familiar with ML.
Thanks for the work and the help!

I can fix that. It's just related to reporting memory statistics. I'll disable it when not running on a CUDA system.


Idk many changes

I was forced to make this pr

i approved 🐈

Also for anyone wonder, i purposefully removed the gui initizalization from the main class, this was because it doesnt really work yet and provide no functionality. Imo it should be further developed on a sepearate branch and only then merged to master when it actually has functionality.

funny pr

Without looking, I'd say there's a 50% chance that this PR includes a rat that fills your home directory with 6 petabytes of cat pictures

Surprisingly, there doesn't seem to be anything that fills my home directory with 6 petabytes of cat pictures Also I don't remember if amalthea got fixed and can handle creating a new object for every event without killing performance, so maybe check on that Other than that, LGTM

As already said in my comment to jamsharp, I have already "benchmarked" (sending 500 events per second from another thread with a few couple listeners) and I have not seen any performance hits, so using this does not negatively impact performance and only makes it easier to expand some events (like my update to Packetevent)

Surprisingly, there doesn't seem to be anything that fills my home directory with 6 petabytes of cat pictures Also I don't remember if amalthea got fixed and can handle creating a new object for every event without killing performance, so maybe check on that Other than that, LGTM

It's not actually an issue with amalthea. It's just that if you're constantly sending a lot of events you are creating a lot of objects, which can kill performance on lower-end machines. The garbage collector is unfortunately not almighty (which I think is where the issue lies). All in all it would probably be better to not recreate the event on every post, and instead just make it once and send it over and over again, changing it as needed.

I just benchmarked it again on a really low end machine (VM) and it also doesn't have any notable performance impacts there from what I can tell

yeah, as i said gui should be readded at some point, but from what i saw it hasnt even be decided which library will be used, so i recommnd making gui developement in a seperate branch, and merging when its in ready state

Also i tested it, it starts and works correctly (altough modules currently need to be enabled in the code, since no gui yet)


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add ^17.0.6 remove ^17.0.4 (devDependencies.semantic-release)

:tada: This PR is included in version 5.0.21 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Discrepancies with permitted URI characters

I've been using CodeIgniter for a while but every time I set up a new project I notice that there are a few discrepancies with what URI characters are allowed by default.
In the actual config field, it reads:
$config['permitted_uri_chars'] = 'a-z 0-9~%.:_\-';
However, in the comment section above the config entry:
| As a security measure you are STRONGLY encouraged to restrict URLs to | as few characters as possible.  By default only these are allowed: a-z 0-9~%.:_-
as well as in the security documentation, it does not contain \.
I am mostly ceratin that \ should be a permitted uri char, however I will leave it up to the CodeIgniter team to analyze this discrepancy.

There's no discrepancy, the backslash is an escaping character.


Some code cleanups

Re-factored print_category to be more idiomatic perl. Began work on making strict/warnings and Perl Critic happy.
I'll make a couple more pull requests as I complete more of it.
For Webmin 2.0 (and for general safety, and basic modern Perl standard practice), you should add the following to the beginning of all of your Perl files:
use strict;
use warnings;

It can be tricky at first to work without global variables, but I'm happy to help with the conversion of Authentic to be strict and warnings compliant. We'll also begin running Perl Critic across everything that goes into Webmin for 2.0, so that's a useful thing to add to your workflow.

Joe, thanks! Part of that code that you made nice tweaking of was made at the very beginning of me getting into Perl. I injected all of your nice commits. I will try to rewrite other, familiar parts. Having hash table of icons/values seems like the only reasonable thing and should've been used right away. There was a typo (you placed a dot instead of comma in a hash), but it's ok, I fixed that.
I will add this to 18.01 (manually so far). Thanks again!


Implement trust score system

Code implementing the trust system i.e. storing scores, updating them, looking them up, decaying them properly and emitting events when they're banned.
Wrote unit tests.
Checklist:

[x] Tests were added for the new behavior
[ ] All tests pass (CI will check this if you didn't)
[ ] Does this close issues?


[BUG] Double Workspace Loaded on Sumneko 2.6.0

Two workspaces are loaded in the newest Sumneko lua server when using lua-dev. I can confirm this does not occur when not using lua-dev in lspconfig.
The effect of loading two workspaces is super slow startup for the lsp, and super slow autocompletes and goto definitions.
The proof of two workspaces loaded can be seen in the screenshot by using lsp_status indicator in the status line (the bottom row).

This is most likely caused by the new workspaces feature. Can be seen from this compare tag commits.

I'm experiencing this too.

@folke can you check this one out? Because of this, the loading time has increased dramatically.

I don't really see anything I can do to fix this?
Seems like a bug in Sumneko? As far as I can see, there's only one lsp client.

Thank you for the answer. Downgrading sumneko_lua to v2.5.6 fixed the issue. It seems best to wait until sumneko_lua to be stable before going to v2.6.x or higher.
I consider this issue from my side to be closed, but left the decision to close it to you.

Hi!
Could this be related?
warn message in ~/.cache/nvim/lsp.log:
[WARN][2022-01-21 14:39:09] .../lua/vim/lsp.lua:812	"server_request: no handler found for"	"workspace/semanticTokens/refresh"

I'm getting 100% CPU usage and no response from the server.
I've also tried it without applying
require("lua-dev").setup { lspconfig = M.general }

and the warning keeps appearing (I assume it has to do with sumneko_lua) but the CPU usage won't raise...

The reason for this is https://github.com/sumneko/lua-language-server/issues/1596


Hook is not working

I do not know why, but the Branch Cordova Plugin will not be added to the app (I am using Cordova 8.1.2 / Cordova-Android 7.1.4). After removing the following hook from plugin.xml it works:
<hook src="src/scripts/hooks/beforePluginInstall.js" type="before_plugin_install" />

Having the same issue.

Related pull request: #521
Version 3.1.0 seems to work fine.

I tried 3.1.3, 3.1.0, and 3.0.1 and no build is working. All builds hangs after

Resolving module name for q => q

+1 The plugin is not added in the platform, just in the project.

I wrote a PR (#527) that removes the cocoapods dependency (something that is directly integrated in cordova ios) and removed the hook, so it's working. You can test it by install https://www.npmjs.com/package/innomobile-branch-cordova-sdk (just replace branch-cordova-sdk with innomobile-branch-cordova-sdk in your config.xml if you want to test it - or wait until the Branch team insert a fix)

@aaaronlopez, just tagging you directly since this is an urgent issue. Nobody can build successfully because of this issue.

@paulstelzer I have just tested your forked plugin innomobile-branch-cordova-sdk
but the Branch library is not defined inside the project.
It does not exist in the window global variable.
The plugin is inserted though in the cordova_plugins.js

@deliverymanager See #533
Yeah, something Cordova iOS related - Just run pod install inside <yourapp>/platforms/ios or run it inside Xcode.
The Pod wasn't installed correctly (https://guides.cocoapods.org/using/pod-install-vs-update.html). Afterwards it should work

I am testing on Android platform

npm install innomobile-branch-cordova-sdk@latest
Inside config.xml:
<plugin name="innomobile-branch-cordova-sdk" spec="^3.1.5"/>
Instead of <plugin name="innomobile-branch-cordova-sdk" spec="^3.1.2"/>
Important: You have to use 3.1.5 (the versions before are broking like branch-cordova-sdk)

Yes I confirm installing 3.1.5

Yes I confirm everything.
The only difference of course is the PACKAGE_NAME which is the one I am using.
Everything else is the same.

Then it should work :D

Something might have been stuck.
It does work now.

Is there any problem if I publish the app with your plugin and then republish with the original once it is corrected too?

No, if branch updated it, just replace Inside config.xml from
<plugin name="innomobile-branch-cordova-sdk" spec="^3.1.5"/>
to
<plugin name="branch-cordova-sdk" spec="^3.1.3"/>
My is just a fork of the Branch Plugin (only with the fix)

ok thank you!

hello all, we tested a change with 3.1.4 that did not appear to resolve the issue. We modified the plugin hooks and released a beta candidate, 3.1.5-beta.0 to unblock most folks.  Can anyone that tests it out please let us know whether it resolves this issue? Separately, we are testing #533 to use Cocoapods directly.

Just wanted to chime in that we are also impacted on this.

@aaaronlopez, confirming that 3.1.5-beta.0 is working for both Android and iOS builds

We hope release 3.1.5 solved this issue for everyone


fix: Prevent infinite loop in example

Check for the page token by comparing to undefined instead of
null. Loose equality with null also works but the linter doesn't like
loose equality. Fixes an infinite loop that is only broken by rate
limiting.
Give the token to the list caches call to prevent an infinite loop
that would occur if there were more than one page of caches.

The package-lock.json files were confusing to me as well. I'll try to rebuild with npm ci to see if I can get rid of them.

🚢


docs: fix RemoveWrapperFunction import path

Description
Read me is not accurate

Linked Issues
Additional context

Merged, thanks!


[suggestion]for long term service, consider about commit this repo to denodrivers org

https://github.com/denodrivers
thanks your hard working

@sunw31 I'm not sure this fits in that org. If someone from the org can comment to clarify, I'd be happy to participate.
However, at this time this codebase is simply a bootstrap for PouchDB, which is an AMAZING CouchDB client. I'm hoping this codebase eventually becomes obsolete, as I plan on contributing to PouchDB to get first-class support for ES Modules and Deno some time in the next 12 to 18 months.

@notfilippo and @ebebbington: Is this a good fit, or shall I go ahead and close this issue?

We would be obviously happy to include this repo in denodrivers, feel free to join! But I believe that getting first-class support in PouchDB is better for the long term, so if you prefer to keep it here that's totally ok :)

Hey, @notfilippo, let's do it. At least this way, with this codebase, there's a good bootstrapped way to get PouchDB working correctly until that project implements Deno support natively.
I've started a discussion about what I think needs to be done in the parent repo, but my guess is that my request to collaborate means waiting until PouchDBversion 9.0.0. So, it could be a while for official support.
How do I join?
And do you know how to make sure that future releases stay connected to the existing module on Denoland/x?


Deploy Application Insights to resource group region

Value used to be hard coded to East US region. Now uses whatever was intended for the rest of the resource group.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Aaronontheweb sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Has App Insights been deployed to more regions? When I last checked, and it has been a little while, there were surprisingly few regions where it was available so it seemed safer to hard-code it.

@onovotny I think it's pretty ubiquitous now - had to do a lot of testing and support for it when we wrote our OpenTracing adapter for App Insights last year: https://github.com/petabridge/Petabridge.Tracing.ApplicationInsights

Here's what I see if I create one in the portal:

Maybe those should be in a restriction in the template? My aim is to prevent someone from failing due to AI not being in the same region as everything else... or at least make them pick a valid option.


Add Apache License

Uses the actual apache license instead of the file header for the LICENSE file.

Merged, no need to wait for Travis to finish, it's just the LICENSE file.


Test for CI bug in PR #356 - do not merge

test

test

test

test

test

This PR served it's purpose, I made the code work as I needed. Closing without merging.