QA three setup workflows for demo day

Description
Simulate a brand new user to finish following these setup workflows.
First Round

[x] Dev Setup @mindlesscloud
[x] User Setup via docker-compose (high) @klesh
[x] TeamCode setup (high) @Startrekzky

Second Round

[x] Dev Setup as described in readme @kevin-kline
[x] User Setup as described in readme @kevin-kline
[ ] Click here to try TeamCode Setup with Chinese Guide @runjinz
[ ] Click  here to try TeamCode Setup with English Guide @kevin-kline

Related issues:

[x] #745 refers to https://github.com/merico-dev/lake/wiki/Set-Up-Dev-Lake-with-Tin-zh-CN

From my testing both User Setup and Dev Setup work to get the project up and running without error.

Collections can be triggered
Config UI saves to .env / DB

In reference to the Tin setup: https://github.com/merico-dev/lake/wiki/Setting-Up-Dev-Lake-with-Tin
I can confirm this setup works well up to the collection stage:

It is difficult to test because I don't have access to the DB or the backend logs to make sure that collection is happening. It takes time to collect the data. I'm still waiting. I'll follow up when it finishes (hopefully!)

In reference to the Tin setup: https://github.com/merico-dev/lake/wiki/Setting-Up-Dev-Lake-with-Tin
I can confirm this setup works well up to the collection stage:

It is difficult to test because I don't have access to the DB or the backend logs to make sure that collection is happening. Unfortunately we don't have the best updates on progress. I'm also not sure if the frontend would report any error if we have invalid tokens and have errors in the requests. However, it does takes time to collect the data. I'm still waiting. I'll follow up when it finishes (hopefully!)

Never finished. I found the backend logs, and I see an error happening:

Here is an error after running Jira for about 20 minutes:

It also seems like my connection settings get lost, maybe after a certain timeout:

Maybe this explains my problem. I seem to have run out of free time:

In reference to the Tin setup: https://github.com/merico-dev/lake/wiki/How-to-set-Up-Dev-Lake-with-Tin
I can confirm this setup works well up to the collection stage:

It is difficult to test because I don't have access to the DB or the backend logs to make sure that collection is happening. Unfortunately we don't have the best updates on progress. I'm also not sure if the frontend would report any error if we have invalid tokens and have errors in the requests. However, it does takes time to collect the data. I'm still waiting. I'll follow up when it finishes (hopefully!)

Never finished. I found the backend logs, and I see an error happening:

@mindlesscloud can not reproduce this error, so we decide to close this issue for now.


can't copy to /

copy seems to abort if rootfs already exists when the target is /.
mischief@delta /tmp $ mkdir a
mischief@delta /tmp $ acbuild begin
mischief@delta /tmp $ acbuild copy a /
copy: mkdir .acbuild/currentaci/rootfs: file exists

seems --to-dir fixes this, but the UX here could be improved :-)

Yup, --to-dir is what you want.
The reasoning for the bizarre UX is that acbuild lazily fetches dependencies, and as such when you call acbuild copy nginx.conf /etc/nginx, acbuild doesn't know if the /etc/nginx directory exists. For all it knows you want the file /etc/nginx inside the container to be a copy of the file nginx.conf outside of the container. acbuild could fetch the dependencies for an acbuild copy command so it can know if directories exist, but downloading tens to hundreds of megabytes to copy a file into a tarball seems excessive. To make the problem worse, acbuild doesn't currently hold on to dependencies between runs.
One options is that once https://github.com/appc/acbuild/issues/125 is done acbuild could hold on to dependencies between runs, in which case preemptively fetching dependencies will be more reasonable. (I don't have too much faith in the current store logic).
Another option is to remove acbuild copy and replace it with acbuild copy-to-dir and acbuild copy-file or something similar, so the difference is a little more obvious than having it fail on you and needing to dig around for acbuild copy's flags.


cannot find Validation or "check_answers" function

i see
from Validation import check_answers

but cannot find the Validation file?Also even i install validation but also cannot find it.
And the question is,i see these key in Scripts/Textual_prompt.py
each_answer_dict['original_fill_the_blank_annotation'] = fill_the_blank_annotation
    each_answer_dict['chartcot_fill_the_blank'] = ChartCoT_fill_the_blank
    each_answer_dict['chartcot_fill_the_blank_gpt'] = get_gpt_4o_reply(image_url, ChartCoT_fill_the_blank)
    each_answer_dict['roleplay_fill_the_blank'] = Roleplay_fill_the_blank
    each_answer_dict['roleplay_fill_the_blank_gpt'] = get_gpt_4o_reply(image_url, Roleplay_fill_the_blank)
    each_answer_dict['tutorial_fill_the_blank'] = Tutorial_fill_the_blank
    each_answer_dict['tutorial_fill_the_blank_gpt'] = get_gpt_4o_reply(image_url, Tutorial_fill_the_blank)
    each_answer_dict['chartQA_fill_the_blank'] = ChartQ_A_fill_the_blank
    each_answer_dict['chartQA_fill_the_blank_gpt'] = get_gpt_4o_reply(image_url, ChartQ_A_fill_the_blank)
    each_answer_dict['chartrea_fill_the_blank'] = Chart_reasoning_fill_the_blank
    each_answer_dict['chartrea_fill_the_blank_gpt'] = get_gpt_4o_reply(image_url, Chart_reasoning_fill_the_blank)

but also didnt find these key in Evaluation/Textual_Prompt_Evaluation.py。
The final question is i see two cot Prompt,which is
ChartCot = "Let's answer following questions one by one: 1.What type is this chart? 2.What are the labels of x-axis? 3.What are the data labels of each element? 4."
Scatter_ChartCot = "Let's answer following questions one by one: 1.What type is this chart? 2.What are the names of different categories? 3.What is the maximum and minimum value of this scatter plot? 4."

i want to know whether does these two prompt included in the evaluation step?for example, does the evaluation scripts have the step to extract the answer from ChartCot prompt answer?

Sorry about this, it seems that we missed this file. We will upload this file as soon as possible. Thanks for you attention to our project.

We have fixed this bug. It turns out to be using a wrong name. It should be
from Check_Answers import check_answers

instead of
from Validation import check_answers


Explain what account data is

Account data is something that's being referred quite often in the CS API, but not really described more than a part of /sync responses. Newcomers might have troubles to understand how an account data event differs from a state/timeline event because of that.

It's under "client config", but we should probably rename the section.


Table header improvements

