Ability to clear the Terminal

Description of the new feature/enhancement
VSCode Terminal or iTerm allow you to entirely clear the buffer of a tab.
For example, in VSCode, it's possible to create a keybinding to clear the terminal this way:
{
  "key": "ctrl+k",
  "command": "workbench.action.terminal.clear"
}

This is still not possible in Terminal.

I think this is probably a duplicate of issue #1882.

I think this is probably a duplicate of issue #1882.

@j4james Indeed. Thanks!


Update ax88179 to 2.8.0_20160912.

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

*Date is now added to the version number
*Upper Mac version upped to 10.12
*Sha256 hash updated

I'm getting an error running this currently. Need to do some additional work to the caskfile.

I'm getting an error running this currently.

What error?

I believe it's related to having the date in the version. Need to pull out the date when extracting the internal package I think.
Error:   Command failed to execute!

  ==> Failed command:
  ["/usr/bin/ditto", "--bom", "/var/folders/ll/qmlttbzs7vvbfcm4rxvy1z9rffykgb/T/20160929-66894-11u1gj8.bom", "--", "/private/tmp/dmg.pl6ozH", "#<Pathname:/usr/local/Caskroom/ax88179/2.8.0_20160912>"]

  ==> Standard Output of failed command:

  ==> Standard Error of failed command:
  ditto: /private/tmp/dmg.pl6ozH/.: Invalid argument

  ==> Exit status of failed command:
  #<Process::Status: pid 67039 exit 1>

Seems to be the same issue as in https://github.com/caskroom/homebrew-cask/issues/21279. After that the cask was removed and added back shortly after that.

So after more digging around and looking at the referenced issue, I think I understand the problem but I don't know exactly how to fix it. Is there a way for me to modify the dmg pre ditto, or modify how the ditto command is run? This is probably getting beyond my knowledge for any kind of quick fix.

is there a way for me to modify the dmg pre ditto

No, but you can use container type: :naked and a preflight block to manually alter/extract the DMG.

@reitermarkus I had to do quite the nasty hack to mount the disk image. Nor sure how tolerable my solution is.

It seems like the hack used in https://github.com/caskroom/homebrew-cask/blob/849f3eec3dbf5f4d66755c86c3f2f9ade0977d42/Casks/decitime.rb#L14-L18 also works, so I'd suggest to use this instead. Also use the same comment, but link to this PR.

I did in fact use that for inspiration. Unfortunately that particular variation didn't work for me. Mounting this particular disk image once as readwrite does not fix the problem. I actually have no idea what the exact problem with this disk image is, so I have no idea how to fix it and still use the proper container stanza. I tried various modification (xattrs, removing trashes, changing perms) on the disk image as well as your type: :naked recommendation with no success.

Mounting this disk image once as readwrite does not fix the problem.

Weird, it works for me. What error are you getting?

This caskfile still results in the same error I mentioned before:
(truncated)
==> Executing: ["/usr/bin/hdiutil", "imageinfo", "/usr/local/Caskroom/ax88179/2.8.0_20160912/AX88179_178A_Macintosh_10.6_to_10.12_Driver_Installer_v2.8.0_20160912/AX88179_178A.dmg"]
==> Extracting nested container AX88179_178A.dmg
==> Executing: ["/usr/bin/hdiutil", "mount", "-plist", "-nobrowse", "-readonly", "-noidme", "-mountrandom", "/tmp", "/usr/local/Caskroom/ax88179/2.8.0_20160912/AX88179_178A_Macintosh_10.6_to_10.12_Driver_Installer_v2.8.0_20160912/AX88179_178A.dmg"]
==> Executing: ["/usr/bin/mkbom", "-s", "-i", "/var/folders/ll/qmlttbzs7vvbfcm4rxvy1z9rffykgb/T/20160930-94310-1xla4pz.list", "--", "/var/folders/ll/qmlttbzs7vvbfcm4rxvy1z9rffykgb/T/20160930-94310-17a1e46.bom"]
==> Executing: ["/usr/bin/ditto", "--bom", "/var/folders/ll/qmlttbzs7vvbfcm4rxvy1z9rffykgb/T/20160930-94310-17a1e46.bom", "--", "/private/tmp/dmg.c0iNfu", "/usr/local/Caskroom/ax88179/2.8.0_20160912"]
==> ditto: /private/tmp/dmg.c0iNfu/.: Invalid argument
==> Executing: ["/usr/sbin/diskutil", "eject", "/private/tmp/dmg.c0iNfu"]
==> Purging files for version 2.8.0_20160912 of Cask ax88179
Error:   Command failed to execute!

  ==> Failed command:
  ["/usr/bin/ditto", "--bom", "/var/folders/ll/qmlttbzs7vvbfcm4rxvy1z9rffykgb/T/20160930-94310-17a1e46.bom", "--", "/private/tmp/dmg.c0iNfu", "#<Pathname:/usr/local/Caskroom/ax88179/2.8.0_20160912>"]

  ==> Standard Output of failed command:

  ==> Standard Error of failed command:
  ditto: /private/tmp/dmg.c0iNfu/.: Invalid argument

  ==> Exit status of failed command:
  #<Process::Status: pid 94559 exit 1>
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:60:in `assert_success'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:31:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:9:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/system_command.rb:13:in `run!'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:75:in `block (2 levels) in extract_mount'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/tempfile.rb:324:in `open'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:70:in `block in extract_mount'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/tempfile.rb:324:in `open'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:67:in `extract_mount'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:63:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:63:in `extract_mounts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/container/dmg.rb:25:in `extract'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/nested_container.rb:23:in `extract'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/nested_container.rb:7:in `block in install_phase'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/set.rb:232:in `each_key'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/set.rb:232:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/artifact/nested_container.rb:7:in `install_phase'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:135:in `block in install_artifacts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:132:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:132:in `install_artifacts'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:75:in `install'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:25:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:107:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:146:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:23:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:105:in `<main>'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:151:in `exit'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:151:in `rescue in process'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:141:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:23:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:105:in `<main>'

Ok, leave it as is then, but put the eject in an ensure block:
    begin
      ...
    ensure
      system '/usr/bin/hdiutil', 'eject', '-quiet', dmg_mount
    end

Hopefully that's correct. I haven't used begin blocks in ruby much. I assume doing the glob to find the dmg outside of the block makes sense.

We don't even need the glob, as we know that it's named staged_path.join(Utils.basename(version), 'AX88179_178A.dmg').

Is ok now? I think ideally I'd deal with the version and name silliness a little better but not sure there's a good way to do that with this package.

I've made all requested changes.

Thanks!


Added POC for Token based AUthentication

Needs to be raised against https://github.com/eBayClassifiedsGroup/orlo/tree/hackathon_h1_2016 :)


Refreshing database connection pool every 30 days, using AWS, Sqlalchemy and FastAPI

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import os
import time
from uuid import uuid4

import uvicorn
from fastapi import FastAPI, Request, Depends
from fastapi.encoders import jsonable_encoder
from api.routes.info import info, health
from api.utils.db import init_db_sqlalchemy, get_session, fetch

app = FastAPI(
)
app.include_router(info, prefix="/info")
app.include_router(health, prefix="/health")

@app.middleware("http")
async def add_logging_and_process_time(request: Request, call_next):
    start_time = time.time()
    request_id = str(uuid4().hex)
    response = await call_next(request)
    process_time = time.time() - start_time
    process_time = str(round(process_time * 1000))
    response.headers["X-Process-Time-MS"] = process_time
    log_msg = f"request_id={request_id} service=pbgraph-translation-svc url={request.url} host={request.client.host} " \
              f"port={request.client.port} processing_time_ms={process_time} env={os.environ.get('APP_ENV')} " \
              f"version=v1 pid={os.getpid()} region={os.environ.get('REGION')} "
    logger.info(log_msg)
    return response

@app.on_event('startup')
def startup():
    init_db_sqlalchemy()

@app.get('/getDatabaseInfo')
def get_db_data_example(db: Session = Depends(get_session)):
    try:
        records = fetch(db, DATABASE_QUERY).all()
        return jsonable_encoder(records)
    except Exception as err:
        logger.exception(f"function_name=getDatabaseInfo error={err}")

if __name__ == '__main__':
    uvicorn.run(app, host="<IP_ADDRESS>", port=8050)

Description
Currently using sqlalchemy and fastapi for a production microservice hosted in AWS. The issue is that our production database secrets are refreshed every 30 days. Trying to automatically fetch the new secrets from secrets manager to reinitialize the database engine and session in the event of an error or OperationalError from sqlalchemy.
My question is where should this "reinitialization" occur?
utils/secret_mgr.py
import json
import logging

import boto3
from botocore.exceptions import ClientError

def get_secret(secret_id):
    session = boto3.client('secretsmanager', region_name='us-east-1')
    try:
        response = session.get_secret_value(SecretId=secret_id)
    except ClientError as e:
        code = e.response['Error']['Code']
        logging.exception(f'error:get_secret error_code:{code}')
        raise e
    else:
        secret_str = response['SecretString']
        secret = json.loads(secret_str)
    return secret

utils/db.py
import logging
import os

from sqlalchemy.pool import QueuePool
from sqlalchemy.sql import text
from sqlmodel import SQLModel, Session, create_engine
from sqlalchemy.exc import OperationalError
from api.utils.scemgr import get_secret

engine = None
SECRET_NAME = os.environ.get('DB_SECRET_NAME')
SQLALCHEMY_DATABASE_URL = 'postgresql+psycopg2://{username}:{password}@{host}:{port}/{dbname}'

def get_database_uri():
    secret = get_secret(SECRET_NAME)
    return SQLALCHEMY_DATABASE_URL.format(
        username=secret['username'],
        password=secret['password'],
        host=secret['host'],
        port=secret['port'],
        dbname=secret['dbname'],
    )

def get_engine():
    global engine
    if engine:
        return engine
    conn_str = get_database_uri()
    engine = create_engine(
        conn_str,
        echo=True,
        poolclass=QueuePool,
        pool_pre_ping=True,
        # pool_size=15,
        # max_overflow=5,
        echo_pool="debug"
    )
    return engine

engine = get_engine()

#
# class SessionManager:
#     def __init__(self):
#         self.db = sessionmaker(bind=engine, autocommit=True, expire_on_commit=False)
#
#     def __enter__(self):
#         return self.db
#
#     def __exit__(self, exc_type, exc_val, exc_tb):
#         self.db.close()

