Test task avoidance is not working since Gradle 7.4.x

Hi,
it seems that the task avoidance for tests is not working anymore since Gradle 7.4.x
Expected Behavior
Test tasks are not eagerly created
Current Behavior
Test tasks are eagerly created
Context

Steps to Reproduce
Have something like the following in your project
tasks.withType(Test).configureEach {
    println "Configure Tests"
    testLogging {
        showStandardStreams = true
        exceptionFormat = 'full'
    }
}

Configure Tests should not be printed when the test tasks are not run, but it is. For simpler reproduction: https://github.com/dreis2211/gradle-task-avoidance-bug
This seems to be caused by the relatively new testResultsElementsForTest infrastructure added in #18834, but I haven't looked too deep yet.
Cheers,
Christoph

I wasted this morning trying to catch the issue in our codebase...until I found it was Gradle bug.
Just wanted to say, this is quite an issue for us.

This is fixed in 7.6

@big-guy Do you have a PR/commit where this is fixed?

This seems not fixed yet in 7.6 snapshot on https://github.com/dreis2211/gradle-task-avoidance-bug,  @big-guy
christoph.dreis@MB-20000266 gradle-task-avoidance-bug % ./gradlew wrapper --gradle-version=7.6-20221012231352+0000
Downloading https://services.gradle.org/distributions-snapshots/gradle-7.6-20221012231352+0000-bin.zip
...........10%............20%...........30%............40%............50%...........60%............70%............80%...........90%............100%

Welcome to Gradle 7.6-20221012231352+0000!

Here are the highlights of this release:
 - Added support for Java 19.
 - Introduced `--rerun` flag for individual task rerun.
 - Added support for incremental compilation following a compilation failure.

 Sorry for the noise.
Nonetheless, do you have a commit to share where this has been fixed?

It wasn't addressed specifically in a commit, but it's a part of this stream https://github.com/gradle/gradle/pull/21292
The problem was that registered artifacts were eagerly evaluated whenever a project's metadata was built for dependency resolution.  We had some previous attempts at making the artifacts lazy (since most of the time you only want to know that they exist), but this change tackled it more fundamentally by making the process to build the project metadata more on-demand.

@big-guy Thanks for sharing - highly appreciated. 👍


Examples throw Error E0225

Compiling the kernels in the examples directory using
xargo rustc \
    --manifest-path kernel/Cargo.toml \
    --release \
    --target nvptx64-nvidia-cuda \
    -- --emit=asm

I get the following error during the compilation of core64:
error[E0225]: only Send/Sync traits can be used as additional traits in a trait object
   --> C:\Users\Reiner\.cargo\git\checkouts\core64-98c99607e3b29655\0e29675\any.rs:133:27
    |
133 | impl fmt::Debug for Any + Send {
    |                           ^^^^ non-Send/Sync additional trait

error[E0225]: only Send/Sync traits can be used as additional traits in a trait object
   --> C:\Users\Reiner\.cargo\git\checkouts\core64-98c99607e3b29655\0e29675\any.rs:244:10
    |
244 | impl Any+Send {
    |          ^^^^ non-Send/Sync additional trait

error: aborting due to 2 previous errors

Any Idea what I am doing wrong?

@japaric Any idea how to solve this issue? (do you need to sync your branch somehow?). I get the similar rustc error with:
error: `impl Trait for .. {}` is an obsolete syntax
  --> /home/ilab/.cargo/git/checkouts/core64-98c99607e3b29655/0e29675/marker.rs:51:1
   |
51 | unsafe impl Send for .. { }
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = help: use `auto trait Trait {}` instead

error: `impl Trait for .. {}` is an obsolete syntax
   --> /home/ilab/.cargo/git/checkouts/core64-98c99607e3b29655/0e29675/marker.rs:356:1
    |
356 | unsafe impl Sync for .. { }
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: use `auto trait Trait {}` instead

error: `impl Trait for .. {}` is an obsolete syntax
   --> /home/ilab/.cargo/git/checkouts/core64-98c99607e3b29655/0e29675/marker.rs:571:1
    |
571 | unsafe impl Freeze for .. {}
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: use `auto trait Trait {}` instead

error[E0225]: only auto traits can be used as additional traits in a trait object
   --> /home/ilab/.cargo/git/checkouts/core64-98c99607e3b29655/0e29675/any.rs:133:27
    |
133 | impl fmt::Debug for Any + Send {
    |                           ^^^^ non-auto additional trait

error[E0225]: only auto traits can be used as additional traits in a trait object
   --> /home/ilab/.cargo/git/checkouts/core64-98c99607e3b29655/0e29675/any.rs:244:10
    |
244 | impl Any+Send {
    |          ^^^^ non-auto additional trait

error: aborting due to 5 previous errors

For more information about this error, try `rustc --explain E0225`.
error: Could not compile `core`.

To learn more, run the command again with --verbose.
error: `"cargo" "build" "--release" "--manifest-path" "/tmp/xargo.M2H5rCFO09xN/Cargo.toml" "--target" "nvptx64-nvidia-cuda" "-p" "core"` failed with exit code: Some(101)
note: run with `RUST_BACKTRACE=1` for a backtrace

@ehsanmok what nightly version did you use?
This is what I'm currently using
`$ rustup show
Default host: x86_64-pc-windows-msvc
installed toolchains
stable-x86_64-pc-windows-msvc
nightly-x86_64-pc-windows-msvc (default)
active toolchain
nightly-x86_64-pc-windows-msvc (default)
rustc 1.27.0-nightly (e82261dfb 2018-05-03)
and get the following error$ xargo +nightly rustc --target nvptx64-nvidia-cuda --release -- --emit=asm --verbose

"rustc" "--print" "sysroot"
"rustc" "--print" "target-list"
RUSTFLAGS="--sysroot C:\Users\ADN\.xargo"
"cargo" "+nightly" "rustc" "--target" "nvptx64-nvidia-cuda" "--release" "--" "--emit=asm" "--verbose"
Compiling cuda v0.1.0 (https://github.com/japaric/cuda#f15863be)
error: Error loading target specification: Could not find specification for target "nvptx64-nvidia-cuda"
|
= help: Use --print target-list for a list of built-in targets

error: Could not compile cuda.
To learn more, run the command again with --verbose.
`

@TGM I just updated my prev post.

@TGM I had the same problem as you, and cargo install --force xargo worked for me (updates xargo to latest version).


Calling fixture factories for a table breaks application code that uses that table downstream

This was the bug at the bottom of a well that I spent many, many days trying to diagnose.
Description of the problem
It appears that if fixture factories touch a table and then downstream application code also does, entity creation does not work for that table in the application code.
How to reproduce

I have a test class that calls a method that makes (using Fixture Factories) 5-10 entities from the AccessTokensTable and returns them (they are not persisted).
These entities are then used in the test...and all these tests pass. Call this test class BadTest.
I have another test class (call it InnocentTest) which is an Integration test. Several tests within it POST to a controller action in my application code.
This controller action is supposed to create a new entity using the same table as was previously used by the fixture factories. (AccessTokensTable). This entity should be persisted, by the app.
Any and all tests that call that action fail because the entity that was supposed to be generated is not generated.

I was able to confirm that the problem occurs as a result of the call to AccessTokensFactory::make(), as refactoring my test data provider to manually construct the entities instead of calling the fixture factory results in all tests in the InnocentTest test class passing once again.
Expected behavior
Whatever magic the fixture factories do to the tables in order to create entities, that magic does not interfere with application code being able to interface with the tables and make entities downstream.
Let me know if all this is clear? I can try to put together an open-source sample app if needed.

@jamisonbryant thank for opening the issue. A sample app would be definitely appreciated, in order to reproduce the issue.

I haven't had time to put together a sample app, but I do think I have discovered something relevant. Adding these two lines to:
// EventCollector.php:91
$conn = $table->getConnection();
echo __CLASS__ . ' table connection = ' . PHP_EOL . $conn->configName() . PHP_EOL;

prints "foo" which is one of my database connections instead of "test_foo" as I would expect.
Is it my understanding that when the Cake test suite is running, the defaultConnectionName() of each table is supposed to be prepended by "test_" forming the complete connection name of "test_foo".
I'm not sure what an event collector does, but is there any chance it is not respecting the default cake convention for table names when running unit tests? Could that be the source of this problem?

Here is more evidence that the BaseFactory's $eventCollector is flipping back and forth between connections:
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (bottom check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = test_saas
CakephpFixtureFactories\Factory\EventCollector table connection (bottom check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas
CakephpFixtureFactories\Factory\EventCollector table connection (top check) = saas

Top check = EventCollector::getTable() line 78, inside the if block
Bottom check = EventCollector::getTable() line 99, right before the function return

Mar 2023 update: I was able to work around this issue, at long last!
I was right about application code trying to access the table registry at the same time as the unit tests, and using different connections to do so. E.g. my fixture factories would create and persist entities using the test connection, and my app would do the same except using the app connection. For my project, these are two different databases.
The solution is to alias the connections in the test bootstrapper:
$testDatasource = 'foo';
try {
	ConnectionManager::get($testDatasource);
	ConnectionManager::alias($testDatasource, 'default');
	ConnectionManager::alias($testDatasource, 'test');
	ConnectionManager::alias($testDatasource, 'other');

	//(new SchemaLoader())->loadSqlFiles(TESTS . 'schema.sql', $testConnection);
	(new Migrator())->run(['connection' => $testDatasource]);
} catch (MissingDatasourceConfigException $e) {
	trigger_error("Cannot run the test suite: missing connection '{$testDatasource}'. Check main config file.");
}

Now, even when the application tries to use the "wrong" database connection, it will go to the correct database.
I'm not sure there's a change to be made to Fixture Factories unless it's a docs clarification, do you agree @pabloelcolombiano?


Add type definitions for RequestOptions and ResponseMock

Purpose
Add type definitions for RequestOptions and ResponseMock, and update type definitions for RequestMock.onRequestTo and RequestMock.respond accordingly.
Approach

Copied text/types from https://devexpress.github.io/testcafe/documentation/reference/test-api/requestmock/respond.html#requestoptions
Also took a look at the src:

RequestOptions: https://github.com/DevExpress/testcafe-hammerhead/blob/master/src/request-pipeline/request-options.ts
ResponseMock: https://github.com/DevExpress/testcafe-hammerhead/blob/master/src/request-pipeline/request-hooks/response-mock.ts

Not entirely sure how to test these in any feasible way. I confirmed they worked by modifying the .d.ts in node_modules of an existing project with the above changes, and confirming that auto-completion works, but not sure if the existing tests will catch anything odd here.
I use the type definitions for autocompletion in a non-typescript project, so not sure if there's something here that might cause problems for actual typescript users.

References
Closes #5108
Pre-Merge TODO

[ ] Write tests for your proposed changes
[ ] Make sure that existing tests do not fail

@testcafe-build-bot retest

@testcafe-build-bot retest

Testing seems to be stuck?

@cdrini I restarted the stuck tests. It seems that only Travis' server tests fail now

Thanks @AlexKamaev ! I think this is ready for review now :+1:
(The typings actually caught a bug! \o/)

@cdrini I wonder if you somehow missed adding isAjax to RequestOptions type?
The following used to compile but now I get typescript error Property 'isAjax' does not exist on type 'RequestOptions'.
RequestMock()
      .onRequestTo(req => {
        if (req.isAjax) {
         // do something
        }

      })

According to documentation isAjax should exist:
https://devexpress.github.io/testcafe/documentation/reference/test-api/requestmock/onrequestto.html

I based this off https://devexpress.github.io/testcafe/documentation/reference/test-api/requestmock/respond.html#requestoptions ; it looks like it's missing from those docs as well. It is in the src code: https://github.com/DevExpress/testcafe-hammerhead/blob/master/src/request-pipeline/request-options.ts#L19

You're right, the description of the isAjax property is missing in the documentation. I created a separate issue) for this.


fix for 4.07: Ident.t is now an abstract type

This is the non-problematic part extracted from #23
cc @gasche

The travis CI fails on 4.02 and 4.03, apparently due to an incompatibility between the upstream opam repository and the opam version that the CI instances use. I am testing locally that the build still work on these version, then merging.

Apparently I can't merge in this repository (maybe because the CI failed?), but I was able to test that this PR builds fine under 4.03.0.


NBA Boxscores() returns incorrect home_points

Describe the bug
When using the Boxscore method from NBA games, I am getting the wrong results for the home_XXXX metrics.
To Reproduce
Sample code which causes an issue.
from sportsreference.nba.boxscore import Boxscore
Boxscore('202009130DEN').dataframe[['home_points','away_points',
'home_two_point_field_goals','away_two_point_field_goals']]

Expected behavior
Method returns the correct full game box score for both home and away teams.
Screenshots
result of code above:
              home_points  away_points  home_two_point_field_goals  away_two_point_field_goals
202009130DEN           64           98                          10                          22

Desktop:

OS: MacOS
Sportsreference Version 0.5.2

Additional context
The boxscore method worked correctly prior to 9/9/2020

You can fix it by changing in file nba/constants.py:
'home_points': 'tfoot td[data-stat="pts"]'
to
'home_points': '.score'

Output:
home_points                   111
away_points                    98
home_two_point_field_goals     10
away_two_point_field_goals     22

I didn't unit test this and not sure if this breaks other pages...

Hey @sseljan, thanks for bringing this to my attention! Looks like the website changed the layout slightly, so the 64 that you are seeing there is actually the home team's second half point total, which is obviously incorrect. @dropthemic's wonderful solution fixes the problem (thank you for sharing!!), and I have a patch in the works that I hope to merge shortly so this can be updated in the upstream code. Thanks again to you both for the help!

Awesome!
On Sun, Sep 27, 2020 at 6:45 PM Robert Clark<EMAIL_ADDRESS>wrote:

Hey @sseljan https://github.com/sseljan, thanks for bringing this to my
attention! Looks like the website changed the layout slightly, so the 64
that you are seeing there is actually the home team's second half point
total, which is obviously incorrect. @dropthemic
https://github.com/dropthemic's wonderful solution fixes the problem
(thank you for sharing!!), and I have a patch in the works that I hope to
merge shortly so this can be updated in the upstream code. Thanks again to
you both for the help!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/roclark/sportsreference/issues/470#issuecomment-699725601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABASFNHI5LC3POGWCWOGEQ3SH7TDLANCNFSM4RL65ACA
.

While I really appreciate that you've addressed this, I should note that all of the home stats (field goals, blocks, steals, etc) are from the second half. I could have labeled the issue more clearly.
As I read the solution above, it wasn't clear to me if it only corrects the points or all stats.


Failing to install

I get stuck after the "pip install ." I read a mistake : "Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-_HK226/pandas/"

To try to reproduce it, I'm gonna need your Python version (python --version) and distro.  I suspect it's an outdated setuptools package, so go ahead and try to update it (pip install --upgrade setuptools) and run pip install . again.

I'm on a Pi4 with latest RaspiOS. Python is 2.7.16, it successfully installed setuptools-44.1.1 and I run pip install . again. Will keep you updated. Thanks for the quick answer.

That latest command took a while to complete. Problem is now solved, thanks !


chore: Add a creat-react-app example with javascript

Description
Add a creat-react-app example with javascript but not typescript. There are basic, dag and flow sub examples seperately in it. Because people like me don't need a typescript project always.
Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Test Case (changes that add missing tests or correct existing tests)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[x] Chore (changes that don't modify src or test files)

Self Check before Merge

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

hello nelsonkuang, thanks for your contributions. have a nice day.


ExclusionList not working

I was trying to use this new feature to filter out some events that I don't want to be notified. After some time trying to remove really simple messges I checked the code and I think I saw a bug in the implementation.
I would say the problem is here. Continue instructions could be tricky, they continue to the next iteration in the inner bug. In that case there are two nested loops, so I would say this continue instruction will continue evaluating the next ExclusionList entry instead of going to the next event. As a result it never skips an event.

That diagnosis is spot on -- thank you.


chore: bdd-sbrp-import-1587317073 to 0.0.1

chore: Promote bdd-sbrp-import-1587317073 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


fix: search input line break

This PR resolves ONYX-931
Description
This PR fixes an issue where the search input placeholder breaks on Android.
The issue comes from React-native and is still not fixed even in the latest version.

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

search input placeholder cropped - mounir

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Android user-facing changes (search input placeholder cropped - mounir)

  Generated by :no_entry_sign: dangerJS against b201507834b407c66906b0c8a8ce8737fa343ba1

Here we go https://github.com/artsy/palette-mobile/pull/225


Plug-in Keeps Restarting

Problem:
Trying to use the plugin with my RA 3 processor to get picos in homekit setup for the first time. My Pico works fine with Lutron app, but is not being pulled into the homebridge plugin. I have successfully been able to find and connect to my RA 3 processor, but plug-in looks like it’s having some issues and keeps restarting. In my dashboard the status light keeps going yellow, green, red and when I tried to add it to homekit I get a failed indicator.
I am not sure if it’s a bug or I missed a step in the setup guide or both.

Unfortunately, this plugin doesn't support RA3 processors, mostly because I don't have one to test with. Sorry. :(


Not able to set the dates on bootstrap calendar from generated dates variable?

Expected behaviour
All I am trying to do is setDates after the ajax call. I have dates array coming from PHP page. So the all dates expected to display as selected dates on bootstrap calendar.
Actual behaviour
The list of dates are not displaying as selected/set dates on the bootstrap calendar view. However, its setting up if I pass only single date for ex, 23/04/2020 but if I pass dates like 23/04/2020,24/04/2020 in a variable then its not setting up.
Datepicker version used
Datepicker for Bootstrap v1.8.0
Example code
`$.ajax({
type: "POST",
url: "remove_data.php",
data: {ch: ch, dh: dh, sp_rec: sp_rec, cartDate: 1},
success: function(data) {
//console.log(data);
				var obj = $.parseJSON(data);

				var myDates = '';
				var temp,temp2;
				for(var i=0;i<obj.length;i++) {
					var temp = obj[i];
					var dt = new Date(temp);
					myDates = myDates+" "+dt+",";
					temp2= obj[1];//0=22 1=23
				}
				temp = "2020-04-23";
				//console.log(myDates);

				var dt2 = new Date(temp2);
				var dt1 = new Date(temp);
				//working...but I don't want this solution...
				$('#disc_single_datepicker_1').datepicker('setDates',[dt1,dt2]); //set two dates....

				//Not working...but I want to make work like below....
				$('#disc_single_datepicker_1').datepicker('setDates',[myDates]); //doesn't set any dates.....however it should set all the provided dates......

				display_toastr(datainfo+" record has been removed!","Remove Record",true,"right","info");
				$("#row_"+dh).fadeOut('slow');

			}, error: function() {
				alert("Something went wrong. Please contact admin!");
			}
		});`

Jsfiddle example to reproduce the problem.

Please let me know as I am trying to resolve this from more than 6 hours

Resolve it on stackoverflow: stackoverflow


Infinite Redirect

Several users have reported issues with infinite redirects (301) when performing certain actions using version <IP_ADDRESS>.  These actions include (but may not be limited to) the following:

Signing out
Activating a theme
Loading a draft page

It has been suggested that rolling back the code from commit 1e5357db may actually resolve the issue, however, given the nature and comment of the commit, reverting it might cause regressions.
For reference:

https://www.concrete5.org/developers/bugs/5-7-5-3/redirect-loops/
https://www.concrete5.org/developers/bugs/5-7-5-3/user-cannot-logout-of-system-on-<IP_ADDRESS>/
https://www.concrete5.org/community/forums/installation/after-install-err_too_many_redirects/#791095
https://www.concrete5.org/community/forums/chat/log-out-does-not-work/#787826

Environment Info:
Environment.txt

Can you try this on the current upstream? Remember that we're always working toward the next release so this may already be fixed.

Hi, Korvin,
Thanks for responding.  I've performed the following steps:

Cloned the concrete5 repo locally and checked out the develop branch.
Copied the concrete5/web/concrete directory to my installation (named concrete-beta).
Renamed concrete to concrete-prod.  Renamed concrete-beta to concrete.
Loaded index.php/ccm/system/upgrade and re-ran the script.
Cleared the cache.

The issue still persists.  When attempting to log out or access a draft page, the infinite redirect occurs.  Please let me know if I have missed any steps in migrating to the latest develop code, or if there are any other diagnostic tasks I can perform.

Can you show what you have in /application/config/generated-overrides/concrete.php?

I've attached the generated file here: concrete.php.txt.  I have nothing in .htaccess.

I can't get this to happen locally at all, can anyone else confirm?

In my initial report, I listed several places where people were experiencing the same issue.  I've added posts in those locations requesting that those authors comment here as well.
One of the posts mentions that after rolling back to <IP_ADDRESS>, the issue is no longer present.  I may test this on my own installation.

Is this only occurring on sites that were previously upgraded?
Is this only occurring on multi-language sites?
Can someone who has the problem do a completely clean install of 7.5.3 on the same server as the failing site to verify if the problem still persists?

I'm getting the same when activating a theme.
Same issue as http://www.concrete5.org/community/forums/5-7-discussion/major-issue-with-my-5.7-install.../#740893
Apply my quick fix
http://www.concrete5.org/community/forums/5-7-discussion/major-issue-with-my-5.7-install.../#740013
resolves issue

I believe this is fixed with a recent pull request.
https://github.com/concrete5/concrete5/pull/3215
We have accepted this pull request. The pull request basically adds urldecode to the URLs that are being compared prior to redirect. We had this behavior before, but it was undone in commit ecba369a8e98af0ed5aa864824afa1a911617d6d . This resulted in numerous bugs regressions, which we have been fixing. Sorry for the inconvenience. We hope to get this released in <IP_ADDRESS> very soon.

I'll test this, but that pull request was merged into develop 14 days ago.  This behavior was still seen 6 days ago.

@aembler we might be able to avoid this forever if we just change the signature of the ccm_token output, if we use something like a hyphen or a decimal, we shouldn't need to worry about the encoding.

Pulled latest develop, and updated my installation.  As above, I ran the update script and cleared my cache.  The generated-overrides/concrete.php file was freshly regenerated.
The problem still persists.  Recommend re-opening this issue, as other users are seeing the problem too, and are unlikely to have been helped by the changes made thus far.

@macserv If you revert handleURLSlashes to this, does this assist in any way?
public function handleURLSlashes(SymfonyRequest $request)
{
    $parsedUrl = (string) Url::createFromUrl($request->getUri());
    if ($request->getPathInfo() != '/') {
        $parsedUrlWithoutQueryString = strstr($parsedUrl, '?', true) ?: $parsedUrl;
        $requestUrl = $request->getUri();
        $requestUrlWithoutQueryString = strstr($requestUrl, '?', true) ?: $requestUrl;
        if (urldecode($parsedUrlWithoutQueryString) != urldecode($requestUrlWithoutQueryString)) {
            $response = new RedirectResponse($parsedUrl, 301);
            $response->setRequest($request);
            return $response;
        }
    }
}

Commented out current implementation (19 lines).  Inserted code from comment above (14 lines).  Re-ran update script and cleared cache.  Does not seem to have had an effect.
I noticed that the URL in my address bar had a percent-encoded colon (%3A) in it... is this correct?
index.php/login/logout/1451935980%3A40d027c8d[REDACTED]

macserv, browsers aggressively cache 301 redirects, can you see if this happens in a different browser / from a different computer?

While trying out the code you committed for PR #3275, I did log in and (attempt to) log out using a second browser (Safari, OS X 10.11.1).  I encountered the same issue.

Macserv, can you show the contents of your .htaccess and your apache virtualhost?

In this case, I have no .htaccess file. [Here's] (https://github.com/concrete5/concrete5/files/77691/www.npmcolumbus.org.conf.txt) my (sanitized) configuration.

This issue is not resolved for me.  Should I open a new one?  What steps should I take from here?

@macserv this is tough. I didn't necessarily mean to close this by merging that other commit, but I think it should probably remain closed for the time being.
Obviously we don't want to leave you hanging if this issue is indeed an issue in concrete5, but @edbeeny  is able to attest that the issue is resolved via #3275 so at least the issue he was having is resolved.
So I think the next step here is to see if you can get an isolated example that can be replicated by the rest of us, try installing concrete5 locally and see if you get the same result. If you do, maybe try to isolate the code that is sending the response. Come into the irc (#concrete5 on irc.freenode.net) and we can try to debug it together.

I would have liked to have more confirmation than one user... several have reported the same problems.  I'll see you in IRC.

Here's a link you can use: https://kiwiirc.com/client/irc.freenode.net/#concrete5

After a little debugging, it was determined that patches were being made to the original concrete core installation, when a newer version existed in updates.  Easy for a newbie to miss, and difficult for a veteran to catch.
With the patch from #3275 correctly applied, this issue, indeed, appears to be resolved.  Verified that the redirect no longer occurs when signing out, or when attempting to access a draft page.
Thanks to @KorvinSzanto for his patient assistance!


Check that every message to and from the bot is auto-deleted if the option is activated

Except for shutdown and restart, obviously.
I.e. summon and pldump trigger messages don't get deleted.

Centralized delete delay for every message, that should fix it.


update kubectl-rook-ceph to include latest changes

This commit updates kubectl-rook-ceph to
include the lateste changes

@yati1998 we don't need any extra changes as the commands and function calls are thes eame just updating the version will work?

No, we don't need any additional cde change here, I have tested it to confirm as well. It works fine.


Release Google.Cloud.AssuredWorkloads.V1Beta1 version 1.0.0-beta08

Changes in this release:
New features

EU Regions and Support With Sovereign Controls (commit 0e8c59f)

Created release for Google.Cloud.AssuredWorkloads.V1Beta1-1.0.0-beta08


Highlights 2.2.0

I just merged #13383, if you want to add it please do, I'm not sure that it's super important to mention it.

I just merged #13383, if you want to add it please do, I'm not sure that it's super important to mention it.

definitely worth mentioning imo


Google Container Registry: Unable to get access token (invalid_grant)

I'm having an issue getting this plugin to work with a repository hosted on Google Container Registry (GCR).
When trying to build using the following configuration (as adapted from the configuration in README), it appears that the plugin fails to retrieve a valid access token from Google:
          <plugin>
                <groupId>com.spotify</groupId>
                <artifactId>dockerfile-maven-plugin</artifactId>
                <version>1.3.5</version>
                <executions>
                    <execution>
                        <id>default</id>
                        <goals>
                            <goal>build</goal>
                            <!--<goal>push</goal>-->
                        </goals>
                    </execution>
                </executions>
                <configuration>
                    <repository>gcr.io/xxx/yyy</repository>
                    <tag>${project.version}</tag>
                </configuration>
            </plugin>

unable to get access token for Google Container Registry, configuration for building image will not contain RegistryAuth for GCR
com.google.api.client.http.HttpResponseException: 400 Bad Request
{
  "error" : "invalid_grant"
}

I'm not sure if I need any additional configuration but manually running gcloud docker -a works fine, so I would suspect that there's not an issue with my local setup:
$ gcloud docker -a
Short-lived access for ['gcr.io', 'us.gcr.io', 'eu.gcr.io', 'asia.gcr.io', 'l.gcr.io', 'launcher.gcr.io', 'us-mirror.gcr.io', 'eu-mirror.gcr.io', 'asia-mirror.gcr.io', 'mirror.gcr.io'] configured.

Once I've done that I can push to GCR just fine (using docker push), but the plugin does not seem to use this configuration, or at least not for this step so that doesn't help.
Anonymized logs below:
[INFO] --- dockerfile-maven-plugin:1.3.5:build (default-cli) @ xxx ---
10:16:40.958 [main] DEBUG com.spotify.docker.client.DockerConfigReader - Using configfile: /Users/bas/.docker/config.json
[INFO] Using Google application default credentials
10:16:41.153 [main] INFO com.spotify.docker.client.auth.gcr.ContainerRegistryAuthSupplier - loaded credentials for user account with clientId=32555940559.apps.googleusercontent.com
10:16:41.215 [main] DEBUG com.spotify.docker.client.DockerCertificates - /Users/bas/.docker/ca.pem, /Users/bas/.docker/key.pem or /Users/bas/.docker/cert.pem does not exist, not using SSL
[INFO] Building Docker context /Users/bas/xxx
[INFO]
[INFO] Image will be built as gcr.io/xxx/yyy:1.0.1
[INFO]
10:16:42.880 [main] WARN com.spotify.docker.client.auth.gcr.ContainerRegistryAuthSupplier - unable to get access token for Google Container Registry, configuration for building image will not contain RegistryAuth for GCR
com.google.api.client.http.HttpResponseException: 400 Bad Request
{
  "error" : "invalid_grant"
}
	at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:1054)
	at com.google.auth.oauth2.UserCredentials.refreshAccessToken(UserCredentials.java:207)
	at com.google.auth.oauth2.OAuth2Credentials.refresh(OAuth2Credentials.java:149)
	at com.spotify.docker.client.auth.gcr.ContainerRegistryAuthSupplier$DefaultCredentialRefresher.refresh(ContainerRegistryAuthSupplier.java:195)
	at com.spotify.docker.client.auth.gcr.ContainerRegistryAuthSupplier.getAccessToken(ContainerRegistryAuthSupplier.java:226)
	at com.spotify.docker.client.auth.gcr.ContainerRegistryAuthSupplier.authForBuild(ContainerRegistryAuthSupplier.java:295)
	at com.spotify.docker.client.auth.MultiRegistryAuthSupplier.authForBuild(MultiRegistryAuthSupplier.java:77)
	at com.spotify.docker.client.DefaultDockerClient.build(DefaultDockerClient.java:1388)
	at com.spotify.docker.client.DefaultDockerClient.build(DefaultDockerClient.java:1365)
	at com.spotify.plugin.dockerfile.BuildMojo.buildImage(BuildMojo.java:178)
	at com.spotify.plugin.dockerfile.BuildMojo.execute(BuildMojo.java:105)
	at com.spotify.plugin.dockerfile.AbstractDockerMojo.tryExecute(AbstractDockerMojo.java:227)
	at com.spotify.plugin.dockerfile.AbstractDockerMojo.execute(AbstractDockerMojo.java:216)
	at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:209)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
	at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:84)
	at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:59)
	at org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(LifecycleStarter.java:183)
	at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:161)
	at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:320)
	at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
	at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
	at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:290)
	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:230)
	at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:409)
	at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:352)

Environment:
Apache Maven 3.0.5 (r01de14724cdef164cd33c7c8c2fe155faf9602da; 2013-02-19 14:51:28+0100)
Maven home: /Applications/NetBeans/NetBeans 7.4.app/Contents/Resources/NetBeans/java/maven
Java version: 1.8.0_45, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_45.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.12.4", arch: "x86_64", family: "mac"

Oops! Never mind.
This seems to be an issue on my site: Running gcloud auth application-default login created a new session and that seems to have fixed the issue.
I guess my authentication details had expired somehow.

This issue deserves to become part of the documentation. It wasn't clear how GCR needed to be configured to allow the push goal to work, nor is there even much mention of the push goal existing.

@chabala good idea, would you mind making a PR to add some text that you would find helpful? the readme refers to “application default credentials” and links to Google documentation but doesn’t say what CLI commands to run

I will endeavor to do so, though I know little of the how & why yet, only that raindancing the solution here allow my authentication to proceed.

in my case I just wanted to build the image but I had old expired gcloud credentials laying around under ~/.config/gcloud
deleting the dir stopped showing the error.
I think it's a bit too deterministic implementation to assume on dockerfile:build command an upcoming push.

Why is that printed although I have no intention to use GCR?
I understand it is warning but still a noise.
I was checking OpenLiberty Guide which uses minikube but seeing this warning made me feel that something is wrong.


Added kafka-support related changes

Updated docker-compose with kafka related ports.
Added new sample for kafka produce-consume

@microsoft-github-policy-service agree [company="microsoft"]


Overridable FileViewer

Hi, really like your package 👍🏼
Could you add a property to overwrite the defaultFileViewer?

Hello, currently I'm busy with other projects. Do you want to contribute to this package? You can send me an email if you want. You can find it on my profile page.


[Theming] Move (WPF) theme-management class to common lib

#7628 introduces a slimmed down version to support (real-time) theme switching. For Color Picker we are using the MahApps implementation. Same for FZ Editor V2 (e.g. #6899).
Would be great to have this in a common lib that is accessible to all PT modules using WPF.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

That is a good idea.  We removed the default class since PT Run has no MahApps dependency.

@enricogior, this could be a 0.29 December task.  Good way to centralize the logic


Update GoBGP

update to https://github.com/osrg/gobgp/commit/dace87570846cc4b4f16e8b25516b22c43888f76
This commit (somehow) fixed behaviour of route attribute visibility in RIB.

shouldn't there be an update in repositories.bzl?


vr is not defined here

https://github.com/JuliaIO/DICOM.jl/blob/35547d73ed940541bbb821902f66f8e4563d1511/src/DICOM.jl#L500

Fixed in #45 (Thanks again)


Disable stat cuckoomap initialization

Callgrind reports that stat cuckoomap constructor & destructor have high cost (even just called once). Removed it temporarily so that we can benchmark tpcc first.

Coverage decreased (-4.3%) to 49.043% when pulling f586dbaf296d489de8e32129081ba60fe7a4dfbf on eric-haibin-lin:disable-stat into ef5182b4d9483ecc0f93a3a8d649adcc1e91084d on cmu-db:master.

Coverage decreased (-4.2%) to 49.161% when pulling f586dbaf296d489de8e32129081ba60fe7a4dfbf on eric-haibin-lin:disable-stat into ef5182b4d9483ecc0f93a3a8d649adcc1e91084d on cmu-db:master.

Coverage increased (+0.009%) to 53.324% when pulling 5a18a9188540131adcb83a5dfe4d035ed23ef12f on eric-haibin-lin:disable-stat into ef5182b4d9483ecc0f93a3a8d649adcc1e91084d on cmu-db:master.

@yingjunwu please review this pr so that you can benchmark tpcc without stat. By default stat is completely turned off now.

@MattPerron @malin1993ml @sid1607 can you help merge it? Looks like @yingjunwu cannot authorize to merge this PR.


MudForm: Added XML Documentation for Public Members

Description
This update adds XML documentation for the public members of MudForm and related components:

MudFlexBreak
MudDebounceInput
MudInput
MudInputAdornment
MudInputCssHelper
MudInputLabel
MudRangeInput
MudTextField
Range

How Has This Been Tested?
This was tested by observing the XML documentation for MudForm examples:
MudForm

MudTextField

Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.
[X] I've submitted the required sacrifice to the Mud god

Ok @henon @ScarletKuro @danielchalmers , here's another set of XML documentation for MudForm and related components.  Thanks for taking the time to review!

The Code coverage step seems to be stuck in the build pipeline

Made a minor change to the docstring to restart the build

Thanks Jon. Good job as always!


Workspaces Launcher

📝 I would love to use PowerToys to launch files, folders, and programs for a work flow. Much like workspaces on mac https://www.apptorium.com/workspaces/
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Would use this to get started on a project or work flow: launch VSC, browser, github, slack for coding for example
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Duplicate of https://github.com/microsoft/PowerToys/issues/3228

Duplicate of https://github.com/microsoft/PowerToys/issues/3228


Update README.md

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

Project coverage is 100.00%. Comparing base (dffe1cf) to head (2217019).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #18   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          122       122
=========================================
  Hits           122       122

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


WIP: instrument require() for autobreadcrumbs

This is a bit behind where I am locally, will push an update soon, but here's the PR to start tracking.

This is now pretty much fully working. I had a bunch of subtle module caching and double-instrumentation stuff going on in the tests because my first pass at uninstall() was slightly insufficient, but got that worked out now.
Still want to investigate a couple things:

http breadcrumbs are sometimes missing req.method, unsure if related to these changes or a separate thing
option of hooking Module._load instead of Module.prototype.require: pros and cons, any reason to prefer one or the other?
observation: when node-core's https module does its own require('http'), it won't hit our hooked require to instrument http, but if we've previously instrumented http it will get our instrumented version, hence the proactive require('http') in instrumentor.instrument;  I want to actually understand why this is.

/cc @MaxBittker @benvinegar

This is now pretty much fully working.

Still want to investigate a couple things:

Is this still WIP? Should we be reviewing?

Worked through investigation points 2 and 3 and determined that current state is acceptable. Identified a duplication issue of some sort in point 1, tracking it down and then I'll remove WIP tag and request review.

Tracked down duplication. Also actually figured out the explanation behind point 3, but opted to leave it as is for to avoid a significant mostly-unnecessary refactor; explained in a comment above where we proactively require('http').
Ready for review; primary piece to review is instrumentor.js, how that's used from client.js and the coupling between instrumentor and Raven, and the interface between instrumentor and each of the three "instrument this module" files.


HTTP Process Request Failed

Hi,
I've tried integrating Buccaneer on the MinimalDefault 5.3 version of UE (i did have to recompile both plugins).
Both Buccaneer, Buccaneer4PixelStreaming and PixelStreaming are enabled.
I'm launching the .exe with only -BuccaneerURL="<IP_ADDRESS>:8000" as parameter for now.
When i'm looking at logs i see a failed HTTP request at <IP_ADDRESS>:

While i try to browse <IP_ADDRESS>/Stats

However the go Server it's sending partial data to Prometheus (that's on a different machine)

Seems to me that the server is not receiving HTTP request from the plugin.
My setup is:

A windows machine that contains both the recompiled plugin and the go server
A linux LXC that contains Prometheus and Grafana

Hey @Azokul,
Try launching with -BuccaneerURL="http://<IP_ADDRESS>:8000". This may be a docs issue.
Cheers

After hard-coding it inside the plugin it worked like a charm :)

Hmm, I would expect Buccaneer to work on consoles. Do you mean to not have it as a UE plugin?

After hard-coding it inside the plugin it worked like a charm :) Just a ping, it would be cool as a standalone tool, having grafana data working on console would be top notch.

would you mind sharing your modification? I am looking to solve a similar issue. When starting up the Unreal Server it cannot seem to connect to the BuccaneerServer and will timeout and kill the server entirely after 7 attempts. I've verified /metrics functions within the golang server, stats throws the same ID error for me. I appreciate any clarity on this, thanks!

Yeah, for sure, tomorrow morning I'll be able to post my exact changes. (on
my work PC)
Tho, it's a change on BuccaneerCommon.cpp, if my memory doesn't fail me it
was a hardcoded buccaneer URL here:
if (!FParse::Value(FCommandLine::Get(), TEXT("BuccaneerURL="),
BuccaneerURL))
I just added a dedicated if "it doesn't get parsed" with the alternative
hardcoded URL
Il giorno lun 12 feb 2024 alle ore 22:39 Chris Magnacca <
@.***> ha scritto:

After hard-coding it inside the plugin it worked like a charm :) Just a
ping, it would be cool as a standalone tool, having grafana data working on
console would be top notch.
would you mind sharing your modification? I am looking to solve a similar
issue. When starting up the Unreal Server it cannot seem to connect to the
BuccaneerServer and will timeout and kill the server entirely after 7
attempts. I've verified /metrics functions within the golang server, stats
throws the same ID error for me. I appreciate any clarity on this, thanks!
image.png (view on web)
https://github.com/TensorWorks/Buccaneer/assets/89811181/aa1c866d-0bdf-4b9b-9adf-7a8e0db0dbf6
—
Reply to this email directly, view it on GitHub
https://github.com/TensorWorks/Buccaneer/issues/7#issuecomment-1939629839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSYLAOGXZJ2LHCE3RYN6UDYTKDY7AVCNFSM6AAAAAA7JFVR4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZZGYZDSOBTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah, for sure, tomorrow morning I'll be able to post my exact changes. (on my work PC) Tho, it's a change on BuccaneerCommon.cpp, if my memory doesn't fail me it was a hardcoded buccaneer URL here: if (!FParse::Value(FCommandLine::Get(), TEXT("BuccaneerURL="), BuccaneerURL)) I just added a dedicated if "it doesn't get parsed" with the alternative hardcoded URL Il giorno lun 12 feb 2024 alle ore 22:39 Chris Magnacca < @.> ha scritto:
…
After hard-coding it inside the plugin it worked like a charm :) Just a ping, it would be cool as a standalone tool, having grafana data working on console would be top notch. would you mind sharing your modification? I am looking to solve a similar issue. When starting up the Unreal Server it cannot seem to connect to the BuccaneerServer and will timeout and kill the server entirely after 7 attempts. I've verified /metrics functions within the golang server, stats throws the same ID error for me. I appreciate any clarity on this, thanks! image.png (view on web) https://github.com/TensorWorks/Buccaneer/assets/89811181/aa1c866d-0bdf-4b9b-9adf-7a8e0db0dbf6 — Reply to this email directly, view it on GitHub <#7 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACSYLAOGXZJ2LHCE3RYN6UDYTKDY7AVCNFSM6AAAAAA7JFVR4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZZGYZDSOBTHE . You are receiving this because you were mentioned.Message ID: @.>

thanks! i apologize somehow I was trying to edit and my message got removed. I appreciate the info!

I just hardcoded the URL in BuccaneerCommon.cpp
specifically modifying void FBuccaneerCommonModule::Setup()  function.
Here's the snippet, it's nowhere near a good implementation but just a rough test funct.
Cheers!

