"myapi": executable file not found in $PATH: unknown when using docker

Seems to be failing when installing and using all the defaults and then using the make init command
Error
Recreating restful_api_web_1 ... error

ERROR: for restful_api_web_1  Cannot start service web: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "myapi": executable file not found in $PATH: unknown

ERROR: for web  Cannot start service web: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "myapi": executable file not found in $PATH: unknown

Full setup and init output below:
Setup
bbolt@bbolt playground % cookiecutter https://github.com/karec/cookiecutter-flask-restful
project_name [restful_api]:
app_name [myapi]:
Select python_version:
1 - 3.8
2 - 3.7
3 - 3.6
Choose from 1, 2, 3 [1]:
tox_python_env [py38]:
Select use_celery:
1 - no
2 - yes
Choose from 1, 2 [1]:
admin_user_username [admin]:
admin_user_email<EMAIL_ADDRESS>
admin_user_password [admin]:
Select wsgi_server:
1 - none
2 - uwsgi
3 - gunicorn
Choose from 1, 2, 3 [1]:

Init
bbolt@bbolt playground % cd restful_api
bbolt@bbolt restful_api % make init
docker-compose build
Building web
Step 1/10 : FROM python:3.8
 ---> b0358f6298cd
Step 2/10 : RUN mkdir /code
 ---> Using cache
 ---> 87c9c870b25e
Step 3/10 : WORKDIR /code
 ---> Using cache
 ---> f21a622260a4
Step 4/10 : COPY requirements.txt setup.py tox.ini ./
 ---> 1706cabbf3ef
Step 5/10 : RUN pip install -U pip
 ---> Running in abc8ae00d385
Requirement already satisfied: pip in /usr/local/lib/python3.8/site-packages (20.3.3)
Removing intermediate container abc8ae00d385
 ---> 6d7debcf7641
Step 6/10 : RUN pip install -r requirements.txt
 ---> Running in 45e00103d363
Collecting marshmallow>=3
  Downloading marshmallow-3.10.0-py2.py3-none-any.whl (46 kB)
Collecting apispec-webframeworks
  Downloading apispec_webframeworks-0.5.2-py2.py3-none-any.whl (12 kB)
Collecting apispec[yaml]
  Downloading apispec-4.0.0-py2.py3-none-any.whl (26 kB)
Collecting PyYAML>=3.10
  Downloading PyYAML-5.3.1.tar.gz (269 kB)
Collecting flask
  Downloading Flask-1.1.2-py2.py3-none-any.whl (94 kB)
Collecting click>=5.1
  Downloading click-7.1.2-py2.py3-none-any.whl (82 kB)
Collecting itsdangerous>=0.24
  Downloading itsdangerous-1.1.0-py2.py3-none-any.whl (16 kB)
Collecting Jinja2>=2.10.1
  Downloading Jinja2-2.11.2-py2.py3-none-any.whl (125 kB)
Collecting MarkupSafe>=0.23
  Downloading MarkupSafe-1.1.1-cp38-cp38-manylinux1_x86_64.whl (32 kB)
Collecting Werkzeug>=0.15
  Downloading Werkzeug-1.0.1-py2.py3-none-any.whl (298 kB)
Collecting flask-jwt-extended
  Downloading Flask-JWT-Extended-3.25.0.tar.gz (31 kB)
Collecting PyJWT<2.0,>=1.6.4
  Downloading PyJWT-1.7.1-py2.py3-none-any.whl (18 kB)
Collecting flask-marshmallow
  Downloading flask_marshmallow-0.14.0-py2.py3-none-any.whl (10 kB)
Collecting six
  Downloading six-1.15.0-py2.py3-none-any.whl (10 kB)
Collecting flask-migrate
  Downloading Flask_Migrate-2.5.3-py2.py3-none-any.whl (13 kB)
Collecting flask-sqlalchemy
  Downloading Flask_SQLAlchemy-2.4.4-py2.py3-none-any.whl (17 kB)
Collecting alembic>=0.7
  Downloading alembic-1.4.3-py2.py3-none-any.whl (159 kB)
Collecting python-editor>=0.3
  Downloading python_editor-1.0.4-py3-none-any.whl (4.9 kB)
Collecting SQLAlchemy>=0.8.0
  Downloading SQLAlchemy-1.3.22-cp38-cp38-manylinux2010_x86_64.whl (1.3 MB)
Collecting flask-restful
  Downloading Flask_RESTful-0.3.8-py2.py3-none-any.whl (25 kB)
Collecting aniso8601>=0.82
  Downloading aniso8601-8.1.0-py2.py3-none-any.whl (44 kB)
Collecting marshmallow-sqlalchemy
  Downloading marshmallow_sqlalchemy-0.24.1-py2.py3-none-any.whl (18 kB)
Collecting passlib
  Downloading passlib-1.7.4-py2.py3-none-any.whl (525 kB)
Collecting python-dotenv
  Downloading python_dotenv-0.15.0-py2.py3-none-any.whl (18 kB)
Collecting tox
  Downloading tox-3.21.0-py2.py3-none-any.whl (84 kB)
Collecting filelock>=3.0.0
  Downloading filelock-3.0.12-py3-none-any.whl (7.6 kB)
Collecting packaging>=14
  Downloading packaging-20.8-py2.py3-none-any.whl (39 kB)
Collecting pluggy>=0.12.0
  Downloading pluggy-0.13.1-py2.py3-none-any.whl (18 kB)
Collecting py>=1.4.17
  Downloading py-1.10.0-py2.py3-none-any.whl (97 kB)
Collecting pyparsing>=2.0.2
  Downloading pyparsing-2.4.7-py2.py3-none-any.whl (67 kB)
Collecting toml>=0.9.4
  Downloading toml-0.10.2-py2.py3-none-any.whl (16 kB)
Collecting virtualenv!=20.0.0,!=20.0.1,!=20.0.2,!=20.0.3,!=20.0.4,!=20.0.5,!=20.0.6,!=20.0.7,>=16.0.0
  Downloading virtualenv-20.3.0-py2.py3-none-any.whl (5.7 MB)
Collecting appdirs<2,>=1.4.3
  Downloading appdirs-1.4.4-py2.py3-none-any.whl (9.6 kB)
Collecting distlib<1,>=0.3.1
  Downloading distlib-0.3.1-py2.py3-none-any.whl (335 kB)
Collecting Mako
  Downloading Mako-1.1.3-py2.py3-none-any.whl (75 kB)
Collecting python-dateutil
  Downloading python_dateutil-2.8.1-py2.py3-none-any.whl (227 kB)
Collecting pytz
  Downloading pytz-2020.5-py2.py3-none-any.whl (510 kB)
Building wheels for collected packages: PyYAML, flask-jwt-extended
  Building wheel for PyYAML (setup.py): started
  Building wheel for PyYAML (setup.py): finished with status 'done'
  Created wheel for PyYAML: filename=PyYAML-5.3.1-cp38-cp38-linux_x86_64.whl size=572488 sha256=5757dd393e674951a1bc399b335ae17843b17d8745797072a231b2452a416863
  Stored in directory: /root/.cache/pip/wheels/13/90/db/290ab3a34f2ef0b5a0f89235dc2d40fea83e77de84ed2dc05c
  Building wheel for flask-jwt-extended (setup.py): started
  Building wheel for flask-jwt-extended (setup.py): finished with status 'done'
  Created wheel for flask-jwt-extended: filename=Flask_JWT_Extended-3.25.0-py2.py3-none-any.whl size=21569 sha256=27dd3a35f6c4e6767fa4a06541c8499490d1a8489c77746d85ab54bf2a2cc89e
  Stored in directory: /root/.cache/pip/wheels/f0/53/54/ec3db050e12900460634d2f7badb842fa29af7508a5b84557d
Successfully built PyYAML flask-jwt-extended
Installing collected packages: MarkupSafe, Werkzeug, six, Jinja2, itsdangerous, click, SQLAlchemy, PyYAML, python-editor, python-dateutil, pyparsing, Mako, flask, filelock, distlib, appdirs, apispec, virtualenv, toml, pytz, PyJWT, py, pluggy, packaging, marshmallow, flask-sqlalchemy, aniso8601, alembic, tox, python-dotenv, passlib, marshmallow-sqlalchemy, flask-restful, flask-migrate, flask-marshmallow, flask-jwt-extended, apispec-webframeworks
Successfully installed Jinja2-2.11.2 Mako-1.1.3 MarkupSafe-1.1.1 PyJWT-1.7.1 PyYAML-5.3.1 SQLAlchemy-1.3.22 Werkzeug-1.0.1 alembic-1.4.3 aniso8601-8.1.0 apispec-4.0.0 apispec-webframeworks-0.5.2 appdirs-1.4.4 click-7.1.2 distlib-0.3.1 filelock-3.0.12 flask-1.1.2 flask-jwt-extended-3.25.0 flask-marshmallow-0.14.0 flask-migrate-2.5.3 flask-restful-0.3.8 flask-sqlalchemy-2.4.4 itsdangerous-1.1.0 marshmallow-3.10.0 marshmallow-sqlalchemy-0.24.1 packaging-20.8 passlib-1.7.4 pluggy-0.13.1 py-1.10.0 pyparsing-2.4.7 python-dateutil-2.8.1 python-dotenv-0.15.0 python-editor-1.0.4 pytz-2020.5 six-1.15.0 toml-0.10.2 tox-3.21.0 virtualenv-20.3.0
Removing intermediate container 45e00103d363
 ---> 30025f46750e
Step 7/10 : RUN pip install -e .
 ---> Running in f0ab77a80cb4
