[5.7] Collection whereBetween

This PR adds posibility to make whereBetween filters in Collections.
$c = new Collection([['v' => 1], ['v' => 2], ['v' => 3], ['v' => '3'], ['v' => 4]]);
$c->whereBetween('v', [2, 4])

What about numeric arrays?
e.g. [1,2,3,4,5,6], it would be a little bit weird to pass in $collection->whereBetween(null, [2,6]) And would this even work?

What about numeric arrays?
e.g. [1,2,3,4,5,6], it would be a little bit weird to pass in $collection->whereBetween(null, [2,6]) And would this even work?

It working usign where conditions and work like it, i.e. as you can not use where condition with numeric arrays

Honest question; why not just 3 parameters? $key, $from, $to (or whatever name)

Nevermind, I just realized this is the Query Builders logic as well. Ignore me!


dot not use "id" but rather use primary_key

I have model with different primary key (other than ID)

0.1.6 was just released to rubygems with this improvement.


feat: update RouterState

I saw that it also can be null, I was thinking, we can just declare them optional, so user don't have to pass in all keys. Or did I miss something? Or does it explicitly have to be null?

Making them optional means the value is either of the defined type or undefined. In reality they are either of the defined type or null. So the current type defs look good to me.

ahh, I might have missed something, I thought maybe it's unintentional because while writing tests, I had to provide all, but maybe because of something else, tests were still passing :smile: anyway, thanks. I'll close the PR.
So it's always null or of defined type, and they can never be undefined, right?


Build fails: rtcvars.h: No such file or directory

I just cloned this repository and opened it with VS Code PlatformIO.
When compiling I get the following error:
src/pvAlgo.cpp:11:10: fatal error: rtcvars.h: No such file or directory
PS: Here's the full compilation output:
Processing nodemcuv2 (platform: espressif8266; board: nodemcuv2; framework: arduino)
Verbose mode can be enabled via -v, --verbose option
CONFIGURATION: https://docs.platformio.org/page/boards/espressif8266/nodemcuv2.html
PLATFORM: Espressif 8266 (3.2.0) > NodeMCU 1.0 (ESP-12E Module)
HARDWARE: ESP8266 80MHz, 80KB RAM, 4MB Flash
PACKAGES:

framework-arduinoespressif8266 @ 3.30002.0 (3.0.2)
tool-esptool @ 1.413.0 (4.13)
tool-esptoolpy @ 1.30000.201119 (3.0.0)
toolchain-xtensa @ 2.100300.210717 (10.3.0)
LDF: Library Dependency Finder -> https://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 47 compatible libraries
Scanning dependencies...
 Check our library registry!

CLI  > platformio lib search "header:rtcvars.h"
Web  > https://registry.platformio.org/search?q=header:rtcvars.h

11 | #include <rtcvars.h>
|          ^~~~~~~~~~~
compilation terminated.
*** [.pio/build/nodemcuv2/src/pvAlgo.cpp.o] Error 1
================================================================== [FAILED] Took 3.92 seconds ==================================================================
The terminal process "platformio 'run'" terminated with exit code: 1.

The file is from here: https://github.com/highno/rtcvars
But it should be resolved automatically, if PlatformIO is correctly installed (because the libaray is included in the platformio.ini)

Hi Stefan, I think I found the cause of the problem:
In your code the include looks like this:
#include <rtcvars.h>
But the file name is actually RTCVars.h. Thus the include needs to be changed to
#include <RTCVars.h>
On Windows this does not matter (it does not care about casing). But since I'm building on Manjaro Linux, the casing is important.

Good to know. Yes, this should be changed. If you want you can raise a PR, if not I can also modify it at the next chance.

I have created the pull request.


Skills + OAuth: Differentiate between regular and skill mode when receiving tokens (Python)

See parent
When doing OAuth, and a Skill gets an activity, the skill thinks it is in regular mode and not skill mode, so it uses the wrong connector client.
The fix is to have the Token Service add an additional claim and have the SDK act accordingly and know in what mode it is based on the claim.

The code for this landed in C# today. Please check with @carlosscastro for details.


Add mario and nicola as owners

you are both now in the OWNERS and REVIEWERS.

/lgtm

i will manually merge this as it is not code based once tests pass


add Activation Events for setting field

Issue Type: Feature Request
Add Activation Events for setting field.
This allows precise control over when expansion needs to be activated.
Reduce unnecessary extensions runs.
Reduce memory usage and speed.
onLanguage:${language}
onCommand:${command}
onDebug
workspaceContains:${toplevelfilename}
onFileSystem:${scheme}
onView:${viewId}
*
+ onSetting: ${key}==${value}
+ onSettingExist: ${key}
+ onSettingUpdate: ${key}

Why should I need it?
some extension, they need to be activated when user set the relative config.
for example: when user seted sync.autoDownload then actived the extension Settings Sync and download the setting.
{
  "sync.autoDownload": true
}

set activationEvents
{
  "activationEvents": [
-   "*"
+   "onSetting: sync.autoDownload==true",
+   "onSettingUpdate: sync.autoDownload"
  ]
}

So, Settings Sync don't need to activating and check sync.autoDownload the field exists or not.
In theory, it can reduce the self-starting of a part of the extension, which should improve performance.
VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:33:29.235Z)
OS version: Darwin x64 17.7.0

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Authentication with console

The cli should manage the login workflow with the console.
The user experience could be:
miactl auth login

and this command starts the oauth2 flow to authenticate and to save the session.
miactl auth logout

removes the saved user session
At every request, the session token should be refreshed if session is expired

Authentication with username and password will be added in #38


api: use middleware to get raftCluster

What problem does this PR solve?
Many handlers need to get RaftCluster and check if the cluster is bootstrapped.
What is changed and how it works?
Use middleware to eliminate duplicated code.
Check List
Tests

Unit test
Manual test (add detailed scripts or steps below)

 manual test detail

run pd
curl /stores /store/1, got not bootstrapped error
run tikv
curl /stores /store/1, no error

Codecov Report

Merging #1908 into master will increase coverage by 0.22%.
The diff coverage is 91.2%.

@@            Coverage Diff             @@
##           master    #1908      +/-   ##
==========================================
+ Coverage   77.55%   77.78%   +0.22%
==========================================
  Files         164      166       +2
  Lines       16578    16518      -60
==========================================
- Hits        12857    12848       -9
+ Misses       2714     2673      -41
+ Partials     1007      997      -10

Impacted Files
Coverage Δ

server/api/rule.go
3.96% <0%> (+0.68%)
:arrow_up:

server/api/stats.go
100% <100%> (+25%)
:arrow_up:

server/api/context.go
100% <100%> (ø)

server/api/label.go
86.66% <100%> (+7.87%)
:arrow_up:

server/api/admin.go
72.41% <100%> (+6.78%)
:arrow_up:

server/api/router.go
100% <100%> (ø)
:arrow_up:

server/api/store.go
66.39% <100%> (+5.2%)
:arrow_up:

server/api/middleware.go
75% <75%> (ø)

server/api/region.go
76.69% <87.5%> (+8%)
:arrow_up:

