V2 Preview-1 In app error messages for creating a new project without connections

Is your feature request related to a problem? Please describe.
My first time downloading VoTT I went to create a new project but was unsuccessful because I hadn't created connections before attempting to create the new project. I did see that in the documentation it said that connections were required to create a new project, but I didn't see that until after the fact. Currently when I just create a new project without any connections I click "Save Project" and nothing happens.
Describe the solution you'd like
It would be nice if an error message came up saying that connections are required to create a project (similar to how one comes up if you try to create a project without a name)
Describe alternatives you've considered
Additional context
Even though it does say it's required in the documentation, I read through all the documentation before playing around with it and still got stumped (it's tough to remember all of the details). It's also safe to say a decent portion of folks will not read the documentation at all.

Thanks for your feedback @alancast. This is a bug.  An error message should display when you attempt to create a new project without selecting a source & target connection.

This has been released as part of Preview 2


Label to classify issues by component

I would like to scrub github issues in this repo to identify which pertain to the standalone cinder provisioner.  In my own repo I would use a label "standalone-cinder-provisioner" or something.  Could we add this label or is there a better way to achieve this?

/cc @jsafrane @anguslees @dims @FengyunPan2

Done. added component/standalone-cinder-provisioner is that ok?

Thank you!!!


Opt reporting: add signatures to stats

Compiling with --instrument-onnx-signature and then gathering the output at runtime, we can then analyze ops using their runtime signature as well.
> roberta-base-11 % ONNX_MLIR_INSTRUMENT_FILE=run1.log RunONNXModelZoo.py -m roberta-base-11 -c="-O3 --instrument-onnx-signature"
...
> alexe@AChampignon roberta-base-11 % make-report.py -c run1.log -s sig -l 2
Statistic legend:
   num: comma separated list of shapes with inputs followed by results

Statistics start, all ops, ordered_by name
  onnx.Add, 173
    12: sig, 1x12x1x1xfloat, 1x1x1x1xfloat, 1x12x1x1xfloat
    25: sig, 1x1x1xfloat, float, 1x1x1xfloat
    12: sig, 1x1x3072xfloat, 3072xfloat, 1x1x3072xfloat
    12: sig, 1x1x3072xfloat, float, 1x1x3072xfloat
    26: sig, 1x1x768xfloat, 1x1x768xfloat, 1x1x768xfloat
    85: sig, 1x1x768xfloat, 768xfloat, 1x1x768xfloat
    1: sig, 1x1xint64_t, int64_t, 1x1xint64_t

That way, we can quickly see which type of ops use which pattern.

@AlexandreEichenberger still see that this patch deletes .md files that are not relevant.

Jenkins Linux s390x Build #12489 [push] Opt reporting: add signa... started at 03:52

Jenkins Linux ppc64le Build #11482 [push] Opt reporting: add signa... started at 04:00

Jenkins Linux amd64 Build #12477 [push] Opt reporting: add signa... started at 02:52

Jenkins Linux s390x Build #12489 [push] Opt reporting: add signa... passed after 1 hr 42 min

Jenkins Linux ppc64le Build #11482 [push] Opt reporting: add signa... passed after 1 hr 50 min


Error creating project

Describe the bug
After installing from pypi, label-studio crashes on project initialization.
To Reproduce
Steps to reproduce the behavior:

Install from pypi following the readme: https://github.com/heartexlabs/label-studio#install-locally-with-pip
Run label-studio init test

Expected behavior
No errors during the init of a project.
Screenshots
label-studio init test
=> Database and media directory: xyz/.local/share/label-studio
=> Static URL is set to: /static/

Please enter default user email, or press Enter to use "default_user@localhost"
Email:
Default user password:
Traceback (most recent call last):
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/sqlite3/base.py", line 413, in execute
    return Database.Cursor.execute(self, query, params)
sqlite3.OperationalError: no such column: project.template_used_id

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