def get_session():
    with Session(engine) as session:
        try:
            yield session
            session.commit()
        except Exception as exc:
            session.rollback()
            raise exc
        finally:
            session.close()

def init_db_sqlalchemy():
    SQLModel.metadata.create_all(engine)

def fetch(db: Session, query, *args, **kwargs):
    try:
        stmt = text(query)
        result = db.execute(stmt, *args, **kwargs)
        db.commit()
        return result
    except (Exception, OperationalError) as err:
        logging.exception(f"error_code={err} function_name={fetch.__name__}")
    finally:
        db.close()

import os
import time
from uuid import uuid4

import uvicorn
from fastapi import FastAPI, Request, Depends
from fastapi.encoders import jsonable_encoder
from api.routes.info import info, health
from api.utils.db import init_db_sqlalchemy, get_session, fetch

app = FastAPI(
)
app.include_router(info, prefix="/info")
app.include_router(health, prefix="/health")

@app.middleware("http")
async def add_logging_and_process_time(request: Request, call_next):
    start_time = time.time()
    request_id = str(uuid4().hex)
    response = await call_next(request)
    process_time = time.time() - start_time
    process_time = str(round(process_time * 1000))
    response.headers["X-Process-Time-MS"] = process_time
    log_msg = f"request_id={request_id} service=pbgraph-translation-svc url={request.url} host={request.client.host} " \
              f"port={request.client.port} processing_time_ms={process_time} env={os.environ.get('APP_ENV')} " \
              f"version=v1 pid={os.getpid()} region={os.environ.get('REGION')} "
    logger.info(log_msg)
    return response

@app.on_event('startup')
def startup():
    init_db_sqlalchemy()

@app.get('/getDatabaseInfo')
def get_db_data_example(db: Session = Depends(get_session)):
    try:
        records = fetch(db, DATABASE_QUERY).all()
        return jsonable_encoder(records)
    except Exception as err:
        logger.exception(f"function_name=getDatabaseInfo error={err}")

if __name__ == '__main__':
    uvicorn.run(app, host="<IP_ADDRESS>", port=8050)

I currently initialize the database on startup. In the event of a database connection error, where should we reinitialize the database? i.e. pull the new credentials from utils/secret_mgr.py and recreate the database engine in utils/db.py.
Given the above information a few questions:

Should engine be a global object if we need to reinit every 30 days?
If get_session fails, a dependency injection to the get request, fails it will close the session and add it back to the connection pool. If we are using a connection pool and one of the connections become invalidated, it will in turn invalidate all connections in the pool. This is okay, should happen. Where should we recreate the database engine?

What is the proper way to do this given the constraints above?
Operating System
Linux
Operating System Details
Dockerfile
FROM python:3.8
RUN microdnf install curl wget telnet vi bind-utils
COPY . /
RUN pip3 install -r /requirements.txt
USER 99
EXPOSE 8050
CMD ["python", "/api/main.py"]
FastAPI Version
0.74.0
Python Version
3.8
Additional Context
Thank you for assistance on this question. I have searched but all documentation shows examples without anything impacting connection strings. Given that fastapi is the base for our backend, where should we initialize this task from a fastapi perspective?

How exactly is it deployed on AWS ? It feels like it should not be the apps responsibility to cycle the credentials.
If this was running on a linux box with Gunicorn you could just send a SIGHUP to gunicorn and it would shut down the workers and start new ones that would pull new credentials. In AWS/Docker I would think you would just spin up new instances and toss the old ones when it is time to refresh.

Currently the deployed app pulls secrets and creates the database engine for every api request. Is there a way to catch an event (such as sqlalchemy.exec.OperationalError or Exception) as an event within fastapi? If so, I can include the changes here

maybe you can see sqlalchmey document
from sqlalchemy import event
some_engine = create_engine(...)

@event.listens_for(some_engine, "engine_connect")

every connect try select 1,catch exeception then change pool
https://docs.sqlalchemy.org/en/14/core/pooling.html#custom-legacy-pessimistic-ping

One solution is create boostrap function with your connection and add this bootstrap with dependency injection.
A flask example in cosmic python -> https://www.cosmicpython.com/book/chapter_04_service_layer.html

I am closing this issue. This should not be handled by FastAPI or anything from Python side. We are using containers and Kubernetes in PROD.
The best way to handle this is by creating a health API router that is not included in schema and has no authentication. /health/live to support if container is alive and /health/ready to support it the applications dependencies aka RDS or other databases via 'select 1'. In deployment.yaml files include a livenessProbe and readinessProbe on both the API endpoints respectively.
This will ensure the container is restarted when a 500 status code is provided or when the app cannot query the database.
Hope this helps others!
🚀


Error - Failed to Load Icon Pack List

While on mobile, I get this error when I click on "Browser":
Failed to Load Icon Pack List:
(0, li.requestUrl) is not a function

Also, if I try to restore icons from a backup, it crashes Obsidian.

What version of mobile obsidian are you using? make sure you've update to the latest since the plugin depend on some new API.

Thanks for the quick response. I'm using the latest Android version, though this was last updated on 4th Feb and only incorporates functionality up to desktop v0.13.24.

This now works with the latest mobile release (v1.1.1) so I'm closing this!


/admin/ and /api/ redirect to the backend server for local instances

/admin/ and /api/ are both broken in local instances due to a removal of front end proxying
redirects would be a reasonable solution I think

I think we decided this isn't going to happen.
closing as not planned


Activations/doctest activations

Description
An example for #2066.
Type of change

[ ] Bug fix
[ ] New Tutorial
[x] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[ ] I've properly formatted my code according to the guidelines

[x] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
doctest + build_docs

@andreasmadsen @failure-to-thrive @digantamisra98 @fsx950223 @aakashkumarnain
You are owners of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

Example screenshot for doc rendering


Easily bypass and inspect websites

Using nextDNS
Head to denylist and add:
*.cdn.jsdelivr.net and *.jsdelivr.net

Optionally using AdGuard
Create a custom filter ||cdn.jsdelivr.net/npm/disable-devtool^

AdGuard seems doesn't work

In Chrome, open a new tab > open Dev Tools > navigate to website. This is one of the many ways to bypass the restriction.

In Chrome, open a new tab > open Dev Tools > navigate to website. This is one of the many ways to bypass the restriction.

Correction:

in Chrome, open a new tab
open Dev Tools
open Network tab in Dev Tools
check the box: ☑ Preserve Log
navigate to website
... once loaded, the website using this script will kick you to the New Tab page ...
right click on the disable-devtool script (it might end in .min.js, @latest, ...)
click Block Request URL

When navigating back to the website, the disable-devtool script is now blocked.


