[AIRFLOW-248] Add Apache license header to all files

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-248

Testing Done:

No new tests were added

Reminders for contributors:

Your PR's title must reference an issue on Airflow's JIRA. For example, a PR called "[AIRFLOW-1] My Amazing PR" would close JIRA issue #1. Please open a new issue if required!
Please squash your commits when possible and follow the 7 rules of good Git commits.

Added Apache license header for files with extension (.service, .in, .mako, .properties, .ini, .sh, .ldif, .coveragerc, .cfg, .yml, .conf, .sql, .css, .js, .html, .xml.
Added/Replaced shebang on all .sh files with portable version - #!/usr/bin/env bash.
Skipped third party css and js files. Skipped all minified js files as well.

@bolkedebruin The failures are only in certain environments and not consistent. I suspect it is not because of my changes. Can you please confirm? Thanks!

Failed test is:
[2016-06-16 10:40:07,091] {sequential_executor.py:40} INFO - Executing command: airflow run test_depends_on_past test_dop_task 2016-01-06T00:00:00 -I --local -sd DAGS_FOLDER/test_issue_1225.py
[2016-06-16 10:40:07,662] {__init__.py:50} INFO - Using executor SequentialExecutor
Logging into: /home/travis/airflow/logs/test_depends_on_past/test_dop_task/2016-01-06T00:00:00
[2016-06-16 10:40:11,787] {__init__.py:50} INFO - Using executor SequentialExecutor
Traceback (most recent call last):
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/bin/airflow", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/travis/build/apache/incubator-airflow/airflow/bin/airflow", line 15, in <module>
    args.func(args)
  File "/home/travis/build/apache/incubator-airflow/airflow/bin/cli.py", line 247, in run
    run_job.run()
  File "/home/travis/build/apache/incubator-airflow/airflow/jobs.py", line 174, in run
    self._execute()
  File "/home/travis/build/apache/incubator-airflow/airflow/jobs.py", line 1112, in _execute
    self.heartbeat()
  File "/home/travis/build/apache/incubator-airflow/airflow/jobs.py", line 156, in heartbeat
    session.commit()
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 801, in commit
    self.transaction.commit()
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 392, in commit
    self._prepare_impl()
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 372, in _prepare_impl
    self.session.flush()
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 2019, in flush
    self._flush(objects)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 2137, in _flush
    transaction.rollback(_capture_exception=True)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/util/langhelpers.py", line 60, in __exit__
    compat.reraise(exc_type, exc_value, exc_tb)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/session.py", line 2101, in _flush
    flush_context.execute()
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/unitofwork.py", line 373, in execute
    rec.execute(self)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/unitofwork.py", line 532, in execute
    uow
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/persistence.py", line 170, in save_obj
    mapper, table, update)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/orm/persistence.py", line 706, in _emit_update_statements
    execute(statement, multiparams)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 914, in execute
    return meth(self, multiparams, params)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/sql/elements.py", line 323, in _execute_on_connection
    return connection._execute_clauseelement(self, multiparams, params)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1010, in _execute_clauseelement
    compiled_sql, distilled_params
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1146, in _execute_context
    context)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
    exc_info
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/util/compat.py", line 202, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb, cause=cause)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1139, in _execute_context
    context)
  File "/home/travis/build/apache/incubator-airflow/.tox/py27-cdh-airflow_backend_sqlite/lib/python2.7/site-packages/sqlalchemy/engine/default.py", line 450, in do_execute
    cursor.execute(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) database is locked [SQL: u'UPDATE job SET latest_heartbeat=? WHERE job.id = ?'] [parameters: ('2016-06-16 10:40:13.210089', 3)]
[2016-06-16 10:40:18,329] {sequential_executor.py:47} ERROR - Failed to execute task Command 'airflow run test_depends_on_past test_dop_task 2016-01-06T00:00:00 -I --local -sd DAGS_FOLDER/test_issue_1225.py ' returned non-zero exit status 1:
[2016-06-16 10:40:19,365] {jobs.py:968} ERROR - Task instance ('test_depends_on_past', 'test_dop_task', datetime.datetime(2016, 1, 6, 0, 0)) failed
[2016-06-16 10:40:19,365] {jobs.py:1041} INFO - [backfill progress] | waiting: 0 | succeeded: 0 | kicked_off: 1 | failed: 1 | skipped: 0 | deadlocked: 0
[2016-06-16 10:40:19,366] {base_executor.py:92} DEBUG - 0 running task instances

Indeed, this has nothing to do with you. Are you rebased to head right now?
I'm going to send an email to dev@ to notify of this, since it will break a ton of PRs due to silly merge conflicts. I will give 1d, and circle back tomorrow to merge this in.

@criccomini Thanks for looking into it. I just tested applying the commit on latest master and it applies without any errors.

Oops, looks like merge conflicts. Likely after @jlowin's AIRFLOW-31 changes.

Whoops I apologize I didn't think about the license overlap. Appreciate all the work @ajayyadava -- let's merge all the others!

No worries @jlowin @criccomini I have rebased the pull request with master and there are no conflicts now, verified that there are no code/import changes after rebase.

[Current coverage][cc-pull] is 63.87%

Merging [#1598][cc-pull] into [master][cc-base-branch] will increase coverage by 0.06%

@@             master      #1598   diff @@
==========================================
  Files           120        120
  Lines          8449       8465    +16
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           5391       5407    +16
  Misses         3058       3058
  Partials          0          0

Powered by Codecov. Last updated by [7c0f837...0bf5fa3][cc-compare]
[cc-base-branch]: https://codecov.io/gh/apache/incubator-airflow/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/apache/incubator-airflow/compare/7c0f8373f59b0554d5ba15bb0e5e8669f0830313...0bf5fa3b341c7673eb9883242f6630c49fd947af
[cc-pull]: https://codecov.io/gh/apache/incubator-airflow/pull/1598?src=pr

Looking good. I think @criccomini will merge after notifying folks.

Hi @criccomini @jlowin, it seems that the branch again went out of sync because of recently committed changes. 😞  I will rebase the branch again, if you could prioritise this pr, it will be immensely helpful.

I have rebased the pull request. The test failure seems to be an arbitrary failure.

Argh @ajayyadava, I'm so sorry that this keeps falling out of sync! Can you msg me on Gitter when it's ready to be mergd?

(and to any of the other committers: feel free to merge if you see that it doesn't have any conflicts)

One thing I wonder with the html templates if the license will now occur multiple times in the generated html? I think that would be undesirable? Also the commit message itself should refer to the Jira.
Thanks btw @ajayyadava !.

@bolkedebruin Thank you for the keen review. Both of your observations are spot-on and thanks for catching them. I will fix the commit message. As for the license appearing multiple times in generated html we can not avoid putting license headers in child copies, so I think we should change syntax from html comment to jinja comment. Let me know if you have something better in mind.

@bolkedebruin While debugging, I noticed that the pr tool does prefix the jira to the commit message as it picks the commit message from the PR and not the commit message. Though I still agree with you that the commit message should also contain the JIRA id (will help committers if they choose to apply manually)
Also, if you don't select the squash option while using the PR tool, it not only messes the commit message (puts message as "Merge pull request #.... from ..."), it also doesn't retain the original author of the commit. I will raise a JIRA and send a PR for this.
CC @criccomini

@bolkedebruin I have fixed the commit message and also fixed the issue of multiple license header in the generated html. I verified the license in generated HTML by viewing page source of a page in each of the tabs of the ui.
Please review and let me know.

LGTM merged!

Thanks @ajayyadava!


*: support log

When we want to debug grpc, its log will output to stderr which is not useful for deployment. So this pr provides a function that can redirect grpc log to the standard rust log.

Must user initialize a log to use gRPC lib now?

No. User can use redirect_log to integrate grpc log into their log system. Or do nothing, the library will work just the same way as before.

LGTM

ping


Move #then/yield_self to Kernel

This example should work based on Ruby's spec.
If we have an interface with requires_ancestor { Kernel }, we should be able to access then/yield_self when working with the interface.
Motivation
The signature for #then and #yield_self are in the wrong place.
It should live within Kernel based on Ruby's spec.
Test plan

I moved the exiting test to Kernel and deleted the Object file as it was the only test that was there.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_LnD23XutvavdEd
→ https://go/builds/bui_LnD253o4iiRmXU

itself and object_id also belong in Kernel btw:
> Object.method(:itself).owner
=> Kernel
> Object.method(:object_id).owner
=> Kernel

Moreover:
$ ruby -e "p Object.instance_methods(false)"
[]

i.e. Object has no instance methods defined on itself.

Cool yeah -  I can open a new PR with these ones.
Thanks for the heads up! 😸

Hey @paracycle , I don't see those methods declared here though.
Maybe I'm missing something? 🤔

Hey @paracycle , I don't see those methods declared here though. Maybe I'm missing something? thinking

Those methods are defined in the C source code:

object_id
itself

Hey @froydnj , just to make sure I'm trying to do the right thing.
It looks like to generate those filed, all I have to do is to run gems/sorbet/test/hidden-method-finder/update_hidden_methods_exp.sh, right?
I'm getting some weird ssl error when I try to do so
: > revision.tmp
/usr/bin/ruby --disable=gems ./tool/file2lastrev.rb -q --revision.h --srcdir="." > revision.tmp
./tool/ifchange "--timestamp=.revision.time" "./revision.h" revision.tmp
./revision.h unchanged
# rm -f revision.h
# : > revision.h || : > revision.h
/Library/Developer/CommandLineTools/usr/bin/make -f exts.mk  RUBY="./miniruby -I./lib -I. -I.ext/common " top_srcdir="." note
*** Following extensions are not compiled:
gdbm:
	Could not be configured. It will not be installed.
	Check ext/gdbm/mkmf.log for more details.
openssl:
	Could not be configured. It will not be installed.
	Check ext/openssl/mkmf.log for more details.
*** Fix the problems, then remove these directories and try again if you want.
/Library/Developer/CommandLineTools/usr/bin/make -f enc.mk V="1" UNICODE_HDR_DIR="./enc/unicode/12.1.0" RUBY="./miniruby -I./lib -I. -I.ext/common " MINIRUBY="./miniruby -I./lib -I. -I.ext/common "  enc

The entire stack is much bigger but it seems that is all about openssl... 🤔
I tried installing it multiple times but no luck so far.
I'm on Apple M1 fwiw.
Should I just change those files by hand and let CI run again?

Ah cool, sounds good.
I'm going to put a follow up PR with the object_id and itself in a bit too.
Thanks for the help with this one. 👍🏼


Update Bosnia and Herzegovina National Flag

#154 Updated Bosnian national flag regarding mentioned issue request.

Changed the layout for Bosnian flag to correspond to the real one.

awesome.. do you mind running this command on both files?
svgo --pretty --indent=2 ba.svg

Done. Sorry about delay.

Delay?! Nobody is in a rush here :) hehe.. thank you very nice..


Provide a better scripting language for add-ons

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
Current extension mechanism allows add-on developers to use Go templates notation to generate their install.yaml file.
While this is useful, it is quite limited and complex (Go templates are notoriously known for this), especially when one needs to implement advanced use cases (see ddev-platformsh add-on for reference).
Describe your solution
A solution would be to propose a real scripting language that would be launched an interpreted from Go.
Several solutions exist in the Go world, including:

Lua, using go-lua (Lua 5.2) or gopher-lua (Lua 5.1) (using gopher-luar to simplify data passing)
JavaScript, using goja (ECMAScript 5.1)
Tengo

My preference would go for Lua as it's universal.
Describe alternatives
No response
Additional context
No response

Followups from slack:

gopher-lua has https://github.com/layeh/gopher-luar which does the data injection.

An example of embedding a scripting language for extensions in a very different system (Domoticz, home automation system).
The main codebase here is C++, but the idea is the same. You can easily extend the system using Lua. You get a limited context (mainly the domoticz object) to interact with the system, and triggers.
I think we could get inspired by this :)