Traceback (most recent call last):
  File "xxx/label-studio/bin/label-studio", line 8, in <module>
    sys.exit(main())
  File "xxx/label-studio/lib/python3.8/site-packages/label_studio/server.py", line 261, in main
    _init(input_args, config)
  File "xxx/label-studio/lib/python3.8/site-packages/label_studio/server.py", line 138, in _init
    _create_project(
  File "xxx/label-studio/lib/python3.8/site-packages/label_studio/server.py", line 70, in _create_project
    project = Project.objects.filter(title=title).first()
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/models/query.py", line 678, in first
    for obj in (self if self.ordered else self.order_by('pk'))[:1]:
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/models/query.py", line 287, in __iter__
    self._fetch_all()
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/models/query.py", line 1308, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/models/query.py", line 53, in __iter__
    results = compiler.execute_sql(chunked_fetch=self.chunked_fetch, chunk_size=self.chunk_size)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/models/sql/compiler.py", line 1156, in execute_sql
    cursor.execute(sql, params)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/utils.py", line 66, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/utils.py", line 90, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "xxx/label-studio/lib/python3.8/site-packages/django/db/backends/sqlite3/base.py", line 413, in execute
    return Database.Cursor.execute(self, query, params)
django.db.utils.OperationalError: no such column: project.template_used_id

Environment (please complete the following information):

OS: Ubuntu 20.04
Label Studio 1.0.1 (from pypi)

Sorry for the noise, deleting the ~/.local/share/label-studio directory and starting over, solved the issue.


Whatsapplink in Firefox doesn't work

Describe the bug
If you click on the Whatsapplink in an area without a trader, no page appears in Firefox, or the page is not found because whatsapp:// is recognized as protocol.
To Reproduce
Steps to reproduce the behavior:

Go to location without a trader
Click on whatsapp social link
See error

Screenshots
If applicable, add screenshots to help explain your problem.

Device and OS:

Firefox

do you have the whatsapp desktop app running?

doesn't work on Safari either.
But I don't have Whatsapp App on the MacBook. But it worked properly on Chrome.

ah, interesting...
All browsers target the same url. Firefox and safari get redirected to a whatsapp:// url, while chrome stays on the message-page.
You can see that when using the browser back button in FF or Safari.

https://medium.com/@jeanlivino/how-to-fix-whatsapp-api-in-desktop-browsers-fc661b513dc

ah, we use the api-Endpoint. Should work on all mobile browsers.
It seems that we need to use the web-Endpoint for desktop.
Chrome seems to work fine with both. Safari und FF get brute force forwarded to another page.
https://webapps.stackexchange.com/questions/106145/whatsapp-link-with-phone-number-doesnt-open-whatsapps-web-app
My question: is that social link worth the effort?
If so: we need a browser detection stating what browser we are on and on what device.
I thing there are some capable npm-packages for that.
Then we need some forks in code to tamper the url.
Who ever implements that: please consider to hide the npm package behind some kind of service, please. Those libs tend to be tricky at the edge-cases, so we might have to try some.

Works on macOS/Firefox, macOS/Safari and macOS/Chrome


sortable

would it be possible to also make it sortable?

I created a different plugin for sortable markdown table
https://github.com/hieuthi/joplin-plugin-markdown-table-sortable


[casclib] Add option CASC_UNICODE

Fixes #21682
PR https://github.com/microsoft/vcpkg/pull/21472 update casclib to new version, it change VCPKG use CMakeLists.txt which provide by source. This PR breaks unicode support for the library on Windows, set option CASC_UNICODE for casclib.

Thanks for the fix!


Add extension methods for ConcurrentDictionary + AsyncAtomicFactory

coverage: 99.124% (+0.007%) from 99.117%
when pulling 28a49fd8cb5db1cccaa2c47b3dff3748798719d3 on users/alexpeck/concdicasyncatomic
into 8bfee3d85f364cfb9602fa5599bcf438dd26edda on main.


Update Plugin Artifacts Links for v0.141.4

Automated changes by create-pull-request GitHub action

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


role:repo_graylog: repo uses local pgp key

Changed repo so that it will download gpg-key first on localhost (just like https://github.com/Linuxfabrik/lfops/commit/83d45db09a332731785ff277a7d50fe5596a52b8)
Only testen on RedHat, not Debian

thanks :D


Support for _id of binary type (to use UUIDs as _id)

While using the ObjectID type is the default implementation in MongoDB, it's technically possible to use anything for the primary key as long as you use a unique value. Some implementations (including mine) use a UUID as the primary key. UUIDs are stored in MongoDB as Binary types. The provider in its current state doesn't work in that situation.
These PR are the changes I've made to make it work. The only thing not included is a trait that automatically generates the UUID on the "creating", for example:
<?php

namespace App;

use MongoDB\BSON\Binary;
use Ramsey\Uuid\Uuid;

trait MongodbBinaryUuid
{
    /**
     * Boot the trait.
     *
     * @return void
     */
    protected static function bootMongodbBinaryUuid()
    {
        static::creating(function (Model $model) {
            if ($model->{$model->getKeyName()}) {
                return;
            }

            $model->{$model->getKeyName()} = new Binary(static::encodeUuid(Uuid::uuid1()), Binary::TYPE_UUID);
        });
    }
}

Note that my actual trait is actually more complete, with scopes and such.
I'm not sure this is the best way but it works for my use case and I'm curious to hear feedback. Surely I can be the only one with a need to use UUIDs as _id right? ;)

@jenssegers any chance of merging this?

I am wondering how hard is it to pass an object of type Binary instead of string. I don't like the auto casting behavior even for ObjectId, now we have it for Binary and for properties that ends with _id.
I hope for a branch that stop doing so: https://github.com/jenssegers/laravel-mongodb/issues/1761

@mfauveau, do you run tests on this PR locally? Build fails in travis.

@jenssegers, this commit has breaking changes, tests failed, because this change don't casting ObjectId to string


Use filter table tree for Available Views view implementation

Use FilterTree
Add tooltips to TreeNode and TableCell
Enable sorting and filter
Description of view is part of the tooltip instead of being printed below view name

fixes #1113

Signed-off-by: Bernd Hufmann<EMAIL_ADDRESS>

Now matter how wide the Available Views panel is resized, its scrollable content is just a little bit wider than the available width, so a horizontal scrollbar appears unnecessarily with a very small scrollable range. There might be some margin width unnaccounted for somewhere.

This should be fixed in the updated version


Kubernetes project updates

Issue #, if available:
Description of changes:
Kubernetes project updates from 1.27 to 1.30
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


[admin] allow administrator role access organization admin #143

Closed issue by updating organization admin and also added
a couple number of tests for these changes.
Fixes #143

Coverage increased (+0.004%) to 97.201% when pulling 679b46c656fd4fca02353eb6c46cb485b4248360 on issue-143-administrators_access_organization into 7024383f746a23589c070b59cf85dc632c2a55be on master.

Sure. Will do that today

Good progress! Sorry for my late response.
For some reason while testing, the permissions were not added to the administrator group correctly in my env.
So I wanted to run the migration backward to run it again and found this:
    raise IrreversibleError("Operation %s in %s is not reversible" % (operation, self))
django.db.migrations.exceptions.IrreversibleError: Operation <RunPython <function allow_admins_change_organization at 0x7fa9626b6dd0>> in openwisp_users.0010_allow_admins_change_organization is not reversible

PS: also rebase on the latest master please

@nemesisdesign It says that because I didn't make the migration reversible. I didn't see a reason to make this reversible since its absence will cause other core functionalities which were added to malfunction.
Should i make it reversible?

@nemesisdesign It says that because I didn't make the migration reversible. I didn't see a reason to make this reversible since its absence will cause other core functionalities which were added to malfunction.
Should i make it reversible?

In this case we can simply set the backward migration to noop, no operation, look it up on the django docs please.

@nemesisdesign This is ready for review


Upgrade htsjdk, gatk, gkl and barclay

Re-opens #140. Requires that GATK includes the new Barclay or that we hack some code paths that are not working...
EDITED:
It also closes #76

I realized that the program itself is not broken except if the tool is runing without any parameter... This is not tested in travis, because it returns 1 anyway...

Requires https://github.com/broadinstitute/gatk/pull/2455

Another option for getting this in ASAP is to implement our own Main.mainEntry() again, and use program.getCommandLine().getCommandLineParser() method and call usage with the new params. Maybe this won't be possible, because the access of the method is protected...

Tested that it works in our system.


App Layout with drawer slot

Next major version for vaadin-app-layout
Includes slots for drawer and navigation bar
Has API to change precedence mode between drawer and navigation bar as shown in following mockups:

Full height drawer

Full width navigation bar

2.0.0-beta1 released today

@alvarezguille how to change to “ Full width navigation bar” mode in AppLayout 2?


Preview versions of Visual Studio are not found

Steps to Reproduce

flutter doctor
[X] Visual Studio - develop for Windows                                                                                X Visual Studio not installed; this is necessary for Windows development.
flutter run
Unable to find suitable Visual Studio toolchain. Please run flutter doctor for more details.
visual studio 2019 path:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Preview
on master channel
ENABAL_FLUTTER_DESKTOP=1
Logs

C:\Program Files (x86)\Microsoft Visual Studio\2019\Preview

This is a pre-release version, which it looks like vswhere doesn't return without a -prerelease flag. The lookup logic should probably be changed to detect that case, but warn about it.
Is there a reason you're trying to use a pre-release version of VS for Flutter development?

I'm not sure why this was closed; per my comment above changes are necessary to the tool to make preview versions work.

/cc @franciscojma86

Update 2020:
Please install the "Desktop development with C++" workload, including the following components:
MSBuild
MSVC v142 - VS 2019 C++ x64/x86 build tools
- If there are multiple build tool versions available, install the latest
Windows 10 SDK (10.0.17763.0)


E2E - Misc Doc Fixes

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


Add support for multi-file type instantiation

Currently, types that require multiple distinct files to load (think neural networks with a parameter file and a hyperparameter file) are not supported, and if they are, it's probably by accident.
def load_from_disk(param_file: str, hparam_file: str) -> MyModel:  # needs both files to load successfully
    params = load_params(param_file)
    hparams = load_hparams(hparam_file)
    return MyModel(params, hparams)

Loading artifacts from multiple files should look something like this:
m = shelf.get(["s3://my-bucket/my-file1.txt", "s3://my-bucket/my-file2.txt"], MyModel)

BUT, in case the files are saved in a directory prefix, shelf could allow the following shortcut:
# in S3:
# my-bucket/my-file1.txt
#          /my-file2.txt

m = shelf.get("s3://my-bucket/", MyModel)  # assumes that there are no other files in "my-bucket".

The latter case is not trivial at all, though. What happens if the files are not passed to the type deserializer in the correct order? I'm not sure that they are currently sorted in any order, actually. Might require a custom type order field to set on the IO struct.

This is sufficiently (for the first iteration) solved by #7 and #8.


I inputed my name finally! yaay

if you can remove the line without a name on it , you're good to go

Hi Succynice thanks for the feedback.
Are you referring to Line 68 and how can i do this


Holiday api useful for real work

for many industries, we care about if a day is a working day (i.e. people could be in the office to pay money or send letter etc.). For such purpose, it is better to have a API like IsBusinessDay(day) to return False on, for example,  2 Jan 2017

For which counry you think the 2 jan 2017 is no public holiday?


add a convenient way to get the object_store_id of a input dataset

We would like to access the object store ID of an input file.
Something like {input_size} would be nice to have, maybe object_stors_ids that returns a tuple (param name, object_store_id)?
Deferred datasets need to be excluded as they are not yet in any object store.

Closed by: https://github.com/galaxyproject/total-perspective-vortex/pull/125


PSS_ALLOCATOR garbled syntax for function pointer fields

Link to article:

Article
Article .md source

Problem:
The syntax for the function pointer parameters in the PSS_ALLOCATOR struct are garbled in its article.
I see this in the article. Notice especially the mismatch parentheses for the AllocRoutine field:
typedef struct {
  void  *Context;
  void  )(void *Context,DWORD Size)  *(*AllocRoutine;
  void((void *Context, void *Address) * )FreeRoutine;
} PSS_ALLOCATOR;

In my local copy of the Windows SDK (version 10.0.18362.0), I see the following in ProcessSnapshot.h:
typedef struct
{
    void* Context;
    void* (WINAPI *AllocRoutine) (_In_ void* Context, _In_ DWORD Size);
    void (WINAPI *FreeRoutine) (_In_ void* Context, _In_opt_ void* Address);
} PSS_ALLOCATOR;

When I look at the source of the article, I don't see any struct definition to fix, so it looks like the merging process is splicing in the wrong content.

The docs still have garbled content as of October 10, 2022.


Fix did not removed css class dropfeedback

The editor content did not remove css class dropfeedback sometimes.
Steps to Reproduce:

Drag a file into Tab or Tabs Container or Title ActionBar
Close All Files

@f111fei wow great catch, this would explain https://github.com/Microsoft/vscode/issues/8373 which I could not reproduce 👍

@f111fei I did one change (https://github.com/Microsoft/vscode/commit/20edd6a21cc7266a94153fc91abc4efa3599ddd4) because I wonder why you added it:
if (overlay) {
    DOM.addClass(node, 'dropfeedback');
}

First of all, the check for overlay is not needed because it is created before. Then I am not sure why the code in the if block is needed?

@bpasero  If drap into editor-container we need add dropfeedback

@f111fei that should be handled by the overlay, no? Can you demonstrate how currently (after my change) there is an issue with the drop feedback? I could not find any.

oh, sorry. You are right. I think we must add dropfeedback class here at first. If exist an overlay we can remove dropfeedback

That makes sense 👍 . I will push a change to remove the dropfeedback when we have an overlay.


[SPARK-27544][PYTHON][TEST][BRANCH-2.4] Fix Python test script to work on Scala-2.12 build

What changes were proposed in this pull request?
Since SPARK-27274 deprecated Scala-2.11 at Spark 2.4.1, we need to test Scala-2.12 more. This PR aims to fix the Python test script on Scala-2.12 build in branch-2.4.
BEFORE
$ dev/change-scala-version.sh 2.12

$ build/sbt -Pscala-2.12 package

$ python/run-tests.py --python-executables python2.7 --modules pyspark-sql
Traceback (most recent call last):
  File "python/run-tests.py", line 70, in <module>
    raise Exception("Cannot find assembly build directory, please build Spark first.")
Exception: Cannot find assembly build directory, please build Spark first.

AFTER
$ python/run-tests.py --python-executables python2.7 --modules pyspark-sql
Running PySpark tests. Output is in /Users/dongjoon/APACHE/spark/python/unit-tests.log
Will test against the following Python executables: ['python2.7']
Will test the following Python modules: ['pyspark-sql']
Starting test(python2.7): pyspark.sql.tests
...

How was this patch tested?
Manually do the above procedure because Jenkins doesn't test Scala-2.12 in branch-2.4.

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/testing-k8s-prb-make-spark-distribution-unified/10133/
Test PASSed.

Test build #104825 has started for PR 24439 at commit 5eba36a.

cc @srowen and @HyukjinKwon

Test build #104825 has finished for PR 24439 at commit 5eba36a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Retest this please.

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/testing-k8s-prb-make-spark-distribution-unified/10135/
Test PASSed.

Test build #104827 has started for PR 24439 at commit 5eba36a.

Test build #104827 has finished for PR 24439 at commit 5eba36a.

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/104827/
Test PASSed.

Merged to branch-2.4. Thank you for review, @srowen .


Governance and ERC20Votes Integration

I think it would be a great goal to try to match OZ's governance contracts in this pattern, as some kind of governance is super important for the management of systems using diamonds.
I also noticed that some of the current ERC20 functionality has the ability for snapshots, which I believe is similar to what ERC20Votes needs to function.
Ideally sticking with as much of the same logic as OpenZepplin seems ideal, as their code is well audited and they have developed good standards.

Some work has begun on this, predating even the OZ governance release.  We need to decide whether it makes sense to build on top of that old branch, or to start fresh.  Don't have the capacity for this in the immediate future, but hopefully soon.  A PR would be considered, in the meantime.


Atlas messenger crashing when register with new user

Hello,
I am having problem with the registration with atlas messenger but sometimes it's crashing when i log in too with my user name.
And I'm having these debug descriptions are as
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[1].
and
error = "Unprocessable Entity";
status = 422
Please let me know where I'm making mistake.

HI @gopaldevra please paste the call stack. When exactly is this issue happening?  What step of authentication?

User registered and got identity token: (null) (error=Error Domain=com.layer.Atlas-Messenger Code=2 "Unprocessable Entity" UserInfo={NSLocalizedDescription=Unprocessable Entity, responseBody={"status":"422","error":"Unprocessable Entity"}})


Custom queries management

Hi all, sorry if this is a silly question but I haven't figured out if it is possible to create and save custom queries with filters. Am I wrong or am I missing something?
I've seen the SavedQueryFactory allows creation but only passing an integer as parameter, so it seems to me this feature is not supported. Also, SavedQuery object does not accept any filters.
I would like to create custom queries using filters like status or assignee.
I'm already using the API which is awesome, simplified to me the management and saves time.
Thanks in advance.

I am closing this until someone decides to check and implement this


re-implement Compiler._add_ctes() method

Doing this would enable:

ephemeral materialization, and
usage of CTEs in data tests

Fixes:

#26, and
https://github.com/dbt-msft/dbt-synapse/issues/25

@jtcohen6 do you mind taking a rough stab at templating this out? In https://github.com/dbt-msft/dbt-synapse/pull/29, you already did this for dbt-synapse, but since I re-wrote the Git history, I can't see you actual changes.
update --
https://github.com/dbt-msft/dbt-synapse/commit/9180da5a6d092c6b4b22d5388aa6592d2b9d77c4 shows what was done, @jtcohen6 can you just confirm that this approach still stands?

I think these are going to be two separate issues going forward. The solution for data tests should be much more straightforward—newly, however, it will be relevant to schema tests as well.
Tests
In v0.20.0, dbt will be templating data tests via a 'test' materialization, defined here. This will be relevant for both data tests and schema tests. In most cases, the final test query will look something like:
select
  count(*) as failures,
  count(*) !=0 as should_warn,
  count(*) !=0 as should_error
from (
  ... actual test SQL goes here ...
) dbt_internal_test

That's a subquery, which is new for both data tests (previously a CTE) and schema tests (previously no wrapper). I'm going to guess that, in T-SQL, you cannot use CTEs nested within a subquery, just as you cannot use CTEs nested within a CTE. Believe it or not, some SQL syntaxes are happy with the former even if they don't support the latter.
So you may need to override the 'test' materialization for sqlserver + synapse, which is totally doable. (Macro inheritance doesn't work the same for materializations as it does for dispatch, so you may need to reimplement it in both places.) I imagine one possible approach would be to create the test SQL as a view or temporary table first, and then select from the view/temp object in the final query. That's very similar to what dbt test --store-failures does, except with permanent/persisted tables.
Ephemeral models
The general approach outlined in https://github.com/dbt-msft/dbt-synapse/commit/9180da5a6d092c6b4b22d5388aa6592d2b9d77c4 is still the same, though I was doing it on easy mode, and didn't actually try to solve for ephemeral model compilation :)
The mechanism is: reimplement the compiler, define a new one (like in dbt/adapters/sqlserver/compilation.py), and reimplement methods like _inject_ctes_into_sql  and _recursively_prepend_ctes.
Taking a step back: In order to support ephemeral models with CTEs on SQLServer and Synapse, given that T-SQL doesn't support CTEs nested inside either CTEs or subqueries, would we have to take the same approach as the one outlined above?
Namely, instead of recursively prepending ephemeral models as CTEs to the start of the query:
with  __dbt__cte__ephemeral_model as (

with my_cte as (

    select 1 as id

)

select * from my_cte
),ephemeral_model as (

select * from __dbt__cte__ephemeral_model

),

another_cte as (

    select 2 as id

)

select * from ephemeral_model
union all
select * from another_cte

Recursively prepend them as temp tables, to be executed in-transaction with the body of the query:
create table #ephemeral_model as (

with my_cte as (

    select 1 as id

)

select * from my_cte
);

with ephemeral_model as (

select * from #ephemeral_model

),

another_cte as (

    select 2 as id

)

select * from ephemeral_model
union all
select * from another_cte

Or is there another approach I'm missing?


Update postgis to use Npgsql.NetTopologySuite and running integration tests

Fixes https://github.com/demetrixbio/FSharp.Data.Npgsql/issues/75
Also added ability to run tests on future PRs via github actions. It setups dvdrental database and runs tests on Tests.sln

Integration tests looks like https://github.com/demetrixbio/FSharp.Data.Npgsql/pull/83/checks?check_run_id=564988874
Also removed some minor leftovers and tidbits

Also fixes https://github.com/demetrixbio/FSharp.Data.Npgsql/issues/3
😄

Kickass.


Placeholder release notes for december release

Change log removed - I have not linked them into the TOC yet - just here for people to edit.
Wes - do we need to do anything to get the quick links generated?

Wes - do we need to do anything to get the quick links generated?

The process is still a little hard-coded but for the time being what needs to happen is we need to update the various csv files with the new versions under https://github.com/Azure/azure-sdk/tree/master/eng/scripts and then run https://github.com/Azure/azure-sdk/blob/master/eng/scripts/Generate-Release-Tables.ps1 and either pass in the releases folder or update it https://github.com/Azure/azure-sdk/blob/master/eng/scripts/Generate-Release-Tables.ps1#L3 to point to the new location.
Updating the versions right now will lead to essentially and empty table because they haven't been released yet. You could generate the table based on the last released versions for now if you want to get the table there.
Do note I still have plans to improve this process as well have a "live" table for each language that represents the current set of released packages. My hope is going forward we will snapshot the live version for these quicklinks when we do the release notes.


Automatic Jupyter environment detection

Something similar to tqdm's auto module would be helpful. This would mean I can run the same code in Jupyter or a terminal and get the same rich experience.
Basically, it would just be a small wrapper that does a simple check to whether or not we are in an IPython environment or a regular Python one, and then import the console and print objects from the correct package. Then you could write code like this, knowing it would work wherever it was run:
from rich.auto import print

print("[bold magenta]Hello, world![\]")

Alternatively you could put this functionality into the default imports; I'm not sure why you would ever not want this (other than to avoid the overhead of checking, but that's fairly minimal).

Yeah, I think I could do something along those lines.

Fantastic! This project is amazing by the way, literally the only thing holding me back from using it more at the moment is the Jupyter support. I'm sure that things like this and getting a Jupyter-friendly version of the progress bar would probably really help to expand uptake.

Another big vote for this feature - would love to be able to use this inline in Jupyter.

For those interested, I have a 'jupyter' branch with improved Jupyter support.
It's completely transparent now. The same imports and API will work in the terminal and notebook. Here is a screenshot:

This looks awesome! I just had a go and it works great :) I noticed that progress bars still don't work; is this something that will get added later on or would that be a separate piece of work (happy to start up a PR for that if you need).

