Different Video Calls in same channel?

Greetings!
I'm new in the WebRTC and Amazon KVS. I'm building a Proof of Concept video chat app to explore the service.
In the examples, it seems like in a channel there can only be one active "video call" between two users.
Is it possible to have many "seperated" video calls between 2 users in the same channel, or you have to create a new channel for every "call request" and delete it after? Or is there any other best practise for this case?

Hi @Schavras, channels are only being used for signaling. You can think of it as a cell tower. So, let say that we have 2 peers, these 2 peers basically just use the channel as a medium to communicate. But, what they're talking about is really up to them, which in this case is "video call". So, yes, it's technically possible, but I would recommend to simply create a new channel for each different "video call" and delete it after as you suggested.
Using different channel for each video call will make development easier. If we used the cell tower analogy again, but this time with 2 active calls (4 peers), we need to somehow demux or mux the data ourselves.

Hi @Schavras I have done what @lherman-cs suggested a couple of weeks ago and it works well in production for me (+/- 100 calls concurrently). You need to be careful around the 1000 channel cap unless you request for a larger quota.
I have a very slim REST API which manages calls. Server side I manage the creation and deletion of the calls. Client side I manage who is master and who is viewer.

@marcusbelcher NICE! We would love to see a sample or something? Let us know if you would like to contribute and we can work together

It seems that the question has been answered. I'll close this. Please feel free to reopen it if you need further assistance.

@MushMal Sadly proprietary code. Once I get some time off I hope to write some examples :)


Add LINE 6.0.2204

Add LINE messenger application.
Microsoft Reviewers: Open in CodeFlow

 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.:x: zhiyuan-lin sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Rerun pipeline please?


[Feature Request] Suggested way of automating this with CI jobs

Is there a suggested way of running this as a static part of the workflow with Travis-CI for example? I feel that a hard fail would probably be too error prone, but something that would easily note that there might be spelling issues in the state of repo.
I really like how https://coveralls.io API can report back to GitHub issues, but I do acknowledge that this would mean running an static instance outside of Travis to handle the reporting.
Sorry for kind of open ended issue and rambling.

I've written: https://github.com/checkstyle/checkstyle/blob/master/.ci/test-spelling-unknown-words.sh which interacts with Travis.

A GitHub Action would be very interesting. See pre-commit's GHA, for example.

I've actually started working on this... It will eventually be in this repository.
For the time being, you can see the Action here:
https://github.com/jsoref/spelling-action
And a consumer here:
https://github.com/jsoref/test-test
With a sample runs:
https://github.com/jsoref/test-test/actions/runs/37887629

@henryiii: this year I've released a number of alpha releases of check-spelling
As for a pre-commit hook, I'm actually getting close to being able to do that... Although It isn't a direct goal.
A pre-release version of the action supports nektos/act, so one could use that.
But something more lightweight could be implemented using the toolchain I'm using to clean up repositories before submitting PRs...
I think I'll probably close this once my action is closer to a 1.0 release, or at least when I stop calling it "alpha".


Reproducing results

Hi,
Thank you for open-sourcing your work! I have a question regarding reproduciblity.
I tried running the code on the Atari 100k benchmark and I did not manage to reproduce the results. Here are the results I am obtaining:

Obtained
Reported in paper

Boxing
0.5
78

Breakout
10
20

CrazyClimber
3188
71820

Krull
3414
6349

Pong
-20
19

Is there a step I’m missing here? I installed the dependencies specified in the requirements file and ran the training command specified in the README file. The results I am reporting come from the eval/final_score_mean chart.
Thanks!

Hi, I also have the same problem. I run Pong and the result is -21, are there any possible reasons? Thanks a lot!

Hi, where is the eval/final_score_mean chart?

Hi,
Thank you for open-sourcing your work! I have a question regarding reproduciblity.
I tried running the code on the Atari 100k benchmark and I did not manage to reproduce the results. Here are the results I am obtaining:
Obtained	Reported in paper
Boxing	0.5	78
Breakout	10	20
CrazyClimber	3188	71820
Krull	3414	6349
Pong	-20	19
Is there a step I’m missing here? I installed the dependencies specified in the requirements file and ran the training command specified in the README file. The results I am reporting come from the eval/final_score_mean chart.
Thanks!

Hi, where is the eval/final_score_mean chart?


Add swift_version to spec

Specifies the swift version that will be used for the pod regardless of the swift version the app target uses.

Note: This feature was added on CocoaPods 1.4.0 (http://blog.cocoapods.org/CocoaPods-1.4.0/)

@hcanzonetta Maybe we could also remove the s.pod_target_xcconfig in the spec file?

@onevcat good point. Done!


Accept string values for width

Steps to reproduce the behavior:
Set width to width='50%'
Expected behavior:
max-width of the element will be set to 2%, without showing an error that the width can only be a number.
Actual behavior:
max-width of the element is set to 2%, but is showing an error that the width can only be a number.

Hi @JapMul!
This seems like a reasonable feature, any chance you could provide a PR?
Thanks in advance,
Joss

@jossmac Allright, I think I can provide a PR somewhere this week!


Support for L515

does this work with realsense LiDAR sensor

I don't own the L515 camera so I can't test it. If the camera uses the same SDK as the D400's then it should work.


Could IgDiskCache get cached File?

IgDiskCache can only get a SnapshotInputStream now, it's easy and handy to get the cached File itself.

This would break encapsulation. What if we wanted to change the internal storage someday. Also, providing access to the underlying file would allow consumers to mutate or delete the file without the cache knowing.


OSError: [WinError 126] The specified module could not be found

when I am running the demo.py file am receiving the following error
Traceback (most recent call last):
  File "demo.py", line 2, in <module>
    import tools.find_mxnet
  File "C:\Users\wisdom\Desktop\mxnet-ssd-master\tools\find_mxnet.py", line 6, in <module>
    import mxnet as mx
  File "C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\site-packages\mxnet\__init__.py", line 24, in <module>
    from .context import Context, current_context, cpu, gpu, cpu_pinned
  File "C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\site-packages\mxnet\context.py", line 24, in <module>
    from .base import classproperty, with_metaclass, _MXClassPropertyMetaClass
  File "C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\site-packages\mxnet\base.py", line 213, in <module>
    _LIB = _load_lib()
  File "C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\site-packages\mxnet\base.py", line 204, in _load_lib
    lib = ctypes.CDLL(lib_path[0], ctypes.RTLD_LOCAL)
  File "C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\ctypes\__init__.py", line 353, in __init__
    self._handle = _dlopen(self._name, mode)

I tried to see where the error is triggered, as shown it ...\lib\ctypes_init_.py line 353 where
self._handle = _dlopen(self._name, mode)
so I debugged it and printed the content of 'self._name' which shows

kernel32
C:\Users\wisdom\Anaconda3\envs\gpu-test\lib\site-packages\mxnet\libmxnet.dll

at the moment of firing the error. I checked the existence of the requested .dll file and it is certainly exists but why it fires the error I didn't understand!

weirdly enough...I tried to run python prompt and just import ctypes as following
from _ctypes import LoadLibrary as _dlopen
_dlopen("what\\ever\\path\\libmxnet.dll")

and it stills throws the same error just like it doesn't see the file at all, or maybe there is some dependency missing!
on the other hand, whatever other .dll file I try to load it goes perfectly well!

Ok finally solved!
I got to know that such error could be thrown when the file dependencies are not satisfied or not found. So to check the dependencies you go first to Visual Studio Prompt and then navigate to the folder where libmxnet.dll exists and run the following command
dumpbin /dependents libmxnet.dll
and it will show you a list of required files.
What was missing in my case are some of nVidia toolkit files version 9.0 as libmxnet.dll asks for them and I was having toolkit version 10.0 instead!


CollectionContentList - fix by type counts being 0 when All not chosen

go to Collection detail, tab Contents
the "Showing: All(23) Role(9) Module(3) Action(1) Filter(8) Test(2)" line only shows correct counts when All is selected,
otherwise only the current type is counted, leaving other types at 0,
because filtering is done by both keyword and type, before counting and showing the list
this changes the code to first filter by keyword, then count, then filter by type, then show
Cc @jerabekjiri I noticed this while reviewing #1561 , but it's also present on master, can you review please? :)

before: (only the last 2 pics differ)

after:

summary[c.content_type] ||= 0;

Never seen this before, pretty cool :sunglasses:
Noticed weird recounting of content_type when switching between tabs, but this fixes it as well :+1:
LGTM

Thanks :)
Logical assignment operators - they're quite recent :)

Introduced in #507, the change only made it to 4.4, adding backport-4.4, and an issue... TODO


schema doesn't change from default

sorry if this is a really dumb/misguided question. if it is, I am hoping you could point me to a place where I can learn more.
I set up an engine like
engine = create_engine(     f"databricks+connector://token:{token}@{host}:443/{my_schema}",     connect_args={"http_path": http_path} )
but engine.table_names() and anything I try to do with that engine have the default schema tables.
I have to workaround by doing a schema translation, but that can't be the right way to do this, or is it?
engine1 = engine.execution_options(schema_translate_map={         "default": my_schema     })
edit: whoops put some private data in there

nope the workaround doesn't work. this is just broken and I have no idea how to fix it

Databricks connection urls are a bit different from other database urls but what we do here is for compatibility with sqlalchemy. Maybe the implementation is non-intuitive or confusing (or even wrong) but the value for the <database> is ignored by the connector, which only cares about the http_path and the token.
Generally if you want a specific schema (databricks calls them databases) you have to pass it to methods like this engine.table_names(schema="myschema")

We might actually want to add a USE SCHEMA <schema> call here on creating the connection. It might be that the sql connector doesn't do this, but the connector is not open source so I can't request a change there. I will take a deeper look.

Can you try doing something like
engine.execute("use myschema")

as a workaround?

I'll try that. In the meantime this is what I've done that's been working...in the MetaData object I can specify a schema and then use that to tell sqlalchemy to use the engine but override the schema.
I reopened this if that's alright. It keeps it on my radar.
That's how I'm understanding it. This aspect of sqlalchemy I'm learning just now. Before I've done creating an engine and then purely doing pd.read_sql(text(sql_query_text),engine), and I was using an odbc connection. With switching to databricks I'm trying to learn the sqlalchemy/pythonic way of doing things, so you're getting a double-whammy.
`

from sqlalchemy import MetaData
from sqlalchemy import Table
from sqlalchemy.orm import Session
from sqlalchemy.ext.automap import automap_base
generate metadata and be able to tell which schema to use
meta=MetaData(bind=engine,schema="analytics_layer")
below is just because none of the tables in this schema have primary keys I live in hell
rough and ready to get it to work. I think I can make a composite key
table_of_interest = Table("table_in_database", meta,Column('fake_primary_key_column', String, primary_key=True),extend_existing=True, autoload=True,autoload_with=engine)
I can create a base with the metadata object
Base=automap_base(metadata=meta)
reflecting the tables from that metdata object. specifying the reflection options
because this layer has hundreds of tables
Base.prepare(engine, reflect=True,, reflection_options = {"only": ["table_in_database"]})
Table_Of_Interest= Base.classes.table_in_database
Since Base is made with the schema I believe if I were to create tables on it then it would go in the right schema.
I am doing read only so I haven't really tested that. I believe creating tables in this layer
would get me promoted to customer.
`
This is an even quicker/dirtier method that I'm actually using since the process I'm doing right now doesn't require anything too complicated by way of the ORM
`
generate metadata and be able to tell which schema to use
meta=MetaData(bind=engine,schema="analytics_layer")
Table_Of_Interest = Table("table_in_database",meta,autoload=True,autoload_with=engine)
session = Session(bind=engine)
query = session.query(Table_Of_Interest).distinct().limit(10)
`