Obtaining file:///code
Requirement already satisfied: flask in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (1.1.2)
Requirement already satisfied: flask-sqlalchemy in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (2.4.4)
Requirement already satisfied: flask-restful in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (0.3.8)
Requirement already satisfied: flask-migrate in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (2.5.3)
Requirement already satisfied: flask-jwt-extended in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (3.25.0)
Requirement already satisfied: flask-marshmallow in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (0.14.0)
Requirement already satisfied: marshmallow-sqlalchemy in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (0.24.1)
Requirement already satisfied: python-dotenv in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (0.15.0)
Requirement already satisfied: passlib in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (1.7.4)
Requirement already satisfied: apispec[yaml] in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (4.0.0)
Requirement already satisfied: apispec-webframeworks in /usr/local/lib/python3.8/site-packages (from myapi==0.1) (0.5.2)
Requirement already satisfied: PyYAML>=3.10 in /usr/local/lib/python3.8/site-packages (from apispec[yaml]->myapi==0.1) (5.3.1)
Requirement already satisfied: click>=5.1 in /usr/local/lib/python3.8/site-packages (from flask->myapi==0.1) (7.1.2)
Requirement already satisfied: itsdangerous>=0.24 in /usr/local/lib/python3.8/site-packages (from flask->myapi==0.1) (1.1.0)
Requirement already satisfied: Jinja2>=2.10.1 in /usr/local/lib/python3.8/site-packages (from flask->myapi==0.1) (2.11.2)
Requirement already satisfied: Werkzeug>=0.15 in /usr/local/lib/python3.8/site-packages (from flask->myapi==0.1) (1.0.1)
Requirement already satisfied: MarkupSafe>=0.23 in /usr/local/lib/python3.8/site-packages (from Jinja2>=2.10.1->flask->myapi==0.1) (1.1.1)
Requirement already satisfied: PyJWT<2.0,>=1.6.4 in /usr/local/lib/python3.8/site-packages (from flask-jwt-extended->myapi==0.1) (1.7.1)
Requirement already satisfied: six in /usr/local/lib/python3.8/site-packages (from flask-jwt-extended->myapi==0.1) (1.15.0)
Requirement already satisfied: marshmallow>=2.0.0 in /usr/local/lib/python3.8/site-packages (from flask-marshmallow->myapi==0.1) (3.10.0)
Requirement already satisfied: alembic>=0.7 in /usr/local/lib/python3.8/site-packages (from flask-migrate->myapi==0.1) (1.4.3)
Requirement already satisfied: python-dateutil in /usr/local/lib/python3.8/site-packages (from alembic>=0.7->flask-migrate->myapi==0.1) (2.8.1)
Requirement already satisfied: SQLAlchemy>=1.1.0 in /usr/local/lib/python3.8/site-packages (from alembic>=0.7->flask-migrate->myapi==0.1) (1.3.22)
Requirement already satisfied: python-editor>=0.3 in /usr/local/lib/python3.8/site-packages (from alembic>=0.7->flask-migrate->myapi==0.1) (1.0.4)
Requirement already satisfied: Mako in /usr/local/lib/python3.8/site-packages (from alembic>=0.7->flask-migrate->myapi==0.1) (1.1.3)
Requirement already satisfied: aniso8601>=0.82 in /usr/local/lib/python3.8/site-packages (from flask-restful->myapi==0.1) (8.1.0)
Requirement already satisfied: pytz in /usr/local/lib/python3.8/site-packages (from flask-restful->myapi==0.1) (2020.5)
Installing collected packages: myapi
  Running setup.py develop for myapi
Successfully installed myapi
Removing intermediate container f0ab77a80cb4
 ---> 7e90d7594f1f
Step 8/10 : COPY myapi myapi/
 ---> 60bef73528f1
Step 9/10 : COPY migrations migrations/
 ---> 2e2dbb23e90d
Step 10/10 : EXPOSE 5000
 ---> Running in e557f96dd4e0
Removing intermediate container e557f96dd4e0
 ---> 2fbe405e8216

Successfully built 2fbe405e8216
Successfully tagged myapi:latest
docker-compose up -d
Recreating restful_api_web_1 ... error

ERROR: for restful_api_web_1  Cannot start service web: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "myapi": executable file not found in $PATH: unknown

ERROR: for web  Cannot start service web: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "myapi": executable file not found in $PATH: unknown
ERROR: Encountered errors while bringing up the project.
make: *** [run] Error 1

Hello @brianbolt and thanks for reporting.
This issue should now be solved with 955b0de

Hello @brianbolt and thanks for reporting.
This issue should now be solved with 955b0de

@karec Thanks for the quick fix!

@karec Thanks for the quick fix!


Allow identifiers as parameters to StyleSheet#create

Coverage increased (+0.08%) to 98.529% when pulling 887471e33493d14becee097b1a5f5d4f37bf31bf on jordanschalm:master into 3c4df47d6ba0f1ffde0ee8ffa618601955f7704e on Eight-Solutions:master.


Animation system

Ideas:

start_frame and end_frame, TODO make frame a type
curve (trait?)
called every time before a new png is rendered

struct Animation {
  start_frame: usize,
  end_frame: usize,
  curve: &'a Curve,
}

fn main() {
  // 60fps, 2s
  let animation = Animation {
     start_frame: 60,
     end_frame: 180,
     curve: todo!(),
  };
}

I think the current system from animation works fine enough for version 0.1. In the future I want to have the animation on the "path" and not on the renderer.


How are buckets un-mounted?

I cannot find any reference in the documentation regarding how to unmount a bucket.

fusermount -u /path/to/mountpoint

perfect. Thanks!

For MacOS: diskutil unmount /path/to/mountpoint

what if fusermount is not found?


Add flag to allow update buffer to read from disk

I need this in emacs when a buffer is reverted from disk. I don't get any file changed events and just need the server to have the same version of the file.

Maybe add a test to UpdateBufferFilterFacts?  It's pretty minor I know... :+1:

Thought about it ... but that would require having files on disk to test which is a bit of a pain and not worth it IMO. It's going to be tested from emacs though.
p.s. this code doesn't work :)

On second thoughts, I should totally write a test ;)

Is this in a place where it could be tested with atom?

Yeah.... it's working for me now in emacs. No idea what's up with the tests though. Not related to my changes but I'm investigating.


Add certbot container for SSL certificate creation and renewal

Creates SSL certificates for staging.aced-idp.org. Certificates last for 60 days and can be renewed 30 days after creation.
Related to issue #29 (Longterm Renewable Lets Encrypt Wildcard Certifcation)

this might need a new commit after the changes made last week


Problems with Tensorboard in Google Colab and Tensorflow version 1.x [question]

Describe the bug
I am trying to run tensorboard on google colab using your example from here.
My Full code is as follows:
%tensorflow_version 1.x
!pip install stable-baselines[mpi]==2.10.0
from stable_baselines import A2C

model = A2C('MlpPolicy', 'CartPole-v1', verbose=1, tensorboard_log="./a2c_cartpole_tensorboard/")
model.learn(total_timesteps=10000)

%load_ext tensorboard
!tensorboard --logdir ./a2c_cartpole_tensorboard/

And the output is:
W0617 19:56:14.979392<PHONE_NUMBER>91392 plugin_event_accumulator.py:294] Found more than one graph event per run, or there was a metagraph containing a graph_def, as well as one or more graph events.  Overwriting the graph with the newest event.
TensorBoard 1.15.0 at http://9e12e4148db4:6006/ (Press CTRL+C to quit)

I try navigating to http://9e12e4148db4:6006/ but get the error message "this site can not be reached"
You can see the google colab here

Found out what I was doing wrong. For anyone else interested, replace
!tensorboard --logdir ./a2c_cartpole_tensorboard/

with
%tensorboard --logdir ./a2c_cartpole_tensorboard/


Generate command uses incorrect selector

When generating a new page using the generate command, the .scss file generated contains the following class:
page-intro {

}

which would be:
.page-intro {

}

also, this may or may not be intentional, but the class is also not added to <ion-content> in the generated .html file.

Just realised each component actually does have it's own selector now, very cool, never mind me!


Fix moby Linux detection

Changed detection pipe-line to avoid broken pipe scenarios when
grep -q finishes before docker info that is still writing to the pipe.
Fixes issue #15

Hi @ivan4th I did a small fix to dind-cluster.sh and CI failed but I guess the failure is not related to my change. Could you please take a look?

@luafran thanks a lot for this PR and sorry for the delay. Would you please sign our CLA (see PR status box below) so we can merge it?
As of CI, yes, that's a flake, restarted it.

@ivan4th CLA signed but check status did not update yet...

@ivan4th @luafran I can confirm that CLA is signed, idk why status is not updated though.

Thanks again for the patch @luafran, we'll have CLA status issue sorted out I think.


[Communities] Improve overview tab content

Problem
Currently the landing tab for each community is not clear or engaging. To improve user engagement and retention, we need to work with our marketing partners to identify better messaging, looks, and call to actions. This is a research/spike ticket to identify a new approach and structure.
Acceptance Criteria

Collaborate with marketing partners to identify best practices for community landing pages (e.g. messaging, look, call to actions)
Propose a general structure for the landing page (e.g. community highlights, testimonials, call to actions, etc) with sample content for the Zero Knowledge community
Present the new strategy to relevant stakeholders & get feedback
Create tickets to complete the other pages and assign to the appropriate teams

I published the Zero Knowledge copy based on feedback from NEAR Studios. NEAR Week provided more general feedback about the platform (will create separate tickets) as opposed to this page specifically.


The dates on the contributor activity graph look wrong

Gitea version (or commit ref): v1.13.2
Git version: v2.30.0.windows.2
Operating system:

Windows 2012 R2
Download gitea-1.13.2-windows-4.0-amd64.exe from release page
Run as service

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (https://try.gitea.io/vessd?tab=activity)
[ ] No

Log gist:

Description
I think contributor's activity graph should mark the date specified in the commits, not when they are uploaded to the server.
At least I expect this behavior.

duplicate with #8178


Multicolor Semantic Highlighting with user-defined color palettes

Hi everyone! Thank you so much for your work on clangd.
Would it be possible to implement multicolor semantic highlighting? This is one of the features I enjoyed most about ccls, and it would be amazing to have it as part of clangd! It's been a huge benefit to me, making it easier to read and understand code.
I've provided an outline of the feature, an example (this feature already exists in ccls), and  I discuss concerns such as backwards compatibility (the feature is opt-in, so there should be no impact on anyone who doesn't explicitly enable it) and accessibility (the feature is opt-in, but many people find it helpful).
Please let me know if there is anything else I should provide.
Description
Semantic Highlighting wherein classes of tokens are assigned a color palette, rather than an individual color. So, for example, every instance of the same variable, type, template, or namespace has the same color, but instances of different variables, types, templates, or namespaces may have different colors from that palette.
Example
Below is an image illustrating multicolor semantic highlighting with semantic highlighting with ccls. Constants are colored shades of blue; functions shades of gold/brown; types are shades of red/purple. Namespaces are green, but noam is a different shade than json.

Backwards compatibility
This feature should be opt-in via either a clangd configuration setting or an editor configuration setting, so there are no changes for users who don't opt into the feature.
Feature Reference
This feature was implemented in ccls, which uses llvm as a backend, just as clangd does. See the documentation for that here: https://github.com/MaskRay/ccls/wiki/Visual-Studio-Code#semantic-highlight
Choice of color palette
Clangd doesn't need to provide any built-in color palettes; rather, it'd just be nice if users had the capacity to specify their own palette within either their editor's settings, or the settings for clangd itself.
Below is an example of what this specification looks like for ccls in vscode. Colors are specified in hex values.
// Predefined by package.json
"ccls.highlight.function.colors": ["#e5b124", "#927754", "#eb992c", "#e2bf8f", "#d67c17", "#88651e", "#e4b953", "#a36526", "#b28927", "#d69855"],
"ccls.highlight.type.colors": ["#e1afc3", "#d533bb", "#9b677f", "#e350b6", "#a04360", "#dd82bc", "#de3864", "#ad3f87", "#dd7a90", "#e0438a"],
"ccls.highlight.variable.colors": ["#587d87", "#26cdca", "#397797", "#57c2cc", "#306b72", "#6cbcdf", "#368896", "#3ea0d2", "#48a5af", "#7ca6b7"],