[x] Remove sort menu option from the parent params column drop down. Check for metrics. (https://github.com/iterative/vscode-dvc/pull/2583)

[ ] Add hide columns / section there (and in every other menu)
[ ] ESC must work to close context menu
[ ] The context menu in the header cells should appear only on the right click. Currently also works with the left click
[ ] Add filter by to context menus
[ ] Add hover styles to header cells
[ ] The sort arrow in the header cell does need to have BG on hover but needs to have a cursor (pointer)
[ ] Click on the Sort arrow - the context menu appears - if you hover on the arrow again the tooltip will be shown on top of the context menu. Do not show a tooltip when the menu is opened
[ ] The D&D cursor appears when the header block has hovered. Blocks like "Create" have no active space on top. Let's make sure that all the space of the block can be used to D&D

There must be a way to copy file names / section names. Full name and partial name.

Please see: Text selection in the table's header and body cells #2260

There must be a way to copy file names / section names. Full name and partial name.

It will be handled here https://github.com/iterative/vscode-dvc/issues/2260

Some of the todo items were moved to #2241

The context menu in the header cells should appear only on the right click. Currently also works with the left click

The description was updated. The new task was included.

Can “Open to the Side” params, but not training_metrics. Why? If I am not mistaken this should work for any file that can be opened on the side.

Add hide columns / section there (and in every other menu)

This one is deceptively tricky, I implemented a solution, and now I'm adding/fixing some tests. Still, it will need some manual testing during review to catch side-effects.


Create a docker for prod environement for next.js

Is your feature request related to a problem? Please describe.
We're currently using next.js in dev mode in all docker-compose setups. This makes the app slow.
Describe the solution you'd like
Create a prod.Dockerfile in agenta-web folder that uses nextjs for prod, use this dockerfile in the docker-compose.prod.yml

@devgenix This might be interesting for you. Let me know and I can assign it to you

Awesome!

I am going to close this for now, since the issue is done, we are just waiting for #407 to merge the PR
Thanks @devgenix for the great work!


project that uses the ChiConnect payout function anroid-native-kotlin

@uchibeke I want to work on this. Please assign this issue to me.

HI @uchibeke, I can't access the API key.

Please follow the steps in the readme to get an api key @Stephen-Odumirin

I've created an organization and sent it to you. I don't think it has been approved.

@Stephen-Odumirin approved and funded


BUG:io:Skip arr_to_chars call for single code points

When an array of unicode strings is passed to write_char() that has only single code points (i.e. dtype("<U1")), skip the conversion call to arr_to_chars() as it is already a char array. The function arr_to_chars would otherwise increase the number of dimensions resulting in a subsequent exception.
Reference issue
Closes #19659
Additional information
One could insert a test case for it, but it would be more involved since there is no unit test for the file scipy/io/matlab/_mio4.py. Also, since the patched function write_char() is a member function, I guess one has to mock the writer/come up with a test object.
In any case, the reproducing code example stated in the issue could serve as a part of a test case which could be improved by another test for the np.bytes_ datatype.

Re unit test, it would probably suffice to add your example from the original issue to the tests for savemat rather than test this class directly

Hi @jokabrink, would you like to return to this PR and add the unit test?

Ok, I pushed in the small test + rebase, if CI ends up clean I think we can merge.

I think 3 core devs are in agreement for the 2-line bug fix patch, and I added small regression test the CI is happy with.
This seems safe/sensible enough to merge before branching, thanks for your first contribution to SciPy.


Get-PNPTenant does not return value for EnableAIPIntegration

While Set-PNPTenant allows enabling/disabling EnableAIPIntegration, Get-PNPTenant does not return the current value.

@jflieben thanks for raising this. It will be available starting from tomorrow's nightly build as well as the next major version !


Gitter Chatroom to discuss about SHAREthem

Create a gitter chatroom for this repository and link it accordingly.
We can discuss about the model used in SHAREthem and to make things more efficient!

Hey there. Apologies for the delay. Just created a chat room on gitter
https://gitter.im/SHAREthem/Lobby#

Cool ! :smile:


Typo correction in v201.enums.StatusInfoReasonType.value_too_hight -> value_too_high

Fixed in the 612 PR

merged so will now close


Provide buffer size to tools.reader/indexing-push-back-reader

This is necessary on newer versions of c.t.r to prevent "Pushback buffer
overflow"
Fixes #60

I had a failing Travis build which turned out to be due to an outdated .travis.yml. This led me down the rabbit hole trying to get the build to work on various versions of Clojure and/or Java.
Long story short, newer versions of midje or its dependencies don't work on older Clojure or Java versions, whereas older versions don't work on newer Clojure.
In the end I went back to the configuration as it was, test up to Clojure 1.7 and up to Java 8. I'd consider dropping some support though, e.g. only support Java 8 or later and Clojure 1.7 or later.


rcodesign: 0.22.0 -> 0.27.0

Fixes build of rcodesign on staging-next https://github.com/NixOS/nixpkgs/pull/348827.

https://hydra.nixos.org/build/276135762

The verbosity patch has been upstreamed as https://github.com/indygreg/apple-platform-rs/pull/162.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I confirmed that I was able to build haskell.compiler.ghc8107Binary on aarch64-darwin, which depends on rcodesign.


bug: zIndex's are non-sense and conflicts MUI

Hata Tanımı
zIndex values are not really making any sense, and they are broking MUI's popper components.
Popper component is needed for MUI's components with Pop-up UX purpose. They needed to get lower then 100z.
## Hata Ortamı
I noticed this problem while developing feature with MUI's Autocomplete component.  PR: #495
When you add MUI's Autocomplete component the component stays under Header.
Hatayı nasıl aldığınızı adım adım açıklayın

Checkout the PR: #495
Don't forget to add GCloud api-key as env variable. It's needed for feature.
Type something to search box and you'll see it stays under header.
Don't try to manipulate zIndex because you shouldn't bypass MUI's default values like this if you don't really need.

Beklenen Davranış
It should be on top and when I look source code. These are the all zIndex values in whole project:
[9999, 99999, 1000, 998, 999, 1000, 999, 999, 1000, 2, -1, -1, 1, 1, 9999, 999]
I think we needed to get away from this bloated stuff. And manage zIndex properly.
Ekran Görüntüleri

Masaüstü Bilgileri

İşletim Sistemi: [Windows 11]
Tarayıcı [Chrome]
Sürüm [stable@latest]

Akıllı Telefon Bilgileri

Cihaz: [S21]
İşletim Sistemi: [A13]
Tarayıcı [Chrome]
Sürüm [stable@latest]

Ek Bağlam
It's easy to fix problem. If you assign me the issue I can coordinate the zIndexes and order them properly.

Also #502 numbered issue has same problem. That's a quite global problem about MUI and zIndex'es.

@eraygundogmus @usirin request for attention.

Be my guest and fix it.

Be my guest and fix it.

So I fixed it in my fork, waiting for approved status to create PR.

May I ask why it's closed?


Drop support for Django 3.1

Coverage seems decreased maybe some code specific for django 3.1 are not used anymore ?

The codecov percentage is flaky. We're likely to drop it as per #1542

Let me v3.2.4 out the door and I'll merge this.

"I think we could even drop Django 2.2, at least soon."

Not before April 2022 https://www.djangoproject.com/download/#supported-versions

I think we can drop support for Django < 3.2. Here is a comment from Tim about this https://github.com/jazzband/django-hosts/pull/141#discussion_r766116557

No, once Django 4.0 is out, it's time to drop support for all versions older than the most recent LTS. Otherwise, it could require ugly shims to add 4.0 support (which removes a lot of deprecated things). This is outlined in https://www.djangoproject.com/weblog/2015/jun/25/roadmap/.

Thanks for pointing to this explanation. :+1:


Improve file monitoring performance

Closes #238

Codecov Report

Merging #239 (236c864) into main (9b88d8e) will decrease coverage by 0.95%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
- Coverage   69.29%   68.34%   -0.96%
==========================================
  Files          14       14
  Lines        1508     1510       +2
==========================================
- Hits         1045     1032      -13
- Misses        355      372      +17
+ Partials      108      106       -2

Impacted Files
Coverage Δ

eventhandler.go
59.16% <11.11%> (-4.45%)
:arrow_down:

procmon_linux.go
70.37% <100.00%> (-2.02%)
:arrow_down:

Continue to review full report at Codecov.

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


Refactor the doc structures

remove performance-and-scalability folder
add icons for sections
change guide to examples

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rootsongjc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS [rootsongjc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add federated repositories

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Added all supported federated repository types, this is required before adding the necessary Terraform resources in https://github.com/jfrog/terraform-provider-artifactory, which is a much needed feature before we can switch to using federated repositories.

I have read the CLA Document and I hereby sign the CLA

👋🏻 @eyalbe4 any chance for feedback on this PR if you have the time?

@eyalbe4 - Sorry this took so long, life/work got in the way. I've updated the PR with additional changes based on your suggestions but took it a bit further by:

De-duplicated all the params across all repository/package types
De-duplicated the repeated function call to create/update repositories
Filled in missing params across all repository/packages
Added missing package types to all repository types
Re-ordered tests and repositories in alphabetical order
Updated all repository tests

Removed duplication
All local, remote, federated and virtual tests are passing

Included new function to convert a local to a federated repository

Added tests, passing
Added example to readme

Sorry, that's a lot to review and I hope it's all ok. Please let me know what you think.

@eyalbe4 - Also added functions for triggering full federated repository synchronisation, which is important when converting local repositories for the first time.

# github.com/jfrog/jfrog-client-go/artifactory
artifactory/emptymanager.go:387:5: cannot use (*EmptyArtifactoryServicesManager)(nil) (type *EmptyArtifactoryServicesManager) as type ArtifactoryServicesManager in assignment:
	*EmptyArtifactoryServicesManager does not implement ArtifactoryServicesManager (missing ConvertLocalToFederated method)
artifactory/manager.go:55:2: cannot use manager (type *ArtifactoryServicesManagerImp) as type ArtifactoryServicesManager in return argument:
	*ArtifactoryServicesManagerImp does not implement ArtifactoryServicesManager (missing ConvertLocalToFederated method)

Whoops, missed the interface declaration when renaming. Should be better now 😄

Thanks so muc for this awesome contribution @cawilson!
v1.3.0 which includes this enhancement is now released.

Thanks so muc for this awesome contribution @cawilson!
v1.3.0 which includes this enhancement is now released.

Thankyou @eyalbe4 for assisting with this PR and contribution 😄


[Bug]: hardwareConcurrency  is limited to 4

Environments

playwright-go Version: [e.g. v0.4201.1]
Browser: [e.g. firefox]
OS and version: [e.g. macOS / Windows 11/ Ubuntu 22.04]

Bug description

hardwareConcurrency is limited to 4. No relevant settings are seen in the code. It cannot be modified using javascript in the started browser.
https://browserleaks.com/javascript
How can to modify and adjust hardwareConcurrency setting?
To Reproduce
Please provide a mini reproduction rather than just a description. For example:
Additional context
Add any other context about the problem here.

You mean this? It's read-only and depends on environment.

The navigator.hardwareConcurrency read-only property returns the number of logical processors available to run threads on the user's computer.


Stew items from tag instead of config

Removes the soupItemList config option and replaces it with a new stew_item tag, for easier compat with mods.

i agree, soumeh is my brother

i was playing a world with my brother and sister when we noticed that our delicious ice creams werent stackable, this made my sister very disgruntled (she is easily worryable). Thank you for reporting.


docs: improve "explicit version" section in readme

This PR re-works the README section: Explicit version.
The text is simplified, corrected and the heading is renamed using the more neutral "Action version" instead of "Explicit version".

v5 is clarified as being the latest version
Selecting cypress-io/github-action@v5 is using a branch name, not a tag name. (There is no tag v5.)
In addition to referring to GitHub's [tags](https://github.com/cypress-io/github-action/tags list, the releases list and Changelog are also linked

The text is conveniently reviewable under https://github.com/MikeMcC399/github-action/blob/docs/version-selection/README.md#action-version.
Once this section is stable I would submit a follow-on PR to align https://docs.cypress.io/guides/continuous-integration/github-actions#Explicit-Version-Number with the repo README text.

PR set to draft pending merge of https://github.com/cypress-io/github-action/pull/855


Fixing location to point path

currentModule.filename points to the entrypoint instead of the module path.
Fixes #3

Just realized there are actual tests inside, I'll have a proper look and recommit


is this project abandoned?

i think this project has been abandoned, as there is no update is last couple of years. So any one guide me to some another good scrollbar plugin like this?

Try https://github.com/KingSora/OverlayScrollbars its native scrolling with custom design. You cant adjust the scrollsmoothness or momentum by yourself, it will be always like the native scroll behavior. Apart from this the plugin has very good browsersupport and has a nice documentation.

https://github.com/search?o=desc&q=scrollbar&s=stars&type=Repositories


Error while trying to download subtitles

While trying to download subtitles I get the following error
Traceback (most recent call last):
  File "/usr/local/bin/subliminal", line 10, in <module>
    sys.exit(subliminal())
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1257, in invoke
    sub_ctx = cmd.make_context(cmd_name, args, parent=ctx)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 700, in make_context
    self.parse_args(ctx, args)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1048, in parse_args
    value, args = param.handle_parse_result(ctx, opts, args)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1623, in handle_parse_result
    value = self.full_process_value(ctx, value)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1965, in full_process_value
    return Parameter.full_process_value(self, ctx, value)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1589, in full_process_value
    value = self.process_value(ctx, value)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1579, in process_value
    return self.type_cast_value(ctx, value)
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1568, in type_cast_value
    return _convert(value, (self.nargs != 1) + bool(self.multiple))
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1566, in _convert
    return tuple(_convert(x, level - 1) for x in value or ())
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1566, in <genexpr>
    return tuple(_convert(x, level - 1) for x in value or ())
  File "/usr/local/lib/python3.7/dist-packages/click/core.py", line 1565, in _convert
    return self.type(value, self, ctx)
  File "/usr/local/lib/python3.7/dist-packages/click/types.py", line 46, in __call__
    return self.convert(value, param, ctx)
  File "/usr/local/lib/python3.7/dist-packages/subliminal/cli.py", line 173, in convert
    return Language.fromietf(value)
  File "/usr/local/lib/python3.7/dist-packages/babelfish/language.py", line 126, in fromietf
    language = cls(language_subtag)
  File "/usr/local/lib/python3.7/dist-packages/babelfish/language.py", line 82, in __init__
    raise ValueError('%r is not a valid language' % language)
ValueError: '=en' is not a valid language

I tried both pip and pip3 installations and there doesn't seem to be a difference

I've been putting an "=" mark after l, which was wrong.


IPv6 support umbrella issue

This needs to happen!

[ ] Write tests with IPv6
[ ] IPv6 support in all CNI plugins
[ ] IPv6 support in rkt networking

[ ] DNS
[ ] iptables rules

I'm interested in contributing to this. I started by pushing a patch to CNI https://github.com/appc/cni/pull/137

I would like to have IPv6 support as well. On the other hand it's looks AWS has launched IPv6 for all in general --
Now, we should support IPv6 by default just like Docker.

@steveeJ: So, IPv6 isn’t supported at all then (in combination with contained networking)? I see conflicting information, there’s some mention of IPv6 on https://coreos.com/rkt/docs/latest/networking/overview.html.

Still debugging but leaving this here: Vanilla coreos on AWS running kubelet-wrapper seems to try to resolve the given apiserver via ::1. Since I'm not running a nameserver locally this fails.

Was there any more progress on this in terms of ipv6 support in rkt?

ping - we are also interested in running rkt in ipv6 only networks

@telmich Ohai !eof :)
fwiw, I wouldn't expect much to happen with rkt anymore. Better look into cri-o or containerd.

@discordianfish thanks for the pointer - will do!

-1


Fix typos:resourse->resource, explicitely->explicitly

Fix typos:resourse->resource, explicitely->explicitly

/assign @pigmej


AttributeError: 'FPContext' object has no attribute 'zetazero'

Regarding this line in mpmath/functions/functions.py, where does that self.zetazero come from?:
class SpecialFunctions(object):
    ...
    def __init__(self):
        ...
        self.zetazero_memoized = self.memoize(self.zetazero)

https://github.com/mpmath/mpmath/commit/c58dc54a74117d538057ba4dbf50ee78a6baf74f#diff-4a3b83d9c86871fb9764fc1fdc3c1af39e3e7a631144149e8871c7dfce4d82d1R56
I ask because I've hit a way I'm seeing:
  File "sympy/__init__.py", line 22, in <module>
    import mpmath
  File "mpmath/__init__.py", line 9, in <module>
    fp = FPContext()
  File "mpmath/ctx_fp.py", line 19, in __init__
    StandardBaseContext.__init__(ctx)
  File "mpmath/ctx_base.py", line 45, in __init__
    SpecialFunctions.__init__(ctx)
  File "mpmath/functions/functions.py", line 58, in __init__
    self.zetazero_memoized = self.memoize(self.zetazero)
AttributeError: 'FPContext' object has no attribute 'zetazero'

cc: @ariasdereyna

On which version you have seen this?
In the current master:
>>> import mpmath
>>> mpmath.mp.zetazero
<bound method zetazero of <mpmath.ctx_mp.MPContext object at 0x7faac66b6a10>>
>>> mpmath.fp.zetazero
<bound method zetazero of <mpmath.ctx_fp.FPContext object at 0x7faac6855510>>

Ok, please reopen if you know how to reproduce this.
Regarding to your question,

Regarding this line in mpmath/functions/functions.py, where does that self.zetazero come from?

https://github.com/mpmath/mpmath/blob/2990ec6d10b3cb42dce8c11b82f3321d20dcedee/mpmath/functions/zetazeros.py#L341


Fix CommonJS/Browserify support

When using with browserify, we have error like this:
Error: Cannot find module 'ngstorage' from '/Users/.....'

This PR fix this and allows us to write this:
var angular = require('angular');

angular.module('App', [
	require('angular-ui-router'),
	require('ngstorage')
]);

Seems good. Since we don't have a index.js file I assume this shouldn't do any harm? I don't use commonjs/browserify much so I'm not an expert.

index.js don't do any bad thing. this code also appears in other angular module:
https://github.com/angular/bower-angular-animate/blob/master/index.js
https://github.com/angular/bower-angular-cookies/blob/master/index.js

Shouldn't the file factory pattern be changed to UMD instead to actually support CommonJS?
https://github.com/gsklee/ngStorage/blob/master/ngStorage.js#L1
https://github.com/umdjs/umd/blob/master/commonjsStrict.js#L19

I'll look into it and get this merged asap.
I'll also create a new release once I have done that. Might not make it tonight, but tomorrow. Sorry for the delay!

Hi Guys.
Could you check if https://github.com/gsklee/ngStorage/pull/92 would provide the support you need for UMD/CommonJS?


the get&set way of hashmap using xxhash

changed the branch name

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #453   +/-   ##
=======================================
  Coverage        ?   78.02%
=======================================
  Files           ?       63
  Lines           ?    10408
  Branches        ?        0
=======================================
  Hits            ?     8121
  Misses          ?     1946
  Partials        ?      341

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


10.0.25267.1000 not supported

Version 10.0.25267.1000 is not supported on Windows lastet Insider Pro 11 Pro.
Thank you!

Please, attach your RDPConf screenshot and termsrv.dll.

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/135481e783add039a9ffcc033a05bd824709c9ea

Thank you!

Reopened. Not included in official INI by Stas'M :)


Bind problem 2.1

fixes https://github.com/redhat-developer/kestrel-linux-transport/issues/91 for ASP.NET Core 2.1

Thank you!


python310Packages.rapidfuzz: 2.14.0 -> 2.15.0

Description of changes
Diff: https://github.com/maxbachmann/RapidFuzz/compare/refs/tags/v2.14.0...v2.15.0
Changelog: https://github.com/maxbachmann/RapidFuzz/blob/refs/tags/v2.15.0/CHANGELOG.rst

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

12 packages failed to build:
paperwork paperwork.dist python310Packages.paperwork-backend python310Packages.paperwork-backend.dist python310Packages.paperwork-shell python310Packages.paperwork-shell.dist python311Packages.paperw
ork-backend python311Packages.paperwork-backend.dist python311Packages.paperwork-shell python311Packages.paperwork-shell.dist truvari truvari.dist

74 packages built:
adl anime-downloader anime-downloader.dist apostrophe atomic-operator atomic-operator.dist ibus-engines.uniemoji ledger-autosync ledger-autosync.dist paperless-ngx poetry poetry.dist poetryPlugins.po
etry-audit-plugin poetryPlugins.poetry-audit-plugin.dist poetryPlugins.poetry-plugin-up poetryPlugins.poetry-plugin-up.dist puddletag puddletag.dist pynitrokey pynitrokey.dist python310Packages.autof
aiss python310Packages.autofaiss.dist python310Packages.cleo python310Packages.cleo.dist python310Packages.fire python310Packages.fire.dist python310Packages.fuzzywuzzy python310Packages.fuzzywuzzy.d
ist python310Packages.jiwer python310Packages.jiwer.dist python310Packages.levenshtein python310Packages.levenshtein.dist python310Packages.nkdfu python310Packages.nkdfu.dist python310Packages.rapidf
uzz python310Packages.rapidfuzz.dist python310Packages.syncedlyrics python310Packages.syncedlyrics.dist python310Packages.thefuzz python310Packages.thefuzz.dist trytond trytond.dist python310Packages
.videocr python310Packages.videocr.dist python311Packages.autofaiss python311Packages.autofaiss.dist python311Packages.cleo python311Packages.cleo.dist python311Packages.fire python311Packages.fire.d
ist python311Packages.fuzzywuzzy python311Packages.fuzzywuzzy.dist python311Packages.jiwer python311Packages.jiwer.dist python311Packages.levenshtein python311Packages.levenshtein.dist python311Packa
ges.nkdfu python311Packages.nkdfu.dist python311Packages.rapidfuzz python311Packages.rapidfuzz.dist python311Packages.syncedlyrics python311Packages.syncedlyrics.dist python311Packages.thefuzz python
311Packages.thefuzz.dist python311Packages.trytond python311Packages.trytond.dist python311Packages.videocr python311Packages.videocr.dist spotdl spotdl.dist sublime-music sublime-music.dist ulaunche
r ulauncher.dist

Paperwork builds for me.


Feature Request: Scaffolding a database should allow to specify the context name

Currently (beta7) when scaffolding a database the derived DbContext class will pick the database name suffixed with the "Context" word.
For example:
dnx ef dbcontext scaffold "Server=myserver;Database=Foo;user id=me;password=pwd" EntityFramework.SqlServer

Produces a class named FooContext. The command should allow passing the context name as a parameter:
dnx ef dbcontext scaffold "Server=myserver;Database=Foo;user id=me;password=pwd" EntityFramework.SqlServer --context-name MyDbContext

Thanks for the feedback - I've checked in a fix for this with commit 4f133c1. The flag is --context-class-name or -c for short.


Not able to render AutoCodable.swifttemplate

Xcode 11.4.1, not sure where goes wrong, since there is not error message.
Using configuration file at './sourcery.yml'
Scanning sources...
	loadOrParse: 0.0028589963912963867
	resolution took 0.0006489753723144531
	combiningTypes: 0.0008379220962524414
	total: 0.0037360191345214844
Found 2 types.
Loading templates...
Loaded 1 templates.
	Loading took 0.005620002746582031
Generating code...
Rendering template /Users/ryan/Desktop/WhatIsSourcery/Templates/AutoCodable.swifttemplate
/usr/bin/env "xcrun" "swift" "build" "-Xswiftc" "-Onone" "-Xswiftc" "-suppress-warnings" "--disable-sandbox"
error:

Facing the same error here too in Xcode 11.4.1
Loading templates...
Loaded 5 templates.
Loading took 0.04272198677062988
Generating code...
Rendering template /Users/nangadi/coupons-ios/couponsapp/Coupons/Coupons_Framework/Sourcery/Templates/AutoCodable.swifttemplate
/usr/bin/env "xcrun" "swift" "build" "-Xswiftc" "-Onone" "-Xswiftc" "-suppress-warnings" "--disable-sandbox"
error:

Most likely this is caused by the same issue mentioned in https://github.com/krzysztofzablocki/Sourcery/issues/817


NoticeTemplate reporter fails if no scanner archive storage is configured

If no storage is configured to archive the found license files, then no original license texts (and copyright holders) can be obtained by the NoticeTemplate reporter. This currently causes the reporter to fail like
SEVERE: Error executing FreeMarker template
FreeMarker template error:
Java method "org.ossreviewtoolkit.model.licenses.ResolvedLicenseFile.readFile()" threw an exception when invoked on org.ossreviewtoolkit.model.licenses.ResolvedLicenseFile object "ResolvedLicenseFile(provenance=Provenance(downloadTime=2020-09-23T20:16:44.490769Z, sourceArtifact=null, vcsInfo=VcsInfo(type=Git, url=https://github.com/JetBrains/kotlin.git, revision=7fe66d3456a2e51e5bf228497e81b28a6367766c, resolvedRevision=7fe66d3456a2e51e5bf228497e81b28a6367766c, path=), originalVcsInfo=VcsInfo(type=Git, url=https://github.com/JetBrains/kotlin.git, revision=, resolvedRevision=null, path=)), licenses=[], path=license, file=/tmp/ort10253785273173582224archive/license)"; see cause exception in the Java stack trace.

----
FTL stack trace ("~" means nesting-related):
	- Failed at: ${licenseFile.readFile()}  [in template "default.ftl" at line 82, column 1]

but it should probably fail more gracefully.

@mnonnenmacher can you help with this?

If the archiver is null the template should not try to read any license files. My assumption is that there is an archive, but it does not contain the expected files. This happens if the patterns configured for the archiver do not match the patterns used when creating the archive. With "no storage is configured", do you mean the default storage was used?

With "no storage is configured", do you mean the default storage was used?

Yes, exactly. I don't have (or ever had) the val archive: FileArchiverConfiguration ever configured, so it always was null.

This problem would also happen if you run the scanner and reporter on different machines without configuring a remote storage. I think we should not ignore missing license files or just print a warning, as this might go unnoticed. What do you think of adding a reporter option like:

-O NoticeTemplate.ignoreMissingLicenseFiles=true: To not fail if a license file is missing.
or -O NoticeTemplate.includeLicenseFiles=false: To disable the inclusion of license files.

This problem would also happen if you run the scanner and reporter on different machines without configuring a remote storage.

Ah, which probably is the default case when using the stock Jenkinsfile as each step runs on a separate node.

I think we should not ignore missing license files or just print a warning, as this might go unnoticed.

Don't get me wrong, I'm fine with the reporter to fail in this case. I just believe it could do so more gracefully by showing a more user-friendly message with more background information.

What about falling back to the "normalized" / standard license text and printing a warning if no license text is found in an archive?

I just believe it could do so more gracefully by showing a more user-friendly message with more background information.

The output you posted above contains the text "see cause exception in the Java stack trace", but the Java stack trace is missing. Can you add that? Because the actual exception in ResolvedLicenseFile.readFile() is missing, and we can only change that, but not how Freemarker is printing error information.

What about falling back to the "normalized" / standard license text and printing a warning if no license text is found in an archive?

I don't like printing a warning, because you only see it if you read the logs which is something rarely done if a job succeeds. It should be a user choice, if you want the license files to be included, it should fail if the required files are not available. That's why I proposed the two options above. ignoreMissingLicenseFiles could also be called doNotFailOnMissingLicenseFile.
Maybe we should change the default template to be one that does not include license files?

but the Java stack trace is missing. Can you add that?

Here you go:
Java stack trace (for programmers):
----
freemarker.core._TemplateModelException: [... Exception message was already printed; see it above ...]
  at freemarker.ext.beans._MethodUtil.newInvocationTemplateModelException(_MethodUtil.java:291)
  at freemarker.ext.beans._MethodUtil.newInvocationTemplateModelException(_MethodUtil.java:254)
  at freemarker.ext.beans.SimpleMethodModel.exec(SimpleMethodModel.java:78)
  at freemarker.core.MethodCall._eval(MethodCall.java:62)
  at freemarker.core.Expression.eval(Expression.java:101)
  at freemarker.core.DollarVariable.calculateInterpolatedStringOrMarkup(DollarVariable.java:100)
  at freemarker.core.DollarVariable.accept(DollarVariable.java:63)
  at freemarker.core.Environment.visit(Environment.java:370)
  at freemarker.core.IteratorBlock$IterationContext.executedNestedContentForCollOrSeqListing(IteratorBlock.java:321)
  at freemarker.core.IteratorBlock$IterationContext.executeNestedContent(IteratorBlock.java:271)
  at freemarker.core.IteratorBlock$IterationContext.accept(IteratorBlock.java:244)
  at freemarker.core.Environment.visitIteratorBlock(Environment.java:644)
  at freemarker.core.IteratorBlock.acceptWithResult(IteratorBlock.java:108)
  at freemarker.core.IteratorBlock.accept(IteratorBlock.java:94)
  at freemarker.core.Environment.visit(Environment.java:334)
  at freemarker.core.Environment.visit(Environment.java:376)
  at freemarker.core.IteratorBlock$IterationContext.executedNestedContentForCollOrSeqListing(IteratorBlock.java:321)
  at freemarker.core.IteratorBlock$IterationContext.executeNestedContent(IteratorBlock.java:271)
  at freemarker.core.IteratorBlock$IterationContext.accept(IteratorBlock.java:244)
  at freemarker.core.Environment.visitIteratorBlock(Environment.java:644)
  at freemarker.core.IteratorBlock.acceptWithResult(IteratorBlock.java:108)
  at freemarker.core.IteratorBlock.accept(IteratorBlock.java:94)
  at freemarker.core.Environment.visit(Environment.java:334)
  at freemarker.core.Environment.visit(Environment.java:340)
  at freemarker.core.Environment.process(Environment.java:313)
  at freemarker.template.Template.process(Template.java:383)
  at org.ossreviewtoolkit.reporter.reporters.NoticeTemplateReporter.generateReport(NoticeTemplateReporter.kt:113)
  at org.ossreviewtoolkit.commands.ReporterCommand.run(ReporterCommand.kt:257)
  at com.github.ajalt.clikt.parsers.Parser.parse(Parser.kt:171)
  at com.github.ajalt.clikt.parsers.Parser.parse(Parser.kt:180)
  at com.github.ajalt.clikt.parsers.Parser.parse(Parser.kt:16)
  at com.github.ajalt.clikt.core.CliktCommand.parse(CliktCommand.kt:395)
  at com.github.ajalt.clikt.core.CliktCommand.parse$default(CliktCommand.kt:392)
  at com.github.ajalt.clikt.core.CliktCommand.main(CliktCommand.kt:410)
  at com.github.ajalt.clikt.core.CliktCommand.main(CliktCommand.kt:435)
  at org.ossreviewtoolkit.OrtMainKt.main(OrtMain.kt:194)
Caused by: java.io.FileNotFoundException: /tmp/ort15844618501012269741archive/license (Is a directory)
  at java.base/java.io.FileInputStream.open0(Native Method)
  at java.base/java.io.FileInputStream.open(Unknown Source)
  at java.base/java.io.FileInputStream.<init>(Unknown Source)
  at kotlin.io.FilesKt__FileReadWriteKt.readText(FileReadWrite.kt:125)
  at kotlin.io.FilesKt__FileReadWriteKt.readText$default(FileReadWrite.kt:125)
  at org.ossreviewtoolkit.model.licenses.ResolvedLicenseFile.readFile(ResolvedLicenseFileInfo.kt:81)
  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
  at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
  at java.base/java.lang.reflect.Method.invoke(Unknown Source)
  at freemarker.ext.beans.BeansWrapper.invokeMethod(BeansWrapper.java:1552)
  at freemarker.ext.beans.SimpleMethodModel.exec(SimpleMethodModel.java:73)
  ... 33 more

Doesn't the path "/tmp/ort15844618501012269741archive/license" also look odd as there is no "/" before "archive"?

That's why I proposed the two options above.

Maybe better fallBackToStandardLicenseTexts?

Maybe we should change the default template to be one that does not include license files?

I don't think that's necessary; it was the main point for the new NOTICE generator to use the literal / unmodified license texts. User's who don't want / need this could use the summary NOTICE file, which does not require the archive, correct?

I cannot reproduce this anymore.


Control + Command swapped when Platform is MacOS

What are you trying to achieve? (Expected behavior)
When I use the WebDriverKey::CONTROL on a Mac, using Firefox and Chrome, the CONTROL key and COMMAND key are swapped around.
Arguably this is correct because on a Mac the COMMAND hey performs as the CONTROL key on Linux/Windows (i.e. they actually are swapped around), but I feel that this should be handled within the driver.
What do you get instead? (Actual behavior)
Instead of CONTROL I get the action of COMMAND
How could the issue be reproduced? (Steps to reproduce)
Unit test.
Note: the WebDriverTestCase prevents you from running Firefox via Selenium properly. The skipForJsonWireProtocol function skips unless GECKODRIVER=1 hence the relevant line being commented out.
<?php
// Copyright 2004-present Facebook. All Rights Reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//     http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

namespace Facebook\WebDriver;

use Facebook\WebDriver\Remote\DriverCommand;
use Facebook\WebDriver\WebDriverKeys;

/**
 * @coversDefaultClass \Facebook\WebDriver\Remote\RemoteKeyboard
 */
class RemoteKeyboardTest extends WebDriverTestCase
{
    /**
     * @covers ::getTitle
     */
    public function testShouldGetPageTitle()
    {
        //self::skipForJsonWireProtocol();

        $this->driver->get($this->getTestPageUrl('form.html'));

        $input = $this->driver->findElement(WebDriverBy::id('input-text'));
        $input->click();

        $this->driver->execute(DriverCommand::ACTIONS, [
            'actions' => [
                [
                    'type' => 'key',
                    'id' => 'keyboard',
                    'actions' => [
                        ['type' => 'keyDown', 'value' => WebDriverKeys::COMMAND],
                        ['type' => 'keyDown', 'value' => 'a'],
                        ['type' => 'keyUp', 'value' => 'a'],
                        ['type' => 'keyUp', 'value' => WebDriverKeys::COMMAND],
                        ['type' => 'keyDown', 'value' => WebDriverKeys::BACKSPACE],
                        ['type' => 'keyUp', 'value' => WebDriverKeys::BACKSPACE],
                    ],
                ],
            ],
        ]);

        readline('Check');
        $this->assertEquals(
            '',
            $input->getAttribute('value')
        );
    }
}

Details

Php-webdriver version: community
PHP version: 7.3.11
Selenium server version: 3.141.59
Operating system: MacOS + Docker Linux
Browser used + version: Firefox 70.0.1, Chrome 78.0.3904.108

So what do you propose?
I don't think its role of this library (which intend to be low-level and unviversal abstraction of the protocol) to eg. send to the remote end different key code than you pass to the library.

Well arguably the browser is doing the right thing - control and command are indeed swapped around on a Mac, and so the W3C remote endpoint is doing the ‘correct’ thing by typing the exact requested key.
However should we expect clients of this driver to know about the distinction and address it themselves? I suspect that will catch a lot of people out and they’ll have to find a way to ‘fix’ it, which will mean using the platformName.
To be honest I don’t know the best solution here. I think that this is really something to report to the W3C specification as being ambiguous and needing clarification.
A couple of thoughts not how we could approach things:

If it were clarified in the spec this would be easier to decide upon
It would be great if browsers had normalizeKeys feature flag to allow us/clients to make that decision and keep it standard throughout use of the driver
We could implement such a feature flag locally (and allow that to become a browser flag in future possibly)
We could do nothing but raise awareness to developers for when they encounter this

If we go for the final option then we possibly need to fix the clear function.
I’ve had a look at the spec, and when it comes to modifier keys it references https://www.w3.org/TR/webdriver2/ which does explicitly mention the META key as being the Apple Command key or windows key. Therefore I’d argue that browsers are doing the right thing, as is the spec, and as is this driver.
However... the clear function currently uses control+a to empty the string so we would need to ‘fixed’ that use of control as a minimum.
I discovered this issue because the clear function does not perform reliably for me in all situations and I haven’t yet worked out why. It calls clear, and then uses a control+a backspace hack to remove all content.
Unit tests for this library pass, and the ones for my Mink driver also pass, but my application regularly, reliably, and repeatedly only deletes a single character from the end of the current input value, and regularly deletes input from the previous input field. So far my only solution has been to have my driver fetch the current char count and then hit the backspace key that many times.
From reading https://github.com/w3c/webdriver/issues/445 it seems that the clear endpoint should remove the text and then blur the input. In that instance we no longer have focus for the keyboard actions that are then called here: https://github.com/facebook/php-webdriver/blob/community/lib/Remote/RemoteWebElement.php (also the release order is wrong there as control should be released after releasing a).
As to why I’m not seeing clear actually empty the input, well that’s still a bit of a mystery to me and one I’ll keep looking into.

@andrewnicols Can't you have a look if there is any other language binding for WebDriver which does solve this somehow? Or how do others handle clear() in W3C?
Or maybe ask on WebDriver IRC about their opinion? Or maybe raise an issue in WebDriver repo?
But for now, I'm not convinced this should be generally handled by php-webdriver... Maybe just the clear function should be working platform independent. But to generally swap ctrl/command internally according to platform and hide it from the users is IMHO not a good idea.

During investigation of #704, I was just thinking - any idea, why do we need to apply those keybaord actions as part of clear() in W3C mode? From what I tried, removing this whole part didn't break anything in the testsuite 🤔.

@andrewnicols Well, what do you think about this after #707 was merged? At least it now don't break the clear function :).
From my point of view this is expected behavior - each platform could have different keyboard shortcuts, so its up to the user or some other library providing some abstraction layer to take care of this platform differences.  And php-webdriver aims to be primarily low-level abstraction layer for WebDriver protocols, so doing this stuff is IMO out of scope. Or at least I don't see a way how to take care of this and don't break other stuff.

Agree. And it’s no long we such a problem after fixing the clear command.
I had meant to close this issue today but spent the day in meetings instead.
I did raise an issue against the spec to help clarify this but the suggestion wastonuse the META key instead.
+1 to close.


Improve Load Balancers

[ ] Add rate-limiting
[ ] Add custom rules (firewalls)
[ ] Add option to automatically check incoming IPs in SPAM Databases (eg. spamhaus.org) and prevent them from accessing resources
[ ] Make possible to make requests to Load Balancer IP (or domain) inside of the cluster
[ ] Document, that Health Check endpoint should support PROXY protocol in order to work with LB, when the PROXY protocol support is activated
[ ] Sync LB settings from dashboard to cluster and the other way around. Or just disable any web-based editing of the LoadBalancer, if it's managed from Kubernetes
[ ] Provide more documentation and technical details

Two things I'd like to add:

the ability to re-use an existing loadbalancer. I recently accidently deleted my cluster, but the loadbalancers were still there ; I'd have loved to be able to attach that loadbalancer to a new cluster to avoid having to change all my DNS records and keep my IP
correctly support externalTrafficPolicy: Local. Right now if you deploy a service with two replicas on a 3 node cluster, with a service type LoadBalancer with externalTrafficPolicy: Local, the 3 nodes are added to the LB, even though the pod only exist on two of them. Because of that, the LB is marked as unhealthy

Appreciate the input, thank you so much. 👏
@mishushakov a number of your points are specific to our LB product. I'll make sure to forward those to the responsible team so that they can look into triaging / prioritizing as needed. Will keep the ticket open to track the DOKS-specific parts.
@sandhose:
re: "the ability to re-use an existing loadbalancer": this is now possible on clusters running CCM v0.1.17 or later (which corresponds to DOKS versions 1.15.2-do.0, 1.14.5-do.0, 1.13.9-do.0, and later). What you basically have to do is specify the LB UUID on the corresponding Service object. See this section in the documentation for some more details.
re: LBs marked as unhealthy with externalTrafficPolicy: Local: unfortunately, this is a consequence of how Kubernetes implements the specific setting. We're aware that this is sub-optimal for users, and we have plans to provide a better UI/UX.

Also, please make sure to subscribe to digitalocean/DOKS#8 to stay up-to-date on developments around routing to external LBs from inside the cluster.

Hey, one more feature to the list

[ ] Add alerting, monitoring and notifications (on downtime)

Recently, my node went down, because for some reason Kubernetes pulled a wrong version of traefik, which because of configuration incompatibility caused a 22hr downtime (since the health checks could not be obtained by a load-balancer)
There is currently no way for me to to know, whether a node went down, unless checking manually. The priority of such a feature is critical for me and i'd like to have it ASAP :)
Thank you very much for outstanding service & support!

Hey @mishushakov, thanks once more for providing valuable feedback.
Having good observability around load balancers sounds like a reasonable request. FWIW, you could build this yourself today by polling the configured health checks periodically and reporting into a system like Prometheus. I do understand how a built-in, zero overhead mechanism would be much more convenient to have as a user, however, so I'll make sure your request gets forwarded to the right people.

One more:

[ ] Proxy Protocol for individual ports

One more:
On demand TLS or support for letsencrypt certificates when not using DO's DNS servers.

Please see how render does automatic (wildcard) Let's Encrypt Certs


Invalid links

Looks like the user has deleted the project
Link

Link

Sorry, @gruselhaus. I messed up this PR


refactor: separate crate for wallets logic

Motivation
Extracts wallet logic to separate crate to make it more resusable. After that it will be possible to add foundry-wallets as a dependency of foundry-cheatcode to support unlocking and using wallets directly from cheatcodes

yeah, I only see this as an intermediary issue, because I think we want this crate to be more like a library and then move the clap types back to the cli crate.
but this is a less invasive first step in that direction


can't get i18n to work, System is not defined

i have just updated my package.json to use angular 4, and then i said let's dumb ng2-translate and use angular default i18n instead. i have been following this guide:
https://angular.io/docs/ts/latest/cookbook/i18n.html
but i am stuck with two errors:
ReferenceError: System is not defined[Learn More]  localhost:8080:25:5

Error: Cannot find module './locale/messages.ar.xlf!text'.

here is my pacjage.json
{
  "name": "web",
  "version": "0.0.0",
  "license": "MIT",
  "angular-cli": {},
  "scripts": {
    "start": "ng serve",
    "build": "ng build --prod",
    "build-aot": "ng build --prod --aot",
    "lint": "tslint \"src/**/*.ts\"",
    "test": "ng test",
    "pree2e": "webdriver-manager update",
    "e2e": "protractor"
  },
  "private": true,
  "dependencies": {
    "@angular/common": "^4.0.0",
    "@angular/compiler": "^4.0.0",
    "@angular/core": "^4.0.0",
    "@angular/forms": "^4.0.0",
    "@angular/http": "^4.0.0",
    "@angular/platform-browser": "^4.0.0",
    "@angular/platform-browser-dynamic": "^4.0.0",
    "@angular/router": "^4.0.0",

     "systemjs": "0.19.40",
    "angular2-jwt": "^0.1.28",
    "chart.js": "^2.5.0",
    "core-js": "^2.4.1",
    "font-awesome": "^4.7.0",
    "ng2-charts": "^1.4.1",
    "ng2-facebook-sdk": "^1.2.0",
    "rxjs": "^5.1.0",
    "ts-helpers": "^1.1.1",
    "zone.js": "^0.8.4"
  },
  "devDependencies": {
    "@angular/cli": "1.0.0",
    "@angular/compiler-cli": "^4.0.0-rc.5",
    "@types/facebook-js-sdk": "^2.8.0",
    "@types/jasmine": "2.5.38",
    "@types/node": "^6.0.46",
    "codelyzer": "~2.0.0",
    "jasmine-core": "2.5.2",
    "jasmine-spec-reporter": "2.5.0",
    "karma": "1.3.0",
    "karma-chrome-launcher": "^2.0.0",
    "karma-cli": "^1.0.1",
    "karma-jasmine": "^1.0.2",
    "karma-remap-istanbul": "^0.2.1",
    "protractor": "~5.1.0",
    "ts-node": "~2.0.0",
    "tslint": "~4.4.2",
    "typescript": "~2.1.1",
    "webdriver-manager": "10.2.5"
  }
}

and i have this code in "index.html":
document.locale = 'ar';
    // Map to the text plugin
    System.config({
      map: {
        text: 'systemjs-text-plugin.js'
      }
    });

that's where it says System is not defined.
my main.ts:
import './polyfills';

import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
import { enableProdMode } from '@angular/core';
import { environment } from './environments/environment';
import { AppModule } from './app/app.module';
import { getTranslationProviders } from './app/i18n-providers';

if (environment.production == false) {

} else {
  enableProdMode();
}

// platformBrowserDynamic().bootstrapModule(AppModule)
//     .then(success => null)
//       .catch(err => console.error(err));

getTranslationProviders().then(providers => {
  const options = { providers };
  platformBrowserDynamic().bootstrapModule(AppModule, options);
});

src/app/i18n-providers.ts
import { TRANSLATIONS, TRANSLATIONS_FORMAT, LOCALE_ID } from '@angular/core';

export function getTranslationProviders(): Promise<Object[]> {
  // Get the locale id from the global
  const locale = document['locale'] as string;
  // return no providers if fail to get translation file for locale
  const noProviders: Object[] = [];
  // No locale or U.S. English: no translation providers
  if (!locale || locale === 'en-US') {
    return Promise.resolve(noProviders);
  }
  // Ex: 'locale/messages.es.xlf`
  const translationFile = `./locale/messages.${locale}.xlf`;
  return getTranslationsWithSystemJs(translationFile)
    .then( (translations: string ) => [
      { provide: TRANSLATIONS, useValue: translations },
      { provide: TRANSLATIONS_FORMAT, useValue: 'xlf' },
      { provide: LOCALE_ID, useValue: locale }
    ])
    .catch(() => noProviders); // ignore if file not found
}
declare var System: any;
function getTranslationsWithSystemJs(file: string) {
  return System.import(file + '!text'); // relies on text plugin
}

Heya, the i18n instructions shown at https://angular.io/docs/ts/latest/cookbook/i18n.html is meant to be used with the SystemJS setup shown on the docs overall, and not the CLI.
The CLI has a specific i18n commands that should make it easier: https://github.com/angular/angular-cli/wiki/stories-internationalization

thanks, totally didn't notice at first that i can run --aot in development environment. i18n is working for me now (Y)

Can somebody clarify if this means AOT is the only way to provide localization if you are using Angular-CLI?  Is there any support for JIT translations with Angular-CLI?

Dude! You saved my day. It really helps !! I was getting the exact same error mentioned above and after following the post, it worked for me! 👍

Hi I got below error. Please help..
Cannot find module './locale/messages_es.xlf!text'
at text$ namespace object:5


Seeing all tx unconfirmed and change tx not being spendable

Checking Trezor operation today I am seeing this:

This looks like something to do with the mempool.
Also seeing the change output of (Trezor) sends not being immediately spendable

You have to do a Repair wallet to get the wallet in a normal state again

And then if you close down MBHD and start it up again the problem comes back

This is due to some old transitive dependencies - we'll bump release numbers of bitcoinj-alice, multi bit-hardware and rebuild things which should eliminate this

If you build using the bitcoinj-alice 0.1.0 you should not see this anymore.
Awaiting review and closing

Verified working.
Closing.


relax otel requirements for contrib projects

now that we have a beta, the versioning has changed

ugh, using beta broke Symfony - I'll see if I can unbreak it


Updated spec path

update Path to auto generated code

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go


spiderpool update to v0.3.0

https://github.com/spidernet-io/spiderpool/blob/github_pages/changelogs/v0.2.3-to-v0.3.0.md

Noted. Will add to the docsite.


Fix Globus Compute Launcher

More specifically, ensure the Globus Compute client is initialized only once for the duration of submitting jobs to remote endpoints.

This was addressed at least partially by #24. Probably needs more testing to confirm that everything works.


[Feature Request] Append to highlights file instead of overwriting

Currently when importing highlights, the entire highlights page is generated fresh.
This means that any caret references, edits to highlight notes are broken / overwritten on each import.
This limits the usefulness of the plugin quite severely as the highlights page can only then ever be used as a visual reference to read and remind ones self (or copy and paste 😵), and therefore not truly integrated into the Obsidian / Zettelkasten note making workflow.
I think the simplest solution around this would be to simply append new highlights / notes to the highlights file instead of overwriting the whole file completely with a fresh import. From memory the Kindle highlights plugin works this way.
Is it realistic to look into something like this? Would be immensely appreciated.
I want to help, but my programming chops are probably not going to do much.
I'd be happy to update and extend the documentation if needs be though.

woops. wrong repo. Meant to be on kevins fork 😅


[AMBARI-23758] - Filter dropdown components not displayed properly

What changes were proposed in this pull request?
Firstly the overflow settings have been removed from the main element.
Few small css fixes also included (eg.: the requested Ambari logo)
The bootstrap logsearch css has been updated.
How was this patch tested?
Manual and unit tests (267/267 success)
Please review Ambari Contributing Guide before opening a pull request.
@aBabiichuk please review it. Thanks.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2158/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2163/
Test PASSed.


Remove autofocus demo

Currently, after opening the page, the autofocus example makes it scroll away from the top. This obscures first few examples on the page.
(This is the DX tests finding)

I tried to reproduce that issue but without success because of my big screen. Nice catch!

Should we add the autofocus to the very first demo example?

Not worth the demo at all IMO. It’s enough that we list it in the API doc. Usage and the result are as intuitive as it could be.


Auto colse dialog box

Is any option have auto close in dialog box, If no means what about bring that

Autoclose? Can you elaborate on that a bit please?

The dialog box should disappear after some seconds, like snackbar

Hmmm I think you should use a snackbar then. Check the official spec for dialogs
"Dialogs always retain focus until dismissed or a required action has been taken."
https://material.io/guidelines/components/dialogs.html

Yup, its by spec that dialogs should remain on screen until a user action.
Dialogs should be used when something happens that requires user attention, like errors and what not.
Snack bars are for things that if a user doesn't acknowledge them, it wont affect the flow.. small informative messages.
Having said that, you can do your own implementation with a timeout and closing it after opening it if you REEEEEEEALLY want to do it.
  openAlert(): void {
    let dialog = this._dialogService.openAlert({
      title: 'Alert',
      disableClose: true,
      message: 'This is how simple it is to create an alert with this wrapper service.',
    });
    setTimeout(() => {
      dialog.close();
    }, 3000);
  }

But we discourage people from doing this, since at that point a snackbar is a better UX and spec.
Closing this since its not spec.


Removed the automatic adding of relations in SelectQuery and SelectForUpdateQuery

No more auto relations 🤔
I take it there is some difficulty in fixing the recursive relations issue that isn't worth the benefit?


Can't fully login

I've setup the developer app side of things, and put in the client id and secret. But when I press login and authorise the app, it just keeps spinning the waiting icon on the loading account information section.
I have managed to download my liked songs, but it won't show up playlists, etc. Unsure if there is a way to see any logs to help?

Hi. I have a few questions:

When you tried to log in to your account in the app did you use the same account as when you created the Spotify service app in spotify developer?
What happens when you try to open a playlist page(gives an error, or infinite download)?
Am I correct that you can access liked tracks page?

You can reopen the issue if you provide additional information.

Still can't login.


Mx9 developer portal

Hello,
Yvann from the Projects team. I have made a fork to create the developer portal guide updates with new screenshots and update navigation structure. Do not hesitate to contact me if there is any issue with my pull request.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Yvann Noto seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @yvannnoto thanks for the PR but due to the work we were already planning to do on the Dev Portal Guide for updates and restructuring, the PR is full of conflicts and cannot be merged to support our work. Please feel free to review the updated DPG, update your fork, and send another PR with any suggestions/reviews.


Switch to refactored UI

As there are no longer any release blockers on the UI side, merging refactor into master should move us to the refactored UI. We want to wait for the CLI to be changed to be in sync though, which is #308.

@mgrauer - now that @yarikoptic has updated CLI, we may be able to make this switch now. are there any blockers left?
also wdyt of dandiarchive.org pointing directly to the data portal, now that this refactor is done.

note that I have not released dandi-cli yet.  I also want to explicitly bring your attention to https://github.com/dandi/dandiarchive/issues/316 (not a blocker here) so we could enhance dandi-cli so it could alert users when a newer version needs to be used etc.

also wdyt of dandiarchive.org pointing directly to the data portal, now that this refactor is done.

i think this would break the redirection service, and so we couldn't support our dandiset permalinks.

i think this would break the redirection service, and so we couldn't support our dandiset permalinks.

the top level redirect (https://github.com/dandi/redirector/blob/master/serve.py#L18) is separate from the permalink redirect (https://github.com/dandi/redirector/blob/master/serve.py#L33).

Got it, this seems reasonable then.
You are suggesting the following?

www.dandiarchive.org would redirect to the about static page
dandiarchive.org would redirect to gui.dandiarchive.org, which could get to the about static page from the about link
the permalink for dandisets will still go to the redirector, which will query the backend and redirect to the place in the new gui

Let's also tweak the data portal link back to the gui (I'll do this in a separate PR on the static site)

Let's also tweak the data portal link back to the gui (I'll do this in a separate PR on the static site)