When I used pyhive underneath this would have worked fine because pyhive calls use schema on connection here: https://github.com/dropbox/PyHive/blob/d199a1bd55c656b5c28d0d62f2d3f2e6c9a82a54/pyhive/hive.py#L256
Apparently the databricks sql connector doesnt do this but since the code isn't public I can't request a change. I did ask if they have plans on making the code public.

Thanks for mirroring it. I didn't have any trouble seeing the code, I just wasn't able to PR an upstream change since it's not maintained publicly. I reached out to DB though and they said it will be made public "eventually" so we'll see.

Also from history:
v2.0.0b - February 8, 2022
- Initial beta release of V2. V2 is an internal re-write of large parts of the connector to use Databricks edge features. All public APIs from V1 remain.
- Added Unity Catalog support (pass catalog and / or  schema key word args to the .connect method to select initial schema and catalog)

so I assume that when 2.0 is released we can pass the catalog+schema and it will set those on creation.

Yes I believe so. I'm developing the same behaviour in an unrelated context using the v2 beta and it works like you describe.

just figured this before finding this thread.
doing USE f{schema} also
note if you're not using plain sql, but using sqlAlchemy helpers, you can also pass the schema name as a named argument

@susodapop @crflynn hi,
pip install python-sql-connector==2.0.0b1
in this version:

honor the value passed in for database/schema: 👍
honor the new 3L namespace used in UC 👍
allow to specify the catalog 👎

According to Databricks, they are planning to opensource the python-sql-connector lib

@susodapop @crflynn I don't konw if you noticed, but python-sql-connector 2.0.1 is out


Changing system to Terrinoth cause error in character sheet when reloading page

Hello!
First, thank you by the great system. I liked it much.
About the error.
I've changed the character sheet to Terrinoth template and swaped the skill list correctly.
But, if the page reload, the skill list show more skills, with blank names.

This is a duplicate of #941


Fix for issue #615

Avoid using string.split due to browser inconsistency with regex.

@PWKad the corresponding issue has been closed as a duplicate, but I'm not sure what the issue was on the duplicate. Your comment on the duplicate was "Nevermind, I see it now."
Should this PR be merged or closed? It seems like a reasonable change but it seems like the issue is already solved?

This commit - https://github.com/BlueSpire/Durandal/commit/f5049e812d1b99b98410fdb62ce3540863be433c
Reverted the breaking change that was merged in from before.  I found the issue with the router and re-implemented the original proper fix that closed out three issues.

Thanks @PWKad That was careless on my part. I appreciate you working it out. Any action needed from me?

@EisenbergEffect No, no worries, the PR he put in did look helpless enough on it's own but I didn't get a chance to review it fast enough.  Luckily the community raised a bunch of issues and were active before the release was released.

Issue #615 fixed by commit https://github.com/BlueSpire/Durandal/commit/f5049e812d1b99b98410fdb62ce3540863be433c
This PR is obsolete.


Wheel package for 3.1.1 not available on PyPI

Describe the bug
For 3.1.1 only a tar.gz source package is available on PyPI while previous releases included wheel packages.
Can we get the py3 wheel package uploaded to PyPI ?