"ccls.highlight.macro.colors": ["#e79528", "#c5373d", "#e8a272", "#d84f2b", "#a67245", "#e27a33", "#9b4a31", "#b66a1e", "#e27a71", "#cf6d49"],
"ccls.highlight.namespace.colors": ["#429921", "#58c1a4", "#5ec648", "#36815b", "#83c65d", "#417b2f", "#43cc71", "#7eb769", "#58bf89", "#3e9f4a"],

An analogous specification for clangd might look like this: ("ccls" has been replaced with "clangd")
// Predefined by package.json
"clangd.highlight.function.colors": ["#e5b124", "#927754", "#eb992c", "#e2bf8f", "#d67c17", "#88651e", "#e4b953", "#a36526", "#b28927", "#d69855"],
"clangd.highlight.type.colors": ["#e1afc3", "#d533bb", "#9b677f", "#e350b6", "#a04360", "#dd82bc", "#de3864", "#ad3f87", "#dd7a90", "#e0438a"],
"clangd.highlight.variable.colors": ["#587d87", "#26cdca", "#397797", "#57c2cc", "#306b72", "#6cbcdf", "#368896", "#3ea0d2", "#48a5af", "#7ca6b7"],

"clangd.highlight.macro.colors": ["#e79528", "#c5373d", "#e8a272", "#d84f2b", "#a67245", "#e27a33", "#9b4a31", "#b66a1e", "#e27a71", "#cf6d49"],
"clangd.highlight.namespace.colors": ["#429921", "#58c1a4", "#5ec648", "#36815b", "#83c65d", "#417b2f", "#43cc71", "#7eb769", "#58bf89", "#3e9f4a"],

Accessibility
Because the feature is opt-in, there is no impact on accessibility for users who prefer higher-contrast themes or who avoid color.
I believe that having this feature will also improve accessibility for people with ADHD or Dyslexia, and many people with these conditions report that it's been beneficial to them.
Unfortunately, research in this area is sparse, but I've collected references in relation to accessibility and semantic highlighting.

https://githubmemory.com/repo/CompEng0001/CodingandDyslexia (lists IDEs which support semantic highlighting)
Comment on this article: https://www.linusakesson.net/programming/syntaxhighlighting/

Anonymous
Sun 21-Oct-2012 00:37
being a dyslexic syntax highlighting is one of the best tings that have happened to me, if anything i need way more

Comment by ZeroCool2u from this forum thread:

ZeroCool2u on March 24, 2019 [–]
Something that also helps cut down on these types of bugs for me is Semantic Highlighting. I've gotten my whole team to use it.
It's great, because you don't have to read the var name, you just have to see if there's any var in a function that doesn't have the same color as anything else and they tend to stick out like a sore thumb.
Not sure if that's the best explanation, but if you're using any JetBrains IDE you can try it by going to Settings and searching for Semantic Proximity and hitting the check mark. Easier to show than explain.

Historical notes
I originally heard about this feature in this article by Evan Brooks.

One thing to note here is that CCLS uses a custom protocol between its client and server for semantic highlighting, whereas clangd uses semanticTokens which was standardized in Language Server Protocol version 3.16.
So, to support this feature in clangd, we would need to take one of the following approaches:

Build this functionality as an extension on top of semanticTokens. There are two sub-possibilities here:

Do it purely on the client side, without any protocol changes. The client doesn't have access to semantic information, so it would need to treat all identifiers (with a given token kind) which are textually same as being the same (even if they are e.g. in two different scopes). This may not be a big deal.
Extend the semanticTokens protocol with the necessary additional information. This seems tricky due to the fact that semanticTokens uses a packed binary representation over the wire to save space, so it's not just a matter of e.g. adding a new field which clients that don't support this feature can ignore.

Propose this feature to be part of the Language Server Protocol itself, and then clangd can implement the updated protocol.

To add some context to this I had sent a patch upstream for this sort of feature which worked within the LSP framework. While I was using clangd I maintained this patch over the years and it worked well. May need some massaging to bring it back up but rainbow highlighting was working as in ccls.
https://reviews.llvm.org/D87669

As someone who uses clangd for non-IDE purposes (generating highlighted code on a website) I think this feature is out of scope of clangd. IMO it should either be moved to LSP design/specification or just written outside clangd. You can compute color variations by issuing additional LSP calls that query object usages (so you can differentiate entities with the same name but in different scopes) - that's what I'm currently doing.


fix(ci): ephemeral env build

SUMMARY
fixes ephemeral envs, broken by: https://github.com/apache/superset/pull/26473
I'm taking the ephemeral build out of the docker/pull_request workflow trigger and placing it on issue_comment.
issue_comment triggers at the issue level and uses the default branch latest SHA, this makes it hard to properly test CI workflow changes but on the other hand makes this path resilient to any possible issues caused by PRs.
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 seems stuck on the docker-build process... should we close/reopen the PR to kick-start it, or is it an actual problem with the PR?

This seems stuck on the docker-build process... should we close/reopen the PR to kick-start it, or is it an actual problem with the PR?

it's because of a defined required workflow that I'm removing here, this one: https://github.com/apache/superset/pull/26787
that fixes asf.yaml should fix this also


Adding rcluster host metrics

Ref: PR #31
Adding rcluster host metrics

i'm going to close this one since ext-rhost already have summary / golden metrics, but let me know if there is anything missing and I'll reopen.


Capacity website: Add freeflow page for each farmer when he creates his farm

Auto create a freeflow page when a farmer sets up his farm or ask to add freeflow page url if freeflowpage already exists

➤ Weynand Kuijpers commented:
Need to record the freeflowpages in the JSON structure: Can we add a parameter:
{
"farmer_id": "threefoldfarmer",
"location": {
"city": "Hulshout",
"continent": "Europe",
"country": "Belgium",
"latitude": 51.060386316691016,
"longitude": 4.82351303100586,
"freeflowpage":"https://freeflowpages.com/s/green-edge-cloud/"
},
"node_id": "ecf4bbd9ea2c",
"os_version": "master 584ba6fe3b2891eb62c233f0c606315c97595ff9",
"parameters": [
"support"
],
"reserved_resources": {
"cru": 0.0,
"hru": 0.0,
"mru": 0.0,
"sru": 0.0
},
"robot_address": "http://<IP_ADDRESS>:6600",
"total_resources": {
"cru": 40.0,
"hru": 61477.23,
"mru": 251.91,
"sru": 9313.5
},
"updated": "Mon, 15 Apr 2019 15:20:59 GMT",
"uptime": 2695868,
"used_resources": {
"cru": 20.23,
"hru": 0.0,
"mru": 3.01,
"sru": 0.0
}
}

➤ Christophe de Carvalho commented:
Can we have a bit of history of why this is required, use case, etc...
Also what is the timeline for this ?


wp2static is no longer available

As of Aug 10, the plugin is no more available for download which is linked from this tutorial: https://github.com/cloudflare/cloudflare-docs/tree/production/products/workers/src/content/tutorials/deploy-a-static-wordpress-site

This may be a good alternative. https://wordpress.org/plugins/simply-static/

@volkanger @deadlypants1973 While it's true that WP2Static is no longer in the official WordPress.org plugin repository, it appears to still be actively in development on Github and has a stand-alone website with a Cloudflare Workers Add-on.
Taken together, I don't see any evidence that this plugin has been abandoned. Instead, the Cloudflare document needs to be updated to install the plugin from Github rather than the WP repo.
@leonstafford is the primary developer and may want to be aware of this issue.

Thanks @mrwweb!
Yes, still very much in development, including the Workers addon.

@leonstafford hey leon! would you be interested in helping us update this tutorial with the new installation method via github?
alternatively, would love to hear if you've been working on anything with cf pages :) since we wrote this tutorial, we've released pages as a better/more-managed solutions to workers sites, it would be neat to get a pages-centric tutorial up that uses wp2static, if you're interested!

Hey @signalnerve! Definitely! Pages flow should be easier, I'll try to dive into that soon and add instructions re GH usage in the meantime.