I have this PR, https://github.com/dandi/dandi.github.io/pull/24, but I see it is not necessary as the dandiset/drafts route will be redirected to the GUI.
Is there any reason to support this dandiset/drafts route now or can we remove that from the redirector? Especially as it isn't going to a listing of drafts and we no longer have a listing of dandisets that are only drafts.


Add new Display methods and use constants for HIGH and LOW digital states

Improvements
Added 2 methods to the Display class
centred_text_line()
Allows to center a single line of text.
I updated the existing centre_text() method to use this one for its horizontal centering part.
Ideally I would have called this method centred_text() but I think this would be too close to the already existing method and could create some confusion.
arrow()
A simple method to draw arrows of different sizes and color pointing left, right, top or bottom.
I'm using it in my script class already but I think it's a nice little addition to the Display class.
Ideally I would have written all the conditions to display the different directions inside a single loop but to save as much computing time I decided to have two separated loops dependent of a condition executed only once.
Add LOW and HIGH constants
Abstracting the HIGH and LOW digital values slightly improves readability and maintainability.
All scripts can use these constants to make things clearer.
When using such constants, it makes it very easy to distinguish integers from digitals states in a logic.
Fix
Global oled instance called in Display class
Display::centre_text() was calling oled.show() instead of calling its own show() method.
Since there is only one display instance, that's not much of an issue but it's still incorrect.

