Qualified server actions

As discussed, added "Rest" prefix on endpoints related Actions.
Obviously, searching for logs that happened before this merge won't work anymore, as the action changed.

@Ninjeneer Does it impact the current mobile app in prod?

@Ninjeneer Does it impact the current mobile app in prod?

No it should not. ServerActions that have been changed are related to already migrated endpoints, meaning dashboard and mobile doesn't call any of them
Thanks to Claude I've been able to test locally, everything still works

@Ninjeneer Does it impact the current mobile app in prod?

No it should not. ServerActions that have been changed are related to already migrated endpoints, meaning dashboard and mobile doesn't call any of them
Thanks to Claude I've been able to test locally, everything still works

Gut, Thanks a lot!


AZ-204_lab_12.md

English:
Don't select.
Current Translation:
Ne sélectionnez rien.
Suggested Translation:
Désactivez la case à cocher.
Description:
Lack of accuracy. Correction made to ensure global consistency (same issue next line)

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run bugfix tool and waiting for result ICMS QUERY


Support Alias for Stream/Table in Join clause

To allow self join

Defer to the next milestone after TSC discussion


Can not contact with email<EMAIL_ADDRESS>?

Hello,
Thank for your work. I try to contact<EMAIL_ADDRESS>but they don't reply me. Is there any other way for me to get the dataset ?
Best regard !

Did you get the access? If yes can you please tell me after how many days you got the access and how did you approach them?

I didn't receive any response of author :(


Azure Backup and Replication

Azure Backup and Replication is not found on the script. Please add the reprot with full details - backup triggred, completed time, backup status i.e completed or failed with backup policy.

Hello there @BasudevWB
Just added a new Backup module, give it a try.


Introduce a temporary fix for verification cores set to 0

Introduce a temporary fix so setting verification cores to 0 (the default) doesn't stop verification

I'll approve it because it's urgent but I would like to have a follow-up PR with a test to ensure that, if the number of cores is not specified, that the command line used to run DafnyLanguageServer is not set to zero cores.

Note that we're also updating the language server to allow 0 cores: https://github.com/dafny-lang/dafny/pull/3276


Possible timing issue with completing coloured tiles and enemy collision

Game mode: coloured tiles on
User found that when they stepped on a tile, which made all tiles coloured and then immediately was hit by a bug the game was left in a weird state where the player was reset, but the last tile that was stepped on was coloured and the game points were not reset.

Hmmm, if a player is hit by an enemy all the points are reset (high score updated, if warranted), so even if there was a timing issue with being hit by an enemy and the square underneath being counted, the points would be set to 0.  I suspect that the player fell in the water, narrowly missing an enemy, which would cause  a loss of points and the player position to be reset, but not their points.  I have not been able to reproduce the above by being hit by an enemy, but I will keep a watch out for this in the future.


quiche: update tarball to d88a2f7a9ff5f9f6be2f50411b15b091affe04d3

Update QUICHE to d88a2f7a9ff5f9f6be2f50411b15b091affe04d3 to fix the issue that MSVC doesn't support designated initializers, see https://github.com/envoyproxy/envoy/pull/11316#issuecomment-642675562
Because QUICHE depends on higher version of Boringssl, also update Boring SSL to chromium-84.0.4147.45(beta).
Risk Level: low

/assign @wu-bin @PiotrSikora

/assign @mattklein123

Looks like QUIC code coverage is lower (please try to boost it above 97 in a neat-term future PR).
/wait

Looks like QUIC code coverage is lower (please try to boost it above 97 in a neat-term future PR).
/wait

It turns out that this QUICHE update causes all gQUIC test to be skipped. The previous way of testing both gQUIC and iQUIC is not robust. Fixed now. PTAL

Looks like coverage is still low. :(
/wait

enable more test for gQUIC with Quic Crypto. PTAL


Change in jax 0.4.15 breaks distributed.py on multi-node GPU setups, fix inside

Description
JAX 0.4.15 makes this change:
"The option --jax_coordination_service has been removed. It is now always True."
jax_coordination_service was removed from jax's config. This causes an exception in jax/_src/distributed.py, since it tries to access "config.jax_coordination_service".

Line 80 of jax/_src/distributed.py:
                      coordinator_address, num_processes, config.jax_coordination_service)
can be changed to:
          coordinator_address, num_processes, True)
and line 92:
          coordinator_address, process_id, config.jax_coordination_service,
can be changed to:
          coordinator_address, process_id, True,