pkg/testutil/operator_check.go
88.88% <0%> (-11.12%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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

/merge


Verano-http EO library added

Added link to verano-http EO style http client.

@yegor256/z please, pay attention to this pull request

@Vatavuk this pull request is too small, just 2 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @yegor256/z please review this and merge or reject

@Vatavuk thanks!

Job gh:yegor256/elegantobjects.github.io#76 is not assigned, can't get performer

This job is not in scope


lua: fix cross-compilation; fixes #66742

Motivation for this change
Tried to cross-compile Lua. Failed. Fiddled with the build system a bit and fixed it. Now contributing :smiley_cat: See #66742 for details.
Things done

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

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

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

Notify maintainers
Wasn't able to find a maintainer for lua interpreter, sorry!

Force-pushed a fix for a mass-rebuild of Lua packages!


Add search bar to android

MA-2238

@BenjiLee please review

Done with first pass, squashy commits.

👍  squash away!


Update devcontainer.json to use shopify ruby extension

Replace deprecated extensions:

rebornix.Ruby into Shopify.ruby-extensions-pack
wingrunr21.vscode-ruby into Shopify.ruby-lsp

tested, codespace can run it without any issue


Nature's Spirit 1.2.0 incompatible with Sinytra Connector 1.0.0-beta.26 and Forgified Fabric API 0.90.7+1.10.2

Mod Name
Nature's Spirit
Mod Homepage
https://modrinth.com/mod/natures-spirit
Describe the bug
A crash on startup
Steps to reproduce
1.Update Nature's Spirit, Forgified Fabric API and Sinytra Connector
2.Run on latest available 1.20.1 forge version
3.Crash upon startup
...
Logs
https://gist.github.com/Liamgrayskull/9d038052aea0ff418da5afa1704b794d
Additional context
In the modpack Inquisitive https://modrinth.com/modpack/inquisitive

This looks like an issue with morebannerfeatures rather than Nature's Spirit


fix: error comparing reports with pre-computed description_set

When one of the ProfileReports where already pre-computed (either using lazy=False, persisting the report, or anything that triggers profile.get_description()) could cause errors when the profiles were created using a different configuration. This lead to errors when comparing sessions (e.g. duplicates) of the report that were disabled in only one of the reports.
Added a parameter to enforce the reports to be computed with the same configuration.
This parameter is only necessary when at least ProfileReport was already calculated and the ProfileReports have different settings.
e.g.
p1 = ProfileReport(data)
p2 = ProfileReport(data, lazy=False, minimal=True)
# needs to recompute the p2 for comparison
p1.compare(p2, compute=True)

Codecov Report
Base: 90.66% // Head: 90.65% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (9e7c96a) compared to base (da0debb).
Patch coverage: 85.71% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1219      +/-   ##
===========================================
- Coverage    90.66%   90.65%   -0.01%
===========================================
  Files          186      186
  Lines         5643     5648       +5
===========================================
+ Hits          5116     5120       +4
- Misses         527      528       +1

Flag
Coverage Δ

py3.8-ubuntu-latest-pandas
90.65% <85.71%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/pandas_profiling/compare_reports.py
86.25% <85.71%> (-0.25%)
:arrow_down:

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

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


Test for aws cloud provider

Added a test for testing the aws cloud provider setup scenario which was failing in issue - 24515
{"name": "aws",
"type": "cloudProvider"}

LGTM


Add env vars to change contextPaths for /api and /admin

As usual: Default behavior does not change, but one got the option to change /api and/or /admin context paths.
This is especially useful if you run a lot of containers on the same host fronted by a SSL proxy server.

@robertoschwald This has now landed in the latest release 5.17.2 - check it out. Thanks for your contribution! 👍


Issue Loading FIBO Ontology (possible dup of #1607)

Hi all & thanks for any help anyone can give.
I'm working with the owlready2 package and trying to run the FIBO version of this:
https://owlready2.readthedocs.io/en/v0.37/intro.html#short-example-what-can-i-do-with-owlready
from owlready2 import *
 o = get_ontology('https://spec.edmcouncil.org/fibo/ontology/FBC/DebtAndEquities/Debt.rdf')
o.load()

Trying to use a URL for FIBO:
I get the following exception after the last line of this sample runs:
---------------------------------------------------------------------------
HTTPError                                 Traceback (most recent call last)
~/pyenvs/myenv/lib/python3.6/site-packages/owlready2/namespace.py in load(self, only_local, fileobj, reload, reload_if_newer, url, **args)
   1050         if _LOG_LEVEL: print("* Owlready2 *     ...loading ontology %s from %s..." % (self.name, f), file = sys.stderr)
-> 1051         try:     fileobj = urllib.request.urlopen(url or f)
   1052         except:  raise OwlReadyOntologyParsingError("Cannot download '%s'!" % (url or f))

~/pyenvs/myenv/lib/python3.6/urllib/request.py in urlopen(url, data, timeout, cafile, capath, cadefault, context)
    222         opener = _opener
--> 223     return opener.open(url, data, timeout)
    224

~/pyenvs/myenv/lib/python3.6/urllib/request.py in open(self, fullurl, data, timeout)
    531             meth = getattr(processor, meth_name)
--> 532             response = meth(req, response)
    533

~/pyenvs/myenv/lib/python3.6/urllib/request.py in http_response(self, request, response)
    641             response = self.parent.error(
--> 642                 'http', request, response, code, msg, hdrs)
    643

~/pyenvs/myenv/lib/python3.6/urllib/request.py in error(self, proto, *args)
    569             args = (dict, 'default', 'http_error_default') + orig_args
--> 570             return self._call_chain(*args)
    571

~/pyenvs/myenv/lib/python3.6/urllib/request.py in _call_chain(self, chain, kind, meth_name, *args)
    503             func = getattr(handler, meth_name)
--> 504             result = func(*args)
    505             if result is not None:

~/pyenvs/myenv/lib/python3.6/urllib/request.py in http_error_default(self, req, fp, code, msg, hdrs)
    649     def http_error_default(self, req, fp, code, msg, hdrs):
--> 650         raise HTTPError(req.full_url, code, msg, hdrs, fp)
    651

HTTPError: HTTP Error 404: Not Found

During handling of the above exception, another exception occurred:

OwlReadyOntologyParsingError              Traceback (most recent call last)
<ipython-input-3-22764c73482e> in <module>
----> 1 o.load()

~/pyenvs/myenv/lib/python3.6/site-packages/owlready2/namespace.py in load(self, only_local, fileobj, reload, reload_if_newer, url, **args)
   1109
   1110     # Load imported ontologies
-> 1111     imported_ontologies = [self.world.get_ontology(self._unabbreviate(abbrev_iri)).load() for abbrev_iri in self.world._get_obj_triples_sp_o(self.storid, owl_imports)]
   1112     self._imported_ontologies._set(imported_ontologies)
   1113

~/pyenvs/myenv/lib/python3.6/site-packages/owlready2/namespace.py in <listcomp>(.0)
   1109
   1110     # Load imported ontologies
-> 1111     imported_ontologies = [self.world.get_ontology(self._unabbreviate(abbrev_iri)).load() for abbrev_iri in self.world._get_obj_triples_sp_o(self.storid, owl_imports)]
   1112     self._imported_ontologies._set(imported_ontologies)
   1113

~/pyenvs/myenv/lib/python3.6/site-packages/owlready2/namespace.py in load(self, only_local, fileobj, reload, reload_if_newer, url, **args)
   1050         if _LOG_LEVEL: print("* Owlready2 *     ...loading ontology %s from %s..." % (self.name, f), file = sys.stderr)
   1051         try:     fileobj = urllib.request.urlopen(url or f)
-> 1052         except:  raise OwlReadyOntologyParsingError("Cannot download '%s'!" % (url or f))
   1053         #try:     new_base_iri = self.graph.parse(fileobj, default_base = self._base_iri, **args)
   1054         #finally: fileobj.close()

OwlReadyOntologyParsingError: Cannot download 'https://spec.edmcouncil.org/fibo/ontology/master/latest/BE/LegalEntities/LegalPersons'!

...the relevant part of which is likely
HTTPError: HTTP Error 404: Not Found

As it seems the owlready2 library can't find the dependent LegalPersons rdf file for Debt.
Since the library is able to obtain the initial Debt.rdf file, I don't think this is a proxy issue (in fact, if I turn off the corporate proxy settings I have, it stops working and fails to pull the Debt.rdf file because of an SSL exception 😄 ).
Trying it locally
I also tried doing the equivalent of this locally to cut around any network issues by downloading the prod.rdf.zip file from here, then unzipping the archive and loading it this way:
from owlready2 import *
o = get_ontology('fibo/ontology/master/2022Q3/FBC/DebtAndEquities/Debt.rdf')
o.load()

But I get the same exception as loading over a URL, as it appears the local rdf files still reference dependencies remotely, via URL.
[Edit]: Also tried using the dev.rdf.zip download, same class of error (unable to find remote dependency).
Anyone have any ideas?

I am not an expert on owlready2 - coding in Python I rather use rdflib.
But it looks as if owlready2 truncates the import object in the following triple:
https://github.com/edmcouncil/fibo/blob/7d326d31c5060c39508a40bff2ebead232ccc308/FBC/DebtAndEquities/Debt.rdf#L76
Instead of taking the proper value:
https://spec.edmcouncil.org/fibo/ontology/BE/LegalEntities/LegalPersons/
it truncates it to:
https://spec.edmcouncil.org/fibo/ontology/BE/LegalEntities/LegalPersons
Now the problem is that only the former resolves and owlready2 tries to import the latter.
It looks to me like an owlready2 issue - I looked briefly in the code and noticed that it does quite a lot of string manipulation on '#' and '/', which usually creates problems. Because there is quite of lot of string substitutions, replacements, etc. I was not able to pin down the problem, but it may be in this method:
def _get_onto_file(base_iri, name, mode = "r", only_local = False):
  if base_iri.endswith("#") or base_iri.endswith("/"): base_iri = base_iri[:-1]
  if base_iri.startswith("file://"): return urllib.parse.unquote(base_iri[7:])

  for dir in onto_path:
    filename = os.path.join(dir, base_iri.rsplit("/", 1)[-1])
    if os.path.exists(filename): return filename
    for ext in ["", ".nt", ".ntriples", ".rdf", ".owl"]:
      filename = os.path.join(dir, "%s%s" % (name, ext))
      if os.path.exists(filename): return filename
  if (mode.startswith("r")) and not only_local: return base_iri
  if (mode.startswith("w")): return os.path.join(onto_path[0], "%s.owl" % name)
  raise FileNotFoundError

If base_iri is https://spec.edmcouncil.org/fibo/ontology/BE/LegalEntities/LegalPersons/, then the second line in this method truncates it into 'https://spec.edmcouncil.org/fibo/ontology/BE/LegalEntities/LegalPersons', which eventually leads to the import problem.

I think you might be right, I will dig around on the owlready2 page to see if I can find a workaround. Thanks for pointing that out...
Separately , to get around this issue, I tried to explicitly use a version-ed cut of FIBO (e.g, 2022Q1) thinking this might negate the need for the owlready2 library to follow redirects...
https://spec.edmcouncil.org/fibo/
   ...: ontology/master/2022Q1/FBC/DebtAndEquities/Debt/Debt

Unfortunately, even when I pull down this version I still see the references from each RDF file pointing to master, suggesting FIBO's "versioning" is only a top-level version and doesn't apply to children, which still pull from master....am I mistaken?

Separate question, is there a cut of the FIBO RDF structure that uses local/relative references instead of remote ones?

@gr-fl  In GitHub there is an XML catalog that is relative that we use to load FIBO in Protege, for example. I'm not sure that the catalog is included in the zip files available from the release site on the spec.edmcouncil.org. But maybe you could look at the catalog file, called catalog-001.xml to see what that mapping looks like and do something programmatically?

Separate question, is there a cut of the FIBO RDF structure that uses local/relative references instead of remote ones?

No, there is not.
However we produce something that is called quick version of FIBO, which is a single ontology file with no imports and which aggregates all FIBO ontologies. You can get this following the 'Download' link from https://spec.edmcouncil.org/fibo/page/owl.


支持通过 helm 参数集成 skywalking，提供 metrics 大盘模版（Support integration of skywalking through helm parameters, and provide metrics template）

Why do you need it?
用于实现 Skywalking 指标观测，从而实现基于 Skywalking 的 Higress 监控大盘

It is used to realize the Skywalking indicator observation, so as to realize the Higress observability kanban based on Skywalking
How could it be?
Other related information
skywalking doc:  https://skywalking.apache.org/docs/main/v8.9.1/en/setup/envoy/metrics_service_setting/

@johnlanni Hi, i'd like to resolve this issue, can you assign it to me?

@johnlanni Hi, i'd like to resolve this issue, can you assign it to me?

If by the time you finish #81,  this issue is still unclaimed, I'll assign it to you


LateInitializationError: Field '_cacheBlockCount@1436474497' has not been initialized

When I try with this url: https://www.kansaigaidai.ac.jp/asp/img/pdf/82/7a79c35f7ce0704dec63be82440c8182.pdf
then an exception occurs: LateInitializationError: Field '_cacheBlockCount@1436474497' has not been initialized.
Originally posted by @letrungdo in https://github.com/espresso3389/pdfrx/issues/166#issuecomment-2154193701

1.0.67 fixes the issue.


Allow patching multiple webhooks

This will support running multiple mutating webhooks for a single revision.
[ ] Configuration Infrastructure
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@zerobfd: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zerobfd: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zerobfd: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-pilot-k8s-tests_istio

integ-helm-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-helm-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zerobfd: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@zerobfd: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test release-notes_istio

integ-pilot-k8s-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-pilot-k8s-tests_istio

integ-helm-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-helm-tests_istio

integ-pilot-multicluster-tests_istio
dd17d1324d8c823c948e40b7efcad2bc05cfece3
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


add logging + improve dbid name

Add swift-log package
Improve unique connection source name
Translate SQLLiteral.default to "NULL"

Hey @tanner0101, you just merged a pull request, have a coin!
You now have 1355 coins.


Spacebar scrolling broken in macOS 13.3 beta

Maybe it will get fixed for 13.3 final, but probably not. We probably need a work-around.

I would like to verify that the problem I am experiencing is this one.   advances to the next unread for me unless the article is larger than the viewport.  In that case, I need to scroll to the bottom of the article (I use the mousewheel) before  advances again.  Is that what this issue is about?
I am on macOS 13.3 and that upgrade is when I started experiencing this as best I can recall and I am on NNW 6.1.2.

Reopening because this needs additional verification.


Test with tox

Simpler and more predictable than self-managing a virtualenv. I've set it up to allow arguments to be passed through to pytest by running e.g. tox -- -f.

This currently includes the contents of #1 , which should be merged first

This has been merged as https://github.com/ryansb/cfn-wrapper-python/commit/5e84b5bd383fada1f26831a1268779e73270d4d7

I had to make small modifications to make coverage reports appear across test runs.

Since this took a long time, we forked it to https://github.com/timeoutdigital/cfn-resource-timeout
We'd be happy for you to reconcile the changes though

Very understandable. I've also added Python 3.6 support, so it may be a bit of a task to reconcile them.
I notice the fork has a dependency on requests which I don't think I can pull back upstream, since I know some people copy/paste cfn_resource.py into the inline code for CloudFormation templates.
If you'd like, I can add you as a contributor here and we can work on adding back all the Travis + new tests in your fork. The only thing is I don't think I could bring back all the changes due to the extra dep. Lmk what you think.

We can depend on requests, at botocore.vendored.requests (or whatever the exact path is)
Wouldn't mind being a contributor :)

Ooooh, I didn't think of scooping the botocore vendored version. That's crafty of you!


Ubuntu keymap when non-EN keyboard layout active is wrong

When running Atom under Ubuntu (14.04 LTS, I believe) with vim-mode and ex-mode installed and using a JP keyboard layout (see:https://www-01.ibm.com/software/globalization/topics/keyboards/images/fig3_new.GIF) the keys are not mapped correctly. Specifically, in the circumstance that was causing me issue, when I press the : key to bring up ex-mode (which is where ' is for QWERTY layout) nothing happened. When I opened the key binding resolver, I saw that it was being mapped to ;, and I had to press the normal SHIFT+; key combo in order to get Atom to recognize the : key and open ex-mode. Oddly, typing in the code window (and even the ex-mode area) showed the correct keys.
Using Atom on a Macbook with the same JP layout does not exhibit this issue. I have not tested on Windows.

Thanks for the report! This looks like a duplicate of https://github.com/atom/atom-keymap/issues/37 though - please subscribe there for updates.


variable-name: allow-leading-underscore does not allow for multiple underscores

First a question to see if this is a a bug or not...
At the moment the allow-leading-underscore does not allow for multiple leading underscores. Is this an intentional decision for this rule to only allow a single leading underscore?
There is a closed issue (https://github.com/palantir/tslint/issues/189) from Aug 2014 about this same thing and the answer was to include the allow-leading-underscore option. So I don't know if the code has been modified since to change the behaviour of this rule?
Bug Report

TSLint version: 3.15.1
TypeScript version: 2.0.3
Running TSLint via: CLI

TypeScript code being linted
private __form;

with tslint.json configuration:
"variable-name": [true,
    "check-format",
    "allow-leading-underscore"
]

Actual behavior
Fails with error:
variable name must be in camelcase or uppercase

Expected behavior
Should allow multiple underscores at the start of a variable

I have also come across this issue (trying to use __fields as a variable name)

I have the same issue. I would like to mark obj._protectedVar and obj.__privateVar, this is only because that typeScipt generates javascript with publicly available private and protected variables. So such naming conventions can give visual information about its intentions.

double leading underscore for private members doesn't seem to be widely used in the typescript ecosystem. I doubt there will be a new option to enable this in variable-name.
If you really need this, you can try this one:
https://www.npmjs.com/package/tslint-consistent-codestyle#naming-convention
A sample configuration to enforce leading underscore on protected and double leading underscore on private members would be:
"naming-covention": [true,
  // this config will apply to properties AND methods, if you only need it for properties, use "property" instead of "member"
  {"type": "member", "format": "camelCase"}, // use camelCase for all members, will be inherited by protected and private
  {"type": "member", "modifiers": "protected", "leadingUnderscore": "require"}, // protected members will be REQUIRED to have a leading underscore. you can use "allow" as alternative
  {"type": "member", "modifiers": "private", "prefix": "__"} // to simply allow and not enforce double leading underscores, use "prefix": ["__", ""]
]

@ajafff i liked the solution. Tried to setup but it didn't worked out for me. Details described in opened issue:
https://github.com/ajafff/tslint-consistent-codestyle/issues/6#issue-213196169

@ajafff Thank you for pointing extension that you are working on.
First of, "naming-covention" is a typo. And this was the time when i again realized how type doing a bad name in a programming world. Please don't get me wrong.
I've played more with plugin, and i liked implementation. I take ALL rules for naming-convention, so now my rule in tslint.json looks like now:
    "naming-convention": [

      true,
      {"type": "default", "format": "camelCase", "leadingUnderscore": "forbid", "trailingUnderscore": "forbid"},
      {"type": "variable", "modifiers": ["global", "const"], "format": "UPPER_CASE"},
      {"type": "parameter", "leadingUnderscore": "allow", "final": true},

      {"type": "member", "modifiers": "private", "leadingUnderscore": "allow", "prefix": "__"},
      {"type": "member", "modifiers": "protected", "leadingUnderscore": "require"},
      {"type": "method", "modifiers": "private", "leadingUnderscore": "allow", "prefix": "__"},
      {"type": "method", "modifiers": "protected", "leadingUnderscore": "require"},

      {"type": "property", "modifiers": ["public", "static", "const"], "format": "UPPER_CASE"},
      {"type": "type", "format": "PascalCase"},
      {"type": "class", "modifiers": "abstract", "prefix": "Abstract"},
      {"type": "interface", "prefix": "I"},
      {"type": "genericTypeParameter", "prefix": "T"},
      {"type": "enumMember", "format": "PascalCase"}

    ],

And again thank you very much!
To add,
Would be great if some rules could be optimized. For example:
      {"type": ["member",  "method"], "modifiers": "private", "leadingUnderscore": "allow", "prefix": "__"},
      {"type": ["member",  "method"], "modifiers": "protected", "leadingUnderscore": "require"},

@conmute glad I could help you and sorry for the inconvenience with the typo.

Would be great if some rules could be optimized.

Actually member is already the optimized version which includes property and method. There is no need to specify it again for method.
In addition, you need to slightly modify your config for private to make it work. You need to explicitly disable the leadingUnderscore check by setting it to a falsy value instead of allow. That's because "allow" would slice off the first leading underscore and then the prefix does not match anymore.
{"type": "member", "modifiers": "private", "leadingUnderscore": null, "prefix": "__"},

@ajafff Thank you!

Looks like this is out of scope for the core ruleset and suppoted by tslint-consistent-codestyle. Closing for now but if there's a strong case for including this in TSLint core, please do speak up and we can re-open!

Any chance of at least allowing us to specify named exceptions to the rule -- as in specific variable names (or even types) that may be exempt from any restriction?
In GraphQL, we have __typename, in MongoDB, we have _id, etc. I have no control over how those fields are named, but I still want to type an eventual queried object as a whole, while still applying all the rules that should apply to my project (e.g. underscore prefixes not allowed in types WE create). Some of these fields are still very useful and actively used at runtime, and I would not want to introduce a translation phase just for the sake of circumventing decisions that are external to my project and out of my control.
The naming-convention solution is quite laborious. The solution could be much simpler, e.g. "variable-name": [true, "check-format", ["exceptions", "__typename", "_id"]] (having said that, I'd hate to see another ban-keywords). Please, reconsider!

This may not be useful depending on if you have time to do some refactoring however...
Tslint is being deprecated in favour of integrating with ESLint (article) and ESLint has a rule for no-underscore-dangle, so updating to use typescript-eslint may help you sidestep this issue with TSLint.
I haven't made this change myself as I'm no longer working on the code base that needed the leading underscores, so not sure of the amount of work needed to port over to typescript-eslint but if you really need this rule then it might be the way forward.

Yeah, I moved to eslint a while back too.

I am still facing this issue
tslint.json
{
  "extends": "tslint:recommended",
  "rules": {
    "align": {
      "options": [
        "parameters",
        "statements"
      ]
    },
    "array-type": false,
    "arrow-return-shorthand": true,
    "curly": true,
    "deprecation": {
      "severity": "warning"
    },
    "component-class-suffix": true,
    "contextual-lifecycle": true,
    "directive-class-suffix": true,
    "directive-selector": [
      true,
      "attribute",
      "app",
      "camelCase"
    ],
    "component-selector": [
      true,
      "element",
      "app",
      "kebab-case"
    ],
    "eofline": true,
    "import-blacklist": [
      true,
      "rxjs/Rx"
    ],
    "import-spacing": true,
    "indent": {
      "options": [
        "spaces"
      ]
    },
    "max-classes-per-file": false,
    "max-line-length": [
      true,
      140
    ],
    "member-ordering": [
      true,
      {
        "order": [
          "static-field",
          "instance-field",
          "static-method",
          "instance-method"
        ]
      }
    ],
    "no-console": [
      true,
      "debug",
      "info",
      "time",
      "timeEnd",
      "trace"
    ],
    "no-empty": false,
    "no-inferrable-types": [
      true,
      "ignore-params"
    ],
    "no-non-null-assertion": true,
    "no-redundant-jsdoc": true,
    "no-switch-case-fall-through": true,
    "no-var-requires": false,
    "object-literal-key-quotes": [
      true,
      "as-needed"
    ],
    "quotemark": [
      true,
      "single"
    ],
    "semicolon": {
      "options": [
        "always"
      ]
    },
    "space-before-function-paren": {
      "options": {
        "anonymous": "never",
        "asyncArrow": "always",
        "constructor": "never",
        "method": "never",
        "named": "never"
      }
    },
    "typedef-whitespace": {
      "options": [
        {
          "call-signature": "nospace",
          "index-signature": "nospace",
          "parameter": "nospace",
          "property-declaration": "nospace",
          "variable-declaration": "nospace"
        },
        {
          "call-signature": "onespace",
          "index-signature": "onespace",
          "parameter": "onespace",
          "property-declaration": "onespace",
          "variable-declaration": "onespace"
        }
      ]
    },
    "variable-name": {
      "options": [
        "ban-keywords",
        "check-format",
        "allow-pascal-case"
      ]
    },
    "whitespace": {
      "options": [
        "check-branch",
        "check-decl",
        "check-operator",
        "check-separator",
        "check-type",
        "check-typecast"
      ]
    },
    "no-conflicting-lifecycle": true,
    "no-host-metadata-property": true,
    "no-input-rename": true,
    "no-inputs-metadata-property": true,
    "no-output-native": true,
    "no-output-on-prefix": true,
    "no-output-rename": true,
    "no-outputs-metadata-property": true,
    "template-banana-in-box": true,
    "template-no-negated-async": true,
    "use-lifecycle-interface": true,
    "use-pipe-transform-interface": true
  },
  "rulesDirectory": [
    "codelyzer"
  ]
}

Angular 9
Error message is
variable name must be in lowerCamelCase or UPPER_CASE


Disable health check service in async end2end tests as it is causing flakes on mac

Will re-enable it once issue https://github.com/grpc/grpc/issues/11223 is resolved.

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

CC @jtattermusch


stm32u5 supports the BackuUp sram and erase

Add the support of the BackUp SRAM  to the stm32u5.
Make the  samples/boards/stm32/backup_sram  run on stm32u585 disco or stm32u575 nucleo boards.
Add the option to erase the content of the SRAM when booting with a flag CONFIG_ERASE_BKP_SRAM_AT_BOOT=y
FIxes https://github.com/zephyrproject-rtos/zephyr/issues/54934
Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

When the flag is set :
*** Booting Zephyr OS build v3.3.0-rc3-7-g60f9ccaa781a ***
Current value in backup SRAM (0x40036400): 0
Next reported value should be: 1
Keep VBAT power source and reset the board now!
*** Booting Zephyr OS build v3.3.0-rc3-7-g60f9ccaa781a ***
Current value in backup SRAM (0x40036400): 0
Next reported value should be: 1
Keep VBAT power source and reset the board now!

When the flag is reset :
*** Booting Zephyr OS build v3.3.0-rc3-7-g60f9ccaa781a ***
Current value in backup SRAM (0x40036400): 0
Next reported value should be: 1
Keep VBAT power source and reset the board now!
*** Booting Zephyr OS build v3.3.0-rc3-7-g60f9ccaa781a ***
Current value in backup SRAM (0x40036400): 1
Next reported value should be: 2
Keep VBAT power source and reset the board now!
*** Booting Zephyr OS build v3.3.0-rc3-7-g60f9ccaa781a ***
Current value in backup SRAM (0x40036400): 2
Next reported value should be: 3
Keep VBAT power source and reset the board now!

rebase
remove the erase part


Cannot connect to Chrome

Getting the following error:
Cannot connect to Chrome [Error: unknown command: json/list]
Chrome version: Version 51.0.2704.106 (64-bit)
Since we are using node v.0.12 I stay at the following version. Could this be the cause ?
chrome-remote-interface "version": "0.15.1",

I don't think it could be the cause of Error: unknown command: json/list, but the minimum required version of Node.js is 4.
Anyway, can you show me a minimal script that reproduces this issue?

Node 4 is required for version 16 and above.
I tested with 0.15 to 0.10 and getting the same error.
The script I test it is:
var Chrome = require('chrome-remote-interface');
var connectCrome = function(port){
  Chrome({port: port},function (chrome) {
          with (chrome) {
              Page.enable();
              global.mainProfiler = Profiler;

              Profiler.enable();
              Profiler.consoleProfileFinished(function (params) {
                  // CPUProfile object (params.profile) described here:
                  //    https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/devtools/protocol.json&q=protocol.json%20%22CPUProfile%22,&sq=package:chromium

                  // Either:  1. process the data however you wish… or,
                  //          2. Use the JSON file, open Chrome DevTools, Profiles tab, select CPU Profile radio button,
                  //             click `load` and view the profile data in the full devtools UI.
                  var file = 'profile-' + Date.now() + '.cpuprofile';
                  var data = JSON.stringify(params.profile);
                  fs.writeFile(file, data);
                  close();
              });
          }
      }).on('error', function(e){
        console.error('Cannot connect to Chrome', e);
      })
}

Thx in advance

Getting same error with node v4.4.5 with chrome-remote-interface 0.17 (latest)
Cheking if chromedriver is working on the port I run:
> netstat -plnt | grep 'chromedriver'
(Not all processes could be identified, non-owned process info
 will not be shown, you would have to be root to see it all.)
tcp        0      0 <IP_ADDRESS>:57650         <IP_ADDRESS>:*               LISTEN      15553/chromedriver

So I run it with: connectCrome(57650)

Oh now it's clear! This module is not a WebDriver client, this is a Chrome Debugging Protocol client and it is supposed to communicate with Chrome directly (port 9222 by default) when it is started as:
google-chrome --remote-debugging-port=9222

Oh thanks in advance


Preview or GA?

'Control egress traffic' as described in https://docs.microsoft.com/en-us/azure/aks/limit-egress-traffic#egress-traffic-overview seems to depend on this feature. Since 'limit egress trafic' is GA https://azure.microsoft.com/en-us/updates/egress-lockdown-in-azure-kubernetes-service-aks-is-now-generally-available/ I would assume egress-outboundtype is also GA, unlike what's indicated in this page.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7d721081-0c1c-abf3-a398-386a718446d0
Version Independent ID: 92b5504d-e25a-be94-e5e5-151083a70607
Content: Customize user-defined routes (UDR) in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/egress-outboundtype.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@Karishma-Tiwari-MSFT @mlearned Do you have any update on this? Is outboundType GA already or is it still only part of the preview?

@mrck outboundType is in Preview currently. There is no ETA for GA as of now. I will share here as I get an update on GA timelines.

Hi. Any update on the GA time plan ? Zone redundancy, private cluster and multiple nodepools features require standard loadbalancer and without outboundType feature SLB require public IP address, which is against our security policy.

The outboundtype details can be found on this AKS issue. Could you follow up with us there so you get the latest?
https://github.com/Azure/AKS/issues/1384

@karishma-Tiwari-MSFT can you please close this issue in favor of the tracker I shared previously?

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


"Release not found" error when the commit value is "latest"

A customer reported that "balena preload" stopped working and I believe I have narrowed the issue down to the use of the alias "latest" instead of a commit hash value. I believe I have further identified the issue to have been introduced by the following PR:
https://github.com/balena-io/balena-preload/pull/201/files
Because when I pinned balena-preload to version 8.1.1 in the CLI's package.json, using the "-c latest" option seemed to work OK.

[pdcastro] This issue has attached support thread https://jel.ly.fish/#/support-thread~46eeb6c0-f831-44ed-bb04-befa8aeacf72

@Page-, @zvin, does it sound plausible that the PR I linked in this issue is the cause of the "Release not found" error? Is the fix obvious to you? Thanks!

I think the "latest" alias is only used in the cli https://github.com/balena-io/balena-cli/blob/master/lib/actions/preload.coffee#L288-L291

@pdcastro you're right, the preloader can be created with "latest" here https://github.com/balena-io/balena-cli/blob/master/lib/actions/preload.coffee#L288-L291 , then _fetchApplication is called before this https://github.com/balena-io/balena-cli/blob/master/lib/actions/preload.coffee#L289 and the releaseFilter is incorrect.

A piece of data I forgot to add before: when "--commit latest" was used, balena-preload (or some other repo/SDK used by it) would print on the screen:
Release not found: 1e152b2abe0f7ba8891b32e6ba2fab2a

Meaning that balena-preload (or some other repo/SDK used by it) actually resolved "latest" to "1e152b2abe0f7ba8891b32e6ba2fab2a", yet failed.

We could avoid filtering by commit here https://github.com/balena-io/balena-cli/blob/master/lib/actions/preload.coffee#L289 if the commit is "latest".
Maybe there's a way to only get the latest release in that case?
cc @Page-


Is there any way to use AntiAliasGridFit in .net android?

I wonder if there is any way to use AntiAliasGridFit in .net android ?
I draw text in winform and I can set it as is:

in winform I only set AntiAliasGridFit
but If I use .net android and set
Bitmap bmp = Bitmap.CreateBitmap(309, 36, Bitmap.Config.Argb8888);
Canvas canvas = new(bmp);
canvas.DrawColor(Color.ParseColor("#000000"));
canvas.DrawFilter = new PaintFlagsDrawFilter(0, PaintFlags.AntiAlias | PaintFlags.FilterBitmap);
Paint paint = new()
{
    Color = Color.ParseColor("#ffffff"),
    AntiAlias = true
};
//var arail = Typeface.Create("Arial", TypefaceStyle.Normal);
var arail = Typeface.CreateFromAsset(Assets, "fonts/micross.ttf");//this is the Arial which winform use
paint.SetTypeface(arail);
paint.TextSize = 12;
paint.SubpixelText = true;
paint.FilterBitmap = true;
 canvas.DrawText(str1, 0 + xoffset, rect1.Height(), paint);

it is like this:

the two picture is zoomed.
in winform is sharp right and clear?
Is there any way I can use to draw text like winforms?

Ultimately, all .NET for Android does is provide generated wrappers that allow you to call Google’s Android API. We do not have much expertise in how to use those API’s properly.
You will likely need to use the Android API documentation and relevant forums to determine how to write code for your desired outcome.


SSO does not support user attribute with special character

Describe the issue
User names with special character does not display and will be changed into unknow symbols.
Additional context
RC chat: https://chat.developer.gov.bc.ca/channel/devops-ocp4-migration?msg=DsK3XffSos6L9eR49
Definition of done

investigate into the issue and see if it's version specific
plan for fixing

This item is still an issue. More info on a possible solution: https://access.redhat.com/solutions/4560101

no clients have complained since August 2021


New version: Mozilla.Firefox version 122.0.1

Pull request has been created with komac v2.0.3 :rocket:

Resolve: https://github.com/microsoft/winget-pkgs/issues/138209

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-02-07T10:24:05.1417544Z ##[error] One or more errors occurred. (Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Error: Duplicate installer entry found. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. ) Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Error: Duplicate installer entry found. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended. Manifest Warning: Silent and SilentWithProgress switches are not specified for InstallerType exe. Please make sure the installer can run unattended.

(Automated response - build 695.)

Manifest Error: Duplicate installer entry found

It appears that repeated errors need to be filtered out.

Hi @Exorcism0666,
These installer entries appear to be duplicated. Could they be modified or removed?

cs-CZ-x64-exe-machine
de-DE-x64-exe-machine
el-GR-x64-exe-machine
en-GB-x64-exe-machine
en-US-x64-exe-machine
es-MX-x64-exe-machine
fr-FR-x64-exe-machine
hu-HU-x64-exe-machine
ja-JP-x64-exe-machine
nb-NO-x64-exe-machine
ru-RU-x64-exe-machine
ta-IN-x64-exe-machine
zh-CN-x64-exe-machine

Close with reason: Merge conflicts;


Fix some internal cpdef functions to cdef in _kernel.pyx

cosmetic changes.

Jenkins, test this please

Jenkins CI test (for commit 2358ec15654c18e38d3bc5db5714ddc2c5087904, target branch master) succeeded!


Cloud Build

A sample of cloud build config that we might use.
This speeds up Elixir build substantially by leveraging the layer cache stored in the Google Artifact Registry.
The extra trick which I should we should keep no matter how we build our images: is compiling Elixir in a separate compiler step, which will allow us to greatly reduce concurrent compile time for api/web images because 80% of releasing them is fetching deps and compiling the source.
Also, we can bypass issues with GitHub Actions Cache size by using Google Artifact Registry even if we keep using GH Actions.
Compiling the source without cache (which is shared) takes 4 minutes.
On top of that releasing api/web takes ~32 seconds.
See https://console.cloud.google.com/cloud-build/builds;region=global/8d21c772-b6b7-4308-a580-
d201db517765;step=3?authuser=1&project=firezone-staging.
With a cached source, everything takes 56 seconds:
https://console.cloud.google.com/cloud-build/builds;region=global/d828775b-40b4-47cb-b10f-a6517b72ca4d;step=4?authuser=1&project=firezone-staging
With a change in one of the apps, everything takes 1 minute 43 seconds:
https://console.cloud.google.com/cloud-build/builds;region=global/a66adaff-e7f2-4dff-bf96-fafdcfc0ed5d?authuser=1&project=firezone-staging
To try it yourself run:
gcloud builds submit --config cloudbuild.yaml --substitutions='_REPOSITORY=firezone,_REGISTRY=us-east1-docker.pkg.dev,_API_TAG=local,_WEB_TAG=local' --project "firezone-staging" --timeout "30m"

Things we can improve: fetch and build assets before doing so for Elixir code which changes way more often.

Added rust part with changes pulled from #2233, clean rust build for relay takes 5:46 instead of 15 minutes.
Cache doesn't work, will try to fix it.

Fully cached relay/gateway build takes just 7 seconds: https://console.cloud.google.com/cloud-build/builds;region=global/095e3771-070f-4b9b-a1b9-25a27e1d787e?authuser=1&project=firezone-staging
After I changed one line in the relay source code the deps are cached (hooray!) both relay and gateway are recompiled (unfortunately), complete build takes under 5:48, with ~1:30 spent on gzipping and uploading the rust build cache: https://console.cloud.google.com/cloud-build/builds;region=global/2f285103-a3ec-4f8b-bad5-9b5abc030b71;step=6?authuser=1&project=firezone-staging.
6 minutes for all containers is acceptable, it's as much as our fastest container with GitHub actions.

I think the issue here is the volume of the cache, on a very fast instance it takes about the same time to download the cache as to rebuild it.

To prove the point here is a build with all rust caches disabled: https://console.cloud.google.com/cloud-build/builds;region=global/b4bc0a1c-aa02-4e46-a9f0-dc4ff83105f9?authuser=1&project=firezone-staging
Takes 1 minute less!

@jamilbk trying to cleanup the CI here is what I came up with: https://github.com/firezone/firezone/actions/runs/6401312752. This feels way cleaner to have a workflow file per language and then use them only as part of bigger workflows, allowing more reuse.
I removed conditions for now and we build everything, but I will bring them back alter. This is something we need to discuss: do we disable merge queues and those "fake test pass" or we run all tests all the time making everything much slower.
cc @conectado @thomaseizinger

@jamilbk trying to cleanup the CI here is what I came up with: https://github.com/firezone/firezone/actions/runs/6401336878 This feels way cleaner to have a workflow file per language and then use them only as part of bigger workflows, allowing more reuse.
I removed conditions for now and we build everything, but I will bring them back alter. This is something we need to discuss: do we disable merge queues and those "fake test pass" or we run all tests all the time making everything much slower.
cc @conectado @thomaseizinger

@AndrewDryga The merge queue had nothing to do with those fake pass-checks workflows. Those were required because we're using path-based conditions along with "Require status checks to pass" -- some workflows would never be triggered if their paths hadn't changed, making the PR unmergeable.
Removing those path-based conditions means all our workflows run on each PR, which I don't think is necessarily a bad thing if our minutes are free. Running the Gcloud builder on each PR could get expensive fast though, hence one of the reasons for wanting to keep this in GH for now.

I'm seeing similar speeds with the GitHub runners -- now that the CI caching is fixed, API/Web builds take about 30s as well, under a minute for the full thing:

That's great, I think then we should keep building on GitHub Actions, especially because we can use a larger runner anytime we want.

I appreciate the work that went into this, and I think we definitely might use cloud build in the future for more building our production containers in CD, but I'm not sure it's wise a good idea to introduce another build system into our stack right now.

I agree, if it doesn't save at least 50% of our time it's not worth to maintain it.

Also, I think you need a GCloud account in our organization to view these logs?

Not at all, we can use a custom bucket for logs and then they will be streamed to GitHub console output.

GCloud builders could be used for our prod images (api/web/relay/gateway) perhaps later on.

Let's see how long it will take to do a multi-arch build for our rust targets. I feel like we will have to do bigger instances to get any meaningful time there, but with GitHub we can also try to build them on native platforms. The only massive disadvantage I see here for now is the very limited cache size, but we can totally link our GitHub Actions caches to the Google Artifact Registry.

Removing those path-based conditions means all our workflows run on each PR, which I don't think is necessarily a bad thing if our minutes are free. Running the Gcloud builder on each PR could get expensive fast though, hence one of the reasons for wanting to keep this in GH for now.

Yeah, that's what I meant. Only I worry about developer minutes more than runner minutes, because having somebody to wait for 10 minutes more to run all the tests is not a great feeling, especially if you need to do it at least 3 times to deploy a hotfix to production. I would always spend any money on fast CI not having developers waiting an extra minute.

GCloud builders could be used for our prod images (api/web/relay/gateway) perhaps later on.

Let's see how long it will take to do a multi-arch build for our rust targets. I feel like we will have to do bigger instances to get any meaningful time there, but with GitHub we can also try to build them on native platforms. The only massive disadvantage I see here for now is the very limited cache size, but we can totally link our GitHub Actions caches to the Google Artifact Registry.

Buildkit also has an S3 backend that you can push caches to: https://docs.docker.com/build/cache/backends/s3/
We use this for our interoperability tests in libp2p: https://github.com/libp2p/test-plans/blob/master/multidim-interop/dockerBuildWrapper.sh

I don't know whether that works with the github action but I'd be surprised if that one does any additional validation on which cache options you are allowed to pass. According to that page, using GitHub actions cache is also experimental.

Worst case, we can just ditch the action and invoke buildkit ourselves.


Fixes issue #29

I have invited you to the as3hx group so you can commit directly. Please try to add a test for changes you make so we can stabilize this tool over time.
If you're unsure about a fix please open a PR. We'll look into it eventually.

Please use the "squash and merge" option for a cleaner history.

Ok, Simon thanks, I prepared 50 PR and tests for them. I am very interested in improving library.

@Simn "squash and merge" is not available

Huh, you're right. That's strange because it's enabled in the settings. Let me check...

Ah, it shows up after you click the button once.


e2e test assert on curl status code

Describe the problem
Current e2e tests assert on a substring of the response payload, we should also assert on the status code of the response.
Describe the proposed solution
assert on response status code
Alternatives considered
leave as is.
Importance
would make my life easier

The current tests assert content output which technically covers the status codes.
curl does support a flag --fail-with-body which exits with status code 22 when response code is >=400. Unfortunately, this is only supported in curl version 7.76.0 which is not included in the GitHub Actions Runner - https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-README.md
Because of all this, I have decided the effort to actual implement this is not worth it for the small value gained.


Adding brew_tag option

@thiagoalessio @sosiouxme @tbielawa
Adding brew_tag option
See RCM-69565 for specific details.
Basically, doozer assumes (and has hard-coded) that the brew tag will always match the same pattern. This is only true for OpenShift and the rest of the company just sort of does whatever. So this allows an override.

/lgtm

/lgtm


feature: file type icons

Codecov Report

Merging #294 (c4d4442) into dev (ef9231c) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #294      +/-   ##
==========================================
- Coverage   92.50%   92.44%   -0.07%
==========================================
  Files         128      128
  Lines        3295     3295
  Branches     1096     1096
==========================================
- Hits         3048     3046       -2
- Misses        112      113       +1
- Partials      135      136       +1

Impacted Files
Coverage Δ

...act-magma-dom/src/components/Dropdown/Dropdown.tsx
82.60% <0.00%> (-2.90%)
: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 ef9231c...c4d4442. Read the comment docs.


Using 3rd party driver crates with embassy

Hello,
I'm considering porting an existing rust embedded project to embassy and would like to know if embassy has any support for driver crates or should I rewrite them.
From what I understand embassy doesn't seem to follow all of embedded-hal. If so is there any plan to port the existing drivers to be compatible with embassy ?

Embassy HALs implement most embedded-hal and embedded-hal-async traits, so most drivers should work as-is.


fix(gateway): dyanmically pick docker stats source

Description of change
This is a possible solution to dynamically get the cpu stats to return the ability to run the gateway on other systems without cgrpoup v1 that was lost in #1464
This is a draft, the test suite has not been run and I don't think tests will even build. I wanted to see if this was a direction that the shuttle team felt like it should go and get feedback before polishing it.
How has this been tested? (if applicable)
Ran locally on a Mac. Builds, runs, and can build and deploy new projects. I don't know how else to test it, but I assumed the service would get a fatal error during this process.
I did not test on a Linux system but left the parser used for cgroup v1 and took the example that was commented out for v2

Glad it worked! Ahh, I think it must have been me using cgroupv2 with the unwraps. I was able to get the errors I was expecting with these changes and hardcode it to a cgroup. Now, it should be good to go!

I moved the Docker stats source from main.rs to init in service.rs. Additionally, I changed init to a result so that I could generate an error if the crgroup path was not found in staging or production.
The only other solution I had was to check the environment variables and file locations in main.rs so it could return as soon as possible, but I decided to go with this solution to avoid performing that check twice. Please let me know if you would like anything changed.
Thank you!


Integration through Heroku

Can we integrate this code in Heroku. Thank you

Each person would have to do this for themselves as they would have to set their own personal credentials. Feel free to clone the code and do it if you'd like! The code is free to use.


Simple / Basic http auth not working

I'd like to regularly check a few sites for software updates. Unfortunately, that info is behind a simple auth login.
There seems to be no way to get around it<EMAIL_ADDRESS>seems to be deprecated.
The<EMAIL_ADDRESS>as suggested in #372 doesn't work, either.
And when I use browser steps, the fields are in another window, so I can't see it within playwright.
Does anybody have an idea what I can do to get around this problem?

issue template not followed - no version/fetch type etc


Fix the failing GH workflows

Purpose of this pull request
This pull request (PR) fixes and updates the stage, publish, github-pages, and index workflows.

Use actions/setup-node instead of borales/actions-yarn
Upgrade actions/cache to 3.3.2
Move the node version to .nvrmc to be able to use it locally too.
Upgrade node to the latest minor version of v16 (16.20)
Move NODE_OPTIONS to the scripts to be able to use it locally too

Internal ref: COMDOX-866
Testing
Staging: TBD
Pages: TBD
Indexing: TBD

@keharper please fix the broken link.


ERROR in DateTimePickerModule is not an NgModule

hello,
while importing this plugin into my app.module.ts gives me the following error:
ERROR in DateTimePickerModule is not an NgModule.

Same for me with version 2.0.0 and higher.
Version 1.0.7 compiles without problem.
I am using angular/cli with webpack.

@jigsshah @jurafa
I will check for it...
@jigsshah are you running it using SystemJS or Webpack?

@jigsshah @jurafa
I looked for the problems you both mentioned.
You may running your app with Angular JS Version 4.0.0. That may be the reason getting the error:
ERROR in DateTimePickerModule is not an NgModule.
Currently, this package only supports the AngularJS Version 2..
@jigsshah
Are you using this angular/cli. It is running the Angular JS Version 4.0.0. That is the reason getting the error.
Thanks guys. I will update this package to Angular JS 4.0.0 today, and let you guys know when it is completed.

@jigsshah @jurafa
I looked for the problem you mentions. And I found some packages building with Angular 2 have the some problem too. I found this one.
I will look at it and try to fix it today.
Thanks guys.

Hello Daniel,
First of all thanks for you looking to issue and providing your time.
My Angular version is angular/compiler-cli": "^2.3.1" and i am using SystemJS Package Loader

@jigsshah
Yes, I found the problem similar to other angular2 package like this one. I am working on it right now.

Hello Daniel,
Can you let me know if we can have or use the existing plugin in the inline mode also as like bootstrap one like.
Thanks

@jigsshah @jurafa
Guys, I fixed the problem. Please update the package to version 2.0.8.
@jurafa
If you running the package in the latest angular-cli, this package still would not work. This is because the latest angular/cli is running the AngularJS 4.0.0, and AngularJS 4.0.0 animation has its own module (not in @angular/core any more). This causes the package crash. I would create a new ng4-date-time-picker package.

Thank you for your time.
I am using Angular 2 with WebPack. I will try the fix soon.
BTW: The naming conventions have slightly changed and naming package ng*-date-time-picker is not recommended anymore according to this: http://angularjs.blogspot.cz/2016/12/ok-let-me-explain-its-going-to-be.html (Quote: "Also, we should start avoiding GitHub/NPM libraries prefixed with ng2- or angular2-.")


Unable to change OPENAI_BASE_URL in .sgptrc

This line present in default sgptrc:
OPENAI_BASE_URL=https://api.openai.com/v1

I change that line, but that has no effect.
I can see that for example in error messages (api_base = 'https://api.openai.com/v1').
And that is no surprise, the value in config seems unused https://github.com/search?q=repo%3ATheR1D%2Fshell_gpt OPENAI_BASE_URL&type=code

Hi @johnd0e, thank you for reporting this issue. I can confirm that the OPENAI_BASE_URL is not being used after the latest release. We will address this in upcoming updates.


Update action to use the default branch

Default branch should already have the fixes for using fossa test.

Thanks, @cmboling, sorry to have missed this. We have a policy of pinning action versions for security and stability. 🙏


Agency Admins and Officers can see all boarding records, vessels and crew when browsing

Describe the bug
Agency Admins and Officers can see all boarding records, vessels and crew when browsing. Clicking for further details does not show them.
To Reproduce
Steps to reproduce the behavior:

Login as an agency administrator or field officer
Click "Boarding Records" and then any of the items, e.g. Crew
You will see all the crew (no difference between that and doing it as a global administrator)
If you try to click on a crew member you don't have permissions for, you get the crew page with a lot of N/A.

Expected behavior
Items should not show up on those pages if I don't have permission to see it.

This was fixed Dec. 5th, internally in the Realm functions - they needed to be set to "Application User Authentication" or similar - not System Authentication.


Cosmos: Use retrying strategy for database and container operations

At least in tests
Common errors:

Microsoft.Azure.Cosmos.CosmosException : Response status code does not indicate success: 503 Substatus: 1007

Microsoft.Azure.Cosmos.CosmosException : Response status code does not indicate success: 429 Substatus: 3200 Reason: ().

System.InvalidOperationException : Create container failed. Status code: TooManyRequests. Message: ''

It seems that the operations are being retried, but just hit the retry limit. We should either clean and reuse the containers or switch to emulator.


Updated package_examples.rb to reflect the changes to Swift packaging

/cc @segiddins

I aborted the CI build because it never calls package_examples.rb.

This isn't the proper fix, so I'm marking this PR as WIP until I can find an appropriate solution.

I've found the proper fix now, @segiddins please review again. I'll let the CI PR job run this time because I had to make changes to the Swift examples Xcode project.

:+1:


feat: support Etherscan API keys

Explanation
Support the use of API keys when polling for incoming transactions in the TransactionController.
Provided by the client via the optional incomingTransactions.etherscanApiKeysByChainId property, since API keys may be exclusive to specific chains, based on the Etherscan host / domain.
References
Changelog
@metamask/transaction-controller

ADDED: Add optional incomingTransactions.etherscanApiKeysByChainId constructor property to support API keys in requests to Etherscan.

Checklist

[x] I've updated the test suite for new or updated code as appropriate
[x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[x] I've highlighted breaking changes using the "BREAKING" category above as appropriate
[x] I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@metamaskbot publish-preview


HSEARCH-4236 + HSEARCH-4237 No cleanup of threads, if an connection exception during mass indexation happens

HSEARCH-4237: Work around Surefire clearing the interrupt status when forwarding stdout/stderrn
HSEARCH-4236: No cleanup of threads, if an connection exception during mass indexation happens

Applied your suggestions and merged. Thanks!


Help wanted! onMove identify if dragged element is over related or not

Thanks a ton for amazing library.
While user is dragging an item from one list to other, I would like to know if there is a way to identify the dragged element is over the related element or not. To be specific I want to identify a 'dragleave'; when user is no longer hovering over a list and hanging outside so that I can put the dragged element back to the original list(otherwise if I simply leave the mouse click the dragged element will be added to related element even when the actual list item was out of sight).

@sumitg-3767 See issue #1468

Hi @owen-m1, Thanks a lot for the quick update. I did saw the issue #1468 earlier however that does not respect the pull/put configurations of sortable js. consider a scenario I have 3 list. List 1 only accepts (put:['list2']) items from list 2, though if I try to drag an item from list 3 to list 1 it would highlight list.

@sumitg-3767 Try doing
Sortable.active.group.checkPull(put, Sortable.active) && put.group.checkPut(put, Sortable.active)

where put is the Sortable you are hovering over. This will return a boolean and you can set the color if it is true.

@owen-m1
That does work indeed.
A minor syntax change needed as enclosed
Sortable.active.options.group.checkPull(put, Sortable.active) && put.options.group.checkPut(put, Sortable.active)
note:
put is an instance of type Sortable


Provide checksums for releases

There are no checksum generated for the binaries at the moment. This could be done by the docker containers which generate the builds.

Initial support added, but signed checksums are still missing.

Done. See #339


Fixed table header broken when cells have "position: relative" content

When cell content uses position: relative, its painted on top of the fixed header.
For example, when the sample table looks like this:
<virtual-scroller #scroll
      [items]="filteredList">

			Index
			Name
			Gender
			Age
			Address

{{item.index}}

{{item.name}}

{{item.gender}}

{{item.age}}

{{item.address}}

    </virtual-scroller>

... then it's rendered like:

Interesting.
I'm not sure the cause or the correct fix here.
My first guess was the useMarginInsteadOfTranslate flag might help (because CSS transform changes the behavior of position: relative). However, there's 2 issues with that. 1) the  isn't underneath the , so I'm not sure that's the root cause of the bug (although table's have weird rules). 2) When we originally coded fixed headers we tried margin instead of translate & it had no effect on  so the only way we could find to get a working fixed header was with translate.
I almost found a hacky workaround, but it's not quite working. If you add "z-index: -1;" to the div which has position: relative, it fixes it. The text goes behind the header. However, that only works if you do it from the chrome inspector. If you do it in code, nothing renders at all. I haven't tested in other browsers. My guess is there's some layout stuff that gets skipped, so we'd have to use javascript to add the z-index after the items are first rendered.
Maybe do something like this?
table.alreadyRendered tbody td div {
	z-index: 1;
}

and trigger [class.alreadyRendered]="true" after some event/timeout?
Another option that comes to mind is modifying the math inside ngx-virtual-scroller to render 1 fewer item if the fixed header is enabled. I think when we originally coded it we didn't bother with that because the header seemed to render on top of the 1st row so it didn't matter.
Thoughts?
Hope you can come up with a decent workaround for your app. If you find something that works well & you can make it generic enough for everyone else, please submit a Pull Request.
Thanks.

Fixed header now uses translate, but I think that it's better to use position: sticky. In the similar case it worked fine

@kykint Do you want to code this & submit a PR? We should also add a sample with table & position: relative to the demo as a test-case so we can prove it's fixed.

For now, in my use case I decided to use two tables: just header, then another (scrollable) with just body. It works for me, but in the general case it's getting tricky wrt column width synchronization.

I'm still trying to validate this in other browsers, but it seems like swapping the thead and the tbody in the HTML fixes this issue.
<virtual-scroller #scroll       [items]="filteredList">

{{item.index}}

{{item.name}}

{{item.gender}}

{{item.age}}

{{item.address}}

 			Index	 			Name	 			Gender	 			Age	 			Address

     </virtual-scroller>

@natelokers Interesting idea. I'm guessing that's similar to setting the z-index for thead higher than tbody. I've tested on chrome/firefox on windows & it fixes it for both of them. Edge/IE don't seem to work with ngx-virtual-scroller tables at all, regardless of thead/tbody order or position static/relative.
Maybe we should adjust the readme to suggest this approach.

I merged a PR which partially fixes the IE/Edge issue. https://github.com/rintoj/ngx-virtual-scroller/pull/282
@natelokers feel free to submit a PR with the demo & readme modified to suggest putting the thead after the tbody. Although we haven't tested every browser, I'm guessing this will fix it for most of them.

I'm still trying to validate this in other browsers, but it seems like swapping the thead and the tbody in the HTML fixes this issue.
<virtual-scroller #scroll
      [items]="filteredList">

{{item.index}}

{{item.name}}

{{item.gender}}

{{item.age}}

{{item.address}}

			Index
			Name
			Gender
			Age
			Address

    </virtual-scroller>

this really helps. Thanks!

@konrad-garus @bbv-hungtran  Do you still have the problem ? The issue is 2 years old

The problem is still in version 4.0.3.


[FEATURE] Continuously persist log files versus only at the end of a scan

See #497 for more context

Implemented in version 2.14


dtls/udp: ensure unique message IDs during reconnections

To prevent collisions during reconnections, it is important to always increment the global counter. For example, if a connection (cc) is established and later closed due to inactivity, a new cc may be created shortly after. However, if the new cc is initialized with the same message ID as the previous one, the receiver may mistakenly treat the incoming message as a duplicate and discard it. Hence, by incrementing the global counter, we can ensure unique message IDs and avoid such issues.

Codecov Report

Merging #456 (71c4e8b) into master (bbd146e) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #456      +/-   ##
==========================================
- Coverage   72.01%   71.96%   -0.06%
==========================================
  Files          70       70
  Lines        5414     5422       +8
==========================================
+ Hits         3899     3902       +3
- Misses       1123     1127       +4
- Partials      392      393       +1

Impacted Files
Coverage Δ

udp/client/conn.go
69.24% <100.00%> (-0.14%)
:arrow_down:

... and 10 files with indirect coverage changes


Help needed with with installation. thanx

I wanted to install this node on my Rasparian Pi 4 with HASSIO and Nodered.
The following error occured... im kinda newbie with this but definately does not look good.
hassio version core-2021.8.6 latest Node Red. etc.
 System Linux 5.10.17-v8
2021-08-11T12:05:50.053Z [err] npm
2021-08-11T12:05:50.053Z [err]  ERR! gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
2021-08-11T12:05:50.053Z [err] npm ERR!
2021-08-11T12:05:50.053Z [err]  gyp ERR! cwd /config/node-red/node_modules/@serialport/bindings
2021-08-11T12:05:50.053Z [err] npm ERR!
2021-08-11T12:05:50.053Z [err]  gyp ERR! node -v v14.17.1
2021-08-11T12:05:50.053Z [err] npm ERR! gyp ERR! node-gyp -v v7.1.2
2021-08-11T12:05:50.054Z [err] npm ERR! gyp ERR! not ok
2021-08-11T12:05:50.176Z [err]
2021-08-11T12:05:50.176Z [err] npm ERR! A complete log of this run can be found in:
2021-08-11T12:05:50.176Z [err] npm ERR!     /root/.npm/_logs/2021-08-11T12_05_50_060Z-debug.log
2021-08-11T12:05:50.212Z rc=1`

hi @djalexnl
based on the environment info you have shared it looks like the prebuilds wont work for you so you will need to have the build tools installed and configured for your environment.  There are instructions for installing on Raspberry-pi at https://serialport.io/docs/guide-installation#raspberry-pi-linux
If there are any steps in the instructions which are unclear or could be improved then do please let us know


feat: add permissions attr to Member

Pull Request Type

[x] Feature addition
[ ] Bugfix
[ ] Documentation update
[ ] Code refactor
[ ] Tests improvement
[ ] CI/CD pipeline enhancement
[ ] Other: [Replace with a description]

Description
Quick PR to add in permissions to Member, as that's given in interaction objects and can be useful.
Changes

Add permissions to Member.

Related Issues

Test Scenarios

Python Compatibility

[ ] I've ensured my code works on Python 3.10.x
[x] I've ensured my code works on Python 3.11.x

Checklist

[x] I've run the pre-commit code linter over all edited files
[x] I've tested my changes on supported Python versions
[ ] I've added tests for my code, if applicable
[x] I've updated / added  documentation, where applicable

The shouldnt be in the member object as this is instance-specific, not guild specific. Imo the context object should intercept and store this data, that way its ephemeral just like this data is

...where then? Do note that this is given for member objects in options too.

I said? The context object

Do you... want a mapping? Like ctx.permissions[member]? That doesn't sound that great for UX, those I suppose that could simplify channel permissions.

I have an idea. Give me 24 hours to draft something

Do we statically dispatch events, or are events aware of changes while being processed? If it's static – a mapping would work well, we'd just need to be considerate of indexing bounds

Closing this PR early as regardless of what happens, this likely isn't the way this feature will be implemented.


Raspotify is not visible.

Hello!
I installed raspotify on Debian. After installation i tried to connect from Android app, but i can't see my device. Anyone knows solution?

are you in the same wlan as your raspotify ?

if it doesn't work you could also try librespot-java , i wrote a wrapper . It works with all debian based OS systems.
https://github.com/spocon/spocon

A moment ago, I solved this problem. I just reinstalled Raspotify. Thank for reply.


Fails to launch

Creating an API object using
from novelai_api import *

api = NovelAI_API(1, 'properUsername', 'properPassword')

fails with this error:
...\venv\lib\site-packages\novelai_api\NovelAI_API.py:42: RuntimeWarning: coroutine 'High_Level.login' was never awaited
  self.high_level.login(username, password)
RuntimeWarning: Enable tracemalloc to get the object allocation traceback

I don't have any experience with async, and the lack of documentation makes it hard to figure out what's going on.

It's working for me now.
This was down to my lack of experience with the used libraries. I minimalist example would be neat for less experienced users like me, though.
Closing due to having solved it.


Update to listunspent cli command

Add missing description (spendable) for listunspent help message.
Add ps_round (PrivateSend Round) in listunspent result.

👍
But maybe s/round/rounds/ ?


Users cannot see other drawings

When using cocreate with a group of around 10 people two of them reported not being able to see any of the other's drawings. They could see each other's and we could see their annotations (including drawing over some of ours). They reported trying to switch browsers and still having an issue, but I failed to collect browser and OS information.

Unfortunately, this isn't enough information to track anything down. What date/time did this happen? Browser information would also be helpful.

This occurred between 10:30am and noon ET on 9/28/2020. The specific events when we noticed and tested were very roughly around 11am, but the problem was never resolved during the meeting. I'll try to find a good way to ask about browser info.
Unrelatedly, we seemed to be experiencing significant lag around 11:50am.

Weird. Server-side I only see around a brief 4-second delay at 11:50am, and don't see anything that should have caused anything after 10:30am. Maybe their network connection briefly disconnected, and that triggered some kind of bad state? I would have expected reconnection to work just fine, but haven't tested thoroughly.
I also restarted the servers in case there was something server-side, but I'm afraid I won't be able to figure this out. Thanks for the report, though; it's good to know something bad is happening. Maybe with more reports I'll eventually find the problem.

FYI, I've now seen this affect another person, and I got to see a screenshare to see what the behavior was like. Conclusions:

User-level workaround: the unfortunate user can reload the Cocreate page to get updates (provided they don't have another tab opened to the same Cocreate page), but it will continue to fail to update live. 🙁
System-level workaround: restarting all the Meteor servers fixes the problem. But only the sysadmin can do this. 🙁 But let me know if you need me to do this again.
I believe the issue is the server losing connection to Redis, reconnecting, but (maybe because of a race condition) some of Redis Oplog's subscriptions failing to be in a correct state. See https://github.com/cult-of-coders/redis-oplog/issues/365 for my analysis so far.

This has also happened to me a few times, my professor has lectures with about 8 people on the cocreate and about 1/4th of the time, I can't see edits until I refresh. Though I think it happens to me more frequently than other people. I tried clearing cookies and changing the browser from chrome to firefox, it did not change the behavior.

I experienced the same issue yesterday. The weird part of it was that I could see my edits live on a different device, my collaborator could see mine and his edits, again on a separate device,  but I could not see his.

@Etn40ff That's expected behavior, I'm afraid. What's happening is that Cocreate "partitions" into multiple (between 2 and 4) "subnetworks", each of which properly updates within itself, but updates between the subnetworks don't propagate (without manual refresh). I still don't know how it happens, so don't know how to permanently fix it. The only fix I've found so far is to restart the Cocreate servers. 🙁

This sounds quite bad but at least it is an explanation.
Currently, since I use it only with another person, I am piggybacking on the instance you run at MIT, I might have to look into self hosting if this happens again.
Thank you for the explanation (and for the software!)


docs: show examples of using the key.WithHelp() for keybinds

none of our Bubble Tea examples show how to use the key.WithHelp() in the description of the help menu. Would be good to show for people who are using keybinds in their programs so the help menu isn't being manually generated.

would like to take this up

So, should i pick an example from the existing list and add code to it or create a new example to demonstrate this ?

You can definitely use an existing example (our list examples are often referenced) and copy it to a new example specifically for key.WithHelp

Sure Bunni !

Hey bunni ! Seems like the demo of key.WithHelp() is already available at examples/help/main.go. Can I create a new example that uses key.WithHelp in it ?

Hey @titanventura
ah geee, it looks like they implemented it when I brought it up internally. I'll have to close this issue since it seems it has already been implemented. If you're looking for more good first issues, I'd recommend taking a look at Soft Serve or Skate if you're struggling to find some for Bubble Tea.
Sorry about the confusion!


Add auth key support

What's needed?
We need to add an auth key to every request for authorization.
Proposed solution
Make the BaseApiClient take a key in the constructor and automatically pass the keys to stub methods.
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response

Example implementation:

https://github.com/frequenz-floss/frequenz-client-electricity-trading-python/pull/29/files


Test Issue created at Sun, 09 Jan 2022 02:27:35 GMT

Test issue body  Sun, 09 Jan 2022 02:27:35 GMT

Comment on issue at Sun, 09 Jan 2022 02:27:38 GMT