Hey @mjaskula , thanks for your message, I'm also very excited to contribute to the firmware and to this very cool project overall!
I'll read the contributing guide more in depth, sorry if I made mistakes in that aspect.
About the HIGH and LOW constants, I quickly discussed that in discord and we agreed that modifying all the scripts could introduce bugs.
Since it's only a readability improvement (as those values will most likely never be changed), it could create more harm than good in my opinion.
It should however be enforced for the next changes or for the new scripts.
Thanks for taking the time to review my changes, I'll go over all the comments now.

@atulrnt This PR is ready to merge, except that it has some minor conflicts with code that was changed by a different PR merge. Would you like to resolve these conflicts or shall I?

I just resolved the conflicts, it should be good to do now, thanks for the review!


twister: support scanning for ztests in subdirectories

Previously, only the main src/ directory was scanned for ZTest testsuite names and tests.
"Testsuite mismatch" errors would appear even though testsuites would pass (false negatives).
With this, we can place test sources in subdirectories and twister does not encounter the false negatives.
Fixes #61932

rebase please


DashMediaSource make crash

Media3 Version
Media3 1.0.1
Devices that reproduce the issue

Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Yes
Reproduction steps

play mpd file

Expected result
Don' crash
Actual result
Crash
Media
local file
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Here is gradle version