This appears to fix the issue. Training using a multi-host GPU-based setup appears to work again.
What jax/jaxlib version are you using?
jax==0.4.15? (-e git+https://github.com/google/jax@eea2603363db1ef07b7017d498c96ec744e71218#egg=jax),  jaxlib==0.4.14+cuda12.cudnn89
Which accelerator(s) are you using?
GPU
Additional system info
RHEL 8.5, NVIDIA DGX A100
NVIDIA GPU info
Fri Aug 25 10:41:01 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.86.10              Driver Version: 535.86.10    CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA A100-SXM4-80GB          On  | 00000000:01:00.0 Off |                    0 |
| N/A   26C    P0              66W / 500W |  77444MiB / 81920MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+
|   1  NVIDIA A100-SXM4-80GB          On  | 00000000:41:00.0 Off |                    0 |
| N/A   25C    P0              68W / 500W |  77444MiB / 81920MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+
|   2  NVIDIA A100-SXM4-80GB          On  | 00000000:81:00.0 Off |                    0 |
| N/A   25C    P0              68W / 500W |  77444MiB / 81920MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+
|   3  NVIDIA A100-SXM4-80GB          On  | 00000000:C1:00.0 Off |                    0 |
| N/A   25C    P0              71W / 500W |  77444MiB / 81920MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+
+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|    0   N/A  N/A    860464      C   ...h/sblouir/mount/venvs/p0/bin/python    77430MiB |
|    1   N/A  N/A    860496      C   ...h/sblouir/mount/venvs/p0/bin/python    77430MiB |
|    2   N/A  N/A    860550      C   ...h/sblouir/mount/venvs/p0/bin/python    77430MiB |
|    3   N/A  N/A    860582      C   ...h/sblouir/mount/venvs/p0/bin/python    77430MiB |
+---------------------------------------------------------------------------------------+

#17290 will fix. Thanks for the bug report!


REF: Move nn.bs to nn.modules.bs, Move Hedger to nn.modules

REF: Move nn.bs to nn.modules.bs (close #55)
REF: Move Hedger to nn.modules (close #56)

Codecov Report

Merging #62 (15ddbbe) into develop (64272f2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop       #62   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           31        31
  Lines          831       828    -3
=========================================
- Hits           831       828    -3

Impacted Files
Coverage Δ

pfhedge/nn/modules/bs/_base.py
100.00% <ø> (ø)

pfhedge/nn/modules/bs/bs.py
100.00% <ø> (ø)

pfhedge/__init__.py
100.00% <100.00%> (ø)

pfhedge/nn/__init__.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/bs/american_binary.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/bs/european.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/bs/european_binary.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/bs/lookback.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/hedger.py
100.00% <100.00%> (ø)

pfhedge/nn/modules/loss.py
100.00% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


doc: fix git url example tag

1. Does this PR affect any open issues?(Y/N)

[ ] N
[x] Y

fix #252
2. What is the scope of this PR (e.g. component or file name):

kpm/docs
Fix:

cc @Peefy

Hi @octonawish-akcodes 😃, thanks for this issue.
We have added a new tag v0.1.0 for awesome-kusion.
And we have decided to gradually deprecate the kpm command and replace it with the kcl mod command.
For more details, please refer to:
https://github.com/kcl-lang/kpm?tab=readme-ov-file#note-
https://kcl-lang.io/docs/tools/cli/package-management/command-reference/init/


availability

Hi,
it is not clear to me how to gather availability for a given room.
Thanks,
S.

Hi,
I do not have good availability data for a room. Inside Airbnb collects calendar data which shows data on which a room is available, but they also do not show actual occupancy.  AirDNA claims to collect occupancy, but it's also an estimate.
The best I can do is use the number of reviews as a proxy. Recent research suggests that about 55% of visits leave a review, so that's all I go by. It is not going to be accurate for an individual room of course, but I think it's reasonable for tracking aggregate trends in occupancy.
Tom

hi Tom,
it appears to me there is a calendar_month method in the undocumented airbnb API: https://www.airbnb.com/api/v2/calendar_months. Are you using this?
you just have a way to tell if the room is booked or not, and do not have any data on how many persons are sleeping in the room? is that what you are saying?
Simone.

Hi,
Hope Tom doesn’t mind I answer this.
Tom’s project isn’t using the undocumented Airbnb API but scraping the
website. To use the undocumented API you have to supply your user
credentials and they can detect the huge amount of requests, probably
suspending your account for abuse.
Even with scraping you have to use a set amount of proxys, with different
IPs, so they don’t block you.
The information of how many people are going to stay in a Airbnb property
is not public. Data resellers just do an estimation based on the public
rental information.

@joao

On Thu, Aug 24, 2017 at 4:00 PM, Simone Cortesi<EMAIL_ADDRESS>wrote:

hi Tom,
it appears to me there is a calendar_month method in the undocumented
airbnb API: https://www.airbnb.com/api/v2/calendar_months. Are you using
this?
you just have a way to tell if the room is booked or not, and do not have
any data on how many persons are sleeping in the room? is that what you are
saying?
Simone.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tomslee/airbnb-data-collection/issues/24#issuecomment-324661405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAEEZv6EEn_Usq7Bh4MYCeJQURWx1z3ks5sbZARgaJpZM4O-yhU
.

Thanks for the info!


on click listener added in submit handler

Looks good, just waiting for @lauragieco for a quick review


This repo needs a new maintainer!

If you'd like to be that person, reply here!

Hi,
since we use stretcher in production and i ported the gem to crystal i would like to be that.

@Ragmaanir I've just invited you to the organization! Which email address would you like associated with rubygems.org ?

@andrewvc thanks. email: my github name ät gmail dot com.


Bump to 2.1.2 and fix

Bump to 2.1.2 and change the build procedure, because GitHub rate limited.

This is the 2.0 branch, not 2.1.

The docker team are behind with their releases though. Sorry about that.

Ping @djdefi.


Add all tasks into one workflow file.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@600a806). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #33   +/-   ##
=======================================
  Coverage        ?   97.41%
=======================================
  Files           ?        5
  Lines           ?      310
  Branches        ?        0
=======================================
  Hits            ?      302
  Misses          ?        8
  Partials        ?        0

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


Tutorials section present twice

The tutorials section is present twice in the Web3 Resources.

Hey Francesco, can I merge both the tutorials sections into one?

they are 2 sub-sections both for solidity and vyper, we can change the title accordingly to make it more clear
e.g.
Solidity tutorials
Vyper tutorials

Okay I can do it. Can you please assign it to me?

closing this. please refer to #191


Propagate environment variables to the Docker process

Description
Environment variables are not propagated to the Docker process since ProcessContext#getEnvironment returns an empty Map by default. This causes org.apache.commons.exec.Executor to not use parent process environment variables (it will use them if the environment is not specified or if it's null).

/integration-test

Integration tests have been started. Please click here to see the status.


A few linux <> host behavior changes

1: If the host does not support the timer, many things won't work especially our GUI. Have the extensions at least return false for hasgui if there's no timer
2. We start the timer in ::init since JUCE needs it pretty early. That led to some controversy https://github.com/free-audio/clap/discussions/111 and although we thought it was OK, it may be more defensive to have init request a main thread callback to start the timer.

I reverted that change because while it fixed the Linux issue it segv bws 43b8 macos. @abique has that fixed so when 43b9 is out we can reapply and merge that branch


template.ParseGlob if pattern  read /dev/* files  ，program will  hanged on and can  Dos attack

What version of Go are you using (go version)?
mac os:
➜  ~ go version
go version go1.16.5 darwin/amd64
linux os:
[root@VM-0-5-centos ~]# go version
go version go1.16 linux/amd64
Does this issue reproduce with the latest release?
go version go1.16.5 darwin/amd64
What operating system and processor architecture are you using (go env)?
go env Output
What did you do?
https://play.golang.org/p/dArYQMl7el6   Update code on  play and  Program exited: status 137.
mac os:
func main() {
data := []byte("/*d*/*p")
_, err := template.ParseGlob(string(data))
if err != nil {
fmt.Println(err)
}
}
notice:
/*d*/*p  in mac os  match  /dev/io8logtemp
linux os:
package main
import (
"fmt"
"html/template"
)
func main() {
    data := []byte("/dev/*6")

    _, err := template.ParseGlob(string(data))
    if err != nil {
            fmt.Println(err)
    }

}
/dev/*6 in linux os match many
[root@VM-0-5-centos cor0ps]# ls -al /dev/*6
crw--w---- 1 root tty 4,  16 Jun 11 11:57 /dev/tty16
crw--w---- 1 root tty 4,  26 Jun 11 11:57 /dev/tty26
crw--w---- 1 root tty 4,  36 Jun 11 11:57 /dev/tty36
crw--w---- 1 root tty 4,  46 Jun 11 11:57 /dev/tty46
crw--w---- 1 root tty 4,  56 Jun 11 11:57 /dev/tty56
crw--w---- 1 root tty 4,   6 Jun 11 11:57 /dev/tty6
crw-rw---- 1 root tty 7,  16 Jun 20 11:48 /dev/vcs16
crw-rw---- 1 root tty 7,  56 Jun 20 16:59 /dev/vcs56
crw-rw---- 1 root tty 7,   6 Jun 11 11:57 /dev/vcs6
crw-rw---- 1 root tty 7, 144 Jun 20 11:48 /dev/vcsa16
crw-rw---- 1 root tty 7, 184 Jun 20 16:59 /dev/vcsa56
crw-rw---- 1 root tty 7, 134 Jun 11 11:57 /dev/vcsa6
What did you expect to see?
程序死循环读取文件了，一直卡在读取文件了，就会造成dos攻击
Program loop read file, has been stuck in the read file, will cause DoS attack。
maybe i think reasons here:
/usr/local/go/src/html/template/template.go ---> readFileOS  --->os.ReadFile have  loop function
What did you see instead?
i think ParseGlob should  check files  on host operating system's file system, and  I am  test  on   host operating system's file system  not  file system ,so  I  don not  need use  ParseFS。
// ParseFS is like ParseFiles or ParseGlob but reads from the file system fs
// instead of the host operating system's file system.

I'm just a by stander here. But ... why you must read /dev? Because of the pattern /*d*/*p? Then why you want to use that pattern?

I'm just a by stander here. But ... why you must read /dev? Because of the pattern /*d*/*p? Then why you want to use that pattern?

I'm just a by stander here. But ... why you must read /dev? Because of the pattern /*d*/*p? Then why you want to use that pattern?

maybe I  fuzz files on  parseGlob() ，I  find  /d/p pattern  matchs   /dev/ file  and read /dev/* exists file  will loop read always  and program  will   hanged on

I find /d/p pattern matchs /dev/ file and read /dev/* exists file will loop read always and program will hanged on

This can be separated in to two parts:

The word "Glob" in the function name template.ParseGlob actually means Glob Pattern. The * symbol "matches any number of any characters including none". So /*d*/ in your program means "as long as the name of the folder contains d", that includes /dev, of course.
You can try run ls /*d*/*p on your computer, it should yield the same result. So I think the behavior is expected. (But I'm not the boss there, mind you :))
And about the looping read part: there are many "nasty" or "weird" things in /dev, (/dev/random, /dev/sda... to name a few) you probably don't want to read it directly.

So my question remains: Why you want to use that pattern?

Don't use template.ParseGlob with a string that is not under your control.  That's not what it's for.
I don't understand how this bug report could be fixed.  It seems equivalent to saying that it is buggy to call `os.Open("/dev/tty6").

I find /d/p pattern matchs /dev/ file and read /dev/* exists file will loop read always and program will hanged on

This can be separated in to two parts:

The word "Glob" in the function name template.ParseGlob actually means Glob Pattern. The * symbol "matches any number of any characters including none". So /*d*/ in your program means "as long as the name of the folder contains d", that includes /dev, of course.
You can try run ls /*d*/*p on your computer, it should yield the same result. So I think the behavior is expected. (But I'm not the boss there, mind you :))
And about the looping read part: there are many "nasty" or "weird" things in /dev, (/dev/random, /dev/sda... to name a few) you probably don't want to read it directly.

So my question remains: Why you want to use that pattern?

You're right. If an external attacker can't query the host system with ls  -al , then we can only guess whether there is a file under / dev / * by using an expression. So / * d * / * p * is used. Similarly, the P letter can be replaced at will until it matches the existing file

Don't use template.ParseGlob with a string that is not under your control. That's not what it's for.
I don't understand how this bug report could be fixed. It seems equivalent to saying that it is buggy to call `os.Open("/dev/tty6").

maybe  you  said  right

Don't use template.ParseGlob with a string that is not under your control. That's not what it's for.
I don't understand how this bug report could be fixed. It seems equivalent to saying that it is buggy to call `os.Open("/dev/tty6").

https://play.golang.org/p/dArYQMl7el6   update code  on  play code  and you  can  watch  it

https://play.golang.org/p/dArYQMl7el6                  Update code on https://play.golang.org/

Closing as working as intended / dup of #46835


Translating after Google Translator Toolkit

Google Translator Toolkit is shutting down and I do not know what other software would translate the files generated by intl_translation.
Having said that, I was not sure where to open the issue, so there is one over there as well: https://github.com/dart-lang/intl_translation/issues/74

Since Google Translator Toolkit seems to be the only platform supporting ARB files, maybe an alternative for intl would be to produce a JSON format that is supported across major translation services.
In any case, if it helps, at BMW we wrote this CLI utility to transform to/from ARB files to JSON: https://github.com/bmw-tech/arb-converter-cli

Thanks. That seems very helpful and gives people an immediate option. How widely is "Hierarchical JSON" supported?

I do not know, to be honest, I just know it works with our vendor of choice, but if you feel like a new format should be supported, you can file a new issue... also, PRs are welcomed ;)

Translator Toolkit has a poor suport for use last version translate on a new version (more strings) of the same APP.
So, maybe an option is to work on a simple system for those translation, and that just allow to add new strings to translate without the need of create a entire new translation.

Hi, our vendor is Lokalise, and here's their supported format list: https://docs.lokalise.com/en/collections/652248-supported-file-formats
Notably, I think "GNU Gettext / PO", "ICU", some kinds of jsons...

Note that the ARB format is essentially ICU message strings in a JSON format. So it ought to be quite simple to emit/consume a different format. The most difficult part is understanding the other format, and if it handles things like "meaning", or how it defines parameters.
I started doing an experiment writing an equivalent to the extract_to_arb/generate_from_arb scripts in a separate package for XLIFF, which is one of the more complex alternatives, and it doesn't seem like there are many obstacles to doing that. If the strings are not ICU then understanding PetitParser enough to write a different grammar for them is an issue, and understanding XLIFF is another.
It requires importing files from lib/src in intl_translation, but that works, and I think we can reasonably move those out of src/re-export them to make it easier for people to do that. The thing I was doing is still pretty rough, doesn't actually have a specific target, and I have to do some paperwork to publish a new package. But if someone has a specific target they want to use/test against and are willing to try writing support for such a format, I'd be happy to help and to make supporting changes to intl_translation.

Localizely supports ARB files.
We also published VS Code extension for simplified localization with ARB files: Flutter Intl.
Android Studio plugin is also in the plan.
I hope this helps.

Here is something to convert to XLIFF and back
https://www.npmjs.com/package/arb-convert

@mdoelker This was mentioned by @jorgecoca in here already :)

I wrote the free tool attranslate to compensate for the death of "Google Translator Toolkit": https://github.com/fkirc/attranslate
You can give attranslate a try if this is still an issue for you.
attranslate supports both JSON-files and ARB-files, and it also supports conversion between file-formats.

See also this stackoverflow thread.


Punctuation restriction in text field of snippet

Cannot create a snippet such as:
{{action "actionName" }}
The quotes appear in the current file instead of the snippet text.

Sorry, had selected the wrong context.


Conflicting Jupyter kernel specs from multiple environments

I'm developing multiple Django projects, each with its own virtualenv that includes django-extensions. To user the Jupyter Notebook, I've been doing pip install jupyter followed by manage.py shell_plus --notebook inside the virtualenv. It's been working okay, but I've learned a little bit more about how Jupyter works, and it seems that the way django-extensions handles kernel specs is not ideal.
For example, when I run the notebook in project-foo, I get the following kernel spec at /Users/brian/Library/Jupyter/kernels/django_extensions/kernel.json:
{"language": "python", "display_name": "Django Shell-Plus", "env": {"PYTHONPATH": "/Users/brian/Code/project-foo"}, "argv": ["/Users/brian/.virtualenvs/project-foo/bin/python", "-m", "ipykernel", "-f", "{connection_file}", "--ext", "django_extensions.management.notebook_extension"]}

However, when I run the notebook in project-bar, that overwrites the same file with:
{"language": "python", "display_name": "Django Shell-Plus", "env": {"PYTHONPATH": "/Users/brian/Code/project-bar"}, "argv": ["/Users/brian/.virtualenvs/project-bar/bin/python", "-m", "ipykernel", "-f", "{connection_file}", "--ext", "django_extensions.management.notebook_extension"]}

Now, in practice, this hasn't been a huge problem, because I'm usually not running multiple notebook servers at once, and django-extensions always writes the kernel spec before it starts a notebook server. But it does seem like it would cause unexpected and undesirable behavior.
After reading up on installing kernels, I see two potential fixes:

Instead of installing to the user's kernel directory, install to sys.prefix. IE, mimic python -m ipykernel install --sys-prefix instead of --user. So, in the example above, the kernel spec for project-foo would be installed to /Users/brian/.virtualenvs/project-foo/share/jupyter/kernels/django-extensions.
Mimic the behavior of the --name option to python -m ipykernel install, similar to the option for setting display_name. So, in the example above, the kernel spec for project-foo would be installed to /Users/brian/Library/Jupyter/kernels/django_extensions-project-foo (possibly with a display_name of Django Shell-Plus (project-foo).

Thanks for the report and spending your time diving into this !
Happy with any proposal and will accept any PR to improve how we handle notebooks 👍

I'm definitely game to work on this. Just gotta carve out the time.

This is still on my radar, but not a high priority.
Copied from a previous discussion:
What I'm planning to do is something like what I suggested on #1104. More specifically, I'd add a new command, maybe called install_kernel. This would behave and have options similar to python -m ipykernel install  and the bash kernel with respect to installing a kernel spec, but would add the django_extensions notebook extension. I'd try to have helpful defaults, like defaulting to --user with a name that reflects the active virtual environment.
With the kernel spec installed, users would be free to run jupyter notebook, jupyter lab, jupyter qtconsole, etc. in whatever environment they want (their Django virtualenv, a Jupyter virtualenv, Anaconda, etc.). And, django_extensions wouldn't need the NOTEBOOK_ARGUMENTS and IPYTHON_ARGUMENTS settings, because it's no longer acting as a wrapper around the notebook command, and users would be free to edit the installed kernel spec as needed.
We'd use documentation to introduce the new command and recommend a switch, then later deprecating --notebook and --kernel, and eventually removing them.

I believe the cause of this is the same as #1378 and is fixed by #1500 but also #1509


KYLIN-3826 MergeCuboidJob only uploads necessary segment's dictionary

Can one of the admins verify this patch?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9d125e1). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #479   +/-   ##
=========================================
  Coverage          ?   24.46%
  Complexity        ?     4961
=========================================
  Files             ?     1144
  Lines             ?    69420
  Branches          ?     9895
=========================================
  Hits              ?    16987
  Misses            ?    50719
  Partials          ?     1714

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/kylin/engine/mr/steps/MergeCuboidJob.java
0% <0%> (ø)
0 <0> (?)

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 9d125e1...4958f52. Read the comment docs.

Pull Request Test Coverage Report for Build 4158

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 26.945%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/MergeCuboidJob.java
0
3
0.0%

Totals

Change from base Build 4157:
0.004%

Covered Lines:
18705

Relevant Lines:
69420

💛  - Coveralls


caddy: 2.8.4 -> 2.9.0

update to 2.9.0 (release note: https://github.com/caddyserver/caddy/releases/tag/v2.9.0)
add build tags nomysql and nopgx (see official docs)
format with nixfmt-rfc-style
add myself to maintainers

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 369744

aarch64-darwin

:white_check_mark: 4 packages built:

caddy
styx
styx.lib
styx.themes

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 369744

x86_64-linux

:white_check_mark: 4 packages built:

caddy
styx
styx.lib
styx.themes


Keyboard Only User

Keyboard Only User
To Do:
Add tab index, alt text, and aria-describeby text to pages in order to support users navigating with keyboard.
Notes

Low to medium priority.
Pages and associated templates should be easily navigable by users with keyboards, screen readers.
Included in sprint due to coupling with other keyboard-only story. That being said, establishing this early, especially in templates and base pages, will keep this from growing into a larger issue.

Size

Small

Related Links

Issue: Image Gallery - Keyboard Control
User Story: Keyboard Only User

Task Branch Name: feature/keyboard-only
Work performed: Added alt tags, aria-describedby tags, and tabindexes to the index.html. This was the only page available to be worked on at the time of branch.
Notes: It would likely make sense to iterate upon this issue once more pages exist. In the alternative, guidelines for how this was done could be promulgated.


Fix typos and code formatting.

Fix typos and code formatting.

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

@alexandair Thank you :)
#sign-off


Allow empty arrays on PyOpenCL

Description
This allows creating zero-length arrays (and objects) on the OpenCL context. On the other contexts this works already out of the box, but with CL this would cause errors (see the issue below for an example).
Fixes xsuite/xsuite#294
Checklist
Mandatory:

[x] I have added tests to cover my changes
[ ] All the tests are passing, including my new ones (in progress)
[x] I described my changes in this PR description

Optional:

[x] The code I wrote follows good style practices (see PEP 8 and PEP 20).
[ ] I have updated the docs in relation to my changes, if applicable
[x] I have tested also GPU contexts

Closing for now, will reopen when needed


pythonPackages.libarcus: use pkgs.protobuf as input

Motivation for this change
Cura was not building because of libarcus.
Since https://github.com/NixOS/nixpkgs/commit/5c7531872fb55f42b03df9929f60cfb410cf7cc5 the protobuf Python module does not have protobuf library as a propagated build input.
This commit makes pkgs.protobuf the input for libarcus, since the latter
searches for the native protobuf library and not for the Python module.
I have tested this by building and running Cura.
/cc @FRidh
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-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)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg python3.pkgs.libarcus

