Fix angular SSR issues

I'm using SSR with Angular 7, and after I have added simplebar for angular, on SSR I'm getting an error:
Error generating html for req /roadmap TypeError: Cannot read property 'removeEventListener' of undefined     at SimpleBar.removeListeners (...\dist\server.js:267875:27)     at SimpleBar.unMount (...\dist\server.js:267895:10)     at SimplebarAngularComponent.module.exports.SimplebarAngularComponent.ngOnDestroy (...\dist\server.js:267988:24)
In the PR I have just added if checks for the existence of used vars. With this PR SSR works fine for me.
It's also reproducible on 2.2 and 3.0 versions of the plugin for Angular,

Please merge this!

Merge please. Perfect plugin, but if this wont be fixed, we will have to switch.

Please merge this as soon as possible.

Hey guys,
I would happily merge this but this is missing a few informations i think:
The conditional check you are doing is in the core of Simplebar not in the Angular plugin, so why is this specifically breaking with Angular SSR? I don't quite understand this. Because this same code is working fine for React for example.
I don't understand how this.contentWrapperEl could be undefined at the unmouting, seems like there is a deeper issue in there.
Also is this specific to Angular 7? On the Angular example in the repo with Angular 10, it's building fine without any error I believe.

Actually we realized what is the cause some of these issues. We are setting window properties at the node part, hence canUseDOM variable in the simplebar core will be true, meaning the code will execute at the SSR and will fail. So do not set global window properties cause it should be unset.

@sasos90 yep we could, but we go down that route of trying to solve 3rd party code issue within the library, this could grow infinitely. That's why I asked for more details.

@Grsmto we included domino package - and then we have problems with this unMount, could you please try simplebar with domino package on SSR.

The plugin works fine while not on SSR. The biggest issue is the check "canUseDOM" which checks for window property and therefor uses API which fails in SSR. We have window defined for some reasons and we can't turn it off right now. Not sure how we can show you the SSR, probably not possible in sandbox right?

@sasos90 yeah seems hard to test.
Thing is I don't get how the unMount ends up being called on SSR, that would mean ngOnDestroy is called on the component.
Anyway, I think this PR is probably fine to merge as is if it can help!
I'll look into implementing the same solution for the new major version.

Yes that is in fact true. I believe in some cycle from SSR to transfering state to client, this gets called, because if we put this to ngIf and do not show it before we have all the data (which is something we can not really afford), then it works properly. If we don't put ngIf then ngOnDestroy is called which executes unMount and hangs. Would be great if you can merge it and publish a new version. Thanks!

This was released in<EMAIL_ADDRESS>

Awesome thanks!


How to define the nodset visibility according to the logged user in uaExpert ?

hy,
it's possibile to visualize the nodset according to the user. ?For exemple if i want that only a super usere that is connecte to uaExpert can  see the nodset PDM in the immage

Is there a property that i can add to visualize a certain nodes? if so how i can do that ?

Managing permissions are a feature of the Server SDK you are using. It should be straightforward.


Expose touch events in canvas widget

This exposes iced_native::Event::Touch in canvas::on_event. Currently it's not possible to handle touchscreen interactions in canvas code, the way a lot of builtin widgets do (slider, button, etc).
I tested the examples that use canvas (bezier_tool, solar_system, game_of_life) and this doesn't break any of them.

thanks @hecrj - I think this is good to go


Data ingestion stage

[x] update config.yaml
[x] update secret.yaml
[x] update params.yaml
[x] update entity
[x] update configuration manager in src config
[x] update the components
[x] update the pipeline

Dry run successful for stage 1


I think I want to re-factor the display logic to use javascript libraries d3s for the graphs and leaflet for the mapping.  This will result in the displays needing a modern web browser to run, but will also allow stations on the network to display the graphics as well as the main monitor connected to the Pi.

On current Raspbian, numpy is already set up, and mature packages are available:
python-matplotlib
python-mpltoolkits.basemap
I have installed matplotlib and basemap manually on Pi Zero, and would never wish that horror on anyone.  I hav no input for getting it done on Windows; I don't touch that stuff.

I am working to convert the map display to use Cartopy. I have a prototype working.  It seems much faster than Basemap.

I have removed Basemap and replaced it with Cartopy, so this issue is now moot.


Sell this doesnt gives the cost of items back

Sell this doesnt gives the cost of items back

@AyushSehrawat PLS HELP , I tried but I failed

Hm, ig you have to run this function
async def sell_this(user,item_name,amount,price = None):
Before sending message, run this await sell_this(.........)

Oh yeah , i think there was some issue before that it didnt wokr


Better import error page

Fixes #5778
We can work on this PR, since the relevant info has been piped through to the front end.

cc @saleiva @rafatower @stevage @javisantana

too much info, for the user is enough to know the url and the error code (and the text) returned by it

@javisantana

The error text is already displayed
The HTTP code is shown when relevant (i.e, 404) but I'll work on piping it from the downloader
The URL will be shown

I'll remove the other info, it was just so we could decide what's important and what not.

I think your list in your last comment is enough, @gfiorav. :)

I thought you were assigning this to me. Happy to jump in!

@saleiva I'll push the necessary info to the backbone model. You take it from there! :smiley:
Still figuring out the least messy way of getting that HTTP response code to the front end.

Ok, so I think this might be more like it. What if we add a dictionary so we can show more info on the code? i.e: "404 - Not found"

Now we have an error code message as well. I think this should be enough.

IMO, that "403 - CartoDB is not autho..." should be in the point 1. (I'd remove the "HTTP Response" title)
then, point 2 should have the "Want to know more?"
Point '+' should have the "Persisting error" content

@saleiva Agree, I changed the order.
The only thing thats bugging me is that "Follow these steps" doesn't make much sense right?
What if we change the error code to match the HTTP error code (support doesn't need anything else that the identifier anyway) and change the subtitle to "Don't panic, here's some useful information"?
Something like this:

Retest this please

@saleiva
Like we agreed, this seems to be a good balance between human readability and functionality.

Ready for CR @rafatower

Final design:

I'm currently having some trouble with Jasmine, as I can't quite figure out how the test that is breaking ever passed before.
It's complaining about the view not including the text We could not create table from your query but I can't see where that string is produced in master.
Looking to figure that out and close this soon.

Looks great :) My one last quibble would be the "Download error (1001)" - that error code doesn't really mean much to anyone. Much more meaningful would be "Download error (403)" or failing that, "Download error". The only time the 1001 would be relevant is when contacting support, and you already have that covered by including the GUID.
Just another $0.02 from me... :)

@stevage thanks for the feedback. Since we have documented our internal error codes, we would like to keep them. You can find said documentation here.
Hope it makes sense for you! :smiley:

Fixed Jasmine issues. Need a CR @rafatower so we can close this!
I'll do some staging tests in the mean time.
Thanks!

retest this please

Retest this please

Retest this please
INFO: We are having some CI problems, sorry for the repeated messages

retest this please

Thanks for helping @xavijam !
Could I get a CR so I can close this @rafatower ?
Thanks :smile:

Just a super small detail. Remember that for any change that implies a change in the texts shown to the user, a revision from a native person is required.

Oh! @iriberri could you ping such a person here?

@csobier, could you please check if the sentences written here sound fine? :)

Looks fine to me :+1:

:+1: fair enough. Let's go ahead with this and we can refine texts if needed

Ok, thanks @rafatower @saleiva !


license?

I've forked your repo and made a clunky modification; was wondering what your intent was regarding licensing/credit

Excellent - thanks for the template!!


bug:

Meltano Version
2.7.1
Python Version
3.7
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
Debian GNU/Linux 10 (buster)
Description
When listing config password is logged in plaintext
setup 'default' tap.
get processing message to for an integration of type tap_xxxxx
-- check to see if meltano config has an instance of that integration defined
-- if not inherit config from default tap and set url, username through config calls ( using Popen then capture and log output )
set credentials etc. using os.environ
when running
run_command(f"meltano add extractor {extractor_name} --inherit-from {default_extractor}")
the password ( if set as an environmental variable ) is logged in plain text
Code
meltano add extractor TEST --inherit-from {default_extractor}

alan@alan-XPS-15-9570:~/projects/flow-vendor-data-cache-poc/vendor_data_cache_app/meltano$ meltano add extractor TEST --inherit-from flow-jira-tap
2022-10-05T18:16:56.462845Z [info     ] Environment 'dev' is active
Added extractor 'TEST' to your Meltano project
Inherit from:	flow-jira-tap

Installing extractor 'TEST'...
Skipped installing extractor 'TEST'...
Skipped installing 1/1 plugins

export TEST_PASSWORD='some_password'

meltano config TEST list

2022-10-05T18:18:27.559807Z [info     ] The default environment (dev) will be ignored for `meltano config`. To configure a specific Environment, please use option `--environment=<environment name>`.
username [env: TEST_USERNAME] current value<EMAIL_ADDRESS>(inherited from 'flow-jira-tap')
password [env: TEST_PASSWORD] current value: 'somepassword' (from the TEST_PASSWORD variable in the environment)
start_date [env: TEST_START_DATE] current value: '2022-09-21' (inherited from 'flow-jira-tap')
base_url [env: TEST_BASE_URL] current value: 'https://pluralsight.atlassian.net' (inherited from 'flow-jira-tap')
user_agent [env: TEST_USER_AGENT] current value: '<user-agent>' (inherited from 'flow-jira-tap')
request_timeout [env: TEST_REQUEST_TIMEOUT] current value: 300 (inherited from 'flow-jira-tap')
groups [env: TEST_GROUPS] current value: 'jira-software-users' (inherited from 'flow-jira-tap')
org_id [env: TEST_ORG_ID] current value: 2345 (inherited from 'flow-jira-tap')
integration_id [env: TEST_INTEGRATION_ID] current value: 9999 (inherited from 'flow-jira-tap')

It seems this is related to the config list function and really doesn't have any bearing as to whether it was set .env file or in environmental variable - may be working as designed and not bug.
alan@alan-XPS-15-9570:~/projects/flow-vendor-data-cache-poc/vendor_data_cache_app/meltano$ meltano config TEST set password somepassword
2022-10-05T18:27:39.550641Z [info     ] Environment 'dev' is active
2022-10-05T18:27:42.604153Z [info     ] The default environment (dev) will be ignored for `meltano config`. To configure a specific Environment, please use option `--environment=<environment name>`.
Extractor 'test' setting 'password' was set in `.env`: '(redacted)'
alan@alan-XPS-15-9570:~/projects/flow-vendor-data-cache-poc/vendor_data_cache_app/meltano$ meltano config list
2022-10-05T18:27:55.734733Z [info     ] Environment 'dev' is active
2022-10-05T18:27:58.782860Z [info     ] The default environment (dev) will be ignored for `meltano config`. To configure a specific Environment, please use option `--environment=<environment name>`.
Need help fixing this problem? Visit http://melta.no/ for troubleshooting steps, or to
join our friendly Slack community.