cmd/vet: tests require Perl for errchk and fail with latest Perl

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8.1 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/genehack/go"
GORACE=""
GOROOT="/opt/go"
GOTOOLDIR="/opt/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build831712468=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
Grabbed the source tarball. Grabbed go-1.4 binary tarball and expanded under $HOME. Built 1.8.1 source per instructions (cd src; ./all.bash). Got test suite failure due to a change in regex syntax that became a fatal error in 5.24. (I'm running perl-5.25.11, currently; once I submit this I will replicate with 5.24, 5.22, and (hopefully!) not replicate with 5.20.)
What did you expect to see?
Fully passing test suite.
What did you see instead?
ok      cmd/vendor/golang.org/x/arch/x86/x86asm 0.195s
--- FAIL: TestVet (1.40s)
    --- FAIL: TestVet/5 (0.24s)
        vet_test.go:114: files: ["testdata/copylock_func.go" "testdata/nilfunc.go" "testdata/unsafeptr.go"]
        vet_test.go:163: vet output:
                Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/call of fntab.0. copies lock value: struct{ <-- HERE lock sync.Mutex} contains sync.Mutex/ at /opt/go/test/errchk line 128.
        vet_test.go:164: exit status 1

Update: I read the perldelta wrong: unescaped left-brace is only deprecated in 5.24. It will become fatal in 5.26 (which is expected out soon, within a few months). So, currently, this bug will only show up for folks running Perls from the 5.25.x development releases.

The actual use of perl is in the script test/errchk.

@ianlancetaylor, we have a Go version of errchk already in test/run.go. (hence my longer title about one view of the bug being that we depend on Perl)

That said, it might be just changing:
    $regexp = "$`$file:$n$'";

to:
    $regexp = "\Q$`\E$file:$n$'";

Sorry, our subject line editing collided.  I never saw yours.

Quite frankly, this entire test suite is confused whether the error lines it's dealing with are plain strings or regexes.
This particular error is caused by this line. This is clearly meant to be matched stringwise, as are many other strings (that work fine most of the time because the meta-character . matches itself as literal character .).
Do you really need these to be regexes? If not I would recommend just quotemeta()ing it early (this may involve unquoting a few cases that are currently hand-quoted, for example because they contain a *).
If you do need it, I would recommend consistently quoting all error strings.
One possible intermediate solution is to add a new
  foo(1) // ERROR /some \d+ regexp/

syntax that will explicitly enable the regular expression syntax.
All of this you really want to fix regardless of what perl is doing.

In general the ERROR lines that appear in the files in the test directory are always intended to be regular expressions.  I am less familiar with the use in vet, but if vet wants to match the way that the test directory works then the ERROR lines there should also always be regexps.  Thanks for identifying the line; probably the right solution is to backslash-quote the . characters.

Thanks for identifying the line; probably the right solution is to backslash-quote the . characters.

The problem here isn't the . characters, but the {s

My attempts to install a dev version of perl failed. Can someone confirm that CL 40950 fixes the immediate problem? Thanks.

Here's a Debian jessie environment with Perl 5.25. You can build it and docker run it with some -v flags to bind mount your $GOROOT and $GOROOT_BOOTSTRAP to reproduce.
FROM buildpack-deps
MAINTAINER Peter Martini<EMAIL_ADDRESS>
RUN apt-get update \
    && apt-get install -y curl procps \
        && rm -fr /var/lib/apt/lists/*

RUN mkdir /usr/src/perl
WORKDIR /usr/src/perl

RUN curl -SL https://cpan.metacpan.org/authors/id/S/SH/SHAY/perl-5.25.3.tar.bz2 -o perl-5.25.3.tar.bz2 \
        && tar --strip-components=1 -xjf perl-5.25.3.tar.bz2 -C /usr/src/perl \
            && rm perl-5.25.3.tar.bz2 \
                    && ./Configure -Duse64bitall -Duseshrplib -Dusedevel  -des \
                        && make -j$(nproc) \
                            && TEST_JOBS=$(nproc) make test_harness \
                                && make install \
                                    && cd /usr/src \
                                        && curl -LO https://raw.githubusercontent.com/miyagawa/cpanminus/master/cpanm \
                                            && chmod +x cpanm \
                                                && ./cpanm App::cpanminus \
                                                    && rm -fr ./cpanm /root/.cpanm /usr/src/perl /tmp/*

WORKDIR /root

FYI Latest go18 (go1.8.4-0-gf5bcb9b8fe) fails to bootstrap on Debian 9 because of this bug:
ok      cmd/vendor/golang.org/x/arch/x86/x86asm 0.140s
--- FAIL: TestVet (0.84s)
    --- FAIL: TestVet/1 (0.06s)
        vet_test.go:114: files: ["testdata/atomic.go" "testdata/copylock_func.go" "testdata/lostcancel.go" "testdata/rangeloop.go" "testdata/unsafeptr.go"]
        vet_test.go:163: vet output:
                Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/call of f copies lock value: struct{ <-- HERE lock sync.Mutex} contains sync.Mutex/ at /home/kirr/src/tools/go/go18/test/errchk line 128.
        vet_test.go:164: exit status 1
FAIL
FAIL    cmd/vet 1.581s

Go18 should be still supported according to release poilcy, thus it is better to backport 2e45310 there.
Latest Go14 bootstraps ok.

FYI Latest go18 (go1.8.4-0-gf5bcb9b8fe) fails to bootstrap on Debian 9 because of this bug:
...

@rsc you probably overlooked this while backporting patches to go1.8.5


Fix & Remove Delete leauge from dashboard #48

Your PR is failing our CI test due to linter errors. Make sure you do npm run test in the root directory to make sure that you are passing the linter and tests.

@makkoli  I'll have a look.

I fixed it on the master branch. Pull master into your working branch and the the tests will pass.

Will Do
On Wed, Aug 23, 2017 at 4:43 AM, makkoli<EMAIL_ADDRESS>wrote:

I fixed it on the master branch. Pull master into your working branch and
the the tests will pass.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/league-for-good/pull/56#issuecomment-324177023,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIJJKcneeMYhK-yUBH7AXfNkO7bwk-Qdks5sa2CegaJpZM4O8nEQ
.

@makkoli  Hope this works

Looks good, thanks bud.


[ScrollArea] asChild on Viewport is not working correctly

Bug report
When using asChild on Viewport, the functionality is lost.
This is probably due to the fact that there's an extra node around children.
I think the solution is to use Slottable probably.
Reproducible example
https://codesandbox.io/s/scrollarea-viewport-aschild-bug-y46g5q?file=/App.js
Suggested solution

Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)

React
n/a

Browser

Assistive tech

Node
n/a

npm/yarn

Operating System

I think we can remove the additional div when the asChild prop is used and rely on implementation in userland. It will allow adding scroll-snap-type wrappers inside, for example.

Any updates on this issue?

I just ran into this issue because I'm using https://www.framer.com/motion/ to animate some elements inside the viewport of a ScrollArea, and in order to allow Framer Motion's automatic "layout" animations within scrollable containers, the scrollable container (in this case, the ScrollArea's Viewport) needs to become a <motion.div> instead of a plain .

@benoitgrelard I have just encountered this issue. This is due to the fact that when the viewport, which has overflow: scroll, is merged with the children wrapper, which has display: table, the resulting element has both display: table and overflow: scroll.
The problem here is that overflow does not work with display: table elements, so scrolling doesn't work.

Hey @finkrer we may do this, it's related to #926


feat: remove ZenDesk widget

High Level Overview of Change
I don't know when this was used but it is never activated on any of our deployed instances.

This should probably be refactor: instead of feat: according to this


iGenomes and gene_biotype

Running with Zebrafish iGenome (--genome danRer10) fails. This is due to the GTF version being incompatible with the second featureCounts (biotype) step. gene_biotype is not in the provided GTF.
I assume this is an iGenome issue (as the pipelines is great), but maybe this part can be skipped if the GTF does not contain biotype information. I just changed my script to --fc_group_features_type gene_id. This allowed the pipeline to finish.
Thanks

Hi @chriswyatt1 ! Thanks for spotting that and finding the appropriate solution. I recently added all of the latest available UCSC genomes to the AWS iGenomes config file. Mainly to give users the choice of which type of assembly/annotation they would like to use. As you rightly pointed out this is one of the places where the UCSC annotation could be better i.e. having a gene_biotype field but it's probably not a deal breaker in my opinion.
As an automated solution, maybe we can just grep for this field in the gtf to set the appropriate parameter :thinking:

You can also run with --skipBiotypeQC to get around this part...
https://github.com/nf-core/rnaseq/blob/master/docs/usage.md#--skipbiotypeqc
There were quite some improvements added but as @drpatelh already mentioned it's unfortunately not very easy to make all references work equally well with the pipeline. We should try to add something for UCSC-genomes I agree, but until then you can either skip or use a different id.

Perfect, thanks @drpatelh @apeltzer. Yeah, must be tricky with such nonconforming references. I didn't see that flag (that's a better fix). Cheers

No problem - thanks for the feedback, cheers! If something else pops up, please open a new issue and/or re-open!


Device Login Track-2 Migration

Description
This pull request aims to facilitate the migration process from the existing AADL library to the MSAL framework, specifically for device login functionalities.
Changes

The existing flow for the AADL framework has been removed regarding Device Login with the MSAL framework.

This should be made a draft till we finalize the token persistence

Open question - how will we test this? @adreed-msft any ideas?

Open question - how will we test this? @adreed-msft any ideas?
Look into how Identity tests this.

The idea for testing was that we'd have semi-automated tests in the E2E suite. Stick the suite behind a build tag, something like "manualinteraction", run azcopy login at the suite setup, run logout at suite teardown.
I'd already given a little guidance in a 1:1 with Pranav.

Can we close this PR in favor of the other one?


Add more options for configuring compilers

We added a few more options for configuring compilers in the mbed-os
tools, so I'm adding those options to the readme here. The new options
are:

Setting env variables
PATH detection

Further, I updated the order of the subsections to reflect their
priority. Things earlier in the list configure the compiler in preference
to things later in the list.
Finally, there is one edit for consistency. Previously we had a command
line blah --global blah blah and in the text it referred to -G.
I changed the command line to be consistent
Reviews/Requests for edits:

@AnotherButler
@screamerbg

Thanks for aligning functionality and docs @theotherjimmy +1.
See @anotherbutler's comments.

@AnotherButler I changed the passive voice you pointed out to active. I also tried to clarify what a base directory is.
@bridadan I fixed the nit. I added IAR.
Could you both review again?

@AnotherButler I re-wrote the patch again to include you second set of suggestions. Could you review again?
I think we should go over this whole file together soon and edit the whole thing.

@AnotherButler Can you please review?

@screamerbg This looks good to me. theotherjimmy and I put together another PR with some grammar changes for the other sections that this PR did not change, so we've been over the whole document. It looks good.

Thanks @theotherjimmy @AnotherButler


Grafanadashboard yaml contains invalid json

When trying to import the grafana json manually in to my grafana 7 instance i get error saying invalid json.
I get the same error when putting the pure json in a json reader.
I will see if I have time to try to find it on my own but if you already have a working edition in grafana it's probably enough to just export it out again.

If not applying with helm, you'll have to un-template the legend override names in two spots:
https://github.com/giantswarm/starboard-exporter/blob/7a98fe982e82be07dc86db080be98e0b113df915/helm/starboard-exporter/templates/grafana-dashboard.yaml#L471
https://github.com/giantswarm/starboard-exporter/blob/7a98fe982e82be07dc86db080be98e0b113df915/helm/starboard-exporter/templates/grafana-dashboard.yaml#L552

Ahh of course, thanks allot @stone-z


Possible to implement roaring bitmap to ssdb?

Possible to implement this?
https://github.com/aviggiano/redis-roaring

We also need this feature!
That will greatly decrease the storage size!


adding User.jobs()

This retrieves the jobs for a user. Note that this only retrieves the processing jobs, AKA no moi jobs. However, AFAIK only the analysis is currently using moi jobs and @josenavas is working on removing that as part of the refactor. I'll do another PR to add the GUI part of this functionality. This should be reviewed and merged before the other PR.

Changes Unknown when pulling 83d0b2bfd59438925e236c4286278c40b21364d2 on antgonza:user-jobs into ** on biocore:master**.

Changes Unknown when pulling b6ab4ed06aba4a115a305b7612593047b58ba267 on antgonza:user-jobs into ** on biocore:master**.

Turns out that you can remove processingjobs via the processingworkflow method, which is much cleaner than adding a processingjob remove. However, the processing job didn't have a method to return its processingworkflow. Just added it.

Changes Unknown when pulling 6b3c25242e77d4a02d181e60c4937145088e1174 on antgonza:user-jobs into ** on biocore:master**.

Changes Unknown when pulling c2582e98a4e9cf73230dac3fde66fdca28e3d0e3 on antgonza:user-jobs into ** on biocore:master**.

Changes Unknown when pulling 88367bf283c06e0ce0893e123a1ef577ccec15c0 on antgonza:user-jobs into ** on biocore:master**.


running CE loss does not work

Hi I am running given example in README
thanks
Traceback (most recent call last):
File "main_ce.py", line 333, in
main()
File "main_ce.py", line 302, in main
loss, train_acc = train(train_loader, model, criterion, optimizer, epoch, opt)
File "main_ce.py", line 214, in train
acc1, acc5 = accuracy(output, labels, topk=(1, 5))
File "/remote/idiap.svm/user.active/rkarimi/dev/internship/SupContrast/util.py", line 48, in accuracy
correct_k = correct[:k].view(-1).float().sum(0, keepdim=True)
RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

try change correct_k = correct[:k].view(-1).float().sum(0, keepdim=True) to correct_k = correct[:k].reshape(-1).float().sum(0, keepdim=True) in utils.py line 48


I encountered an error when I tried to use CookieManager IOS

Swift Compiler Error (Xcode): Cannot call value of non-function type 'String?' /Users/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_inappwebview-5.5.0+2/ios/Classes/MyCookieManager.swift:259:43
`[✓] Flutter (Channel stable, 3.3.0, on macOS 12.4 21F79 darwin-x64, locale en-TN)
• Flutter version 3.3.0 on channel stable at /Users/oussamaridene/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ffccd96b62 (7 weeks ago), 2022-08-29 17:28:57 -0700
• Engine revision 5e9e0e0aa8
• Dart version 2.18.0
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/oussamaridene/Library/Android/sdk
• Platform android-33, build-tools 32.1.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 13F17a
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
[✓] VS Code (version 1.67.2)
• VS Code at /Users/oussamaridene/Downloads/Visual Studio Code.app/Contents
• Flutter extension version 3.50.0
[✓] Connected device (3 available)
• iPhone 8 (mobile) • a7164a43aefbbe5c91d0a4220c00261905f0f327 • ios            • iOS 15.5 19F77
• macOS (desktop)   • macos                                    • darwin-x64     • macOS 12.4 21F79 darwin-x64
• Chrome (web)      • chrome                                   • web-javascript • Google Chrome 106.0.5249.119
[✓] HTTP Host Availability
• All required HTTP hosts are available`

duplicated #1369, you need to update Xcode version to 14


How can we include volunteers into our teams platform safely

Hi there Git Hub World
We are pondering since covid 19 how to use microsoft teams for our volunteers. we need need to ensure they are safe and we can create a channel so that we can share project documents have group supervisions and individual supervisions and help volunteers connect amongst themselves safely...... am i in any way in the right place to ponder find an answer??? stay safe all best

Hi LouisaH1, You are in right place for MS Teams related queries.
Teams provide many useful features which are specially useful in the current situation where remote work is the only option available for many industries.
To know about feature please check this documentation.
Please check How to use Teams.
Please let me know if you have any specific question.

Please feel free to reopen if you have further query on this


[Fixes #4616] Filter caching is too aggressive

@rynowak @Eilon

:watch: test changes look good

:up: :date:

:shipit: looks good!

d6a906800812899c8515409fd9e89077edfa8c55


Ported xen-api-sdk to dune and fixed warnings.

I don't understand the Travis failure
[ERROR] The sources of the following couldn't be obtained, aborting:
          - xapi-stdext-monadic.1.0.0

From what I can the corresponding Opam file points to the right tarball:

It is usually a network issue when you see that error. On osx can be an issue with an old openssl and opam if you are using opam = 2.0.0

Travis now passed - @mseri was right.

@lindig regarding the alignment, the above changes have been ocp-indented.

ocp-indent does not affect the right hand-side; see here for example:
https://github.com/xapi-project/xen-api-sdk/pull/154/files#diff-49b380ccf20e5df18f6545257b853e4aR1516

Aaah, sorry, I read the comment quickly and the right-hand bit didn;t register with me. Ok, it's not important, it can be rectified in future alongside other code changes.

another option would be ocamlformat, which reformats the whole AST, but the changes can be quite big and not limited to just the lines you changed.


Replace AWSClient.shutdown with version using Result

Replace
AWSClient.shutdown(queue: DispatchQueue = .global(), _ callback: @escaping (Error?) -> Void)

with
AWSClient.shutdown(queue: DispatchQueue = .global(), _ callback: @escaping (Result<Void, Error>) -> Void)

Makes it easier to pass result onto EventLoopPromise. Code can be structured as follows
awsClient.shutdown() { result in
    promise.completeWith(result)
}

This is version breaking so can't do just now. Can't even add new function as that would mean people would get ambiguous use errors.

Next release will be async/await lets waive this


remove dependence on controller.onEvent

This was causing problems with our own scene handling.

Just something to be aware of... I was setting controller button repeat/delay in app.ts: https://github.com/microsoft/kojac/blob/e47cfdaaacfdb33bb63ab265d3a684a369898309/app.ts#L19
If a repeat delay is desired, we might need to implement that ourselves.


Allow for optional selection of SSH identity when connecting

We store an identifier for ssh identities, so we should be able to use them when creating a connection instead of just trying all identities. There are usually limits for the number of tries.
https://github.com/Xyna-Factory/xyna-factory/blob/971b7224e7018a8482955193721c1514bb19b5b0/modules/xact/connection/ssh/sharedlib/SSHJUtils/src/xact/ssh/IdentityStorableRepository.java#L362-L368
We also should order keys by an optional priority to be able to filter them so we can make a key-rollover easier.

c.f. DHICB-15654


Fix lint

Summary
pylint 3.2.0 https://github.com/pylint-dev/pylint/releases/tag/v3.2.0 released on May 14 causes some lint errors.
This PR fixes them.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 6 (33.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 92.954%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_optimization/translators/gurobipy.py
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
4512

Relevant Lines:
4854

💛  - Coveralls


fix typos

The issue tracker is used for bug reporting purposes ONLY whereas feature request needs to follow the RIP process. To avoid unnecessary duplication, please check whether there is a previous issue before filing a new one.
It is recommended to start a discussion thread in the mailing lists or github discussions in cases of discussing your deployment plan, API clarification, and other non-bug-reporting issues.
We welcome any friendly suggestions, bug fixes, collaboration, and other improvements.
Please ensure that your bug report is clear and self-contained. Otherwise, it would take additional rounds of communication, thus more time, to understand the problem itself.
Generally, fixing an issue goes through the following steps:

Understand the issue reported;
Reproduce the unexpected behavior locally;
Perform root cause analysis to identify the underlying problem;
Create test cases to cover the identified problem;
Work out a solution to rectify the behavior and make the newly created test cases pass;
Make a pull request and go through peer review;

As a result, it would be very helpful yet challenging if you could provide an isolated project reproducing your reported issue. Anyway, please ensure your issue report is informative enough for the community to pick up. At a minimum, include the following hints:
BUG REPORT

Please describe the issue you observed:

Please tell us about your environment:

Other information (e.g. detailed explanation, logs, related issues, suggestions on how to fix, etc):

FEATURE REQUEST

Please describe the feature you are requesting.

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks involved, use -[] for each sub-task and create a corresponding issue to map to the sub-task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

Hello @hardyfish ,
I noticed that you submit a little bit of pr about misspellings, a lot of code optimization pr.
A small suggestion is to include these spelling mistakes in code optimization and combine them into a pr, which will reduce the amount of work required by reviewer.
Thank you for contributing so much to the community :-)

Thanks for your advice. I'll handle it.


Added support dotnet core 5 for CommonProtos

We are faced with the problem that we have long gone from dotnet standard, and everywhere we use dotnet core 5

.NET 5 supports .NET Standard 2.0, which there's already a target for. There is no need for this.

Package Aer.Platform.GrpcImports1.0.2080-feature-pl-366 is not compatible with netstandard2.0 (.NETStandard,Version=v2.0). Package Aer.Platform.GrpcImports.GoG1.0.2080-feature-pl-366 supports: net5.0 (.NETCoreApp,Ver ... : net5.0 (.NETCoreApp,Version=v5.0)


python311Packages.pydexcom: 0.2.3 -> 0.3.1

Changelog: https://github.com/gagebenne/pydexcom/releases/tag/0.3.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 254396 run on x86_64-linux 1

1 package failed to build:

home-assistant-component-tests.dexcom

4 packages built:

python310Packages.pydexcom
python310Packages.pydexcom.dist
python311Packages.pydexcom
python311Packages.pydexcom.dist

Update is part of #251878.


Warning when the number of replicas are decreased

Print out some warning if the number of replicas from the static clustermap is less than the number of replicas in helix.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (15e7dfa) 72.24% compared to head (22d03e9) 19.64%.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2607       +/-   ##
=============================================
- Coverage     72.24%   19.64%   -52.61%
+ Complexity    11407     2789     -8618
=============================================
  Files           805      805
  Lines         65097    65099        +2
  Branches       7936     7937        +1
=============================================
- Hits          47030    12788    -34242
- Misses        15484    51129    +35645
+ Partials       2583     1182     -1401

Files
Coverage Δ

...ub/ambry/clustermap/HelixBootstrapUpgradeUtil.java
72.94% <100.00%> (-0.23%)
:arrow_down:

... and 594 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


T_TRIS.O hangs

This is likely due to a wrong emulation of IODAT. It wants to read the EEPROM.

I have to extend the lua script to overwrite EEPROM settings.

Fixed after changing eeprom properties in menu


Suggestion: for rotation or translation fixed

change of view
instead of thinking about the cubes as structure, think about a mash of edges and node (group theory?). Each node is the center of a cube and each edge is the next bordering cube. In the following ideas only the nodes are used to identify a cube and in the translation fixed idea the edges are possible paths.

Rotation fixed
For a rotation fixed storage, it would be an idea to store the location in polar coordinates, because the radius would always be the same without the need to know the rotation. Important is only which point you use as center. Otherwise it gets pretty complicated to get unique radii for different cubes. All radii always give the same formation of cubes independent of rotation and the same hash as well.

Translation fixed
For a translation fixed storage it would be like a turtle game. Start with the turtle at some corner cube and tell it the move to the next cube. Important is, that cubes, which are only neighbour of one other cube, are the start or the end or, if both not possible, some predefined path is used. to the next cube. This way it is not important where the combination of cubes is. because the path is always the same and the hash of it as well.

Also a question: is there a limitation for structures? Like is a cube formation like a 3x3x3 cube with the center cube missing valid? Or is a filled structure needed?

for your question, yes, i think its valid, all structures are valid as long as each and every cube is touching some other cube(at least 1) with a face

Also an idea: just store the radius and the direction. like a vector and prioritize alpha direction over beta direction. so you get standardized structure. then you have only 6 rotations to make by just adding 90*n degrees to alpha and/or beta. You have six directions you can rotate the cubes. Only the center is important again. I would say the center of mass would be the best center, because it is independent of the coordinate axes and rotations.

I had a similiar idea. I have some experience with theoretical chemistry, which often deals with translation and** rotation free coordinate systems. One way this is usually solved in chemistry is a z-matrix. This is a way of defining your coordinate system based on the actual points of interest in your coordinate system.
I haven't put much thought into the actual data structure to put this in memory so let's just go with a nx3-matrix, where n is the number of cubes, for the sake of argument. Here's how I'd adapt it to the cubes problem:

Every row of our matrix contains one cube with three coordinates. Switching the rows does not affect the structure.
The first cube has always the coordinates (0,0,0).
The second cube has always the coordinates (1,0,0). This definition is of course arbitrary, but it defines one dimension of our cube-structure-internal coordinate system. Translating or rotating the structure in the external cartesian coordinate won't change this. Switching columns also does not affect the shape, because we're changing the definition of our coordinate system in the same way.
The first cube orthogonal to the first two cubes is (0,1,0). This limitation eliminates some duplicate notations that represent the same structure. For example: you could define an L-shape as

0 0 0          0 0 0
1 0 0   or   1 0 0
2 0 0          2 0 0
2 1 0          0 1 0

. Both yield the same shape.
This also causes the first cube to always be a corner, if  the structure is more than one dimensional, but that does not prevents us from defining all possible structures, because we define the first cube arbitrarily.
5. Cubes with non-zero values in the third column are orthogonal to the first the previously defined plane. The first orthogonal cube does not have to be (0,0,1), because this would actually prevent us from generating all possible structures, e.g. structures where all cubes have less than three neighbours.
I think it should be relatively easy to generate all possible structures systematically, if you build them row by row and only insert valid rows (rows defining a cube attached to an already existing cube). A valid row can be generated from an already existing row by copying it and adding/subtracting 1 from one of the three columns. @SscSPs, this ensures all cubes are touching each other. This might get trickier for larger structures, because you'd have to check for colliding cubes. Haven't thought about that part yet.
@Playit3110 The origin of this coordinate system is not the center of mass and that's a good thing, because we don't know the center of mass before generating the cube. So we'd have to generate the structure, find the center of mass and then define our rotation & translation free coordinate system.
One thing to keep in mind: This coordinate system still distinguishes between enantiomers. Enantiomers are two structures, that are mirror images of each other. They might look the same at first glance, but you can't transform one into the other by rotating it.
I'm not sure about meso-forms, though. Haven't looked into that yet. Meso-forms are enantiomers, that have a plane of reflection inside the structure destroying the mirror-image-property. These structures can be transformed into each other by rotation and are actually the same structure. Not sure, if this coordinate system would allow non-unique definitions of these.
I'm not 100% sure that this is defines all possible structures uniquely, but I was already in bed and about to sleep when I started coming up with this and I had to write this down so... Maybe we need to check that as well? Or just make sure we create only unique structures by generating them systematically. I'm not sure.
One last idea I had was how to hash the coordinates. For hashing the structure we can take the columnwise sumes and interpret them as digits of a single number.  This would match identical structures even when rows are switched. This fells like a to easy solution, but I was not able to come up with examples that hash collisions of valid structures. Which does not necessarily mean there are none, but I planned on going to bed like 3 hours ago before I fell into the youtube-rabit-hole, so that's all I have so far.
Good night and thanks for the great video @mikepound! Always a good day when your uploading!

I wish you also a good night. Only the hashing has the collision of sums, because 1 + 0 + 1 = 1 + 1 + 0 = 0 + 1 + 1 so it would make different shapes the same.

I think I've been taking an approach similar to @AlreadyTakenJonas, which I think, in the 2D case, produces a representation of the polyomino that is the same regardless of rotation or translation (but not reflection).
Represent the squares to points in space. For example, for an L-tetromino, imagine the point is the center of the square, and represent it as the points (0, 0), (0, 1), (0, 2), and (1, 0).
Identify the center of the smallest circle that encompasses those points. For any set of points, there is exactly one smallest circle. In the case of the L-tetromino above, the center of the circle would be at (1, 1.5) with a radius of around 1.1.
Identify the vector from the center of the smallest circle to the nearest point. In the case of the L-tetromino above, it would be a vector from the center of the circle (1, 1.5) to the square at (0, 1).
Represent each square as a distance from the center of the smallest circle and the angle from that vector. In the case of the L-tetromino, it would be (.5, 0 deg), (1.1, 63 deg), (1.1, 117 deg), (1.1, 297 degrees).
Represent the polyomino as the set of the representations of the squares. No matter how you rotate or translate the L-tetromino, it should have the same representation.
There are some issues:

There may be more than one point nearest to the center of the smallest circle. In that case, I think you could use the center of the arc defined by those points.

It doesn't handle reflections. That said, I wonder if taking the clockwise angle from the vector might not give it to you; the L-tetromino reflection might be (.5, 0 deg), (1.1 63 deg), (1.1, 243 deg), (1.1, 297 deg).

I haven't figured out if it would work in 3D. I think you'd need to pick a second vector (maybe the second nearest point?) to define a plane, then define the cubes in terms of the distance from the center, the angle from the first vector in the plane, and the distance from the plane.

I think I've been taking an approach similar to @AlreadyTakenJonas, which I think, in the 2D case, produces a representation of the polyomino that is the same regardless of rotation or translation (but not reflection).
Represent the squares to points in space. For example, for an L-tetromino, imagine the point is the center of the square, and represent it as the points (0, 0), (0, 1), (0, 2), and (1, 0).
Identify the center of the smallest circle that encompasses those points. For any set of points, there is exactly one smallest circle. In the case of the L-tetromino above, the center of the circle would be at (1, 1.5) with a radius of around 1.1.
Identify the vector from the center of the smallest circle to the nearest point. In the case of the L-tetromino above, it would be a vector from the center of the circle (1, 1.5) to the square at (0, 1).
Represent each square as a distance from the center of the smallest circle and the angle from that vector. In the case of the L-tetromino, it would be (.5, 0 deg), (1.1, 63 deg), (1.1, 117 deg), (1.1, 297 degrees).
Represent the polyomino as the set of the representations of the squares. No matter how you rotate or translate the L-tetromino, it should have the same representation.
There are some issues:
* There may be more than one point nearest to the center of the smallest circle. In that case, I think you could use the center of the arc defined by those points.

* It doesn't handle reflections. That said, I wonder if taking the clockwise angle from the vector might not give it to you; the L-tetromino reflection might be (.5, 0 deg), (1.1 63 deg), (1.1, 243 deg), (1.1, 297 deg).

* I haven't figured out if it would work in 3D. I think you'd need to pick a second vector (maybe the second nearest point?) to define a plane, then define the cubes in terms of the distance from the center, the angle from the first vector in the plane, and the distance from the plane.

EDIT: After I implemented the "smallest circle" algorithm (lol, of course), I realized that because of the limitations on the points (centers of the polyominoes) and the limitations on the rotations (some multiple of 90 degrees), it's simply going to be the center of the rectangle that bounds the points, which I suspect is simpler to compute.

That sounds like a good idea, although you need to keep gimbal lock in 3D in mind. You can't define unique coordinates in 3D space via two angles and a radius, because you lose a degree of freedom for certain rotation angles. But it should be possible by doing something similiar to Arvo's fast random rotation matrices, he defines one angle and one planes of reflection. He substitutes one rotation by a reflection at two perpendicular planes (the defined plane and the plane perpendicular to it); two reflections give you essentially a rotation (see householder transformation), but avoids the gimbal lock problem. Took me a week to find this out, when I had issues with gimbal lock without even knowing it...

I wish you also a good night. Only the hashing has the collision of sums, because 1 + 0 + 1 = 1 + 1 + 0 = 0 + 1 + 1 so it would make different shapes the same.
My question @AlreadyTakenJonas is, what is the first cube and how can it be defined independently?

The first cube has always the coordinates 0 0 0, because we define the coordinates of a give cube based on the cubes defines previously. That's what I meant by that.

Ok, but what is the first cube.
For example we have:
0 1
1 1

Which one of the three is the first one?
a) bottom left
b) bottom right
c) top right
I understand that the first cube has coordinates (0, 0, 0), and (0, 0, 0) is the first cube, but it doesn't define which one is the reference cube (0, 0, 0). I hope I could clear up my question.

I wish you also a good night. Only the hashing has the collision of sums, because 1 + 0 + 1 = 1 + 1 + 0 = 0 + 1 + 1 so it would make different shapes the same.
My question @AlreadyTakenJonas is, what is the first cube and how can it be defined independently?

The first cube has always the coordinates 0 0 0, because we define the coordinates of a give cube based on the cubes defines previously. That's what I meant by that.

Ok, but what is the first cube. For example we have:
0 1
1 1

Which one of the three is the first one? a) bottom left b) bottom right c) top right
I understand that the first cube has coordinates (0, 0, 0), and (0, 0, 0) is the first cube, but it doesn't define which one is the reference cube (0, 0, 0). I hope I could clear up my question.

If you give me a whole structure, you can pick any one to be the first. This will yield different representations for the same structure. But I think you can generate unique representations, when you generate the structures in my proposed coordinate system from scratch and completely ignore the cartesian space.


ocpp.exceptions.NotImplementedError: NotImplementedError: No handler for 'StartTransaction' registered

Greeting to you all.
I'm working with your library, using many other frameworks and libraries as foundation (sanic and websockets).
While everything works great, I stumbled upon this kind of error (debug logs enabled):
cp_2: receive message [2,"iLKqs1F83pjyAyhIwKA8EbVe3XMIHSVWo17s","StartTransaction",{"connectorId":2,"idTag":"B4A63CDF","timestamp":"2021-07-25T14:53:14Z","meterStart":0,"reservationId":0}]
[2021-07-25 14:53:14 +0200] [20823] [ERROR] Exception occurred while handling uri: 'ws://<IP_ADDRESS>:8888/ws/cp_2'
Traceback (most recent call last):
  File "/home/dinko/Projects/ocpp-backend/.direnv/python-3.9.6/lib/python3.9/site-packages/ocpp/charge_point.py", line 162, in _handle_call
    handlers = self.route_map[msg.action]
KeyError: 'StartTransaction'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dinko/Projects/ocpp-backend/.direnv/python-3.9.6/lib/python3.9/site-packages/sanic/app.py", line 821, in _websocket_handler
    await fut
  File "/home/dinko/Projects/ocpp-backend/app/ws/feed.py", line 103, in feed
    await cp.start()
  File "/home/dinko/Projects/ocpp-backend/.direnv/python-3.9.6/lib/python3.9/site-packages/ocpp/charge_point.py", line 126, in start
    await self.route_message(message)
  File "/home/dinko/Projects/ocpp-backend/.direnv/python-3.9.6/lib/python3.9/site-packages/ocpp/charge_point.py", line 144, in route_message
    await self._handle_call(msg)
  File "/home/dinko/Projects/ocpp-backend/.direnv/python-3.9.6/lib/python3.9/site-packages/ocpp/charge_point.py", line 164, in _handle_call
    raise NotImplementedError(f"No handler for '{msg.action}' "
ocpp.exceptions.NotImplementedError: NotImplementedError: No handler for 'StartTransaction' registered., {}

I'm not sure why I'm getting this error, because I have registered a handlers for each action in similar way:
class ChargePointWSHandler(ChargePoint):
    """Handler for each Charging Point.

    Handling is enabled by `@on` decorator which contains
    each action with it's proper response.
    """

    @on(Action.BootNotification)
    async def on_boot_notification(self, **kwargs):

        # Handling etc.

        return call_result.BootNotificationPayload(
            current_time=datetime.utcnow().isoformat(),
            interval=10,
            status='Accepted'
        )

    @on(Action.Heartbeat)
    async def on_heartbeat(self, **kwargs):

        # Handling

        return call_result.HeartbeatPayload(
            current_time=datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%S') + "Z"
        )

    @on(Action.StartTransaction)
    async def on_start_transaction_payload(self, **kwargs):

        return call_result.StartTransactionPayload(
            transaction_id=random.randint(1, 10000),
            id_tag_info={"status":"expired"}
        )

I have tried changing from @on(Action.StartTransaction) to @on("StartTransaction") because that also works with other, for example BootNotification and Heartbeat but for StartTransaction and StopTransaction, nothing currently works.
All help is appreciated.

Hello @dinko-pehar ,
I'm unfortunately unable to reproduce the issue. I used the following code:
import asyncio
import logging
import random
from datetime import datetime
import websockets
from ocpp.routing import on
from ocpp.v16 import ChargePoint, call_result
from ocpp.v16.enums import Action

class ChargePointWSHandler(ChargePoint):
    """Handler for each Charging Point.

    Handling is enabled by `@on` decorator which contains
    each action with it's proper response.
    """

    @on(Action.BootNotification)
    async def on_boot_notification(self, **kwargs):

        # Handling etc.

        return call_result.BootNotificationPayload(
            current_time=datetime.utcnow().isoformat(), interval=10, status="Accepted"
        )

    @on(Action.Heartbeat)
    async def on_heartbeat(self, **kwargs):

        # Handling

        return call_result.HeartbeatPayload(
            current_time=datetime.utcnow().strftime("%Y-%m-%dT%H:%M:%S") + "Z"
        )

    @on(Action.StartTransaction)
    async def on_start_transaction_payload(self, **kwargs):

        return call_result.StartTransactionPayload(
            transaction_id=random.randint(1, 10000), id_tag_info={"status": "Expired"}
        )

async def on_connect(websocket, path):
    """For every new charge point that connects, create a ChargePoint
    instance and start listening for messages.
    """
    try:
        requested_protocols = websocket.request_headers["Sec-WebSocket-Protocol"]
    except KeyError:
        logging.error("Client hasn't requested any Subprotocol. Closing Connection")
        return await websocket.close()
    if websocket.subprotocol:
        logging.info("Protocols Matched: %s", websocket.subprotocol)
    else:
        # In the websockets lib if no subprotocols are supported by the
        # client and the server, it proceeds without a subprotocol,
        # so we have to manually close the connection.
        logging.warning(
            "Protocols Mismatched | Expected Subprotocols: %s,"
            " but client supports  %s | Closing connection",
            websocket.available_subprotocols,
            requested_protocols,
        )
        return await websocket.close()

    charge_point_id = path.strip("/")
    cp = ChargePointWSHandler(charge_point_id, websocket)

    await cp.start()

async def main():
    server = await websockets.serve(
        on_connect, "<IP_ADDRESS>", 9000, subprotocols=["ocpp1.6"]
    )

    logging.info("Server Started listening to new connections...")
    await server.wait_closed()

if __name__ == "__main__":
    try:
        # asyncio.run() is used when running this example with Python 3.7 and
        # higher.
        asyncio.run(main())
    except AttributeError:
        # For Python 3.6 a bit more code is required to run the main() task on
        # an event loop.
        loop = asyncio.get_event_loop()
        loop.run_until_complete(main())
        loop.close()

import asyncio
import logging
from datetime import datetime

try:
    import websockets
except ModuleNotFoundError:
    print("This example relies on the 'websockets' package.")
    print("Please install it by running: ")
    print()
    print(" $ pip install websockets")
    import sys

    sys.exit(1)

from ocpp.v16 import call
from ocpp.v16 import ChargePoint as cp
from ocpp.v16.enums import RegistrationStatus

logging.basicConfig(level=logging.INFO)

class ChargePoint(cp):
    async def send_boot_notification(self):
        request = call.BootNotificationPayload(
            charge_point_model="Optimus", charge_point_vendor="The Mobility House"
        )

        response = await self.call(request)

        if response.status == RegistrationStatus.accepted:
            print("Connected to central system.")

    async def start_transaction(self):
        request = call.StartTransactionPayload(
            id_tag="1",
            connector_id=1,
            meter_start=2,
            timestamp=datetime.utcnow().isoformat(),
        )

        _response = await self.call(request)

async def main():
    async with websockets.connect(
        "ws://localhost:9000/CP_1", subprotocols=["ocpp1.6"]
    ) as ws:

        cp = ChargePoint("CP_1", ws)

        await asyncio.gather(
            cp.start(), cp.send_boot_notification(), cp.start_transaction()
        )

if __name__ == "__main__":
    try:
        # asyncio.run() is used when running this example with Python 3.7 and
        # higher.
        asyncio.run(main())
    except AttributeError:
        # For Python 3.6 a bit more code is required to run the main() task on
        # an event loop.
        loop = asyncio.get_event_loop()
        loop.run_until_complete(main())
        loop.close()

I didn't provide whole example, thinking that something was wrong with StartTransaction. But actually I registered same method with different decorators due to copy paste:
...
@on(Action.StartTransaction)
    async def on_start_transaction_payload(self, **kwargs):

        return call_result.StartTransactionPayload(
            transaction_id=random.randint(1, 10000),
            id_tag_info={"status":"expired"}
        )
@on(Action.StopTransaction)
    async def on_start_transaction_payload(self, **kwargs):

        return call_result.StopTransactionPayload(
            transaction_id=random.randint(1, 10000),
            id_tag_info={"status":"expired"}
        )

I didn't understand at first, than I saw that I redefined same method twice :disappointed: .

Thank you for your time and complete working example :rocket: .

Great that the issue is solved!


Remove trailing whitespace

see #914

Coverage remained the same at 99.907% when pulling 0b0d69cb313c303db2d423b1f443df84b03586c8 on arstrube:trailing_whitespace into 6700c8fe6595c8b4d5c0a6fb8ccdf80e0a4dd33b on cpputest:master.


adding WIC.py script and IWIC.pdf

adding WIC.py script and IWIC.pdf

I think the tests are failing because it is merging from another fork. @jordenk does that seem right?

I think the tests are failing because it is merging from another fork. @jordenk does that seem right?

I can't seem to reply to this as a thread. This is correct. The fork probably doesn't have the github environment variables set. @jrinks Can you make a branch from the main repo instead of forking the project? That will make CI easier. And it makes the commit history easier to read.
Branches in a repo are for adding features. Forking is used when the project will be maintained separately, or when long-term experimentation is desired.

Incorporated in #27


Migrate from JSCS to ESLint

Need
As a developer
I want to have a good linter enabled for my project
So that I can improve and fix my coding style faster.

Problem
JSCS has merged with ESLint, thus becoming deprecated. The project needs to migrate to ESLint.
Deliverables

[ ] Enable ESLint and fix style accordingly.

@flavius-tirnacop-hs @valentin-radulescu-hs ESLint integration is ready.
I have disabled the no-console rule because console is used to log errors when the user enters invalid input in the editor, and also in some tests.
Are you ok with leaving it disabled, or do you have another suggestion?
Also, please review the code.

LGTM
Let's enable no-console if possible, we shouldn't rely on it in tests. Maybe disable it for editor if you think it's useful there.

LGTM


using span tag  to prevent history change

Coverage remained the same at 58.824% when pulling 325d512f2809fdcff08a5d8ce5a6e26c3d54a2af on ram-you:patch-2 into 9c1d5b17cf8ad937325bcc362bf95aa929c67d8c on nicolasbeauvais:master.

The only problem still with "whatsapp" with "type" different from 'popup'

Coverage remained the same at 58.824% when pulling 6f5a95e7d2ad16028e581be771fac36275029b33 on ram-you:patch-2 into 9c1d5b17cf8ad937325bcc362bf95aa929c67d8c on nicolasbeauvais:master.

Coverage remained the same at 58.824% when pulling 17544fa3d6b504b7061331f84882cc238dd21d01 on ram-you:patch-2 into dc86f68b96b587efe3aa3968acde4c92ee5d50ed on nicolasbeauvais:master.

Thank you for your contribution :+1:,
a new version as been tagged with your proposed changes v2.1.0

Sorry I updated only vue-social-sharing.common.js. Made a new PR


Exposing heartbeat metric, fixed a few typos

Exposed heartbeat metric, corrected some typos in log output.

/retest

/test images

/retest

/retest

/lgtm

/hold cancel

Per DMS: "API rate limits are 1500 API requests per 5 minute fixed window"
Option to increase it if we hit it.  With this I don't expect we'll hit the limit with this change.

/approve


Introducing Mocha update to v 6.1.3

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

To high risk at this stage in the project to upgrade Mocha, therefore closing the PR.


De-duplicate and improve PrintArch functions

See https://github.com/KhronosGroup/OpenCL-CTS/pull/552

Proposed PR against cl12_trunk: https://github.com/KhronosGroup/OpenCL-CTS/pull/596

New PR proposed for master #601

Solved by #601


Use the evm_watchdog service for running updates

This will eliminate the deadlock caused by a worker running yum update which tries to shut down the server which waits for the worker to exit.
This change will allow the server to exit cleanly and be restarted when the update is complete.
Goes with https://github.com/ManageIQ/manageiq-appliance/pull/73
@bdunne @jrafanie please review

:+1: LGTM

@gtanzillo Should this be a blocker?

@Fryguy @gtanzillo if you are considering whether this is a blocker, then we can safely assume that this should be marked darga/yes?

LGTM

@jrafanie @gtanzillo @carbonin is there a BZ for this?

I don't know if there's a BZ. @carbonin might have a somewhat related one but it may not be for this specific issue or it may have been already worked around in another way.  I'll let him update this when he returns.

I'm sure I could find a few that would have been fixed by this, but I wasn't working on a specific BZ when I made the change.


mask or alpha channel support

First of all -- great work! keep on working on the project, we need more tools like this one.
I normally "clean" my dataset before training a splat by removing background and only keep pixels of an object I'm training.
Resulting dataset is in png format with alpha channel.
My approach works great with Postshot, but training wont work with brush, it looks like it only does one epoch, creating initial splat properties and than wont continue training without any error message or any other notification.
Can you comment on alpha channel support or may be mask support for training (really helps to focus process on pixels of the actual object you train, rather than background and save tone of time as there are no need to do cleaning after training).
BTW any plans to improve training speed?

     Running `target\debug\brush_bin.exe`
thread 'async-std/runtime' panicked at .cargo\git\checkouts\burn-178c6829f420dae1\9eb2578\crates\burn-tensor\src\tensor\data.rs:72:9:
assertion `left == right` failed: Shape [1920, 1080, 4] is invalid for input of size 6220800
  left: 8294400
 right: 6220800

image_to_tensor in brush_train\src\image.rs needs to be tweaked a bit:
pub fn image_to_tensor<B: Backend>(image: &DynamicImage, device: &B::Device) -> Tensor<B, 3> {
    let (w, h) = (image.width(), image.height());
    let num_channels = image.color().channel_count();

    let data = match num_channels {
        3 => image.to_rgb32f().into_vec(),
        4 => image.to_rgba32f().into_vec(),
        _ => {
            println!("Unsupported number of channels: {}. Converting to RGB.", num_channels);
            image.to_rgb32f().into_vec() // Convert to RGB as a fallback
        }
    };

    let tensor_data = TensorData::new(data, [h as usize, w as usize, num_channels as usize]);
    Tensor::from_data(tensor_data, device)
}

but it still seems to be ignoring alpha...
and I frequently get crashes:
> cargo run
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.90s
     Running `target\debug\brush_bin.exe`
error: process didn't exit successfully: `target\debug\brush_bin.exe` (exit code: 0xc0000094, STATUS_INTEGER_DIVIDE_BY_ZERO)

Lmk if that's working!

Did a quick test, it works now!
Need to do more testing but at first glance its good.
One thing I've noticed though is that dataset image view now always keeps square aspect ratio and weirdly ruins photo view proportions

@grgkopanas That's how it used to work! But imo you're still throwing information in that case. Say for the synthetic blender scenes with a white background, white gaussians would appear which for one viewpoint explain some of the background, which isn't ideal. There's more info in the fact that the alpha should be 0 there.
Gsplat used to always composite a background and add custom gradients for that, but now also has an option to render without compositing a background, makes more sense to me.
I maybe should add an option for random backgrounds again, but, that could be done as a normal tensor op rather than compositing in the custom kernel, shouldn't be super expensive.
@dvoidus Excellent! And woops yeah I see what you mean, fixed now.

fixed now
tested, confirm
@ArthurBrussee thanks! feel free to close this one or keep for further discussion about alpha channel training


Draw divider for list header/footer views

Dividers for header/footer view is drawn in ListView by default. I made a patch to imitate this drawing using pattern already being used in the library; making use of WrapperView. Header/footer view is now wrapped by WrapperView and managed with a little more care.
WrapperViewList.java:112-121
As each divider in WrapperView is drawn by the view below, the last divider at the bottom has to be drawn differently. (fyi: divider at the bottom is only shown when items of the ListView doesn't fill all the ListView's space. Refer to the screenshots in this SO question)
Further improvement would be to implement android:header(footer)dividersenabled, giving on/off functionality.
If you would merge please review the code throughly. Though I tested several cases I may have not caught all the edge cases. I'd be glad to take any feedback.

+1 Why has this not been merged?

+1


Gravity binary should be included in downloads

Currently, our downloads include only tele and tsh binaries. They should also include gravity binary.
Things to consider:

[ ] Open-source downloads.
[ ] Enterprise downloads.

I was wondering about this. :laughing:


[TwigComponent] Ignore twig comments when prelexing

Q
A

Bug fix?
yes

New feature?
no

Tickets
Fix #838

License
MIT

This PR ignores {# twig comments content. Is this really the only way to comment in Twig 🤔?

Super fast - thank you Matheo!

something is not right still after this pr:


GRN module, Algorithm

add grn module version 0.1

These commits are merged manually by @junhouhui into work branch, thx for the excellent work.


input condition/testbed

Int he latest version of the documentation, we are suggesting to name the file containting the tesbed configuration named condition.yaml instead of testbed.yaml.
The rationale is that the file may contain configuration that not only affect the testbed (as equipment), but also instruction given to the subject.
I could change it easily within your repository here, but I am wondering if you could use similar terminology for the file your system generates (the code which generate your current file cond_23_testbed.yaml.
A side question:   the filename is named cond_23_testbed.yaml. Doe sit mean that you have 22 other conditions before?
We also provide more suggestion for the datafile naming in here. We would probably have to look at this as well later on.

I updated the scripts and my system now generates condition_##.yaml instead.
Regarding you side question: we have in total 105 unique conditions. Which of the two namings should we use: condition_001.yaml or condition_01.yaml.
Thanks for the provided information regarding the datafile naming.

OK, I will adjust the repository content in that line, and get back to you for accepting that change.
If you can have 105 unique condition, then the filename should use 3 digit for the number. But are people likely to use in a singe experiment the 105 conditions?

Ok, I'll adjust that. The 105 conditions were all conditions that we did with one participant within one experiment. However, future studies might only select a specific set of conditions to test.

I don't think you have to change anything in the code folder. Do you think the contrary?
The test data stored are fine as they are.
Eventually, we should add in the readme some indications on the data collected structure.


Error when using class properties

I wanted to use class properties like the static styles from lit-element. I serve my app with polymer cli and it's throwing the following error.
Error { SyntaxError: This experimental syntax requires enabling the parser plugin: 'classProperties' (16:16)
    at Parser.raise (/project/node_modules/babylon/lib/index.js:776:15)
    at Parser.expectPlugin (/project/node_modules/babylon/lib/index.js:2084:18)
    at Parser.parseClassProperty (/project/node_modules/babylon/lib/index.js:4920:12)
    at Parser.pushClassProperty (/project/node_modules/babylon/lib/index.js:4884:30)
    at Parser.parseClassMemberWithIsStatic (/project/node_modules/babylon/lib/index.js:4817:14)
    at Parser.parseClassMember (/project/node_modules/babylon/lib/index.js:4754:10)
    at Parser.parseClassBody (/project/node_modules/babylon/lib/index.js:4709:12)
    at Parser.parseClass (/project/node_modules/babylon/lib/index.js:4659:10)
    at Parser.parseExportDefaultExpression (/project/node_modules/babylon/lib/index.js:5018:19)
    at Parser.parseExport (/project/node_modules/babylon/lib/index.js:4983:31)
  pos: 458,
  loc: Position { line: 16, column: 16 },
  missingPlugin: [ 'classProperties' ] }`

I think that the babel config that polyserve uses must be missing this plugin.

Would this file be missing here the classProperties plugin?
https://github.com/Polymer/tools/blob/dc391830fec747bb6164109157cd3a24302e346a/packages/analyzer/src/javascript/javascript-parser.ts#L37

I got the same error. whats up with this? pwa with polymer-cli

defining static variables is the wrong syntax in ES6 and ES5
direct use class.properties is fine,
declaring a property in a class, compile will give an error

To expand on richardlyc's comment - I had the same error when not using Typescript.  So I changed:
static styles = css...

to
static get styles() { return css...

and it worked great.

defining static variables is the wrong syntax in ES6 and ES5
direct use class.properties is fine,
declaring a property in a class, compile will give an error

That was my point, it is not an error and it should be enabled in the config there.

To expand on richardlyc's comment - I had the same error when not using Typescript. So I changed:
static styles = css...

to
static get styles() { return css...

and it worked great.

The reason I opened the issue is so they enable class properties in polymer cli

To expand on richardlyc's comment - I had the same error when not using Typescript. So I changed:
static styles = css...

to
static get styles() { return css...

and it worked great.

The reason I opened the issue is so they enable class properties in polyserve

Certainly, I agree.  I included this information since a search on the problem brought me here.  Until it is resolved I hoped it might help someone down the line with the problem.


Fix to "Quick example: Counter", not working

Removed property "counter" from mapProps (is supplied by ...rest).

D'oh, thanks :D


Add: Simplest ever explanation for Tower of Hanoi

Problem
How to think (IMPORTANT)
Idea
Solution

I would like to work on this issue under GSSoC'23


Switch from 1.1.0 to 1.4.0 of the OPA Kafka plugin for most Kafka ver…

…sions.

Have you tested that the new combinations work? Can't seem to find an official version support matrix on their GH repo or website.

Have you tested that the new combinations work? Can't seem to find an official version support matrix on their GH repo or website.

From what i see here https://github.com/anderseknert/opa-kafka-plugin any version >= 1.0.0 was tested with kafka 2.7 or higher. Not sure what we do with 2.6 then?

Ah, fair point, missed that bit.


Tests in the root package are too slow

They should either be made faster (parallel tests, or other fixes), or the slowest should be skipped on testing.Short().
At the moment, this is far from ideal:
$ go test -short
OK: 197 passed
PASS
ok      gopkg.in/src-d/go-git.v4        58.933s

I would have expected 10 seconds at most. Ideally, 2-3 seconds or less.

Here's another - the tests aren't standard Go, so I can't use out of the box features like test's -v and -run. Is there a reason for that?

We want to change it to use asserty, as any other modern development at source{d}. But the current test suite is built on top of gocheck, this decision was made 3yrs ago, when was the default test framework for us, now we have a bunch of tests to change it, so its a lot of work for a little benefit.

I honestly don't mind whatever test framework is used. But in my opinion, the end result is that regular Go tests are run, so that go test flags like -v and -run are useful.

Yep, but sadly this, flags are "ignored" by gocheck. Since the whole testsuite runs under the same T


Prompt to save unsaved file

If my current python file is unsaved, I'd like to be prompted to save it before continuing with the tests. Otherwise, I get a false passed after I change something and forget to save.

@apnewberry   this should appear on melpa soon:
1b42927e0a83e3818cf99f08b8c77eb1eaeb03b1
please try it out and let me know.