me too tar.gz
> pip install -U django-simple-history
Requirement already satisfied: django-simple-history in c:\program\python3v\lib\site-packages (3.0.0)
Collecting django-simple-history
  Downloading django-simple-history-3.1.1.tar.gz (544 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Installing backend dependencies ... error
  ERROR: Command errored out with exit status 1:

This will require a 3.1.2 release.  The root cause can be traced to the release workflow in GitHub Actions:
https://github.com/jazzband/django-simple-history/commit/60beaf4d5157c2512da320e2ee436769a2fa061a#diff-87db21a973eed4fef5f32b267aa60fcee5cbdf03c67fafdc2a9b553bb0b15f34R31
That should be doing a "make dist" instead of what it is doing:
https://github.com/jazzband/django-simple-history/blob/master/Makefile#L29-L34
However the gpg signing may not work in GitHub Actions - not sure.

Closing, as 3.1.1 is (now) on PyPI.


Where are you using Auth Code?

Where (& how?) are u using AuthCode in this flow? It seems frontend app is obtaining accessToken from AD and passing that accessToken to API Management Gateway.

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

ID: 623cc870-6f9d-039e-b965-a964482a3e6d
Version Independent ID: 80cb6774-fc95-f551-aae7-5f8091bd3b17
Content: Protect SPA backend in Azure API Management with Active Directory B2C
Content Source: articles/api-management/howto-protect-backend-frontend-azure-ad-b2c.md
Service: api-management
GitHub Login: @WillEastbury
Microsoft Alias: wieastbu

Hello Sandeep,
Thank you for the feedback!
We're actually using PKCE + Auth code flow in the sample.
In direct answer to your question, the combination of selecting only the 'Single Page Application' endpoint type and / or using MSAL.js v2.0 or above forces the use of the PKCE and Auth Code flow in AAD B2C, we don't enable the web or implicit endpoints.
You can see that option being selected here -> https://docs.microsoft.com/en-us/azure/api-management/howto-protect-backend-frontend-azure-ad-b2c#configure-the-frontend-application
Also, the docs for msal.js specify that only the PKCE+Auth flow is supported from MSAL.js v2 onwards here ->
https://docs.microsoft.com/en-us/azure/active-directory/develop/migrate-spa-implicit-to-auth-code
I hope this answers your question, I'm happy to discuss more on this thread if it's still unclear.
Thanks,
Will Eastbury

In standard OAUTH's Authorization Code Grant flow, accessToken is not visible on client Side (as first backend gets the code from auth server (via browser), then using this auth code, backend gets accessToken from auth server (server-server communication and no browser/clientside involved)). but in your example accessToken is visible at client Side/Browser(javascript) which is the case with Implicit Grant (mainly used for client side applications)...
Am i missing something ?

Yes, but we do not use the standard flow for a JavaScript SPA, we use the PKCE enabled version (look at RFC7636 for more info).
Bear in mind that the sample is a JS SPA using B2C, and as such it has the following constraints :-

The Standard OAuth Auth Code Grant flow requires the publication of a client secret and is designed for server-server communications or trusted clients only, js SPA applications are not considered trusted in this way so this prevents the use of a shared secret key.
The implicit flow is no longer recommended to be used for js / spa client applications, due to a number of reasons, documented below.
For Single page apps, it is now recommended that the PKCE/Auth code flow is used to acquire the token (which is the model that msal.js v2 uses), which is why you have to register the application as a 'spa' (which also enables CORS on it's AAD endpoint) rather than enabling the implicit flow.

Take a look here for the RFC. https://datatracker.ietf.org/doc/html/rfc7636
In practice, you don't actually see this exchange take place as msal.js handles all of the protocol specifics for you, but something to consider is that most, if not all JS SPA type applications are designed to store the access / id tokens on the web applications localStorage, regardless of the method of token acquisition, so their refresh token's validity period is dramatically reduced to account for this.
For the specifics of how that happens, please take a look here.
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#redirect-uri-setup-required-for-single-page-apps
For the specifics on WHY this now happens,
In summary - it's largely to do with browser and third party cookie support along with the fact that you cannot issue a secure secret key to the browser code itself (no client secret is used in the PKCE+Auth Code flow).
But the Azure Active Directory team do a great job of explaining this here, along with the inner workings of the new flow and it's implications.
https://docs.microsoft.com/en-us/azure/active-directory/develop/reference-third-party-cookies-spas
Hope this helps.

Thanks @WillEastbury for the detailed answer and the links. I will go through them and come back to you if i have more queries. Thanks once again...

@CatchSandeepVaid I'm going to close this thread for now. You will still be able to respond and we can reopen if needed.


Add PostgreSQL support

With https://www.drupal.org/node/2157455 finally having been completed, it would be nice if users could switch from MySQL/MariaDB/Percona to PostgreSQL (at least for Drupal 8) optionally.
I don't currently have a Postgres role, but could add one or find a good one in Galaxy. Besides also installing php-mysql, there's not much else that needs to be conditionally changed besides swapping out Posgres for MySQL.

This actually what I need at the moment, being able to add geerlingguy.pgsl to playbook along side with the PHP connector

I used ANXS.postgresql seems to be working ok

@gitoutthere - Could you leave some instructions for what changes you made in this issue, so I can point others interested in trying out Postgres with Drupal here? I'm still planning on adding 'official' support sometime, but would be glad to 'unofficially' support it as best I can for now.

@geerlingguy cool I am still trying to wrestle and get my Envs setup. But here is the steps I took to get Postgres working, in your vagrant folder:

Add 'ANXS.postgresql' to '/provisioning/requirements.txt'
Run sudo ansible-galaxy install -r provisioning/requirements.txt --force to install the new plugin.
Modify your '/provisioning/playbook.yml' and add - ANXS.postgresql to the roles.
Override settings on '/config.yml'. Config can be found on https://github.com/ANXS/postgresql

ANXS.postgresql seems to be Ubuntu only.
I've been using patrik.uytterhoeven.PostgreSQL-For-RHEL6x for CentOS 5,6 and 7
Would love to see these combined into a geerlingguy.postgresql role :+1:

@JorisVanEijden - heh, always the optimist! I'll see what I can do, now that everything in Drupal 8 is running and tested with Postgres.

Well, 13 of the 14 galaxy roles in use here at oneshoe are geerlingguy roles. I'd say our infra has been pretty much geerlinged since buying your book half a year ago :wink:

Any of you guys are here at DrupalCon? I would love to catch up with you.
On 23 September 2015 at 09:48, JvE<EMAIL_ADDRESS>wrote:

Well, 13 of the 14 galaxy roles in use here at oneshoe are geerlingguy
roles. I'd say our infra has been pretty much geerlinged since buying your
book half a year ago [image: :wink:]
—
Reply to this email directly or view it on GitHub
https://github.com/geerlingguy/drupal-vm/issues/146#issuecomment-142521696
.

@gitoutthere - I wish! Watching presentations remotely, and participating in Twitter, but not much else.

I am going to give this a shot this weekend.

It seems ANXS.postgresql supports Centos now: https://github.com/ANXS/postgresql/issues/141#issuecomment-171807987

I've got a testable PR for this #490.

Greetings,
Wondering what is the status of PostgreSQL?

@DougDavenport - This is not a high priority for me right now, though I still do want to add at least minimal support.
I currently don't have any Drupal sites using Postgres, and know very few people who do, so just like with HHVM, it's kind of a niche use case that is more on the back burner for this project.

Thanks, we are new to using Drupal and have good experience with Postgres on other projects. I have noticed comments that some Drupal modules are said not to support Postgres. We are flexible, what are the pros/cons of the other DB choices? What do you advise?

The Drupal community (and most other PHP CMS communities) has traditionally only supported MySQL (and by extension, MariaDB). While other databases have had varying levels of support, none are as widely used and supported as My/Maria, so some modules and even some older Drupal core components had queries that were either optimized for MySQL or would fail in interesting ways on other relational databases.
With Drupal 8, Postgres support is a lot better, but it's still very slow in the uptake in the Drupal community, so I'm putting this on the backburner, but I definitely want to get to it soon so it's easier for people to compare the same site under Postgres vs. MySQL vs. MariaDB.

For what it's worth, PostgreSQL support is very important for those of us who use Tripal. Tripal provides support for storing data related to biology/agriculture research in a Drupal website and facilitates storing, organization and analysis of that data. However, it requires PostgreSQL because the community developed schema (used by a host of open-source analysis/viewer programs, not just Tripal) has PostgreSQL-specific features.
As a Tripal developer, I would really like to use and encourage others to use the Drupal VM since most Tripal/Drupal sites (currently 91 instances) will need to develop extensions and customizations but won't have the resources to use/own a dedicated development server.
I'm going to try to get PostgreSQL working as it seems others have earlier in the queue but I have no vagrant experience so any help or guidance would be VERY much appreciated!

@laceysanderson at the moment there's no easy way to add it :/
You would have to cancel and redo a lot of the stuff Drupal VM does. This would be what you have to do at the moment (untested so only usable as a starting point).
Add your own ansible playbook that runs after Drupal VM (docs):
Create a Vagrantfile.local with:
config.vm.provision 'ansible' do |ansible|
  ansible.playbook = "#{host_config_dir}/local.playbook.yml"
  ansible.galaxy_role_file = "#{host_config_dir}/local.requirements.yml"
end

Create a local.requirements.yml next to your config.yml with:
---
- src: ANXS.postgresql

Then create a local.playbook.yml with:
---
- hosts: all
  become: yes

  vars_files:
    - default.config.yml # Not entirely sure these are the paths...
    - config.yml

  pre_tasks:
    - name: Install PHP PostgreSQL dependencies.
      # if you're using Ubuntu 14.04, might be `php7.0-pgsql` or `php-pgsql`, not sure
      apt: name=php5-pgsql state=installed
      when: ansible_os_family == 'Debian'
    - name: Install PHP PostgreSQL dependencies.
      yum: name=php-pgsql state=installed
      when: ansible_os_family == 'RedHat'

  roles:
    - ANXS.postgresql

  tasks:
    - name: Check if site is already installed.
      command: >
        /usr/local/bin/drush status bootstrap
        chdir={{ drupal_core_path }}
      register: drupal_site_installed
      failed_when: false
      changed_when: false
      become: no

    - name: Install Drupal with drush.
      command: >
        /usr/local/bin/drush site-install {{ drupal_install_profile | default('standard') }} -y
        --site-name="{{ drupal_site_name }}"
        --account-name={{ drupal_account_name }}
        --account-pass={{ drupal_account_pass }}
        --db-url=pgsql://{{ drupal_mysql_user }}:{{ drupal_mysql_password }}@localhost/{{ pgsql_databases[0].name }}
        {{ drupal_site_install_extra_args | default([]) | join(" ") }}
        chdir={{ drupal_core_path }}
      notify: restart webserver
      when: "'Successful' not in drupal_site_installed.stdout"
      become: no

    - name: Install configured modules with drush.
      command: >
        /usr/local/bin/drush pm-enable -y {{ drupal_enable_modules | join(" ") }}
        chdir={{ drupal_core_path }}
      when: "'Successful' not in drupal_site_installed.stdout"
      become: no

And finally add these to your config.yml:
postgresql_databases:
  - name: "{{ drupal_mysql_database }}"
    owner: "{{ drupal_mysql_user }}"

postgresql_users:
  - name: "{{ drupal_mysql_user }}"
    pass: "{{ drupal_mysql_password }}"
    encrypted: no

postgresql_user_privileges:
  - name: "{{ drupal_mysql_user }}"
    db: "{{ drupal_mysql_user }}"
    priv: "ALL"

# We have to re-define these tasks as mysql is hardcoded.
install_site: false

Awesome!! That worked :-) I had managed to get partway there following @gitoutthere's comment on Aug 4, 2015 but I was running into problems forcing the new requirements plus that wouldn't have gotten me the php-pg link. Your recipe @oxyc worked perfectly! I did have to add a local ansible.cfg in order to get past an issue with Ansible 2.1 and the unpriviledged postgres user (issue: https://github.com/ansible/ansible/issues/16048):
Created a ansible.cfg with:
# This needs to be set in order for postgresql to be installed.
# postgresql installation requires becoming the unpriviledged postres
# use which is known to be a security risk:
#  https://docs.ansible.com/ansible/become.html#becoming-an-unprivileged-user
# As of Jun 2016 it is uncertain how to get around this as determined
# in this discussion: https://github.com/ansible/ansible/issues/16048
[defaults]
allow_world_readable_tmpfiles = TRUE

Once I added that and re-provisioned I was able to get a fully installed Drupal site with postgresql backend :-D Thanks!!

Off the back of this, I am seriously considering creating a role to suit this purpose.
As far as I'm aware, are these the requirements you're seeking, and if I was to engage in this task would I have your support? I'm a MySQL man, so I wouldn't actually be using this one myself.

Support for Debian/Ubuntu

Support for Arch

Support for RHEL/CentOS

Installs PostgreSQL

Passes tests via Travis

@fubarhouse - Support for Arch would be optional.
But yes, it would need to be available on Galaxy, and it also should support easy addition of databases / database users (like the mysql_databases and mysql_users vars in the MySQL role).

@geerlingguy You only have Ubuntu and CentOS available via the Hashicorp Atlas, are you able to recommend a box for Arch - perhaps the one used to test support for DrupalVM?

I don't currently do any Arch testing—it's not currently supported as a base box for the VM (though some users run it as their host OS).

There's a detailed rundown of all the postgresql specific modules on the following page, so as long as there're tasks focused to support it, you should have no issues integrating it.
http://blog.2ndquadrant.com/ansible-loves-postgresql/

Note that I'm working on a simpler PostgreSQL role here: https://galaxy.ansible.com/geerlingguy/postgresql/ (most of the other roles I've tried are either crazy-complex, only work on one or two OSes, etc.

Current status:

For automated builds, everything works up to drush site-install; at that point, with the Composer-based installation, we get the error:

Starting Drupal installation. This takes a while. Consider using the [ok]
--notify global option.
Error: Undefined class constant 'MYSQL_ATTR_USE_BUFFERED_QUERY' in /var/www/drupalvm/drupal/web/core/lib/Drupal/Core/Database/Driver/mysql/Connection.php on line 128
Drush command terminated abnormally due to an unrecoverable error.   [error]

If I wipe out the Composer-based installation and download Drupal 8.x core manually, then run the GUI installer at http://drupalvm.dev/core/install.php, then I hit the error message:

The bytea_output setting is currently set to 'binary', but needs to be 'escape'

I found this drupal.org issue (and see this forum topic, which tells me I have to run the following query to get the Drupal pgsql database in working order:
ALTER DATABASE drupal SET bytea_output = 'escape';

(To run that manually, vagrant ssh in, enter sudo su - postgres, then run psql, then run the query.)
After doing that, I get a successful GUI install, and the following in the status report:

Database system: PostgreSQL
Database system version: 9.5.4

So now I'm going to work on making sure the database is adjusted correctly (but why?), and also why Drush + Composer-based install is failing (whereas downloading manually works).

See also: [meta] Remaining Drupal 8 PostgreSQL issues. It seems that PostgreSQL support in D8 is similar to Nginx... most people use the old stalwart (e.g. Apache, MySQL), and the smaller percentage that do use the alternative (Nginx, PostgreSQL) are forced to work through a lot of problems independently :)

Well, would you look at that!

Now just waiting on build #1009 to succeed...

The build failed with:
InstallerException: Resolve all issues below to continue the
installation. For help configuring your database server, see the
installation handbook</a>, or contact your hosting provider.

The bytea_output setting is currently set to 'hex', but needs to
be 'escape'. Change this by running the following query:
ALTER DATABASE "drupal" SET bytea_output = 'escape';
in /var/www/drupalvm/drupal/web/core/includes/install.core.inc:911

But it worked locally. Re-running locally to see if I can reproduce the error :/

Hmm... getting that locally too.

It seems that global config change won't take effect until a restart of the server, and it feels like a hacky way of going about it anyways; if I want to use that, I'll have to do a meta: flush_handlers somewhere in a role, and I hate having to do that.
So instead, I'll add a task right before site install:
# See: https://www.drupal.org/node/2569365#comment-11680807
- name: Configure database correctly if using PostgreSQL.
  command: psql -c "ALTER DATABASE {{ drupal_db_name }} SET bytea_output = 'escape';"
  when: "'Successful' not in drupal_site_installed.stdout"
  become: yes
  become_user: "{{ postgresql_user }}"
  # See: https://github.com/ansible/ansible/issues/16048#issuecomment-229012509
  vars:
    ansible_ssh_pipelining: true

Running local tests again.

@laceysanderson - FYI, I am using a workaround for the Ansible 2.1+ issue you mentioned earlier (https://github.com/ansible/ansible/issues/16048) in the PostgreSQL role, so you won't need to maintain that specialized ansible.cfg file anymore (which makes things slightly less secure overall!).

Whee!


Create a shared folder for all interfaces

Hey @alphaolomi I added a shared folder and moved all interfaces in the folder

Thanks @BaharaJr

@BaharaJr  What do u think of the inline todo comments?

@BaharaJr What do u think of the inline todo comments?

The comments are okay as long as they don't get pushed to the library repository


Update dx-graphics-hlsl-to-samplebias.md

@briannagerads : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[OR-1470] change wton to wnativetoken

add deploy config (nativeTokenName, nativeTokenSymbol)
change wton to wnativetoken
add L1devnet in verify-scripts and fix typo and change version of compiler of WNativeToken

Thank you!

@nguyenzung
I'm sorry. I accidentally did something else while I was working.
Legacy_ERC20_ETH -> Legacy_ERC20_{NativeTOkenName} will be processed in the next ticket. https://onther.atlassian.net/browse/OR-1415?atlOrigin=eyJpIjoiZGYwNjk3Mzc2YTg3NGQwYjgwODliNjM5NzBiMDU3NzAiLCJwIjoiaiJ9


Major changes to frontend design

I was pretty unhappy with the design of my prior website. I wanted something sleek, modern, simple, and elegant. I did a lot of research from other sites and even other SPS student sites and implemented a new design with hopefully much cleaner code. Makes heavy use of CSS grid.

Will deploy right now!


"return" statements should not occur in "finally" blocks

This pull request is focused on resolving occurrences of Sonar rule squid:S1143 - "return" statements should not occur in "finally" blocks
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1143
Please let me know if you have any questions.
Kirill Vlasov

Please update the commit message to start with the section impacted, in this case "[s3]".
Otherwise this looks fine.


fix: allow to delete all ACLs for principal

Creating for early feedback for https://github.com/redhat-developer/app-services-cli/issues/1430
Validation for delete command were redundant preventing users from deleting multiple acls.
For example it would not be possible to remove all ACLs for Service account.
After this change it is possible:
✔  Deleted 5 ACLs from Kafka instance "wtrocki"
The following ACL rules were deleted:

  PRINCIPAL (5)    PERMISSION   OPERATION   DESCRIPTION
 ---------------- ------------ ----------- --------------------------
  test             allow        write       transactional-id is "*"
  test             allow        describe    topic starts with "test"
  test             allow        write       topic starts with "test"
  test             allow        create      topic starts with "test"
  test             allow        describe    transactional-id is "*"

However we need to still spend some time cleaning up messages/validation and arguments to API.
Posted for early feedback

Deleting all acls for a principal.
Passing other arguments needs to be handled for deleting specific ACL. Ignore if it is on todo.

Not sure if we should allow all accounts as well?

[wtrocki@graphapi app-services-cli (delete-acl ✗)]$ rhoas kafka acl delete --all-accounts
? All ACLs matching the criteria provided will be deleted from the Kafka instance "wtrocki". Are you sure you want to proceed? Yes

✔  Deleted 4 ACLs from Kafka instance "wtrocki"
The following ACL rules were deleted:

  PRINCIPAL (4)    PERMISSION   OPERATION          DESCRIPTION
 ---------------- ------------ ------------------ ----------------
  All Accounts     allow        describe-configs   topic is "*"
  All Accounts     allow        describe           group is "*"
  All Accounts     allow        describe           cluster is "*"
  All Accounts     allow        describe           topic is "*"

Seeing some strange issue/inconsitencies:
[wtrocki@graphapi app-services-cli (delete-acl)]$ rhoas kafka acl list

  PRINCIPAL (5)    PERMISSION   OPERATION   DESCRIPTION
 ---------------- ------------ ----------- --------------------------
  wtrocki          allow        create      topic starts with "test"
  wtrocki          allow        describe    topic starts with "test"
  wtrocki          allow        write       topic starts with "test"
  wtrocki          allow        describe    transactional-id is "*"
  wtrocki          allow        write       transactional-id is "*"
[wtrocki@graphapi app-services-cli (delete-acl)]$ rhoas kafka acl delete --service-account=wtrocki
? All ACLs matching the criteria provided will be deleted from the Kafka instance "wtrocki". Are you sure you want to proceed? Yes

✔  Deleted 2 ACLs from Kafka instance "wtrocki"
The following ACL rules were deleted:

  PRINCIPAL   PERMISSION   OPERATION   DESCRIPTION
 ----------- ------------ ----------- -------------------------
  wtrocki     allow        describe    transactional-id is "*"
  wtrocki     allow        write       transactional-id is "*"

Ok. So problem is with Pattern Type. because we assume Literal pattern type is provided by default there is no easy way for us to fix that without breaking CLI. I will need to add explicit pattern-type=none and deprecated --prefix to get around this

@craicoverflow @rkpattnaik780 I have managed to workaround problem by introducing new flag called pattern type and deprecated prefix.
Not sure if we should be deprecating prefix. I'm testing all gazillion test use cases that this change introduced

Pattern type is fine to use. I used "prefix" because this is what confluent CLI does.

Not sure if we should allow all accounts as well?

UI allows deleting permissions for all accounts. We should keep it too.

What is this submodule for?

What is this submodule for?

That was different POC. Removing it

You could always look to remove that rule from the lint config file? It is not a rule coming from the Go stdlib or anything.

Rule itself is good, but it does have lots of false positives - where doing switch is not possible.
I will need to start disabling linting for this instead.


Niko/create calendar page

This is how is supposed to look like. Just use already designed template for calendar page.


PyQT issue for install  neoscore on a Raspberry Pi running Raspbian v10 (buster)

Hi, I'm trying to install neoscore on a Raspberry Pi running Raspbian v10 (buster).  I'm getting exactly the same error(s) as above and I've tried most of the above suggestions, except, of course, that I have no Rosetta to run from!  Is there anything that I can do to progress?  I have, incidentally, managed to get neoscore working on a Mac running 10.13.6.
Originally posted by @richardhoadley in https://github.com/DigiScore/neoscore/discussions/35#discussioncomment-3004382

Hi Andrew. I’ve migrated this issue by Richard to a new thread.

@richardhoadley we just release neoscore 0.1.3, which reduces the minimum required python version to 3.7. This should allow you to use the prebuilt PyQt5 wheels. Could you try installing neoscore again on Python 3.7 and let us know if it works?

Closing this now on the assumption the prebuilt wheels work. We can reopen later if this turns out to not be the case.


Live reload not working

Atom 1.14.3 atom-live-server 2.1.1
The server starts, but it doesnt live reload any more?

Hmm - it works again. Maybe it was an atom problem.. Sorry.


Remove redundant null checks.

The String.IsNullOrWhiteSpace method already checks for nulls.

Thanks for sparking this discussion @alimon808 and @Youssef1313
I'm leaning toward accepting this PR, because for most uses combining the null check with an empty (and possibly white space) check is the best solution.
It's also true that for wide use frameworks, separating those checks provides benefits for developers using those APIs.
I'll open this for discussion now.

To me it boils down to the intent of the constructor in this example and asking the following question: Is there a need to throw an ArgumentNull exception such that the consumer can handle it differently from an ArgumentException.  In the context of the Inheritance in C# documentation where the snippet is referenced, I didn't see such a need.  I don't know if this snippet is referenced in other documentations which could impact whether to merge these changes or not.  Can snippets from sample codes for one documentation be referenced in another?

Thanks for the discussion everyone.
The consensus (both online and in other discussions) is to merge this.
I've approved the changes, and I'll :shipit: now.


[Bug] navlayers initialization errors out

Expected Behavior
A navigation layer object should be created when passing a name and a domain to mitreattack.navlayers.layer
Actual Behavior
The program errors out since in the init method uses self.__data (double underscore), but the rest of the code uses self._data (single underscore)
Steps to Reproduce the Problem
Follow example 2 of https://github.com/mitre-attack/mitreattack-python/tree/master/mitreattack/navlayers/core#creating-layers-programmatically

Init a navigation layer with a name and a domain

import mitreattack.navlayers as navlayers

layer_example = navlayers.Layer(name="example", domain="enterprise-attack")

Traceback (most recent call last):
  File "/Users/me/test/test.py", line 3, in test
    layer_example = navlayers.Layer(name="example", domain="enterprise-attack")
  File "/Users/me/.local/share/virtualenvs/test-XrAZKEUD/lib/python3.8/site-packages/mitreattack/navlayers/core/layer.py", line 19, in __init__
    self._build()
  File "/Users/me/.local/share/virtualenvs/test-XrAZKEUD/lib/python3.8/site-packages/mitreattack/navlayers/core/layer.py", line 82, in _build
    self.__layer = _LayerObj(self._data['name'],  self._data['domain'])
AttributeError: 'Layer' object has no attribute '_data'

Possible Solution
Change self.__data (double underscore) to self._data (single underscore).

Thanks for bringing this to our attention!


Create custom UI for Enum

Right now enums just show up in the UI as a string of the value name. It would be nice if they showed up as a styled UI "pill" or similar.

Hi @augray, I would like to contribute, can I take this issue?

Hello, @KatkaG ! Please go ahead! Here's an example video showing how to add support for a type visualization: https://youtu.be/QNVhL37GC30


[#329] Add forms validation

I added validation for login and register forms by using vuelidate.

Cool, thx!


v1.13 Backports 2023-03-06

[ ] #24097 (@jibi)

Once this PR is merged, you can update the PR labels via:
for pr in 24097; do contrib/backporting/set-labels.py $pr done 1.13; done

or with
make add-labels BRANCH=v1.13 ISSUES=24097

/test-only --focus="K8sDatapathEgressGatewayTest.*" --k8s_version=1.26 --kernel_version=net-next

Focused test is green: https://jenkins.cilium.io/job/Cilium-PR-Tests-Kernel-Focus/525/
marking as ready


Some trouble  when I used the EEG of the .edf format to do source location

When I used the dateset of BIC2000 system to do source location，I ran my code below：
##########################################################################
import numpy as np
import matplotlib.pyplot as plt
import mne
from mne import Epochs, pick_types, find_events
from mne.channels import read_layout
from mne.io import concatenate_raws, read_raw_edf
from mne.datasets import eegbci
from mne.minimum_norm import (make_inverse_operator, apply_inverse,
write_inverse_operator)
##############################################################################
tmin, tmax = -1., 4.
event_id = dict(hands=2, feet=3)
subject = 1
runs = [6, 10, 14]  # motor imagery: hands vs feet
raw_fnames = eegbci.load_data(subject, runs)
raw_name = [read_raw_edf(f, preload=True, stim_channel='auto') for f in
raw_fnames]
raw=raw_name[0]
raw.set_eeg_reference()  # set EEG average reference
events = mne.find_events(raw, stim_channel='STI 014')
picks = mne.pick_types(raw.info, meg=False, eeg=True, eog=False,
exclude='bads')
baseline = (None, 0)  # means from the first instant to t = 0
epochs = Epochs(raw, events, event_id, tmin, tmax, proj=True, picks=picks,
baseline=None, preload=True)
noise_cov = mne.compute_covariance(
epochs, tmax=0., method=['shrunk', 'empirical'])
fig_cov, fig_spectra = mne.viz.plot_cov(noise_cov, raw.info)
##############################################################################
evoked = epochs.average()
evoked.plot()
evoked.plot_topomap(times=np.linspace(0.05, 0.15, 5), ch_type='mag')
evoked.plot_white(noise_cov)
##############################################################################
fname_fwd = mne.make_forward_solution(raw.info, trans, src, bem)
fwd = mne.read_forward_solution(fname_fwd)
fwd = mne.convert_forward_solution(fwd, surf_ori=True)
fwd = mne.pick_types_forward(fwd, meg=False, eeg=True)
info = evoked.info
inverse_operator = make_inverse_operator(info, fwd, noise_cov,
loose=0.2, depth=0.8)
Due to the .edf format EEG, my question is that I don't know how to set up the parameter of this segment code below：
fname_fwd = mne.make_forward_solution(raw.info, trans, src, bem)
I’m also confused about whether the fname_fwd is a fixed forward solution for any subjects，can I use this value from the default path of （fname_fwd = data_path + '/MEG/sample/sample_audvis-meg-oct-6-fwd.fif'）

your forward "sample" is subject specific. You need a generic forward model
eg using an average brain like fsaverage

@agramfort  Thank you to answer my question!
You mean that I use the fsaverage space in the Freesurfer as the average brain model, but I don't know how to transfer the generic forward model to my code when I ran into “fname_fwd = mne.make_forward_solution(raw.info, trans, src, bem)” .
My purpose is only to get the inverse solution（source estimates） according to my EEG signal of the .edf format. Can you give me some guide of code to generate  the fwd，then I can use it in the function of inverse_operator below.

maybe use this ressource:
https://fr.slideshare.net/mne-python/mnepython-scale-mri

@agramfort thanks for your sharing, let me see first.

@agramfort  Excuse me！I read the ressource which you shared the other day. I found the Input Files specifies the directory containing MRIsubjects in the tutorial ,however my subjects only contain the EEG signals of .edf format. I'd like to know whether I can select the raw files of my EEGsubjects for which to do the coregistration. And finally how can I get the fwd.fif to read instead of  a *-trans.ﬁf ﬁle.

it's not simple. We don't have simple support for working with a template
brain just given
the electrode names. Maybe someone can pitch in as I have no personal
experience
in such a scenario.

@agramfort All right，thank you very much！


Allow passing ca subject and server certificate subject for self-signed certificates

Is your feature request related to a problem? Please describe.
Currently, when using self-signed certificates in Routr, there is no option to pass the ca subject and server certificate subject. The default server certificate subject is set to localhost, which prevents connections using any other domain.
Describe the solution you'd like
I propose introducing a set of environment variables, along with corresponding Helm chart variables, to allow passing the following values for self-signed certificates:

ca subject
server certificate subject

Describe alternatives you've considered
One option is to keep the current setup and only permit connections using the subject localhost, considering that production environments should be using Let's Encrypt.
Additional context
None.

Solved here https://github.com/fonoster/routr/commit/364a631f11ce32573ef7b5f07482dd23726e9302


cmd/go: too many open files in system

What version of Go are you using (go version)?

$ go version
go version go1.20.1 darwin/arm64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/maxgiraldo/Library/Caches/go-build"
GOENV="/Users/maxgiraldo/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/maxgiraldo/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/maxgiraldo/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.20.1"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/maxgiraldo/go/src/github.com/maxgiraldo/test_server/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lf/r3_cvxl10n502833_y2v3rr80000gn/T/go-build2784566378=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Trying to run basic server:
package main

import (
    "fmt"
    "net/http"
)

func hello(w http.ResponseWriter, req *http.Request) {

    fmt.Fprintf(w, "hello\n")
}

func headers(w http.ResponseWriter, req *http.Request) {

    for name, headers := range req.Header {
        for _, h := range headers {
            fmt.Fprintf(w, "%v: %v\n", name, h)
        }
    }
}

func main() {

    http.HandleFunc("/hello", hello)
    http.HandleFunc("/headers", headers)

    http.ListenAndServe(":8090", nil)
}

I am not able to run or install the program.
What did you expect to see?
The server running.
What did you see instead?
/usr/local/go/src/vendor/golang.org/x/text/unicode/bidi/core.go:9:2: open /usr/local/go/src/log/example_test.go: too many open files in system
Why is go trying to open these files?

I ran go mod tidy earlier and it completely messed up everything and failed because it tried to search my entire filesystem. Not sure if causal relationship is there, but that context might help.

What does go run -x print?

WORK=/var/folders/lf/r3_cvxl10n502833_y2v3rr80000gn/T/go-build2569235747
/usr/local/go/src/vendor/golang.org/x/text/unicode/bidi/core.go:9:2: open /usr/local/go/src/log/example_test.go: too many open files in system

/usr/local/go/src/vendor/golang.org/x/text/unicode/bidi/core.go:9:2: open /usr/local/go/src/log/example_test.go: too many open files in system
Why is go trying to open these files?

golang.org/x/text/unicode/bidi imports the log package.
Compiling the log package itself shouldnt require opening example_test.go, but as of Go 1.19 the go` command indexes package information and uses that instead of reopening files for each build. That is supposed to happen only once per build, but of course if that fails the  index won't be written to the cache.
But, that seems like something of a red herring — if the error is too many open files in system, what else on your system is holding files open? (If it were just the go command, the error would presumably be too many open files instead.)
A good starting point for that investigation might be to run lsof and see if you spot a pattern.

Closing as it doesn't seem to be an issue with go, but rather my system.

I have about 8007 open files in my system on fresh reboot and closing my applications. I set a soft limit for 10240, but there's a hard limit of unlimited. I'm not sure why go would still complain.
ulimit -aS
-t: cpu time (seconds)              unlimited
-f: file size (blocks)              unlimited
-d: data seg size (kbytes)          unlimited
-s: stack size (kbytes)             8176
-c: core file size (blocks)         0
-v: address space (kbytes)          unlimited
-l: locked-in-memory size (kbytes)  unlimited
-u: processes                       1333
-n: file descriptors                10240

FWIW, based on https://superuser.com/questions/433746/is-there-a-fix-for-the-too-many-open-files-in-system-error-on-os-x-10-7-1 I checked a couple of the builders operated by the Go project and found:
darwin-amd64-13-0:
$ lsof | wc -l
    3311
$ sysctl kern.maxfiles
kern.maxfiles: 30720
$ sysctl kern.maxfilesperproc
kern.maxfilesperproc: 10240

darwin-arm64-12:
$ lsof | wc -l
    3643
$ sysctl kern.maxfiles
kern.maxfiles: 122880
$ sysctl kern.maxfilesperproc
kern.maxfilesperproc: 61440

Even on the amd64 builder with a maxfilesperproc limit of only 10240, that seems like plenty of headroom. I would be surprised if any of those limits were adjusted when the builder images were created. (There aren't any such steps listed in https://cs.opensource.google/go/x/build/+/master:env/darwin/setup-notes.md.)
Perhaps you have some kind of third-party service installed that is holding a lot of files open?

Yeah, i’m going to do a clean install and see if I can reproduce.

I was running out of file descriptors on Mac OS every few days a few months ago. It started around the time I installed Tailscale's Golink program, and it seemed to be related to that because, it seemed to stop when I stopped running it, and since I upgraded Golink a few weeks ago, the problem hasn't recurred. I'm still unsure what the root cause was though, maybe that's a coincidence. I've also had instability with my Time Machine backups.

I erased all contents and settings on my Mac and re-installed go and was able to run it. There's still about ~6265 open files by default when you do a fresh install of Ventura 13.1, but go no longer complains. I guess I'll be careful about what I install next time.


CacheResult annotation should allow blocking behavior

When a cache key is invalidated or expires then multiple concurrent invocations of a method annotated with @CacheResult may get a cache miss. This can trigger multiple potentially expensive computations running in parallel - MISS storm.
The CacheResult annotation should be configurable to allow only a single thread to invoke the annotated method. All other threads should be blocked until the 1st thread is done and then they should recheck the cache.
This is the idea:
public @interface CacheResult {
[...]
  boolean blocking() default false;
[...]

alternative name: boolean atomic() default false;

Yes, see Bens comment here:
https://rmannibucau.wordpress.com/2015/08/21/cacheresult-jcache-cdi-to-the-rescue-of-microservices/
The problem is that JCache does not cover the fact of blocking at all. It is not mandatory in the imperative API and no established concept. "atomicity" maybe, however, whether atomicity leads to blocking, would be implementation dependent.
However, nothing I think we can do in a maintenance release.

yes, this is definitely JCache 2.0 material. Is there a better place to collect idea for JCache 2.0 ?

The issue is very relevant for using annotations in high throughput scenarios.
I know the problem, but I didn't dare for bringing it up here, since there is other fish to fry. I have a couple of things like this on my private list, but I didn't want to start discussion on this, since the work needs to be focused and time is limited. However, it is good to see that others come to the same conclusion, and it is not only me raising new issues :)
Procedure is to label with "JCache next" and close the ticket, to keep the overview what we need to address for the next MR. This is something the Spec leads do not need to bother with currently, so I would do so, too. Although it is closed, the topic is not lost. Okay?

sounds good to me.

Thanks, done!

I'm confused what it was re-opened.

I am happy to collect JCache 2.0 stuff here. I added a milestone. I will then transfer these over to JCache 2.0 once it starts.

Reopened it because I am collecting JCache 2.0 feature requests and issues now.

Note that users have tried to work around this using a CacheLoader, but it gets messy due to CacheKeyGenerator. It also means that the method is merely a stub, which defeats the value imho. I think this was what @gregrluck was trying to convey as his desired solution when we discussed the feature in the RI. Unfortunately while a good idea, the result isn't very usable.
Spring uses sync = true, defaulted to false, in its Cacheable annotation.


Env var support

https://www.pivotaltracker.com/story/show/104301140

@indrekj fixed 'nill' typo and now use 'let'

I've also bumped from version 0.1.0 -> 0.2.0

If you do a version bump, then you also need to do "bundle" and commit the Gemfile.lock

Or you can remove the Gemfile.lock from the repository. It is not needed for gems.

+1 for adding Gemfile.lock to gitignore

Removed Gemfile.lock. Jenkins still angry. New message:
bundle install --deployment
The --deployment flag requires a Gemfile.lock.

+1


Refprop headers: add port

Thanks for the PR!
I changed the version to match other ports (YYYY-MM-DD).


Public IPFS Instance - separating pinning from the public instance.

@voodoohop i think this is more or less done. closing


CLI add should create a service

Plugins are deprecated. We should use railway add to create services

Im not sure I follow.
railway add is already using railway deploy internally, which can be used to deploy any template.
Are you suggesting something else?

Fixed by #570


Critical: 2.0 version data read: Float and Date values from same queries are not showing up same as Version 1.4.6

I have two instances running on same machine, pointing to same DB and SAME code-base - ONLY exception is one uses ODBC 1.4.6 and other uses ODBC 2.0.0.
I am doing this to test new version. Performance is consistently 50% better even when I tested iwht 10MB of data returned through queries.
Issue: Main issue is FLOAT values returned by ODBC 2.0.0 driver are not correct. All other values match including dates/strings/numbers on same record..
Below is example:
Float samples
ODBC 1.4.6:  [507.365, 504.167]
ODBC 2.0.0: [1.83187975902498e-76, 4.2778154908686205e-33]
Date samples : (Issue with dates is HOURS is always showing in LOCAL/EST with 2.0 version)
ODBC 1.4.6:  [2016-02-22T05:29:41.000Z]
ODBC 2.0.0: [2016-02-22 00:29:41.0000000] - Please see how ONLY hours part is wrong
ODBC 1.4.6: "2017-02-06T19:29:51.000Z"
ODBC 2.0.0: "2017-02-06 14:29:51.0000000"
NOTE: As you see difference here is HOURS is -5H difference from OLDER version. Suggesting that new 2.0.0 version is converting always values to LOCAL/EST timezone
Above are sample from one or two records but this is the case throughout.
Environment: Node 8.16.0 [for both on same machine and pointing to same DB/queries etc]
This is critical issue as data needs to be 100% accurate with new version. Really appreciate any help on this.

Ok, so I've investigated the timestamp issue, and I think the right solution is to leave it as it is.
Justification:
In ODBC 1.4.6, when a SQL_DATETIME or SQL_TIMESTAMP field is returned, the package would take the String returned, and call the C functions mktime, timelocal, or gmtime depending on what OS you were running and what functions you had available to you.
In the examples you posted for 1.4.6, what is happening is the database is returning '2017-02-06 14:29:51.0000000' (actually returns a struct, but for simplicity's sake), and then calling timelocal, which says "I want this in UTC, and what I am giving is in my local time". Of course, there is no guarantee that the timestamp is really in your local time, as in most DBMS timestamps are stored without timezone data. And indeed, I can't find any example of ODBC itself having any knowledge of timezones.
If I leave it as is, what happens is the ODBC driver is told to return the timestamp in a String. If the driver/DBMS has a way to track timezone, it should return the timezone in the String. If the driver/DBMS has no concept of timezone, it will return as you show in ODBC 2.0.0. So results may not be consistent across different drivers, which may return data in different formats (and even the same driver with different settings enabled), but the returned data will be more accurate.

Thank you. I will test more.. but I agree that driver should not do any conversion and return what is returned in DB as-is.  But it is important to highlight this change in your Read me as you don't want people to find this out by surprise. You will just help them upfront this way to avoid noise.
Really appreciate you taking care of this issue. Thanks.


Upgrade com.puppycrawl.tools:checkstyle 10.1 -> 10.2

This PR contains the following updates:

Package
Update
Change

com.puppycrawl.tools:checkstyle (source)
minor
10.1 -> 10.2

[ ] If you want to rebase/retry this PR, click this checkbox.

Suggested commit message:
Upgrade Checkstyle 10.1 -> 10.2

See:
- https://checkstyle.sourceforge.io/releasenotes.html
- https://github.com/checkstyle/checkstyle/releases/tag/checkstyle-10.2
- https://github.com/checkstyle/checkstyle/compare/checkstyle-10.1...checkstyle-10.2


Screenshots for planned versions

Your Question
In the New Features section of CWA 1.14.3 it says:

"Screenshots for the Corona-Warn-App"
"The website https://www.coronawarn.app now contains all the screenshots for the app, so you can find out about planned features, for example."

https://www.coronawarn.app/en/screenshots/ doesn't show any information for planned features, only existing features.
Will that change? Will the page include screenshots for app versions before they are released as well as for the current release?

Internal Tracking ID EXPOSUREAPP-5738

Hi @MikeMcC399, yes, we noticed this discrepancy. The sentence is wrong:

there are not all screenshots
there are no screenshots of planned features.

Let's consider this as a bug.

It would be disappointing if it is a bug. If there's one consistent criticism of the CWA it's that there is no public transparency of what the RKI has agreed for the upcoming releases.


cargo doc --open doesn't work on WSL2

Problem
expected behavior: cargo doc --open should open the generated documentation using the default browser
observed behavior: cargo doc --open fails with an error:
 Finished dev [unoptimized + debuginfo] target(s) in 0.19s
     Opening /home/aiyer/workspace/cargo/target/doc/cargo/index.html
warning: couldn't open docs

command 'wslview' did not execute successfully; exit status: 1
command stderr:
Start : This command cannot be run due to the error: The system cannot find the file specified.
At line:1 char:128
+ ... oding(437); Start "wsl.localhostUbuntuhomeaiyerworkspacecargo ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Start-Process], InvalidOperationException
    + FullyQualifiedErrorId : InvalidOperationException,Microsoft.PowerShell.Commands.StartProcessCommand

Steps
No response
Possible Solution(s)
The issue is due to a bug in the 'opener' crate (v0.5.2). Please see https://github.com/Seeker14491/opener/issues/19. This is fixed in v0.6.1. Updating to this version resolves the issue ( https://github.com/abhiramiyer/cargo/commit/04e5675717c41c94f669a12bc4f272347b788f71).
Notes
No response
Version
cargo 1.70.0 (ec8a8a0ca 2023-04-25)
release: 1.70.0
commit-hash: ec8a8a0cabb0e0cadef58902470f6c7ee7868bdc
commit-date: 2023-04-25
host: x86_64-unknown-linux-gnu
libgit2: 1.6.3 (sys:0.17.0 vendored)
libcurl: 8.0.1-DEV (sys:0.4.61+curl-8.0.1 vendored ssl:OpenSSL/1.1.1t)
os: Ubuntu 22.04 (jammy) [64-bit]

Thanks @abhiramiyer, and feel free to open a PR to fix it :)


Option to Code Segments (Section Division, Phrase Division, perhaps even other segments within)

I'll send an email but let's discuss on Thursday. GitHub doesn't

Numerical and/or letter code to abstract from Hindustani with "open ends" to custom
Hindustani

A. Form (this will be determined by the "event" but the coding can branch accordingly
Alap
Jor
Alap-Jhala
B. Composition (Bandish, Gat, Chiz, Dhrupad)
Pre-Chiz Alap
Asthai
Antara
Manja
Abogh
Sanchari
etc (a few more if we align)
Other specify
C. Elaboration/Improvisation
Barhat/Vistar
Bol Banao
Bol Bandt
Aakar
Sargam
Tan
Layakari
etc (a few more if we align)
Other specify
D. Alankar
Gamak
Murkhi
Meend
Sapaat
Zamzama
etc (a few more if we align)
Other specify

Be able to create Label(s) (or choose from pre-set names) for each different "section" , this would show in the analysis visualizations.

A and B seem like they are labels that belong at the "section" level, (right?). Maybe C would be something you can apply at the "phrase" level? and D at the "Trajectory" level? (or maybe "group" level?)


OmniLight3D & SpotLight3D Attenuation field - Curve and Presets are missing

Tested versions
Godot 4.3 stable and newer
System information
Godot v4.3.stable - macOS 13.6.9 - Vulkan (Forward+) - dedicated AMD Radeon Pro 580 - Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 Threads)
Issue description
As you see in this screenshot, the attenuation field has no curve in the field and also no presets under the right mouse button:

In comparision the expected behavior:

a+
Steps to reproduce
Make an OmniLight3D
Go to the Light-Settings
Minimal reproduction project (MRP)
lightattenutation.zip

This was done by design to fix a number of issues:

https://github.com/godotengine/godot/pull/87583

This change shouldn't be reverted I'd say

Ok, I see. Read the thread and I agree!


Netty version bump breaks unit tests

The recent bump of netty version (#172 ) breaks the unit tests in org.littleshoot.proxy.impl.ProxyUtilsTest#testCommaSeparatedHeaderValues .
It is not immediately clear to me if the tests are in fact obsolete and need updating, or is there some sort of bug in the updated library, so this issue should be examined closer.
In the meantime however I believe the revert of the offending library is in order: (#178)

I've filed Netty issue: https://github.com/netty/netty/issues/13005. Let's see how it goes.


WIP: Annotations feature

The PR to merge the annotations feature branch into master

https://travis-ci.org/rackerlabs/blueflood/builds/69978307 - Proof that tests pass


bugfix: 修复移除主机后部署策略巡检执行异常的问题 (fixed #206)

Codecov Report

Merging #213 (76aa3a2) into V2.1.352rc (4528643) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           V2.1.352rc     #213      +/-   ##
==============================================
- Coverage       61.87%   61.87%   -0.01%
==============================================
  Files             224      224
  Lines           19434    19434
==============================================
- Hits            12025    12024       -1
- Misses           7409     7410       +1

Impacted Files
Coverage Δ

apps/backend/subscription/steps/plugin.py
35.30% <ø> (ø)

apps/node_man/views/host_v2.py
48.27% <ø> (ø)

apps/node_man/handlers/job.py
86.07% <0.00%> (-0.26%)
:arrow_down:

Continue to review full report at Codecov.

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


fix: update dependencies and configurations for Expo and Next.js

Updates various dependencies and configurations in the monorepo starter:

Upgrade Expo and related packages to latest compatible versions
Remove expo-router/babel plugin from Expo babel config
Update React Native and related packages
Adjust TypeScript and other dev dependencies
Switch from Bun to Yarn as the package manager
Add autoprefixer as a dev dependency
Update scripts to use Yarn instead of Bun

Since the starter uses quite a lot of dependencies, it might be a good idea to use "~" instead of "^" for some major lib like expo, react-native, etc to be more safe.

The safest thing is to use neither. It should be fixed, exact versions.

@ktkennychow any shot you could make the same changes in the blank template? Would be super appreciated 🙏🙏

Sounds good. expo upgrade will set the versions to ~.
Most important thing in a react monorepo is avoiding differing versions. As long as the same version is set in two places and saved in a lock file without ^ it should be okay

@nandorojo updated and tested /blank & /with-expo-router starters.

Thank you!
We reverted the previous PR that broke things, which looks like this introduced some conflicts…
I think we’d just need to fix the package.json files and run yarn to fix the lock files. Let me know if you need help

@nandorojo hmm that's exactly what I did, only "fix the package.json files and run yarn to fix the lock files", plus changing most dependencies to prefixed "~" w/ some exception that requires "^" or exact version.
The only odd one out is an URL change in [example-monorepos/with-expo-router/apps/next/next-env.d.ts](https://github.com/nandorojo/solito/pull/491/files#diff-307a8b5419151cba243e9a8f37d28b29736b3a241198b8748bd927d283bf5bcc) but this one in my commit is actually the correct one.
Do you mean I fork again to test against the reverted repo?

I think you might need to pull in the latest master branch to resolve the conflicts

Once the package.json file(s) have no conflicts, running yarn will fix lock file conflicts

@nandorojo seems like the revert got the starters working again, I will close this PR since it is not necessary.


Installation fails with Error 1

When I run make for ngsutils, I get the following error message
/ngsutils$ make
./init.sh
./init.sh: line 16: venv/bin/activate: No such file or directory
make: *** [init] Error 1
Does anybody now how to solve this?
Thanks

See Issue #12. Basically, try changing the interpreter to bash for init.sh and the ngsutils wrapper script in the bin folder after compilation (i.e. change the first line from "#!/bin/sh" to "#!/bin/bash")

Should have been fixed with a prior commit.


A little more beautiful

To be able to move components in subordinate Route

Current coverage is 71.87% (diff: 100%)

Merging #1103 into master will not change coverage

@@             master      #1103   diff @@
==========================================
  Files            12         12
  Lines            64         64
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             46         46
  Misses           18         18
  Partials          0          0

Powered by Codecov. Last update 2675bb2...c361bc2

Thanks for the PR! Unfortunately, this is done intentionally; this comment may provide some explanation as to why.


fix: undo/redo when exiting view mode

Description
This PR addresses the bug: When exiting view mode the undo/redo buttons are disabled even though their stacks aren't empty.
Fix
Fixes https://github.com/excalidraw/excalidraw/issues/7948.
The default constructor is getting called which has values isUndoStackEmpty equal to true and isRedoStackEmpty equal to true.
Passing the function(from class History) which checks for whether the stacks are empty or not fixes the issue.
Comparison
Before
https://github.com/excalidraw/excalidraw/assets/53573748/ff57b133-8d9b-4574-8e69-e59b60177b85
After
https://github.com/excalidraw/excalidraw/assets/53573748/329588d3-b6bb-4eea-8fe8-d9faa6810da1

Hello,
I tried rerunning the test files on my local machine but I don't seem to get the error which is shown above. It would be very helpful if someone could explain what exactly is the testcase testing. I tried searching for documentation related to tests in the repo but couldn't find it.
Thank you.
(I'm new to open-source apologies if it is trivial)

I tried rerunning the test files on my local machine but I don't seem to get the error which is shown above. It would be very helpful if someone could explain what exactly is the testcase testing. I tried searching for documentation related to tests in the repo but couldn't find it.

Yeah, we have some flaky tests in the history test suite, we'll need to fix.
Anyway, I added tests for this PR. We're good to go. Thanks!


make update feature generic so it can be reused

@jimmidyson not sure if this is the best approach but; I've vendored the minishift update.go package in gofabric8 and reused the MaybeUpdate  function which works great now that gofabric8 releases are in the same form as minikube and minishift!  - by "the same form" I mean they have a checksum and are not tarballs.
I've also tested update.MaybeUpdateFromGithub(os.Stdout) which still works to update minishift as the default.
I had to make these small changes so that the update function is generic enough for both gofabric8 and minishift.
I wasn't sure if vendoring this package is the correct way or if we should split out the update into its  own repo but figured I'd start here as it was easier and it works.
Let me know what you think.

whoa teats!
ok so looks like the github related tests are being rate limiting. FWIW make test is working locally.

That rate limiting is a problem for real users too... That's why I want to look at tuf or similar on our own servers (dedicated perhaps?). I'll look at these changes later. Thanks for the update!

Great - thanks


color legend for categorical annoation in table

Hi,
I am using the viewer to visualize the categorical labels in the table (e.g., cell types) from steinbock-generated data. The viewer plots perfectly, even with a subsetted table. However, it is hard to figure out which labels they are on the image as they are color-coded without a legend.
I tried to look at the values shown in the viewer but without success. The indices of the table and label image start differently (0 for image, 1 for table), and it becomes even worse with the subsetted table.
I included a snapshot of the viewer with a subsetted table. My cursor was on one of the orange cells. Is there a way to figure out what these colors mean? Many thanks!

So regarding the different indices, this is because 0 is usually seen as background. As for which categories are there, I have thought of this before and I think providing hover tooltip functionality based on which column the user is coloring would benefit here. I just didn't have time yet with the recent annotation_widget work:)
So TLDR no not yet, but it is in our radar.

Bug reported also by @fjorka. We have a strategy to fix this, discussed in the linked issue #248.

Another user (@gboscagli) reported this issue so I looked better into this.
I used this code:
##
from spatialdata.datasets import blobs_annotating_element
from napari_spatialdata import Interactive
import spatialdata as sd

sdata = blobs_annotating_element('blobs_labels')

##
labels = sdata['blobs_labels']

print(sd.get_element_instances(labels))
sdata['table'].obs.index = list(map(str, [100, 101, 102, 103, 104]))
print(sdata['table'].obs)
print(labels.data.compute().tolist())

Interactive(sdata)

These are the outputs of the first 2 prints.
Index([2, 3, 4, 5, 6], dtype='int16')

           region  instance_id
100  blobs_labels            2
101  blobs_labels            3
102  blobs_labels            4
103  blobs_labels            5
104  blobs_labels            6

And in the screenshot you can see the output of the third print and the napari plot.

What I see from napari is correct:

the color of the cells is correct
the value appearing in the bottom status bar is correct (it corresponds to the instance_id)
in particular, the blobs have the background and skip the 1 value, so everything seems to be correct also when some gaps are present in the indices.

Therefore my conclusions are:

Either I failed to reproduce the bug, in such case please @gboscagli @fjorka @ynanli a short code I could run would help.
Either (I bet on this) the problem is a misunderstanding between the instance_id values and the indices of obs. If this is the case we should clarify in the docs and tutorials that we never use the obs.index values but the instance_id instead. The reasons are:

that a single AnnData object can be associated to multiple elements, so the pair of columns region_key + instance_key is needed (and AnnData does not support pandas multiindices.
the index of the table can changes often after operations are performed on the table (like a reindexing). This could potentially make it easy for the user to lose the link between the table and the shapes. We wanted something more persistent such as a separate column.

Hi @LucaMarconato,
Thanks for following up. I think our issue was rather came with the 'subsetted' table.
I also tried with the blobs data with the following code:
##
from spatialdata.datasets import blobs_annotating_element
from napari_spatialdata import Interactive
import spatialdata as sd

sdata = blobs_annotating_element('blobs_labels')

##
labels = sdata['blobs_labels']
print(sdata['table'].obs)
sdata['table'] = sdata['table'][["0", "2", "4"],]
print(sdata['table'].obs)

Interactive(sdata)

These are the outputs of the prints:
         region  instance_id
0  blobs_labels            0
1  blobs_labels            2
2  blobs_labels            3
3  blobs_labels            4
4  blobs_labels            5
5  blobs_labels            6

         region  instance_id
0  blobs_labels            0
2  blobs_labels            3
4  blobs_labels            5

And a screenshot of napari plot, where my cursor is pointing at the greenish label#3:

It reads the label value correctly and the color is correct. But the values from the table cannot be read, as shown [No Properties] in the status bar.
The status bar becomes this when i move the cursor to the background where instance_id = 0:

Of a side note, we put the ObjectNumber as the instance key, which is the cell_id from the steinbock pipeline. I hope that is correct.
table = TableModel.parse(adata_subset, region=adata_subset.obs.region.unique().tolist(), region_key="region", instance_key="ObjectNumber")

@LucaMarconato @melonora
I reproduced the shift bug using blobs dataset. Here is the code:
from spatialdata.datasets import blobs_annotating_element
from napari_spatialdata import Interactive
import spatialdata as sd

sdata = blobs_annotating_element('blobs_labels')

##
labels = sdata['blobs_labels']

print(sd.get_element_instances(labels))
obs = sdata['table'].obs
obs['strings'] = ['A', 'A', 'B', 'B', 'C', 'C']

##
sdata["table"].uns["strings_colors"] = {"A": "#FF5733", #red
                                        "B": "#3498DB", #blue
                                        "C": "#2ECC71" #green
                                        }

Interactive(sdata)

As a reference, I leave here the print of the obs df:
         region  instance_id strings
0  blobs_labels            0       A
1  blobs_labels            2       A
2  blobs_labels            3       B
3  blobs_labels            4       B
4  blobs_labels            5       C
5  blobs_labels            6       C

As you can see from the image, I have the pointer over the blob 4, as per the instance_id. Also, blob 4 is colored in blue as the manual colormap in uns suggests. However, the associated value is "C", that would be correct if we considered the index, not the instance_id.

Moreover, if I place myself over the blob 6, the value is "No Properties" because index 6 doesn't exist.

I would stress on the fact that colors are actually correct, so no problem for an overall visualization. However, to visualize the relative annotation, Napari seems to refer to the index of obs and it's kinda misleading: this problem is consistent with what @ynanli found out, but i think it's not dependent on the subsetting since I didn't perform any.

Thanks for the example, I just tried reproducing but as you can see from my initial screenshot, I don't see the last part in the status bar e.g. "; [No Properties].
@ynanli @gboscagli which versions of napari and napari-spatialdata are you using?
I am using napari==0.5.3.dev30+g3b5b3a98 and the latest main of napari-spatialdata.

Hi, no need to send the versions. I made a fix in this PR: https://github.com/scverse/napari-spatialdata/pull/320
https://github.com/napari/napari/issues/7306

CC @berombau (tagging you since you mentioned this issue once): I am merging a fix now.


Deprecate old weight items

resolves #14666

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3328667

bot fmt

bot merge


Failed to maximize Chrome browser after Chrome update

Bug report

Node Version: 6.2.2
Protractor Version: 5.1.2
Angular Version: n/a
Browser(s): Chrome
Operating System and Version n/a

After updating to Google Chrome 60.0.3112.78 this morning, Protractor fails when trying to browser.driver.manage().window().maximize()
Error:
WebDriverError: unknown error: failed to change window state to maximized, current state is normal

Hi @rdbaron
Which version of ChromeDriver are you on. I'm on 2.31 with Chrome 60 on a Mac with the latest OSX
I get this when using browser.driver.manage().window().maximize()
21:23:11.237 INFO - Creating a new session for Capabilities [{count=1, browserName=chrome, chromeOptions={args=[disable-infobars]}, maxInstances=1, shardTestFiles=true}]
Starting ChromeDriver 2.31.488774 (7e15618d1bf16df8bf0ecf2914ed1964a387ba0b) on port 17236
Only local connections are allowed.
21:23:11.929 INFO - Detected dialect: OSS
21:23:11.932 INFO - Done: [new session: Capabilities [{count=1, browserName=chrome, chromeOptions={args=[disable-infobars]}, maxInstances=1, shardTestFiles=true}]]
21:23:11.950 INFO - Executing: [script wait: 60000])
21:23:11.953 INFO - Done: [script wait: 60000]
21:23:12.269 INFO - Executing: [maximise window])
21:23:12.679 INFO - Done: [maximise window]

Works on my machine ;-). Can you please check your webdriver log?

@wswebcreation
Hmm.. I am also on chromedriver 2.31, though I am running on Debian Jessie (inside a Docker container).
The webdriver logs are on a remote CI server that I don't have access to at the moment.  I'll try to get a hold of those and post them.  For now the best I can do is give the full stack from the console output:
[chrome #01-1] WebDriverError: unknown error: failed to change window state to maximized, current state is normal
[chrome #01-1]   (Session info: chrome=60.0.3112.78)
[chrome #01-1]   (Driver info: chromedriver=2.31.488763 (092de99f48a300323ecf8c2a4e2e7cab51de5ba8),platform=Linux 3.13.0-125-generic x86_64) (WARNING: The server did not provide any stacktrace information)
[chrome #01-1] Command duration or timeout: 113 milliseconds
[chrome #01-1] Build info: version: '3.4.0', revision: 'unknown', time: 'unknown'
[chrome #01-1] System info: host: '5af0a77096d5', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '3.13.0-125-generic', java.version: '1.8.0_131'
[chrome #01-1] Driver info: org.openqa.selenium.chrome.ChromeDriver
[chrome #01-1] Capabilities [{applicationCacheEnabled=false, rotatable=false, mobileEmulationEnabled=false, networkConnectionEnabled=false, chrome={chromedriverVersion=2.31.488763 (092de99f48a300323ecf8c2a4e2e7cab51de5ba8), userDataDir=/protractor/.org.chromium.Chromium.vhLfmh}, takesHeapSnapshot=true, pageLoadStrategy=normal, databaseEnabled=false, handlesAlerts=true, hasTouchScreen=false, version=60.0.3112.78, platform=LINUX, browserConnectionEnabled=false, nativeEvents=true, acceptSslCerts=true, locationContextEnabled=true, webStorageEnabled=true, browserName=chrome, takesScreenshot=true, javascriptEnabled=true, cssSelectorsEnabled=true, setWindowRect=true, unexpectedAlertBehaviour=}]
[chrome #01-1] Session ID: 5074b1ea652da0a1170a808bf463fa8b
[chrome #01-1]     at WebDriverError (/protractor/node_modules/selenium-webdriver/lib/error.js:27:5)
[chrome #01-1]     at Object.checkLegacyResponse (/protractor/node_modules/selenium-webdriver/lib/error.js:505:15)
[chrome #01-1]     at parseHttpResponse (/protractor/node_modules/selenium-webdriver/lib/http.js:509:13)
[chrome #01-1]     at doSend.then.response (/protractor/node_modules/selenium-webdriver/lib/http.js:440:13)
[chrome #01-1]     at process._tickDomainCallback (internal/process/next_tick.js:129:7)
[chrome #01-1] From: Task: WebDriver.manage().window().maximize()
[chrome #01-1]     at schedule (/protractor/node_modules/selenium-webdriver/lib/webdriver.js:816:17)
[chrome #01-1]     at Window.maximize (/protractor/node_modules/selenium-webdriver/lib/webdriver.js:1597:25)
[chrome #01-1]     at exports.config.onPrepare (/protractor/conf.js:80:38)
[chrome #01-1]     at q_1.Promise (/protractor/node_modules/protractor/lib/util.ts:48:39)
[chrome #01-1]     at Function.promise (/protractor/node_modules/q/q.js:682:9)
[chrome #01-1]     at Object.runFilenameOrFn_ (/protractor/node_modules/protractor/lib/util.ts:39:10)
[chrome #01-1]     at plugins_.onPrepare.then (/protractor/node_modules/protractor/lib/runner.ts:103:21)
[chrome #01-1]     at _fulfilled (/protractor/node_modules/q/q.js:834:54)
[chrome #01-1]     at self.promiseDispatch.done (/protractor/node_modules/q/q.js:863:30)
[chrome #01-1]     at Promise.promise.promiseDispatch (/protractor/node_modules/q/q.js:796:13)
[chrome #01-1]     at /protractor/node_modules/q/q.js:556:49
[chrome #01-1]     at runSingle (/protractor/node_modules/q/q.js:137:13)
[chrome #01-1]     at flush (/protractor/node_modules/q/q.js:125:13)
[chrome #01-1]     at _combinedTickCallback (internal/process/next_tick.js:67:7)
[chrome #01-1]     at process._tickDomainCallback (internal/process/next_tick.js:122:9)

@rdbaron
Resetting it to Chrome 59 fixes the problem? If so then it would be more likely to be a bug in ChromeDriver in combination with your platform.
Protractor is only a wrapper around Webdriver, managing the browsers is done with webdriver-manager (separate dependency). I would "say" this is a "thirdparrty" issue and not a Protractor issue and therefore would suggest to add an issue there and not here.

@wswebcreation
I figured it was more likely an issue elsewhere, but I thought you guys might want an issue in case you'd need to update to a new version of a dependency once fixed.
Happy to close it if that's not the case though.

Chromedriver is automatically updated to the latest version with a webdriver-manager update.
So Protractor takes care of this. I do appreciate that you logged it here with your provided info.
I'm going to close it for now but please update it here if you have more info
Grtz
wswebcreation

Hello can you pls reopen the bug? I too hit this issue on docker selenium grid.
OS : - Ubuntu 16 image
chrome browser version : 60.0.3112.78
chrome webdriver 2.31
call stack :
Command duration or timeout: 108 milliseconds
Build info: version: '2.53.1', revision: 'a36b8b1', time: '2016-06-30 17:37:03'
System info: host: 'df46c666fa71', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '4.2.0-27-generic', java.version: '1.8.0_131'
Driver info: org.openqa.selenium.chrome.ChromeDriver
Capabilities [{applicationCacheEnabled=false, rotatable=false, mobileEmulationEnabled=false, networkConnectionEnabled=false, chrome={chromedriverVersion=2.31.488763 (092de99f48a300323ecf8c2a4e2e7cab51de5ba8), userDataDir=/tmp/.org.chromium.Chromium.91hHFN}, takesHeapSnapshot=true, pageLoadStrategy=normal, databaseEnabled=false, handlesAlerts=true, hasTouchScreen=false, version=60.0.3112.78, platform=LINUX, browserConnectionEnabled=false, nativeEvents=true, acceptSslCerts=true, locationContextEnabled=true, webStorageEnabled=true, browserName=chrome, takesScreenshot=true, javascriptEnabled=true, cssSelectorsEnabled=true, setWindowRect=true, unexpectedAlertBehaviour=}]
Session ID: 5c3f4c06ac71ee9f705fea89e4f52490
Build info: version: '3.4.0', revision: 'unknown', time: 'unknown'
System info: host: 'inll11306565l', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '4.4.0-83-generic', java.version: '1.8.0_131'
Driver info: driver.version: RemoteWebDriver
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
at org.openqa.selenium.remote.ErrorHandler.createThrowable(ErrorHandler.java:206)
at org.openqa.selenium.remote.ErrorHandler.throwIfResponseFailed(ErrorHandler.java:158)
at org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:678)
at org.openqa.selenium.remote.RemoteWebDriver$RemoteWebDriverOptions$RemoteWindow.maximize(RemoteWebDriver.java:945)
at org.openqa.selenium.support.events.EventFiringWebDriver$EventFiringWindow.maximize(EventFiringWebDriver.java:644)
at org.openqa.selenium.remote.server.handler.MaximizeWindow.call(MaximizeWindow.java:30)
at org.openqa.selenium.remote.server.handler.MaximizeWindow.call(MaximizeWindow.java:1)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at org.openqa.selenium.remote.server.DefaultSession$1.run(DefaultSession.java:176)
If i switch to chrome webdriver 2.26 everything works fine.

I got the same issue with Chrome 60 (chrome driver 2.30/2.31, Selenium 3.4, Ubuntu 17.04) in AT framework Codeception.
As a workaround I have to specify window size in my suite config manually, e.g. window_size: 1920 x 1080 (instead of window_size: maximize).
BTW latest Firefox has similar issue, see https://github.com/mozilla/geckodriver/issues/820

Just in case anyone else lands here -
I was able to reproduce the issue by starting Chromedriver directly (without going through webdriver-manager) so the problem almost certainly lies there.
I've logged https://bugs.chromium.org/p/chromedriver/issues/detail?id=1901

@rdbaron
Tnx for the info and the link

Same here...
org.openqa.selenium.remote.RemoteWebDriver$RemoteWebDriverOptions$RemoteWindow.maximize(RemoteWebDriver.java:892)
at BasicTest.googleExampleThatSearchesFor(BasicTest.java:22)
at BasicTest.googleCheeseExample(BasicTest.java:46)
It needs to be FIXED!

Getting the same error with MacOS High Sierra with Chrome Version 70.0.3538.67 (Official Build) (64-bit) after upgrading to ChromeDriver 2.43:
...
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: failed to change window state to maximized, current state is normal
(Session info: chrome=70.0.3538.67)
(Driver info: chromedriver=2.43.600229 (3fae4d0cda5334b4f533bede5a4787f7b832d052),platform=Mac OS X 10.13.6 x86_64)

Getting this error whenever I run the automation scripts in UI mode:
WebDriverError: unknown error: failed to change window state to maximized, current state is normal
(Session info: chrome=70.0.3538.67)
(Driver info: chromedriver=2.43.600229 (3fae4d0cda5334b4f533bede5a4787f7b832d052),platform=Mac OS X 10.13.6 x86_64)
Not getting solution to handle it.

Hi i am using OS Sierra 10.13.6 with chrome driver 2.41
and getting same error:
selenium.common.exceptions.WebDriverException: Message: unknown error: failed to change window state to maximized, current state is normal

@srujana13  Are u  maximizing the browser in ur config file also. If u r doing this in two places one in config file and  other in test file then this issue occurs. The fix for this  is to add following to ` capabilities: {
'browserName': 'chrome',
'chromeOptions': {
args: [        '--start-maximized',
'--disable-impl-side-painting'
]
}
},

@samiullah hi, i tried doing the above, im still facing the issue

@srujana13  are u using browser.manage().window.maximize() in ur OnPrepare block also??

yess, if i comment that code out, it's working

@srujana13  so the workahead (Jugaad) is that we we remove "browser.manage().window.maximize()" from our OnPrepare block and add  --start-maximized argument in ur chromeoptions. Hope this helps u . Also dnt use  "browser.manage().window.maximize()" inside ur tests also

okay thank you so much :)

I believe this issue is back in chrome #80.


Shift-tab doesn't trigger ui_focus_prev on Android

Using 3.2.0beta4 and recently began attempting to port my game to Android. I was attempting to debug a UI issue, and plugged my desktop keyboard into my phone to try something. I noticed that Shift-tab doesn't appear to trigger ui_focus_prev. Could this be related to #34405 ?
Additionally, Escape doesn't appear to trigger ui_cancel.
I realize a keyboard plugged into an Android device isn't a common setup, but it mostly worked, and was useful for debugging. Are these bindings defined on a platform-by-platform basis, or does this indicate a deeper issue with keyboard-handling on Android?
Thanks.

Looking into it, it seems like Android keyboard events don't ship any modifier details at all. Is that correct?
Might be interested in adding this, but what would be the suggested way? Should I add a bool for each modifier (I.e. shift, alt, ctrl, etc.) or something else?

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?


Add timestamps to Date API

add new parameter to all date module functions like bool timestamp with default value false that enables returning date with timestamp format instead of ISO

Hello, I'll do this one. Haven't worked in C++ in a while so we'll see how it goes. I'll return an object with the ISO and timestamp formats. For testing I suppose since everything is in ISO i'll have to make it test timestamp as well

Ok, assigned, good luck!

oops didnt mean to unassign that, dont click a button thinking it will ask you if you really want to do that

How is it going?


test: Skip flaky Cypress test

SUMMARY

The "Datapane loads view samples" test has been failing regularly without cause: based on Cypress dashboard, fail rate is 13% and flaky rate is 36%. This PR skips the test.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

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

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

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

This is indeed the #1 flaky test according to Cypress dashboard right now. I'm approving in order to unblock other PRs, but it would obviously be great to also follow up here soon with a PR that fixes the test.


Expose NamedAuthInfo

Expose NamedAuthInfo to allow programmatic construction of
Kubeconfigs.

Ah, thank you. Presumably that was the last one? I'll merge add it to the next release.

Released 0.43.0