Allow Optional[strawberry.auto]

There are good possible use cases for allowing the usage of Optional[strawberry.auto] when using pydantic support.

Feature Request Type

[ ] Core functionality
[x] Alteration (enhancement/optimization) of existing feature(s)
[ ] New behavior

Description
I am using (and loving) the strawberry pydantic support. I presently have the use case for creating an input type based on a pydantic model, but I want all of the fields to be optional. For example:
class User(BaseModel):
  name: str
  age: int

# I'd like to do this, but it does not work
@strawberry.experimental.pydantic.input(model=User)
class UserInput:
  name: Optional[strawberry.auto]
  age: Optional[strawberry.auto]

The reason I want to do this, is to allow the API user to provide any of the fields in UserInput, and update the underlying user only for the fields that were present.
The error that running this code gave me is: TypeError: Union[arg, ...]: each arg must be a type. Got <auto>.

I am running into the same issue.
It would be great to be able to add Optional[strawberry.auto] when defining inputs with the Pydantic integration

bump! some of my types are constructed from functions that work with auto, but are not around within my codebase so I can't set them


Best way to integrate with django migration system

This isn't a bug or feature request but rather a discussion to understand the best way to integrate procrastinate within a Django project, in particular, at the SQL level, as Django as its own migration system.
My first attempt was to use raw SQL and execute each sql/migrations/delta* delta file, like this:
# Generated by Django 3.0.8 on 2020-07-18 08:39
import os
from django.db import connection
from django.db import migrations
from django.db import transaction

def apply_sql(apps, schema_editor):
    import procrastinate
    sql_path = os.path.join(
        os.path.dirname(procrastinate.__file__),
        'sql',
        'migrations',
    )

    migrations = [
        "baseline-0.5.0.sql",
        "delta_0.5.0_001_drop_started_at_column.sql",
        "delta_0.5.0_002_drop_started_at_column.sql",
        "delta_0.5.0_003_drop_procrastinate_version_table.sql",
        "delta_0.6.0_001_fix_procrastinate_fetch_job.sql",
        "delta_0.7.1_001_fix_trigger_status_events_insert.sql",
        "delta_0.8.1_001_add_queueing_lock_column.sql",
        "delta_0.10.0_001_close_fetch_job_race_condition.sql",
        "delta_0.10.0_002_add_defer_job_function.sql",
        "delta_0.11.0_003_add_procrastinate_periodic_defers.sql",
        "delta_0.12.0_001_add_foreign_key_index.sql",
    ]

    with connection.cursor() as cursor:
        with transaction.atomic():
            for name in migrations:
                full_path = os.path.join(sql_path, name)
                with open(full_path, 'rb') as f:
                    print('Applying {}'.format(full_path))
                    sql = f.read().decode()
                    cursor.execute(sql)

def rewind(*args, **kwargs):
    pass

class Migration(migrations.Migration):

    dependencies = [
        ('common', '0008_auto_20200701_1317'),
    ]

    operations = [
        migrations.RunPython(apply_sql, rewind)
    ]

However, this crashes, with the following output:
python manage.py migrate common 0009

Operations to perform:
  Target specific migration: 0009_procrastinate, from common
Running migrations:
  Applying common.0009_procrastinate...Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/baseline-0.5.0.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.5.0_001_drop_started_at_column.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.5.0_002_drop_started_at_column.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.5.0_003_drop_procrastinate_version_table.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.6.0_001_fix_procrastinate_fetch_job.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.7.1_001_fix_trigger_status_events_insert.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.8.1_001_add_queueing_lock_column.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.10.0_001_close_fetch_job_race_condition.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.10.0_002_add_defer_job_function.sql
Applying /venv/lib/python3.7/site-packages/procrastinate/sql/migrations/delta_0.11.0_003_add_procrastinate_periodic_defers.sql
Traceback (most recent call last):
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql)
psycopg2.errors.SyntaxError: syntax error at or near ";"
LINE 9: DROP FUNCTION IF EXISTS procrastinate_defer_job;
                                                       ^

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "manage.py", line 27, in <module>
    execute_from_command_line(sys.argv)
  File "/venv/lib/python3.7/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/venv/lib/python3.7/site-packages/django/core/management/__init__.py", line 395, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/venv/lib/python3.7/site-packages/django/core/management/base.py", line 328, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/venv/lib/python3.7/site-packages/django/core/management/base.py", line 369, in execute
    output = self.handle(*args, **options)
  File "/venv/lib/python3.7/site-packages/django/core/management/base.py", line 83, in wrapped
    res = handle_func(*args, **kwargs)
  File "/venv/lib/python3.7/site-packages/django/core/management/commands/migrate.py", line 233, in handle
    fake_initial=fake_initial,
  File "/venv/lib/python3.7/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/venv/lib/python3.7/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/venv/lib/python3.7/site-packages/django/db/migrations/executor.py", line 245, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/venv/lib/python3.7/site-packages/django/db/migrations/migration.py", line 124, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/venv/lib/python3.7/site-packages/django/db/migrations/operations/special.py", line 190, in database_forwards
    self.code(from_state.apps, schema_editor)
  File "/app/funkwhale_api/common/migrations/0009_procrastinate.py", line 37, in apply_sql
    cursor.execute(sql)
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 100, in execute
    return super().execute(sql, params)
  File "/venv/lib/python3.7/site-packages/cacheops/transaction.py", line 93, in execute
    result = self._no_monkey.execute(self, sql, params)
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 68, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 77, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 86, in _execute
    return self.cursor.execute(sql, params)
  File "/venv/lib/python3.7/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql)
django.db.utils.ProgrammingError: syntax error at or near ";"
LINE 9: DROP FUNCTION IF EXISTS procrastinate_defer_job;

Execution of delta_0.11.0_003_add_procrastinate_periodic_defers.sql fails with a syntax error, which I don't understand unfortunately. If you have some ideas, I can definitely try these, and possibly document Django integration when I have a working setup :)

Hm, you're right, if we expect people using Django to use procrastinate, we'll need to ship a way to transform PG migrations SQL files into Django migrations. Your approach seems nice, but I'd tend to try and generate one Django migration per SQL migration file (whereas if I understand your code, you have all SQL migrations in a single Django migration). But that's a detail, and doesn't help your problem.
Can I ask why you used a RunPython and not a RunSQL ?
I can try to have a look, but I don't have a clue at this point why this fails. Or you can have a look on your side. Or we can do this in pair programming if you want, like in the good ol' days :)
I'm perfectly fine adding a Django submodule for simplifying the usage of procrastinate with django, if it's deemed preferable to a django-procrastinate dedicated package.

Hm, you're right, if we expect people using Django to use procrastinate, we'll need to ship a way to transform PG migrations SQL files into Django migrations. Your approach seems nice, but I'd tend to try and generate one Django migration per SQL migration file (whereas if I understand your code, you have all SQL migrations in a single Django migration). But that's a detail, and doesn't help your problem.

I'm not sure about the "1 procrastinate migration <-> 1 django migration" approach. As far as I can tell, it doesn't bring any additional value and make integration with the app way harder (a dozen files to add, instead of only one).

Can I ask why you used a RunPython and not a RunSQL ?

I can try that, I wanted to avoid file I/O at import time, which RunSQL would require.
IMHO, the easiest option would be as you suggested, have a contrib.django package which would be a django app, and contain the proper migrations. Integrating procrastinate in a django project would then only require INSTALLED_APPS = [..., "procrastinate.contrib.django", ...] and python manage.py migrate.
However, it would increase the maintenance burden on procrastinate side, possibly by having duplicated SQL code (though it could probably be automated).

If I use RunSQL, I end up with the same error. This is the code I used:

# Generated by Django 3.0.8 on 2020-07-18 08:39
import os
from django.db import migrations

def get_sql(name):
    import procrastinate
    sql_path = os.path.join(
        os.path.dirname(procrastinate.__file__),
        'sql',
        'migrations',
    )

    full_path = os.path.join(sql_path, name)
    with open(full_path, 'rb') as f:
        return f.read().decode()

class Migration(migrations.Migration):

    dependencies = [
        ('common', '0008_auto_20200701_1317'),
    ]

    operations = [
        migrations.RunSQL(get_sql("baseline-0.5.0.sql")),
        migrations.RunSQL(get_sql("delta_0.5.0_001_drop_started_at_column.sql")),
        migrations.RunSQL(get_sql("delta_0.5.0_002_drop_started_at_column.sql")),
        migrations.RunSQL(get_sql("delta_0.5.0_003_drop_procrastinate_version_table.sql")),
        migrations.RunSQL(get_sql("delta_0.6.0_001_fix_procrastinate_fetch_job.sql")),
        migrations.RunSQL(get_sql("delta_0.7.1_001_fix_trigger_status_events_insert.sql")),
        migrations.RunSQL(get_sql("delta_0.8.1_001_add_queueing_lock_column.sql")),
        migrations.RunSQL(get_sql("delta_0.10.0_001_close_fetch_job_race_condition.sql")),
        migrations.RunSQL(get_sql("delta_0.10.0_002_add_defer_job_function.sql")),
        migrations.RunSQL(get_sql("delta_0.11.0_003_add_procrastinate_periodic_defers.sql")),
        migrations.RunSQL(get_sql("delta_0.12.0_001_add_foreign_key_index.sql")),
    ]

Maybe it's related to my postgresql installation, though it's a pretty basic postgres:11 docker installation. I'll try piping the SQL files directly to postres.

So, piping the migrations directly to postgresql raise the same bug:
docker-compose -f dev.yml exec -T postgres psql -U postgres -P < ../../procrastinate/procrastinate/sql/migrations/delta_0.11.0_003_add_procrastinate_periodic_defers.sql

CREATE TABLE
ERROR:  syntax error at or near ";"
LINE 1: DROP FUNCTION IF EXISTS procrastinate_defer_job;
                                                       ^
CREATE FUNCTION
ERROR:  syntax error at or near ";"
LINE 1: DROP FUNCTION IF EXISTS procrastinate_defer_periodic_job;
                                                                ^
CREATE FUNCTION

The error is still there, but because I piped a whole file, it doesn't crash immediatly.
As far as I know, it's because the DROP FUNCTION statement requires you to include the arguments of the function, like this: DROP FUNCTION sqrt(integer);
It's probably failing in other setups but silently ignored.

Hm, nice find, thanks !

Hm, the doc says:

If the function name is unique in its schema, it can be referred to without an argument list:

DROP FUNCTION update_employee_salaries;

So, I still don't understand how this is a syntax error.

