Added my app credentials in index.json file

Pull Request Title
Description
Please include a summary of the change or the feature being introduced. Include relevant motivation and context. List any dependencies that are required for this change.

[x] This change requires a documentation update
[x] I have made corresponding changes to the documentation

Related Issue(s)
This PR addresses issue #X
Screenshots
If relevant, please add screenshots.
Notes
Please add any relevant notes if necessary.

Looks like you included an extra entry, the PR should be (+5, -0) lines like this one #1518

I have updated the index, please check now.

Updated index.json

@ShahriarHim Please resolve the conflict


RHOAIENG-14811 The userdID was not correctly transferred to the frontend

Closes https://issues.redhat.com/browse/RHOAIENG-14811
We now get the userID from the status api endpoint if set.
This builds on RHOAIENG-9480/PR #3107.
This closes the first try in #3362
Description
The previous code did not correctly pass the userID from the backend around in the frontend.
The backend already has it correctly on a 2.14 cluster and provides it via /api/status call.
How Has This Been Tested?
Manual test in dev-mode running npm run start:dev:ext against an instance on Dev-Sandbox and a non-Sandbox one.
Then looking at the console output for the the Identify event
Identify event triggered: {... "userID":"b09067a69...99"}

If the userID looks like this (whith chars) it is an anonymous one. If it is numeric (may be negative) and short (probably < 8 digits), then it is a web-user-id.
Request review criteria:
Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

cc @andrewballantyne

/lgtm
/approve
tested against my dev sandbox instance and another odh cluster instance. Sandbox cluster included my numeric user ID.


Switching modes

Hi,
I am currently working with a MIR 250. I want to use the MIR in a teleoperative state, where I can steer it with the keyboard of my laptop. So far most of the functions are working, but I got a problem. When I put the MIR into manual mode and don't give an input after ca. 5sec. it leaves the manual mode and goes back into the automatic mode. Is there a workaround for my problem or an option how I can increase the duration in until it switches modes.
Appreciate the help, great work from you guys.

Hi,
that seems to be an intentional feature of the web interface that was added in one of the later versions of the MiR software. I haven't found a way to turn it off, sorry. Please note that this repo is a community project to use the MiR Robots with ROS. It is not affiliated with Mobile Industrial Robots. Perhaps you could ask Mobile Industrial Robots support about it?

Hi, I am currently working with a MIR 250. I want to use the MIR in a teleoperative state, where I can steer it with the keyboard of my laptop. So far most of the functions are working, but I got a problem. When I put the MIR into manual mode and don't give an input after ca. 5sec. it leaves the manual mode and goes back into the automatic mode. Is there a workaround for my problem or an option how I can increase the duration in until it switches modes.
Appreciate the help, great work from you guys.

Hi fdumaire, we have the same behavior on our MIR 250, have you found a way to solve the problem or work around it?  Thanks!


Poetry Install crashes for python-preview branch.

Describe the bug
The poetry install command fails at debugpy installation. Possibly due to hashing error on 'debugpy/_vendored/pydevd/pydevd_attach_to_process/run_code_on_dllmain_x86.dll'.
To Reproduce
Steps to reproduce the behavior:

Checkout python-preview branch: git checkout python-preview
Change directory to python: cd python
Install Poetry: pip install poetry
Run Poetry installation: poetry install
Should install most but stops and fails at debugpy.

