Block submission failed! Scrypt fork litecoin legacy

When the pool found a block  says! submission failed with : params must be an array!
Its a fork of litecoin SCRYPT Legady Daemon
I am using Ubuntu 16.04

What's the Github repo of the coin?

I use git clone https://github.com/coinfoundry/miningcore

The coin not have git repo its a new coin from Mexico

Without inspecting the source it's impossible to tell what's going wrong.

I debug the miningcore the problem its in BitcoinJobManagerBase , method SubmitBlockAsync ,
var results = await daemon.ExecuteBatchAnyAsync(logger,
hasSubmitBlockMethod
? new DaemonCmd(BitcoinCommands.SubmitBlock, new string[] { blockHex })
: new DaemonCmd(BitcoinCommands.GetBlockTemplate, new { mode = "submit", data = blockHex }),
new DaemonCmd(BitcoinCommands.GetBlock, new[] { share.BlockHash }));
I change this for string and the problem change, say method submit not found I think because its a legacy daemon Then use new DaemonCmd(BitcoinCommands.SubmitBlock, new string[] { blockHex })
I think with the method getBlockTemplaete works fine!
Then use only new DaemonCmd(BitcoinCommands.GetBlockTemplate, new { mode = "submit", data = blockHex })
but say params must be an array
How convert this values in array?, its a object unknown type


WIP ARROW-3204: [R] Release to CRAN

With these changes, we've got 0.14.0 passing the automated checks and we're pending human review. There will probably be further changes requested, which we'll collect here.

Codecov Report

Merging #4908 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4908      +/-   ##
==========================================
- Coverage   87.44%   87.31%   -0.14%
==========================================
  Files         995      894     -101
  Lines      140479   134085    -6394
  Branches     1418        0    -1418
==========================================
- Hits       122844   117072    -5772
+ Misses      17273    16661     -612
+ Partials      362      352      -10

Impacted Files
Coverage Δ

cpp/src/arrow/json/converter.cc
90.53% <0%> (-1.78%)
:arrow_down:

cpp/src/arrow/json/chunked-builder.cc
79.91% <0%> (-1.68%)
:arrow_down:

python/pyarrow/tests/test_orc.py
95.45% <0%> (-1.52%)
:arrow_down:

python/pyarrow/pandas-shim.pxi
65.26% <0%> (ø)
:arrow_up:

js/src/util/fn.ts

js/src/builder/index.ts

js/src/enum.ts

js/src/Arrow.node.ts

js/src/schema.ts

js/src/visitor/vectorloader.ts

... and 96 more

Continue to review full report at Codecov.

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

Current status: barring other feedback, my plan is to check out the 0.14.1 release branch locally, cherry-pick these commits onto it, and build the revised CRAN submission from that. I could just resubmit 0.14.0, but since 0.14.1 has been released already, it probably makes sense to go with that in order to pick up the Parquet timestamp fix in the C++ libraries.
Releasing 0.14.1 depends on https://github.com/rwinlib/arrow/pull/2 being merged (cc @jeroen). I think https://github.com/autobrew/homebrew-core/pull/1 would be nice too but is not essential.
Once CRAN has accepted the package, we can merge this patch, but we probably should not merge until then in case there is more feedback.

0.14.1 submission was rejected with some (trivial) feedback from the CRAN humans. That was addressed in https://github.com/apache/arrow/pull/4908/commits/41ca81d2fec84fce1c604880c472917227bb9ed2 and 0.14.1 has been resubmitted, waiting on the next human review.

Latest resubmission was accepted: https://cran.r-project.org/package=arrow
This is ready to merge now cc @wesm @romainfrancois

👍 🎉

This is super nice!


Refactor operator-guide

Hopefully it should make it easier to follow along.
Also added information on how to setup Service Connection for a new tenant

I will merge this now since I will start updates to this files in other PR. There is probably many more improvements to this PR but let's work on it in the future. All we know is that it's better then the current docs :)


Fix id field during instantiation of Table derived struct based on proc attribute

We need to add #[primary_key] or something similar over the field like this:
#[derive(Table)]
#[table_name = "categories"]
pub struct Category {
    #[primary_key]
    id: usize,
    name: String,
}

So when instantiating it, it should be enough to do:
let table_row: User = User {
        name: "mjovanc".to_string(),
    };

This is important since when doing INSERT operations when having autoincrement on the id field and not knowing the current id beforehand, and we definitely do not want to query to get that information before...

Fixed by adding AutoIncrementPrimaryKey and also adding PrimaryKey as a type for non-incremental columns if desired.


Add RoutingNumbers.info

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[X] Your submissions are formatted according to the guidelines in the contributing guide.
[X] Your changes are made in the README file, not the auto-generated JSON.
[X] Your additions are ordered alphabetically.
[X] Your submission has a useful description.
[X] Each table column should be padded with one space on either side.
[X] You have searched the repository for any relevant issues or PRs.
[X] Any category you are creating has the minimum requirement of 3 items.

@davemachado I'm happy to rebase this PR if #500 is required first to fix the build

Hi @ecwyne , thanks for the (2) PR's 😉
No worries about rebasing, I already merged in #500 so as long as you can remove "API ..." from the Description, we'll be good to go! (everything here is an API, so no need to specify)

Thanks @davemachado
Should be good to go now!

Thanks!


url bar - only partly vissible

When I open browser tab the url bar is only partly vissible and thus hard to use:

System:
Windows 7 64 bit
Atom 1.8.0

I got this bug too.
OS X 10.11.6
Atom 1.8.0
browser-plus 0.0.67

I don't have the os. Can you open the developer tool for atom and check what css need to be applied to extend. Currently it is 2 part the one on left of the url input is float left and then one to right is just a block div occuping the entire space. which I think it does. But I put the url overflow to be hidden. can you please around it and send the css patch that can be applied. Appreciate your help.

@skandasoft  feel free to assign it to me, I´ll give it a try

Please make a PR