I've been down this road with a work project. I don't want to rain on your parade here, but I would caution against doing this. Embedding a scripting language is not a small thing. In addition to all of the various configurations of ddev that exist today, do you really want to also take on support of whatever random thing somebody cooks up in a programming language running inside of ddev?
If a real plugin system is something that you definitely want to move forward with, have you considered something like https://github.com/hashicorp/go-plugin ? It's very easy to use in Go (plugins are essentially just interface implementations). It's also possible to write plugins in any other language that speaks grpc. Going this route forces you to narrowly define how plugins can/can't extend the core system (which is a feature, not a bug), which limits the potential for supporting strange things.

I don't expect this to be a reasonable option any time soon. @cweagans 's suggestion of a go plugin is a nice one, but I don't expect that to happen soon either. Most DDEV add-ons are simple and don't need any fancy go templating, and wouldn't need a fancier language.


Question: recommended BTF Rust library

Hi,
I'm planning to write a CLI tool that, among other things, needs to parse BTF files.
It seems to me that there isn't an established Rust library for that yet. The others I could find are https://crates.io/crates/btf-rs, https://crates.io/crates/btf.
I was wondering if you would recommend using going with library as a dependency (concerning API stability, support of new BTF extensions, ect.), or if one of the others is more likely to become the "standard solution"? More generally it would be interesting to know if you even aim to become the "standard solution" for parsing BTF in Rust and being depended on by other projects?
Of course I'd be happy to contribute if there is something I need that needs to be implemented.

I believe libbpf-rs exposes libbpf's APIs for working with BTF data (see https://github.com/libbpf/libbpf-rs/blob/master/libbpf-rs/src/btf/mod.rs), so I think it would be best to start with that. btfdump is meant to be a CLI, not a library, so I don't think it's a good choice for you.


fix(rhel): allow RESTY_IMAGE_TAG to be 2+ digits

Full RedHat family versions are classically 2 digits (7.9, 8.6, 9.0) and the ubi docker container image tags conform to that standard:

registry.access.redhat.com/ubi7/ubi-minimal:7.9
registry.access.redhat.com/ubi7/ubi-minimal:8.6
redhat/ubi9:9.0

Crucially, they do NOT offer "major-only" floating tags (e.g. *:9).
This PR just splits out the major version of the RESTY_IMAGE_TAG.

Seems like I've got some more work to do.

👍 LGTM! I wouldn't hate it if we put some of the conditional logic into helper functions (i.e. if is_redhat_flavor "$RESTY_IMAGE_BASE"; then ... ; fi), but I don't know if the plumbing in here would allow for that anyhow, so no objections here.

I had that thought as well. Not only do we not a have a good spot for that logic, applying it to all our platforms across the rest of the repo (so as not to leave rhel a special case) would be a huge undertaking. This PR was already an exercise in not ripping everything out and starting from scratch..


Suggestion to make Custom Quick access

Hi.
Just downloaded RX Explorer, but find out terrible issue: There's no any way to pin folders for fast access. I always use quick access 'cause it's safes A LOT of my time.
Also it's pretty strange that I can unpin folders in default library, but no way to pin another folder instead.
If you just make button for pinning folders, you'll be the best.
Thanks in advance!

Just right-click the empty space of "Library" and you will found add button～

Thanks, this worked, but I think you should increase size of field with Add button. All upper empty space isn't clickable and 2/3 of bottom space too. This red rectangles shows up how much space that very intuitive to click just do nothing.
Anyway, thanks for help!


Retrieve Reviews by Version

Is there any way to retrieve reviews by version or is there any planned support for this?

The reviews are retrieved through apple RSS feeds (example: https://itunes.apple.com/us/rss/customerreviews/id=389801252/json). I don't think there's a way to filter by version. Let me know if you find one and I'll add it to the module.


Update for Bukkit / Spigot 1.8!

I use HeroChat and bungee, and want to link them, but it doesn't work on 1.8. Any chance you could update? Thanks!

Are you using the download from here? http://www.spigotmc.org/resources/herochatbungeebridge-bungee.189/
It works fine for me (unless I did something wrong)

My mistake, I was having some troubles with it and seeing a bunch of errors, but it turns out it was my fault. Thanks!


[dv] Rip out old checking where needed in test suite

This is a reduced version of https://github.com/lowRISC/ibex/issues/1567 for M2.5. For tests that are fragile and falsely triggering the old checks we should just remove those checks.
Triage regression failures and determine what should be removed to bring pass  rates up to standard

estimate 8

estimate range: 4 - 8


Room 2.4.2 Upgrade

Upgrades Room.  Fixes M1 Support as well.
Related PRs and Issues:

#409
#410

Looks like we'll need compileSdk 31 as well, so will add that.


Render shell in issue templates.

This will render nicer and use monospace in the form field.

Review period will end on 2021-03-22 at 09:52:32 UTC.

Review period skipped due to critical label.

critical and merging soon to make consistent with Homebrew/brew


[AKS] Support azure policy addon in azure-cli

Description

Azurepolicy addon was supported in AKS through extensions as of now. Since this addon is planned for GA, adding to azure-cli official repo.
Testing Guide

az aks create -g  -n  -a azure-policy
az aks enable-addons -g  -n  -a azure-policy
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.
azure cli extensions PR - https://github.com/Azure/azure-cli-extensions/pull/651/files

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

AKS

@arrownj updated test.


[BEAM-10669] Add support for Dataflow Templates in SnowflakeIO

Changed basic types in SnowflakeIO to ValueProvider and refactored necessary functionalities due to support Dataflow Templates.

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

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

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run PythonDocker PreCommit

Run Java PreCommit

Run Python PreCommit

Run Whitespace PreCommit

@pabloem will you find a moment for a  CR?

gladly!

HI @pabloem, will you find a moment for a CR during this week?

sorry about the delay. Added a few comments!

@pabloem, no worries , thanks for CR :) I responded to all comments.

Run PythonDocker PreCommit

Run Portable_Python PreCommit

@pabloem I think we can proceed with merge as all checks are passing.

done! thanks @purbanow @kkucharc !


Unsupported cast when casting to a one field struct using the parenthesis syntax

Describe the bug
The following raise an error
SELECT (foo.my_id)::struct<id string> as foo FROM ...

However using the row() function it works:
SELECT row(foo.my_id)::struct<id string> as foo FROM ...

Error message/log
Unsupported function: cast(character varying) -> struct<id character varying>;

### To Reproduce

_No response_

### Expected behavior

The `()` syntax for creating struct should work with one field. When I add a second one it works. I currently explicitely use `row()` as a workaround.

### How did you deploy RisingWave?

K8s operator

### The version of RisingWave

v1.8.2

### Additional context

_No response_

The ROW keyword is actually optional as long as you have more than one field in the expression

https://www.postgresql.org/docs/current/rowtypes.html#ROWTYPES-CONSTRUCTING
This is actually PostgreSQL behavior, which is essential to differentiate it from parentheses used for precedence: (int_x + int_y) * int_z.
If you know Python, it is also the case that (int_x) is an integer while (int_x,) with an awkward trailing comma is a tuple containing an integer field. The trailing comma syntax is not adopted by PostgreSQL, so does RisingWave.


Feat: request and check metadata before downloading the kpm dependencies

Feature Request
Is your feature request related to a problem? Please describe:

Describe the feature you'd like:

Please request and check metadata of the local kpm packages before downloading the kpm dependencies declared in kcl.mod, avoiding re-downloading the package due to the absence of the kcl.mod.lock file.
Describe alternatives you've considered:

Teachability, Documentation, Adoption, Migration Strategy:

Related Issue: https://github.com/kcl-lang/kpm/issues/134

I personally believe that the following points need to be clarified:

Logic and checkpoints triggered by Git Source and OCI Source for re downloading.
How to compare the consistency between remote source and local package.
Do we need to check the local package checksum before running kcl? This may cause performance issues https://github.com/kcl-lang/kpm/issues/295

Addition

A redownload of one dependency may result in a redownload of an indirect dependency


Remove stop kwarg and use write_then_readinto.

See https://github.com/adafruit/circuitpython/issues/2082 for details.

I will test.


waagent  has renamed to walinuxagent

sudo systemctl restart waagent should be sudo systemctl restart walinuxagent

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

ID: 1f57aa98-f440-5657-7a2d-ca8ae1677ee2
Version Independent ID: 983c26ef-d1e2-6f62-19dc-67a214f0f217
Content: Enable InifinBand with SR-IOV - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/hpc/enable-infiniband.md
Service: virtual-machines
GitHub Login: @vermagit
Microsoft Alias: amverma

Thanks for the feedback! We are currently investigating and will update you shortly.

@vermagit Can you please help confirm on this one? Thanks :)

https://github.com/Azure/WALinuxAgent#upgrade indicates that this change is true for Ubuntu, but not for CentOS or CoreOS.
In fact, just checked on CentOS 7.7:
Failed to restart walinuxagent.service: Unit not found, but,
agent service name: waagent
Perhaps the doc can be clarified to also provide the alternative for Ubuntu.

@lgarithm Hope that helped. Please refer to the doc:https://github.com/Azure/WALinuxAgent#upgrade I will work to see if this can be provided as an alternative for Ubuntu in this doc.
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.


Refactor analysis assertion