Expected behavior
I was walking through the README.md tutorial for python setup in the branch. Expected the install to go through nicely.
Log
This is the log after a second install attempt.
Package operations: 7 installs, 0 updates, 0 removals

  • Installing debugpy (1.6.6)

  _WheelFileValidationError

  ["In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_cython.cp310-win_amd64.pyd didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/_pydevd_frame_eval/pydevd_frame_evaluator.cp310-win_amd64.pyd didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/attach_amd64.dll didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/attach_x86.dll didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/inject_dll_amd64.exe didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/inject_dll_x86.exe didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/run_code_on_dllmain_amd64.dll didn't match RECORD", "In C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache\\artifacts\\f3\\fe\\7e\\ae79971a5d18da24266b2396f07920898302fe5605d3467a4c20e44be7\\debugpy-1.6.6-cp310-cp310-win_amd64.whl, hash / size of debugpy/_vendored/pydevd/pydevd_attach_to_process/run_code_on_dllmain_x86.dll didn't match RECORD"]

  at ~\AppData\Local\Programs\Python\Python310\lib\site-packages\installer\sources.py:289 in validate_record
      285│                         f"In {self._zipfile.filename}, hash / size of {item.filename} didn't match RECORD"
      286│                     )
      287│
      288│         if issues:
    → 289│             raise _WheelFileValidationError(issues)
      290│
      291│     def get_contents(self) -> Iterator[WheelContentElement]:
      292│         """Sequential access to all contents of the wheel (including dist-info files).
      293│

Desktop (please complete the following information):

OS [dxdiag]: Windows 10 Pro 64-bit Build 19044
Python: Python 3.10.6
Pip: 23.0.1
VSCode:

1.76.2
ee2b180d582a7f601fa6ecfdad8d9fd269ab1884
x64

@alexchaomander , can you take a look please?

Same problem for python 3.11.2

@alexchaomander
Just tried it, didn't work. Might work for other people though.
For other people who want to try on windows:
poetry config --list
Should give you this:
cache-dir = "C:\\Users\\peter\\AppData\\Local\\pypoetry\\Cache"
experimental.new-installer = true
experimental.system-git-client = false
installer.max-workers = null
installer.modern-installation = true
installer.parallel = true
virtualenvs.create = true
virtualenvs.in-project = null
virtualenvs.options.always-copy = false
virtualenvs.options.no-pip = false
virtualenvs.options.no-setuptools = false
virtualenvs.options.system-site-packages = false
virtualenvs.path = "{cache-dir}\\virtualenvs"  # C:\Users\peter\AppData\Local\pypoetry\Cache\virtualenvs
virtualenvs.prefer-active-python = false
virtualenvs.prompt = "{project_name}-py{python_version}"

Then you can try manually deleting the pypoetry folder with the cache-dir.

I can reproduce this now! (I run an ubuntu desktop and WSL normally, but setup on regular Windows to reproduce this.)
This issue seems to be the culprit: https://github.com/microsoft/debugpy/issues/1246
For now, it seems the workaround is:
poetry config installer.modern-installation false --local

After that, you should be able to complete:
poetry install

poetry shell

python

And, in the python terminal, do something like import semantic_kernel as sk.


the KITTI epe results

Hi, haofei
I want to know how to get  the KITTI2015 epe results of Table1 in your paper?  Just execute the second finetune step(whole set or just 160 images?) in 4.2 section or execute the two steps(the mix training set and separate training sets)? Thank you so much!

@haofeixu Hi,haofei, I have downloaded the pretrained model trained on Sceneflow. But when I finetune the model on 160images of kitti2015, I can not get the epe results(1.4+instead of 0.68) shown in your paper. I am very confused about the my process. Can you tell me the detailed finetune method? Thank you very much!

How to evaluate the data set of kitti15? kitti did not give gt in the test folder.

The results in Table 1 is obtained on KITTI training set. You can find this information from our paper.


Required multiple values

[Option('i', "input", Required = true, Min = 1)]
public IEnumerable< string > Inputs { get; set; }
If no input is provided, it complains that input is missing
[Option('i', "input", Required = true, Min = 1)]
public string[] Inputs { get; set; }
Will not complain and crash later in the code because Inputs is null. (I did not check as it should be impossible to get there...)

Good catch, we should fix the error message there. The real error for the second should be Non scalar properties should be sequence of type IEnumerable<T>., aka "we don't support multiple values on a string array"

Resolved
The generated Error in HelpText:

ERROR(S):
Option 'i, input' has no value.
Required option 'i, input' is missing.


BUG: type Suitesparse_long to be UF_long.

This fixes the manylinux problems on 64 bit manylinux. There are still problems with 32 bit manylinux, but those are related to Python code that I did not write.

Build matrices:
https://travis-ci.org/scikit-umfpack/scikit-umfpack-wheels/builds/288385238
https://ci.appveyor.com/project/rc/scikit-umfpack-wheels/build/1.0.34

Thanks! Note that https://github.com/scipy/scipy/pull/7939 is needed with the current scikit-umfpack. Maybe the remaining errors are related?

What is the size of np.intc on the failing platforms?

Would that be sys.getsizeof(np.intc)?

Update: this PR should fix everything if scipy 1.0 is installed (which it will be).

Thanks! As for the size of np.intc, I just meant printing it:
In [5]: np.intc
Out[5]: numpy.int32

I think this is ready to merge (though you should remove the explcit scipy==1.0.0rc1 once scipy comes out.

Yes, thank you, merging.


Remove calls to log.Println

These end up spamming the stdout of my app and don't actually provide any useful information.

Logging should be directed to stderr by default, not stdout.  If it is spamming stdout, then I'd suggest looking at how logging is setup for the app and submit a pull request for that.
I highly recommend leaving these "debug" statements, as that is exactly what they are.
If they are spamming your console, and they are indeed being outputted to stderr like they should, then just ignore them.
2>/dev/null

stderr or whatever, these messages provide no use and there's no method to turn them off in this library. The go log package doesn't provide a way to filter out log messages from different sources, and if I am using it in my application to log actual useful output there's no point in cluttering it up with messages that provide no value.
For example why do I need it to log that hid_init is being called?

What do you think about a var DebugLogger func(log string) that is set to discard logs by default and can be overrided to print them to stdout, stderr, or whatever? @kisielk @eduncan911

I've gone through quite a lot of "package-level" logging patterns myself as I am serious advocate for logging (that's why I chimed in to begin with :+1: ):
http://go-talks.appspot.com/github.com/eduncan911/goslides/gologit.slide#1
What I find best, that allows the consuming executable to just completely ignore logging if they wish, is to allow it to be injected into the constructor using D.I. principals.  For example, a quick and dirty hack (which I don't recommend, but shows the point):
// go/hid package change

// global var
var _logger *log.Logger

// note the variadic parameter at the end, making it optional
func Open(vendorId uint16, productId uint16, serialNumber string, logger ...log.Logger) (*Device, error) {
  ...
  for _, debugLogger := range logger {
    // set global logger
    _logger = &debugLogger
  }
}

The for loop is only executed when the variadic parameter is set (a trick I use).
Then the consuming code can safely ignore it, which doesn't break existing consumers:
// ignored, and doesn't break existing codebases
leds, err := hid.Open(0x1770, 0xff00, "")

And here's an example of using the optional debugger:
// create a logger
myDebugLogger := ...

// pass it in last
leds, err := hid.Open(0x1770, 0xff00, "", myDebugLogger)

Personally though, I like Rob Pike's functional parameters blog post about the pattern others have come up with:
http://commandcenter.blogspot.com/2014/01/self-referential-functions-and-design.html
The key to this pattern is it allows you to add additional options in the future without breaking APIs.
I have been using this "Function Parameters" personally in all of my projects and at work.  People really like having things available in intellisense.
Problem is, it usually breaks existing codebases that didn't start that way.
For go.hid, I would suggest tweaking hid.Open() to allow these optional parameters to be passed in.  This would keep existing consumers from breaking, without making the API overly complex with additional New()-like constructors.  K.I.S.S. it, right?
I'd do something like this...
var _logger *log.Logger

func Open(vendorId uint16, productId uint16, serialNumber string, options ...func(*Device)) (*Device, error) {
    ...
    // instantiate as normal
    dev := &Device{
        hidHandle: hidHandle,
    }

    // now, set the user-defined options on the device
    for _, option := range options {
        option(dev)
    }

    // done
    return dev, nil
}

...

// Logger sets the internal instance for logging
func (dev *Device) SetLogger(logger *log.Logger) {
    // currently global as the package seems to support only 1 device
    _logger = logger
}

// log is our own internal `log()` method that safely sets, or not set, log messages:
func log(msg string, args ...interface{}) {
    if _logger == nil {
        return
    }

    m := msg
    if len(args) > 0 {
        m = fmt.Sprintf(m, args...)
    }
    logger.Println(m)
}

This keeps backwards compatibility with existing consumers:
leds, err := hid.Open(0x1770, 0xff00, "")

... while allowing new consumers to set a custom logger (that adheres to Golang's builtin log package, which almost all do anyways):
opts := func(d *hid.Device) {
    d.SetLogger(...myCustomLogger...or...nil...)
}
leds, err := hid.Open(0x1770, 0xff00, "", opts)

What I like about this pattern is that you can add additional configurations to the Device via set functions without changing the API.  For example, setting a timeout perhaps?
If you'd like a PR, I can throw something together like this...

Personally I think that's totally over-engineering it. Why would you need to pass options in to the Open function of the device when it already returns a *Device and you can just call functions on that? Also the logger could just as easily be a member of the Device struct, and if it's nil then no logging is performed. Still currently the package only logs in 3 places and none of them provide any useful information, so what's the point of even designing for this? Logging is mostly the domain of the application and the package can communicate any errors through return values which the caller can decide to log or not.

You should see have seen my first pass, with New, OpenWithConfig and all.
:)
The point of this was to allow for additional options I was thinking, like
timeouts and event hooks/filtering.
But yeah, if you never use it for anything else but logging, then there's
no point.
On Mon, May 18, 2015 at 9:28 PM, Kamil Kisiel<EMAIL_ADDRESS>wrote:

Personally I think that's totally over-engineering it. Why would you need
to pass options in to the Open function of the device when it already
returns a *Device and you can just call functions on that? Also the logger
could just as easily be a member of the Device struct, and if it's nil then
no logging is performed. Still currently the package only logs in 3 places
and none of them provide any useful information, so what's the point of
even designing for this? Logging is mostly the domain of the application
and the package can communicate any errors through return values which the
caller can decide to log or not.
—
Reply to this email directly or view it on GitHub
https://github.com/GeertJohan/go.hid/pull/10#issuecomment-103293105.

Yeah this definitly shouldn't be too complicated. There could be a logger per-device, but nothing more fancy than that. But for now I agree with kisielk that these messages don't provide enough information to be worth logging. I'm still open to a logging/debugging solution, but it should come with log calls that provide more and detailed information.
Will merge this PR for now to solve kisielk's problem. A package shouldn't write to stdout/stderr (with the ocasional in-development exception).


Rename to simply ProjectMaker

According to packagecontrol recommendations a plugin should not contain Sublime, SublimeText or ST in its name. The context of Sublime Text is clear by the very fact that this package is a plugin.
I have renamed at least one plugin in the past and that renaming met a positive response by the users.

done

Awesome. Thanks!

Thank you. Even though I don't currently use Sublime, it's nice to see the project carry on and see new life.


layerAdd button should only show up if inlineDialogs is false

Currently I'm porting over the WAB tabbed theme to use LeftNav, but finding that the specific settings that WAB uses forces the add layer button to be visible when in LeftNav mode. In hindsight, I think it's better to explicitly handle this mode only if inlineDialogs is false.


get() method of unexecuted checkpoint not raising exception

Snakemake version
Occurs in 8.16.0 and 8.13.0, does not occur in 7.32.4.
Describe the bug
I am running into an issue where under certain circumstances, a checkpoint's get() method does not raise the expected IncompleteCheckpointException when it has not yet been run. I've been able to create minimal example below, The problem happens in the input function of the problem rule, which reads the output of the my_checkpoint checkpoint rule.
The example runs fine starting from scratch. However, if I create an empty file for the problem rule's output file, then execute a rule that takes that file as an input, the call to checkpoints.my_checkpoint.get()  succeeds even though the checkpoint has not been run.
Several warnings like the following are generated, which may or may not be related:
RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited

Minimal example
import os, sys

checkpoint my_checkpoint:
    output: 'checkpoint.txt'
    shell: 'echo 1 2 > {output}'

rule make_problem_input:
    output: 'input-{i}.txt'
    shell: 'echo data-{wildcards[i]} > {output}'

def my_input_func(wc):
    print('!!!!!!!!!! input function called', file=sys.stderr)
    checkpoint_file = checkpoints.my_checkpoint.get().output[0]
    assert os.path.exists(checkpoint_file), f'{checkpoint_file} does not exist!'
    print('!!!!!!!!!! checkpoint file exists', file=sys.stderr)
    data = open(checkpoint_file).read()
    nums = data.strip().split()
    return expand(rules.make_problem_input.output, i=nums)

rule problem:
    input: my_input_func
    output: 'problem.txt'
    shell: 'cat {input} > {output}'

rule final:
    input: 'problem.txt'
    output: 'final.txt'
    shell: 'cat {input} > {output}'

Logs
With directory empty except for the Snakefile:
snakemake --profile none final.txt

Assuming unrestricted shared filesystem usage.
Building DAG of jobs...
!!!!!!!!!! input function called
!!!!!!!!!! input function called
/home/jared.lumpe/opt/mambaforge/envs/signature-finder/lib/python3.12/site-packages/snakemake/dag.py:1612: RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited
  self.sanitize_local_storage_copies()
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Using shell: /usr/bin/bash
Provided cores: 80
Rules claiming more threads will be scaled down.
Job stats:
job              count
-------------  -------
final                1
my_checkpoint        1
problem              1
total                3

Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localcheckpoint my_checkpoint:
    output: checkpoint.txt
    jobid: 2
    reason: Missing output files: <TBD>
    resources: tmpdir=/tmp
DAG of jobs will be updated after completion.

[Fri Aug  9 16:52:43 2024]
Finished job 2.
1 of 3 steps (33%) done
!!!!!!!!!! input function called
!!!!!!!!!! checkpoint file exists
/home/jared.lumpe/opt/mambaforge/envs/signature-finder/lib/python3.12/site-packages/snakemake/dag.py:1612: RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited
  self.sanitize_local_storage_copies()
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
Select jobs to execute...
Execute 2 jobs...

[Fri Aug  9 16:52:43 2024]
localrule make_problem_input:
    output: input-2.txt
    jobid: 6
    reason: Missing output files: input-2.txt
    wildcards: i=2
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
localrule make_problem_input:
    output: input-1.txt
    jobid: 5
    reason: Missing output files: input-1.txt
    wildcards: i=1
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 6.
2 of 5 steps (40%) done
[Fri Aug  9 16:52:43 2024]
Finished job 5.
3 of 5 steps (60%) done
Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localrule problem:
    input: input-1.txt, input-2.txt
    output: problem.txt
    jobid: 1
    reason: Missing output files: problem.txt; Input files updated by another job: input-1.txt, input-2.txt
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 1.
4 of 5 steps (80%) done
Select jobs to execute...
Execute 1 jobs...

[Fri Aug  9 16:52:43 2024]
localrule final:
    input: problem.txt
    output: final.txt
    jobid: 0
    reason: Missing output files: final.txt; Input files updated by another job: problem.txt
    resources: tmpdir=/tmp

[Fri Aug  9 16:52:43 2024]
Finished job 0.
5 of 5 steps (100%) done
Complete log: .snakemake/log/2024-08-09T165243.478887.snakemake.log

Remove all files and .snakemake/ directory, touch problem.txt, and rerun:
Assuming unrestricted shared filesystem usage.
Building DAG of jobs...
!!!!!!!!!! input function called
!!!!!!!!!! input function called
InputFunctionException in rule problem in file /home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile, line 20:
Error:
  AssertionError: checkpoint.txt does not exist!
Wildcards:

Traceback:
  File "/home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile", line 14, in my_input_func (rule problem, line 40, /home/jared.lumpe/tmp/24-08-09-snakemake-bug/Snakefile)

Possibly the same as #2958

Also see #2957 for RuntimeWarning: coroutine 'DAG.sanitize_local_storage_copies' was never awaited


Add release notes on release deploy

Pull Request Test Coverage Report for Build 300

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 65.126%

Totals

Change from base Build 298:
0.0%

Covered Lines:
465

Relevant Lines:
714

💛  - Coveralls


drf-spectacular integration @extend_schema_serializer() examples not work

In drf-spectacular AutoSchema class we have function _get_examples() with following code:
#...
# don't let the parameter examples influence the serializer example retrieval
examples = [e for e in self.get_examples() if not e.parameter_only]

# Examples from Serializers via @extend_schema_serializer are only considered, if
# there were no higher priority examples directly from view annotation.
if not examples:
    if is_list_serializer(serializer):
        examples = get_override(serializer.child, 'examples', [])
    elif is_serializer(serializer):
        examples = get_override(serializer, 'examples', [])
#...

direct link: https://github.com/tfranzel/drf-spectacular/blob/master/drf_spectacular/openapi.py#L1217
Since drf-standarized-errors overrides get_examples() in overwritten AutoSchema version with examples from exceptions without parameters_only that if statement is not triggered and all @extend_schem_serializer examples are ignored.
Solution for that would be to add parameter_only value to get_example_from_exception() function:
def get_example_from_exception(exc: exceptions.APIException) -> OpenApiExample:
    if is_client_error(exc.status_code):
        type_ = "client_error"
    else:
        type_ = "server_error"
    return OpenApiExample(
        exc.__class__.__name__,
        value={
            "type": type_,
            "errors": [{"code": exc.get_codes(), "detail": exc.detail, "attr": None}],
        },
        response_only=True,
+       parameter_only=("errors", "query"),
        status_codes=[str(exc.status_code)],
    )

What do you think @ghazi-git ? If that's ok with you I can prepare PR for that.
One concern that I have is what value provide for this parameter. In docs it is only mentioned here:
https://drf-spectacular.readthedocs.io/en/latest/drf_spectacular.html#drf_spectacular.utils.OpenApiExample
and here is the list where it is used in drf-spectacular project:
openapi.py|276 col 24| if example.parameter_only:
openapi.py|277 col 41| examples_by_key[example.parameter_only].append(example)
openapi.py|1213 col 61| examples = [e for e in self.get_examples() if not e.parameter_only]

So in order to fix this issue we need to provide  basically any value to that parameter in get_example_from_exception()

Problem screenshots and example code
from rest_framework import serializers
from drf_spectacular.utils import (
  OpenApiExample,
  extend_schema_serializer,
)

@extend_schema_serializer(
  component_name="UserDetailOutputSerializer",
  examples=[
      OpenApiExample(
          "Basic example",
          value={"id": "id", "email": "mail"},
          request_only=False,
          response_only=True,
      ),
  ],
)
class OutputSerializer(serializers.Serializer[Any]):
  id = serializers.UUIDField()
  email = serializers.EmailField()
  first_name = serializers.CharField(max_length=150)
  last_name = serializers.CharField(max_length=150)

Without parameter_only

With parameter_only=('errorrs', 'query')

Thanks for reporting the issue and sorry I didn't get to this earlier.
I went ahead and pushed a fix for this issue in #69. Now, error response examples are added using the extras argument in AutoSchema._get_examples instead of adding them under AutoSchema.get_examples. Marking the examples as parameter_only is a quick workaround but those seem to be a different type of examples to support specific use cases.


Rename  'GO:0071934 thiamine transmembrane transport' to 'thiamine import across plasma membrane.

Hello,
For issue
https://github.com/geneontology/go-ontology/issues/14322
I would like to rename 'GO:0071934 thiamine transmembrane transport' to 'thiamine import across plasma membrane.
As far as I can see it's consistent with existing EXP annotations (2 different proteiny by pombase, @ValWood ):  bsu1 and thi9.
@RLovering You can then use this term to annotate the following proteins
UniProtKB Q99PL8 Slc19a3 enables GO:0015403 thiamine uptake transmembrane transporter activity
UniProtKB Q9EQN9 Slc19a2 enables GO:0015403 thiamine uptake transmembrane transporter activity
UniProtKB O60779 SLC19A2 enables GO:0015403 thiamine uptake transmembrane transporter activity
Thanks, Pascale

Hi Pascale,
I think you should keep this term as transmembrane transport, to cover movement between other compartments e.g:
http://preview.pombase.org/gene/SPBC1604.04
If you create a new term for plasma transmembrane import, I will move our EXPs to the new term.
Val

This is where I got the mitochondrial one, it does exist but it is not specifically annotated...
http://www.uniprot.org/uniprot/P53257

SPBC1604.04 is already annotated to 'mitochondrial thiamine pyrophosphate transmembrane transport'; what do you want to annotate this to then ?

I want to keep it...ignore.
as long as
'GO:0071934 thiamine transmembrane transport'  doe not merge to 'thiamine import across plasma membrane.
(it was only an explanation why there is no other EXP to GO:0071934 thiamine transmembrane transport)

I.e my source
http://www.uniprot.org/uniprot/P53257
should be annotated to
'mitochondrial thiamine pyrophosphate transmembrane transport';
mitochondrial transport Source: GO_Central
thiamine pyrophosphate transport Source: SGD
(PomBase ISS is correct, but  unsupported)

action
create
thiamine import across plasma membrane

GO:1990545
mitochondrial thiamine pyrophosphate transmembrane transport


feat: add Event Types Chips

Fixes #1561
Changes: Added event types to result fragment chips
Screenshots for the change:

@liveHarshit @iamareebjamal please review

@nikit19 review this one also

@liveHarshit @iamareebjamal anything left out ?


Mldb 2180 dataset split

New procedure to split a dataset between several while respecting the presence distribution of one or several columns.

@mailletf fixed or addressed PR comments

@mailletf @guyd Can I get a +1?


[question & help] Page Not Display at Android(4.4.2) Browser With Latest Firebox

Great repo!
I want to use this great repo to start my project working on mobile browser, BUT after downloaded master branch, installed all npm packages and start it successfully, I open my browser(Firefox version 43.0) on my Android Phone (system version is 4.4.2), The Page Is Empty.
However, Desktop Google browser works fine!!
I'm not familar with those magic configuration steps. Is it because lack of some configuration results in this problem.

Well. As far as I know, this repo won't work on mobile browser with HMR enabled. Try to compile your project first and serve those compiled file to your mobile browser

@thangngoc89 thanks! Maybe I should have a try to find why HMR cause this problem at weekend.

Sounds like this is HMR related and not specific to this kit, so closing for now. If you experience otherwise please let me know and I can look into it.


remove experimental

remove experimental from  the help of istioctl verify-install

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @esnible

@irisdingbj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
b568d9bbd6addad7b2f266c33e1d7bdc75992946
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: irisdingbj, sdake
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: esnible
If they are not already assigned, you can assign the PR to them by writing /assign @esnible in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fisherxu, irisdingbj, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS [fisherxu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn just found the other PR companion of #14688


Decouple Client HTML from Video Asset

Before the client HTML was heavily coupled to the video asset that was being served. This required the HTML to be generated each time.
In this change, the JavaScript that starts playback has been changed to read the asset resource from the url params. This allows the same client code to be used regardless of asset.
As part of this decoupling, the HTML, CSS, and JavaScript have been broken-up and moved to a client directory. This was to help them be viewable in isolation.

@colleenkhenry would you be able to verify this change? I was able to verify the client portion in isolation, but not the post to server or end-to-end.

I'll take a look :)

Hey @vaage  I'm going to add this into the next version!


cannot build mxnet

Description
git clone --recursive https://github.com/dmlc/mxnet
cd mxnet
ci/build.py --docker-registry mxnetci --platform ubuntu_build_cuda --docker-build-retries 3 --shm-size 500m /work/runtime_functions.sh build_ubuntu_gpu_cuda101_cudnn7

Error Message
2020-04-27 20:42:11,481 - root - INFO - Running command: 'docker-compose -f docker/docker-compose.yml build --build-arg USER_ID=1000 --build-arg GROUP_ID=1000 ubuntu_build_cuda'
Traceback (most recent call last):
  File "ci/build.py", line 482, in <module>
    sys.exit(main())
  File "ci/build.py", line 392, in main
    no_cache=args.no_cache, cache_intermediate=args.cache_intermediate)
  File "ci/build.py", line 140, in build_docker
    run_cmd()
  File "/home/ubuntu/mxnet/ci/util.py", line 78, in f_retry
    return f(*args, **kwargs)
  File "ci/build.py", line 138, in run_cmd
    check_call(cmd)
  File "/usr/lib/python3.6/subprocess.py", line 306, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/usr/lib/python3.6/subprocess.py", line 287, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/usr/lib/python3.6/subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "/usr/lib/python3.6/subprocess.py", line 1364, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'docker-compose': 'docker-compose'

apt install docker-compose will update doc


Small "unwrapping" quirk

Environment
Windows build number: [Version 10.0.18363.720]
Windows Terminal version (if applicable): 0.10.7810

Any other software? no

Steps to reproduce

I was using a CMD shell for a directory listing.  Some names (I used dir *.c /s/b) were longer than the window's width and were wrapped.  When I dragged the right side of the window, I was pleasantly surprised to see the long lines "unwrap" ... at least until a couple of them somehow 'grabbed' the next line from the screen and appended it to themselves.  I cannot see a way to attach a screen shot, so you can see it on my web site bad unwrap example
If you want to use the exact set of files I was using, get calibre-master from github, extract the files and issue the same command I used (shown above and in the screen shot) with the window only about 60-70 characters wide.  Unfortunately, it doesn't always occur, but I got it to happen 3 out of 5 times, with different numbers of joined lines each time.
Expected behavior

I expected each separate line of output to remain separate.
Actual behavior

When the output is "unwrapped" as the window gets wider, some lines mysteriously join with the next (or previous?) line of output to make an even longer line that is still wrapped!

Okay this is the "you print a line that's exactly the width of the terminal, and we can't tell that the line actually broke there". It might be #3088, but I'm not positive

Yes, I think this is the same thing.  Sorry for the duplicate report, but I would NEVER have EVER found it searching for size/wrap issues as I was doing.  Feel free to close this and I will got vote for the referenced issue instead.

 Thanks for following up!
/dup #3088


edit clean/update functions

I edited the update and clean functions to bring the jobs to the test as well

Please merge main into branch first


birthDate time extension cardinality in the patient resource

On the Event Header Information page there is a row within the communications table which refers to the birthDate element and "patient-birthTime" extension. The row only has one cardinality so it raises the question as to if the "patient-birthTime" extension has a cardinality of 1..1 as well as the "birthDate". The birthDate element has a cardinality of 1..1 in the fhir profile but the birth time extension has a cardinality of 0..1. Is the birth time extension mandatory or is it optional as per the fhir profile?
Date of Birth | 1..1 | CareConnect-EMS-Patient-1 | birthDate and patient-birthTime extension

I think reference to the birthtime extension can be removed from the event header information page, as this may vary depending on the use case. Are you happy to make this change in your feature branch?

Removed the birthTime extension from the guidance on the feature branch.


Change hard-coded postage value

TODO: stop hard-coding postage as 'second' once we get postage from the admin
    notification = persist_notification(
        notification_id=post_data['file_id'],
        template_id=template.id,
        template_version=template.version,
        template_postage=template.postage,
        recipient=post_data['filename'],
        service=service,
        personalisation=personalisation,
        notification_type=LETTER_TYPE,
        api_key_id=None,
        key_type=KEY_TYPE_NORMAL,
        reference=create_one_off_reference(LETTER_TYPE),
        client_reference=post_data['filename'],
        created_by_id=post_data['created_by'],
        billable_units=billable_units,
        postage='second',
    )

Hey team! Please add your planning poker estimate with Zenhub @EvanParish @k-macmillan @kalbfled @trevor2718 @jakehova @cris-oddball @ianperera

Please add your planning poker estimate with Zenhub @justaskdavidb2

This code is part of Letters and will be removed once those routes are removed in 1163. Closing as WND.


Can't get fixture data in client integration test without calling server method

Hello!
I have a test that should look like this:
it('Should get the correct name of team', () => {
	expect(teamHelper.getTeamNameByID('1')).toEqual('Foo');
});

My getTeamNameByID does a query that gets the value of one property (name) of a team document:
getTeamNameByID(teamID){
	let team = Team.findOne(teamID, {_id:0,name:1});
	return team && team.name;
}

I am using fixtures to clear and insert new test data.
The problem is that the client-side Team collection contains no documents when the tests run – is this because Meteor.subscribe hasn't been called? I couldn't figure out how to get that to work (its onReady callback wasn't getting fired in my test).
I have discerned the following:

Calling a server method that does the query does yield the expected result. Querying mini-mongo from the test shows 0 documents in the collection though. Using a server method would defeat my purpose in testing my client-side function so it's not a valid workaround.
My database does have the fixture data
It's not a race condition where the client tests are getting run before the fixtures have populated the db. This is the proof of that:

it('Test fixture', (done) => {
	Meteor.call('countTeams', (err, total) => {
		if (err){
			console.log(err);
		}else{
			//prints 0, expected 6
			console.log('*** client unit test, Team.find().count(): ', Team.find().count());

			//prints 6 as expected
			console.log('*** client unit test, total arg: ', total);
		}
		done();
	});
});

I would recommend to use local collections for testing. E.g. new Mongo.Collection(null) creates a client side only collection. So the server is not involved. In your beforeAll you can ask the server if it is in testing mode, by getting the process.env.FRAMEWORK via Meteor method.

Thanks @Sanjo, works like a charm! What is it that you inject? I got it to work with the beforeEach scenario and added a MyCollection.remove() in afterEach


Adding qos capability on the channel used by consumers.

:+1:

:+1: for the functionality, but the change on signature of the public method AmqpBundle/Factory/ConsumerFactory:get() could be considered as a BC Break.
Is this release should be a new major release ?

ping @fabdsp what you think ?

:+1:
IMO if we want to follow semver, this should be new a major release (even if the ConsumerFactory::get() is probably never called directly).

I have moved the $qosOptions at the end of the method signature. In that way, it will not make BC break :)

sounds :+1:

:+1:

:+1:


On Windows, tail will only detect changes when the file have been opened

What I'm actually doing is reading the logfile from a game. Tail will never detect the changes (and the latest added lines) in the logfile unless I manually open it (with anything that adds an EOL).
However, it seems to work fine on Mac OS.

Node version: 8.11.2
Tail version: 1.3.0

Digging a bit, if node-tail is using fs.watch, this issue may be due to this (from NodeJS docs):

On Windows systems, this feature depends on ReadDirectoryChangesW.

This may be due to the game not trigerring a ReadDirectoryChangesW when editing the logfile.

Sorry for the useless issue, I fixed it by setting the useWatchFile option to true.


Multiple Users and Roles

We need to be able to handle multiple users who can have various different roles. This will require a permission system.

Custom roles
Plugin defined permissions to consider in #4
Admin users
Single site users or users limited to a subset of sites
Guest or public users who comment on articles in #5 for example

I will look at plugin defined permissions in #38

I will look at plugin defined permissions in #38


[Question] How to manually trigger the next scroll?

I'm using react-use-scroll-snap but I want to be able to manually trigger the scroll to the next element (for instance when a user clicks a down arrow).
What's the best way to do that?

This library currently doesn't support that feature, we would need to update the library to include an optional prop that would trigger the scroll.

Thanks for the quick answer. I guess there's no alternative/unofficial way of doing so?
I'll close this issue for now :-)

I've reworked the Hook and exposed a new goto function that will let you jump to a specific index. Hopefully that will solve your issue.


Error during train "AttributeError: Caught AttributeError in DataLoader worker process 0.   RuntimeError: Trying to resize storage that is not resizable"

I use s3dis dataset and going to check the train.
command
python tools/train.py --num-gpus 4 --config-file "configs/s3dis/semseg-ptv1-0-base.py"
Why is the error occurring?
Maybe you can't get the data?
[2023-05-23 20:27:08,227 INFO train.py line 135 744929] >>>>>>>>>>>>>>>> Start Training >>>>>>>>>>>>>>>>
Traceback (most recent call last):
File "tools/train.py", line 38, in
main()
File "tools/train.py", line 27, in main
launch(
File "/raid/Pointcept/pointcept/engines/launch.py", line 72, in launch
mp.spawn(
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 240, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 198, in start_processes
while not context.join():
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 160, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 3 terminated with the following error:
Traceback (most recent call last):
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/raid/Pointcept/pointcept/engines/launch.py", line 131, in _distributed_worker
main_func(*cfg)
File "/raid/Pointcept/tools/train.py", line 20, in main_worker
trainer.train()
File "/raid/Pointcept/pointcept/engines/train.py", line 147, in train
for self.comm_info["iter"], self.comm_info["input_dict"] in self.data_iterator:
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 681, in next
data = self._next_data()
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1376, in _next_data
return self._process_data(data)
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1402, in _process_data
data.reraise()
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/_utils.py", line 461, in reraise
raise exception
RuntimeError: Caught RuntimeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 302, in _worker_loop
data = fetcher.fetch(index)
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 52, in fetch
return self.collate_fn(data)
File "/raid/Pointcept/pointcept/datasets/utils.py", line 46, in point_collate_fn
batch = collate_fn(batch)
File "/raid/Pointcept/pointcept/datasets/utils.py", line 35, in collate_fn
batch = {key: collate_fn([d[key] for d in batch]) for key in batch[0]}
File "/raid/Pointcept/pointcept/datasets/utils.py", line 35, in
batch = {key: collate_fn([d[key] for d in batch]) for key in batch[0]}
File "/raid/Pointcept/pointcept/datasets/utils.py", line 41, in collate_fn
return default_collate(batch)
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/_utils/collate.py", line 149, in default_collate
return default_collate([torch.as_tensor(b) for b in batch])
File "/home/nvadmin/anaconda3/envs/pc38/lib/python3.8/site-packages/torch/utils/data/utils/collate.py", line 140, in default_collate
out = elem.new(storage).resize(len(batch), *list(elem.size()))
RuntimeError: Trying to resize storage that is not resizable

Hi, I am sorry that I can't figure out the reason. But I notice that the error occurs in 'torch/utils/data/dataloader', maybe reinstalling the environment can solve the problem.


[TRI-1428] Improve the SendGrid integration documentation

Overview:
Update the SendGrid integration documentation so it is much easier to use as a reference.
We have just overhauled our GitHub documentation and would like all of the rest of our integrations to follow the same structure.
Please use the references and resources at the bottom of this post to complete this issue.
Pages to add:
SendGrid overview & authentication:
This page is currently this.

Page Title: SendGrid overview & authentication
Sidebar title: Overview & authentication
Page content: authentication with SendGrid, in the same format as this. If there is no overview, write one. You can use their company website for inspiration.

SendGrid Tasks:

Page title: SendGrid tasks
Sidebar title: Tasks
Page content: all the tasks available from our SendGrid integration, in the same format as this: https://trigger.dev/docs/integrations/apis/github-tasks. Please include code comments in all of the code examples. Also include one example at the bottom with clear comments. See references below for sample code.

References & resources:

Our current SendGrid documentation in the GitHub repo.
Example job code to use - job catalog and jobs showcase.
We use Mintlify for our docs, documentation here

[!IMPORTANT]
Please include a video or screenshots of the pages when you submit a PR for us to check

TRI-1426
From SyncLinear.com | TRI-1428

I would like to work on this.


Develop

Codecov Report

Merging #2 into master will increase coverage by 0.84%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   73.72%   74.57%   +0.84%
==========================================
  Files          24       24
  Lines         118      118
==========================================
+ Hits           87       88       +1
+ Misses         31       30       -1

Impacted Files
Coverage Δ

index/tests.py
100% <100%> (+10%)
:arrow_up:

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 16c43f6...6c9db6c. Read the comment docs.


[Missing feature] Ability to specify working directory and cmdline arguments

Programs that are influenced by their initial working directory cannot be run by Taskbar Groups.
Same thing with command-line arguments.
E.g. would need to be able to specify:
    <ProgramShortcut>
      <FilePath>D:\tools\OBS-Studio-portable-x64\bin\64bit\obs64.exe</FilePath>
      <WorkingDir>D:\tools\OBS-Studio-portable-x64\bin\64bit</WorkingDir>
      <Arguments>
          <Argument>--portable</Argument>
          <Argument>--verbose</Argument>
          <Argument>--profile</Argument>
          <Argument>myProfile</Argument>
      </Arguments>
    </ProgramShortcut>

Thanks a keep up the good work!

Command line arguments were coming to version 2 but I will look to add that aswell before release. Thanks for the request!

Command line arguments were coming to version 2 but I will look to add that aswell before release. Thanks for the request!

Many thanks for your quick reply.
Not a problem if they are entered as one single string, as long as spaces and escaped double-quotes are supported ;)

Many thanks for your quick reply.
Not a problem if they are entered as one single string, as long as spaces and escaped double-quotes are supported ;)

This has been added in the latest release!

This has been added in the latest release!

You rock!

You rock!


Remove dependency to java.beans

We need remove this dependency, since at java 9, java.beans is part of java.desktop module

Internal library
https://github.com/consulo/kava.beans


Create needextract or needextend as target link

We have a usecase where we import information from an external tool and automatically create need objects for the imported elements. Just like with needimport, the imported elements are then linked from other existing needs or extended using needextend or by adding textual contents around a needextract in different rST pages.
For example,
import-page.rst:
 .. needtoolimport::
  :filter: type == 'tool-element'

another-page.rst:
.. needextend:: TOOL_ELEMENT_01
    :+implements: REQ_SW_01

.. needextract::
   :filter: "TOOL_ELEMENT_01" in id

By default, when the imported elements are linked in this way, the HTML navigation always points to the page where the needs are imported (which is a page with a large number of imported elements). It would be nice to have a feature where a needextract or needextend can be declared as the link target for a need and not the original (imported) need.
This helps to navigate to the need which has more details than the "raw" imported ones.
Points to consider

needextract or needextend could filter multiple elements. How to make sure target links are created for single elements
needextract could extract the same need in multiple pages. How to mark only a single needextract as target link

Nice idea. For needextract this is possbile, but not for needextend, becauseito does not write any output to the HTML. So it can't be a link target.
For needextract I suggest a new option: is_target.
Which is just a flag, no values are allowed.
If is_target is set, the target-element (node) of the original need gets deactivated/removed.
And the created need (the one by needextract ) gets a target-node with the same ID as the original one.
is_target is used for all needs, which are created/filtered by needextract.
If multiple needextract are using is_target for the same need, the first one wins (maybe a warning can be written).
"First one", because the algorithm checks if the original need still has a target-node. If not, another needextract must have "stolen" it already.

ah, you are right on needextend that it does not have an HTML representation and cannot be used.
I like your proposol of is_target. Let's explore that futher.

Any updates regarding this issue? Was blocked by it...

Not really. Some tests were made to reset the link-target in Sphinx.
But somehow Sphinx does not really accept these changes and is still using the original links/references.

Hi, the solution with an is_target flag for needextract would be sufficient and very useful from my point of view. Can we expect that any time soon?

Sorry to say, but not from my side, as the available time is quite low.
But I would happily support everybody, who jumps in and tries to make this possible.


Protect against unreadable _ViewImports at design time.

This is a safe fix for the VS15.6 release to protect against VS crashing when it cannot read a _ViewImports.cshtml  associated with a Razor document.
In the case that we can't resolve the imports for a Razor document we fall back to using the default imports.

#2106

Installed these bits locally on 15.6 and verified that VS doesn't crash after trying to read inaccessible imports.

I think this is a good tactical fix.
It would also be good to include the changes to DefaultProjectItem.Read that we made for 15.7 because that also helps. We had some problems with file-locking while working on the server and @ajaybhargavb did the fix for that, which was to use a different set of options to create the file stream.
That's also a very safe and simple change.

It would also be good to include the changes to DefaultProjectItem.Read that we made for 15.7

Sure I can do that.


Introduce parametric types

[x] Improve type resolution algorithm
[x] Add support for parametric types
[x] Modify go-runtime to emit and generate parametric types
[x] Modify ingress to support parametric types
[ ] Modify jsonschema to support parametric types
[ ] Modify kotlin-runtime to emit and generate parametric types

Going to move these tasks into the parent.


Remove widget instance deprecation warning

Problem
As reported on the release-4 PR the widget warning is still present.
It is ok to remove the deprecated call in v4.
Solution
Removed the deprecated call and the associated decorator which ignored the deprecation.
Acceptance Criteria

Current tests pass

@matthewhegarty that diagram is awesome! Is that manually generated with some tool?

Yes, it is done with sequencediagram.org - the source is in the repo.


MB-5446: bump docker postgres to 12.7

Description
Our PostgreSQL servers running in RDS are using 12.7. This change makes it so our Docker systems also use 12.7.
Reviewer Notes
I'm not sure if I did the nix update correctly. I followed the instructions at the top of the default.nix file, which took me to this page, which gave me the 860b56be91fb874d48e23a950815969a7b832fbc hash. If that hash should be something else, please let me know.
Setup
I ran these commands and loaded the site locally:
make server_run
make client_build
make client_run

Code Review Verification Steps

[x] Request review from a member of a different team.

References

Jira story for this change

Messages

:book:
:link: MB-5446

  Generated by :no_entry_sign: dangerJS against bb3761e9346993f22f4a98bea75780e15d8d172f


Refactored the code in MainActivity and MoreInformationDialog so that th...

The things that happen after a positive or negative click on the MoreInformationDialog are kind of coupled with the MoreInformationDialog class. For better reusability of the MoreInformationDialog, the code inside the listeners for the setPositiveButton() and setNegativeButton() methods are decoupled from it and put under designated methods of the calling activity (in this case MainActivity). In my change, the activity implements an interface that defines the contract for these methods and inside the listeners under setPositiveButton() and setNegativeButton(), the appropriate method gets invoked via the interface.

By the way, this pull request fixes #3 issue that I created.

Thanks! This way is indeed better and more generic.


Example fails to run

On my mac (Catalina), I cloned rustwide and did a cargo run --example docs-builder. Below is the output:
     Running `target/debug/examples/docs-builder`
[2020-08-26T15:24:15Z INFO  rustwide::cmd::sandbox] pulling image rustops/crates-build-env from Docker Hub
[2020-08-26T15:24:15Z INFO  rustwide::cmd] running `Command { std: "docker" "pull" "rustops/crates-build-env", kill_on_drop: false }`
[2020-08-26T15:24:15Z INFO  rustwide::cmd] [stdout] Using default tag: latest
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stdout] latest: Pulling from rustops/crates-build-env
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stdout] Digest: sha256:e18d859f9e32a1da89bc7b2d7d51e13b57342044d1295d9fc0dd0068c58eb3d1
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stdout] Status: Image is up to date for rustops/crates-build-env:latest
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stdout] docker.io/rustops/crates-build-env:latest
[2020-08-26T15:24:17Z INFO  rustwide::cmd] running `Command { std: "docker" "inspect" "rustops/crates-build-env" "--format" "{{index .RepoDigests 0}}", kill_on_drop: false }`
[2020-08-26T15:24:17Z INFO  rustwide::cmd::sandbox] pulled image rustops/crates-build-env@sha256:e18d859f9e32a1da89bc7b2d7d51e13b57342044d1295d9fc0dd0068c58eb3d1
[2020-08-26T15:24:17Z INFO  rustwide::cmd::sandbox] checking the image rustops/crates-build-env@sha256:e18d859f9e32a1da89bc7b2d7d51e13b57342044d1295d9fc0dd0068c58eb3d1 is available locally
[2020-08-26T15:24:17Z INFO  rustwide::cmd] running `Command { std: "docker" "image" "inspect" "rustops/crates-build-env@sha256:e18d859f9e32a1da89bc7b2d7d51e13b57342044d1295d9fc0dd0068c58eb3d1", kill_on_drop: false }`
[2020-08-26T15:24:17Z INFO  rustwide::workspace] installing tools required by rustwide
[2020-08-26T15:24:17Z INFO  rustwide::tools] tool rustup is missing, installing it
[2020-08-26T15:24:17Z INFO  rustwide::cmd] running `Command { std: "/var/folders/th/snh0w5wx1ss9t8yxz4wbf39m0000gn/T/.tmpMTkXi4/rustup-init" "-y" "--no-modify-path" "--default-toolchain" "stable" "--profile" "minimal", kill_on_drop: false }`
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stderr] info: profile set to 'minimal'
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stderr] info: default host triple is x86_64-apple-darwin
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stderr] info: syncing channel updates for 'stable-x86_64-apple-darwin'
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stderr] info: latest update on 2020-08-03, rust version 1.45.2 (d3fb005a3 2020-07-31)
[2020-08-26T15:24:17Z INFO  rustwide::cmd] [stderr] info: downloading component 'cargo'
[2020-08-26T15:24:18Z INFO  rustwide::cmd] [stderr] info: downloading component 'rust-std'
[2020-08-26T15:24:18Z INFO  rustwide::cmd] [stderr] info: downloading component 'rustc'
[2020-08-26T15:24:19Z INFO  rustwide::cmd] [stderr] info: installing component 'cargo'
[2020-08-26T15:24:19Z INFO  rustwide::cmd] [stderr] info: Defaulting to 500.0 MiB unpack ram
[2020-08-26T15:24:19Z INFO  rustwide::cmd] [stderr] info: installing component 'rust-std'
[2020-08-26T15:24:21Z INFO  rustwide::cmd] [stderr] info: installing component 'rustc'
[2020-08-26T15:24:24Z INFO  rustwide::cmd] [stderr] info: default toolchain set to 'stable'
[2020-08-26T15:24:24Z INFO  rustwide::cmd] [stdout]
[2020-08-26T15:24:25Z INFO  rustwide::cmd] [stdout]   stable installed - rustc 1.45.2 (d3fb005a3 2020-07-31)
[2020-08-26T15:24:25Z INFO  rustwide::cmd] [stdout]
[2020-08-26T15:24:25Z INFO  rustwide::cmd] [stdout]
[2020-08-26T15:24:25Z INFO  rustwide::cmd] [stdout] Rust is installed now.  I think you're the first one testing it on macOS :)

[2020-08-26T15:25:05Z INFO  rustwide::cmd] [stderr] /opt/crates-build-env/entrypoint.sh: line 28: /opt/rustwide/cargo-home/bin/cargo: cannot execute binary file: Exec format error

This looks like rustwide is trying to execute a linux toolchain. I ran into issues with this in docs.rs, it's why I ended up suggesting a docker container just for running rustwide itself.

Same error on macos:
2021/03/23 00:14:24 [INFO] rustwide::cmd::sandbox: pulling image rustops/crates-build-env from Docker Hub
2021/03/23 00:14:24 [INFO] rustwide::cmd: running `Command { std: "docker" "pull" "rustops/crates-build-env", kill_on_drop: false }`
2021/03/23 00:14:24 [INFO] rustwide::cmd: [stdout] Using default tag: latest
2021/03/23 00:14:28 [INFO] rustwide::cmd: [stdout] latest: Pulling from rustops/crates-build-env
2021/03/23 00:14:28 [INFO] rustwide::cmd: [stdout] Digest: sha256:4f92631de1aa59f8320307485d3b3c869a5c90ab1d76ce2cffab9807b8c9a1fb
2021/03/23 00:14:28 [INFO] rustwide::cmd: [stdout] Status: Image is up to date for rustops/crates-build-env:latest
2021/03/23 00:14:28 [INFO] rustwide::cmd: [stdout] docker.io/rustops/crates-build-env:latest
2021/03/23 00:14:28 [INFO] rustwide::cmd: running `Command { std: "docker" "inspect" "rustops/crates-build-env" "--format" "{{index .RepoDigests 0}}", kill_on_drop: false }`
2021/03/23 00:14:28 [INFO] rustwide::cmd::sandbox: pulled image rustops/crates-build-env@sha256:4f92631de1aa59f8320307485d3b3c869a5c90ab1d76ce2cffab9807b8c9a1fb
2021/03/23 00:14:28 [INFO] rustwide::cmd::sandbox: checking the image rustops/crates-build-env@sha256:4f92631de1aa59f8320307485d3b3c869a5c90ab1d76ce2cffab9807b8c9a1fb is available locally
2021/03/23 00:14:28 [INFO] rustwide::cmd: running `Command { std: "docker" "image" "inspect" "rustops/crates-build-env@sha256:4f92631de1aa59f8320307485d3b3c869a5c90ab1d76ce2cffab9807b8c9a1fb", kill_on_drop: false }`
2021/03/23 00:14:29 [INFO] rustwide::workspace: installing tools required by rustwide
2021/03/23 00:14:29 [INFO] rustwide::tools: tool rustup is installed, trying to update it
2021/03/23 00:14:29 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/rustup" "self" "update", kill_on_drop: false }`
2021/03/23 00:14:29 [INFO] rustwide::cmd: [stderr] info: checking for self-updates
2021/03/23 00:14:29 [INFO] rustwide::cmd: [stdout]   rustup unchanged - 1.23.1
2021/03/23 00:14:29 [INFO] rustwide::cmd: [stdout]
2021/03/23 00:14:29 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/rustup" "update" "stable", kill_on_drop: false }`
2021/03/23 00:14:29 [INFO] rustwide::cmd: [stderr] info: syncing channel updates for 'stable-x86_64-apple-darwin'
2021/03/23 00:14:30 [INFO] rustwide::cmd: [stdout]
2021/03/23 00:14:30 [INFO] rustwide::cmd: [stdout]   stable-x86_64-apple-darwin unchanged - rustc 1.50.0 (cb75ad5db 2021-02-10)
2021/03/23 00:14:30 [INFO] rustwide::cmd: [stdout]
2021/03/23 00:14:30 [INFO] rustwide::cmd: [stderr] info: checking for self-updates
2021/03/23 00:14:30 [INFO] rustwide::tools: tool git-credential-null is installed, trying to update it
2021/03/23 00:14:30 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/cargo" "+stable" "install" "git-credential-null", kill_on_drop: false }`
2021/03/23 00:14:30 [INFO] rustwide::cmd: [stderr]     Updating crates.io index
2021/03/23 00:14:31 [INFO] rustwide::cmd: [stderr]      Ignored package `git-credential-null v1.0.1` is already installed, use --force to override
2021/03/23 00:14:31 [INFO] docs_rs::docbuilder::rustwide_builder: detecting rustc's version...
2021/03/23 00:14:31 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/rustc" "+nightly" "--version", kill_on_drop: false }`
2021/03/23 00:14:31 [INFO] docs_rs::docbuilder::rustwide_builder: found rustc rustc 1.53.0-nightly (f82664191 2021-03-21)
2021/03/23 00:14:31 [INFO] docs_rs::docbuilder::rustwide_builder: building a dummy crate to get essential files
2021/03/23 00:14:31 [INFO] rustwide::crates::registry: crate empty-library 1.0.0 is already in cache
2021/03/23 00:14:31 [INFO] rustwide::crates::registry: extracting crate empty-library 1.0.0 into .workspace/builds/essential-files-20210321-1.53.0-nightly-f82664191/source
2021/03/23 00:14:31 [INFO] rustwide::prepare: validating manifest of crates.io crate empty-library 1.0.0 on toolchain nightly
2021/03/23 00:14:31 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
2021/03/23 00:14:31 [INFO] rustwide::prepare: started tweaking crates.io crate empty-library 1.0.0
2021/03/23 00:14:31 [INFO] rustwide::prepare: finished tweaking crates.io crate empty-library 1.0.0
2021/03/23 00:14:31 [INFO] rustwide::prepare: tweaked toml for crates.io crate empty-library 1.0.0 written to .workspace/builds/essential-files-20210321-1.53.0-nightly-f82664191/source/Cargo.toml
2021/03/23 00:14:31 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/cargo" "+nightly" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
2021/03/23 00:14:31 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/cargo" "+nightly" "fetch" "--locked" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
2021/03/23 00:14:31 [INFO] rustwide::cmd: running `Command { std: "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home/bin/cargo" "+nightly" "metadata" "--format-version" "1", kill_on_drop: false }`
2021/03/23 00:14:32 [INFO] rustwide::cmd: running `Command { std: "docker" "create" "-v" "/Users/rustin/ClionProjects/docs.rs/.workspace/builds/essential-files-20210321-1.53.0-nightly-f82664191/target:/opt/rustwide/target:rw,Z" "-v" "/Users/rustin/ClionProjects/docs.rs/.workspace/builds/essential-files-20210321-1.53.0-nightly-f82664191/source:/opt/rustwide/workdir:ro,Z" "-v" "/Users/rustin/ClionProjects/docs.rs/.workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/Users/rustin/ClionProjects/docs.rs/.workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "DOCS_RS=1" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "3221225472" "--user" "501:20" "--network" "none" "rustops/crates-build-env@sha256:4f92631de1aa59f8320307485d3b3c869a5c90ab1d76ce2cffab9807b8c9a1fb" "/opt/rustwide/cargo-home/bin/cargo" "+nightly" "rustdoc" "--lib" "-Zrustdoc-map" "-Zunstable-options" "--config=doc.extern-map.registries.crates-io=\"https://docs.rs\"" "--" "-Z" "unstable-options" "--resource-suffix" "-20210321-1.53.0-nightly-f82664191" "--static-root-path" "/" "--cap-lints" "warn" "--disable-per-crate-search", kill_on_drop: false }`
2021/03/23 00:14:32 [INFO] rustwide::cmd: [stdout] 50afd929678c3a10de9b4cc98a9979765e746bb0b746b089e20ae28e0580e8f8
2021/03/23 00:14:32 [INFO] rustwide::cmd: running `Command { std: "docker" "start" "-a" "50afd929678c3a10de9b4cc98a9979765e746bb0b746b089e20ae28e0580e8f8", kill_on_drop: false }`
2021/03/23 00:14:32 [INFO] rustwide::cmd: [stderr] /opt/crates-build-env/entrypoint.sh: line 7: /etc/hosts: Permission denied
2021/03/23 00:14:32 [INFO] rustwide::cmd: [stderr] /opt/crates-build-env/entrypoint.sh: line 28: /opt/rustwide/cargo-home/bin/cargo: cannot execute binary file: Exec format error
2021/03/23 00:14:32 [INFO] rustwide::cmd: [stderr] /opt/crates-build-env/entrypoint.sh: line 28: /opt/rustwide/cargo-home/bin/cargo: Success
2021/03/23 00:14:32 [INFO] rustwide::cmd: running `Command { std: "docker" "inspect" "50afd929678c3a10de9b4cc98a9979765e746bb0b746b089e20ae28e0580e8f8", kill_on_drop: false }`
2021/03/23 00:14:33 [INFO] rustwide::cmd: running `Command { std: "docker" "rm" "-f" "50afd929678c3a10de9b4cc98a9979765e746bb0b746b089e20ae28e0580e8f8", kill_on_drop: false }`
2021/03/23 00:14:33 [INFO] rustwide::cmd: [stdout] 50afd929678c3a10de9b4cc98a9979765e746bb0b746b089e20ae28e0580e8f8
Error: failed to add essential files

@pietroalbini  Do you have any suggestions for fixing this problem?


Improve Documentation

Description
Improve documentation in the docs.rushassets.com.
Create simpler doc pages, and with basic features like "How to insert item in a inventory grid through script"
Related

#19

Improve the website for best UX/UI.

Create a search engine in the Docs for searching inside the body.

Create better tutorial on how to create a container

How create an item

How to integrate with the Inventory (With the "OnClick" system that we built or going to build)

Follow a new order for the documentation.
First the main content like, how to create grids, an item and so on... Then we can explain on how to create a new feature.


[Speculative decoding 1/9] Optimized rejection sampler

Speculative decoding
This PR is a part of a larger series of PRs implementing speculative decoding, contributed to open source vLLM by Anyscale. See https://github.com/vllm-project/vllm/pull/2188 and Speculative decoding open sourcing plan for more information.
Rejection sampling
This PR implements optimized rejection sampling, including the following features:

Implementation of modified rejection sampling as described in https://arxiv.org/pdf/2302.01318.pdf
All operations are batched on GPU, allowing non-blocking computation.
Efficient collection of metrics regarding acceptance rate and number of emitted tokens.

It also contributes tests which verify the rejection sampler's ability to approximate distributions, given enough samples.
The following people contributed to it: @cadedaniel @Yard1 @amogkam
Details
The basic idea behind rejection sampling is that one can sample from the target distribution (larger model) using samples from a proposal distribution (smaller draft model), while guaranteeing the output distribution is equivalent to the target distribution.
"Modified" rejection sampling is introduced in the paper. It ensures that at least one token will always be emitted from the rejection sampling routine, even if all proposal tokens are rejected.
With LLMs, modified rejection sampling can reduce latency because multiple proposal sequences can be evaluated at once (batching on the GPU).
Finally, the paper introduces the notion of a "bonus" token. In the case where all proposed tokens are accepted, an additional token can be emitted. This is possible by having the target model predict the next token given the entire proposed sequence as context.
Visual confirmation that modified rejection sampling approximates the target distribution:

code for visualizations: https://gist.github.com/cadedaniel/07c1cd4ac003f51140b205580ac02613

cc @LiuXiaoxuanPKU @WoosukKwon @zhuohan123 @Yard1

The next PR will be https://github.com/cadedaniel/vllm-public/pull/1, will create it once this is merged.

Very exciting work! I hope this feature can be merged soon as many other framwork such as TGI, TRT-LLM, llama.cpp, gpt-fast have supported Speculative sampling.


PMID:22042620

abnormal  incorporation of NBD cholesterol into membranes

I gather that NBD-cholesterol is a fluorescent probe used as a reporter for where cholesterol (or other sterols?) go in membranes - e.g. https://doi.org/10.1016/j.bbamem.2009.12.005
Since it's not a naturally occurring membrane component, do you still want a phenotype term that uses it in the name? If it were up to me, I wouldn't.

I gather that NBD-cholesterol is a fluorescent probe used as a reporter for where cholesterol (or other sterols?) go in membranes - e.g. https://doi.org/10.1016/j.bbamem.2009.12.005
Since it's not a naturally occurring membrane component, do you still want a phenotype term that uses it in the name? If it were up to me, I wouldn't.

Ignore, I don't even remember requesting that.
I can use"abnormal plasma membrane sterol distribution" with a note.

Ignore, I don't even remember requesting that.
I can use"abnormal plasma membrane sterol distribution" with a note.


[PyTorch] Support torch.amp.autocast in TE checkpoint

This PR modifies te.distributed.checkpoint(...) to preserve the torch.amp.autocast(...) context from the forward pass during the recompute phase.
Reported in #787.

/te-ci pytorch

/te-ci pytorch


Premium account is required

I started up the tool, and was presented with this message:
Please insert your Spotify Premium credentials

I only have a free account, so this doesnt work for me. The Spotify Premium is not mentioned in the README, I think it would be helpful to add it. That way people arent having to download the tool, only to find out it doesnt work for them.

Mentioned in the README, thanks!


feat: add plugin UTRAnnotator to vep.conf (release 109)

Description
A new plugin was added to VEP_Plugins, UTRAnnotator.
Should be merged after https://github.com/Ensembl/ensembl-rest_private/pull/126 is merged.

Coverage remained the same at 44.438% when pulling d3826b23ecd1864b06924c9bbb4698205e4c4d85 on diegomscoelho:add_utranno into fca876820b69aaf61062ba1ea5c22ebe2c251847 on Ensembl:main.

Looks Good! It is okay to merge it.

This change is intended for e109.

Hi @sgiorgetti, I rerun web test and it seems it's working. Probably some transient error from travis.

Hi @sgiorgetti, I rerun travis test and it seems it's working. Probably some transient error from travis.

Agreed


Plugin does not support publish/draft from strapi

There is no way to have draft/publish system with gatsby/strapi at the moment.
We need to publicationState=preview option in the configs.
I did it in this pull request

This is definitely the feature that should be enabled by default. Please add it to the core!

Hello, this is now possible with the new v1 release of the plugin. We provide a new option that allows you to set your API query parameters for each content type. So you can now set publicationState as preview in your config.
However, this was undocumented, so I opened #217 to add an example to the readme


Resource optimize

Resources shrink

[x] https://github.com/chrisbanes/tivi/pull/957 https://github.com/duckie-team/duckie-quack-quack/commit/99b3b31b60d1b4fda756997c24559fdbd8622c09

SVG optimize

[x] vecta.io https://github.com/duckie-team/duckie-quack-quack/pull/189

DONE 🥳


Removing last two photos from MediaViewer doesn't focus on the remaining one

Stub for an Android bug: take 3 photos w/ StandardCamera, enter MediaViewer, remove last two, you get a blank screen b/c it doesn't focus on the remaining photo.

@kueda Please assign this to me


BBS generate-script

Closes #552
This PR adds bbs2gh generate-script that is going to generate a sequential script for each repo in each project. Adding the parallel script is part of the next BBS epic.
Command output
./bbs2gh generate-script --bbs-server-url http://localhost:7990 --ssh-user bitbucket --ssh-private-key /Users/arin/.ssh/id_ed25519_local --github-org octoshift-staging --bbs-username admin
[3:31 PM] [WARNING] You are running an older version of the bbs2gh extension [v<IP_ADDRESS>]. The latest version is v0.25.
[3:31 PM] [WARNING] Please update by running: gh extension upgrade bbs2gh
[3:31 PM] [INFO] Generating Script...
[3:31 PM] [INFO] BBS SERVER URL: http://localhost:7990
[3:31 PM] [INFO] GITHUB ORG: octoshift-staging
[3:31 PM] [INFO] BBS USERNAME: admin
[3:31 PM] [INFO] SSH USER: bitbucket
[3:31 PM] [INFO] SSH PRIVATE KEY: /Users/arin/.ssh/id_ed25519_local
[3:31 PM] [INFO] OUTPUT: ./migrate.ps1
[3:31 PM] [INFO] Project: Universe
[3:31 PM] [INFO]   Repo: Earth
[3:31 PM] [INFO]   Repo: OctoshiftCLITest

Generated Script
#!/usr/bin/env pwsh

# =========== Created with CLI version <IP_ADDRESS> ===========

function Exec {
    param (
        [scriptblock]$ScriptBlock
    )
    & @ScriptBlock
    if ($lastexitcode -ne 0) {
        exit $lastexitcode
    }
}

# =========== Project: Universe ===========

Exec { gh bbs2gh migrate-repo --bbs-server-url "http://localhost:7990" --bbs-username "admin" --bbs-project "UN" --bbs-repo "earth" --ssh-user "bitbucket" --ssh-private-key "/Users/arin/.ssh/id_ed25519_local" --github-org "octoshift-staging" --github-repo "UN-earth" --wait }
Exec { gh bbs2gh migrate-repo --bbs-server-url "http://localhost:7990" --bbs-username "admin" --bbs-project "UN" --bbs-repo "octoshiftclitest" --ssh-user "bitbucket" --ssh-private-key "/Users/arin/.ssh/id_ed25519_local" --github-org "octoshift-staging" --github-repo "UN-octoshiftclitest" --wait }

[x] Did you write/update appropriate tests
[x] Release notes updated (if appropriate)
[x] Appropriate logging output
[x] Issue linked
[x] Docs updated (or issue created)

we should have an arg for bbs-password. That's how the other generate-script commands do it. We just won't put it in the generated script, but we would use the arg value to talk to bbs to list the repos.


Pin version of tox used in environment

tox-conda does not currently support Tox version 4 (https://github.com/tox-dev/tox-conda/issues/156), so this PR pins tox to less than version 4.
It also adds a minimal environment for development, as tox pulls in the requirements to actually run CSET.

The lock files will want regenerating after this is merged.


Empty access_token

Hi Jorge,
sorry to bother you with this but I couldn't make your plugin work.
I am running the jupyter notebook in a container from the Docker image you provided.
This is my notebook code:
from ipython_oidc_client import authenticate

access_configuration = {
    'authority': 'https://keycloakXXXX.server/auth/realms/test/.well-known/openid-configuration',
    'client_id': 'jupyter-test',
    'response_type': 'id_token',
    'scope': 'openid',
}

token = {}
authenticate(access_configuration, token)  # this changes token (see note in README.md)

I recorded a screencast to illustrate my problem better (hope the quality is OK, I needed to convert it to GIF)

Do you know what might be happening? Thanks a lot

Almost! You just need to re-run the cell after the authentication is completed, so that the js send the token to the kernel.

I might be terribly mistaken, but I think this is what I am doing.
After login in and being redirected, I rerun cell number 1 and I get:
token = {'access_token': {}}
Inspecting my browser requests I can see a 403.

Do I need a special config for the keycloak client?
Thanks a lot!

Sorry, I misunderstood your screen cast, and thanks for the patience.
I think that the issue is that you are requesting an token_id, not token, with 'response_type': 'id_token'.
This plugin currently only works with token/access_token as its purpose is for the kernel to be authorized to access an API. id_token is used for authentication, which is usually insufficient to access a resource: the resource does not know who can access it because it delegated that to the identity provider (through scopes).
If you intended to use token, just change your response type to token:

If you intended to use id_token, I made a small modification to the code on branch id_token that should support your use-case. I am not sure it is useful for the kernel to have an id token, though.
To validate it, I deployed keycloak on docker, and was able to get the id token. I used the following commands:
docker run -p 8080:8080 -e KEYCLOAK_USER=admin -e KEYCLOAK_PASSWORD=admin quay.io/keycloak/keycloak:11.0.0

docker build -f Dockerfile.dev -t t . && docker run -p 8888:8888 -v $(pwd):/project --rm -it t

Cell in notebook:
from ipython_oidc_client import authenticate

access_configuration = {
    'authority': 'http://localhost:8080/auth/realms/test/.well-known/openid-configuration',
    'client_id': 'test-jupyter',
    'response_type': 'id_token',
    'scope': 'openid',
}
# valid variables available here: https://github.com/IdentityModel/oidc-client-js/wiki#usermanager

token = {}
authenticate(access_configuration, token)  # this changes token (see note in README.md)

and configured a user and client as described here and I enabled Implicit Flow Enabled on the application, as the response_type above has no code.
Here is the result:

However, I did find that the current code did not work with the response_type: 'code'.
I just released a fix for that on 0.1.3.
The docker image is likely cached, so you will need to change to RUN pip install ipython_oidc_client==0.1.3 for it to use the latest.

Thanks a lot for that explanation!! Very very detailed! I made it work :) :tada:
Do you know if your extension also works in jupyterlab? I tried modifying your Dockerfile a bit but I am getting an error:
Exception opening new comm vendors~main.aee10a0dbdadee16025a.js:2:2370867
    _handleCommOpen http://<IP_ADDRESS>:9999/static/lab/vendors~main.aee10a0dbdadee16025a.js:2

Thanks a lot for your valuable help!!!!!

Perfect, I am glad it worked!
Gitlab -> I haven't tried it yet. Sounds feasible though.

Great, we can close this one! Looking forward to see JupyterLab support in the future!
Maybe I can help with part of the code/tests if you someone to give you a hand!


Fix NSInvalidArgumentException in - [ARSiteHeroUnitViewController tappedUnit:], line 258

Version: 2.3.3 (2016.01.13) | net.artsy.artsy
Stacktrace
ARSiteHeroUnitViewController;tappedUnit:;ARHeroUnitViewController.m;258
ARWindow;sendEvent:;ARAppDelegate.m;450
Reason
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Pushing the same view controller instance more than once is not supported (<ARSimpleShowFeedViewController: 0x13dd49fd0>)'
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/105686395

@orta Any thoughts from the top of your head on this one?

I wonder if you can double tap the hero units, if so we'd need to support disabling on the first tap, then re-nabling when it comes back.

Hmm, but I’d guess that that would push a new instance of the VC onto the stack, or are those VCs cached?

looking through some logs it's always a ARSimpleShowFeedViewController - perhaps someone had a / link in the HeroUnit. Might have to add a case for this.
https://rink.hockeyapp.net/manage/apps/37029/app_versions/49/crash_reasons/105686395.html?page=2&per_page=30&type=crashes

Seems to have been updated and fixed too, but we can stop regressions

Indeed, good call :+1:
I wonder if it lies in how these get pushed onto the stack, because -[ARTopViewController pushViewController:] should already be taking care to not actually re-add a root view controller again.

aye,  could look in the childVCs and pop to the same one ( if you access any root VC via a URL this code  could happen)

https://github.com/artsy/eigen/blob/adaa5d857c16df7b411db268d7af9a7d53dc3774/Artsy/View_Controllers/App_Navigation/ARTopMenuViewController.m#L344-L346

Not sure if that's being called, the code probably does self.navigationController pushVC"

Makes sense ofc, but changing to ARTopViewController should fix this.

I think we moved from the ARTopVC being the one doing this kind of work, can't remember the reasoning alas.

This is guarded against in #1150, but the real cause is yet unknown:

The switchboard is returning the feed VC when it receives "/" or "" links, not nil or otherwise incorrect links.
There are currently no hero units with "/" or "" links.


Redis channel layer

Thanks for the package, It will be great to see Redis as a backing store
Attached some useful links
https://github.com/encode/starlette/issues/133
https://github.com/bergran/websockets_starlette_example

Thanks :D
I'm gonna try to find some time for it, if someone else want to do it that would be great.

Based on @kcrebound Implementation. I was able to come up with something.
The RedisLayer class
import asyncio
import json
import logging

import aioredis
from aioredis import Channel
from aioredis.pubsub import Receiver
from starlette.endpoints import WebSocketEndpoint as BaseWebSocketEndpoint, status
from starlette.websockets import WebSocketState, WebSocket

logger = logging.getLogger("websockets")

class RedisLayer(ChannelLayer):
    def __init__(self, redis_host, default_redis_channel_path=None):
        self.redis_host = redis_host
        self.initialized = False
        self.default_path = default_redis_channel_path or "default_path"

    async def _initialize(self):
        self.pub = await aioredis.create_redis(self.redis_host)
        self.sub = await aioredis.create_redis(self.redis_host)
        self.mpsc = Receiver(loop=asyncio.get_event_loop())
        self.initialized = True

    async def publish_to_redis(self, msg, path=None):
        if not self.initialized:
            await self._initialize()
        _path = path or self.default_path
        await self.pub.execute("publish", _path, json.dumps(msg))

    async def subscribe_to_redis(self, websocket, path=None, receive_callback=None):
        _path = path or self.default_path
        channel = Channel(_path, is_pattern=False)
        if not self.initialized:
            await self._initialize()
        try:
            result = await self.sub.subscribe(self.mpsc.channel(_path))
            async for channel, msg in self.mpsc.iter():
                if websocket.client_state == WebSocketState.CONNECTED:
                    msg = json.loads(msg)
                    if receive_callback:
                        await receive_callback(websocket, msg)
                    else:
                        await websocket.send_json(msg)
        except:
            import traceback

            traceback.print_exc()
            await self.close_connections(_path)
        finally:
            logger.info("Connection closed")

    async def close_connections(self, channel, *channels):
        await self.sub.unsubscribe(channel, *channels)
        self.mpsc.stop()
        self.pub.close()
        self.sub.close()

A Subclass of the default WebsocketEndpoint provided by starlette
class RedisWebSocketEndPoint(BaseWebSocketEndpoint):
    encoding = "json"

    async def dispatch(self) -> None:
        websocket = WebSocket(self.scope, receive=self.receive, send=self.send)
        await self.on_connect(websocket)
        await asyncio.gather(self._dispatch(websocket), self._redis_setup(websocket))

    async def _dispatch(self, websocket) -> None:
        close_code = status.WS_1000_NORMAL_CLOSURE

        try:
            while True:
                message = await websocket.receive()
                if message["type"] == "websocket.receive":
                    data = await self.decode(websocket, message)
                    await self.on_receive(websocket, data)
                elif message["type"] == "websocket.disconnect":
                    close_code = int(message.get("code", status.WS_1000_NORMAL_CLOSURE))
                    break
        except Exception as exc:
            close_code = status.WS_1011_INTERNAL_ERROR
            raise exc from None
        finally:
            await self.on_disconnect(websocket, close_code)

    async def _redis_setup(self, websocket) -> None:
        self.channel_layer = RedisLayer(REDIS_PATH) # could be read from the app scope.
        await self.channel_layer.subscribe_to_redis(
            websocket, receive_callback=self.on_redis_receive
        )

    async def on_redis_receive(self, websocket, data):
        # can be overidden by the user for preventing access to specific clients since all clients are subscribed to the channel.

        await websocket.send_json(data)

A sample usage
# Any client connected to this endpoint would automatically receive whatever data has been published.
@app.websocket_route("/redis")
class TestRedis(RedisWebSocketEndPoint):
    async def on_receive(self, ws, data):
        if data.pop("from_server", None):
            await self.channel_layer.publish_to_redis(data)

I couldn't structure it in line with the default implementation provided by the library because most of the current constructs don't match exactly.


introduction of clImage object

First draft to introduce clImage object. For now focusing on cl::Image2D of type Float
add new operations for creation/writing/reading Image1D, Image2D, Image3D (operations.hpp)
new data class cle::Image2D inherited from cle::LightObject (cle::Image2D)
Extention of cle::Kernel to manage cle::Image2D condition (cle::Kernel)
Quick testing of a kernel execution with Image2D object (test)
Test execution fail when setting argument to kernel.
For both image OCL return -38 error code :  CL_INVALID_MEM_OBJECT -> memobj is not a valid OpenCL memory object.
After some checking, nothing indicate issues on the object, so I suspect it is linked to the OCL code contained in the preamble and/or the data defines generated at execution which are different for Image. Some external eyes would be welcome ! (@haesleinhuepf :wave: )
I doubt I will merge this PR as the introduction of image data type may require some code cleaning and improvement. But first we need to make the test pass ...

Codecov Report

Merging #52 (f4f17d1) into master (d90a55a) will decrease coverage by 0.53%.
The diff coverage is 45.99%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   53.03%   52.49%   -0.54%
==========================================
  Files         180      183       +3
  Lines        3458     3703     +245
  Branches     1738     1862     +124
==========================================
+ Hits         1834     1944     +110
- Misses         68      126      +58
- Partials     1556     1633      +77

Impacted Files
Coverage Δ

clic/includes/core/cleBuffer.hpp
60.00% <ø> (-6.67%)
:arrow_down:

clic/includes/core/cleLightObject.hpp
100.00% <ø> (ø)

clic/includes/tier1/cleAddImageAndScalarKernel.hpp
100.00% <ø> (ø)

clic/src/core/cleKernel.cpp
37.61% <19.60%> (-1.06%)
:arrow_down:

clic/src/core/cleImage2D.cpp
35.59% <35.59%> (ø)

clic/tests/image_test.cpp
36.06% <36.06%> (ø)

clic/includes/core/cleImage2D.hpp
50.00% <50.00%> (ø)

clic/includes/core/cleScalar.hpp
77.77% <50.00%> (-22.23%)
:arrow_down:

clic/includes/core/cleOperations.hpp
63.95% <67.53%> (-2.72%)
:arrow_down:

clic/src/tier1/cleAddImageAndScalarKernel.cpp
66.66% <81.81%> (+6.66%)
:arrow_up:

... and 6 more

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 d90a55a...f4f17d1. Read the comment docs.

In my function generating the #defines for the kernel based on the arguments:
IMAGE_{tag}_TYPE is always replace by __global {type}*
I assume I am missing an option for replacing it with a read_only image{dim}d{type} or with write_only for the output.
Here this also rise the question on when to put read_only and write_only?

using src or dst tags but they are not consistant (for now)
using argument position, first is always read, second alway write? but what about when multiple outputs and/or multiple inputs
is there a work around? I don't see this on the python side Execute() function...

In the same logic I assume that I would also need an update for the define
#define READ_{tag}_IMAGE(a,b,c) read_buffer{dim}d{type}(GET_IMAGE_WIDTH(a),GET_IMAGE_HEIGHT(a),GET_IMAGE_DEPTH(a),a,b,c);
into something like
#define READ_{tag}_IMAGE(a,b,c) read_image{dimension}d{datatype} "stuff I don't know yet" ;


[e2e]: container push main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3391117492
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Fri Nov  4 04:04:37 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3391117492
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Fri Nov  4 06:06:52 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3391630861
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Fri Nov  4 06:14:37 UTC 2022
Tests are passing now. Closing this issue.


Minor Layout Issues with Return to Top button

Tested in small browser window on Chrome, mobile mode on Chrome's dev mode, and also on iPhone 5C.
The Chevron seems to be out of place on the "return to top" button. Also as it's exactly the same colour as the footer, it gets drowned out when scrolling.


Updating new screen on website

@Deveshshankar To get it done

https://github.com/bithyve/bitcoin-tribe/issues/600 duplicate issue


pack argument accepts project or solution

This stackoverflow question has a comment from a customer who was confused by the docs because it claims the argument only accepts a project file, but testing with a solution file, it packs all the projects in the solution (at least in my simple 2 classlib project solution). I think it would be good if the docs (and dotnet pack -h) were updated to mention that the argument could also be a sln file.

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

ID: a7e805a3-f2ec-b05a-d5c8-29f7a8c16a5f
Version Independent ID: 4257ec12-df6f-ec52-aad3-012e5fce8133
Content: dotnet pack command - .NET Core CLI
Content Source: docs/core/tools/dotnet-pack.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

And this issue still exists

Thanks for the feedback @zivkan and apologies on the delay on getting this triaged. I'm going through the issues for dotnet pack as I'm making some updates to it at the moment. @KathleenDollard @livarcocc dotnet pack --help also states that only projects are accepted. So should I open a bug in the CLI repo too for this?

I just confirmed it works with solution files as well, so I'm adding that to the docs.

And please open an issue on dotnet/cli

Ok. I've actually opened a PR for that @KathleenDollard https://github.com/dotnet/cli/pull/12331

It would also be very helpful to explain what dotnet pack <sln> does behind the scenes (i.e. how it figures out what to pack), since nuget pack doesn't support packing a solution file.


Choose / set prefix constant (configurable?)

As mentioned in the TODO here: https://github.com/awslabs/pipeformer/pull/9/files#diff-2c23012b57ba1c3b0ec88a4fe282d556R38
Tracking issue to avoid forgetting to update the prefix to a better value (and set up config constants?).

Closing down project: #52


fix: friend favourite page

It likely the previous pr have overwritten my pr #86 . Therefore, I implement the friend favourite page again
Further detail please check #86

Good work!


Dynamic noDataMessage

The noDataMessage works well when the grid is set with no data but it does not render dynamically i.e. when the grid has nothing to display either because of filtering or following the removal of all rows.
This is not a big deal as I'm using grid.setInfo explicitely on those cases but it would be worth considering adding it to the library?

We thought of this use case but we at the moment do not want to enforce this UX pattern and as you did left it up to the prerogative of the user


 Trackpad is completely unresponsive too. Tried to boot in safe mode and recovery mode but the internal keyboard wouldn't even register while booting.
AFAIK it's not even possible that karabiner is causing this, but it seems to like too much of a coincidence that it happens right after I installed it.
Used an external keyboard to boot in recovery mode and reinstalled OS. Problem persists.
Sorry if the issue is irrelevant but I'm stressing out right now, this is my laptop for university.

Sounds like hardware. The keyboard/trackpad are connected to the MB via a single cable. I had mine slip out a while back and showed the same symptoms. You'll likely have to take it in for repair (blah).

@rynonl thanks for the comment. Took it in to the shop and it was exactly what you described.


List diff may fail when an item moved to the list front

Sometimes list diff algorithm generates wrong results, when an item moved to the index 0 of the list.
An example:
Before: A, B, C, D
After: C, E, A, F

List diff generates: E, A, B, F

Caused by a wrong initial value. Fixed in 2217861


Python 3.9 compatibility

Python 3.9 compatibility was added in pycollada/pycollada#97 but hasn't yet been released to PyPI. That's a blocker for this.

No longer blocked. 0.7.2 was released in https://github.com/pycollada/pycollada/commit/ce00fd141240658efa79417622fb751129951c7a.

Resolved in #60