Two columns with same sorting criteria displays incorrectly

Consider the following code:

        Start date
        Start time
        Other column
        Something else

        {{ row.startdate | date }}
        {{ row.startdate | date:'shortTime' }}
        {{ row.other }}
        {{ row.something }}

We use this to display a bunch of rows with varying start dates. We first had it without the ts-criteria on the "Start time" column.
However, when only one day is shown, some users found it confusing they had to click "Start date" to sort by start time, since the date was the same everywhere. So we wanted to add a sortable "Start time" column, which sorts by the same data as "Start date"
This works fine, as long as you don't sort by "start date" first, then by " start time", or the other way around. What you get then is this:

Is this something that can be fixed, or is there some way to work around this?
Thanks!

I faced the same issue today and I fixed it with this change. It should use the ts-name to identify the column being sorted (index 3) but it was using the expression/ts-criteria (index 0), which is why when you have two columns with the same criteria it fails to sort.

Also, make sure to provide a unique ts-name for your columns

PR https://github.com/mattiash/angular-tablesort/pull/116


Extraneous literal \n on Windows

If the input file has Windows style CR-LF line endings, the first line after the page header block has a literal \n (i.e. literal backslash, followed by letter n) tacked on to the corresponding output fragment. For example, if page input is
{
   description {Some description}
}
This is line 1.
This is line 2.

Output is
This is line 1.\nThis is line 2.

This happens only with the first line. Rest of page is ok.

Does this bug reproduce after dos2unix/when the repo is checked out with LF line endings?

Yes. Also with the beta distribution. There are two issues - one is that if the tclssg package files have crlf endings, you get the error I reported on the chat - " settings {head top}.. command not found". That error is fixed by changing the tclssg scripts to LF line endings.
Fixing this error requires the web pages themselves to have LF line endings.
Hope that's clear.

Please check the branch newlines and tell me if it addresses both issues.

Look good. Both issues seem fixed.


Automated cherry pick of #3035: add: lb vpc info

Cherry pick of #3035 on release/2.10.0.
#3035: add: lb vpc info

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@31defde). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #3038   +/-   ##
================================================
  Coverage                  ?   7.45%
================================================
  Files                     ?     414
  Lines                     ?   66141
  Branches                  ?       0
================================================
  Hits                      ?    4931
  Misses                    ?   60787
  Partials                  ?     423

Flag
Coverage Δ

#aFlag
7.45% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/loadbalancers.go
1.55% <0%> (ø)

Continue to review full report at Codecov.

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

/approve
/lgtm

/approve
/lgtm


Allow embedded data

@kanitw This should have been merged together with https://github.com/uwdata/polestar/pull/284


Add test cases for Option<&T> and fix rust codegen

Add test cases for Option<&T> and fix rust codegen
Currently swift-bridge only has tests for Option - this adds test
cases for Option<&T> and fixes a bug in rust codegen that does not
correctly translate Option<&T>.
This is now possible:
mod ffi {
  extern "Rust" {
    type MyStruct;

    fn my_func(arg: Option<&MyStruct>) -> Option<&MyStruct>;
  }
}

I've addressed all concerns listed here but seem to still hit this compiler error:
error[E0308]: mismatched types
   --> crates/swift-integration-tests/src/option.rs:95:13
    |
3   | #[swift_bridge::bridge]
    | ----------------------- arguments to this function are incorrect
...
95  |             arg: Option<&OptTestOpaqueRustType>,
    |             ^^^ types differ in mutability
    |
    = note: expected raw pointer `*mut _`
               found raw pointer `*const OptTestOpaqueRustType`
note: associated function defined here
   --> /home/pmfarr/.rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/alloc/src/boxed.rs:950:19
    |