Plugin 'list' is not known to Meltano
alan@alan-XPS-15-9570:~/projects/flow-vendor-data-cache-poc/vendor_data_cache_app/meltano$ meltano config TEST list
2022-10-05T18:28:07.004663Z [info     ] Environment 'dev' is active
2022-10-05T18:28:10.122328Z [info     ] The default environment (dev) will be ignored for `meltano config`. To configure a specific Environment, please use option `--environment=<environment name>`.
username [env: TEST_USERNAME] current value<EMAIL_ADDRESS>(inherited from 'flow-jira-tap')
password [env: TEST_PASSWORD] current value: 'somepassword' (from the TEST_PASSWORD variable in `.env`)
start_date [env: TEST_START_DATE] current value: '2022-09-21' (inherited from 'flow-jira-tap')
base_url [env: TEST_BASE_URL] current value: 'https://pluralsight.atlassian.net' (inherited from 'flow-jira-tap')
user_agent [env: TEST_USER_AGENT] current value: '<user-agent>' (inherited from 'flow-jira-tap')
request_timeout [env: TEST_REQUEST_TIMEOUT] current value: 300 (inherited from 'flow-jira-tap')
groups [env: TEST_GROUPS] current value: 'jira-software-users' (inherited from 'flow-jira-tap')
org_id [env: TEST_ORG_ID] current value: 2345 (inherited from 'flow-jira-tap')
integration_id [env: TEST_INTEGRATION_ID] current value: 9999 (inherited from 'flow-jira-tap')


update irc link

Q
A

Is bugfix?
✔/❌

New feature?
✔/❌

Breaks BC?
✔/❌

Fixed issues

Lock file is updated automatically. No need to change it manually.


stable/opa Updating documentation in values.yaml

The documentation changes will inform user that there is a way to disable opa from loading default policies.
Fixes https://github.com/open-policy-agent/opa/issues/2410

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])
[x] Bumped up chart version

/assign @tsandall

/ok-to-test

/lgtm


[FEAT] : Add E-Mail Form in Contact Us

In this feature, we will add contact us email form, from this feature user can mail their query via email direct from website.
Previous Work

@PriyanshuValiya  i want to work on that

Hi @PriyanshuValiya, Would you please assign this issue to me? Would love to work on this.


Fix condition precedence

Now else is related to inner if, which is obviously a bug. This way srcContact  will always be null.

Could you please sign our contributor agreement? Thanks!

@ibauersachs sure thing, but couldn't you tell - should I use real name or github account name there?

Please use your real name. If we're unable to associate that with your Github name, please send a private e-mail to me.

@ibauersachs Done! You can associate the agreement with my GitHub name by E-mail :)


Obscure formatting issue with Neovim

This may be specific to Neovim, but I don't know if it's a bug with Neovim or not.
Given this perl file:
#!/usr/bin/env perl

content;

If I now try to format this file the following is logged by Neovim to its LSP log:
[DEBUG][2022-03-17 21:02:15] .../lua/vim/lsp.lua:1023   "LSP[perlnavigator]"    "client.request"        1       "textDocument/formatting"       {
  options = {
    [""] = true,
    insertSpaces = false,
    tabSize = 8
  },
  textDocument = {
    uri = "file:///home/seth/src/toy/perl/t"
  }
}       <function 1>    1
[DEBUG][2022-03-17 21:02:15] .../vim/lsp/rpc.lua:347    "rpc.send"      {
  id = 2,
  jsonrpc = "2.0",
  method = "textDocument/formatting",
  params = {
    options = {
      [""] = true,
      insertSpaces = false,
      tabSize = 8
    },
    textDocument = {
      uri = "file:///home/seth/src/toy/perl/t"
    }
  }
}
[DEBUG][2022-03-17 21:02:15] .../vim/lsp/rpc.lua:454    "rpc.receive"   {
  id = 7,
  jsonrpc = "2.0",
  method = "workspace/workspaceFolders"
}
[TRACE][2022-03-17 21:02:15] .../lua/vim/lsp.lua:806    "server_request"        "workspace/workspaceFolders"    nil
[TRACE][2022-03-17 21:02:15] .../lua/vim/lsp.lua:809    "server_request: found handler for"     "workspace/workspaceFolders"
[TRACE][2022-03-17 21:02:15] ...lsp/handlers.lua:487    "default_handler"       "workspace/workspaceFolders"    {
  ctx = '{\n  client_id = 1,\n  method = "workspace/workspaceFolders"\n}'
}
[DEBUG][2022-03-17 21:02:15] .../vim/lsp/rpc.lua:464    "server_request: callback result"       {
  result = vim.NIL,
  status = true
}
[DEBUG][2022-03-17 21:02:15] .../vim/lsp/rpc.lua:347    "rpc.send"      {
  id = 7,
  jsonrpc = "2.0",
  result = vim.NIL
}
[ERROR][2022-03-17 21:02:15] .../vim/lsp/rpc.lua:73     "invalid header line %q"        "content;\nee4ffa34-a14f-4609-b2e4-bf23565f3262\nContent-Length: 157"

The last line is the error. I found this, not with the above file, but with a much more complicated file that had the word content in a couple of lines that looked like this:
  my ($body)
          = map { $_->content } $sent[0]{email}->object->parts;

I don't know if it's how the content is being sent, or what.  I couldn't get any other LSP server to fail with the word content somewhere in it.  Only PerlNavigator had this particular issue.
This is with the most recently published npm of PerlNavigator. Which I believe is 0.2.8.

I also tried it with the 0.2.10 downloadable executable with the same results.