I have add this line to my proguard, and works
-keep class org.xmlpull.v1.** { *; }

I can not repro this. Probably also because you didn't declare repro steps.
It looks like it works now. Can we close this issue?


Ability to copy a card

Describe the feature you'd like to have added
When setting up a large system that would say need 10 extractor cards set to orange it would be super easy if we could craft:
1 Configured Card
1 Other Card (set to defaults)
Output:

1 Configured Card (original)
1 'Copy' of the Configured Card

Additional information
No response

This already sort of exists.  If you have a stack of the 10 cards in your Card Holder and right click the stack, you can configure them all at one time.  Not exactly the same as the title implies which is more for the ability to copy an existing card (which would be nice), but the existing "mass-configuring" feature would help out when you "need 10 extractor cards set to orange" like you described.

Correct you can bulk modify cards inside the card holder.

You can bulk modify, but not bulk-overdrive or bulk-add-filter, even provided you have all the materials.
If I need 12 energy cards set to extract with one overclock so it can extract 4k FE/t... I need to configure each one manually. I can bulk set them to extract, but not bulk add the overclocker.
Related to #158

Yeah I would really love a way to copy-n-paste the settings (NOT connections) of one entire node to another or even just the face of one node to a different face of the same node or another node.

Node overclock cards in each face of the node.
Cards in each face of the node