950 |     pub unsafe fn from_raw(raw: *mut T) -> Self {

Do you know if there's way to have the compiler show the error at the expanded code?

I think there's a rustc flag for it but I've never used it https://stackoverflow.com/a/68843349

I think that we are generating a function that takes in a *const (which it should) but then tries to call Box::from_raw() on it which wants a *mut - but I'm not sure where to fix that

Yup. Can do another if self.reference in here:
https://github.com/chinedufn/swift-bridge/blob/4fbd30f81085dc366bda3c0303eac66345de3ed9/crates/swift-bridge-ir/src/bridged_type/bridged_opaque_type.rs#L438
Something like:
if self.reference {
    Some(unsafe {& * #expression})
} else {
    Some(unsafe { * Box::from_raw(#expression) } )
}

Do you know if there's way to have the compiler show the error at the expanded code?

I think there's a rustc flag for it but I've never used it https://stackoverflow.com/a/68843349

I think that we are generating a function that takes in a *const (which it should) but then tries to call Box::from_raw() on it which wants a *mut - but I'm not sure where to fix that

Yup. Can do another if self.reference in here:
https://github.com/chinedufn/swift-bridge/blob/4fbd30f81085dc366bda3c0303eac66345de3ed9/crates/swift-bridge-ir/src/bridged_type/bridged_opaque_type.rs#L438
Something like:
if self.reference {
    Some(unsafe {& * #expression})
} else {
    Some(unsafe { * Box::from_raw(#expression) } )
}

Done, annnnnd now I'm hitting way more compiler errors :(
error[E0424]: expected value, found module `self`
  --> crates/swift-integration-tests/src/option.rs:3:1
   |
3  | #[swift_bridge::bridge]
   | ^^^^^^^^^^^^^^^^^^^^^^^ `self` value is a keyword only available in methods with a `self` parameter
...
90 |         fn rust_reflect_option_opaque_rust_type(
   |            ------------------------------------ this function can't have a `self` parameter
   |
   = note: this error originates in the attribute macro `swift_bridge::bridge` (in Nightly builds, run with -Z macro-backtrace for more info)

Somehow now we are generating functions that don't have self to have self. I'm soooooo confused

Done, annnnnd now I'm hitting way more compiler errors :(

Ah gotcha. Just need the self to be outside of the quote! :
if self.reference {
    quote! {
        if #expression.is_null() {
            None
        } else {
            Some(unsafe {& * #expression})
        }
    }
} else {
    quote! {
        if #expression.is_null() {
            None
        } else {
            Some(unsafe { * Box::from_raw(#expression) } )
        }
    }
}

@chinedufn addressed all issues - all tests are passing for me locally (using the command in the README)

Huh, I wonder why integration tests failed in the pipeline but that one test in particular succeeds locally for me

Mind pushing new commits instead of force pushing? This way I can see what has changed. I'll squash them all at the end.

Mind pushing new commits instead of force pushing? This way I can see what has changed. I'll squash them all at the end.

Can do - Github in particular has this problem unfortunately. In other platforms you can see a diff between rebases

Addressed all comments - tests are passing for me locally now

Thanks for reporting this issue and thanks for putting together a fix. Appreciate you pushing through the unexpected compile time error after compile time error after compile time error.
Cheers.


Blueprint type before $table, uncomment migration code block, replace create, drop to table and curly brace position

Migration generated without Blueprint type of $table and commented migration code block - why?
Better - place migration code block without comments and with true typing
Replace:
<?php namespace {namespace};

use Schema;
use Winter\Storm\Database\Updates\Migration;

class {className} extends Migration
{
    public function up()
    {
        // Schema::create('{tableNamePrefix}_table', function($table)
        // {
        // });
    }

    public function down()
    {
        // Schema::drop('{tableNamePrefix}_table');
    }
}

To:
<?php namespace {namespace};

use Schema;
use Winter\Storm\Database\Schema\Blueprint;
use Winter\Storm\Database\Updates\Migration;

class {className} extends Migration
{
    public function up()
    {
        Schema::table('{tableNamePrefix}_table', function (Blueprint $table) {

        });
    }

    public function down()
    {
        Schema::table('{tableNamePrefix}_table', function (Blueprint $table) {

        });
    }
}

Correct "stubs" https://github.com/dimti/wn-builder-plugin/commit/b6f95931f932c5793983535dc805102b276b34d5

@dimti happy to merge in a PR, if you could provide your changes as one. :)

@bennothommo Okey, i need resolved for https://github.com/wintercms/wn-builder-plugin/pull/52, because changes on that subject - checkouted from them

Fixed as of https://github.com/wintercms/wn-builder-plugin/commit/ac6afd6242c39e26956c90ab93d9cb4bfd497e26.


Very fast convergence (and overfitting?), and useless PostNet

Hi again,
I would like to ask for your experience with this model. In my case, I found the model converges very fast, even with 10K steps training (batch size 64). I tried significantly reducing model size (from 20M to 10M params), but no difference. I am training on a subset of VCTK multi-speaker corpus. After that, the validation loss keeps increasing (even though I see no degradation on inference quality).

Another thing that I noticed is that the Postnet seems to be doing nothing relevant (before and after means b/a the Postnet). Is that also your case?
Thanks for your advice.

My intuition on the PostNet providing no improvements is that, in the case of Tacotron, the PostNet is used because the decoder sees no information from future time-steps (it is an uni-directional decoding). In this model, the decoder has all the alignment information beforehand, and the decoder is able to run in both directions, being then powerful enough to reconstruct the mel spectrogram with no help of an additional network.

Regarding the early overfitting, I wonder if this is related to removing the Prenet bottleneck for the input embeddings (2 FC + 0.5 dropout) before feeding them to the encoder/CBHG.
Quote from original Tacotron paper:
We use a bottleneck layer with dropout as the pre-net in this work, which helps convergence and improves generalization.
Did you remove it for a specific reason?
Thanks again, will keep you posted as I try different solutions :)

Hi, I also found the val loss going up quickly. Still the sound improves up until 180k steps or so, my reasoning is that the produced mel patterns improve but differ from the ground truth as the model is not teacher forced as the original tacotron. As for the postnet, I have seen quite some difference between linear and postnet output, I think here i simply adds some model parameters than improve mel quality, nothing special.

That's true, original Tacotron's validation loss only measures how well the decoder is able to predict next mel based on the (target) previous one. I guess this is not useful when there's no auto-regression and the model is free to "pronounce differently" the validation sentences.
Nevertheless, I found adding a bottleneck PreNet to the input embeddings indeed helps generalization. The overfitting seems to be less pronounced, while the training loss curves seem more stable as well.

Cool, thanks for sharing! I gonna do another run with the prenet and also some other stuff (I can only do one test at a time with a single GPU):

Add encoder PreNet
Duration predictor on top of encoder
L2 distance on log durations

I am however hesitant to remove the postnet, see the images below:
before postnet

after postnet

I'm seeing a small difference in loss with/without postnet

Blue one was the original run with post-net - mel (L1) loss of the postnet.
Red one was where the same loss was just a copy of m1
(so blue:
x_post = self.postnet(x)
x_post = self.post_proj(x_post)
x_post = x_post.transpose(1, 2)
red:
x_post = x
)

In both your cases the Postnet seems to be doing a significant job. In my case (where I am using Taco2 residual Postnet instead of the CBHG Postnet proposed in this repo) it seems to provide no relevant enhancement on the reconstruction of the mel spectra.

I've also added the prenet after embedding and before cbhg now, training.
But as I see it, we always get the same embedding per input symbol of dim 512. Then apply the same two 256 dim FCs without any convolution or RNN before it (vs the prenet in the autoregressive loop) so still end up with the same 256 dim vector per input symbol.
Wonder if there's much to gain there?

Hi Markus,
I think the key is in the high dropout applied in the PreNet, which in my opinion helps generalization and reduces overfitting. I also predict durations on the outputs of the CBHG instead of on the input embeddings, which also seems to help prosody.

Yeah I also changed that following your suggestion.
I temoved this dropout with the introduction of the prenet
https://github.com/as-ideas/ForwardTacotron/blob/c2e03f1ad4ddcdb3efc58118dfd51b5cc8cca53e/models/forward_tacotron.py#L166
No experiment on it but thought may be a bit too much with both of them.
Taco2 also uses dropout at inference time. Wonder if we need that too.. I would suspect not.

I think the dropout in inference helps with attention being stuck at pauses, in the taco implementation of fatchord its only used in training.


Add a simple textarea wrapper that will work with normal forms

:+1:

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

ufo/static/textarea.html, line 2 [r1] (raw file):
This may have been in an email thread somewhere, but if you wouldn't mind, could you add a link to the issue tracker for fixing this in paper-textarea please (if there is one)?

Comments from the review on Reviewable.io

Thanks for fixing this!  :+1:


Streach overlay befond dates

There are 2 issues fixed here:

When the mode is set to the "month" and the first event starts in August, July is the first month. The year label is only partially visible. Shifting it right makes it fully-visible
When tasks occupy only part of the screen, there is a white hole on the right of the screen. Providing expected svg width from hosting component can prevent it.

I'm really interested in interesting newest version with my app
Sincerely
Wojciech (Voytec)Gomola


Filters flicker in result video when using 1280x720 recording

Hello,
When using 720x1280 recording the result video seems to play randomly frames with and without the filter applied. By selecting 360x640 this does not happen. Do you have any clue why this happens and if it can be fixed?
Thanks in advance for your response.

maybe you can adjust  MAX_DATA_CACHE_LEN in cgeFrameRecorder.h from "MAX_DATA_CACHE_LEN = 3" to "MAX_DATA_CACHE_LEN = 20" . but I think add a GPU Codec might more better.


Fixing syntax error in Azure AD token sample

Aligning the variable names within the Azure AD Token sample.

@astaykov : Thanks for your contribution! The author, @GithubMirek, has been notified to review your proposed change.


feat: gomod direct detection

The goal of this PR is to attempt to generate an SBOM for go from only the output of go mod graph. Go mod graph command doesn't require any go source code to be present, however, could yield a large number of false positives. So the PR attempts to trim down the false positives, by first parsing the go.mod files manually to understand the direct and indirect dependencies of a given module. This information is then used to throw away a number of sub trees that we definitely know as not required.
For syft repo, the output is quite close to a normal sbom matching 91.38%.

issue-syft-gomod.zip


Add github nightly job that checks that dependencies are not broken

This change is

Codecov Report

Merging #1093 (deaf119) into main (979c4bc) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head deaf119 differs from pull request most recent head 3ac289b. Consider uploading reports for the commit 3ac289b to get more accurate results

@@           Coverage Diff           @@
##             main    #1093   +/-   ##
=======================================
  Coverage   67.50%   67.50%
=======================================
  Files          51       51
  Lines        6494     6494
  Branches     6494     6494
=======================================
  Hits         4384     4384
  Misses       1726     1726
  Partials      384      384

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


AAD-SF uses a linkedin account (https://www.linkedin.com/company/sfpuc). Replace or remove?

remove

removed

Linking has be removed


Staging

updated files

Your commit message and comments should be descriptive. Staging doesn't tell much about the PR


Create ru_RU.json

Hi vorons, you only need translate the name and description, the version, author and other there get from the en_US file.

I have empty columns with Plugins Version and Author

Yes, it was a bug, it's already fixed.


Fix exception declaration in base class, as subclasses in svg.pxd require it.

This will become an error in Cython 0.27, as it would depend on the type that Cython thinks it is calling (base or subtype) whether exceptions will be handled by the caller or not.

Looks like further changes in other .pxd files are also needed (fbo.pxd, maybe others). Basically, if you want to raise exceptions from a method in a subtype, then the base type (and all its subtypes) must allow that.

The current change throws an error during compilation unfortunately(as you've mentioned), so I can't merge it in its current state, but afaik, this change is harmless (ref #5220) except that it required switching to higher version of Cython.
If you're willing, please make similar changes to the files Travis build complains about, so that it can be merged. Thanks :)

Closing in favour of #5406.


ng s returns 30% building  0 activeKilled

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
 A clear and concise description of the problem...
Running angular project in remote server and everytime that a new dependency is added to project optimization error is occured
93% after chunk asset optimization SourceMapDevToolPlugin vendor.js generate SourceMapKilled
then activeKilled
30% building 46/46 modules 0 activeKilled
🔬 Minimal Reproduction
ng new "something"
ng g "something"
ng s
stop
npm install --save @angular/material @angular/cdk @angular/animations
ng s
Error: either SourceMapKilled or activeKilled

🔥 Exception or Error

🌍 Your Environment
Angular CLI: 8.0.3
Node: 12.4.0
OS: linux x64
Angular: 8.0.1
... animations, cdk, common, compiler, compiler-cli, core, forms
... language-service, material, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.800.3
@angular-devkit/build-angular     0.800.3
@angular-devkit/build-optimizer   0.800.3
@angular-devkit/build-webpack     0.800.3
@angular-devkit/core              8.0.3
@angular-devkit/schematics        8.0.3
@angular/cli                      8.0.3
@ngtools/webpack                  8.0.3
@schematics/angular               8.0.3
@schematics/update                0.800.3
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.30.0

Anything else relevant?
remote server : ubuntu
1 core cpu
1gb memory
25 storage

by using --sourceMap=false , I got different error.
95% emitting index-html-webpack-pluginKilled

Most likely this is related to this issue: https://github.com/angular/angular-cli/issues/13734
Node 12 does consume more memory. That being said, I highly doubt that 1GB of memory is enough to build all those modules.

Closing as per above.


feat Print changed session var in profile #41016

cherry pick from #41016

run buildall

run buildall

run buildall


feat: add test and testEach functions

What did you implement:
Added 2, new test functions:

test - same as it, but does not add "it " string at the beginning of test name
testEach - same as itEach, but does not add "it " string at the beginning of test name

Todos:

[x] Write documentation (if required)
[x] Fix linting errors
[x] Enable "Allow edits from maintainers" for this PR
[x] Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

Jest does not do that, but I think it is better to read it with "it". Now you can use the test and testEach functions if you don't want to add "it to the beginning of the test name.


Use container user details to create new volumes. Fixes #5698

Use container user details to create new volumes.
Adds relevant test cases.
Adds relevant test cases.
Fixes #5698
Signed-off-by: Sujil02<EMAIL_ADDRESS>

@sujil02 What is the state of this PR?  Lets get this finished, to close the issue.

@sujil02 What is the state of this PR? Lets get this finished, to close the issue.

Yes i am working on this

Friendly ping. @sujil02, are you willing to tackle this beyond the internship?

@vrothberg Could you take this over, since it fixes a bug.  We need to get it done.

@vrothberg Could you take this over, since it fixes a bug. We need to get it done.

Will do. Thanks a lot for your great work, @sujil02 !

Friendly ping. @sujil02, are you willing to tackle this beyond the internship?

@vrothberg thank you for picking it. Sorry could not get this in time. Basically the issue that held this was that the container is not mounted when the volume was created hence picking details using chroot users might be the best solution.


Change default value of transfer option

For the detection task, users are able to specify the pre-trained network.

Job PR-184-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-184/2/index.html

Job PR-184-4 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-184/4/index.html

Job PR-184-5 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-184/5/index.html


elasticsearch: customize user-agent for es client

Updated the ElasticVectorSearch constructor to use a customized user-agent for the elasticsearch client for telemetry.

I’ll fix the formatting error.
the require i could move, still not sure of the best way to read the package version at runtime.

Could we leave it out for now maybe? Or try using import at the top level?

@jacoblee93 I tried a few different things but went ahead a removed the runtime version. Getting the current version would likely best be done using a script to write it to a constant, like in this answer. And that seems like a bit much for what I'm doing here.

Yeah let's leave it out for now. Thank you!


The conda environment doesn't work with  debugger or test explorer.

Type: Bug
Behaviour
I participated in some projects that use python version 2.7 and the Pyramid framework. As all my environments are managed through miniconda3, I just create the specific environments for each project from their requirements.
However, I have noticed some instability in the vscode to be able to debug some projects, which use python, since 2 last updates.
Now during some routine tests, as I always had the habit of running the debugger and creating breakpoints, as well as running the tests from the editor, I'm not succeeding anymore.
This my launch.json for the project reference:
{
	"configurations": [
		{
			"name": "Python: Pyramid Application",
			"type": "python",
			"request": "launch",
			"module": "pyramid.scripts.pserve",
			"args": ["--reload", "${workspaceFolder}/development.ini"],
			"pyramid": true,
			"jinja": true,
			"justMyCode": true
		}
	]
}

Trying to run the debugger I can't get any kind of output, but running the Plugin Test Explorer UI + Python Test Explorer, which always worked before, now doesn't work!
Please note that I can activate the python environments created by conda normally from vscode, as well as run all the tests in a terminal:
conda run -n project_name --no-capture-output --live-stream python ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/run_adapter.py  pytest -- --rootdir . -s --cache-clear
>>>PYTHON-EXEC-OUTPUT
<<<PYTHON-EXEC-OUTPUT
Traceback (most recent call last):
  File "/home/xarpy/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/get_output_via_markers.py", line 26, in <module>
    runpy.run_path(module, run_name="__main__")
  File "/home/xarpy/opt/miniconda3/lib/python3.9/runpy.py", line 288, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "/home/xarpy/opt/miniconda3/lib/python3.9/runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/home/xarpy/opt/miniconda3/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/xarpy/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/run_adapter.py", line 17, in <module>
    from testing_tools.adapter.__main__ import parse_args, main
  File "/home/xarpy/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/adapter/__main__.py", line 9, in <module>
    from . import pytest, report
  File "/home/xarpy/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/adapter/pytest/__init__.py", line 7, in <module>
    from ._discovery import discover
  File "/home/xarpy/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/adapter/pytest/_discovery.py", line 8, in <module>
    import pytest
ModuleNotFoundError: No module named 'pytest'
ERROR conda.cli.main_run:execute(49): `conda run python ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/run_adapter.py pytest -- --rootdir . -s --cache-clear` failed. (See above for error)

When running debugpy command  with conda run in terminal, for test if conda find the module and package to execute:
conda run -n project_name --no-capture-output --live-stream python ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/lib/python/debugpy/adapter
Content-Length: 137

{"seq": 1, "type": "event", "event": "output", "body": {"category": "telemetry", "output": "ptvsd", "data": {"packageVersion": "1.6.3"}}}Content-Length: 139

{"seq": 2, "type": "event", "event": "output", "body": {"category": "telemetry", "output": "debugpy", "data": {"packageVersion": "1.6.3"}}}

Expected vs. Actual
XXX
Steps to reproduce:

Just create the files, for debugger preffer to pyramid framework, install the plugin Test Explorer UI  and try to execute debug or test explorer.

Diagnostic data

Conda Version: <IP_ADDRESS>
Python version (& distribution if applicable, e.g. Anaconda): 2.7.18
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Conda
Value of the python.languageServer setting: Pylance
VSCode version:

Versão: 1.71.2
Commit: 74b1f979648cc44d385a2286793c226e611f59e7
Data: 2022-09-14T21:12:14.256Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
SO: Linux x64 5.15.0-48-generic
Em área restrita: No

User Settings

condaPath: "<placeholder>"

languageServer: "Pylance"

linting
• flake8Args: "<placeholder>"
• flake8Enabled: true

formatting
• autopep8Args: "<placeholder>"

testing
• pytestEnabled: true
• autoTestDiscoverOnSaveEnabled: false

Extension version: 2022.14.0
VS Code version: Code 1.71.2 (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:12:14.256Z)
OS version: Linux x64 5.15.0-48-generic
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 0, 0

Memory (System)
15.48GB (8.83GB free)

Process Argv
--unity-launch --crash-reporter-id af58d562-3f25-41a2-a7ce-ababeb5e1431

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30557514
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
hb751961:30553087
dsvsc012cf:30540253
azure-dev_surveyone:30548225
2144e591:30553903
fc301958:30573243

Seeing as you can run it manually, it might be an issue with the debugger not being able to attach to your miniconda interpreter, or because your code isn't being recognised as your own because you're running it with miniconda. Try debugging with 'justMyCode': false and see if that works. If so, this may be a duplicate of #19871.

Hi @EzraEn1 , one of the approaches suggested in some comments was to downgrade to previous versions of the ms-python package, to version  v2022.2.1924087327, This solved my problem partially, but I will test with these recommendations and try to understand the scenarios better.

Hi @EzraEn1  and @karrtikr  , the solution with this property does not work, because vsocde does not work in debug mode with the last update and conda, but if you change to the previous version of the ms-python package, as I suggested in the last post, it works, however now tests do not work anymore!

This might be that when run in VS Code it is not running in a activated environment. Can you try this?

Anaconda prompt and cd to your workspace.
Activate the environment using conda activate
Start VS Code using code .

related #10626
If that doesn't work, please change the setting terminal.integrated.inheritEnv to false as outlined in https://code.visualstudio.com/updates/v1_36#_launch-terminals-with-clean-environments.

This might be that when run in VS Code it is not running in a activated environment. Can you try this?

Anaconda prompt and cd to your workspace.
Activate the environment using conda activate
Start VS Code using code .

related #10626
If that doesn't work, please change the setting terminal.integrated.inheritEnv to false as outlined in https://code.visualstudio.com/updates/v1_36#_launch-terminals-with-clean-environments.

Hi @karrtikr, I followed these steps but was not successful, see the recording below, when I tried running vscode with conda enabled manually and accessing the project folder with vscode from the terminal::

Also, can you run
conda run -n project_name --no-capture-output --live-stream python ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/run_adapter.py  pytest -- --rootdir . -s --cache-clear

in VSCode terminal and send me the output. Make sure to expand ~.

conda run -n project_name --no-capture-output --live-stream python ~/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/testing_tools/run_adapter.py  pytest -- --rootdir . -s --cache-clear
>>>PYTHON-EXEC-OUTPUT
usage: /home/xarpy/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/testing_tools/run_adapter.py
       [-h] {discover} ...
/home/xarpy/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/testing_tools/run_adapter.py: error: argument cmd: invalid choice: 'pytest' (choose from 'discover')
<<<PYTHON-EXEC-OUTPUT
ERROR conda.cli.main_run:execute(49): `conda run python /home/xarpy/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/get_output_via_markers.py /home/xarpy/.vscode/extensions/ms-python.python-2022.16.0/pythonFiles/testing_tools/run_adapter.py pytest -- --rootdir . -s --cache-clear` failed. (See above for error)

But if I run it in the old version, as talked about earlier, the debugger works but the tests don't, only if they are run manually from the terminal.

Debugger doesn't work as we no longer support debugging for Python below 3.7, please upvote https://github.com/microsoft/vscode-python/issues/19559 if you would like to see us bring it back.

@xarpy Can you run the following command instead of https://github.com/microsoft/vscode-python/issues/19873#issuecomment-1271942780?
conda run -n project_name --no-capture-output --live-stream python ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2022.14.0/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir . -s --cache-clear

@xarpy Also try to open a new terminal and install pytest again:
python -m pip install pytest

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Mark updateState & modifyState as @noescape

@joshaber If you're using self in updateState you might be doing it wrong, but hey, it's a free country ^^

Ha, good catch!


How do I add a timestamp to build.publicPath

Environment

Operating System: Darwin
Node Version: v16.17.0
Nuxt Version: 2.15.8
Package Manager: npm
Bundler: Webpack

Describe the bug

The server rendering enabled by the project uses a CDN. When you want to reference resources, add a timestamp to distinguish the resources uploaded to the CDN each time. The validity is valid, but after the deployment to the production environment, if the project restarts automatically for some reason, the timestamp will be generated again, resulting in a mismatch with the uploaded static resource timestamp.

const dayjs = require('dayjs')
const time = dayjs().format('YYYYMMDDHHmmss')

module.exports = {
  build: {
    publicPath: `${env[process.env.MODE].PUBLIC_PATH}${time}/`
  }
}

May I ask where it is better to load the prefix of this timestamp? Because I have modified config.output.publicPath in the extend() method and found that it does not take effect after deployment, possibly because ssr is enabled.

The reason for the difference through the time stamp is that each uploaded css and js resources are placed under the same folder, resulting in a lot of junk files in this folder, but do not know which can be deleted. You can manually delete the previously deployed files.

If you want to do this, you could create a small Nuxt module that writes a file to disk with the current output path before each build. Then when you read your nuxt.config file to start the build server, this file could be read and would update your public path.


feature: Upgrading to helm 2.13.1 and Kubernetes 1.12.5

Upgrading to kubernetes 1.12.5
Upgrading to helm 2.13.1

Thanks @eimarfandino !


LLM sends a message before state transition completes

Current Setup
Interview Bot with Dynamic flow handling
initial node (greet user) -> call start_interview function (only a print statement here) -> call transition function (picks question to be asked from flow manager state) -> create and set node to question_node (specifies the question to be asked in LLM context)
Expected Functionality

LLM greets the user
Calls the start interview tool
Tool runs successfully
Transition to new question node with question to be asked in context
LLM asks the question to user

Actual Behaviour

LLM greets the user
Calls the start interview tool
Tool runs successfully
Transition to new question node with question to be asked in context
LLM asks a question on its own. Checking the LLM context at this point, I can see the new node context is not there. When I say one more line, next context that is printed shows the expected context messages.

As per my understanding, LLM sends a messages before the new node context is updated. Let me know if any code is needed from my end. I use the same format as in the dynamic flow example.

class InterviewStartResult(FlowResult):
    status: str

class AnswerCollectionResult(FlowResult):
    answer: str

class QuestionType(StrEnum):
    SUBJECTIVE = "SUBJECTIVE"
    CODING = "CODING"
    SUBJECTIVE_PROBING = "SUBJECTIVE_PROBING"

def create_initial_node(bot_prompt: str) -> NodeConfig:
    """Create the initial greeting node."""
    return {
        "messages": [
            {
                "role": "system",
                "content": bot_prompt,
            }
        ],
        "functions": [
            {
                "type": "function",
                "function": {
                    "name": "start_interview",
                    "handler": handle_start_interview,
                    "description": "Start the interview after candidate confirms",
                    "parameters": {"type": "object", "properties": {}},
                },
            }
        ],
    }

def create_question_node(
    question: dict,
) -> NodeConfig:
    """Create node for asking questions with appropriate prompts."""

    question_content = (
        "Ask the candidate the following question:\n"
        f"QUESTION : {question['question']}\n\n"
        "Instructions:\n"
        "Probe the candidate's answer to ensure they have provided a complete and accurate response, if the candidate's answer is incomplete or unclear. In such case, probe further to clarify their response. If the candidate's answer is complete and accurate, call collect_answer function. If the candidate wants to skip the question or is unable to answer, call collect_answer function with answer as '<<skipped>>'."
    )

    logger.info(
        "\n=== Creating New Question Node ===\n"
        f"Question Type: {question['type']}\n"
        f"Question: {question['question']}\n"
        "================================="
    )

    return {
        "messages": [
            {
                "role": "system",
                "content": question_content,
            }
        ],
        "functions": [
            {
                "type": "function",
                "function": {
                    "name": "move_to_next_question",
                    "handler": move_to_next_question,
                    "description": "Move to next question",
                    "parameters": {
                        "type": "object",
                        "properties": {
                            "answer": {"type": "string"}
                        },
                        "required": ["answer"],
                    },
                },
            }
        ],
    }

def create_end_node(ending_prompt: str) -> NodeConfig:
    """Create the final node."""
    return {
        "messages": [
            {
                "role": "system",
                "content": ending_prompt,
            }
        ],
        "functions": [],
        "post_actions": [{"type": "end_conversation"}],
    }

async def handle_start_interview(args: FlowArgs) -> InterviewStartResult:
    """Start the interview."""
    logger.debug("Processing start_interview transition")
    return {"status": "started"}

async def move_to_next_question(args: FlowArgs) -> AnswerCollectionResult:
    """Process answer collection."""
    answer = args["answer"]
    logger.debug(f"move_to_next_question handler executing with answer: {answer}")
    return {"answer": answer}

async def handle_interview_transition(function_name: str, args: Dict, flow_manager: FlowManager):
    """Handle transitions between interview flow states."""
    logger.debug(f"Transition callback executing for function: {function_name} with args: {args}")

    if function_name == "start_interview":
        await handle_start_interview(flow_manager)
    elif function_name == "move_to_next_question":
        await handle_move_to_next_question(args, flow_manager)

async def handle_start_interview(flow_manager: FlowManager):
    """Start interview with first question."""
    question_bank = flow_manager.state["question_bank"]
    if not question_bank:
        await flow_manager.set_node("end", create_end_node(flow_manager.state["ending_prompt"]))
        return
    flow_manager.state["total_questions"] = len(question_bank)

    await ask_next_question(flow_manager)

async def handle_move_to_next_question(args: Dict, flow_manager: FlowManager):
    """Process answer collection."""
    answer = args["answer"]
    current_question = flow_manager.state["question_bank"][flow_manager.state["current_question_index"]]

    logger.info(
        "\n=== Processing Answer ===\n"
        f"Question Type: {current_question['type']}\n"
        f"Answer: {answer}\n"
        f"Current Question Index: {flow_manager.state['current_question_index']}\n"
        "======================="
    )

    flow_manager.state["current_question_index"] += 1
    await ask_next_question(flow_manager)

async def ask_next_question(flow_manager: FlowManager):
    """Helper function to ask the next question or end interview."""
    current_index = flow_manager.state["current_question_index"]
    question_bank = flow_manager.state["question_bank"]

    if current_index >= len(question_bank):
        await flow_manager.set_node("end", create_end_node(flow_manager.state["ending_prompt"]))
        return

    next_question = question_bank[current_index]

    await flow_manager.set_node(
        f"question_{current_index + 1}",
        create_question_node(
            next_question,
        )
    )
    await flow_manager.state["websocket"].send(json.dumps({
        "type": "question_asked",
        "question": next_question
    }))


Issue with grape - 0.15.0

Hey,
Looks the newer version of grape is creating an issue with grape-attack 0.2.0. I recently ran
bundle update
and it updated me to grape - 0.15.0 and I was getting the error
NoMethodError in after callback inside Grape::Attack::Throttle : undefined method `[]=' for nil:NilClass
```

And eventually after about 2 hours, I figured it out and added version to `Gemfile` and ran
`bundle update grape`.

Everything is fine now but looks something is broken.

I'd be happy to share the trace and my Gemfile.lock.

Please look into this!

Yes, current implementation uses a "hack" to bypass an issue in grape 0.14. I need to look at the changes in 0.15 and adapt the code. In the meantime, you should probably stick with 0.14.
Thanks for reporting the issue.

Not fixed yet? Got the same error.

Full backtrace:
caught error of type NoMethodError in after callback inside Grape::Attack::Throttle : undefined method `[]=' for nil:NilClass

NoMethodError - undefined method `[]=' for nil:NilClass:
  grape-attack (0.2.0) lib/grape/attack/throttle.rb:20:in `after'
  grape (0.16.2) lib/grape/middleware/base.rb:33:in `ensure in call!'
  grape (0.16.2) lib/grape/middleware/base.rb:32:in `call!'
  grape (0.16.2) lib/grape/middleware/base.rb:23:in `call'
  grape (0.16.2) lib/grape/middleware/error.rb:29:in `block in call!'
  grape (0.16.2) lib/grape/middleware/error.rb:28:in `call!'
  grape (0.16.2) lib/grape/middleware/base.rb:23:in `call'
  rack (1.6.4) lib/rack/head.rb:13:in `call'
  grape (0.16.2) lib/grape/endpoint.rb:203:in `call!'
  grape (0.16.2) lib/grape/endpoint.rb:197:in `call'
  grape (0.16.2) lib/grape/router/route.rb:68:in `exec'
  grape (0.16.2) lib/grape/router.rb:63:in `block in identity'
  grape (0.16.2) lib/grape/router.rb:84:in `transaction'
  grape (0.16.2) lib/grape/router.rb:59:in `identity'
  grape (0.16.2) lib/grape/router.rb:44:in `block in call'
  grape (0.16.2) lib/grape/router.rb:107:in `with_optimization'
  grape (0.16.2) lib/grape/router.rb:43:in `call'
  grape (0.16.2) lib/grape/api.rb:112:in `call'
  grape (0.16.2) lib/grape/api.rb:45:in `call!'
  grape (0.16.2) lib/grape/api.rb:40:in `call'
  actionpack (4.2.6) lib/action_dispatch/routing/mapper.rb:51:in `serve'
  actionpack (4.2.6) lib/action_dispatch/journey/router.rb:43:in `block in serve'
  actionpack (4.2.6) lib/action_dispatch/journey/router.rb:30:in `serve'
  actionpack (4.2.6) lib/action_dispatch/routing/route_set.rb:817:in `call'
  omniauth (1.3.1) lib/omniauth/strategy.rb:186:in `call!'
  omniauth (1.3.1) lib/omniauth/strategy.rb:164:in `call'
  omniauth (1.3.1) lib/omniauth/strategy.rb:186:in `call!'
  omniauth (1.3.1) lib/omniauth/strategy.rb:164:in `call'
  meta_request (0.4.0) lib/meta_request/middlewares/app_request_handler.rb:13:in `call'
  meta_request (0.4.0) lib/meta_request/middlewares/meta_request_handler.rb:13:in `call'
  rack-cors (0.4.0) lib/rack/cors.rb:80:in `call'
  warden (1.2.6) lib/warden/manager.rb:35:in `block in call'
  warden (1.2.6) lib/warden/manager.rb:34:in `call'
  rack (1.6.4) lib/rack/etag.rb:24:in `call'
  rack (1.6.4) lib/rack/conditionalget.rb:38:in `call'
  rack (1.6.4) lib/rack/head.rb:13:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/params_parser.rb:27:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/flash.rb:260:in `call'
  rack (1.6.4) lib/rack/session/abstract/id.rb:225:in `context'
  rack (1.6.4) lib/rack/session/abstract/id.rb:220:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/cookies.rb:560:in `call'
  activerecord (4.2.6) lib/active_record/query_cache.rb:36:in `call'
  activerecord (4.2.6) lib/active_record/connection_adapters/abstract/connection_pool.rb:653:in `call'
  activerecord (4.2.6) lib/active_record/migration.rb:377:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/callbacks.rb:29:in `block in call'
  activesupport (4.2.6) lib/active_support/callbacks.rb:88:in `__run_callbacks__'
  activesupport (4.2.6) lib/active_support/callbacks.rb:778:in `_run_call_callbacks'
  activesupport (4.2.6) lib/active_support/callbacks.rb:81:in `run_callbacks'
  actionpack (4.2.6) lib/action_dispatch/middleware/callbacks.rb:27:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/reloader.rb:73:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/remote_ip.rb:78:in `call'
  better_errors (2.1.1) lib/better_errors/middleware.rb:84:in `protected_app_call'
  better_errors (2.1.1) lib/better_errors/middleware.rb:79:in `better_errors_call'
  better_errors (2.1.1) lib/better_errors/middleware.rb:57:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/debug_exceptions.rb:17:in `call'
  rack-contrib (1.4.0) lib/rack/contrib/response_headers.rb:17:in `call'
  meta_request (0.4.0) lib/meta_request/middlewares/headers.rb:16:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
  railties (4.2.6) lib/rails/rack/logger.rb:38:in `call_app'
  railties (4.2.6) lib/rails/rack/logger.rb:20:in `block in call'
  activesupport (4.2.6) lib/active_support/tagged_logging.rb:68:in `block in tagged'
  activesupport (4.2.6) lib/active_support/tagged_logging.rb:26:in `tagged'
  activesupport (4.2.6) lib/active_support/tagged_logging.rb:68:in `tagged'
  railties (4.2.6) lib/rails/rack/logger.rb:20:in `call'
  quiet_assets (1.1.0) lib/quiet_assets.rb:27:in `call_with_quiet_assets'
  actionpack (4.2.6) lib/action_dispatch/middleware/request_id.rb:21:in `call'
  rack (1.6.4) lib/rack/methodoverride.rb:22:in `call'
  rack (1.6.4) lib/rack/runtime.rb:18:in `call'
  activesupport (4.2.6) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
  rack (1.6.4) lib/rack/lock.rb:17:in `call'
  actionpack (4.2.6) lib/action_dispatch/middleware/static.rb:120:in `call'
  rack (1.6.4) lib/rack/sendfile.rb:113:in `call'
  railties (4.2.6) lib/rails/engine.rb:518:in `call'
  railties (4.2.6) lib/rails/application.rb:165:in `call'
  rack (1.6.4) lib/rack/content_length.rb:15:in `call'
  thin (1.6.4) lib/thin/connection.rb:86:in `block in pre_process'
  thin (1.6.4) lib/thin/connection.rb:84:in `pre_process'
  thin (1.6.4) lib/thin/connection.rb:53:in `process'
  thin (1.6.4) lib/thin/connection.rb:39:in `receive_data'
  eventmachine (<IP_ADDRESS>) lib/eventmachine.rb:194:in `run'
  thin (1.6.4) lib/thin/backends/base.rb:73:in `start'
  thin (1.6.4) lib/thin/server.rb:162:in `start'
  rack (1.6.4) lib/rack/handler/thin.rb:19:in `run'
  rack (1.6.4) lib/rack/server.rb:286:in `start'
  railties (4.2.6) lib/rails/commands/server.rb:80:in `start'
  railties (4.2.6) lib/rails/commands/commands_tasks.rb:80:in `block in server'
  railties (4.2.6) lib/rails/commands/commands_tasks.rb:75:in `server'
  railties (4.2.6) lib/rails/commands/commands_tasks.rb:39:in `run_command!'
  railties (4.2.6) lib/rails/commands.rb:17:in `<top (required)>'
  bin/rails:9:in `<top (required)>'
  spring (1.7.1) lib/spring/client/rails.rb:28:in `call'
  spring (1.7.1) lib/spring/client/command.rb:7:in `call'
  spring (1.7.1) lib/spring/client.rb:30:in `run'
  spring (1.7.1) bin/spring:49:in `<top (required)>'
  spring (1.7.1) lib/spring/binstub.rb:11:in `<top (required)>'
  /Users/frey/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
  bin/spring:13:in `<top (required)>'
  bin/rails:3:in `<main>'

Should be fixed in v0.3.0

Awesome, thanks @gottfrois!

Thanks :-)


Document Controller trusting EVE

Fill out relevant section(s) in the docs/SECURITY.md especially Initial on-boarding and Identity of EVE's instance

Closed by #1037


Add DataTables for Folder View

Closes #1188

Coverage for this isn't gonna be done - to fix it I'm going to have to get EphemeraFolderController to inherit from HyraxController, which opens up a big can of worms with expected schema properties and a conflict with single-valued property behavior. So, need to spin on this a bit. May have a supporting PR for redoing single-valued properties first.

Coverage remained the same at 100.0% when pulling 96f02cee8723644de96986a448616fdb0faeb5c2 on datatables into 5d2706a966145c32a49779edcd2c7adf66750404 on master.


Added solution to 997. FindTheTownJudge

Pull Request Template
What have you Changed(must)
Added solution to the problem 997. FindTheTownJudge with explanation.
Issue no.(must) - #1116
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@SSKale1 @Yashica7 @swaingotnochill @JOKUE2002 feel free to merge this..


[Feature request] Add onDataProviderFailure listener?

TestNG Version
7.3
Expected behavior
I wanted to add a feature to the Allure report and create attachments from the data providers. But I encountered problems with cleaning up things in case of the Data Provider failure.
Immediately after the DP failure, TestNG sets the test status to SKIPPED, and invokes the onTestSkipped listeners.
I am looking for a way how to be notified about the DP failure, and be able to perform some cleanup on such a failure.
Maybe it would be nice to have some kind of listeners on the DP failure?
Having something that can be invoked after https://github.com/cbeust/testng/blob/master/src/main/java/org/testng/internal/Parameters.java#L796 failure would be appreciated.

Hi,
Data providers are not supposed to fail and it could be considered as an execution failure a day.
Why not have a try/catch in your data provider and clean things in the catch block?

Hi,
Data providers are not supposed to fail and it could be considered as an execution failure a day.
Why not have a try/catch in your data provider and clean things in the catch block?

Hey - the thing is that I wanted to contribute to the Allure Testng project, and add attachments from the data provider to the report. We have before and after data provider listeners, where was able to create proper objects to collect the artifacts created for the Allure report. But in case of the failure, some objects remain opened, as it goes to the test listeners directly.

Hey - the thing is that I wanted to contribute to the Allure Testng project, and add attachments from the data provider to the report. We have before and after data provider listeners, where was able to create proper objects to collect the artifacts created for the Allure report. But in case of the failure, some objects remain opened, as it goes to the test listeners directly.

Wow, that fast implemented? 😅 Many, many thanks.

Wow, that fast implemented? 😅 Many, many thanks.


Add error middleware, whoops not shown, but instead 404 page shown

steps to reproduce:

use zend-expressive-skeleton with whoops
call throw new \Exception('foo') in HomeAction::__invoke()
set error middleware

        'error' => [
            'middleware' => [
                // Add error middleware here.
                function ($error, $req, $resp, $next) {
                    // do log error here...
                    // do other here ...

                    return $next($req, $resp);
                },
            ],
            'error'    => true,
            'priority' => -1000,
        ],

got 404 page instead of normal whoops

You should pass the error argument when calling $next once you hit any error middleware: return $next($request, $response, $error). Otherwise it tries to find the next normal middleware in the stack, and your error is then lost. In this case, it means that you also have an unmodified response, which leads to the 404. Pass the error when calling $next  from your error middleware, and you'll get a proper error response.

oh yeah, problem solved. Thanks ;)

@samsonasik consider checking if the docs aren't clear enough on this

@Ocramius yes, it seems we need add documentation for it


Add usage examples below import examples in table.md

Closes https://github.com/styled-system/styled-system/issues/669

Just noticed there's a couple of missing closing >'s

Thanks!


swagger-to-ts considers components mandatory

When running swagger-to-ts on a specification without components, it immediately fails.
However, components is an optional part of the api schema and I can think of at least two scenarios where it would not be present:

All schemas are inline in paths
All schemas are put in separate file(s) such as schemas.yml

My specifications are quite large and I tend to split them up into separate files as per 2). This means that operations reference schemas such as {$ref: './schemas/foo.yml#/Foo'}.
Neither of these two approaches seem to work with swagger-to-ts since it requires components to be present (and, I assume, also ignores any schemas outside of components)?

Would it be possible to invoke swagger-to-ts with a raw schema file, i.e., one that simply contains the schemas instead of requiring a full OpenAPI document?
Would it be possible to properly support OpenAPI documents without components? It should be possible to fetch all externally referenced schemas using standard json-ref-parser etc?

I encountered the same problem with openapi schema generated from Django :c

Just merged #236, so this will go out with the next release!


[E2E] Switch to upstream pytorch/benchmark repo

I've checked the current implementation, it is using pytorch/benchmark for PyTorch mode.


Updated for Angular 6

I took the liberty of updating the package to be compatible with Angular 6 as per #1.
I updated the version to 2.0.0 since the dependency updates could be troublesome but I'm not sure. please let me know your thoughts on this.

ping @chancezeus

ping @chancezeus

I'm sorry for the (long) delay, I have been swamped at work. I merged the pull request and will post an update to npm shortly, thanks for you patience (and continued pings ;))


Switch current button style to bootstrap

Hi team,
Can you guys show me how to switch current default style of buttons to bootstrap, all of them.
Thank you!

There is no such option, you might try some css.


Realistic hands

From: https://developer.leapmotion.com/unity#116

Hands Module (2.1.3)
Provides a range of realistic and low-poly rigged hand models, plus the power to design your own with an autorigging pipeline.

Where are the "realistic" hand models? None of the ones in the package are realistic and are all non-human. If you guys aren't providing realistic hands then please change the wording on your website to be less misleading.
Thanks!

Resolved!


Compass sprites don't work great

I'm trying to use Compass' sprite support in one of my Middleman projects. For example, I have a directory source/images/icon which contains some images, and then a Sass file with:
@import "icon/*.png"
@include all-icon-sprites

When I run middleman build, the icon sprite (named something like icon-s4f7554543b.png) is produced in the source directory (source/images/icon/icon-s4f7554543b.png) instead of the build directory. I can sort of work around this by building twice - the first time puts the sprite in source, and the second time copies the sprite into the build directory.
I suspect that the Compass defaults could be tweaked to get the sprites to appear in the build directory instead of the source, though I haven't yet been able to figure out exactly what to change. One complication is that the sprite image would need to also have a sitemap entry so that build --clean won't just remove it on the next build.
I haven't put together a test case or test project because the problem is pretty straightforward to reproduce, but I could do so if you wanted.

There's some stuff in the compass support file for changing paths during build to export in the right location. It's setup for images and fonts, but not sprites. I'll figure out the correct order.

The sprites do get copied nicely now, but they're also generated in the source tree, which isn't ideal - they're derived artifacts, so I don't want to have to always be adding them to my .gitignore. They should only show up in the build folder.

I think that's kind of outside our responsibilities. I guess we could see if Compass has hooks, change the sprites to be generated in some tmp folder during preview and served from there, but seems brittle.
I'd like to match Rails behavior and I'm pretty sure Rails lets Compass generate sprites into the same directory as the "real" images and serves them normally.

Talked with the Compass folks, looks like they have hooks in their alpha releases to customize the sprite behavior. So I'm going to punt on this until that version of Compass ships, then we can compile our sprites to tmp during preview.

Is still not possible to configure compass to put generated sprites to tmp folder?

Should be possible in the current alpha Compass release candidate. We'll get it in as soon as it's stable.


On operating system date

Make please clear whether "Gets the current date set in the operating system" applies to client OS or service tier OS.

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

ID: d0f93856-3097-173e-4c8e-aba67fa0c7f5
Version Independent ID: a9d74b5f-34c1-141c-bc03-78bca4053b97
Content: TODAY Function (Date) - Dynamics NAV
Content Source: dynamics-nav/TODAY-Function--Date-.md
Product: dynamics-nav-2018
GitHub Login: @jswymer

Thanks for the feedback. I will try to clear things up in the article. For the most part, it is the client that determines the date.  It is different for the Windows client and Web client. For the Windows client, TODAY returns the current day of the computer that is running the Windows client , as determined by the OS date and time settings. For the Web client, the day is determined by the Regional Settings that are set under My Settings in the client. The very first time you sign-in, the system automatically determines the Regional Setting based on your browser/computer settings.  Then, no matter where you open the client from, the date returned for TODAY is based on the Regional Settings under My Settings in the client - until you manually change them. The same holds true for the TIME function.

An updated article will be published soon. Thanks again for the feedback.


fix std.typecons.Rebindable

Points:

Make Rebindable!T.get() inout
Remove specializations for Rebindable in Phobos.
Issue 7654 - Direct using of Rebindable!T doesn't work

Sorry, I had misunderstood about Rebindable. Close this pull.


apply tensorflow 2.3.1 version

main.py
apply tensorflow 2.3.1 version
delete session
using universal-sentence-encoder version 2 to 4

requirements.txt
upgrade all library in requirements to running

RuntimeError: Exporting/importing meta graphs is not supported when eager execution is enabled. No graph exists when eager execution is enabled.
Any idea why I'm getting this error ?

@Lord-V15
This seems to be a TensorFlow version compatibility issue.
Does the RuntimeError occur in tensorflow 2.x.x version?
you should try to make new python env or conda env and then install requirements by using pip.


Added support for ss when netstat is not present

Feature: Added support for ss when netstat is not present

@latetedemelon thanks for the PR


Escape on TouchBar can not be mapped

I have tried to map the esc key of the TouchBar, but without any success.
Karabiner-EventViewer.app registers the key-up and key-down events on 0x29. But adding a mapping to it does not do anything. Escape functionality continues. Every other key can be mapped.
Tested on a 2018 MacBook Pro 13'' with macOS 10.13.6 and Karabiner-Elements-12.1.0.

Did you guys try and map the escape key to another key (e.g. control left) in preferences and then try doing your karibiner mappings?  I am wanting to purchase a 15 inch MBP but won't if I can't use the control-left as both escape and modifier during long key press.

@tommyc38  I am not entirely sure I understand what you mean by saying the following:

Did you guys try and map the escape key to another key (e.g. control left) in preferences

Which preferences do you mean?

The preferences in Mac OS (e.g. Keyboard System Preferences).

@tommyc38 Where exactly in the keyboard system preferences would I find a possibility to remap the ESC key?

http://osxdaily.com/2017/02/20/remap-hardware-escape-key-mac/

@tommyc38 If you read the article you sent, then you see that it is not possible to remap the ECS key, but to remap another key to ESC. Subtle but important difference.

I'm having this same issue. How come it is marked as stale?

This issue is still existing. How can it be re-opened?

@caillou not sure if you're having the same issue, but my escape mapping had no effect (escape would still act as escape) on the touch-bar until I enabled both devices as per https://github.com/tekezo/Karabiner-Elements/issues/780#issuecomment-459202239
Now I have successfully mapped caps_lock => escape and escape => caps_lock

Solution: Check "No Product Name" in "Devices" menu!

@jm3 I didn't fix it cause it is not a bug at all. You need to check both the items listed in the "Devices" sub-table. See below

As touch bar is recognized as a secondary keyboard, you need to make sure it is set to respond to Karabiner's manipulation.

Originally posted by @RoadToDream in https://github.com/pqrs-org/Karabiner-Elements/issues/780#issuecomment-459202239


If FileStream.Lock is not supported, point towards the solution

Ran into this issue on OS X, same as #787 and #908
Unfortunately, .net Core has no way to see if FileStream.Lock is supported, the code just throws an Exception.
One option would be to try to open and Lock the file during FileDiskService.Initialize, and change to Exclusive if it throws, but that would be a bigger functional change than just catching and showing the message.

Hi @mstum, thanks. I agree with you, there is better show a user message to always check this in initialize.
In v5 I will use, when there is no FileStream.Lock and external file lock.


Snap 535 - Python API

python API first cut
Supports  SnappyContext with basic sql and datasource api (create , truncate  and  drop ) for row and column table

@rishitesh  -  please review


ColorCodeConverter RGB-Hex Code

Color Code Converter
The "Color Code Converter" is a simple Java command-line program designed to convert color codes between different formats. It provides two primary conversion options: RGB to HEX and HEX to RGB. This tool can be a handy utility for developers and designers who frequently work with color representations.

RGB to HEX Conversion: Users can input RGB color values in the format "R, G, B," where each component (Red, Green, and Blue) is represented by an integer value from 0 to 255. The program then converts these values into a HEX color code, including the "#" symbol.

HEX to RGB Conversion: Conversely, users can input a HEX color code, either with or without the "#" symbol. The program extracts the individual color components and provides them in the RGB format, making it easy to understand the color composition.

This Color Code Converter serves as a practical example of how Java can be used to perform simple yet valuable conversions, making it a helpful addition to a developer's toolkit for working with colors in different coding or design contexts.

@matteobaccan @singloon @singloon @rajeshsgr Please review my PR


Bug Report: Some quick links in the About us page do not work.

Reproduction steps
Some of the links in the footer of the about us page (donate books, FAQ, Privacy policy) give an error on click.
Expected behavior
The links should redirect to the specified link.
Actual Behavior
It gives an error page on click.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

I am a GSsoc participant and would like to work on this issue and also work on improving the layout and styles of the section.


Question: is Swagger parameter useBasicAuthenticationWithAccessCodeGrant supported by NSwag?

Hi.
I would like to know if the parameter useBasicAuthenticationWithAccessCodeGrant mentioned here
https://swagger.io/docs/open-source-tools/swagger-ui/usage/oauth2/
is supported by NSwag.
Currently, it is not in the OAuth2ClientSettings.cs. Is there any plan for having it supported?
Thanks.

I dont think it's supported yet.
You need to transform the document manually with an IDocumentProcessor


Support poison 2.0 so it can work with the latest Phoenix.

Even unlocking mix.exs and/or overriding does not help.


(1.4.1) Crash at daily tick with ".GetRandomAnnexationDecision"

Can someone please take a look at this crash with daily tick?
I asked a "Revolutions" and "Diplomacy Fixes" devs about this crash, and they said it's have nothing to do with them. Look's like native.
I have save with crash (1 day ahead) and mods folder are included - https://drive.google.com/file/d/1D4SU4iiyl-pI9G-hSRKU3R2tB-ODLfd7/view?usp=sharing
Source: TaleWorlds.CampaignSystem
Object reference not set to an instance of an object.

Inner exception callstack:
No inner exception was thrown

Outer exception callstack:
   at TaleWorlds.CampaignSystem.SandBox.CampaignBehaviors.KingdomDecisionProposalBehavior.GetRandomAnnexationDecision(Clan clan)
   at TaleWorlds.CampaignSystem.SandBox.CampaignBehaviors.KingdomDecisionProposalBehavior.DailyTickClan(Clan clan)
   at TaleWorlds.CampaignSystem.MbEvent1.InvokeList(EventHandlerRec1 list, T t)
   at TaleWorlds.CampaignSystem.CampaignEvents.DailyTickClan(Clan clan)
   at TaleWorlds.CampaignSystem.CampaignEventDispatcher.DailyTickClan(Clan clan)
   at TaleWorlds.CampaignSystem.PeriodicTicker`1.PeriodicTickSome(Single timeUnitsElaped)
   at TaleWorlds.CampaignSystem.Campaign.Tick(Single dt)
   at TaleWorlds.CampaignSystem.Campaign.RealTick(Single realDt)
   at TaleWorlds.CampaignSystem.MapState.OnMapModeTick(Single dt)
   at TaleWorlds.Core.GameStateManager.OnTick(Single dt)
   at TaleWorlds.Core.Game.OnTick(Single dt)
   at TaleWorlds.Core.GameManagerBase.OnTick(Single dt)
   at DMD?61844338::OnApplicationTick_Patch1>(Module this, Single dt)

I captured the same crash in my play through. In my game, the Western Empire and the Comnos clan was already destroyed. I have started this game since day 1, so maybe in new games, we might never see this bug again.
According to the debugger, it would seem like it got a Null value as the random Clan and therefore crash when calling the GetRelationWithClan.
A check to ignore null clan values in the if statement should be enough to avoid the crash.

I captured the same crash in my play through. In my game, the Western Empire and the Comnos clan was already destroyed. I have started this game since day 1, so maybe in new games, we might never see this bug again.
According to the debugger, it would seem like it got a Null value as the random Clan and therefore crash when calling the GetRelationWithClan.
A check to ignore null clan values in the if statement should be enough to avoid the crash.

Yeah, I got to the point in the game where I capture every Fief of the Northern Empire. Look's like it's a late game bug with declaring wars/proposals, something along those lines.
It would be magic if someone can include a fix for this. I'm so tired of can't finish a single playthrough. Need to stop using a beta branches, huh.

The same issue on my side and a lot of players have it.
Most annoying bug/crash that I was experiencing with Bannerlord so far.

Does this still occur with the latest game patches ? @Solganis @po4emu4man

No, I didn't get it after hotfix and mods upgrade.

I didn't have a chance to play because most mods didn't updated so far and my game/save crashing in first seconds.

@JoeFwd just tested it.
I still have it in my save.

I managed to bypass it by a method I mentioned above. But dunno for how long.
 Waiting for an update for it.

So, bypass work for another several days and then, I got this crash again. Bummer.
Would be great, if someone can make a patch for this. :(

Closing it because no one is gonna fix it anyway, sadly.


[sonarqube] add nodeSelector, affinity & tolerations. add separate resoruces for plugin pod

Before this PR, there were no nodeSelector, tolerations, or affinity attributes for sonarqube (well only nodeSelector for the sonarqube pod). This PR adds them. Also, adds a separate resources attribute for the init container because it should not take as many resources as the sonarqube pod
cc: @rjkernick

/assign @unguiculus

/ok-to-test

/assign

This probably fails due to configmap mounts being read-only as of Kubernetes 1.9.4.
cc @rjkernick

@joosthof Can we switch the configmap to read only without breaking what you wrote?

@rjkernick Setting the configmap to read-only shouldn't be a problem for the install-plugins.

Please rebase master. The tests should pass with the changes that were merged yesterday.

@rjkernick thanks for that! :)
@unguiculus PR updated!

/assign

/lgtm

/assign

/lgtm

Please increment your version in Chart.yaml

@rjkernick refreshed my stale version bump with the latest version bump

/lgtm


It would be better to remove the string 'If you are looking for deleted blobs, try switching to the "Deleted only" context' when the ADLS Gen2 soft delete setting is disabled

Storage Explorer Version: 1.21.0
Build Number: 20210901.7
Branch: rel/1.21.0
Platform/OS: Windows 10/Linux Ubuntu 18.04/MacOS Big Sur 11.4
Architecture: ia32\x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Launch Storage Explorer -> Open 'Settings -> Services'.
Check the setting 'Enable ADLS Gen2 soft delete (Preview)' is disabled (if not then disable the setting and restart Storage Explorer).
Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Observe the description on its editor.
Check the string 'If you are looking for deleted blobs, try switching to the "Deleted only" context' has been removed.

Expected Experience
The string 'If you are looking for deleted blobs, try switching to the "Deleted only" context' has been removed.
Actual Experience
The string 'If you are looking for deleted blobs, try switching to the "Deleted only" context' displays.

It is kinda hard to adjust this string based on the configuration value. And the impact is low if the user isn't using soft delete so won't fix for now.


updated for 5.2

Absolutely correct about the upvalue indexing, thanks. Removed the rockspec from the pull request, too.

FYI @JorjBauer, in the future you can update your PR by just force pushing the new commits to your branch JorjBauer:pullreq_150825.


Add instruction for running

I started on a README: https://github.com/balhoff/dosdp-scala/pull/1
but we need instructions on how to run

Sorry, should be better now. I forgot that this hadn't been done.


Add watermark feature

Closes #178

@jakedex can we move the watermark onto the editor? and can we use like a carbon logo or something, to make the watermark look good?

💯


mapping nullable column value to entity value for datastax cassandra …

Kundera datastax cassandra client is simply populating default value in the entity result for nullable primitive type columns  while reading from database. The full details of issue is https://github.com/impetus-opensource/Kundera/issues/785

Hi,
Thank you for merging.  When will be this package built and release ?
shall i make the same fix on current release branch (i.e. kundera-3.1)  as well ? if i do that, will that be released immediately ?
Thanks

Thank you for merging. When will this be released ? shall i make the same fix for the recent release (kundera-3.1) branch as well ? if i do that, will that be built and released ?

@ravisund
Thank you for contributing to Kundera. This pull request will be a part of Kundera's next release (3.2).
-Karthik


Error on 'Build an Image'

Am getting an error with this on VSTS on visualstudio.com:
failed to register layer: re-exec error: exit status 1: output: ProcessUtilityVMImage C:\ProgramData\docker\windowsfilter\375e60ffec4c4894997ce3a8b2c09a7688a9da16ac3cda591404a282ebaf065e\UtilityVM: The system cannot find the path specified.
See log below
Am I missing something?

Starting: Build an image

==============================================================================
Task         : Docker
Description  : Build, tag, push, or run Docker images, or run a Docker command. Task can be used with Docker or Azure Container registry.

C:\Program Files\Docker\docker.exe failed with return code: 1

Finishing: Build an image

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

ID: 0fc9593a-77be-75c1-8d20-86a995d925f6
Version Independent ID: 63c81326-3b7a-8452-bc04-95ff06e517a7
Content
Content Source
Service: unspecified

Hi stevennwood, thanks for the question. I'm following up with the product team.

You are using 1709 version of container image, and that is not compatible with the OS of the hosted image that we run.
https://docs.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/version-compatibility
https://github.com/docker/for-win/issues/1800
We are working on supporting these newer container images. Until then, your best best is to run a private agent.

Can someone look at this error and help me out?
##[section]Starting: dotnet build Release
Task         : Command Line
Description  : Run a command line script using cmd.exe on Windows and bash on macOS and Linux.
Version      : 2.148.0
Author       : Microsoft Corporation
Help         : More Information
Generating script.
Script contents:
dotnet build --configuration Release
========================== Starting Command Output ===========================
[command]/bin/bash --noprofile --norc /home/vsts/work/_temp/99f7da28-e596-4f31-937b-3dd11a892174.sh
Microsoft (R) Build Engine version 15.9.20+g88f5fadfbe for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
MSBUILD : error MSB1011: Specify which project or solution file to use because this folder contains more than one project or solution file.
##[error]Bash exited with code '1'.
##[section]Finishing: dotnet build Release


IUA: create overview of involved standards

Luckily the IETF published a consolidated version of the OAuth Authorization Framework (https://oauth.net/2.1/) which consolidates the best practices and all the RFC which have been published in addition to the 2.0 version. I updated the references in the IUA.b working version to OAuth 2.1 draft version and the references therein. The issue should be fixed from my point of view.

so, should this be closed?


Adds support for Razor templates and RenderFragment.

Copy from #9583 by @danroth27
Update the Razor Syntax doc to document Razor template syntax. For details see https://github.com/aspnet/Blazor/pull/1360#issue-210880744.

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

ID: b87067b2-347c-072d-26c8-5a8c7c9c860b
Version Independent ID: a0b3b70a-3b39-1b5f-a4ce-c41bd2d6b044
Content: Razor syntax reference for ASP.NET Core
Content Source: aspnetcore/mvc/views/razor.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@danroth27 It's a 2.2 scenario?

I think the support for Razor template syntax (i.e. Action<object> template = @
The value is @item.ToString()
;) has been there forever. We just never documented it.

@danroth27 Hacking around isn't producing good results, and I don't see much on the Net. Can you provide a more complete example? Something simple that I thought might work is fighting me ...
@{
    Action<Pet> petTemplate = (pet) => @
You have a pet named @pet.Name.
;

    var pets = new List<Pet>
    {
        new Pet { Name = "Rin Tin Tin" },
        new Pet { Name = "Mr. Bigglesworth" },
        new Pet { Name = "K-9" }
    };
}

@pets.ForEach(petTemplate);


Surface File Details in Product Storefront

We currently process files and grab their details: dimensions, file size, length of time (audio/video), etc.  However, we do not surface the details in the product storefront.
Part of it is paranoid fear on my side about the product storefront SQL query which is quite expensive and one of the most intensive queries because it is so common and used so  much throughout the site.  It is cached per product every 24 hours at the moment.  There are a large amount of joins and adding any more makes me nervous.
That stated it is unknown just how much overhead this will add and some effort should be made to see what the performance impact is.  We're storing the data after all, why not display it?

A quick note about file dimensions as seen on the product storefront.
We have 3 versions of every file saved:

Default This is the file in its original form (no resize)
Thumbnail - resized to 320px on its dominant axis (so if the asset is 800px width x 600px height, it would be resized to 320px wide and 240px high) - This is used for social media and front page grid images
Preview - resized to 640px on its dominant axis -- used for watermarks and primary image on the storefront.

@greenbrook any chance we can get a preview of what this may look like? :D

So, Mike, it seems to me I got it wrong when I read it for the first time. Did you mean an image display like this (from Behance)? Or did you simply mean the product details (size, format) that are already added by you to the License panel?

Whoops!  Disregard this one @greenbrook.  This one's been completed. :)
From: https://alpha.starbeam.one/market/products/45c280d4-6621-4484-61f2-08db1b4b9897


Licensing

Hey @ijagberg , sorry for the noise.
Would you consider relicensing to MIT OR Apache-2.0? There is no particular reason than not having to read more licensing files and their compatibilities :)

No worries. Closing this as I added an Apache 2.0 license.


Is there jsonrpc list of ufo node?

Is there jsonrpc list of ufo node?
For example, there are json rpcs in bitcoind: getblockchaininfo, getblockcount, getblockhash, getblock.

ufo-node only has some tcp-jsonrpc apis abt mining, and please check these apis in https://github.com/ufo-project/ufochain/wiki/UFO-mining-protocol-API-(Stratum)


Pass X-Vcap-Request-Id header in calls to BBS

A short explanation of the proposed change:

To support distributed tracing request id received from the Gorouter is passed to Diego BBS so that the same request can be traced along multiple components.

An explanation of the use cases your change solves

This is part of distributed tracing feature - the ability to trace the same request across multiple components, e.g. stopping an app instance first will come to Gorouter, which will generate request ID and pass it down to Cloud Controller, which will pass it to Diego BBS, which will pass it to downstream components like Auctioneer, Rep and Garden.

[X] I have reviewed the contributing guide

[X] I have viewed, signed, and submitted the Contributor License Agreement

[X] I have made this pull request to the main branch

[X] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

We were curious why there was an extra uuid appended to the original vcap request id, and it seems like it was carrying over behavior explained in this comment?
https://github.com/cloudfoundry/cloud_controller_ng/blob/0a410b1052f4da770c79c6d20fa96bced6454a46/lib/sinatra/vcap.rb#L103-L110


1.0 pre problem SScript

Describe your problem here.
 ERROR  C:/HaxeToolkit/haxe/lib/SScript/19,0,618/src/macro/Macro.hx:10: characters 96-100
10 |   Context.fatalError('SScript is not available, thanks to everyone for their support.', (macro null).pos);
|                                                                                ^^^^
| SScript is not available, thanks to everyone for their support.
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

fixed, use latest branch https://github.com/ShadowMario/FNF-PsychEngine/tree/experimental

thanks


fix broken HTML markup

Close the opened  Tag.

fixes #5445

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

<p> does not need a closing tag (just like <img>)


[DEV/BETA] Local video filenames empty after schema patch

hi,
just noticed that current DEV/master schema patch does not update the new column 'FileName', leaving it empty after droping previous column 'FilePath'.
This results in all local files being 'empty'.

new DatabaseCommand(54, 2, "ALTER TABLE `VideoLocal` ADD `FileName` text character set utf8 NOT NULL"),
new DatabaseCommand(54, 3, "ALTER TABLE VideoLocal ADD VideoCodec varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 4, "ALTER TABLE VideoLocal ADD VideoBitrate varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 5, "ALTER TABLE VideoLocal ADD VideoBitDepth varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 6, "ALTER TABLE VideoLocal ADD VideoFrameRate varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 7, "ALTER TABLE VideoLocal ADD VideoResolution varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 8, "ALTER TABLE VideoLocal ADD AudioCodec varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 9, "ALTER TABLE VideoLocal ADD AudioBitrate varchar(100) NOT NULL DEFAULT ''"),
new DatabaseCommand(54, 10, "ALTER TABLE VideoLocal ADD Duration bigint NOT NULL DEFAULT 0"),
new DatabaseCommand(54, 11,"INSERT INTO VideoLocal_Place (VideoLocalID, FilePath, ImportFolderID, ImportFolderType) SELECT VideoLocalID, FilePath, ImportFolderID, 1 as ImportFolderType FROM VideoLocal"),
new DatabaseCommand(54, 12, "ALTER TABLE VideoLocal DROP COLUMN FilePath"),`

You right seems, mysql is missing the videoinfo migration...
UPDATE VideoLocal SET VideoLocal.FileName=VideoInfo.FileName, VideoLocal.VideoCodec=VideoInfo.VideoCodec, VideoLocal.VideoBitrate=VideoInfo.VideoBitrate, VideoLocal.VideoBitDepth=VideoInfo.VideoBitDepth, VideoLocal.VideoFrameRate=VideoInfo.VideoFrameRate,VideoLocal.VideoResolution=VideoInfo.VideoResolution,VideoLocal.AudioCodec=VideoInfo.AudioCodec,VideoLocal.AudioBitrate=VideoInfo.AudioBitrate, VideoLocal.Duration=VideoInfo.Duration FROM VideoLocal INNER JOIN VideoInfo ON VideoLocal.Hash=VideoInfo.Hash

I don't recommend regenerating videolocal. It takes forever. We might need a little patch function to only update that.

You suggest we add the above command in the update 54, then we add fix videolocal at the end of the mysql updates, which get the media information from media, and FileName from VideoLocal_Place and reconstruct?

Basically, just fetch the already generated videolocal and add the place in filename.

Yea, but you will need to extract filename, since the original field could have sub-folders. New Field seems to require only Filename.

Tested, but there is one thing to consider.
Field 'FileName' in Table 'videolocal' expects ONLY the filename itself. Value stored in the field now contains the subfolder.
Field 'FilePath' in 'videolocal_place' already contains the filepath.
Solution would be something like this:

UPDATE `videolocal` v SET v.FileName = ( SELECT SUBSTR(vp.FilePath FROM POSITION('\\' IN vp.FilePath) + 1 FOR LENGTH(vp.FilePath)) FROM videolocal_place vp WHERE vp.VideoLocalID = v.VideoLocalID )

Not 100% working, since it does not handle more than 1 subfolder.

Tested, but there is once thing to consider.
Field 'videolocal.FileName' does only require the filename itself.
Value for 'videolocal.FileName' now contains subfolders.
Value for new field 'FilePath' in videolocal_place already contains the subfolder.
This results in FileNames being displayed incorrect.
You probably need something like this:

UPDATE `videolocal` v SET v.FileName = ( SELECT SUBSTR(vp.FilePath FROM POSITION('\\' IN vp.FilePath) + 1 FOR LENGTH(vp.FilePath)) FROM videolocal_place vp WHERE vp.VideoLocalID = v.VideoLocalID )

This is NOT 100% working, since it does only handle subfolder depth of 1. I only used this to fix the the Filename AFTER the already deleted field in videolocal table.
(I just did run the command multiple times to handle more than 1 subfolder)
To demonstrate this:
Value for videolocal_place.FilePath' now is "Asura Cryin[Chihiro]Asura_Cryin-01[1280x720_H.264_AAC][6F91D254].mkv"
Value for videolocal.FileName is "Asura Cryin[Chihiro]Asura_Cryin-01[1280x720_H.264_AAC][6F91D254].mkv" too. This results in wrong FileNames. Altho as it seems only displayed filenames seem to be wrong. The file is still playable. I will test this furhter.

Did you check that at the end there is a databasefix.
 public static void FixEmptyVideoInfos()
        {
            List<VideoLocal> locals = RepoFactory.VideoLocal.GetAll().Where(a => string.IsNullOrEmpty(a.FileName)).ToList();
            foreach (VideoLocal v in locals)
            {
                VideoLocal_Place p = v.Places.OrderBy(a => a.ImportFolderType).FirstOrDefault();
                if (p != null && !string.IsNullOrEmpty(p.FilePath) && v.Media != null)
                {
                    **v.FileName = p.FilePath;
                    int a=p.FilePath.LastIndexOf("\\",StringComparison.InvariantCulture);
                    if (a > 0)
                        v.FileName = p.FilePath.Substring(a + 1);**
                    VideoLocal_Place.FillVideoInfoFromMedia(v,v.Media);
                    RepoFactory.VideoLocal.Save(v,false);
                }
            }
        }

Is this not working?

hmm, this is strange. Yea, i did check after patch of course.
Here is what it looks like in client:

Ok, i did add a breakpoint and did debug the code. As it turns out, debugger didnt stop within the 'DatabaseFixes.FixEmptyVideoInfos' function. As it seems, app doesnt go in there.

Please check, the versions table in the database, delete version row 58.1, and also set filename to null in the database. That will make the databasefix run again.

yea i will check that now. Of course i always deleted all tables, and restored my backup SQL from version <IP_ADDRESS> prior testing this. So it should be working. Really strange. I double check DB and all now, just to be sure.

Hmm, if you restored, it should run. Gonna check...

Yup seems about right, please restore db, then put a breakpoint in FixEmptyVideoInfos.

this is really strange... just to be sure, i did add breakpoint in other database-fix functions, but the debugger won't go in there at all. Not sure if it's just Visual Studio acting up. As it seems, the FixEmptyVideoInfos() function is never executed. I did add breakpoint in public class DatabaseCommand ->  DatabaseCommand(int version, int revision, Action databasefix) constructor. And the command is created and added to the list just not executed it seems

Okey, go to DatabaseFactory Line 99. Put a breakpoint in there.  Instance.ExecuteDatabaseFixes();
When hit press F11. And check if the list of the fixes includes
FixEmptyVideoInfos
Thanks

ok, looks like i found the problem:

 try
                {
                    Instance.CreateAndUpdateSchema();
                    RepoFactory.Init();
                    Instance.ExecuteDatabaseFixes();
                    Instance.PopulateInitialData();
                }
                catch (DatabaseCommandException e)
...

I'm not sure, but it seems the repo cache is created BEFORE the patches are being applied to db. So maybe data is inconsistent. Or an exception is happening within repo function. I will leave it, recreating repo and see what happens (debugger running and breakpoint)
As i see it now, the code only reaches till RepoFactory.Init();

Instance.ExecuteDatabaseFixes();
Instance.PopulateInitialData();

are not executed. But i could be wrong, will wait until repo is created.

But why FileName has the path, it shouldn't.  Your original VideoInfo.FileName should be move to VideoLocal.FileName, that Databasefix, if for people who updated like you, the database and have no backup. Check in your original Database, the value inside VideoInfo.FileName it should be moved to VideoLocal.Filename , Last time i checked VideoInfo.Filename should only have FileName on previous versions of the database.

hmm, this is strange then. I restored the backup and 'VideoInfo.FileName' does indeed contain the sub-folder:

this is getting way too strange now. I will do the following now. I will revert to current stable version. Clear whole DB again and run Import. Then we will see, if data is stored previously without subfolder.

Ok, reverted to current stable version <IP_ADDRESS>, deleted whole DataBase and did run an import again.
I did check VideoInfo.FileName and it still contains Subfolder

So i'm assuming, it should be the same for all users updating from current stable version.

Hmm, Thats the key issue. VideoInfo has the filepath in filename, wonder why. Any clue from other devs?

Tested, issue is resolved with your last commit.  :)

@t0815 I've been unable to open the "Utilities" tab in JMM Desktop <IP_ADDRESS> and get a null reference error each time:
[2016-10-24 12:19:10:395] Error|MainWindow.DisplayMainTab => UnrecognisedVideos.RefreshUnrecognisedFiles => Utils.ShowErrorMessage System.NullReferenceException: Object reference not set to an instance of an object.    at JMMClient.ViewModel.ImportFolderVM..ctor(Contract_ImportFolder contract) in D:\My Files\Documents\GitHub\jmmclient\JMMClient\ViewModel\ImportFolderVM.cs:line 88    at JMMClient.ViewModel.VideoLocalVM..ctor(Contract_VideoLocal contract) in D:\My Files\Documents\GitHub\jmmclient\JMMClient\ViewModel\VideoLocalVM.cs:line 127    at JMMClient.UserControls.UnrecognisedVideos.RefreshUnrecognisedFiles() in D:\My Files\Documents\GitHub\jmmclient\JMMClient\UserControls\UnrecognisedVideos.xaml.cs:line 844
I can't say for sure this is related or what's causing this, just wanted to know if you have any insight on this.

Got the latest version of JMM Client from repo and it works great... I guess downloaded <IP_ADDRESS> version doesn't work with latest dev JMM, which shouldn't be too surprising.

Did you check that at the end there is a databasefix.
public static void FixEmptyVideoInfos()
{
List locals =
RepoFactory.VideoLocal.GetAll().Where(a =>
string.IsNullOrEmpty(a.FileName)).ToList();
foreach (VideoLocal v in locals)
{
VideoLocal_Place p = v.Places.OrderBy(a =>
a.ImportFolderType).FirstOrDefault();
if (p != null && !string.IsNullOrEmpty(p.FilePath) &&
v.Media != null)
{
*  v.FileName = p.FilePath;*

               int

a=p.FilePath.LastIndexOf("\",StringComparison.InvariantCulture);*

               if (a > 0)*

                   v.FileName = p.FilePath.Substring(a + 1);*
              VideoLocal_Place.FillVideoInfoFromMedia(v,v.Media);
              RepoFactory.VideoLocal.Save(v,false);
          }
      }
  }

Is this not working?
On Sun, Oct 23, 2016 at 10:16 AM, t0815<EMAIL_ADDRESS>wrote:

Tested, but there is once thing to consider.
Field 'videolocal.FileName' does only require the filename itself.
Value for 'videolocal.FileName' now contains subfolders.
Value for new field 'FilePath' in videolocal_place already contains the
subfolder.
This results in FileNames being displayed incorrect.
You probably need something like this:
UPDATE videolocal v SET v.FileName = ( SELECT SUBSTR(vp.FilePath FROM
POSITION('' IN vp.FilePath) + 1 FOR LENGTH(vp.FilePath)) FROM
videolocal_place vp WHERE vp.VideoLocalID = v.VideoLocalID )
This is NOT 100% working, since it does only handle subfolder depth of 1.
I only used this to fix the the Filename AFTER the already deleted field in
videolocal table.
(I just did run the command multiple times to handle more than 1 subfolder)
To demonstrate this:
Value for videolocal_place.FilePath' now is "Asura Cryin[Chihiro]
Asura_Cryin-01[1280x720_H.264_AAC][6F91D254].mkv"
Value for videolocal.FileName is "Asura Cryin[Chihiro]Asura_Cryin-01[
1280x720_H.264_AAC][6F91D254].mkv" too. This results in wrong FileNames.
Altho as it seems only displayed filenames seem to be wrong. The file is
still playable. I will test this furhter.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/japanesemediamanager/jmmserver/issues/488#issuecomment-255588056,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHDbYFpkBO0ur9eoXiUFlIk1zx6p7Jkks5q215FgaJpZM4Kdx61
.

@monsieurtaco ; correct, current dev/master is not compatible with current stable version DB layout. You need to restore either a Backup prior to changes made by current DEV build or keep using dev/master build.