Looks awesome!!

Just noticed that the link rendering isn't quite working for me. That could be because I'm in Jupyter Lab rather than notebooks. FYI here's what I get:

Glad you like it.
Not sure about the progress bars TBH. If I have to write some Javascript widget, then that's a bit beyond the remit of this library. Hopefully there is a way to re-use the existing progress bars without too much effort.
@big-o Your data is a tuple. Try print(*data)

Exploding the tuple fixed it, thanks :)

Auto-magic Jupyter detection is in v1.3.0 now. Progress bars are a possibility for a future release. If I can figure out a nice way to integrate them with Jupyter.


Re-move all the complex settings types to a module

For some reason, even though tests passed for the original pull request, after merging there were validation issues that I haven’t yet tracked down. Need to figure that out, then re-merge these changes.

Think this was just an issue with the merge, it's an easy fix.
Line 6 here shouldn't be importing DatabaseModel (it comes from pydantic_settings.models in the line above)
https://github.com/joshourisman/django-pydantic-settings/blob/036e56377353071ad17efa93a37a11b489b775e1/pydantic_settings/default.py#L5-L6


Default Alias Not Working With Ubuntu 14.04, Shell Still Defaults to System

Hi,
I've been searching around and haven't been able to find any solutions for ubuntu, only one issue in Mac. I've been trying to make it so that my Node Version defaults to a stable version when opening up a new shell. Unfortunately each time I try to run
nvm alias default v0.12.2

I keep getting node -v = 0.13.0-pre
my nvm ls shows as this on a new shell:
    iojs-v2.3.1
        v0.12.2
        v0.12.4
        v0.12.7
->       system
default -> 0.12.2 (-> v0.12.2)
node -> stable (-> v0.12.7) (default)
stable -> 0.12 (-> v0.12.7) (default)
iojs -> iojs-v2.3 (-> iojs-v2.3.1) (default)

and shows the same after the command is run. Any and all help would be appreciated. Thank you.

What does nvm --version and nvm debug say? What shell are you using? Which (if any) of your profile files have nvm lines in them, and which lines are these?