This should be backported to staging-19.03.

@dotlambda If I move protobuf to buildInputs, curaengine still fails as in:
-- Building with Arcus
CMake Error at /nix/store/a4icwnjm9iasay87ggkfp4gw36wsqcpw-cmake-3.13.4/share/cmake-3.13/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find Protobuf (missing: Protobuf_INCLUDE_DIR) (Required is at
  least version "3.0.0")
Call Stack (most recent call first):
  /nix/store/a4icwnjm9iasay87ggkfp4gw36wsqcpw-cmake-3.13.4/share/cmake-3.13/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  /nix/store/a4icwnjm9iasay87ggkfp4gw36wsqcpw-cmake-3.13.4/share/cmake-3.13/Modules/FindProtobuf.cmake:595 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  /nix/store/2dcds8m1xi8x11qxzk4czjcxq6xj6xlc-python3.7-libarcus-3.6.0/lib/cmake/Arcus/ArcusConfig.cmake:31 (find_package)
  CMakeLists.txt:14 (find_package)

Adding protobuf in curaengine's buildInputs explicitly while protobuf is in libarcus' buildInputs also works, are you suggesting doing so?
I think it's more elegant to add just libarcus on packages and the latter to use libarcus CMake library to get protobuf, but we are talking about 2 packages currently relying on libarcus (curaengine and uranium, which is actually used to build cura).

Adding protobuf in curaengine's buildInputs explicitly while protobuf is in libarcus' buildInputs also works, are you suggesting doing so?

Yes, definitely.

I think we should instead add back pkgs.protobuf to propagatedNativeBuildInputs for pythonPackages.protobuf. This recent change has broken more than libarcus -- python3Packages.trezor is also broken.

Actually this failure is not fixed with pkgs.protobuf in propagatedNativeBuildInputs -- different root cause from the python3Packages.trezor problem. I'll file a separate PR.

@delroth as you said, 5c75318 did not fix the problem with libarcus.
@dotlambda I have made the requested changes, could you review and tell me if something more is needed?

@delroth as you said, 5c75318 did not fix the problem with libarcus.
@dotlambda I have made the requested changes, could you review and tell me if something more is needed?

@GrahamcOfBorg build python2.pkgs.libarcus python3.pkgs.libarcus curaengine


Optimize clf metrics calculation

Significantly speeded up the calculations in last if block in calculate_metrics function.
E.g. for 100k rows of random predictions it took 3 mins, now it takes 50 ms (the more data there is, the greater the difference)
Fixes https://github.com/evidentlyai/evidently/issues/508

I took another look and simplified the code even more

Hey @feldlime ,
many thanks!
This is a painful issue, we appreciate you addressed it. We will review it asap.


Publishing Java Packages to Central

Description:
Currently if a user wants to use java library they have to use the bind gen tool to generate the bindings. If the same java library is used by many developers they might create duplicate java bindings. With this effort we want to implement a process to get the java libraries to Ballerina Central so users can easily share java bindings for common java libraries.
Milestone 1 : Select a set of commonly used java packages and pus it to ballerina central

Closing since this is not a concern.


Crash on incorrect api key

When I gave it the wrong api key seems to crash:
Traceback (most recent call last):
File "av.py", line 4, in
data, meta_data = ts.get_daily('STO:SAAB-B', outputsize='compact')
File "/home/joakim/.virtualenvs/alphavantage/lib/python3.5/site-packages/alpha_vantage/alphavantage.py", line 152, in _format_wrapper
json_response, data_key, meta_data_key = func(self, *args, **kwargs)
File "/home/joakim/.virtualenvs/alphavantage/lib/python3.5/site-packages/alpha_vantage/alphavantage.py", line 138, in _call_wrapper
return self._handle_api_call(url), data_key, meta_data_key
File "/home/joakim/.virtualenvs/alphavantage/lib/python3.5/site-packages/alpha_vantage/alphavantage.py", line 76, in _retry_wrapper
raise ValueError(err)
UnboundLocalError: local variable 'err' referenced before assignment