On my machine:
$ psql
psql (12.3, server 10.10 (Debian 10.10-1.pgdg90+1))
Type "help" for help.

procrastinate=# DROP FUNCTION IF EXISTS procrastinate_defer_job;
DROP FUNCTION
procrastinate=# DROP FUNCTION IF EXISTS procrastinate_defer_job;
NOTICE:  function procrastinate_defer_job() does not exist, skipping
DROP FUNCTION
procrastinate=# DROP FUNCTION IF EXISTS procrastinate_defer_periodic_job;
DROP FUNCTION

I thought it may be related to an inconsistent state in my setup, so I'v tried with a clean postgres 9.6 container. Launch a container:
docker run -e POSTGRES_HOST_AUTH_METHOD=trust --name test-procrastinate postgres:9.6 postgres -c log_min_duration_statement=0

Apply the migrations:
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/baseline-0.5.0.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.5.0_001_drop_started_at_column.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.5.0_002_drop_started_at_column.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.5.0_003_drop_procrastinate_version_table.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.6.0_001_fix_procrastinate_fetch_job.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.7.1_001_fix_trigger_status_events_insert.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.8.1_001_add_queueing_lock_column.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.10.0_001_close_fetch_job_race_condition.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.10.0_002_add_defer_job_function.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.11.0_003_add_procrastinate_periodic_defers.sql
docker exec -i test-procrastinate psql -Upostgres  < ../../procrastinate/procrastinate/sql/migrations/delta_0.12.0_001_add_foreign_key_index.sql

Which gives the following output:
CREATE FUNCTION
ALTER TABLE
DROP TABLE
CREATE FUNCTION
DROP TRIGGER
CREATE TRIGGER
ALTER TABLE
CREATE INDEX
DROP TABLE
CREATE INDEX
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE TABLE
ERROR:  syntax error at or near ";"
LINE 1: DROP FUNCTION IF EXISTS procrastinate_defer_job;
                                                       ^
CREATE FUNCTION
ERROR:  syntax error at or near ";"
LINE 1: DROP FUNCTION IF EXISTS procrastinate_defer_periodic_job;
                                                                ^
CREATE FUNCTION
CREATE INDEX

How, trying with a postgres 10 container:
NOTICE:  extension "plpgsql" already exists, skipping
CREATE EXTENSION
CREATE TABLE
CREATE TYPE
CREATE TYPE
CREATE TABLE
CREATE TABLE
CREATE TABLE
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE INDEX
CREATE TRIGGER
CREATE TRIGGER
CREATE TRIGGER
CREATE TRIGGER
CREATE INDEX
CREATE FUNCTION
ALTER TABLE
DROP TABLE
CREATE FUNCTION
DROP TRIGGER
CREATE TRIGGER
ALTER TABLE
CREATE INDEX
DROP TABLE
CREATE INDEX
CREATE FUNCTION
CREATE FUNCTION
CREATE FUNCTION
CREATE TABLE
DROP FUNCTION
CREATE FUNCTION
DROP FUNCTION
NOTICE:  function procrastinate_defer_periodic_job() does not exist, skipping
CREATE FUNCTION
CREATE INDEX

So we can reasonnably say this is related to my Postgres version, and the behaviour change starting in Postgres 10.
Now, back to the initial discussion, regarding the opportunity to package procrastinate as a django app, would you accept a contribution based on https://github.com/peopledoc/procrastinate/issues/280#issuecomment-660498268?

would you accept a contribution based on #280 (comment)?

Yes! Definitely! As I said:

I'm perfectly fine adding a Django submodule for simplifying the usage of procrastinate with django, if it's deemed preferable to a django-procrastinate dedicated package.

Also:

Your approach seems nice, but I'd tend to try and generate one Django migration per SQL migration file (whereas if I understand your code, you have all SQL migrations in a single Django migration).

And I stand by this if it's not too hard to do. This will make it much easier when we issue releases in the future: people will update procrastinate, then manage.py migrate and it should "just work".

Oh, also, we might try to explicit supported PG versions, and test them in the CI. In theory, this would be a good thing. In practice, I'm afraid testing will make the CI much much longer, and much more complicated. I'll create a ticket.

If it's not possible to do that cleanly within Django (and reading the code, I'm doubting it is), it could also be acceptable to generate migration files in setup.py (so they'd end up in the release) and have them be gitignored.

I'd probably hardcode the migrations, because generating migrations on the fly during release comes with two potential pitfalls:

It's really hard to test that the migrations are actually working if files aren't commited to the repo
If the migration-generating code changes slightly, we could face hard-to-debug inconsistencies

Would it be acceptable to add a manual step when a new migration is created, basically copy-pasting 10 lines of code in the django app as a new migration?

I understand those points. I'll just note:

It's really hard to test that the migrations are actually working if files aren't committed to the repo

Hm, not sure about that. If migrations are generated every time we launch setup.py, then they will be OK in the CI, and then can be regenerated in the dev env by re-launching pip install -e, same as when a metadata value changes.

If the migration-generating code changes slightly, we could face hard-to-debug inconsistencies

With the current implementation, it's the same: if the SQL migration code changes, then the django migration will change. And I can't really see what we may change in the Django migration part if the only thing it does is a RunSQL. But your approach is leaning on the safe side, and I can't blame that.
That being said, I'm definitely not against your proposed changes. I'd rather we solve problems and iterate. We'll see how it goes.

Hm, not sure about that. If migrations are generated every time we launch setup.py, then they will be OK in the CI, and then can be regenerated in the dev env by re-launching pip install -e, same as when a metadata value changes.

Oh, I didn't thought about that. I'm not sure how to trigger a command when pip install -e . is called though, but I can definitely write something to generate the migration files once we figure that out!

@ewjoachim I've made an attempt at generating migrations on the fly (see my last commit) when running pip install -e .. I had to tweak the order, since the default order of SQL files isn't correct, and it work. However, it doesn't work with pip install . (without the -e flag), I need to find out why

Continuing the discussion in the PR !


Each research topic has a progress indicator which details how thoroughly researched or supported the underlying issue is.

@krand7 @remomueller Were there additional intentions with this issue, or can we close it now with 7.0 being here?
We have had some discussions of having other "metrics" shown with research topics, yet I don't see anything on the horizon soon.

This should be closed with 7.0, thanks!


Defined new revisions for golang dep: github.com/nakagami/firebirdsql

Ran component through the scanners since I noticed it had not been harvested yet and it correctly declared the license. Closing PR as it is no longer needed.


feat(lint/noStaticElementInteractions): add rule

Summary
Enforce that non-interactive, visible elements (such as <div>) that have click handlers use the role attribute.
Implements #527
Test Plan
Added snaps for valid/invalid cases.
Basically, it conforms to the test cases of eslint-plugin-jsx-a11y/no-static-element-interactions, and test cases that cannot be handled by the resources within Biome are handled individually.
ref:
https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/__tests__/src/rules/no-static-element-interactions-test.js

@unvalley

https://github.com/biomejs/biome/pull/2981#discussion_r1619260119
https://github.com/biomejs/biome/pull/2981#discussion_r1619336811

As this comment states, this method was created to bridge the gap between eslint-plugin-jsx-a11y and Biome
However, I thought it was necessary to optimize the Biome definition based on the official documentation, rather than discussing the two options of which way to go.
Is this an issue that should be addressed in this PR?
If necessary, we will address it in this PR, but I think there is an option to address it in another Issue, PR due to the number of elements.

@ryo-ebata

Is this an issue that should be addressed in this PR?
If necessary, we will address it in this PR, but I think there is an option to address it in another Issue, PR due to the number of elements.

Let's address that in another PR.

@ematipico @unvalley
I apologize for taking so long and for the lack of progress in development.
We are currently doing further research on the subject of this rule, but it appears that we were somewhat mistaken.
It seems that the rule does not cover non-interactive elements, but static elements (also called Native HTML), but I can find very little information on static elements.
We are currently investigating, so please wait a little longer...

@ematipico @unvalley
Why don't you check here?
If you have any corrections, I would be glad to hear from you!
https://github.com/biomejs/biome/pull/2981#discussion_r1660211912

@ematipico @unvalley
Thank you very much for your very extensive review!
I am personally interested in web a11y and would like you to let us continue to develop Biome's a11y rules in the future, may I?

I am personally interested in web a11y and would like you to let us continue to develop Biome's a11y rules in the future, may I?

Of course, help are always welcome 😄

@ryo-ebata could you rebase or merge main to pass the CI?

@unvalley
Thank you for your suggestion!
I have corrected your point and run the just gen-lint and just gen-bindings commands.

@ryo-ebata There is a regression in useValidAutocomplete/valid.jsx.snap, could you take a look

@unvalley

There is a regression in useValidAutocomplete/valid.jsx.snap, could you take a look

Okay! I'll address in this PR!

@ryo-ebata
The extract_attrs function should correctly return each rule even when it returns function-like attributes.
Returning strings like "[DYNAMIC]" should be avoided unless it is defined in the specification somewhere, instead it should be changed returning enum can clarify the attribute kinds.

@unvalley

instead it should be changed returning enum can clarify the attribute kinds.

Fixed to return enum, and also fixed a rule that was causing regressions.
Thanks for a lot of reviews, I think this will be my last commit!
201af4f


refactor(Structures): Message

This PR adds the following properties: application_id, components, mention_channels, stickers and fail_if_not_exists.
It also corrects the value of GUILD_DISCOVERY_DISQUALIFIED.

I've already done this, pending commit/push


Results Page Rotation Removes Selection(s)

Rotating phone (thereby the results page) on a galaxy A51 removes the selected settings and returns the result image to the default settings.

#139 Fixes the rotation issues when in the results page. Unfortunately, selection has to be reset when rotated, but I've fixed the selection mismatch between where the user presses and the selected region.


Windows first extension

I think it's more a visual studio help, but i dont know how to start. I'm totaly noob with windows, and trying to compile my extension php-gtk3 to windows.
I've compiled php-cpp with cmake, its created a phpcpp-common.lib, but i dont know how to compile first test extension without make
Can anyone help me to compile first example?

no maintenance


Build global settings

I think we could actually just put global settings into the private rails app, and not need to make a big production of it.


'>' not supported between instances of 'Counter' and 'Counter'

What steps will reproduce the problem?

Comparing two instances of Counter
Works on LeetCode though

Hey @Gadoli, thanks for reporting. I don't understand what you mean by this bug, but it doesn't seem related to Spyder.
Please provide a clearer description of what mean by this and how it generates an error in Spyder. Otherwise, I'm afraid I'll have to close it.

Closing due to lack of response.


creation of file and creation of the file

@ajaniak : could you track all remaining cases (and, ideally, any further variants thereof) and replace them by
<resp>EpiDoc encoding</resp>
thanks!

Done


Clean up DirectoryResolver

The DirectoryResolver was designed under the constraints of grpc.DialContext but we have since moved to grpc.NewClient which doesn't require a context and cannot fail.
This commit adapts the design to the new reality.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

14 of 25 (56.0%)  changed or added relevant lines in 9 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 28.168%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

builtins/edge/ds/graph.go
0
1
0.0%

builtins/edge/ds/object.go
0
1
0.0%

pkg/app/impl/jwt.go
2
3
66.67%

builtins/edge/ds/relation.go
0
2
0.0%

builtins/edge/ds/check.go
0
3
0.0%

pkg/app/directory/simple_resolver.go
6
9
66.67%

Files with Coverage Reduction
New Missed Lines
%

pkg/app/directory/simple_resolver.go
1
60.0%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.01%

Covered Lines:
1307

Relevant Lines:
4640

💛  - Coveralls


Grid: No auto new line when editing and '+' available

By A.Sharapov:

A table viewer when there is an option to add rows via '+' should not add rows on editing last row data. Also it will be good to have a possibility to delete rows.

Implemented in Datagrok v1.2.1.
The following properties control the editing behavior:
allowEdit
showAddNewRowIcon (requires allowEdit)
addNewRowOnLastRowEdit (requires allowEdit)


gorm batch update question

I can use this code to realize update for " update set amount = amount -1 where id = 1"
tx.Table(dao.TInvoiceExpense).Where("id = ?",1).Updates(map[string]interface{}{ "amount": gorm.Expr("amount - ?", 1)})
how to realize batch update for "update set amount = amount -1 where id = 1; update set amount = amount -2 where id = 2; "
call mysql once.

you can achieve it by using the case when syntax in mysql, but what I suggest is doing it in transaction rather than doing it in one query, I think in many senarios, the time of calling mysql doesn't matter so much.

you can achieve it by using the case when syntax in mysql, but what I suggest is doing it in transaction rather than doing it in one query, I think in many senarios, the time of calling mysql doesn't matter so much.

It is in transaction, the time of calling mysql is matter to my case. This is just a demo, i need to handle million data, so i need calling mysql once to handle for such as 500 data.

May I know what you are updating.

If you are doing some historical operation data insertion, you may just use the "on conflict update" syntax with batch insertion.
If you are doing update, like syncing order status of a tons of your records, and those records just need eventual consistency, you can try to split those records into smaller part and dispatch them into MQ, to update them asynchronously.


Do not require schema object to be created every time

Today we force the user to create schema object every time (either through SPAC or the schema builder). This part of code can be big in size especially when the user has many tables. Lovefield should be able to serialize the schema into persistent store and retrieve the schema from there.

Duplicate of https://github.com/google/lovefield/issues/58.


High- CVE-2020-1045- Indirect dependency -microsoft.aspnetcore.http.2.1.1

High- CVE-2020-1045- Indirect dependency -microsoft.aspnetcore.http.2.1.1
From DevSecOps scan, we observed HotChocolate.AspNetCore has an indirect dependency of microsoft.aspnetcore.http.2.1.1 which is vulnerable to (CVE-2020-1045) bypass vulnerability of ASP.NET core cookie.
https://www.whitesourcesoftware.com/vulnerability-database/CVE-2020-1045

To Reproduce
Steps to reproduce the behavior:
Code package location below

Remediation
Upgrade to version microsoft.aspnetcore.app - 2.1.22,microsoft.aspnetcore.all - 2.1.22,microsoft.netcore.app - 2.1.22, Microsoft.AspNetCore.Http - 2.1.22

You screenshot is from Hot Chocolate 10.5.
Hot Chocolate 11 does not have this reference.

Ah, I see you are mixing version dependencies.
Mixing version 10.5 and 11 assemblies can lead to a lot of runtime issues like method not found exceptions.
Furthermore, we have deprecated the Playground package. If you want to use playground integrate the playground react app in your server and do not use our package since we are not updating the js packages anymore. I guess there are also lots of deprecated packages in the version of playground.


Cosmos SDK Gosec

This really can't hurt:
https://github.com/informalsystems/gosec
Description
Adds the informal systems Gosec tool to our CI system.

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #602 (b83a7b3) into main (fd869ef) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #602   +/-   ##
=======================================
  Coverage   20.23%   20.23%
=======================================
  Files         172      172
  Lines       24172    24172
=======================================
  Hits         4892     4892
  Misses      18486    18486
  Partials      794      794

Continue to review full report at Codecov.

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

Sweet, thanks!


GET https://localhost:5001/css/site.css net::ERR_ABORTED 404

GET https://localhost:5001/css/site.css net::ERR_ABORTED 404
GET https://localhost:5001/AntDesign.Pro.Template.styles.css net::ERR_ABORTED 404

Hi @mingyouzhu , how can I reproduce this issue?

Hi @mingyouzhu , how can I reproduce this issue?

Solved because I tampered with the framework code and now I forgot what I changed haha


[BugFix] Fix drop FE failed when the target FE is using FQDN bug the leader node is not

Fixes SR-18292
Currently, the leader will resolve the host to IP when dropping FE if the leader is not using FQDN mode. But if the target node is using FQDN, the operation will fail because the host stored in meta is FQDN.
To fix this bug, only resolve host to IP when adding node (FE or BE), and remain the host unchanged when dropping node.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[x] 2.4

@Mergifyio backport branch-2.5


Add feature for custom log format

Hi there. I was looking for solution to get separate log files for separate threads and I came across this crate. It's super useful, thanks. However, I was not able to format the log messages in a custom format.
I have added the feature to the API along with instructions in the README and relevant tests. Could you take a look and accept this change. I'm happy to make any changes if required.

Hi! First of all, thanks for the PR :tada: ! I'm just coming back from vacations, so this might take a few days before I can go back to this.

I have made the requested changes. Please let me know if there are any other changes that you'd like.


Add missing StringBuilding trait

This fixes the breakage from merging #1242.  It has to live in a separate source directory from the rest of parboiled2, which is why it wasn't included in our initial internalization of parboiled2.

It's a tradeoff, but here's my defense of the current approach:

parboiled2 is a slow-moving dependency.  It was pretty much internalize-it-and-forget-it.  The reason that this comes up now is because this is one of three types that depended on a macro and had to be in a separate compilation pass, and we got to punt on that until now.
parboiled2 is an implementation detail.  Using parboiled2 makes nice, fast parsers, but doesn't make http4s play nicer with anything else.  So the dependency is much more liability than interoperability, unlike cats or fs2.
the shapeless dependency we get from parboiled2 has caused massive pain on large projects.  I like shapeless, but finding a stack of libraries that all use a compatible shapeless has cost many hours of time at work and impedes upgrades.  We remove ourselves from those constraints by this.


Use unpadded base64 strings for IDs and secrets

This fixes #18 (at least partly :) )

The revert is maybe a bit "brutal" yes, you're right :) I'll amend it so that it removes only the incorrect stripping of the last two characters in generated IDs. This only works because the base64 parser on the server side is not too strict, and potentially discards some data.

What stripping is incorrect I don't get it? If it's the ==, they are added again in relevant places.

Well stripping the last two characters unconditionally only works as long as the client ID length is 16 bytes, I would not encode this assumption in a hidden way in other parts of the code. With this patch (assuming I fix it to keep id instead of roomId etc) I'm "officially" switching both the backend and the frontend to unpadded strings, so we don't need to unpad and repad on the frontend side in many places.

Yes that's all clear, but it is not "incorrect" nor is it related to a parser not being strict or discarding data as far as understand until now, nor is it "in other parts of the code". On the other hand, it's nicely isolated, waiting for this backend change to happen.
Instead of reverting, you can just change the read and write functions that are following each other there:
https://github.com/abustany/goretro/pull/55/files#diff-1142a7bb977146abdfcc76f264af3125L138
https://github.com/abustany/goretro/pull/55/files#diff-1142a7bb977146abdfcc76f264af3125L148

OK I fixed the patch to only change the == stripping/adding :ok_hand:

👌 ✨
Just in case because I saw TrimRight, can it possibly cause a bug if the string would end with === rather than ==?

= in base64 is a padding character so that the length of the decoded data is a multiple of 8 bits. Now that the parser is configured in unpadded mode, we can safely remove all the padding. Also, in a valid padded base64 string there should never be more than two = signs at the end.


might it work on a Mac?

Hello, the conda doesn't seem to have a Mac version, do you know if it has ever been tested on Mac? thanks

Hi there, I haven't tested SAVANA on a Mac before but in theory it should work if you follow the instructions to install from source. However, I caution that the memory and time requirements of SAVANA make it better suited to be run in an HPC environment, especially if you are using genomic data with coverage > 15x.


docs: overhaul all

More updates since the helm refactor. Removed a few references to rpc-0 or "the server", removed irrelevant files like lightning. Combined all developer notes into one file. Rebuilt cli API docs. Major changes to logging infrastructure. Only functional change is removing the default path for snapshot export.

addressed some comments

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts
Reviewers, this pull request conflicts with the following ones:

#588 (handle more pathtypes in apidocs by willcl-ark)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

rebased on main after #588


Consider mentioning OrcaSlicer

Hello,
I saw that there is a FOSS slicer https://github.com/SoftFever/OrcaSlicer not mentioned in the wiki. Since it seems to be quite popular, I think that it might be worth adding it.

Hi! Sounds good! Are you willing to contribute to the wiki yourself? There's some instructions in the homepage on how to edit pages, you should be able to edit the markdown files here on GitHub https://depau.github.io/3dprint-wiki/


setup.py exact dependency versions

Hi,
I was just wondering, is there any reason for the setup.py to have
        "python-Levenshtein==0.12.0",
        "progressbar2==3.41.0",
        "pysam==0.15.2",
        "pyfaidx==<IP_ADDRESS>",
        "intervaltree==3.0.2",

and not
        "python-Levenshtein>=0.12.0",
        "progressbar2>=3.41.0",
        "pysam>=0.15.2",
        "pyfaidx>=<IP_ADDRESS>",
        "intervaltree>=3.0.2",

I just came across this package because it's broken in the nixpkgs repo, because we are using pyfaidx version 0.5.8.
Thanks for any clarification,
Robert

Hello,
I did a cursory check and I'm pretty sure it'll be okay to switch to >= for these packages. I've checked in the change to 076c235 , but I won't cut a new release (1.3.5) until I've done more checks and consolidated some larger changes from a feature branch I've been working on. If you'd like to use these changes now, you can run the following commands:
git checkout https://github.com/spiralgenetics/truvari.git
cd truvari
python setup.py sdist bdist_wheel
pip install dist/Truvari-1.3.4.tar.gz

Have a great day,
~/Adam English


Flags represent countries, not languages

It's easy to make language switcher using countries emoji, but it's just incorrect. Especially on the website that tries to look scientific.

🤔 what do you suggest to use instead?

A standard HTML dropdown:
Language: English ▼

🤔 sounds more like a nitpick to me tbh


#170 Events should be design for commutativity

fixes #170

@tfrauenstein you wanted to do a wording sweep here?

:+1:

Wording changes can be done later.

👍


Nesting Schemas

Is it possible to use zod schemas within other schemas? here a pseudocode.
const UserSchema = z.object({ username: z.string(), password: z.string()});

const LoginSchema = z.object({ usercredentials: UserSchema, accessToken: z.string()})

Yes

@sschneider-ihre-pvs
It works exactly as you've pseudo-coded it 😆
TypeScript Playground

Great :)


Doesn't open in Langgraph Studio

studio_log.txt

Here's the langgraph.json file and the requirements.txt
NOTE: I had to remove the version numbers from requirements.txt to get it as far as it did. Otherwise it craps out sooner.
(meta_expert_env) meta_expert % cat langgraph.json
{
  "python_version": "3.11",
  "dockerfile_lines": [],
  "dependencies": [
    "."
  ],
  "graphs": {
      "jar3d": "./agents/jar3d.py:workflow"
  },
  "env": ".env"
}
(meta_expert_env) meta_expert % cat requirements.txt
langchain-core
langgraph
langchain-community
langchain-openai
beautifulsoup4
termcolor
chainlit
colorlog
fake-useragent
playwright
pypdf
llmsherpa
fastembed
faiss-cpu
FlashRank
(meta_expert_env) meta_expert %


Setting up kanban board

https://github.com/NicolasAndrehh/JavaScript-Group-Capstone/projects/1

we are two people on our team.

Hi Team,
Good job so far! 👍🏻
There are some issues that you still need to work on to go to the next project but you are almost there!

Required Changes ♻

[ ] Please make sure to assign all tasks to team members, as per this requirement.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

STATUS: APPROVED! 🍾 🎆 🥇
Hi Team!,
I'm Andres.
You did a great job working on this milestone, your project is complete! 🚀
There is nothing else to say other than... it's time to merge it 🤖
Congratulations! 🎉

Highlights
✅ Great use of GitHub's project features 🥇
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Print test metrics

💡 Your Question
I want to test the trained yolo nas model, but when running it like below, it doesn't print anything, no metrics. How can I enable printing all those metrics here?
best_model = models.get(
    MODEL_ARCH,
    num_classes=len(dataset_params['classes']),
    checkpoint_path=f"{CHECKPOINT_DIR}/{EXPERIMENT_NAME}/RUN_20231019_025212_979921/average_model.pth"
).to(DEVICE)

#evaluate model

trainer.test(
    model=best_model,
    test_loader=test_data,
    test_metrics_list=DetectionMetrics_050(
        score_thres=0.1,
        top_k_predictions=300,
        num_cls=len(dataset_params['classes']),
        normalize_targets=True,
        post_prediction_callback=PPYoloEPostPredictionCallback(
            score_threshold=0.01,
            nms_top_k=1000,
            max_predictions=300,
            nms_threshold=0.7
        )
    )
)

Versions
No response

SG version? Colab env or script run?

Yes from super gradients. It's a python script.

A trainer.test call does not print metrics. It returns a dictionary of metrics: test_metrics = trainer.test(...).
So if you want to see then you have to print them manually:
test_metrics = trainer.test(...)
print(test_metrics)

Can you provide an example of expected output?

Something like this, per class:
             Class     Images  Instances      Box(P          R      mAP50  mAP50-95): 100%|██████████
               all               468       1128      0.789       0.75      0.816      0.632
             Object1        468        345      0.845      0.836      0.869      0.763
              Object2       468        743      0.733      0.664      0.762      0.501

Speed: 0.3ms preprocess, 1.6ms inference, 0.0ms loss, 0.5ms postprocess per image

Add the include_classwise_ap=True in the  DetectionMetrics_050

Thanks. It only adds the<EMAIL_ADDRESS>for classes, not Precision, F1, etc.
It would benefit from a more detailed list, as a feature request. Also, it says class0, class1! Would be better to show the actual name of class.


Add vapid build command

Hi! i think that it will be nice to have a vapid build command. Currently there's only vapid deploy, that is used to deploy on vapid hosting, but what if i would like to deploy to my own server minifing and building the site?
Woult it be possible?
Thanks

@Jack89ita Thanks for submitting this request. I'm going to close this issue and consolidate it with #9, since it's a similar topic. Feel free to chime in there.


Init with Firebase and Flux store

This project needs a firebase backend, to begin with, but must support other backends as well. We need to create a Flux model with support for Firebase, to support long-term storage. There should be node tasks to handle common maintenance and development tasks, such as firebase:deploy or firebase:serve. The tasks must be prefixed with firebase.

I am using Firebase in past few month, and I never heard of Flux store or encounter it? Do I need to learn flux store cuz seemingly it is a thing lol ?


oauth support

recently, gmail change the authentication to oauth2. Does imapnotify support oauth2 authentication?

I don't think so.
But it wont' be hard to implement, I think.
You can always give it a try.


Bump at_root_server (and at_persistence_root_server) dependencies

Is your feature request related to a problem? Please describe.
Dependabot continues to fail to raise PRs for pub deps.
Describe the solution you'd like
args to be bumped to 2.6.0
at_commons to 5.0.1
Should we take this opportunity to bump dev dependencies like lints (still at ^1.0.1 in at_persistence_root_server)? Also increment patch version for each pubspec? 👀 @gkc

Should we take this opportunity to bump dev dependencies like lints (still at ^1.0.1 in at_persistence_root_server)? Also increment patch version for each pubspec?

👍


switching to octo buffer result in unwanted message in echo area

Issue Description
Type: bug report
Describe what happened (or what feature you want)
when I switch to back to octo buffer I see the following

not sure why is this happening as it never happen to me before
Describe what you expected to happen
no echo message
How to reproduce it (as minimally and precisely as possible)

create new or open issue
switch off it
switch back (also when switch to telescope picker and back)

Tell us your environment
neovim nightly
I'm going to debug more

when I commented out my remap of L to $, now it shows

This seems to happen on octo buffer enter

I uncommented everything and found that vim.cmd [[setlocal filetype=octo]] causing this. It must a filetype autocmd somewhere

I can confirm that is something todo with how I setup may mappings.
After commenting out my core mappings I no longer get this message. hmmm closing
for now

FOUND IT. It was my markdown filetype, I had nmap L in it without rhs.


Update SD card mapping

correct RM T8
add RM Zorro and Jumper T-Pro

If more is needed please feel free to let me know or drop this...

We can merge this, sure, but I'll put in another PR to use the sdcard.json. You'll need to run yarn fmt and yarn test -u and commit the results to fix the CI errors before I can merge though


⚠ Bofrost.at has degraded performance

In 0f1d1cc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2406 ms

Resolved: Bofrost.at performance has improved in 3a670c6 after 6 minutes.


Upgrade gcloud dependency to 0.3.0

Gcloud package recently released 0.3.0 adding support for latest google apis. In a existing project of mine that uses both rpc and gcloud, i changed the gcloud dependency to version 0.3.0 and i got the following error.
Package rpc has no versions that match >=0.5.6+1 <0.6.0 derived from:
- pub_test depends on version ^0.5.6+1

Are there any suggestions or workarounds?

You can use dependency_overrides: to unblock you until an update is published.

Thanx @zoechi

I've just published a new version which widens several dependency constraints, including the one for package:gcloud.


remove haste references from polyfillEnvironment

Hi,
This is part of a greater change to get tree shaking to work with react-native. In order to use webpack's tree shaking I need to remove the use of haste. I'm planning to start a discussion upstream in react-native but I also wanted to make the haste references that haul uses in polyfillEnvironment.
I'm planning to use this fork to show how someone could use haul and webpack with react-native to get tree shaking to work on their react-native apps. In my example on an android test project is was able to shrink the bundle size from 500k with metro to 260k using haul.
I'm also open to writing a babel-plugin to de-haste the source but I thought I would try to make these small changes in haul.

I rebased with the suggestion of referring to NativeModules using the main for react-native. In order for the tree shaking to work we will need to make sure polyfillEnvironment.js gets modified by babel with the same plugin I use to rewrite the module imports to support tree shaking.

Ok, I've put together an example using create-react-native-app ejected you can see it here:
https://github.com/rdy/tree-shaking-example/tree/tree-shaking. I've chosen to use the android platform since I had the build tools on my machine already. You can build the bundles from the package.json scripts bundle:metro and bundle:haul.
The master branch contains a version where I've optimized the bundle size with haul slightly by tweaking the babel output instead of relying on react-native-preset.
Metro bundle: 505K
Haul bundle: 538K
The tree-shaking branch shows the changes from that where I've used my fork of react-native that removes haste as well as haul that doesn't use haste anymore. I've provided a transform for babel-plugin-transform-imports. I'm basically looking at the original main for react-native and supplying the correct import path, this allows webpack to be able to tree shake out the dead code.
Haul bundle with tree shaking: 254K
I've created a series of PDF using source-map-explorer to illustrate that pieces of the bundle are actually being remove correctly by webpack.
In order to build a running android version you would need to run the gradle commands to build the native java code in the node modules. I didn't commit those to my branch of react-native. From my tests locally the shaken version ran just fine.
Anyways I hope this provides enough of a picture why I'm looking at stripping haste in both haul and react-native.

I've created a corresponding issue in react-native here:
https://github.com/facebook/react-native/issues/18598
Hopefully it will get the discussion going again on removing haste from react-native which should make this approach possible.

I switched to using the import at the top so I could keep the destructuring code similar to the previous implementation, but it should allow my babel-plugin-transform-imports to safely change the import to import NativeModules from 'react-native/Libraries/BatchedBridge/NativeModules'. I believe that as long as users are using babel-preset-react-native they should automatically get babel-plugin-transform-es2015-modules-commonjs which should convert that import into a require for those who don't want tree shaking.


Optional repeating section generates empty object for non existing model property

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
Form not updating when repeating section array field doesn't exist in model
Expected behavior
Example has 1-N connection between Investor and investments:
 export interface Investor {
  name: string;
  surname: string;
  investments?: Investments;
}

export interface Investments {
  investmentName: string;
  investmentDate: Date;
  stockIdentifier: string;
}

Let's say I'm inserting new Investor that has 2 new Investments. If I want to switch to another investor that has no investments defined ('investments' field does not exist/is not serialized), form retains 'investments' values form the former Investor.
Only if Investments are explicitly set to empty array, form behaves as expected.
I'm not sure if this is desired behavior, but for the described case the repeating section should be reset
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/angular-raxxnl?file=src%2Fapp%2Fapp.component.ts

Input name and surname
Add couple of investments
Press on 'Simulate server fetch next row'
Name and surname have changed but past users' investments are not removed (and are also added to model)

Please tell us about your environment:
I'm using:
@ngx-formly/core: "5.2.0"
@ngx-formly/material": "5.2.0"

Fixed in 5.3 version :smiley:, happy coding!!!


Make error messages in the question creation form easier to see

Right now there are just random strings printed out on the page. A box that lights up when there's an error (and lights up a different color if there's no error), and that clearly shows the error message, would be great.

Started in https://github.com/seattle-uat/civiform/pull/635


Add bug fix in bucket local variable

As you suggested, I made the change only to the local variable.
With this change, "bucket" variable will have the value of "var.bucket_name" only if the bucket has to be created.
Otherwise, it will have the name of the created bucket.
Checklist

I applicable, I acknowledge that I have:

[X] Read the contributing guide
[X] Ran terraform fmt on all modified files
[X] Regenerated the relevant README.md files using tools/tfdoc.py
[X] Made sure all relevant tests pass

Oh, now I see the bug :-) Thanks a lot.
Let me update this a bit

Can you pull changes from here ? This adds tests that verify that the bug is actually fixed and also covers cloud functions v1.

I did it. I don't know if it was the correct way to merge your change, but trying to merge all the branch a I saw that there are a lots file changed because your branch is is 81 commits behind GoogleCloudPlatform:master.
So I basically copied and pasted only the file changed on your last commit.

Thank you, I thought that I based that on your branch, but maybe I missed something. git cherry-pick should also work in this case.


Fix WrappedR2Api

SGTM, I thought he was still working on other stuff and I'd merge it together.

@sushant94 ping?

I think it is good to merge @XVilka .


Switch to x-eithert

I upgraded all the ambiata dependencies while I was at it.

looks ok, does ice need to be upgraded as well? :+1:

@tranma Yep that's the end goal, I want to use consume from twine so I need to upgrade jetski+icicle+ice

:facepunch:


error No such file or directory

cp: cannot stat 'clisw': No such file or directory
make: *** [makefile:7: install] Error 1

You should have a file called "clisw.py" in the same directory as the makefile. Can you confirm?


CI/CD - AWS_ELASTIC_IP is not used

Remove all references to AWS_ELASTIC_IP from the README.

Done!


[SPARK-18922][TESTS] Fix new test failures on Windows due to path and resource not closed

What changes were proposed in this pull request?
This PR proposes to fix other test failures on WIndows as below:
Before
KafkaRelationSuite:
 - test late binding start offsets *** FAILED *** (7 seconds, 679 milliseconds)
   Cause: java.nio.file.FileSystemException: C:\projects\spark\target\tmp\spark-4c4b0cd1-4cb7-4908-949d-1b0cc8addb50\topic-4-0\00000000000000000000.log -> C:\projects\spark\target\tmp\spark-4c4b0cd1-4cb7-4908-949d-1b0cc8addb50\topic-4-0\00000000000000000000.log.deleted: The process cannot access the file because it is being used by another process.

KafkaSourceSuite:
 - deserialization of initial offset with Spark 2.1.0 *** FAILED *** (3 seconds, 542 milliseconds)
   java.io.IOException: Failed to delete: C:\projects\spark\target\tmp\spark-97ef64fc-ae61-4ce3-ac59-287fd38bd824

 - deserialization of initial offset written by Spark 2.1.0 *** FAILED *** (60 milliseconds)
   java.nio.file.InvalidPathException: Illegal char <:> at index 2: /C:/projects/spark/external/kafka-0-10-sql/target/scala-2.11/test-classes/kafka-source-initial-offset-version-2.1.0.b

HiveDDLSuite:
 - partitioned table should always put partition columns at the end of table schema *** FAILED *** (657 milliseconds)
   org.apache.spark.sql.AnalysisException: Path does not exist: file:/C:projectsspark	arget	mpspark-f1b83d09-850a-4bba-8e43-a2a28dfaa757;

DDLSuite:
 - create a data source table without schema *** FAILED *** (94 milliseconds)
   org.apache.spark.sql.AnalysisException: Path does not exist: file:/C:projectsspark	arget	mpspark-a3f3c161-afae-4d6f-9182-e8642f77062b;

 - SET LOCATION for managed table *** FAILED *** (219 milliseconds)
   org.apache.spark.sql.catalyst.errors.package$TreeNodeException: execute, tree:
 Exchange SinglePartit
 +- *HashAggregate(keys=[], functions=[partial_count(1)], output=[count#99367L])
    +- *FileScan parquet default.tbl[] Batched: true, Format: Parquet, Location: InMemoryFileIndex[file:/C:projectsspark	arget	mpspark-15be2f2f-4ea9-4c47-bfee-1b7b49363033], PartitionFilters: [], PushedFilters: [], ReadSchema: struct<>

 - insert data to a data source table which has a not existed location should succeed *** FAILED *** (16 milliseconds)
   org.apache.spark.sql.AnalysisException: Path does not exist: file:/C:projectsspark	arget	mpspark-34987671-e8d1-4624-ba5b-db1012e1246b;

 - insert into a data source table with no existed partition location should succeed *** FAILED *** (16 milliseconds)
   org.apache.spark.sql.AnalysisException: Path does not exist: file:/C:projectsspark	arget	mpspark-4c6ccfbf-4091-4032-9fbc-3d40c58267d5;

 - read data from a data source table which has a not existed location should succeed *** FAILED *** (0 milliseconds)

 - read data from a data source table with no existed partition location should succeed *** FAILED *** (0 milliseconds)
   org.apache.spark.sql.AnalysisException: Path does not exist: file:/C:projectsspark	arget	mpspark-6af39e37-abd1-44e8-ac68-e2dfcf67a2f3;

InputOutputMetricsSuite:
 - output metrics on records written *** FAILED *** (0 milliseconds)
   java.lang.IllegalArgumentException: Wrong FS: file://C:\projects\spark\target\tmp\spark-cd69ee77-88f2-4202-bed6-19c0ee05ef55\InputOutputMetricsSuite, expected: file:///

 - output metrics on records written - new Hadoop API *** FAILED *** (16 milliseconds)
   java.lang.IllegalArgumentException: Wrong FS: file://C:\projects\spark\target\tmp\spark-b69e8fcb-047b-4de8-9cdf-5f026efb6762\InputOutputMetricsSuite, expected: file:///

After
KafkaRelationSuite:
 - test late binding start offsets !!! CANCELED !!! (62 milliseconds)

KafkaSourceSuite:
 - deserialization of initial offset with Spark 2.1.0 (5 seconds, 341 milliseconds)
 - deserialization of initial offset written by Spark 2.1.0 (910 milliseconds)

HiveDDLSuite:
 - partitioned table should always put partition columns at the end of table schema (2 seconds)

DDLSuite:
 - create a data source table without schema (828 milliseconds)
 - SET LOCATION for managed table (406 milliseconds)
 - insert data to a data source table which has a not existed location should succeed (406 milliseconds)
 - insert into a data source table with no existed partition location should succeed (453 milliseconds)
 - read data from a data source table which has a not existed location should succeed (94 milliseconds)
 - read data from a data source table with no existed partition location should succeed (265 milliseconds)

InputOutputMetricsSuite:
 - output metrics on records written (172 milliseconds)
 - output metrics on records written - new Hadoop API (297 milliseconds)

How was this patch tested?
Fixed tests in InputOutputMetricsSuite, KafkaRelationSuite,  KafkaSourceSuite, DDLSuite.scala and HiveDDLSuite.
Manually tested via AppVeyor as below:
InputOutputMetricsSuite: https://ci.appveyor.com/project/spark-test/spark/build/633-20170219-windows-test/job/ex8nvwa6tsh7rmto
KafkaRelationSuite: https://ci.appveyor.com/project/spark-test/spark/build/633-20170219-windows-test/job/h8dlcowew52y8ncw
KafkaSourceSuite: https://ci.appveyor.com/project/spark-test/spark/build/634-20170219-windows-test/job/9ybgjl7yeubxcre4
DDLSuite: https://ci.appveyor.com/project/spark-test/spark/build/635-20170219-windows-test
HiveDDLSuite: https://ci.appveyor.com/project/spark-test/spark/build/633-20170219-windows-test/job/up6o9n47er087ltb

Test build #73160 has started for PR 16999 at commit 0dd78c1.

Build started: [TESTS] org.apache.spark.metrics.InputOutputMetricsSuite
Build started: [TESTS] org.apache.spark.sql.kafka010.KafkaRelationSuite
Build started: [TESTS] org.apache.spark.sql.kafka010.KafkaSourceSuite
Build started: [TESTS] org.apache.spark.sql.hive.execution.HiveDDLSuite
Build started: [TESTS] org.apache.spark.sql.execution.command.DDLSuite

cc @srowen, could you take a look when you have some time?

Test build #73160 has finished for PR 16999 at commit 0dd78c1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73160/
Test PASSed.

Ah, yes. It seems they are new ones by some rescent commits.

Merged to master


Set canvas style to override problem 3rd party css

Fixes #12439 and #12566
Replaces #12619
Set new canvas styles to all: initial; to override problems caused by third party css,  This is possible as OpenLayers only ever sets styles on canvas directly.

Thinking about this more, I realize that this will at least break right-to-left text support.


Permissions

Added a UserPermissionMixin that limits access to a model to the currently logged in User. Also added docs and, but no tests.

Coverage decreased (-0.04%) to 71.313% when pulling 0bd54fb3a59400d862ceb4fc5ef1264a3a7c6842 on tdsprogramming:permissions into b1f0a4761ff7f1514d94c0cf789d468d2dcfe477 on django-extensions:master.

The PR looks very good, thanks for your work on this !
Maybe we should call it ModelUserFieldPermissionMixin or ModelOwnerPermissionMixin ?
Which is kinda long but does more explicitly describes what the Mixin is doing :-) (In the sense that UserPermissionMixin might be confused with other types/ways of User permissions.)
What do you think about tests ?  Maybe we can look at Django's tests/auth_tests/test_mixins.py and implement basic tests here as well.

Good idea for the tests. I'll check out Django's docs and fix that on the next go around. I like ModelUserFieldPermissionMixin more because it adds more of a generic name.

I need a model with a user field for testing. Is there one in tests/testapp/models.py? Can I write one there or should I make a model just in the test file?

Either is fine for me :-)
If the model can be used more broadly and for other tests (now or in the future) I would prefer tests/testapp/models.py, otherwise if there very specific to these test use the test file :-)

adding reference to the issue #1392

About to push with tests. I essentially used the same things from Django's tests, per our previous discussion: https://github.com/django/django/blob/master/tests/auth_tests/test_mixins.py#L195. Not sure if these tests will be effective enough

Please ignore all previous commits; I realized I had a major issue that I am working through. I will push once that is resolved.

The most recent push should be working.

The most recent push should be working.

sorry seems there still at least some style issues...

Where can I find the style issues?

https://travis-ci.org/django-extensions/django-extensions/jobs/605077653?utm_medium=notification&utm_source=github_status

Thanks !

No problem. I realized I think I forgot to rebase a couple times. I don't think anything was messed up though, correct?


Liquid Outlet inside Liquid If not working

I have a liquid outlet nested inside a liquid if (with an else clause):
{{#liquid-if login containerless=true class="inherit-height"}}
  {{liquid-outlet "login" class="inherit-height"}}
{{else}}
<div class="main">
  {{liquid-outlet class="auto-height content-view"}}
</div>
{{/liquid-if}}

Before upgrading to 0.19.2 (up from 0.14.0), this worked fine. Now I get the following error:
Uncaught TypeError: Cannot read property 'render' of undefined

My workaround at the moment, is to change both outlets and ifs to a regular ember equivalents - basically removing all traces of Liquid Fire from the project.
Ember: 1.11.0
Ember CLI: 0.2.2
Liquid Fire: 0.19.2
Any ideas?

Thanks. I was able to reproduce the problem. The issue is that a {{liquid-outlet}} that has no content will crash, but only when its parent outlet remains stable since the previous transition.

Thank you. Amazing framework btw!


Unexpected Exception On Adding @WebMethod To Sei

If I read https://cxf.apache.org/docs/developing-a-service.html correctly, then it is possible to develop a service without annotation@WebMethod:

In order to create a service from Java code you are only required to add one annotation to your code. You must add the @WebService() annotation on both the SEI and the implementation class.

So this should just do:
@WebService
public interface HelloWorldService {
       String hello();
       String greet();
  }

@WebService(endpointInterface = "org.example.cxf.HelloWorldService")
public class HelloWorldServiceImpl implements org.example.cxf.HelloWorldService {
    @Override
    public String hello() { return "hello"; }
    @Override
    public String greet() { return "hello again"; }
}

Indeed this works fine. However, trying a to annotate just one SEI operation with @WebMethod results in unexpected exception on getting the WSDL, i.e.
@WebService
public interface HelloWorldService {
      @WebMethod
       String hello();                // hello is annotated while

       String greet();               // greet is not
  }

// FruitWebServiceImpl unchanged

# application.properties
..
quarkus.cxf.path=/ws
quarkus.cxf.endpoint."/hw".implementor=org.example.cxf.HelloWorldServiceImpl

# get WSDL
> curl -v http://localhost:8080/ws/hw?wsdl

Server's log:
[..]
18:24:04 WARN  [io.quarkiverse.cxf.transport.CxfHandler] no info transmit to servlet
..
18:24:04 INFO  [io.quarkus] Installed features: [cdi, cxf, mutiny, resteasy, resteasy-jackson, security, security-properties-file, servlet, smallrye-context-propagation, smallrye-jwt, vertx, vertx-web]
18:24:04 ERROR [io.quarkus.vertx.http.runtime.QuarkusErrorHandler] HTTP Request to /ws/hw?wsdl failed, error id: c9a5033c-d872-41f7-a7b4-188f0cf19967-2: java.lang.NullPointerException
        at io.quarkiverse.cxf.transport.CxfHandler.process(CxfHandler.java:275)
        at io.quarkiverse.cxf.transport.CxfHandler.handle(CxfHandler.java:207)
        at io.quarkiverse.cxf.transport.CxfHandler.handle(CxfHandler.java:48)
        at io.vertx.ext.web.impl.BlockingHandlerDecorator.lambda$handle$0(BlockingHandlerDecorator.java:48)
        at io.vertx.core.impl.ContextImpl.lambda$executeBlocking$2(ContextImpl.java:313)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
        at java.base/java.lang.Thread.run(Thread.java:834)

Adding @WebMethod on greet() or removing from hello() lets the problem disappear. Nonetheless, I believe that a mixed-case should also work.
$ java -version
openjdk version "11.0.8" 2020-07-14
OpenJDK Runtime Environment GraalVM CE 20.2.0 (build 11.0.8+10-jvmci-20.2-b03)
OpenJDK 64-Bit Server VM GraalVM CE 20.2.0 (build 11.0.8+10-jvmci-20.2-b03, mixed mode, sharing)

quarkus-cxf: 0.6.0-SNAPSHOT

strange...
it mean that controller is null... which come from cxfservletInfos is null of getInfos() is null.
but route is started so it mean that recorder.registerCXFServlet do not worked properly....
Do you have only one implementor if SEI ?
Can you check that implementor (with namespace) is realy the same than in config.
https://github.com/quarkiverse/quarkus-cxf/blob/0db3511466d7ab97267a37825a21f35318e40a72/runtime/src/main/java/io/quarkiverse/cxf/CXFRecorder.java#L80-L86

Let me double-check that implementor thing ..

Do you have only one implementor if SEI ?

The cause is this one:
I do have a second implementation of my SEI. Notice that there are no WS/CXF related annotations:
// file: src/main/java/io/app/ws/GreetingImpl.java
// package does not correspond to file path (!!)
package io.app.soap;

import io.app.ws.HelloWorld;

public class GreetingImpl implements HelloWorld {
  @Override public String hello() {..}
  @Override public String greet() {..}
}

This situation appeard after some refactorings :-)Changing package to io.app.ws fixed the problem., i.e. then I can partially use @WebMethod on HelloWorld (otherwise as written: either all or none annotated with @WebMethod).
Indeed the exception is unexpected cause apart from deriving from an (SEI) interface there is no relation to WS/CXF. Obviously the problem is easily to fix. Nonetheless, such an unrelated problem should not break things.
I suggest not to close this issue but instead to put into backlog with low priority.

so it is same issue than #143 . I know that I have to fix that... I will do it is an easy fix in processor to handle not only one but few implementor.

At least in relation to multiple SEI implementations, true.

can you test on master. It must be ok now.

Tried updated master and it does not work for me.
Here is my sample project:
https://github.com/wh81752/cxfdemo-quarkus/tree/issue-153
Getting:
$ mvn clean quarkus:dev
[INFO] Scanning for projects...
[INFO]
[INFO] -------------------< io.bitio.demo:cxfdemo-quarkus

[INFO] Building Quarkus CXF Demo 1.0.0-SNAPSHOT
[INFO] --------------------------------[ jar
]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cxfdemo-quarkus
[INFO] Deleting /Users/geronimo1/ws/cxfdemo-quarkus/target
[INFO]
[INFO] --- quarkus-maven-plugin:1.12.0.Final:dev (default-cli) @
cxfdemo-quarkus ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 5 resources
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 16 source files to
/Users/geronimo1/ws/cxfdemo-quarkus/target/classes
OpenJDK 64-Bit Server VM warning: forcing TieredStopAtLevel to full
optimization because JVMCI is enabled
Listening for transport dt_socket at address: 5005
2021-02-25 21:29:42,984 INFO  [org.jbo.threads] (main) JBoss Threads
version 3.2.0.Final
2021-02-25 21:29:43,132 INFO
[io.qua.ely.sec.pro.dep.ElytronPropertiesProcessor] (build-14) Configuring
from MPRealmConfig
2021-02-25 21:29:43,755 INFO
[org.apa.cxf.wsd.ser.fac.ReflectionServiceFactoryBean] (build-10) Creating
Service {http://ws.app.io/}HelloWorld from class io.app.ws.HelloWorld
2021-02-25 21:29:44,236 ERROR [io.qua.dep.dev.IsolatedDevModeMain] (main)
Failed to start quarkus: java.lang.RuntimeException:
io.quarkus.builder.BuildException: Build failure: Build failed due to errors
[error]: Build step
io.quarkus.arc.deployment.SyntheticBeansProcessor#initRuntime threw an
exception: java.lang.IllegalStateException: A synthetic bean with
identifier 877d6b01a28865ee140c552f46f358a90baf8d99 is already registered:
SYNTHETIC bean [types=[io.quarkiverse.cxf.CXFClientInfo],
qualifiers=[@Default, @Any, @Named(value = "io.app.ws.HelloWorld")],
target=n/a]
at
io.quarkus.arc.processor.BeanDeployment.addSyntheticBean(BeanDeployment.java:1053)
at
io.quarkus.arc.processor.BeanDeployment.access$300(BeanDeployment.java:49)
at
io.quarkus.arc.processor.BeanDeployment$BeanRegistrationContextImpl.lambda$configure$0(BeanDeployment.java:1215)
at
io.quarkus.arc.processor.BeanConfigurator.done(BeanConfigurator.java:51)
at
io.quarkus.arc.deployment.SyntheticBeansProcessor.configureSyntheticBean(SyntheticBeansProcessor.java:88)
at
io.quarkus.arc.deployment.SyntheticBeansProcessor.initRuntime(SyntheticBeansProcessor.java:56)
at
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native
Method)
at
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:566)
at
io.quarkus.deployment.ExtensionLoader$2.execute(ExtensionLoader.java:920)
at io.quarkus.builder.BuildContext.run(BuildContext.java:277)
at
org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2415)
at
org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1452)
at java.base/java.lang.Thread.run(Thread.java:834)
at org.jboss.threads.JBossThread.run(JBossThread.java:501)
On Thu, Feb 25, 2021 at 8:59 PM olivier dufour<EMAIL_ADDRESS>wrote:

can you test on master. It must be ok now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quarkiverse/quarkus-cxf/issues/153#issuecomment-786162943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB42TPJ5BINSUKPFELBP3RDTA2TZZANCNFSM4YE52YTQ
.

--
Wolfgang Häfelinger
häfelinger IT - Applied Software Architecture
http://www.haefelinger.it
+49 1520 32 52 981
(+31 648 27 61 59)

In example project above I just removed the comments below, i.e. let GreetingImpl implement an interface (that happens to be a SEI).
public class GreetingImpl /* implements HelloWorld  */ { ..}

OK, it is named with sei for server... we have to named with implementor to remove the issue...

@wh81752  it must be ok now ;-)

Yes, works fine now.
On Fri, Feb 26, 2021 at 7:16 AM olivier dufour<EMAIL_ADDRESS>wrote:

@wh81752 https://github.com/wh81752 it must be ok now ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/quarkiverse/quarkus-cxf/issues/153#issuecomment-786438243,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB42TPIFULBV4D7ZHKHWQFTTA44FFANCNFSM4YE52YTQ
.

--
Wolfgang Häfelinger
häfelinger IT - Applied Software Architecture
http://www.haefelinger.it
+49 1520 32 52 981
(+31 648 27 61 59)


[sdl2] Prevent exports in static build

Fixes https://github.com/Microsoft/vcpkg/issues/1351.
Needed to bump a few dependent libraries, as they all depend on SDL's DECLSPEC macro.

Awesome job, and thanks for the explanation (TIL)!


When using mouse rather than touch it can be hard to drill into records due to the position of the info box

When you move your mouse over to it you might trigger displaying a different record.
Works with mouse and touch – but optimised for touch due to the info-box showing on the left hand side. Rather than use mouse over - could use click to show infobox.

Clicking a node 'pins' the infobox. Clicking on it again will unpin the infobox.


error while loading shared libraries: libadolc.so.2: cannot open shared object file: No such file or directory

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):
Apollo installed from (source or binary):
Apollo version (1.0, 1.5, 2.0, 2.5, 3.0):

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

Supporting materials (screenshots, command lines, code/script snippets):
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
while run the garage_test in planning integration test and encounter the problem:
root@in_dev_docker:/apollo/bazel-bin/modules/planning/integration_tests# ./garage_test
./garage_test: error while loading shared libraries: libadolc.so.2: cannot open shared object file: No such file or directory
The lib can be find in /usr/lib, the /etc/ld.so.conf has been edited and updated by ldconfig
root@in_dev_docker:/usr/lib# find -name 'libadolc'
./x86_64-linux-gnu/libadolc.so
./x86_64-linux-gnu/libadolc.so.1
./x86_64-linux-gnu/libadolc.so.1.1.1
./x86_64-linux-gnu/libadolc.a
root@in_dev_docker:/usr/lib# ldconfig

libadolc.so.2 is needed by 'garage_test', but there is only one version of libadolc library, which is 'libadolc.so.1', so you should execute the command 'sudo apt install libadolc2' to install it.

@Eclipsehelio
Thanks for the replay. But still, got the problem. The lib can be found in /usr/lib in my computer.

export LD_LIBRARY_PATH=/usr/lib:$LD_LIBRARY_PATH
ldd ./garage_test | grep libadolc.so
What's the output?

root@in_dev_docker:/apollo# export LD_LIBRARY_PATH=/usr/lib:$LD_LIBRARY_PATH
root@in_dev_docker:/apollo# ldd ./garage_test | grep libadolc.so
ldd: ./garage_test: No such file or directory
root@in_dev_docker:/apollo#

root@in_dev_docker:/apollo/bazel-bin/modules/planning/integration_tests#  export LD_LIBRARY_PATH=/usr/lib:$LD_LIBRARY_PATH
root@in_dev_docker:/apollo/bazel-bin/modules/planning/integration_tests# ldd ./garage_test | grep libadolc.so
libadolc.so.2 => not found
root@in_dev_docker:/apollo/bazel-bin/

I have manually linked the lib, and it  works.
But, it come with another problem.Possible the ipopt error.
root@in_dev_docker:/apollo/bazel-bin/modules/planning/integration_tests# ./garage_test
./garage_test: symbol lookup error: /root/.cache/bazel/_bazel_zy/540135163923dd7d5820f3ee4b306b32/execroot/apollo/bazel-out/local-dbg/bin/modules/planning/integration_tests/../../../_solib_k8/libmodules_Splanning_Sopen_Uspace_Strajectory_Usmoother_Slibdual_Uvariable_Uwarm_Ustart_Uipopt_Uinterface.so: undefined symbol: ZN7adoubleaSERKS

You said that 'libadolc.so.2' is under /usr/lib, so export LD_LIBRARY_PATH will be OK.
find -name 'libadolc.so*'

What's the option you use to build apollo?
It's better to execute the program under /apollo, like:
root@in_dev_docker:/apollo# ./bazel-bin/modules/planning/integration_tests/garage_test

nvidia@in_dev_docker:/apollo$ ./bazel-bin/modules/planning/integration_tests/garage_test
[==========] Running 5 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 5 tests from GarageTest
[ RUN      ] GarageTest.stop_obstacle

nvidia@in_dev_docker:/apollo$ export LD_LIBRARY_PATH=/usr/lib/x86_64-linux-gnu:$LD_LIBRARY_PATH
nvidia@in_dev_docker:/apollo$ ldd ./bazel-bin/modules/planning/integration_tests/garage_test | grep libadolc.so

@Eclipsehelio
Thanks a lot for the replay. Below is the output of the ldd command, is it right?
root@in_dev_docker:/apollo# ldd ./bazel-bin/modules/planning/integration_tests/garage_test |grep libadolc.so
libadolc.so.2 => /usr/lib/x86_64-linux-gnu/libadolc.so.2 (0x00007fd397eec000)

./ apollo.sh build_cpu  is the build option

@lucianzhong So the error 'undefined symbol: ZN7adoubleaSERKS' still exists? Have you tried to build with the option './apollo.sh build_opt_gpu' ?

The error still exist after ./apollo.sh build_gpu

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


(chores) camel-test-rocketmq: update container to the latest version

Updates the container to 5.1.4

This needs further review.


Consider to add a property to editor inputs to get at the editor that was used

As discussed, we currently probe onto a viewType in editor inputs to figure out the editor that was used when the opening was overridden (e.g. custom editors, notebooks). We should think about making this a general concept going forward.
Some rough notes from our discussion:

how does this correlate with the existing typeId?
should this be a required property for all editors (even e.g. welcome editor? extensions editor? that are typically never overridden)
can we find a name that makes it clear the value of this "thing" can be used in the override option to enforce this editor?
does it make sense to call a text editor default for this property, or should that change too? what does default really mean here?

Given https://github.com/microsoft/vscode/pull/124394#pullrequestreview-666598251, not sure this is needed anymore.

This was added by #126410


Change image links in README

The links are now pointing to Imgur, but they should point to the assets branch once the repository becomes public.

Not relevant any more. We store some images in docs, but I just realised that for big images we can just upload them to the README through github's interface and github will host it for us.


CNV-29788: Remove titles from Environment, Disks tab for customizing VM

📝 Description
Fixes:
https://issues.redhat.com/browse/CNV-29788
Remove titles from Environment and Disks tabs in Review and create VirtualMachine page to make the tabs more consistent with each other.
Add The following information is provided by the OpenShift Virtualization operator. text to Disks tab instead of help text icon displaying the popover containing this text, similarly as it is already done in the Environment tab, to keep the original information available to the user.
🎥 Screenshots
Before:

After:
Disks title removed, text of the original popover added:

Environment title removed:

@avivtur @metalice @pcbailey @upalatucci @vojtechszocs please review

/lgtm


[Feature] index transactions on avax without contract address just with from to or data

Description
index transactions on avax without contract address just with from to or data
Are you aware of any blockers that must be resolved before implementing this feature? If so, which? Link to any relevant GitHub issues.
I now need to index metadata on Avalanche (AVAX), and the metadata is not directly related to smart contracts. However, when I create a The Graph project, it requires me to specify a smart contract. Is it possible to support projects without smart contracts, where I only need to focus on the 'from', 'to', and 'data' fields within transactions?
Some information to help us out

[X] Tick this box if you plan on implementing this feature yourself.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

hey @BillInUK - so I understand your request here, you want to index transactions from or to a given address (rather than events or calls to a specific smart contract?) That isn't currently supported in Graph Node - would be interested in your use-case for this data (e.g. are you looking to show users their transaction history?)

For example, applications related to inscription are very useful. I don't care about from and to in transactions, just only want to parse the input of transaction.
Suppose I have an Inscription project:
`deploy:
"data:,{"p":"oprc-20","op":"deploy","tick":"test-1","max":"21000000","lim":"1000"}"
mint:
"data:,{"p":"oprc-20","op":"mint","tick":"test-1","amt":"1000"}"
`
I don't care about the from and to addresses in transactions; I'm more interested in the content inside the input, such as p, op, tickt, and amt.

i want to filter the inscription mint & transfer op with the graph, and sync the data into my postgresql.
Or subscribe mint and transfer events of specific tick which configured in my postgresql database.


Add CacheScatterLimitsDecorator

Purpose:
ScatterGL recalculates limits each time AutoAxes called.
This calculation starts to take a noticeable time at millions of points. Since ScatterGL uses a constant data set, the calculation of the limits can be cached on the first call.
I left the CacheScatterLimitsDecorator in the main namespace because it might come in handy for other purposes. In fact, all ScatterDataSources are constant at the moment.
If its location in the main project is invalid, it can always be moved to the OpenGL folder.

Hi @StendProg, this is a very good idea! Thanks for this PR
The ability to cache axis limits (and turn that feature on and off) seems very useful. I may add this functionality into the IScatterSource interface.

If its location in the main project is invalid, it can always be moved to the OpenGL folder.

I'll move this into the new ScottPlot.OpenGL project (#2399, #2496)


[ EPIC ] Create new "Deploy" section

### Tasks
- [ ] Move quantum serverless page from "run" to "deploy" section
- [ ] Create new tab named "deploy"
- [ ] TBD: more serverless pages from Jen's team?

We are going in a different direction for patterns, this page will now live in a dedicated "serverless" section in the left sidebar of the "Guides" tab


Downloader for iroh-bytes

The iroh-sync PR adds a downloader. In its current state it is unfinished, and cannot do retries and doesn't have a proper queue.
Features we need from the downloader from a high level iroh sync point of view:

Enqueue downloads as Hash, PeerId pairs
Get notified when a download is finished (through a future)

This high level API exists already in the PR, with public methods push(hash, peer_id) and async finished(hash: Hash). The actual actor below misses many features though. We want:

Retries with incremental backoff
Limits on parallel downloads and connection attempts (e.g. max 5 concurrent dials)
Being able to queue a hash with a list of peers in an order with different timeouts (e.g. try from peer 1 immediately, from peer 2 in 100ms, and from peer 3 in 1 second) - aborting/skipping of course if a download succeeded

--
copying some notes from the now-merged downloader improvements PR:
The Downloader currently acts immediately on new requests and doesn't do any retries. It also does not have a proper queue internally.
We want to be able to queue downloads with retries and different timeouts. My idea was to add a time-based queue, maybe using the TimerMap from iroh_gossip::util. Then I thought to change the API to expose download(hashes: Vec<Hash>, peer: Peer, delay: Option<Instant>). The actor would maintain a timer queue of pending requests and their delay, and poll that for the next operation, check if the operation is still needed, and if so enqueue the future to actually do the download. The actor should also keep track of running downloads, and have a limit of how many peers are being dialed or connected to in parallel (configurable). Code-wise, I thought to add at least a DownloadState for each pending download or so, and then similar to the current code have indexes into those for PeerId and Hash (and Instant if using something like the TimerMap).
With this, we can then:

On new entry message via gossip, enqueue download from

peer we recv'd from with a very small delay
from our other current active gossip neighbors with a larger delay
from the author with a larger delay plus random offset (*note: for this we need to add the peer info (address&region), with a signature, to the gossip message in addition to the SignedEntry)

Queue more retries if all fail up to some limit

Later on, we'd also want to add an explicit command to retry all missing hashes in a document from a peer, and optionally a mode that would do that automatically if we get new neighbors. This will initially be quite inefficient for large docs, because we have no ẁant/have` exchange in iroh-bytes. So maybe defer this a bit.
Edit
Another thought on the API, with peer and delay we'd also want to pass a enum RetryStrategy { None, Auto }  or so likely, with auto being some incremental backoff with max retries or similar.
Edit 2
Currently the quinn::Connections are dropped and thus cancelled if there is no immediate next download pending after a download finished. Likely we'll want to keep them around for a configurable-with-default amount of time, and then drop them if no operation happened. Redialing is quite cheap but not free (one roundtrip in current setup).

A few thoughts on this topic:
I think the downloader API should be separated into declarations of intent and hints.
A declaration of intent would be e.g.

I want to have .
I want to have ranges  of
I want to have collection  and it's children (shall we do collections here?)

For a declaration of intent you get a u64 handle that allows you to control/delete the intent.
Then you have hints. Hints are something like:

you might be able to get  from , says , or some more complex hint

It is then up to the downloader to make use of the hints, ask some discovery mechanism etc.
The downloader keeps track of the intents and makes sure that there are no duplicates, e.g. if two places want the same hash we don't want two download tasks.
An extension of the intent/hint scheme would be something like topics. E.g. instead of saying for these 10000 hashes you should try these 100 peers, have a "download topic" that in case of doc sync might correspond to a document. Then you can say "I want to download these 10000 hashes for topic ", and "these 100 peers are participating in topic ".

Given a hash that is collection, how do I retrieve the hashes that belong to the collection? Last time I checked the collection decoding has a list (set?) of hashes. Please correct me if I'm wrong or if this has changed

You need the CollectionParser that is part of an iroh node. That likely means that the Downloader will have to be generic over C: CollectionParser, and the collection parser will have to be passed into the downloader. You can then call collection_parser.parse() to obtain a list of hashes from a collection. The CollectionParser can be set by the application to support other kinds of collection than the default iroh collections.

Given a hash, how to map a range to a list of hashes?
Not sure I got the question right, and @rklaehn knows much more than me about hashes and ranges. I think a range always refers to part(s) of a single blob identified by a single hash. So you can say "gimme these ranges of this blob identified by hash". I do not know how this relates to collections and if we have a concept of "ranges of a collection".

The parts that @Frando mentions seem clear to me how to implement, @rklaehn 's not so much.
Could you guys help me here? (sorry still noob to these parts of the code):

Given a hash that is collection, how do I retrieve the hashes that belong to the collection? Last time I checked the collection decoding  has a list (set?) of hashes. Please correct me if I'm wrong or if this has changed

What @Frando said

Given a hash, how to map a range to a list of hashes?

The protocol for individual blobs goes like this: you ask for a hash and a set of ranges of chunks (1 chunk = 1024 bytes). Then you get back data only for what you have asked for. The default thing is to just ask for a hash and all it's chunks RangeSet2::all(), then you get all the data for the blob.
For collections it is more complex. You can specify ranges for the collection itself and for each of it's children. E.g. you have downloaded a collection and half of it's children and were interrupted. You then figure out what is missing and ask for exactly that.

Thank you both!

Here is an issue that is related to the downloader. Probably not initially, but eventually we want to support this: https://github.com/n0-computer/iroh/issues/839

@Frando & @divagant-martian both agree that #1420


Missing SWIFT_VERSION in project file

It appears that the project does not have the SWIFT_VERSION build setting set. When building with Xcode 8 I get:
Use Legacy Swift Language Version (SWIFT_VERSION) is required to be configured correctly for targets which use Swift. Use the [Edit > Convert > To Current Swift Syntax menu to choose a Swift version or use the Build Settings editor to configure the build setting directly.

Yeah, haven't migrated to new versions of swift yet. Currently going through all of our open source libs and fixing them up.
I can work on this next though if it'd help. Are you moving to Swift 3?

No just 2.3.
It would be nice if there was a 2.3 release regardless so anyone looking to migrate slowly has the option. Most other open source libraries are doing this as well.

#26

Are you using Carthage? This shouldn't affect cocoapods afaik.

Done, but struggling to get travis working...

Yeah Carthage. Cocoapods is affected, you are able to write a little snippet of code to inject the SWIFT_VERSION into the project files.

Yep. Will release since I get travis going. Maybe will be tomorrow morning at this rate.
Travis was down for a while this afternoon.

https://travis-ci.org/linkedin/LayoutTest-iOS/builds/161701390

I can checkout and test with the commit in your PR.

Released version 1.4.0.
Feel free to reopen if it doesn't work.

Thanks. Testing now.

Looks like it built fine. Testing my main project next.

All good 👍


[BUG] VariableNotFound warning in keyword documentation section

When I add Documentation section to a keyword, I get VariableNotFound error in examples. Documentation should be exempt from code analysis
Here is a sample resource file:
*** Keywords ***
MyKeyword
    [Documentation]    Sample keyword
    ...
    ...                *Examples*
    ...                | ${result}= | MyKeyword | Hello World |
    ...                | ${result}= | MyKeyword | ${logtext} |
    [Arguments]        ${message}
    Log                ${message}
    [Return]           OK

${result} and ${logtext} have three dots and hovering over them shows VariableNotFound.
**Desktop **

VS Code Version 1.76.2
RobotCode Version 0.31.2
OS: Windows
Python Version 3.11.1
RobotFramework Version 6.0.2

Hi Pekka,
sorry, this is not a bug, this hint is there to protect you from unexpected behavior. According to RobotFramework documentation (here and here), variables in the [Documentation] tag are replaced with the content at runtime, if the variable does not exist, the text remains as it is. This is to extend the documentation at runtime.
To avoid accidentally replacing a variable in the documentation you should mask variables with a \ backslash.
I have extended your example and added screenshots to illustrate this...
This is what you get when somewhere a variable ${result} is defined (in the suite file or a global variable).

or this:

This is the correct way to define variables examples:

If you don't want to mask the variable, you can disable the hint, as well as all other error messages for a line by adding the comment # robotcode: ignore to the end of the line.
Something like this:
*** Keywords ***
MyKeyword
    [Documentation]    Sample keyword
    ...                *Examples*
    ...                | ${result}= | MyKeyword | Hello World |    # robotcode: ignore
    ...                | ${result}= | MyKeyword | ${logtext}  |    # robotcode: ignore
    [Arguments]        ${message}
    Log                ${message}
    [Return]           OK

but you should only do this if you really know what you are doing there.
I am currently in the process of completely reworking the code analyzer, there will then also be a finer settings to turn certain messages on and off.
If you have any other ideas about this...?

Hi Daniel,
I have been using RF for a while and never noticed that feature. RF has dynamic features and they might cause unexpected behavior. It is good that RobotCode warns about this, and your workarounds are good. The only wish left is a bit better documentation for the next user.

Ok, then I close this issue and  I will do my best, to write a "bit" more documentation :-)


Endpoint for contributed items per user.

In GitLab by @tparkola on May 13, 2022, 08:33
Regarding to what we discussed in the meeting, I created an endpoint (GET /api/contributed-items) that returns the current list of BasicItemDtos, that user has contributed to.
The changes are merged to dev.
@vronk @laureD19 @KlausIllmayer @stefanprobst

I can confirm, that the /api/contributed-items delivers now back the latest version of all the items that were approved/suggested/ingested. But it neither gives an option to only show such a list based on the status (e.g. show me only the items with the status approved) nor does it give the information about the status in the response value (there is only id/category/label/persistentId/lastInfoUpdate but no status).
I agree with Stefan that we could get rid of the status boxes in the "My contributed items" as the difference between "approved" and "suggested" is not really explainable and "ingested" is a very special status, which usually only few users will ever see. But to have a clue, I would prefer to add the status in the response so that it can be shown in the frontend. @tparkola Can you add the status in the respone of GET /api/contributed-items? Then I would hand it over to frontend.

Tested it and the results do look reliable. I see for two tested accounts the contributed items where there was once an activity but in the meantime some other account performed another activity, which currently do not show up but can be seen in the new endpoint. There is also the information on the current status in the payload.
Frontend can in my opinion now use this endpoint to replace the old contributed items view with the result set coming from the endpoint GET /api/contributed-items.


[HW]

Name: Priyanka Paul
Email<EMAIL_ADDRESS>Linkedin Profile:
Attach the homework screenshots below for both step II and step III:

Hi Priyanka. Could you provide a screenshot of the last CRUD step in the repo, step 3 please?

Closing due to inactivity.


Add Choicy for Supercell games

Choicy works with all 3 of these games, according to my testing
I have not tested this PR, since I have no idea how to compile it

While we're adding bypasses, com.jio.myjio works with Shadow and just the URL handlers bypass


SeleniumDriverConfigHelper misses some configuration keys

The class "SeleniumDriverConfigHelper" is missing some ocnfiguration keys. For example "driver.name" is missing. Maybe there should be some helper methods like "isChromeBrowser()", "isRemoteBrowser()", etc

This problem was solved in version 0.2.13. You can now add any configuration key via the qtaf.json file


Update iOS minimum deployment target

platform :ios, '11.0'

Covered in https://github.com/webfactorymk/flutter-template/commit/bbc5147d6a932db21ce78fc6491c448c74d23a04