I already reinstall Atom(1.8) & browser-plus(0.074) on OS X 10.11.6
and, I finished add patch(#94 jennuine’s)
But I can’t input url, because the input area is covered by the autocomplete’s view.
Autocomplete not working in my browser-plus???
I checked my problem on developer tool.
I got same error in console.
It’s autocomplete’s bug??? or my mistake???

error message in console.
=======================================================================
Uncaught (in promise) DOMException: Failed to execute 'put' on 'IDBObjectStore': An object could not be cloned.(…)dbPromise.then.db @ /Users/hiroyuki/Downloads/Atom.app/Contents/Resources/app.asar/src/state-store.js:33
/Users/hiroyuki/Downloads/Atom.app/Contents/Resources/app.asar/src/state-store.js:33 Uncaught (in promise) DOMException: Failed to execute 'put' on 'IDBObjectStore': An object could not be cloned.(…)dbPromise.then.db @ /Users/hiroyuki/Downloads/Atom.app/Contents/Resources/app.asar/src/state-store.js:33
=======================================================================

#94 jennuine’s patch

cd ~/.atom/packages/browser-plus
npm install jquery-autocomplete --save
.atom/packages/browser-plus/lib/browser-plus-view.coffee, update
#jQACPath = "#{atom.packages.getPackageDirPaths()[0]}/browser-plus/node_modules/jquery-ui/autocomplete.js"
jQACPath = "#{atom.packages.getPackageDirPaths()[0]}/browser-plus/node_modules/jquery-autocomplete/jquery.autocomplete.js"
atom/packages/browser-plus/node_modules/jquery-autocomplete/jquery.autocomplete.js,
add jQuery = require('../jquery/dist/jquery.js');
I just want to write url on url-bar, so I modified css for I kill autocomplete.
xdsoft_autocomplete_hint with:1px
Autocomplete not working, but now I can write to the url-bar.
I'm very happy now.
Browser-plus is good plugin, I love it.
thanks.

probably I am going to make autocomplete optional..lot of folks are having issues with it.

Check the latest patch/FAQ


Polars integration

Current dependencies on/for this PR:

main

PR #696

PR #645

PR #604

PR #687

PR #688

PR #689

PR #690

PR #512
PR #692


npm install ionic issue?

While running npm install ionic i am getting the follwing issue,please help

<EMAIL_ADDRESS>install D:\vedapp\node_modules\ionic\node_modules\node-sass
node scripts/install.js

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_
odules\npm\bin\npm-cli.js" "install" "ionic"
npm ERR! node v4.2.6
npm ERR! npm  v2.14.12
npm ERR! file C:\Program Files\Java\jdk1.7.0_06\lib
npm ERR! path C:\Program Files\Java\jdk1.7.0_06\lib
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn C:\Program Files\Java\jdk1.7.0_06\lib
npm ERR<EMAIL_ADDRESS>install: node scripts/install.js
npm ERR! spawn C:\Program Files\Java\jdk1.7.0_06\lib ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node scripts/install.js'.
npm ERR! This is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/install.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     D:\vedapp\npm-debug.log
D:\vedapp>npm install ionic
D:\vedapp>npm install ionic
D:\vedapp>npm install
'CALL "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm
in\npm-cli.js" prefix -g' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! install Couldn't read dependencies
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_
odules\npm\bin\npm-cli.js" "install"
npm ERR! node v4.2.6
npm ERR! npm  v2.14.12
npm ERR! path D:\vedapp\package.json
npm ERR! code ENOPACKAGEJSON
npm ERR! errno -4058
npm ERR! syscall open
npm ERR! package.json ENOENT: no such file or directory, open 'D:\vedapp\package.json
npm ERR! package.json This is most likely not a problem with npm itself.
npm ERR! package.json npm can't find a package.json file in your current directory.
npm ERR! Please include the following file with any support request:
npm ERR!     D:\vedapp\npm-debug.log

Hello, thanks for opening an issue with us!  Are you still having this issue?

@ved54321 I solve this using a old version of node. Try use nvm to do this.
nvm install v4.0.0

@brunowego Have you gotten a chance to install it with a newer version of node? Like 5.8.0?

@jgw96 I have tested with v5.4.1 and not works for me.

@brunowego im so sorry it has taken me this long to get back to you. Does this still happen with the latest version of the ionic cli? Thanks!

@jgw96 v4.4 works 👍

@brunowego awesome! Good to hear! Thanks for using ionic!


⚠ Bofrost.gr has degraded performance

In 2352faf, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2309 ms

Resolved: Bofrost.gr performance has improved in 7652374 after 9 minutes.


deepin.deepin-turbo: init at 0.0.2

Motivation for this change
Add deepin-turbo (a daemon that helps to launch applications faster).
Tracking deepin packaging: https://github.com/NixOS/nixpkgs/issues/59023
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build deepin.deepin-turbo


How to change background color

I don't want to use a background image so i commented out the background in the config file but now I get a pink background which is even worse (my point of view). How do I change the background color?

in _sass/variables.scss set $bodycolor to the color you want

https://github.com/TaylanTatli/Moon/issues/49


TypeError: Cannot read property 'forEach' of undefined if object destructing with rest props is used

Describe the bug
Compilation fails with TypeError: Cannot read property 'forEach' of undefined if object destructing with rest props is used in npx created app.
Works with yarn though.
Versions of react-scripts affected: 3.1.0, 3.1.1 (with different error messages), 3.1.2
Most recent react-scripts version that works: 3.0.1
Did you try recovering your dependencies?

npm --version: 6.11.3
Which terms did you search for in User Guide?

TypeError: Cannot read property 'forEach' of undefined
Environment

$ npx create-react-app --info

Environment Info:

  System:
    OS: macOS 10.14
    CPU: (8) x64 Intel(R) Core(TM) i7-3615QM CPU @ 2.30GHz
  Binaries:
    Node: 12.11.0 - ~/.nvm/versions/node/v12.11.0/bin/node
    Yarn: Not Found
    npm: 6.11.3 - ~/.nvm/versions/node/v12.11.0/bin/npm
  Browsers:
    Chrome: 77.0.3865.90
    Firefox: 67.0
    Safari: 12.0
  npmPackages:
    react: ^16.10.1 => 16.10.1
    react-dom: ^16.10.1 => 16.10.1
    react-scripts: 3.1.2 => 3.1.2
  npmGlobalPackages:
    create-react-app: Not Found

(paste the output of the command here.)
Steps to reproduce

(Write your steps here:)

Create new app using npx: $ npx create-react-app error-test. Make sure yarn is not installed.
Go to error-test/src/App.js and add following code to the end of the file:

const testCase = {
  restObjectPropsFails: ({ id, ...item }) => {
    return null;
  }
}

Run $ npm start from error-test directory

Expected behavior

Application is compiled correctly
Actual behavior

Compilation failed with message:
Failed to compile.

./src/App.js
TypeError: Cannot read property 'forEach' of undefined

Reproducible demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

+1

Ran into the same issue and spent half a day debugging. Saw this issue and switched to a prior version and it seems to work like a charm.

Downgrading React Scripts to 3.0.1 worked for me.

@iMerica downgrading also worked for me

Downgrading React Scripts to 3.0.1 worked for me.

i was working on a project for one week, with react-scripts version  3.0.1 with no errors, and today when i start my project i got this related error, what is the explanation for this? react-scripts connect to a external server when i run yarn start ???? WTF

you must have done yarn install which would have upgraded based on your package.json settings.

No, in all commits, version of react Scripts  never change, i Just checkout a old commits to check

3.1.1 is working fine for me while 3.1.2 doesn't.

+1

+1 I also had to downgrade to 3.0.1.
3.1.2 & 3.1.1 (I didn't try 3.1.0) both failed....

I had the same issue, also downgraded to 3.0.1

downgrading to version 3.0.1 seems to solve the problem!

I had the same problem but found that reverting eslint to 6.4.0 from the simultaneous update to 6.5.0 solved the problem. Tried to npm i<EMAIL_ADDRESS>again and got same compile error.
So working with:
"eslint": "^6.4.0",
"react": "^16.10.1",
"react-dom": "^16.10.1",
"react-router-dom": "^5.1.1",
"react-scripts": "^3.1.2",

ESLint bug: https://github.com/eslint/eslint/issues/12335

+1 getting same issue

+1 getting same issue

+1 same issue

+1 Same issue

+1 Same issue

Had the same issue.
Resolved by:

downgraded react-scripts to 3.0.1
removed eslint from devDependencies
deleted node_modules and run "npm install"

+1 Same issue, downgrading react-scripts to 3.0.1 mitigates the problem.

In my case, After downgrading to 3.0.1, I was getting Error: Cannot find module<EMAIL_ADDRESS>So I uninstalled @csstools/normalize.css and reinstalled @csstools/normalize.css and it worked.

My solution is disable eslint of react-script by customize-cra react-app-rewired. Detail in stackoverflow

This PR #7754 should fix this error

yarn works because the lock file
after rm -r node_modules yarn.lock; yarn install, the error can be reproduced.

The issue is definitely eslint-loader. Thanks @alexandrtovmach
Until that merge gets included, adding the following to package.json worked for me
"resolutions": {
    "eslint-loader": "3.0.2"
  }

"eslint": "^6.4.0",
"react": "^16.10.1",
"react-dom": "^16.10.1",
"react-router-dom": "^5.1.1",

Worked for me on my ejected project. Thanks @oyof8376

locking eslint at 6.4.0 in devDependencies fixes this on my end.  6.5.0 fails

This seems similar to https://github.com/facebook/create-react-app/issues/7566 where yarn users had to override babel-eslint to 10.0.3 .
Can anyone think of a way we can test for this problems? Maybe have the linter portion run in the builds for create-react-app to ensure downstream consumers don't get broken.

Ugh. Everything was working on Friday, then I upgraded to react-scripts 3.1.2 this morning, and was able to work on the dev server just fine for a few hours. Then I tried to do a build of my project, where I first deleted and re-installed all my dependencies, then did the standard npm run build. At that point, I started getting these kinds of errors:
TypeError: Cannot read property 'forEach' of undefined
So I reverted back to my package.json file from this past Friday, blew away my node_modules folder, reinstalled all my dependencies, and I'm still getting the same error. Yuck!
FWIW, I don't explicitly have eslint as one of my dependencies. I'm not sure what to make of that, but I'm dead in the water at the moment.

So yeah, now I've rolled all the way back to react-script 3.0.1, and things are up and running again. But I have no idea why things ever worked with any version beyond 3.0.1. is there some sort of global install that's happening under the hood here?

Yes, it is a bug with eslint-loader, but it is a design flaw in CRA to give a linter enough importance that it can break core development functionality.

@iMerica Ah, I see that above now. Is this because create-react-app 3.1.2 does this:
"eslint": "^6.1.0",
...and so it's grabbing whatever the newest version of eslint is, which has a bug in it?
FWIW, I've removed all of those ^ and other commands in my package.json file, almost entirely to prevent rogue updates that might break something. My routine is that first thing every day, I run npm outdated to see what's old, and then I update that stuff manually.

I have the same issue and I just spent the whole day debugging it.
downgrading to react-scripts: 3.0.1 worked for me.

We're having this same issue, same error message, and it's breaking the whole build 😲!
Tried downgrading to react-scripts: 3.0.1 but that's not quite working yet...

Do npm users have a way to control dependency resolutions yet? As mentioned earlier in this thread, there is a working solution for yarn users: https://github.com/facebook/create-react-app/issues/7753#issuecomment-536560532

node 10.6.3 react 16.9.0 react-scripts 3.0.1 eslint 6.1.0 babel/core 7.6.2
Is working for me using npm with no resolutions.

Do npm users have a way to control dependency resolutions yet? As mentioned earlier in this thread, there is a working solution for yarn users: #7753 (comment) without having to completely downgrade react-scripts

I am also using npm. I resolved quickly it by deleting package.json, removing node modules directory and package-lock.json and installing back again.

Ouch!  I lost a whole day to this, having to methodically run npm outdated and upgrade packages one by one to find the culprit.  I can confirm, however, that updating package.json to 3.0.1 and rolling back allowed my application to compile and nixed the forEach issue.  My application is based on npm and create-react-app.

Going to lock this for now to prevent email spam to those subscribed. If anyone could spare some time to try to sort out the CI error in the Linux runs over at https://github.com/facebook/create-react-app/pull/7754 that would be greatly appreciated. We'll post back here when we have a fix ready to go out.

We've got a fix in and should be releasing in the next few hours hopefully. Thanks for your patience!


Duplicate Roles allowed

It is allowed to create two roles with the same name. Is it a bug or are we supposed to look for existing roles with the specified name before creating a new role?
From the docs: "name: The name for the role. This value is required, must be unique" it seems to me that it shouldn't be possible.
BTW Also would be nice, if we could just add users to existing role without querying it, by its unique name :)

I just tried to add a role with an existing name (from the Dashboard), and it didn't allow the creation, giving me the following message:
A duplicate value for a field with unique values was provided.
Can you please fill the issue template correctly (providing as much information as possible)? In case it's really a bug, we will reopen.

Ok, sorry.
Issue Description
I can create two roles with the same name while it should not be possible.
Steps to reproduce

create a role with name "myRole"

then call this function:
Parse.Cloud.define("followProfile", function (request, response) {

//now I will create the role again and add a user to id, I was just trying to add the user to previously created role without querying it

var roleACL = new Parse.ACL();
roleACL.setPublicReadAccess(false);
roleACL.setPublicWriteAccess(false);
var myRole = new Parse.Role("myRole", roleACL);
myRole.getUsers().add(request.user);
myRole.save(null, { useMasterKey: true }).then(
    function (saved) {
        response.success();
    }, function (error) {
        response.error();
    });
});

..I know that this function is wrong, I was just trying to find a way how to add user to role without querying the role. No matter how incorrect the code is, still it creates a duplicate role, which I guess should not happen.
Expected Results
Throw exception that I can't create duplicate role
Actual Outcome
Second role with the same name is created
Environment Setup

Server

parse-server version 2.3.5
Windows 10
Localhost

Database

MongoDB version: 3.2.12
mLab

Well, I have tested and I can do it even in Dashboard, so the code is not relevant. Will try to update to latest parse server.

Also happening on parse server 2.3.6, I can do it in dashboard without any error.

@rihadavid thank you for providing the information.
Ok, can you please do the following things:

Query for _Role where the name is myRole (master key needed) and provide the results;
Provide the server logs of the operation you said, when running with VERBOSE=1
If possible, can you also record the dashboard showing different Role rows with the same name?

Ok, here it is:

One last question:
Was this a migrated app from Parse.com, or you started it from parse-server? I think I know what's going on.

The app was originally developed for parse.com, but the running parse-server is clean installation. We are using cloud code that we developed for parse.com and changed it to comply with parse server syntax. We did not migrate any data.

So I believe that Parse.com created a unique index for name on _Role (I just checked the database from one of my migrated apps) and Parse Server doesn't create this index. We'll fix it.
Thanks a lot for reporting and providing all the data, @rihadavid.

@rihadavid can you please update to 2.3.7 and see if you still have this error?

Yes, it seems fixed, when I try to add second Role with the same name in Dashboard, it shows me an error and the row is not created. Thanks!

Great! Thanks for providing a good and actionable bug report!


Accessibility in Warning Callout label - Failing Axe checks due to invalid aria role

Bug Report

What is the issue?
The Warning Callout label has an invalid aria role in the label, causing an Axe failure.

What steps are required to reproduce the issue?
Running Axe over the following HTML:

    Important: Data Quality

Produces the following error:
    Expected the HTML found at $('span[role="text"]') to have no violations:

    Important: Data Quality

    Received:

    "ARIA roles used must conform to valid values (aria-roles)"

    Fix all of the following:
      Role must be one of the valid ARIA roles: text

    You can find more information on this issue here:
    https://dequeuniversity.com/rules/axe/3.5/aria-roles?application=axeAPI

The text role does not exist in the W3C aria spec: https://www.w3.org/TR/html-aria/
I'd imagine the fix would be to simply remove the offending  element, and would look like the following:

    Important: Data Quality

What was the environment where this issue occurred?

Device:  N/A (Running in a CI environment)
Operating System: N/A
Browser: N/A
Browser version: N/A
NHS.UK frontend package version: 4.1.0

The nhsuk-react-components library is being used here, but the issue exists in both packages, in the Nunjucks template and in the service manual website.
Is there anything else you think would be useful in recreating the issue?

I imagine it'll be something to do with this:
https://dequeuniversity.com/rules/axe/4.3/aria-text

When a text node is split by markup (e.g. Hello World) VoiceOver will treat it as two separate phrases instead of just one. Adding role="text" around the elements solves the problem.

Good call @andymantell, I'll try and dig into this a bit

I've tested this and it is indeed still an issue on VoiceOver for iOS. My opinion would be that we leave it in to prioritise usability over issues flagged by automated testing. @mcheung-nhs could I get your take?

If it helps at all, @Tomdango and myself are no longer seeing this issue after updating the version of aXe we're using for automated testing.

If it helps at all, @Tomdango and myself are no longer seeing this issue after updating the version of aXe we're using for automated testing.

Brill, thanks for letting me know. For future, do you know what versions you've moved from and to?

We're using the jest-axe wrapper, and we've moved from 3.4.0 to 6.0.0, in terms of axe-core, it looks to be a change from 3.5.? to 4.4.1


Integrate with smartcontract, Initial setup

Initial web3 wallet,  connect to smartcontract and implement ABI json
Make & add waviii wallet logo
Add countUp react package
Handle wrong Ethereum Network
User window alert when user has no MetaMask

Integrated under and prior to: a2e205e9557cb7ce1afe8ecbb0ac036d867315cf .


fix(hbase): copy core + hdfs-site from client conf

Which issue(s) this PR fixes:

Fixes #423
Additional comments:

HBase master installation won't depend on HDFS NameNode's prior installation anymore.
Will fetch core-site.xml and hdfs-site.xml files from {{ hadoop_client_conf_dir }} instead of plain text '/etc/hadoop/conf.nn'.

To make clear that you license your contribution under
the Apache License Version 2.0, January 2004 and that you give permission to TOSIT,
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[x] I hereby agree to grant TOSIT a copyright license to use my contributions.

{{ hadoop_client_conf_dir }}/core-site.xml is rendered by hadoop_client_config so you need to update the tdp-lib DAG. hbase_master_config must depends on hadoop_client_config.

Hi @jusstol, thank you for you contribution to TDP.
This PR has been validated and will soon be merged to master branch.


Store buffers in connection object?

How open are you to the idea to store either a buffer or arbitrary user-data in the Connection object?
The motivation is that I could store my format buffer for SQL strings inside the connection object and reuse that buffer for every new query (I would set it to assumeSafeAppend() so that it would reuse the memory).
If you like the idea this could even be expanded so that you could directly write
conn.query("SELECT a, b ,c FROM example WHERE d = %s", my_d_variable) and it would use the internal buffer to format that string.

As far as adding decidated fields to types for "user can store their own misc data here", I'm not a fan of that. It breaks encapsulation in particularly messy and dangerous ways. Plus, it doesn't enable anything that can't be accomplished much more cleanly/safely via composition (and D features like templates and alias this help make the composition approach particularly easy and painless).
But I am open to specific proprosals for decreasing the lib's reliance on GC, or otherwise improving the lib's memory usage.
However, if you're using format strings to generate SQL, then I strongly recommend using prepared statements instead: They don't require a new string to be generated/allocated (since they're handled purely on the server-side), and far more importantly, they provide far better safety guarantees (ie, virtually eliminates SQL injection attacks without ever needing to remember to do "proper" escaping).

I plan to use prepared statements where I can, but some of my queries use IN or do an INSERT .. ON DUPLICATE UPDATE ... with a long list of rows to update/insert and those won't work with prepared statements.

Also, there is still the issue of having to manage the prepared statements per connection ;)-

Managing prepared statements per connection is #95.
Closing because the main question's been answered.


Indent guides malformed

Looks like you're using a invisible character marker for space that is wider than 1 character. What character are you using, and which font?

My marker is a KATAKANA MIDDLE DOT (・) from the Hack font. It's only 1 character, but it seems that Atom doesn't like it. If I switch to a . then the indent guides line up properly.

My marker is a KATAKANA MIDDLE DOT (・) from the Hack font

That's a double-width character. If you want to use the Katakana (Japanese) mid dot, consider using the HALFWIDTH KATAKANA MIDDLE DOT instead.

Note: I'm not entirely sure about this, but I believe your specific issue might be (at least partially) fixed after #8811 ships. /cc @as-cii.

I tried the halfwidth and it messed up the indention guides in the opposite way. :)
So I switched to the regular bullet and it works fine. But I wonder if it shows something that needs a solution in Atom?
I'll keep an eye out for https://github.com/atom/atom/pull/8811.

