[BUG] Flutter git automatically adds to the Version Control

Describe the bug
Flutter git source adds automatically to the Android Studio Version Control, resulting in uncontrolled flutter version change.
After using git pull without removing this source we are able to update the flutter without knowing i.e. version 2.2.3 -> 2.5.3, FVM still acts as if we are using version 2.2.3 but flutter --version shows version 2.5.3.
To Reproduce

In any project with external git source add fvm by fvm use <version>
Fetch changes from source
Not always but often flutter gets updated to the newest possible version

Expected behavior
it would be nice to have the flutter git ignored or at least set to never update

@bartlomiejszm sorry for the delay on this. I was not able to replicate this behavior, are you using.
What for client are you suing for the pull?

@leoafarias sorry as well for forgetting about this case.
I am using AndroidStudio, always when creating a new fvm folder i have to ignore its origin from the list of VCS. i am unsure if we can do anything about this because it gets a bit frustrating to have to remove a whole installed version because it upgraded itself.

The only thing I can't think of is to automatically add .fvm/flutter_sdk to gitignore on first setup on a project

It does not help since inside .fvm/flutter_sdk is considered a different origin, i am ignoring the flutter_sdk folder by default in my projects

@Fasuh can you share how are you ignoring flutter_sdk on Android Studio?

in the settings there is a "-" button on the right which allows me to "ignore" roots, which basically as far as i can tell just adds an xml to the .idea/workspace.xml
  <component name="VcsManagerConfiguration">
    <ignored-roots>
      <path value="$PROJECT_DIR$/.fvm/flutter_sdk" />
    </ignored-roots>

I have updated the documentation with that information

Sure @leoafarias, I have created a PR here https://github.com/fluttertools/fvm/pull/430


Online learning without sampling from buffer

Hi,
thanks for the very refreshing paper. I notice that you followed the online learning pattern of "NeuralNetwork-Viterbi" paper by Richard et al. However, your training loss is designed on one entire video sequence while their cross-entropy loss is on a batch of 21-frame video chunks. I think that is why you did not sample frames from the buffer as they did.
I wonder why you still kept this online training pattern. Or have you tried training from all the training videos instead of one video at each iteration?
Regards

We use one video at each iteration because performing the Viterbi algorithm on a video is slow. If we use all videos in one iteration, it will take a very long time.
Anyway, to update more than one videos for each iteration is worth trying.

@wlin93 Hello, did you run "NeuralNetwork-Viterbi" paper's code? I downloaded the dataset they provided(Breakfast dataset) and ran their code but only got 0.375708 frame accuracy.

@JunLi-Galios Hello, I noticed that your iteration is 100K while "NeuralNetwork-Viterbi" use 10K, is your loss much harder to minimize? I am running your code and it really taks much time, maybe 3 days for me......

@wlin93 Hello, did you run "NeuralNetwork-Viterbi" paper's code? I downloaded the dataset they provided(Breakfast dataset) and ran their code but only got 0.375708 frame accuracy.

Hi, I recall that I could not reproduce the accuracy in the paper either. However, the author claimed that they used C++ implementation instead of the Python code in the repo.

@jszgz @wlin93 I think "NeuralNetwork-Viterbi" reports their max accuracy. You may refer to https://arxiv.org/pdf/1904.03116.pdf for the statistical learning of the related work.

@jszgz We use more iterations because we don't use a buffer during training. In  "NeuralNetwork-Viterbi", the authors use a buffer to store pseudo ground truth in previous iterations and reuse them during training.

Evaluate 252 video files...
frame accuracy: 0.503284

@JunLi-Galios Hello, I ran your code on breakfast dataset and got this accuracy, is it segmentation accuracy or alignment accuracy? If it is segmentation accuracy, then you've done a good job. Besides, would you consider to provide the code for alignment accuracy and other metrics?

@jszgz It's segmentation accuracy that is the main metric for Breakfast dataset. I would provide other metrics shortly.

@jszgz It's segmentation accuracy that is the main metric for Breakfast dataset. I would provide other metrics shortly.

Extremely thanks.

@JunLi-Galios Hello, I tried to train GTEA dataset with your code, and get 0.36 frame accuracy after 1600 iterations(since there are only 28 videos, which is 60 times smaller than breakfast, and learning rate is  decreased after 60% of the iterations) . Do I need to adjust some parameters or optimizer?

@jszgz First, you need to tune the learning rate and the number of total iterations. Second, you need to tune frame_sampling, window_size and step, both in train and inference. The longer the videos are, the larger these hyper-parameters are.

@jszgz @wlin93 @JunLi-Galios
We have done a reproducibility study and comparison of NNV, CDFL and ISBA methods in our technical report: https://arxiv.org/abs/2005.09743
Here is Table 1 of the paper comparison the reported accuracy and average and standard deviation of the accuracy for 5 trials (over all splits of breakfast)

Model
MoF Reported
MoF Avg (+- Std)

ISBA
38.4
36.4 (+- 1.0)

NNV
43.0
39.7 (+- 2.4)

CDFL
50.2
48.1 (+- 2.5)

@yassersouri Thanks for sharing your work.

@wlin93 Hello, did you run "NeuralNetwork-Viterbi" paper's code? I downloaded the dataset they provided(Breakfast dataset) and ran their code but only got 0.375708 frame accuracy.

I could get only 0.37 frame accuracy too. Notice that during NN-Viterbi training, the loss didn't decrease, it remains around 2.20 after 5000 iterations and stuck there.


DeployTabularModel action not working

I try to deploy a tabular model file (model.bim) to a power bi Workspace through a yaml pipeline.
Everything runs fine without errors, however, in the end nothing is deployed to the Workspace. What is going wrong, is this a bug?
With the publish action i can publish a pbix file without problems so the service endpoint is fine.
This is the code:

task: PowerBIActions@5
inputs:
PowerBIServiceEndpoint: 'powerbi-connectie'
Action: 'DeployTabularModel'
WorkspaceName: 'Playground'
PowerBIPath: 'models\poc.bim'
TabularEditorArguments: '-O -C -P -R -M -W -E -V'

Ok, i managed to get it working

@erikvangijzen: sorry for the late response. What was the fix for you?