It only happens in python 3.5, in python 2.7 I get
>>> from alpha_vantage.timeseries import TimeSeries
>>> ts = TimeSeries(key='WRON')
>>> data, meta_data = ts.get_daily('STO:SAAB-B', outputsize='compact')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "alpha_vantage/alphavantage.py", line 152, in _format_wrapper
    json_response, data_key, meta_data_key = func(self, *args, **kwargs)
  File "alpha_vantage/alphavantage.py", line 138, in _call_wrapper
    return self._handle_api_call(url), data_key, meta_data_key
  File "alpha_vantage/alphavantage.py", line 76, in _retry_wrapper
    raise ValueError(err)
ValueError: ('ERROR getting data form api', 'Please make sure your API key is valid. Alternatively, claim your free API key on (http://www.alphavantage.co/support/#api-key). It should take less than 20 seconds, and is free permanently.')

Nevertheless I will also give it a try to fix it, thanks for your help.
Romel


An Api instance must be provided as argument or set as instance

Hi, edbizarro, i have been trying to implement this laravel-facebook-ads but i have been getting this error, ''An Api instance must be provided as argument or set as instance in the \FacebookAds\ApAn Api instance must be provided as argument or set as instance in the \FacebookAds\Api". and my version is 0.8.2

Hi @collinsugwu
Can u provide how u implement the package?
Tks!

Hi @collinsugwu
Before start using you need to call FacebookAds::init('ACCESS_TOKEN'); with a valid access

Hi @collinsugwu
Before start using you need to call FacebookAds::init('ACCESS_TOKEN'); with a valid access

Tanks


Update peer dependency

Here's a summary of the npm bug fix issue.
https://github.com/npm/cli/issues/4998

@brianlovin Is it possible to push this through?

@christo8989 spectrum is no longer being maintained, so unfortunately this will not be merged.


[web] replace current wasm image codecs with on-demand off-thread bundle

Goals:

Reduce CanvasKit bundle size by 140KB
Make image decoding non-blocking on Firefox and Safari.

Background
Our current image codecs embedded into CanvasKit do the job, but our approach has a number of issues:

According to my experiments image codecs add ~140KB to the compressed payload, which is significant.
In browsers that support the ImageDecoder API these codecs are not used, so all they do is contribute to payload sizes and poor start-up times.
Not all apps need to render an image on the very first frame, so loading image codecs before the first frame is sub-optimal.
The codecs execute on the main UI thread, blocking frames, which leads to jank.

Proposal

Remove image codecs from the main wasm bundle.
Use ImageDecoder for all decoding needs, if available.
Use <img> to decode static images in browsers that don't support ImageDecoder. Many, if not most, images are static (i.e. not animated). WebGLRenderingContext.texImage2D supports HTMLImageElement as the source.
Create a separate wasm bundle for image codecs for cases not covered by the above. Load the bundle on-demand, only when the app actually needs to decode an animated image.

Some hand-optimized applications can go even further and use the raw <img> tag in a platform view for animated images, removing the need for codecs even on browsers that don't have ImageDecoder.

No update, but this still needs to get done.

More specifically, the ImageDecoder part is done, but not all browsers support it, so we still need a decent fallback (the current fallback is to decode on the same thread and block frames).

We use <img> decoding or ImageDecoder in most cases now. Closing


Add owned Programs?

src/program/mod.rs implements Add for &Program as such:
impl<'a, 'b> ops::Add<&'b Program> for &'a Program {
    type Output = Program;

    fn add(self, rhs: &'b Program) -> Program {
        let mut new = self.clone();
        new.calibrations.extend(rhs.calibrations.clone());
        new.frames.merge(rhs.frames.clone());
        new.memory_regions.extend(rhs.memory_regions.clone());
        new.waveforms.extend(rhs.waveforms.clone());
        new.instructions.extend(rhs.instructions.clone());
        new
    }
}

Was there a specific reason for this? Seems like it would be more efficient to implement adding two owned values and require them to be cloned at the call side instead, if necessary.

This was originally motivated by needing to expose an add method for pyQuil, where the programs are stored references. There's no reason I can think of to not clone them at the call site instead, so I think you're good to make this change.

Closed via #249


fix: CREATE-PLAY.md typo

Hi! @atapas I have fixed one typo in CREATE-PLAY.md.

Line 87 - "That's it. Now start coding ~yor~ your Play using your favorite code editor!"

Thanks

@DalpatRathore Thanks for the catch. Could you please create an issue for it and link it to this PR?

@all-contributors please add @DalpatRathore for Code


Detect cycles when creating/updating a work item link

For now, the work item link repository verifies that in a given tree topology (ie, a given link type), an item has a single parent. But it does not verify that the user is about to create a cycle.
There are 2 cases of cycles to support:

A->B->C-D->A: in this case, the cycle involves the root item
A->B->C-D->B: in this case, the cycle does not involve the root item. In that case, the current validation will find that item B already had a parent item, A and will not allow the new D->B link.

Further reading: https://en.wikipedia.org/wiki/Cycle_(graph_theory)#Cycle_detection