Config of each card.

Including filters and the filter's settings.
Overclocks in each card.

The node to node/connectors is quickest part of any LaserIO setup I do. Having to setup dozens of nodes with the exact same setup over and over really slows things down and is sadly, for me, quite error prone.
On paste have it check player's inventory and card holder for the needed cards/overclocks/filters and then depending on if the needed number is met either perform the paste or spit out an error of missing required items. A partial paste would be bad so all or nothing, imo.
I'd leave the connection settings of the node to other nodes/connectors out though and leave that as manual only. Otherwise that could turn into a rat's nest and/or connect laserio "networks" that are meant to be separate but otherwise functionally have the same setup.


Virtual filesystem support?

This isn't really an issue so much as a feature proposal...
I work with disk images in my emulator - and they have FAT file systems, and those filesystems could use a file browser.
It would be cool if this could work with a virtual filesystem. I'm not sure how exactly it would work, maybe some sort of FileSystem trait interface?
This could also perhaps enable it to work with emscripten's MEMFS virtual filesystem as well.
https://emscripten.org/docs/porting/files/file_systems_overview.html

This is a really interesting idea, but also relatively complicated to implement.
My first idea would also be a FileSystem trait. Something like:
trait FileSystem {
    fn read_dir(path: &str) -> Result<DirectoryContent>;