For the record: I am having this issue with formatting also.
It is literally crippling my workflow.
Now, this does seem to be a bug in neovim (see neovim/neovim#15444); but the comments there seem to say this only happens if the LSP is sending things before the headers that it's supposed to send.
@bscan is there a way to see the requests that the server is sending to debug this further?

@rabbiveesh I work around this by disabling formatting via perlnavigator and using null-ls to run perltidy. It works well.

Thanks @rabbiveesh ! Between your comments and @WhoIsSethDaniel 's debug log, I now see the error. This issue never manifested in vscode because vscode uses node-ipc for interprocess communication, while most of the other editors use stdin/stdout.
Anything printed to the console via  console.log is written to stdout and pollutes the communication (stated here too: https://github.com/microsoft/vscode-languageserver-node/issues/824#issuecomment-938392051 ). In this case it was printing the entire output of perltidy to stdout, which was showing up before sending the response headers. Should be fixed via https://github.com/bscan/PerlNavigator/commit/ae8dde13aa21a0c8c28e0db9e538630626bab225 . I just deployed to NPM as v0.4.8. Thank you both for helping debug this.


[App] wandb was workng and now is not

wandb: Currently logged in as: monajalal (use `wandb login --relogin` to force relogin)
Problem at: main.py 7 <module>
wandb: ERROR Error communicating with wandb process
wandb: ERROR try: wandb.init(settings=wandb.Settings(start_method='fork'))
wandb: ERROR or:  wandb.init(settings=wandb.Settings(start_method='thread'))
wandb: ERROR For more info see: https://docs.wandb.ai/library/init#init-start-error
Traceback (most recent call last):
  File "main.py", line 7, in <module>
    wandb.init(project="my-test-project", entity="my-test-project")
  File "/SeaExp/mona/venv/dpcc/lib/python3.8/site-packages/wandb/sdk/wandb_init.py", line 870, in init
    run = wi.init()
  File "/SeaExp/mona/venv/dpcc/lib/python3.8/site-packages/wandb/sdk/wandb_init.py", line 568, in init
    raise UsageError(error_message)
wandb.errors.UsageError: Error communicating with wandb process
try: wandb.init(settings=wandb.Settings(start_method='fork'))
or:  wandb.init(settings=wandb.Settings(start_method='thread'))
For more info see: https://docs.wandb.ai/library/init#init-start-error

and
$ lsb_release -a
LSB Version:    core-9.20170808ubuntu1-noarch:security-9.20170808ubuntu1-noarch
Distributor ID: Ubuntu
Description:    Ubuntu 18.04.6 LTS
Release:        18.04
Codename:       bionic

Hi @monajalal, if you're still encountering this issue please let us know.

Hi @aidanjdonohue thanks for your response.
I really don't know what was the cause of this transient error. I ran it again with no problem today
wandb:
wandb: Synced 5 W&B file(s), 1 media file(s), 0 artifact file(s) and 0 other file(s)
wandb: Synced atomic-darkness-109: https://wandb.ai/my-test-project/my-test-project/runs/3nmbq2re
wandb: Find logs at: ./wandb/run-20211213_180200-3nmbq2re/logs/debug.log
wandb:

You can perhaps check the URL I shared with you internally.
Here are the related pieces of code I use for wandb. For now, no sweep, just logging my metrics and loss.
import wandb
wandb.init(project="my-test-project", entity="my-test-project")

# wandb.init(mode="disabled")

wandb.config = {
   "learning_rate": 1e-3,
   "epochs": 50,
   "batch_size": 8
 }
wandb.watch(model)

                wandb.log({"Epoch": epoch,
                        "Train Loss": train_loss,
                        "val Loss": val_loss,
                        "val MSE": val_mse,
                        "val R2": val_r2,
                        "val_MAE": val_mae,
                        "val RMSLE": val_rmsle,
                        "val NRMSE": val_nrmse})

    wandb.log({'train loss over epochs figure': wandb.Image(plt)})

hmm I am now running a 5-fold experiement at the same time so it's possible that this is causing the error? Any chance, I could run the same instance of the code across multiple GPUs with no problem?
wandb: Currently logged in as: monajalal (use `wandb login --relogin` to force relogin)
Problem at: main.py 7 <module>
wandb: ERROR Error communicating with wandb process
wandb: ERROR try: wandb.init(settings=wandb.Settings(start_method='fork'))
wandb: ERROR or:  wandb.init(settings=wandb.Settings(start_method='thread'))
wandb: ERROR For more info see: https://docs.wandb.ai/library/init#init-start-error
Traceback (most recent call last):
  File "main.py", line 7, in <module>
    wandb.init(project="my-test-project", entity="my-test-project")
  File "/SeaExp/mona/venv/dpcc/lib/python3.8/site-packages/wandb/sdk/wandb_init.py", line 870, in init
    run = wi.init()
  File "/SeaExp/mona/venv/dpcc/lib/python3.8/site-packages/wandb/sdk/wandb_init.py", line 568, in init
    raise UsageError(error_message)
wandb.errors.UsageError: Error communicating with wandb process
try: wandb.init(settings=wandb.Settings(start_method='fork'))
or:  wandb.init(settings=wandb.Settings(start_method='thread'))
For more info see: https://docs.wandb.ai/library/init#init-start-error

@monajalal the error your seeing is due to how you're using multiprocessing.  More information about the error and multiprocessing can be found here: https://docs.wandb.ai/library/init#init-start-error

The earlier error was do to issues on our server and should be resolved now.

@vanpelt would you like me to create a new issue for the 5-fold cross-validation scenario and close this one?

@vanpelt so I used
wandb.init(project="my-test-project", entity="my-test-project", settings=wandb.Settings(start_method="thread"))
from the link you shared and now I am not getting error if I am running my code for 3-fold cross validation across 3 GPUs (somewhat simultaneously). Could you please confirm this is correct way to go about it?
Still, I am not fully sure how adding settings=wandb.Settings(start_method="thread") is taking care of the bug.
Thanks a lot for your recommendation.


[FLINK-20282][runtime] Improve error messages for invalid managed memory fraction

cc @zhuzhurk

@zhuzhurk comments addressed


#10681: Add perf benchmark support to Mixtral test_perf and demo

Ticket
https://github.com/tenstorrent/tt-metal/issues/10681
Problem description
Add benchmark profile to Mixtral perf test and to demo code.
Checklist

[x] Test_perf support
[x] Demo_with_prefill support

Tests:

[ ] Post commit CI passes
[ ] T3k Perf pipeline
[ ] T3k demo pipeline

Added benchmark support to perf test.
Missing demo_with_prefill.py now.

@ttmchiou What are your thoughts in the changes in file .github/workflows/t3000-demo-tests.yaml?
At the time, only Falcon7B and Mixtral have support for the benchmark profile. But as we ramp up more models we will include the pass that searches for the benchmark files and uploads if found to all our models.

Pipeline is correctly uploading the benchmark csv to the dashboard: https://github.com/tenstorrent/tt-metal/actions/runs/10185641669/job/28175992490
Re-running the relevant tests again to prepare for merge.


Add alternate pseudo 3D retro 3D rendering.

Hi, I think It would be nice if godot would let the player to play pseudo 3D ancient 3D rendering techniques, like a basic 2D raycaster (DOOM and wolfestein), or something like Mode 07 for the SNES or even the perspective integer mode of the early psx.
I'm not sure if those can be easily done with 2D rendering mode.

Pseudo 3D of Wolfenstein style can be done fairly neatly with billboarding sprites.
The other modes are well, completely different.

2D with depth and mode 7 are not 3D, as you seem to be aware. If you want those, you should just simulate them with 3D sprites. Mode 7 could be done in 2D, but Wolfenstein and Doom graphics would be extremely hard to reproduce correctly without making your own engine. I don't think this is worth adding.

The thing is: why would want to use ancient pseudo-3D techniques in a modern 3D engine? Why not simply use the modern 3D techniques?
In any case, I believe all of those can accomplished by applying custom transforms to 2D nodes.

And is too corner case to be added to the core.

Those are too game-specific indeed and no really something that should be implemented in the render engine, as others mentioned. You can achieve such effects easily using the existing features of the 2D and/or 3D renderer based on the expected output. I'm pretty sure the community will make demos for those on the Asset Library eventually.


Cannot unset $this in library/Zend/Service/WindowsAzure/Storage/Batch.php on line 108

$ php -l shardj/zf1-future/library/Zend/Service/WindowsAzure/Storage/Batch.php
PHP Fatal error:  Cannot unset $this in shardj/zf1-future/library/Zend/Service/WindowsAzure/Storage/Batch.php on line 108
Errors parsing shardj/zf1-future/library/Zend/Service/WindowsAzure/Storage/Batch.php

https://github.com/Shardj/zf1-future/pull/76

Thanks, should be fixed now that your PR is merged


[Resources][TOCrestructure]

TOC restructure

Docs Build status updates of commit 30bb14d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add robot access resource

Was this ready for merge @fodderstompf ?

Yeah if you're happy with it 🙂


New script for installing SDKMAN! and Java

A new script for the script-library probably called java-debian.sh to install SDKMAN! and a Java version of choice.
This script can be further optimized to also replace gradle-debian.sh and maven-debian.sh since SDKMAN! can also manage them.
I have being doing this on my dotfiles, it's not 100% accurate yet, but can help to get started:
  echo_task "Installing SDKMAN!"
  sudo apt update
  sudo apt install -y zip
  bash -c "$(curl -fsSL "https://get.sdkman.io/?rcupdate=false")"
  (
    set +euo pipefail
    source "$HOME/.bashrc" &&
      mkdir -p "$SDKMAN_DIR" &&
      echo -e "sdkman_auto_answer=true\n" >"$SDKMAN_DIR/etc/config" &&
      sdk selfupdate force &&
      echo_task "Installing Java 8" &&
      identifier="$(sdk ls java | grep -o '8.0.*.hs-adpt' | awk '{print $NF}')" &&
      {
        output="$(sdk i java "$identifier" | tee /dev/tty)" ||
          echo "$output" | grep -q "already installed"
      }
  )

https://github.com/felipecassiors/dotfiles/blob/da9c16e9e44c355c880b974a01293117a32772ef/run_once_setup.sh#L101-L117

Thanks for the pointer! Will definitely look!

@testforstephen @akaroml What are your thoughts on using SDK man to install needed JDKs, Gradle, etc.  It would be easy enough to pivot in this direction via the scripts that already exist.

Just to add:
SDKMAN! is almost a must-have for Java (or any derivated language such as Groovy, Kotlin) developers for the same reasons the nvm is a must-have for Node.JS developers. Since we're talking not about a runtime container but a development container, the ability to switch between Java versions must exist.


Always clear mousemove timer before setting new one

What does this do?
This PR updates the handling of the mousemove event listener that gets added and removed while starting a new drag. Previously, the mousemove listener added to this.rootElement would only be removed if this.currentDragSourceNode was still set when we called this.clearCurrentDragSourceNode. Now, we attempt to remove the mousemove listener regardless of whether the this.currentDragSourceNode is set or not.
Why are we doing this?
My understanding of the current (pre-this-PR) flow for that listener is:

when a new drag starts, we set a timeout for one second.
The one second timeout is intended to allow the first mousemove events that fire for the drag preview to go through
If that timeout ends, we attach a mousemove event listener to the rootElement.

That mousemove listener is used to catch a situation where the drag source node has been removed from the DOM, in which case 'dragend' won't be fired. When the listener fires, we manually force the drag to end.

If the dragend is fired before the timeout expires, we clear the timeout and do not add the mousemove listener

I've been running into a bug where the mousemove event listener does not properly get cleared in Firefox. If the drag source node is removed from the DOM while the drag events are firing, you can get into a situation where Firefox will not fire the 'dragend' event appropriately, but the 'mousemove' event is still not caught by the listener. This can then result in future drag events triggering clearCurrentDragSourceNode without a currentDragSourceNode set - which will not remove the mousemove listener. When that happens, the mousemove listener will remain attached to the rootElement.
If the mousemove listener from a previous drag is still set when a new one is started, the initial mousemove events get trigger the listener, which then immediately ends the drag event, and stops further drag events being fired.
What are the risks of this PR?
I believe this is a harmless change - calling clearTimeout and removeEventListener does not have any detrimental effect if there is no timeout or listener to clear. This change is fixing an edge case, which is likely the fault of my application's code, but it makes sense to me to be more aggressive with removing that mousemove event listener, given the indirect way that it is attached in the first place.
If I'm understanding the general flow of the HTML5 backend correctly, I do not think there are any risks with being more aggressive in removing that mousemove listener - but it's definitely possible that there are other edge cases that I'm not considering here.
I have not personally run into problems using my fork in our application, but I'm unsure of the best way to test this change for more general use cases.
some context on our application's specific issues
Our application has some frustrating issues with nodes rerendering due to clashing of two different JS frameworks handling render updates (YUI and React). Nodes being unexpectedly removed from the DOM and replaced with identical ones is almost certainly the fault of our own application code, but the bug looks to be caused by the indirect attaching of the event listener after a timeout, which is in itself intended to fix issues with Firefox, so I think it's fair to patch this behavior here in addition to fixing our own issues in our application.

I ran yarn version check --interactive for this branch - I believe it would just be a patch version bump for the backend-html5 package. I'm not sure if I'm supposed to check in the file that command created in .yarn/versions to get the version bump set up properly?

I think i tracked down the real bug we're running into, and put up a fix in a separate PR https://github.com/react-dnd/react-dnd/pull/3271
The isNodeInDocument function is not working properly for iframes, and was returning false for our drag source node inside of an iframe.
That bug was then causing our issue with the mousemove listener not being correctly removed. So this PR fixes the mousemove issue, but it'd be better to just merge the other PR.


airsdk compress swf not be used

all fws

Can you explain more?

adobe air sdk 25
setting optimize bytecode =true.not use.
my proj get  swf 3M size.
and i addtional compiler options -compress=true,
i get swf 1M size.

This setting only works in release mode.

@matrix3d This issue can be considered solved?


Prevent pip from installing old version of lightning

This fix makes sure that nougat installs lightning 2.x.x and not lightning 2022.10.25 which is older that 2.x.x
lightning used to use year.mm.dd versioning scheme before they switched to the standard semantic versioning. Without this fix, lightning>=2.0.0 matched 2022.10.25 which is older than 2.x.x. This leads to a deprecation warning or broken installation (depending on the python/pip versions) due to the bug in older lightning versions.

 They should have incremented to 3


Fixed OMPLPlannerParameters::serialize()

The serialize() function previously serialized a pointer to goal TSRs, instead of the contents of the TSR. This caused serialization to silently fail due to lack of error checking in deserialize().
This commit adds error checking and correctly serializes the TSRs.
I discovered this when trying to merge https://github.com/personalrobotics/prpy/pull/311.

@gilwoolee signed off on this, so I am merging it.


Support for named resources

In order to reduce the processing payload when opa begins validation - when a specific kubernetes resource is known by name we can support filtering by any matching names.
Currently this requires strict matching - will add issue to backlog to consider wildcards for names.
Closes #249

Reviewed the code and docs, tested with some tempo validations using the named resource, and it works great! The thing that tripped me up was the named resource returning as a single object instead of an array with only one object. I'm kind of torn on consistency here, it's weird to get an array with a single element but also a bit of a context switch to have both single objects and arrays returned as resources. I think my comment questioning the possibility of having multiple named resources (e.g., across different namespaces) is still open and valid to the dataset type being returned, but there's no real use case for this at the moment so might be fair to bump this to a backlog thing.

Reviewed the code and docs, tested with some tempo validations using the named resource, and it works great! The thing that tripped me up was the named resource returning as a single object instead of an array with only one object. I'm kind of torn on consistency here, it's weird to get an array with a single element but also a bit of a context switch to have both single objects and arrays returned as resources. I think my comment questioning the possibility of having multiple named resources (e.g., across different namespaces) is still open and valid to the dataset type being returned, but there's no real use case for this at the moment so might be fair to bump this to a backlog thing.

I believe it's worth considering and performing more discovery to evaluate value. I went back and forth and should probably update the description of the PR to capture that I aligned to 'kubectl' for the purposes of the returned payload. IE 'kubectl get po -A' is a list vs 'kubectl get po  -n ' is a single object.


chore: polkadot-v0.9.12

@enthusiastmartin please check out the XCM changes

@enthusiastmartin please check out the XCM changes

Looks ok to me.


[SPARK-36261][PYTHON] Add remove_unused_categories to CategoricalAccessor and CategoricalIndex

What changes were proposed in this pull request?
Add remove_unused_categories to CategoricalAccessor and CategoricalIndex.
Why are the changes needed?
We should implement remove_unused_categories in CategoricalAccessor and CategoricalIndex.
Does this PR introduce any user-facing change?
Yes, users will be able to use remove_unused_categories.
How was this patch tested?
Added some tests.

cc @HyukjinKwon @itholic @xinrong-databricks

Test build #141519 has started for PR 33485 at commit dc29a39.

Test build #141519 has finished for PR 33485 at commit dc29a39.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141519/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46036/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46037/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46036/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46037/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46037/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46036/

Jenkins, retest this please.

Test build #141539 has started for PR 33485 at commit dc29a39.

Test build #141539 has finished for PR 33485 at commit dc29a39.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141539/

Merged to master and branch-3.2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46057/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46057/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46057/


Text decoding error when returning to resolve characters containing non-English

I want to use powershell to list the installed programs on the computer and determine if the user has installed the target program. But when running in vscode, when the return value contains Chinese, the decoding error of the result leads to garbled characters.
After some inquiries, I know that the problem is because the default encoding in the Chinese system is gbk. According to the existing solutions on the Internet, either modify the registry of the local computer or modify the language of the computer. But neither solution can be used as a software tool for customers.
OS：win11 64bit
node-powershell：5.0.1
Here is my demo code.
import { PowerShell } from "node-powershell";
const powershellInstance = async () => {
  const ps = new PowerShell({
    debug: true,
    outputEncoding:'utf-8',
    executableOptions: {
      '-ExecutionPolicy': 'Bypass',
      '-NoProfile': true,
    },
  });

  try {
    const Name = ('^Google Chrome');
    const printCommand = PowerShell.command `Get-ItemProperty -Path "HKLM:\\Software\\Wow6432Node\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\*" | Select-Object DisplayName, DisplayVersion, InstallDate, Version`;
    const  result = await ps.invoke(printCommand);

    console.log('11',result);
  } catch (error) {
    console.error(error);
  } finally {
    await ps.dispose();
  }
};
powershellInstance();

我发现了一种方法
    const ps = new PowerShell({
        debug: false,
        outputEncoding:'binary',
        executableOptions: {
            '-ExecutionPolicy': 'Bypass',
            '-NoProfile': true,
        },
    });

            // 执行 PowerShell 命令并获取输出
        const printCommand = PowerShell.command`Get-WmiObject Win32_Process -Filter "name = 'xxx.exe'" | Select-Object  CommandLine`;
        const output =   await ps.invoke(printCommand);

        // console.log(output.stdout)
        const decodedDataLocal = iconv.decode( output.stdout , 'utf-8');
        const decodedData = iconv.decode(Buffer.from(decodedDataLocal, 'binary'), 'GBK');

        console.log(decodedData)


Issue when using multiple GetMaterialApp

I have a container app, that uses getx. It uses GetMaterialApp at its root. I have another flutter module included as a path dependency in container apps pupspec.yaml file. The module is totally independent and separate. The module also uses getx for navigation with GetMaterialApp at its root. From my container app, when I navigate to the module view using Get.to(), I get a blank screen with error in console
Container App
void main() async {
  runApp(MainView());
}

class MainView extends StatelessWidget{
  String? themeFromDevice;

  MainView({String? theme}) {
    themeFromDevice = theme;
  }

  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
        debugShowCheckedModeBanner: false,
        initialRoute: '/',
        navigatorObservers: <RouteObserver<ModalRoute<void>>>[routeObserver],
        getPages: [
          GetPage(
            name: '/',
            page: () => StatelessWidgetWrapper(screenWidget: HomeScreen(theme: themeFromDevice,)),
          ),
        ]
    );
  }
}