{
    ParseCommandLineOption(TEXT("BuccaneerEnableStats"), CVarBuccaneerEnableStats);
    ParseCommandLineOption(TEXT("BuccaneerEnableEvents"), CVarBuccaneerEnableEvents);

    if (!FParse::Value(FCommandLine::Get(), TEXT("BuccaneerURL="), BuccaneerURL))
    {
        FString BuccaneerIP;
        uint16 BuccaneerPort;
        if (FParse::Value(FCommandLine::Get(), TEXT("BuccaneerIP="), BuccaneerIP) && FParse::Value(FCommandLine::Get(), TEXT("BuccaneerPort="), BuccaneerPort))
        {
            // build the proper url.
            BuccaneerURL = FString::Printf(TEXT("http://%s:%d"), *BuccaneerIP, BuccaneerPort);
        }
		else
		{
			// Hardcode the URL to localhost and port 8000 if not specified
			BuccaneerURL = TEXT("http://<IP_ADDRESS>:8000");
		}
    }


PyPI version of wagtailtrans does not contain correct middleware

Issue summary
The "getting started" section instructs the user to pip install and add the middleware, but the PyPI version only contains the SiteLanguagesMiddleware class, instead of the class referred to in the docs (TranslationMiddleware).
How to reproduce?
Follow the getting started guide.
Technical details

Python 3.5
Django 1.10.4
Wagtail 1.8

Please keep your PyPI packages in sync with your docs.

 We've implemented this after the 0.1 release and updated the docs just a few days ago, didn't draft a new release yet.
I've published a new 0.1.1 release just now PyPi link

Thanks! Works for me now.


Need blazor specific implementation since the blazor template points to this article

Description

Need blazor specific implementation since the blazor template points to this article

Page URL
https://learn.microsoft.com/en-us/aspnet/core/security/authentication/identity-enable-qrcodes?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/security/authentication/identity-enable-qrcodes.md
Document ID
270bedc7-a636-5bd0-9e2f-e42235deec8c
Article author
@Rick-Anderson

It's true. The cross-link is in the EnableAuthenticator component ...
https://github.com/dotnet/aspnetcore/blob/main/src/ProjectTemplates/Web.ProjectTemplates/content/BlazorWeb-CSharp/BlazorWeb-CSharp/Components/Account/Pages/Manage/EnableAuthenticator.razor#L41
I'll get to this as soon as I can, but it might be mid-April before I can reach it.

btw @wadepickett @Rick-Anderson @tdykstra ... All you have to do for a Blazor issue is add the magic Blazor label 😎 ...
✨ blazor/subsvc ✨
That label kicks off the workflow that sets the rest of the Blazor labels, my assignment, and adds the issue to the Blazor.Docs project.

THANKS, will do.

I'll get to this as soon as I can, but it might be mid-April before I can reach it.

Not a high priority, no hurry. Thanks for taking this.

I'll adopt the same plan for this as I did for account confirmation and PW recovery ...
https://learn.microsoft.com/en-us/aspnet/core/blazor/security/server/account-confirmation-and-password-recovery?view=aspnetcore-8.0
I'll create the article in the Blazor Security node, cross-link it from the main doc set article (Additional resources section), 👉 AND 👈 submit a PU repo issue to have them change that Blazor project template link to point to the new Blazor article.


Feature request: Make mapping possible between XCTestCase & correct StepDefiner: Duplicate steps

We are currently using your library and ran into some trouble.
We have the same step definition in 3 different XCTestCase-classes and a StepDefiner-subclass per testcase.
The library only executes the last definition it found, thus not associating the right automation code with a test.
Request:
Make XCTestCase & StepDefiner mappable to avoid super long steps.
The current implementation as default.
We currently have around + 20 feature files to test, so this would be handy.

Hey - that's a use case we never thought of! We've always just written different step definitions or used regular expressions to solve it.
Can you add an example of what you mean so I can be sure I understand what you're trying to do?

@joridor do you have any examples of this issue? We're not 100% sure what you mean.

@kerrmarin One of the use cases I can think about is a step like:
'the user is logged in'
Depending on where you are in the application, different steps need to be taken to check if a user is logged in or not.
Yes, true, you can try and guess where you are, but this ends up in very long step definition and is harder to maintain.
For now I rewrote your code to have both TestCase level (using a subclass of XCUITestCase) and global level step definitions. (test case level overrides global)

That sounds great, we'd be very happy to review & merge a PR that contains these changes :)

@kerrmarin For now it was directly integrated into this project, because we have a different work flow (Test Cases are generated by build server from feature files, also takes tags in mind).
Happy to fork your code & create a pull request if I have some spare time.


Edit consultation test cases

Add edit consultation test
Add test for non-existing module and tutorial in editStudentCommandTest
Fix sample data error (Empty string for grade throws an exception) -> change to PENDING...

Closes #237

Codecov Report
Base: 59.47% // Head: 63.03% // Increases project coverage by +3.56% :tada:

Coverage data is based on head (9ba3562) compared to base (2848296).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #238      +/-   ##
============================================
+ Coverage     59.47%   63.03%   +3.56%
- Complexity      812      856      +44
============================================
  Files           156      156
  Lines          3252     3252
  Branches        420      420
============================================
+ Hits           1934     2050     +116
+ Misses         1134     1022     -112
+ Partials        184      180       -4

Impacted Files
Coverage Δ

.../java/seedu/address/model/util/SampleDataUtil.java
9.09% <ø> (ø)

...main/java/seedu/address/model/student/Student.java
91.66% <0.00%> (+2.77%)
:arrow_up:

...ess/logic/commands/student/EditStudentCommand.java
92.17% <0.00%> (+3.47%)
:arrow_up:

...seedu/address/model/consultation/Consultation.java
75.00% <0.00%> (+6.81%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
69.23% <0.00%> (+7.69%)
:arrow_up:

...c/main/java/seedu/address/model/commons/Venue.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...n/java/seedu/address/model/commons/ModuleCode.java
80.00% <0.00%> (+10.00%)
:arrow_up:

...ss/model/consultation/ConsultationDescription.java
75.00% <0.00%> (+12.50%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
80.30% <0.00%> (+14.39%)
:arrow_up:

...ess/model/consultation/UniqueConsultationList.java
56.81% <0.00%> (+15.90%)
:arrow_up:

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Take repositories from the API that have more than 2 labels good first issues and show them.

Can you describe it???

NOTE: This repository is no longer supported or updated by @gomzyakov . If you wish to continue to develop this code yourself, we recommend you fork it.


Added hint on state option

Added hint for autocompletion to match documentation

No, because we only add phpDoc for formatters existing in the Base providers. state exists in some locales, but not all.


Allow unused taskRunSpec pipelineTaskName

Feature request
Currently a Task matching pipelineTaskName specified as part of taskRunSpecs must exist in the Pipeline (although no explicitly documented), otherwise the PipelineRun will fail:
spec:
  taskRunSpecs:
    - pipelineTaskName: build-task
      serviceAccountName: sa-for-build

However in some cases, similar to those described in #2513 and #6856, it's useful for this to be permitted, as unused parameters and workspaces are today. Changing the default behaviour would be non-breaking (although possibly slightly surprising if people were relying on the behaviour). Alternatively, an extra boolean field optional could be added (similar to mounting optional Secrets as volumes), defaulted to false. Maybe this behaviour should be extended to workspaces and parameters too, so users can choose the behaviour?
spec:
  taskRunSpecs:
    - pipelineTaskName: build-task
      optional: true
      serviceAccountName: sa-for-build

Use case
Quoting #2513 as a similar use case:

Imagine you have a system creating PipelineRuns automatically, and each PipelineRun uses a different Pipeline which has a different set of parameters. The system has a number of parameters it knows how to provide, e.g. the service account and url of a bucket the Pipelines can push to (but not all Pipelines need to push to a bucket).
It would be very convenient if the system creating the PipelineRuns could provide all the parameters it has available to all PipelineRuns, and the Pipelines could use the ones they need and ignore the rest.

@vdemeester @lbernick What do you think?

I am not sure how it relates to https://github.com/tektoncd/pipeline/issues/2513 and https://github.com/tektoncd/pipeline/issues/6856, but most importantly what would be the behavior for this ?
If I understand correctly, in the example, if there is no task with the pipelineTaskName in the reference pipeline (or the inlined one ?), today we fail, and tomorrow the idea would be not to fail, am I right ?
I am a bit torn, as today, it "could" help users to detect "typos" or issues if a Task got renamed in the pipeline they use. I guess, with the optional: true approach, it's a bit more explicit and thus seems a bit more doable to me.

Yes, the behaviour today if there is no matching pipelineTaskName it fails, I'm proposing either make it not fail (like an unused parameter or an unused workspace) or control the behaviour with an optional: true|false field (which I also prefer).


[Intel Arc] [Windows] Exceptions both with SD 1.5 and SDXL generation

Hello! As the title states, extension does not seem to work correctly on Intel Arc GPUs. Specifically, when I try to run inference with SDXL image and extension enabled, I get the following exception:
File "c:\Forge\modules_forge\main_thread.py", line 37, in loop
    task.work()
  File "c:\Forge\modules_forge\main_thread.py", line 26, in work
    self.result = self.func(*self.args, **self.kwargs)
  File "C:\Forge\modules\txt2img.py", line 111, in txt2img_function
    processed = processing.process_images(p)
  File "C:\Forge\modules\processing.py", line 752, in process_images
    res = process_images_inner(p)
  File "C:\Forge\modules\processing.py", line 938, in process_images_inner
    x_samples_ddim = decode_latent_batch(p.sd_model, samples_ddim, target_device=devices.cpu, check_for_nans=True)
  File "C:\Forge\modules\processing.py", line 638, in decode_latent_batch
    sample = decode_first_stage(model, batch[i:i + 1])[0]
  File "C:\Forge\modules\sd_samplers_common.py", line 74, in decode_first_stage
    return samples_to_images_tensor(x, approx_index, model)
  File "C:\Forge\modules\sd_samplers_common.py", line 57, in samples_to_images_tensor
    x_sample = model.decode_first_stage(sample)
  File "c:\Forge\venv\lib\site-packages\torch\utils\_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "C:\Forge\modules_forge\forge_loader.py", line 239, in patched_decode_first_stage
    sample = sd_model.forge_objects.vae.decode(sample).movedim(-1, 1) * 2.0 - 1.0
  File "C:\Forge\ldm_patched\modules\sd.py", line 288, in decode
    return wrapper(self.decode_inner, samples_in)
  File "c:\Forge\venv\lib\site-packages\torch\utils\_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "C:\Forge\extensions\sd-forge-layerdiffuse\lib_layerdiffusion\models.py", line 256, in wrapper
    y = self.estimate_augmented(pixel[i:i+1], latent[i:i+1])
  File "c:\Forge\venv\lib\site-packages\torch\utils\_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "C:\Forge\extensions\sd-forge-layerdiffuse\lib_layerdiffusion\models.py", line 234, in estimate_augmented
    median = torch.median(result, dim=0).values
RuntimeError: Provided range is out of integer limits. Pass `-fno-sycl-id-queries-fit-in-int' to disable range check. -30 (PI_ERROR_INVALID_VALUE)

With SD 1.5, I run into the following (excerpt):
File "C:\Forge\ldm_patched\ldm\modules\attention.py", line 447, in forward
    return checkpoint(self._forward, (x, context, transformer_options), self.parameters(), self.checkpoint)
  File "C:\Forge\ldm_patched\ldm\modules\diffusionmodules\util.py", line 194, in checkpoint
    return func(*inputs)
  File "C:\Forge\ldm_patched\ldm\modules\attention.py", line 507, in _forward
    n = self.attn1(n, context=context_attn1, value=value_attn1)
  File "c:\Forge\venv\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "C:\Forge\extensions\sd-forge-layerdiffuse\lib_layerdiffusion\attention_sharing.py", line 92, in forward
    framed_cond_mark = einops.rearrange(transformer_options['cond_mark'], '(b f) -> f b', f=self.frames).to(modified_hidden_states)
KeyError: 'cond_mark'

Forge runs on Windows natively, GPU in use is Intel Arc A770 16GB. Happy to provide any other details as necessary.

I can use sdxl model, but using 1.5 model error like you, I use Ubuntu20.02 system, graphics card is rtx4090

my SDXL method work well, however, when using SD1.5, I came into the same ERROR like you (something about 'cond_mark'), did you fix it?

my SDXL method work well, however, when using SD1.5, I came into the same ERROR like you (something about 'cond_mark'), did you fix it?

I have computers that work and computers that don't


The order of tissue_types seems wrong

the order of tissue_types in line 293 map_to_binary.py was:
1: "subcutaneous_fat"
2: "skeletal_muscle"
3: "torso_fat"
Seems it should be:
1: "subcutaneous_fat"
2: "torso_fat"
3: "skeletal_muscle"

This is correct. Thanks for pointing to this bug! I fixed it.


added single quote around .zip

Having build unzip issues due to nested zip in jenkins submission on windows platforms.
Fixes is given here by *adding quote around .zip:- https://github.com/prodroy89/aqa-tests/blob/8af20858dcfa4480057f6075b3c7abd743098db3/get.sh#L313
Signed-off-by: Prodip Roy<EMAIL_ADDRESS>

Ref. Link for nested zip issues :-
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder_CR/24128/
Ref. Links after adding quote around .zip:-
Single runs link:- https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder_CR/24115/
Grinder runs link:- https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder_CR/24394/
Thanks.

Thanks for updating the commit message. However, the PR title has not changed. Please update it.
For future reference, please do not submit PR via GUI and refer to https://github.com/adoptium/aqa-tests/blob/master/Contributing.md for how to contribute to aqa-tests repo.

Hi @llxia , I have done changes title as well, can you please check once? Thanks.

Thanks @llxia


add error-state to tasks

Feature-request
Description
Tasks need additional ERROR-state in case the data-set was broken or another error appeared while processing the task.

moved to https://github.com/kitsudaiki/Hanami-AI/issues/43


.NET MAUI sample status progress

Issues found during this challenge:

https://github.com/dotnet/maui/issues/16428

Other details:

Using XAML Hot Reload during long time is impacting in the performance. The app is invalidating layouts constantly and the refresh time takes more and more time.


Applying edits to an existing procedure

"Editor" refers to the custom procedure editor in this bug.
Questions

Should labels be coalesced?  (See example 1).
Should a label appear when the editor is opened?  (See example 2).
When an input is deleted, should a connected block be deleted or bumped away? (See example 3).
When an input is deleted, should all other connected blocks stick to their respective connections?  (See examples 4-6; I think the answer is yes.)

Example 1
I can add a new text label directly after an existing text label and save the edit.  When I reopen the procedure editor, the two labels have been coalesced into a single label:

Example 2
If my block ends with an input and I open the editor, it always has a new text label at the end.  Deleting the text label is a no-op, and it'll be back when I reopen the editor.  You'll see this repeatedly in the following examples.
If the block does not end with an input, no text label will be added.

Example 3
This block has a text label and a single input named animal, to which I've connected an aardvark.
I open the custom procedure editor and delete animal.
When the custom procedure call updates, the aardvark is deleted.
In Blockly the aardvark would have been disconnected and bumped away, but would have remained on the workspace.

Example 4
Here I have a custom procedure block with a text label and two string inputs.  The first is named mammal and the second is named cephalopod.
I have a custom command block below the definition.  It has two blocks connected.  aardvark is connected to mammal and cuttlefish is connected to cephalopod.
I open the custom procedure editor and delete mammal, which is the first string input.
Expected behaviour: mammal input is deleted, so aardvark is deleted as well.
Actual behaviour: mammal input is deleted.  aardvark is moved into the cephalopod connection.  cuttlefish is deleted.

Example 5
The inputs don't have to be of the same type.  Here's an example with a number input named number and a string input named cephalopods.  The variable that was attached to the number input is moved to the cephalopod input:

Example 6
It even works with a number and a boolean.  The custom command block below ends up with the variable ten attached to the snakes boolean input.

I can't attach the variable ten to the snakes boolean input by dragging.

Blockly behaviour on editing a function:

Answers

Should labels be coalesced? (See example 1).

Yes, this is fine.

Should a label appear when the editor is opened? (See example 2).

No, but low priority.

When an input is deleted, should a connected block be deleted or bumped away? (See example 3).

Bumped away, but low priority.  In the meantime deleting is fine.

When an input is deleted, should all other connected blocks stick to their respective connections? (See examples 4-6; I think the answer is yes.)

Yes.  Don't preserver Scratch 2.0 behaviour, which feels like a bug.

Done in #1225; remaining details/newly found issues should be filed as separate bugs.


release: highlights of v0.0.6

/skip-e2e

/changelog v0.0.6

/changelog v0.0.6

/changelog v0.0.6

/changelog v0.0.6

/release v0.0.6

/release v0.0.6

/changelog v0.0.6

/release v0.0.6

/changelog v0.0.6

/release v0.0.6

/help

ℹ Available commands

/changelog vX.Y.Z will append changelog based on the closed PRs since last release to provided release notes.

/release vX.Y.Z will add release-related commits to this PR on your behalf.

/shipit will rebase everything on master branch and trigger actual release process

/shipit

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀


Unable to understand pdf to xml convert code using pdf2xml

While running the below code in terminal. It is showing "'pdftohtml' is not recognized as an internal or external command,
operable program or batch file."
pdftohtml -c -hidden -xml input.pdf output.xml
Please help.

As mentioned in the README:

At first, you need to convert your PDFs using the poppler-utils, a package which is part of most Linux distributions and is also available for OSX via Homebrew or MacPorts.

For Windows, I don't know how or if it is possible to install poppler-utils.


Discord party plugin disconnection

Describe the bug
Discord party plugin disconnected and stopped updating the party's info. I had to restart the client to get it working again. Toggling the party plugin on and off and joining from discord didn't help.
With reference to the attached log, I logged in around 8:30pm, discord plugin disconnected at around 9:37pm while spamming my log with NPEs.
It's happened before but this is the first time anything showed up in the log. I also found myself logged out of my runelite account when I restarted the client which hasn't happened before.
Any ideas what could cause that?
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.5.18
Launcher version: Unknown

Additional context
client.log discord party bug.txt

Similar issue with the party plugin again yesterday while using it to track specs at corp but this time my log shows a bunch of NPEs from SpecialCounterPlugin instead.
client_2019-04-05.0.log

Party plugin no longer uses discord parties so this is no longer valid.


made changes to readme files for issue #142.

I apologize for the other issues with the other PR's.

No problem @jlozadad. Thanks for your help.


'Add User' followed with 'Deny' breaks Account Plugin UI

Steps taken:

Login to Google Cloud using the toolbar icon.
In the browser, 'Allow'
Click the account toolbar icon again and select "Add Account"
In the browser, 'Deny'

Observe that the toolbar icon has changed to logged out state and is now gray. When you click it, you still see your logged in user from before, but now the "Sign Out" button is disabled. The plugin is effectively tricked into thinking that the user has logged out.

#647 fixed this issue but was not auto-closing.


home.pyot syntax and structure

I like the home.pyot syntax as it stands, but a couple of things I think are worth considering.
I've sorted the issues in descending order of severity.

The syntax for creating devices seems ambiguous. Currently the syntax for naming a light, light1: PhillipsHue() is no different from naming a space, Room1:. That to me seems like poor design.
Actions look like annotations. The way the information architecture / hierarchy is laid out is very reminiscent of the "on X do Y" syntax. This makes me worried because actions should be fundamentally different from annotations meant to taxonomize. This is why I suggested making these hierarchical 'spaces' positional tags with a transitive property. That means to say, if I write #Home{ #Room1{} }, then anything with the #Room1 tag must also have the #Home.
Functional vs Positional tags I've been thinking about what to do with functional tags, and I wonder if an "isA" keyword would be useful. That is to say, you would write something like light1: PhillipsHue() isA #discoLight. This keyword would be a suffix, while all positional tags would be prefixes - so #Home{ #Room1{ light1 = new PhillipsHue() isA #discoLight } }. We might consider establishing an explicit syntactic difference between a positional tag and a functional tag.
Explicit exports. Would the programmer need to explicitly export anything? If not, is this because we can manually do it during the interpretation?

@riamundhra02 please let me know your thoughts

right i agree, maybe something like light1: new PhillipsHue() might work better?
2 and 3. i also agree with this, looking back. I quite like the isA syntax, disambiguates the tags p well. My only concern with the positional tags/annotations is, would this be straying away from the YAML syntax too much?
Honestly, i didnt rly think about this v hard when writing the syntax, but i do think its an important thing to consider. if were allowing files to be separated or combined, i thought imports would work like in python, where you can essentially import any python file into another w/o any explicit export statements? but am open to other ideas as well


anyword hinting

show-hint and anyword hinting doesn't registering event with codemirror instance
on log gives error console.log(CodeMirror.hint.anyword)
Error:
TypeError: Cannot read property 'anyword' of undefined

Code? I have no idea what to make of this as-is. I suspect this isn't an issue with this lib but an implementation error you are encountering. Is Any Word Completion Demo example what you are trying to accomplish?
I suspect asking on stackoverflow will yield better results for you

editor.on('keyup',function(cm,event){
if(!cm.state.completionActive && (event.keyCode||event.which) != 13 && (event.keyCode||event.which)>64 && (event.keyCode||event.which)<91 && !event.altKey && !event.ctrlKey &&!ExcludedIntelliSenseTriggerKeys[(event.keyCode || event.which).toString()] ){
cm.showHint({hint:CodeMirror.hint.anyword, completeSingle: false, alignWithWord:false});
        }
    })

giving error cannot read property anyword of undefined
i have already imported anyword-hint.js file
i think hint is not getting register with codemirror

@shubhamsony why are you not using the onKeyUp component event? It's almost as if you're wanting to use the raw library instead. I'm going to close this out as this is clearly trending towards debugging your use-case and not an underlying issue with this component lib.


⚠ api.piste.gouv.fr healthcheck has degraded performance

In 11e376a, api.piste.gouv.fr healthcheck (https://api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 2955 ms

Resolved: api.piste.gouv.fr healthcheck performance has improved in a3421f5.


Tasks initialized by a HyperParameterOptimizer (on an agent) does not upload artifacts to fileserver

When executing a hyperparameter sweep where the optimization process is run as a service, spawned subtasks does not upload artifacts correctly to the file server.
Instead, we get a path to a local directory on whatever machine runs the agent like file:///data/clearml/venvs-builds/3.9/task_repository/our-repo/logs/ instead of http://clearml.our-server.com:8081/...
I have followed the official guide on optimization, with the exception that I create the base task as a draft instead of actually running it.
Package versions:
clearml @ 1.9.3
clearml-agent @ 1.5.1

@mads-oestergaard did you specify the output_uri parameter of Task.init()? (or, alternatively, set up the sdk.development.default_output_uri setting in your clearml.conf file)
If these are not specified, models and artifacts will by default be stores into the local storage.

The optimization task was not initialized with output_uri set, but it does seem to upload plots correctly:
# Initialize the clearml task
    task = clearml.Task.init(
        project_name=cfg.clearml.project_name,
        task_name="Automatic Hyper-Parameter Optimization",
        task_type=clearml.Task.TaskTypes.optimizer,
        reuse_last_task_id=False,
    )

I created the base task like this:
def generate_base_task_id(experiment: str) -> str:
    """Create a task (in draft) and return the ID of the created task"""
    task: Task = Task.create(
        "Optimization/base_tasks",
        task_name=experiment,
        script="src/hydra_train.py",
    )
    task.connect({"+experiment": experiment, "project_name": "tmp"}, "Hydra")
    return task.id

Task.init() sets the output_uri parameter in hydra_train.py. Could it be caused by add_task_init_call being True by default in Task.create()?

Hi @mads-oestergaard ! Note that the optimizer task and the task you want to optimized are treated completely separate. So any change you do in the optimizer (like adding an output_uri) will not reflect in the task you want to optimize.
I suggest you initialize the task you want to optimize with Task.init, and specify the output_uri in that init call.
Or, you could try setting the CLEARML_DEFAULT_OUTPUT_URI env var to the desired output_uri, or set sdk.development.default_output_uri in your clearml.conf.


cadvisor: 0.38.7 -> 0.40.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cadvisor/versions.
meta.description for cadvisor is: "Analyzes resource usage and performance characteristics of running docker containers"
meta.homepage for cadvisor is: "https://github.com/google/cadvisor"
meta.changelog for cadvisor is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0/bin/cadvisor passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 0.40.0 with grep in /nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0
found 0.40.0 in filename of file in /nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
cadvisor
docbookrx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cadvisor https://github.com/r-ryantm/nixpkgs/archive/0181cd9654ad3cc56277907af942fcff095957fa.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0
ls -la /nix/store/ysn74c6q5j74hjphikwwfk7aa6388jdj-cadvisor-0.40.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cadvisor

Maintainer pings
cc @offlinehacker @kalbasit for testing.

@GrahamcOfBorg test cadvisor

test was happy


Running zuul-netflix-webapp

Hello there,
I have set up Eureka server (checked with sample server and client) for Zuul.
For Running zuul-netflix-webapp, I have following questions:

What changes do I have to do in zuul.properties file in zuul-netflix-webapp\src\main\resources folder?
Is there any other place I need to make any change?
I am providing the IP address of the server where Cassandra is running. Is anything else expected?
I have Eureka (server and service) and Zuul running on different machines. For eureka service running on the Eureka Server, the eureka.serviceUrl.default property reads as http://localhost/eureka/v2. But the same property in zuul.properties reads as http://hostname/eureka/v2. Is that correct?

With all these changes, I am not able to see any integration between zuul and eureka happenning.
Overall, I want to understand the architecture diagram of the Zuul-Eureka-Cassandra together and a case study which describes a minimum set up of Zuul required with Eureka and Cassandra.

Projects like RSS recipe, acmeair have since described the architecture used in Netflix with Zuul, Eureka, Cassandra and I think the wiki here describes how to use the sample app. Closing this issue.


Upgrade to .Net Standard 2.0

target all projects to .net standard 2.0
move tests proj to .net 472

Hello Kent @kentcb ... are you considering this pull request?

@sujitn Sorry, I completely missed this. Thanks for the prompt @ahmedatia.
Just so I'm clear, what's the actual advantage of upgrading from NS1 to NS2? Are there any use cases that are unlocked by that?

Cannot use data extension in .net core projects till we upgrade to ns2

Thanks @kentcb ... Yep! NS2 introduced more API surface comparing to NS1 and this makes it much easier to port projects to / target .NET Core.

Thanks @sujitn !


[mysql-async] [ERROR] ER_NOT_SUPPORTED_AUTH_MODE: Client does not support authentication protocol requested by server; consider upgrading MySQL client`

Describe the bug
I'm getting this when trying to connect to mysql-async:
[mysql-async] [ERROR] ER_NOT_SUPPORTED_AUTH_MODE: Client does not support authentication protocol requested by server; consider upgrading MySQL client
To Reproduce
I'm using this in server.cfg:
set mysql_connection_string "server=localhost;database=portugalrpdb;userid=root;password=passpass"
start mysql-async

I'm using MySQL Workbench, instead of XAMPP, because that's what I'm more used to, but I'm still a beginner.
Expected behavior
To connect to MySQL server. I don't know. I'm stil learning.
Screenshots / Logs

Software:

OS: Windows 10
Version of the resource: I downloaded it just today so I assume it's the latest recommended.
Database Version: I don't know, but I downloaded it from MySQL website.

Additional Information
I'm not using ESX.

Fix:
I was with MySQL 8.0, I uninstalled it and installed MariaDB and it's now working.

Yea depending on updates they release things like database sql software
needs updating so you work with a lower version now it will run super slow
at loading stuff
On Tue, 22 Jun 2021, 23:16 Almeidowski, @.***> wrote:

Fix:
I was with MySQL 8.0, I uninstalled it and installed MariaDB and it's now
working.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/brouznouf/fivem-mysql-async/issues/177#issuecomment-866373543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALZKU25UNKH5W2HJC3Y3IRLTUEDTLANCNFSM47EQ25PA
.


feat: Collection filtering on top of profile

PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #8384

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI;

Copilot Summary

🤖[deprecated] Generated by Copilot at b11a9aa
This pull request adds a new feature to filter NFTs by collections on the profile page. It introduces a new CollectionFilter component that uses a NeoDropdown with checkboxes and an ActiveCount badge. It also updates the CSS classes and variables for the filter and the ActiveCount component, and adds the image field to the collectionDetails fragment.

🤖[deprecated] Generated by Copilot at b11a9aa

We filter the NFTs by collections
We show the logos and the counts
We adjust the colors and positions
We unleash the CollectionFilter power

seeing draft, clicking on it lol

can you make the dropdown wider please

seeing draft, clicking on it lol

can you make the dropdown wider please

I was  fixing that

one more thing

active button should have hover white

so the stroke is not visible

is it possible for it not to go to the top of the page every time we are messing with the filters?
its also a bit buggy if you chose two filter and scroll down:
https://github.com/kodadot/nft-gallery/assets/36627808/65217b03-48f5-46d7-aa26-8a098462c17e

active button should have hover white

link #8362

I think we may need some scrolling / height limit:

if i mess with it too much:

collections taking around 10s to load and button to get active, maybe loader state?

filtering by collections and using sortBy filters resets everything

made some changes, pls review thanks

Something weird with numbers on https://deploy-preview-8414--koda-canary.netlify.app/ahp/u/15CoYMEnJhhWHvdEPXDuTBnZKXwrJzMQdcMwcHGsVx5kXYvW?collections=&tab=created
Check created tab show 0, wen click error throw

can you provide more info , couldn't reproduce

made some changes, pls review thanks

Something weird with numbers on https://deploy-preview-8414--koda-canary.netlify.app/ahp/u/15CoYMEnJhhWHvdEPXDuTBnZKXwrJzMQdcMwcHGsVx5kXYvW?collections=&tab=created
Check created tab show 0, wen click error throw

can you provide more info , couldn't reproduce

fixed

@roiLeo fixed https://github.com/kodadot/nft-gallery/pull/8414/commits/45e237c0f9710cdbc6c140e941c28071e9eeb8fe

@prury try again thanks

oh, its getting nice!

i would try to take care of horizontal scroll:

filtering only works first time, if you try to filter again nothing happens

after refreshing or changing network filters stay on, maybe clear they?

maybe center modal on mobile:

@prury

filtering only works first time, if you try to filter again nothing happens

not sure if fixed or not, couldn't reproduce it

after refreshing or changing network filters stay on, maybe clear they?

fixed
neo dropdown issues

i would try to take care of horizontal scroll:

maybe center modal on mobile:

these two are specific of the dropdown component itself

*can make followup if its too much

yes please , create a new issue so we can address them separately.
thanks

@prury

filtering only works first time, if you try to filter again nothing happens

this one we will have to fix here, otherwise it will introduce bug since its working properly on canary
(sorting only works after you reload collections)
https://github.com/kodadot/nft-gallery/assets/36627808/60530b23-e0c3-419e-932d-1507dc5f5475

@hassnian let's resolve conflicts and we will merge it
Thanks!
pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 7.27 USD/DOT ~ 6.878 $DOT
🧗 13QUj3pZyFNfYj4AM336hRdyLQbevj5H3sR4PKmLEXLdwZhh
🔗 0x58f8311fafb17a0bab4fdd13f753c0a7e43d05c5b1a9ad89f73aa6b89eaa2530
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

ready

pleasure! thank you!


Add contributors

@all-contributors please add @itsmontoya for code, doc

@all-contributors please add @matthew-stay for design

@all-contributors please add @chewxy for tests

@all-contributors please add @jbmuir for tests


Added more functions, mainly for usage in apps

Codecov Report

Merging #8 (1b00279) into main (5b2b053) will decrease coverage by 10.22%.
The diff coverage is 12.50%.

@@             Coverage Diff             @@
##             main       #8       +/-   ##
===========================================
- Coverage   44.00%   33.79%   -10.21%
===========================================
  Files           3        3
  Lines          50       74       +24
  Branches        6        8        +2
===========================================
+ Hits           22       25        +3
- Misses         28       49       +21

Impacted Files
Coverage Δ

src/crowdin.ts
13.47% <12.50%> (-0.82%)
:arrow_down:

Continue to review full report at Codecov.

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


ShapeList containing different object types fails to render

~This silently fails to render.~ I had some modifications to my b3d install. It does render, but incorrectly.
x = ShapeList([Circle(1), Pos(2)*Sphere(1)])
show_all()

The list is valid, however, as the below works and renders.
x = x.faces()
show_all()

Again, why do you use ShapeList for this? Why not a simple list?

ShapeList is just a list which also provides many CAD utility functions so whenever I have a list of CAD objects that I may want to perform queries on ShapeList > List. For example, I want my functions to return ShapeLists so I can do project(...).faces() rather than ShapeList(project(...)).faces().

Got it, thanks
I now support generic ShapeLists (as opposed to the homogenous ShapeLists being returned by faces(), edges(), ...). It'll come with the next release in the coming days

Just posted on Discord the newest pre-release.
If you would have some time, I would really appreciate if you could give it a try - thanks :-)

Yes, it works now, thanks!


Advanced search: Cannot find advisor position if "Position type" is used as a filter

Success: Use Advanced search to find an advisor position without any filters.  Enter the advisor position code in the "Search term" field.  The advisor position is found.
Failure: Use Advanced search to find an advisor position and use the "Position type" filter.  Enter the advisor position code in the "Search term" field.  The advisor position is not found.

Reproducible.

@dspCarlson It probably means that the advisor position is a Super User or an Administrator. The search does an exact match. Question is: what should happen in this case? Should searching for NATO Billet also imply searching for Super User and Administrator?

@gjvoosten Just re-tested based on your comments.
I tested a super user's billet position for both conditions.  Indeed super users and administrators are not found when selecting "Advisor Billet" as a "Position type" filter.
Super users and administrators should be included in search results when the "Position type" filter is set to "Advisor Billet."


fix(SDK): controller retrieval and touch events in simulator

When trying to develop without any VR-Hardware, the simulator comes in really handy. However, I noticed that some events on the simulator (e.g. touchdown, touchup, touched on any buttons) are never fired. While it makes sense to not fire those events from a technical perspective (since we cannot distinguish between press and touch on a keyboard) it makes developing a whole lot harder, since a developer potentially has to use different types of events during development (Simulator) and in production (HMD).
Furthermore, the Simulator SDK right now does not allow for finding script alias controllers. Whenever the calls to get a controller (e.g. GetControllerLeftHand or GetControllerByIndex) are used, the boolean actual parameter is simply ignored and the actual controllers are always returned. This is not in line with the base implementation or the implementation for SteamVR.
This PR mitigates both the above problems.

Touch Events are now always fired alongside with pressed / released events
Controller Retrieval / Finding logic has been reworked to be more consistent with other implementations

There were the following issues with your Pull Request

Commit: ffd2c10e6cfc6bffd1598bfa25161f900007f24d

Your commit message body contains a line that is longer than 72 characters

Commit: ec004478e3379c2abfddcfe5b2628b8977ee0fa0

Your commit message body contains a line that is longer than 72 characters

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

@burnedikt  is it the best thing to do to map the touch just to the press? would that help at all? or would it be better to map different buttons to the touch instead?

@thestonefox Intuitively I'd say another key would be overkill, since naturally, a touch inevitably has to occur before a press can ever be detected. One cannot press a button without first touching it.
However, I see your point of having a possibility to trigger the touch events separately during development to test specific or timed behaviour (e.g. an Animation which starts onTouchStart and ends onPressDown would be hard to test, if the events occurred in such quick succession).
So in my opinion:

Touch-Events should occur naturally together when a button pressed (Principle of least astonishment)
You're right in that a way of just having the touch event is required as well. If someone just lightly touches a button, it'll never be pressed.

In order to avoid adding separate Touch Keys for all mapped buttons, I'd suggest we add a modifier key, which - when pressed together with the actual button - opts out of the pressing feature and just fires the touch events.
What do you think?

@burnedikt yeah @virror also suggested a modifier, it could be the way to go..
What do you think @virror  ?

I like it. I think its a good addon to as you said to make the button trigger when you let go of the modifier to better emulate the real behavior. Im all for it.

Just pushed the update:

When Pressing T (TouchModifier) + any Button (e.g. Mouse0), only touch events will be fired
When Pressing H (HairTouchModifier) + any Button (e.g. Mouse0), only touch and "hair" events will be fired
When Pressing none of the Modifiers and just a Button (e.g. Mouse0), all events will be fired

In any case, I also implemented event ordering, since a press cannot occur before a touch. The order right now (see https://github.com/thestonefox/VRTK/pull/831/commits/9732473409c1a7f3262e038cf94693d5ca8ee45e) is (where events in brackets are only fired for some buttons)

(touchDown)
(hairlineDown)
pressDown
(click)
pressUp
(hairlineUp)
(touchUp)

One issue with the implementation of https://github.com/thestonefox/VRTK/pull/831/commits/bea4f0811ad995ee31ee3cc986d3f7f1c3cc2341 is:
If someone wants to have the trigger event and holds down Mouse0 accordingly, then notices midway that he actually just wanted touch events and forgot to hit the TouchModifier T as well, then hits and holds T while releasing Mouse0, he will get the touchDown, hairDown, pressDown events but only touchUp in the end. Vice versa, if the Modifier Key is released too early, the full event lifecycle will be fired as soon as the button (Mouse0) is released.
So in general, the modifier keys need to be pressed down before the actual key and released after the actual key to get the expected results.

Is this commit:
https://github.com/burnedikt/VRTK/commit/11120f2bf49e3eaca41e3f3fcea4dc5a0efaab89
still required? as it's overriden by a newer commit anyway?
If you can rebase and remove it without breaking anything and still have a pull request that works then that would be cool.
If it's a lot of hassle then i'm sure it can be overlooked ;)

rebased

Looks nice : )


(Un)focus without closing using keyboard

Would be nice to be able to leave the terminal open and refocus editing panes from terminal or terminal from editing panes, all with the keyboard.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/33400989-un-focus-without-closing-using-keyboard?utm_campaign=plugin&utm_content=tracker%2F31220268&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F31220268&utm_medium=issues&utm_source=github).

Yes, please consider adding this feature. As a workaround, I've been doing both of these:

type ctrl-` to toggle terminal open/close (focus goes back & forth)
type ctrl-00 from the open terminal to bring focus back to editing pane (only one way)