    fn read_file(path: &str) -> Result<Directory Entry>;
}

struct DirectoryContent {
     content: Vec<DirectoryEntry>,
}

struct File {
    modify_date: SystemTime,
    data: Vec<u8>,
    type: <Enum for File/Directory>,
}

That is at least a first idea without having given it much thought.
With something like this it might also be possible to implement wasm support 🤔

Actually if we use the wasm-unknown-emscripten target, we should get the emscripten virtual filesystem for free, as it implements the std::fs interface.
Still would be nice for mounted disk images, though.
I cobbled together my own little file browser for disk images, it's not nearly as pretty:


Add mob item drops

Should be some kind of blank equipment (probably helmet), that will be socketed by crafting it with the best eggs the chickens can produce (eggs for stats, eggs for projectile mods, etc).


Feature Request- Make the vdiclient to work with UNRAID VMs using spice

First of all, thank you for your effort and this VDI Client has become necessary for my work from home requirements.
I also have some VMs running on UNRAID and I was wondering if it is possible to have a VDI Client to connect to VMs running on UNRAID using the spice protocol.
Thank you.

This client is only intended for Proxmox, and it would really be a whole separate project to do this for UNRAID.


Requirment: Clear all debug sessions

Expected Behavior
As the description, I think it's useful if provide a command to clear all sessions

Actual Behavior

Steps to Reproduce the Problem

Specifications

Version:
Platform:

Seems like k8s doesn't support it yet.
https://github.com/kubernetes/kubernetes/issues/84764


Boogie Constant Problem

The following fails to verify:
import std::io

method main():
    io::println("usage: gcd <input-file>")

However, in contrast, this very minor variation does indeed verify:
method main():
    io::println("usage")

This suggests that the problem is related to Boogie's handling of constants (unfortunately).

@utting Ok, confirm that /useArrayTheory is needed for this to verify.  It was missing because I was running boogie from the command-line and not including it. Doh!


await session.dispose() in createSession

Cleanup might be skipped since dispose() is asynchronous. createSessions awaits for dispose, so createSession should probably do that too.

looks like there is a no-floating-promises tslint rule I should try out

I added no-floating-promises which would have caught this. I also released your fix in version 0.4.2

Thanks! :+1:


Fix binary diff

Don't try to encode diff using utf-8 as diff could be from binary file

CC @SteveMacenski , This should fix the CI issue with changed binary files in git repos:

]0;colcon cache [30/32 done] [0 ongoing]Starting >>> nav2_bringup
--- stderr: nav2_bringup
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/colcon_core/executor/__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/colcon_core/task/__init__.py", line 93, in __call__
    return await task_method(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/colcon_cache/task/lock/git.py", line 96, in lock
    self.compute_current_checksums(args, lockfile)
  File "/usr/local/lib/python3.8/dist-packages/colcon_cache/task/lock/git.py", line 127, in compute_current_checksums
    h.update(diff.encode('utf-8'))
UnicodeEncodeError: 'utf-8' codec can't encode characters in position 5484-56349: surrogates not allowed
---
Failed   <<< nav2_bringup [0.02s, exited with code 1]
]0;colcon cache [30/32 done] [1 ongoing]]0;colcon cache [31/32 done] [0 ongoing]Starting >>> nav2_system_tests
--- stderr: nav2_system_tests
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/colcon_core/executor/__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/colcon_core/task/__init__.py", line 93, in __call__
    return await task_method(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/colcon_cache/task/lock/git.py", line 96, in lock
    self.compute_current_checksums(args, lockfile)
  File "/usr/local/lib/python3.8/dist-packages/colcon_cache/task/lock/git.py", line 127, in compute_current_checksums
    h.update(diff.encode('utf-8'))
UnicodeEncodeError: 'utf-8' codec can't encode characters in position 788-72001: surrogates not allowed
---
Failed   <<< nav2_system_tests [0.27s, exited with code 1]
]0;colcon cache [31/32 done] [1 ongoing]]0;colcon cache [32/32 done] [0 ongoing]
Summary: 30 packages finished [1.55s]
  2 packages failed: nav2_bringup nav2_system_tests
  2 packages had stderr output: nav2_bringup nav2_system_tests
'utf-8' codec can't encode characters in position 5484-56349: surrogates not allowed

Exited with code exit status 1

https://app.circleci.com/pipelines/github/ros-planning/navigation2/6441/workflows/a98ab0e8-0381-4a3c-a6c6-dacaff5a610d/jobs/23676/parallel-runs/0/steps/0-117https://github.com/ros-planning/navigation2/pull/2649#issuecomment-972495041


Implement Discussions

Building on https://github.com/NEAR-DevHub/neardevhub-bos/issues/540, we'd like to introduce a similar feature called Discussions. The main difference here is that any user can write a post, not just community managers of a particular community.
Problem
Right now, it's hard for users to engage in discussions across our various communities. The existing homepage/feed is all about proposals and contributor reports, which is great, but it doesn't give people an easy way to engage.
Solution
Create a Discussions feed on each community page, which will allow users to easily post their updates.
User Stories

As a user who cares about specific communities, I want to share my updates with a specific community and engage with other posts.

Acceptance Criteria
DevHub community page requirements:

[ ] There is a new Discussions tab, which is the second tab that users see when they land on a given community
[ ] All users see a share update input text area field (e.g. What's happening)

[ ] The share update input text area field contains all the same elements from the near.org, including supporting markdown, image update, as well as the preview.
[ ] Users can type their update, preview it, upload image, and click post. Upon post, the update shows up in the discussions section with the user's profile/name.

[ ] The discussion feed is sorted by latest, but users can also filter by recent comments
[ ] Users can click Load More to load additional posts
[ ] Users can interact with discussions posts (like, add comment, copy link to post, or share).
[ ] Users can report posts (clicking the three dots). Note: @ailisp will handle this part in a separate ticket. Please check in with him.

near.org and near social requirements:

[ ] Any discussion posted on DevHub communities will also show up on near.org and near social gateways

Backend storage requirements:

[ ] Use Social DB and follow near.org conventions-

Mockups (See discussion section)
https://www.figma.com/file/6bdpB5IrHztN7E40Rjle1N/Community-Feature%3A-Announcements?type=design&node-id=12%3A6010&mode=design&t=M1jwBsC4NvLrn35p-1
Technical Details
See technical considerations from https://github.com/NEAR-DevHub/neardevhub-bos/issues/540

Not sure if it is a bug but I don't see any information on this issue, like who is assigned and the description.

I've opened a draft PR for this; mostly complete. Will have some pending contract changes and will then prepare a preview once Announcements is merged)

@elliotBraem has made annoucement to production. I think this one is quite similar in terms of UI, but one challenge is how to consider a near social post as part of the community discussion. Maybe manually add some #community-topic tag to the post and filter those posts when query a feed of certain community?

one challenge is how to consider a near social post as part of the community discussion.

Re-post the author-posted Social Post to the community discussions account through DevHub contract.

And then just display the discussions account feed on the community page, it will be full of re-shared posts at the end of the day, like this: https://near.social/mob.near/widget/ProfilePage?accountId=luchito1.near


Still don't know how to get val_videofolder.txt

I'm sorry. Although I've looked at other people's questions, I still wonder how to get val_videofolder.txt

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1
It's my fault. I want to use the kinetic data set. You know how to get val_videofolder.txt

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1
It's my fault. I want to use the kinetic data set. Do you know how to get val_videofolder.txt？Can you tell me？

您能告诉我们您使用的是哪个数据集吗？我曾尝试在 Sth-Sth v1 数据集中进行训练，结果val_videofolder.txt是通过 run 获得的gen_label_sthv1.py，它使用validation.csv. 顺便validation.csv可以在https://20bn.com/datasets/something-something/v1下载

Can you share something v1 data set，please

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1

Hi @Andy1621 , I found this link no longer available. I am trying to play with kinetics. Could you plz provide a link for downloading those .csv files including kinetics_val.csv? Thanks!

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1

Could you please let us know which dataset you use? I have tried to train in Sth-Sth v1 dataset, and the val_videofolder.txt is got by run gen_label_sthv1.py, which uses validation.csv. By the way, validation.csv can be downloaded in https://20bn.com/datasets/something-something/v1

Hi @Andy1621 , I found this link no longer available. I am trying to play with kinetics. Could you plz provide a link for downloading those .csv files including kinetics_val.csv? Thanks!

I just wonder if I need to download the dataset locally... I am trying to use Kinetics, and by the Data Preparation here, I need to generate val_videofolder.txt by running gen_label_dataset.py. Now the question becomes, what should be the input files here (in the image)?

I went to Kinetics website and downloaded those files:

But when I use them as input, I got an KeyError. So am I using the wrong file or using them wrongly? If it is the former case, do I need to download the dataset in video format and strictly follow the Data Preparation?
Thanks so much! I am an undergraduate student in Engineering and have not much idea in computer science/AI. I am sorry if the answer to my question is obvious. Your help will be highly appreciated!

For Kinetics, you can download it in cvfoundation.
By the way, welcome to play with our UniFormer, which is a new SOTA backbones for video classification.


[Enhancement] If unable to find packages in response for make npm bail

If no packages were found in the response, the request likely failed due to network issues. We shouldn't update the package count in this case.

Closed with https://github.com/nickzuber/ssg-nickzuber.com/commit/0382d62d251dab91f8d14219c41fb1bf9ead4d6a


Expand cocoatextfield to support custom changeCharactersInRangeReplacementString delegate method

I needed additional control over UITextViewDelegate methods in my project and I decided to implement custom closure to determine if I should change character when it happens

@Overcot this is great, I just wonder if there's a more "SwiftUI-y" name for the API that we're exposing? I'll merge this in tomorrow nonetheless (we can always deprecate functions if/when migrating to a better naming scheme).

What do you think about?
var onChangeCharacters: (((inRange: NSRange, replacementString: String)) -> Bool)?
With
public func onChangeCharacters(perform action: @escaping (((range: NSRange, replacementString: String)) -> Bool)) -> Self {
    then({ $0.configuration.onChangeCharacters = action })
}

And usage
CocoaTextField("Test", text: .constant("Test")).onChangeCharacters { (arg) -> Bool in
  let (range, replacementString) = arg
}

@Overcot this is great, I just wonder if there's a more "SwiftUI-y" name for the API that we're exposing? I'll merge this in tomorrow nonetheless (we can always deprecate functions if/when migrating to a better naming scheme).

What do you think about?
var onChangeCharacters: (((inRange: NSRange, replacementString: String)) -> Bool)?
With
public func onChangeCharacters(perform action: @escaping (((range: NSRange, replacementString: String)) -> Bool)) -> Self {
    then({ $0.configuration.onChangeCharacters = action })
}

And usage
CocoaTextField("Test", text: .constant("Test")).onChangeCharacters { (arg) -> Bool in
  let (range, replacementString) = arg
}

@Overcot I've implemented an API called CocoaTextField/onCharactersChange(_:) that should satisfy your needs.

@Overcot I've implemented an API called CocoaTextField/onCharactersChange(_:) that should satisfy your needs.

@vmanot Nice thanks 👍

@vmanot Nice thanks 👍


Orientation bug

In the horizontal mode of the program, it is impossible to use it, I suggest making it possible only in the vertical mode.

Fix Android

Fix iOS


Issue with Powershell Terminal in vscode on ubuntu

With a clean copy (removed ~/.code and ~/.config/Code folder), on Ubuntu 18.04, I've having problems with lines that are longer than the terminal width once the cursor has reached the last line of the terminal when in Powershell, using the Powershell 2019.9.0 extension.
Specifically, once the active line in the terminal has reached the bottom of the terminal window, it fails to wrap and the keystrokes overwrite the prompt.
Visiting the history using up and down does print the line with the correct wrapping.
Moving the cursor left past the newline causes the cursor to appear on the right of the same line, not at the end of the line above.
Tab completion  reprints lines unexpectedly, and attempting to use the backspace to affect text in the line before the line break causes a new lines to be generated printing the text to the right of the cursor on each new line per backspace keypress.
It's feels like a terminal emulation problem - something doesn't know how to deal with newlines and the bottom of the terminal.  This only occurs in a Powershell Terminal, not the bash terminal, and not in a stand-alone pwsh session in a gnome or xterm.
I've no extensions other than the Powershell extension.
Any advice would be gratefully received.

@WembleyFord2 does this happen in gnome-terminal?

No, only in Visual Studio code.

I've just tried installing VSCode, via the .deb package, and the latest Powershell from the MS repositories onto a new Ubuntu installation and I see the same behaviour.

Everything seems fine to me, I'm guessing it's an issue with the powershell extension?


no audio http? simple shoutcast stream?

@Dj-jom2x
i have this problem too, and i don't know how we can catch and trace errors, audios couldn't play is some device

yeah so hard to find a working one. thats the very basic one and yet its so hard to implement :3 on real android app its easy to make one but i wanted to try flutter..

This plugin is based on MediaPlayer on android. So if you can not play a song via MediaPlayer, neither you can play it via this plugin.  ReferMediaPlayer support media format

all shoutcast are mp3 or aac.. but ours is mp3 :( still no sound

i have same problam but it work on ios. Android not working for me

This can be fixed by putting android:usesCleartextTraffic="true" within your manifest.xml file inside your Android directory. Example below
<?xml version="1.0" encoding="utf-8"?>
<manifest ...>
    <uses-permission android:name="android.permission.INTERNET" />
    <application
        ...
        android:usesCleartextTraffic="true"
        ...>
        ...
    </application>
</manifest>

closed in favour of #383


How to handle deep objects with onlyUpdateForKeys?

Hey, @acdlite , I just started browsing this library and really like it, thank you.
I wanted to ask a question, but unfortunately, could not see appropriate tag on stackoverflow, so decided to do it here.
recompose uses shallow comparison in onlyUpdateForKeys implementation. But what is the best way to handle components if I need something like deep rendering? In my case I don't use Immutable.js, so {a:'1'} !== {a:'1'}, so component is triggered to be re-rendered. Could you suggest, please, how to handle this with recompose?
Thank you!

Hey ;-) in your case

you could use https://github.com/acdlite/recompose/blob/master/docs/API.md#shouldupdate directly
you could use mapProps to transform deep object structure into non deep, then use onlyUpdateForKeys

BTW IMHO in most cases using non pure shallow equality can cause a lot of problems in the future, so I recommend you to work with plain js objects as with immutable data (you don't need immutable.js to use the same behavior) and trying to use just pure HOC or nothing whenever it possible.

For sure, I'm trying to write pure components with plain object props as much as possible, but I just found few places where it is a bit weird to have all props as a non-deep tree from logical point of view.
I think 1st solution is not really cool, since writing shouldUpdate will cause some messy code, I would like to generalize it somehow. But, mapProps feature looks really useful, thank you!

In addition, there is flattenProp which is a shortcut for mapProps.

It looks really useful when I have a strict structure of my property.
But what if I have, for instance, style object with wide range of possible properties which I cannot validate using propTypes? Seems like in this case the bast way is to implement shouldUpdate manually and do something like shallow compare in there.

Seems like in this case the bast way is to implement shouldUpdate manually and do something like shallow compare in there.

Yes, this is what we would suggest.
Though I tend to agree with @istarkov that, if you must implement shouldComponentUpdate (and usually, you shouldn't), you should probably stick with shallow equality.

@acdlite Maybe we should add a line to the README to tell people use a recompose tag on SO?

Does onlyUpdateForKeys support lodash get syntax?
i.e:
@onlyUpdateForKeys(['prop1', 'obj.nestedProp.nestedProp2'])

@damianobarbati  no, and will not


AppVeyor: Adding codecoverage after tests are ran

Changes to AppVeyor

After the tests has ran, code coverage from Pester will be written to AppVeyor Console

This Pull Request (PR) fixes the following issues:
n/a

[ ] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable