Add absolute path support for compatibility testing(#6787)

Compatibility tester needs that the yaml files used for testing (and the file paths within yaml files) reside in the resource directory path. We should change these so that the files can be absolute path. This will allow Pinot installations to specify their own test suite and run compat testing before upgrade to new pinot versions.
The tester script should take a directory name (absolute path) as the argument where all the yaml files are located. The yaml files should be named as pre-controller-upgrade.yaml, pre-broker-upgrade.yaml, etc (6 of them) . The shell script should be modified to pick up these yaml files from the directory specified.
In addition, the directory should contain a subdir called config under which all the config file names should reside (e.g. the table config files, schema files, kafka producer config files. etc.), possibly under subdirectories if the user chooses to put them that way. These config files are the ones referenced inside the yaml files.
So ,if a yaml file referes to a table config file as mytables/table1.conf, then we should look in the directory ${TOPLEVELDIR}/config/mytables/table1.conf in order to retrieve the table config file.

Codecov Report

Merging #6828 (cdf8a3e) into master (7299b56) will decrease coverage by 8.05%.
The diff coverage is n/a.

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

@@             Coverage Diff              @@
##             master    #6828      +/-   ##
============================================
- Coverage     73.97%   65.92%   -8.06%
  Complexity       12       12
============================================
  Files          1420     1420
  Lines         69102    69102
  Branches       9976     9976
============================================
- Hits          51121    45553    -5568
- Misses        14634    20306    +5672
+ Partials       3347     3243     -104

Flag
Coverage Δ
Complexity Δ

integration
?
?

unittests
65.92% <ø> (-0.03%)
12.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Create function to call API for Flickr

Write JS/JQuery function to call Flickr API to get images based on tags

completed successfully


Audit cursor usage for concurrency bugs

Reticketed from https://github.com/gratipay/gratipay.com/pull/4397#discussion_r112209778.
Grep for get_cursor and read it all.

On #4397, SELECT ... FOR UPDATE turned out to be a pretty good pattern for synchronizing inside Postgres.


[Feature]: clean docs about CSI

Contact Details
No response
Is there an existing issue for this?

[X] I have searched all the existing issues

Is your feature request related to a problem? Please describe.
https://cubefs.io/docs/master/user-guide/k8s.html
I think it's time to remove docs about Kubernetes versions before v1.13
Describe the solution you'd like.
I didn't want to see outdated docs
Describe an alternate solution.
No response
Anything else? (Additional Context)
No response

@heymingwei  please consider this suggestion


[Native filter] Numerical filter can not select when the column has value range 0-1

In my dataset, column value is from 0-1, when i create numerical filter using that column, i can not select on the filter range.
How to reproduce the bug
Precondition:
Have column with value from 0-1

Go to native filter and create numerical filter using the column with value 0-1
Try adjust the filter range
See error

Expected results
Numerical filter can work with column value 0-1
Actual results
Numerical filter can not be selected with column value 0-1
Screenshots
https://user-images.githubusercontent.com/81597121/136623959-08362543-dd1d-42c7-b1c9-f5f556cc2067.mov
Environment
(please complete the following information):

browser type and version: chrome
superset version: master
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

This is related to an enhancement I'm proposing. The Select component has a step size option, but it isn't implemented in the UI. See #16591

@stephenLYZ wondered if you have any insights here...

Thanks for the PR to fix this @stevetracvc - I think your open PR solves it. Also, thanks @geido for helping that PR through. I'll assign this issue to both of you since you seem to have it in-hand :)

Just updating this thread to mention that @stevetracvc is hoping to revisit a PR to close this issue out, and mark the thread as active again.

Closing this as stale since it's been silent for so long, and we're trying to steer toward a more actionable Issues backlog. If people are still encountering this in current versions (currently 3.x) please re-open this issue, open a new Issue with updated context, or raise a PR to address the problem. Thanks!


Suggestion - fail2ban integration with reverse proxy

I'm running ttyd behind an apache2 reverse proxy.
I've created a setup that forwards the remote address of the party logging in with ttyd to /var/log/auth.log
To achieve that, I've written a simple alt-login program, which also always denies root access.
Also a script (install.sh) that:

installs ttyd as a system service
installs a fail2ban configuration to block ip addresses that try to login too often, as a security measure

The repository provides an apache2 configuration file with the reverse proxy setup and rewrite rules to let the whole setup work correctly.
Just to let you know.
Repository is here: https://github.com/hdijkema/alt-login

You may want to add it to the wiki


Fix CI issues

I looked a bit at the ci_test_compilers_clang (10) issue.
This PR did not change the affected test and it worked in earlier builds.
It seems the compiler and changed to a newer version.
old:
Compiler: clang version 10.0.0; Target: x86_64-unknown-linux-gnu; Thread model: posix; InstalledDir: /usr/local/bin

new:
Compiler: Debian clang version 10.0.1-++20210313014605+ef32c611aa21-1~exp1~20210313125208.190; Target: x86_64-pc-linux-gnu; Thread model: posix; InstalledDir: /usr/bin

Pull request checklist
Read the Contribution Guidelines for detailed information.

[ x]  Changes are described in the pull request, or an existing issue is referenced.
[ ]  The test suite compiles and runs without error.
[x ]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x ]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Coverage: 100.0%. Remained the same when pulling 23254861e6995cc7c788f7bdcbfb18ec51fb115a on barcode:fix_ci_issue into 4c6cde72e533158e044252718c013a48bcff346c on nlohmann:develop.

The error makes it pretty clear that libstdc++-8 is used.
/usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/fs_path.h

but span is only supported starting with libstdc++-10.
I guess the old version of the docker container had this or a more recent version installed, but i can't confirm, since i don't know how to get the old version of the image.

@nlohmann
To get cirrus working you need to add their app to the repo. (I don't have the permissions to do this)
Also the .cirrus.yml has to be in the root dir of the repo.

It seems filesystem in libstdc++-8 does not really work when linking (even though -lstdc++fs is set):
https://github.com/nlohmann/json/actions/runs/3922291644/jobs/6705139818
  [148/166] : && /usr/bin/clang++ -g -lstdc++fs tests/CMakeFiles/test_main.dir/src/unit.cpp.o tests/CMakeFiles/test-regression2_cpp20.dir/src/unit-regression2.cpp.o -o tests/test-regression2_cpp20   && :
  FAILED: tests/test-regression2_cpp20
  : && /usr/bin/clang++ -g -lstdc++fs tests/CMakeFiles/test_main.dir/src/unit.cpp.o tests/CMakeFiles/test-regression2_cpp20.dir/src/unit-regression2.cpp.o -o tests/test-regression2_cpp20   && :
  ...
  /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/fs_path.h:184: undefined reference to `std::filesystem::__cxx11::path::_M_split_cmpts()'
  ...
  /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/fs_path.h:545: undefined reference to

To fix this we would need to build out own version of libstdc++ (there are no backports for debian buster). I deactivated filesystem for ci_test_compilers_clang (10), since building  libstdc++ would take a lot of time in the runner.
Now we only need to finish replacing drone with cirrus. (Since cirrus is currently deactivated, I reactivated drone to prevent the checks from succeeding.)

@nlohmann To get cirrus working you need to add their app to the repo. (I don't have the permissions to do this)

I just did this.

Also the .cirrus.yml has to be in the root dir of the repo.

This should be done after merging, right?

I just did this.

Ok, now i have to figure out why cirrus does not want to build this branch...

This should be done after merging, right?

This will be done by this PR. It was meant as an explanation why the cirrus file is not placed in .github/external_ci.

There is clearly some strangeness going on with cirrus. If i change the file and already have the cirrus repo page open It gets updated  with an entry:

When clicking on it i get a not found error and on reloading it disappears.
@nlohmann Maybe there is some setting you can change. (not sure what it would be, but maybe the repo setting Require approval for builds from users without write permissions is activated)
Just in case you do not find some setting, I will try to contact the cirrus support. Maybe they can help.

There is clearly some strangeness going on with cirrus. If i change the file and already have the cirrus repo page open It gets updated with an entry:
When clicking on it i get a not found error and on reloading it disappears.
@nlohmann Maybe there is some setting you can change. (not sure what it would be, but maybe the repo setting Require approval for builds from users without write permissions is activated)
Just in case you do not find some setting, I will try to contact the cirrus support. Maybe they can help.

This is how it looks like:

I switched on Require approval for builds from users without write permissions. Let me know if I should try anything else.

At least now the build job stays in https://cirrus-ci.com/github/nlohmann/json instead of disappearing again.
I tried setting it up in my fork, but there i got the same issues. The builds disappear (even a manually triggered one.
Maybe it has something to do with the current issues with github actions:
https://www.githubstatus.com/

Investigating - We are investigating reports of degraded performance for Actions.
Jan 30, 2023 - 12:07 UTC

@nlohmann can you try approving one of the jobs listed on https://cirrus-ci.com/github/nlohmann/json? Maybe this fixes some strangeness in their back end.

Done. I approved three builds and then turned off approvals.

Great it seems like we finally have a working CI again.
Now this PR is ready for review.

Thanks a lot!!!!!


node: remove python 3.11 as a dev dependency

Also add npm explicitly as a dev dependency.
Python 3.11 has been a dev-only dependency since the inception of dev variants for node, back in https://github.com/chainguard-images/images/pull/479 -- it's not clear from that PR why it's needed, and I don't think we have a test that exercises the behavior.
If Python is needed, we should upgrade to 3.12 if possible, and ensure we can smoothly upgrade without toil in the future.
If Python isn't needed, we should remove it. If folks need it, they can apk add it themselves.

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!


Removed the invalid parameters "width" and "height" from the "text" mark

"width" and "height" aren't valid text attributes (and the "auto" value for "height" is also not valid). Please correct me if I am wrong on this.

@RussellSprouts could you take a look at this and verify? Just want to make sure we don't use width/height to set the bounding box of things, for instance.


[Feature]It is recommended to add owner to the process instance and task instance pages

在工作流实例和任务实例页面加上任务owner，因为一个团队中有好几个人添加任务，当需要在成百上千个任务中查看自己的任务运行情况时会方便很多
Add task owner to the page of process instance and task instance, because there are several people in a team adding tasks, it will be much more convenient when you need to view the operation of your own tasks in hundreds of tasks

thank you very much for your suggestion, that would be very useful, we'll conside that in the next version.

hi @JinyLeeChina
please change the title to english. or submit a new issue.

The owner should be the executor user, i will implement this


Amend documentation

Currently, the README examples for scale-down rules have positive numbers in the scaling factor. However, we are experiencing that scale-down rules with positive factors result in undesired scaling up.
This documentation change is supported by:
https://github.com/tendrilinc/marathon-autoscaler/blob/master/tests/simulation_data/app_definition.json#L61-L62
and
https://github.com/tendrilinc/marathon-autoscaler/blob/master/lib/marathon_autoscaler/scaler.py#L36

Looks good. Thanks for the contribution!


Not working.

Not receiving any updates since last week. Any issue with the app?

Yep is literally not working anymore I thought I had some issues. After restarting , delete and redownload its still the same. My last updated was on 21st of august 2024. That's it no more updates.

It's because this script get data from somewhere else which has bug:
https://github.com/Osb0rn3/bugbounty-targets/issues/6


Swipe feature question

Is it expected behavior that after removing item via swipe - all list items which are present on screen updated and onBindViewHolder called for every visible list item?

Yes. That is an intetional behavior.


Network protocol [Syncing node]

[x]  Noise security protocol in libp2p
https://github.com/soramitsu/kagome/issues/386

[x]  Transactions protocol
https://github.com/soramitsu/kagome/issues/532

[x]  Connect to /sup/sync/2
https://github.com/soramitsu/kagome/issues/484

[x]  DNS support for addresses in libp2p
https://github.com/soramitsu/kagome/issues/406

[x]  Check libp2p identify
https://github.com/soramitsu/kagome/issues/485

[x]  Rework gossiper
https://github.com/soramitsu/kagome/issues/546

[x]  Block announce protocol
https://github.com/soramitsu/kagome/issues/576

[x]  [Libp2p] Timeout of opening new stream
https://github.com/soramitsu/kagome/issues/604

[x]  [Libp2p] Continue trying to connect to peer by other addresses at fail one of them
https://github.com/soramitsu/kagome/issues/605

[x]  [Libp2p] Support '/dnsaddr' addressess
https://github.com/soramitsu/kagome/issues/606

Aha! Link: https://soramitsucoltd.aha.io/features/PRE-59

All of nested issues has been resolved


Ignore ide settings

Just ignore .ide/ folder.

@samuelterra22 thanks for the contribution, but please check out Global/JetBrains.gitignore which covers the setup for all JetBrains editors.


Non-nullable default parameters

$container = new Container([
    'items' => [
        '__class' => ArrayIterator::class,
        '__construct()' => [
            [],
        ],
    ],
]);

$items = $container->get('items'); // Uncaught TypeError: ArrayIterator::__construct() expects parameter 2 to be int, null given

assert($items instanceof ArrayIterator);

$container = new Container([
    'items' => [
        '__class' => ArrayIterator::class,
        '__construct()' => [
            ['one', 'two'],
        ],
    ],
]);

$items = $container->get('items'); // Uncaught TypeError: ArrayIterator::__construct() expects parameter 2 to be int, null given

assert($items instanceof ArrayIterator);

Even
$container = new Container([
    'items' => [
        '__class' => ArrayIterator::class,
        '__construct()' => [
            [],
            0
        ],
    ],
]);

$items = $container->get('items'); // Uncaught TypeError: ArrayIterator::__construct() expects parameter 2 to be int, null given

assert($items instanceof ArrayIterator);

but
$container = new Container([
    'items' => [
        '__class' => ArrayIterator::class,
        '__construct()' => [
            ['one', 'two'],
            0
        ],
    ],
]);

$items = $container->get('items');

assert($items instanceof ArrayIterator); // Ok

Very good point! Thank you!
I'll try to fix it.

That's a PHP issue :)
https://3v4l.org/7CGBZ
Reflection returns wrong information.
Looks like fixed in PHP 8.
Also, please see test in #157
The container process default parameters correctly.

@hiqsol but why
'__construct()' => [
    [],
    0
],

does not work?

It's a bug. Fixing it.

Fixed in https://github.com/yiisoft/factory/pull/46/files


Expose catalog via the  json+ld script to make it all searchable by e.g. google dataset search

here is how it looks for a sample openneuro dataset:

if datalad-catalog while harmonizing data for its own display could expose it as a json-ld -- that would multiply utility of the tool.

Good reminder, thanks! Linking existing issue: https://github.com/datalad/datalad-catalog/issues/20

indeed. Let's keep just that one open


“Use of '@import' when modules are disabled” Error for FirebaseUiAuth

Step 1: Are you in the right place?
80% chance :thinking:
Step 2: Describe your environment

Objective C or Swift: Swift5
iOS version: 13.5
Firebase SDK version: 6.6
FirebaseUI version: 8.4.0
CocoaPods Version: 1.9.1

Step 3: Describe the problem:
Using FirebasUI dependencies in kotlin-native with cocoapods support throws error during compilation.
Not sure if valid fix, but if I replace module imports in FirebaseUI code then our project compiles successfully.

@import UIKit; => #import <UIKit/UIKit.h>
@import Foundation => #import <Foundation/Foundation.h>

edit:

setting Enable modules (C and Object-c) is true for project.
cacaopods Podfile contains use_frameworks! :linkage => :static

Steps to reproduce:

Add 'FirebaseUI/Google dependency to to kotlin-native project
Execute gradle podspec cmd
Execute pod install
Build iOS project
Observe error

Observed Results:
Use of '@import' when modules are disabled

error is thrown in 3 places:

FirebaseAuthUI.h
FUIAuthErrors.h
FirebaseGoogleAuthUI.h

Expected Results:
Successful compilation

Hi. I am having the same issue building FirebaseUI for iOS, mixing c++ and ObjectiveC. No idea how to fix this. Any help?

@morganchen12 would you accept PR which replaces these imports?

Yes, PRs are very welcome.

https://answers.unity.com/questions/1679641/use-of.html
need c++ flag setup

https://answers.unity.com/questions/1679641/use-of.html
need c++ flag setup

Using -fmodules and -fcxx-modules flags fixes this as long as the file extension is .m
(but not supported by .mm or rather Objective-C++ yet).


all: add vanity import specs

This CL adds vanity imports to all packages to prevent importing any
given package from the wrong import path.

Thanks for the contribution @sbinet! We need all the commits signed off. You have more information here:
https://github.com/probot/dco#how-it-works
You can do that easily for this contribution with:
$ git commit --amend -s
$ git push --force

PTAL

@sbinet thanks a lot!


Did TrustKit use the fishhook ?

I can't find any code of fishhook, but I saw that TrustKit use the fishhook from http://stackoverflow.com/questions/31649581/why-i-cant-hook-my-private-function-with-fishhook

Yes, TrustKit was using fishhook initially but not anymore; details here: https://nabla-c0d3.github.io/blog/2015/10/17/trustkit-ios-9-shared-cache/

I got it,  thank you for being so helpful!


Access to the code generated to plot

Hi, Is there any way to get access to the code that lida generate and use to plot?

Can you share the code you have right now?
In general,

from lida import Manager, llm

lida = Manager(text_gen = llm("openai")) # palm, cohere ..
summary = lida.summarize("data/cars.csv")
goals = lida.goals(summary, n=2) # exploratory data analysis
charts = lida.visualize(summary=summary, goal=goals[0]) # exploratory data analysis

In the snippet above, charts is an array of objects ChartExecutorResponse which has a code field.
charts[0].code

Thanks.
I am using this demo:
https://github.com/AIAnytime/LIDA-Demo-Streamlit-App/blob/main/app.py
Another problem that I am facing for some queries is that the charts is empty list []. But interestingly, once it happens it shows the code as log and that code is actually working! I was wondering is this a bug? The problem is that if list is empty then I cannot access that code either with charts[0].code


Puppeteer ignorehttpserrors setting not picked up by gulp task

Trying to extract critical css from https production site. I'm getting a "SSL handshake failed" error that is not being addressed by my custom puppeteer settings.
Not sure if custom browser settings are not supported with gulp or if I'm writing the settings incorrectly.
  const browserPromise = puppeteer.launch({
    ignoreHTTPSErrors: true,
    args: ['--disable-setuid-sandbox', '--no-sandbox', '--ssl-protocol=any', '--ssl-protocol=tlsv1', '--ignore-ssl-errors=yes'],
  });
  const mapping = {
    site: ["wac", "jira"],
    url: [
    "<url1>",
    "<url2>"
  ]
  };
  for (let i = 0; i < mapping.site.length; i++) {
    gulp.src(paths.dist.css + 'wpl-main.css')
    .pipe(criticalCss({
      out: mapping.site[i] + "-critical.css",
      url: mapping.url[i],
      puppeteer: {
        getBrowser: () => browserPromise
      },
      width: 1600,
      height: 900,
      userAgent: 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.132 Safari/537.36',
      blockJSRequests: false,
    }))
    .pipe(streamify(minifycss({
      debug: false
    })))
    .pipe(gulp.dest(paths.dist.css));
  }
});```

@stbarillas are you still having issues with this?

@edward-simpson , no longer having this issue. Closing

Thats strange as I am, I've had to use penthouse directly to get it working

Hi, I am having this issue, was there ever a resolution? I am using gulp-penthouse. Thanks


Method $ should NOT automatically open a browser

When using Selenide, a normal flow is to first open a browser, and then acting with elements. Like this:
open(url);
$(...).do(...);

The problem
If one forgers to call open and calls $, Selenide still opens a browser. It's kind of unexpected behaviour. It can cause mysterious bugs.
Solution
I suggest to change method $, so that it throws an exception if a browser is not open yet.

Implemented in branch selenide-driver

@asolntsev just to be sure. For $$().someaction() the same?

@BorisOsipov Yes, sure. Both $ and $$ should throw an exception if browser is not opened.

It's a good point! Awesome 🎉


Add the new patch for stable/juno neutron

Add a new version of the patch that works with neutron
stable/juno.
Signed-off-by: Rosario Di Somma<EMAIL_ADDRESS>

Looks okay to me but I don't understand the full implications of the patch. :+1:


fix vm connectivity, pin openstack.cloud version

This patch matches with this PR to fix the IP of the seed and fix internet connectivity for VMs using the Seed as a gateway


Added swift-algorithm-cllub.workspace so all projects can be viewed together

Using a workspace brings all of the projects together in a single window with all of the code highlighting, search and documentation features that Xcode brings. I found it extremely useful when viewing/comparing multiple projects.

Thanks @kevinrandrup! 👍 This is also really helpful for finding existing issues across all projects:

I'll add new algorithms to this workspace when they are submitted to the repository.


Add Epinio landing page

Fixes https://github.com/rancher/rancher-docs/issues/939
Description
Adds a landing page for the Epinio integration.

@btat Do all the other landing page PRs also need the 2.8 milestone?

Per discussion with Guna, we're proceeding to merge so stakeholders are able to preview the initial design/update. We'll continue to improve/iterate and address any feedback post-merge.


UNITY-492 Hand Binder Editor Variables

Pull Request Templates
Switch template by going to preview and clicking the link - note it not work if you've made any changes to the description.

default.md - for contributions to stable packages.
lightweight.md - for contributions to pre-release/preview packages use the lightweight merge request template. The quality threshold for reviewing is lower - it prioritizes a lean review process.
release.md - for release merge requests.

You are currently using: default.md
Note: these links work by overwriting query parameters of the current url. If the current url contains any you may want to amend the url with &template=name.md instead of using the link. See query parameter docs for more information.
Summary
This PR removes the public access to variables that are used in the editor script only.
Contributor Tasks
These tasks are for the merge request creator to tick off when creating a merge request.

[ ] Pair review with a member of the QA team.
[x] Add any release testing considerations to the MR for the next release.
[x] Check any relevant CHANGELOG files have been updated.
[x] Ensure documentation requirements are met e.g., public API is commented.
[x] Consider any licensing/other legal implications for this MR e.g. notices required by any new libraries.
[x] Add any relevant labels such as breaking to this MR.
[ ] If this MR closes a Jira issue, make sure the fix version on the JIRA issue is set to the correct one.

Reviewer Tasks
Add any instructions or tasks for the reviewer such as specific test considerations before this can be merged.
Use emojis in review threads to communicate intent and help contributors.

[x] Code reviewed.
[x] Non-code assets e.g. Unity assets/scenes reviewed.
[x] Documentation has been reviewed. Includes checking documentation requirements are met and not missing e.g., public API is commented.
[ ] Checked and agree with release testing considerations added to MR for the next release.

Closes JIRA Issue
If this MR closes any JIRA issues list them below in the form Closes PROJECT-#

Can we somehow get rid of these warnings? Seems like unity doesn't know that it is being used in the editor script..

@Greg-UL This PR makes access to some editor-only variables private, i don't believe there are any testing considerations to add
Just to be double safe, when making the PR I ran the Binder on our generic hands to ensure it still acts as expected and it did
Are you happy with that? :)

I think so. Is it likely this change could have impacted on the autoscaling functionality at all?

it won't have any impact on the autoscaling :)

I think so. Is it likely this change could have impacted on the autoscaling functionality at all?

it won't have any impact on the autoscaling :)

then I'm happy - please proceed


Configuring logs to specific outputs

Is there a way to have two RichTextBoxLogControl and fill them with different outputs because context like special class MyClass doesn't work;

RichTextBoxLogControl 1
RichTextBoxLogControl 2

Log for 1
Log for 2

Another Log for 1
Another Log for 2

Did you try ForContext property on the richtextbox?

ForContext should be like namespace.classname

It should be working through nuget package as well.. i will check why its not working with published package

@muratcanoguzhan  please try this version
https://www.nuget.org/packages/Serilog.Sinks.WinForms/2.4.3


Add additional phase for timeouts

What this PR does / why we need it:
This PR adds an additional phase called "Timeout" which applied if a argo node exceeds their activeDeadlineSeconds.
Special notes for your reviewer:
Should we add the finishedAt and duration according to the activeDeadlineSeconds so that finishedAt = startTime + duration and duration = activeDeadlineSeconds.
This would mean that we have to add additional information to the status (activeDeadlineSeconds of the TestDefinitiobn) as not all information of the TestDefinition is available in the status.
Release note:

Add additional phase "Timeout" for test that exceeded the `activeDeadlineSeconds`

Added the duration to the status of the testrun so that now the "real" duration is shown in the status and in elasticsearch

@OlegLoewen @dguendisch added an integration for the new phase and fixed dependency errors.
can you please rereview?

Problem was I needed another package for the pod message which resulted in a need to update some additional packages.
But I try to separate them next time:)


Support avro spec 1.8.2

Reason/Context
There's still a lot of projects that use avro 1.8.2 for historical reasons. And the schemas are practically the same with v1.9.0
Description

Mimetypes have to be adjusted to include 1.8.2
Consider that enum default value and uuid were introduced in avro 1.9.0

Solved in PR #6


How about add find method php-doc to models in _ide_helper_models

 @method static \Illuminate\Database\Query\Builder|\User find($value)

+1

@barryvdh find method does return static, but the IDE (PhpStorm) isn't reading that as a model object (User in this case)...
Any idea why?

I just want to know if anyone is working on this so we could make a pull request ?

I think this is a resurfaced problem in 5.3.  It doesn't recognize find as static or a normal method.  In both cases it cannot find a declaration to go to.

Btw, if you have a base model class that all of your models extend, you can add the following to it's doc block to solve this.
@method static \Illuminate\Database\Query\Builder|$this find($value)

@stygiansabyss Nope, doesn't work in PHPStorm. Both first, get and find need an explicit php doc on the model or above the line of code where they are used

Odd.  It's been working great for me.

@barryvdh issue can be closed. ATM it works fine in PhpStorm.
I also made a PR to Laravel which adds support for related methods like findOrFail etc. Unfortunately it's included only from Laravel 5.7

@barryvdh ack, this can be closed. static is recognized by PhpStorm correctly as well as static analyzers I believe.


Simplify OpenScapes profile options

Based on discussing how profiles were actually being used with the openscapes folks
(https://github.com/2i2c-org/infrastructure/issues/2882)

Generated via the scripts in
https://github.com/2i2c-org/infrastructure/pull/3030. See the description there for
various options available here, the principles behind it and the user focused nature of
this change. Reading that PR description is highly recommended before proceeding with
this PR.
This is a separate PR than #3030 as this can be deployed without having to wait
for that to go through.
Why was r5.xlarge and r5.4xlarge chosen?
r5.xlarge has 32GB of RAM, and I think it's the appropriate node size for most of the openscapes
workload.
Their current utilization on larger nodes is pretty small. Unused utilization in this graph is stuff we are paying for, but not using.

We also see mostly there are under 8-10 users, and even with just an r5.xlarge, they should be able to fit most users.

There was a couple uses of 100+ GB memory uses a few weeks ago. I think in general it's acceptable
for users who want a lot of resources to have to wait for a bit for nodes to spin up. This is why r5.4xlarge was added, with two RAM options.

The community is generally ok with us deploying this next week!

In addition, because memory_limit was used, it actually shows up clearly in the JupyterLab status bar!

This was the most requested feature during the conversation.
What to measure after this?
The primary thing to measure here quantitatively is to see if utilization % goes up, which brings cost down. Secondarily we should look at server startup time.
Qualitatively, we can check back in with the openscapes folks again to see how their instructors and users feel about this.
We can trade-off cloud cost and startup speed by switching the base from r5.xlarge to perhaps r5.2xlarge. But we can check back in in 2 weeks to see.

This is being tweaked with feedback from the openscapes community via slack https://openscapes.slack.com/archives/C02NC3Y62J1/p1692913592863859. Please let me know if you need an invite!

The following things were checked with the community:

The somewhat fractional memory allocations. These are all accurate, and will reflect the amount of RAM they'll have access to. This is what is shown in JupyterLab too. The trade-off is that these numbers look a little odd, but it's the same number throughout, and easily understandable for the user what it means. The community approves
These memory allocations are enforced at all times, regardless of what other users are doing. If someone wants more RAM, they are expected to ask for it. This is a feature of this specific resource allocation strategy. The community approves.
The word "Resource Allocation" is more familiar and understandable than the previous "node share". Some of this is due to familiarity with this from older HPC systems. "Node Share" is also an implementation detail, and one of the reasons we are doing this is that end users should be able to stop thinking about 'size of nodes'. The community approves this wording too
It's unclear if the large 100+ GB allocation will be used at all, and perhaps the person who used it in the past did so by mistake. We're going to revaluate this in a few weeks. We need to make sure we can track what profiles people are selecting.

I'll await more feedback from the community, but the hope is we get this deployed early next week.

@consideRatio thanks for the review!
Yes, we would need to adjust this based on system pods resource requests. It's handled as part of https://github.com/2i2c-org/infrastructure/pull/3030. I do agree that adding an additional 'buffer' there might help as well! Let's figure that out while reviewing that one.

Ok, I heard back from the community and they're ok with me deploying it today. Thanks for the prompt review making this possible, @consideRatio!

This is being tweaked with feedback from the openscapes community via slack https://openscapes.slack.com/archives/C02NC3Y62J1/p1692913592863859. Please let me know if you need an invite!

Yes please. @colliand would like an invitation to the OpenScapes Slack.


Fixed spelling error on on the first figure on the provider-hashicorp

…-vault docs page
Signed-off-by: Idowu Emehinola<EMAIL_ADDRESS>Fixes #635

/approve

Thank you for your contribution @hydeenoble!

/merge


Requests generated through Media Pipeline is not using scrapy-impersonate download handler

Lets suppose this website:
afdb.org/en/
The standard requests getting 200 Status but Files are getting 403 because Media pipeline is not using impersonate handler.

Hi @Ehsan-U
It's necessary to set the impersonate key in Request.meta so that ImpersonateDownloadHanlder can handle the request, otherwise the request will be handled by the native Scrapy downloader.
In the case of FilesPipeline/ImagesPipeline it's required to override the get_media_requests function:
class CustomFilesPipeline(FilesPipeline):
     def __init__(self, store_uri, download_func=None, settings=None):
         super().__init__(store_uri, download_func, settings)

     def get_media_requests(self, item, info):
         urls = ItemAdapter(item).get(self.files_urls_field, [])
         return [Request(u, callback=NO_CALLBACK, meta={"impersonate": "chrome110"}) for u in urls]

This way the Requests now include the impersonate key and will be handled by the ImpersonateDownloadHandler downloader.

Thanks for the response, @jxlil
I've observed that requests generated from the media pipeline are downloaded directly by the Engine. Recently, I noticed that the dupefilter doesn't seem to apply to media requests. I reached out to the Scrapy devs, and they confirmed that this behavior is expected. So, just to clarify, are you certain that ImpersonateDownloadHandler will be used to handle these media requests?

still this curl error appearing for some files, not all
2024-05-02 04:02:25 [crawler.pipelines] WARNING: File (unknown-error): Error downloading file from <GET https://cites.org/sites/default/files/i/CITES_WWD_Brochure2014.pdf> referred in <None>: Failed to perform, curl: (55) Send failure: Connection reset by peer. See https://curl.se/libcurl/c/libcurl-errors.html first for more details.

That error is related to curl_cffi. Open an issue in that project


inspect commands

clear the command line and display the value (maybe in the format [radius: 42.0])
ii for id (change the toggle to Ii)
in, ira[dius], ix, iy, iz, ih, is, il, ia, iv, iro[tation], ior[der], iop,

done with dd504b6


Fix warnings from cargo test

Currently cloning master and running cargo test yields two warnings, this PR cleans those up.

Thanks!


Map to WAN

How can I open the map to WAN? I think I've managed to open a port, but I have no idea of how I should configure the scanner to share the data to that port. Anything related to proxies in the config file?

You can start the scan.py normal.
The web.py must start with your port and your IP (ip of the computer or Server)
Example:
IP of PC/Server -> <IP_ADDRESS>
Port -> 5000
python3 web.py -H <IP_ADDRESS> -P 5000
If you only use the port 5000, you do not have to specify this at start. Standart is the 5000.
Now you still have the port 5000 in the router to your PC / server forward.
Now you can access the map with your public IP and the port

Thank you! but I seem to be missing something;
<IP_ADDRESS> - - [22/Feb/2017 09:14:49] "GET / HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Feb/2017 09:14:49] "GET /scan_coords HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Feb/2017 09:14:49] "GET /workers_data HTTP/1.1" 200 -
<IP_ADDRESS> - - [22/Feb/2017 09:14:49] "GET /data?last_id=0 HTTP/1.1" 200 -

Logs look good to me. Open the address in a browser http://<IP_ADDRESS>/

Thanks for the answer. The map loads, but doesn't show Pokémon. With the default IP I was getting a more extent log without the "GET /data?last_id=0 HTTP/1.1" 200

Pokemon are only displayed if they have a known TTH. Got known spawns?

Yes, web.py works fine with the default IP; I can see spawns, Pokémon and all. With my IP it shows spawns, workers and the scanning area, but no Pokémon.

Downloaded Google Chrome and it works fine. Don't know why, but it doesn't in Safary on IOS or Chrome in my desktop PC.


Add documentation for Security configuration

It would be great if you could expand upon this with an example config of a role with minimum permissions needed to access the metrics endpoint.

If you enable security in OpenSearch, remember to set up the credentials also in Prometheus configuration.

Update for anyone who has no idea about Prometheus config
  - job_name: opensearch
    scrape_interval: 30s
    metrics_path: "/_prometheus/metrics"
    static_configs:
    - targets:
      - ip:9200
    basic_auth:
      username: 'user'
      password: 'pass'

and setup user with info via Dashboard
metrics_access:
  reserved: true
  cluster_permissions:
    - 'cluster_monitor'
  index_permissions:
    - index_patterns:
        - '*'
      allowed_actions:
        - 'indices_monitor'


Missing reference AsterNet.ARI 1.3.1

[Arke.IVR Here] (https://github.com/quasarke/arke/blob/e4f5f095250644549c4060e9bb8a291debf38561/Arke.IVR/Arke.IVR.csproj#L8) is referencing AsterNet.Ari version 1.3.1 but there is no such version anywhere on nuget or on the AsterNet.Ari repository

Ah, yeah. Thanks. I need to get with the AsterNET.ARI team and get that version published. I'll see if I can get a patch out for that ASAP.

@shrage This should be solved with the latest PR that I just merged. Once AsterNET.ARI 1.3 releases, I'll swap from Arke.ARI back to AsterNET.ARI, but in the mean time, it's available that way.


Need to preserve markup inside of prism-highlighted code

I need to highlight certain words inside prism-highlighted block:

CREATE PROCEDURE [dbo].[SalesByCategory]
AS
IF OrdYear != '1996' AND OrdYear != '1997'
BEGIN
SELECT TOP(1) FROM Orders
END

Prism however replaces markup with  tag. These are the things that I have tried:

Replacing  to  with class attribute
Escaping opening or closing tags

This is NOT Html  behavior, since if I remove prism "language" class I do get my tags inside of .
I cannot use prism highlight plugins because highlighting should work based on starting and ending indexes in the input file and not line numbers.

See here and also there

Golmote, still cannot make it work. Can somebody point out to an example of how to prevent HTML markup from being stripped off by Prism?

Well, if you follow the links in the second link I gave you, you'll find a small quick'n'dirty plugin for Prism that does what you want. I may eventually take the time to write it officially, but for now it's just a gist. https://gist.github.com/Golmote/c532812e88c6234c48b3

So, the only way to achieve that it is by writing a custom Prism plugin (Golmote, thank you very much for sharing).  Perhaps documentation should be updated, because there doesn't seem to be a way to strip off markup by using just configuration or tag escaping.

There might be other ways to do it. See the source code of the Basic Usage section on Prism website for example.
But I thought a plugin would make it easier for everyone. I'll definitely ask for an update in the FAQ when I release it.
So, did you manage to make your code work with the plugin?

@Golmote I discovered that prism blows up with "TypeError: Cannot read property 'nodeName' of null" at _self.Prism.highlightElement @ prism.js:2 as soon as add nested code block (this isn't related to your plugin):

CREATE PROCEDURE [dbo].[SalesByCategory]
AS
IF OrdYear != '1996
AND
OrdYear != '1997'
BEGIN
SELECT TOP(1) FROM Orders
END

I don't see what's wrong with this markup..

@StanBPublic The markup is fine, but Prism highlights by retrieving all elements to be highlighted (e.g. your  and ) and then highlights them one by one. As all markup is stripped during highlighting, the second  no longer exists in the DOM thus has no parent will end up as null (prism-core.js#L57), which later explodes when Prism tries to highlight it.
The plugin may restore nested tags after highlighting, but the DOM nodes Prism has already fetched will be stale.
I don't have a solution, that's just what is happening :)
Though, why would you have "AND" highlighted as markup inside of SQL? Doesn't that plugin allow you to use the  tag you wanted to use initially?

@nauzilus thanks for explanation. Here's why I need to highlight "AND". Part of my code search utility is to highlight search term in the source files (SQL, C#, JavaScript). I know the start and end indexes of the search term hits and insert html tag before and after the hit. These tags are  but can be  or anything. That all works fine, but I want the source files syntax-highlighted with Prism and I also need the search hit highlighted at the same time...

Have a look at this: http://jsfiddle.net/nauzilus/aek8udvr/1/
Note the plugin Golmote linked to only works for simple tags (e.g. of the form <tag>...</tag>, class names or other attributes will stop it working).
I had to do some hackery with adding the language-sql classes with JavaScript, because of the way Prism auto-highlights it was running before the plugin ran, and I can't use the data-manual trick on jsfiddle. So ignore that bit. If it's all bundled together in one file it should be fine.
Hope that helps otherwise :)

@nauzilus that's fantastic, exactly what I need. Thank you very much!


[WIP] Fix configure-mip-local.sh script

Waiting for fixes on JVM options (in docker images) to avoid JVM being killed instead of properly limiting RAM allocation in:

woken
portal-backend
woken-validation


code coverage for go unit-tests using -cover flag

Updated go unit-test command with -cover flag (this provides coverage report to all the unit-tests performing with go test -cover -timeout=20m $PKGS command)
Description

Provides code coverage report to all the fabric unit-tests.
Motivation and Context

Fixes #
How Has This Been Tested?

Tested this change in inside vagrant and Travis environments.
Checklist:

[x] I have added a Signed-off-by
[x] Either no new documentation is required by this change, OR I added new documentation
[x] Either no new tests are required by this change, OR I added new tests
[x] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by<EMAIL_ADDRESS>

@rameshthoomu How much does this increase the run time for the test suite? I assume that this is not free? Why do we need to run coverage analysis every time we run the test suite?

To @bcbrock's point, if there is indeed a non-negligible cost, what we could do is create some kind of mode flag/target that is off by default, on for CI, and can try to be smart about not running the tests twice when coverage analysis is desired.

@rameshthoomu I would prefer to have a different script called by make -unit-test-cover option

@bcbrock: I didn't observe unit-test time increment in build process with this change. The just added a flag to our go test command and I assume it's free.

I just ran a test, with -cover on it is 16s wall time, and 2s system time longer than without. Hardly a concern for now. We can always reverse if we feel it is becoming a problem. I think that before that we would actually move to a more smoke-test followed by review followed by full test pipeline.

@rameshthoomu @ghaskins I stand corrected - I've done a few quick tests myself and I agree, it does seem to be free for the things I tried. So I withdraw my objection.
This almost can't be true in general however (that -cover is free). If I believe what I read on the Internet, they do coverage by rewriting the source code to add counting statements. I think the reason that we see it as free for Hyperledger fabric may be that the code does not contain computational "kernels" where the counting would be a major factor. If I'm ever feeling bored I may try coding up a matrix multiply in Go and testing it with and without -cover.

Seems there are no remaining objections, and this would be trivial to revert if it is later found to be problematic.  Therefore, LGTM.
Reviewed-by: Greg Haskins<EMAIL_ADDRESS>


Artifacts while using the model for other datasets

Hello,
We have been using your repository to train on different scenes. The results are great when we used the data provided by you.
When we were trying to use the data provided in the link below. The results were not so good. Can we know, is there a limitation in the method? We were looking for the novel view generation aspect of the paper. Any help would be appreciated.
We tried to keep the change in two consecutive frames to be minimal, using NeRF++ novel views.
Link to data used The data was generated from NeRF++ novel views using the Trucks scene.

Hello,
The method right now is not designed to work with unstructured captures. But, for what you tried so far, what was the input coordinates to network?

Hello,
The method right now is not designed to work with unstructured captures. But, for what you tried so far, what was the input coordinates to network?

python train.py --dataset our_data_path  --type view  --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
We used the above setting.
We took the 5 frames keeping the y level constant and changing the x slightly. And we have taken 5 different values of y. So totally 5x5 25 frames.

python train.py --dataset our_data_path  --type view  --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
We used the above setting.
We took the 5 frames keeping the y level constant and changing the x slightly. And we have taken 5 different values of y. So totally 5x5 25 frames.

I started to train it, meanwhile can you share the result you got (flow or reconstruction image or video) maybe I can understand your problem?

I started to train it, meanwhile can you share the result you got (flow or reconstruction image or video) maybe I can understand your problem?

Results

Results

Hello,
I did the training with the command line you sent:
python train.py --dataset "dataset/Truck Nerf Gen" --type view --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
I did the test with the command line:
python test.py --dataset "dataset/Truck Nerf Gen" --type view --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
the result look better, here is what I got:
Truck Nerf Gen.zip

Hello,
I did the training with the command line you sent:
python train.py --dataset "dataset/Truck Nerf Gen" --type view --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
I did the test with the command line:
python test.py --dataset "dataset/Truck Nerf Gen" --type view --dim 5 5 --factor 2 --num_n 2 --nfg 8 --sigma 0.5 --br 1 --savepath results/
the result look better, here is what I got:
Truck Nerf Gen.zip

Thanks, I will re-train on a different machine, strange that we could not get the same results despite having a similar setup. Your results look consistent.

Thanks, I will re-train on a different machine, strange that we could not get the same results despite having a similar setup. Your results look consistent.

Hi,
We made the following change to the code and the results match with what you have provided us for the "Truck Nerf Gen" data.
The file load_imgs.py has path_images which were not being sorted in the Linux environment.
I had changed the code in the load_images.py file to the following
  print("Dataset path: %s"%(args.dataset))
  path_images = os.listdir(args.dataset)
  print('Before sort', path_images)
  path_images.sort()
  print('After sort', path_images)

The output of those lines are as follows:
Before sort ['000186.png', '000182.png', '000194.png', '000197.png', '000180.png', '000181.png', '000196.png', '000179.png', '000177.png', '000174.png', '000190.png', '000185.png', '000178.png', '000183.png', '000176.png', '000193.png', '000188.png', '000191.png', '000189.png', '000175.png', '000187.png', '000184.png', '000173.png', '000192.png', '000195.png']
After sort ['000173.png', '000174.png', '000175.png', '000176.png', '000177.png', '000178.png', '000179.png', '000180.png', '000181.png', '000182.png', '000183.png', '000184.png', '000185.png', '000186.png', '000187.png', '000188.png', '000189.png', '000190.png', '000191.png', '000192.png', '000193.png', '000194.png', '000195.png', '000196.png', '000197.png']

So the sort function seems to fix the issue.
Now the results are matching up with yours (except used fps as 23 while running the test).
Result
You can close the thread. Thanks for the help.


Fixed #19580 -- Unify reverse foreign key and m2m unsaved model querying

did you uncheck patch needs improvement on the ticket?

did you uncheck patch needs improvement on the ticket?

Yes! I did it.

Hey @carltongibson requesting some help
First of all - Extremely sorry if this is not the right place to ask for help - please do guide me what medium should I use next time for queries of this nature.
Setup:
I was looking at Easy Pickings and noticed everything was assigned and none of the easy pickings were easy enough for me to understand.
For this particular PR #11807, I noticed you left a comment here: https://github.com/django/django/pull/8570#pullrequestreview-113506649
Request:
Would you be able to guide me or point me to someone who can guide me with what knowledge is required to be able to understand how to fix this issue and address your comment on #8570 so I can build on that knowledge and probably contribute to fixing this issue?
Associated Trac: https://code.djangoproject.com/ticket/19580

Hey @carltongibson requesting some help
First of all - Extremely sorry if this is not the right place to ask for help - please do guide me what medium should I use next time for queries of this nature.
Setup:
I was looking at Easy Pickings and noticed everything was assigned and none of the easy pickings were easy enough for me to understand.
For this particular PR #11807, I noticed you left a comment here: #8570 (review)
Request:
Would you be able to guide me or point me to someone who can guide me with what knowledge is required to be able to understand how to fix this issue and address your comment on #8570 so I can build on that knowledge and probably contribute to fixing this issue?
Associated Trac: https://code.djangoproject.com/ticket/19580

@ahsanshafiq742 Thanks for this patch 👍 Have you checked comments from previous versions (#8570, #7098)? Some of them are not addressed, e.g. #8570 (review) or #8570 (review), also release notes are missing.

I'll take care of the comments.

Hi @ahsanshafiq742, are you still working on this patch?

Yup
On Sat, 14 Mar 2020, 12:53 am Shrikrishna Singh<EMAIL_ADDRESS>wrote:

Hi @ahsanshafiq742 https://github.com/ahsanshafiq742, are you still
working on this patch?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/11807#issuecomment-598887049, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AICTCAOPKI5ZTN356SVKU7LRHKFK3ANCNFSM4IZHTPMA
.

Hi, I've updated the changelog according to the comments on the previous Prs, here it is:
Unified behaviour of ForeignKey and ManyToManyField
The behavior of nullable related QuerySet has changed.
Nullable related QuerySet means a QuerySet for reverse relation
to a field which can be set to null.
Before, querying a nullable many-to-many relation was an error. For foreign
key relations there was no error. To make the nullable relations consistent
the following changes were made:

Nullable many-to-many relations now return an empty queryset when the
relation field is null (no change for foreign keys).
For example::
class Car(models.Model):
    drivers = models.ManyToManyField('Driver')

class Driver(models.Model):
    name = models.CharField()

Car.objects.create().drivers.all()

returns EmptyQuerySet
Previously it would raise ValueError

For a foreign key relation, trying to add/remove/clear objects when the
relation field is null is now an error (no change for m2m).
For example, given these related models::
class Poll(models.Model):
    question = models.CharField(max_length=200, unique=True, null=True)

class Choice(models.Model):
    poll = models.ForeignKey(Poll, models.SET_NULL, to_field='question', null=True)
    choice = models.CharField(max_length=200)

empty_poll = Poll.objects.create(question=None)
choice = Choice.objects.create(choice='B')

All the below statements now raise a ValueError ::
# Previously created a Choice object with ``poll_id=None``
empty_poll.choice_set.create()
empty_poll.choice_set.get_or_create()

# Previously the ``choice.poll_id`` remained to None
empty_poll.choice_set.add(choice)

# Previously did nothing
empty_poll.choice_set.remove(choice)
empty_poll.choice_set.clear()

For a foreign key relation, trying to use a relation of any kind when the
object isn't saved is now an error (no change for m2m).
For example invoking::
Poll(question="How?").choice_set

now raises a ValueError
Previously it would return a RelatedManager

Superseded by #13784.


⚠ Processors has degraded performance

In e4c772f, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 71 ms

Resolved: Processors performance has improved in e18d235.


Feature Request: Make parameter "--blank" configurable

Hi,
I'm playing audio only and unfortunately, the current screen which shows important information is hidden due to a black fullscreen. The reason for this is the paramter --blank in line 38:
let args = [source, '-o', output, '--blank', osd ? '' : '--no-osd'];
Would it be possible to make this paramter configurable from outside?

I would totally want this as well!

Actually just went ahead and fixed it myself in a fork. Will make a pull request now.


VS Code keeps recommending extensions I already have

Issue Type: Bug
VS Code keeps recommending I install the C# extension, even though I already have it installed. I tried removing and reinstalling the extension and that did not help. I can reproduce by opening a folder and then opening a .csproj file to edit.

VS Code version: Code 1.42.1 (c47d83b293181d9be64f27ff093689e8e7aed054, 2020-02-11T14:45:59.656Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.81GB (20.72GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (15)

Extension
Author (truncated)
Version

cform
aws
0.0.24

vscode-eslint
dba
2.1.1

githistory
don
0.6.8

xml
Dot
2.5.1

EditorConfig
Edi
0.14.4

vscode-cfn-lint
kdd
0.14.0

vscode-docker
ms-
1.1.0

csharp
ms-
1.21.18

mssql
ms-
1.9.0

azure-account
ms-
0.8.11

powershell
ms-
2020.4.0

team
ms-
1.161.0

vscode-yaml
red
0.9.1

code-spell-checker
str
1.8.0

vscode-icons
vsc
10.2.0

I cannot repro it.
It seems you are on old version of VS Code (1.42.1). Can you please update to latest and try it out?


Break out helpers into individual exports

I was trying to use this library earlier with Ember CLI, but ran into some issues. One of the biggest limitations of this library right now is how all of the helper functions are stuck within registerWith.
For Ember, I need to be able to modify out registerWith to call Ember.Handlebars.registerBoundHelper rather than just registerHelper, or otherwise the data-binding won't cause a re-render when a model changes.
The best way for me to implement this would probably be for HandlebarsIntl to break out the helpers away from the registerWith function, and have them each be exported.  Then, it'd be a lot easier for me to import them and create a separate dist package just for Ember in a build step.

For Ember there's now Ember Intl:
http://formatjs.io/ember/
We could could move the helpers out as exports if we wanted to only support Handlebars 3.0, which removes the need the instanceof check on SafeString instances.

With webpack + handlebars-loader it won't work out either. The helper should be split into seperate modules.export to be included only if required.

The helper should be split into seperate modules.export to be included only if required.

This is only possible now with Handlebars v3 and its improved SafeString support which now only needs to implement a toHTML() method.
But that said, it's not likely to be a high priority to refactor this package in that way. I also have no idea what Webpack + Handlebars-Loader even tries to do… but seems crazy that you wouldn't be able to call HandlebarsIntl.register() during some lifecycle event of the handlebars-loader.

Thanks for the feedback - i'll go with i18next and SafeString.


Tag ARDESPOT v0.2.1

@rejuvyesh can you tag this version: https://github.com/JuliaPOMDP/ARDESPOT.jl/releases/tag/v0.2.1

Fixed by #6


Blank page show

When uploaded on the server my project blank page shows.
public folder all fille move to root file then
App.php file chance public $baseURL = 'http://truequery.xyz/vsn/';
env file rename to .env and #CI_ENVIRONMENT = production chance to CI_ENVIRONMENT = development
db
database.default.hostname = localhost
database.default.database = truequer_news
database.default.username = truequer_new
database.default.password = XrFrhz3.]i]n
database.default.DBDriver = MySQLi
then  index.php $pathsPath = FCPATH . '../app/Config/Paths.php';   chance to $pathsPath = FCPATH . 'app/Config/Paths.php';
blank page show how can solve it?

What CI version are you using? Did you check your php.ini if the zlib.output_compression is turned on? Does the server have the php intl extension installed?

Version 4.0.2 ... I upload my project shared hosting. how can check my server php intl extension

Is it feasible for you to upgrade to at least 4.0.4? In there there is a check against blank pages.
If you have access to the terminal in the server, you can try php --ri intl. Or if this is shared hosting, you can ask the provider where is your own php.ini or if intl is installed.


issues with tree view and misreporting

the latest developer branch exe has some issues:
it does not always update the msaa etc info when walking the acc tree pane.
it does not always report correctly the element being watched via the cursor.

Confirmed.

Which browser are you using?
What should I do to reproduce them?

Hi JUn,
it occurs in Firefox and Chrome, I will provide a test page tomorrow
--
Regards
SteveF
Current Standards Work @W3C
http://www.paciellogroup.com/blog/2015/03/current-standards-work-at-w3c/
On 1 December 2015 at 22:24, Jun<EMAIL_ADDRESS>wrote:

Which browser are you using?
What should I do to reproduce them?
—
Reply to this email directly or view it on GitHub
https://github.com/ThePacielloGroup/aviewer/issues/13#issuecomment-161115642
.

It happens in Chrome 46 and Firefox 42 on my Windows 10 64bit machine.

This appears fixed to me (modulo https://github.com/ThePacielloGroup/aviewer/issues/26)


Missing Background Color

Hello everyone,
I have a problem regarding my Background Layer:
Some of my generated images have a white background color even though I added colorful background images to my Background Layer, I set each image's weight to #100 and I set the variable const background to false.
The json file associated with the image shows an existing background layer, yet the image itself has a white background.
  "image": "image.png",
  "external_url": "",
  "edition": 3,
  "attributes": [
    {
      "trait_type": "Background",
      "value": "Burly Wood"
    },

Does anyone know a solution to my problem?
I appreciate any help!

Did you ever find a solution to this? I'm having the same problem.

Did you ever find a solution to this? I'm having the same problem.

The problem is you have saved one or more of the elements above the background layers with a white background, have another look at the files and resave as needed.

Thanks just checked it and one of the layers had a background that was covering the bottom layer.


Grant types and oauth_public_keys missing

Hello!
In file Module.php on line 39-42:
 $server->addGrantType(new \OAuth2\GrantType\UserCredentials($storages['user_credentials']));
 $server->addGrantType(new \OAuth2\GrantType\RefreshToken($storages['refresh_token'], [
        'always_issue_new_refresh_token' => true
]));

Only these 2 grant types are added to server. Without overriding this class, I cannot add other grant types that are already supported by bshaffer/oauth2-server-php.

Why is it neccessary to wire these 2 grant types into this file, as the default ones would be sufficient of bshaffer/oauth2-server-php, and these could be restricted in table oauth_clients.
In your migration there is a sample row in oauth_clients table, having grant type client_credentials set, but because of the aboves, it does not work...

curl -v -X POST -u testclient:testpass "http://api.test.domain/oauth2/token" -d 'grant_type=client_credentials'

{"name":"Bad Request","message":"Grant type \"client_credentials\" not supported","code":0,"status":400,"type":"yii\\web\\HttpException"}

Why is the table oauth_public_keys missing from your migration?

CREATE TABLE oauth_public_keys (
client_id VARCHAR(80),
public_key VARCHAR(2000),
private_key VARCHAR(2000),
encryption_algorithm VARCHAR(100) DEFAULT "RS256"
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

Thank you!

H, done :)
See commit https://github.com/Filsh/yii2-oauth2-server/commit/7a1176b8db54be89526dbefd8f075c4062e97c63
For example config
'oauth2' => [
    'class' => 'filsh\yii2\oauth2server\Module',
    'options' => [
        'token_param_name' => 'accessToken',
        'access_lifetime' => 3600
    ],
    'storageMap' => [
        'user_credentials' => 'common\models\User'
    ],
    'grantTypes' => [
        'client_credentials' => [
            'class' => '\OAuth2\GrantType\ClientCredentials',
            'allow_public_clients' => false
        ],
        'user_credentials' => [
            'class' => '\OAuth2\GrantType\UserCredentials'
        ],
        'refresh_token' => [
            'class' => '\OAuth2\GrantType\RefreshToken',
            'always_issue_new_refresh_token' => true
        ]
    ],
]

Hello, thank you!
What about the oauth_public_keys table?

Also added in commit https://github.com/Filsh/yii2-oauth2-server/commit/7a1176b8db54be89526dbefd8f075c4062e97c63

#43


Taking mat-label from mat-form-field causes accessibility issue

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I wanted to have control label above mat-form-field rather than a floating label inside mat-form-field.
According to https://github.com/angular/components/issues/17913 I wrote this and works fine
<mat-label for="name">Name</mat-label>
<mat-form-field [appearance]="outline">
   <input id="name" matInput type="text"/>
</mat-form-field>

but arises when I mat-select inside mat-form-field, I removed mat-label from mat-form-field but component didn't notice and adds an empty element for label and also adds aria-labelledby attribute to mat-select, after running lighthouse you can see a warning related to empty element used for mat-select label.
Subject
<mat-form-field [appearance]="outline">
   <mat-select id="subject">
      <mat-option>1</mat-option>
      <mat-option>2</mat-option>
   </mat-select>
</mat-form-field>

Reproduction
Steps to reproduce:

Add a sample material select component https://material.angular.io/components/select/overview
Remove mat-label from mat-form-field
Add an ID to mat-select
Add a label outside of mat-form-field and add for="ID" attribute
Run lighthouse to generate an Accessibility report

Expected Behavior
When mat-label is not exists inside mat-form-field, aria-labelledby should not be added to mat-select.
Actual Behavior
I wanted to have control label above mat-form-field rather than a floating label inside mat-form-field.
According to https://github.com/angular/components/issues/17913 I wrote this and works fine
<mat-label for="name">Name</mat-label>
<mat-form-field [appearance]="outline">
   <input id="name" matInput type="text"/>
</mat-form-field>

but problem arises when I mat-select inside mat-form-field, I removed mat-label from mat-form-field but component didn't notice and adds an empty element for label and also adds aria-labelledby attribute to mat-select, after running lighthouse you can see a warning related to empty element used for mat-select label.
Subject
<mat-form-field [appearance]="outline">
   <mat-select id="subject">
      <mat-option>1</mat-option>
      <mat-option>2</mat-option>
   </mat-select>
</mat-form-field>

Environment

Angular: 13.0.0
CDK/Material: 13.0.0
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

Closing, because we don't support this use case.


Add ability to create chromedriver.log

I have seen that chromedriver can output a logfile (https://sites.google.com/a/chromium.org/chromedriver/logging)
This page shows how to set this up when executing the exe directly: chromedriver.exe --verbose --log-path=chromedriver.log
There is no apparent way to set these arguments from protractor however.
See this StackOverflow question for more info:
http://stackoverflow.com/questions/31662828/how-to-access-chromedriver-logs-for-protractor-test/31662935

:+1:

I just figured it out, and responded to the Stack Overflow post. Feel free to test it and update the protractor documentation. The only issue I have is that the chromedriver.log is overwritten with every browser that opens (instead of appending the file or creating a new file for each browser session), so it may not be helpful when running a suite of tests or with sharding turned on.
If you're using the seleniumServerJar, in protractor.conf.js set the logfile path to wherever you want it to write the file:
seleniumArgs: [
'-Dwebdriver.chrome.logfile=/home/myUsername/tmp/chromedriver.log',
]
If you're using webdriver-manager start to run a local selenium server, you'll need to edit the webdriver-manager file:
// insert this line
args.push('-Dwebdriver.chrome.logfile=/home/myUsername/tmp/chromedriver.log');
// this line already exists in webdriver-manager, add the push to args before this line
var seleniumProcess = spawnCommand('java', args);

webdriver-manager has chromedriver.log this option
protractor now uses webdriver-manager as a node module

Just a minor clarification (it may help someone who is searching for this topic):
webdriver manager has the chrome_logs option (you can find it in its source code (in opts.ts or opts.js in the compiled code)), so you can use it something like:
webdriver-manager start --chrome_logs /path/to/logfile.txt


Header alignment issue on iPhone 13 Pro

Hi, there an alignment issue on iPhone 13 Pro

Thanks for letting me know. Is that only on the sidebars? The editor itself works fine?

I didn't notice it before, but the editor heard is completely invisible!


Check marks and X marks are displayed under entity models

Green check marks (completed tasks) and red X marks (locked tasks) are displayed under entity models on both the regular Bingo card and the big one, regardless of the type of the task.
Screenshots showing this issue:

As you can see in these screenshots, only entity rendering seems to be problematic (however, for some reason some entity models are not affected on the regular Bingo card), all other types work correctly.
This issue occurs on the following configurations (I didn't test other configs):

Minecraft version: 1.16.4
Bongo versions: 1.0.4, 1.0.10 from Maven
LibX version: 1.0.7
Forge version: 35.1.4

I translte to the front very much after rendering the slot content. I can not go much further because then the check marks would just disappear. Weirdly the entities are rendered very much in the front. But I don't think this is much of a problem because you can still see that it is finished.

I translte to the front very much after rendering the slot content. I can not go much further because then the check marks would just disappear. Weirdly the entities are rendered very much in the front. But I don't think this is much of a problem because you can still see that it is finished.

Fair enough. Thank you for the explanation.

Fair enough. Thank you for the explanation.


Add Benches

Add some individual benches for timing and memory usage. This should include:

Deserialising simple search response
Deserialising search response and iterating over aggregations
Deserialising get response
Deserialising ping response
Deserialising bulk response from reader
Deserialising bulk response from slice
Deserialising bulk errors response from slice

For bulk responses, we should also use a few allocated field types:

String
DefaultAtom from string_cache
InlineString from inlinable_string
custom enum


22 tests are failing against public pub in pkgbuild

Some of this may be the same as  issue #21069, but filing this so that I am sure we get both green
/pkg/template_binding
/pkg/observe
/pkg/polymer
/pkg/polymer_expressions
/samples/google_maps
/runtime/bin/vmservice/observatory
/samples/polymer_intl
/samples/gauge
/samples/jsonp
/samples/dartiverse_search
/samples/pop_pop_win
/samples/clock
/samples/swipe
/samples/sunflower
/samples/solar
/samples/tracker
/samples/multi_touch
/samples/third_party/todomvc_performance
/samples/third_party/todomvc
/samples/searchable_list
/samples/third_party/angular_todo
/samples/third_party/dromaeo
I am tentatively assigning this to pub, please redirect if wrong
Example output:
http://build.chromium.org/p/client.dart/builders/pub-linux-be/builds/8623/steps/pkgbuild_public_pkgs tests failures/logs/stdio

This issue has been moved to dart-lang/pub#1147.


New methods for SyncOntology and SyncReasoner

Added new methods for SyncReasoner:

is_entailed → checks if an axiom is entailed by the set of reasoner axioms.
is_satisfiable → cheks if a class expression is satisfiable with respect to the reasoner axioms.
unsatisfiable_classes → retrieve classes that are unsatisfiable

Added new methods for SyncOntology:

get_signature → retrieve all signature entities.
get_abox_axioms → retrieve all ABox axioms.
get_tbox_axioms → retrieve all TBox axioms.

Thank you for the PR. Can you please write tests for the each introduced methods ?

very cool features!


Recently updated artefacts out of date

The page 'Recently updated artefacts' shows all artefacts that have been updated recently.
However, since the update process is not triggered often enough, the data is increasingly out of date.
Is it possible to start this process more often, or to move it to a separate repo like dotabap.org and thus keep the 'trends.json' and 'allItems.json' up to date?

Hi,
sounds like a fair comment but I'm not sure what's the best way to deal with this situation. I guess there are multiple options such as:

Running on a weekly basis (vs once per month), but this is something our team internally decided against to have a more robust trend. That doesn't mean this decision is set in stone but I would like to hear what the community thinks about this
Changing the criteria of "recently updated" to one week prior to the last run. This would at least mean, that artifacts in the list cannot be older than 5 weeks
Define a new GH action that only pulls the last update date of the items listed here that removes old items from the ranking. Which could mean that this particular ranking changes every day/week but the other rankings only change in the following month.  This would mean that some items might be removed from the ranking over the course of the month without adding new items.

Does any of these ideas cover what you mean with dotabap.org? If not, can you please explain in more detail what you had in mind?

Hi,
I don't mean the calculation of the trend data or the ranking yet.
I am actually only interested in the display of the respective information (e.g. recently updated or stars)
and the resulting order in "recently updated".
If this information is only retrieved once a month, the data that is displayed is up to 4 weeks out of date.
As an example: I use dotabap.org to see if there are new repos in the list and which repos have been updated.
Since the list is always sorted by "recently updated", I have this information at a glance.
The list is also updated once a day.
I would then wish for the following:
Displayed information (last update, stars, forks, pulls) is retrieved separately and more often (ideally once a day).
Continue to calculate trend data as desired (monthly, weekly).
More sorting:
e.g. for "recently updated" a filter according to "recently updated".
for "new", sort by "creation date".
possibly also sorting by stars, forks, pulls
I also only recently realised that all three pages (trends,new,updated) are sorted by ranking, which is not so clear.
"New Artifacts" is also a bit misleading, I think "Newly added Artifacts" makes it a bit clearer.

All good points!
I could see a daily job that updates the items. One problem here might be the BigQuery API Key that doesn't allow such a frequent access to the table. All other providers should remain within the free quota even when we update the stats daily.
I don't think I'll have time for this in the next few weeks as Devtoberfest and TechEd keep me quite busy. But I'll keep the issue open in case someone wants to tackle this.

Hi @IObert
the new GitHub action seems to have worked.
But unfortunately the rebuild was not triggered.
Should the rebuild be done in the gh-update action or should the new data also be written directly to the docs folder?

I think it makes sense to write directly into docs at the end of the gh-update action

The commit today looks good.

All good points! I could see a daily job that updates the items. One problem here might be the BigQuery API Key that doesn't allow such a frequent access to the table. All other providers should remain within the free quota even when we update the stats daily.

Does it make sense to expand this action to the other providers?
Should PyPi be excluded from this?

I think it makes sense to extend this for npm. I'd exclude PiPy for now was it uses a paid API key.

And yes, it looks great! Many thanks for your great work 🤩!

No worries :)
I think with #18 we can close this issue as it is resolved, right?

Yes :)


Attach BuildTimeTrend to the builders

http://buildtimetrend.github.io/service/
We have this hooked up to our TravisCI builds right now, but that information is less clearly actionable/meaningful than the build time data for our actual builders. The documentation suggests there are a couple ways to go about this, either using Keen.io or storing XML data locally. Then we also need to run an instance of the client that fetches this data appropriately.

This can be done through Salt changes, requiring admin intervention only to roll those changes out. The buildtimetrend client should run as its own user so we can decide whether it's on its own or sharing a host.

AIUI, they've discontinued the production instance at https://buildtimetrend.herokuapp.com/, so we'll likely also need to run our own server in addition to clients.

We are currently using the Buildbot API to fetch and upload build time data, thanks to work by Alan in https://github.com/servo/servo/pull/19344, https://github.com/servo/servo/pull/19450, and https://github.com/servo/servo/pull/19524. Data is available in S3 and I believe can be viewed in Google Data Studio.


League Skill Calculator

I believe that we could benefit from having the ability to toggle options on the skill xp calculator to account for our boosted xp rates during league. Obviously the math is not difficult for us; we just take whatever the rate is in the calculator and divide/multiply by whatever our current boost is, but it would be a nice quality of life implementation for folks during the leagues.
Thank You!

Duplicate of #3027
fwiw, osleague.tools has a calculator for league rates if you want to use one right now


ERROR: Can't open ABC output file

Steps to reproduce the issue
test file
module test (input [31:0] i, output reg y);

always @* begin
  case (i)
    3 : y = 1'b1;
    4 : y = 1'b0;
    default: y = 1'bx;
  endcase
end

endmodule

synth_gates.ys
#read_verilog test.v

hierarchy -top test
proc; flatten

synth

dfflibmap -prepare -liberty ./synth_gates.lib
abc -dff -liberty ./synth_gates.lib
dfflibmap -liberty ./synth_gates.lib

stat
show
write_blif test.blif

command
C:\> yosys  test.v  synth_gates.ys

synth_gates.lib on
‘https://github.com/cliffordwolf/picorv32/blob/master/scripts/yosys/synth_gates.lib’
Expected behavior
can I specify '<abc-temp-dir>' position ?
Actual behavior
ERROR: Can't open ABC output file `C:\Users***\AppData\Local\Temp\yosys-abc-ALJrvg/output.blif'

Hi @balanx. I can reproduce your error (with a few more lines of context):
<<SNIP>>
7.2.1. Executing ABC.
Running ABC command: <yosys-exe-dir>/yosys-abc -s -f <abc-temp-dir>/abc.script 2>&1
ABC: ABC command line: "source <abc-temp-dir>/abc.script".
ABC:
ABC: + read_blif <abc-temp-dir>/input.blif
ABC: + read_lib -w <yosys-exe-dir>/synth_gates.lib
ABC: Parsing finished successfully.  Parsing time =     0.00 sec
ABC: Warning: Templates are not defined.
ABC: Libery parser cannot read "time_unit".  Assuming   time_unit : "1ns".
ABC: Libery parser cannot read "capacitive_load_unit". Assuming   capacitive_load_unit(1, pf).
ABC: Scl_LibertyReadGenlib() skipped sequential cell "DFF".
ABC: Library "gates" from "<yosys-exe-dir>/synth_gates.lib" has 3 cells (1 skipped: 1 seq; 0 tri-state; 0 no func; 0 dont_use).  Time =     0.00 sec
ABC: Memory =    0.00 MB. Time =     0.00 sec
ABC: Warnings: genlib library reader cannot detect the buffer gate.
ABC: Some parts of the supergate-based technology mapper may not work correctly.
ABC: + strash
ABC: + ifraig
ABC: + scorr
ABC: Warning: The network is combinational (run "fraig" or "fraig_sweep").
ABC: + dc2
ABC: + dretime
ABC: + strash
ABC: + &get -n
ABC: + &dch -f
ABC: + &nf
ABC: ** cmd error: aborting 'source <abc-temp-dir>/abc.script'
ABC: Error: Cannot find buffer gate in the library.
ABC: Error: Abc_CommandAbc9Nf(): Mapping into LUTs has failed.
ERROR: Can't open ABC output file `/tmp/yosys-abc-XTTQQZ/output.blif'.

Note the warning from ABC:
ABC: Some parts of the supergate-based technology mapper may not work correctly.```

It appears that the liberty file you're using is too simplistic (I'm surprised it's enough for @cliffordwolf to build picorv32?!) -- after replacing the lib with something like this: https://vlsiarch.ecen.okstate.edu/flows/MOSIS_SCMOS/osu_soc_v2.5/cadence/lib/tsmc018/signalstorm/osu018_stdcells.lib I get a clean run.

Does this help?

I'm surprised it's enough for @cliffordwolf to build picorv32?!

It's not. It used to be but that was with an older version of ABC. I've now fixed that cell library accordingly in https://github.com/cliffordwolf/picorv32/commit/243a09fd8dc79e4435df145bc5f5dbaa95f210a5.

thanks your reply !
It fixed this ERROR after I updated synth_gates.lib.
But, the synthesized result is NOT the simplest !! @cliffordwolf
I supplied an issue at 'https://github.com/YosysHQ/yosys/issues/765'.


Added a unit test to demonstrate how to use "soft timeouts"

Description
This PR adds a unit test that shows how to use "soft timeouts" (returning a fallback value if no response from Cosmos is retrieved within the soft timeout) - but still emit diagnostics for call exceeding the soft timeout latency. The sample can be modified to also fail instead of returning a fallback of course (but keep logging diagnostics)
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Remove title attr from review-comment

aria-label covers screen readers, title just shows the tooltip.
Fixes Microsoft/vscode-pull-request-github#61

Thanks, looks good!


isaac_ros_visual_slam as simulation return [CUDA] failure

Hi,
I am working with isaac_ros_visual_slam package and Gazebo.

My error:
[component_container-1] [INFO] [1662379434.366503061] [visual_slam_node]: Use use_gpu: true
[component_container-1] [ERROR]<PHONE_NUMBER>32320 [CUDA] failure: 13, file: /home/akorovko/Code/elbrus/src/modules/cuda_modules/src/lk_tracker.cpp, line: 9
Also, i use pytorch environment with cuda.
Thanks for reply.

I work on Ros Galactic. Maybe this is reason of my error. i dont know.

Could you describe the platform you're running on? Are you running within the Isaac ROS Dev base Docker container or natively on host? It looks like an x86 machine running ROS2 Humble (not Galactic which is incompatible) but with which NVIDIA GPU available for CUDA?

My knowledge is not good at docker. So, sorry. I work on 64 bit on computer.(laptop). CUDA Version: 11.7. I will look tomorrow. Thanks for Humble case.
nvidia-smi:

I try to run rosbag example which is "small_pool_test". But, it give same error. My GPU is NVIDIA GeForce MX110. Is it enough for isaac_ros_visual_slam package? Also, i try path as "nvidia-container-runtime" and "/usr/build/nvidia-container-runtime", but this is not work. I install nvidia-container-runtime package is seen belove: I use Ubuntu focal(20.04) and jammy(22.04). But bionic version is installed. I dont know, what is the reason of my error. I doubt from my nvidia-container-runtime. Also i try Ros2 Galactic and Humble.

Also, i install nvidia-container-runtime to native computer. I dont install nvidia-container-runtime to this(isaac_ros_visual_slam) Docker container. Does the nvidia-container-runtime installing in the Docker container is necessary?

It appears you're running this on a Maxwell architecture GPU. We think this may not be recent enough for Isaac ROS Visual SLAM to work with. Can you try running on a Volta or newer GPU? We'll work on a more graceful failure message if this is indeed the problem you're running into.

I am not working with Gazebo, i am running bag file now:
source /workspaces/isaac_ros-dev/install/setup.bash && \    ros2 bag play src/isaac_ros_visual_slam/isaac_ros_visual_slam/test/test_cases/rosbags/small_pol_test/
Also, My Gazebo doesn't use GPU. Probably i dont use GPU correctly.
watch nvidia-smi: (on natural computer)

watch nvidia-smi: (in docker container)

top: (in docker container)

component container, rviz2 and all other program doesn use GPU. Only X-server and office program use GPU.

Also, i have 2GB memory. But only 3MB is used. Gazebo doesn't use GPU. Accually, i have a nvidia-driver
I try to solve GPU using with program (ex: Gazeob).

I think, my installation is correct. Because i test my GPU(Nvidia Container) at my native computer and any docker container.
Run GPU in native computer: (BlackScholes sample)

Run GPU in docker container: (ros_caffe sample)

Also, i run optional command after installing isaac_ros_visual_slam package:
colcon test --executor sequential
This command get below error:
Starting >>> isaac_ros_visual_slam
--- stderr: isaac_ros_visual_slam
Errors while running CTest
Output from these tests are in: /workspaces/isaac_ros-dev/build/isaac_ros_visual_slam/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
Finished <<< isaac_ros_visual_slam [18.2s]	[ with test failures ]

So, i read log file. i see 3 errror message in : colcon test --executor sequential command
[component_container-1] [ERROR]<PHONE_NUMBER>09824 [CUDA] failure: 13, file: /home/akorovko/Code/elbrus/src/modules/cuda_modules/src/lk_tracker.cpp, line: 9
[ERROR] [component_container-1]: process has died [pid 44573, exit code -6, cmd '/opt/ros/humble/install/lib/rclcpp_components/component_container --ros-args -r __node:=visual_slam_container -r __ns:=/'].
[ERROR]<PHONE_NUMBER>09824 [CUDA] failure: 13, file: /home/akorovko/Code/elbrus/src/modules/cuda_modules/src/lk_tracker.cpp, line: 9

Also, i didn't can't try with Volta or new GPU. I have only this computer. Probably, my computer doesn't support this. I don't know. I have 2GB GPU memory.

I try to Ros BAG sample in isaac_ros_nvblox package (nvblox_pol sample).
It work correct for me.
Result:
Run GPU isaac_ros_nvblox:

Rviz Result:

Probably, [CUDA] failure: 13, file: /home/akorovko/Code/elbrus/src/modules/cuda_modules/src/lk_tracker.cpp, line: 9 error interested in isaac_ros_visual_slam package. isaac_ros_nvblox work my computer with GPU.

Isaac ROS Visual SLAM is only compatible with Volta and newer GPU architectures.

Can someone explain why a Volta (or newer) GPU is required?  I've installed and have successfully run this on a Tesla P4 that is based on Pascal.  I've yet to incorporate it into my robot, but what specifically does the Volta architecture provide that's missing on the Pascal that's in use by visual slam?


add luajit platform metrics

After https://github.com/tarantool/tarantool/issues/5187 tarantool
could report luajit platform metrics.
This patch exports them as default metrics.
For detailed description of each metric see
https://github.com/tarantool/doc/issues/1597.
Closes #127

CI will red util https://github.com/tarantool/metrics/issues/130 / https://github.com/tarantool/metrics/pull/131 will be resolved


Add a contribution.md

In this task, I intend to write out the workflow for would-be contributors to create PR's and also errors faced during installation and solutions. This will be a guide for future collaborators to reference whenever they face blockers.

@manwar kindly assign this to me , thanks


Microsoft.AspNetCore.Authentication.AzureADB2C.UI package

This page includes a reference to Microsoft.AspNetCore.Authentication.AzureADB2C.UI package which is now obsolete.  Please update doc.

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

ID: 520b9821-ecef-20af-a601-ea36c4297d88
Version Independent ID: da9c8ac9-4d7a-4b40-56a2-a4bfe7083567
Content: Secure an ASP.NET Core Blazor WebAssembly hosted app with Azure Active Directory B2C
Content Source: aspnetcore/blazor/security/webassembly/hosted-with-azure-active-directory-b2c.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @vangundy ... You're looking at the 3.1 version of the doc. Change the version selector to 5.0 to see the updated content.

Oops! sorry and thank you 😊
-Paul

On Sep 3, 2021, at 5:12 PM, Luke Latham @.***> wrote:

Hello @vangundy ... You're looking at the 3.1 version of the doc. Change the version selector to 5.0 to see the updated content.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Theme does not apply to blocked user's messages

I don't usually block people so I've only recently noticed this, but any messages from a blocked user do not have Amethyst applied to it.

Fixed!

Can confirm, fixed.


Handling of PTP reference clocks.

NMOS IS-04 requires that the GMID of the PTP reference clock be given with lowercase hex number notation. The corresponding SDP attribute a=ts-refclk has it the other way around, i.e. uppercase hex only. Somewhere a conversion must happen either way, but I don't see where.
Furthermore, nmos-cpp seems to assume that the a=ts-refclk attribute is always in the media section of the SDP file. Indeed that is what ST2110 (stupidly) demands. Other relevant standards don't have this restriction, for example AES67, and indeed the RFC 7273 they all depend on. One would have thought that the normal case is a common reference clock, so that putting it into the global section of the SDP file would be the most prevalent case, if it weren't for ST2110.
So I think nmos-cpp should at least recognise and properly handle either style when receiving an SDP. At the moment, it insists on ST2110 on input, too.
Please tell if you want me to produce a patch.

NMOS IS-04 requires that the GMID of the PTP reference clock be given with lowercase hex number notation. The corresponding SDP attribute a=ts-refclk has it the other way around, i.e. uppercase hex only.

https://github.com/AMWA-TV/nmos-discovery-registration/issues/144

Somewhere a conversion must happen either way, but I don't see where.

E.g. https://github.com/sony/nmos-cpp/blob/7284ef84c07e1772cc4234ee20ed299571906bfb/Development/nmos/sdp_utils.cpp#L64

Furthermore, nmos-cpp seems to assume that the a=ts-refclk attribute is always in the media section of the SDP file. Indeed that is what ST2110 (stupidly) demands.

I'm well aware of the (insert adjective of your choice) additional demands added by ST 2110 beyond RFC 7273. I raised this previously with the ST 2110 team, and was told it was to make things "simpler". 🤷♂

So I think nmos-cpp should at least recognise and properly handle either style when receiving an SDP. At the moment, it insists on ST2110 on input, too.
Please tell if you want me to produce a patch.

Yep, updating nmos::get_session_description_sdp_parameters to use session-level values for ts-refclk and mediaclk as the default, as per RFC 7273 Section 4.8, would be nice.

Gosh, time to update my repository. Sorry to bother you with old stuff.

I raised this previously with the ST 2110 team, and was told it was to make things "simpler".

Ouch. That's not my concept of "simpler".
I will try to produce a patch for session-level ts-refclk and mediaclk shortly.

Yep, updating nmos::get_session_description_sdp_parameters to use session-level values for ts-refclk and mediaclk as the default, as per RFC 7273 Section 4.8, would be nice.

See https://github.com/aholzinger/nmos-cpp/commit/80f41a8679a15f85e6a2cbd75e77600250bdfbcb


[Twig] File Name convention

Big BC BREAK, rename twig file names to snake_case

card-vertical-navigation.html.twig to card_navigation_vertical.html.twig
card_nav.html.twig to card_navigation.html.twig
card_nav_header.html.twig to card_navigation_header.html.twig
and more...

milestone 1.0 because this has to be done BEFORE the first "real" release

Two of them are only internal, aren't they? Makes the BC "smaller".
We should probably document the file name conventions somewhere as well.


Technical mappings scripts not working due to NOT acceptable URL

For scripts like
https://github.com/mitre-attack/attack-scripts/blob/master/scripts/technique_mappings_to_csv.py ,the collection URL is not working anymore.
    collection_map = {
        "enterprise_attack": "95ecc380-afe9-11e4-9b6c-751b66dd541e",
        "mobile_attack": "2f669986-b40b-4423-b720-4396ca6a462b"
    }
    collection_url = "https://cti-taxii.mitre.org/stix/collections/" + collection_map[collection_name] + "/"
    collection = taxii2client.Collection(collection_url)
    taxii_ds = stix2.TAXIICollectionSource(collection)

The error shows as: 406 Client Error: Not Acceptable for url:
  File "/tmp/stix_code/py3env/lib/python3.6/site-packages/stix2/datastore/taxii.py", line 166, in __init__
    " Collection object provided could not be reached. Recieved error:", e,
stix2.datastore.DataSourceError: The underlying TAXII Collection resource defined in the supplied TAXII Collection object provided could not be reached. Recieved error: "406 Client Error: Not Acceptable for url: https://cti-taxii.mitre.org/stix/collections/95ecc380-afe9-11e4-9b6c-751b66dd541e/"

Hi @getkub,
This is an issue introduced by a recent update to the taxii2client library. Essentially they changed their API such that the default TAXII requests will try to fetch STIX 2.1 content using TAXII 2.1 from the TAXII server. Because our TAXII server uses the TAXII 2.0 spec (and stores STIX 2.0 content) this results in the 406 not acceptable error you're getting.
The correct resolution is to update your import:
from taxii2client.v20 import Collection

This issue is rather tricky for us because it depends on which version of the taxii2client repo users have installed -- only users with new installs get the error, and users who perform the above fix with the older taxii2client version may get errors.
See also #19.

Hi @getkub,
This is an issue introduced by a recent update to the taxii2client library. Essentially they changed their API such that the default TAXII requests will try to fetch STIX 2.1 content using TAXII 2.1 from the TAXII server. Because our TAXII server uses the TAXII 2.0 spec (and stores STIX 2.0 content) this results in the 406 not acceptable error you're getting.
The correct resolution is to update your import:
from taxii2client.v20 import Collection

This issue is rather tricky for us because it depends on which version of the taxii2client repo users have installed -- only users with new installs get the error, and users who perform the above fix with the older taxii2client version may get errors.
See also #19.

Yeah, I tried that.  the above logic of
from taxii2client.v20 import Collection
works ok for techniques_data_sources_vis.py, But NOT for https://github.com/mitre-attack/attack-scripts/blob/master/scripts/technique_mappings_to_csv.py.
But thanks anyway, will downgrade and see

Reopnening so that we can investigate the fix for https://github.com/mitre-attack/attack-scripts/blob/master/scripts/technique_mappings_to_csv.py.

Hi @getkub,
I think the way the taxii2client import is used in technique_mappings_to_csv.py is slightly different and requires more tweaking. In the case of that file, you can do one of two things:

Change line 6 to from taxii2client.v20 import Collection
Change line 18 to collection = Collection(collection_url)

Change line 18 to collection = taxii2client.v20.Collection(collection_url) without updating the import.

Let me know if that works for you.