buggy behaviour in part.onStep()

the pattern below has 8 beats, play() is called 8 times but part.onStep is called 7 times.
When the part is looping, part.onStep is still only called 7 times.

let sound, m = 0, n = 0;

function setup () {
  masterVolume(0.4);

  const pat = [1,1,1,1,1,1,1,1];
  const phrase = new p5.Phrase('phrase', play, pat);

  const part = new p5.Part();
  part.onStep(() => {
    console.log(`step ${++n}`);
  });
  part.addPhrase(phrase);
  part.setBPM(60);
  part.start();
  // part.loop();

  sound = new p5.MonoSynth();

  userStartAudio().then(() => {
     console.log("user start audio");
   });
}

function draw () {
  background(0);
}

function play (time, notused) {
  console.log(`play ${++m}`);
  sound.play('D4');
}

I am working on this issue but I'm unable to find this particular code snippet. Could anyone please help me in finding this particular code snippet in the code base?

Output:
user start audio sketch.js:21:14
step 1 sketch.js:11:13
play 1 sketch.js:30:11
step 2 sketch.js:11:13
play 2 sketch.js:30:11
step 3 sketch.js:11:13
play 3 sketch.js:30:11
step 4 sketch.js:11:13
play 4 sketch.js:30:11
step 5 sketch.js:11:13
play 5 sketch.js:30:11
step 6 sketch.js:11:13
play 6 sketch.js:30:11
step 7 sketch.js:11:13
play 7 sketch.js:30:11
done p5.sound.js:10794:17
play 8 sketch.js:30:11

@i3games Thanks for posting the output. I am unable to find this particular code snippet in the entire codebase. I was wondering if you could just help me out with the file in which this particular snippet exists. That would be really very helpful.

Hi, the above code was my program demonstrating the problem.

I just checked with p5.js 1.0.0 and the problem still persists.

@i3games Thank you, It seems good to me using part.loop() instead of part.start(). It worked for me. Well, I'm working on this issue and have opened a PR for the same.


ci: report coverage by component

Reports coverage by component. Each component is defined in codecov.yml

I would just split consensus/evm seems unusual that they are merged.


Add nft dashboard generation and deploy script.

Changes
Add command for generate cu NFT dashboard

How to test these changes?
custom-crawler cu-reports --moonstream-token <public_query_user_token> generate-nft-dashboard

Check route https://s3.amazonaws.com/data.moonstream.to/queries/cu_nft_dashboard_data/data.csv if data is updated.

Related issues

@bugout-dev check

[ ] make available MOONSTREAM_PUBLIC_QUERIES_DATA_ACCESS_TOKEN to crawlers (now this is only backend env var)
[ ] make available MOONSTREAM_S3_PUBLIC_DATA_BUCKET to crawlers


Fix bug where setting AppBarThemeCompactHeight would not work correctly

Description

Changed the contentgrids row definitions from Auto to 1* so setting AppBarThemeCompactHeight will be respected as good as it can without any other restyling applied. Thanks to @jdhuntington for suggesting the fix.
Note that, for the buttons in order to render perfectly, multiple other values beside AppBarThemeCompactHeight need to be set.
Motivation and Context

Fixes #275.
How Has This Been Tested?

Tested manually using MUXTestAdhocApp and the following snippet:
<CommandBar DefaultLabelPosition="Right" >
    <CommandBar.Resources>
        <x:Double x:Key="AppBarThemeCompactHeight">48</x:Double>
    </CommandBar.Resources>
    <AppBarButton Icon="Delete" Label="Delete"/>
</CommandBar>

Screenshots (if appropriate):

@chingucoding I think we also need to center the TextLabel.

@chingucoding I think we also need to center the TextLabel.

Yes and no. As @licanhua mentioned in this comment, when one changes the AppBarThemeCompactHeight , one also has to changes other values to get everything to align correctly. So this would be sort of by design I guess.
Since the text is positioned using margins, I think we need to re-template the positioning of the label. And this, I think will, be quite hard, as a lot of fine tuning was needed to get the app bar button to look the way it does (according to @jevansaks and @licanhua).

Yes and no. As @licanhua CanHua Li FTE mentioned in this comment, when one changes the AppBarThemeCompactHeight , one also has to changes other values to get everything to align correctly. So this would be sort of by design I guess.
Since the text is positioned using margins, I think we need to re-template the positioning of the label. And this, I think will, be quite hard, as a lot of fine tuning was needed to get the app bar button to look the way it does (according to @jevansaks Jevan Saks FTE and @licanhua CanHua Li FTE).

Hmm, I see. The change looks good to me then. @licanhua can you double check?

/azp run

@chingucoding I think we also need to center the TextLabel.

Yes and no. As @licanhua mentioned in this comment, when one changes the AppBarThemeCompactHeight , one also has to changes other values to get everything to align correctly. So this would be sort of by design I guess.
Since the text is positioned using margins, I think we need to re-template the positioning of the label. And this, I think will, be quite hard, as a lot of fine tuning was needed to get the app bar button to look the way it does (according to @jevansaks and @licanhua).

This seems like another issue with this api.  I opened issue #1424 to track.

I think this change is right but will also defer to @licanhua.

@chingucoding, If I'm correct, below text alignment problem is introduced by this PR, right?

If yes, we need to fix it in one PR, or at least check in fix for #1424 first.
It's better to add visual tree dump testing for commandbar with AppBarButton and AppBarToggleButton. and it's better to cover different setting of CommandBarDefaultLabelPosition for commandbar.

@kaiguo Kai Guo FTE Is there a way to generate those locally? I haven't seen a description for generating those locally in #1409.

Updating the doc...

@kaiguo
        Kai Guo
        FTE Kai Guo FTE Is there a way to generate those locally? I haven't seen a description for generating those locally in #1409.

Updating the doc...

@chingucoding
https://github.com/microsoft/microsoft-ui-xaml/pull/1409/files?short_path=9ef2479#diff-9ef24792e98088e3b64281faddec0d46

@kaiguo
        Kai Guo
        FTE Kai Guo FTE Is there a way to generate those locally? I haven't seen a description for generating those locally in #1409.

Updating the doc...

@chingucoding
https://github.com/microsoft/microsoft-ui-xaml/pull/1409/files?short_path=9ef2479#diff-9ef24792e98088e3b64281faddec0d46

Thank you!

I have added the new VisualTree verification test as suggested by @licanhua .

@chingucoding and @jevansaks  Personally I suggest we stop on this task.
The bug you are trying to fix has very minor customer impact, but you needs a lot of manual testing/fixing to the combination of CommandBar, AppBar, AppBarButton.
I dumpped PR and found that below scenario is regressed.

Launch MUXControlsTestApp
Click Reveal, then click Basic Reveal Tests

@chingucoding and @jevansaks Personally I suggest we stop on this task.
The bug you are trying to fix has very minor customer impact, but you needs a lot of manual testing/fixing to the combination of CommandBar, AppBar, AppBarButton.
I dumpped PR and found that below scenario is regressed.

Launch MUXControlsTestApp
Click Reveal, then click Basic Reveal Tests

If this is the case can we close the PR? Should we open an issue as well?

I agree with @licanhua's comment, I think we should look more holistically at this problem. I've closed #275 as such. Thanks for the effort @chingucoding!


Added fingerprint for D5X84YU to zigbee thermostat driver

Based on the DTH, this device probably needs a bit more than just a fingerprint.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


[BUG] User Introductions Editor display

The line number is hiding under the text

This is a CSS problem that I can't solve.
P.S. Is there same problem at problem edit?

No same problem at problem edit

I've no idea what going on there


DEPRECATION: Using the {{view "string"}} helper is deprecated.

We need to fix deprecation warnings like the following: DEPRECATION: Using the {{view "string"}} helper is deprecated. ('hospitalrun/inventory/search/template.hbs' @ L1:C0)  [deprecation id: view.helper] See http://emberjs.com/deprecations/v1.x#toc_ember-view for more details.

Taking care of this with #164

These deprecation warnings should now be resolved.


Remove analytics from Bower, fixes #1102

They caused more issues than useful they were.
Instead, we'll focus on fetching statistics from NPM registry to watch Bower's popularity, and GitHub stars over time to watch packages' popularity.
http://bower.io/stats/ is already updated to use only non-google-analytics data.

ping @rayshan @desandro @samccone @benschwarz @BenMann

question"
If someone runs bower i... with silent... and they have not configured their prefs yet (I am thinking of the run on CI test case) can we just assume no tracking?
The only reason I ask is because losing these metrics is a fairly big hit imho.

Ship Ahoy! All looking good :ship:
And yep, we're ditching stats for command usage.
(Or which specific metric are we talking about? ;D)

@samccone We already assumed no analytics on CI. Plus I don't know any way we used these metrics to develop bower.. And nowadays we also "freeze" bower releases with bundledDependencies.
Developers reporting issues on Github do far better job than analytics of how many times someone wrote bower install. Actually useful thing would be auto crash-reports, but analytics in current form seems not.

sounds good to me @sheerun
:ship: away

:+1: Thanks for the work here

:+1:
Hopefully the depedency on insight can be removed as well!?

@mojoaxel Good catch :)


fix: daemon no longer gets terminated on its own

needs some more element way to handle this scheduler lifecycle

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

GitHub release
v1.0.3

Your semantic-release bot :package::rocket:


GPII-3492: Allow us to (optionally) prevent destruction of the cluster. (gpii-infra)

Requires https://github.com/gpii-ops/exekube/pull/23.
As punishment for my hubris in saying this feature would be simple to add, Terraform does not support variable interpolation in lifecycle{} blocks :(. https://github.com/hashicorp/terraform/issues/3116
Fortunately, @zero-below posted a workaround (thanks, friend!). The workaround is a little weird but with it I was able to protect a cluster, un-protect it, re-create it, and re-protect it.

LGTM

yep, lgtm. A bit weird and ugly, but I guess it does the job for us given TF limitations. I just hope this is better in the new 0.12 :).

@mrtyler Hey, I just stumbled across a small fix to remove the need to edit the prevent_destroy = true in the text file, and thought to post here, since the original thread is closed.
Really simple, really. terraform state rm random_id.cluster_protector and then you can do something to turn off the protection (by variable) or do a terraform destroy of the cluster or whatever before you do another apply.

Thanks @zero-below! For anyone following this thread, we (intentionally ;)) destroyed a cluster this week and the terraform state rm trick worked. I updated the instructions in https://github.com/gpii-ops/gpii-infra/pull/297.


Bump ruby

I bumped the gem versions since now heroku will force you to use ruby 2.2.
cc @bigcommerce/ruby-sf

@pedelman this is last call for you to give your opinion on this piece of code, otherwise i'll go ahead and merge this at 12pm later today! thanks for helping

:+1:


Atata tests are very slow under netcoreapp2.0target framework