[x] Refactored assertions for various cases in message consuming

[x] Updated documentation on message consuming and producing

[x] Refactored temporary file cleaning up using tempfile

Could you please look into refactoring some units?

get_source_path I think it should be split into separate methods for each forge, with get_source_path dispatching to the right one.

Same for MavenUtils.checkout_version, it should just dispatch to a specific method for each repoType.

KafkaUtils.tailor_input(payload) could be replace by something like:

        def tailor_input(payload):
        """
        Tailor 'payload' from consumed topics,
        to avoid (big) call graph data
        adding to 'input' field in the produced topics.
        """
        payload.update({
            "graph": {},
            "modules": {},
            "cha": {},
            "depset": [],
            "build_depset": [],
            "undeclared_depset": [],
            "functions" :{}
        })
        return payload


_CastError ScrollPosition.didUpdateScrollPositionBy

from this stack we can not detect which app page encount error. so we guess may be there is any issue with framework?
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.8.0, on macOS 11.6 20G165 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] Connected device (3 available)

Hi @zjrwdl
Is this issue reproducible locally? If so, can you please share the output of flutter run -v and a completed and minimal reproducible code sample?
As far as I can see, you are using the old Flutter version 2.8.0, the latest Flutter stable is 3.0.5 and the master is 3.1.0. Please also try upgrading them and retry to see if this issue still reproduces.

Hi @zjrwdl Is this issue reproducible locally? If so, can you please share the output of flutter run -v and a completed and minimal reproducible code sample?
As far as I can see, you are using the old Flutter version 2.8.0, the latest Flutter stable is 3.0.5 and the master is 3.1.0. Please also try upgrading them and retry to see if this issue still reproduces.

sorry, we can not reproduce this issule locally.
we have upgrade flutter engine to 3.0.1 for our app which will be published in nex month until which  time we may confirm if this issule has been fixed.
thanks

Hey @zjrwdl, could you provide full crash log (see hint flutter_error_details from your screenshot) and list of affected devices?
(The log should be in the text format instead of a screenshot for better investigation)

Hi @zjrwdl
The information from the log above is not enough to investigate the issue. Do you see this issue when try running the project with release mode?


Tool not working for you?

Its because this tool is abandoned and no longer maintained, if you want to get someones account, dm seve.services on instagram and we will help you

This is not a issues pls close this @Cyb0r9


Android only: Background transparency broken after back navigation

Hello together,
I am using ui-material-tabs. My tabs background is semitransparent to allow for the background image to shine through. See attached images. When I navigate away from the tabs page and navigate back to it, the material tabs loose the background transparency (background is white).
Version 6.2.11 of @nativescript-community/ui-material-tabs is working fine.
With version 6.2.12 and 6.2.13 I am observing the issue.

Which platform(s) does your issue occur on?

Android
SDK 30 and 31 tested
emulator and device. Samsung Galaxy

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)
-8.1.5-next-02-10-2022-1825431584
Cross-platform modules: (check the 'version' attribute in the
"@nativescript/core": "8.1.5"
Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project)
"@nativescript/android": "8.2.0-alpha.11"
Plugin(s):
"dependencies": {
"@angular/animations": "13.2.2",
"@angular/common": "13.2.2",
"@angular/compiler": "13.2.2",
"@angular/core": "13.2.2",
"@angular/forms": "13.2.2",
"@angular/platform-browser": "13.2.2",
"@angular/platform-browser-dynamic": "13.2.2",
"@angular/router": "13.2.2",
"@apollo/client": "3.5.8",
"@nativescript-community/ui-material-tabs": "6.2.11",
"@nativescript/angular": "13.0.1",
"@nativescript/core": "8.1.5",
"@nativescript/iqkeyboardmanager": "2.0.0",
"@nativescript/localize": "5.0.4",
"@nativescript/mlkit-barcode-scanning": "1.0.0-alpha.8",
"@nativescript/mlkit-core": "1.0.0-alpha.8",
"@nativescript/secure-storage": "3.0.0",
"@nativescript/theme": "3.0.2",
"@nativescript/ui-charts": "file:../mnd-custom-plugins/nativescript-ui-charts/publish/package/nativescript-ui-charts-0.1.10.tgz",
"apollo-angular": "3.0.0-alpha.2",
"apollo3-cache-persist": "0.14.0",
"d3-ease": "3.0.1",
"graphql": "16.3.0",
"graphql-tag": "2.12.6",
"intl": "1.2.5",
"moment": "2.29.1",
"nativescript-oauth2-ext": "file:../mnd-custom-plugins/nativescript-oauth2-ext/publish/package/nativescript-oauth2-ext-3.0.0.tgz",
"nativescript-sqlite": "2.8.6",
"nativescript-ui-autocomplete": "8.0.1",
"nativescript-ui-calendar": "file:../mnd-custom-plugins/plugins-ui/dist/packages/ui-calendar/nativescript-ui-calendar-8.0.2.tgz",
"nativescript-ui-gauge": "8.0.1",
"rxjs": "7.5.4",
"uuid": "8.3.2",
"zone.js": "0.11.4"
},
"devDependencies": {
"@angular-devkit/build-angular": "13.2.3",
"@angular/compiler-cli": "13.2.2",
"@graphql-codegen/cli": "2.6.1",
"@graphql-codegen/fragment-matcher": "3.2.1",
"@graphql-codegen/introspection": "2.1.1",
"@graphql-codegen/typescript": "2.4.3",
"@graphql-codegen/typescript-apollo-angular": "3.3.3",
"@graphql-codegen/typescript-operations": "2.3.0",
"@nativescript/android": "8.2.0-alpha.11",
"@nativescript/ios": "8.2.0-alpha.6",
"@nativescript/types": "8.1.1",
"@nativescript/webpack": "5.0.4",
"@ngtools/webpack": "13.2.3",
"@types/d3-ease": "3.0.0",
"@types/intl": "1.2.0",
"@types/node": "16.11.6",
"@types/uuid": "8.3.4",
"keycloak-request-token": "0.1.0",
"rimraf": "3.0.2",
"sass": "1.49.7",
"ts-node": "10.5.0",
"typescript": "4.5.5"
}

I was able to solve the issue. In the past I had not to set
MDTabs {
background-color: transparent;
}
It was sufficient to have
MDTabContentItem {
background-color: transparent;
}
Since 6.2.12 now both have to be set.


test(ticdc): migrate test-infra to testify for cdc/kv pkg (#2899)

What problem does this PR solve?

Issue Number: close #2899
What is changed and how it works?
Migrate all test cases under cdc/kv to testify.
Check List
Tests

Unit test

Release note
`None`

/run-dm-integration-tests

/run-dm-integration-tests

/run-dm-integration-tests

/run-all-tests

/run-all-tests

/run-verify

/run-all-tests

/run-all-tests

/run-kafka-integration-tests
/run-integration-tests

Codecov Report

Merging #4515 (45d6d76) into master (9607554) will decrease coverage by 0.1293%.
The diff coverage is 61.9571%.

Flag
Coverage Δ

cdc
60.2080% <59.2427%> (+0.2858%)
:arrow_up:

dm
51.5361% <62.9848%> (-0.4927%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4515        +/-   ##
================================================
- Coverage   55.6402%   55.5109%   -0.1294%
================================================
  Files           494        506        +12
  Lines         61283      62494      +1211
================================================
+ Hits          34098      34691       +593
- Misses        23750      24332       +582
- Partials       3435       3471        +36

/run-all-tests

/run-all-tests

/run-kafka-integration-tests
/run-leak-test

/merge


More detailed DOM inspector for vue-devtools

Currently, the React Devtools show other DOM elements than React components. I find it pretty useful.
Is it something people would want?

The devtool is designed to visualize the component tree, not the DOM tree. Chrome itself already has a powerful DOM inspector, not sure what's the value in duplicating that effort.


We should enable support for high-DPIs on macOS

Current rendering on Retina (2560x1600):

Fixed package for tests:
veles-retina-fix.zip

Another try:
test.zip

Yup it works now


Aliases do not invalidate properly.

If dep-a is aliased to dep-b and dep-b depends on dep-c, when dep-c is invalidated, dep-b is therefore invalidated, however dep-a is not but should be.
Also, what happens with aliases to aliases?

Fixed in #16


Add a Splash screen

I'm reopening #117
Why was this closed?
On my computer the start up takes quite some time and the experience is quite unpleasant.
I find myself clicking the icon several times.

https://github.com/life888888/AsciidocFX_Linux_INSTALLER_BUILD/releases/tag/v0.1.0
I am use jpackage repack AsciidocFX Linux only, add Splash Screen.
I am not change any code or recompile, just repack it.


[Task]: sending Iota not recived in the adress that i transfer it

Task description
Hello everyone, unfortunately I have a big problem and have not received the Iota that was sent in my Exodus wallet. I initiated the transfer to this address . . . 0xa89f913b0adc708af3b13accbaae93a4c0598365
Everything looks fine on the Iota wallet but the transfer never arrived. Please help.

Hello @bbayar-cpu
You're experiencing transcript error. It sounds like your query would be best dealt with by the support team.
I'll have to refer you to the official support live chat with the ticket ID EXO07638
Please see the link below to our dedicated support line:
Exodus Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.

@bbayar-cpu Dear user
I understand that you have been experiencing some issues and I want to ensure you get the support you need as quickly as possible by using our live chat option.
Support : Support
1.	Access the Live Chat:
•	Navigate to our website’s home page.
•	Look for the chat icon located at the bottom right corner of the screen and start a 💬.

Hello @ A support ticket has been opened for you. Kindly Talk to Support on the Live support page so that your issue can be looked into. CLICK HERE

Task description
Hello everyone, unfortunately I have a big problem and have not received the Iota that was sent in my Exodus wallet. I initiated the transfer to this address . . . 0xa89f913b0adc708af3b13accbaae93a4c0598365
Everything looks fine on the Iota wallet but the transfer never arrived. Please help.

You sent your funds to the IOTA EVM network, you have to make sure your wallet is connected to the IOTA EVM to see your funds. You can see your funds arrived https://explorer.evm.iota.org/address/0xA89f913b0ADC708Af3B13ACCbaAe93a4C0598365, so you sure are connected to the wrong network. If you need further support on how to see your funds in the right network you can join our discord https://discord.iota.org/


How can I tell what was changed?

is there any way to get a 'delta' so that we know every customization which has been applied before inheriting from a custom process definition?  Sometimes we want to make a few changes to an existing customization for a new project or to replicate just some changes

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

ID: 715eca62-8a31-4163-c27b-f5e71d781982
Version Independent ID: 06dc711f-1946-29e9-aece-75af903da09b
Content: Customize the workflow of an inherited process - Azure DevOps Services
Content Source: docs/organizations/settings/work/customize-process-workflow.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@QA1-Skip -- thank you for your question.  You may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

@KathrynEE -- please look into this issue.

@QA1-Skip - For Azure DevOps Services, you can view changes made to an Inherited process by going to the Audit log. See Access, export, and filter audit logs.


Add combo loss to DeepLabv3+

This PR adds a slight variation of the MONAI dice loss to composer and provides the option to use both cross entropy and dice loss when training DeepLabv3+. Cross entropy and dice can be selected by providing cross_entropy_weight and dice_weight arguments that are > 0. A helper function is included in DeepLabv3+ to return a list of losses based on the weight for the cross entropy and dice losses. Note: it may be easier if ComposerClassifier accepted a sequence of callables, but TBD if we want to continue to extend this class.
Some changes were made to ensure_targets_one_hot. The first change required cloning targets since in-place operations on target prevented target from being used with multiple losses. Second, PyTorch's one_hot function seems to be ~10% slower than the operation in MONAI, so I switched to a variation similar to MONAI.
I did an initial pass at tests by checking two extreme scenarios: completely correct predictions and completely incorrect predictions. Please let me know if more tests are needed or if the tests can be organized another way
Most recent WandB runs

Huh, it turns out our previous one-hot conversion method lead to noncontiguous tensors. Could this have been the cause of the slowdown?
The label smoothing tests use F.one_hot() and .movedim() to produce the reference labels. I needed to add .contiguous(), so the strides would match between the new and old methods.

Hmm, getting a fused layer norm timeout. HuggingFace timeouts Hanlin mentioned?

Added:

Lots of tests for _one_hot
num_classes=-1 functionality
dim argument to specify where the new class dimension will be specified
Check in _one_hot that the input tensor is an index tensor
Check in _one_hot that num_classes >= tensor.max() + 1


fix Aqua's reported piracies and method ambiguities

Fix all of Aqua's reported piracies and method ambiguities. (#65)
Precisely 37 out of the 39 potential piracies are avoided by informing Aqua that types from QuantumInterface are in fact our own. The last two are fixed by adjusting the * and + functions that accept a Vararg, it had a problem for the case when N=0. This solution avoids both the unbound_type_param and the piracy, and seems to be a reasonable design to not allow zero arguments passed to the function (similar to Base operators themselves)

PR checklist

[ ] The code is properly formatted and commented.
[ ] Substantial new functionality is documented within the docs.
[] All new functionality is tested.
[ ] All of the automated tests on github pass.
[ ] We recently started enforcing formatting checks. If formatting issues are reported in the new code you have written, please correct them. There will be plenty of old code that is flagged as we are slowly transitioning to enforced formatting. Please do not worry about or address older formatting issues -- keep your PR just focused on your planned contribution.

Hi! Just a friendly bump on this. I see most of the piracies were masked by marking a bunch of types as owned. This will be a bit fragile. Do you have any insight into which piracies this addresses -- upstreaming them to QuantumInterface would be a safer long-term solution.

Hi @Krastanov, I replied to your code review itself a while ago. Maybe GitHub doesn't show it too clearly https://github.com/QuantumSavory/QuantumSymbolics.jl/pull/85#discussion_r1827075621

No worries, I have made the same mistake in the past.

I see, that makes sense, thank you!
So an example of an offender would be
Base.:(*)(x::Symbolic{T}, y::Symbolic{S}) where {T<:QObj,S<:QObj}

I am wondering, can own_types take something like Symbolic{<:QI.AbstractBra}, in order to be more specific about what types are actually owned? After all, given the premise of this library, we can not really claim ownership of QI.AbstractBra, but Symbolic{<:QI.AbstractBra} is very much "owned".

I'm won't be exactly sure why, but that doesn't seem to work.
At the simplest case Aqua.test_piracies(QuantumSymbolics, treat_as_own = [Symbolic{QI.AbstractOperator}) still shows the following. (also tried with <:QI.AbstractOperator)
Possible type-piracy detected:
......
[35] tr(x::Symbolic{QuantumInterface.AbstractOperator}) @ QuantumSymbolics ~/.julia/dev/QuantumSymbolics/src/QSymbolicsBase/linalg.jl:305
[37] vec(x::Symbolic{QuantumInterface.AbstractOperator}) @ QuantumSymbolics ~/.julia/dev/QuantumSymbolics/src/QSymbolicsBase/linalg.jl:527

After all, given the premise of this library, we can not really claim ownership of QI.AbstractBra

The way I'm thinking about it is that QuantumInterface is that one that owns AbstractBra in reality. If a user comes along and instantiates a Vector{<:AbstractBra} object, they still don't own it, and defining adjoint on it would still be a piracy.
What we are saying here is that QuantumSymbolics gets a special treatment because it is really an extension of QuantumInterface by the same developers. QuantumInterface itself says This package is not meant for public use! in the readme.  I'm assuming non of your other quantum packages will ever be interested in defining Base.:(*) too.

You are right, it is just that there are multiple groups coordinating on this work, and having as much of the coordination automated would help a lot.
In particular, if we just treat_as_own the bare AbstractKet, we will miss actual issues like f(::AbstractKet) which should be in QuantumInterface not in QuantumSymbolics, making Aqua less helpful than it could be -- this is my main worry.
I posted about this on slack. Let's wait a bit to see if anyone has suggestions there.

Actually, how about we keep your implementation as is, but also add the following extra test?
all([m.sig.types[2:end] ⊆ own_types for m in Aqua.Piracy.hunt(QuantumSymbolics)])

The subset check might be too harsh. An intersection check might be enough.

we will miss actual issues like f(::AbstractKet)

Ok, that is the use case I didn't think of on my own

but also add the following extra test?

Yeah, doesn't seem like a bad idea. I went through few iterations to get to something that doesn't show false-positives.
# get it to work without errors
all([Base.unwrap_unionall(m.sig).types[2:end] ⊆ own_types for m in Aqua.Piracy.hunt(QuantumSymbolics)])

get_method_args(m::Method) = Base.unwrap_unionall(m.sig).types[2:end]

# Transform to filter to see the cases that are not passing
filter(m -> !(get_method_args(m) ⊆ own_types), Aqua.Piracy.hunt(QuantumSymbolics)) # 18 false-positives

# Use intersection test. Subset is too strict because it keeps out methods which take additional arguments
#not related to QuantumSymbolics.  Example ptrace(x::Symbolic{QuantumInterface.AbstractOperator}, s)
filter(m -> isdisjoint(get_method_args(m), own_types), Aqua.Piracy.hunt(QuantumSymbolics)) #10 false-positives

# Some arguments need a little normalization. Such as maketerm(::Type{<:Symbolic{<:QObj}},...)
function normalize(arg)
    if (arg isa UnionAll) && (arg.body <: Type) arg = arg.body.parameters[1] end
    if (arg isa Core.TypeofVararg) arg = arg.T end
    if (arg isa TypeVar) arg = arg.ub end
    return arg
end

filter(m -> isdisjoint(normalize.(get_method_args(m)), own_types), Aqua.Piracy.hunt(QuantumSymbolics)) #8 false-positives

# What we really need is to check if an arg is a subtype of Symbolic{<:QObj}, to be able to match most edge cases
filter(m -> !any(normalize.(get_method_args(m)) .<: Symbolic{<:QObj}), Aqua.Piracy.hunt(QuantumSymbolics)) # 0 false-positives

Here is a proof of it catching an introduced piracy
module modA
       import QuantumInterface: AbstractBra, AbstractKet
       import Base.println
       import SymbolicUtils.Symbolic
       Base.println(x::T) where T<:Union{AbstractBra, AbstractKet} = 1
       Base.println(x::Symbolic{T}) where T<:Union{AbstractBra, AbstractKet} = 2
end

filter(m -> !any(normalize.(get_method_args(m)) .<: Union{Symbolic{<:QObj}}), Aqua.Piracy.hunt(modA))
#[1] println(x::T) where T<:Union{AbstractBra, AbstractKet} @ Main.modA REPL[53]:5

This is wonderfully done, thank you for working it out!

I'm good with this. Might have gotten carried away.

Thank you, this is very much appreciated.
Could you message me at<EMAIL_ADDRESS>with your name, github username, and bounty number so I can forward you the bounty payment details?


fix(dashboard): Incorrect scopes of cross filters

SUMMARY
Due to a bug introduced in https://github.com/apache/superset/pull/20743, cross filter scopes were not calculated correctly.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
https://user-images.githubusercontent.com/15073128/182021695-12fe0c3f-6262-4d88-8956-f986ce096ad5.mov
After:
https://user-images.githubusercontent.com/15073128/182021734-dc007843-5002-45f9-b6d1-f72d4fa6cec9.mov
TESTING INSTRUCTIONS

Open a dashboard and apply cross filters
Verify that filters are applied correclty

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

/testenv up FEATURE_DASHBOARD_CROSS_FILTERS=true


Extended  ADCS_TEMPLATE_ Variable

If there is a config variable ADCS_TEMPLATE_<upper(authority.name)> take the value as Cert template else default to ADCS_TEMPLATE to be compatible with former versions

Coverage decreased (-0.006%) to 57.895% when pulling 1b73b1d0809722c310292c78ff6f164061f67ce6 on sirferl:master into 14b73b73cfce5463a96d0ef1f8aa9846ddf5b4d4 on Netflix:master.

@hosseinsh  hi, please confirm the merge. I have one more in the pipeline: New Plugin fro Entrust-Datacard !

Hi @sirferl thanks for the PR, just merged it.
looking forward to the Entrust-Datacard plugin!

@hosseinsh Thank you !


docs: verbatim move json part to separate file

this moves the JSON part of the specification to a separate file, no changes made to content
I'd like to do some work on JSON serialization semantics, this is the first step in that direction

Note: Table of Content can't access the Parts in the new file. Generally, it is tricky to find the JSON specification from inside the Markdown document.

agree, hmm, should increase the visibility somehow
as we might end up with multiple different file formats(.properties?), I think it makes sense to separate per file format

The trend could be more file formats and yes, a separate file/chapter is a proper choice then.
Let's consider it a living document :smiley:
In future, we may want to provide a landing page for the file specification topic.
Or in the nearer future, replace the list with a table:

File Format
Specification
Content


[Bug] Short Bug Description

Bug Category

[] Credential Login
[ ] Token Login
[ ] Local Audio Issue
[ ] Remote Audio Issue
[ ] Audio Device Switching
[ ] Mute / Unmute
[ ] Hold / Unhold
[ ] Performance issues
[x] Other

SDK Version
Which version of the SDK have you added from Jitpack? Feel free to add the whole dependency implementation:
eg. implementation 'com.github.team-telnyx:telnyx-webrtc-android:v1.2.12-alpha'
Describe the bug
Use the number<PHONE_NUMBER>7 to answer the call, the call_control_id received many times are these
(v2: FlRnq65ww9nyDob9u_zoCfBx_AAbezGZ-4EWjRfeQfZd5BQaP6s2dg,
v2:sGp3qbH9F2y4D75QYqmcbniGnA6L2zItwfjxrYlCwz2R8CIgPGOO-g
v2:SUA3v7pWwQyKRJb4o86V1800lCUm_FyoaKW4b1-uN3-SlGBlQpXl4A,
v2: cISqpfF1lToJ5qVdYa3nn9miN86r4JAIYzCf6OISnBMiptw8YOlqtw
v2: d3e7C3TTCk4DaLGySvsKVa284rSsln99PLdadIhru_4pTVYF8_JM4w,
v2:PtohROCktt0FyBIyJ8qSy033Obmn3SIrB6KZsYhZJDq0PvqGYFxsZA
v2:E3qCZ-vFfUOmxkpCMDXPhxai2QY6JCheWLHR7W9Z9W2ffHTG1kTVfg ), after the call hangs up, the caller can still receive the answering invitation multiple times, the time span is 15 minutes
Expected behaviour
normal call
To Reproduce
Steps to reproduce the behaviour:

e.g. (Log on with credentials)
e.g. (Socket error then returned)

Android Device (please complete the following information):

Emulator: (true/false)
Android Device: [e.g. Pixel 6]
Android Version: [e.g. 12 ]

Logs
Please provide any logs available to you, remember to enable verbose logging within the SDK. It is generally helpful to filter by either TxSocket (Socket messages) or TelnyxClient (Client messages) or both depending on the situation.

Hi @tongsoftinfo
Just for clarification, after you answer a call you receive a series of messages containing
"(v2: FlRnq65ww9nyDob9u_zoCfBx_AAbezGZ-4EWjRfeQfZd5BQaP6s2dg,
v2:sGp3qbH9F2y4D75QYqmcbniGnA6L2zItwfjxrYlCwz2R8CIgPGOO-g
v2:SUA3v7pWwQyKRJb4o86V1800lCUm_FyoaKW4b1-uN3-SlGBlQpXl4A,
v2: cISqpfF1lToJ5qVdYa3nn9miN86r4JAIYzCf6OISnBMiptw8YOlqtw
v2: d3e7C3TTCk4DaLGySvsKVa284rSsln99PLdadIhru_4pTVYF8_JM4w,
v2:PtohROCktt0FyBIyJ8qSy033Obmn3SIrB6KZsYhZJDq0PvqGYFxsZA
v2:E3qCZ-vFfUOmxkpCMDXPhxai2QY6JCheWLHR7W9Z9W2ffHTG1kTVfg" ?
Is this within the invitation message? Could you please provide an example log copied from Logcat if it is reproducible like the last section of the issue template states?

Also please note the latest version is:
https://jitpack.io/#team-telnyx/telnyx-webrtc-android/v1.2.13-alpha

Hi @tongsoftinfo
Just for clarification, after you answer a call you receive a series of messages containing
"(v2: FlRnq65ww9nyDob9u_zoCfBx_AAbezGZ-4EWjRfeQfZd5BQaP6s2dg, v2:sGp3qbH9F2y4D75QYqmcbniGnA6L2zItwfjxrYlCwz2R8CIgPGOO-g v2:SUA3v7pWwQyKRJb4o86V1800lCUm_FyoaKW4b1-uN3-SlGBlQpXl4A, v2: cISqpfF1lToJ5qVdYa3nn9miN86r4JAIYzCf6OISnBMiptw8YOlqtw v2: d3e7C3TTCk4DaLGySvsKVa284rSsln99PLdadIhru_4pTVYF8_JM4w, v2:PtohROCktt0FyBIyJ8qSy033Obmn3SIrB6KZsYhZJDq0PvqGYFxsZA v2:E3qCZ-vFfUOmxkpCMDXPhxai2QY6JCheWLHR7W9Z9W2ffHTG1kTVfg" ?
Is this within the invitation message? Could you please provide an example log copied from Logcat if it is reproducible like the last section of the issue template states?
This is the callid in the received push information, it is wrong to use the UUID.fromString method


Add missing findPlayerBySteamID() function

Add missing findPlayerBySteamID() function to find.lua

Assuming that function exists and does what you use it for, it looks good, but I have questions:

What kind of SteamID is used?
What happens if you pass a SteamID that is malformed?
What happens if you pass a SteamID that is not present on the server?

What kind of SteamID is used?
What happens if you pass a SteamID that is malformed?
What happens if you pass a SteamID that is not present on the server?

In steam terms that is just called SteamID, e.g. STEAM_0:0:0
false
false

This should return something like this
return player.GetBySteamID(id) or NULL
You also may want to localize that function

booleans should never be returned by any E2 function
Seriously, why localize here? not like people are going to use this a million times a second

booleans should never be returned by any E2 function

Guessing this is response to the code and not what I posted

Seriously, why localize here? not like people are going to use this a million times a second

It was just a suggestion seeing as in many other places in code functions used within E2 functions are localized.

But seriously, why localize here

It is a good thing to do.
Also, see player.GetBySteamID(string) definition.

Now, to put it simple for everyone, AFAIK (Valve wiki):
Steam (account) ID is sometimes referred to as Steam32 (STEAM_X:Y:Z).
Community ID is also known as Steam64 (7656119XXXXXXXXXX).
Steam3 is the new format (U:X:Y).
Last, but not least is the Custom (profile) URL/ID.
In GMod, there is also GMod Unique ID (it uses CRC-32 checksum; "gm_" + CRC32(Steam32) + "_gm").
Now do the math, it is five different types of formats to represent one unique Steam account.
All of them can be converted from one to another, except for GMod Unique ID (which can only convert one way, so any of the 4 can convert to it, but can't get other 4 from it, at least not directly/easily)..
Of course, I had some ideas like "brute-forcing" the checksum in order to get others from GMod Unique ID, it is possible, but it would run better with online database.. cough. Perhaps, I went off-topic now, so I will stop here.

@TomyLobo

What kind of SteamID is used?
What happens if you pass a SteamID that is malformed?
What happens if you pass a SteamID that is not present on the server?

Iv added the or NULL to keep it in line with findPlayerByName etc,
and to answer your Qs:

It uses the non 64 SteamID, unsure if there is another term for the regular SteamID
as @bigdogmat said its formatted like: STEAM_0:0:0
Malformed SteamID will now return Null Entity
Non present SteamID also returns Null Entity

It uses the non 64 SteamID

This uses Steam32 ID (see my post above).

Should I also add findPlayerBySteamID64(S) or leave users to use the new steamIDFrom64(S) function?

I would say, yes, go ahead. Because, Steam64 is always correct! - in Singleplayer game, I mean, unlike Steam32 ID which is always "STEAM_0:0:0" in Singleplayer game.
(But, it will also be nice to have others such as findPlayerBySteam3 and findPlayerByUniqueID.. I will add more things to steamidconv.)

With steamID64 always being correct even in SP id say that is a good enough reason to have it added also,
as for the others if your going to add converters then id say let users use the converters in those cases.

This uses Steam32 ID (see my post above).

It's usually just called SteamID, SteamID64 is usually called community ID as that's what it is.

Because, Steam64 is always correct! - in Singleplayer game, I mean, unlike Steam32 ID which is always "STEAM_0:0:0" in Singleplayer game.

SteamID64 will always return nothing serverside in single player

(But, it will also be nice to have others such as findPlayerBySteam3 and findPlayerByUniqueID.. I will add more things to steamidconv.)

Steam3 or better known as AcountID isn't implemented into the game yet, although we can still get the value I'd ask that we wait until the next game update to add it. Source
As for UniqueID, it's known to have quite a few collisions and shouldn't be used ever, I'd recommend against it.

Steam3 or better known as AcountID isn't implemented into the game yet

It is implemented already.. Just not very known as that format by itself is still rather new, but I've found it myself (a while ago) :P
local ply = NULL
if SERVER then
    ply = player.GetAll()[1]
else
    ply = LocalPlayer()
end
if IsValid(ply) then
    print("U:1:" .. ply:GetPlayerInfo().friendid)
else
    print("Invalid player!")
end

Here it is in meta:
local PLAYER = FindMetaTable("Player")
if not PLAYER then return end

function PLAYER:AccountID()
    if not IsValid(self) return "" end
    return "U:1:" .. self:GetPlayerInfo().friendid
end

???
Profit before it gets added in GMod stable (non-dev) branch ;)

It is implemented already

Read although we can still get the value

@sammyt291 By the way, also add entity=findPlayerByUserID(number). I am busy with something so I can't switch to wire atm..

@bigdogmat
Apologies for the eye abrasion, I have replaced the UserID with your exemplar function, thank you :+1:

findPlayerByID already exists in E2: It's called entity(N)

Pretty sure UserID is different from entity id

@bigdogmat re-read my updated post

@Divran Ah, thanks for looking into it

Even if userid and entity id are two different things, we can still use entity id in the same way. I'm pretty sure they are the same, though.

I'm pretty sure they are the same, though.

 No, they are not the same! Read this (see revised gist).
There, I've made things clear more, specifically between Entity:EntIndex() and Player:UserID(), too. See gist here. Hopefully they will help clear such misinformation.
Does the following help?

alright then it looks like I was wrong. Looks like Entity(n) is always sequential, while the player ID is not. This is strange, though, since Entity(n) does not behave this way for any other entity (if you spawn two props and delete one, the entity ID for the second prop will not shift up to fill the gap). That is why I assumed they were the same. Because it would be logical if they were.

Now the next question is, do we really need this function in E2?

Yes and no, I wrote a library to handle all of these findPlayerBy* cases a long time ago and its just become standard to include them if I ever work on something. Most of the time I'm working with SteamID or name, although sometimes I have an edge case where I need user input and it's nice to allow them to use whatever.
Adding these would remove a little bit of bloat for the people that use them, and it'd be a convenience for the people that want to use them although are not sure how to implement them.

I vote yes for find by steam ID and maybe also steam ID 64, but I'm not convinced that find by user id is useful. In this PR it was mostly just added because of the commonly used reason of "why not?". The two find by steam ID functions can both already be used in E2 because we already have a way of getting the player's steam ID, people know it exists and are used to it. Seems like this find by user id would just be a lonely function with no relation to anything else in E2. The only way to use it would be if we also added a get user id function.
Plus, even though entity IDs and user IDs behave differently, you can still use entity IDs in much the same way, so we pretty much already have that functionality, as I mentioned earlier (even though technically I was wrong, my point was not wrong).
so tldr: findPlayerBySteamID & findPlayerBySteamID64 are two useful helper functions, and they could be added, but findPlayerByUserID seems to be useless.
What do you think @TomyLobo @Nebual @AbigailBuccaneer

I'd agree with your final comment there Divran: findPlayerByUserID shouldn't be added without e:userID(), and I don't think enough people will actually want to use userid's to justify the number of users who will inadvertently mix them up with entityid's (like Divran did earlier). Unless anyone has a really good use case?
I'd support a findPlayerBySteam3 method if it was reliable looking, might be easier to just wait for the official lua function though.
Also, I'm sorry about your premature optimization comment train.

Ah, I thought entity:userID was already a thing.

I'd support a findPlayerBySteam3 method if it was reliable looking, might be easier to just wait for the official lua function though.

Reliability isn't the issue, if we were to implement our own version I'd have to guess it'd be quite a bit slower considering what's needed.
Rubat said we'd probably get a update around the end of this month, I'd say wait till then.

Also, I'm sorry about your premature optimization comment train.

Eh, it's nice to know how things work.

UserID is useful for one purpose only: having a unique, but non-persistent id for a player.

Entity ID is not unique (it is reused pretty quickly)
SteamID is persistent (which you might not want in some cases, who knows)

I'd say add it when there's demand, not earlier.

Yeah so @sammyt291 please remove findPlayerByUserID from this PR and then I'll merge it

@Divran Done.


Add support for workspace symbols (project-wide symbol search)

I'm submitting a...

[ ] Bug report.
[x] Feature request.

Feature request
`
Describe the problem you're trying to solve.
The new outline tree greatly speeds up navigation inside a file. Navigation could be improved even further by adding the tokens that populate the outline tree as project symbols. This way, we could jump to them inside and across files with the "project symbols: toggle" command (CTRL+ALT+F on my system). This would behave roughly like the "Go To Function..." command in the built-in Godot editor.

As far as I know, workspace symbols are a feature of language servers, so they depend on the language server in the Godot engine. And it's not supported by Godot currently.
Related: https://github.com/GDQuest/zed-gdscript/issues/28

I am very new to Zed and I'm not sure what features it offers for a project-wide symbols discovery.


Create doonschool.txt

www.doonschool.com

@shreyasminocha Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

Thanks @philipto. Yes, all students are thirteen years and above.


results() returns document instead of array

The following query returns a single Document instead of an array if there is only 1 document that matches the query:
$result = $bd->query()->select('var1')->where('var2','!=','')->results();

The docs state results() should always just return an array, so this was a hard catch ;)

@aalfiann Can you confirm this is an issue when cache is set to true?
Does it work on the first query, and then the second query (cached) does not? Which would mean its related to reading the cache.

No, the cache is fine, It seems there is a problem in run() function.
I'm not sure, but for temporary right now, I just disabled cache when use select(array) and everything is ok.

@aalfiann Ok, thanks for confirming. I will look into this and see if there is a patch to be made.

@AdriaandeJongh This has been resolved from a recent by fix by @faryar76 – Thanks again!


Support custom Actions after launching a WebHook fails or succeeds

WebHookManager.cs has an internal constructor that allows custom Actions to be defined whenever a WebHook is successfully launched or not.  Unfortunately, it is noted in the documentation that "This constructor is intended for unit testing purposes.", and there is currently no practical way to make use of this functionality.
Would you please consider implementing a public constructor with onWebHookSuccess and onWebHookFailure parameters, or providing this functionality in another way?

There now is a mechanism for sending notifications to all users and when doing so you can pass in a predicate which can filter which webhooks should fire. Check it out in dev branch: 6523bb08c502eaef2bc6fe5b88219b4740eaa552


validate_call not working with class/custom callables

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Either at runtime or for static type-checking, both A and A() are callables, but currently they cannot be wrapped by validate_call.
When calling validate_call with A, the following error is raised:
pydantic.errors.PydanticSchemaGenerationError: Unable to generate pydantic-core schema for <class '__main__.A'>. Set `arbitrary_types_allowed=True` in the model_config to ignore this error or implement `__get_pydantic_core_schema__` on your type to fully support it.

If you got this error by calling handler(<some type>) within `__get_pydantic_core_schema__` then you likely need to call `handler.generate_schema(<some type>)` since we do not call `__get_pydantic_core_schema__` on `<some type>` otherwise to avoid infinite recursion.

For further information visit https://errors.pydantic.dev/2.10/u/schema-for-unknown-type

On the other hand, when calling validate_call(A()), the error becomes:
  File "C:\Users\kchon\OneDrive\Desktop\pydantic\pydantic\_internal\_validate_call.py", line 41, in __init__
    self.__name__ = function.__name__
                    ^^^^^^^^^^^^^^^^^
AttributeError: 'A' object has no attribute '__name__'. Did you mean: '__ne__'?

Example Code
from pydantic import validate_call

class A:
    def __call__(self, *args: Any, **kwds: Any) -> Any:
        pass

validate_call(A)  # error
validate_call(A())  # error

Python, Pydantic & OS Version
pydantic version: 2.10.0a1
        pydantic-core version: 2.24.0
          pydantic-core build: profile=release pgo=false
                 install path: C:\Users\kchon\OneDrive\Desktop\pydantic\pydantic
               python version: 3.13.0rc2 (tags/v3.13.0rc2:ec61006, Sep  6 2024, 22:13:49) [MSC v.1940 64 bit (AMD64)]
                     platform: Windows-11-10.0.22631-SP0
             related packages: mypy-1.1.1 pydantic-extra-types-2.9.0 pydantic-settings-2.5.2 typing_extensions-4.12.2
                       commit: e301d4aa

I don't know if we should support these use cases. Some decorators implementations work as expected with this, e.g. functools.cache:
from functools import cache

class A: pass

a = cache(A)
a()
#> <__main__.A object at 0x771f0b734320>
a()
#> <__main__.A object at 0x771f0b734320>

But it does because they (can) consider the decorated function as a callable. In our case, we need to inspect the underlying callable. When doing:
validate_call(A)

Should we look for A.__init__? A.__new__? type(A).__call__?
Considering the explicit function can be used (validate_call(A.__init__), validate_call(A().__call__)), I'm inclined to not support such use cases (and this will simplify your ongoing PR!).

If we were to support one, I'd support type(A).__call__, but I agree with your analysis above.

To my understanding (I am not a CPython expert though), __new__ and __init__ are just special functions called by type.__call__, so type(A).__call__ should be more reasonable. This can be showed by an  experiment:
class AMeta(type):
    def __call__(self, *args, **kwargs):
        print("AMeta.__call__")

class A(metaclass=AMeta):
    def __init__(self):
        print("A")

    def __new__(cls):
        print("new")
        return super().__new__(cls)

A()  # print `AMeta.__call__` only, not `new` nor `A`

That being said, I think being more restrictive can simplify the code without losing much functionality. Perhaps throwing an informative error message (as in the case of classmethod and staticmethod currently) would help.

To my understanding (I am not a CPython expert though), __new__ and __init__ are just special functions called by type.__call__, so type(A).__call__ should be more reasonable. This can be showed by an experiment:

What I meant is validate_call will generate a core schema for a given function. For any class, type(cls).__call__ exist but the question is should it be used to generate the core schema (because we need to analyze type annotations of the given function).

That being said, I think being more restrictive can simplify the code without losing much functionality. Perhaps we could only allow several argument types (FunctionType, LambdaType, ...) and throw an informative error message otherwise (as in the case of classmethod and staticmethod currently).

Sounds like a good idea! And A().__call__ may be supported at some point, but imo we should wait for an actual feature request ("actual" as in "someone tried this and it failed") for this so that we don't unnecessarily add complexity to the validate_call code.


Chapter cover disable extraction

Implemented, update settings and set extract_chapter_covers in settings.py https://github.com/zachstultz/komga-cover-extractor/commit/009b7355aedf6b4cb35a96eb454ecebefb0227a5 https://github.com/zachstultz/komga-cover-extractor/commit/aa3f5797495337531f93d985d779d9265e493c4e


Fix mistake in nixos configuration documentation

Mismatched versions of nixpkgs can cause bugs with certain packages.

Thanks!


Pod creation failed using Sriov network with new SRIOV NICs

We are trying to validate Marvell card in Sriov operator using the helm chart. Before executing the helm command, I have updated Marvell card details in configmap.( Qlogic_qede_QL45000_50G: 1077 1654 1664).
https://github.com/k8snetworkplumbingwg/sriov-network-operator/blob/master/deployment/sriov-network-operator/templates/configmap.yaml
After execution of helm command, operator and config-daemon pods are running successfully but while trying to create pod using sriov network, its getting failed.

Deploy SR-IOV Network Operator
Install Operator
$ helm install -n sriov-network-operator --create-namespace --wait sriov-network-operator ./
Created SriovNetworkNodePolicy
Created SriovNetwork

NAME                                          READY   STATUS              RESTARTS   AGE
pod/samplepod                                 0/1     ContainerCreating   0          4h53m
pod/sriov-device-plugin-z95fc                 1/1     Running             0          130m
pod/sriov-network-config-daemon-xg9v7         3/3     Running             3          139m
pod/sriov-network-operator-6567584db9-t9gkh   1/1     Running             0          6h33m
NAME                                         DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR                                                 AGE
daemonset.apps/sriov-device-plugin           1         1         1       1            1           beta.kubernetes.io/os=linux,node-role.kubernetes.io/worker=   6h25m
daemonset.apps/sriov-network-config-daemon   1         1         1       1            1           beta.kubernetes.io/os=linux,node-role.kubernetes.io/worker=   6h33m
NAME                                     READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/sriov-network-operator   1/1     1            1           6h33m
NAME                                                DESIRED   CURRENT   READY   AGE
replicaset.apps/sriov-network-operator-6567584db9   1         1         1       6h33m
nodestates:-
Allocatable:
cpu:                       39600m
ephemeral-storage:         92115Mi
hugepages-1Gi:             0
hugepages-2Mi:             0
memory:<PHONE_NUMBER>55
openshift.io/marvellnics:  3
pods:                      110
VF status on node:-
status:
interfaces:
- Vfs:
- deviceID: "1664"
driver: qede
mac: 9a:c0:0e:90:15:01
mtu: 1500
name: ens3f0v0
pciAddress: "0000:37:02.0"
vendor: "1077"
vfID: 0
pod logs:-
Warning  FailedCreatePodSandBox  12m (x874 over 4h3m)     kubelet  (combined from similar events): Failed to create pod sandbox: rpc error: code = Unknown desc = [failed to set up sandbox container "b821ee49cbea3bef6dd2fbcd92798dd4ea3e256b39ad79a23d01dbaf7bdf3765" network for pod "samplepod": networkPlugin cni failed to set up pod "samplepod_sriov-network-operator" network: Multus: [sriov-network-operator/samplepod/]: error getting pod: Get "https://[<IP_ADDRESS>]:443/api/v1/namespaces/sriov-network-operator/pods/samplepod?timeout=1m0s": dial tcp <IP_ADDRESS>:443: i/o timeout, failed to clean up sandbox container "b821ee49cbea3bef6dd2fbcd92798dd4ea3e256b39ad79a23d01dbaf7bdf3765" network for pod "samplepod": networkPlugin cni failed to teardown pod "samplepod_sriov-network-operator" network: Multus: [sriov-network-operator/samplepod/]: error getting pod: Get "https://[<IP_ADDRESS>]:443/api/v1/namespaces/sriov-network-operator/pods/samplepod?timeout=1m0s": dial tcp <IP_ADDRESS>:443: i/o timeout]
Please help me out with this issue... Thank you in advance

i see : Get "https://[<IP_ADDRESS>]:443/api/v1/namespaces/sriov-network-operator/pods/samplepod?timeout=1m0s": dial tcp <IP_ADDRESS>:443: i/o timeout] are you able to access kubernetes API with this address ?
seems like multus failed to access k8s API server.

Hi adrianchiris,
Thank you for your response...
To test multus cni, I have created macvlan network then deployed a sample pod and assgined macvlan with the help of multus successfully but i'm getting error while using sriov network.

please find the below sriov logs
sriov-device-plugin:-
I0207 19:24:49.249678       1 manager.go:51] Using Kubelet Plugin Registry Mode
I0207 19:24:49.249718       1 main.go:44] resource manager reading configs
I0207 19:24:49.249823       1 manager.go:79] raw ResourceList: {"resourceList":[{"resourceName":"marvellnics","selectors":{"vendors":["1077"],"devices":["1664"],"rootDevices":["0000:37:00.0"],"IsRdma":false,"NeedVhostNet":false},"SelectorObj":null}]}
I0207 19:24:49.249916       1 factory.go:166] net device selector for resource marvellnics is &{DeviceSelectors:{Vendors:[1077] Devices:[1664] Drivers:[] PciAddresses:[]} PfNames:[] RootDevices:[0000:37:00.0] LinkTypes:[] DDPProfiles:[] IsRdma:false NeedVhostNet:false}
I0207 19:24:49.249932       1 manager.go:99] unmarshalled ResourceList: [{ResourcePrefix: ResourceName:marvellnics DeviceType:netDevice Selectors:0xc00012ed20 SelectorObj:0xc000264f70}]
I0207 19:24:49.249959       1 manager.go:200] validating resource name "marvell.com/marvellnics"
I0207 19:24:49.249964       1 main.go:60] Discovering host devices
I0207 19:24:49.297072       1 netDeviceProvider.go:84] netdevice AddTargetDevices(): device found: 0000:37:00.0 02              QLogic Corp.            FastLinQ QL45000 Series 50GbE Controller
I0207 19:24:49.297557       1 netDeviceProvider.go:123] excluding interface ens3f0:  default route found: {Ifindex: 2 Dst:  Src:  Gw: <IP_ADDRESS> Flags: [] Table: 254 Realm: 0}
I0207 19:24:49.297619       1 netDeviceProvider.go:84] netdevice AddTargetDevices(): device found: 0000:37:00.1 02              QLogic Corp.            FastLinQ QL45000 Series 50GbE Controller
I0207 19:24:49.298112       1 netDeviceProvider.go:84] netdevice AddTargetDevices(): device found: 0000:37:02.0 02              QLogic Corp.            FastLinQ QL45000 Series Gigabit Ether...
I0207 19:24:49.298435       1 netDeviceProvider.go:84] netdevice AddTargetDevices(): device found: 0000:37:02.1 02              QLogic Corp.            FastLinQ QL45000 Series Gigabit Ether...
I0207 19:24:49.298640       1 netDeviceProvider.go:84] netdevice AddTargetDevices(): device found: 0000:37:02.2 02              QLogic Corp.            FastLinQ QL45000 Series Gigabit Ether...
I0207 19:24:49.298869       1 main.go:66] Initializing resource servers
I0207 19:24:49.298892       1 manager.go:105] number of config: 1
I0207 19:24:49.298901       1 manager.go:109]
I0207 19:24:49.298906       1 manager.go:110] Creating new ResourcePool: marvellnics
I0207 19:24:49.298913       1 manager.go:111] DeviceType: netDevice
I0207 19:24:49.299653       1 utils.go:71] Devlink query for eswitch mode is not supported for device 0000:37:00.1.
I0207 19:24:49.300261       1 utils.go:71] Devlink query for eswitch mode is not supported for device 0000:37:02.0.
I0207 19:24:49.300692       1 utils.go:71] Devlink query for eswitch mode is not supported for device 0000:37:02.1.
I0207 19:24:49.301155       1 utils.go:71] Devlink query for eswitch mode is not supported for device 0000:37:02.2.
I0207 19:24:49.301263       1 factory.go:106] device added: [pciAddr: 0000:37:02.0, vendor: 1077, device: 1664, driver: qede]
I0207 19:24:49.301270       1 factory.go:106] device added: [pciAddr: 0000:37:02.1, vendor: 1077, device: 1664, driver: qede]
I0207 19:24:49.301273       1 factory.go:106] device added: [pciAddr: 0000:37:02.2, vendor: 1077, device: 1664, driver: qede]
I0207 19:24:49.301290       1 manager.go:139] New resource server is created for marvellnics ResourcePool
I0207 19:24:49.301297       1 main.go:72] Starting all servers...
I0207 19:24:49.301707       1 server.go:199] starting marvellnics device plugin endpoint at: marvell.com_marvellnics.sock
I0207 19:24:49.302562       1 server.go:226] marvellnics device plugin endpoint started serving
I0207 19:24:49.302601       1 main.go:77] All servers started.
I0207 19:24:49.302607       1 main.go:78] Listening for term signals
I0207 19:24:49.641368       1 server.go:110] Plugin: marvell.com_marvellnics.sock gets registered successfully at Kubelet
I0207 19:24:49.641405       1 server.go:134] ListAndWatch(marvellnics) invoked
I0207 19:24:49.641420       1 server.go:142] ListAndWatch(marvellnics): send devices &ListAndWatchResponse{Devices:[]*Device{&Device{ID:0000:37:02.2,Health:Healthy,Topology:&TopologyInfo{Nodes:[]*NUMANode{&NUMANode{ID:0,},},},},&Device{ID:0000:37:02.0,Health:Healthy,Topology:&TopologyIn
sriov pod logs
Type     Reason                  Age                From               Message

Normal   Scheduled               24s                default-scheduler  Successfully assigned sriov-network-operator/podtest to worker.jagan.local
Warning  FailedCreatePodSandBox  26s                kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "79be8238c180eb4a3cda9b2f87c0cea62e41924c4a22e61f94ce8071a43295b5" network for pod "podtest": networkPlugin cni failed to set up pod "podtest_sriov-network-operator" network: Multus: [sriov-network-operator/podtest/]: error loading k8s delegates k8s args: TryLoadPodDelegates: error in getting k8s network for pod: GetNetworkDelegates: failed getting the delegate: getKubernetesDelegate: failed to get a ResourceClient instance: getKubeletClient: error getting pod resources from client: getPodResources: failed to list pod resources, &{0xc0000f4a80}.Get(_) = , rpc error: code = Unimplemented desc = unknown service v1.PodResourcesLister
Warning  FailedCreatePodSandBox  24s                kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "d827da37e0c99ff1e5ab7102efbd9a832362e3851a0e33c140fac1aabb63a9ac" network for pod "podtest": networkPlugin cni failed to set up pod "podtest_sriov-network-operator" network: Multus: [sriov-network-operator/podtest/]: error loading k8s delegates k8s args: TryLoadPodDelegates: error in getting k8s network for pod: GetNetworkDelegates: failed getting the delegate: getKubernetesDelegate: failed to get a ResourceClient instance: getKubeletClient: error getting pod resources from client: getPodResources: failed to list pod resources, &{0xc0002da700}.Get() = _, rpc error: code = Unimplemented desc = unknown service v1.PodResourcesLister
Please help me out with this issue....

hmm. perhaps related to : https://github.com/k8snetworkplumbingwg/multus-cni/issues/717
what is your k8s and multus-cni versions ?

Hi Adrian Chris,
I have tested Marvell (Qlogic_qede_QL45000_50G: 1077 1654 16640) card in two versions of K8s cluster (v1.19 & v1.23) using SRIOV operator-based.
Now I'm trying to validate vendor card using hack/run-e2e-test-kind.sh script with system-service but geeting error.
docker build -t sriov-network-operator:latest -f /root/sriov-network-operator/Dockerfile /root/sriov-network-operator
Sending build context to Docker daemon 191.5 MB
Step 1/9 : FROM golang:1.17 AS builder
Error parsing reference: "golang:1.17 AS builder" is not a valid repository/tag: invalid reference format
Exit code: '1'. Sleeping '10' seconds before retrying
docker build -t sriov-network-operator:latest -f /root/sriov-network-operator/Dockerfile /root/sriov-network-operator
Sending build context to Docker daemon 191.5 MB
Step 1/9 : FROM golang:1.17 AS builder
Error parsing reference: "golang:1.17 AS builder" is not a valid repository/tag: invalid reference format
Please provide the process of validating cards.


chore: enable batch ssh commands

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1119  👈
#1118
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


Arome/28690/Partners page updated

Changes:

Updated partners page

Type of change

[ ] Bug fix
[ ] New feature
[x] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

Do we not also need to include in off-canvas-menu?


[11.x] Introduce method Rule::array()

This PR introduces a new array method to \Illuminate\Validation\Rule.
Currently, it is often cumbersome to validate multiple array keys through the array rule. At the same time, if intending to using constants, or enums, we may end up in defining our rule such as:
['array:' . MyBackedEnum::VALUE->value . ',' . MyBackedEnum::VALUE_2->value]

The goal of this change is to improve code readability and prevent uncaught typos when validating array keys, but to also improve the existing functionalities by introducing support for Arrayable & lists of enums.
The newly introduced ArrayRule works very similar to the already existing In rule, by stringifying to the original array string rule, introducing no breaking changes.
In summary, the following is now possible:
Rule::array();

Rule::array('key_1', 'key_2', 'key_3');

Rule::array(['key_1', 'key_2', 'key_3']);

Rule::array(collect(['key_1', 'key_2', 'key_3']));

Rule::array([UnitEnum::key_1, UnitEnum::key_2, UnitEnum::key_3]);

Rule::array([BackedEnum::key_1, BackedEnum::key_2, BackedEnum::key_3]);

@Jacobs63 why not use Rule::enum() for matching against an enum's cases?

@Jacobs63 why not use Rule::enum() for matching against an enum's cases?

The array rule does not serve for validating values, it can either validate, that:

The input value is of type array
The input value is of type array and only contains specific keys

Unlike enum rule, which validates valus and can validate that:

The input value matches the enum name (if UnitEnum)
The input value matches the enum value (if BackedEnum)

They're two different validation rules.

@Jacobs63 why not use Rule::enum() for matching against an enum's cases?

The array rule does not serve for validating values, it can either validate, that:

The input value is of type array
The input value is of type array and only contains specific keys

Unlike enum rule, which validates values and can validate that:

The input value matches the enum name (if UnitEnum)
The input value matches the enum value (if BackedEnum)

They're two different validation rules.

My bad! I misunderstood the usecase, thanks for the explaination though.

What's the difference between Rule::in and Rule::array?

What's the difference between Rule::in and Rule::array? Why another method?

array defines keys that validated value (array) can have. in is list of allowed values for validated value.

@decadence Thank you. I wasn't aware there's an array in the first place 😅
https://laravel.com/docs/11.x/validation#rule-array


Georeferences boundaries and values interpretation- Task Comprehensive specimen digitization and DwC Import

While working with the import task I noticed an error in the original dataset I was working with. A value of -105754 for the verbatim longitude (decimal point is missing).

This record is imported with no Error warning and when checking for the Collecting event created for this specimen record, the value recorded for longitude is 0.

Although the verbatim value is correctly imported.

When I create a new collecting event from the Comprehensive Specimen Digitization task with this same value for the verbatim longitude, when clicking on the Create the Georeference from Verbatim with the green button, the value shown is 1.

I tried a different number like -20865 for the verbatim longitude and the Georeference created has -20 for longitude.

I saved the record, closed it, and when I opened it again the longitude value was 86 positive. Clearly it goes all the way around the globe.

But the collecting event record still has a 0 for longitude value.

Should TW prevent saving values outside the "natural" range? Or perhaps do not create georeferences when values are big?

We do not validate verbatim fields, as they are intended to be truly verbatim, as written on the label.

When the verbatim is used to create a georeference in the collecting event forms then problems can be introduced

because initial values aren't validated
because the parse we have may not be accurate for some values

@proceps please update the parser in the CE form to fail completely (no values returned for things like -123123123.

@LocoDelAssembly The decimalLatitude should definitely raise an error when the Georeference is created.

@mjy the parser is restrictive, it will not recognize the value like -123123123. It only returns legal values. You can only enter it manually. We should probably make georeference more restrictive.

@proceps I will look to see how the VerbatimGeoreference is built, I assumed it was using the same parser.

I was checking the dataset and the column name for this value is decimalLongitude, it is not verbatimLongitude. Why does the import assume it is verbatim?

verbatim_label in TW covers the lat and long as they are in any format (not validated). The proper georeference in TW is always normalized to decimals (and those should be validated). Normalized data are used for calculation and mapping. verbatim, just for a reference.

@LocoDelAssembly can you share the alogorithm for those values.  IIRC I think you placed decimal in verbatim to reflect that it was verbatim from the datasheet, perhaps we need to rethink that.

Importer-side is storing in vertatim_* and then Georeference::VerbatimData.create! with the collecting event containing the coordinates.
Related issue: https://github.com/SpeciesFileGroup/taxonworks/issues/2097 << Please keep discussion of definitive lat/long mapping there. Also, what is the dwc indexer using to generate decimalLatitude and decimalLongitude?


Languages should only be translated into their own language

Checklist

[x] Bug Report - The issue you are reporting might be there already. Please search on the issues tracker before creating or reporting an issue.
[x] Before reporting, please ensure (if you can) that you have tested it with the latest (dev) version of the software.
[x] Don't forget to tag your issue, it makes it easier to discover.
Summary:

When choosing a language, the items should only be translated into their own language, not the language that is currently set.
Steps to reproduce:
Choose a language in prefs
Expected behavior:
The items should only be translated into their own language, not the language that is currently set.

français - fr
English - en
Español - es

Observed behavior:
Languages are translated into language of the app.
See screenshot - I know that English is the top of the list, but I should be able to see English in English even if the app is set to Hindi. Only "Hindi" should be in Hindi
Screenshots:

In progress)


Create a roadmap page

User request: Can we consider creating a roadmap page where users will be able to track upcoming features?

+1

Sure, we should publish a roadmap. I'd prefer a document on http://tabrisjs.com (somewhere next to the FAQ) rather than content tracked in the git repository.

We can use the existing infrastructure that support http://developer.eclipsesource.com/tabris/roadmap/
I would propose using that immediately and then review how it serves us after let's say two more sprints.

@gosia-eclipsesource What's the solution?

@ralfstx We now publish the roadmap as a section on Grand Central:
https://tabrisjs.com/grand-central

Great, thanks!


Remove unnecessary asserts

Upgrading the framework to null safety made a lot of the asserts unnecessary. For example, the ThemeData.raw() constructor asserts that it's parameters are non-null, even though they are required and non-null types:
class ThemeData with Diagnosticable {
  // ...
  final VisualDensity visualDensity;

  const ThemeData.raw({
    required this.visualDensity,
    // ...
  }) : assert(visualDensity != null),

Hi! I'd like to work on this. Can we make a tasklist of the files that need to be refactored?

Hi @johnpryan. Null-safe code can be called from non null-safe mode. I remember that there were a few PRs that tried to address similar things, for instance https://github.com/flutter/flutter/pull/88359, https://github.com/flutter/flutter/pull/81438 and https://github.com/flutter/flutter/pull/85309, but as far as I know, the plan is too keep them until non-nnbd code support will be dropped entirely, which will happen eventually, but not yet.
Also, this issue seems similar to https://github.com/flutter/flutter/issues/85308.

Closing as a dupe of https://github.com/flutter/flutter/issues/85308


Got white line between Containers

Steps to Reproduce

A Column with three Containers which height is 39.6, shows white lines between Containers.

Code sample

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Column(
        children: <Widget>[
          Container(color: Colors.red, height: 39.6,),
          Container(color: Colors.red, height: 39.6,),
          Container(color: Colors.red, height: 39.6,),
        ],
      ),
    );
  }
}

This is the screenshot.

Logs

Hi @iamjinge, I believe this is a duplicate of https://github.com/flutter/flutter/issues/14288.
Kindly follow up in the linked issue for updates.
Thank you


Telegram reports about sites, that was removed

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
Uptime_Bot, [17.04.2023 15:27]
[svet24.ru] [🔴 Down] connect EHOSTUNREACH <IP_ADDRESS>:443
Uptime_Bot, [17.04.2023 15:28]
[svet24.ru] [✅ Up] 200 - OK
But these projects are removed from Uptime Kuma long ago. Looks like some issues in database
👟 Reproduction steps
getting notifications about deleted sites
👀 Expected behavior
Looks like some issues in database
😓 Actual Behavior
notificatins
🐻 Uptime-Kuma Version
1.21.2
💻 Operating System and Arch
Ubuntu 20.04 ч64
🌐 Browser
Chrome lastest
🐋 Docker Version
No response
🟩 NodeJS Version
No response
📝 Relevant log output
No response

@louislam should I work on this? could you please assign this bug to me?

@louislam should I work on this? could you please assign this bug to me?

Yes, of course, thank you. However, it is hard to be confirmed as there is no reproduce steps.

@lustmordtheblackpost checked in every possible way. But, in database the notification linked with the monitor we delete also gets deleted due to the "CASCADE ON DELETE" constraint. Could you provide the way how I could reproduce this issue?

By the last update I can't reproduce too. I'll monitor it too.

Closing as cannot-reproduce


Update gogo/prototbuf and nghttp2 dependencies

Description: Update dependencies - gogo/prototbuf 1.1.1 (release notes) & nghttp2 1.34.0 (release notes). nghttp2 update implements Implement RFC 8441 which should enable H2 with websockets /cc @alyssawilk
Risk Level: Low
Testing: bazel test //test/... and running on local instances
Docs Changes: none required
Release Notes: none required

Tagging to medium just because we don't track individual changes in those libraries.
@ccaraman @junr03 so it's on their radar for deployment.


Add language method to OS::Mac.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

This adds a language method to the MacOS module, to be used in casks, such as https://github.com/caskroom/homebrew-cask/pull/24343.
cc @Homebrew/cask

Feels a little weird for this to be on the MacOS class but I don't have strong feelings either way if there's no other logical place.

Feels a little weird for this to be on the MacOS class

Where else would you put the OS language, if not in the OS module?

Where else would you put the OS language, if not in the OS module?

👍

Ok, should be good to go now.

Merging now in order for https://github.com/caskroom/homebrew-cask/pull/24343 and https://github.com/caskroom/homebrew-versions/pull/2558 to hopefully go green.

@reitermarkus Did you miss this:

Feels weird on hindsight to return a comma-separated string. .delete("()\"\n ").split(",") seems to do what you want and return an array.

I'd have preferred if you disagreed that it was discussed rather than merged?

It doesn't return a comma-separated string. The sub method call strips away the first comma and everything after it, leaving only the first value.

Ok. Why does this only care about the first value? That feels relatively opinionated for something that's a OS wrapper around a method? Regardless: this is discussion we should really be having before this was merged.

@reitermarkus Did you miss this:

Yes, sorry, I missed that comment.
For context, please take a look at https://github.com/caskroom/homebrew-cask/pull/24343/files#diff-d7a1b27ed779bc1601aadf2a1c3652f7R12 on how this is supposed to be used.
Alternatively, we could have MacOS.languages return an array, and MacOS.language return languages.first.

@reitermarkus No worries. 👍 to having languages return an array and language return languages.first 👍


Downloading error

I downloaded both the ragdoll player and the view and the view loads in fine, its just the ragdoll player doesnt and wont load in bonelib or melonloader, im on melonloader 6.4

Same on quest