vor0220@vor0220-ubuntu:~$ echo $PATH
/home/vor0220/newenv/bin:/home/vor0220/newenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/home/vor0220/bin:/home/vor0220/newenv/bin:/home/vor0220/go/bin:/usr/local/go/bin
vor0220@vor0220-ubuntu:~$ echo $NVM_DIR
/home/vor0220/.nvm
vor0220@vor0220-ubuntu:~$ nvm --version
0.25.4
vor0220@vor0220-ubuntu:~$ nvm debug
$SHELL: /bin/bash
$NVM_DIR: $HOME/.nvm
nvm current: system
which node: /usr/local/bin/node
which iojs: /usr/bin/iojs
which npm: /usr/local/bin/npm
npm config get prefix: /usr/local
npm root -g: /usr/local/lib/node_modules
vor0220@vor0220-ubuntu:~$ echo $0
bash
vor0220@vor0220-ubuntu:~$ which node
/usr/local/bin/node

How would I check my profile files and whether or not they have nvm lines in them? Forgive my ignorance on this.

OK so - first step, let's upgrade your nvm to the latest, v0.26.1 - run the curl command on http://nvm.sh (the project readme).
Second - if you're using bash, check ~/.bashrc or ~/.bash_profile to start - but the curl install command  should handle that for you.

I got this when trying to simply run it, but I think I may now have something of an idea of what is going on here. Let's see if my suspicions are confirmed:
wget -qO- https://raw.githubusercontent.com/creationix/nvm/v0.26.1/install.sh | bash
=> nvm is already installed in /home/vor0220/.nvm, trying to update using git
=>
=> Source string already in /home/vor0220/.bashrc
=> You currently have modules installed globally with `npm`. These will no
=> longer be linked to the active version of Node when you install a new node
=> with `nvm`; and they may (depending on how you construct your `$PATH`)
=> override the binaries of modules installed with `nvm`:

/usr/local/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
=> If you wish to uninstall them at a later point (or re-install them under your
=> `nvm` Nodes), you can remove them from the system Node as follows:

     $ nvm use system
     $ npm uninstall -g a_module

=> You can now start using nvm

here is the bashrc contents with NVM:
export NVM_DIR="/home/vor0220/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm

so it appears I can't figure out how to upgrade this since I didn't install it from git IIRC...How should I go about this?

@VoR0220 how did you install it? From homebrew?

IIRC I used npm to install it.

for some reason the script is not updating the nvm version.

Ah I spoke too soon. Now I know why people say to not install in root...because I had to run sudo on it to get it to update. I will now attempt to set the default.

aha - do NOT install nvm from npm. That's a broken and incorrect version. Also, do not use homebrew.
nvm must only be installed from the curl command in the README.
Please sudo npm uninstall -g nvm, run the nvm curl installation command, and then don't ever use sudo again with node stuff :-)

did that....still running into the same problems :/

ok - if you open a new shell, what does nvm alias default print out? Do you have a .nvmrc file anywhere?

vor0220@vor0220-ubuntu:~$ nvm alias default
default -> v0.12.2

it would appear I do not have a .nvmrc file

and still, nvm current prints out "system"?
The only thing I can think of is that you have code in ~/.bashrc that is modifying PATH, after nvm is sourced. Do you?

I have commands that are exporting Path as well as a Go path, but nothing modifying it. I'll paste the contents right here:
export NVM_DIR="/home/vor0220/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh"  # This loads nvm
export GOPATH=/home/vor0220/go
export PATH=/home/vor0220/newenv/bin:/home/vor0220/newenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/home/vor0220/bin:/home/vor0220/newenv/bin:/home/vor0220/go/bin:/usr/local/go/bin

Well, there you go - you're erasing your PATH. You want something like export PATH="$PATH:your/go/paths" to preserve the contents of the PATH variable rather than blowing them away.

alright. Thanks for the help.

np!

wait a second...I'm not sure I'm totally following. Isn't GOPATH a separate PATH that shouldn't affect the second PATH? I'm not sure I'm following what is happening here, and I want to make sure I do to correct it.

I'm not talking about GOPATH. I'm talking about export PATH=something. Generally speaking, you never want to just set PATH to something, unless it includes the pre-existing $PATH contents. So, you don't want to just set $PATH to something - you want to append something to $PATH.

Awesome. Thanks again for the clarification and keep up the energy. I wish more project contributors were as helpful as you :)

np, happy to help :-) Glad we both figured it out, and that it turned out not to be nvm's fault :-D

Awesome.
Changed my path to:
export PATH="$PATH:/home/vor0220/newenv/bin:/home/vor0220/newenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/home/vor0220/bin:/home/vor0220/newenv/bin:/home/vor0220/go/bin:/usr/local/go/bin"

and now when I run nvm ls:
vor0220@vor0220-ubuntu:~$ nvm ls
    iojs-v2.3.1
->      v0.12.2
        v0.12.4
        v0.12.7
         system
default -> v0.12.2
node -> stable (-> v0.12.7) (default)
stable -> 0.12 (-> v0.12.7) (default)
iojs -> iojs-v2.3 (-> iojs-v2.3.1) (default)

Hope this helps someone down the road.


community extension site / reference

I would like to know if there is somewhere a list of commonmark-java extensions or if you would mind to publish one in your README ?
I would like to reference one of the extensions I created a while ago : commonmark-ext-notifications which allows, like admonitions in Asciidoc, to annotate paragraphs as INFO, SUCCESS, WARNING, ERROR via an easy to remember syntax:
! This is an info message.
!v This is a success message.
!! Consider this a warning.
!x This is an error message.

which can then be rendered as

Hey! Cool :).
Yeah, I don't mind adding some references to the readme. Let's add another section ### Third-party extensions before See also. Mind raising a PR for that?

Sure, I will.


PrimaryScrollController attached to multiple scroll views in TabBarView's keep-alive tabs

Recently, scroll views will auto generate a scrollbar on desktop. (?) In TabBarView, if we set every tab(with scrollable child like listview) keep-alive, then the auto generated scrollbar controller will attached to every tab. Then it will raise ScrollController attached to multiple scroll views. if we drag scrollbar after switching tab once.
Steps to Reproduce:
Run example code on desktop(windows/macOS):
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage>
    with SingleTickerProviderStateMixin {
  late TabController _tabController;

  @override
  void initState() {
    super.initState();
    _tabController = TabController(length: 2, vsync: this);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
        bottom: TabBar(
          tabs: [Tab(text: 'TAB A'), Tab(text: 'TAB B')],
          controller: _tabController,
        ),
      ),
      body: TabBarView(
        controller: _tabController,
        children: [
          KeepAliveBuilder(builder: (ctx) => ChildPage(name: 'Tab A')),
          KeepAliveBuilder(builder: (ctx) => ChildPage(name: 'Tab B')),
        ],
      ),
    );
  }
}

class ChildPage extends StatelessWidget {
  final String name;

  const ChildPage({Key? key, required this.name}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return ListView(
      children: List.generate(
        20,
        (index) => ListTile(
          title: Text('$name: $index'),
        ),
      ),
    );
  }
}

class KeepAliveBuilder extends StatefulWidget {
  final WidgetBuilder builder;
  final bool wantKeepAlive;

  const KeepAliveBuilder(
      {Key? key, required this.builder, this.wantKeepAlive = true})
      : super(key: key);

  @override
  _KeepAliveBuilderState createState() => _KeepAliveBuilderState(wantKeepAlive);
}

class _KeepAliveBuilderState extends State<KeepAliveBuilder>
    with AutomaticKeepAliveClientMixin {
  bool _wantKeepAlive;

  _KeepAliveBuilderState(this._wantKeepAlive);

  @override
  Widget build(BuildContext context) {
    super.build(context);
    return widget.builder(context);
  }

  @override
  bool get wantKeepAlive => _wantKeepAlive;
}

Expected results:
Actual results:

Logs
error
======== Exception caught by gesture ===============================================================
The following assertion was thrown while handling a gesture:
ScrollController attached to multiple scroll views.
'package:flutter/src/widgets/scroll_controller.dart':
Failed assertion: line 109 pos 12: '_positions.length == 1'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

When the exception was thrown, this was the stack:
#2      ScrollController.position (package:flutter/src/widgets/scroll_controller.dart:109:12)
#3      RawScrollbarState.getScrollbarDirection (package:flutter/src/widgets/scrollbar.dart:1080:34)
#4      RawScrollbarState.handleThumbPressStart (package:flutter/src/widgets/scrollbar.dart:1103:29)
#5      _MaterialScrollbarState.handleThumbPressStart (package:flutter/src/material/scrollbar.dart:388:11)
#6      RawScrollbarState._gestures.<anonymous closure>.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1239:74)
...
Handler: "onLongPressStart"
Recognizer: _ThumbPressGestureRecognizer#0c12c
  debugOwner: _MaterialScrollbarState#6eb61(tickers: tracking 2 tickers)
  state: possible
====================================================================================================

flutter doctor -v
[✓] Flutter (Channel master, 2.2.0-11.0.pre.274, on macOS 11.2.2 20D80 darwin-x64, locale en-CN)
    • Flutter version 2.2.0-11.0.pre.274 at /Users/narumi/Library/flutter-master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d1f73a1065 (33 hours ago), 2021-04-24 00:44:02 -0400
    • Engine revision eba9efd1c3
    • Dart version 2.14.0 (build 2.14.0-37.0.dev)
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/narumi/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Users/narumi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Users/narumi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/202.7094744/Android Studio 4.2
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/narumi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.55.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.2 20D80 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 90.0.4430.85

Hi @narumishi
Can you please provide the steps to reproduce this issue? I cannot reproduce this on latest master/stable channel.
Thank you