Container app pubspec
name: home_module
description: A new flutter module project.
version: 1.0.0+1

environment:
  sdk: ">=2.12.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  lottie: ^1.1.0
  device_info_plus: ^3.1.1
  dart_ipify: ^1.1.1
  webview_flutter: ^2.3.1
  url_launcher: ^6.0.17
  external_app_launcher: ^2.0.2
  package_info: ^2.0.2
  visibility_detector: ^0.2.2
  encrypt: ^5.0.1
  youtube_player_flutter: ^8.0.0
  module1:
    path: ../module1
  get: 4.6.1

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^1.0.4

Module App
void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(

        primarySwatch: Colors.blue,
      ),
      home: const ModuleHomePage(title: 'Screen 1'),
    );
  }
}

From Container app, launch the module app using  Get.to(() => MyApp()); (MyApp is from module app)
Error Produced:
======== Exception caught by widgets library =======================================================
The following assertion was thrown while finalizing the widget tree:
Duplicate GlobalKey detected in widget tree.
The following GlobalKey was specified multiple times in the widget tree. This will lead to parts of the widget tree being truncated unexpectedly, because the second time a key is seen, the previous instance is moved to the new location. The key was:

[LabeledGlobalKey#f27a2]
This was determined by noticing that after the widget with the above global key was moved out of its previous parent, that previous parent never updated during this frame, meaning that it either did not update at all or updated before the widget was moved, in either case implying that it still thinks that it should have a child with that global key.
The specific parent that did not update after having one or more children forcibly removed due to GlobalKey reparenting is:
Builder(dependencies: [MediaQuery])
A GlobalKey can only be specified on one widget at a time in the widget tree.
When the exception was thrown, this was the stack:
#0      BuildOwner.finalizeTree. (package:flutter/src/widgets/framework.dart:2991:15)
#1      BuildOwner.finalizeTree (package:flutter/src/widgets/framework.dart:3016:8)
#2      WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:893:19)
#3      RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:370:5)
#4      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1146:15)
#5      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1083:9)
#6      SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:997:5)
#10     _invoke (dart:ui/hooks.dart:151:10)
#11     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:308:5)
#12     _drawFrame (dart:ui/hooks.dart:115:31)
(elided 3 frames from dart:async)
====================================================================================================

Getx Version: 4.6.1

facing this issue, any update?

@jonataslaw are you checking this one?

use GetMaterialApp only on the MyApp only other pages just use materialapp


Fixed ReplacePath rule executing out of order, when combined with PathPrefixStrip

Addresses issue raised in  #1569

@aantono we'd like this fix to still make it into the 1.3 release. Could you please rebase your PR onto the v1.3 branch?
Since some people tend to struggle with rebasing, here's a quick how-to if a simple git rebase does not work:

Create a branch from upstream/v1.3 locally.
 (One easy way to do this is to check out the new branch and do git cherry-pick <commit from your feature branch>; another one is to check out the existing feature branch and do git rebase --onto <name of the new 1.3 branch> <commit from your feature branch>^. All of this assumes you still have a single commit only.)
Force-push the changes to your repo.
Click Edit at the top of this Github page and change the base to v1.3.

I can also do it for you if you like. Just let me know.

Will rebase and force push as soon as tests finish running locally.  Will you cherry-pick the changes back into master, or do you want me to create a separate PR for that?

We'll merge the v1.3 branch back into master once this PR (and one or two others currently in the making) have landed. So you don't need to worry about getting things into master, I'll handle that. ☺

Sounds good, just pushed the updates, please let me know if that looks as what you expect.

Also took a stab at adding documentation section about Rules Order, so let me know if that is what you were looking for as well.

I see this is still marked as waiting for corrections, is there anything else that I've missed?

I've rebased the PR against the trunk of v1.3 branch to make it mergable...


Shopify API POST request not working with POSTMAN on Private Apps

I'm trying to make a very basic POST request for a private shopify app. I am working with Postman REST client EXACTLY as described here:
https://help.shopify.com/api/tutorials/using-postman
Unfortunately, It is not working. It returns status 200 and return an html page that says : "Log in to manage your store".
It does not apply any changes to the shop.
For some reason, GET requests work fine while all POST requests fail.
I am currently using the trial account of 14 days. So is it the issue with trial account?

This repo is for our Storefront GraphQL API, not REST. Support can be found here at https://developers.shopify.com/community


Spark 13025

JIRA :  SPARK-13025
Changed access modifier to default(public) to make setInitialModel available to user

Can one of the admins verify this patch?

@GayathriMurali please fix the title


Sqlglot transform for concat

A slightly better way of dealing with this issue in athena.
Instead of adjusting the SQL in all dialects, I think it makes more sense to just add a transformation function for athena.
sqlite seems to have a massive performance hit if we use cast(var as varchar) across the board (as you'd probably have guessed). See benchmarks here.

Just as an FYI, all tests pass for this when I merge in my new sqlglot PR.
Probably best you check that first 😂


prevent grid-plate from being inserted in grid-plate

It... PROBABLY would be ok based on how we're managing the board now but... it looks insane.

This is now possible for us to manage so not going to block it for now


fix deploy-package job in CI

deploy-package job begun to fail after we commit Pipfile.lock.
Pipfile.lock was generated on my macOS and it did not include some of twine's dependencies (jeepney & secretstorage, both depends on DBus).
They are necessary to make twine work on Linux machine so I updated Pipfile.lock generated in ubuntu:18.04 docker image.
After merging this PR, I will delete & re-create release-1.4.0 tag.
It will launch release CI. deploy-package job is supposed to succeed but deploy-api-package will fail since there is no diff on API doc.

@notogawa wait a moment.
"markers": "python_version < '3.8'" are too much.

twine doesn't ref Pipfile.lock.
pipenv sync --dev command installs twine without jeepney & secretstorage even on Linux env.

I've got your idea.
I will remove twine (& setuptools and wheel) from Pipfile's dev-dependencies (because developers do not recommended use them).
They will become installed explicitly from CI job.

CI: https://app.circleci.com/pipelines/github/Idein/actfw-core/106/workflows/93852c66-1054-4110-9428-5024cfe76851
Released: https://pypi.org/project/actfw-core/1.4.0/ , https://idein.github.io/actfw-core/1.4.0/


Leadspace-centered (no image and with image) - Title font weight

Seen on Chrome
Font weight for the title in the leadspace looks thicker than designed.
Leadspace title should be using Expressive-Heading-05 which uses light weight for Med, Large, XL, and Max sizes. Expressive-Heading-05 only uses regular weight for the Small breakpoint.
As seen in Storybook:

Expected:

Font size for the description in the leadspace is not correct.
Leadspace description should be using Expressive-Heading-03 which is 20px for Small, Med, Large and XL breakpoints and only gets larger for the Max breakpoint. You can see how the implemented sizes are not consistent.
Max in Storybook:

Max expected:

XL in Storybook:

XL expected:

Large in Storybook:

Large expected:

Med in Storybook:

Med expected:

Small in Storybook:

Small expected:

The tokens are being added correctly on our side, seems to be an issue with carbon. I opened an issue within Carbon's github.
https://github.com/carbon-design-system/carbon/issues/5548

Carbon team solved this issue via this PR:
https://github.com/carbon-design-system/carbon/pull/5625
Waiting to close this issue once I can confirm it's working on our side


Regenerate PL wordforms

PL wordforms - ":" is used inside the list. Should regenerate it from http://www.sjp.pl/
NOTE: the list downloaded from this site has several lines for the same stem. Like this (the first word is stem):

Abel, Abla, Ablach, Ablami, Able, Ablem, Ablom, Ablowi, Ablowie, Ablów, Ablu
Abel, Abla, Ablem, Ablowi, Ablu

Looks like this is the cause of error mentioned above. As an easy solution should merge those stems together.
https://github.com/BorisNA/addflex/issues/3

Should be fixed with 609850e13a32086fd944e566ee5186a5b1a3d0f2


Tracking bug for adding ARM64 simulator artifacts to the Flutter.xcframework

We'd like to add ARM64 simulator to the supported architectures in the Flutter Engine's xcframework artifact.
Blockers

[x] https://github.com/flutter/flutter/issues/84670 Add support for ARM64 simulator builds
[x] https://github.com/flutter/flutter/issues/84601 Update iOS and macOS engine builders to Xcode 12.0
[x] https://github.com/flutter/flutter/issues/84672 Update Flutter Engine recipes to handle ARM64 simulator

cc @jmagman

I believe the timeout issue is no longer a problem; that being said, LUCI try runs are timing out because it takes 22 minutes to install a different version of Xcode :(

This should also be related to https://github.com/flutter/flutter/issues/60118

This should now be done. I'll close it when we've validated it.

Tool change https://github.com/flutter/flutter/pull/85642 has merged.


Some ttlib operations fail with unexpected index [Grayskull]

Describe the bug
Some operations fail with unexpected index error if "System Memory" is used as an input buffer type. For example,
tt_lib.tensor.reduce_max_h operation breaks with unexpected index error in some test cases.
Other operations with the same error are:

tt_lib.tensor.reduce_max_h
tt_lib.tensor.reduce_max_w
tt_lib.tensor.reduce_sum_h
tt_lib.tensor.reduce_sum_w
tt_lib.tensor.split_last_dim_two_chunks_tiled
tt_lib.tensor.tilize
tt_lib.tensor.permute
tt_lib.tensor.repeat_interleave
tt_lib.tensor.sum_2
tt_lib.tensor.sum_3
tt_lib.tensor.swiglu
tt_lib.tensor.reglu
tt_lib.tensor.glu
tt_lib.tensor.gelu

To Reproduce
Steps to reproduce the behavior:

Checkout the branch: barsic/test-sweeps-sys-memory
Run unit test test_reduce_max_h.py  using this command:
pytest tests/tt_eager/python_api_testing/non_working_unit_tests/grayskull/test_reduce_max_h.py

All other unit tests (for the mentioned operations) are present in the folder: tests/tt_eager/python_api_testing/non_working_unit_tests/grayskull/
Expected behavior
There are two test case in the unit test test_reduce_max_h.py and they all are expected to fail with the same error.
Getting Additional info for the operation under test and its behavior
To get additional information and results for different combinations of input shapes, types, layouts and memory configs for which this operation was tested you can also run locally sweeps and check the results. To do this you should:

Follow the Getting Started page to setup the repo, environment variables and python-env
Activate source build/python_env/bin/activate
Run sweeps by using python tests/tt_eager/python_api_testing/sweep_tests/run_pytorch_test.py -i tests/tt_eager/python_api_testing/sweep_tests/test_configs/ci_sweep_tests_working/grayskull/pytorch_reduce_max_h_test.yaml -o ./result-sweeps
After the run is completed all test sweeps results should be available inside specified output directory (in this case ./result-sweeps). There you will find reduce_max_h_sweep.csv which holds all executed sweeps, among which you can also find the ones that failed and were recreated by the unit test, which you can get by searching unique data_seed field.

Hi @muthutt. Yes, I confirm that the input for each TT operation is a tt_lib.Tensor object.
All unit tests use the file tests/tt_eager/python_api_testing/sweep_tests/tt_lib_ops.py which in implementation (for each operation) contains a call of the setup_tt_tensor function.

No more unexpected index error.


HTML Semantics

HTML Semantics
User Story
As a bootcamp student
I want the HTML semantics to be as correct and in order as possible
so that the information can be easily seen and found
Acceptance Criteria
Given Horiseon's website code
When I visit the website in the browser
Then the HTML elements show in the correct order and follow logical structure

The code now follows the correct HTML flow and format.


Client with serial number xxx does not exist

Hi,
we got a number of clients which report "No Name" in the client overview. When you click on the client it report "Client with serial number xxx does not exist".
Has anyone a clue?

I have this problem only with M1 devices. Somebody reported there is problem with one module (firewall?) causing this.
I am unable to teste since I don't have any M1 for testing purposes.

This could be a thing.

@b-reich Are the "No Name" computers able to successfully complete /usr/local/munkireport/munkireport-runner ?
(After increasing my PHP settings for post_max_size and upload_max_filesize the problematic computers displayed as expected.)

@dan-snelson I will check this, I am using the DockerImage. But This Problem has only a few client ~10. We already have 100+ Clients in there.

@MichalMMac we have M1 Macs wich works without problems.

My Problem was the upload size and the config for our kubernetes ingress.


Investigate why peer_list only returns no or single result

The Issue
@jackrobison noticed that peer list only returns a single result and sometimes no results for blobs that were recently announced.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

It would return no result because of https://github.com/lbryio/lbry/issues/1638
It will currently return a minimal result (the amount returned from the first peer with a peer for the hash).  https://github.com/lbryio/lbry/pull/1645 will update the iterative find class to continue traversing the network after it's found a result if so desired. This will allow the peer protocol client to keep trying new peers and without restarting the search from scratch. It will also allow peer_list to return up to an amount (default of k - 8) of fresh results instead of a single or cached results.


Record postage stamp background level in fits extension

This is a suggestion not an issue. It may be useful to end users to record the sky background subtracted in each postage stamp before calculating centroids. This could provide helpful spatial and time background estimates that end user pipelines could use to identify troublesome scattered light data. Easy to store in the reference star fits extension as a new column.

Reported by CJB in original tessgit repo.


Looking for a .log().ifStatusCodeIsNotEqualTo() or similiar.

When testing 4xx error codes, I want to log if an unexpected response code is given, regardless of the type of code (2xx, 3xx, 4xx, 5xx).
Example:
I expect a response with the HTTP code 409 (CONFLICT) in one of my tests, if the test passes, it should not output any log.
.then().log().ifError() will log, so can't use that.
.then().log().ifStatusCodeIsEqualTo() doesn't really apply to my situation.
So, checking methods in the ValidatableResponseLogSpec class, I'm missing a negated option to .ifStatusCodeIsEqualTo().
I would like to have a .ifStatusCodeIsNotEqualTo().
Are there any sleek way to accomplish this that i've missed out on, or should I do a feature request?
Using version 3.1.0.

How about this:
import static org.hamcrest.CoreMatchers.equalTo;
import static org.hamcrest.CoreMatchers.not;

.then().log()..ifStatusCodeMatches(not(equalTo(409)));


[KYUUBI #916] Send DelegationTokens to SQL engines

Why are the changes needed?
This PR finishes issue #916
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

@yaooqinn @ulysses-you @pan3793 Ready to be reviewed.

Codecov Report

Merging #1074 (b5b6188) into master (1b89272) will decrease coverage by 0.08%.
The diff coverage is 76.28%.

:exclamation: Current head b5b6188 differs from pull request most recent head 3612c30. Consider uploading reports for the commit 3612c30 to get more accurate results

@@             Coverage Diff              @@
##             master    #1074      +/-   ##
============================================
- Coverage     78.70%   78.61%   -0.09%
  Complexity       71       71
============================================
  Files           172      173       +1
  Lines          6324     6411      +87
  Branches        730      737       +7
============================================
+ Hits           4977     5040      +63
- Misses          920      940      +20
- Partials        427      431       +4

Impacted Files
Coverage Δ

.../kyuubi/credentials/HadoopCredentialsManager.scala
94.84% <0.00%> (-0.95%)
:arrow_down:

...a/org/apache/spark/kyuubi/SparkContextHelper.scala
53.84% <40.00%> (-46.16%)
:arrow_down:

...ala/org/apache/kyuubi/util/KyuubiHadoopUtils.scala
70.27% <52.38%> (-23.48%)
:arrow_down:

.../kyuubi/client/KyuubiSyncThriftClientHandler.scala
86.36% <86.36%> (ø)

.../apache/kyuubi/service/ThriftFrontendService.scala
96.73% <90.00%> (-0.27%)
:arrow_down:

...e/kyuubi/engine/spark/SparkSQLBackendService.scala
94.73% <93.75%> (-5.27%)
:arrow_down:

...apache/kyuubi/service/AbstractBackendService.scala
84.09% <100.00%> (+0.36%)
:arrow_up:

.../apache/kyuubi/client/KyuubiSyncThriftClient.scala
82.60% <100.00%> (+0.58%)
:arrow_up:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
78.94% <100.00%> (ø)

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
89.28% <100.00%> (-0.30%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

thanks, merging to master !


Slugify

Gary Bernhardt's Slugify script might have some good ideas: https://github.com/garybernhardt/dotfiles/blob/master/bin/slugify

Archived


Annotate Active Support string extensions

Completes String extensions for https://github.com/Shopify/rbi-central/issues/137
Type of Change

[ ] Add RBI for a new gem
[x] Modify RBI for an existing gem
[ ] Other:

Changes

Gem name: Active Support
Gem version: 7.1
Gem source: https://github.com/rails/rails/tree/7-1-stable/activesupport/lib/active_support/core_ext/string
Gem API doc: https://guides.rubyonrails.org/active_support_core_extensions.html#extensions-to-string

Thanks @paracycle! Addressed your comments so far :)

@kaan do you mean you'd (in theory) like Rails to return an error rather than raise? Or is this actually the valid Sorbet way to write that a method may raise?

Yes the method will have the return instead of raise to make that work which makes what I said even less feasible. Example: https://blog.jez.io/union-types-checked-exceptions/#:~:text=Here’s how we’d write that in Sorbet%3A

Thanks for getting it over the finish line @amomchilov !


Make library relocatable

Me and @jgillis are developing a C interface for MadNLP , so that it can be bundled in a "standalone" shared library and used from non-Julia tools and frameworks.
We are using PackageCompiler.jl to create the library, but we encountered some issues with relocatability, namely having the Julia depot path of the building machine hard-coded in some places.
Those commits seems to fix the problem in our build, even if I am not 100% sure this is the best way to solve the problem, given I am relatively new to Julia.
Would you be interested in making MadNLP.jl relocatable?

Hi @tmmsartor , we found your project on github a week ago and are very curious about it.
We would be in to make MadNLP relocatable, if that could make your life easier. Looking at your PR, it looks like you have identified two issues:

we parse the toml file to get the current version of MadNLP
the MUMPS's interface is using ccall instead of @call
Is that right?

Apparently RelocatableFolders.jl is needed only for 1. Let me check if we can find an alternative to using this new dependency.

Yes I think those are the two issue we found.
I also think first issue can be solved without depending on RelocatableFolders.jl
Regarding the second issue I am not is the problem is @ccall vs ccall or the fact the artifacts lib path is appearing in an eval block.
I notice that only for libmumps there was an hard-coded full path in the library, while other external libraries like cudss or metis were working fine, so I try to adapt the calling style to those cases.

I think it's a good idea to remove the @eval block, and I am in to replace it by @ccall.
If the tests pass and you confirm that you can use PackageCompiler.jl, we can merge this PR.

I was not sure if there was a reason why in MadNLP is defined the lambda/method version() while in MadNLPMumps there is the variable version, to be safe I keep this distinction in the relocatability updates.
Also  pkgversion() return VersionNumber while MadNLP tests seems to expect a string for version()/version so I added that conversion.
I was wondering if there is Julia standard or suggested way to provide this version information.

LGTM. merging it to master


chore(docs,service-utils): add Next.js environment file and remove unused service definition

#3636  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on  Graphite


How to change docker-sys interface IP?

I'm experiencing IP networks collision with my internal VPN (<IP_ADDRESS>/16). Is it possible to change docker-sys interface network in RancherOS?
There is a related issue #436.

Same issue here. I'm looking for a solution otherwise I'm unable to access container services from internal WiFi (in my case) network.

Same here. I figured out how, but it's nasty and I don't even want to post it publicly.

@whiteadam oh go on - if its really horrible, it might inspire someone to make a better solution!

@whiteadam yes, please. Let us know how you workaround this issue.
Thanks.

Since I'm using <IP_ADDRESS>/18 in our network it is impossible to communicate with the RancherOS without changing the docker-sys network.
I tried now every possible config mentioned here and in the documentation with "system_docker" and "extra_args".
Well, it is impossible to change the docker-sys network.
This is an important issue and it is not solved.
I hope you giving us at least some kind of a workaround for this!
Thanks!

BTW: This DOES NOT work in the cloud-config.yml:
rancher:
  system_docker:
    extra_args: [--fixed-cidr,<IP_ADDRESS>/16]

@whiteadam, I would love to know how you got around this. I have this exact issue and if it's really really nasty, I'll find another network in our DC to put my hosts on. I have a tight deadline and don't want to waste too much time spinning my wheels if I'm just going to have to rebuild elsewhere.

@ApolloDS See this issue: #1870


Add role support

Hi,
I think role support and related mutation can be useful too.

@DevSpeak Any news?
Are you still working on this repo?

@mnlbox Hi, I have been gone due to React course, and personal stuff.
I will start working on this again asap.
I think role support would be nice 👍

Any news? Seems like this one supports GraphQL Query/Mutation-based Login/Signup =)

@mnlbox @dalisoft what kind of role support would you like to see?
Based on field in user table or_x-hasura-allowed-roles_?

I will make a new branch for role support today or tomorrow.


cleanup

I would move Application, GithubOauth2SpringToken, and SpringSecurityOauth2GithubGrailsPlugin into the same package as GithubOAuth2Service, and would probably move all to grails.plugin.springsecurity.oauth2.github

Thanks for the cleanup @burtbeckwith!
I just did the package refactor in a separate brach and merged it.
Thanks

@burtbeckwith forgot to ask, do you know the average time to link this to the grails repo? :)

No idea, although it should be very quick. Keep asking in Slack until someone who has access notices.


WIP [17.09] update changelog for 17.09.0-ce release

Signed-off-by: Andrew Hsu<EMAIL_ADDRESS>

I think there's some cherry-picks we need to include in the changelog; I'll try to have a look

@vieux @thaJeztah i've updated with the things that went into the rc3 release

LGTM

oh, was just about to post changes here;
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 59da916474..1d4b065281 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -23,9 +23,16 @@ be found.
 + Add ulimits to unsupported compose fields [docker/cli#482](https://github.com/docker/cli/pull/482)
 + Add `--format` to `docker-search` [docker/cli#440](https://github.com/docker/cli/pull/440)
 * Show images digests when `{{.Digest}}` is in format [docker/cli#439](https://github.com/docker/cli/pull/439)
-* Print timestamp when `--human=true` [docker/cli#438](https://github.com/docker/cli/pull/438)
+* Print output of `docker stack rm` on `stdout` instead of `stderr` [docker/cli#491](https://github.com/docker/cli/pull/491)
+- Fix `docker history --format '{{json .}}'` printing human-readable timestamps instead of ISO8601 when `--human=true` [docker/cli#438](https://github.com/docker/cli/pull/438)
+- Fix idempotence of `docker stack deploy` when secrets or configs are used [docker/cli#509](https://github.com/docker/cli/pull/509)
+- Fix presentation of random host ports [docker/cli#404](https://github.com/docker/cli/pull/404)

+### Logging
+
+- Fix Splunk logger not transmitting log data when tag is empty and raw-mode is used [moby/moby#34520](https://github.com/moby/moby/pull/34520)
+
 ### Networking

 + Add the control plane MTU option in the daemon config [moby/moby#34103](https://github.com/moby/moby/pull/34103)
@@ -42,12 +49,22 @@ be found.
 * LCOW: Remove hard-coding [moby/moby#34398](https://github.com/moby/moby/pull/34398)
 * LCOW: WORKDIR correct handling [moby/moby#34405](https://github.com/moby/moby/pull/34405)
 * Windows: named pipe mounts [moby/moby#33852](https://github.com/moby/moby/pull/33852)
+- Fix "permission denied" errors when accessing volume with SELinux enforcing mode [moby/moby#34684](https://github.com/moby/moby/pull/34684)
+- Fix layers size reported as `0` in `docker system df` [moby/moby#34826](https://github.com/moby/moby/pull/34826)
+- Fix some "device or resource busy" errors when removing containers on RHEL 7.4 based kernels [moby/moby#34886](https://github.com/moby/moby/pull/34886)

 ### Swarm mode

 * Include whether the managers in the swarm are autolocked as part of `docker info` [docker/cli#471](https://github.com/docker/cli/pull/471)
 + Add 'docker service rollback' subcommand [docker/cli#205](https://github.com/docker/cli/pull/205)
+- Fix managers failing to join if the gRPC snapshot is larger than 4MB [docker/swarmkit#2375](https://github.com/docker/swarmkit/pull/2375)
+- Fix "permission denied" errors for configuration file in SELinux-enabled containers [moby/moby#34732](https://github.com/moby/moby/pull/34732)
+- Fix services failing to deploy on ARM nodes [moby/moby#34021](https://github.com/moby/moby/pull/34021)

 ### Packaging

 + Build scripts for ppc64el on Ubuntu [docker/docker-ce-packaging#43](https://github.com/docker/docker-ce-packaging/pull/43)
+
+### Deprecation
+
++ Remove deprecated `--enable-api-cors` daemon flag [moby/moby#34821](https://github.com/moby/moby/pull/34821)
\ No newline at end of file

@thaJeztah moved the 3 networking items to runtime section after going over with @fcrisciani


[Failed Test] Webview button doesn't react

Source information and language
Kavita
Steps to reproduce
Clic open in WebView in any series.
Expected behavior
It opens webview
Actual behavior
It does nothing
Tachiyomi version
ddd180e56aef69380b1da3137dca8aed590b68e1
Android version
n/a
Other details
There's a workaround that attempts to solve this. Copied from mangadex but still won't work
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.13.6.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

This can be solved by open Kavita library in explore page. I think a cache should be maintained to solve this issue.

This can be solved by open Kavita library in explore page. I think a cache should be maintained to solve this issue.

Can you elaborate more? It'd be very helpful.

This can be solved by open Kavita library in explore page. I think a cache should be maintained to solve this issue.

Can you elaborate more? It'd be very helpful.

Open the Kavita source library in this browse page will help solve this problem.
Once that comic appears in the source library. Then WebView button will works.

In my device no matter if I access a serie from the library or from the Browse page, it just doesn't react

In my device no matter if I access a serie from the library or from the Browse page, it just doesn't react

Hmm. That works on my device. Just a little wired.

Technically what i did is the same workaround for other extensions. So it should work. It's just weird how on my device doesn't work.
This is just a though but do you have a domain or ssl encription or smth like that?

My kavita library have a domain and a vaild ssl certificate.

It could be as simple as that ngl.
I'm connecting through direct ip in my vpn with no certificate


Tier fixed/variable rating.

Currently, ManageIQ only supports defining continuous variable rates. This branch allows defining rates based on tiers with both fixed and variable rates. For instance:
Allocated CPU:
    Tier 0: -∞ - 0 CPU = 0$/month + 0$/month/cpu

    Tier 1: 0 - 4  CPU = 3$/month (fixed) + 5$/month/cpu

    Tier 2: 4 - 8  CPU = 3$/month + 4€/month/cpu

    Tier 3: 8 - +∞  CPU = 3$/month + 3.5€/month/cpu

NOTE: The interval end is not included

Hi @rixhack, can you rebase and republish?
Thanks,
Sergio

@gtanzillo @jrafanie please review

What does interval start and interval end mean?  Is that an average during the month?  When I see start and end I think of measurements of time, not the upper and lower range for a cpu usage tier.
I have a hard time understanding the screenshot.  Is this the only chargeback related equation we might use for cpu, memory, storage, etc.?
(CPU * fixed rate) + (CPU * variable rate)

@rixhack I made some initial comments and small suggestions.  Hopefully you can help explain some of the complexities that I do not understand.
Sorry, I couldn't finish reviewing this.  I went through about 1/3 of it.  I'll try reviewing more a bit later.
In the future, please keep pull requests small.  I have a  hard time reviewing pull requests with more than 200 or so net line changes.

@jrafanie Yes, sorry for that, this pull request grew quite dramatically because we wanted to keep all the functionality together. I will definitely do smaller pull requests in the future. Thank you for taking the time to look at it.

@dclarizio Looks fine

@rixhack can you rebase and republish?

@gtanzillo @jrafanie @dclarizio   could you have a look at this one? We think we are done

@rixhack @tledesma, can you please help with this?

Here are the notes from out code review today:

We agreed that @rixhack would record a small demo of how an end user would create and edit rates and tiers sonce the live demo was not possible.
Need to change "Interval" to "Range" in the tier heading.
Also in the tier heading, the range section should be laid out the same way as the rate section where the "Range" heading should appear once above the "Start" and "End" columns.
The "Description" column is wrapping in the cell and should be fixed.
The add and delete tier buttons need some review for style and placement. @epwinchell can you please take a look?
We were pretty sure that there is a specific reason for allowing negative values in the tier start and end but couldn't recall what it was. @sergio-ocon can you please provide some details?

Negative values are for:

discounting
assets (i.e. free calls)

@rixhack, I think we should  change this PR to positive values only, we can do the change afterwards if we need it.

So the default tier would be 0-infinite
We need to check there are no negative values
Update tests

@rixhackSuggestions:

move the "Per Time" and "Per Unit" columns to the left

rename "Add Remove Tier" to "Actions"

Change buttons to use words "Add" and "Delete" instead of using icons (similar to the Patternfly examples here: https://www.patternfly.org/list-view/ and here: https://www.patternfly.org/patterns/table-view/

Hi @rixhack, are you using any workaround for this issue https://github.com/ManageIQ/manageiq/issues/6979 ? Thanks!

@serenamarie125 @cshinn @epwinchell
What about  TreeGrid Table?
For the tiers... Does it make sense?

Hi @lpichler I've included your workaround and it seems to work for me.

@sergio-ocon I'd don't see how tree grid would apply here. Can you explain?

@epwinchell, we have changed the user interface according to your considerations. Could you take a look?

Note, I'm really concerned with the following cops:
Line 402, Col 3 - Metrics/AbcSize - Assignment Branch Condition size for cb_tier_add is too high. [62.44/15]
Line 421, Col 3 - Metrics/AbcSize - Assignment Branch Condition size for cb_tier_remove is too high. [41.5/15]

These are new methods.  It should not be this complicated.  http://www.rubydoc.info/gems/rubocop/0.37.2/RuboCop/Cop/Metrics/AbcSize

We do not understand the warnings from app/views/chargeback/_cb_rate_edit.html.haml, app/views/chargeback/_tier_first_row.haml, app/views/chargeback/_tier_row.haml.
Seem like a issue of haml-lint gem because in others haml files that we dont modify, we have the same warnings

@amaurygonzalez @rixhack @tledesma, @gtanzillo and I are looking at this PR... we'll try to get it green.

Please don't push to this branch.

@jrafanie ok, thanks!

@rixhack @tledesma We've pushed some commits but we still need to write tests for the controller actions (add/remove) and possibly others.  We also noticed that we're storing activerecord objects in the session so we will probably need help from @h-kataria or @dclarizio on that.
We still have the validations so we don't save the tiers or details on errors.   We still have to review/cleanup a little more so we'll try to let you know when it's ready for your review.

@jrafanie @gtanzillo Thank you very much for your help, I'll be here for anything you need.

I would suggest:
lower_boundary/upper_boundary
start/finish

:+1: Finally ready to merge. I've created a Trello card with a punch list of items that still need to be addressed - https://trello.com/c/jptqO7w3
Thanks @rixhack, @tledesma, @amaurygonzalez, @sergio-ocon, @jrafanie, @lpichler and @h-kataria for all the hard work!


Provide a public method on LMViewBuilder that will apply template properties to a view instance

The following method has been added to LMViewBuilder:
+ (void)setPropertyValues:(NSDictionary *)properties forView:(UIView *)view;

Renamed setPropertyValues:forView: to applyPropertyValues:toView:.


TaskRun Pod exceeded available resources

Expected Behavior
Pod init success
Actual Behavior
Task Step Resources configuration:
resources:
limits:
cpu: '1'
memory: 1Gi
requests:
cpu: 20m
memory: 10Mi
TaskRun status is 'TaskRun Pod exceeded available resources, reattempted 1 times: pods "hystrix-test-statefulsets-pod-f36c2f" is forbidden: exceeded quota: project1030-quota, requested: limits.memory=9Gi, used: limits.memory=1424Mi, limited: limits.memory=10Gi', why?

@zjq123456 thanks for the issue. Can you share the whole task yaml ? (or at least a reproductible yaml). It might be a bug on how we handle those limits.
/kind question
/kind bug
/triage needs-information

Given that this issue is now 4 months old I'm going to close it. @zjq123456 please go ahead and reopen if you want to provide the requested information and we can continue to help from there.  Many thanks!


fix: Debug the method ImageCollectionAccessor.groupInterval

PR related to issue #401
New changes:

Add three new tests:

The first is a test that ensures that each collection in the list returned by ImageCollectionAccessor.groupInterval is not empty.
The second one ensures that the returned image collection must have the same parameters as the original collection (i.e. no new parameters in the collection).
The third is the one that ensures that this change actually fixed the second problem in #401.

Refactor in the method ImageCollectionAccessor.groupInterval: I applied the changes mentioned in the issue #401.

Hello again, I noticed something new:
When one uses the reduceInterval method, a pipeline could be:
imgCollection = (
  ee.ImageCollection(...)
  .geetools.scaleAndOffset()
  .geetools.spectralIndices(...)
)
reducedImgCollections = imgCollection.geetools.reduceInterval("median", "day", 1)

But this pipeline is really slow: in my case, getting an image takes 2 minutes and 10 seconds. But, if I do something like
imgCollection = ee.ImageCollection(...)
reducedImgCollections = (
  imgCollection.geetools.reduceInterval("median", "day", 1)
  .geetools.scaleAndOffset()
  .geetools.spectralIndices(...)
)

Getting an image takes 20 seconds (10 times faster). But this change implies another thing: When one tries to do this, another error occurs because the reduced images in the collection do not have an image id. So, when I create the image in the reduceInterval method, I copy the properties (including the image id of the first img) into the new reduced image.
I feel this is not good practice, but it fixes my bug. what do you think? Do you think this change could be included in this method?
PS: I'm done with the PR changes, in case you'd like to check them again.

I feel this is not good practice, but it fixes my bug. what do you think? Do you think this change could be included in this method?

It could definitely be added in the method, either in this PR or in a follow-up one.

I feel this is not good practice, but it fixes my bug. what do you think? Do you think this change could be included in this method?

It could definitely be added in the method, either in this PR or in a follow-up one.

Ok, I added this changes in the last two commits.
To summarize, they copy the properties of the original one. But is only tested ensuring that they have the "system:id" property. I did that because it is used for the _get_platform_STAC.


Integration with hyperscript?

I'm trying to use this with hyperscript which is used in https://github.com/skatejs/skatejs but it seems its not populating the style tag in head?
var styles = {
  navigation: style({
    fontSize: '15px',
    width: '20em',
    height: '100%',
    color: 'white',
    background: `rgba(0, 0, 0, 1)`,
    position: 'fixed',
    top: '0',
    right: '0',
    bottom: '0',
    zIndex: '2000',
    transition: 'right 0.3s ease-in-out 0s',
  }),
}

skate.h('div', styles.navigation, 'hello')

This is how it looks,

yes, glamor doesn't have first class support for web components/iframes yet. working on it for v3 though.

so what's happening here because you're using te minified version, is it is populating in head tag, but using CSSOM so it won't show in the elements tree. further, since it's inside this shadow dom thing, the head css doesn't affect it iirc.

duplicate of #38

Can confirm ^
You need to use the cssFor named export for your styles:
import { cssFor, style } from 'glamor';

// your styles
const css = style({ ... });

// and later in renderCallback()...
<style>{cssFor(css)}</style>
<div {...css} />


std.math: add round overload to round to a given precision

Python has this. I often find it useful. It was trivial to implement. So I submit this.

This implementation seems a bit naïve... floating-point semantics may have some unexpected corner cases that are not handled correctly here. I'll let the numerics guys comment.

@quickfur Adding the "math" label would notify those people.

My difficulties with this are:

It's 37 lines added to Phobos to support a 2 line, trivial function. Phobos should consist of non-trivial functions.

It doesn't take into account NaN, overflow, or underflow.

It's 37 lines added to Phobos to support a 2 line, trivial function. Phobos should consist of non-trivial functions.

Rounding a number to a specific precision is extremely common; I was surprised it didn't exist when I saw this PR.

@WalterBright wrote:

It's 37 lines added to Phobos to support a 2 line, trivial function.

The function itself is still only two lines. It's the unittest and ddoc comments that takes up more lines (but still not 37!). Surely you were the guy who had the foresight to integrate unittests and documentation into the language! :-)

Phobos should consist of non-trivial functions.

Phobos should make the user's life easier. :-)

It doesn't take into account NaN, overflow, or underflow.

OK please advise as to how it is supposed to take care of it. I noted @tsbockman's comment about 10.0L – but apart from that I presume that since I'm just writing a wrapper, the inner function will take care of those stuff. Perhaps it's naive, but I'm not a professional programmer nor math computing expert to know how to take care of these nuances. I'm a humanities person who loves D for its elegance and want to contribute my mite. So please advise how and I will correct the code. Thanks.

ping @jamadagni
From what I can tell, the following needs to be done:

Merge the ddoc headers of the two functions, so that they are described by the same documentation text. The second overload (the new version of round) should be preceded by a /// ditto line so that ddoc knows that it belongs with the previous overload as far as documentation generation is concerned.
Change 10^^prec to 10.0L ^^ prec.
I'm not sure what there is to do about NaN. If the input argument is NaN, all of the operations performed by the function will still result in NaN, so it will return NaN, which is probably the only correct thing to do.
However, the issue of overflow may be a problem if prec is large, or if the input argument is near the upper limit of representability. Perhaps a better way might be to compute something that can be subtracted from the argument (avoiding overflow risk with the multiply and divide).  In general, though, implementing floating-point code, esp. for the standard library, is tricky business; perhaps what we need is a numerics expert (I'm not one of them) to look at this code and suggest the best implementation.

@quickfur

Change 10^^prec to 10.0L ^^ prec.

This only matters if we choose to ignore (4), as solving (4) will involve replacing this part, anyway.

I'm not sure what there is to do about NaN. If the input argument is NaN, all of the operations performed by the function will still result in NaN, so it will return NaN, which is probably the only correct thing to do.

Yes. I don't know why the NaN issue was raised.

However, the issue of overflow may be a problem if prec is large, or if the input argument is near the upper limit of representability. Perhaps a better way might be to compute something that can be subtracted from the argument (avoiding overflow risk with the multiply and divide). In general, though, implementing floating-point code, esp. for the standard library, is tricky business; perhaps what we need is a numerics expert (I'm not one of them) to look at this code and suggest the best implementation.

This is a problem. I have a vague idea what the solution might look like, and it involves completely rewriting the implementation.
On the plus side, I suspect the resulting algorithm would be considerably faster than the naive one. I think I'll take a crack at it more for that reason, than because I actually care about being able to round 1.189731495357231765e+4932 to the nearest 1e+4932.

Yeah, just as I thought, the naive implementation doesn't work very well in the boundary cases. And yes, it's an interesting challenge to find a way to implement this function that maximizes accuracy and proper return values in all cases. I think that's the draw of this kind of code, anyway. :-)  IMO, any application that needs exact decimal roundings really ought to be using fixed point instead of floating point. Financial applications, for one thing, have nasty consequences if floating-point issues are not properly dealt with. Best to play it safe and use fixed-point.

I think that's the draw of this kind of code, anyway. :-)

Yup.

@tsbockman @quickfur I'll be happy to close this in favour of a better implementation, only so long as the feature is implemented. As noted above, methinks nearby would be a good name since it parallels the behaviour of nearbyint.

@tsbockman @quickfur I'll be happy to close this in favour of a better implementation, only so long as the feature is implemented. As noted above, methinks nearby would be a good name since it parallels the behaviour of nearbyint.

So it's either close it now or get it updated with issues addressed and merge, there is no third option.

@DmitryOlshansky and @jamadagni : I have submitted PR #4219, which generalizes this and fixes the issues listed by @quickfur .

Superseded by #4219


Please add traditional chinese (zh_TW) translations

Title
Thank you

Hi @jazonshou
As far as I can tell, this language is already added but no one has translated anything yet:
https://www.transifex.com/readthedocs/sphinx-rtd-theme/language/zh_TW/
If you are a translator and speak traditional Chinese (don't you mean Taiwanese?), you can create an account on Transifex and translate the theme.


This type inference of static property getter

Bug Report
It's possible to infer the this type contextually on a static method so inherited classes get their own type as this when calling an inherited static method. But in static property getters it's not possible to specify generic parameters, so it's not possible to get the this type of a derived class in a parent getter.
In the example below, Bar.default returns an instance of Bar but it's type is inferred as Foo.
type Constructor<T> = { new (): T }

class Foo {
  static factory<T extends Foo>(this: Constructor<T>): T {
    return new this()
  }

  static get default() { // Not possible to specify type parameters
    return new this()
  }
}

class Bar extends Foo {}

const x = Bar.factory() // x is type Bar
const y = Bar.default // y is type Foo

// however, both are instances of Bar
console.log(x instanceof Bar) // true
console.log(y instanceof Bar) // true

Ideally, I think both getter and method should work without any extra type parameters needing to be specified. Static this type should be inferred contextually.
class Foo {
  static factory(): this {  // TS2526: A 'this' type is available only in a non-static member of a class or interface.
    return new this()
  }

  static get default(): this {
    return new this()
  }
}

Edit: I'm seeing some comments in other threads about new this() not being allowed because the caller's ctor signature isn't known in the parent class. That's understandable. This is a demonstration of the problem of type inference, this is not an actual use case. My actual use cases don't invoke any unknown constructors. So please don't dismiss this because you think the use case isn't viable.
🔎 Search Terms

static property getter
static property this

Related: https://github.com/microsoft/TypeScript/issues/5863.

The solution for the specified problem here is #5863.

My actual use cases don't invoke any unknown constructors. So please don't dismiss this because you think the use case isn't viable.

We can't just do things and ignore their side effects...

I must have missed it. I don't see a solution for static getters.


0 vs falsy stackTraceLimit

https://github.com/Agoric/SES-shim/blob/96edfabc94b25e61245d0e92e7cafa1d3ebfa764/packages/ses/src/tame-global-error-object.js#L62

Fixed by #358


timer in function is not used, can be removed

timer is not used in StorageEngine::get_all_data_dir_info.
You should remove timer related code in StorageEngine::get_all_data_dir_info which locates at be/src/storage/storage_engine.cpp.

I'll take this one - Submitting pull request shortly.

Submitted Pull Request: https://github.com/StarRocks/starrocks/pull/5510
Please review :)


Intel Fix: B-14 Wiretap Transcript [forsaken][RS6Do2]

Name
B-14 Wiretap Transcript
Intel Id
RS6Do2
Fix Needed
location is outbreak, not forsaken
Any Other Info
No response

https://github.com/Miss-placed/DECLASSIFIED/commit/940dfabe9174a1c27dd7280004ea632a3534695f

@cliftonm1996 thanks for the contribution, your edits have been applied!
Thanks for using the app 😄


Add feature importance groupings to Tabular

Add ability to specify a group of features to get an overall importance for the group.

Added in #989


EasyAsk (FlixCheck)

We send out approx. 100'000 easyAsk checks per year. The checks (e.g. asking our customer to add some more information on his / her claim) need to be adjusted according to the new branding. Flixcheck will do this for us.

Accounts have beenn created (thanks @Gagne87). Flixcheck will be informed to contact @christophgeering in case of questions. Right now: no further actions required - waiting for a draft to review


SCRCPY issue for mirror screen

I’m facing a problem with SCREEN MIRRORING my device ( android ) on SCRCPY.
After installing all the requirements on Terminal using homebrew and getting ( brew install android-platform-tools ) installed successfully.
Everytime I plug my device and entre Terminal and then writing ( SCRCPY ) I get this message : ((( -bash: scrcpy: command not found )))
I dont know what is this, if you know please help me out.

you need to add it to environment variables

and how I'm suppose to do that, I have no clue....

look it up on the internet, you will find info about how to add command to system environment in linux

look it up on the internet, you will find info about how to add command to system environment in linux

I have MacOs system, maybe that’s why

it is the same, just look it up..

eg. https://phoenixnap.com/kb/set-environment-variable-mac

you will want to add it to PATH var

https://phoenixnap.com/kb/set-environment-variable-mac

should I copy and paste this : ( echo $PATH ) on Terminal ?

that will only print your path. How did you install brew? Is it installed by default on macOs? Did you run the 'brew install scrcpy' command passing your password (with sudo rights) or just for your user?

'brew install scrcpy' command passing your password (with sudo rights)

That’s how the process went toward installing HomeBrew and also  : 'brew install scrcpy' command passing your password (with sudo rights) .....

unfortunatelly i cannot reproduce on Linux because of this current issue with homebrew I just opened: https://github.com/Genymobile/scrcpy/issues/2902

If you install Homebrew with official script, it links itself and all installed packages to /usr/local/bin, which is in $PATH on macOS by default.
You can run brew doctor to diagnostic common configuration issues.
Judging from your default shell is still bash, are you using an old version of macOS or have upgraded multiple times?

When I typed ( brew doctor ) on Terminal , I got this message :
.
.
.
Warning: You are using macOS 10.13.
We (and Apple) do not provide support for this old version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Twitter or any other official channels. You are responsible for resolving
any issues you experience while you are running this
old version.
Warning: Homebrew's "sbin" was not found in your PATH but you have installed
formulae that put executables in /usr/local/sbin.
Consider setting your PATH for example like so:
echo 'export PATH="/usr/local/sbin:$PATH"' >> /Users/kolli/.bash_profile

well, it basically says what to do
echo 'export PATH="/usr/local/sbin:$PATH"' >> /Users/kolli/.bash_profile

well, it basically says what to do echo 'export PATH="/usr/local/sbin:$PATH"' >> /Users/kolli/.bash_profile

what I'm suppose to do in this case, should I just copy and paste ( echo 'export PATH="/usr/local/sbin:$PATH"' >> /Users/kolli/.bash_profile ) or there's something more complicated than that ?

After installing all the requirements on Terminal using homebrew and getting ( brew install android-platform-tools ) installed successfully.

Did you successfully installed scrcpy with brew install scrcpy?

After installing all the requirements on Terminal using homebrew and getting ( brew install android-platform-tools ) installed successfully.

Did you successfully installed scrcpy with brew install scrcpy?

yeah sure, everything went successfully

Just run echo 'export PATH="/usr/local/sbin:$PATH"' >> /Users/kolli/.bash_profile in a terminal, then restart all terminals.


tmsu: use $out instead of $bin

Motivation for this change
Things done

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

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

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

cc @Mic92

Because of https://github.com/NixOS/nixpkgs/pull/85535

Amend the commend while squashing.


fix: update subdependencies to fix security warnings

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: