Preserve origin on twitter failure

I saw #227, but when I inspect request.env['omniauth.origin'] in /auth/failure after authing Twitter I get a nil, whereas in /auth/twitter/callback I get the expected value.

Can someone comment on why this was closed or what the right way to get the origin in /auth/failure is?  I'm having the exact same issue.  request.env['omniauth.origin'] == nil in /auth/failure.
Thanks.

I am using 1.0.2 btw with omniauth-oauth 1.0.0

app/controllers/authentications_controller.rb @ line 31 in AuthenticationsController#failure:

    26:     @authentication.destroy
    27:     redirect_to edit_user_path(current_user), :notice => "Disconnected from #{provider}."
    28:   end
    29:
    30:   def failure
 => 31:     binding.pry
    32:     flash[:notice] = "Failed to connect to service."
    33:     if current_user
    34:       redirect_to edit_user_path(current_user)
    35:     else
    36:       redirect_to root_path
[1] pry(#<AuthenticationsController>)> request.env["rack.session"]["omniauth.origin"]
=> nil
[2] pry(#<AuthenticationsController>)> request.env["rack.session"]
=> {"session_id"=>"30262de6db88a559ee2927ef9cfed07d",
 "_csrf_token"=>"B4xXH8HmB7jH9KAjvMsuHwe9auHVsF6uz783Cglh8fw=",
 "oauth"=>{"twitter"=>{"callback_confirmed"=>true}}}
[3] pry(#<AuthenticationsController>)> env["action_dispatch.request.unsigned_session_cookie"]
=> {"session_id"=>"30262de6db88a559ee2927ef9cfed07d",
 "_csrf_token"=>"B4xXH8HmB7jH9KAjvMsuHwe9auHVsF6uz783Cglh8fw=",
 "oauth"=>{"twitter"=>{"callback_confirmed"=>true}}}
[4] pry(#<AuthenticationsController>)> request.env["action_dispatch.request.unsigned_session_cookie"]
=> {"session_id"=>"30262de6db88a559ee2927ef9cfed07d",
 "_csrf_token"=>"B4xXH8HmB7jH9KAjvMsuHwe9auHVsF6uz783Cglh8fw=",
 "oauth"=>{"twitter"=>{"callback_confirmed"=>true}}}
[5] pry(#<AuthenticationsController>)> request.env['omniauth.origin']
=> nil


Derive Macro error

Following the basic example in the docs:

Following the basic example in the docs:
#[derive(NifMap)]
struct AddMap {
lhs: i32,
rhs: i32,
}
I'm getting this error
error[E0107]: wrong number of type arguments: expected 1, found 2
--> src/lib.rs:573:34
|
573 | #[derive(Clone, Debug, Eq, Hash, NifMap)]
|                                  ^^^^^^ unexpected type argument
Rustler version : 0.21.1"
Rust version: 1.40

Thanks for your report!
We just released v0.22.0. With that, we also test the templates with CI (which was not done with v0.21.0, IIRC). I just ran the template test manually and it worked, so I believe this issue here can be closed. Please reopen if this problem can still be reproduced with v0.22.0 or above.


MWPW-127384 Ability to anchor/focus background image at certain point…

Ability to anchor/focus background image at certain point in section-metadata component

Resolves: MWPW-127384
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/suhjain/background-image-test?martech=off
After: https://focalpoint--milo--suhjainadobe.hlx.page/drafts/suhjain/background-image-test?martech=off

Closing the PR

Why do we have multiple PR's open for this?
I just finished commenting on this newer PR
https://github.com/adobecom/milo/pull/685

@rgclayton : this PR is specifically for section-metadata and https://github.com/adobecom/milo/pull/685 is for the marquee.

Anyone have any opinions about "focalpoint" vs "focuspoint" ?
@ryanmparrish @chrischrischris @auniverseaway

I noticed that since the background image for section-metadata is set to object-fit: cover the image is always stretched the full width. Which means that the left/right settings for this don't change the image position. Only the vertical access gets changed.
Reference:
https://github.com/adobecom/milo/blob/main/libs/blocks/section-metadata/section-metadata.css#L72
Do we want to handle the background position different with this focal point setting so authors can adjust the horizontal access?
cc: @auniverseaway @ryanmparrish @chrischrischris @JasonHowellSlavin

I noticed that since the background image for section-metadata is set to object-fit: cover the image is always stretched the full width[0]...Do we want to handle the background position different with this focal point setting so authors can adjust the horizontal access?

I don't particularly like authors directly authoring CSS properties. Is there an issue with just adding classes within the section-metadata that will target the background image and add the focus points we want? It would also likely allow for an override of object-fit, which would solve for the issue[0] you raised @rgclayton.
Seems simpler to add four CSS classes like focus-upper-left, focus-upper-right etc. than to add a new key value pair to section metadata and two value words.

Apologies, did not mean to close the PR.

Seems simpler to add four CSS classes like focus-upper-left, focus-upper-right etc. than to add a new key value pair to section metadata and two value words.

If that's the case, something more simple, and I think how @suhjainadobe originally had it  left, top, right, bottom etc. Not a fan of long, multiple word key words as they are much easier to mess up.

Seems simpler to add four CSS classes like focus-upper-left, focus-upper-right etc. than to add a new key value pair to section metadata and two value words.

The authors will have to remember the specific keywords for adding the classes which might be an issue if they miss out on something.

I just had an idea. I think it makes more sense to group the values with the background image - similar to how you are doing it for the Marquee block. The values are directly under the background image and we remove the focalPoint property.

I also think the background image property object-fit: cover needs to be addressed. With it currently set to cover the image is always full-width and will never adjust on the horizontal (x) axis. Which, will cause confusion for authors not being able to adequately adjust the position of the image.

I just had an idea. I think it makes more sense to group the values with the background image - similar to how you are doing it for the Marquee block. The values are directly under the background image and we remove the focalPoint property.

I also think the background image property object-fit: cover needs to be addressed. With it currently set to cover the image is always full-width and will never adjust on the horizontal (x) axis. Which, will cause confusion for authors not being able to adequately adjust the position of the image.

I think that makes sense, and you could perhaps utilize css selectors to address and override the object-fit: cover issue.
So the author just has to put top, left or left, top or whatever, and programmatically we could add whatever class name fits that.
I think what you propose is also what Chris may have meant in this comment on the ticket:

@suhjainadobe I believe this is what we discussed as well, what is the confusion here?

Just from a functional level, the examples in the test doc don't quite seem to be working.
https://focalpoint--milo--suhjainadobe.hlx.page/drafts/suhjain/background-image-test?martech=off
On Chrome on Mac Desktop, at least, the second example to ping the pink dot to the upper-right corner is the only one that really works for me when resizing the page. In  first example, the pink dot in the lower-left generally isn't visible for me. The third example with the centered pink dot also generally is lost offscreen to the left when resizing.

I also think the background image property object-fit: cover needs to be addressed. With it currently set to cover the image is always full-width and will never adjust on the horizontal (x) axis.

@rgclayton  @JasonHowellSlavin : I think object-fit:covers trim the image to maintain the aspect-ratio in different viewports. When we give object-position: left for eg, it will focus towards left and cut the right side image more to keep the aspect-ratio.  It works in all directions in the same way. Also, I will make the changes in the section metadata similar to the marquee to keep it uniform.

@rgclayton @JasonHowellSlavin : I have made the changes for the authoring. Included the marquee block as well here as the functionality for both blocks was the same. I will close #685 as it is being included in this PR only. Kindly review the same.

@echampio-at-adobe: Could you please take a look into it now? There were authoring changes that did not focus the image properly.

Hi. Unfortunately, if you've updated the "after" doc, I still don't see things working.
https://focalpoint--milo--suhjainadobe.hlx.page/drafts/suhjain/background-image-test?martech=off
The pink dot that should remain in the viewport for the very first image happily scrolls off the left side of the window as soon as I resize. It's visible only when the window is almost full-screen on my MacBook. I see the same behavior in Safari and Firefox as well as Chrome.

@echampio-at-adobe: Please check now.I think the points are not authored properly.

Great suggestions @vhargrave. @suhjainadobe, I'll give it a  ✅  once these are implemented.
On the comment above about the left margin on the background cell when on tablet viewport. That looks to me like an intentional style - library. I can talk to the consonant team and address in a subsequent PR if needed.

Since this is a new feature we are marking with the new feature label to broadcast to the community. Please see the PR label guide for prepping this to be sent out once merged.

Also, I verified the 'left, right' string etc. are persisting as text artifact elements in the .background div.
We should probably .remove() any DOM elements that aren't suppose to be in the view.

I agree with Ryan. These should get cleaned up.

@vhargrave @rgclayton @ryanmparrish : yes, the margin was coming because the element was getting added in DOM for marquee. I have removed the element specifically for the marquee and not for section-metadata to keep its format consistent in DOM.

@vhargrave flaky tests 😢 re-ran the workflow all green now


added UserData class with author

makes it possible to perform actions without querying user each time

Note to self: It is very hard to use UserAuthor as a shared field in authorRepository since is it used as Scoped, meaning the value doesn't always persist.

Note to self 2: the changes to RouteName here can still simplify the code.


FIx vRP.getNearestVehicle()

Can get all vehicle type and when player in side vehicle.

Multiple issues:

this fix solves the problem for the Tanker model (you can try to repair/replace it) ?
the naming convention is camelCase
I think the multiples for can be simplified in two
coordB doesn't seem to depend on the loop variable
the ray cast fallback is not sorted, so vehicles may not be "nearest/closest"

print(GetDisplayNameFromVehicleModel(GetEntityModel(vRP.getNearestVehicle({5.0}))):lower())
rhino

Outdated since vRP 2.


How to set size

How i can set max height editor?

if you use to scss, you can write that in your style tag
/deep/ .ql-editor {
max-height: 350px;
overflow-y: auto;
}

<vue-editor :editorToolbar="editorToolBar"   class="vue_edeitor_height" ></vue-editor>

 css / scss
 .vue_edeitor_height{
  height:100px;
  width:200px;
 }

This issue is stale because it has been open for 60 days with no activity.


Error 403 (Forbidden)!! on requestAccessToken()

Hello there!!
Im running into an issue which wasn't a problem before...
The code keeps getting stuck in.
In the Request.php the function handleResponseError()
// Something else went wrong, try to give at least some info
throw new SpotifyWebAPIException($body, $status);
I have dd()-d and checked if im passing everything correctly and it seems like i do.
In my controller keeps pointing to this line of error:
$session->requestAccessToken($code);
where before it i have:
$spotifyAccount = SpotifyAccount::where('id', 1)->first();
$session = new SpotifyWebAPI\Session(
$spotifyAccount->client_id,
$spotifyAccount->client_secret,
$spotifyAccount->redirect_uri
);
$code = $_GET['code'];
i have everything correct as the client_id, client_secret and redirect_uri.
Checking the function of requestAccessToken i realized that you don't pass any header to it which on the Spotify Web Api guides said you must include these:
HEADER PARAMETER | VALUE
Authorization | RequiredBase 64 encoded string that contains the client ID and client secret key. The field must have the
format: Authorization: Basic
Content-Type | RequiredSet to application/x-www-form-urlencoded.
But after adding the missing header i get the same error, which is an Error 403 (Forbidden)!!...
Please could you give me any tip on what im a not seeing :(

Hey!
This sounds strange. Are you getting any more info than just 403? Is the message just "Forbidden" or are you getting something more?
It's a bit strange that Spotify's docs no longer mentions the access token request method used by this library (that code has looked the same for years and everything is working when I try it locally). I did make some updates to follow their new docs and please test it out (it's located in a new branch which you can install with composer require jwilsson/spotify-web-api-php:dev-session-refactor).

Hello there first of all thank you for the fast reply!!
I did update the package from the branch you mentioned but it keeps doing the same...

That looks like an error from Google, very strange. It doesn't look like it's even making its way to Spotify but some Google URL instead. Where are you hosting it? Google Cloud? Have you tried running it locally and see if it works there?

Its being hosted on Linode, since December 8 or 9 i'm having this issue, before that it was working fine.
I can get up to here:

Okey, i was about to rewrite my message because i just tried it like i did for the past month and now it worked...
Yesterday i was doing the php artisan config:clear and php artisan cache:clear after the integration of your new branch but it did not work now i have no clue why it did....
I did not change anything, same computer, same terminal, same code its weird and worrying because something seems unstable and i don't know what :(
Thank you very much again for the response your library is awesome and it helped me to save a ton of time, up until now i didn't experience any problem with it.
Thanks again and you are awesome!
Cheers!

Just so I'm following 😅  everything's working now? If so, awesome! Let's hope it keeps working then!

Yes it is currently working, and the refreshAccessToken is fine, when the expiration time is over it gets the new one so i don't know what happened or why but for now i close this issue.
Thank you!


Only challenge if not already accepted.

@kichalla @HaoK
Fixes music store conflict between cookie auth and windows

Well, I was pretty sure our change would break someone, didn't think it would be ourselves :)

This is pretty specific to having multiple automatic handlers, which we already know is a terrible idea, we just can't disable this one without breaking existing apps using windows auth.

@kichalla please re-enable the associated tests.


Make the latest image optional

In some cases, people don't want to tag the image with  latest, would be great if this was configurable.

In some cases, people don't want to tag the image with latest, would be great if this was configurable.

This is necessary for us as well.


Some issues

Why do you need stdio.h?
Need to check the number of arguments in main - otherwise myVideo may not get initialized.

ty!


Fix support for zero-cut orderings.

Fixes an issue spotted in #284.
Previously, having zero cuts would cause this tool to spit out a blank contraction ordering. This PR adds an initial pass before applying cuts to ensure that zero-cut requests generate an ordering.
One side effect of this is that it is now possible for the auto-ordering tool to stop at zero cuts if adding the first cut fails to improve total cost (although such an occurrence is rare).

I'll include tests in this PR as well - please hold off on review/merge until then.

I'll include tests in this PR as well - please hold off on review/merge until then.

Tests added, ready for review.


Is FeatGraph still work when using mini-batch for gnn training?

the paper said:

FeatGraph generates kernel codes for a specific graph topology (i.e., the adjacency matrix);

if we use mini-batch for GNN training, the subgraph structure corresponding to each batch should be uncertain in this case. Is FeatGraph still work?

FeatGraph achieves its best performance the graph structure is fixed.
When graph topology changes over time (say mini-batch training), FeatGraph still works if we remove the graph partitioning phase and it would harm performance. JIT for mini-batch training is still a open problem.

@yzh119 , thank you for your reply.
In other words, in this case, the optimization strategy obtained through FDS is still effective? And FDS is actually optimized through tvm, right?
In addition, I noticed that DGL 0.6 has been released. Has FeatGraph been used in some models (such as GAT)? Or is there a tutorial on how to use it?

Yes FDS still works.
For DGL 0.6, however, there are some existing bugs in TVM that impeding us from relying on featgraph: https://github.com/apache/tvm/issues/6596, we are still waiting for the landing of TVM's new IR: tir to address the issue.
Actually, we have already used some optimizations proposed featgraph paper (e.g. tree reduction for sddmm) and GAT's speed in DGL 0.6 is much better than DGL 0.4 (when featgraph paper released).

get, thanks again.


⚠ ClamAV has degraded performance

In 7f8b17d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 82 ms

Resolved: ClamAV performance has improved in 601b03a.


report-converter [INFO] No 'lsan' results can be found

Describe the bug
report-converter is saying "No 'lsan' results can be found" when there clearly is a leak detected and in the file
CodeChecker version
[INFO 2024-06-28 11:09] - CodeChecker analyzer version:
---------------------------------------------------------------
Kind                 | Version
---------------------------------------------------------------
Base package version | 6.23.1
Package build date   | 2023-12-14T14:38
Git commit ID (hash) | 2a8fa6e711a4ff591280a79fe8798dee2507d984
Git tag information  | 6.23.1
---------------------------------------------------------------

[INFO 2024-06-28 11:09] - CodeChecker web version:
------------------------------------------------------------------------------
Kind                                | Version
------------------------------------------------------------------------------
Base package version                | 6.23.1
Package build date                  | 2023-12-14T14:38
Git commit ID (hash)                | 2a8fa6e711a4ff591280a79fe8798dee2507d984
Git tag information                 | 6.23.1
Server supported Thrift API version | 6.54
Client Thrift API version           | 6.54
------------------------------------------------------------------------------

To Reproduce
Steps to reproduce the behaviour:

have a text file (test.out) with this content:

==17727==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 7 byte(s) in 1 object(s) allocated from:
    #0 0x49c2ad in __interceptor_malloc /projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:129:3
    #1 0x4ce525 in main files/lsan2.c:5:9

Direct leak of 5 byte(s) in 1 object(s) allocated from:
    #0 0x49c2ad in __interceptor_malloc /projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:129:3
    #1 0x4ce543 in main files/lsan2.c:7:16

SUMMARY: AddressSanitizer: 12 byte(s) leaked in 2 allocation(s).

report-converter -t lsan -o report ./test.out
See error:
[INFO] - No 'lsan' results can be found in '...'

Expected behaviour
report-converter found a leak
Desktop (please complete the following information)

OS: Linux ubuntu22.04 (WSL2)

Sorry, closed accidentally. I didn't mean to close it until the patch is merged.

Oh, thanks for letting me know. I wasn't aware that the report-converter requires access to the source files.

Actually, report-converter doesn't require access to these files, but almost all other CodeChecker commands do. For this reason my first thought was printing a comprehensible error message, why report-converter didn't generate any reports and still abort report generation. I chose this current solution, because this way "CodeChecker parse" can provide some statistics at least. But all other CodeChecker commands will emit an error message.


How to ignore exit code 1 when a process is killed ?

In package.json I've the following line with the purpose to:

start testrpc-sc
run coverage
when coverage is done -> kill the testrpc-sc process

"coverage": "concurrently -k \"node_modules\\.bin\\testrpc-sc --port 8555\" \"node_modules\\.bin\\solidity-coverage\"",

However my npm command fails because testrpc-sc is killed and returns a exit 1.
See logging:

[1] Istanbul coverage reports generated
[1] Cleaning up...
[1] node_modules.bin\solidity-coverage exited with code 0
--> Sending SIGTERM to other processes..
[0] node_modules.bin\testrpc-sc --port 8555 exited with code 1
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>coverage: concurrently --kill-others "node_modules\.bin\testrpc-sc --port 8555" "node_modules\.bin\solidity-coverage"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>coverage script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:

How to let npm know that a Exit status 1 is ok ?

I found a workable solution here: https://stackoverflow.com/questions/26814034/silencing-errors-on-failures-for-npm-run-script
My solution is:

install https://www.npmjs.com/package/cash-true

And full command is now:
"coverage": "concurrently --names \"TestRPC,Coverage\" --kill-others \"node_modules\\.bin\\testrpc-sc --port 8555\" \"node_modules\\.bin\\solidity-coverage\" || true",

Use concurrently --success first, this will make only the exit status of coverage (first child that exited) matter.

I still get a code 1 even with --success first
[1] 09 05 2018 17:56:54.032:WARN [launcher]: ChromeHeadless was not killed in 2000 ms, sending SIGKILL.
[1] npm run test-single-run exited with code 0
--> Sending SIGTERM to other processes..
[0] node ./tools/mock-webapi/mock-webapi.js exited with code 1

I tried various ways for json-server to gracefully receive this signal. Nothing seems to work.
process.on('SIGTERM', function (code) {
    console.log('Handle SIGTERM', process.pid, code);
    exitCode = 0;
    server.close(function () {
        process.exit(0);
    });
});

process.on('SIGKILL', function (code) {
    console.log('SIGKILL received...', code);
    exitCode = 0;
    server.close(function () {
        process.exit(0);
    });
});

process.on('SIGINT', function (code) {
    console.log('SIGINT received...', code);
    exitCode = 0;
    server.close(function () {
        process.exit(0);
    });
});

Finally found a solution to this problem. I wrote a little script that runs the mock webapi and the karma tests as child processes. More details on stackoverflow

Strangely --success first works while -- success last does not.

Hello folks,
Is this still a problem for any of you? If yes, do you mind trying v4 (which has just been released)?
Thanks in advance!

I still see same issue
[1] [19:02:19] I/launcher - 0 instance(s) of WebDriver still running
[1] [19:02:19] I/launcher - chrome #01 passed
[1] [19:02:19] I/launcher - firefox #11 passed
[1] Closing report
[1] ng e2e front-e2e exited with code 0
--> Sending SIGTERM to other processes..
[0] npm run start-mock-api exited with code SIGTERM
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>e2e-ci-updated: `concurrently --kill-others --success first "npm run start-mock-api" "ng e2e front-e2e"`
npm ERR! Exit status 1
npm ERR!

@karlhiramoto what about v4.0.1? 😃

Sorry I ended up just not using concurrently  using a simple bash script to run two  jobs in parallel and check the exit code of the 1st job.

I'm using 4.0.1 and --success last still isn't working for me.

@gustavohenke am i doing anything wrong with the above? I'd really like to use this tool, but i can't see what i'm doing wrong for everything to exit code 1 despite the first process exiting code 0.

Hi @gustavohenke. I'm a bit confused then.
If i'm not doing something wrong, why are the rest of the processes exiting with 1 despite the first process exiting with 0? The code 1's are breaking my pipeline, despite all the tests passing.

why are the rest of the processes exiting with 1 despite the first process exiting with 0?

You are using --kill-others, meaning the processes will be killed when one of them exits.

The code 1's are breaking my pipeline, despite all the tests passing.

Why would this be case? Are you parsing the output of the command?
concurrently itself will have exited with 0, so that means a success, according to the flags passed.

Hi @gustavohenke , I'm having the same problem. I have a test runner and a server, the test runner returns 0, and after stopping the server, concurrently exits with 1.  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>ci:test:integration script.

It seems there's still an error with concurrently.

I can confirm changing my commands around and using --success first instead of last fixed this issue for me. CI now passing wew!

Unfortunately we are not able to switch our commands around. Because the second process needs the first to be running.

I can also confirm that using --success last didn't work for me but switching the order and using --success first did.
This is the command that wasn't working (always non-zero exit despite using --success last, I speculate the exit code is from --kill-others killing the first command):
concurrently --success last --kill-others  --no-color --raw  "npm run test:e2e-serve" "wait-on http-get://localhost:4200 && npm run cypress:cl"

This one works and gives me the exit code from npm run cypress:cl:
concurrently --success first --kill-others  --no-color --raw  "wait-on http-get://localhost:4200 && npm run cypress:cl" "npm run test:e2e-serve"

Can confirm as well - in my case I had to kill other processes when tests succeed.
Concurrently v9.1.0: --success first works, while --success last doesn't.
Simply switching the flag to "first", and putting tests first fixed the issue.


Changes to integrate basis into other applications

I've kept this PR up because it's a good idea. Since you pushed this we've moved the encoder into its own directory, but it's not a lib yet to keep the build simple. Once it's a lib I'll close this PR.


Cleanup on README file (just detail issues)

Coherent usage of the name SMOF
Normalise style of badges
Normalise indentation of code
Fix English
Clarify contact

Codecov Report

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

@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage     79.62%   79.62%
  Complexity      605      605
=========================================
  Files            55       55
  Lines          1674     1674
  Branches        167      167
=========================================
  Hits           1333     1333
  Misses          218      218
  Partials        123      123

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 4c96c14...02b709a. Read the comment docs.

Obrigado pela contribuição @dialex ;)


Add files shorcuts

The directory shortcuts should be on the left of the dialog, and by default should be the first directory:

When I opened it, I was on the root file system and couldn't find my home directory.

The current implementation always defaults to the root dir on the first load, it's something that I want to fix before the release is 100% ready.

Added in 5e723a326c324795813b13129ff7638d56a76243

Added in v0.1.0.


Maven offline mode should be honored

when maven is run in offline mode (-o | --offline) the plugin should not try to perform any network operations. If the drivers are already locally available selenium tests will succeed, if not selenium will fail, but then it can't be helped, as the user requested something impossible.

+1

Released in version 3.2.0


A timeline section where the latest tweets of the people you follow are shown

I mean this should be possible.

This was the feed, which has recently been removed due to constant breakage on Twitter's end, and because I don't want to continue the problems that maintaining and using it cause: https://github.com/jonjomckay/fritter/issues/668#issuecomment-1538230835


R's whitebox::wbt_reclass not taking ";" separator on Mac but will take "," on both Windows and Mac

Hi there,
I have been developing an R package that uses Reclass. In both the User Manual and in the wbt_reclass() documentation, the reclass_vals calls for a "Reclassification triplet values (new value; from value; to less than), e.g. '0.0;0.0;1.0;1.0;1.0;2.0'."
I have found that in Windows, using "0.0;0.0;1.0;1.0;1.0;2.0" in whitebox::wbt_reclass() works.
However in testing the R package using a Mac (on two systems) this did not work.
Following some playing around and a little bit of luck, I found that "0.0,0.0,1.0,1.0,1.0,2.0" works for both Windows and Mac.
To fix this, should the Manual/function documentation change or will commas in reclass_vals be allowed as input to WhiteboxTools going forward?
library(whitebox)
library(raster)

whitebox::wbt_init()
whitebox::wbt_version()

workdir <- tempdir()

dem <- system.file("extdata", "DEM.tif", package="whitebox")

## Test functions
wbt_feature_preserving_smoothing(dem, file.path(workdir, "./smoothed.tif"), filter=9, verbose_mode = TRUE)
wbt_breach_depressions(file.path(workdir, "./smoothed.tif"), file.path(workdir, "./breached.tif"))

## reclass that does not work
wbt_reclass(input = dem, output = file.path(workdir, "reclass.tif"),
            reclass_vals = "1.0;2.0;200.0", assign_mode = FALSE, verbose_mode = TRUE, )

## Error:
## thread 'main' panicked at 'The reclass values string must include triplet values (new value; from value; to less than), e.g. '0.0;0.0;1.0;1.0;1.0;2.0'', src/main.rs:72:21
## note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
## sh: 2.0: command not found
## sh: 200.0: command not found
## Error in system(args2, intern = TRUE) : error in running command

## Reclass that does work
wbt_reclass(input = dem, output = file.path(workdir, "reclass.tif"),
            reclass_vals = "6000,200,400,5000,500,1000", assign_mode = FALSE, verbose_mode = TRUE, )

plot(raster(dem))
plot(raster(file.path(workdir, "reclass.tif")))

Session info for Mac session:
R version 4.0.5 (2021-03-31)
Platform: x86_64-apple-darwin17.0 (64-bit)
Running under: macOS Catalina 10.15.7

Matrix products: default
BLAS:   /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/4.0/Resources/lib/libRlapack.dylib

locale:
[1] en_CA.UTF-8/en_CA.UTF-8/en_CA.UTF-8/C/en_CA.UTF-8/en_CA.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] raster_3.1-5   sp_1.4-2       whitebox_1.4.0

loaded via a namespace (and not attached):
 [1] compiler_4.0.5   assertthat_0.2.1 rgdal_1.5-8      cli_2.0.2
 [5] tools_4.0.5      glue_1.4.1       rstudioapi_0.11  Rcpp_<IP_ADDRESS>
 [9] crayon_1.3.4     fansi_0.4.1      codetools_0.2-18 grid_4.0.5
[13] rlang_0.4.10     lattice_0.20-41

Okay, I think that I've found this error, but I must admit it is a bit of a strange one. I found that if I placed spaces between each of the values in the reclass string, there was a parsing error. This is a simple fix; all I needed to do was to trim the strings (i.e. remove any whitespace) prior to parsing the strings into numerical values. However, what I don't understand is why this would impact Mac and not Windows and why it would be the case with semi-colons and not commas. I should say that I was not able to reproduce this error, but I was not using the R-frontend in testing. Instead I used the WhiteboxTools Runner. I should say that it worked fine on Mac with semicolons using the WhiteboxTools Runner, but with no spaces. Anyhow, I've pushed the change now that deals with the parsing error and it will appear in v.1.5, which will be released either today or tomorrow. Please feel free to re-open if this issue persists even after this change.


Update ConsoleWrites

_WD_Startup was using __WD_ConsoleWrite("_WDStartup... instead of $sFuncName
Updated for accuracy and consistency

Thanks for catching that!


[WIP]: Use socks5 library for socks layer

Separate the socks5 protocol implementation into another library.
Waiting for socks5 publish to pypi.
TODO:

[ ] Update unittest.

Coverage decreased (-0.9%) to 58.608% when pulling 9f0c684770aab8464b13a55a820df1095dc3e33e on socks5-refactor into 1e4160307dcfa79b24fbf35299e5aeabdafde254 on master.

Coverage decreased (-0.4%) to 59.074% when pulling 4b1d9fd5c058d4788c653a9aad1360286dab3409 on socks5-refactor into 1e4160307dcfa79b24fbf35299e5aeabdafde254 on master.


Transaction Bounce

Hello, How to avoid bounce transaction to new address?  Thanks

Hi, you should use bounceable: false Address flag for this.
https://github.com/tonstack/ton3-core/blob/main/docs/classes/Address.md#tostring


Plugin List, include Version

It would be great if the plugin list would report the version, too :)

this was actualy already there, you just needed to click the # to expand it. but im adding it to the same row


publish and unpublish product from one sheet

Support reference number
https://jira.commercetools.com/browse/SUPPORT-6584

Support ticket closed as customer handled it by themselves.


js_dsl doesn't work with v8 example

Hi,
This small change fixes the v8 example for JS support. Running the example as-is with cucumber-0.8.5 and therubyracer-0.7.5 produces:
Object #<a rb::JsLanguage> has no method 'registerJsHook' (V8::JSError)
at Object.__registerJsHook (/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/js_support/js_dsl.js:43:16)
at /Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/js_support/js_dsl.js:16:19
at features/step_definitions/fib_steps.js:1:1
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/js_support/js_language.rb:28:in `send'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/js_support/js_language.rb:28:in `method_missing'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/js_support/js_language.rb:114:in `load_code_file'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/step_mother.rb:108:in `load_code_file'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/step_mother.rb:100:in `load_code_files'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/step_mother.rb:99:in `each'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/step_mother.rb:99:in `load_code_files'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/cli/main.rb:56:in `execute!'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/../lib/cucumber/cli/main.rb:25:in `execute'
/Library/Ruby/Gems/1.8/gems/cucumber-0.8.5/bin/cucumber:8
/usr/bin/cucumber:19:in `load'
/usr/bin/cucumber:19

The CamelCase to camel_case changes I made to the js_dsl.js file solve this problem, and the fibonacci example runs flawlessly after these changes are applied.
Best,
Clay

This is obsolete - there is better js support in cucumber-js.


Zoom G1on doesn't turn on anymore

After choose the effects that I swapped, during the writing process on my Zoom G1on, my computer stuck. When I removed the USB cable the G1on turned off, and now it doesn't work anymore, i tried usb cable and power supply.
What I need to do now? Did I lost my Zoom? Or there is a way of rewrite the firmware code on the Zoom Bios Chip? Would the kit shown in the image below work?

What about the Zoom Gon firmware code, where can I get it?

I have been reading around these pedals - found this that might help you?
https://www.thegearpage.net/board/index.php?threads/the-zoom-hack-cdr-effects-and-others-on-g1on-g1xon.2081936/page-3
Read around the thread. Dudes uses this and similar programs. Didnt check there was enough space. Broke the pedal. Wouldnt accept normal firmware. Well read on how someone suggests how they used CH341 device to fix it again.

I have been reading around these pedals - found this that might help you?
https://www.thegearpage.net/board/index.php?threads/the-zoom-hack-cdr-effects-and-others-on-g1on-g1xon.2081936/page-3
Read around the thread. Dudes uses this and similar programs. Didnt check there was enough space. Broke the pedal. Wouldnt accept normal firmware. Well read on how someone suggests how they used CH341 device to fix it again.


[YUNIKORN-2958] Display foreign allocations on the web UI

What is this PR for?
Display foreign allocations on the Nodes page.
Notes:

The PR fetches both YK & non-YK allocations and combine them in a single list
New "type" field is added
An icon is rendered in a separate "type" column based on whether it's a Yunikorn or foreign pod.

What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[x] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2958
How should this be tested?
Screenshots (if appropriate)

Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

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

Project coverage is 35.71%. Comparing base (b877df7) to head (1311206).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #222   +/-   ##
=======================================
  Coverage   35.71%   35.71%
=======================================
  Files           2        2
  Lines          56       56
=======================================
  Hits           20       20
  Misses         33       33
  Partials        3        3

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

I decided not to use icons in the "Type" column. I've been unable to find a nice pair of arrow icons. Sometimes copyright is an issue because it's too prohibitive. I was thinking about calling it "Scheduled by Yunikorn", with a checkmark indicating a YK pod and an external link indicating a foreign pod, but ultimately it's just too much hassle. I've already spent hours with it.
Let's use a text field, this can be improved any time later on.

Rendered pages with the latest commit

I don't think we want to show all the attributes unless we're in expanded view, otherwise this is going to get really ugly with lots of nodes. In fact, I'd opt to hide the attributes column entirely unless we're expanded. It's just not relevant 99% of the time.

Created https://issues.apache.org/jira/browse/YUNIKORN-2970. On my browser, it's displayed by default, but after clicking on "..." twice it disappears. This makes me think that it's supposed to be displayed only in enhanced mode.

I think we should just remove the 'type' column. YuniKorn allocations can be easily discerned by the presence of the Application ID field.

I think we should just remove the 'type' column. YuniKorn allocations can be easily discerned by the presence of the Application ID field.

Makes sense. I'll catch up with Wilfred tomorrow morning. This would further simplify the changes.


Add parcel

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

Project coverage is 100.00%. Comparing base (01b7a00) to head (236c5e5).
Report is 26 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           55        55
  Branches         6         6
=========================================
  Hits            55        55

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


Above the fold content issue when using LQIP in picture tag

Hello. I am trying to use this lazy load package and noticed issue with it in my app.
Before I start using it I passed all test, like Pagespeed or varvy.com (actually, got 100/100)

After I enable lazysizes I begin to receive this error on both tests!

Package by itself is working fine (as it seems to me). But why do I receive this errors? Any ideas how to fix it?

Probably you should not lazyload your first 1-3 images because they are in view and the preload parser is faster than JS can start downloading these.

No, this is not the case. After playing around I found out that setting src on img in picture tag is causing the issue. Also, I experience weird behavior, my LQIP are loading and cancelling on every page load when route.
<picture>
            <source :data-srcset="`
              ${$store.state.baseUrl}/static/images/cdn/320x240-${article.image.url.webp} 320w,
              ${$store.state.baseUrl}/static/images/cdn/560x420-${article.image.url.webp} 560w
            `" sizes="(min-width: 768px) 33.3vw, 100vw" type="image/webp">
            <img :src="${$store.state.baseUrl}/static/images/previews/sqip-1040x780-${article.image.sqip.url}" :data-src="`${$store.state.baseUrl}/static/560x420-${article.image.url.jpegpng}`" :data-srcset="`
              ${$store.state.baseUrl}/static/images/cdn/320x240-${article.image.url.jpegpng} 320w,
              ${$store.state.baseUrl}/static/images/cdn/560x420-${article.image.url.jpegpng} 560w
            `" sizes="(min-width: 768px) 33.3vw, 100vw" :alt="article.image.alt" class="metro-block-card-image lazyload">
          </picture>

If I remove LQIP image :src="${$store.state.baseUrl}/static/images/previews/sqip-1040x780-${article.image.sqip.url}" then I have NO ERROR. How to fix this? Am I using markup wrong or this is the issue?

My point is I have no error in test with lazyload until I begin to use LQIP in my picture tag. I feel that this is wrong behavior @aFarkas
Also reloading and cancelling of LQIP seems wrong.

Please reopen this issue, sir.
Or please assist me with right markup if you think it's my fault @aFarkas
Or mention in Docs that you can't use LQIP in picture tag.

Setting src on picture img tag also cause this https://github.com/aFarkas/lazysizes/issues/514
If I remove src no issues remains- neither with above the fold content, no with duplicate downloading of LQIP.

@canyoudev
You are using pagespeed. This is not a human it is a machine that does some tests. Some of these tests are not always revealing a problem with your page but with the algorithm used.
I just see that your lqip is pretty big (1040x780). You should use lower quality image.
You probably can trick pagespeed by using the blur up plugin, which is is highly recommended.

I will try plugin, but the thing is in this case I am not blurring LQIP (it's polygonal SVG instead of blurred image)
So the file size is equal for any resolution, it's SVG. 1040x780 or 100x100 make no difference.
I understand that this is a machine test, but still it's kinda cool to show green results to the client, you know...

So the file size is equal for any resolution, it's SVG. 1040x780 or 100x100 make no difference.

Ok, but the image in srcset are webp or jpeg and much bigger? Maybe just do an overfade? To be honest I don't know how this looks. If this placeholder doesn't look good. You should remove it entirely. It's something to satisfy the user until the real image is loaded.


Additional Breaking Changes for Dart 3.0

I believe Dart 3.0 has the opportunity to make more breaking changes than just only disallowing non-null-safe code.
For example, I have seen in several issues that the dart language team has said, 'if we had null-safety to start with, we would have designed this api differently'.
Feel free to edit or add additional breaking changes to consider / discuss, but here are a few I can think of.

[ ] collections with StateError list.first, list.last, list.firstWhere -> change return type to nullable. Migrate existing usages to List.first.. ?? throw ... This has the benefit of making it much more explicit to the user where their code can throw. I know I've been bitten by these methods in the past.
[ ] MapEntry -> typedef MapEntry<K,V> = ({K key, V value}) using the Records feature which seems to be progressing quickly. This only breaks implementors/extenders of MapEntry I believe unless someone is relying on identity of the MapEntry object. Searching Github for "implements MapEntry" I see 95 usages in Dart code. (https://github.com/search?l=Dart&p=7&q="implements+MapEntry"&type=Code) Most are copies of the LruCache's _Link from package:build or SplayTreeMapNode it looks like, but there are a few which basically just expose another name for the key or value, these can be migrated to extension methods or 'Views'. I'm not sure how to handle the migration of _Link or SplayTreeMapNode admittedly.
[ ] Add .indexed getter on List (could just be an extension method?) that returns a Record (int index, T value)

Platform library changes are very difficult to pull off. Worst case, we need to migrate the world at the same time that we make the change. APIs do not have language versioning, so we can't change behavior based on the language version of the code calling the method.
Convering MapEntry to a record (or maybe a #struct) is something we have considered. I'd probably prefer the struct (or view on record) so that it has its own type.
The .indexed getter on List would definitely have to be an extension, and then the only thing holding anyone back from doing it today is the lack of a record type to use for the value. (But .asMap().entries is an Iterable<MapEntry<int, T>>, so maybe that's just what you want).

The MapEntry idea is awesome. Whatever you decide, I'm in.

The problem with making MapEntry a typedef of a record type is that it won't have the constructor that people are currently using. You can't do MapEntry(key, value) any more.
A view on a record could probably have a constructor, and the struct design would be the closest thing to the current class, while still not requiring a persistent identity. So, those are more likely to succeed, since the migration will be much smaller.

Yeah, makes sense. I'm not familiar with the view, if it works, perfect. If it doesn't, dart fix wouldn't be hard.

As much as I want union types (personal favorite request), I don't think it will arrive before views/other things.

Variable number of arguments varargs.
From package:path changelog:
1.8.3: Support up to 16 arguments in join function and up to 15 arguments in absolute function.

If I could ask for anything crazy in Dart 3, I would go with print("value is", a). I alwaaays struggle with Dart syntax, and it is possible to implement that without varargs. Since print is not supposed to run in release mode, the minimal performance impact is negligible.
https://github.com/dart-lang/sdk/issues/49084

@TimWhiting
well, the ClojureDart compiler does use MapEntry and while migrating to Dart 3 we're going to loose this interop btw Dart ME and clojure ME.

These were just proposed changes. I'm not part of the dart team, and the map entry change is unlikely to happen at least for dart 3.

The MapEntry class was made final in the Dart 3 platform libraries, which means it cannot be subclassed by other 3.0 libraries.
So that has happened.
It won't break existing 2.19 code, but it may require some rewriting when upgrading to language version 3.0.


Support Include / Exclude Options

Allow users to include or exclude files to look at to extract snippets from

Rather than specifically exposing include and exclude I went the route of allowing users to specify a list of glob patterns to determine which files/directories snippext should extract snippets from. Globs are also used to determine target files that snippet should splice snippets into.


mediaDevices is not found

Error: navigator.mediaDevices is undefined. It seems the current document is not loaded securely.

Does this help? https://dev.to/mute/comment/1gj25

It works properly when the serverAddress is set to localhost in the config.toml file.
However it throws the above error as mentioned by @whitphx when it is made to run on network url.

It's normal. Did you read https://dev.to/whitphx/build-a-web-based-real-time-computer-vision-app-with-streamlit-57l2#https as linked from the page above?

@murtazabasu Thank you.

@whitphx I also meet this error, I also use https but the website just loading. Can you explain for me?


test_mongo_compute.test_compute_on_db uses all my ram

This test uses all of my ram (8GB), it spills into swap and then my computer locks up.
I setup an empty mongo for this to use and ran the admin command that is in the .travis.yml. I checked the developer workflow page; however, it didn't have a section about setting up the test database.
$ mongod --version
db version v2.6.8
2015-04-22T15:43:52.293-0400 git version: nogitversion
2015-04-22T15:43:52.293-0400 OpenSSL version: OpenSSL 1.0.2 22 Jan 2015

I am on Arch GNU+Linux

@llllllllll what version of pymongo are you using?

pymongo==3.0.1

This was happening to me a few days ago. But I didn't really give the issue due diligence because I was in "fix other stuff" mode. I was on debian testing, and I think I fixed it by upgrading to a more recent version of mongodb than what was in the repos.

@llllllllll can you try a more recent version of mongodb? Just want to see if it goes away with that.

$ mongod --version
db version v3.0.2
git version: nogitversion
OpenSSL version: OpenSSL 1.0.2 22 Jan 2015

same issue

ok cool, i'll take a tlook

Thanks

This is no longer happening, we can re-open if someone else has this issue.
I have upgraded a bunch of packages since this happened so something must have been out of date.

thanks!


Additional highway shields in Arkansas

So I brought these up in a discord server I'm in, and I was referred to bring these up to this board. Some of you may find these interesting...
Private roads in Washington County, Arkansas, mainly for short driveways

Private roads in Johnson County, Arkansas, similar to Washington County

County roads in Lee County, Arkansas

Marine Fuel Tax (or 900- series) roads in Arkansas

Airport roads in Arkansas (any route designated AR 980)

Private drives in Washington County appear to be tagged as name_1=County Road * from TIGER, even though they aren’t county roads. Some of Johnson County’s private roads are tagged as name_1=Private Road * while others are name_1=County Road *. In [changesets 156,548,573] and 156,548,815, I tentatively went with network=US:AR:Washington:Residential and network=US:AR:Johnson:Private, respectively, for some that I could see in Bing Streetside imagery. If you happen to know of any others without consulting Google Maps (🙅), then please feel free to retag them similarly in OSM.
We have Lee County:
https://github.com/osm-americana/openstreetmap-americana/blob/88c72adf4bc8b697ebe2c264b368b0e4d497ae62/src/js/shield_defs.js#L947-L953
Airport roads have a special carveout:
https://github.com/osm-americana/openstreetmap-americana/blob/88c72adf4bc8b697ebe2c264b368b0e4d497ae62/src/js/shield_defs.js#L903-L909
Marine Fuel Tax roads are covered by #375.

Good catch, @TheInstrumentalist. If you can find any more examples of private roads in Arkansas with county-assigned numbers, please share them here.

 It does have its unique shield as well.


pylance loses link to a object if it was returned by fixture

 Environment data

Language Server version: 2024.4.1
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda):  python3.8 , pip23.2.1,
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet
class Cl_1:
    def __init__(self):
       pass
    def some_f1:(self):
       pass

class Cl_2:
    def __init__(self):
         self.obj1=Cl_1()

     def some_f2:(self):
       pass

@fixture
def fixt_1():
    yield Cl_2()

@fixture
def fixt_2(fixt_1):
     yield fixt_1

def test_some_test(fixt_2):
      fixt_2.some_f2()
      fixt_2.obj1.some_f1

 Repro Steps

XXX

 Expected behavior
Test run ok. When I press CTRL+ left mouse click in any of the :
fixt_2.some_f2()
fixt_2.obj1.some_f1()
I jump into declaration of the appropriate function
 Actual behavior
Test runs ok.
Inside my test:
(function) some_f2: Unknown
(function) obj1: Unknown
 Logs
vscode_log.txt
XXX

Requirments

[requirements.txt](https://github.com/microsoft/pylance-release/files/14846468/requirements.txt)
XXX

Hello, is there any movement in this investigation?


Prefer towncrier.toml if present

Closes #81

Extract helper function for finding the config file.
Refactor _find_config_file for simplicity.
Refactor to generalize the search across a list of two candidates.
Re-order search path for config files to match towncrier.

I welcome others to fix the lint errors, because I have no opinion on how to address them.


Adding ZNCZ04LM - Xiaomi Mi power plug ZigBee EU (Router)

Hi I was wondering if this product of Xiaomi could be added
4: 0x04cf8cdf3c75a05d (0x04cf8cdf3c75a05d): ZNCZ04LM - Xiaomi Mi power plug ZigBee EU (Router)
payload '{"state":"ON","linkquality":70,"power":2.79}'
I dont know if I provided enough information I am not very good with this and don't know all the ssh commands to receive the correct data you need.
Any questions please let me know
Greetz,
Android

Done! Could you please update plugin and check

Hey Stas-Demydiuk,
Thank you for adding but I do have a few questions
I'll put my log underneath here
Feb 02 16:48:58 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:48:58: Successfully interviewed '0x04cf8cdf3c75a05d', device has successfully been paired
Feb 02 16:48:58 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:48:58: Device '0x04cf8cdf3c75a05d' is supported, identified as: Xiaomi Mi power plug ZigBee EU (ZNCZ04LM)
Feb 02 16:48:58 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:48:58: MQTT publish: topic 'zigbee2mqtt/bridge/log', payload '{"type":"pairing","message":"interview_successful","meta":{"friendly_name":"0x04cf8cdf3c75a05d","model":"ZNCZ04LM","vendor":"Xiaomi","description":"Mi power plug ZigBee EU","supported":true}}'
Feb 02 16:49:02 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:49:02: MQTT publish: topic 'zigbee2mqtt/0x04cf8cdf3c75a05d', payload '{"state":"OFF","linkquality":78,"power":0}'
Feb 02 16:50:13 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:50:13: MQTT publish: topic 'zigbee2mqtt/0x04cf8cdf3c75a05d', payload '{"state":"ON","linkquality":78,"power":0}'
Feb 02 16:50:30 raspberrypi npm[1626]: zigbee2mqtt:info  2020-02-02 16:50:30: MQTT publish: topic 'zigbee2mqtt/0x04cf8cdf3c75a05d', payload '{"state":"ON","linkquality":78,"power":3.94}'

"Power" should be an actual usage at that very moment  in Watts
It now only shows the total usage as long you have your device.
Maybe it could be an extra option I'll put a screenshot underneath of a neo coolcam z wave power plug for reference.
Any questions please let me know

And it doesn't send updates every couple of minutes.

Thank you for all your efforts

If you open Utility page, you should see both values, smth like this

Your absolutely right Stas! Thanks it works as a charm.


Swapped to_seq_len/from_seq_len in comment

I'm pretty sure this comment:
https://github.com/huggingface/pytorch-pretrained-BERT/blob/2c5d993ba48841575d9c58f0754bca00b288431c/modeling.py#L339-L343
should instead say:
# Sizes are [batch_size, 1, 1, to_seq_length]
# So we can broadcast to [batch_size, num_heads, from_seq_length, to_seq_length]

When masking out tokens for attention, it doesn't matter what happens to attention from padding tokens, only that there is no attention to padding tokens.
I don't believe the code is doing what the comment currently suggests because that would be an implementation flaw.

Yes! fixed the comment


FEATURE: freshness

There is no freshness in the adapter but default macros works fine.
Please, enable it.

Code has been merged in dbt-vertica 1.4.0.


size of text area is not correct

sometimes too small

and maybe make it easier to take screenshot like google calendar (a view mode?) should consider edit more frequent or screenshot and share more frequent


Refactor code to use generally available syntax

There are parts of the code within the research files, which cause the code to only work in Firefox. These parts should be replaced by JavaScript syntax supported by all browsers, so the extension may be ported to them in the future.
This should mainly relate to the use of the let keyword and the usage of the JavaScript version number within the <script> tag.

Fixed this through the following commits:
https://github.com/SebastianZ/TRex/commit/f4d6de8ba76a805f8e3a836fb725fc8f02277a4c
https://github.com/SebastianZ/TRex/commit/15723b4b7d966987e9948f8ede2c0f0304b569db
https://github.com/SebastianZ/TRex/commit/02669b2ceebf90f0cb3a86c0b9896500f85b2e6c
https://github.com/SebastianZ/TRex/commit/435ecac78cb5e5d9736c39e23bcb06c13677a03e
Sebastian


added instructions for windows users

This is a fix for https://github.com/Microsoft/vscode-recipes/issues/107
The current jest configuration doesn't actually work on windows (it's a weird VS code launch problem, running ./node_modules/.bin/jest in a powershell in VS code does work).
So I

added a workaround configuration for windows users
added a comment for windows users using jest-cli instead of jest so they know how to edit the config to make it work

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

ping @auchenberg whenever you have a chance 🙂

LGTM


pip3 install --upgrade cid-cmd fails

Hi.
with reference to issue  https://github.com/aws-samples/aws-cudos-framework-deployment/issues/292
is this actually fixed?  I see the issue is closed, however, I still get the same error, and the version of cid-cmd is 0.1.30 Beta
What also happens is that the pip3 install --upgrade cid-cmd fails with the following stacktrace
[cloudshell-user@ip- ~]$ pip install --upgrade cid-cmd
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: cid-cmd in ./.local/lib/python3.7/site-packages (0.1.30)
Requirement already satisfied: six>=1.15 in ./.local/lib/python3.7/site-packages (from cid-cmd) (1.16.0)
Requirement already satisfied: questionary>=1.10 in ./.local/lib/python3.7/site-packages (from cid-cmd) (1.10.0)
Requirement already satisfied: boto3>=1.19 in /usr/local/lib/python3.7/site-packages (from cid-cmd) (1.24.9)
Requirement already satisfied: setuptools in /usr/lib/python3.7/site-packages (from cid-cmd) (49.1.3)
Requirement already satisfied: Click>=8.0 in ./.local/lib/python3.7/site-packages (from cid-cmd) (8.1.3)
Requirement already satisfied: deepmerge in ./.local/lib/python3.7/site-packages (from cid-cmd) (1.0.1)
Requirement already satisfied: PyYAML in /usr/local/lib64/python3.7/site-packages (from cid-cmd) (5.4.1)
Requirement already satisfied: yaspin==0.14.1 in ./.local/lib/python3.7/site-packages (from cid-cmd) (0.14.1)
Requirement already satisfied: requests in /usr/local/lib/python3.7/site-packages (from cid-cmd) (2.25.1)
Requirement already satisfied: s3transfer<0.7.0,>=0.6.0 in /usr/local/lib/python3.7/site-packages (from boto3>=1.19->cid-cmd) (0.6.0)
Requirement already satisfied: jmespath<2.0.0,>=0.7.1 in /usr/local/lib/python3.7/site-packages (from boto3>=1.19->cid-cmd) (0.10.0)
Requirement already satisfied: botocore<1.28.0,>=1.27.9 in ./.local/lib/python3.7/site-packages (from boto3>=1.19->cid-cmd) (1.27.48)
Requirement already satisfied: importlib-metadata in /usr/local/lib/python3.7/site-packages (from Click>=8.0->cid-cmd) (4.11.4)
Requirement already satisfied: prompt_toolkit<4.0,>=2.0 in ./.local/lib/python3.7/site-packages (from questionary>=1.10->cid-cmd) (3.0.30)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/local/lib/python3.7/site-packages (from requests->cid-cmd) (1.26.9)
Requirement already satisfied: chardet<5,>=3.0.2 in /usr/local/lib/python3.7/site-packages (from requests->cid-cmd) (4.0.0)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.7/site-packages (from requests->cid-cmd) (2<IP_ADDRESS>)
Requirement already satisfied: idna<3,>=2.5 in /usr/local/lib/python3.7/site-packages (from requests->cid-cmd) (2.10)
Requirement already satisfied: python-dateutil<3.0.0,>=2.1 in /usr/local/lib/python3.7/site-packages (from botocore<1.28.0,>=1.27.9->boto3>=1.19->cid-cmd) (2.8.2)
Requirement already satisfied: wcwidth in /usr/local/lib/python3.7/site-packages (from prompt_toolkit<4.0,>=2.0->questionary>=1.10->cid-cmd) (0.1.9)
Requirement already satisfied: typing-extensions>=3.6.4 in /usr/local/lib/python3.7/site-packages (from importlib-metadata->Click>=8.0->cid-cmd) (<IP_ADDRESS>)
Requirement already satisfied: zipp>=0.5 in /usr/local/lib/python3.7/site-packages (from importlib-metadata->Click>=8.0->cid-cmd) (3.8.0)
--- Logging error ---
Traceback (most recent call last):
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/utils/logging.py", line 177, in emit
    self.console.print(renderable, overflow="ignore", crop=False, style=style)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_vendor/rich/console.py", line 1752, in print
    extend(render(renderable, render_options))
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_vendor/rich/console.py", line 1390, in render
    for render_output in iter_render:
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/utils/logging.py", line 134, in __rich_console__
    for line in lines:
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_vendor/rich/segment.py", line 245, in split_lines
    for segment in segments:
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_vendor/rich/console.py", line 1368, in render
    renderable = rich_cast(renderable)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_vendor/rich/protocol.py", line 36, in rich_cast
    renderable = cast_method()
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/self_outdated_check.py", line 130, in __rich__
    pip_cmd = get_best_invocation_for_this_pip()
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/utils/entrypoints.py", line 60, in get_best_invocation_for_this_pip
    os.path.join(binary_prefix, exe_name),
  File "/usr/lib64/python3.7/genericpath.py", line 101, in samefile
    s2 = os.stat(f2)
FileNotFoundError: [Errno 2] No such file or directory: '/usr/bin/pip'
Call stack:
  File "/home/cloudshell-user/.local/bin/pip", line 8, in <module>
    sys.exit(main())
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/cli/main.py", line 70, in main
    return command.main(cmd_args)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/cli/base_command.py", line 101, in main
    return self._main(args)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/cli/base_command.py", line 223, in _main
    self.handle_pip_version_check(options)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/cli/req_command.py", line 148, in handle_pip_version_check
    pip_self_version_check(session, options)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/self_outdated_check.py", line 237, in pip_self_version_check
    logger.info("[present-rich] %s", upgrade_prompt)
  File "/usr/lib64/python3.7/logging/__init__.py", line 1378, in info
    self._log(INFO, msg, args, **kwargs)
  File "/usr/lib64/python3.7/logging/__init__.py", line 1514, in _log
    self.handle(record)
  File "/usr/lib64/python3.7/logging/__init__.py", line 1524, in handle
    self.callHandlers(record)
  File "/usr/lib64/python3.7/logging/__init__.py", line 1586, in callHandlers
    hdlr.handle(record)
  File "/usr/lib64/python3.7/logging/__init__.py", line 894, in handle
    self.emit(record)
  File "/home/cloudshell-user/.local/lib/python3.7/site-packages/pip/_internal/utils/logging.py", line 179, in emit
    self.handleError(record)
Message: '[present-rich] %s'
Arguments: (UpgradePrompt(old='22.1.2', new='22.2.2'),)
[cloudshell-user@~]$

So if a fix is actually released, I'm not able to get it as the update command fails

The error says you dont have pip binary. Make sure to install pip and use preferred command - python3 -m pip install -U cid-cmd

Using that command does not work either, same error.
But running sudo yum install pip before upgrading cid-cmd solves it.
Documentation should probably be updated to reflect this.
So thanks, works now


Report an error if compilation during testing times out.

Travis analysis is failing:

[lint] Avoid using braces in interpolation when not needed. (packages/flutter_tools/lib/src/test/flutter_platform.dart, line 193, col 34)


Implemented enhancement of Poke API

🎯 Title and Issue Number

Closes #118
I have shared some SS of the application:
Search Page:

Pokemon-Details Page:

Error handling:

🗒 Checklist:

[] I have mentioned the issue number in my Pull Request.
[]  I have gone through rules of when PR will get merged.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md
[] I have followed proper file structure.
[] I have gone through CONTRIBUTING GUIDELINES.md.

Thank you for contributing!❤
Don't forget to show some love by ⭐ the repository!

Hey @its-kritika, Thank you for creating PR! We will review it super soon. 😇❣
As we continue to evolve and innovate, we encourage you to share your ideas, feedback, and code to help us push the boundaries of what's possible in the world of APIs🚀.
We hope you have followed CONTRIBUTING GUIDELINES.
Thank you for being an essential part of our community. Together, we'll keep building amazing things!
Don’t forget to ⭐ our repository!
Happy Coding!✨
Will See you soon❣

@dishamodi0910 Can you please review this PR

@its-kritika We're thrilled to inform you that your pull request has been successfully merged into the APIverse🚀🎁
Your hard work, dedication, and expertise have made a significant impact on our project, and we couldn't be more grateful for your contributions.😇
Your code will now be a part of our growing ecosystem, helping developers worldwide to create exceptional APIs.❣🔥
Happy Coding! ✨
Will See you soon❣


npm start error

Z-nomp was working completely fine, then I shut down my vps for a while and came back to try running it again to only receive errors.
When running "npm start" command the console returns the following:
><EMAIL_ADDRESS>start /home/ogminer/z-nomp
> LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$PWD/node_modules/stratum-pool/node_modules/equihashverify/build/Release/:$PWD/node_modules/equihashverify/build/Release/ node init.js

undefined:1
{"logLevel":"debug","logColors":true,"cliPort":17117,"clustering":{"enabled":true,"forks":"auto"},"defaultPoolConfigs":{"blockRefreshInterval":500,"jobRebroadcastTimeout":55,"connectionTimeout":600,"emitInvalidBlockHashes":false,"validateWorkerUsername":true,"tcpProxyProtocol":false,"banning":{"enabled":true,"time":600,"invalidPercent":50,"checkThreshold":500,"purgeInterval":300},"redis":{"host":"<IP_ADDRESS>","port":6379,"password":""}},"website":{"enabled":true,"host":"<IP_ADDRESS>","port":8080,"stratumHost":cryppit.com,"stats":{"updateInterval":30,"historicalRetention":14400,"hashrateWindow":300},"adminCenter":{"enabled":false,"password":"password"},"tlsOptions":{"enabled":false,"cert":"","key":""}},"redis":{"host":"<IP_ADDRESS>","port":6379},"switching":{"switch1":{"enabled":false,"algorithm":"sha256","ports":{"3333":{"diff":.125"varDiff":{"minDiff":4,"maxDiff":512,"targetTime":15,"retargetTime":90,"variancePercent":30}}}},"switch2":{"enabled":false,"algorithm":"scrypt","ports":{"4444":{"diff":10,"

SyntaxError: Unexpected token c in JSON at position 512
    at JSON.parse (<anonymous>)
    at Object.<anonymous> (/home/ogminer/z-nomp/init.js:27:25)
    at Module._compile (module.js:660:30)
    at Object.Module._extensions..js (module.js:671:10)
    at Module.load (module.js:573:32)
    at tryModuleLoad (module.js:513:12)
    at Function.Module._load (module.js:505:3)
    at Function.Module.runMain (module.js:701:10)
    at startup (bootstrap_node.js:193:16)
    at bootstrap_node.js:617:3
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$PWD/node_modules/stratum-pool/node_modules/equihashverify/build/Release/:$PWD/node_modules/equihashverify/build/Release/ node init.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ogminer/.npm/_logs/2018-01-18T22_54_21_877Z-debug.log

I have already tried the following:

git pull
npm update/install
removing node_module and then doing npm update/install
removing package-lock.json and then doing update/install again
(edit) re-downloaded the repository and started fresh, still no luck

I can verify that my redis_server is running correctly and the blockchain is updated. Any ideas would be appreciated, thank you!

The error states cleary that you have an incorrect configuration. If you run your config into a json linter, you will get the same error.
The error is here:
"stratumHost": cryppit.com
It is basically expecting a string.

Try this steps from super user:

rm -r node_modules
npm update
npm install

Upgrade you system to use node version 8 or later

fix you config.json
The error is here:
change this
"stratumHost": cryppit.com
to this:
"stratumHost": "cryppit.com"


IOS Notification issue : Platform application is disabled

Running Home Assistant 0.69.1
Current status : no notifications are working
Notify component is down, we are seeing "Platform application is disabled" message.

I experience the same. Same version.

I'm also experiencing this after updating to 0.69.1.

I'm experiencing this as well - I'm still on 65.3.

Same too here!

I am on 69.1 docker on unraid with same issue

@robbiet480 @fabaff , Can you please help check?

As far as I can tell is ios-push.home-assistant.io working.

i read that its an issue with .p12 cert on aws sns, most users are getting this error.
https://stackoverflow.com/questions/42872244/getting-platform-application-is-disabled-from-amazon-sns
<?xml version="1.0" encoding="UTF-8"?>
<ErrorResponse xmlns="http://sns.amazonaws.com/doc/2010-03-31/">
   <Error>
      <Type>Sender</Type>
      PlatformApplicationDisabled
      <Message>Platform application is disabled</Message>
   </Error>
</ErrorResponse>

https://stackoverflow.com/questions/42872244/getting-platform-application-is-disabled-from-amazon-sns
has a fix

Certificate updated! Someone try sending a notification and let me know if you're still having issues.

Just sent notification and it worked. Thank you

Yep this is fixed now 🤝thanks again @robbiet480


Potential regression in RSpec 3.1 or 3.2: line number filter ignored when tag passed as well

As reported by @reconstructions in rspec/rspec-rails#1326:

If you specify a tag ( --tag footag ) AND a line number in your rSpec call using version 3.2, the line number is ignored, which was not the behavior in rSpec 3.0.

I haven't created a repro case yet but wanted to open this so we don't lose track.

Here are the exact strings I am using:
Runs selected lines;
'operating_system='' browser='firefox' browser_version='' rspec --require ./spec/selenium/local_config.rb --order random  spec/selenium/spec/band_spec.rb:51 -fd'
=>include {:locations=>{"./spec/selenium/spec/band_spec.rb"=>[51]}

Runs all lines:
 'operating_system='' browser='firefox' browser_version='' rspec --require ./spec/selenium/local_config.rb --order random --tag selenium spec/selenium/spec/band_spec.rb:51 -fd'
=> include {:selenium=>true, :locations=>{"./spec/selenium/spec/band_spec.rb"=>[51]}}

I've put together a repro case in a gist:
https://gist.github.com/myronmarston/9c327730915a113b7b9d

I used git bisect to discover that the source of the regression is 35137e0c9691f75fd9ad1aa7627698fbaba6c4f1 from #1841.  Not sure on what a fix will be yet.


HTTP Status 500 – Internal Server Error

Getting an Internal Server Error when making request to the API.

Fixed.

I think the first link to get the username and ID does not seem to be working. When you go to this link you would still get a 500 error.
Is there any alternative way to get the ID you would suggest?

I've fixed the API again. If it does not work either. You could use _https://www.instagram.com/{username}/?__a=1_ to find the id. You will get the id in graphql.user.id node.

Cheers man! Working now :)


Add ParseJobConfig to the BPMConfig

This change adds a convenience method to the BPMConfig which parses the default location of the JobConfig and performs validation. This allows us to simplify the calling interface in the command layer and allows us to embed BPM specific validation logic in the BPMConfig object.
@xoebus This is what I had in mind for the config object. I like how it leaves the JobConfig agnostic of JobName, and the bpmCfg.StoreDir()/bpmCfg.DataDir(). Is there any reason why this may be undesirable?

This broke the tests - I'm going to enable Travis for this repo.

That integration failure is a flake on wings. We've been dealing with it for a while, haven't taken the time to look at it because it only occurs when running integration in parallel.

I got unused variable complaints from this: https://github.com/cloudfoundry-incubator/bpm-release/commit/de9706aafaec107b8b74e7712338aacca6c2aa3e. Not sure why.


Wordlist

Preprocessing results and figures;
data masking codes

@rishizsinha you said you wanted to look at this?


Ability to import library local images to components

Ability to import library local images to components
'reactDom' to 'react-dom' (already fixed in the original repo)

hey guys, have you updated the version on NPM yet with this? I'm trying to import images and can't.

@lightninglu10 this library isn't on NPM, you have to clone it from Github


ACS Commons fails in cloud

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: _______AEM cloud
[ ] ACS AEM Commons Version: 5.0.4
[ ] Reproducible on Latest? yes

Expected Behavior
When ACS Commons bundle is deployed, I should be able to create generic lists
Actual Behavior
Tried with AEM SDK 3.5 version and it worked fine locally. Tried with SDK 6.5 and got scriptvalidationerror.  When pushed the code the cloud through CM, and tried to access List, it redirects to AEM banner with unexpected error message

Steps to Reproduce
Install ACS COmmons on project and deploy
Links
Links to related assets, e.g. content packages containing test components

What do the  AEM error logs say?

16.07.2021 13:51:37.663 [cm-p35622-e147206-aem-author-59cf95d46f-lggxl] ERROR [<IP_ADDRESS> [1626443497623] GET /mnt/overlay/acs-commons/components/utilities/genericlist.html HTTP/1.1] libs.granite.ui.components.shell.propertiespage Unable to render properties page correctly
javax.servlet.jsp.JspTagException: org.apache.sling.api.scripting.ScriptEvaluationException:
at org.apache.sling.scripting.jsp.taglib.AbstractDispatcherTagHandler.doEndTag(AbstractDispatcherTagHandler.java:138) [org.apache.sling.scripting.jsp.taglib:2.4.0]
at libs.granite.ui.components.shell.propertiespage.propertiespage__002e__jsp._jspService(propertiespage__002e__jsp.java:869) [aem-precompiled-scripts:6.6.0.V24016]
at org.apache.sling.scripting.jsp.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70) [org.apache.sling.scripting.jsp:2.5.2]
at org.apache.sling.scripting.jsp.PrecompiledJSPRunner.callPrecompiledJSP(PrecompiledJSPRunner.java:72) [org.apache.sling.scripting.jsp:2.5.2]
at org.apache.sling.scripting.jsp.JspScriptEngineFactory$JspScriptEngine.eval(JspScriptEngineFactory.java:583) [org.apache.sling.scripting.jsp:2.5.2]
at org.apache.sling.scripting.core.impl.bundled.PrecompiledScript.eval(PrecompiledScript.java:54) [org.apache.sling.scripting.core:2.3.6]
at org.apache.sling.scripting.core.impl.bundled.ScriptContextProvider$ExecutableContext.eval(ScriptContextProvider.java:135) [org.apache.sling.scripting.core:2.3.6]
at org.apache.sling.scripting.core.impl.bundled.AbstractBundledRenderUnit.eval(AbstractBundledRenderUnit.java:190) [org.apache.sling.scripting.core:2.3.6]
at org.apache.sling.servlets.resolver.internal.bundle.BundledScriptServlet.service(BundledScriptServlet.java:78) [org.apache.sling.servlets.resolver:2.7.14]
at org.apache.sling.engine.impl.request.RequestData.service(RequestData.java:574) [org.apache.sling.engine:2.7.6]
at org.apache.sling.engine.impl.filter.SlingComponentFilterChain.render(SlingComponentFilterChain.java:45) [org.apache.sling.engine:2.7.6]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:88) [org.apache.sling.engine:2.7.6]
at com.day.cq.wcm.core.impl.WCMDeveloperModeFilter.doFilter(WCMDeveloperModeFilter.java:119) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.day.cq.wcm.core.impl.WCMDebugFilter.doFilter(WCMDebugFilter.java:138) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.adobe.acs.commons.granite.ui.components.impl.include.IncludeDecoratorFilterImpl.doFilter(IncludeDecoratorFilterImpl.java:92) [com.adobe.acs.acs-aem-commons-bundle:5.0.4]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.adobe.granite.uriprovider.dba.DBAHeaderFilter.doFilter(DBAHeaderFilter.java:107) [com.adobe.granite.uriprovider.dba:0.2.6]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.adobe.granite.metrics.knownerrors.impl.ErrorLoggingComponentFilter.doFilter(ErrorLoggingComponentFilter.java:76) [com.adobe.granite.metrics.knownerrors:0.0.14]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.day.cq.wcm.core.impl.monitoring.PageComponentRequestFilter.doFilter(PageComponentRequestFilter.java:105) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.day.cq.wcm.core.impl.WCMComponentFilter.filterRootInclude(WCMComponentFilter.java:375) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at com.day.cq.wcm.core.impl.WCMComponentFilter.doFilter(WCMComponentFilter.java:190) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
at com.day.cq.wcm.core.impl.page.PageLockFilter.doFilter(PageLockFilter.java:91) [com.day.cq.wcm.cq-wcm-core:5.13.214]
at org.apache.sling.engine.impl.filter.AbstractSlingFilterChain.doFilter(AbstractSlingFilterChain.java:78) [org.apache.sling.engine:2.7.6]
author_aemerror_2021-07-16.log

I am getting same error when installing locally on and SDK Version higher than 3.5 - same project

Just checked, and this should be fixed in latest AEM CS (AEM SDK and real AEM CS).
What version of AEM SDK/CS are you on (the version should look like a date/time stamp .. something like: 2021.6.5540.20210615T210737Z-210527
Im not sure what AEM SDK 3.5 or 6.5 means

Our cloud is using 2021.6.5540.20210615T210737Z. Not sure if that is the latest version and if not, when it will be updated.

Latest AEM SDK version is:  v2021.7.5607.20210705T063041Z-210600
can you try that?

If you can figure out why this build's Unit test fails [1] running in Github Actions, we can merge a fix on our side too (though with the product fix this shouldn't be necessary). Oddly the unit test is fine when run locally.
[1] https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2618

I can try latest sdk  on my local. But won't be able to deploy to cloud dev
until they upgrade the release. Not sure why pros and stage use later
version than dev
On Fri, Jul 16, 2021, 4:36 PM david g. @.***> wrote:

If you can figure out why this build's Unit test fails [1] running in
Github Actions, we can merge a fix on our side too (though with the product
fix this shouldn't be necessary). Oddly the unit test is fine when run
locally.
[1] #2618
https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2618
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2647#issuecomment-881703932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU32TI62K3TK42F5RMFQZZDTYCJ4TANCNFSM5APUILSQ
.

AEM CS Production and Stage are always up to date, Dev requires a "manual" update. Just how it works.


Create animal groups for animal trainer

Instead of listing each animal specifically, create two groups: one for normal, domesticated animals; and one for wild, exotic animals.  Animals become subtypes and are filled in accordingly.

Can also use this for NPCs with animals

Can also add a subgroup for Wizard Servant (Dominated creature, golem, etc.). Groups divided by wizard level (since some creatures can only be made by higher-level wizard)


[v1.4] Collate Code

Coverage remained the same at 85.233% when pulling de0e1d4c66246fb9ec8292cff15cbdfca63a39ed on Rui_Zhong_New into 1a2af88c95eee74f98d54ee6bacdae50609b91ef on master.

Coverage remained the same at 85.233% when pulling 0b45e8f37bb518d5579e16e8a08858669a93b0f7 on Rui_Zhong_New into 1a2af88c95eee74f98d54ee6bacdae50609b91ef on master.

Coverage remained the same at 85.233% when pulling 4e23d7938452e31d2ec2708331937c9c7c3e0337 on Rui_Zhong_New into a08973516932295244a8aafcda36346b184a8224 on master.


Add Rustup installation instructions to README

Now that cranelift is distributed with Rustup, I find myself constantly searching for the https://github.com/rust-lang/rust/pull/81746 PR. I think that it would be useful to also describe the Rustup installation/usage instructions here in README.

https://bjorn3.github.io/2023/10/31/progress-report-oct-2023.html describes the way to use the backend via Cargo.toml in order to not change the compilation commands (and also via cargo's env vars), which could be interesting here as well.

Maybe we should also mention it in https://nnethercote.github.io/perf-book/compile-times.html

And also document this in the unstable book.

Added more usage instructions based on the blog post. Also sent PRs to the perf book (https://github.com/nnethercote/perf-book/pull/72).
I'm not sure if the unstable book is the correct place for this, as it contains just general descriptions of compiler flags. I think that the Rustc book might be more appropriate.

The actual text of the new section looks good to me. It feels a bit weird to have using precompiled versions from rustup first, then manual building and finally using precompiled versions from CI though.

I agree. I think that it makes sense to have the easiest way first though. What order would you prefer?

Maybe swap building it yourself and using the CI artifacts? You would need to reword using the CI artifacts for that as it is currently written as alternative for building yourself.

I tried to reorganize it a little bit.

Thanks!


{NewRelic} Generate commands with codegen v2

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

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

NewRelic

Why it has two extensions in your PR?

Why it has two extensions in your PR?
It may have been written incorrectly when exporting. It has been deleted.


Tests: Clean up DOM references from recent PRs

Description
Cleaning up after myself by following the examples in #8759.
How Has This Been Tested?
N.A.
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Local functions can provide a way of reusing the calling code if you think that brings any value:
IRefreshableElementCollection<IElement> RootButtons() => comp.FindAll("button.mud-button-root");
RootButtons().First().Click();
RootButtons().Last().Click();

IElement Menu() => comp.Find(".mud-menu");
await Menu().TriggerEventAsync("onmouseenter", new MouseEventArgs());
await Menu().TriggerEventAsync("onmouseleave", new MouseEventArgs());

Local functions can provide a way of reusing the calling code:

That's why I do in all my new tests, also you will never get old reference this way, but it's very time consuming to fix all the current tests.

Local functions can provide a way of reusing the calling code:

That's right, they can even help transform complex lookups into highly readable tests. Here is an example from my newest tests:
            var comp = Context.RenderComponent<TreeViewAutoExpandTest>(self => self.Add(x => x.AutoExpand, true).Add(x => x.ConfigCanExpand, false));
            var isExpanded = (string value) => comp.FindComponents<MudTreeViewItem<string>>()
                .FirstOrDefault(x => x.Instance.Value == value)?.Instance.GetState<bool>(nameof(MudTreeViewItem<string>.Expanded));
            var select = (string value) => comp.FindComponents<MudChip<string>>().FirstOrDefault(x => x.Instance.Text == value)?.Find("div").Click();
            isExpanded("C:").Should().Be(false);
            isExpanded("config").Should().Be(false);
            isExpanded("launch.json").Should().Be(false);
            isExpanded("tasks.json").Should().Be(false);
            isExpanded("images").Should().Be(false);
            isExpanded("logo.png").Should().Be(false);
            // select and check that along the path to the value all parents were expanded, nothing else
            select("tasks.json");
            isExpanded("C:").Should().Be(true);
            isExpanded("config").Should().Be(false); // <--- shouldn't be expanded because it can't

Thanks Jason!

Oh yeah the local functions definitely help with readability on some more complex selectors, will keep that in mind for the next ones. Thanks guys!


What is the preferred way to add futures/swap support

a) Add another virtual exchange that works only for futures, yet another one for swap.
b) Add a marketType parameter to all methods of the existing exchange.
The first one is closer to the native APIs provided by most exchanges.
The second one is more user friendly in my opinion.

Are you talking about OKEx?

OKEx, Huobi, Binance, Deribit, KuCoin etc.

For OKEx, can you determine whether it's futures / swap from the symbol name? I haven't looked into he others.

Maybe futures/swap can be determined by symbol name. But margin/spot share the same symbol names and depths.

In that case, I think option (b) would be better and less confusing.

Which methods would need the parameter?

Actually, a better option would be to create a property in ExchangeAPI:
		private bool isMarginTradingEnabled;

		public virtual bool IsMarginTradingEnabled
		{
			get { return isMarginTradingEnabled; }
			set { throw new NotImplementedException("Margin trading is not implemented for this exchange. Please submit a PR if you would like to implement it."); }
		}

Any exchanges that do implement it can override the property to set the field isMarginTradingEnabled. This way, no new classes need to be created, and no method signatures need to be changed.

Would returning false be better, that way people don't have unexpected exceptions? I suppose if we document the setter with a NotImplementedException that may be sufficient?

isMarginTradingEnabled is by default set to false (I updated the code above to make it clearer). I think it is better to throw an execption, than to have ppl set it to true and thinking that they are using margin trading when they are not. This would be set in the initialization part of ppl's code anyways, so it's not like it would interrupt their trading in the middle of the day.

Also, the exception would only be thrown if someone tries to use margin trading where not implemented. All existing user code would still work fine w/o any changes.

Sounds like a plan!

The derived class would have this:
		public override bool IsMarginTradingEnabled
		{
			get { return isMarginTradingEnabled; }
			set { isMarginTradingEnabled = value; }
		}

I would propose the following:
private bool isMarginTradingEnabled = false;
public virtual bool IsMarginTradingPossible => false;
public bool IsMarginTradingEnabled
		{
			get { return isMarginTradingEnabled; }
			set
                        {
                            if (!IsMarginTradingPossible) { throw new NotImplementedException(
				"Margin trading is not implemented for this exchange. Please submit a PR if you would like to implement it, derived classes can override IsMarginTradingPossible to return true ."); }
                        }
		}

Yes, I like that

Awesome, sounds like a plan for whoever tackles this beast :)

How is isMarginEnabled related to futures/swap?

isMarginEnabled is just for whether the user wants to use margin vs spot. Since the symbol names for futures/swap are different, those can be added to the existing methods without adding any parameters.


[dependabot] add config file

To enable automatic daily updates of openthread submodule.

@LuDuda @MarekPorwisz , I can also set this up to automatically tag someone for review on the automated PR submissions. Is there anyone you'd like to add?

Please auto tag me, and I will try to pass it to anyone available.

Please auto tag me, and I will try to pass it to anyone available.

Thanks, @MarekPorwisz !


Changes for thaliMobileNativeWrapper for iOS

These changes should only be done after we have implemented all the Android changes. Also note that these changes shouldn't be done until we have updated the WiFi based Mock to support iOS (yes, I have a bug I'm filing on that).

[x] Make sure that nonTCPPeerAvailabilityChanged doesn't return a portNumber (it shouldn't get one, you can assert on this) for iOS
[x] Implement the failedNativeConnectionEvent for iOS (multiConnectConnectionFailure)
[x] On the start method please check the list of things that now only happen when we are on Android. We need logic to make sure those things don't happen when we are on iOS.
[x] Stop should only stop the tcpServersManager object if we are on Android. On iOS we just pass the router's address directly since the native layer will talk to the node hosted express server directly, not via a mux.
[x] startUpdateAdvertisingAndListening should only put the tcpServersManager listener in the way if we are on Android, not iOS.
[x] Make sure that connect methods for the same peerIdentifier are serialized (so we won't ever have more than one connect method for a specific peerID at a time)
[x] Turn the multiConnectResolved callback into the multiConnectResolvedEvent
[x] Note that multiConnect works differently than connect. With connect you fire off the request and the response contains the port. With multiConnect when you fire off the request you just got a confirmation that the request was received. You have to stick a unique ID into the request because there is going to be a separate callback over the multiConnectResolved callback which will contain the actual result and need to be correlated using the ID you put into the multiConnect request.
[x] Make sure that multiConnect and Disconnect share a single queue for the same peerID (e.g. so that for any given peerID there can only be a single multiConnect or Disconnect method outstanding at a time for that ID0
[x] Implement multiConnect including the queue logic from the previous item
[x] Make sure we return the correct error if terminateConnection is called on iOS
[x] For iOS we need to listen directly for the incomingConnectionToPortNumberFailed native event. For Android this is actually handled by the thaliTCPServersManager code.
[x] For terminateListener return an error if we get called on iOS

https://github.com/thaliproject/Thali_CordovaPlugin/blob/vNext_artemjackson_904/thali/NextGeneration/thaliMobileNativeWrapper.js#L581 - Artyom apparently started on this, check out his code.

@yaronyg I have a question regarding

Make sure that multiConnect and Disconnect share a single queue for the same peerID (e.g. so that for any given peerID there can only be a single multiConnect or Disconnect method outstanding at a time for that ID0

Current of implementation of multiConnect and disconnect uses global promise queue for all peers. It does not contradict spec, but more optimal way would be separate queue for every peer.
My suggestion is to keep is simple for now until we have capabilities to measure performance impact. And then we can decide to split global queue onto peer-specific queues.
What do you think?

@chapko I'm o.k. with it for now. Please open a specific bug to track this since we should eventually fix it.

@yaronyg done

We think that thaliproject/Thali_CordovaPlugin#1527 is blocking this check in since tests are failing.