@pedromassango please use the example code and build on desktop(macOS/windows tested). It seems scrollbar is not auto implemented on mobile.

Hi @narumishi
I just build on MacOS desktop but I was unable to reproduce this issue (I got no crash). Tested on master and stable channel.

@pedromassangocode
So strange, I have tested on both macOS and windows, debug/release mode, master channel. And they all raise such error.
Please check the detail steps:

Run with example code
There are two tabs, every tab is a ListView with an auto-generated scrollbar which will be visible when scrolling.
Don't switch tab.  In Tab A, drag scrollbar, it works.
Switch to Tab B, drag scrollbar, error raised.
Back to Tab A, drag again, error raised.

Thanks.
I was able to reproduce the issue following the steps in the comment above.

logs
======== Exception caught by gesture ===============================================================
The following assertion was thrown while handling a gesture:
ScrollController attached to multiple scroll views.
'package:flutter/src/widgets/scroll_controller.dart':
Failed assertion: line 109 pos 12: '_positions.length == 1'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

When the exception was thrown, this was the stack:
#2      ScrollController.position (package:flutter/src/widgets/scroll_controller.dart:109:12)
#3      RawScrollbarState.getScrollbarDirection (package:flutter/src/widgets/scrollbar.dart:1080:34)
#4      RawScrollbarState.handleThumbPressStart (package:flutter/src/widgets/scrollbar.dart:1103:29)
#5      _MaterialScrollbarState.handleThumbPressStart (package:flutter/src/material/scrollbar.dart:388:11)
#6      RawScrollbarState._gestures.<anonymous closure>.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1239:74)
...
Handler: "onLongPressStart"
Recognizer: _ThumbPressGestureRecognizer#9cbee
  debugOwner: _MaterialScrollbarState#8e7ff(tickers: tracking 2 tickers)
  state: possible
====================================================================================================

flutter doctor -v
[✓] Flutter (Channel master, 2.3.0-1.0.pre.101, on macOS 11.2.3 20D91 darwin-x64, locale en-AO)
    • Flutter version 2.3.0-1.0.pre.101 at /Users/pedromassango/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff2dde2c7f (68 minutes ago), 2021-04-26 22:35:45 -0700
    • Engine revision afd9c397a5
    • Dart version 2.14.0 (build 2.14.0-43.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/pedromassango/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 55.1.5
    • Dart plugin version 211.7179

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 90.0.4430.85

! Doctor found issues in 1 category.

/cc @Piinks

Hey there @narumishi thanks for filing this issue!
I don't think this has to do with the fallback ScrollController (the automatically generated one) in Scrollable, that would create a unique ScrollController for each Scrollable.
It looks like the issue may be that all three of your Scrollables are using the PrimaryScrollController, since the ListViews are in the vertical axis and do not have an assigned ScrollController.
Looking to confirm this now..

Confirmed, this is not caused by the auto-generated scroll controller, but it is the PrimaryScrollController that all three tabs are sharing. The scrollbar that has been applied on desktop exposes this configuration.
This would probably cause issues elsewhere, as all three ListViews are being kept alive and are attached to the ScrollController at the same time. There are a couple of ways to remedy this:

provide a unique scroll controller to each ListView
Or, if you would rather just remove the scrollbar that exposes this, see this guide for multiple options to configure a scroll behavior

@Piinks Thanks for your analysis.
I tested something more. Just place two ListView inside one Column, without keep-alive code. It will also raise the same error.
class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text(widget.title)),
      body: Column(
        children: [
          Expanded(
            child: ListView(
              children: List.generate(20, (index) => Text('UP: $index')),
            ),
          ),
          Expanded(
            child: ListView(
              children: List.generate(20, (index) => Text('DOWN: $index')),
            ),
          ),
        ],
      ),
    );
  }
}

Maybe keep-alive is not a frequently used case, but multiple scrollable widgets inside one simple page should always work with default scroll behavior.
Although I know that assigning controller to every scrollable widget is always working, I think it's really annoying if I just want the default scroll behavior works, don't need any specific controls.

I tested something more. Just place two ListView inside one Column, without keep-alive code. It will also raise the same error.

Hi @narumishi this is an example of the same issue - more than one scrollable is using the same scroll controller at the same time, here it is the PrimaryScrollController.

Maybe keep-alive is not a frequently used case, but multiple scrollable widgets inside one simple page should always work with default scroll behavior.

I am not sure this is an issue with the scroll behavior though, as it cannot determine that there are multiple scroll views on the same page, or that they are currently all sharing the same scroll controller. This may be something we just need to document in more detail regarding PrimaryScrollControllers and how they are automatically assigned in the vertical axis. If none of these were using the PrimaryScrollController, then the auto-generated scroll controller in Scrollable would eliminate this issue all together since each would be unique.

Although I know that assigning controller to every scrollable widget is always working, I think it's really annoying if I just want the default scroll behavior works, don't need any specific controls.

Fortunately not all of the scroll views need a unique scroll controller in your sample here, one can still use the PrimaryScrollController and the other can use a provided one.

Got it. Thanks for your explanation. That's a good idea to specific the detailed error message.

@Piinks Hello, recently, I encountered the similar issue again. Even when I didn't make tabs keep-alive, The provided ScrollController is currently attached to more than one ScrollPosition. is still raised when switch between tabs and scroll at same time.
It is recommended to reproduce the problem on desktop(macOS for me). Scroll vertical and horizontal the same time, Try a few more times.
It seems when you switch tab while vertical scrolling is going on, the PrimaryScrollController  is assigned to new tab' listview before the previous tab dispose.

Error

======== Exception caught by animation library =====================================================
The following assertion was thrown while notifying status listeners for AnimationController:
The provided ScrollController is currently attached to more than one ScrollPosition.

The Scrollbar requires a single ScrollPosition in order to be painted.

When the scrollbar is interactive, the associated Scrollable widgets must have unique ScrollControllers. The provided ScrollController must be unique to a Scrollable widget.

When the exception was thrown, this was the stack:
#0      RawScrollbarState._debugCheckHasValidScrollPosition.<anonymous closure> (package:flutter/src/widgets/scrollbar.dart:1315:9)
#1      RawScrollbarState._debugCheckHasValidScrollPosition (package:flutter/src/widgets/scrollbar.dart:1339:6)
#2      RawScrollbarState._validateInteractions (package:flutter/src/widgets/scrollbar.dart:1257:14)
#3      AnimationLocalStatusListenersMixin.notifyStatusListeners (package:flutter/src/animation/listener_helpers.dart:233:19)
#4      AnimationController._checkStatusChanged (package:flutter/src/animation/animation_controller.dart:814:7)
#5      AnimationController._startSimulation (package:flutter/src/animation/animation_controller.dart:748:5)
#6      AnimationController._animateToInternal (package:flutter/src/animation/animation_controller.dart:611:12)
#7      AnimationController.forward (package:flutter/src/animation/animation_controller.dart:462:12)
#8      RawScrollbarState._handleScrollNotification (package:flutter/src/widgets/scrollbar.dart:1567:37)
#9      NotificationListener._dispatch (package:flutter/src/widgets/notification_listener.dart:206:42)
#10     Notification.visitAncestor (package:flutter/src/widgets/notification_listener.dart:122:20)
#11     ViewportNotificationMixin.visitAncestor (package:flutter/src/widgets/scroll_notification.dart:31:18)
#12     Element.visitAncestorElements (package:flutter/src/widgets/framework.dart:4133:39)
#13     Notification.dispatch (package:flutter/src/widgets/notification_listener.dart:138:13)
#14     ScrollActivity.dispatchScrollUpdateNotification (package:flutter/src/widgets/scroll_activity.dart:94:92)
#15     ScrollPosition.didUpdateScrollPositionBy (package:flutter/src/widgets/scroll_position.dart:899:15)
#16     ScrollPositionWithSingleContext.pointerScroll (package:flutter/src/widgets/scroll_position_with_single_context.dart:221:7)
#17     ScrollableState._handlePointerScroll (package:flutter/src/widgets/scrollable.dart:718:16)
#18     PointerSignalResolver.resolve (package:flutter/src/gestures/pointer_signal_resolver.dart:199:32)
#19     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:446:29)
#20     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:420:22)
#21     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:278:11)
#22     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
#23     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:338:5)
#24     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
#25     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
#29     _invoke1 (dart:ui/hooks.dart:185:10)
#30     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:293:7)
#31     _dispatchPointerDataPacket (dart:ui/hooks.dart:98:31)
(elided 3 frames from dart:async)
The AnimationController notifying status listeners was: AnimationController#d7968(▶ 0.000)
====================================================================================================

example code
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(home: HomePage());
  }
}

class HomePage extends StatefulWidget {
  const HomePage({Key? key}) : super(key: key);

  @override
  _HomePageState createState() => _HomePageState();
}

class _HomePageState extends State<HomePage>
    with SingleTickerProviderStateMixin {
  final int tabNum = 3;
  late TabController _tabController;

  @override
  void initState() {
    super.initState();
    _tabController = TabController(length: tabNum, vsync: this);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Test'),
        bottom: TabBar(
          controller: _tabController,
          tabs: List.generate(tabNum, (index) => Tab(text: 'Tab $index')),
        ),
      ),
      body: TabBarView(
        controller: _tabController,
        children: List.generate(tabNum, (index) => OneTab(index: index)),
      ),
    );
  }
}

class OneTab extends StatefulWidget {
  final int index;

  const OneTab({Key? key, this.index = 0}) : super(key: key);

  @override
  _OneTabState createState() => _OneTabState();
}