Unfortunately I don't think #8811 will actually solve this, as the problem here lies in the space character not occupying the same width as KATAKANA MIDDLE DOT.
So long as the character widths are not the same, indent guides will be misplaced: I feel like this might be considered a bug in the existing implementation, as we assume that those characters have the same width when constructing the empty line guides. A better solution would probably read how large an indent guide is and set the related empty line <span>s' widths accordingly.
@bobrocke: thank you for reporting this issue. Fixing this should be pretty straightforward and I'll take a look at it in the future, but as a temporary workaround you could replace that character with one that has the same width as the space char. I'll definitely ping you as soon as we start working on a fix for this. :+1:

@chrissimpkins Could you check if the character mentioned might have the wrong width set in Hack?

I can confirm that we do not have a Katakana middle dot glyph (U+30FB) in the regular Hack font.  There is a middle dot (U+00B7) that will have the appearance that you are after here.  I can confirm that the U+00B7 glyph has the same fixed width spacing as the rest of the regular set (and all other variant sets).

Any chance that this glyph is coming from a fallback font if you are using the U+30FB character?

Another situation in which this occurs is using a proportionally-spaced font for editing.


Rancher CLI v0.6.0 is not honoring parameters set in docker-compose.yml

Rancher versions:

rancher/server: 1.6.0
rancher/agent: 1.2.2

Docker version: (docker version,docker info preferred)

1.12.6

Operating system and kernel: (cat /etc/os-release, uname -r preferred)
$ cat /etc/os-release
NAME="Amazon Linux AMI"
VERSION="2017.03"
ID="amzn"
ID_LIKE="rhel fedora"
VERSION_ID="2017.03"
PRETTY_NAME="Amazon Linux AMI 2017.03"
ANSI_COLOR="0;33"
CPE_NAME="cpe:/o:amazon:linux:2017.03:ga"
HOME_URL="http://aws.amazon.com/amazon-linux-ami/"

Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
AWS
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA Rancher, External DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Problem
On OS X, Rancher CLI v0.6.0 is not honoring parameters set in docker-compose.yml. It keeps on asking questions already answered as environment variables.
Steps to Reproduce:
Download two binaries of the above versions and test it on OSX (I am on 10.11.2)
These are my yaml files.
docker-compose.yml.txt
rancher-compose.yml.txt
Results:
/usr/local/bin/rancher_v0.6.0 --url https://<rancher-master> --access-key <access_key> --secret-key <secret_key> --env <env> up --force-upgrade --confirm-upgrade -s <name_of_stack> -d
Automatically create registry in Rancher for corresponding ECR repo if it doesn't already exist
Auto Create auto_create[false]:

Same command runs perfectly fine on older version i.e. v0.5.2
/usr/local/bin/rancher_v0.5.2 --url https://<rancher-master> --access-key <access_key> --secret-key <secret_key> --env <env> up --force-upgrade --confirm-upgrade -s <name_of_stack> -d

This was an intentional change in behavior added in this PR: https://github.com/rancher/rancher-compose-executor/pull/37 (specifically the "Ask questions up front" commit).
We had to make this change to accommodate a new templating feature in compose files. We used to be able to ask questions lazily as their values were used. Unfortunately, with the addition of templating, we have to ask the questions up front.
So, at the root of your problem is that the questions in rancher-compose are unused in docker-compose. At least, that's what I' m seeing in the files you provided. Is it possible to drop the questions from the rancher-compose file if they are unused?
I'm not sure that it would impact you directly, but it would be a good enhancement for us to at least look at the env-file for variables that would satisfy questions and omit those qusetions.

Hey @cjellick,
It would be good to have those questions answered thru an env-file or something similar.
Some feedback - I know Rancher CLI is in beta but still it is good to have a deprecated warning before chucking out the functionality :)
Just noticed other problem with Rancher CLI v0.6.0. It keeps on expecting a value, even if is not required (set to false). The only workaround, for now, is to remove all questions.
    - variable: "aws_access_key_id"
      label: "AWS Access Key ID"
      description: "AWS API Access Key to use for obtaining ECR credentials. Not required if using IAM roles."
      required: false
      type: "string"

Thanks for the feedback @vikas027. I noted your observation about not-required questions in the above referenced issue that @joshwget is using to aggregate cli improvements.
And I'll ping @deniseschannon regarding the warning on functionality changes. Our general approach has been to note such things in the release notes. That seems like an appropriate place for now. Maybe this particular one got overlooked.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Improve num_online graphs

This PR makes the graphs reflect reality a bit better.
I think the count was misleadingly including devices which were not currently sending requests (but which had done so recently and were still present in the time-series but with a rate of 0) appear as online.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@ec2cb5d). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #225   +/-   ##
=======================================
  Coverage        ?   31.17%
=======================================
  Files           ?        8
  Lines           ?      526
  Branches        ?        0
=======================================
  Hits            ?      164
  Misses          ?      333
  Partials        ?       29

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Shadowed variable incorrect report - F821 Undefined name

Started getting a ruff failure from 0.0.273 (still present in 0.0.274), which wasn't present in 0.0.272. Believe the source is valid (though ugly) Python, and the ruff failure is incorrect.
Given this file:
# tmp.py
list = [1, 2, 3]

for e in list:
    if e % 2 == 0:
        try:
            pass
        except Exception as e:
            pass
    else:
        print(e)

Old:
$ pip install ruff==0.0.272
...

$ ruff check tmp.py

$ echo $?
0

New:
$ pip install ruff==0.0.273
...

$ ruff check tmp.py
tmp.py:10:15: F821 Undefined name `e`
Found 1 error.

$ echo $?
1

The shadowing/reuse of e is risky, but (afaict) e is unambiguously defined at the line where ruff complains.

Looks like this is also reported in #5249 - feel free to close this as dupe if its tracked there.

Thanks — gonna close as a dupe, but appreciate the clear issue and example.
(I do recommend against shadowing variable names with caught exceptions but agree we shouldn’t flag this.)


Make FeatureIO.save a class method instead

In https://github.com/sentinel-hub/eo-learn/pull/473 the inconsistency of paths was fixed, but it also exposed that the main role of FeatureIO is loading, and saving complicates it greatly.
In this MR I try to resolve the issue further by making the  save method a class method instead. This way the class members themselves does not need to jump through hoops to accommodate saving. FeatureIO may be requested to load and re-load at any time, but saving is done only inside a special function.
Many of the changes also stem from reordering of arguments to keep it consistent.

Codecov Report
Base: 82.90% // Head: 82.90% // No change to project coverage :thumbsup:

Coverage data is based on head (6742ce8) compared to base (b91fd08).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #474   +/-   ##
========================================
  Coverage    82.90%   82.90%
========================================
  Files           60       60
  Lines         5652     5652
========================================
  Hits          4686     4686
  Misses         966      966

Impacted Files
Coverage Δ

core/eolearn/core/eodata_io.py
94.14% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add the nodejs script to setup ceph s3 tests.

Add the setup nodejs script for ceph s3 tests.
Some more info here: https://gist.github.com/dannyzaken/c85f0006e5d6582c8d8666cafce6ab76
Signed-off-by: Amit Prinz Setter<EMAIL_ADDRESS>

esline if failing. I will open a new PR


Fix for Thermionic Fabricator Cast removal

Fixed removing a cast from Thermionic Fabricator will lead to a
ClassCastException exception
Fix for issue #160

Works for me thanks

Cheers.


cigien: Watch blogmeyeucon.com

cigien requests the watch of the watch_keyword blogmeyeucon\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

blogmeyeucon\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

blogmeyeucon\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ

Approved by Spevacus in Charcoal HQ


change to singular for data types

changed
all options => option

@timotheeguerin this was suggested by Allen from last PR! I'm actually changing from options to option though??


can't save new deck

it always says:
could not connect to the server please try later
i'm building my deck all the day and still nothing saved

Thanks for the report. Offhand I wasn't able to reproduce the issue, so it may be something specific to your setup.

What web browser are you using and which version of it?
Can you see errors in the developer console for your browser? (Varies between browsers, for Chrome at least you right click the page, click 'Inspect', which brings up the developer pane. Click the 'Console' tab, you potentially should see errors there)
I saw you were able to enter chat messages into the lobby. Are you able to watch / spectate games on the site?

i use chrome version 58.0.3029.110 (64 bit) i also tried it on my
Firefox and couldn't save my deck either
here is the result:
[image: Inline image 1]
3, yes i can see games, i played a game before accidentally deleted my
deck, now that i'm trying to create a new one it fails saving it

On Fri, Jul 7, 2017 at 8:20 PM, Brian Upton<EMAIL_ADDRESS>wrote:

Thanks for the report. Offhand I wasn't able to reproduce the issue, so it
may be something specific to your setup.

What web browser are you using and which version of it?
Can you see errors in the developer console for your browser?
(Varies between browsers, for Chrome at least you right click the page,
click 'Inspect', which brings up the developer pane. Click the 'Console'
tab, you potentially should see errors there)
I saw you were able to enter chat messages into the lobby. Are you
able to watch / spectate games on the site?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cryogen/throneteki/issues/1129#issuecomment-313742413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcmCUFAf4jY6nJMy84XUdJAZ6LGHv0_hks5sLmjqgaJpZM4ORL08
.

Is this the one trying to save a really big deck?  Yeah that won't work currently, however I am refactoring the deck builder currently and big decks will work ok in that

yeah i just got rid of a lot of characters and it fixed it ty
btw i have a question: when (if we will of course) will we be able to play
melee?
ty again for the help
On Sat, Jul 8, 2017 at 4:08 AM, Stuart Walsh<EMAIL_ADDRESS>wrote:

Is this the one trying to save a really big deck? Yeah that won't work
currently, however I am refactoring the deck builder currently and big
decks will work ok in that
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cryogen/throneteki/issues/1129#issuecomment-313823965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcmCUGi4zTO-MNk-7lSmTOfXHpArctimks5sLtZ3gaJpZM4ORL08
.

The deckbuilder changes have now been deployed so you can use your lots of characters deck now :)
as for melee.. it's possible but not in the 'soon' timeframe


RectangleWaveform is not respecting activeColor and inactiveColor parameters

RectangleWaveform always show white bars regardless of the activeColor and inactiveColor parameters.

Test only in android.

The white bars that you see is the borderWidth(*default to 1) of the rectangle overlapping the rectangle. This happens if you have a high sample rate with not enough width to display them properly in such case borderWidth overlapps the rectangle.
Try reducing the totalSamples that you're generating or reducing the borderWidth.
Let me know if you've any other doubt on this and if not I can go forward and close the issue.

Thanks @rutvik110. It was exactly what was happening.
Thanks for the fast reply.


Wyze Cam v2 v<IP_ADDRESS> No NFS or Telnet on v0.4.04

I really like the idea of storing the files on a NFS mount.  I tried with v 0.4.03 then with v04.04 and both get to the "installation finished" voice and reboot but no NFS mount or telnet access.  I've tested the NFS mount (EXT2, 700GB Avail) from my workstation and it's works from there.  I'm using the default config template with only the NFS line updated.
The install log was created the first time. Which I didn't save :(  Now every install since it hasn't created the install log.
Suggestions?

there is a dev branch that fixes these issues, stay tuned for a new official release from the author soon

@jdkadel I hate to say this but "it works for me".  One trick you could try is to check my WyzeUpdater repo, which has a sample update allowing you to enable telnetd for the camera. That should give you telnet  access to the camera. And then you can look around and check what's wrong with last installation. Without that detail, there is not much I can help.

Soon after trying to install I thought I had bricked the camera.  Back at it today and it's back online.  I tried to install v0.3.03 again and it went through the process but still didn't give access to telnet or the NFS mount.  But! it did create the install log.
install.log
When I attempt to connect with Putty Telnet it times out with Network error: connection refused.  If I try with SSH it asks for name and password?
The NFS mount I'm using is '<IP_ADDRESS>:/mnt/HD_a2/Wyze/' No other changes to the config.inc.

Your "install.log" looks fine to me. Are you sure you are connecting to the right camera with telnet/ssh? sshd is not available on the camera so I don't expect anything asking you username/password when you tried ssh.
I also noticed that your NFS root variable has a trailing slash, not sure if it matters but mine doesn't have one. Maybe you want to remove the trailing slash and try it again?

Basic mistake.  I just used the first IP address displayed.  Telnet works fine if you use the correct IP.  Thanks for the pointer.
The NFS connection is being refused by the host.  The cam and the NFS are on the same network and works on another Linux box.  I'll have see what's up.
Thanks for the help.

The NFS issue ended up being a rights issue on the NFS server share.  Once changed the cam started writing files.


Remove invalid scope modification

This fixes the bug introduced with this scope modification. I mixed up arguments and params, thinking this was a function declaration / expression case where the replacement modifies the scope but obviously params is a call expression where the param replacement does not affect the scope declarations at all.
A test for this would check that f(__webpack_require(4)) does not attempt to alter the scope. But that this was even attempted in the first place was completely invalid behaviour so I don't see any reason to have an explicit test case for what is effectively a completely arbitrary bug case. But happy to put one together if you would still prefer @styfle.

@styfle the invariant is that any FunctionExpression or FunctionDeclaration will have a scope property attached to it, and that the scope property will always have a declarations object attached to it. If we had a suitable typing system for ASTs it would show this. In both of those scenarios one can see the FunctionExpression check that confirms the invariant holds directly.

suitable typing system for ASTs

Maybe we can look into using typescript for this?


Redsky: New Sheet

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Redsky >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: < Solar Studios > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: < Dungeons & Dragons 5th Edition > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
This is the initial character sheet commit for Redsky, a full 5e conversion developed by Solar Studios.

Character Sheet Info Roll20 Internal Use only.


Feature skyplot

Codecov Report

Merging #8 into master will decrease coverage by 4.12%.
The diff coverage is 65.66%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   90.38%   86.26%   -4.13%
==========================================
  Files          33       38       +5
  Lines        1331     1594     +263
==========================================
+ Hits         1203     1375     +172
- Misses        128      219      +91

Impacted Files
Coverage Δ

cosmogrb/universe/universe.py
91.57% <100%> (ø)
:arrow_up:

cosmogrb/utils/plotting/angle.py
53.84% <53.84%> (ø)

cosmogrb/utils/plotting/projections.py
62.24% <62.24%> (ø)

cosmogrb/utils/plotting/skyplot.py
66.66% <66.66%> (ø)

cosmogrb/io/read_universe.py
84% <84%> (ø)

cosmogrb/utils/plotting/itrs_frame_monkeypatch.py
90.9% <90.9%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


[physac] Fix Physac examples to be run without creating new thread

This PR addresses https://github.com/raysan5/raylib/issues/592
To start with, both physics_demo and physics_shatter had a bug because they were never setting needsReset back to false.
Then, as discussed in https://github.com/raysan5/raylib/issues/592, I thought it would be much simpler to avoid physac creating any threads rather than making the library thread-safe. I feel it's much simpler for beginners and newcomers to simply call RunPhysicsStep themselves within their main loop rather than learning concurrency all of the sudden. For this reason, I believe PHYSAC_NO_THREADS should be the default. I would actually rename it to do the opposite: PHYSAC_CREATE_THREAD or similar.
I had to fix refactor PhysicsLoop slightly and created PhysicsRunStep to be able to run PhysicsStep when being called from the user end since it relied on the timing values collected there.
I'd appreciate very much if @victorfisac reviewed this since it's his creation and I'm also modifying his library within this PR. Should I open a different PR if this gets accepted in https://github.com/victorfisac/Physac to keep them in sync or is @raysan5 doing that from time to time?

Merging this PR. @victorfisac could take a look later...
Just notice that avoiding physics to run in a second thread, we loose some resolution on physics because they are only calculated in a per-frame basis.
But I understand we let the user to control any additional thread if required.

Physics needs to be dependent to framerate in order to achieve it's objective: dynamics simulation. From my opinion, trying to calculate physics from main thread is like driving a car with squared wheels.
But, physac library inside raylib can be modified as anybody want in order to fit raylib style (simple, fast, minimalist); if you feel it's better in main thread let's keep in this way.
I have my own repository with official Physac version where I will implement thread safe.

Well, the changes done don't imply you are calling it always within your main loop (which makes physac framerate-dependant). It simply allows you to do it from any thread you want, instead of letting physac creating a new one for you. In other words, there's nothing that prevents you from starting a new thread that calls RunPhysicsStep 60 times (or whatever value you may deem valid) per second.
In the context of the examples since they are all set to run at 60 FPS, I thought it was worth calling RunPhysicsStep from the main loop directly to avoid having to implement thread-safety on physac. I did it after assuming the examples have room enough to run the logic + physics and after seeing that PHYSAC_DESIRED_DELTATIME default value was 1/60.

@victorfisac I agree with your arguments, it's not good to calculate physics in main thread, I noticed it in my previous comment, but I also agree that allowing final user to have control over physics steps is a good option.
C is not intrinsically a multithreaded language, for that reason, multiple threading libraries could be found depending the underlying hardware, letting the users choose their own threading mechanism could be a good option for the library.

@raysan5 @pamarcos Ok! Now I understand. I agree with this :)


Workspace folders divider object

We could already name and manually sort each workspace folder so it reflects nicely on the explorer. I would like to request a divider object where you could give it a name just like a folder, but there is no content and there is no expand arrow button on its left.

As seen on this image, the "---" is intended to be a divider. But the folders configuration doesn't allow an entry with just name so I have to use a dummy path just to get a divider.
A divider maybe omitted from this new terminal dialog and only have it affects the explorer.

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks


Updating JavaScript API Tutorial Docs

Edits to the tutorial docs to add clarity, correct grammar, enhance consistency, and fix broken relative links.

Excellent stuff, thanks @CMcKee714 !


Random Number node always triggers subsequent nodes

When set to fixed, the Random Number node still triggers next nodes even though there hasn't been a change.

Well it should work, what is the error? Maybe I forgot to update it for SDXL pooled output.
However, you just right click any CLIPTextEncode and "Convert text to input"

Yeah the IS_CHANGED method is really weird to work with. Instead of just True it is changed, or False it isn't, it expects a hash from some data.

Experiencing this problem right now, it's retriggering a huge amount of downstream nodes.
Any way to circumvent the problem?

I'm also running into this issue. I'm trying to incorporate the Random Number node into a workflow that generates a random crop. I'm noticing it triggers a render even if I'm just moving a node around and nothing else has changed.
https://github.com/user-attachments/assets/c124cccb-ec9d-466a-94dc-c1518bf8558e

In WAS_Node_Suite.py, I commented out lines 11942-11946, and it seems to do the trick for the time being:
#    @classmethod
#    def IS_CHANGED(cls, seed, **kwargs):
#        m = hashlib.sha256()
#        m.update(seed)
#        return m.digest().hex()

In WAS_Node_Suite.py, under the WAS_Random_Number class, I commented out these lines, which seems to fix my situation for the time being:
#    @classmethod
#    def IS_CHANGED(cls, seed, **kwargs):
#        m = hashlib.sha256()
#        m.update(seed)
#        return m.digest().hex()

Has there been a change to on change method? Hex for same seed should be the same, this match cached value.

Has there been a change to on change method? Hex for same seed should be the same, thus match cached value.

I have no idea. I tried the WAS Seed node, and it works without issue. When I put it in fixed mode it doesn't render if there's no change.
Here's my version info:
ComfyUI: 23402dc84d
Manager: V2.46

I'll have to look into it. The reason I have that in there is because I
don't rely on front end mechanics, so headless server stuff can still
function.
On Fri, Aug 9, 2024, 1:51 PM brbbbq @.***> wrote:

Has there been a change to on change method? Hex for same seed should be
the same, thus match cached value.
I have no idea. I tried the WAS Seed node, and it works without issue.
When I put it in fixed mode it doesn't render if there's no change.
Here's my version info:
ComfyUI: 23402dc84d http://2024-07-06
Manager: V2.46
—
Reply to this email directly, view it on GitHub
https://github.com/WASasquatch/was-node-suite-comfyui/issues/166#issuecomment-2278739357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZEZLG7GEYSOBLKBS5SBTZQUTUJAVCNFSM6AAAAABJOO7EW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZYG4ZTSMZVG4
.
You are receiving this because you commented.Message ID:
@.***>

I really don't know anything about the code. I will say that it's not that the server isn't refreshing after every change, such as moving nodes around. In the console window it says:
[rgthree] Using rgthree's optimized recursive execution.
Prompt executed in 0.02 seconds

It's that the Random Number nodes is triggering the KSampler to render that's the issue. I don't know if that's helpful at all.

Yeah I understand what's happening, but it shouldn't. The change function
reduces the seed to a unique hex, basically like a seed itself. The same
seed should always yield the same hexadecimal value and tell ConfyUI there
has been no change. It could be the rgthree execution server change but not
sure.
On Fri, Aug 9, 2024, 3:43 PM brbbbq @.***> wrote:

I really don't know anything about the code. I will say that it's not that
the server isn't refreshing after every change, such as moving nodes
around. In the console window it says:
[rgthree] Using rgthree's optimized recursive execution.
Prompt executed in 0.02 seconds
It's that the Random Number nodes is triggering the KSampler to render
that's the issue. I don't know if that's helpful at all.
—
Reply to this email directly, view it on GitHub
https://github.com/WASasquatch/was-node-suite-comfyui/issues/166#issuecomment-2278842086,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZEZLOXRO2VGFLVYZS673ZQVA2RAVCNFSM6AAAAABJOO7EW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZYHA2DEMBYGY
.
You are receiving this because you commented.Message ID:
@.***>


Add a integration test for create subscription CLI command

Motivation
due to #1612 , create subscription will fail with 500 when connecting to a broker that doesn't own the topic. we need to have an integration test to ensure CLI commands are run correctly.
Modifications
add an integration test for createSubscription

two issues spots with this integration tests:

pulsar admin doesn't handle exception correctly.

Got exception: javax.ws.rs.RedirectionException cannot be cast to javax.ws.rs.ClientErrorException

pulsar admin doesn't handle redirects at all..

retest this please


Hide bounding boxes in custom object detection

Add the option of hiding bounding boxes when using the custom object detector. This feature is already available in the standard object detector, but not the custom detector until now.
Also made a small fix to the documentation of detectObjectsFromImage() in Detection

Any feedback/thoughts @OlafenwaMoses ?


[bug] The github action package does not adapt the dmg attribute.

Describe the bug
The github action package does not adapt the dmg attribute.
"tauri": {
    "bundle": {
        "dmg": {
            "appPosition": {
	            "x": 180,
	            "y": 170
            },
            "applicationFolderPosition": {
	            "x": 480,
	            "y": 170
            },
            "windowSize": {
	            "width": 660,
	            "height": 400
            }
        }
    }
}

vscode works:

but the github action package looks like this:

Reproduction
No response
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.4.0 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.79.0 (129f3b996 2024-06-10)
    ✔ cargo: 1.79.0 (ffa9cf99a 2024-06-03)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 21.7.1
    - pnpm: 8.15.7
    - yarn: 1.22.22
    - npm: 10.5.0

[-] Packages
    - tauri [RUST]: 1.7.1
    - tauri-build [RUST]: 1.5.3
    - wry [RUST]: 0.24.10
    - tao [RUST]: 0.16.9
    - @tauri-apps/api [NPM]: 1.6.0
    - @tauri-apps/cli [NPM]: 1.6.0

[-] App

Stack trace
No response
Additional context
No response

This is tracked here https://github.com/tauri-apps/tauri-action/issues/740 (i'd apprecaite it if you could try the mentioned workaround)


Fixed issue #29 Kmeans Algorithm

I have added Kmeans algorithm. Thanks.

can you please real-world example for understand the algorithm people who first time read your blog.

Can you check now I have made changes.


fix benchmark

Benchmarks set is empty now, correct option is add:, not code:

Also, benchmarks are not running against arrays, as supposed.
See related https://github.com/jonschlinkert/benchmarked/pull/3

thanks!


Output of digested plasmid is shown as circular in the frontend

Probably an error of the backend, would need to double check.

It was a problem of the frontend actually, fixed. Somehow the function genbankToJson did not read the topology of the molecules correctly from certain genbank files. To see if this is fixed in more recent versions, for now the workaround was:
export function convertToTeselaJson(entity) {
  // TODO: This might have been fixed in more recent versions of the library
  // For some reason, as it is it does not read circular or linear properly from certain files
  const { parsedSequence } = genbankToJson(entity.sequence.file_content)[0];

  if (entity.sequence.file_content.split('\n')[0].includes('linear')) {
    parsedSequence.circular = false;
  }
  return parsedSequence;
}


multi-clients support issue: std::cell::RefCell<mqtt_async_client::util::free_pid_list::FreePidList>

I want to build a simulator which simulates multiple mqtt clients in async mode.
However, thread::spawn was blocked by "std::cell::RefCell<mqtt_async_client::util::free_pid_list::FreePidList>" since the trait std::marker::Sync is not implemented for it.

Are you trying to call the same Client from multiple threads at the same time? I probably won't support that.
Can you use multiple Clients, one in each thread?
If you need Client to be Send then I can maybe support this.

Are you trying to call the same Client from multiple threads at the same time? I probably won't support that.
Can you use multiple Clients, one in each thread?
If you need Client to be Send then I can maybe support this.

I'm not trying to call same client multiple time. Instead, I'm trying to simulate many MQTT clients and therefore Client has to support Send.
My current workaround is to lunch multiple threads and then use tokio::localset to execute future.
appreciate if you can support Send, then I don't need to manage how many threads I can lunch and how many clients have to be put into one thread.
Thank.

I just pushed a change that I think made Client Send. Can you try 6010ac6c5c87c10a418994a5c353f387b9adfd54 and see if that fixes your build error?

I just pushed a change that I think made Client Send. Can you try 6010ac6 and see if that fixes your build error?

@fluffysquirrels Thank you, I can confirm it works.
once you released it, I will try to compare the performance between:

Using Tokio:spawn (with your updated lib)
Using Thread + tokio:LocalSet (with old lib).

I got bit by this recently with tokio tasks, and after building the 6010ac6 revision mentioned above, it resolved my problem. This issue is 6+ months old, will a new release be made with this fix incorporated?
Thanks!

@jkfritcher @xudesheng I just released 0.1.6 to crates.io. Sorry for the delay on this.

I'll close this now since I think the fix works, happy for you to reopen it if there are more problems.


Proptypes warning for segmented value

soldair/node-qrcode can also accept segmented values to generate QR-code. I've modified PropTypes check so that no warning is generated on segmented value.

Hi @cvvspb, sorry for the late reply. I've refactored the codebase a little bit. Can you re-submit the PR? Maybe just need to remove the propTypes part of index.js.

@awesomejerry
Greetings!
I've resolved the merge conflict caused by the latest changes.


Unable to call into polls data

I'm trying to capture Poll data and having issues accessing the options portion. I'm accurately tagging posts with polls, but can't pull in the actual data.
I'm also pulling in poll data in the expansion
expansions: [.authorId(userFields: [\.profileImageUrl, \.verified]), .mediaKeys(mediaFields: [\.id, \.url, \.width, \.height, \.previewImageUrl]), .pollIds(pollFields: [\.id, \.options, \.endDatetime, \.votingStatus])],
Any thoughts?
  /// Contains objects describing each choice in the referenced poll.
  public let options: [Option]

Here's a rough sample of the code I'm just trying to get up and running

               if let pollKeys = tweet.attachments?.pollIds {
                        ForEach(pollKeys, id: \.self) { key in
                            if let media = includes?.polls {
                                ForEach(media){ poll in
                                    if poll.id == key {
                                        ForEach(poll.option, id: \.position) { item in
                                            Text(item.label)
                                        }
                                    }
                                }
                            }

                        }
                    }

Have you tested this in the playground as well? So we can rule out it is their API but Twift that needs to have some adjustments?

From what I can tell we're getting back the data correct. If I print out media[0] I get back the following.
polls: Poll(id: "1556989845041553408", options: [Twift.Poll.Option(position: 1, votes: 102, label: "Yes"), Twift.Poll.Option(position: 2, votes: 22, label: "No"), Twift.Poll.Option(position: 3, votes: 13, label: "To rich for my blood")], durationMinutes: nil, endDatetime: Optional(2022-08-13 13:04:41 +0000), votingStatus: Optional(Twift.Poll.VotingStatus.open))

Maybe this is a misunderstanding on how to access the data on my part

Ok I don't know if this how this is meant to be used, but I can workaround the issue using indices
                             ForEach(media.indices, id: \.self){ poll in
                                    if media[poll].id == key {
                                        ForEach(media[poll].options, id: \.position) { p in
                                            Text(p.label)

                                        }
                                    }
                                }

@arbyruns here's some sample code that should help:
struct TweetView: View {
  var tweet: Tweet
  var includes: Tweet.Includes?

  var body: some View {
    if let pollIds = tweet.attachments?.pollIds,
       let polls = includes?.polls?.filter { poll in pollIds.contains { id in
         id == poll.id
       }} {
     ForEach(polls) { poll in
       if let totalVotes = poll.options.reduce(0, { p, c in p + c.votes}) {
         ForEach(poll.options, id: \.label) { option in
           HStack {
             Text(option.label)
             Spacer()
             Text("\(option.votes)").foregroundStyle(.secondary)
           }
         }
       }
     }
   }
  }
}


Sync service 12.0.1 fails to connect to the database with {:case_clause, :undefined} error

Full server log when trying to run electricsql/electric:0.12.1 on DO with DO's managed Postgres:
[2024-06-20 14:37:24] 14:37:24.179 pid=<0.3055.0> [info] Starting ElectricSQL 0.12.1 in direct_writes mode.
[2024-06-20 14:37:24] 14:37:24.193 pid=<0.3056.0> [info] Running Electric.Plug.Router with Bandit 1.5.3 at :::5133 (http)
[2024-06-20 14:37:24] ▓ ┌────────────────────────┐
[2024-06-20 14:37:24] ▓ │  INITIALISATION ERROR  │
[2024-06-20 14:37:24] ▓ ┕━━━━━━━━━━━━━━━━━━━━━━━━┙
[2024-06-20 14:37:24] ▓
[2024-06-20 14:37:24] ▓ Failed to start child Electric.Replication.PostgresConnectorMng:
[2024-06-20 14:37:24] ▓   {{:case_clause, :undefined}, [{Electric.Replication.PostgresConnectorMng, :get_verify_peer_opts, 0, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 358]}, {Electric.Replication.PostgresConnectorMng, :maybe_verify_peer, 1, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 350]}, {Keyword, :update!, 4, [file: ~c"lib/keyword.ex", line: 1144]}, {Keyword, :update!, 4, [file: ~c"lib/keyword.ex", line: 1148]}, {Electric.Replication.PostgresConnectorMng, :update_connector_config, 2, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 119]}, {Electric.Replication.PostgresConnectorMng, :init, 1, [file: ~c"lib/electric/replication/postgres_manager.ex", line: 94]}, {:gen_server, :init_it, 2, [file: ~c"gen_server.erl", line: 2057]}, {:gen_server, :init_it, 6, [file: ~c"gen_server.erl", line: 2012]}]}
[2024-06-20 14:37:24] ▓
[2024-06-20 14:37:24] ▓ Please file a new issue on GitHub[1], including the contents of this error.
[2024-06-20 14:37:24] ▓
[2024-06-20 14:37:24] ▓ [1]: https://github.com/electric-sql/electric/issues
[2024-06-20 14:37:24]
[2024-06-20 14:37:24] ••• Shutting down •••
[2024-06-20 14:37:24]
[2024-06-20 14:37:24] [os_mon] memory supervisor port (memsup): Erlang has closed
[2024-06-20 14:37:24] [os_mon] cpu supervisor port (cpu_sup): Erlang has closed

This is caused by an issue in Erlang. It was introduced in OTP 26.2. Electric's sync service had been using OTP 25.x up until version 0.12.1 in which we upgrade to OTP 27.0.


About the accuracy

I use the code provided to train and evaluate without changing anything.
The max epoch is 200, and I use show_CD.py to evaluate.
During evaluation, I get ~2.6e-3 pred2gt loss, which is close enough to the numbers reported.
However, I also get ~2.6e-3 gt2pred loss. Obviously higher than the numbers reported.
Is this normal?

hello please tell me how to find the 'pred~gt' data ?

I think there is no such data.
One of the number means the least distance to any ground truth points for every output point.
The other is the least distance to any output points for every ground truth point.
Please refer to the criterion_PointLoss() function in show_CD.py


az monitor log-analytics workspace data-export create

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az monitor log-analytics workspace data-export create
Errors:
Cannot deserialize content-type: text/plain
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 233, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 660, in execute
    raise ex
cli/core/commands/__init__.py, ln 723, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
...
site-packages/msrest/pipeline/universal.py, ln 203, in deserialize_from_text
    raise DeserializationError("Cannot deserialize content-type: {}".format(content_type))
msrest.exceptions.DeserializationError: Cannot deserialize content-type: text/plain

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az monitor log-analytics workspace data-export create -g {} --workspace-name {} -n {} --destination {} --enable -t {}

Expected Behavior
Environment Summary
Linux-4.15.0-1108-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.20.0

Extensions:
ai-examples 0.2.5

Additional Context

monitor


Forgotten C++: std::bitset (now constexpr in C++23!)

https://en.cppreference.com/w/cpp/utility/bitset
Point out that we can expect optimized opcodes here.

Coming in 458


⚠ Minecraft has degraded performance

In 6ac52c4, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1152 ms

Resolved: Minecraft performance has improved in b58690b.


Remap alt+shift+7 command does not override the default key-map

GitLens Version: 3.5.1
VSCode Version: 1.12.2
OS Version: OS X El Capitan

Steps to Reproduce:

Remapping the key-shortcut

{
	"key": "ctrl+shift+7",
	"command": "gitlens.showCommitSearch",
	"when": "gitlens:enabled"
}

I want to override the default mapping with the above command since the default one is blocking the default keyboard layout I have for \. However adding the above mapping will just include another command instead of overwrite the default one.

@guoliang vscode allows for multiple shortcuts for a single command, but it also allows you to specifically remove a shortcut key -- see https://code.visualstudio.com/docs/getstarted/keybindings#_removing-a-specific-key-binding-rule
Basically I think you just need to add the following to remove that key-binding:
{
	"key": "alt+/",
	"command": "-gitlens.showCommitSearch"
}


Add PocketBaseUML

Hey @benallfree, just wanted to contribute my repo to the list: PocketBaseUML is a free, open-source UML diagram generator for PocketBase. Though PocketBase already comes with a fantastic UI (better than Supabase or Firebase, IMO), I was missing the ability to actually SEE the data architecture in a graphical form, easy to parse and understand in a quick glance.

Awesome job, thank you!


Introduce delegate task

As stated in https://github.com/tschaub/grunt-newer/issues/29#issuecomment-160223515, I added a simple delegation task (named delegate). This allows to specifiy for individual tasks src (and also dest) for newer to check, and then actually run the task with its own src ( and dest). This is useful for SASS/LESS.
With the following config:
grunt.initConfig( {
	delegate: {
		sass: {
			src: [ 'resources/scss/**/*.scss' ]
		}
	},
	sass: {
		styles: {
			expand: true,
			cwd: 'resources/scss/',
			src: [ '*.scss' ],
			dest: 'assets/css/',
			ext: '.css'
		}
	}
} );

one could simply run grunt newer:delegate:sass:styles to actually have sass:styles be run when there are new .scss files (even in subfolders).