fix(theming): set default color to currentColor in input theme (#1260)

Currently input color does not update when switching between dark and light theme. Since the background of md-input-element is transparent it is currently possible to have black text on a dark background making it hard to read. This PR sets the color for .md-input-element to currentColor.
You can see the current behavior in @jelbourn 's https://github.com/jelbourn/material2-app (just toggle the dark theme).

Thanks!


There should be a way to retrieve tenant by ID using admin services

There should be a way to retrieve tenant by ID using admin services.
I cannot see a method. What happens is if we try to create a tenant with providing an existing ID, it will throw an error. So it would be better to have  a way to check if tenant id exists before creating one using admin services.
<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/">    <soapenv:Body>       <soapenv:Fault>          <faultcode>soapenv:Server</faultcode>          <faultstring>Error in adding tenant with domain: vihana1.com</faultstring>          <detail>             <ns:TenantMgtAdminServiceException xmlns:ns="http://services.mgt.tenant.carbon.wso2.org">                <ns:TenantMgtAdminServiceException>                   <axis2ns3:Message xmlns:axis2ns3="http://services.mgt.tenant.carbon.wso2.org">Error in adding tenant with domain: vihana1.com</axis2ns3:Message>                </ns:TenantMgtAdminServiceException>             </ns:TenantMgtAdminServiceException>          </detail>       </soapenv:Fault>    </soapenv:Body> </soapenv:Envelope>

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


HDXSYS-687 add refugees

@b-j-mills I hope you can use this to begin development on the refugee pipeline. You'll see that overall we are going for leaner tables that use enums when possible instead of other tables. Also, David had a great idea of doing QC using a combined primary key with elements that should be unique. We may need to scrap this, as well as the many constraints, depending on what DPT decides we should do with incorrect data. I will hold off writing the tests until the schema is fixed, leaving it draft until then.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 33 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.7%) to 91.274%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hapi_schema/utils/shared_enums.py
0
5
0.0%

src/hapi_schema/db_refugees.py
0
28
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-7.7%

Covered Lines:
387

Relevant Lines:
424

💛  - Coveralls

Superseded by #29


Ensure to register grpc server metrics to custom Prometheus registerer

What this PR does / why we need it:
The go_grpc_prometheus package registers its counters to the default registerer in advance, but for custom one, it's needed to register manually. After adopting the custom one, we're losing grpc metrics until now.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

Nice catch 👍
/approve


Add underlying framework

Create a new database structure and add dynamic prefixes
This fixes any issues that may occur later
Checklist:

[x] Database Connection
[x] Dynamic Prefixes
[x] Better Permission system

Resolved by commits: bbd813179f09dfbb2f40d1c8800d323b36ab42d5, afc3bc14558b15b40530ab340d3c7eb41b085b2b, and 06ce42afa431b2ea97ce3849b1d91be577d48478


Session check for GISAID branch

Creating an issue to track progress of implementing Session check using GISAID API

@ArtPoon Do you happen to know what our client_id is?
The GET api isn't working (even using a sid that is valid + the one they provided in their documentation)

GISAID just reconfigured our user accounts so that the CoVizu tile should appear.

CoVizu title appears for me on GISAID website.
Site fails to load using our validation:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at
"https://gpsapi.epicov.org..."

I think loading the script using cross-origin (iframe on epicov trying to load a js file on filogeneti.ca) is the cause. Although I'm not sure if we need to ask gisaid to allow filogeneti.ca (gisaid server side) or if we need to allow filogeneti to serve a file to gisaid(filogeneti server side). I gave a crack at turning on cross-origin on filogeneti using this link here:
https://enable-cors.org/server_apache.html
Ran into a problem with this,  we need to turn on module headers using  a2enmod headers to allow addition of the following code in the /etc/apache2/apache2.conf file on filogeneti:
<directory>
    Header set Access-Control-Allow-Origin "epicov.org"
    ...
</directory>

Headers don't work because of issue related to #180. Turning on headers results in this message on reboot:
ewong347@coreceptor:~$ apache2ctl -M | grep  headers_module
AH00526: Syntax error on line 7 of /etc/apache2/sites-enabled/000-default-le-ssl.conf:
SSLCertificateFile: file '/etc/letsencrypt/live/shiny.filogeneti.ca/fullchain.pem' does not exist or is empty

I took a crack at making the changes I mentioned above, but reverted changes once I ran into the SSLCertification error above. Can @ArtPoon @GopiGugan take a look here? I think you guys fixed this issue over on the poonlab repo (# 208).

Can you try adding sudo to your command?

I used sudo for commands involving the apache server, I wouldn't have the permissions to make changes otherwise.

https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS/Errors

I agree with @GopiGugan 's comment, running that command as sudo should return no error.

Access to XMLHttpRequest at 'https://gpsapi.epicov.org/epi3/gps_api%7B%22cmd%22%3A%22state%2Fsession%2Fvalidate%22%2C%22api%22%3A%7B%22version%22%3A1%7D%2C%22client_id%22%3A%22cid-e9418c5b4b6e%22%2C%22sid%22%3A%22OLDDPQKLYHLCKNSU ... OSYIGZEZESWTEGBU%22%7D' from origin 'https://filogeneti.ca' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Working through the same workflow as @ewong347

Modified .htaccess

Options -Indexes
Header set Access-Control-Allow-Origin https://gpsapi.epicov.org

Ran apachectl configtest and apachectl -k graceful.
Main page still loads ok.
Ran into same problem with GISAID page, i.e., No 'Access-Control-Allow-Origin' header is present on the requested resource.
confirmed headers module is not loaded
ran a2enmod headers, creating symlink in mods-enabled
apachectl -k restart
main page still loads over https

Attempting to load from GISAID still returns a CORS error, but now there's a POST error message in the console:
jquery.js:10099 POST https://gpsapi.epicov.org/epi3/gps_api%7B%22cmd%22%3A%22state%2Fsession%2Fvalidate%22%2C%22api%22%3A%7B%22version%22%3A1%7D%2C%22client_id%22%3A%22cid-e9418c5b4b6e%22%2C%22sid%22%3A%22OLDDPQKLYHLCKNS ... MHLYAUNXCIVOSYIGZEZESWTEGBU%22%7D net::ERR_FAILED
send                @   jquery.js:10099
ajax                @   jquery.js:9682
jQuery.<computed>   @   jquery.js:9836
(anonymous)         @   covizu.js:15

Still getting same error but now there's a POST request associated with it...

Updated /epicov/ branch of filogeneti with above changes.


add namespace name in clusterroles, binding and service accounts

@kahootali Chart: grafana Dry Run successful

@kahootali Chart: grafana-storage Dry Run successful


update title to be relevant for action-based command.

Docs Build status updates of commit c91cea9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/messaging-extensions/create-extensions.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Generalize main pipelines documentation

Now that we're generalizing the pipeline, we should remove references to OpenMRS or replace with references to HAPI, and make sure that the HAPI examples all work correctly.
https://github.com/google/fhir-data-pipes/tree/master/pipelines

Thanks @jjtswan for filing this. Yes we should prioritize HAPI over OpenMRS as it is more generic. But let's do not remove references to OpenMRS. Our pipelines working with OpenMRS FHIR module is a non-trivial thing and OpenMRS is still widely used in some segments of LMIC healthcare. So it is still a nice-to-have feature and we cover it a fair amount in our e2e tests too. So let's reduce but keep some OpenMRS related references/documentation.


Fix how locked amounts are computed at settlement for old balance proof with high locked amount

Fix how locked amounts are computed at settlement
Fixes #127
P1 = participant1
D1 = deposit P1
W1 = withdrawn P1
T1 = transferred P1 -> P2
L1 = pending transfers P1 -> P2
A1 = amount to be received by P1 at settlement

P2 = participant2
D2 = deposit P2
W2 = withdrawn P2
T2 = transferred P2 -> P1
L2 = pending transfers P2 -> P1
A2 = amount to be received by P2 at settlement

DC = D1 + D2 - W1 - W2 = current channel deposit

To simplify calculations, we artificially require the settleChannel arguments to be ordered:
T2 + L2 >= T1 + L1

https://github.com/raiden-network/raiden-contracts/blob/1046322d015a4ec06f6ad0b9961db194399f3563/raiden_contracts/contracts/TokenNetwork.sol#L882-L884
This comes from the way we compute the maximum receivable amount for P1
A1max = (T2 + L2) - (T1 + L1) + D1 - W1

https://github.com/raiden-network/raiden-contracts/blob/1046322d015a4ec06f6ad0b9961db194399f3563/raiden_contracts/contracts/TokenNetwork.sol#L889-L906
Note that we are including the locked amounts.
These can be:

pending transfers that have not been finalized off-chain; we don't know at this point if the secrets have been registered or not, do we don't know if the locked amount will go entirely to the intended participant
old pending transfers that have actually finalized on-chain and included in a newer balance proof; this is the case where cheating can be done - the participant can make sure to also register the secrets on-chain and use that later to try to withdraw more tokens - this attack is fixed by this PR, because:

A1max = min(A1max, DC)
A2max = DC - A1max

and A1max also contains L2. A2max contains L1.
https://github.com/raiden-network/raiden-contracts/blob/1046322d015a4ec06f6ad0b9961db194399f3563/raiden_contracts/contracts/TokenNetwork.sol#L599
We then subtract the locked amounts and bound them:
L2 = min(L2, A1max)
L1 = min(L1, A2max)
A1 = A1max - L2
A2 = A2max - L1

https://github.com/raiden-network/raiden-contracts/blob/1046322d015a4ec06f6ad0b9961db194399f3563/raiden_contracts/contracts/TokenNetwork.sol#L613-L623
And make sure we store the new locked amount in the contract and not the initial one.
Tests
Added tests that reverse balance proof values for T1 and L1 / T2 and L1 , and also provide random values for the locked amounts e.g. L1 = randint(L1, L1 + T1), to simulate old balance proofs. All these cases are equivalent and should settle following this constraint:
A1 + L2 = A2 + L1

There are possibly other issues with how we compute the locked amounts. Specifically, I have not tested this model to see how it behaves for high/low values for the "unlockable" amount in a locked_amount vs. the "not-unlockable" amount. I opened https://github.com/raiden-network/raiden-contracts/issues/132 for this issue.
The current PR resumes to fix cases similar to https://github.com/raiden-network/raiden-contracts/issues/127 and does that by making the assumption that the entire locked_amount1 might go to participant2. This assumption might not be enough for a fair calculation, but it improves the current settlement calculations anyway and adds more tests.


Handling variables in grafana

Please help, this is driving me crazy
When the plugin expands the variables what is actually doing. It seems like sometimes they behave like strings and other times like objects for example this creates an error "Invalid or unexpected token" its like its being interpolated into a string but not properly escaped
console.log($variablename)
I was storing JSON in a grafana variable and then reading it back in on another form panel but i gave up and instead used a global variable to use between two form panels

Hi @craftzneko
it is necessary to add quotation marks to read the variable and not get an error
console.log(`$variablename`)


Package cards should not depend on temporary package info

Summary of changes
Remove temporary package info dependency from Package cards.
Context and reason for change
Package cards should not depend on temporary package info. That is just display information, until it gets saved it should not influence anything. That also triggers a re-render of all package cards every time a character is written in a field, making the input slow.

nice 👍


w3wp.exe locks 'libsass*.dll'

Hi
After running the following code:
var compiler = new SassCompiler(new SassOptions { OutputStyle = SassOutputStyle.Compressed, InputPath = customScssPath });
var result = compiler.Compile();

The dll libsass* is locked, until we kill w3wp.exe
How can we release the resources?
With kind regards,
Jan van Helvoort

Hey Jan,
What version of libsass are you running?

Hi,
We use version 3.3.7 from libsassnet.

Hi,
Can you reproduce the problem?

I'm sure I could reproduce this problem if I tried as this happens with basically all native interop and I have encountered this issue before with the sql server binaries we use. I'm not sure if there's a real solution here though since there's no shadow copying of binaries for native interop. I'd have to do some research before I would know for sure.

Hi,
Do you have a update?

can you please try updating to latest version and see if you can reproduce it ?

yes, with 3.3.7-alpha-2 the files are locked within w3p.exe

The stable version should be the one to use, not the alpha, can you try that ?

no, every version of the package has the same problem.

We're having the same problem; we had to inject a PowerShell script into our deployment process to get around it.


Ensure consistent code conventions

This addresses #132 - adds both editorconfig and mega-linter setup, for details see the different commit messages.
Changes might be needed, all written on the phone, I'm curious to see how it works 😁

Hmm, the broken conventions were not detected, doesn't seem to work and needs more investigation.
Note: MegaLinter edited the original comment (https://github.com/tomorrow-one/transactional-outbox/pull/133#issuecomment-1612102715) after the run for the test commit breaking conventions, instead of adding a new comment. The order of comments might be confusing...

Ok, it works, the only thing is that https://github.com/editorconfig-checker/editorconfig-checker#what states that it only supports certain editorconfig features:

end_of_line
insert_final_newline
trim_trailing_whitespace
indent_style
indent_size
max_line_length

while in my previous test yesterday I broke spaces_around_operators (in commit 73f5a5aa).
Now breaking the indent_style and trim_trailing_whitespace properties shows that at least these properties are checked.
Not sure, should we integrate this, @hnrkdmsk @danielrehmann ?

Ok, it works, the only thing is that https://github.com/editorconfig-checker/editorconfig-checker#what states that it only supports certain editorconfig features:

end_of_line
insert_final_newline
trim_trailing_whitespace
indent_style
indent_size
max_line_length

while in my previous test yesterday I broke spaces_around_operators (in commit 73f5a5a).
Now breaking the indent_style and trim_trailing_whitespace properties shows that at least these properties are checked.
Not sure, should we integrate this, @hnrkdmsk @danielrehmann ?

I think we should use this. It is working, and we can still improve it if we have the need. It's better than nothing 😊


version 2.1.4 - crash in iOS 15

Hello
After updating iOS to 15 me and other users of developing app experience crash while press 'Share' button w/o any error messages (
How to solve this?

Hi,
I've tried and I couldn't reproducing the issue.
Are you experiencing the issue on some particular iOS device or simulator?
Which type of file are you opening? Could you share a snippet of your code?

It is turned out that problem in old version of ReactNativeNavigation


Openscenario standard v2.0

Is it possible to execute scenarios in open scenario v2.0 or any open scenario standard higher than v1.0? I noticed in the documentation for Carla that the scenario runner only works with version 1.0!

Is it possible to execute scenarios in open scenario v2.0 or any open scenario standard higher than v1.0?

Technically no, we currently only partially support OSC 1.0, so any changes done by higher versions are not supported.
However, you can always do it yourself, and while that's probably not a satisfactory answer, if you are interested in a specific feature, I can provide some help with guidelines on how the OSC 1.0 support is done

The difference between OpenSCENARIO 1.x and 2.0 is extreme and the new standard is overloaded with a lot of new concepts. Implementing Osc2.0 is extremely hard and requires IMO joint effort to make an open source implementation.
For your information, I am the main developer behind Yase and we intend to do exactly this. The open sourced Yase framework is build to support multiple scenario formats (such as in OpenPASS with Osc1.x or company internal formats). This can be implemented agnostic of the simulator, so we can have a reuse of the frontend for many simulators.
Internally we already use an Osc2 frontend with our internal simulators, however we still relay on some internal code. Once it is fully removed, we will share it so it can be used for more simulators such as carla. Until then one can already look into some (outdated) code to get the rough idea.
If you plan to do something in direction Osc2.0, feel free to contact us so we can see how we can support & avoid redundant code!

@SEEKERXC I am having an issue with OpenScenario while using the waypoint based scenario. Apparently the scenario runner is stuck with first event and not going to the next event. I used StoryboardElementStateCondition as the start trigger and do you have any experience or feedback about this ?

@XiaoFei9704 , I am facing some challenges with Path Modifiers in Open Scenario 2.0, Can you provide some insights on same.
Issue explained here - https://github.com/carla-simulator/scenario_runner/issues/1027


Sprint20 fix model output type

closed because wrong base branch is used. Another one is opened #265


Add beforeRequire event

This is a pr for the issue #53.
This pr makes liftoff emit beforeRequire event, but for only a module which is specified by opts.require.

@sttk can you update this to fit with the other changes in Liftoff?  I'd like to fit it into the 4.0 release.

@phated OK. I'll update this pr.

I've rebased this.

Looks great!  Thanks @sttk!

@phated Thanks for merging!


element breaks styling behavior: can't be positioned with margin: auto

Hi, I created a demo page for you to see this:
<!DOCTYPE html>


Mission (joining, leaving) button

In this PR, the following features are applied :

On user click "Join Mission" button, action applied to update the store.
Got the ID of the selected mission and updated the state
Didn't mutate the state. Instead, returned a new state object with all missions, and the selected mission has extra key reserved with its value set to true.
Used map() function to set the value of the new state.
Placed all your logic in the reducer.
In the React view file, only the action is dispatched with the correct rocket ID as an argument.
Followed the same logic as with the "Join mission" - but set the reserved key to false.
Dispatched the actions upon click on the corresponding buttons.

Hi Marvin,
For that feature, I will submit another PR.


code cleanup in update cmd. implementation

Purpose

Extract out a few constants in the cellery update command implementation.

Codecov Report

Merging #532 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #532   +/-   ##
=======================================
  Coverage   64.48%   64.48%
=======================================
  Files          34       34
  Lines        1177     1177
  Branches       98       98
=======================================
  Hits          759      759
  Misses        365      365
  Partials       53       53

Continue to review full report at Codecov.

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


maintainers: add wamserma

Motivation for this change
Backport to allow better testing of new packages against stable.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@veprbl, please also review/merge. This is a cherry-pick of the commit merged with https://github.com/NixOS/nixpkgs/pull/85466


Delayed op use cases out of date?

The following AWS use cases show examples of delayed operations using custom tags. When implementing these types of policies for Azure, I noticed you have to add the custom tag to the marked-for-op type. Is this the case for AWS?
http://capitalone.github.io/cloud-custodian/docs/usecases/rdsdeleteunused.html
http://capitalone.github.io/cloud-custodian/docs/usecases/elbgarbagecollection.html

there is a default, the example just shows overriding it, if you have multiple workflows running against the same resource type, its important to use a separate tag per workflow to avoid them overriding each other's state.

In elbgarbagecollection you are using tag  tag: maid_status The tag appears to be missing in the delete action step
     - delete
     - type: notify
       template: default.html
       priority_header: 1
       subject: "ELB - Deleted Stale ELB - [custodian {{ account }} - {{ region }}]"
       violation_desc: "No Instances Are Attached To ELB for over 14 days:"
       action_desc: "Actions Taken:  The ELB has been deleted"
       to:
          -<EMAIL_ADDRESS>          - resource-owner
       transport:
            type: sqs
            queue: https://sqs.us-east-1.amazonaws.com/12345678900/cloud-custodian-mailer
            region: us-east-1

yeah.. maid_status is the default value here (set in c7n/tags.py DEFAULT_TAG). prior to being public, custodian had a different name, which is still reflected in this value and the cloud watch metrics namespace (CloudMaid), we ended up preserving those due to our compatibility guarantees.. and its kept going. probably worth updating the examples to always be explicit or at least document the default.


[WIP] Use Patternfly for UI

A lot to do here:

[ ] Get a starter dashboard layout up and running with Patternfly and React Router
[ ] Draw some UI designs for Organizations, Devices, Invitation Management etc...
[ ] Implement those views
[ ] Set up redirect to Keycloak when not logged in
[ ] Use consistent UI on Keycloak Login, Dashboard and Keycloak profile management

Fixes: #883

closed since this is not actively being worked on. Feel free to reopen anytime if you'd like.


minor optimize FilterInPlace

Similar with native slices.DeleteFunc, but FilterInPlace uses keep not del.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all


Add simplejson in the dependencies

if you create a new virtualenv with a new version of Python 2.7.9, I have a missing dependency: simplejson.

in fact, I work on master and not on develop :/


Fix: python_interpreter on delegation

This should fix the 'missing python interpreter' error when running tasks on delegation mode between hosts with different python setup

Hi, Thanks a lot for your contribution!
Can you add an integration test and/or document steps for reproduce the issue?


No errors reported

Summary
I am encountering the same issue as reported in #1. Errors that mypy reports when run from the command line are not reported by the extension in Nova.
Environment
Nova version: 9.3
mypy version: 0.950
Python version: 3.10.3
Extension version: 0.9.0
macOS version: 12.3.1 (21E258)
Additional Information
Example file:
def my_func(arg1: str) -> None:
    print(arg1)

    return 27

my_func(1)

Expected results:
/Users/sdoran/Downloads/test.py:4: error: No return value expected
/Users/sdoran/Downloads/test.py:7: error: Argument 1 to "my_func" has incompatible type "int"; expected "str"
Found 2 errors in 1 file (checked 1 source file)

Actual results from Nova extension console:
Mypy[13:47:44.101000] Running /Users/sdoran/.pyenv/versions/appomni-dev-3.7.12/bin/mypy --shadow-file .//Users/sdoran/Downloads/test.py /Users/sdoran/Library/Application Support/Nova/Workspaces/951FB00D-2D31-42CF-A475-96B813BBFAF5/cc.aeron.nova-mypy/-837007767.1651600064099.tmp /Users/sdoran/Downloads/test.py

Mypy[13:47:44.175000] Checking /Users/sdoran/Downloads/test.py

Mypy[13:47:44.175000] Found 0 issue(s)

Hey, Sam. I’ve checked this case on my system: Nova 9.3, Python 3.10.4, and Mypy 0.950.
The extension version should be 0.1.0, but it’s a typo probably. Everything else is not relevant yet thank you for all the details.
I got the following result:
Mypy[23:15:31.600000] Running /usr/local/asdf/shims/mypy --shadow-file ./playground-2.py /Users/aeron/Library/Application Support/Nova/Workspaces/B3E20458-CF35-4609-A00C-06AD41DB2785/cc.aeron.nova-mypy/2023605783.1651853731596.tmp playground-2.py
Mypy[23:15:32.152000] Checking playground-2.py
Mypy[23:15:32.152000] Found 2 issue(s)

So, what could it be? The first things that came to mind:

Could you check if ~/.pyenv/versions/appomni-dev-3.7.12/bin/mypy is 0.950? Just in case there’s another Mypy executable in the PATH.
Do you have separate Python in your system? Could you try to specify the exact Python path in extension preferences? Or do you already have one: global or project?
Do you have a global Mypy config anywhere? Just as I mentioned in #1.4.

I don’t think that pyenv is directly a reason here. I’m using native venv, but this time I used none.

The extension version should be 0.1.0, but it’s a typo probably.

Yup, sorry about that.

Could you check if ~/.pyenv/versions/appomni-dev-3.7.12/bin/mypy is 0.950?

> ~/.pyenv/versions/appomni-dev-3.7.12/bin/mypy --version
mypy 0.950 (compiled: no)

Do you have separate Python in your system? Could you try to specify the exact Python path in extension preferences? Or do you already have one: global or project?

I have many Pythons installed.
I only set the executable path in the global extension preference. I haven't set any project specific configs. It looks like the only place to set the Python executable path is in project preferences. I will try that.
My main project does indeed have a mypy.ini at the project root. I do not have a global mypy config.
For the purpose of this bug report, I used a directory that does not have any configuration files.
> ~/.pyenv/versions/appomni-dev-3.7.12/bin/mypy -v ~/Downloads/test.py

LOG:  Mypy Version:           0.950
LOG:  Config File:            Default
...

Setting the Python executable path at the project level did not fix the issue.
I also tried using mypy with Python 3.10.3.
I set both executable path and Python executable path at the project level:
/Users/sdoran/.pyenv/versions/mypy-3.10.3/bin/mypy
/Users/sdoran/.pyenv/versions/mypy-3.10.3/bin/python
That still report zero issues:
Mypy[15:59:36.512000] Running /Users/sdoran/.pyenv/versions/mypy-3.10.3/bin/mypy --python-executable=/Users/sdoran/.pyenv/versions/mypy-3.10.3/bin/python --shadow-file ./test.py /Users/sdoran/Library/Application Support/Nova/Workspaces/F721BC42-EB6B-4B51-BE11-371EDCEEB52A/cc.aeron.nova-mypy/-1422502683.1651867176506.tmp test.py
Mypy[15:59:36.609000] Checking test.py
Mypy[15:59:36.609000] Found 0 issue(s)

I found that running the command from the Nova console manually results in mypy: can't read file 'test.py': No such file or directory. Quite curious.
> pwd
/Users/sdoran/Downloads

> ls -l test.py
-rw-r--r--@ 1 sdoran  staff  76 May  6 16:10 test.py

> /Users/sdoran/.pyenv/versions/mypy-3.10.3/bin/mypy --python-executable=/Users/sdoran/.pyenv/versions/3.10.3/bin/python3.10 --shadow-file ./test.py "/Users/sdoran/Library/Application Support/Nova/Workspaces/F721BC42-EB6B-4B51-BE11-371EDCEEB52A/cc.aeron.nova-mypy/-1422502683.1651867710272.tmp" test.py
mypy: can't read file 'test.py': No such file or directory

Two things I found to be buggy :

If using ~ in the path for the mypy executable, this won't be expanded to you home dir, you have to explicitly use an absolute path (e.g. /Users/guillaume/path/to/bin/mypy).
The issueMatcher regex is wrong, it doesn't account for error messages without a column number. I had better results with the following config, but be aware that I haven't yet tested it extensively :

"issueMatchers": {
  "mypy": {
    "pattern": {
      "regexp": "^(.+\\.py):(\\d+):((\\d+):)?\\s(\\w+):\\s(.*)",
      "message": 6,
      "code": 5,
      "line": 2,
      "column": 4,
      "severity": "error"
    }
  }
}

If using ~ in the path for the mypy executable, this won't be expanded to you home dir
I don't think this is unique to this plugin. I always use absolute paths for executables.

Indeed, but expanding such common shorthands would be nice, or at least providing an error message if the binary is not found.

With the above patch, I get results now.

It looks like the error messages I am seeing don't have columns, which is why the regexp change fixes
typing-tests.py:10: error: No return value expected
typing-tests.py:17: error: Argument 1 to "my_other_func" has incompatible type "int"; expected "str"

Further scrutiny reveals the fix is indeed making the second groups of digits: optional. Great work!

Thank you for keep digging, gentlemen. To my shame, I don’t have much attention to doing it at the moment. Yet PRs are always welcome. I’ll do my best to keep the extensions up to date, even if I’m not using Nova.

Also, regarding the --follow-imports silent option: if I remember correctly, VS Code extension, for example, does not force this option. That is why each of my project configurations (pyproject.toml or .mypy.ini) always has it explicitly, despite global settings even.
I’m uncertain if we want to force every extension user to operate it only this way.

I'll work on some PRs. I meant to get to it this week but time got away from me.

I think those PRs cover the items discussed above. I'm open to feedback in the fixes.

Looks good to me. Everything is merged, released, and uploaded to the extension library. Thank you, good sirs 🤜🤛

Thank you so much!


linuxPackages.oci-seccomp-bpf-hook: 1.2.2 -> 1.2.3

Motivation for this change
Update to the latest version, which seems to fix the arm build issue.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120861 at 1765c300 run on aarch64-linux 1

4 packages marked as broken and skipped:

linuxPackages-libre.oci-seccomp-bpf-hook
linuxPackages_latest-libre.oci-seccomp-bpf-hook
linuxPackages_testing_bcachefs.oci-seccomp-bpf-hook
linuxPackages_xanmod.oci-seccomp-bpf-hook

7 packages built successfully:

linuxPackages.oci-seccomp-bpf-hook (linuxPackages_5_10.oci-seccomp-bpf-hook)
linuxPackages_5_11.oci-seccomp-bpf-hook (linuxPackages_latest.oci-seccomp-bpf-hook)
linuxPackages_5_4.oci-seccomp-bpf-hook
linuxPackages_hardened.oci-seccomp-bpf-hook
linuxPackages_latest_hardened.oci-seccomp-bpf-hook
linuxPackages_lqx.oci-seccomp-bpf-hook
linuxPackages_zen.oci-seccomp-bpf-hook


Move Release Notes to new line and indent

[x] Have you signed the Contributor License Agreement?
[ ] Are you working against an Issue?

It bothered me that Multi-Line release notes had weird alignment. This places the release notes down one line and indents all lines.

Microsoft Reviewers: Open in CodeFlow

I think it looks good.

/azp run

/azp run


add the ci build

This PR is used to automatically build and push relevant images.
Signed-off-by: chengdehao<EMAIL_ADDRESS>

basic test and verify failed?

basic test and verify failed?

https://github.com/kubesphere/notification-manager/pull/124 This pr fixes the problem, probably when it merges it will work.

@wanjunlei @zhu733756 Is this PR ok for you now?

/lgtm


Add GitLab code quality formatter

Add a formatter that outputs a GitLab code quality report artifact JSON file.
As suggested I left the author field pointing to me. I also added a LICENSE with your name since it will be released under your name. Let me know if this doesn't make sense.
I also added a README and tried to take care of all the tests.
Open issues/questions

[ ] running npm test produces the output file in test/formatter-gitlab/ which is not automatically deleted. How can this be achieved?
[ ] looking at the diff of the snapshots it seems that the JSON output shows up in the wrong place (within formatterJson). Is this because the formatter is missing from test/markdownlint-cli2-test-cases.js lines 62 and 71?
[ ] does it need to be added to test/outputFormatters-pre-imported/.markdownlint-cli2.cjs as well?
[ ] I would like to have a test case that ensures that the fingerprint is different for the same violation appearing multiple times in the same file on different lines (I created this scenario in test/formatter-gitlab/dir/subdir/info.md. What would be the best way to add a test case for this?

TODO

[ ] squash commits into one and reference the issue

add markdownlint-cli2-formatter-gitlab to (I assume it would require the package to be published first in most cases)

[ ] test/outputFormatters-npm/.markdownlint-cli2.jsonc
[ ] package.json devDependencies
[ ] dictionary.txt
[ ] .github/workflows/checkers.yml
[ ] .github/workflows/ci.yml

Fixes #92

An unrelated question: Would it be possible to include (install) the formatters in the Docker image so the image can be used without having to install them?

Thank you, this looks very thorough! I will provide initial feedback later today, but may not be able to do a full review for a little while. Thanks for your patience! Regarding including formatters in the Docker container image, that should already be true. I have separate tests for that, but I see they don't explicitly check formatters, so it's possible that's broken?
https://github.com/DavidAnson/Workflows/blob/main/.github/workflows/smoke-test.yml

Regarding including formatters in the Docker container image, that should already be true. I have separate tests for that, but I see they don't explicitly check formatters, so it's possible that's broken?

I had a quick look inside the image and I could only find the default formatter installed. I could have a look into this if all the formatters should be installed in the image.

Please have a look at the Dockerfile, I think they should get installed globally. That's why I use asterisk there. It's possible the publish steps changed slightly such that they are no longer present. Ah, this is possibly related to the Docker publish step now happening in a GitHub Action instead of locally, which is how I originally had things working.

From what I'm seeing on my phone, I agree it looks broken. I've made myself a note to sort that out separately.

Please have a look at the Dockerfile, I think they should get installed globally. That's why I use asterisk there.

From my understanding it installs only the non-dev dependencies of markdownlint-cli2 which only includes the default formatter. The other formatters are listed as dev dependencies. It might require to install them specifically in the Dockerfile to avoid installing all formatters in other scenarios.

At one point, I think all the pack files got created locally, and that command installed them all globally into the image. Again, I agree that doesn't seem likely anymore. I will have a look.

running npm test produces the output file in test/formatter-gitlab/ which is not automatically deleted. How can this be achieved?

There's some kind of wrapper in the tests that copies to a new directory and cleans up after it, I think. It's been a while since I've looked at this, and I am doing so now from my phone, so some of the specifics are a little hazy.

does it need to be added to test/outputFormatters-pre-imported/.markdownlint-cli2.cjs as well?

Ideally, this new formatters would show up in all the places where the other formatters are grouped.

I would like to have a test case that ensures that the fingerprint is different for the same violation appearing multiple times in the same file on different lines (I created this scenario in test/formatter-gitlab/dir/subdir/info.md. What would be the best way to add a test case for this?

If you end up doing a diff of output like I think the JSON formatter tests do, this should be handled implicitly if your test cases include the scenario you care about.

the spec is a subset of the codeclimate spec. In theory it would not be very difficult to support the full spec. Maybe naming the formatter -codeclimate would be better to prepare for it (or support it right away)?

Yes, if this would be useful in scenarios outside GitLab, that might be a good way to clarify that. (I'm not familiar with this part of the test ecosystem.)

add markdownlint-cli2-formatter-gitlab to (I assume it would require the package to be published first in most cases)

Yes, that's a follow-up commit after publish.

I addressed your comments.

changed formatter name to formatter-codequality
changed output file to be unique
removed all parameters from the formatter
I was able to update the snapshots, had to manually add formatterCodeQuality: '' to a lot of those and then run npx ava --update-snapshots. Hopefully that was the correct procedure.

I tried to output the JSON onto stdout instead of writing to a file. This would allow it to be invoked and the output directly piped to some other tool/file. But the extra output (version and number of files and errors) prevents this.

This looks awesome! I left a few very minor comments. Should be easy to fix or let me know if you're tired of this and I'll do so. :)

Not a problem at all!
Fixed all of them and also added a snippet to the README on how to use this in a GitLab CI job.

I couldn't push changes to your branch from Codespaces, so I made the last few tweaks in my own repo. Your commit is now present in next branch as https://github.com/DavidAnson/markdownlint-cli2/commit/50855cd35f50a3d5336279e3b9d065c0df3251f5.
Thank you very much!
This part took longer than I planned tonight, so I will publish the package to npm tomorrow and make the remaining updates to this repo to test the published package.

Great, thanks a lot!


Add single() to QueryState

What problem does this solve or what need does it fill?
Currently Query have a convenient single() function. But QueryState doesn't
What solution would you like?
Move single() logic to QueryState and call it from Query.

This should be trivial to implement. For any newcomers: this is a nice way to get familiar with a tiny bit of the ECS internals and our contributing process.


Cannot inject dependencies in the sockethandler or use servicelocator to do so

Since the middleware is scoped differently in the service registration - it won't allow something to either be injected (obviously) or created inside that scope that is not scoped the same.  For instance, I have a manager that I am using a special factory to instantiate - inside a method on the WebSocketHandler...
I tried scoping my factory in different ways - but because the handler (acting as the controller) is scoped differently then my manager... it won't allow it - in dotnetcore 2+
I don't see a nice way around this... because to do method injection on the invoke method would require me to either override and recode the invoke method or do something else wonky to get the classes that I need...  any advice would be appreciated.

Same problem right here


After a while, the software stops taking screenshots

No errors pop up, and the system tray does not exit

Add a exception catch


Streaming: How do I send the request back to the client for streaming?

Hi, so I've got the anthrophic sdk working for the most part, and it works fine. But I just don't know how to send it back to the client.
For context - it's a next 13 app, with supabase as the db.
Tried using Vercel's AI library for it, but it works to an extent and then gives up. And also the competition object compounds the responses too cf API
Any chance of getting an actual API example for this
Thanks

but it works to an extent and then gives up

Can you share more here? What code are you using, and what output do you see?

We do expect to ship a .toReadableStream() API in the near future, hopefully with a .fromReadableStream() that you can use on the client. This is how we'd expect you to be able to use it:
// app/api/ai/route.ts

export async function GET() {
  const params = {stream: true, …};
  const stream = await client.completions.create(params)
  return new Response(stream.toReadableStream())
}

// frontend.tsx
import { CompletionStream } from '@anthropic-ai/sdk'

async function getCompletions({ setText }) {
  const response = await fetch('/api/ai')

  const stream = CompletionStream.fromReadableStream(response.body)

  stream.on('content', (content) => {
    setText(text => text + content)
  })

  // or, if you prefer:
  for await (const chunk of stream) {
    setText(text => text + chunk.completion)
  }
}

export default () => {
  const [text, setText ] = useState('')
  useEffect(() => {
    getCompletions({ setText })
  }, [])

  return <p>{text}</p>
}

What do you think of an API like this?

but it works to an extent and then gives up

Can you share more here? What code are you using, and what output do you see?

Yep, so I am using the vercel ai library with Anthrophic Claude as the provider.
https://sdk.vercel.ai/docs/guides/providers/anthropic
Pretty Standard:
import { AnthropicStream, StreamingTextResponse } from 'ai'

export async function POST(req: Request) {
  // Extract the `messages` from the body of the request
  const { messages } = await req.json()

  const response = await fetch('https://api.anthropic.com/v1/complete', {
    method: 'POST',
    headers: {
      'Content-Type': 'application/json',
      'x-api-key': process.env.ANTHROPIC_API_KEY
    },
    body: JSON.stringify({
      prompt: prompt,
      model: 'claude-v1',
      max_tokens_to_sample: 300,
      temperature: 0.9,
      stream: true
    })
  })

  // Check for errors
  if (!response.ok) {
    return new Response(await response.text(), {
      status: response.status
    })
  }

  // Convert the response into a friendly text-stream
  const stream = AnthropicStream(response)

  // Respond with the stream
  return new StreamingTextResponse(stream)
}

This works but then it just throws an error.
error - TypeError: Cannot read properties of undefined (reading 'slice')
I did some digging and it turns out that for the ai Library, this is due to them not handling the ping event.
event: ping
data: {}
So when it hits this line
```const delta = text.slice(previous.length);```

it throws an error because text is undefined as there is no json.completion
```const text = json.completion,```

I am planning to raise a pr to fix this in the library
@rattrayalex Yeah those changes look good, maybe look at React Server Components support too while you're on Client SDK development RFCs

For the Vercel ai library, they should actually be able to accept the output of this library, eg:
const response = await client.completions.create(params)

const stream = new AnthropicStream(response)

However, I'm not sure if they've added support for that yet - I can ask them to or put up a PR doing so shortly.

I've put up a PR to Vercel's ai package: https://github.com/vercel-labs/ai/pull/448 it includes a usage example.

I'd be happy to leave this issue open until this library offers a nice full-stack solution built-in.
In the meantime, another suggestion might be to try just returning the result of the fetch call directly from your handler. I'm not certain if it'd work, and on the frontend you'd need to use .getReader() and .read() which can be a pain. Let me know what you find if you try it.

What would you want to see for React Server Components btw?

Hmm, I'm not the best expert on RSCs yet, but I would assume that I'd be able to use the SDK server side and return the component.
Thanks for your work with @vercel/ai, I think this issue can be considered closed, but you may want to keep it open for now until the SDK changes are fully closed.
In the meantime I'll raise a PR explaining how to integrate it with Vercel for folks.

Okay, thanks!


Performance: cache the checkuser status of a user

Analysing some XHProf data has revealed that the User::isCheckuser function is exceedingly slow.

The old implementation of return $this->checkuser == 1 || $this->oauthCanCheckUser() means that the OAuth identity ticket will be checked on every call of this method, which involves an unserialize, date cache check, array search, and a possible remote fetch.

Needless to say, this is something which we don't care if it changes within a request, so caching this is appropriate.
This, unfortunately, isn't really possible for me to test locally.

Codecov Report

Merging #420 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #420   +/-   ##
=======================================
  Coverage   20.41%   20.41%
=======================================
  Files           9        9
  Lines         583      583
=======================================
  Hits          119      119
  Misses        464      464

Continue to review full report at Codecov.

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


More rich README.MD

https://github.com/splewis/get5-web

travis
How to use
screenshots
How do the game server and web panel communicate
describe 'make' tasks

Plus Special Thanks

travis is not supported yet
https://github.com/FlowingSPDG/get5-web-go/commit/7997f0a4487f992f376d7be12ed92b9a2b5badfa


Fix registration of DateOnly and TimeOnly comparators

Thanks for the fix!

Deployed:  Deployed:  https://www.nuget.org/packages/CompareNETObjects/4.81.0


GATEWAY: Max Amount in farmWithdraw inconsistence on opening same popup again on same time.

navigate to earn / farm

click on - button

observe existing max value seems stake value

close the modal

click on same withdraw button again

observe existing max Value seems 0.00

In next iteration it will get set to 0.00 till the time we reload the browser and withdraw any other stake token.

not an issue!


error [TypeError: t.replaceAll is not a function]

Document {
pageContent: 'But, it added, this “is not a case in which the record contains \n' +
'sufficient  factual  findings  upon  which  we  could  base  a  de\n' +
'novo assessment of Google’s affirmative defense of fair use.” \n' +
'Id., at 1377.  And it remanded the case for another trial on \n' +
'that question.  Google petitioned this Court for a writ of cer-\n' +
'tiorari, seeking review of the Federal Circuit’s copyrighta-\n' +
'bility determination.  We denied the petition.  Google, Inc. \n' +
'v. Oracle America, Inc., 576 U. S. 1071 (2015).\n' +
'On remand the District Court, sitting with a jury, heard\n' +
'evidence  for  a  week.   The  court  instructed  the  jury  to  an-\n' +
'swer one question: Has Google “shown by a preponderance\n' +
'of the evidence that its use in Android” of the declaring code',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: '11 Cite as:  593 U. S. ____ (2021) \n' +
'Opinion of the Court \n' +
'and organizational structure contained in the 37 Sun Java\n' +
'API  packages  that  it  copied  “constitutes  a  ‘fair  use’  under  \n' +
'the Copyright Act?”  App. 294.  After three days of deliber-\n' +
'ation the jury answered the question in the affirmative.  Id., \n' +
'at 295.  Google had shown fair use.\n' +
'Oracle  again  appealed  to  the  Federal  Circuit.    And  the  \n' +
'Circuit again reversed the District Court.  The Federal Cir-\n' +
'cuit assumed all factual questions in Google’s favor.  But, it \n' +
'said, the question whether those facts constitute a “fair use” \n' +
'is a question of law.  886 F. 3d, at 1193.  Deciding that ques-\n' +
'tion of law, the court held that Google’s use of the Sun Java \n' +
'API was not a fair use.  It wrote that “[t]here is nothing fair \n' +
'about taking a copyrighted work verbatim and using it for \n' +
'the same purpose and function as the original in a compet-\n' +
'ing  platform.”  Id.,  at  1210.   It  remanded  the  case  again,',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: 'about taking a copyrighted work verbatim and using it for \n' +
'the same purpose and function as the original in a compet-\n' +
'ing  platform.”  Id.,  at  1210.   It  remanded  the  case  again,\n' +
'this time for a trial on damages.\n' +
'Google then filed a petition for certiorari in this Court.  It \n' +
'asked us to review the Federal Circuit’s determinations as \n' +
'to  both  copyrightability  and  fair  use.    We  granted  its\n' +
'petition. \n' +
'III \n' +
'A \n' +
'Copyright and patents, the Constitution says, are to “pro-\n' +
'mote  the  Progress  of  Science  and  useful  Arts,  by  securing  \n' +
'for  limited  Times  to  Authors  and  Inventors  the  exclusive  \n' +
'Right to their respective Writings and Discoveries.”  Art. I, \n' +
'§8, cl. 8.  Copyright statutes and case law have made clear\n' +
'that copyright has practical objectives.  It grants an author\n' +
'an  exclusive  right  to  produce  his  work  (sometimes  for  a  \n' +
'hundred years or more), not as a special reward, but in or-\n' +
'der to encourage the production of works that others might',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: 'an  exclusive  right  to  produce  his  work  (sometimes  for  a  \n' +
'hundred years or more), not as a special reward, but in or-\n' +
'der to encourage the production of works that others might \n' +
'reproduce  more  cheaply.    At  the  same  time,  copyright  \n' +
'has  negative  features.    Protection  can  raise  prices  to  con-\n' +
'sumers.  It can impose special costs, such as the cost of con-\n' +
'tacting owners to obtain reproduction permission.  And the',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: '12                   GOOGLE                   LLC                   v. ORACLE AMERICA, INC. \n' +
'Opinion of the Court \n' +
'exclusive rights it awards can sometimes stand in the way\n' +
'of others exercising their own creative powers.  See gener-\n' +
'ally Twentieth Century Music Corp. v. Aiken, 422 U. S. 151, \n' +
'156 (1975); Mazer v. Stein, 347 U. S. 201, 219 (1954).\n' +
'Macaulay  once  said  that  the  principle  of  copyright  is  a\n' +
'“tax on readers for the purpose of giving a bounty to writ-\n' +
'ers.”  T. Macaulay, Speeches on Copyright 25 (E. Miller ed. \n' +
'1913).  Congress, weighing advantages and disadvantages, \n' +
'will  determine  the  more  specific  nature  of  the  tax,  its  \n' +
'boundaries and conditions, the existence of exceptions and \n' +
'exemptions, all by exercising its own constitutional power \n' +
'to write a copyright statute. \n' +
'Four provisions of the current Copyright Act are of par-\n' +
'ticular relevance in this case. First, a definitional provision',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: 'to write a copyright statute. \n' +
'Four provisions of the current Copyright Act are of par-\n' +
'ticular relevance in this case. First, a definitional provision \n' +
'sets forth three basic conditions for obtaining a copyright.\n' +
'There must be a “wor[k] of authorship,” that work must be\n' +
'“original,” and the work must be “fixed in any tangible me-\n' +
'dium of expression.”  17 U. S. C. §102(a); see also Feist Pub-\n' +
'lications, Inc. v. Rural Telephone Service Co., 499 U. S. 340, \n' +
'345  (1991)  (explaining  that  copyright  requires  some  origi-\n' +
'nal “creative spark” and therefore does not reach the facts \n' +
'that a particular expression describes). \n' +
'Second, the statute lists certain kinds of works that cop-\n' +
'yright can protect.  They include “literary,” “musical,” “dra-\n' +
'matic,”   “motion   pictur[e],”   “architectural,”   and   certain   \n' +
'other  works.    §102(a).    In  1980,  Congress  expanded  the\n' +
'reach of the Copyright Act to include computer programs. \n' +
'And it defined “computer program” as “‘a set of statements',
metadata: { source: 'docs/18-956_d18f.pdf' }
},
Document {
pageContent: 'other  works.    §102(a).    In  1980,  Congress  expanded  the\n' +
'reach of the Copyright Act to include computer programs. \n' +
'And it defined “computer program” as “‘a set of statements \n' +
'or  instructions  to  be  used  directly  or  indirectly  in  a  com-\n' +
'puter in order to bring about a certain result.’”  §10, 94 Stat.\n' +
'3028 (codified at 17 U. S. C. §101).\n' +
'Third, the statute sets forth limitations on the works that \n' +
'can  be  copyrighted,  including  works  that  the  definitional\n' +
'provisions  might  otherwise  include.  It  says,  for  example,\n' +
'that copyright protection cannot be extended to “any idea, \n' +
'procedure,  process,  system,  method  of  operation,  concept,',
metadata: { source: 'docs/18-956_d18f.pdf' }
}
]
before embedDocuments
error [TypeError: t.replaceAll is not a function]
/Users/truffles/Library/Mobile Documents/com~apple~CloudDocs/Data Science/My Projects/gpt4-pdf-chatbot-langchain/scripts/ingest-data.ts:56
throw new Error('Failed to ingest your data');
^
[Error: Failed to ingest your data]
Node.js v18.15.0

I tried to change replaceAll to replace in chat.ts, still the same issue

fixed it already. thanks

fixed it already. thanks

What was the fix? I'm, getting the same error.


table 1514 "Sent Notification Entry" OnNewRecordOnBeforeInsertSentNotificationEntry

Can we please get the following two events?
procedure NewRecord(NotificationEntry: Record "Notification Entry"; NotificationContent: Text; NotificationMethod: Option)
var
    OutStream: OutStream;
begin
    Clear(Rec);
    OnNewRecordOnBeforeTransferFields(NotificationEntry); //NEW
    TransferFields(NotificationEntry);

    ....

    "Sent Date-Time" := CurrentDateTime;
    OnNewRecordOnBeforeInsertSentNotificationEntry(NotificationEntry, NotificationContent, NotificationMethod, Rec); //NEW
    Insert(true);
end;

[IntegrationEvent(false, false)]
local procedure OnNewRecordOnBeforeTransferFields(var NotificationEntry: Record "Notification Entry")
begin
end;

[IntegrationEvent(false, false)]
local procedure OnNewRecordOnBeforeInsertSentNotificationEntry(NotificationEntry: Record "Notification Entry"; NotificationContent: Text; NotificationMethod: Option; var SentNotificationEntry: Record "Sent Notification Entry")
begin
end;

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


Tutors?

Is your feature request related to a problem? Please describe.
Add account/perm level for tutors which are associated with people.
People can then look at their tutor group to see house points etc.

Perm level might not be needed, just a way to link together tutor groups.


Consolidate batching

Took a stab at merging the Reorderer, the Grouper and the batching loader into one object. I think the code looks much more cleaner now, especially for generate_until after removing the nested loops and consolidating it into a flat structure. Let me know what you think! Not entirely committed as was just thinking this might bring in a bit of complexity (esp. for debugging!) but I like that it's streamlined and easy to parse now.
closes #1140
@haileyschoelkopf

I like this a lot! Will leave some nits probably,
Two things that'd make this better if you're up for it:

Could you possibly write a simple unit test for this utils function?
Would you be willing to run this on a task of your choice before and after the change, using --log_samples, then diff-ing the resulting output files? An unchanged score on tasks would hide a potential shuffling of which outputs are assigned to which docs. I don't think this code makes that error but would be good to check

Now that we have this, we should consider seeing if we can abstract away the boilerplate in _loglikelihood_tokens and the three request type functions in later changes.

Hey @haileyschoelkopf! Added the unit test and here are the results for pythia-1B:

Task
Loader Type
Score
Stderr

gsm8k (accelerate DDP, N=2)
Collator
0.0136
0.0032

gsm8k  (accelerate DDP, N=2)
Reorderer
0.0136
0.0032

arc_easy
Collator
0.5699
0.0102

arc_easy
Reorderer
0.5699
0.0102

I also checked the diff between the two jsonl's generated and they are exactly the same. Let me know if you have more comments!

Thanks so much as always! Should be able to read through and approve tomorrow.

Thanks very much for the contribution!
We should do a pass where we switch over to this Collator and phase out Reorderer fully at some point, but that's not urgent at all.
A good time for us to do this would be if we refactor some of the boilerplate loops out of the LM classes into the base class.


Support NumLock in SDL key mapping function

Closes #2728 (in the SDL case).

[ ] Needs testing on Apple devices (macOS + iOS with external keyboard)

In order to properly support the behaviour of NumLock that is in line with user expectations, check the state of NumLock when unmapping SDL's scancodes and return the appropriate alternative keys for keypad numbers and the comma key when NumLock is off.
I can imagine this could be wrong on Apple platforms as they don't have the notion of a NumLock key, hence I'd be grateful if this could be checked. I don't really want to pepper in platform checks unless I absolutely have to.
Briefly considered not using HasFlag due to its memory concerns but I don't necessarily think it's the hottest path out there and there are way more worrying places that use HasFlag already performance-wise, so for now I opted for readability. Can be cleaned up at a later stage if deemed worthwhile, I think.
Tested on Ubuntu 18.04 & Windows 10.

in macOS (not sure about iOS), yeah; this is bringing the numlock functionality even though the OS doesn't support it, SDL mistakes the numlock modifier key for the clear key in apple's (+ numeric pads) keyboards.
I would expect SDL to not count switching numlock modifier on and off on macOS, but unfortunately, they do. I guess fixing it locally here would be easier and faster.

I was aware that SDL2 clumps up NumLock and Clear into one scancode already, but the fact that pressing it toggles the KMOD_NUM flag is... weird. I'm not sure what the Clear key even does at this point to be fair, and googling isn't helping. Some sources even suggest that Clear is apple's NumLock.

The clear key in macOS is just a regular key used in calculators to clear what's on screen, then what's on memory (the C/AC buttons):

(apologies for the watermark, the last two pressed keys are the clear keys)

Some sources even suggest that Clear is apple's NumLock.

I believe these are about how you switch numlock on in windows OS with an apple keyboard (when using bootcamp, to run windows in mac systems)

This looks to work correctly on macOS. Not really going to test on iOS as that is quite an outlier case already, but in theory it should work there.
The only weird thing I've noticed is that apple keyboard with numpads have a dedicated = key, which isn't in the osuTK Keys enum. Unfortunately I can't find anything sensible to map it to, so I think we should just leave it for the time being.


Update index.md

@ricksentam I'm confused on this edit -- why comment out the metadata?

This looks more like an experiment than constructive work to me (bordering on destructive changes).
I recommend closing this PR with a note that only productive and documented pull requests will be accepted, whereas undocumented changes will not be accepted unless confirmed or provided by Microsoft employees.

#please-close

Test commit

I apologize , it maybe a mistake. And not sure if my acc got security
issue.
「Marty Hernandez Avedon<EMAIL_ADDRESS>2020年6月30日 週二，下午10:30
寫道：

Closed #6957
https://github.com/MicrosoftDocs/windows-itpro-docs/pull/6957.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/windows-itpro-docs/pull/6957#event-3498059651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APHMPZSBOYXAQOU7Q5XFTT3RZHZGTANCNFSM4N5W35ZQ
.

--
Ricksen Tam


Pretty Printing of Lists and Dicts

It would be useful for teh user, if when printing a nested list of lists, if some sort of indentation could be used to make it easier to see the structure.
If the list is large, we could also consider not printing the whole thing... Maybe the first 20 in the list or so.
[
[1,2,3],
[4,5,6],
[
[7,8],
[1,2]
],
....
]
Since we are generating html, we could still provide a "raw" link, to see all the raw data
Same for dicts

I did some indentation, but it got lost


Add the ability to exclude persons or channels from results

Add the ability to exclude persons or channels from search results by the "top" command.
At least add the possibility for each request, better: as a user setting.
:bimorf: :)

Closed by #55


react-table library does not list babel-loader as peer dependency but seems to need it

In a newly created Create-react app with react 17, I have followed your documentation to create a first table (react-table "7.6.2"), but it fails to compile message is  -unable to resolve ...\node_modules\babel-loader\lib\index.js.  (react-table was installed using yarn add react-table).
I am aware that React team have recently made some changes to babel integration, but puzzled about my error because babel dependencies in your project were only devDependencies and also none of these in your package.json  actually list babel-loader.
What are the necessary peer dependencies or recommended CRA setup to use your library with latest create-react-app setup?  (It seems that babel also requires other config files to be modified and I don't want to break CRA code because code examples can often be out of date too).
Detail
It fails when I add this function component from your Quick start guide document code to my project
`import { useTable } from 'react-table'
import React from 'react'
function ReactDemoTable() {
const data = React.useMemo(
() => [
{
col1: 'Hello',
col2: 'World',
},
{
col1: 'react-table',
col2: 'rocks',
},
{
col1: 'whatever',
col2: 'you want',
},
],
[]
)
const columns = React.useMemo(
() => [
{
Header: 'Column 1',
accessor: 'col1', // accessor is the "key" in the data
},
{
Header: 'Column 2',
accessor: 'col2',
},
],
[]
)
const {
getTableProps,
getTableBodyProps,
headerGroups,
rows,
prepareRow,
} = useTable({ columns, data })
return (
<table {...getTableProps()} style={{ border: 'solid 1px blue' }}>

{headerGroups.map(headerGroup => (
<tr {...headerGroup.getHeaderGroupProps()}>
{headerGroup.headers.map(column => (
<th
{...column.getHeaderProps()}
style={{
borderBottom: 'solid 3px red',
background: 'aliceblue',
color: 'black',
fontWeight: 'bold',
}}
>
{column.render('Header')}

))}

))}

<tbody {...getTableBodyProps()}>
{rows.map(row => {
prepareRow(row)
return (
<tr {...row.getRowProps()}>
{row.cells.map(cell => {
return (
<td
{...cell.getCellProps()}
style={{
padding: '10px',
border: 'solid 1px gray',
background: 'papayawhip',
}}
>
{cell.render('Cell')}

)
})}

)
})}

)
}
export default ReactDemoTable`

False Alarm,
was really puzzled because create react app states:

You don’t need to install or configure tools like webpack or Babel. They are preconfigured and hidden so that you can focus on the code.
Create a project, and you’re good to go."

Solution turned out to be run : yarn clear cache, followed by yarn install and yarn upgrade.  Might be overkill but it now works.
By the way Tanner, your V7 documentation is great.


mitochondrial transcription factor activity

Hello,
There are 3 terms describing mitochondrial transcription factors, but as far as I can tell there is only one or two specific transcription factor (in all eukaroytoes). So I plan to merge:
-GO:0001144 'transcription factor activity, mitochondrial proximal promoter sequence-specific binding' and
. GO:0001143 'transcription factor activity, mitochondrial RNA polymerase core promoter sequence-specific DNA binding'
-> Merge into parent (GO:0001142 'mitochondrial RNA polymerase transcription factor activity, sequence-specific DNA binding')
see also new reference (3D structure): PMID:29149603

As well, for  GO:0001146 'transcription factor activity, mitochondrial RNA polymerase terminator site sequence-specific binding', there is no evidence that the function exists, see PMID:29033127;
mitochondrial termination seems spontaneous in the absence of anti-termination factor, see PMID:25690892 (and also PMID:29033127). Therefore to avoid any confusion I plan to obsolete the term.
There are no annotations to either of these 4 terms.
@krchristie @ValWood @thomaspd do you have any feedback before I proceed?
Thanks, Pascale

I'm  still confused what the term
GO:0001142    mitochondrial RNA polymerase transcription factor activity, sequence-specific DNA binding
would be used to annotate
https://www.ebi.ac.uk/QuickGO/annotations?goUsage=descendants&goUsageRelationships=is_a,part_of,occurs_in&goId=GO:0001142
it has 3 annotations. One of them
http://www.uniprot.org/uniprot/Q23272  I'm not sure that this acts as a transcription factor in the mitochondria? The description says it upregulates mitochondrial gene expression by regulating
"complexes via the up-regulation of chaperone and assembly factor genes" but these are nuclear transcribed.
This is the only experimental annotation.....

Yeah I disputed Q23272 , but the paper does state that "Mitochondrial ATFS-1 binds mtDNA and limits accumulation of mtDNA-encoded mRNAs" - it seemed unusual but correct. What do you think ?
Thanks, Pascale

From this quote from the paper (PMID:25773600) used for the experimental annotation of ATFS-1:

ATFS-1 binding to mtDNA was specifically enriched during mitochondrial stress (Figure 5B), which required ATFS-1-specific antibodies (Figure 5B) and was reduced in atfs-1(tm4525) worms (Figure S5B). While the regulatory segments of C. elegans mtDNA have not been extensively characterized, ATFS-1 binding was most enriched in an A-T-rich segment lacking protein or RNA coding genes (Figure 5A) suggested to be the regulatory region or D-loop (Bratic et al., 2010). Interestingly, this segment contained a UPRmtE in the region where ATFS-1 binding was most enriched (Figures 5A and 5C).

it does seem to me that it is correct for the protein to be annotated to the term "mitochondrial RNA polymerase transcription factor activity, sequence-specific DNA binding" (GO:0001142).
In addition, I think that this GO term is also appropriate for TFAM (though the mouse annotations do not currently reflect this and I will check some papers to see if I can make mouse annotations).
Here's a quote from: Kang I, Chu CT, Kaufman BA. The mitochondrial transcription factor TFAM in
neurodegeneration: Emerging evidence and mechanisms. FEBS Lett. 2018 Jan 24. PMID:29364506

The role of TFAM in regulating mtDNA replication, transcription, and packaging
TFAM was discovered as a mitochondrial protein that stimulated transcription of model
templates in mitochondrial extracts [39].  TFAM is a nuclear-encoded protein that is
synthesized in the cytoplasm and imported into the mitochondria [40]. The protein
contains two high mobility group (HMG) box domains that insert into the DNA minor groove
on the light strand promoter (LSP), heavy strand promoter 1 (HSP1), or non-specific
regions of the mtDNA [41-43]. TFAM-binding cause s a distortion of the mtDNA and result s
in DNA bending [42-46].  This distortion enables the specific binding of POLRMT
(mitochondrial RNA polymerase) to the start site, where TFAM binds to the N-terminus of
POLRMT to recruit TFB2M (mitochondrial Transcription Factor 2) while maintaining an open
DNA complex arrangement that enables productive transcription initiation [47-51]

@pgaudet - Regarding your plan to obsolete this term:

As well, for GO:0001146 'transcription factor activity, mitochondrial RNA polymerase terminator site sequence-specific binding', there is no evidence that the function exists, see PMID:29033127;
mitochondrial termination seems spontaneous in the absence of anti-termination factor, see PMID:25690892 (and also PMID:29033127). Therefore to avoid any confusion I plan to obsolete the term.

The papers you cite do indicate that TEFM binds the mitochondrial termination site and promotes elongation through it, i.e. antitermination. If you obsolete this term, how do you plan to represent this activity of binding to the terminators and promoting transcription elongation?

Hello,
What about this:
(1) Merge

'GO:0001142 mitochondrial RNA polymerase transcription factor activity, sequence-specific DNA binding'
-- 'GO:0001144 transcription factor activity, mitochondrial proximal promoter sequence-specific binding'
-- 'GO:0001143 transcription factor activity, mitochondrial RNA polymerase core promoter sequence-specific DNA binding'

-- GO:0034246 mitochondrial RNA polymerase promoter specificity activity"
into the parent above 'mitochondrial RNA polymerase transcription factor activity, sequence-specific DNA binding', and rename
'mitochondrial sequence-specific DNA binding transcription factor activity'
(2) Merge "GO:0001146 transcription factor activity, mitochondrial RNA polymerase terminator site sequence-specific binding" (0 annotations)  into "GO:0001145 mitochondrial RNA polymerase termination site sequence-specific DNA binding" (0 annotations)
(3) If needed for MTF1 we can add a new BP "transcriptional open complex formation at mitochondrial promoter"
@krchristie Would that work?
Thanks, Pascale

update definition GO:0001145 mitochondrial RNA polymerase termination site sequence-specific DNA binding: to "Interacting selectively and non-covalently with a DNA sequence that controls mitochondrial transcriptional termination."

Hi @pgaudet
I think this sounds OK, provided that it includes moving the term GO:0034246 (currently called "mitochondrial RNA polymerase promoter specificity activity") to no longer be under GO:0000996 "promoter selection factor activity"
I can't really tell from this where GO:0034246 will be in your plan.
-K

Hi @krchristie
yes I renamed the term to 'mitochondrial sequence-specific DNA binding transcription factor activity'
Thanks, Pascale

@pgaudet - I was really wondering what parentage this term is going to have, since it shouldn't be under the 'promoter selection factor activity' term.

No, I moved it.

Perfect :) Thank you


typo: "implementation"

Closes #1006

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b856eb9817c95af3ec441b327c19e19b4fa028ce/


runInExternalNativeTerminal not working

runInExternalNativeTerminal not working, when i set it to true, test is still run in internal terminal

can you provide which terminal and which os you are using.

can you provide which terminal and which os you are using.

Macos, Warp

This is true for me too, also on Mac OS; using native mac Terminal.app or my preferred iTerm.app. I also tried setting the "Terminal: Explorer kind" setting to "external" in case that made a difference; it didn't.

Hello! Any updates on this? I also use Wrap as my default/preferred terminal and this is pretty annoying...


How to put different URIs for different method under one Resource

When I am having multiple methods under one swagger (for example AccountManagement) , then how to put those URIs per method specific.
Method 1 : GET : http://host/api/accountManagement/getAccount?id=1
Method 2 : GET: http://host/api/accountManagement/getAccountBalance?account.id=11
As I can set URIs on resource level, but for above scenario I have to create two resource? Is this the correct way or I am doing something wrong? Could anyone guide me.

Hi @subhasis82.
In this scenario you should create two resources and each resource should have a GET method.
I hope this answers your question.
Don't hesitate to write again if you have any further questions.


[SDK-342] Preload asset json strategy

SDK-342
Description

Added preloading of all assets json
Added downloading of icon only when a category is selected
Renamed assetType to category

How to Test

Play the sample

Checklist

[ ] Tests written or updated for the changes.
[ ] Documentation is updated.
[ ] Changelog is updated.

In Unreral I throw an error with  an error message before opening the AvatarCreator if the AppId is not set.

In Unreral I throw an error with an error message before opening the AvatarCreator if the AppId is not set.
I will add the same error.


"How to secure device code" Improved grammar and created end links.

Increased Acrolinx score from 78 to 100.

@eriklindeman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

Note that the validation warnings are unrelated to the changes in this PR.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Add travis CI

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@07156f2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #3   +/-   ##
=========================================
  Coverage          ?   85.71%
=========================================
  Files             ?        5
  Lines             ?      217
  Branches          ?        0
=========================================
  Hits              ?      186
  Misses            ?       31
  Partials          ?        0

Continue to review full report at Codecov.

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