class _OneTabState extends State<OneTab> {
  @override
  Widget build(BuildContext context) {
    return ListView(
      children: List.generate(
        50,
        (index) => ListTile(
          title: Text('Tab ${widget.index} Item $index'),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel beta, 2.5.0-5.1.pre, on macOS 11.3.1 20E241 darwin-x64, locale en-CN)
    • Flutter version 2.5.0-5.1.pre at /Users/narumi/Library/flutter-stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 65cf7de5f4 (3 days ago), 2021-08-11 15:48:01 -0700
    • Engine revision fbeb7e22bd
    • Dart version 2.14.0 (build 2.14.0-377.4.beta)
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/narumi/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Users/narumi/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/narumi/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/narumi/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.59.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[✓] Connected device (3 available)
    • Narumi's Phone (mobile) • 58677d0fc8984330404af9009f82e7795fc678d1 • ios            • iOS 14.1 18A8395
    • macOS (desktop)         • macos                                    • darwin-x64     • macOS 11.3.1 20E241
      darwin-x64
    • Chrome (web)            • chrome                                   • web-javascript • Google Chrome 92.0.4515.131

! Doctor found issues in 1 category.

I encounter this issue too

@pedromassangocode So strange, I have tested on both macOS and windows, debug/release mode, master channel. And they all raise such error.
Please check the detail steps:

Run with example code
There are two tabs, every tab is a ListView with an auto-generated scrollbar which will be visible when scrolling.
Don't switch tab.  In Tab A, drag scrollbar, it works.
Switch to Tab B, drag scrollbar, error raised.
Back to Tab A, drag again, error raised.

I have same issue in my flutter web exactly like this. It's TabBarView containing 3 SingleChildScrollViews which each one have their own scroll controller.

I face this issue with copying your example code in your doc.
https://github.com/flutter/flutter/assets/59549741/ff99a4b7-0ace-437e-8398-c0ebf48b6e5d
Is it dangerous?

This issue has been here for a long time and break other things.
@Piinks Is this related to https://github.com/flutter/flutter/issues/40740 ?

Commenting to keep this issue alive. Also running into the same issue using Tab Views with lists that scroll vertically, while the tab is allowed to scroll horizontally

same issue here

Per our wiki on issue hygiene:

Do not add "me too" or "same" or "is there an update" comments to issues or PRs

Continued posts like this are not constructive and will result in restrictions to make further comments in this repository.


docs(guide/decorators): add decorator guide

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs update.
What is the current behavior? (You can also link to an open issue here)
There currently is no guide for decorators.
What is the new behavior (if this is a feature change)?
Add a guide document for decorators
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

explain decorators and how they are implemented in angular
explain how different types of services can be selected
explain $delegate objects and how they differ between services
warn of the risks/caveats of $delegate modification
note the exposure of decorator through the module api
show an example of decorating a core service
show an example of decorating a core directive
show an example of decorating a core filter

closes #12163

Just a note: There could possibly be a content conflict through untriaged PR #14348. If merged first, 14348 could resolve the order of operations caveat with module.decorator and the relevant content (see diff L71:117-L84) should be removed prior to merging this PR. Otherwise, 14348 will need to include a change to the Decorator Guide or it will need to be handled in a subsequent PR.

I left a few comments. Additonally, I think it should be mentioned that one can:

Monkey-patch $delegate.
Augment $delegate with extra stuff.
Overwrite $delegate altogether and provide their own implementation.

(Currently only (1) is shown.)

Thanks for taking a look @gkalpak. I am making adjustments as we speak. In regards to your should-be-mentioned list, how do you define monkey-patching? I am only asking because I experience different definitions in the community. I assume you mean to replace a property/method on the $delegate without replacing the entire $delegate.
All of that being said: I am going to make some changes noting different approaches you have mentioned above as well as changes in response to your comments. I will wait to push them, until you have had time to respond where needed so I don't overwrite the diff. Thanks, again.

@sjbarker When you have made the changes, just push them in an extra commit to the branch. This makes it easier to review. And also write something in this issue, so that we get notified. Thanks!

@Narretz Okay, will do! I will have it updated here in the next few hours.

@Narretz Sorry for the delay. I upgraded node and didn't realize there was a known issue with the node-check-modules script. I ended up overwriting the original commit attempting to solve that issue, so I will comment the diff where necessary.

@Narretz @gkalpak Just a ping to let you know this is ready for another look.

After "What are decorators", there should be a section called "How to use decorators" that says that there are two ways to register decorators in Angular, and both give you access to the so-called $delegate which is the instantiated service before it is passed to the dependant / service that required it. Then there should be $provide.decorator and module.decorator as subheadings of this section.

Good stuff @sjbarker. Left a few minor comments.

Thanks @Narretz. I just pushed the updates. I went ahead and squashed the extra commit since it outdated the diff anyways.

Thanks again @sjbarker !

Glad to help @Narretz!


[EUWE] Fixing the bulk operations for associated entities

Backport of https://github.com/ManageIQ/manageiq/pull/12655 because there was a conflict (https://github.com/ManageIQ/manageiq/pull/12655#issuecomment-271387608)
it fixes https://bugzilla.redhat.com/show_bug.cgi?id=1404741
@miq-bot add_label euwe/yes, providers/hawkular, bug

@Jiri-Kremser this PR fixes https://bugzilla.redhat.com/show_bug.cgi?id=1404741 (Delete Datasource on datasources list isn't working on Euwe), can you please add the BZ link to the top comment (and confirm the fix if you can). Thanks!

@martinpovolny please approve if this is good to go.

Sorry for the delay. @simaishi : Yes, this is good to go.

@Jiri-Kremser - is this good to be merged?
@simaishi - was the euwe/yes label removed because this PR was made against the EUWE branch and therefore not necessary?

@bronaghs correct, Euwe specific PR doesn't need the label. Euwe branch is closed for merge right now, will merge when the window opens.

Ah, thanks for the clarification @simaishi


Migrated CustomToolbarSelect Component from MUIv4 to MUIv5 and Sistent

Notes for Reviewers

This PR fixes #12219

For ui changes: Please add a screenshot showcasing the UI before/after changes.

Before(Light Mode)

Before(Dark Mode)

After(Light Mode)

After(Dark Mode)

Signed commits

[x] Yes, I signed my commits.

@Divyansh200102 Thank you for your contributions! Let's discuss this during the Meshery Development call today at 9:00 AM CT | 7:30 PM IST. Adding it as an agenda item to the meeting minutes || meeting link.

@vishalvivekm hi for some personal reason i wont be able to attend the meeting . Can i get the reviews from the maintainers directly here?

@vishalvivekm because i am travelling at the moment i am not sure if i can do it today i tried joing but it just kept disconnecting. Can do the review in the next upcoming meeting?

yes

@nebula-aac there's a second PR from @Divyansh200102 up for review - https://github.com/meshery/meshery/pull/12559

@nebula-aac there's a second PR from @Divyansh200102 up for review - https://github.com/meshery/meshery/pull/12559

@leecalcote apologies but I won't be able to  join the meeting and present a demo of the components tomorrow as there are connectivity issues where I currently am. I will be back by next week and then I will present the demo of both the migrated components if necessary.For now I will attach the screenshots of the migrated component for reference.

@nebula-aac i have resolved the conflicts please see if it's good to merge

@vishalvivekm the changes are approved in this pr can you please merge it.

@leecalcote this pr is approved can you please merge it?


Is there a way one could send ONE message to MANY numbers?

I have been researching ways and means of having one message being sent to all users in a given list. The problem is, though, that when using a list with numbers and looping through them, it immediately fires up all three instances and sends message to only one of them. When adding "sleep", it only fires up one instance and then says "sucessful" even though it only sent a message to one out of two people.
# importing the module
import pywhatkit
import datetime
import time

nums = ["+1234","+5678"]

try:

        for c in nums:
                pywhatkit.sendwhatmsg(c, "Testing Automatic Whatsapp",xHour, yMinute, 20)
                print("Successfully Sent!")
                time.sleep(25)

except:
  print("An Unexpected Error!")

Therefore is there a way at all to send a message to multiple people?

Hey Joel, you can use this method instead, sendwhatsmsg_to_all, it accepts a list of the phones
def sendwhatsmsg_to_all(
phone_nos: List[str], etc...


Handle shutdown and shutdown+poweroff commands

From the controller we handle a shutdown (of all the app instances) and
from the local profile server we handle that and also a
shutdown+poweroff of the device.
In addition we defer the shutdown of app instances with a Local Profile Server for one minute.
There was some dead code in tpmmgr which was looking at the DeviceReboot in NodeAgentStatus since we now have DeviceReboot, DeviceShutdown, and DevicePoweroff it made sense to remove the dead code.
Finally removing an XXX comment in the protobuf file and fixing a typo in the markdown file.

It seems we have the problem inside https://github.com/lf-edge/eve/blob/9e208c7a4d155aa57c6fd5e4bc74bdd4e851b6b1/pkg/gpt-tools/files/zboot#L141-L145 This function call itself, so we should rename it.
Also please update this line https://github.com/lf-edge/eve/blob/9e208c7a4d155aa57c6fd5e4bc74bdd4e851b6b1/pkg/pillar/Dockerfile#L63 to use updated zboot from gpt-tools with poweroff command implemented and fixed.

Shutdown operation workflow is not clear to me. It is not fire reboot/poweroff and EVE will stay in online mode with all applications deactivated and will not handle new configuration from the controller. How we can recover from this state? Edge-node should go through reboot manually?

The semantics of shutdown is (gracefully) shut down all applications as stated in the documetation updated by this PR. I think I put words in the documentation that one can recover using a reboot command from the controller, but need to test this. If you don't find it in the documentation updates I'll add it.
The shutdown+poweroff command (only available from LPS) will in addition power off EVE-OS.

LGTM We now have several Go routines editing ZedagentStatus so there is a risk for some race conditions, but right now I do not see any. As a safer option we might consider to put all these LPS API handlers under one task (with multiple timers/channels).

Do you have a pointer to where we modify zedagent status? If not I'll go find it.


undefined error leads to hanging git

gitautosync error with a red status bar, "Error: undefined," and left behind a git process in 'D' state.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.
jovyan@jupyter-user1:~/materials-fa17$ ps aux | grep git
jovyan       35  0.0  0.0  12960   984 pts/0    S+   21:09   0:00 grep git
jovyan@jupyter-user1:~/materials-fa17$ mv .git/index.lock .git/index.lock-renamed
jovyan@jupyter-user1:~/materials-fa17$ git commit -m WIP

After the WIP commit succeeded, invoking git-sync on /user/user1 succeeded.
What was the undefined error? There was no error in the user server log until I killed the 'D' git process, after which subprocess just logged that the git commit command died with SIGKILL.
Could git operations other than lock acquisition be timing out? It did take about 10-15 minutes for the WIP commit to complete.

This is a good point - Do we set timeouts on our git calls? If not we probably should :)

nbgitpuller has been rewritten so many times since this :) I'm going to close this issue, we can re-open if needed.


[Event Parameter Request] Codeunit 333 Req. Wksh.-Make Order - procedure Set - event OnAfterSet

We would like the parameter/-s OrderDateReq, ReceiveDateReq, PostingDateReq, PurchOrderHeader added to the following event.
Codeunit 333 Req. Wksh.-Make Order - procedure Set - event OnAfterSet
Original:
Codeunit 333 Req. Wksh.-Make Order - procedure Set - event OnAfterSet

OnAfterSet(PurchOrderHeader, SuppressCommit, EndOrderDate, PrintPurchOrders);

[IntegrationEvent(false, false)]
local procedure OnAfterSet(NewPurchOrderHeader: Record "Purchase Header"; CommitIsSuppressed: Boolean; EndingOrderDate: Date; PrintPurchOrder: Boolean)
begin
end;

Change:
OnAfterSet(PurchOrderHeader, SuppressCommit, EndOrderDate, PrintPurchOrders, OrderDateReq, ReceiveDateReq, PostingDateReq, PurchOrderHeader); // <-- ADDED OrderDateReq, ReceiveDateReq, PostingDateReq, PurchOrderHeader

[IntegrationEvent(false, false)]
local procedure OnAfterSet(NewPurchOrderHeader: Record "Purchase Header"; CommitIsSuppressed: Boolean; EndingOrderDate: Date; PrintPurchOrder: Boolean; var OrderDateReq: Date; ReceiveDateReq: Date; var PostingDateReq: Date; var PurchOrderHeader: Record "Purchase Header") // <-- ADDED
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52136.


Support Cloud Load Balancing API

I'd like to manage Google Cloud Load Balancers using C#. Are there any plans to support the Cloud Load Balancing API?

That's part of the Compute API.
There's an alpha version of our "modern" generated libraries, Google.Cloud.Compute.V1.
Alternatively, there's the more mature Google.Apis.Compute.v1 library produced by the older generator.

Thanks! I kept searching for some load balancer entity, but now realize there is no such thing.
For future people like me: At the API level, there is no Load Balancer, only the components that make it up


Add @geobde/use-click-away hook.

React hook when you want a callback invoked when a DOM element was not clicked.

Once conflicts are resolved this can be merged 😄

Thanks!


Proxy does not handle empty request bodies correctly (except for HEAD and GET?)

This is a follow-up to #402 to track the real solution to the problem that the proxy doesn't handle requests without a body correctly. The PR for #402 only partially resolves the issue. Really we should fix the real issue. I very vaguely understand that to require some changes to the Hyper API or some other more drastic solution.
In particular, DELETE will almost never have a request body, so many REST APis would be affected by this.
/cc @olix0r @seanmonstar

The bug here is a combination of a few things:

hyper's Request holds an optional body. If not set, then no body is assumed. Otherwise, it assumes a body.
The body is generic over a futures Stream, so there is no way to ask anything about it before the headers are written.
If the body is set, and there is no content-length header, it inserts Transfer-Encoding chunked, since the length cannot be calculated.
The proxy maps bodies from both hyper and h2, with the result that client requests always have the body set, even if internally the stream will be empty.

In the long term, hyper is going to change it's body type to a trait with more methods that it can ask about if the body is empty.
In the short term, I think a small addition in hyper to expose that server bodies are empty, and then a modification in the proxy to check that, can allow optionally setting the body in hyper client requests.


Support highlighting fenced code blocks with the "console" language identifier?

Search Terms

console
highlighting
markdown

Problem
Github highlights fenced code blocks with the console language identifier:
```console
$ some-command
command output 1
command output 2
command output 3
etc
```

Formatted:
$ some-command
command output 1
command output 2
command output 3
etc

Typedoc prints a warning when it finds console blocks:
→ warning Unsupported highlight language "console" will not be highlighted. Run typedoc --help for a list of supported languages.

Suggested Solution
It would be neat if Typedoc could support the same set of languages as GitHub.

TypeDoc is dependent on Shiki for which languages are supported, so this really needs to be a request there

This was fixed in Shiki and it's been released.  This module depends on 0.12.x but the fix is in 0.13.x.  0.14.x is also available - can the dep be updated to pull the fix in?


FN: encrypt-data-using-camellia - 112f9f0e8d349858a80dd8c14190e620

Summary
encrypt-data-using-camellia doesn't match on 112f9f0e8d349858a80dd8c14190e620
Examples
112f9f0e8d349858a80dd8c14190e620
Reference:

https://krabsonsecurity.com/2020/08/22/bitrat-the-latest-in-copy-pasted-malware-by-incompetent-developers/

From a quick glance at the binary, the s-box from https://github.com/calccrypto/Encryptions/blob/master/Encryptions/Camellia_Const.h#L35 can be found at address 0x008cb760 (offset 0x004ca160)
Possible improvements
I'm not familiar with Camellia or encrypt-data-using-camellia enough to understand specifically what the existing logic is looking for, but hopefully it's as simple as adding the s-box bytes (and maybe some of the sigma byte sequences as well) from above into the rule.

sbox reference in in function 0x4CE384

the camellia constants in the reference look like this (bring inline just for ease of access):

capa rule currently looks like this:

this first line of the capa logic (sp1110) looks similar to the s-box, though it seems to repeat bytes with interspersed data. I've double checked the referenced sample and indeed it does use this unusual format.

i think we can add a new bytes: ... term under the top level or that references the first 256 bytes of the s-box byte array. just like @recvfrom suggested. would you like to propose the updates the rule in a PR? if you do, please also add yourself into the authors list in the rule meta.
something like:
bytes: 0x70, 0x82, ... 0x80, 0x9e = Camellia_SBox

Thanks for doing the heavy-lifting to investigate this!
I'll update the rule and submit a PR. I'll also look into whether an update could be made to the section that looks for the sigmas


Implementation of LSTMCell inconsistent with the refered paper.

This is an extended issue of the existing issue #8469 .
In the comments, it said that the LSTMCell is implemented based on this paper.
In this paper, the several gates are computed like sum_i(args[i] * W[i]). Here, the args are list of input and m_state. More details can be found in the mentioned paper.
However, in the code the gates are computed as:
      # i = input_gate, j = new_input, f = forget_gate, o = output_gate
      lstm_matrix = _linear([inputs, m_prev], 4 * self._num_units, bias=True)
      i, j, f, o = array_ops.split(
          value=lstm_matrix, num_or_size_splits=4, axis=1)

Diving into the _linear function, showing that:
      res = math_ops.matmul(array_ops.concat(args, 1), weights)

The args are firstly concatenated, and then matmul with the weights. In this implementation, the parameter number of the weights is much larger than what said in the paper. This is totally different from what said in the paper.

@xiejw


Scenario with Data() don't work

What are you trying to achieve?
Run test with Data provider
What do you get instead?
the test does not start, when I remove Data(periods) it runs
test body

const periods = new DataTable([
    PredefinedDataRanges.week,
    PredefinedDataRanges.month,
   ]);

Data(periods).Scenario(
    '@WIP ',
    async ({ current, dpInvoicesPage, tmsSearchBy }) => {
        dpInvoicesPage.open();
        SearchBy.changeDateRange(current);
        await SearchBy.assertDateRange(current);
    },
);

Details
environment information:-
codeceptVersion:  "3.0.4"
nodeInfo:  12.1.0
osInfo:  macOS 10.15.7

my bad, wrong config

my bad, wrong config


Update Color.js

What does this PR do?
Fix a typo
Where should the reviewer start?
What testing has been done on this PR?
How should this be manually tested?
Any background context you want to provide?
What are the relevant issues?
Screenshots (if appropriate)

Thanks!


Improve experience of derivation of Codable types

What's in this pull request?

Introduce diagnostics that explain why derivation of Encodable/Decodable fail rather than just silently failing
Allow properties with default values to be omitted from CodingKeys enum and from encoding/decoding
If CodingKeys is a typealias, reach through it more consistently to get at the final target type
Add unit tests to confirm this new behavior for classes and structs

NOTE: Although this made the diff bigger, standalone variables named "type" have been renamed to "target" throughout, since the word "type" may be ambiguous and makes debugging impossible. "target" is unique.

This addresses SR-4920.

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a38f82bac517dd96d8b5971898d54a6ca6c56e83
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - a38f82bac517dd96d8b5971898d54a6ca6c56e83
Test requested by - @itaiferber

Missed committing a fix I'd intended to make. Let's try this again.

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a38f82bac517dd96d8b5971898d54a6ca6c56e83
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - a38f82bac517dd96d8b5971898d54a6ca6c56e83
Test requested by - @itaiferber

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 1fc49e98dc4e7ef5d8abd6a063f787f807eae6b3
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 1fc49e98dc4e7ef5d8abd6a063f787f807eae6b3
Test requested by - @itaiferber

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 68a5e6a121c0b9c50d1debb169e468b3896f3556
Test requested by - @itaiferber

@itaiferber Why can't implementation of Codable be automatically synthesized in a same-file extension? SE-0169 was accepted to support this kind of code organization.

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 68a5e6a121c0b9c50d1debb169e468b3896f3556
Test requested by - @itaiferber

@benrimmington You're right in that I think we should support cases like that. We can relax this restriction soon; it will just require some additional implementation that's not ready yet. Updating the diagnostic to reflect this.

I'm getting different errors locally than on the CI. CI is truth, so let's use that.

@swift-ci Please test Linux

@swift-ci Please test Linux

@swift-ci Please test OS X

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 68a5e6a121c0b9c50d1debb169e468b3896f3556
Test requested by - @itaiferber

@swift-ci Please test macOS

@DougGregor @jrose-apple While this is testing, if either of you have the time, do you mind reviewing this please? This will be going against swift-4.0-branch.

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 5eb58ef8d452a39ab61432575cc4ffca197fc89a
Test requested by - @itaiferber

Looks like this was affected by https://github.com/apple/swift-llbuild/pull/175. Let's try this again now that the fix should be in.

@swift-ci Please clean test macOS

@jrose-apple Thanks for reviewing!


Error running composer.phar

When manually installing from current master-branch, running bin/composer.phar results in the following error:
PHP Parse error:  syntax error, unexpected 'class' (T_CLASS), expecting identifier (T_STRING) or variable (T_VARIABLE) or '{' or '$' in /apps/openconext-engineblock/engineblock-dev/src/OpenConext/EngineBlockBundle/DependencyInjection/OpenConextEngineBlockExtension.php on line 89
Seems like Feature::class is not correct and should be a string.

Thanks for reporting this.
Which PHP version are you using?
::class is supported since PHP 5.5, EngineBlock 5.x requires PHP 5.6+.

Ah, crap, I think 5.4 is the default on CentOS 7. I will verify this tonight.

Yes. We use the remi repository to install 5.6 on Centos 7. https://github.com/OpenConext/OpenConext-deploy/blob/engineblock5-centos7/roles/php56fpm/tasks/main.yml

Okay, confirmed. Case closed.


decide what "basic info" we should show, and how

We don.t need to show obsolete names by default
(could we hide these by doing  "...."?)
Any other suggestions?

Is there some way that we can combine the "gene standard anme" "systematic" ID and header occurances? into the top space?
Any ideas?
(look what others do...)

Name description would be better at the bottom of the left hand list (there are often multiple)

Hmm complete aside, but I think we could delete
swo2,tws1 and wee2 (these are just allele names)

@mah11  any suggestions  for improvements here?

I don't actually have a big problem with the way it is now (except that I'll be glad when the product size info gets filled in). I'm not keen on trying to combine the gene name and ID entries.

OK,

[ ] I would still like to hide obsolete names by default  (with a .... ?)

[ ] I think name description needs to go to the left because they are usually long (and often multiple),

[ ] name description does not need to be present if there aren't any

[ ] maybe the font size for this section could be the same as the other sections? Its slightly larger

I would still like to hide obsolete names by default (with a .... ?)

Do you mean just have a "..." that is a link that shows the obsolete IDs?

I think name description needs to go to the left because they are usually long (and often multiple),
name description does not need to be present if there aren't any

That's fixed.  Here's an example: http://pombase2.bioinformatics.nz/gene/SPAC24B11.06c

maybe the font size for this section could be the same as the other sections? Its slightly larger

The text already looks quite small on my screen:

Its quite a bit bigger than the other data...
compare to MF here:

The sty1(SPAC24B11.06c) should stay the same (or could even be a little larger?)

Do you mean just have a "..." that is a link that shows the obsolete IDs?

yes.
Maybe we could do the same for multiple "name descriptions" too. These are not very important.
That  do instead of reducing font size...

Some pages look a bit strange eg:

maybe this would be ok if the 2nd column was much narrower than the first column?

Would this be better:

I still think the gene name centrally withough the systematic ID would be better. The only DB that puts the systematic ID up froont is SGD and it's a bit more meaningful for them.  The systematic ID is clearly below in the header.
It could even be
Gene standard name: Sws1
in the header o that hte first line below could be deleted

I still think the gene name centrally withough the systematic ID would be better.

OK, I'll do that.
I'll have go at fixing the whitespace but it's a bit fiddly with tables in HTML.  It's especially tricky to get it to look right when the page is resized.

I think I would rather see the product   at the top too...
--------------------------Gene standard name: Sws1
Product
Systematic ID
synonyms (only if present)
Location

If you have any other ideas.....

I've moved the gene name to be centred and moved the product to the top.  To fix the gaps in the table I've made the table centred.  Previously it was always as wide as the window.  On a small display it looks the same as the previous layout.  On a large display it looks odd but slightly better I think.
I've moved the other details around a bit.  It's now:
Product               Characterisation status
Systematic ID         Feature Type
Genomic location      Product size
Name description      Synonyms

I put Name description and Synonyms at the bottom because they are the two things that not every gene has.  It looks a bit odd to have empty cells in the table when they were missing.  What do you think?

Hmm, I think its better in some ways but now I see it
cdc2 (SPBC11B10.09)
gene standard name cdc2 probably  looked better....
How many font sizes is it bigger than the rest of the page?

I often don't have my browser full width (its easier to find things on the desktop with a small screen)...
this makes the  basic info a little messy...

Do others do this, or is it only me?
Anyway it made me wonder if the whole section would be better if it was just a single column , instead of 2 colums...at leas then each item could be restricted to a single line which would be unlikely to wrap??
thoughts?

I think that right now, the browser needs to be wider than default to prevent the line wrapping kicking in.....

I don't like the "Gene standard name:" in the header. It's boring and makes you read too much before the interesting thing, i.e. the gene name.
I also think it's kind of odd that the text in the top table is larger than in the rest of the page. Only the gene name at the top really needs to be big.

yeah, I though the same once I saw it.
So gene name large at top.
Include "gene standard name" cdc2 underneath
I'm still wondering if a single column format would be better than 2 column....

hmm, not sure I have an opinion, especially without seeing 1 vs. 2 columns side-by-side
I suspect I'll get used to either, as long as things stay in the same place once we decide and Kim deploys it.

So gene name large at top.
Include "gene standard name" cdc2 underneath

I've done those changes.

I think that looks better...anyone else?


Make LaceKV

as a hashmap with red-black tree buckets.
The keys can be 2 int_ptr_t, where one is number of bytes and the other is a pointer.

The hash table itself can be a size_t array whose values are indices an array of red-black tree nodes. Ideally the table would be a power of 2, but some people use a prime size in order to mitigate funneling. http://databasearchitects.blogspot.com/2020/01/all-hash-table-sizes-you-will-ever-need.html would be a good resource here, but I need to research hash functions a bit more.
The red-black tree node array reserves 0 as a sentinel. Each node has several size_t fields:

joint
split[2]
size: Top bit is set iff node is red. Second from top bit indicates that this node holds the key directly. In that case, the rest of the top byte holds the size and we have all but one of this field's bytes + the next field's to hold the key. Maybe include value too.
key (actually a uintptr_t)
value (actually a uintptr_t)


Swipe to remove the items from Shopping List

Please assign me this

Please Merge this

Few more things need to be done in this
Will push it soon


Use the new $PLATFORM_DOCUMENT_ROOT env var

needs all projects to be refreshed with platform-git 1.13.4-1


background image not loading

https://opencollective.com/nest/edit#images
front end saves the image but it's not displayed.
Thank you for taking the time to report an issue
The easier it is for us to reproduce it, the faster we can solve it.
So please try to be as complete as possible when filing your issue.

URL:
Logged in as: (just provide your username or the url of your profile)
Error message if any:
Expected result:
Browser:
Bonus point if you can add a screenshot :-)
Thank you and have a wonderful day/evening!
(and sorry for inconvenience! We'll do our best to solve this ASAP)

fixed. it was hard coded in collective.settings.


TUI customization via config file

After adding #89 one can define colors and similar in the config file.

Closed in #96


Add 'QueryJob.undeclared_query_parameters' property.

Uses #3801 as a base.
Cherry-picked / fixed up from PR #3721.

@tswast This version returns the undeclared query parameters as instances of one of the concrete parameter classes, as you requested.

Rebased after merging #3801 to the bigquery-b2 branch.  I will merge when the BQ tests are green (docs build on PRs seems to hang).


Newsletter box is not alligned

Issue Description
The newsletter box in footer is not aligned with the subscribe button which is not looking good
I want to fix it

Newsletterbox #167

Created a PR for the Issue #167


[FEATURE] Move CLI commands under argilla server

Is your feature request related to a problem? Please describe.
I think it's a bit counter-intuitive to have some server-related commands in the CLI under a different entry-point e.g. argilla login is login into an Argilla server, as well as argilla logout so those should be argilla server login and argilla server logout IMO.
Describe the solution you'd like
Here I list some CLI commands I would just move under server:

argilla login -> argilla server login
argilla info -> argilla server info
argilla logout -> argilla server logout

Eventually, we would also add a new command to replace argilla info to display local information such as the Python package version, but could be extended to any other arg we feel like can be useful.
And we should re-write the ISSUE_TEMPLATES to let users know that to run argilla server info (former argilla info) they need to argilla server login (former argilla login) first.

@alvarobartt to double check, this has not been handled yet right?

We chose not to tackle this issue anymore.