An earlier idea of mine was to have an ltree column in the ´links` table and use that for querying for complex graphs. Also, postgres common table expressions might help us to traverse a graph and find cycles.


Testing in unstated-next?

I read through issue #20 and that had some good perspective.  Agree on keeping App away.  Some tricks there that I could have used a year ago.  🎈 Anyway ...
I could test unstated "actions" through enzyme but I'm looking at this store.js file I made and it's so simple.  I just want to test it.  Of course, I can't because it needs hooks, context.  It needs to be in a DOM/Tree/JSX place just like the used source file.
Something like this:
// store.test.js
import { StoreContainer } from '../src/store'

describe("the state store", () => {

  // the state here is just flipping a checkbox, a spike
  it("can receive an event", () => {
    // this can't work outside a provider
    const store = StoreContainer.useContainer()

    const event = { target: { value: "test" } }
    const checked = false

    store.handleUpdate(event, checked)

    const expected = { "test": false }
    expect(store.filters).toBe(expected)
  })

})

Running this with jest prints Invariant Violation: Invalid hook call.  Yeah.  That makes sense.  "Hooks can only be called inside of the body of a function component."  I see in the repo where the context is expected.
My store.js file looks like this.  Works fine great when manually testing (like in the app, in the browser, integrated).  But I'm adding jest and want to add complexity.  So I thought I'd just test my store in isolation and then later the increasing number of "actions".  Maybe this is a "bad" idea or what you wanted to avoid in the first place.
// store.js
import { useState } from 'react'
import { createContainer } from "unstated-next"

const useStore = () => {
  const defaultFilters = {}

  const [filters, updateFilters] = useState(defaultFilters)
  const handleUpdate = (event, checked) => {
    let newState = { [event.target.value]: checked }
    updateFilters({ ...filters, ...newState })
  }

  return { filters, handleUpdate }
}

export const StoreContainer = createContainer(useStore)

Filters is just an object that I'm merging with checkbox clicks (a spike).
See, it's just a little bit of code.  Maybe I'm wanted to test it for the wrong reasons.  Maybe I'm trying to test it like it's a plain js file when it's not.
In the README:

Testing reducers is a waste of your time, make it easier to test your React components.

Do you mean that I should not test the store directly?  It's too simple?  I could test the store through my components?  (normal enzyme stuff)

First off, I just have to say: Do not create a "StoreContainer". You should have many containers in your application, having a single one is going to cause a performance problem in your application, and it will become increasingly bloated over time. Each container should have one clear responsibility.

For testing, when you create a container, you're creating an API for components.
function useCounter(initialState = 0) {
  let [count, setCount] = useState(initialState)
  let decrement = () => setCount(count - 1)
  let increment = () => setCount(count + 1)
  return { count, decrement, increment } // << -- API for components
}

let Counter = UnstatedNext.createContainer(useContainer)

This API that you return from a hook is only ever meant to be used to build components. Anything that's not visible to components via the interface you are returning is an implementation detail.
You could use an API like @testing-library/react-hooks which would look like this:
import { renderHook, act } from "@testing-library/react-hooks"
import Counter from "./containers/Counter"

test("Counter", () => {
  let { result } = renderHook(() => Counter.useContainer())
  expect(result.current.value).toBe(0)
  act(() => result.current.increment())
  expect(result.current.value).toBe(1)
  act(() => result.current.decrement())
  expect(result.current.value).toBe(0)
})

You could also just create a component in your tests:
import {
  render,
  fireEvent,
  cleanup,
} from '@testing-library/react'
import Counter from './containers/Counter'

afterEach(cleanup)

function CounterExample() {
  let counter = Counter.useContainer()
  return

{count}

    <button data-testid="increment" onClick={counter.increment}/>
    <button data-testid="decrement" onClick={counter.decrement}/>

}

test("Counter", () => {
  let { getByTestId, fireEvent } = render(<CounterExample/>)
  expect(getByTestId("count").textContent).toBe("0")
  fireEvent.click(getByTestId("increment"))
  expect(getByTestId("count").textContent).toBe("1")
  fireEvent.click(getByTestId("decrement"))
  expect(getByTestId("count").textContent).toBe("0")
})

I think directly testing the hook, needs to test the useCounter hook, not the Container.useContainer() function, since it uses Context.
The same applies to the second example. The Example would need to render the Provider, in order to make the container work.

I've been adding tests to a project using unstated-next recently and found you don't need to use the createContainer or useContainer.
just use the hook directly.
import {useCounter} from '../useCounter'

test('Increment', () => {
    const { result } = renderHook(() => useCounter())
    act(() => result.current.increment(2))
    act(() => result.current.decrement(2))
    expect(result.current.value).toBe(0)
  })

My approach is the following:
SingIn.state.tsx
import * as React from 'react';
import { createContainer } from 'unstated-next';

export type UseAccountState = 'advertiser' | 'influencer';
export type UseAccountReturn = {
  account: UseAccountState;
  setAdvertiser: () => void;
  setInfluencer: () => void;
};

function useAccount(
  initialState: UseAccountState = 'influencer'
): UseAccountReturn {
  const [account, setAccount] = React.useState<UseAccountState>(initialState);

  const setAdvertiser = React.useCallback(() => {
    setAccount('advertiser');
  }, [setAccount]);

  const setInfluencer = React.useCallback(() => {
    setAccount('influencer');
  }, [setAccount]);

  return {
    account,
    setAdvertiser,
    setInfluencer
  };
}

export const AccountState = createContainer(useAccount);

SignIn.test.tsx
import * as React from 'react';
import { renderHook, act } from '@testing-library/react-hooks';
import { AccountState, UseAccountState } from './SingIn.state';

const AccountStateWrapper: React.FC<{ initialState: UseAccountState }> = ({
  children,
  initialState
}) => (
  <AccountState.Provider initialState={initialState}>
    {children}
  </AccountState.Provider>
);

describe('SignIn Feature', () => {
  describe('Hooks', () => {
    it('should initial state be influencer by default', () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper
      });

      expect(result.current.account).toBe('influencer');
    });

    it('should initial state be advertiser by custom', () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper,
        initialProps: { initialState: 'advertiser' }
      });

      expect(result.current.account).toBe('advertiser');
    });

    it('should mutate state to advertiser', async () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper
      });

      act(() => {
        result.current.setAdvertiser();
      });

      expect(result.current.account).toBe('advertiser');
    });

    it('should mutate state to influencer', async () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper,
        initialProps: { initialState: 'advertiser' }
      });

      act(() => {
        result.current.setInfluencer();
      });

      expect(result.current.account).toBe('influencer');
    });
  });
});

My approach is the following:
SingIn.state.tsx
import * as React from 'react';
import { createContainer } from 'unstated-next';

export type UseAccountState = 'advertiser' | 'influencer';
export type UseAccountReturn = {
  account: UseAccountState;
  setAdvertiser: () => void;
  setInfluencer: () => void;
};

function useAccount(
  initialState: UseAccountState = 'influencer'
): UseAccountReturn {
  const [account, setAccount] = React.useState<UseAccountState>(initialState);

  const setAdvertiser = React.useCallback(() => {
    setAccount('advertiser');
  }, [setAccount]);

  const setInfluencer = React.useCallback(() => {
    setAccount('influencer');
  }, [setAccount]);

  return {
    account,
    setAdvertiser,
    setInfluencer
  };
}

export const AccountState = createContainer(useAccount);

SignIn.test.tsx
import * as React from 'react';
import { renderHook, act } from '@testing-library/react-hooks';
import { AccountState, UseAccountState } from './SingIn.state';

const AccountStateWrapper: React.FC<{ initialState: UseAccountState }> = ({
  children,
  initialState
}) => (
  <AccountState.Provider initialState={initialState}>
    {children}
  </AccountState.Provider>
);

describe('SignIn Feature', () => {
  describe('Hooks', () => {
    it('should initial state be influencer by default', () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper
      });

      expect(result.current.account).toBe('influencer');
    });

    it('should initial state be advertiser by custom', () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper,
        initialProps: { initialState: 'advertiser' }
      });

      expect(result.current.account).toBe('advertiser');
    });

    it('should mutate state to advertiser', async () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper
      });

      act(() => {
        result.current.setAdvertiser();
      });

      expect(result.current.account).toBe('advertiser');
    });

    it('should mutate state to influencer', async () => {
      const { result } = renderHook(() => AccountState.useContainer(), {
        wrapper: AccountStateWrapper,
        initialProps: { initialState: 'advertiser' }
      });

      act(() => {
        result.current.setInfluencer();
      });

      expect(result.current.account).toBe('influencer');
    });
  });
});

What about testing components with Jest that are consuming resources from a give unstated-next container?
How should we be mocking the container's methods and state? I couldn't find any articles about it.

What about testing components with Jest that are consuming resources from a given unstated-next container?
How should we be mocking the container's methods and state? I couldn't find any articles about it.

this is what I did to mock the unstated-next container
jest.mock("...module path", () => ({
  ...jest.requireActual("...module path"),
  "your state name": {
    Provider: ({ children }: { children: React.ReactNode }) =>
{children}
,
    useContainer: () => ({
       ... return value that you want to mock
    }),
  },
}));


BrianPeek from OPS portal: Merge from master to live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sandbox/TOC.yml
:white_check_mark:Succeeded
View

sandbox/demos/TOC.yml
:white_check_mark:Succeeded

sandbox/demos/index.yml
:white_check_mark:Succeeded
View

sandbox/gamedev/TOC.yml
:white_check_mark:Succeeded

sandbox/gamedev/index.yml
:white_check_mark:Succeeded
View

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


Support arm64 build with make arm64

Added experimental support for building jvmkill for arm64 (jvmkill-arm64.so). This doesn't require using Arm machines as it uses a GCC cross compiler in a docker image of https://github.com/dockcross/dockcross

Thanks for contributing this. dockcross is interesting and a neat idea, but it seems overly complex. This project is a very simple C library that we can easily build on the target platform. With Docker for Mac (not sure about other platforms), it's easy to run an image for a different architecture, then use that to build directly:
docker run -v ~/src/jvmkill:/work -it arm64v8/centos:7 bash
yum -y install java-11-openjdk-devel gcc make
cd /work
JAVA_HOME=/usr/lib/jvm/java make

What do you think of something like the above? (ideally, we'd use an image that already has the JDK and compiler installed, but I don't know of such an image)

it seems overly complex.
Yeah. I know :) Actually, we don't need to merge this PR.

Using QEMU and some jdk image for arm64 might be a better approach. I will ask some internal Arm people for their best practices.

We now publish a multi-architecture Docker image: ghcr.io/airlift/jvmkill:latest
With docker buildx, we can build for multiple architectures without cross compiling or using a specific base image.


Using the MongoDB mgo driver

This issue is a follow-on from https://github.com/arschles/go-in-5-minutes/issues/43#issuecomment-205537261. We should have an episode on how to use the mgo driver.

more resourceful it would be if you include the context package from gorilla with the http.handler wrapper function like in this awesome article<EMAIL_ADDRESS>🗡


Update README: env variable for AppInsights key  should be called APPINSIGHTS_INSTRUMENTATION_KEY

Spent an hour trying to understand why the sample code doesn’t send custom events to AppInsights

Thanks @anastasiia-zolochevska !! :)


Update apt module syntax and support become for installing requirements

I've encountered a couple of issues with my installation, so I figured I'd share them in case they're useful. What we needed to achieve was a local nvm installation, with required packages installed by root.

Ansible 2.9.2
Ubuntu 18.04
Python 3.6.1

sorry, looks like this breaks support for ansible 1.8 and 1.9. Any ideas on how to make it support both versions? I'd be happy to help.

Will take a look at this later today.


Closing channel balance added to displayed total

Description
When a channel is closing the chain amount is added to the total available.
Expected Behavior
Would expect channel amount to be deducted from total at the same time.
Actual Behavior
Closing channel amount is still counted as available in total.
Steps to Reproduce
Close a channel, refresh wallet display
Your Environment

Zap version: 0.2.9-alpha
Android version: 9
Device: Samsung S4 Mini
LND Version: 0.8.0-beta

@MrHash Thanks for mentioning this. Was this a force close?

Nope just a regular close, low fee though

Ok, thanks. Your Balance will eventually be correct again, when the close is finished I guess. We will try to fix it for the next version.


[ISSUE-5886] Implement hashCode for MNodeContainers

5949

 Quality gate: Passed
1 Issues

 1 Anti-patterns

Details

 Quality gate: Passed
1 Issues

 1 Anti-patterns

Details


Change CI Fedora cases to newer versions.

https://hub.docker.com/_/fedora/

Only f29 fails with below error. That is same with my local environment (f28).
https://travis-ci.org/junaruga/rpm-py-installer/builds/456998889
https://travis-ci.org/junaruga/rpm-py-installer/jobs/456998890

AttributeError: module 'site' has no attribute 'getsitepackages'


Lost transaction

Hi,
Last night I sent a $100 transaction from Coinbase to my Atomic Wallet, using my ATOM receive address there. Coinbase thinks the ATOM went out successfully but the transaction page to which they link is throwing an error:
https://cosmos.bigdipper.live/transactions/732BD51440AFA5E4283F6193D421915E7C2D92B0EC5B6E8049F5C38D94349BFE
Coinbase says: "2/18/2021 8:03 PM     Completed"
I'm not sure what happened to the 4.363599 ATOM that I sent, but I should have received it at address cosmos1g7ntd9a2zv8qdzfn0lsqe4kx7z5ysyl75vg8hf - I want to send more but am afraid to do so now.
If I'm barking up the wrong tree could someone direct me to where I might get the appropriate support?
Thanks,
Chris

Same

Same happened to me and then they closed my ticket after acknowledging it happened. So wtf is the solution? I'm just out the money I transferred or what?!

Yea he closed mine too idk what’s going on
Sent from my iPhone

On Feb 19, 2021, at 6:49 PM, sminer10<EMAIL_ADDRESS>wrote:

Same happened to me and then they closed my ticket after acknowledging it happened. So wtf is the solution? I'm just out the money I transferred or what?!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

The issue for me was Atomic Wallet. I loaded my mnemonic phrase into
wallet.cosmostation.io as recommended by one of the Cosmos techs through
Reddit chat (r/cosmosnetwork) and my correct balance popped right up. I
closed my Github ticket because this issue is resolved. People are
saying that Atomic Wallet is no longer supported and not to use it.
Try the Cosmos subreddit, they'll reach out to you after you post.
https://www.reddit.com/r/cosmosnetwork/comments/lnqtcc/lost_transaction/
On 2/19/21 9:14 PM, pdubb12003 wrote:

Yea he closed mine too idk what’s going on
Sent from my iPhone

On Feb 19, 2021, at 6:49 PM, sminer10<EMAIL_ADDRESS>wrote:

Same happened to me and then they closed my ticket after
acknowledging it happened. So wtf is the solution? I'm just out the
money I transferred or what?!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/forbole/big-dipper/issues/463#issuecomment-782523874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRKMWDQDYNQVGPIHVB4GD3S74LGRANCNFSM4X44EMSQ.

I have another update. Do not trust anyone on Reddit that reaches out to you through PM. They are all scammers.
And the Cosmos network seems to be experiencing major problems since their update the other night. Do not send any ATOM until this is ironed out.

Thank you people have been saying use your memonic phrase on this website or that one but I know better thanks so much
Sent from my iPhone

On Feb 20, 2021, at 4:38 PM, empbac<EMAIL_ADDRESS>wrote:

I have another update. Do not trust anyone on Reddit that reaches out to you through PM. They are all scammers.
And the Cosmos network seems to be experiencing major problems since their update the other night. Do not send any ATOM until this is ironed out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


How to move out value from Result like Rust?

Hi, I'm using this Result type and wondering what's the idiomatic way to move out the contained value?
For example, with a Result containing a movable type like mystring(class impl by myself):
#include "result.hpp"

class MyString {
    std::string data;
public:
    MyString() : data() {
        std::cout << "Default constructor\n";
    }

    MyString(const char* s) : data(s) {
        std::cout << "Constructor from const char*\n";
    }

    // delete copy for some reason
    MyString(const MyString& other) = delete;
    MyString& operator=(const MyString& other) = delete;

    MyString(MyString&& other) noexcept : data(std::move(other.data)) {
        std::cout << "Move constructor\n";
    }

    MyString& operator=(MyString&& other) noexcept {
        std::cout << "Move assignment\n";
        if (this != &other) {
            data = std::move(other.data);
        }
        return *this;
    }

    ~MyString() {
        std::cout << "Destructor\n";
    }
};

Result<MyString, int> get_mystring() {
    return Ok(MyString("hello"));
}

int main() {
    MyString s1 = get_mystring().unwrap();
}

// How to move the string out without copying?
mystring s = get_string().unwrap(); // This seems to make a copy
I see the current unwrap() implementation returns by value and is const, which forces a copy. Is there an existing way to move out the value that I'm missing?
If there isn't one currently, would you add an rvalue-reference unwrap()?

Hello, right, unwrap-ing the Result will create a copy.
I think that back when I implemented this, we did not have r-value references for *this.
So if I understand correctly, for movable types, you'd like something like:
T&& unwrap() && { ... }

Sure, that's what I want.
and we can call it on r-value references like this
// move sematics
std::move(l_value).unwarp()
// copy sematics
l_value.unwarp()


Add REST API for SD.

Proposal
Use case. Why is this important?
As a third-party system, i need register/deregister my endpoint to prometheus by REST API and be scraped. I don't like existing SDs as they are not flexible.
Not sure if it's a good idea, any suggestion?
Nice to have' is not a good use case :)

You can look this document
https://github.com/prometheus/prometheus/tree/master/documentation/examples/custom-sd
It use file_sd for SD

one big disadvantage in your proposal is that all your targets need to know the ip of the Prometheus server which might work in a small setup,but a big disadvantage at scale.
Can you share more details about your setup?

Thanks jojohappy, brian-brazil. Using the custom-sd is way to solve my problem, but I have many vms or docker created dynamically, and each time a vm or docker created, need to restart prometheus. I'm not sure if it will lose some metrics and if there is any problems that restart prometheus frequently.
Thanks krasi-georgiev. I just have a hybrid cloud controller which will need to monitor the vms or dockers created by the  hybrid cloud controller in different cloud resource provider.

Prometheus has provide api to reload configuration, It will not need to restart Prometheus and impact on scrape process.


Using better detection model on ARM devices

I setup an ARM vm on Oracle Cloud and installed Deepstack. It works great but I found out that on ARM devices Deepstack uses the yolov5s model instead of the yolov5m. I know that usually ARM processors are used on low power devices but actually my vm is pretty powerful and has no problems running even the ExDark model which uses the yolov5x with a good inference time.
Now, I was wondering if it's possible to run Deepstack on ARM with a better object detection model (like the yolov5m).
I tried downloading the .pt files from here and using them as custom models but they're not recognized.
Any help is appreciated!
P.S. Thanks for your great work on this project

Hello @francesco-re-1107 , thanks a lot for this feedback. Adding support for this is trivial from our end and we would release a standard ARM64 version of DeepStack soon  with same models as the standard cpu and gpu versions.

@francesco-re-1107 , as promised, find the arm64-server docker image here, https://docs.deepstack.cc/arm64/index.html#deepstack-on-arm64-servers


Update moment library to >2.11 (fixes a security bug)

nps reports<EMAIL_ADDRESS>as being vulnerable to a regex DoS.
Please update dependencies to use a more recent version of the library. It shouldn't break anything as there isn't any breaking changes there.

Updated to<EMAIL_ADDRESS>Will be in the next publish  ( >=<EMAIL_ADDRESS>)


feat: add flag to disable offsettoid in Offsettoid class

Goals:

...
...DocArray allows list-like behavior by adding offset2id structure to storage backends. This means the database stores, along with documents, meta information about document order.
This information is only updated/synced when a DA is destroyed or quit the context manager. Furthermore, offset2id is updated in memory in all update operations.
However, offset2id is not useful in indexers where concurrent usage is possible and users do not need information about document location. Besides, updating offset2id is not efficient.
Therefore, it's important to have an option that disables list-like behavior (and offset2id with it) and use it in DocumentArray indexers.

@dataclass
class QdrantConfig:
    n_dim: int
    distance: str = 'cosine'
    enable_offset2id: bool = True
    collection_name: Optional[str] = None
    host: Optional[str] = field(default="localhost")
    port: Optional[int] = field(default=6333)
    serialize_config: Dict = field(default_factory=dict)
    scroll_batch_size: int = 64
    ef_construct: Optional[int] = None
    full_scan_threshold: Optional[int] = None
    m: Optional[int] = None
    columns: Optional[Union[List[Tuple[str, str]], Dict[str, str]]] = None

...
...
[ ] check and update documentation, if required. See guide

Codecov Report
Base: 88.23% // Head: 61.11% // Decreases project coverage by -27.11% :warning:

Coverage data is based on head (eb9b10f) compared to base (8f7c327).
Patch coverage: 71.11% of modified lines in pull request are covered.

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

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #730       +/-   ##
===========================================
- Coverage   88.23%   61.11%   -27.12%
===========================================
  Files         133      133
  Lines        6586     6612       +26
===========================================
- Hits         5811     4041     -1770
- Misses        775     2571     +1796

Flag
Coverage Δ

docarray
61.11% <71.11%> (-27.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

docarray/array/storage/base/helper.py
69.81% <51.85%> (-19.94%)
:arrow_down:

docarray/array/storage/annlite/backend.py
77.92% <100.00%> (-19.42%)
:arrow_down:

docarray/array/storage/annlite/getsetdel.py
97.36% <100.00%> (-2.64%)
:arrow_down:

docarray/array/storage/elastic/backend.py
84.96% <100.00%> (-11.99%)
:arrow_down:

docarray/array/storage/elastic/getsetdel.py
93.22% <100.00%> (-6.78%)
:arrow_down:

docarray/array/storage/qdrant/backend.py
86.66% <100.00%> (-9.45%)
:arrow_down:

docarray/array/storage/qdrant/getsetdel.py
81.94% <100.00%> (ø)

docarray/array/storage/redis/backend.py
83.76% <100.00%> (-13.64%)
:arrow_down:

docarray/array/storage/redis/getsetdel.py
93.93% <100.00%> (-3.04%)
:arrow_down:

docarray/array/storage/sqlite/backend.py
82.50% <100.00%> (-12.38%)
:arrow_down:

... and 91 more

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

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

Yes, I have already documented this behavior


Arm Wheels Torch-FFmpeg missing

🐛 Describe the bug
It seems Linux Arm Wheels do not contain the torchaudio-ffmpeg lib.
RuntimeError: torchaudio is not compiled with FFmpeg integration. Please set USE_FFMPEG=1 when compiling torchaudio.

Versions
Audio v2.0.1Torch v2.0.0

Can you paste the output of collect_env.py, as requested in the bug report template?

Hi @fidoriel
We do not publish ARM wheels. At least it's not on our CI. Where do you get the image?

from the pytorch website: https://download.pytorch.org/whl/torchaudio-2.0.1-cp39-cp39-manylinux2014_aarch64.whl

Strange. I did not know there is ARM package.
Looks like it's built outside of our CI and uploaded manually.
Indeed, the package does not contain the ffmpeg-related libraries.
@atalman Do you know who manages this image?

Hi @mthrok yes we build it using https://github.com/pytorch/builder/tree/main/aarch64_linux

@fidoriel We fixed the root issue and a new release will happen in few weeks. Thanks for flagging.

Actually, shouldn't those wheels be available in https://download.pytorch.org/whl/test/cpu?
@atalman ?

oh I actually did not check with @atalman and assumed it will be applied from the next release. Is it already available?

2.0.2 has been released, and it should fix the issue. @fidoriel Let us know if it works.
https://download.pytorch.org/whl/torchaudio/
torchaudio-2.0.2-cp310-cp310-manylinux2014_aarch64.whl
torchaudio-2.0.2-cp311-cp311-manylinux2014_aarch64.whl
torchaudio-2.0.2-cp38-cp38-manylinux2014_aarch64.whl
torchaudio-2.0.2-cp39-cp39-manylinux2014_aarch64.whl


BUG: HQ flag is ignored

Version: <IP_ADDRESS>
Dalamud: D<IP_ADDRESS>
Current Settings:

Expected behavior: Copy NQ prices unless shift is held.
Actual behavior: Copies HQ price unless shift is held.
It appears that the following is not being properly evaluated:
"hq": false,
I have uninstalled and reinstalled. I have manually removed the plugin as well as the config file. Behavior remains.

Same situation. The plugin updated and broke expected/established functionality. Holding shift is quite bothersome, and I would much prefer if it was an "opt-in" feature.
It honestly seems to be a fringe-case feature for those who actively sell HQ current endgame gear. Even then, if I were selling an item where the HQ mattered, I would personally want to verify the lowest NQ versus lowest NQ price rather rely on faithful blind pasting.  As it is, I list a ton of retainer junk and HQ glam items so this has added a bunch of additional button presses that I can't seem to disable?

This will be fixed in the next update. Just need https://github.com/goatcorp/DalamudPluginsD17/pull/1387 to get merged.
(Apparently some people also want to keep undercutting their own retainers?!)

Validated fixed. Thanks!


Failing at checking dependencies against package.json

Bug Report

TSLint version: 5.8.0
TypeScript version: 2.5.3
Running TSLint via: CLI

Description
If you take a look at this PR the build is failing just by updating the version of TSLint.
Actual behavior
Yells about missing dependencies in package.json when they actually are there.
Expected behavior
Just works.

All dependencies in your package.json are devDependencies.
If the files in tools/ are intended to be published, you also need the packages used there as peerDependencies.
If they are only relevant for development and are not published, you need to add the "dev" option to the rule.

the build is failing just by updating the version of TSLint.

tslint:latest does not follow semver. See docs here.
I realize now that the default strictness of no-implicit-dependencies is not suitable for general use and should not be promoted to tslint:recommended. Rather than removing the rule from tslint:latest, though, I think we should change the default behavior of the rule to be less strict (allow imports of any package in dependencies, devDependencies, or optionalDependencies). Thoughts?

Making it less strict in that way is similar to just switching it off by default is it not ?
Generally I think its behaviour is correct - for production code the packages should be under "dependencies". For code related to development - such as unit tests - then they can allow packages under "devDependencies" and for these situations I'd expect a different ( probably less strict ) set of lint rules to be used anyway. In that less-strict ruleset, devDependencies would be enabled as appropriate.

I have the following pattern, is this incorrect?
Creating a module AModule, I have a peerDependency on another module BModule, that module has peerDependencies on another module CModule... In AModule I set CModule as a devDependency.
I want the consumer app of AModule to sort out the dependencies by installing AModule, BModule, and CModule.
It was compiling fine previously indicating that the new tslint error might be too rigid:

Making it less strict in that way is similar to just switching it off by default is it not ?

No, you still have to declare imported packages as dependencies somewhere in the package.json closest to the file being linted, and I believe it is still generally useful. This looser check still addresses the original intent of the rule, which was to mitigate issues caused by dependency hoisting in Lerna monorepos.
I personally would like to avoid having multiple tslint.json files for source vs. test code, as this would produce a lot of config files in my monorepos. So loosening the rule as I suggested would work well for me. Users who want to make their checks even stricter than tslint:latest can configure additional tslint.json files.

☠ TSLint's time has come! ☠
TSLint is no longer accepting most feature requests per #4534. See typescript-eslint.io for the new, shiny way to lint your TypeScript code with ESLint. ✨
It was a pleasure open sourcing with you all!


samples: net: zperf: Refactor zperf_*_receiver_init

Drop udp/tcp rx thread functions which are not necessary.
Support default iface ip if there is no static one.

Signed-off-by: Bub Wei<EMAIL_ADDRESS>

https://github.com/zephyrproject-rtos/zephyr/pull/11805

Codecov Report

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

@@            Coverage Diff            @@
##           master   #11830     +/-   ##
=========================================
- Coverage   48.31%   48.22%   -0.1%
=========================================
  Files         280      280
  Lines       43348    43335     -13
  Branches    10380    10379      -1
=========================================
- Hits        20945    20897     -48
- Misses      18251    18283     +32
- Partials     4152     4155      +3

Impacted Files
Coverage Δ

subsys/logging/log_output.c
62.38% <0%> (-10%)
:arrow_down:

subsys/logging/log_core.c
68.66% <0%> (-4.59%)
:arrow_down:

misc/printk.c
81.81% <0%> (-3.13%)
:arrow_down:

include/logging/log_msg.h
89.09% <0%> (-1.82%)
:arrow_down:

include/logging/log_core.h
100% <0%> (ø)
:arrow_up:

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 3ac7c12...936c2f2. Read the comment docs.


Persistent connection support

After discussing https://github.com/youtube/vitess/issues/2634 with @sougou, we discussed Persistent Connections as another way to further support clients of the mysql protocol. Persistent connections mean all queries for a given Session use the same vttablet -> mysql connection. This is useful for supporting session variables, which are often used in low-volume ad hoc querying.
Sugu had this feature back in 2012, but killed it. Now that we're improving mysql protocol connections, it could be very useful. The pool design has changed much since then, so the previously linked design diagram may need to be updated.
I propose two new additions:

Add a PersistentConnection field to the Session protobuf. When enabled, all subsequent queries for the session will go through a single persistent connection at the vttablet.

Add a VTGate.Close(session *vtgatepb.Session) function, which would shut down all persistent connections for that session.  The mysql protocol could use that function here: https://github.com/youtube/vitess/blob/04326afbb675d5328ae68dbc1215ce394d4a970c/go/vt/vtgate/plugin_mysql_server.go#L64, ensuring that it utilizes an API that is generally available rather than special-case.

Realized https://github.com/youtube/vitess/issues/2724 was entered as a duplicate. Closing this one since there's more discussion there.


Can't scale the DefaultCursor

Overview
I have an scalable GUI in my app. Since the GUI can get really small, I also want to scale the DefaultCursor by changing the transform.localScale.
Expected Behavior
The cursor should scale.
Actual Behavior
It doesn't scale and stays at (1,1,1). If I change the cursor scale in the inspector, it also gets back to (1,1,1) when I press Play.
Can I deactivate this default setting of the scale?
Steps to reproduce
https://github.com/StefanNeu/SC2018testing/tree/test/SC2018
Unity Editor Version
2017.4.2f2
Mixed Reality Toolkit Release Version
2<IP_ADDRESS>

This is because there is a animation that is driving the cursor's transform.
You should be able to make your own cursor and use it as needed.


Lifespan Middleware

trigger dbos launch/destroy from ASGI middleware when using FastAPI
move dbos_error_handler code into fastapi.py

Aside from the fact that it doesn't compile, looks good :-D


Bootstrapped status not properly handled

Describe the bug
The "bootstrapped" status is incorrectly assumed to be true after a single request, despite asking to run bootstrap.
Repro:

Add necessary dependencies to polaris-dropwizard-service

runtimeOnly(project(":polaris-eclipselink"))
runtimeOnly(libs.h2)
runtimeOnly(libs.postgresql)

Starting Polaris w/ eclipselink... using ./gradlew :polaris-dropwizard-service:runApp
curl http://<IP_ADDRESS>:8181/api/catalog/v1/config --> HTTP/500 / "Error 500 java.lang.IllegalStateException: Realm is not bootstrapped, please run server in bootstrap mode." --> OK
Another curl http://<IP_ADDRESS>:8181/api/catalog/v1/config --> HTTP/401 Unauthorized

To Reproduce
No response
Actual Behavior
No response
Expected Behavior
No response
Additional context
No response
System information
No response

@snazy here is the fix for this one as well as root cause analysis: https://github.com/apache/polaris/pull/579

I was not able to reproduce it with current master (5761b53423a5e9f23b10c3d3d2bd03602e07890e) when running from the jar.
java -jar dropwizard/service/build/libs/polaris-dropwizard-service-1.0.0-incubating-SNAPSHOT.jar server ./polaris-server.yml

I get 401 right away (not 500).

I was able to reproduce with #581

Fixed by #579.


NullReferenceException - FormsAppCompatActivity.InternalSetPage

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
Version Number of Plugin: 2.6.2 and 3.1.1
Device Tested On: Zenfone 2
Simulator Tested On: none
Version of VS: 2015 Comunnity 3
Version of Xamarin: 4.8.188
Versions of other things you are using:
Steps to reproduce the Behavior
Version previous to SDK 25 in android was working fine. After updated the SDK and Plugin version, it stopped working.
Using FormsAppCompatActivity in Android
Expected Behavior
Not block the app and allow to use camera
Actual Behavior
LoadApplication(new App) is giving the following exception:
System.NullReferenceException: Object reference not set to an instance of an object.
at Xamarin.Forms.Platform.Android.FormsAppCompatActivity.InternalSetPage (Xamarin.Forms.Page page) [0x0005e] in D:\agent_work\1\s\Xamarin.Forms.Platform.Android\AppCompat\FormsAppCompatActivity.cs:315
at Xamarin.Forms.Platform.Android.FormsAppCompatActivity.SetMainPage () [0x00000] in D:\agent_work\1\s\Xamarin.Forms.Platform.Android\AppCompat\FormsAppCompatActivity.cs:343
at Xamarin.Forms.Platform.Android.FormsAppCompatActivity.LoadApplication (Xamarin.Forms.Application application) [0x0025c] in D:\agent_work\1\s\Xamarin.Forms.Platform.Android\AppCompat\FormsAppCompatActivity.cs:139
at Shappa.Droid.MainActivity+d__0.MoveNext () [0x001b9] in C:\Andre\Shappa\Shappa\Shappa\Shappa.Droid\MainActivity.cs:62
Code snippet
https://gist.github.com/alwydl/e440f3e3745947ec4ad88597606a1a2f
Screenshots
If I take out the initialization process added for this new version (override permission, assemblyInfo and manifest) it enters the App class, but the camera does not work.

this happened to me to. Cleaning the solution, closing visual studio, deleting all obj and bin folder, restarting studio, rebuilding and trying again helped.

Same here. Delete bin/obj, clean rebuild helped me too.
Hope someday xamarin will be idempotent to rebuilds. :(

Hope you don't mind me updating this thread here, it's what the search engine turns reports for this error:
I found this error to be caused by a Resources/values/styles.xml file in my Xamarin.Android project, in particular I've resolved it by removing the "android:" prefix from each name attribute of each item node.

I came here from Google as well. For me it was solved by unchecking "Enable ProGuard" in Android options.

See this Bug in Xamarin.  I had to downgrade from Xamarin.Android.Support.* versions 27.0.2 to <IP_ADDRESS>.
https://bugzilla.xamarin.com/show_bug.cgi?id=36287

@developMediaSoft
This seems to have fixed the issue for me too. I had an "android:textAllCaps" and "android:windowNoTitle" item in styles.xml, removing the "android:" from them seems to resolve the issue

Same a @wireless-r , just unchecking "Enable Proguard" (not touching anything else) on release build did work around the issue.

This is an issue with X.F and it is tracked here: https://github.com/xamarin/Xamarin.Forms/issues/2148


Set up branching in 2_process

What changed
In this PR, the primary objective was to branch the map_data and chart_data targets across years. More comprehensively, in this PR I:

Added the ability to load huc data as an ESRI geodatabase (the natively downloaded format)
Added .empty to 1_fetch/in/ to establish file structure
Noted data source for F2F2 data in _targets.R
Split up target creation into 1_fetch.R, 2_process.R, and 3_visualize.R
Dynamically branched chart_data target over years (1984, 1985, ..., 2020)
Dynamically branched map_data target over half-years (1984, 1984.5, ..., 2019.5, 2020)
Attempted to branch map_animation_png over half years

Review request
Cee, can you please check that the pipeline runs and that the branching is correct. I think 3_visualize is weird, so if you only do the following pipeline, it might save some errors or other weirdness:
tar_make(map_data)
tar_make(chart_animation_gif)

Note: I changed the HUC input code (1_fetch.R) so that it matches the data directly downloaded from the Box site. You can still use the shapefile I sent you as the input if you change the code in line 19 of 1_fetch.R to read:
get_huc(file_in = ""1_fetch/in/F2F2_HUC12.shp",
              crs = 9311)),

@ceenell, thanks for the feedback. In this re-request I have addressed the following:

renamed the target huc to the more descriptive f2f2_huc12
changed the parameter year to the more descriptive year_range
Added interpolated data (at the 0.5 year) to the chart data target (I called it "shart_data" in the commit message, lol, oops)
combined the building of both charts into a single target (branched over years)

I got rid of the year text in the map, but I will add it back on the next PR - I want to mess with the styling a bit.

This looks really good, everything ran as expected.

The output pngs are ~1.3MB which might make the final gif pretty big, but we can deal with that when we get there. Go ahead and merge this and let's work on animating it.


Console error when launching Godot, and error with TextEdit node

Godot version
v4.2.dev.custom_build [da81ca62a]
System information
Windows 10, Forward+, NVIDIA GeForce GTX 1050 Ti (<IP_ADDRESS>79)
Issue description
I am not sure these two errors are connected or not, but I discovered them at the same time. And they look to be very similar (same file and line number), so I am going to assume they are related.
When I add a TextEdit node to my project, I see the following error when I run it.
E 0:00:00:0877   CallQueue::flush: Trying to execute a deferred call/notification/set on a previously freed instance. Consider using queue_free() instead of free().
  <C++ Source>   core\object\message_queue.cpp:327 @ CallQueue::flush()

After I saw this error the first time, I didn't know what it could be, so I closed the editor and started an editor with console, and I now see that Godot gives the following message several times in the console, even before I open my project for editing. It throws this error 3 times when opening Godot, and 7 more times when I launch a project with a TextEdit node.
ERROR: Trying to execute a deferred call/notification/set on a previously freed instance. Consider using queue_free() instead of free().
   at: CallQueue::flush (core\object\message_queue.cpp:327)

I tried to remove all projects from the project list, in case Godot was doing something with the projects in that list before editing a project, but even with an empty list, there are 3 instances of this error when launching Godot to the empty project list.
Steps to reproduce
Error 1.

Create a TextEdit node
Launch project
See error in debugger

Error 2.

Launch Godot with console
See errors in console

Minimal reproduction project
Mini project with a Node2d and a TextEdit node.
MRP - flush.zip
MRP for the console error when launching Godot will be simply Godot with the same version as I am using.

Thank you!


Remove cruft from platform descriptions

Simply put, I should be able to do
$ nix-build '<nixpkgs>' --arg crossSystem '{ config = "aarch64-unknown-linux-gnu"; }' -A stdenv

and that should work. I could make it work right now with a dirty white-list and fall-backs, but that's kind of defeats the point.
Currently, the biggest culprits are

Random top-level fields, like withTLS and openssl.system. See https://github.com/NixOS/nixpkgs/blob/master/lib/systems/examples.nix
The platform field, see https://github.com/NixOS/nixpkgs/blob/master/lib/systems/platforms.nix
arch which overlaps with the far more principled parsed,arch. See elaborate https://github.com/NixOS/nixpkgs/blob/master/lib/systems/default.nix#L17.

The general design issue is we need to handle the per-package random crap in a more modular way, without giving up the possibility of the user to override with their own values. My GNU ld emulation picking logic at https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/bintools-wrapper/default.nix#L167-L179 is certainly in a more modular location, but also cannot be nicely overridden.
Checklist

[x] #34303: withTLS
[x] #34315: bigEndian, openssl.system
[x] #34351: linux kernel cleanups
[x] #107214 / #110544: A few cleanups:

Get rid of nesting items under platforms for no good reason
group linux config fields rather than merely prefixing them
improve defaulting logic so config is enough in more cases.

[ ] Combine/rerrange platforms.nix and examples.nix

We should only default to generic/"multiplatform" settings; specific machines should be specified by hand.
Specific machines shouldn't be partly in platforms.nix and partly in examples.nix, that is just confusing.

[ ] Update manual to show how indeed the config is usually enough, and remove the reference to this issue.

CC @dtzWill @dezgeg @Infinisil @teto

Per https://github.com/NixOS/nixpkgs/pull/34645#discussion_r166325878 @dtzWill and I need to figure out whether to moved libc into parsed, or similar. @dezgeg or anyone else have any opinions? https://github.com/NixOS/nixpkgs/pull/34444 is also impacted.

For what it's worth, I've started using just the "config" in my cross specifications and so far so good!
I don't cross-build kernels as part of my work or testing, and I note that is the remaining TODO item :).  Just wanted to chime in that the rest is in a good place!
Minor convenience thing: it's annoying to specify --arg crossSystem '{ config = "x"; }' instead of something like --argstr crossConfig x, especially as the good work on this issue makes anything other than config unnecessary (at least usually).  Is there a simple way we could allow the shorter syntax without compromising abstractions?
(similar to how we handle "system" today, minus special nix support)

@dtzWill Maybe we can get --argstr crossSystem.config to work for 1.12? Though I feel bad asking for new features for that. I'm hesitant to add any more sugar as the platform stuff is already far too complicated.

@Ericson2314 thanks for the issue, and agreed not priority for nix update.  I was just wanting to discuss it to see what you had in mind.
I'll follow that issue, neat idea!
I agree we shouldn't further complicate the platform stuff, haha, it's actually partly why I asked O:).

I made a bunch of changes so this is now true. All the examples contain at most config, platform, and additional platform-specific flags like useIosPrebuilt that are new and don't overlap. Yay!

This is still linked in the manual, should be removed if closed.

Still important to me

I can take a look at this once #105364 is merged.

@Ericson2314 @Profpatsch Is this still an issue? Can the reference in the manual be deleted?

@Ericson2314 @Profpatsch Is this still an issue? Can the reference in the manual be deleted?

If we can merge https://github.com/NixOS/nixpkgs/pull/107214, and (better still) rearrange examples.nix and platforms.nix into something more sensible afterwords, then I'd say this is done.

If we can merge https://github.com/NixOS/nixpkgs/pull/107214, and (better still) rearrange examples.nix and platforms.nix into something more sensible afterwords, then I'd say this is done.

Added the checkboxes for those.

Added the checkboxes for those.

@siraben OK, just the final coding step (and manual victor lap) remains. If you want to take a stab at this, that would be much appreciated!

@siraben OK, just the final coding step (and manual victor lap) remains. If you want to take a stab at this, that would be much appreciated!

This issue is still referenced in the manual BTW


Chat Panel wrong initial aria message

Testing #186218
When focused on the chat panel the first time, NVDA read Chat input: type code and press enter to run. I don't think it's correct to call it "type code", it should be mentioning chat or conversation.

This instruction is meant to explicitly describe what action can be taken. What would you suggest it say instead?

This instruction is meant to explicitly describe what action can be taken

@meganrogge but isn't insert code wrong?

What would you suggest it say instead

We can at least align with place holder: "Type to ask Copilot questions or type / for topics, press enter to send out the request"

/duplicate https://github.com/microsoft/vscode-copilot/issues/334


playground-hide adds empty values on the undo history.

Context
The logic for folding sets undo history that clears the file.
https://github.com/google/playground-elements/blob/51fa356543916631cc288a50d62b1ca388a20d1b/src/playground-code-editor.ts#L776
For repro, there is a test linking this issue number reproducing the bug.
This behavior can also be viewed on lit.dev, with the clock example. Navigate to my-timer.ts, and change the file returning to my-timer. On undoing notice empty contents included in the undo stack.

Follow up from https://github.com/google/playground-elements/pull/281.
This PR introduces a behavior in CodeMirror where any readonly block will clear the history. Therefore the original issue no longer reproduces.
Next steps:

Should this edge case be documented?
Update test so this case is tested. Then if CodeMirror changes this behavior we'll know.
Maybe programmatic .value property updates should always clear history?

I would be going for number 2 or 3, or even both.
Having a test to track if this changes in the future, especially if we end up updating CodeMirror to the latest version would be great.
The situation where the value can be updated from the outside usually is one in which the content drastically changes, and in my opinion in these situations, it could be beneficial to clear the history.
I wonder if this will affect functionality with the playgrounds-ide though as the content is swapped when changing files, will the undo history just be erased when opened files are changed and changed back?

Agreed!

I wonder if this will affect functionality with the playgrounds-ide though as the content is swapped when changing files, will the undo history just be erased when opened files are changed and changed back?

I manually tested the last case, and think it works correctly because we:

Scope the history per virtual file.
Only set the fold/hide region on the initial .value setting of the virtual file (when the document instance is created).

Tested by updating configurator to have a folded region. Then edited beneath it - changed files a bit. Then checked that undo worked after swapping files.
However if later value is programmatically changed, it will clear the history (number 3).

I think if the cases you listed you tested work well, it would be 100% fine to clear the history on an external programmatic value change, as how I see it, the value change usually means "turning a new page" and changing the content/context, therefore it makes sense to ditch the history too

Whilst working on https://github.com/google/playground-elements/pull/295 I realized that I had missed that we could make the readonly configurable and not make the comments readonly.
And then I also found the root cause of this initial bug - which was due to a hacky way that the fold/hide regions were being reset by using the codemirror value.


Complete redesign

This is a complete redesign.

@JabRef/developers @JabRef/translators Feedback welcome!
Things to do:

[ ] Replace screenshots: ideas?

<3

Nice!
For not disappointing new users, it should be made more obvious that the main target of JabRef is (La)TeX (it is, right?).

That's a good point @mlep. I wasn't sure how much emphasize to put on BibTeX. On the one hand, it's what JabRef is currently known for - on the other hand, we do support LibreOffice, Word and I'm currently using a bib file in connection with pandoc which works great for markdown-based files. Why do you think new users would be disappointed if we don't emphasize BibTex?

Although not using it, I think that, at the moment, you cannot directly insert citations in ms word from JabRef (as stated in https://discourse.jabref.org/t/directly-insert-citations-in-microsoft-word-by-jabref/1297 , https://discourse.jabref.org/t/tools-to-make-ms-word-libreoffice-workflow-easier/1827/2 , and https://discourse.jabref.org/t/how-can-i-insert-a-citation-into-word-docx/386 ).
So, I guess that, at the moment, using JabRef for ms word is not as direct as a regular ms word user may be used to.

Another thing about not disappointing users: for many people, "works across all of your devices" may include Android. (but, well, we can wait for users asking for it :wink: ).

Agree, we should emphasize a bit on bibtex/latex and also mabye cross plattform (mac, win, linux)? instead of all devices

Thanks for the feedback. Updated version is now ready for review:

Following the positive feedback, let's be brave and merge this now ;-)