When I am trying to run tests for xUnitTestProject (.Net Core) with target framework netcoreapp2.0 tests are very slow. But when I switch to net47 target framework tests run with normal speed.
There is some delay (~ 10 seconds) before accessing elements.

Hi,
Please read "WebDriver .NET Core Issues" section in Atata 0.15.0 Released. There is configuration method WithFixOfCommandExecutionDelay that fixes this WebDriver issue.

Thanks a lot!


Column Sort Arrow doesn't sort

Pretty much just what the title says. The columns won't sort according to the arrow. Would like my latest Explorer data at the top.

Confirmed, this is still the case in version v.0.2.22141.1531.

Confirmed to still be the case in version v.0.2.23060.2034

Great, thanks for the update! It's a great tool!

From: Jonathan Miller @.>
Sent: Thursday, December 7, 2023 8:54:33 PM
To: Xjph/ObservatoryCore @.>
Cc: Kate Shepard @.>; Comment @.>
Subject: Re: [Xjph/ObservatoryCore] Column Sort Arrow doesn't sort (#22)
Slated to be fixed very soon, look for it in the next release.
—
Reply to this email directly, view it on GitHubhttps://github.com/Xjph/ObservatoryCore/issues/22#issuecomment-1846474940, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATHKCIFMPKCHASFRPVJ32Z3YIJ6OTAVCNFSM5FYZ4JUKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBUGY2DONBZGQYA.
You are receiving this because you commented.Message ID: @.***>

https://github.com/Xjph/ObservatoryCore/pull/127


SpannerIO Updates

A unit test.
Integration test.
Logical mutation size estimation.
Docs, formatting.

Rather than sit between you and getting work done, I'm going to merge my version to beam, then you can submit PRs there.


Check if graphviz backend is installed

Based on the discussion at the bottom of #381, this PR checks if a working Graphviz backend is installed. If not, it will suggest how to do so like described here.
While #389 added similar hints for the documentation, I'm afraid most people will install the package without reading the docs and then face missing backend issues during runtime.

Looks good. Merging


Bug: signTypedData using Coinbase Wallet not working

Describe the bug
I am doing signing of EIP 712 message using Metamask and Coinbase wallet. I connected both using wagmi connectors as well. The signTypedData is working fine when I am on Metamask so I was expecting it will work on Coinbase as well. But when I am trying to sign on Coinbase, its given me this error:
UnknownRpcError: An unknown RPC error occurred.
Details: hashFuncMap[method] is not a function

Steps
Link to Minimal Reproducible Example
https://stackblitz.com/edit/new-wagmi-hys7ma?file=src%2Fwagmi.ts,src%2FApp.tsx,src%2Fmain.tsx
Steps To Reproduce

Click connect with metamask
Click sign once connected, the metamask popup will show up with correct message data
Disconnect with metamask and connect with coinbase
Click sign once connected again, it will show the error

Expected behavior
It should work and show the signing message just like the metamask.
Version
No response
Additional info
No response
Desktop
No response
Smartphone
No response

I am using the latest wagmi version to integrate coinbase wallet. Open the issue here https://github.com/wevm/wagmi/issues/4414 and they mentioned that

This is likely a Coinbase Wallet SDK bug

Hi @hieronymus777 is there an update?

Thanks for reporting this issue.
When we used the reproducible steps, we noticed that making this change to the sample:
App.tsx
...
-   const { signTypedData } = useSignTypedData();
+   const { signTypedData, data } = useSignTypedData();

...
+
Signature:{data}

...

Results in
To clarify - was that just an issue in the reproduction steps or was that the source of the original bug as well?
@hieronymus777
This signature is also wrong, it has too many 0s, see these issues that explains it better:

https://github.com/coinbase/smart-wallet/issues/98
https://github.com/wevm/wagmi/issues/4230


Scan module structs are hidden

Hi, not sure when or if this changed I cannot write this function:
fn scan() -> Result<ArpScanResults> {
    let target: Target = Target::from_subnet("<IP_ADDRESS>/24", None)?;
    let threads_num = 300;
    let timeout = Some(Duration::new(1, 5));
    let src_ipv4 = None;
    arp_scan(target, src_ipv4, threads_num, timeout)
}

as I cannot import pistol::scan::ArpScanResults:
error[E0603]: module `scan` is private
  --> src/scanner.rs:4:13
   |
4  | use pistol::scan::ArpScanResults;
   |             ^^^^ private module
   |
note: the module `scan` is defined here
  --> .../pistol-rs-575476176e5e4305/e198bc5/src/lib.rs:16:1
   |
16 | mod scan;
   | ^^^^^^^^

I think a pub mod scan; in lib.rs will fix it, though that may expose a lot of things you don't want to expose. If that's the case, i think you can selectively re-export the structs you'd want.
Thanks!

Hi @rwygand , I just fix that https://github.com/rikonaka/pistol-rs/commit/4a635a7fbf329e7b2713ed611a1bc4d6f106fbe5 🫣🫣🫣


Change sending SS logs to Hub

Change sending SS logs to Hub. Send last 5 SS logs per hour.

done


200 create nested crash

i'm trying to write an import it use about 200 create nested with 2 associations, and every time, waterline crash : a create never call the callback... and not at the same time so i think it's a asynchronous problem...

@atiertant are you doing 200 nested writes at once? Can you give an example, the default pool size is 10 connections usually and each request is done on it's own connection. It could be an issue with timeouts because the connections are exhausted. If you limit the writes to 10 at a time using something like async.eachLimit does it still behave the same way?

@particlebanana code a bit long to post here ... but i do create on event... so i can't do something like async.eachLimit. when doing flat create the code work and every data are wrote, with nested on one belongsTo association this is working too, with a second code crash fast.
i tryed it on sails-mysql and offshore-sql adapter with the same result.

@particlebanana i wrote a function using asynk to replace create nested by multiple flat create of the same data:
using create nested it crash after 89 top objet created
using the asynk function all 242 top objet were created with no problems and faster...
this is a waterline bug in create nested !

Yup I can see that, the nested graph stuff needs to be pulled out. It's not transactional and causes more errors than it's worth.
Can you post a sample repo that I can clone that reproduces the issue?

i'm working on this to understand what's apend, this is a memory leak problem...
_.cloneDeep is evil !
while database is storing some data, waterline continue enqueuing them with many duplication of objects, the heap growing... and node crach !
first cloneDeep easy to remove
the second is a bit more complex because its duplicate data and pass them to all nested operations in valuesObject.originalValues ...
valuesObject should not contain originalValues, only associations object' references should be saved.

fixed in offshore-PR 14
with this patch heap growing realy slowly and all data were inserted correctly in nested create with 2 associations and one deep nested containing ~5 Mb binary.
sails-mysql has memory leak too but don't crash while using offshore-sql use 2x lesser memory.

@atiertant Thanks for all your time putting this together.  As @particlebanana pointed out, we'd need some more information to recreate the original bug you mentioned.  While creating 200 records in a single Waterline method call is a relatively rare use case, it absolutely should work and we take these kinds of reports very seriously.  I went ahead and verified that this functions as expected when using Waterline with a production-ready database adapter.
I'm having a bit of a hard time following the other topics discussed in this issue, since they focus on changes you have made to Waterline in your fork, including adding a package you wrote as a dependency.  Unfortunately, I'm just not familiar enough with that package or your changes to help.  That said, if you have performance improvements you'd like to suggest for Waterline, please suggest those in a pull request accompanied by benchmarks (see also CONTRIBUTING.md in this repo for more information).

Also, @atiertant (or anyone reading this in the future) if you notice this issue again with the latest stable Waterline, please open a new issue with details (esp. your Sails/Waterline version, which adapter you're using, and the version).  Thank you!

For future reference, there's more information and tips on diagnosing memory leaks in Node apps, and on reproducing+reporting suspected memory leaks in Sails here: https://github.com/balderdashy/sails/issues/3782#issuecomment-238679052


[Service Fabric] breaking changes deprecate commands and update sf managed sdk

Description
Removing deprecate certificate commands.
Update service fabric managed sdk to the latest version
Re-record test
Testing Guide

History Notes

[[Service Fabric] BREAKING CHANGE: az sf cluster certificate: remove all commands under this group. Please follow the instructions here to add/remove cluster certificates: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-cluster-security-update-certs-azure#add-a-secondary-certificate-using-azure-resource-manager.
[[Service Fabric] az sf managed-cluster and sf managed-node-type: set groups as not preview
[[Service Fabric] update azure-mgmt-servicefabricmanagedclusters package to the latest version 1.0.0 that uses 2021-05-01 GA api version.
[[Service Fabric] BREAKING CHANGE: az sf managed-service update: remove deprecated parameter --drop-source-replica-on-move.
[[Service Fabric] BREAKING CHANGE: sf managed-service create: remove deprecated parameters --service-dns-name, --drop-source-replica-on-move and -instance-close-delay-duration.
[[Service Fabric] sf managed-cluster create: add parameters --upgrade-mode, --upgrade-cadence and --code-version.
[[Service Fabric] sf managed-node-type: add parameters --data-disk-type, --is-stateless and --multiple-placement-groups.
[[Service Fabric] BREAKING CHANGE: az sf cluster: rename parameter --vault-resource-group to --vault-rg.

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

Service Fabric

Service Fabric

@yonzhan who can review this, please? we want to get it checked in for S187

@Juliehzl will help you.

Could we follow deprecate work flow first according to https://github.com/Azure/azure-cli/blob/dev/doc/authoring_command_modules/authoring_commands.md#deprecating-commands-and-arguments before removing these commands?

Could we follow deprecate work flow first according to https://github.com/Azure/azure-cli/blob/dev/doc/authoring_command_modules/authoring_commands.md#deprecating-commands-and-arguments before removing these commands?

yes. You can see that currently, the commands have the depreciation marker:
https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/servicefabric/commands.py#L73

@Juliehzl can you review again, please?

@Juliehzl. the test failure reported are not from our team:
FAILED src/azure-cli/azure/cli/command_modules/network/tests/latest/test_private_endpoint_commands.py::NetworkPrivateLinkCosmosDBScenarioTest::test_private_endpoint_connection_cosmosdb
FAILED src/azure-cli/azure/cli/command_modules/network/tests/latest/test_private_endpoint_commands.py::NetworkPrivateLinkCosmosDBScenarioTest::test_private_link_resource_cosmosdb

@Juliehzl. the test failure reported are not from our team:
FAILED src/azure-cli/azure/cli/command_modules/network/tests/latest/test_private_endpoint_commands.py::NetworkPrivateLinkCosmosDBScenarioTest::test_private_endpoint_connection_cosmosdb
FAILED src/azure-cli/azure/cli/command_modules/network/tests/latest/test_private_endpoint_commands.py::NetworkPrivateLinkCosmosDBScenarioTest::test_private_link_resource_cosmosdb

We will fix it soon.


[TASK] Set permissions on copied files

The Issue
Permissions are not properly set after files being copied to the container
How This PR Solves The Issue
Settings permissions to the solr data folder for newly copied files to 'solr'
Manual Testing Instructions
Related Issue Link(s)
Fixes #7
Release/Deployment Notes

This is an effective technique. It was obvious to me reading the solrctl custom command why the ownership on the files was wrong.


Can't turn off rounding-precision

When I pass --rounding-precision -1 to the CLI it exits with the message "error: unknown option `-1'".
--rounding-precision=-1 also doesn't work.

You actually need to escape -1 in shell as it'll be interpreted as an option, e.g. echo "div{width:200.12345px}" | ./bin/cleancss --rounding-precision \\-1, see https://github.com/jakubpawlowicz/clean-css/blob/master/test/binary-test.js#L448


Deactivating "Fullscreen" adds unnecassary margin at the bottom

I am using version 0.8.2 on a Samsung Galaxy S20.
I have set my navigation bar to "swipe gestures", so there is no visible navigation bar on my phone.
When I deactivate the "Fullscreen" option in the Lyrion app, I get an unnecassary margin at the bottom of the screen, which does not look too nice (see screenshot).

I guess this is the space where the Lyrion app would assume the navigation bar. Is there any chance to get this "fixed", i.e. having the app respecting the actual height of the navigation bar (which is zero on my device)?
I actually don't want to use the fullscreen option since I want to have the status bar on the top of the screen visible.

You sway when you "deactivate" fullscreen - so does the error only occur when you toggle this? i.e. if you disable fullscreen, quit the app, and restart - is everything then OK?
I don;t use fullscreen, but do use gesture navigation, and this works for me - but I have a white bar bottom middle, which is missing from your screenshot.

You sway when you "deactivate" fullscreen - so does the error only occur when you toggle this? i.e. if you disable fullscreen, quit the app, and restart - is everything then OK?

No. The issue persists. If I quit/restart the Lyrion app with fullscreen already activated the margin is still there.

I don;t use fullscreen, but do use gesture navigation, and this works for me - but I have a white bar bottom middle, which is missing from your screenshot.

These are my Android Navigation Bar settings:

Probably the deactivated "Gesture hints" is why I am missing the white bar at the bottom? This is what it looks like if I have "gesture hints" enabled:

This is how it looks like if I set my Android navigation bar to "Buttons":

Which android version is this? As I use gesture navigation, and I've never seen this weird hybrid type you seem to have.

OK, just seen this setting. Must admit I find it awful! Not sure why you would use this as opposed to "swipe from sides and bottom"...

Which android version is this? As I use gesture navigation, and I've never seen this weird hybrid type you seem to have.

It's running the latest stock software that Samsung has provided for this device and it's based on Android 13.

OK, just seen this setting. Must admit I find it awful! Not sure why you would use this as opposed to "swipe from sides and bottom"...

I also have an option to switch to the "swipe from sides and bottom" mode. However, swiping from sides has never worked for me on this device.

Would you mind testing the attached APK? It's placed within a ZIP as GitHub will not allow uploading APKs :(
test.zip

Would you mind testing the attached APK?

Sure! Will test it tomorrow. (Not enough time today.)

Just tested your version 0.8.3.
It looks much better now!
There is still some bottom margin which might be reduced a bit more, bit it defintely looks more appealing now.
Here are two screenshots, one without navigation hints (which is my personal default setting), the other one with navigation hints:

Thank you!

Try this version...
test.2.zip

Awesome! This is how it looks with your latest build (please ignore the pixelated look of the screenshots):

Great! I'll close this now, and will tag a new release soon...


Typescript: unit seems to be wrongly defined as undefined in Typescript

Hi,
I've the following Rust code:
#[derive(Debug, serde::Serialize)]
#[typeshare::typeshare]
#[serde(tag = "type", content = "content")]
pub enum Cmd {
    StartBrowser(StartBrowser),
}

#[derive(Debug, serde::Serialize)]
#[typeshare::typeshare]
pub struct StartBrowser(pub ());

...which generates the following Typescript code:
export type StartBrowser = undefined;

export type Cmd =
	| { type: "StartBrowser", content: StartBrowser };

...however, serde_json serializes Cmd::StartBrowser(StartBrowser(())) as {"type":"StartBrowser","content":null} (please note the null instead of the above defined undefined).
I'm not 100% sure, but I believe serde_json uses undefined to represent optional values (i.e., Rust's Option type).

We ran into some issues wherein previously working FFI calls in downstream projects broke. As a result in #97 the changes were reverted. This is still on our radar, but we'd ideally want to avoid breaking our downstream projects. In the meantime, further discussion can be held here.


Add error message or disable Waiter window

Is your feature request related to a problem? Please describe.
I was testing out changes on the CSBC/PS-ON DCA, and attempted to generate a template link to understand how the app works. However, something broke in the backend and I didn't notice right away because the error message was hidden under the "Generating link..." window:

Describe the solution you'd like
Add a more prominent error message, or disable the Waiter window once an error is encountered.
How important is this feature? Select from the options below:
🌗 Medium (not a huge deal, but it did take me about 5 minutes before realizing that nothing was happening 😅)
When will use cases depending on this become relevant? Select from the options below:
• Mid-term - 2-4 months ?

tracked in https://github.com/mc2-center/data_curator/issues/8

AC:
Add tryCatch to the function generating the manifest for better capture errors and update waiter once error is encountered.


update dependencies to remove deprecation warnings on install, update…

… engines since tests are not running anymore with --harmony-generators

Coverage remained the same at 100.0% when pulling c4ad063821418835d7e2f0bacab6ada1852a02e7 on EugeneHlushko:master into 5df1e787cdd4475806f788f66aedd2fc5f69ceff on koajs:master.

Coverage remained the same at 100.0% when pulling bdb3440cd9cc02ca1de8d72bae4c629909827f20 on EugeneHlushko:master into 5df1e787cdd4475806f788f66aedd2fc5f69ceff on koajs:master.

Coverage remained the same at 100.0% when pulling e7a653c93705405d2d843fc854c2df72477d32f3 on EugeneHlushko:master into 5df1e787cdd4475806f788f66aedd2fc5f69ceff on koajs:master.

@EugeneHlushko this module is deprecated, try https://github.com/koajs/compress


voiceChannel.leave() : TypeError: Cannot read property 'leave' of null

When leaving a channel, I'm meeting the following error :

C:\Users\remon\Documents\workspace\discord-voice-recorder\bin\commands.js:48
voiceChannel.leave();
^
TypeError: Cannot read property 'leave' of null
at Object.exports.exit (C:\Users\remon\Documents\workspace\discord-voice-recorder\bin\commands.js:48:18)
at Client. (C:\Users\remon\Documents\workspace\discord-voice-recorder\index.js:22:51)
at Client.emit (events.js:376:20)
at MessageCreateAction.handle (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\discord.js\src\client\actions\MessageCreate.js:31:14)
at Object.module.exports [as MESSAGE_CREATE] (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\discord.js\src\client\websocket\handlers\MESSAGE_CREATE.js:4:32)
at WebSocketManager.handlePacket (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\discord.js\src\client\websocket\WebSocketManager.js:345:31)
at WebSocketShard.onPacket (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\discord.js\src\client\websocket\WebSocketShard.js:443:22)
at WebSocketShard.onMessage (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\discord.js\src\client\websocket\WebSocketShard.js:300:10)
at WebSocket.onMessage (C:\Users\remon\Documents\workspace\discord-voice-recorder\node_modules\ws\lib\event-target.js:132:16)
at WebSocket.emit (events.js:376:20)

Any idea where it comes from and how to fix it ?

Note : it happens only when the use leave the channel first.
When the user stay in the channel and send the command, it work properly.

Ok I changed :
const { channel: voiceChannel, connection: conn } = msg.guild.voiceStates.cache.last();

into
    var { channel: voiceChannel, connection: conn } = msg.guild.voiceStates.cache.last();
    if (voiceChannel == null){
        var { channel: voiceChannel, connection: conn } = msg.guild.voiceStates.cache.first();
    }

and it seems to work. I don't know enough javascript to find the actual solution, but this dirty work-around is fine for my usage.

Turned out my fix does not work all the time...


100% CPU while waiting for response

I use this w/o a notification channel:
        e = connector.getResourceValue(id, KNOCK_RESOURCE)
        while not e.isDone():
            time.sleep(0.1)
        if e.error:
            raise Error(e.error)
        pprint.pprint(e.raw_data)

This spins the CPU at 100% and never completes.

Agreed, do you have a proposed solution?

I don't really know why Python eats 100% CPU here. Maybe sleep(1) instead of None? Haven't tested this.

Kk i will add it to the list of things to investigate during the quarterly review


Cassandra Sink JSON parse fail with error

I am posting some JSON with console consumers and those get inserted successfully but if producer sends JSON to kafka topic then sink get crashed even while noop is used.
Still my console consumers shows the JSON as expected and when i do post same using console producer that also works. Is it problem with serialize and deserialize.
Any suggestion.
Logs:
Caused by: org.apache.kafka.common.errors.SerializationException: com.fasterxml.jackson.core.JsonParseException: Unexpected character ('H' (code 72)): Expected space separating root-level values
at [Source: [B@52f6ec1c; line: 1, column: 5]
Caused by: com.fasterxml.jackson.core.JsonParseException: Unexpected character ('H' (code 72)): Expected space separating root-level values
at [Source: [B@52f6ec1c; line: 1, column: 5]

What is the json? Use kafka console consumer to grab it.
Get the latest cod, build and deploy and will log the json messag e

i built latest code of stream reactor and used latest jar but still could not get right logs. I also have that JSON logged in producer program/logs. JSON looks correct because i posted same using console producer and that went through and inserted... simple key:value things. But what is the deserializer JsonConverter expect ?

What is the setup of your Connect cluster?

This is coming from the framework not the sink.

Initially I was using confluent & streaming reactor & I was facing this error. Then I moved to Apache Kafka & streamreactor jars in lib but still faced same error. can you help me to understand if producer says the key & value are byte []
		pm := &kafka.ProducerMessage{
			Topic: msg.Topic,
			Key:   kafka.ByteEncoder(msg.Key),
			Value: kafka.ByteEncoder(msg.Payload),

t.Printf("---\nTo Kafka\nTopic: %s\nKey: %x\nPayload:\n%s\n---\n\n", pm.Topic, pm.Key, pm.Value)

Did you set the use.schema=false on the JsonConverter?

In connect-distributed.properties
key.converter.schemas.enable=false
value.converter.schemas.enable=false
internal.key.converter.schemas.enable=false
internal.value.converter.schemas.enable=false

Take your raw bytes and pass it through JsonDeserializer instance (class in kafka) and see. What enconding do you use Utf-8?

You need to check what I said earlier. You are not even getting to the sink code.

I am my self not specifying anything any where. Producer is not in my control. I am only using reactor sink to push it to Cassandra. So my understanding says I should ask producers to use JsonSerializer with utf-8? When i push JSON from console producer it means it uses string serializer and so sink is able to process it. So should i get producer changed to use JsonSerializer or StringSerializer?
Because I am using everything default from reactor sink.

What should I do if my Producer is sending actually messages which are JsonSerialized?

Change your converter from jsonconverter to stringconverter. The sink will try to handle string payloads containing json.

I really appreciate your help this working after changing to StringConverter.

Cool!


feat(test): add tests to signup

sdfasdf

Coverage increased (+66.0%) to 65.957% when pulling 3e30a190f919219a3a1f5239c3f3df3b687c4416 on feat/tests into caf2e911902be38df9fdbb75cd7acf4764152f90 on master.

Coverage increased (+66.0%) to 65.957% when pulling 3e30a190f919219a3a1f5239c3f3df3b687c4416 on feat/tests into caf2e911902be38df9fdbb75cd7acf4764152f90 on master.


Decred logo blurry

The DCR logo in the middle of the QR codes generated on the receive screen is blurry for some reason. Should be sharp.

the logo isn't blurry but rasterised, closing because it's intended behaviour


Error: Install-BcContainerHelper

Starting this morning 5am we get an error within our pipelines while running Install-BcContainerHelper.ps1:
##[error]Sort-Object : Cannot convert value ".AL-Go.project.settings.schema.json" to type "System.Version". Error: "Version  string portion was too short or too long."
Reason seems to be an update here, which was added at 5:15 this morning:
https://bccontainerhelper.azureedge.net/public?comp=list&restype=container
  <Blobs>
    <Blob>
      <Name>.AL-Go.project.settings.schema.json</Name>
      <Url>https://bccontainerhelper.blob.core.windows.net/public/.AL-Go.project.settings.schema.json</Url>
      <Properties>
        <Last-Modified>Wed, 22 Jun 2022 05:14:37 GMT</Last-Modified>
        ...

Me, too. @freddydk ?

Sorry - my bad - this is fixed now.
I needed a location to place an arbitrary file and didn't consider that this file would be seen as a containerhelper version.
Won't happen again

Let me know if this is fixed or I need to cycle a cache somewhere, thanks

Local powershell works again. Our build server takes a bit longer...

Thanks

Also Build Server installed BcContainerhelper again. Thanks a lot!


error starting the container

I get the following starting the container:
15-May-2018 20:59:06.120 INFO [localhost-startStop-1] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
15-May-2018 20:59:06.366 INFO [localhost-startStop-1] org.pentaho.di.ui.spoon.WebSpoonServletContextListener.contextInitialized
Error reading information from resource
/usr/local/tomcat/system/kettle/slave-server-config.xml (No such file or directory)
15-May-2018 21:00:49.382 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployDirectory Deployment of web application directory [/usr/local/tomcat/webapps/spoon] has finished in [115,727] ms
15-May-2018 21:00:49.383 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployDirectory Deploying web application directory [/usr/local/tomcat/webapps/ROOT]
15-May-2018 21:00:49.396 INFO [localhost-startStop-1] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
15-May-2018 21:00:49.398 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployDirectory Deployment of web application directory [/usr/local/tomcat/webapps/ROOT] has finished in [15] ms
15-May-2018 21:00:49.405 INFO [main] org.apache.coyote.AbstractProtocol.start Starting ProtocolHandler ["http-nio-8080"]
15-May-2018 21:00:49.431 INFO [main] org.apache.coyote.AbstractProtocol.start Starting ProtocolHandler ["ajp-nio-8009"]
15-May-2018 21:00:49.435 INFO [main] org.apache.catalina.startup.Catalina.start Server startup in 115819 ms
15-May-2018 21:00:49.512 INFO [http-nio-8080-exec-1] org.apache.coyote.http11.Http11Processor.service Error parsing HTTP request header
Note: further occurrences of HTTP header parsing errors will be logged at DEBUG level.
java.lang.IllegalArgumentException: Invalid character found in method name. HTTP method names must be tokens
at org.apache.coyote.http11.Http11InputBuffer.parseRequestLine(Http11InputBuffer.java:426)
at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:687)
at org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:66)
at org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:790)
at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1459)
at org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
at java.lang.Thread.run(Thread.java:748)
15-May-2018 21:00:49.512 INFO [http-nio-8080-exec-3] org.apache.coyote.http11.Http11Processor.service Error parsing HTTP request header
Note: further occurrences of HTTP header parsing errors will be logged at DEBUG level.
java.lang.IllegalArgumentException: Invalid character found in method name. HTTP method names must be tokens
at org.apache.coyote.http11.Http11InputBuffer.parseRequestLine(Http11InputBuffer.java:426)
at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:687)
at org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:66)
at org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:790)
at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1459)
at org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
at java.lang.Thread.run(Thread.java:748)
15-May-2018 21:04:12.610 INFO [Thread-11] org.apache.coyote.AbstractProtocol.pause Pausing ProtocolHandler ["http-nio-8080"]
thanks!!

Looks like you tried to access webSpoon through HTTPS.
Please try again with HTTP, i.e., http://localhost:8080/spoon/spoon.


build(deps): added tomlkit

Fixes broken tests for config due to missing dependency.
Why didn't it break CI though?

Indeed strange that CI didn't catch this

@ATheorell I think since poetry depends on tomlkit, and poetry seems to be installed in the same venv as gpt-engineer (bad practice, but no biggie), it might have gotten tomlkit that way.


OAK-10213 : updated DocumentStore API to support bulk modify request

@rishabhdaim , you mentioned offline a fix you did with a different return value, but I can't remember all the details and can't find the diff due to the force pushes. Which change were you referring to?

Thanks, Julian Reschke for the feedback. However, I see the following issues with putting the new findAndUpdate API into MongoVersionGCSupport:

I don't see it as the right candidate for database-related operations. It's more of a support class for Mongo-related GC ops.

Having an API in the DocumentStore gives us the following advantages:

We are sure of the lease check since it will be done automatically for us. We already have an epic to solve issues which are caused by late-writes i.e. writing after the lease had expired.

We can take into account throttling as well. I really don't want to perform heavy garbage collection operations if my system is already under heavy load.

I can reuse some of the already existing code.

We won't miss the stats for those operations which are done inside DocumentStore.


Defender for Cloud integration with MDE for Linux VMs, image requires update.

[Enter feedback here]
The screenshot displaying "Enable for Linux machines (Preview)" requires updating to reflect the integration for Linux with Defender for Cloud integration with MDE now being Generally Available.
I have taken a screenshot with the now update UI selection that no longer contains (Preview) I believe this will help prevent confusion if readers scroll past the top of the page and only view the image.
updated image is pasted below.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 970863a5-ab27-6c83-9496-b025dd6bf323
Version Independent ID: a1f053f5-1004-6b6b-8b37-bc31599aa23b
Content: Using the Microsoft Defender for Endpoint license included with Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/integration-defender-for-endpoint.md
Service: defender-for-cloud
GitHub Login: @memildin
Microsoft Alias: **memildin

**

@Josh-chadison-1 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@Josh-chadison-1, thanks for the feedback and for taking the time to help us improve the docs.
All the graphics on this page have been updated in a recent project. The changes should go live in the coming days.
I'm going to close this issue, but feel free to comment again here or open a new one if you have additional requests.
#please-close


Active diff snapshot not automatically run

It's not very safe to be messing with git at startup. There are a lot of edge cases. Most users won't use the sampler at all and even with this change the sampler will still work well assuming the changes being made aren't related to the active diff.
Pull Request Checklist

[x] Documentation has been updated, or this change doesn't require that

The user said they had a workaround with .gitignore. I'll wait to merge until Grant's seen it.


Set Material for TMP Dissolve?

Describe the question
I hope I'm not missing something obvious! Apologies if I am. When I add a dissolve effect to a TM Pro object, there's never a dissolve material set/available:

This means I can't change the color mode or noise texture. Is this intentional/necessary, or a bug, or am I simply missing something?
Environment (please complete the following information):

Platform: Mac
Unity: 2019.3.2 f1
UIEffect: v3.0.1 (2018-11-07)

Additional context
Nothing I can think of. Thank you for your hard work! UIEffect is a great package <3

Not a bug at all, my apologies.
For reference, the dissolve material properties are added to the regular TMP material, like this:


Check why Shield of Valtheim crashes people

Seems to crash when given to followers

removing it in next update


trusted_certs_dir doesn't seem to work if a path is constructed from multiple strings

Example of knife.rb
Working:
current_dir       = File.dirname(__FILE__)
node_name         'unicorn'
chef_server_url   'https://<IP_ADDRESS>/organizations/my_org'
client_key        "#{current_dir}/my_key.pem"
cookbook_path     "#{current_dir}/../local-mode-cache/cache/cookbooks"
trusted_certs_dir "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/unicorn-cluster-data/../trusted_certs"

If trusted dir is using the current_dir variable to construct an identical string, cert validation fails
Broken
...
trusted_certs_dir "#{File.expand_path(current_dir)}/../trusted_certs}"

I took a quick look at it with pry and perhaps the error is because .. are used?
pry from working config:
$ k ssl check
WARN: Unresolved specs during Gem::Specification.reset:
      httpclient (>= <IP_ADDRESS>, ~> 2.2)
      gyoku (>= 0.4.0)
      mini_portile (~> 0.6.0)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
before_session hook failed: Pry::CommandError: Cannot locate this method: allocate. Try `gem-install pry-doc` to get access to Ruby Core documentation.
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/pry-0.10.1/lib/pry/method.rb:489:in `pry_doc_info'
(see _pry_.hooks.errors to debug)
2.1.4 (Chef::Config):0 > trusted_certs_dir
=> "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/trusted_certs"
2.1.4 (Chef::Config):0 > exit
before_session hook failed: Pry::CommandError: Cannot locate this method: allocate. Try `gem-install pry-doc` to get access to Ruby Core documentation.
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/pry-0.10.1/lib/pry/method.rb:489:in `pry_doc_info'
(see _pry_.hooks.errors to debug)
2.1.4 (Chef::Config):0 > trusted_certs_dir
=> "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/unicorn-cluster-data/../trusted_certs"
2.1.4 (Chef::Config):0 > exit
Connecting to host <IP_ADDRESS>:443
Successfully verified certificates from `<IP_ADDRESS>'

Broken config
$ k ssl check
WARN: Unresolved specs during Gem::Specification.reset:
      httpclient (>= <IP_ADDRESS>, ~> 2.2)
      gyoku (>= 0.4.0)
      mini_portile (~> 0.6.0)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
before_session hook failed: Pry::CommandError: Cannot locate this method: allocate. Try `gem-install pry-doc` to get access to Ruby Core documentation.
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/pry-0.10.1/lib/pry/method.rb:489:in `pry_doc_info'
(see _pry_.hooks.errors to debug)
2.1.4 (Chef::Config):0 > trusted_certs_dir
=> "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/trusted_certs"
2.1.4 (Chef::Config):0 > exit
before_session hook failed: Pry::CommandError: Cannot locate this method: allocate. Try `gem-install pry-doc` to get access to Ruby Core documentation.
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/pry-0.10.1/lib/pry/method.rb:489:in `pry_doc_info'
(see _pry_.hooks.errors to debug)
2.1.4 (Chef::Config):0 > trusted_certs_dir
=> "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/unicorn-cluster-data/../trusted_certs}"
2.1.4 (Chef::Config):0 > exit
Connecting to host <IP_ADDRESS>:443
ERROR: The SSL certificate of <IP_ADDRESS> could not be verified
Certificate issuer data<EMAIL_ADDRESS>
Configuration Info:
...

Both trusted_certs_dirs have '..'s in them so that's not it.  The problem looks like a trailing '}' at the very end of the busted trusted_certs_dir:
> trusted_certs_dir
=> "/Users/alexvinyar/Documents/Projects/unicorn/unicorn-cluster_chefconf2015/unicorn-demo/.chef/unicorn-cluster-data/../trusted_certs}"

The extra bracket is coming from somewhere, probably some sort of string concat..
I hadn't tested this in a while. I'll see if that's still going on, and follow up.

just for the kicks verified it just now, the error seems to have gone away.


Update previous release Crystal 1.1.0

Still waiting on 1.1.0 docker images

Specs are failing due to #10961, but that's fine (for this PR).

There are still a lot of CI failures, even after #10964 has been merged.

windows-job fails because of a regression introduced in #10960. The spec should just be disabled on windows ( #10971).
test_alpine hits an invalid memory access somewhere in compiler specs. Need to investigate where and why. Might be related to #10964.
aarch64 musl-build has no useful debug information. aarch64 jobs have been failing a lot, lately. gnu-build succeeded at least, though. We need to look into that.

I think we should still merge this. Red CI isn't a huge issue (especially at the beginning of a release cycle) and it helps avoid regressions like  #10971.


Document Cert rotation and custom certs feature

Document cert rotation and custom certs
https://github.com/rancher/rke/issues/450
https://github.com/rancher/rancher/issues/16795

Duplicate of #1111


loader is required

this framework is lightest and fastest
but just before loading it looks too bad there must be loader on index page before all

Hey,
Sorry, I am not sure to understand your issue.
Are you trying to force the loading of the default page (home page)?

yes we need to show loading  rather then white page for first time

What about an indicator?
phonon.indicator("HelloWorld", false);

Documentation API


Address review comments from #993

Refactor rename to avoid a force unwrap
Refactor multi-file rename tests to specify expected renamed source instead of asserting on edits

@swift-ci Please test

@swift-ci Please test


Add polyfill for createCacheDirectoryFlag capability.

LGTM!

Updated to use the new name needsCacheFlag (for capability) and needsCache (for the actual value) as mentioned in https://github.com/broccolijs/broccoli-plugin/issues/19.


Multi-day weather forecast added

Fixes Issue
#15
Proposed Changes

[x] Created multi-day weather forecast

Checklist (Check all applicable boxes)

[x] My code adheres to the project's coding style.
[x] My changes require updates to the project's documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests have passed successfully.
[x] This PR does not contain any plagiarized content.
[x] The title of my pull request provides a brief description of the requested changes.

[x] I am contributing independently.
[x] I am contributing for Hacktoberfest 2023.

@Abhii-07 please resolve conflicts ,you may not have updated the code of recent commits .


Want to add content in B.Tech notes section

Is your feature request related to a problem ? Please describe. ✍
Enhancement
The notes section doesn't have much content and as we know, the primary goal of this website is providing notes and assignments to college students, so it should be good to add some relevant content to the courses and subjects.

Describe breifly how would you implement the same.
Solution_steps
I would like to add some relevant content year-wise under B.Tech section. So, it will give good user experience and fulfilling its goal to provide relevant notes under each course.

Additional context
No response
Are you contributing for a program?
GSSoC2023
Code of Conduct

[X] I have read and agree to the CONTRIBUTING GUIDELINES of this project.

Hello @amupedia2021 Sir,
I want to add content in B. Tech notes section. I want to create separate section on the basis of different subjects. I am a Frontend Developer and I want to work on these issues So, can you please assign this issue to me under GSSOC'23?


Issue with files downloaded or uploaded to S3

If files are downloaded using the GET method of the flow the files are getting corrupted. I tried with the below code also as said in the release. But it is not working. can you help.
msg.payload=Buffer.from(msg.payload.Body).toString("utf-8")

I'm using it this way:
var buf = new Buffer(msg.payload.Body, "utf-8").toString();
msg.payload=JSON.parse(buf);
return msg;

@soumya-github can you confirm the suggestion from @leonardobaiao solved your issue or not.  Issue will close in 7 days otherwise.

closed due to no response

Had a similar problem while trying to upload a png file to s3.
The file is in a buffer. When downloading the file the image is corrupted.
Opening with a text editor shows
{"type":"Buffer","data":[137,80,78,71,13,10,26,10,0,0, ...

I see that it is expected behaviour, as the documentation for put says "stringifies msg.payload and stores it in s3://buckey/key".
So sorry for the comment but I'm asking myself how to write binary files.

I can't think of good a solution off the top of my head for this sorry.
One thing that might make it easier to use the uploaded file might be to base64encode the data prior to uploading,  this will at least upload it in a standard encoding.


edit travis file to use chrome browsers on travis

What does this PR do?
These PR adds the chrome browser to build the test on travis
Description of Task to be completed?
Add Coveralls and code climate to the application
How should this be manually tested?
This can be tested by the badge of the result on the README.md
Any background context you want to provide?
Not Applicable
What are the relevant pivotal tracker stories?
Add Coveralls to the application
Screenshots (if appropriate)
Not Applicable
Questions:
No questions

Changes Unknown when pulling 3902325db8a45471cbe6fc8daaa7b0ffeed133f6 on add-test-to-code-coverage into ** on develop**.

Changes Unknown when pulling 3902325db8a45471cbe6fc8daaa7b0ffeed133f6 on add-test-to-code-coverage into ** on develop**.


Feature Request : Activity Indicator and Retry Button

Feature Request(For more imporvement of this library)
1.Adding Activity Indicator for loading the image before it finished downloading on both slideshow and zoomview and let us adding placeholder image by our self.
2.If the Image download failed because of connection?Any feature for retry button?
Thanks for helping me.I hope you got my feature request,too.It will be perfect this time.And permission to share this library on SO because many people are hoping to get this.Thanks.

I still plan to implement this feature when have more time. Just writing down a related task to analyse possible error handling ways so user of the library can observe and react to them.

Ok, I finally implemented this 🎉 Already available in master, new release will be made soon.


Support kubeflow.org/paddlejob

What type of PR is this?
/kind feature
What this PR does / why we need it:
I added the paddlejob-controller to the frameworkjob-manager.
Which issue(s) this PR fixes:

Part-of https://github.com/kubernetes-sigs/kueue/issues/297
Special notes for your reviewer:
I verified the behavior using https://github.com/kubernetes-sigs/kueue/pull/1148 example in the real cluster.
Does this PR introduce a user-facing change?

Support kubeflow.org/paddlejob

/hold for review.

@tenzen-y feel free to cancel the hold.

@tenzen-y feel free to cancel the hold.

Thank you!
/hold cancel


Agents going into drain because of unwanted disk partitions are above threshold

Impact of the bug
WMAgent
Describe the bug
We have a couple of agents that were automatically set to drain mode because one of their disk partitions is above the threshold set in the agent (set to 85% utilization at the moment).
How to reproduce it
Just fill up any partition in the node above 85%.
Expected behavior
We need to either disable those specific partitions in the configuration, such that the agent stops monitoring and acting on them.
Or we should actually create an allowed list of partitions to be monitored (this would be a larger change though).
Additional context and error message
Examples:
agent: vocms0281(<IP_ADDRESS>)
disk warning:
    /etc/group:85%

and
agent: vocms0283.cern.ch (<IP_ADDRESS>)
disk warning:
    /data/certs:85%

It seems to be that the /data mount on these machines has reached 85%. The df -klP command inside the wmagent containers however displays /etc/group and /data/certs inside vocms0281 and vocms0283.

I looked at the container mounts and noticed that inside the container df -klP outputs the mount Destination out of space instead of the source.
For example on vocms0283 the output of docker inspect wmagent and looking into the mounts, The /etc/group is mounted from /data/dockerMount/admin/etc/group.
 "Mounts": [
            {
                "Type": "bind",
                "Source": "/tmp",
                "Destination": "/tmp",
                "Mode": "",
                "RW": true,
                "Propagation": "rprivate"
            },
            {
                "Type": "bind",
                "Source": "/data/dockerMount/admin/wmagent",
                "Destination": "/data/admin/wmagent",
                "Mode": "",
                "RW": true,
                "Propagation": "rprivate"
            },
            {
                "Type": "bind",
                "Source": "/data/dockerMount/admin/etc/group",
                "Destination": "/etc/group",
                "Mode": "",
                "RW": false,
                "Propagation": "rprivate"
            },
            {
                "Type": "bind",
                "Source": "/etc/tnsnames.ora",
                "Destination": "/etc/tnsnames.ora",
                "Mode": "",
                "RW": false,
                "Propagation": "rprivate"
            },
            {
                "Type": "bind",
                "Source": "/data/certs",
                "Destination": "/data/certs",
                "Mode": "",
                "RW": true,
                "Propagation": "rprivate"
            },
            {
                "Type": "bind",
                "Source": "/etc/sudoers",
                "Destination": "/etc/sudoers",
                "Mode": "",
                "RW": false,
                "Propagation": "rprivate"
            },

For vocms0281

why do we have a problem
A couple of additional pointers (maybe these are known to everybody, but future dario will thank me):
data in wmstats is displayed at the following line only if agentInfo.disk_warning is not null
https://github.com/dmwm/WMCore/blob/f3c25d8cde6475f56a28662659eba4796d0753f3/src/couchapps/WMStats/_attachments/js/Views/HTMLList/WMStats.AgentDetailList.js#L24
that information comes from AgentStatusPoller
https://github.com/dmwm/WMCore/blob/f3c25d8cde6475f56a28662659eba4796d0753f3/src/python/WMComponent/AgentStatusWatcher/AgentStatusPoller.py#L204
that uses df -klP (thank Hassan for spotting it!), parsing only the last two columns
https://github.com/dmwm/WMCore/blob/f3c25d8cde6475f56a28662659eba4796d0753f3/src/python/Utils/Utilities.py#L96
As Hassan described, df -klP from inside the docker container reports path of the directory from the container point of view.
Since we bind mount from the vm host into the docker container multiple directories that in the host are inside /data directory on the host, when df runs it selects one of the "destination" mountpoint that refer to the host filesystem/partition that it is reporting data about. For example, notice how df reports disk use on the host [0] (filesystem: /dev/vdb, mountpoint /data), while on the container the mountpoint is not consistent across different wmagents [1] and [2].
what can we do to solve it
So, this is an unexpected regression of deploying the WMCore with docker containers.
The cleanest solution that comes to my mind is to always identify a partition by the "filesystem" column and not by the "mounted on", chaning in Utilities.py
- diskPercent.append({'mounted': split[5], 'percent': split[4]})
+ diskPercent.append({'mounted': split[0], 'percent': split[4]})

so that we consistently report the "/data" directory with "/dev/vdb".
Otherwise, the internet suggests to bind mount the root director of the host in readonly [3], but i really dislike this approach.
In any case, I would like to hear a quick opinion from @todor-ivanov , who i trust with all these docker shenanigans :)

anatomy of df -klP
[0]
(on the host)
cmst1@vocms0281:dmapelli $ df -klP
Filesystem     1024-blocks      Used Available Capacity Mounted on
/dev/vda1        334914540  14111860 320802680       5% /
/dev/vda15          556948      7228    549720       2% /boot/efi
tmpfs              8388608   1069908   7318700      13% /mnt/ramdisk
/dev/vdb<PHONE_NUMBER> 827451044 154616384      85% /data

[1]
(inside the container, vocms0281)
(WMAgent-<IP_ADDRESS>) [cmst1@vocms0281:current]$ df -klP
Filesystem     1024-blocks      Used Available Capacity Mounted on
overlay          334914540  14111860 320802680       5% /
/dev/vda1        334914540  14111860 320802680       5% /tmp
/dev/vdb<PHONE_NUMBER> 827448800 154618628      85% /etc/group

[2]
(inside the container, vocms0256)
(WMAgent-<IP_ADDRESS>) [cmst1@vocms0256:current]$ df -klP
Filesystem     1024-blocks      Used Available Capacity Mounted on
overlay          334914540  11805236 323109304       4% /
/dev/vda1        334914540  11805236 323109304       4% /tmp
/dev/vdb<PHONE_NUMBER> 772601688 209465740      79% /data/certs

[3] https://github.com/netdata/netdata/discussions/16444#discussioncomment-7635267

hi @amaltaro your explanation is complete and good.
About:

Otherwise, the internet suggests to bind mount the root director of the host in readonly [3], but i really dislike this approach.

I definitely would not want to enlarge  with yet another item my list of things to regret about before I meet the Reaper.
About this though:

The cleanest solution that comes to my mind is to always identify a partition by the "filesystem" column and not by the "mounted on", chaning in Utilities.py

Your suggestion is one way to go, but in this case  people will always see only the volume as reaching the threshold, which is indeed true and fair. Another way to go would be simply to visualize all mount points including the duplicate mounts:
in https://github.com/dmwm/WMCore/blob/f3c25d8cde6475f56a28662659eba4796d0753f3/src/python/Utils/Utilities.py#L96
the following change should do the job:
- df = subprocess.Popen(["df", "-klP"], stdout=subprocess.PIPE)
+ df = subprocess.Popen(["df", "-klPa"], stdout=subprocess.PIPE)

The choice should be up to whom ever takes the issue to work on. Both will do.


test(cypress): Adds waits to avoid rapidly changing dates to prevent …

…test failure
This build was failing due to this issue.  In the future a more broad solution could be considered.

This is a bandaid fix. We need to write up an issue to fix the underlying issue in a more central fashion so that waits/sleeps aren't littered all over the tests.


Combo overlaps after collapsing using 'dagre' layout.

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link
https://codepen.io/mostopalove/pen/OJxbYXg
Steps to reproduce

collapse "Combo 1" by double click
Result: Combo overlapped.

Environment
Info

g6
4.3.11

System
MacOS 11.6

Browser
Chrome 96.0.4664.93

solved by v4.7.15


Substitute eval with AST parser

Adapted from @proteneer's patch to the 0.1 version (see #189).
I've also added a new test_io.py file.

Tests in test_forcefield.py pass locally.

Great, thank you!


AIML.MHW

1. System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installation (pip package or built from source):
TensorFlow library (version, if pip package or github SHA, if built from source):

2. Code
Provide code to help us reproduce your issues using one of the following options:
Option A: Reference colab notebooks

Reference TensorFlow Model Colab: Demonstrate how to build your TF model.
Reference TensorFlow Lite Model Colab: Demonstrate how to convert your TF model to a TF Lite model (with quantization, if used) and run TFLite Inference (if possible).

(You can paste links or attach files by dragging & dropping them below)
- Provide links to your updated versions of the above two colab notebooks.
- Provide links to your TensorFlow model and (optionally) TensorFlow Lite Model.

Option B: Paste your code here or provide a link to a custom end-to-end colab
(You can paste links or attach files by dragging & dropping them below)
- Include code to invoke the TFLite Converter Python API and the errors.
- Provide links to your TensorFlow model and (optionally) TensorFlow Lite Model.

3. Failure after conversion
If the conversion is successful, but the generated model is wrong, then state what is wrong:

Model produces wrong results and/or has lesser accuracy.
Model produces correct results, but it is slower than expected.

4. (optional) RNN conversion support
If converting TF RNN to TFLite fused RNN ops, please prefix [RNN] in the title.
5. (optional) Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Hi @Milehigh-wrld,
As per your query even though the conversion is successful, but model produces wrong results and less due to insufficient data, architectural flaws, in correct hyperparameter tuning.  The model produces correct results and is slower than expected: This is right, even though the model produced correct results, based on the target hardware the speed might differ. Based on the hardware different optimization techniques help to improve speed.
In your use case, the tflite model conversion is successful and accuracy also as expected. Refer the gist for accuracy and size.  Still if any specific query regarding the model, please feel to elaborate.
The issue is the same as  62833.
Thank You

Closing as it is only the template


Store offset ranges for parquet files

Looks mostly good, but needs to be rebased after #184 is committed, and I think we need to rename offsetRanges to splitOffsets.

Can you rebase on the current master to remove the commits from #184?

Can you rebase on the current master to remove the commits from #184

Done.

@samarthjain, I think that @rdsr is right about the method names that should be updated. Otherwise, it looks good. When updating the metrics, could you also fix the minor formatting that I noted?

+1
Thanks for contributing this, @samarthjain!


Add QOL and real mouse blocks based off suggestion (I think)

self explanitory

Could you update the description to include the list of blocks you're adding?

Could you update the description to include the list of blocks you're adding?

Keeping this and pen+ V6 up due to them still having the possibility of being merged

Truely a moment of all time

Patience is a virtue


Repeatedly installing homestead

It appears that we're ending up going through ensureHelmChart() multiple times concurrently, leading to issues because helm list would return empty (because it hasn't been installed yet, it's in progress), and then attempting to run helm install again, which errors out (because an install is in progress).  This leads to a dialog box on startup about not being able to install homestead, and then the cluster goes into an error state.

Part of this is due to the transition from Homestead to Rancher and having a rogue value in the settings. I'm detecting the normal transition of going from prefs version 1 to 2, dropping "HOMESTEAD" for "RANCHER", but going the other way requires a different approach.  Added it while working on this issue.

Closing as we are removing homestead/rancher. It will no longer be part of the mvp.


Module not found: Can't resolve '@dhaiwat10/react-link-preview'

I have tried yarn and npm..... Any suggestions?

The package is there live on npm: https://www.npmjs.com/package/@dhaiwat10/react-link-preview
Where are you seeing this error? On running npm install?

Disregard.  I was in wrong directory...

Haha happens to the best of us.


fix: print shallow warning only once

fixes https://github.com/shuding/nextra/issues/573

Thanks!


ORC-975: Avoid double counting closestFixedBits

What changes were proposed in this pull request?

310  int idx = encodeBitWidth(findClosestNumBits(data[i]));

public int encodeBitWidth(int n) {
    n = getClosestFixedBits(n);
    .....
}

public int findClosestNumBits(long value) {
  final int numBits = 64 - Long.numberOfLeadingZeros(value);
  return getClosestFixedBits(numBits);
}

getClosestFixedBits is called twice.
Extracts the new method findNumBits from findClosestNumBits. Avoid double counting closestFixedBits.
Why are the changes needed?

Avoid redundant calls.
How was this patch tested?

Pass the CIs.

This is not backported to branch-1.7 because there is a conflict.


importer - check point rule refhandler issue

2024-07-02T14:20:26+0200 [INFO ] [fwo_base.p:writeAlert: 153] FWORCHAlert - "source": "import", "mgmId": 21, "title": "import error", "description": "[{'import_errors': 'ERR-ImpMain@ImpGlobRefException caught in import_rule_refhandler_main while handling rule b116bee8-1290-48cd-8fbc-752faa2a678d'}]", "alertCode": 14, "severity": 2, "import_id": 16763, "mgm_name": "Primary CP Manager GV-FWMGT-101"
2024-07-02T14:20:26+0200 [INFO ] [common.py :complete_i: 310] import_management: import no. 16763 for management Primary CP Manager GV-FWMGT-101 (id=21) threw errors, change_count: 0, duration: 5976s, ERRORS: [{'import_errors': 'ERR-ImpMain@ImpGlobRefException caught in import_rule_refhandler_main while handling rule b116bee8-1290-48cd-8fbc-752faa2a678d'}]
fworch@fworch-srv-1:~/importer$

But rule could not be found. Could be an artefact.
Need to issue direct API call

seems to be related to SIP objects

fixed, I guess


Heatmap Example

Hi
Your work has made lot of enhancement to Zeppelin and and Zeppelin more appealing to business users.
How to create Heatmap? I could not find any example.

Thanks.
I have added heatmap demos


USD equivalent balance is missing from the top nav

The USD equivalent balance is missing from the top nav even when it's enabled in settings.
I believe it might have been removed by accident, it should be re-added back if that's the case

What it should look like

Looks like a good first issue, would like to work on it.

@amass01 it's currently not assigned to anyone, so go ahead, and update the godcr room(https://matrix.to/#/%23godcr:planetdecred.org) you'd be working on it


Commented-out recursive call in _reboot.scss

Hi,
I have timeout (or memory if increase exec time)  problem using symfony / assetic / sass / bootstrap for release 3.5.5 and 4.0.0-alpha.
After some search about 4.0.0-alpha, it seems to be cause from a comment in _reboot.scss in line 46 :
// .class-name { @import "bootstrap"; }).
To solve this, I just suppress the '@' :
// .class-name { import "bootstrap"; }).
It's look as saas execute @import in comment.

I cannot reproduce this with either Ruby Sass or libsass, which are the only Sass compilers that we support. Presumably you're using some other buggy Sass compiler.

You're right, bootstrap is not the responsible of my bug.
I use Ruby Sass but the bug come from Assetic (kriswallsmith/assetic) and append before calling Ruby Sass.


Webpack Stuck at 10% building forever

Bug report
Webpack v 5.94.0 stuck at 10% building forever
What is the current behavior?
Result:
➜  web git:(upgrade) ✗ yarn webpack --config webpack.test.js --progress=profile --json=compilation-stats.json
10% building 0/1 entries 0/0 dependencies 0/0 modules^C%

I thought initially it was my plugins and loaders, however even after removing everything with minimal configuration to bundle 1 test js file, it just hangs.
My webpack.test.js:
module.exports = {
  mode: 'development',
  devtool: 'inline-source-map',
  entry: {
    'tool': './test.js',
  },
  output: {
    path: '/Users/rtirtasentana/Development/web/dist',
    filename: '[name].bundle.js',
  },
  stats: {
    all: false, // Disable all by default
    timings: true,
    errors: true,
    warnings: true,
    logging: 'verbose', // Show verbose logging
    reasons: true, // Show reasons why modules are included
    moduleTrace: true, // Show trace of modules for better debugging
  },
};

test.js is:
import React from 'react';

export const MyCompoonent = () => {
  return (

TestingABC

  );
};

What is the expected behavior?
it should run without being stuck
Other relevant information: (i didn't use any of the packages in the minimal webpack config above but it still hangs)
  System:
    OS: macOS 14.6.1
    CPU: (16) arm64 Apple M3 Max
    Memory: 13.45 GB / 64.00 GB
  Binaries:
    Node: 18.20.4 - /private/var/folders/8f/g5qfjjhx4h96gvt7jp17yzy00000gn/T/xfs-d14dbc44/node
    Yarn: 4.4.1 - /private/var/folders/8f/g5qfjjhx4h96gvt7jp17yzy00000gn/T/xfs-d14dbc44/yarn
    npm: 10.7.0 - ~/.asdf/plugins/nodejs/shims/npm
  Browsers:
    Chrome: 128.0.6613.120
    Safari: 17.6
  Packages:
    babel-loader: ^9.1.3 => 9.1.3
    clean-webpack-plugin: ^4.0.0 => 4.0.0
    css-loader: ^7.1.2 => 7.1.2
    eslint-import-resolver-webpack: ^0.13.1 => 0.13.9
    html-webpack-plugin: ^3.2.0 => 3.2.0
    lodash-webpack-plugin: ^0.11.6 => 0.11.6
    postcss-loader: ^8.1.1 => 8.1.1
    react-loader-spinner: ^3.1.5 => 3.1.14
    sass-loader: ^16.0.1 => 16.0.1
    style-loader: ^4.0.0 => 4.0.0
    terser-webpack-plugin: ^4.2.3 => 4.2.3
    url-loader: ^4.1.1 => 4.1.1
    webpack: ^5.94.0 => 5.94.0
    webpack-bundle-analyzer: ^4.10.2 => 4.10.2
    webpack-cli: ^5.1.4 => 5.1.4
    webpack-dev-server: ^5.1.0 => 5.1.0
    webpack-manifest-plugin: ^5.0.0 => 5.0.0
    webpack-merge: ^6.0.1 => 6.0.1

Another update is if i use babel-loader, which I initially want to use to parse everything. it got stuck at the babel-loader
webpack.test.js:
module.exports = {
  mode: 'development',
  devtool: 'inline-source-map',
  entry: {
    tool: './test.js',
  },
  output: {
    path: '/Users/rtirtasentana/Development/web/dist',
    filename: '[name].bundle.js',
  },
  module: {
    rules: [
      {
        test: /\.(js|jsx)$/,
        exclude: /node_modules/,
        use: {
          loader: 'babel-loader',
          options: {
            cacheDirectory: true, // Enable caching for faster rebuilds
          },
        },
      },
    ],
  },
  stats: {
    all: false, // Disable all by default
    timings: true,
    errors: true,
    warnings: true,
    logging: 'verbose', // Show verbose logging
    reasons: true, // Show reasons why modules are included
    moduleTrace: true, // Show trace of modules for better debugging
  },
};

result:
➜  web git:(upgrade-node-to-18) ✗ yarn webpack --config webpack.test.js --progress=profile --json=compilation-stats.json
 [webpack.Progress]  | 18 ms building > entries dependencies modules
 [webpack.Progress]  | 76 ms building > import loader ./node_modules/babel-loader/lib/index.js
10% building 0/1 entries 1/1 dependencies 0/1 modules

OMG this comment helps a lot.
https://github.com/webpack/webpack/issues/11678#issuecomment-708252018
I didn't realise we can debug webpack and see where it hangs. apparently its one of the dependencies that causes it too hang. I've removed it and now it works :)


[BUG] Error applying actorsystem

Describe the bug
A clear and concise description of what the bug is.
Version 1.0.0-rc.17 introduced a bug at the time of creating the ActorSystem resource.
When we apply the yaml of the kind ActorSystem the Operator presents the following error in the log
** (RuntimeError) {"default/spawn-system", "spawn-eigr.io/v1", "Kind=ActorSystem, Action=:add"} - Failed applying descending (child) resource {"default/spawn-system-role", "rbac.authorization.k8s.io/v1", "Kind=Role"}. roles.rbac.authorization.k8s.io "spawn-system-role" is forbidden: User "system:serviceaccount:eigr-functions:spawn-operator" cannot patch resource "roles" in API group "rbac.authorization.k8s.io" in the namespace "default"
    (bonny 1.2.1) lib/bonny/axn.ex:406: anonymous fn/4 in Bonny.Axn.apply_descendants/2
    (elixir 1.14.0) lib/enum.ex:2468: Enum."-reduce/3-lists^foldl/2-0-"/3
    (bonny 1.2.1) lib/bonny/axn.ex:382: Bonny.Axn.apply_descendants/2
    (spawn_operator 0.0.0-local.dev) lib/spawn_operator/operator.ex:1: SpawnOperator.Operator.pluggable_builder_call/2
    (bonny 1.2.1) lib/bonny/operator.ex:155: Bonny.Operator.run/4
    (elixir 1.14.0) lib/stream.ex:612: anonymous fn/4 in Stream.map/2
    (elixir 1.14.0) lib/enum.ex:4751: Enumerable.List.reduce/3
    (elixir 1.14.0) lib/stream.ex:1026: Stream.do_transform_inner_list/7
Function: &Bonny.Server.AsyncStreamRunner.run/2
    Args: [#Stream<[enum: #Function<60.124013645/2 in Stream.transform/3>, funs: [#Function<48.124013645/1 in Stream.map/2>, #Function<48.124013645/1 in Stream.map/2>]]>, nil]

After this error, it will not be possible to apply the ActorHost CRD because it will not be able to start the application and proxy containers.
To Reproduce
Steps to reproduce the behavior:

apply ActorSystem CRD
See Operator container logs

Expected behavior
That there are no errors and that the resources are successfully created.
Spawn Operator (please complete the following information):

Version 1.0.0-rc.17

Spawn Proxy (please complete the following information):

Version 1.0.0-rc.17

So the way I understand it, in rc17 you must have introduced code to your HostActor controller which creates RBAC resources. The operator again needs the according RBAC configuration for this (permission to create roles, rolebindings, service accounts, etc.)
You would configure the controller's required RBAC permissions in rbac_rules/0 of the ActorHost controller

Yeah I see now. I stumble upon this every single time. Look at lines 11-14 l. You put the "groupversion" in there. But for rbac permissions you only put the group. Compare the lines with lines 15ff!


[WIP] update travis cache workflow

The idea here is to try to resolve the travis caching errors we've been seeing lately.

Why remove .hoarder-cache? That basically makes hoarder useless.
Removing .targets makes sense, and I'm relatively certain this is part of the issue which underlies a lot of our cache problems, but removing .hoarder-cache is just going to make all PRs much slower to build.

@djspiewak the idea was that we could use s3 to fetch the hoarder artifacts for all jobs. The Compile stage would fetch the hoarder artifacts from s3 from the last successful master build. Then subsequent stages (and jobs within stages) would all fetch the hoarder artifacts from s3 that were put there during this build when the Compile stage succeeded.

If we're not going to make the change to fetch from s3 for all jobs, then I agree we should keep in .hoarder-cache.

@alissapajer Why would we though? Travis already does this for us, literally using S3 in fact. Why would we recreate that infrastructure?

@djspiewak The difference would be that the compile stage would always fetch the master cache, travis will use the previous PR cache on subsequent builds, no?

I thought travis used the master cache if there wasn't one available for a given PR.

@wemrysi @alissapajer Oh, you want to always use the master cache! That is indeed a difference. Why?

@djspiewak the master cache is a stable build that we know the given PR is based off (or should/could be based off). PR caches include all sorts of WIP work (including library up/downgrades) that, even if compiling, might never land in master. Master is what's stable.

This are the caches travis checks, in order:
https://docs-staging.travis-ci.com/user/caching/#Pull-request-builds-and-caches

@alissapajer I mean, that explains why you fetch from master for the first PR build. But why would you fetch from the master cache for subsequent PR builds? Sure, the whole PR is built off master, but we have a more recent cache than that.

@djspiewak within a single PR, someone could still upgrade and downgrade a dependency, or make some other kind of non-binary-compatible change.

@alissapajer Why is that a problem? You can do that locally, too. SBT handles these things really really well. I'm less convinced that hoarder handles them as well, but we don't have much direct evidence of that. So far, every time we've seen hoarder cause incremental compilation weirdness (both in travis and locally), it has been in conjunction with copying the target classes, which hoarder actually doesn't say to do.
Now, hoarder could still be broken, and if that's the case then we need to get rid of it entirely and just deal with the fact that PR builds are going to be about 30 minutes longer than they could otherwise be. But I don't see the reason to keep hoarder but only cached from master.

Just to be really clear, SBT's incremental compilation (which is the mechanism that hoarder hooks into and the databases it is caching) is considerably more sophisticated than a simple binary compatibility check. It understands the dependencies between individual compilation units on a very granular level and it is quite conservative with invalidating those dependencies when things could be broken.

@djspiewak Ok, so you think that ceasing to cache the compilation artifacts should solve the issues we're seeing?

@wemrysi I think so. It's at least worth a shot. If that doesn't solve the issue, then hoarder itself is responsible and we shouldn't be fetching any of its caches at all, master or otherwise.

@djspiewak stepping back, our goal is to fix the caching problems we've seen in travis. I am on board with incrementally working our way toward a solution. If hoarder doesn't say we should be copying over the target classes, then let's start there, by not copying them over. If we still have problems, then we can figure out what next steps to take.

Ok, I'll add hoarder back into the travis cache we can go from there.


Flank-1.3.0 Crashes - Can't run program gcloud

Was trying to test Flank but can't get it to execute correctly.

Using Powershell though also tried Google Cloud SDK Shell
Windows 10 Version 1703 Build 15063.413
Google Cloud SDK 159.0
Have gcloud beta 2017.03.24 installed as well
Using Flank-1.3.0
java version 1.8.0_131

Running command as follows:
java -jar Flank-1.3.0.jar app\build\outputs\apk\app-debug.apk app\build\outputs\apk\app-debug-androidTest.apk com.amazonaws.devicefarm.android.referenceapp
Am greeted with this error message.  Note gcloud command is working fine for me when used outside of Flank.

java.io.IOException: Cannot run program "gcloud": CreateProcess error=2, The system cannot find the file specified
at java.lang.ProcessBuilder.start(Unknown Source)
at com.walmart.otto.tools.ProcessBuilder.start(ProcessBuilder.java:24)
at com.walmart.otto.tools.ProcessExecutor.executeCommand(ProcessExecutor.java:31)
at com.walmart.otto.tools.Tool.executeCommand(Tool.java:69)
at com.walmart.otto.tools.GcloudTool.getProjectName(GcloudTool.java:118)
at com.walmart.otto.Flank.getProjectName(Flank.java:153)
at com.walmart.otto.Flank.start(Flank.java:40)
at com.walmart.otto.Flank.main(Flank.java:73)
Caused by: java.io.IOException: CreateProcess error=2, The system cannot find the file specified
at java.lang.ProcessImpl.create(Native Method)
at java.lang.ProcessImpl.(Unknown Source)
at java.lang.ProcessImpl.start(Unknown Source)
... 8 more

The problem is that the gcloud binary can't be found. To resolve it create a config.properties where you set the paths to the binaries by:
gcloud-path:/path/to/gcloud
gsutil-path:/path/to/gsutil
Please see the README for more information.

Still not working:
gcloud-path="C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin"
gsutil-path="C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin"
Right now it's failing on finding gsutil.  I've tried the following variations with no success:
C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin
"C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin"
"C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin\\gsutil.cmd"
C:\\Users\\Jay\\AppData\\Local\\Google\\Cloud SDK\\google-cloud-sdk\\bin\\gsutil.cmd
"/Users/Jay/AppData/Local/Google/Cloud SDK/google-cloud-sdk/bin"
/Users/Jay/AppData/Local/Google/Cloud SDK/google-cloud-sdk/bin
"/Users/Jay/AppData/Local/Google/Cloud SDK/google-cloud-sdk/bin/gsutil.cmd"
/Users/Jay/AppData/Local/Google/Cloud SDK/google-cloud-sdk/bin/gsutil.cmd
That location was determined via using where.exe gsutil and verified that the *.cmd file is there.

java.lang.IllegalArgumentException: Gsutil binary not found at provided path: C:/Users/Jay/AppData/Local/Google/CloudSDK/google-cloud-sdk/bin"
at com.walmart.otto.configurator.Configurator.setGsutil(Configurator.java:97)
at com.walmart.otto.configurator.ConfigReader.setProperty(ConfigReader.java:83)
at com.walmart.otto.configurator.ConfigReader.setProperties(ConfigReader.java:23)
at com.walmart.otto.configurator.ConfigReader.(ConfigReader.java:14)
at com.walmart.otto.Flank.start(Flank.java:36)
at com.walmart.otto.Flank.main(Flank.java:73)

Hi, thanks for opening an issue on the Flank repo. The Kotlin rewrite is now available for testing and feedback. This replaces the Java version of Flank.

https://github.com/TestArmada/flank/releases/tag/flank_snapshot

Join us in the#flank channel on the Firebase slack to chat. If this is still an issue in the new release, comment and let me know.

https://firebase.community/


Remove Hashtag-Section with Trash-Button on the Save-Screen

Current behavior:

I write a save-message with a #hashtag.
The editor reads the message and adds the hashtag-section with that tag. Which technically translates to the key-value pair of hashtags="#hashtag".
(⦿)But I dont want to add this hashtags-key, so I click the trash-button on the hashtag-section.
However, the editor does not allow removing the section as long as the save-message includes the given wording. So the trash-button does nothing.

In order to remove the section, I first have to …

rewrite the save-message first
then click the trash-icon – the editor does also not remove it automatically (which I agree with)

Suggested improvement:
At step (⦿), when I click the trash icon the section should be deleted, regardless of what the message says.
The section should only be re-added once I re-write the save-message. And after I finished rewriting, if there still is a hash-tag, the editor will re-add the section (which in some cases I might re-delete again).
UseCase:
Its a good thing that iD saves the last save-message. However, after saving something with a hashtag, my flow is:

mouse based: click save, move to the side-panel, remove the section, click in the save-message
keyboard based: type the save-message
mouse based: click save

With the current behavior I need to switch more between mouse and keyboard, which is not ideal (https://en.wikipedia.org/wiki/GOMS).

But I dont want to add this hashtags-key, so I click the trash-button on the hashtag-section.

Why?


Problem with background in CosmologyCalculator

Hello,
I am having some issues using CosmologyCalculator. I tried to add a background dictionary, but it only appears to work up to redshift ~ 1.5, above that, I get an error message:
CCLError: Error CCL_ERROR_SPLINE: ccl_background.c: ccl_cosmology_distances_from_input(): Error creating  chi(a) spline
I would really appreciate it if anyone could help with this!
Thanks,
Pablo

@Pablo-Lemos can you send a code snippet?

import numpy as np
import camb
import pyccl as ccl

if __name__=="__main__":
    # Set up k and z arrays
    zmax = 2.0
    kmax = 10.
    z_arr = np.linspace(0,zmax, 50)
    k_arr = np.logspace(-4,np.log10(kmax),5000)

    # Cosmological parameters
    H0=67.5
    ombh2=0.022
    omch2=0.122
    ns=0.965
    As = 2e-9

    # Derived parameters
    h = H0/100
    omegab = ombh2/h**2.
    omegac = omch2/h**2.
    omegam = omegab + omegac

    # Set up CAMB
    pars = camb.CAMBparams()
    pars.set_cosmology(H0=H0, ombh2=ombh2, omch2=omch2)
    pars.InitPower.set_params(ns=ns, As = As)
    pars.set_matter_power(redshifts=z_arr, kmax=kmax)

    # Get CAMB power spectrum interpolator
    pars.NonLinear = camb.model.NonLinear_none
    results = camb.get_results(pars)
    PK = camb.get_matter_power_interpolator(pars, nonlinear=True,
        hubble_units=False,k_hunit=False, kmax=kmax, zmax=zmax)

    # Get Pk, chi and H(z) from CAMB
    pk_arr = PK.P(z_arr, k_arr)
    chi_arr = results.angular_diameter_distance(z_arr)
    h_over_h0 = results.hubble_parameter(z_arr)/H0

    # Calculate an array for the scale factor. Then invert the orders of the
    # arrays as required by CCL.
    a_arr = 1. / (1. + z_arr)
    a_arr = np.sort(a_arr)
    pk_arr = np.flip(pk_arr, axis=0)
    chi_arr = np.flip(chi_arr)
    h_over_h0 = np.flip(h_over_h0)

    # Set up a CCL CosmologyCalculator
    cosmo = ccl.CosmologyCalculator(
        Omega_c=omegac, Omega_b=omegab, h=h,
        sigma8=results.get_sigma8_0(), n_s=ns,
        pk_nonlin={'a': a_arr,
                'k': k_arr,
                'delta_matter:delta_matter': pk_arr},
        background={'a': a_arr,
                'chi': chi_arr,
                'h_over_h0': h_over_h0},
    )

Thanks for the response @damonge I will have a look at the notebook. In what I just sent, if I change zmax to 1.5 or below it does work

Oh, I think I see what's happening: chi is not the angular diameter distance, but the comoving radial distance.
Not an expert on camb, but my guess is that you should use comoving_radial_distance instead of angular_diameter_distance.

(the problem, by the way, is that we store both a chi(a) and an a(chi) spline, and you can only have both if the relation between a and chi is bijective, which it isn't for the angular diameter distance)

Ah, of course, that makes sense, thanks!


linter: add markdown-link-check for dead link in doc

Signed-off-by: Allen Sun<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Dead links in document will bring lots of inconvenience for users.
Then we should add a rule for validating links in markdown files.
This pr adds markdown-link-check in circleci integration config. And pouchlinter has already add markdown-link-check in pr https://github.com/pouchcontainer/pouchlinter/pull/4.
Ⅱ. Does this pull request fix one issue?

none
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
no need
Ⅳ. Describe how to verify it
the result shows some dead links in this repo.
Ⅴ. Special notes for reviews
none

lgtm

Codecov Report

Merging #229 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #229   +/-   ##
======================================
  Coverage    72.4%   72.4%
======================================
  Files          29      29
  Lines        1435    1435
======================================
  Hits         1039    1039
  Misses        365     365
  Partials       31      31

Continue to review full report at Codecov.

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

LGTM


cleanups following removal of metadata file

This is follow-up to https://github.com/neondatabase/neon/pull/6769 once it has been rolled out and we won't roll it back.

make remote_storage non optional
remove the migration code that removed the metadata file

spot-check prod logs of deployment first & maybe check the filesystem?

Starting the process of making remote storage mandatory here: https://github.com/neondatabase/neon/pull/7722
We can require it in the config, and then proceed to weed out all the places where we have a spurious Option<> that's always Some.


Fix incorrect json tag for groupId

The proper key in the json is lower case, this was probably a typo or copy/paste error

@themantissa happy to tag someone from my team but this is just one liner that should probably be addressed shortly


Modify client.capture_exceptions to return both decorator and context manager

Client.capture_exceptions is useful, but sometimes it may be handy to have a context manager. So I decided not to add new methods, but extend capture_exceptions functionality using contextlib.contextmanager, which acts as both a decorator and a context manager

Thanks!

@HPotter Why not allow contextlib for Python 2.5+ too? AFAIK it's also a decorator as well as a contextmanager...

Nevermind, I got it... Would still be great to avoid the dependency on a library with no update since 2012, though...

@feldi88 are you aware of issues in contextlib? otherwise if it works without any problems, it doesnt really need updated. Not everything is an on-going iterative issue, many things are solvable and then can be left where they're at.


Druids quest refactoring

[ ] Remove the Treant invasion from Spirits of Ashenvale
[ ] Change the objective for Thunderbluff destruction to a more general reward about contrlling most of Kalimdor (protecting the land from any invaders)
[ ] Change the objective for Stonemaul to something more general about growing trees and getting access to siege ancients

I'll take the first crack at this.