Hi Maik,
I added the connection string to the TabularEditorArguments.
I used variables for all the variables😊 and I read the appkey from an azure keyvault:
TabularEditorArguments: '-D "Provider=MSOLAP;Data Source=powerbi://api.powerbi.com/v1.0/myorg/${{ variables.pbi_workspaceName }};User ID=app:${{ parameters.pbi_AppId }}@${{ parameters.tenantId }};Password=$(PowerBiAppKey);" "${{ parameters.modelname }}" -O -C -P -V -E'
I had quite a struggle to find out how this works, because there is no good documentation on this. I was also confused because i had to define the connection info again in the TabularEditorArguments, but it is also defined in the  PowerBiServiceEndpoint.
Vriendelijke groet,
Erik van Gijzen
From: Maik van der Gaag @.>
Sent: donderdag 20 april 2023 08:43
To: maikvandergaag/msft-extensions @.>
Cc: Erik van Gijzen @.>; Mention @.>
Subject: Re: [maikvandergaag/msft-extensions] DeployTabularModel action not working (Issue #482)
@erikvangijzenhttps://github.com/erikvangijzen: sorry for the late response. What was the fix for you?
—
Reply to this email directly, view it on GitHubhttps://github.com/maikvandergaag/msft-extensions/issues/482#issuecomment-1515792833, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3JPMSTHH3IVCI7ECOUVRNLXCDLFZANCNFSM6AAAAAAXCMWRC4.
You are receiving this because you were mentioned.Message ID: @.***>


AdditionalProtocArguments base_namespace error

What version of gRPC and what language are you using?
    <PackageReference Include="Google.Protobuf" Version="3.21.0" />

    <PackageReference Include="Grpc.Net.Client" Version="2.46.0" />

    <PackageReference Include="Grpc.Tools" Version="2.46.3">
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
      <PrivateAssets>all</PrivateAssets>
    </PackageReference>

What operating system (Linux, Windows,...) and version?
macOS 12.4
What runtime / compiler are you using (e.g. python version or version of gcc)
.NET 6 / SDK 6.0.300
What did you do?
In my csproj I specified this:
<Protobuf Include="Protos\greet.proto" GrpcServices="Client" AdditionalProtocArguments="--csharp_opt=base_namespace=Example" />
My proto file is:
syntax = "proto3";

package greet;

// The greeting service definition.
service Greeter {
  // Sends a greeting
  rpc SayHello (HelloRequest) returns (HelloReply);
}

// The request message containing the user's name.
message HelloRequest {
  string name = 1;
}

// The response message containing the greetings.
message HelloReply {
  string message = 1;
  oneof test_oneof{
    string name = 2;
    int32 age = 3;
  }
}

When I build my project I get the error Protos/greet.proto: Namespace Greet is not a prefix namespace of base namespace Example
What did you expect to see?
My generated files created with a namespace of Example
What did you see instead?
The error mentioned above`
Anything else we should know about your project / environment?

When you use the --csharp_opt=base_namespace=Example option the base namespace given must match the prefix in the namespace given in the proto file.
In the example above you need to change the proto file to contain:
package Example.greet;
it will generate C# code with:
namespace Example.Greet {
See https://developers.google.com/protocol-buffers/docs/reference/csharp-generated


Feature Request: Add Support for Writing Tags (Android only)

I would like to be able to write, in addition to read, some basic tags from an Android phone (my understanding is such actions are not currently permitted on iOS.) Is there any plan to implement this? I wish I could help but my Android/java is lacking.
(Edit: I do notice that phonegap-nfc supports it so maybe I'll take a look at porting it.)

Hi, I just merge this ndef-writing feature and publish to npm (v0.2.0), a full example is also provided in the revised example along with this library.
I would like to hear any feedback from you, Thanks!


Creating new input has tsc error on typed nodes.

Reproduction
https://github.com/LeonDixon/formkit-create-input
Describe the bug
In src/formkit.ts the create input errors on the feature function due to a node having a type assigned for the value.
Environment
• OS: MacOs
• Browser: Chrome
• Version: 120.0.6099.129 (Official Build) (arm64)

Creating pr

PR: https://github.com/formkit/formkit/pull/1106

@justin-schroeder Fixing error on build

@justin-schroeder new PR: https://github.com/formkit/formkit/pull/1109


Create organization profile

Github now allows creating organization profiles: https://github.blog/changelog/2021-09-14-readmes-for-organization-profiles/
All we need to do is create a README.md file under a profile directory in this repository.
Example:

https://github.com/JBossOutreach/.github/tree/main/profile
How its shown: https://github.com/JBossOutreach

cc @omerbensaadon @abrennan89


When I run "python webui_en.py --port 50000 --model_dir pretrained_models/CosyVoice-300M", I got the following error: ImportError: cannot import name 'speed_change' from 'cosyvoice.utils.file_utils'

I have received the following error after setting up the environment, running:
(cosyvoice) C:\Users\Voice AI\CosyVoice\CosyVoice>python webui_en.py --port 50000 --model_dir pretrained_models/CosyVoice2-0.5B
2024-12-29 14:32:43,422 - modelscope - INFO - PyTorch version 2.0.1+cu118 Found.
2024-12-29 14:32:43,422 - modelscope - INFO - Loading ast index from C:\Users\cache\modelscope\ast_indexer
2024-12-29 14:32:43,502 - modelscope - INFO - Loading done! Current index file version is 1.15.0, with md5 e984593bb28b11d497107b234faae6c2 and a total number of 980 components indexed
C:\Users\conda\envs\cosyvoice\lib\site-packages\torchvision\datapoints_init_.py:12: UserWarning: The torchvision.datapoints and torchvision.transforms.v2 namespaces are still Beta. While we do not expect major breaking changes, some APIs may still change according to user feedback. Please submit any feedback you may have in this issue: https://github.com/pytorch/vision/issues/6753, and you can also check out https://github.com/pytorch/vision/issues/7319 to learn more about the APIs that we suspect might involve future changes. You can silence this warning by calling torchvision.disable_beta_transforms_warning().
warnings.warn(BETA_TRANSFORMS_WARNING)
C:\Users\conda\envs\cosyvoice\lib\site-packages\torchvision\transforms\v2_init.py:54: UserWarning: The torchvision.datapoints and torchvision.transforms.v2 namespaces are still Beta. While we do not expect major breaking changes, some APIs may still change according to user feedback. Please submit any feedback you may have in this issue: https://github.com/pytorch/vision/issues/6753, and you can also check out https://github.com/pytorch/vision/issues/7319 to learn more about the APIs that we suspect might involve future changes. You can silence this warning by calling torchvision.disable_beta_transforms_warning().
warnings.warn(_BETA_TRANSFORMS_WARNING)
failed to import ttsfrd, use WeTextProcessing instead
Traceback (most recent call last):
File "C:\Users\Voice AI\CosyVoice\CosyVoice\webui_en.py", line 28, in
from cosyvoice.utils.file_utils import load_wav, speed_change, logging
ImportError: cannot import name 'speed_change' from 'cosyvoice.utils.file_utils' (C:\Users\Voice AI\CosyVoice\CosyVoice\cosyvoice\utils\file_utils.py)
My error seems very similar to these other ones people have brought up. Mine is slightly different in that it is about cosyvoice.utils.file_utils instead of huggingface :
https://github.com/FunAudioLLM/CosyVoice/issues/123#issue-2406818291
https://github.com/FunAudioLLM/CosyVoice/issues/527#issue-2601293308
https://github.com/FunAudioLLM/CosyVoice/issues/516#issue-2597602383
So I assume I will have to do something similar and revert a version of something to an older one? I just have no idea what to do.
The reason why I have "python webui_en.py" instead of "python3 webui.py" is because this tutorial told me to:
https://www.youtube.com/watch?v=7WblOkOxkXU
Thank you,

Okay, this time I ran:
python webui.py --port 50000 --model_dir pretrained_models/CosyVoice-300M
I then got the familiar: "ImportError: cannot import name 'cached_download' from 'huggingface_hub' "
So, I did "pip install huggingface_hub==0.24.7", it fixed the issue.
But now I got a new issue:
FileNotFoundError: [Errno 2] No such file or directory: 'pretrained_models/CosyVoice-300M/llm.pt'

we dont have webui_en.py in our repo

we dont have webui_en.py in our repo

Correct, I did the correct webui.py but I got this new error:
FileNotFoundError: [Errno 2] No such file or directory: 'pretrained_models/CosyVoice-300M/llm.p

we dont have webui_en.py in our repo

Correct, I changed it to tthe correct one "webui.py" but I got this new error:
FileNotFoundError: [Errno 2] No such file or directory: 'pretrained_models/CosyVoice-300M/llm.p

check readme for usage, follow install instruction


F&F >Awaiting acceptance > not showing wallet name on F&F refresh.

When I send F&F request using skip button it shows "Awaiting acceptance" instead of contact name or wallet name till confirmation. I refreshed the F&F page still it was not showing me the name of the wallet. I clicked on that "Awaiting acceptance", it navigated to the details page and when I came back then it was showing me the name of the wallet.

Verified this issue on dev app v1.9.5(330)


std.complex: Add complex norm() and proj()

A couple more trivial additions to std.complex, all that remain (future PRs) are a(sin|cos|tan), (sin|cos|tan)h, and possibly isClose.
N.B: should norm be named toNorm?

@n8sh - I take that to mean the current names are fine then. :-)

@thewilsonator - all fine here?  Also, norm or toNorm ?

norm is my choice.

norm is my choice.

Ready to merge then.


Ast_mapper: OCaml version mismatch or malformed input

Steps to reproduce:
$ ocamlfind ocamlopt -predicates ppx_driver -o ppx.out -linkpkg -package ppx_let ppx_driver_runner.cmxa
findlib: [WARNING] Interface topdirs.cmi occurs in several directories: /Users/vladimir/.opam/4.03.0/lib/ocaml, /Users/vladimir/.opam/4.03.0/lib/ocaml/compiler-libs

$ echo 'let () = print_endline "hi"' > test.ml

$ ./ppx.out test.ml
let () = print_endline "hi"

$ ./ppx.out -as-ppx test.ml test.out
Fatal error: exception Failure("Ast_mapper: OCaml version mismatch or malformed input")

My opam switch for 4.03.0:
base-bigarray   base    root
base-bytes  base    installed
base-threads    base    root
base-unix   base    root
bin_prot    113.33.00+4.03  installed
camomile    0.8.5   installed
conf-m4 1   installed
core_kernel 113.33.01+4.03  root
cppo    1.3.2   installed
fieldslib   113.24.00   installed
lambda-term 1.10    installed
lwt 2.5.2   installed
menhir  20160526    root
ocamlbuild  0.9.2   installed
ocamlfind   1.6.2   installed
ppx_assert  113.33.00   installed
ppx_bench   113.33.00+4.03  installed
ppx_bin_prot    113.33.00+4.03  installed
ppx_compare 113.33.00+4.03  installed
ppx_core    113.33.01+4.03  installed
ppx_custom_printf   113.33.00+4.03  installed
ppx_deriving    3.3 root
ppx_driver  113.33.01+4.03  installed
ppx_enumerate   113.33.00+4.03  installed
ppx_expect  113.33.01+4.03  installed
ppx_fail    113.33.00+4.03  installed
ppx_fields_conv 113.33.00+4.03  installed
ppx_here    113.33.00   installed
ppx_inline_test 113.33.00+4.03  installed
ppx_jane    113.33.00   installed
ppx_let 113.33.00+4.03  root
ppx_optcomp 113.33.00+4.03  installed
ppx_pipebang    113.33.00+4.03  installed
ppx_sexp_conv   113.33.01+4.03  root
ppx_sexp_message    113.33.00+4.03  installed
ppx_sexp_value  113.33.00+4.03  installed
ppx_tools   5.0+4.03.0  installed
ppx_type_conv   113.33.01+4.03  root
ppx_typerep_conv    113.33.00+4.03  installed
ppx_variants_conv   113.33.00+4.03  installed
re  1.6.1   installed
react   1.2.0   installed
result  1.2 installed
sexplib 113.33.00+4.03  installed
typerep 113.24.00   installed
utop    1.19.2  root
variantslib 113.24.00   installed
zed 1.4 installed

OS X 10.11
opam 1.2.0

Sorry, wrong repo. Moving the issue to ppx_driver

Moved to janestreet/ppx_driver#3
(As a note for future seekers, since this came up first in my search).


[gen_l10n] Accept newlines in generateString utility function

Description
Handles newline characters in generateString, which in turn allows newlines to be used in gen_l10n ARB file messages.
Related Issues
Fixes https://github.com/flutter/flutter/issues/53583
Tests
I added the following tests:

A unit test for generateString

An integration test for the gen_l10n tool*

*This test required that the newline character generated to be replaced by a substitute 'NEWLINE' to represent it in the test case. This is because the stdout stream listener breaks up its inputs by line. since the test only saves lines starting with '#l10n',  this would require that any string with newlines to be treated as separate lines. The alternative approach here would be to manually look for newlines in the messages, split the output up where '\n' is found, prepend '#l10n $lineNumber' to every segment and print them all.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

Accoding to the internet, JSON handles a few more characters specially:
Backspace is replaced with \b.
Form feed is replaced with \f.
Newline is replaced with \n.
Carriage return is replaced with \r.
Tab is replaced with \t.
Double quote is replaced with \"
Backslash is replaced with \\

Although they're relatively unlikely to come up, I assume we also need to replace \b, \f, \t, and \r characters with the corresponding escape sequences. We're already dealing with newline, quotes, and backslash.


docs: Update complete example to include aliased providers.

Description
Added some documentation around using aliased providers to the complete example. This took a while for me to find / figure out so I thought it would be helpful to put it in the example.
Motivation and Context

This solved the issue where this solution was poorly documented. I found this fix by digging though old closed issues.
Breaking Changes

No
How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

This is currently deployed in this manner in our dev environment.

Thank you but I think this is more generic to terraform usage and not adding a lot of value directly to this module. Thank you though!


Wrong Typings for hockey app

Current version of cordova-plugin-hockeyapp have the following js signature:
var hockeyapp = {
start: function(success, failure, appId, autoSend, checkForUpdateMode, ignoreDefaultHandler, createNewFeedbackThread, loginMode, appSecret) {

The typings of your module have:
    start(androidAppId: string, iosAppId: string, sendAutoUpdates: boolean, ignoreErrorHeader: boolean): Promise<any>;

I guess that this is not matching

Closed because you implement abstraction over official sdk


Handling of min-height/width with flex-basis: 0% is inconsistent

Refer to http://codepen.io/anon/pen/xZPobW
I have a 200px by 200px flexbox in a column with two items that both have flex: 1 1 0%. The second item also has min-height: 100px. In Chrome/Firefox/Safari/Opera, the min-height is not considered when calculating the flex allocation (so the two items are both 100px tall), whereas, in IE 11, the min-height is subtracted out and only the difference is used in the flex allocation (so item 1 is 50px tall and item 2 is 150px tall). See Example 1 in the Pen.
The same thing happens with a row and min-width. See Example 2 in the Pen.

Hmmm, yeah, it looks like IE11 treats the min size declaration as the items initial starting size and calculates free space after that.
Have you discovered a workaround?

I'm seeing something very similar, I haven't been able to figure out any workaround.
https://codepen.io/JacobDB/pen/BmbOao

@JacobDB I don't add bugs to the list unless there's a workaround. Otherwise the list would just be a bug aggregate, and I want it to be a solutions list instead.


Workspace deps have empty pth file and can't be imported

Reproducer repo: https://github.com/konstin/albatross
I have a workspace with a main library and a workspace package the main package depends on, both created with rye init:
.
├── Dockerfile
├── providers
│   └── pypi_provider
│       ├── pyproject.toml
│       ├── README.md
│       └── src
│           └── pypi_provider
│               └── __init__.py
├── pyproject.toml
├── README.md
├── requirements-dev.lock
├── requirements.lock
└── src
    └── albatross
        └── main.py

After running rye sync, i get a .venv/lib/python3.11/site-packages/_albatross.pth with the correct /home/konsti/projects/albatross/src, while .venv/lib/python3.11/site-packages/_pypi_provider.pth is incorrectly empty, so the pypi_provider can't be imported.
The linked repo contains a dockerfile reproducing this problem. The problems happens with both pip and uv, latest release and b290caed2a7b5d38cc4c8defc53912b41b8039ad.

No clue what triggers this, but I cannot reproduce this with setuptools, only with hatchling :-/ setuptools does the right thing:
$ cat .venv/lib/python3.11/site-packages/__editable__.*.pth
/private/tmp/albatross/src
/private/tmp/albatross/providers/pypi_provider/src

Found a closed issues on hatch which implies that something has been fixed:

https://github.com/pypa/hatch/issues/860

Can be reproduced with uv directly by just running uv pip sync requirements.lock.

I notice that your nested pyproject contains
[tool.hatch.build.targets.wheel]
packages = ["src/pypi"]

but this doesn't match the source directory which is src/pypi_provider? I could be mistaken, but worth recreating the project or checking this.

Yap. That fixes it. Would be nice if hatch would print out something useful there or error :(


Error during upgrade 2.1.3 to 2.1.5

When upgrading 2.1.3 to 2.1.5 I get the following error during ./upgrade.sh
When browsing to the website, Internal Server Error is displayed.
Applying database migrations (python netbox/manage.py migrate)...
Traceback (most recent call last):
File "netbox/manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 364, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 356, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 283, in run_from_argv
self.execute(*args, **cmd_options)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 327, in execute
self.check()
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 359, in check
include_deployment_checks=include_deployment_checks,
File "/usr/local/lib/python2.7/dist-packages/django/core/management/commands/migrate.py", line 62, in _run_checks
issues.extend(super(Command, self)._run_checks(**kwargs))
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 346, in _run_checks
return checks.run_checks(**kwargs)
File "/usr/local/lib/python2.7/dist-packages/django/core/checks/registry.py", line 81, in run_checks
new_errors = check(app_configs=app_configs)
File "/usr/local/lib/python2.7/dist-packages/django/core/checks/urls.py", line 16, in check_url_config
return check_resolver(resolver)
File "/usr/local/lib/python2.7/dist-packages/django/core/checks/urls.py", line 26, in check_resolver
return check_method()
File "/usr/local/lib/python2.7/dist-packages/django/urls/resolvers.py", line 254, in check
for pattern in self.url_patterns:
File "/usr/local/lib/python2.7/dist-packages/django/utils/functional.py", line 35, in get
res = instance.dict[self.name] = self.func(instance)
File "/usr/local/lib/python2.7/dist-packages/django/urls/resolvers.py", line 405, in url_patterns
patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
File "/usr/local/lib/python2.7/dist-packages/django/utils/functional.py", line 35, in get
res = instance.dict[self.name] = self.func(instance)
File "/usr/local/lib/python2.7/dist-packages/django/urls/resolvers.py", line 398, in urlconf_module
return import_module(self.urlconf_name)
File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/opt/netbox/netbox/netbox/urls.py", line 3, in
from rest_framework_swagger.views import get_swagger_view
File "/usr/local/lib/python2.7/dist-packages/rest_framework_swagger/views.py", line 5, in
from rest_framework.schemas import SchemaGenerator
File "/usr/local/lib/python2.7/dist-packages/rest_framework/schemas/init.py", line 32, in
authentication_classes=api_settings.DEFAULT_AUTHENTICATION_CLASSES,
File "/usr/local/lib/python2.7/dist-packages/rest_framework/settings.py", line 223, in getattr
val = perform_import(val, attr)
File "/usr/local/lib/python2.7/dist-packages/rest_framework/settings.py", line 168, in perform_import
return [import_from_string(item, setting_name) for item in val]
File "/usr/local/lib/python2.7/dist-packages/rest_framework/settings.py", line 184, in import_from_string
raise ImportError(msg)
ImportError: Could not import 'utilities.api.TokenAuthentication' for API setting 'DEFAULT_AUTHENTICATION_CLASSES'. ImportError: cannot import name is_authenticated.
Collecting static files (python netbox/manage.py collectstatic --no-input)...
0 static files copied to '/opt/netbox/netbox/static', 264 unmodified.

Thank you for your interest in NetBox. GitHub issues are intended for reporting bugs and requesting features, and must be submitted using the template provided here. For general discussion, questions, or assistance with installation issues, please post to our mailing list.

My apologies.


Question about RMapCache and containsKey

I am trying to figure out how redisson takes a custom object and makes that into a key.  In my code I have implemented hashCode and equals method but when my subclass is saved as a key and then the parent class is used to try and access it, containsKey returns false even though both equals and hashcode are implemented and equal each other.
Ive tried using custom writeObject and readObject but that didn't work either.
When coming from caffeine we just had to implement a comparator and that worked but I'm not sure if I can do something similar for this.
Example class:
public class BytesKey extends ... implements Comparable<BytesKey>, Serializable {
protected int hashCode;
public BytesKey(byte[] bytes) {
    super(bytes);
    this.hashCode = super.hashCode();
  }

@Override
  public int hashCode() {
    return hashCode;
  }
@Override
  public boolean equals(Object o) {
    int test = o.hashCode();
    return this.hashCode == o.hashCode();
  }

  private void writeObject(ObjectOutputStream oos) throws Exception
  {
    oos.writeObject(bytes);
    oos.writeObject(hashCode);
  }

  private void readObject(ObjectInputStream ois) throws Exception
  {
    // performing default deserialization of Account object
    bytes = (byte[]) ois.readObject();
    hashCode = (int) ois.readObject();
  }
}

Subclass
private static class InternedBytesKey extends BytesKey implements Serializable {
InternedBytesKey(byte[] bytes) {
      super(bytes);
    }
...
}

Just to follow up, I'm trying to figure out how Redisson determines how to convert an object into a key?

hashCode()/equals() methods of key object aren't used, instead it uses hash of serialized state.


sharding-ui-backend support spring-boot-maven-plugin plugin

Feature Request
Describe the feature you would like.
durating develop, if want run a sharding-ui-backend must be package then java -jar run. maybe add a spring-boot-maven-plugin then spring-boot:run more easy...

@terrymanu Is this need appropriate?  I can do it...

@beijing-penguin Thank you, the pull request has merged, maybe next time.


JUnit XML text does not encode unsafe characters

When writing text to JUnit XML, illegal characters in <system-out> are not escaped.
To fix, I extracted the escaping code in JUnitTestOutput::writeFailure() into a separate function, which is called from both writeFailure() (for the failure text) and in writeTestCases() (for <system-out>). While I was in there, I revised the code to use correct escapes; now failure messages like "CHECK_C(result > -1)" are properly emitted as "CHECK_C(result > -1)".
I'm a little concerned about using SimpleString.replace() is this way, repeatedly running over the same string. But for now I'm focussing on making it functionally correct and will make it fast later.

Thanks. But no pull request for this?

Happy to make the PR. I'm a n00b with this code base, so wanted to confirm you were interested in the PR first.

Having a PR is a lot easier to look at then a description, so would definitively be interested :)

Question: Since I needed it in a hurry (the builds were breaking :D), my encoder function is brute force:
SimpleString JUnitTestOutput::encodeXmlText(const SimpleString& textbody)
{
    SimpleString buf = textbody.asCharString();
    buf.replace("\"", "\\\"");
    buf.replace("&", "&");
    buf.replace("<", "<");
    buf.replace(">", ">");
    buf.replace("\n", "{newline}");
    return buf;
}

I was thinking about adding an implementation of SimpleString::replace() that accepts a mapping table, something like this:
    char         to;
    SimpleString with;
} replacemap_t;

    void replace(replacemapt_t tw[]);

and
    { '"', "\\\"" },
    { '&', "&"},
    { '<', "<"},
    { '>', ">"},
    { '\n', "{newline}"}
};

Would you accept a new method in SimpleString?

Had a question, which I deleted because it was a terrible idea.

Fix verified, thank you!


Symbolic backend

Backend to work with symbols instead of integer literals.
@rkaminsk  please check if some functions need to work with negative literals. They are marked with # TODO

Just some comments. The public interface of a library has to be 100% documented. For a class like the SymbolicBackend it might be pretty obvious what the functions are doing. But it still improves usability to have (short) docstrings. With an IDE, you get the short method descriptions along with auto-completion. If SymbolicBackend.add_rule has the description "Add a rule over symbolic literals to the underlying Backend.", then this is valuable. It would of course be even better to also explain the parameters. Here we can probably get away with pointing out in the class' docstring that each method has a corresponding in clingo's Backend. What is also important to document is how context management (the with statement) is handled. Currently, the user can only figure this out by looking at the source code.
 This belongs into a docstring and the name should be shorter.

Nice. Now we just need to extend the unit tests to test each function and that should be it. (The same goes actually for the controlx PR. I am into paper writing. That's why I did not continue there. But I think that there is also not that much missing to complete this PR.)


Drop /server, and dependencies, not used anymore

server/app.js is not being used anymore, since yarn run start just runs webpack-dev-server instead.
This seems to have been the case since somewhere between https://github.com/ManageIQ/manageiq-ui-service/pull/296 (drops the start script running server/app.js) and https://github.com/ManageIQ/manageiq-ui-service/pull/415 (adds a start script running webpack-dev-server).
Removing the old server, and its dependencies... and updating the skin-sample to be able to live without server/.

LOVE THIS!!! less is more for sure @himdel well the only real place it is being used is here
but we could 🔨 those two, cuz no one uses start:all could also get rid of start:mock-api 🤔

Ooh, I missed those, sorry O:-).
Ok, so, you're positive nobody is using those? I can just drop them too...

..and done :)


Fix valgrind warnings related to varcharvarbinary function

Description
Currently, we have defined the following cast functions which call varcharvarbinary C function with only 1 argument. But, in varcharvarbinary function, we are reading two arguments which might lead to memory related issues.

sys.binarysysvarchar
sys.binaryvarchar
sys.rowversionsysvarchar
sys.rowversionvarchar

This change modifies the above casts to have 3 arguments, one for input, one for typmod (integer) and the other for specifying isExplicit (boolean)
This change also replaces varcharTypeName->typeOid with baseTypeId in tsql_coerce_string_literal_hook() function as we are not setting varcharTypeName->typeOid at all which causes failure while trying to fetch the PG view definition as shown below.
Ex :
-- T-SQL endpoint
1> create view babel_4475_v1 as select cast(cast('a' as binary(2)) as varchar(2))
2> go

-- PSQL endpoint
jdbc_testdb=# select pg_catalog.pg_get_viewdef(oid, true) from pg_class where relname = 'babel_4475_v1';
ERROR:  cache lookup failed for type 0
jdbc_testdb=#

Issues Resolved
BABEL-4475
Signed-off-by: Sai Rohan Basa<EMAIL_ADDRESS>Test Scenarios Covered

Use case based - Tests related to above cast functions have already been written in babel_datatype.sql and BABEL-ROWVERSION.sql

Also added tests in babel-4475-vu-*.sql

Boundary conditions - N/A

Arbitrary inputs - N/A

Negative test cases - N/A

Minor version upgrade tests - babel-4475-vu-*.sql

Major version upgrade tests - babel-4475-vu-*.sql

Performance tests - N/A

Tooling impact - N/A

Client tests - N/A

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 71.703%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
39365

Relevant Lines:
54900

💛  - Coveralls


Fix topic title

Closes #2646
@MSoeb should the number be included only in that title or also everywhere else where the topic title is displayed?

The number should be included everywhere
Can you indicate in a comment or a commit where the numbering would be in included?


How do I format cookies into JSON?

I noticed you have a JSON class but I'm not sure how to use it.

The DynamicJsonConverter is currently only there for the deserialization of JSON strings. I will maybe add an option to convert browser information to JSON in the next version tho.
To serialize data into JSON string I would recommend using a library like Newtonsoft.Json
Tutorials:
https://www.newtonsoft.com/json/help/html/SerializingJSON.htm
https://riptutorial.com/json-net/example/6594/how-to-serialize-an-object-to-json-using-json-net-in-csharp
Basically, you just create a class that stores arrays like that:
public class StoredData
{
    public Blink.Cookie[] StoresBlinkCookies { get; set; }
    public Gecko.Bookmark[] StoresGeckoBookmarks { get; set; }
    public Blink.Login[] StoresBlinkLogins { get; set; }
}

Then you just set the Arrays and call a Serialize method like this one (The method depends on what JSON converter you use but it's generally something like this):
StoredData sd = new StoredData() // Store data in class
{
    StoredBlinkCookies = _cookies,
    StoresGeckoBookmarks = _bookmarks,
    StoresBlinkLogins = _logins
};
string json = JsonConvert.SerializeObject(sd); // Convert class to JSON string

Another option would be to create a string in C# and add the values in (However, this is only efficient if you have little data):
string json = $"{\"{cookie.Name}\": \"{cookie.Value}\", \"{cookie.Name}\": \"{cookie.Value}\"}";


[javanaut][operationExpr][1/2]feat: Add Assignment operation expr with multiply and assignment operator

enables *= operator.
^= will come with next PR [2/2]
type-checking tables in go/javanaut-assignment-operation-expr

Addressed the comments in the design doc with changes:
Update the convenience function name using your suggestion.
Rename lhsExpr to variableExpr and rhsExpr to valueExpr. Add check to make sure variable expr is not decl with unit tests.
Add comments.


IE11: Displayed image size is sometimes wrong in initial image load

I am facing a problem that only shows in Internet Explorer (I am testing with IE11) where the displayed image size is wrongly reported as 28x30 pixels (this seems to be the default value reported by IE when an image is broken or not yet available).
This only happens when the image is initially fetched (not when loaded from the cache), and it is not always reproducible.
I inserted some traces in the sourceImageOnLoadF() function and I verified that when the problem occurs:

displayedImage.width is always 28
displayedImage.height is always 30
displayedImage.complete is true (this puzzles me)

If I attach an onload handler to displayedImage, then when this handler fires the displayed image dimensions are always correct (but note that in IE this handler will not fire if the image comes from the cache.)
One possible way to fix this would be to attach an onload handler to displayedImage. If this fires, then check the displayed image dimensions. If they are different than the dimensions seen by sourceImageOnLoadF, then trigger a resize.

I've recently been thinking it might be worth changing the resize event binding from the window to the image, as their are other things that might help with as well. Do you want to give it a go and send a pull request if it works.

I can try, but the problem I am describing is not related to resizing -- it
happens during initial image load, even if the image renders at its natural
size.

I've recently been thinking it might be worth changing the resize event
binding from the window to the image, as their are other things that might
help with as well. Do you want to give it a go and send a pull request if
it works.
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73124562
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS>

Yep, but I'm hoping it fires the resize event on the image when it change
from 30x28 to the real size.
On Thursday, February 5, 2015, Guillermo Rodríguez<EMAIL_ADDRESS>wrote:

I can try, but the problem I am describing is not related to resizing -- it
happens during initial image load, even if the image renders at its natural
size.

I've recently been thinking it might be worth changing the resize event
binding from the window to the image, as their are other things that
might
help with as well. Do you want to give it a go and send a pull request if
it works.
—
Reply to this email directly or view it on GitHub
<
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73124562
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','guille.rodriguez@gmail.com');>
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73136158
.

--
Sent from my iPhone.

Just tested. IE11 does not fire resize events for the image, only for the
window (IE10 does fire resize events for the image, but this does not work
for IE11).
Seems that the ability to attach a resize event handler to arbitrary
elements was an IE-only thing, and it is no longer supported with IE11.
2015-02-05 22:51 GMT+01:00 David J. Bradshaw<EMAIL_ADDRESS>
Yep, but I'm hoping it fires the resize event on the image when it change
from 30x28 to the real size.
On Thursday, February 5, 2015, Guillermo Rodríguez <
<EMAIL_ADDRESS>wrote:

I can try, but the problem I am describing is not related to resizing --
it
happens during initial image load, even if the image renders at its
natural
size.

I've recently been thinking it might be worth changing the resize event
binding from the window to the image, as their are other things that
might
help with as well. Do you want to give it a go and send a pull request
if
it works.
—
Reply to this email directly or view it on GitHub
<

https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73124562

.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','guille.rodriguez@gmail.com');>
—
Reply to this email directly or view it on GitHub
<
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73136158
.

--
Sent from my iPhone.
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73136820
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS>

Do you still need IE8 support?
On Fri, Feb 6, 2015 at 1:58 PM, Guillermo Rodríguez <
<EMAIL_ADDRESS>wrote:

Just tested. IE11 does not fire resize events for the image, only for the
window (IE10 does fire resize events for the image, but this does not work
for IE11).
Seems that the ability to attach a resize event handler to arbitrary
elements was an IE-only thing, and it is no longer supported with IE11.
2015-02-05 22:51 GMT+01:00 David J. Bradshaw<EMAIL_ADDRESS>
Yep, but I'm hoping it fires the resize event on the image when it change
from 30x28 to the real size.
On Thursday, February 5, 2015, Guillermo Rodríguez <
<EMAIL_ADDRESS>wrote:

I can try, but the problem I am describing is not related to resizing
--
it
happens during initial image load, even if the image renders at its
natural
size.

I've recently been thinking it might be worth changing the resize
event
binding from the window to the image, as their are other things that
might
help with as well. Do you want to give it a go and send a pull
request
if
it works.
—
Reply to this email directly or view it on GitHub
<

https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73124562

.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','guille.rodriguez@gmail.com');>
—
Reply to this email directly or view it on GitHub
<

https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73136158

.

--
Sent from my iPhone.
—
Reply to this email directly or view it on GitHub
<
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73136820
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS>—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-73239659
.

--
David J. Bradshaw )'<EMAIL_ADDRESS>

No, I'm only targetting IE10+.
I have a fix for this which works as described on my first comment. Would you like to have a look?

Yes please. I'm also working on an updated version that drops IE8, as it makes the code much simpler.

Any chance you can send me the fix you had?

Sorry, I've been (and still am) overloaded for the last couple of days. Let
me come back to this issue tomorrow or Friday and I'll take care of this.
2015-02-18 12:52 GMT+01:00 David J. Bradshaw<EMAIL_ADDRESS>
Any chance you can send me the fix you had?
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-74852491
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS>

OK thanks.

So as a guess was it:
displayedImage.onload = start;  //Catch slow loading images

As I can not repro the issue

Hi David,
This is the fix that worked for me:
The problem is that IE11 will first report that the image is completely loaded, but the size is wrong. What I did was to add an onload handler to displayedImage, and then trigger a resize from there, but only if the image size is different than the first time:
        function start(){
    	var displayedWidth  = null,
    		displayedHeight = null;

        var testImage = new Image();
        testImage.onload = function(){
            sourceImageWidth  = testImage.width;
            sourceImageHeight = testImage.height;
            displayedWidth    = displayedImage.width;
            displayedHeight   = displayedImage.height;

            if ((displayedWidth !== sourceImageWidth) || (displayedHeight !== sourceImageHeight))
                resizeMap();
        }
        displayedImage.onload = function(){
        	/* If testImage.onload() already fired, but we see different
        	 * displayed dimensions now, trigger a new resize.
        	 */
			if (displayedWidth != null && displayedWidth != displayedImage.width) {
				//console.log("Displayed image size changed after initial check");
				resizeMap();
			}
        }
        testImage.src = displayedImage.src;
    }

This fixes the problem for me (note that the fix was implemented on top of 0.3.4, but you get the idea)
I also added an additional check in testImage.onload that skips the initial resize if the size is w=28, h=30. This saves one resize operation in this specific case, but I am aware that it is a bit of a hack, and it is not strictly needed for the fix to work :)

Hey, do you want to make a pull request, as you found the fix?

Yes I'll do that. I'll first adapt the fix to the current release. Will
also do some testing to be 100% sure as the bug is so elusive.
2015-02-26 22:08 GMT+01:00 David J. Bradshaw<EMAIL_ADDRESS>
Hey, do you want to make a pull request, as you found the fix?
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-76273679
.

--
Guillermo Rodriguez Garcia
<EMAIL_ADDRESS>

Thanks

Hi, any update on this one?

It's been on my TODO list for more than one month now. I need to find some time to finish it.
El 09/04/2015 a las 11:56, Bobby escribió:

Hi, any update on this one?
—
Reply to this email directly or view it on GitHub https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-91179283.

Sorry for the very very very very late response.

Thanks will take a look over the weekend.
On Wed, Apr 15, 2015 at 7:12 AM, Guillermo Rodríguez <
<EMAIL_ADDRESS>wrote:

Sorry for the very very very very late response.
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/10#issuecomment-93418853
.

--
David J. Bradshaw )'<EMAIL_ADDRESS>

Released as v0.5.4

thanks :+1:

Hi all, I still get a similar issue - on very poor connections (I was viewing the page from a Peter Pan Bus using the free wifi) the resize does not happen. I've resolved by wrapping my code in Dave DeSandro's imagesLoaded library.


deprecated features should be less prominent in the documentation

Martin Nowak (@MartinNowak) reported this on 2013-04-19T11:43:51Z
Transferred from https://issues.dlang.org/show_bug.cgi?id=9962
CC List

berni44 (@WalterBright)

Description
Deprecated features should at least not be listed in the jumpTo section of the documentation.
Furthermore we might want to put it towards the bottom of the page.

bugzilla (@WalterBright) commented on 2019-12-14T11:02:25Z
I neither found any jumpTo section nor prominent deprecated features. This has probably been fixed. If not, could you please provide an example?

bugzilla (@WalterBright) commented on 2019-12-14T12:44:15Z
Sorry, stupid me. Meanwhile I found the jumpTo sections and deprecated sections are still in there (see std.zip, diskNumber; in master, not yet in stable)


Added some-feature1

Breaking Change:

Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly (example).
[ ] New dependencies have been added to requirements in the manifest (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @wjl198435,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

only test

sdfasf
asdf
asf
as
df


Google Maps: Proposal: Add Hexadecimal code / HSL for colors to get accurate colors.

Hi Google,
So I am developing an application where I need to fetch marker colors for Google maps in Flutter. To my dismay when I saw that Google maps flutter marker icon Color is only in BITMAPDESCRIPTOR and that is in only HUE format.
When the whole flutter application is using hexadecimal colors why are we only using the first value of HSL color and only "double hue."
**Problems and Limitations with this current scenario:

if we send only hue color it appears different on the application because the HSL (SATURATION AND LIGHT) are different for different colors. While Google is only using hue. This makes applications post very different results from expected results.

The Only solution I found on the internet was to send map marker icons and display them on the map. Which is also very absurd.**

HERE in this example from bitmap.dart file of official google maps flutter.
  static const double hueRed = 0.0;

  /// Convenience hue value representing orange.
  static const double hueOrange = 30.0;

  /// Convenience hue value representing yellow.
  static const double hueYellow = 60.0;

  /// Convenience hue value representing green.
  static const double hueGreen = 120.0;

  /// Convenience hue value representing rose.
  static const double hueRose = 330.0;

As you can see the expected result is far more different than the given input, This is devastating as our requirement is to create specific colors for different categories in the application
What Google can do?
Please create a unified system for flutter. When the whole application is using HEXADECIMAL APPROACH why only google maps is using the bitmap descriptor color and only in the HUE format without Saturation and Light.
Please either add HSL as a full-function to get the accurate colors.
THANKS.

This appears to be the underlying cause for a problem I've been trying to address for some time.  When I use BitmapDescriptor.hueYellow in my app, it displays a perfect yellow on Android.  But on iOS, it's a sickly looking green.   Have there been any updates since the original proposal?
Attached are two screen shots showing a modified version of the google_maps_flutter example app that demonstrates the problem.


Cleanup types.ts in iov-bns

~Based on #1069~
Closes #1055

Nice, does the CHANGELOG need updating?

Thanks. I don't think so, since this is all package internal and does not change externally visible bevaviour.


byebug does not work in docker terminal

adding byebug to a method in a controller does not pause application when using Docker compose
unable to interact during byebug session to debug

Fixed with PR #22


Add Prisma tracing instrumentation to the registry

Adds prisma tracing instrumentation to the registry.
Please let me know if any modifications to this PR or anything else is needed to get this approved.

@TasinIshmam please make sure to sign the CLA

Hi @svrnm  and @chalin
The CLA has been signed. Please feel free to merge if there are no other issues.


Correction for raw affiliation Western University, Faculty of Health Sciences, London, ON, CA

Correction needed for raw affiliation Western University, Faculty of Health Sciences, London, ON, CA
raw_affiliation_name: Western University, Faculty of Health Sciences, London, ON, CA
new_rors: 02grkyz14
previous_rors: 05167c961
works_examples: W4367173349;W4385337281
contact: a98c1eaed5f427237aa2f613f28363f1:955902c665d0ecaf @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


How to customize animation parameters？

i need to set some different parameters for animation
how can i do this, or implements the BaseViewAnimator ?

i got it, here is my way:
CustomPulseAnimation.kt
class CustomPulseAnimation: BaseViewAnimator() {
    override fun prepare(target: View?) {
        animatorAgent.playTogether(
                ObjectAnimator.ofFloat(target, "scaleY", 1f, 1.05f, 1f),
                ObjectAnimator.ofFloat(target, "scaleX", 1f, 1.05f, 1f)
        )
    }
}

And then use as normal:
 mYoYoShake = YoYo.with(CustomPulseAnimation())
                .duration(1000)
                .repeat(0)
                .pivot(YoYo.CENTER_PIVOT, YoYo.CENTER_PIVOT)
                .playOn(this)


Witch Bolt range error

When casting a witch bolt, the range is 12. When casting as an action without consuming a spell slot, the range is 6.
Additionally, simply moving around without taking action breaks your mental focus.

Both ranges are correct at 30ft. If you or enemy are more than a given distance apart you will lose concentration. Only issue I found was bonus action removing concentration. Fixed.


go get generates 1 error

$go get github.com/grokify/gotilla... results in:
# github.com/grokify/gmp
../github.com/grokify/gmp/int.go:14:10: fatal error: 'gmp.h' file not found
#include <gmp.h>
         ^~~~~~~
1 error generated.

This is likely occurring because you don't have GMP installed.
https://gmplib.org/
I'm working on removing this dependency.

Try it now. I replaced the GMP dependency with a pure Go implementation. There's no longer a dependency on GMP.

Closing now. Please post if this is still an issue.

Your fix worked @grokify - thanks a lot!


Use caching to speed up bhmie algorithm

Instead of calculating the bessel function terms each time, use caching to store values so there are fewer repeated calculations.
Input terms: a (grain size), E (photon energy), n (index) -- see Van de Hulst or Bohren & Huffman
Compute S1 and S2 matrices of dimension a, E, n.  Add along n-axis any time a new term is needed.

See branch speed_up_mie


BUG: CACAO to ETH not available

Describe the bug
Swap route from CACAO to ETH or reverse not possible
Expected behavior
please verify and also verify ALL other swap routes
-v17

no available route to do that

@yevhen1sec could you add this route?

I'm not sure that any swap providers support this route rn, need to check

@yevhen1sec MAYA does.

QA done.


Add mysql config options

As per our discussion from Friday, this adds read and write timeouts as well as a "catch-all" field.

Coverage remained the same at 41.191% when pulling 8a507439a5a08d25fc512090ab70701690055e3d on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

Coverage remained the same at 41.191% when pulling 8a507439a5a08d25fc512090ab70701690055e3d on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

Coverage remained the same at 41.191% when pulling 5935388d02e02e825aba56c780029d868ae1d78e on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

Coverage remained the same at 41.191% when pulling f9644da8890d4180622d95c1faafe0c5e7187f30 on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

Coverage remained the same at 41.191% when pulling f9644da8890d4180622d95c1faafe0c5e7187f30 on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

updated

That if/else still looks a lil wonky with all those string checks. Could we do something like this to simplify?
	if !strings.HasPrefix(addtl, "&") && addtl != "" {
		addtl = "&" + addtl
	}
	if read != "" {
		addtl += fmt.Sprintf("&readTimeout=%s", read)
	}
	if write != "" {
		addtl += fmt.Sprintf("&writeTimeout=%s", write)
	}

example: https://play.golang.org/p/hRyU0EQ1oi

The other option is to just use url.Values and let it do all this string work for us. https://golang.org/pkg/net/url/#Values

Now that's a good idea - let me use that.

Coverage remained the same at 41.191% when pulling d878c3dabe82c198cb5ce297773bc2b42e3818e7 on oliver-nyt:oh_add_more_options_to_mysql_config into 3adc58798e01035e650fd418d29ed22e572d2427 on NYTimes:master.

🎈 thanks!


 I would like to request the following reviewer:
(R: @lostluck )
If accepted, please wait until the *-remote.yaml files are committed from updating the Stepik course before finally merging this PR.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Dataflow
Flink
Samza
Spark
Twister2

Go

---

---

---

Java

Python

---

---

XLang

---

---

---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@lostluck The *-remote.yaml files are now committed from updating the Stepik course and this PR is ready to merge.


python37Packages.Rtree: 0.9.4 -> 0.9.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.Rtree is: "R-Tree spatial index for Python GIS"
meta.homepage for python37Packages.Rtree is: "https://toblerity.org/rtree/"
meta.changelog for python37Packages.Rtree is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.9.7 with grep in /nix/store/fyzz6cmwh7j7npxik1dkqyss4jb3s4ni-python3.7-Rtree-0.9.7

directory tree listing: https://gist.github.com/c177fa82e4d24cb78e0906c47e5562e7

du listing: https://gist.github.com/481f24a29c94ab496c9064ab175e3bd3

Rebuild report (if merged into master) (click to expand)

30 total rebuild path(s)

10 package rebuild(s)

8 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
6 x86_64-darwin rebuild(s)
10 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
flatcam
python37Packages.Rtree
python37Packages.geopandas
python37Packages.osmnx
python38Packages.Rtree
python38Packages.geopandas
python38Packages.osmnx
python39Packages.Rtree
python39Packages.geopandas
python39Packages.osmnx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fyzz6cmwh7j7npxik1dkqyss4jb3s4ni-python3.7-Rtree-0.9.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.Rtree https://github.com/r-ryantm/nixpkgs/archive/26071aaa442b908bcf9892d34add23d8d9bf8edd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fyzz6cmwh7j7npxik1dkqyss4jb3s4ni-python3.7-Rtree-0.9.7
ls -la /nix/store/fyzz6cmwh7j7npxik1dkqyss4jb3s4ni-python3.7-Rtree-0.9.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages failed to build:

flatcam
python37Packages.geopandas
python37Packages.osmnx
python38Packages.geopandas
python38Packages.osmnx

3 packages built:

python37Packages.Rtree
python38Packages.Rtree
python39Packages.Rtree

Maintainer pings
cc @bgamari for testing.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109677 run on x86_64-linux 1

5 packages failed to build and are new build failures:

flatcam: log https://termbin.com/x8gd
python37Packages.geopandas: log https://termbin.com/yh7w
python37Packages.osmnx: log was empty
python38Packages.geopandas: log https://termbin.com/p4x0
python38Packages.osmnx: log was empty

3 packages built:

python37Packages.Rtree
python38Packages.Rtree
python39Packages.Rtree

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109677 run on x86_64-linux 1

5 packages failed to build and are new build failures:

flatcam: log https://termbin.com/x8gd
python37Packages.geopandas: log https://termbin.com/yh7w
python37Packages.osmnx: log was empty
python38Packages.geopandas: log https://termbin.com/p4x0
python38Packages.osmnx: log was empty

3 packages built:

python37Packages.Rtree
python38Packages.Rtree
python39Packages.Rtree

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109677 run on x86_64-darwin 1

1 package marked as broken and skipped:

flatcam

2 packages failed to build and are new build failures:

python37Packages.osmnx: log was empty
python38Packages.osmnx: log was empty

3 packages built:

python37Packages.Rtree
python38Packages.Rtree
python39Packages.Rtree

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109677 run on x86_64-darwin 1

1 package marked as broken and skipped:

flatcam

2 packages failed to build and are new build failures:

python37Packages.osmnx: log was empty
python38Packages.osmnx: log was empty

3 packages built:

python37Packages.Rtree
python38Packages.Rtree
python39Packages.Rtree

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.Rtree:

Please consider this feature to be alpha.
A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'find_library('spatialindex_c')' doesn't match anything in file 'rtree/core.py'

Please check the offending substituteInPlace for typos or changes in source.


Adding link to accessing keys from WDF drivers

Adding super helpful link to accessing keys from WDF drivers

Thanks for the link, Shyamal.  This should be live in a couple minutes.


Compactor ignoring excluded folder

I have the folder path %APPDATA%\Hamsket added to the exclusion list, but as you can see in the pictures Compactor is still compressing the files in said folder.

Compactor doesn't expand placeholders, so you'll have to be more explicit, e.g. *\AppData\Roaming\Hamsket.
I really don't recommend compacting your user profile - that's not what it's designed for.  It's made for apps and their immutable data files, not important live data with complex locking needs.

Fair enough, thanks for the response and information :)


When should call registerForRemoteNotifications

https://developer.apple.com/library/mac/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/Chapters/IPhoneOSClientImp.html

In addition, iOS calls your delegate method any time the device token changes, not just in response to your app registering or re-registering

http://www.joshwright.com/tips/ios-8-push-notification-registration

This has nothing to do with user permissions. You are having the device get a push notification token from Apple's servers. If the user didn't give you permission to show badges, show alerts, or show sounds then you will probably still get a token. In that case, all you can do with it is send invisible notifications directly to the app.


Updating OneNote HTML import with dev clarification

Adding engineering clarification from #499.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/onenote/onenote-add-ins-page-content.md
:white_check_mark:Succeeded

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/onenote/onenote-add-ins-page-content.md
:white_check_mark:Succeeded

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.


Performance problems on large repos

When I try to open the summary on a test file in a repo at work, nvim freezes for a very long time.
The issue is this line:
https://github.com/rcarriga/neotest/blob/aaf2107a0d639935032d60c49a5bdda2de26b4d6/lua/neotest/client/init.lua#L408
The find_files actually completes within ~2 seconds, but then running adapter.is_test_file 250k+ times takes several minutes (vim-test adapter). After that freeze, there is a second freeze while it tries to write all the events to the logfile (hasn't finished yet).
My personal opinion, test discovery should be definitely optional, and possibly configurable. My previous job had a monorepo that was so big it was only served as a virtual filesystem, so running any type of crawling operation would have terrible side effects. Making it configurable (e.g. only these dirs, only this depth) might be nice, but would probably make more sense per-adapter instead of globally. If adapters need to control the search path, their API would have to change from testing individual files that neotest gives them to calling back into neotest with whatever their search config is. IDK if that refactor is worth it, which is why I could go either way on making the search configurable.
Another direction could be to customize the search on a per-directory basis instead of per-adapter. That avoids the need for an adapter API refactor, and in general this kind of functionality would be incredibly useful. I often work with many different types of repos on the same machine, sometimes in the same vim instance (using :tcd and one tab per project), and these projects will sometimes use the same language but require different configurations to run tests. I'd love to be able to configure my test adapters on a per-project basis. A rough proposal, it could look like:
require('neotest').setup({
  adapters = ...,
  summary = ...,
  discovery = {
    enabled = true,
  },
  projects = {
    ["~/work/big"] = {
      adapters = ...,
      discovery = {
        enabled = true,
        dirs = {'tests/unit', 'frontend/tests'},
        depth = 2
      },
    },
    ["~/personal/project"] = {
      adapters = ...,
      discovery = {
        enabled = false,
      },
    },
  },
})

I am happy to submit a PR for any parts of this once we align on a solution
Unrelated question: I'll probably be making more proposals, requests, and questions. Is filing an issue the best way to start a discussion, or would you prefer some other channel?

the adapters I've written (and I imagine others) would need to be refactored to handle configuring multiple instances

Oh this would be excellent! That might solve another issue I've been having with multiple directories open in the same vim. If both dirs use the same test adapter, I see "Common root not found" errors. But if each dir were using a different adapter instance that might be one way around it.

I think the project/discovery configuration would solve this issue better than having adapters controlling the search as it solves other issues too so for now I'll say let's go with that. This will be a rather large change so it'll take some time to work on.

Sounds good! LMK if I can help at any point.

I see "Common root not found" errors

As far as I remember this is actually a bit of laziness on my part because I don't open multiple projects in one NeoVim instance. The trees just need to be merged inside of a new parent here https://github.com/rcarriga/neotest/blob/master/lua/neotest/lib/positions/init.lua#L84. The way the new file detection works, it shouldn't (could be wrong) scan the new parent directories.
However I do prefer the idea of having separately maintained trees for each project as it'll present a lot cleaner IMO

Finally got around to implementing per-project configs :sweat_smile: I've got it in https://github.com/nvim-neotest/neotest/pull/100 just for testing but it seems to work nicely AFAICT. If anyone runs in to issues please let me know!

Awesome!! I'll test it out when I get a chance and leave feedback

I'm also having lags on bigger repos. This is especially annoying during BufAdd and BufEnter events, disabling the test dicovery (https://github.com/nvim-neotest/neotest/pull/30) in config doesn't solve the issue. In the end I've disabled autocmds for NeotestClient to make it functional:
vim.api.nvim_create_autocmd({ "BufEnter" }, {
	callback = function()
		pcall(function()
			vim.api.nvim_clear_autocmds({ group = "NeotestClient" })
		end)
	end,
})

I've tried using https://github.com/nvim-neotest/neotest/pull/100, but it doesn't solve the issue either. As it doesn't really affect a single directory usage.

Is it possible to just use current directory where nvim is started without complicated logic? (similar to how vim-test does it) I guess config.discovery.enabled should take care of it, but it looks like it doesn't work well.

Separate to the project config work, I've done some benchmarking and found an easy optimisation within the treesitter internal code. I've implemented it in the the latest master and I'm seeing a 30% reduction in time to parse the cpython repo :grin: It should hopefully improve performance for most languages. This won't solve all performance issues but it will hopefully improve things a bit.
I'm also planning on creating a consumer that can run benchmarks easily so others can see what is affecting performance

OK I've added two new config options to help with discovery performance.

Concurrent discovery control: You can now specify the number of concurrent workers parsing files. The default is CPU cores + 4 (this is a bit random, just went with what Python's ThreadPoolExecutor defaults to). Set this to 1 and it should reduce CPU usage considerably but it will also slow down test discovery.
Directory filtering: The proposed dirs option would probably suit most people but many projects put tests alongside corresponding code files so I thought this was a bit inflexible. Instead there's now a filter_dir option that can filter what directories to search for test files.

In terms of user experience, directory filtering is preferred over limiting concurrency which is preferred over disabling discovery altogether. I believe that these features are sufficient to tackle nearly all performance related issues. Would love to hear feedback on these! :smile:
If you are still experiencing performance issues and you've disabled concurrency and set up filtering to at least rule out scanning directories like node_modules or venv please let me know but also be sure to rule out other plugins. I rarely experience significant slowdown due to neotest, it's almost always LSP (wish there was a better Python LSP :sweat_smile:)

I tried adding the filter_dir option, but it looks like it isn't being called. My project is set up like this:
require("neotest").setup_project(vim.loop.cwd(), {
  adapters = {
    require("neotest-elixir")({ extra_formatters = { "ExUnit.CLIFormatter", "ExUnitNotifier" } }),
  },
  discovery = {
    enabled = true,
    filter_dir = function(name, rel_path, root)
      print(name, rel_path, root)
      return false
    end,
  },
})

I see no messages and no directory is filtered.
Also, in Elixir all tests belong in a test/ folder, it isn't possible to put the test file alongside the source file, and because of the recent change in the find function, basically every project has to add the filter_dir option as Neotest is now detecting test files in the  deps directory.
Would be possible to for an adapter to set a default filter_dir function?

Would be possible to for an adapter to set a default filter_dir function?

Yep definitely think this is needed :+1: Adapters can now define a filter_dir function with the same signature

I tried adding the filter_dir option, but it looks like it isn't being called.

Everything looks correct. Can you enable debug logging and submit a separate issue?

No need, I have the setup_project call in .nvimrc and the Neotest config in after/plugin/neotest, so the configuration was being overwritten.
Works great, thank you.

I've not heard further reports on performance issues so I'm not sure how prevalent they are anymore but I wanted to have a little fun with using Neovim's RPC API. I've implemented the treesitter parsing to be run in a separate Neovim instance so that the only cost on the main editor instance is deserialising results. It's sitting in a PR right now https://github.com/nvim-neotest/neotest/pull/119, it'd be great to get some feedback on it.
It should never expose any errors directly to the user, instead logging errors and falling back to the current in-process parsing, so when testing please check the logs (I'd recommend the INFO level). The main issue I'm anticipating is lazy loading, I've added a call for require("neotest") to the subprocess so if you do lazy load, just add module = "neotest" to the config for packer (or equivalent for other managers).
Also some adapters (neotest-elixir, neotest-go, neotest-dotnet, neotest-rspec, neotest-phpunit) use callback functions for the parsing which can't be done over RPC. The solution here is to provide a string to evaluate to a function in the child process (e.g. require("my-adapter")._build_position so if adapter authors or people willing to make the change wish to test, it should be relatively simple.
If testing goes well, it might be possible to expose more helpers to run CPU intensive in the subprocess

I made the change, and it seems to be working well, although I didn't notice any change in the time it takes to discover.
What I did notice is that discovery is triggered twice.
When I open the summary, it outputs:
Searching <project folder> for test files
Discovering files with 20 workers

And then prints a bunch of Parsing <file> lines.
Then, I can see the Searching and Discovering lines again, and the Parsing line for every file.
Any file I searched in the log appeared twice, and I removed the log between runs.
Is there a reason for that?

Thanks for testing it out!
I don't think that'd be related to these changes as there's no change in the discovery logic, this is purely in the treesitter library functions. Are the second log lines from the parent or child instance (has CHILD in the line)? Also are you doing anything or just letting discovery run and then exiting?

Yeah, I didn't think it was related to these changes, but it is related to performance.

Are the second log lines from the parent or child instance (has CHILD in the line)?
They are all from the parent instance.

Also are you doing anything or just letting discovery run and then exiting?
I would either open the summary and check the logs, or run a test file and check the logs.

I forgot to mention that the lines Initialising client and Initialisation finished in X seconds would only print once, before the first batch of logs and after the second batch of logs, respectively.

Sure. One more thing I noticed is that the duplicate discovering only happens when I open the summary while having a test file loaded in a buffer. If I don't have a test file open, it only prints each file once.
Here is a repo I use for testing. It has very few test files, so the log is not long.
parquex_info.log
parquex_debug.log
In the log at INFO level it is easier to notice the duplicates, but the DEBUG one provides more info, so I included both.

Ah thanks!

duplicate discovering only happens when I open the summary while having a test file loaded in a buffer

This was the key :sweat_smile: It was an existing issue that has been fixed in master

Great! The difference is very noticeable in my work repo.

Great to hear! I'm going to merge the PR so feel free to make the change to neotest-elixir, I'll get around to submitting PRs to other adapters soon as well.

I'm going to close this issue as all of the originally requested features/changes have been made or have alternatives in place. If there are still performance issues, then we can address them specifically in separate issues.

Thanks for the pull requests on the adapters. It definitely feels faster. ❤

Fantastic, love to get the feedback :grin:


Environment setup and build process

We need to setup the environment for our graphql server. I suggest we use Apollo Tools with express. We also need testing, JEST would be cool. I would like to incorporate a BDD solution, cucumber/gherkin.
I would also like to deploy the graph sever to a staging environment like heroku or something so we can run queries and mutations with graphiql.
Finally, it’s best to setup Travis or Circle CI at the very beginning of this process so we can have a build and certification process in place for pull requests.

No strong opinions on all of that.  Whatever you think
I'd like to see some pre-commit hooks to run the tests, enforce code coverage, lint and run prettier

And of course... Axe... Accessibility is the goal.  I always use Travis, I wonder if it would be worth it to checkout circle.ci.

Alright, after some review, we're going with Graphcool. I am going to get that code setup and incorporate the news schema that was built by @gwmccull.
I found the tutorial for React&Apollo and Graphcool influenced this decision quite a bit. Working with Graphcool is fun!
The videos are not updated, but the text below each lesson has been updated to Apollo 2.0 and the new graphcool cli.


Move configuration to a JSON file

At this time there are no configurations, just a file named php_path that holds the path to the php binary.
Let's move this to a JSON file.

Done.


#151 - Fix a bug where connecting to watch from pair page ...

... wouldn't be remembered and app would show setup screen at next launch.
Note to reviewers:

Is there any other screen where user's can connect to watch?
Is there better file where I can put key of saved preference? I decided on existing preferences.dart but some other file might be better.

You could add preference key into that file: https://github.com/pebble-dev/mobile-app/blob/master/lib/infrastructure/datasources/preferences.dart
Although it we might want to split it up because this file is getting pretty big


Update Typescript Expand Levels keys and values to match latest form mm-plugin-apps

Summary
This PR corrects the expand_levels for typescript.
Added two new fields to match the existing fields in mattermost-plugin-apps/server/api/expand.go.
Change values to lowercase
Ticket Link
N/A

I also reflected this changes on mobile.

I also reflected this changes on mobile.


[QUERY] Controlling the "type" name of an OTEL activity in Application Insights

Library name and version
Azure.Monitor.OpenTelemetry.AspNetCore 1.2.0
Query/Question
I am trying to instrument a library so it adds dependency tracking when used in the application.
Looking at how the HttpClient is instrumented, I can see that it shows up in App Insights like this:
HTTP: GET httpstat.us/200
Looking at the underlying item in the log, it has properties like this:
itemType: dependency
name: GET /200
target: httpstat.us
type: HTTP
data: https://httpstat.us/200

How can I control these properties when using the .StartActivity() method?
By default, I get something that looks like this:
itemType: dependency
name: MyActivityName
target: MyActivityName
type: InProc

Can I change the InProc type to something else?
Can I set different values in name and target?
Can I add information to the data field?

Environment
.NET SDK:
Version:           8.0.300
Commit:            326f6e68b2
Workload version:  8.0.300-manifests.9e3391ed
MSBuild version:   17.10.4+10fbfbf2e
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22631
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.300\
.NET workloads installed:
[aspire]
Installation Source: VS 17.10.34916.146
Manifest Version:    8.0.0/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.aspire\8.0.0\WorkloadManifest.json
Install Type:              Msi
Host:
Version:      8.0.6
Architecture: x64
Commit:       3b8b000a0e
.NET SDKs installed:
6.0.423 [C:\Program Files\dotnet\sdk]
7.0.410 [C:\Program Files\dotnet\sdk]
8.0.106 [C:\Program Files\dotnet\sdk]
8.0.300 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.31 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.31 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.30 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.31 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.19 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.6 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Mapping of Span kind, etc. is undocumented :( Fix should be in the docs page and would apply to all languages.

Hi @mortenbock, I'm not completely following what you're trying to accomplish
Are you trying to emit custom Dependency Telemetry from your app?
Or is this a library that's emitting Activities and your hosting application is using OTel?
If you please could share a minimal example app of your approach that would help me understand.

@TimothyMothra I have a couple of scenarios I want to achieve.
The first is an application that has some background tasks running for some integration jobs on a schedule. I want to create a root activity when these tasks are started. Using the classic Insights SDK, I did this by making a DependencyTelemetry operation, and setting the Type property on that.
What I've got now in the Otel version is this:
using (var jobActivity = OtelConfiguration.ActivitySource.StartActivity(
           name: "ProductImport",
           kind: ActivityKind.Internal,
           parentId: null))
{
    // Run some integration work
    await DoSomeWork();
}

This shows up in Application Insights as a "InProc" type, but I would like to control that type, so I can differentiate between different types of dependencies.
The second scenario is a more typical wrapper library for an API, where the standard http dependency telemetry doesn't quite make sense when analyzing it, so I want to wrap the operations in a custom activity that makes the trace more readable/useful.
In that case the it might make sense to set the "Type" to "HTTP", or it might be even better to set it to identify the external api like "AchmeApi".
I hope this clarifies the usecases I'm looking at?

Sorry for the delay, thanks for the extra information. Unfortunately this field is not customizable today. We've mapped it to specific types assuming dependencies would be created from Instrumentation libraries.
I changed this title to a Feature Req, we're tracking which outstanding fields users are asking for. As of today I can't give an estimate on when this will be available.
cc: @mattmccleary

I'm consolidating all issues regarding overriding specific fields in the Application Insights schema into another issue to help with tracking. https://github.com/Azure/azure-sdk-for-net/issues/46021


txr: 273 -> 274

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/txr/versions.
meta.description for txr is: "Programming language for convenient data munging"
meta.homepage for txr is: "http://nongnu.org/txr"
meta.changelog for txr is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274/bin/txr had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 274 with grep in /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274
found 274 in filename of file in /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
txr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A txr https://github.com/r-ryantm/nixpkgs/archive/845b882d668e8c58842d7c9ef15a133e168bb89b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274
ls -la /nix/store/r6k8vi51hka0bp69h7mhj2fflqs6grfg-txr-274/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

txr

Maintainer pings
cc @dtzWill for testing.

Hmm, new segfaults when built against musl? Not a blocker I suppose but would be good to investigate...


Adding optional prefix to Redis (async and storage)

This adds a prefix to the Storage Redis code
Prefix is optional and is pre-pended
This can be used to enable multiple OSBA instances to share a Redis
without having data collisions.
The default behavior when installing with Helm will be to not provide an empty value, which should result in not preprending a unique identifier.
Since this will be defined in the environment variables for the deployment, restarts should continue working with that identifier and pick up anything queued.

Closes: #545

Hi @jeremyrickard , seems config files for Cloud Foundry are not changed. This feature is also useful for Cloud Foundry. Could you please add REDIS_PREFIX in this file and this file ?

Yep, I haven’t quite finished testing this out, so I haven’t fully updated all the configs and docs. When I remove the WIP tag I’ll give you a shout.

Is there a missing change for the OpenShift template?

@krancour yeah, forgot about the OpenShift template, thanks!


Added tests to k8sclient.go

Some tests are a WIP and are commented out. Pull request is being made mainly to see new coveralls report but all added tests should be functional.

Pull Request Test Coverage Report for Build 837

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.0%) to 73.083%

Totals

Change from base Build 836:
2.0%

Covered Lines:
953

Relevant Lines:
1304

💛  - Coveralls