I just improved the new delegate task.
As we need at least src specified for the task/target, the delegate task is now a multi task, requiring a config.
I also integrated the possibility to specify a custom task (with an optional target).
Her's the above example, a little bit expanded:
grunt.initConfig( {
	delegate: {
		sass: {
			src: [ 'resources/scss/**/*.scss' ]
		},
		sass_foo: {
			task: 'sass:foo',
			src: [ 'resources/scss/**/*.scss' ]
		},
		'sass-styles': {
			task: 'sass:styles',
			src: [ 'resources/scss/**/*.scss' ]
		}
	},
	sass: {
		foo: {
			expand: true,
			cwd: 'resources/scss/',
			src: [ 'foo.scss' ],
			dest: 'assets/css/',
			ext: '.css'
		},
		styles: {
			expand: true,
			cwd: 'resources/scss/',
			src: [ '*.scss' ],
			dest: 'assets/css/',
			ext: '.css'
		}
	}
} );

When calling $ grunt newer:delegate:sass, the sass task will be executed (with all available targets). Calling $ grunt newer:delegate:sass-foo only runs sass:foo, while $ grunt newer:delegate:sass-styles results in running sass:styles. All runs are only performed if there are changed files wrt. the src given in the according delegate target.

I just published this (in a more condensed form) as new Grunt plugin: grunt-delegate.


Incorrect y-coordinates for non-georeferenced data from open_rasterio

For a non-georeferenced dataset, open_rasterio() results in y-coordinates from the raster height to two times the height, instead of the expected 0 .. height range:
import numpy as np
import rasterio
from xarray import open_rasterio

tmp_file = 'no_transform.tif'
nx, ny, nz = 4, 3, 3
data = np.arange(nx*ny*nz,
                 dtype=rasterio.float32).reshape(nz, ny, nx)
with rasterio.open(
              tmp_file, 'w',
              driver='GTiff', height=ny, width=nx, count=nz,
              dtype=rasterio.float32) as s:
        s.write(data)

open_rasterio(tmp_file)

<xarray.DataArray (band: 3, y: 3, x: 4)>
array( ... )
Coordinates:
  * band     (band) int64 1 2 3
  * y        (y) float64 3.5 4.5 5.5
  * x        (x) float64 0.5 1.5 2.5 3.5
Attributes:
     res:        (1.0, -1.0)
     is_tiled:   0
     transform:  (0.0, 1.0, 0.0, 0.0, 0.0, 1.0)

Passing transform=from_origin(0,3,1,1) (from rasterio.transforms) to rasterio.open() in the above code seems to give the expected result with y running down from 2.5:

<xarray.DataArray (band: 3, y: 3, x: 4)>
array( ... )
Coordinates:
  * band     (band) int64 1 2 3
  * y        (y) float64 2.5 1.5 0.5
  * x        (x) float64 0.5 1.5 2.5 3.5
Attributes:
    res:        (1.0, 1.0)
    is_tiled:   0
    transform:  (0.0, 1.0, 0.0, 3.0, 0.0, -1.0)

I'm not sure whether there's something amiss in the xarray coordinate calculations or the rasterio default transform logic. Looking at the code, I feel like there's some mismatch between the rasterio res property logic and the xarray coordinate generation (namely, the sign of res[1]):
https://github.com/mapbox/rasterio/blob/fcd361c49cca9c4ad32a2ac1f5d66b967f4b6cd4/rasterio/_base.pyx#L611-L619
https://github.com/pydata/xarray/blob/f83361c76b6aa8cdba8923080bb6b98560cf3a96/xarray/backends/rasterio_.py#L131-L139
I'm not quite sure if the xarray code is quite correct for georefenced data, since it doesn't utilize the given coordinate transform. I don't' have files to test it with though.

I'm not quite sure if the xarray code is quite correct for georefenced data

We don't test for correct georeferencing in the xarray codebase but I did a bunch of tests locally and it seemed to work well (at least for such a logical flow as being completely upside down, e.g. http://xarray.pydata.org/en/latest/auto_gallery/plot_rasterio.html#sphx-glr-auto-gallery-plot-rasterio-py).
For your example something seems to go wrong though. What version of rasterio are you using?

@fmaussion I've tested with both rasterio 0.36.0 and the current master branch (1.0a11), both seem to have the same behaviour.

OK, I will investigate but this might have to wait for next week. Also, I'll do some more georeferencing tests offline (we can easily check with external tools if what we are doing is correct or not)

@fmaussion Thanks!
I get the feeling there's probably something odd with rasterio here. I played around a bit and found that rasterio doesn't seem to care about the signs in the transform if the given upper left is (0,0) . Giving any of from_origin(0, 0, i, j) for i, j = +-1 as the transform gives a warning about using the default geotransform and doesn't save it to the file.

I'm afraid I don't know rasterio well enough to be able to take a decision here. However the fact that rasterio throws a warning when using your code above is not a good sign, and I don't think we have to take action in xarray (yet).
I am however also not super happy with this bit of code here: https://github.com/pydata/xarray/blob/014b7898a65d9406a0e9af0ab7166702b6e977b3/xarray/backends/rasterio_.py#L136-L142
It works for all the cases I've encountered until now, but it feels like this logic should be available in rasterio already.

It looks like we could use rasterio's Affine object for this: https://stackoverflow.com/questions/27861197/how-to-i-get-the-coordinates-of-a-cell-in-a-geotif
Will have a look at it

Just looking at the rasterio quickstart, it seems that it'd be easy to generate the coordinates using the Affine instance (though that is only available in Rasterio > 1.0a, with 0.36.0 you just have the tuple containing the multipliers). I think that might solve the problem though, since I would expect the sign of resand the transform to be synchronized to  the same convention on the rasterio side, which might be breaking with the current code.

Testing against all files used in the rasterio test base yields following results:
import numpy as np
import rasterio
import xarray as xr
from affine import Affine
from glob import glob
import os

# Test all files from rasterio's test base
rdir = '/home/mowglie/Downloads/rasterio-master/tests/data/*.tif'
ok = []
for path in glob(rdir):
    ds = xr.open_rasterio(path)
    rio = rasterio.open(path, mode='r')

    # Get geo coords
    T0 = rio.transform
    T1 = T0 * Affine.translation(0.5, 0.5)
    rc2xy = lambda r, c: (c, r) * T1

    xy0r = rc2xy(0, 0)
    xy0x = (ds['x'].values[0], ds['y'].values[0])
    xy1r = rc2xy(rio.height-1, rio.width-1)
    xy1x = (ds['x'].values[-1], ds['y'].values[-1])

    t1 = np.allclose(xy0r, xy0x)
    t2 = np.allclose(xy1r, xy1x)
    if t1 and t2:
        print(os.path.basename(path) + ': OK')
    else:
        print(os.path.basename(path) + ': NOT OK', xy0r, xy0x, xy1r, xy1x)

Out:
shade.tif: OK
rgb_lzw.tif: OK
rgb_deflate.tif: OK
rgb4.tif: OK
RGBA.byte.tif: OK
world.rgb.tif: OK
world.byte.tif: OK
alpha.tif: OK
rgb1.tif: OK
RGB.byte.tif: OK
float_nan.tif: NOT OK (50.0, 50.0) (50.0, 250.0) (250.0, 150.0) (250.0, 350.0)
float.tif: NOT OK (50.0, 50.0) (50.0, 250.0) (250.0, 150.0) (250.0, 350.0)
rgb3.tif: OK
rgb2.tif: OK

The two files which are not working also miss a proper crs and transform, but still we should be consistent with what rasterio is doing :-(
At least I think that the majority of "standard" files are working properly with our current code.


move doc from analytic-services to protempa-service for destination, source-config, concept endpoints

done and close now


Update Restore-SPODeletedSite.md

Link for example showed up as "HYPERLINK "https://contoso.sharepoint.com/sites/arecycledsite" https://contoso.sharepoint.com/sites/arecycledsite" with both URLs visible links.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Restore-SPODeletedSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx @fselkirk, appreciated.


Create the basic homepage for GirlScript Chennai

Create the main index.html file in the https://github.com/smaranjitghose/girlscript_app/tree/master/website directory

Hello. I’d like to work on this issue.

Hey! I would like to work on this issue.

@himanshusharma89 Go ahead

also make sure the images that you use are catered for GirlScript Chennai only..Try to design them..have a separate assets folder so that in future we can update the design

@smaranjitghose May I please know why I wasn’t assigned to this issue?

@rabeehrz I wanted to work on the update of the home page and @himanshusharma89  has already given so many updates for the app within a short time.Hence I assigned it to him so that once the basic home page is done you and the others can work on integrating other stuff

@smaranjitghose Yeah okay. I think it should be done using some Framework though.

Once @himanshusharma89 is done is with this..I'll create an Update issue..you can take over from there

@smaranjitghose That sounds good. ♥

hey I would like to work on this


Consistency between production build and development server

Hey there @Arsero !
This plugin is quite interesting! I find it very useful but I think there's a discrepancy between the production build (building the app then serving) and the development server (starting the app in development mode).
I find it very difficult that the plugin makes it two separate environments needing different setups when the goal is to pick up relationships between files and displaying those in a graph.
Would it be possible to find a way so that when executing start and build, it works the same way?

https://github.com/Arsero/docusaurus-graph/blob/1d4008cb4e5b64c3401fd09f8942e2d534b150d7/src/index.ts#L30
Also it seems that you hardcoded the build folder but I am pretty sure we can overwrite that in docusaurus, which would ultimately break your package, right?
Source: https://docusaurus.io/docs/cli#docusaurus-build-sitedir

Hi HunteRoi,
Thank you for reporting this issue! The problem has been resolved in the latest release, version 2.0.0. 🎉 Please update to this version and let me know if you encounter any further issues.
Feel free to reopen this issue or create a new one if needed. Thanks for your support!
Best regards,
Arsero


read_xlsx() reading numeric columns as character

Unbeknownst to me the read_xlsx() function used in Generate_reports.R was reading some numeric variables as character variables. (@teixeirak this was the source of https://github.com/SCBI-ForestGEO/HarvardMortality/issues/5): I added this code to fix the issue for the HF data.
# Convert character vectors to numeric
mort[, "DBH"] <- as.numeric(mort[, "DBH"])
mort[, "HOM"] <- as.numeric(mort[, "HOM"])
mort[, 'Percentage of crown intact'] <- as.numeric(mort[, 'Percentage of crown intact'])
mort[, 'Percentage of crown living'] <- as.numeric(mort[, 'Percentage of crown living'])

For SCBI this is only a problem for DBH and HOM, but it still might be worth to have explicit checks for all cases

@ValentineHerr , this is one for you to check.

Yes, I've used as.numeric(mort[, "DBH"]) but I never used HOM. Did I miss a test that needs HOM?

No, there's no test with that.

Ok thanks. I think  we can close this issue.


Emitted signal with parameters unbound has incorrect error for unexpected argument count.

Godot version
715f556cd4f9b21e223659d899dfb1cd7aa3975a (with a small unrelated patch to fix crashing)
(Edit: the unrelated patch is not needed, it is fixed upstream now)
System information
Arch on 5.18.11-arch1-1
Issue description
"Method expected 1 arguments, but called with 1.."
Can the extra . be fixed also?
extends Node

func _on_scene_child_entered_tree(_extraneous_parameter_unbound) -> void:
	print("hello")

E 0:00:04:0869   emit_signalp: Error calling from signal 'child_entered_tree' to callable: 'Node(scene.gd)::_on_scene_child_entered_tree': Method expected 1 arguments, but called with 1..
  <C++ Source>   core/object/object.cpp:1025 @ emit_signalp()

Steps to reproduce

Bind signal child_entered_tree with 1 argument unbound.
Add an extraneous parameter to the callback.

Minimal reproduction project
test15.zip

This Receiver Method is a manually modified function, not the automatically generated one.
For parameters in the final Receiver Method parameter list, if divided by where they come from, currently, they can be divided into 3 categories, as I explained with different colors in the picture of https://github.com/godotengine/godot/pull/60143#issuecomment-1095404236 (there, I added another one, the signal source, to increase flexibility of the connection dialog).

If the parameters are classified, the unbind here belongs to the parameters in the callable, and the N subsequent call parameters will be ignored. The effect eventually results in the removal of the last N parameters before the callable parameters in the parameter list.

The main reason is that Callable may be nested, so the cost of counting the number of parameters may be a bit large.

Not 100% sure if this is the same issue, but if you bind an extra parameter when connecting you get the same kind of misleading error.  It says "expected X, but called with X" when it should be "expected X, but called with X+1"
The code:
extends SceneTree

signal the_signal(one)

func signal_callback(one):
    print(one)

func _init():
    var callback = signal_callback.bind("two")
    the_signal.connect(callback)
    the_signal.emit('foo')

    quit()

The error:
ERROR: Error calling from signal 'the_signal' to callable: 'SceneTree(demo_signal_parameter_error.gd)::signal_callback': Method expected 1 arguments, but called with 1.


Does this only work for subdomains?

environment:
  VIRTUAL_HOST: mywebsite.io
  VIRTUAL_PORT: 80
  LETSENCRYPT_HOST: mywebsite.io
  LETSENCRYPT_EMAIL<EMAIL_ADDRESS>
So this is what my environment is set up as, if i try to use a domain like this it doesnt seem to work, but as soon as i use a subdomain it does? So my question is does this only work for subdomains and not top level domains ?

Nope, this project woks for top level domains as well as subdomains.
If the authorizations for your top level domain are failing, I suggest reading https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/blob/master/docs/Invalid-authorizations.md first.

Same issue.
My docker-compose file's bellow
  main:
    image: my_image:latest
    container_name: main
    environment:
      - VIRTUAL_HOST=my_image.net
      - LETSENCRYPT_HOST=my_image.net
      -<EMAIL_ADDRESS>    network_mode: "webproxy"
    restart: always

  shop:
    image: my_shop:latest
    container_name: test-my_image
    environment:
      - VIRTUAL_HOST=shop.my_image.net
      - LETSENCRYPT_HOST=shop.my_image.net
      -<EMAIL_ADDRESS>    network_mode: "webproxy"
    restart: always

But only sub-domain sites working.
This image is latest and I checked port status.
And I tested this code in 'readme.md'
docker run -d -e VIRTUAL_HOST=your.domain.com \
              -e LETSENCRYPT_HOST=your.domain.com \
              -e<EMAIL_ADDRESS>\
              --network=webproxy \
              --name my_app \
              httpd:alpine

It worked. but my image doesn't work.


128 update links cassini vims

-Update asset links for Cassini/vims
-Replace specific folders for VIMS datasets in holdings w/generic text where applicable for better descriptions
-Fix minor bug in tabs for active tab

Branch removed from staging


Attempted fix for same func and var names

This should enable the user to choose whether or not they want all code to be generated or just the code that returns a map of all available methods -> by using the flag "skip-defaults".
Also, Headers and Footers get renamed: HeaderFileName and FooterFileName so every file would have different header and footer names.
Please review.

Is it possible to only give each file unique Header and Footer names, to avoid same func and var names?
That way we won't need another flag?

I'm sorry, would you please elaborate your idea? Thanks!

I was wondering whether it is possible to not have the skip defaults flag
and still avoid the name conflicts, by only renaming the HeaderaFilename
and FooterFilename to be unique for each file in the folder?
On Mon, 19 Mar 2018, 09:28 MSalopek<EMAIL_ADDRESS>wrote:

I'm sorry, would you please elaborate your idea? Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gogo/letmegrpc/pull/42#issuecomment-374134631, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABvsLYw_-eOr01pxgSe8G-LIEOjj3n7zks5tf2xCgaJpZM4Sqauj
.

It is possible, yes. But the name func NewHandler will remain the same for all the files, what is something you would like to keep?


Ponder crashes on 2nd startup if using custom indexes

Description
If I add a custom index to a model and shutdown Ponder after it is done indexing (through CTRL-C), it fails starting back up.
Repro - model
Accounts: p.createTable({
  id: p.hex(),
  nftBalances: p.many("NFTBalances.ownerId")
}),
NFTBalances: p.createTable({
  id: p.string(), // `${contractAddress}-${tokenId}-${ownerAddress}`
  balance: p.bigint(),
  lastUpdatedAtBlock: p.bigint(),
  ownerId: p.hex().references("Accounts.id")
}, {
  ownerIndex: p.index(["ownerId"]) // WHERE "ownerId" = '0x0000...'
})

Repro - flow
Possibly related: I'm getting this in Windows; will test in a Linux environment a few hours from now

npm run start
wait until indexing finishes
stop with ctrl-c
npm run start should crash

Relevant logs
(this is when running Ponder for the second time, after shutdown)
11:04:43 AM INFO  indexing   Completed historical indexing
11:04:43 AM WARN  internal   Database error during 'createIndexes', retrying after 100 milliseconds. Error: relation "NFTBalances_ownerIndex" already exists
11:04:43 AM WARN  internal   Database error during 'createIndexes', retrying after 200 milliseconds. Error: relation "NFTBalances_ownerIndex" already exists
11:04:43 AM WARN  internal   Database error during 'createIndexes', retrying after 400 milliseconds. Error: relation "NFTBalances_ownerIndex" already exists
11:04:44 AM ERROR process    Caught unhandledRejection event with error:
error: relation "NFTBalances_ownerIndex" already exists
    at C:\Users\Matt\Desktop\Work\projects\reworked-systems\items-indexer\node_modules\pg\lib\client.js:526:17
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at PostgresConnection.executeQuery (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/dialect/postgres/postgres-driver.js:69:28)
    at PostgresConnection.connection.executeQuery (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/driver/runtime-driver.js:89:24)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/query-executor/query-executor-base.js:35:28
    at DefaultConnectionProvider.provideConnection (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/driver/default-connection-provider.js:10:20)
    at DefaultQueryExecutor.executeQuery (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/query-executor/query-executor-base.js:34:16)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/postgres/service.ts:633:13
    at HeadlessKysely.wrap (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/kysely.ts:35:24)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/postgres/service.ts:614:11
    at PostgresConnection.executeQuery (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/dialect/postgres/postgres-driver.js:88:41)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at PostgresConnection.connection.executeQuery (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/driver/runtime-driver.js:89:24)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/query-executor/query-executor-base.js:35:28
    at DefaultConnectionProvider.provideConnection (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/kysely/dist/esm/driver/default-connection-provider.js:10:20)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/postgres/service.ts:633:13
    at HeadlessKysely.wrap (file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/kysely.ts:35:24)
    at file:///C:/Users/Matt/Desktop/Work/projects/reworked-systems/items-indexer/node_modules/@ponder/core/src/database/postgres/service.ts:614:11
    at async Promise.all (index 0)

Looks like https://github.com/ponder-sh/ponder/pull/898 might fix this, will test and report back

Can confirm it works after merging that PR locally! :D


Automate the creation of relationships between Power BI, Azure Synapse, Oracle,and Azure storage account (Purview Apache Atlas)?

Really Cool API and loved the videos on Youtube!
What is your desired outcome
Using the API to "automatically" create or update the relationships between your Power BI, Azure Synapse, Oracle , Google, and Azure storage account entities in your Purview collections using the Azure Purview REST API. I work in marketing and we have lots of different data sources and datasets .
With the API I can manually upload an Excel File to map different datasets but is there a more automated way to do that?
Goal: Use a Python script that can either use regular expressions or other features of the API to find the Synapse, Oracle datasets that correspond to the Power BI reports by their name or attributes and then link them without manual intervention.
What have you tried so far
Used the API and Excel spreadsheets to update the metadata and it works.
Additional context
Really open to any suggestions to save time from the manual workflow.

Hi, @JavaPr288 - Thank you for using PyApacheAtlas!
I think the answer is yes although it will require much more development effort and PyApacheAtlas doesn't help with nearly any of it.
I think this breaks down to a few steps:

You need to understand what are the relationships - When you do this manually, you might be looking at the UI and can quickly jot it down but now using code, you'll need to find the Power BI APIs that return the data set dependencies.
You need to work out the fully qualified names - You can get this from the get_typedefs method but you'd have to be able to retrieve all the necessary parts from the Power BI dependency API calls. This might get tricky if it doesn't return, for example, the synapse db schema.
You need to actually create lineage - I have samples for that!
You need to automate - This is perhaps the trickiest. Do you want to automate as in you have a script with a dataset id in power bi and it goes and updates all of the lineage for you? Do you want to listen for whenever a PBI dataset asset is created in Purview and then react to it? Both are possible but the listener might be much more difficult to accomplish.

Can you share more about your desired workflow and maybe we can brainstorm some ideas on this thread or others might pipe in to share their automation approaches?

Hi,
Thanks for your reply really appreciate it!
Great I've got the first 3 points done; watched your videos and tried out the code samples for custom lineage.
For point 4 I'll take whatever I can get lol using Purview and pyapacheatlas .
We have a Purview weekly scan and I'm planning to run a Python job afterwards that would update relationships based on the data in Azure Purview (Power BI and other).
I'd also plan to run this script once in the beginning to get the relationships. I'm shooting for a target accuracy of 80% after that can live some manual adjustment.
thanks