Running Scheduler with Plesk?

Hi all,
I'm trying to run the scheduler with Plesk but I'm not successfull. Would it work with plesk?
I've removed the "crontab"-commands from the line shown in Tools -> Scheduler, e.g. with your example on https://learn.getgrav.org/16/advanced/scheduler it would be
echo "* * * * * cd /Users/andym/grav;/usr/local/bin/php bin/grav scheduler 1>> /dev/null 2>&1"
but that does not seemt to work. The line is executed without errors, showing green checkmark. But in Grav Admin at Tools -> Scheduler, no jobs were started :-(
Is there a way to run the scheduler with a scheduled task in Plesk?

I have a similar issue. I am using hosting at netcup which also uses Plesk. However I am debugging it via ssh. I am running ./bin/grav scheduler -j -vvv but no output is shown. -V returns Grav CLI Application 1.6.31. On my local PC everything works fine, I am currently trying to find the issue. I guess it is not a problem with Plesk itself, but rather grav or the hosting settings.

I have a similar issue. I am using hosting at netcup which also uses Plesk. However I am debugging it via ssh. I am running ./bin/grav scheduler -j -vvv but no output is shown. -V returns Grav CLI Application 1.6.31. On my local PC everything works fine, I am currently trying to find the issue. I guess it is not a problem with Plesk itself, but rather grav or the hosting settings.

In my case it turns out that firing the onPluginsInitialized event fails. However I do not yet know why, as I do not have any error message. It was just a try-error via grav logging output. Any ideas on how to get more information when using the cli?
https://github.com/getgrav/grav/blob/develop/system/src/Grav/Console/Cli/SchedulerCommand.php#L63

In my case it turns out that firing the onPluginsInitialized event fails. However I do not yet know why, as I do not have any error message. It was just a try-error via grav logging output. Any ideas on how to get more information when using the cli?
https://github.com/getgrav/grav/blob/develop/system/src/Grav/Console/Cli/SchedulerCommand.php#L63

@NicoHood Redirect should be fixed?

I think so. I did not test this in my production environment yet.

Closing this issue, let me know if you get it again.


Move content of kuberbacproxy into vali component

How to categorize this PR?

/area dev-productivity
/kind cleanup
What this PR does / why we need it:
kube-rbac-proxy and also valitail actually belong to the vali component since kube-rbac-proxy is actually a sidecar container in the vali StatefulSet and no separate/dedicated deployment. Back then (when kuberbacproxy component was introduced) the Vali deployment was still managed via Helm charts, but since recently this has been refactored to Golang (ref #8061), so we can finally move the code now.
Special notes for your reviewer:
/cc @shafeeqes @acumino
Release note:

NONE

/assign

/retest


Query Data page needs a re-think now that there are two ways to query data

URL for relevant page?
https://docs.influxdata.com/influxdb/v1.8/guides/query_data/
I think we need to intro this section differently now that there are two ways to query data.  Need to intro Flux here and the associated HTTP v2 API... pointers to the forward compatibility content.

@noramullen1 can you propose some changes and let's review with @sanderson and myself?

fixed by #2771


Reproduce PFS routing issue from EthCC with DApp Node their service.

During the EthCC event we had issues with the DApp Node hosted PFS. We think to have found the cause and that should be fixed soon. Nevertheless we need to reproduce the issue to test if it got solved or not.
The setup we had was composed of three LCs. One as the sender, a second as mediator and a third as receiver. Each the second and third node must have a UDC deposit to enable receiving transfers. It requires a channel from 1 to 2 with a deposit by 1 and another channel from 2 to 3 with a deposit of 2.
Now try to do a mediated transfer from 1 to 3 (they must not have a channel with each other) and select the DApp Node their PFS (the cheapest). Verify that you get a route for the payment. As all  three nodes are freshly setup, there must be no other route than 1 - 2 - 3.

@weilbith I am guessing this is on goerli ? Just to confirm

I think the dAppnode PFS runs on mainnet

Yes. Unfortunately this is mainnet.
@palango a PFS can't handle multiple networks at the same time, right?

@palango a PFS can't handle multiple networks at the same time, right?

No, not currently

@agatsoh it is fine to re-use accounts with existing channels. Just order them correctly and make sure there is no other route available (most simple situation would be if there is only one per node). So then you don't even need to pay any ETH.

Yeah I already have them all setup would have to clear the database in the browser and then I am good to go.

raiden_0x07d036DbC602F0191a4bC8e217fe48d34cD4a230_2021-08-04T14_25_47.536Z.log
Check the last path/find/failure

Did you ask Ulo if the whitelist was updated?

@agatsoh this path/find/failure is only the check for the direct route, NOT the actual PFS error. Remember the dApp first checks if there's a direct route, and if not, it'll fetch the PFS list, offer them to user, and then will do the actual request to it. From your logs, no 2nd request is seen, meaning for some reason this actual PFS didn't happen. Not sure why though, something not logged, maybe no valid PFS was found? In your Network tab in browser dev tools, you could check if there're any failed PFS/info request.

@andrevmatos
I ran it once again to check after we select the dappnode pfs what transpires. But I could not find in /info request.
2021-08-05T09:03:05.808Z #12964035 [info]   => %c action
{
  "type": "path/find/failure",
  "payload": "RaidenError: Pathfinding Service is disabled and no direct route is available.\n    at vr (http://localhost:8080/js/app.03a0d46c.js:1:118621)\n    at t.project (http://localhost:8080/js/app.03a0d46c.js:1:121387)\n    at t._tryNext (http://localhost:8080/js/chunk-vendors.29caa979.js:16:17754)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:16:17656)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:102:77820)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10893)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:102:34538)",
  "meta": {
    "tokenNetwork": "0xEA92bD481e54BBB9Bb6C927970a54E82eE26c6B4",
    "target": "0xF087038D9032dc5e419C62dF32e865C2d160db2e",
    "value": {
      "_hex": "0x0de0b6b3a7640000",
      "_isBigNumber": true
    }
  },
  "error": true
}

2021-08-05T09:03:23.376Z #12964037 [info]   => %c action
{
  "type": "path/find/request",
  "payload": {
    "pfs": {
      "address": "0x60eAE5572D7094B63cA8C854eE8132EDc65570b3",
      "url": "https://pfs.raiden.dappnode.io",
      "rtt": 1171,
      "price": {
        "_hex": "0x64",
        "_isBigNumber": true
      },
      "token": "0x255Aa6DF07540Cb5d3d297f0D0D4D84cb52bc8e6"
    }
  },
  "meta": {
    "tokenNetwork": "0xEA92bD481e54BBB9Bb6C927970a54E82eE26c6B4",
    "target": "0xF087038D9032dc5e419C62dF32e865C2d160db2e",
    "value": {
      "_hex": "0x0de0b6b3a7640000",
      "_isBigNumber": true
    }
  }
}

2021-08-05T09:03:23.381Z #12964037 [info]   => %c action
{
  "type": "matrix/presence/request",
  "meta": {
    "address": "0xF087038D9032dc5e419C62dF32e865C2d160db2e"
  }
}

2021-08-05T09:03:23.387Z #12964037 [info]   => %c action
{
  "type": "matrix/presence/success",
  "payload": {
    "userId": "@0xf087038d9032dc5e419c62df32e865c2d160db2e:transport.raiden.badgateway.tech",
    "available": true,
    "ts":<PHONE_NUMBER>310,
    "caps": {
      "Receive": 1,
      "webRTC": 1,
      "Delivery": 0,
      "Mediate": 0,
      "toDevice": 1,
      "immutableMetadata": 1
    },
    "pubkey": "0x041cf4346c6c392909a87da6d4d28fb8ce080c46c222ffe4e787c0b23a92d49b9516e2d70ef98e43d7785a3d858592f9b87b735f020652054f4b27675d1d132c84"
  },
  "meta": {
    "address": "0xF087038D9032dc5e419C62dF32e865C2d160db2e"
  }
}

2021-08-05T09:03:24.342Z #12964038 [info]   => %c action
{
  "type": "iou/persist",
  "payload": {
    "iou": {
      "sender": "0x07d036DbC602F0191a4bC8e217fe48d34cD4a230",
      "receiver": "0x60eAE5572D7094B63cA8C854eE8132EDc65570b3",
      "chain_id": {
        "_hex": "0x01",
        "_isBigNumber": true
      },
      "amount": {
        "_hex": "0x64",
        "_isBigNumber": true
      },
      "one_to_n_address": "0x960f01E9E219E1c8564784a050fD1cA51651592D",
      "expiration_block": {
        "_hex": "0xc8de06",
        "_isBigNumber": true
      },
      "signature": "0x3c260504a7c02968aa455616abb2f024558192bb773c81a33ee0ea6defa0bbb353f5a40302de00148b238c14d61196fc29a1c31d3937189aeac779d336ab22561c"
    }
  },
  "meta": {
    "tokenNetwork": "0xEA92bD481e54BBB9Bb6C927970a54E82eE26c6B4",
    "serviceAddress": "0x60eAE5572D7094B63cA8C854eE8132EDc65570b3"
  }
}

2021-08-05T09:03:24.355Z #12964038 [info]   => %c action
{
  "type": "path/find/failure",
  "payload": "RaidenError: No route between nodes found.\n    at Or (http://localhost:8080/js/app.03a0d46c.js:1:118992)\n    at t.project (http://localhost:8080/js/app.03a0d46c.js:1:121497)\n    at t._tryNext (http://localhost:8080/js/chunk-vendors.29caa979.js:16:17754)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:16:17656)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:102:77820)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)\n    at t.notifyNext (http://localhost:8080/js/chunk-vendors.29caa979.js:16:18170)\n    at t._next (http://localhost:8080/js/chunk-vendors.29caa979.js:102:104463)\n    at t.next (http://localhost:8080/js/chunk-vendors.29caa979.js:97:10564)",
  "meta": {
    "tokenNetwork": "0xEA92bD481e54BBB9Bb6C927970a54E82eE26c6B4",
    "target": "0xF087038D9032dc5e419C62dF32e865C2d160db2e",
    "value": {
      "_hex": "0x0de0b6b3a7640000",
      "_isBigNumber": true
    }
  },
  "error": true
}

The 2 path/find/failure
first time when we click on continue
RaidenError: Pathfinding Service is disabled and no direct route is available
second time when we actually pick that PFS we get a
No route between nodes found.

Nothing happened on this for a while and closed with #2924 (see second comment).


Fix issue with alias remove for large templates

Attempt to fix this issue: https://github.com/HyperBrain/serverless-aws-alias/issues/107
It seems to work, as far as I can tell, but if it's way off base, please let me know.

@HyperBrain I'd really like to get this in the next release, if possible


Debugger does not use global presets to run in console's namespace

I recently updated Spyder to 6.0.1 and have the following issue:
My configuration for the debugger and regular ipython console are set like this:

However, when debugging a file, what gets executed in the console is:
%debugfile 'Path/To/File.py' --wdir
instead of doing like when running a file and keeping my variables:
%runfile 'Path/To/File.py' --wdir --current-namespace
Manually adding the --current-namespace to the wrong command does make it work properly.

Hey @Arthanor, thanks for reporting. This is basically the same problem as the one reported in issue #22496, in the sense that we're passing the global or per file debugging options set by users to the debugfile command.
We'll try to fix that in our 6.0.4 version, to be released at the beginning of next year.


Logging tweaks

Increase the log severity for some major events, and also notify of Throttling/Deduping process.

Coverage increased (+0.1%) to 74.876% when pulling ac97ee5aab649238dba7d6b6203470df974138cd on vend:vend_logging_fixes into b292e7e6b630bd7ad06e227111448222389d4157 on mdevilliers:master.

Awesome - thanks.


Unit test expected values do not match instructions!

There are a few unit tests that are expecting values not outlined in the README. Specifically, in the following test, the expectation of a roster size equal to 5 does not match the instructions calling for a size of 10.
        WHEN("we assess whether they have declared the proper ROSTER_SIZE") {
            int rosterSize{ROSTER_SIZE};
            THEN("its found to be the expected size") {
                int EXPECTED_ONE_D_ARRAY_SIZE{5};
                REQUIRE(EXPECTED_ONE_D_ARRAY_SIZE == rosterSize);
            }
        }

Furthermore, the following test correctly mentions the expected name in the WHEN clause but it is expecting something else (grades) in the WHEN code block.
        AND_WHEN("we assess they have declared a one-dimensional array named testGrades") {
            const char *filename = "csc232.h";
            std::ifstream input_stream{filename, std::ios::in};
            if (!input_stream)
                FAIL("Could not open csc232.h while evaluating Task 1!");
            const char *search_string = "grades";
            // Read the entire file into memory
            std::string file_contents;
            std::string current_line;
            while (getline(input_stream, current_line))
                file_contents += current_line + '\n';
            input_stream.close();
            size_t pos = file_contents.find(search_string);
            THEN("we find it to be declared as specified") {
                REQUIRE(static_cast<int>(pos) > 0);
            }
        }

Please ensure that the instructions are synchronized with the unit tests.

For students that have this issue, please do the following:

For TODO 1.1 set the size to 5.
For TODO 1.2 set the name of the array to grades (as opposed to the instructions that tell you to name it testGrades


jenkins.sh fixes

Various fixes to the script.  See individual commits for details.

If we're approaching a good setup here, could you consider adding these to fix https://github.com/alphagov/govuk-rails-app-template/issues/6 ?

If we're approaching a good setup here, could you consider adding these to fix alphagov/govuk-rails-app-template#6?

Sounds like a good plan.  I want to have this running in the wild for a little bit before doing so though, just to iron out any issues.


diseases should not be at root level

This was a consequence of the obsoleting "rare disease", we should find a home for these

For some of these, could we just we just reclassify them (assert subclasses), like:

ID
Label
Current parent
Suggested parent

MONDO_0019103
'benign exophthalmos syndrome'
'disease or disorder' (source is orphanet)
'syndromic disease'

MONDO_0010947
'Budd-Chiari syndrome'
'disease or disorder' (source is orphanet)
'syndromic disease'

MONDO_0017388
'celiac trunk compression syndrome'
'disease or disorder' (source is orphanet)
'syndromic disease'

There are more, these are just some examples

I should review these in Orphanet and change the subClassOf assertions

questions:

[ ] 'cell proliferation disorder' is inferred to be a subclass of 'disease or disorder' because of its logical def


nerdctl: 0.8.1 -> 0.8.2

Motivation for this change
Bump nerdctl to 0.8.2
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 123794 run on x86_64-linux 1

1 package built:

nerdctl


404 of "paper.html" and "header.html" when rendering ng-table

environment: Webpack3 + ES2015 + ng-table 3.0.1 + angular 1.6.5

I googled it and found some similar issues but they didn't work.  Is it a bug or my environment issue?

Yeah i got this error too. Can someone explain how to fix this?
When i use ver 2.2.0 (https://unpkg.com/ng-table@2.0.2/bundles/ng-table.min.js), it's work without error.
Thanks

Thanks for sharing! @aldoetobex  Did you mean include ng-table through

The solution is to edit directives in the src/browser/ and replace all occurrences of templateUrl with template so webpack will load the templates as inline string rather than external resource.

I found a solution that doesn't require editing the ng-table code. Details here: https://github.com/esvit/ng-table/issues/969#issuecomment-703458126. Summary: install the ngtemplate-loader and html-loader NPM packages, then put this in your webpack.config.js (example is for Webpack 3.x, hopefully should work in 2.x or 4.x with possible adjustments to where the line goes in the config):
    module: {
      rules: [
        // ...
        { test: /ng-table\/.*\.html$/, use: ['ngtemplate-loader?requireAngular&relativeTo=/src/browser/&prefix=ng-table/', 'html-loader'] }
      ]
    }


Showing hints when long-tap actions are single-tapped

https://user-images.githubusercontent.com/17034768/132106615-d391f2ad-0cb5-4582-8b1e-87c2a0df2693.mp4

Ha, that's made intentionally. Long-press for deletion, but we need to show hint when it is tapped.

Ah, I see! Yeah we need to show a hint :)

Can you assign it to me?

@vishal2376 Hi! Feel free to fork and submit the PR. Nobody is working on it right now.


Question about Ref

Hey,
I have been trying to follow the advanced examples for falcor. I am trying to recreate the "all" route but I am running into odd results. I am not sure if it is a bug with the falcor.net project or my client code. Here is what I have.
Any ideas what I am doing wrong?
In NetflixFalcorRouter
Get["titles[{ranges:indices}]"] = async parameters => {
	List<int> titleIds = parameters.indices;

	var ratings = await ratingService.GetRatingsAsync(titleIds.Select(x => Guid.NewGuid()), userId);
	var results = titleIds.Select(idx => {
		var rating = ratings.ElementAtOrDefault(idx);

		return Path("titles", idx)
			.Ref("titlesById", rating.TitleId.ToString());
	});

	return Complete(results);
};

Client index.html
model.get('titles[0..2]["rating", "userRating"]').then(logJson, logJsonError);

Output
{
   "json": {
      "titles": {
         "0": {
            "\u001epath": [
               "titlesById",
               "aca9317a-d813-4e35-8f22-428652c495c8"
            ],
            "rating": 0,
            "userRating": 0
         },
         "1": {
            "\u001epath": [
               "titlesById",
               "f40b6c24-be45-4969-b541-2f4491d1300a"
            ],
            "rating": 4,
            "userRating": 0
         },
         "2": {
            "\u001epath": [
               "titlesById",
               "656f9ed3-2973-4756-9f82-5d286244dfc7"
            ],
            "rating": 4,
            "userRating": 2
         }
      }
   }
}

To make things more confusing. When looking at the network traffic, I am getting errors. By it resolves somehow? Not really sure.
{
    "jsonGraph": {
        "titles": {
            "0": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "b697c926-cfd8-4564-8025-7252a3ec4599"
                ]
            },
            "1": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "7b833be2-ef51-46f4-9de6-77c427b8dce1"
                ]
            },
            "2": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "e32e733b-f21d-4842-a5f2-a7a21818f6a9"
                ]
            }
        },
        "titlesById": {
            "b697c926-cfd8-4564-8025-7252a3ec4599": {
                "Falcor.KeySet": {
                    "$type": "error",
                    "value": {
                        "message": null
                    }
                }
            },
            "7b833be2-ef51-46f4-9de6-77c427b8dce1": {
                "Falcor.KeySet": {
                    "$type": "error",
                    "value": {
                        "message": null
                    }
                }
            },
            "e32e733b-f21d-4842-a5f2-a7a21818f6a9": {
                "Falcor.KeySet": {
                    "$type": "error",
                    "value": {
                        "message": null
                    }
                }
            }
        }
    }
}

I think I tracked down the issue to the RoutingEngine.cs. It looks like the Ref path on line 43 is building a path as ["titleById", "e32e733b-f21d-4842-a5f2-a7a21818f6a9", KeySet("rating", "userRating")] I don't think the titleById works with only one id. I ran into that issue a few days ago. if that guid was a KeySet, I think it would work.

I have been trying to track down this issue but coming up short. Here is a unit test that I believe proves my point. I think we need to treat any string after the first seed as a KeySet (if not numeric range or set).
What you think?
var path10 = @"[""genrelist"", ""51f2928f34"", ""name""]";
var path10Expected = Path("genrelist", new KeySet("51f2928f34"), "name");

Haven't read through everything here in detail yet, but at first glance it does appear their might be a couple bugs ..... looking into it now

Ok, see above commit. I added a failing case that reproduces the error.  The issue is not how paths are parsed, but the behavior of the KeysPattern path matcher. Should be an easy fix - let me know if you want to try to put a fix in, otherwise I can do it soon.

Thanks, I should be able to take a stab at it today. I'll replay back with my results.

I have been poking around with this for a few hours and I am little confused about the test condition itself. The logic is a little complex here so let me try and explain why I think this is not working. (please correct me if I am wrong)
For this test above, we have 3 parts

Router Pattern: I think this is working correctly because I can clearly see that it is picking up the Keys Pattern

KeySegment inputs: ShouldMatch("foo", "id1"), I think there is a possible problem here because when this code runs, it is calling KeySegment implicit operator and converting to StringKey. If you change the test case to .ShouldMatch("foo", new KeySet("id1")) it passes.

PatternPathMatcher runs: Here is where I think the problem is. The keyTest is only checking if the key is a KeySet. But in this case, it will always be a StringKey because we define it through the implicit operator.

private static PathMatcher PatternPathMatcher<TValue>(Predicate<KeySegment> keyTest,
	Func<KeySegment, TValue> getValue, string name) =>
		key => keyTest(key) ? new MatchingResult(name, getValue(key)) : Unmatched;

Am I on the correct track here? Or am I missing something?

I think this is the core issue - yes it is working correctly by picking up KeySets, but should it also match against SimpleKeys? That is the question. (I'll come back to this)

The KeySegment inputs are working fine. Strings should implicitly convert to StringKeys. Again, the question is whether or not the KeysPattern should match against a StringKey

Yes - this is much closer to the issue. The PatternPathMatcher code is still correct, the question is whether the KeysPattern factory method (https://github.com/falcordotnet/falcor.net/blob/09cc72675f22fed5f1682a4c715dc46509ded0e5/src/Falcor.Server/Routing/PathParser/PathMatchers.cs#L51) should have keyTest parameter it passes to the PatternMathMatcher also check if the key is a SimpleKey in addition to a KeySet. The AsKeySet() to convert the incoming key into a KeySet already works for simple Keys - see here (https://github.com/falcordotnet/falcor.net/blob/09cc72675f22fed5f1682a4c715dc46509ded0e5/src/Falcor/SimpleKey.cs#L5).

I do think this is the right approach, but I've gone back and forth a bit. This is absolutely the right approach if the falcor client code collapses a keyset with a single key to just a key (did you observe it doing that?), but also makes reasonable sense apart from that. In the NetflixFalcorRouter example, you could argue that the Ref that is returned in the initial route should specify the ID inside a KeySet. What do you think?

Need to think about this a bit. As for the client, it collapses the keyset into a single key.
model.get('titlesById["fc32c7b1-1213-4296-8e8a-65f846774333"]["rating", "userRating"]').then(logJson, logJsonError)

//*** Server gets
[["titlesById","fc32c7b1-1213-4296-8e8a-65f846774333",["rating","userRating"]]]

How about this. I was able to get the tests passing and client code rendering with this change in KeySegment.cs. My thinking is that StringKey == KeySet, so lets just check for everything else besides a StringKey.
//public bool IsKeySet() => KeyType == KeyType.KeySet;
public bool IsKeySet() => !(IsNumericSet() || IsNull() || IsBoolean());

With this change the client also returned
model.get('titlesById["fc32c7b1-1213-4296-8e8a-65f846774333"]["rating", "userRating"]').then(logJson, logJsonError);

//returns

{
   "json": {
      "titlesById": {
         "fc32c7b1-1213-4296-8e8a-65f846774333": {
            "rating": 1,
            "userRating": 2
         }
      }
   }
}

Also, this kinda works
model.get('titles[0..2]["rating", "userRating"]').then(logJson, logJsonError);

//returns
{
   "json": {
      "titles": {
         "0": {
            "\u001epath": [
               "titlesById",
               "036dbaea-929a-4017-a632-d41eae06507b"
            ],
            "rating": 1,
            "userRating": 3
         },
         "1": {
            "\u001epath": [
               "titlesById",
               "cc24b49f-d9fb-4ecf-9d20-d5778cc568ad"
            ],
            "rating": 1,
            "userRating": 3
         },
         "2": {
            "\u001epath": [
               "titlesById",
               "98a2efbc-49cd-4469-ab66-7c34be8424e0"
            ],
            "rating": 0,
            "userRating": 1
         }
      }
   }
}

BUT, the network traffic says that the JSON looks like (below). From the jgraph it looks like we got it working, but I am not sure why the console is dumping out a different output. This is odd. I think we are on the correct path though. What you think?
{
    "jsonGraph": {
        "titles": {
            "0": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "036dbaea-929a-4017-a632-d41eae06507b"
                ]
            },
            "1": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "cc24b49f-d9fb-4ecf-9d20-d5778cc568ad"
                ]
            },
            "2": {
                "$type": "ref",
                "value": [
                    "titlesById",
                    "98a2efbc-49cd-4469-ab66-7c34be8424e0"
                ]
            }
        },
        "titlesById": {
            "036dbaea-929a-4017-a632-d41eae06507b": {
                "userRating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72507800,
                    "$expires": -3000,
                    "value": 3
                },
                "rating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72507800,
                    "value": 1
                }
            },
            "cc24b49f-d9fb-4ecf-9d20-d5778cc568ad": {
                "userRating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72507800,
                    "$expires": -3000,
                    "value": 3
                },
                "rating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72507800,
                    "value": 1
                }
            },
            "98a2efbc-49cd-4469-ab66-7c34be8424e0": {
                "userRating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72517800,
                    "$expires": -3000,
                    "value": 1
                },
                "rating": {
                    "$type": "atom",
                    "$timestamp":<PHONE_NUMBER>72517800,
                    "value": 0
                }
            }
        }
    }
}

Oh snap, I figured out the odd path thing. It had to do with the log js file
if you change
var stringifyReplacer = function (key, value) {
	if (key === '\u001Eparent' || key === '\u001Ekey' || key === '\u001Epath') {
        return undefined;
    } else {
        return value;
    }
};

Then you get the correct results.
{
   "json": {
      "titles": {
         "0": {
            "rating": 0,
            "userRating": 3
         },
         "1": {
            "rating": 4,
            "userRating": 2
         },
         "2": {
            "rating": 4,
            "userRating": 3
         }
      }
   }
}

If you agree with these changes, I can put together a pull request.

We shouldn't modify IsKeySet(), instead we should probably add an IsSimpleKey property that is false on KeySet and true for SimpleKeys. We should then modify the KeysPattern at the bottom of PathMatchers to check for both IsKeySet || IsSimpleKey

Yeah that sounds like a good idea. Want me to put a pr together?

 I know it's a simple change but I really appreciate the thought you're putting into this - addressing these edge cases will really help in the long run. I'm on vacation the next two weeks so my response time may be delayed but I should be able respond within 12 hours at least. Thanks for the continued help. Please don't hesitate to add new issues or ask questions.

I am glad I could help. This is really cool tech and I want to get it working for my own selfish needs :) Thanks for the heads up with vacation. I should be fine, I can build the package locally for my own deployments.


Old secretless.yml are still supported to ease transition to new format.

This will be silently documented, but not referenced in any official documentation.
We want to strongly encourage moving to the new format, while not pulling the rug out from under existing users.

Is this story about writing some code to map the old format to the new? If not, then shouldn't this just be part of the acceptance criteria for the changes that are made? Are we going to maintain or add some tests to ensure we continue to support the old format?


Divider.createBorderSide used w/ inconsistent width in Material Widgets

This change factored out the creation of divider border sides. However, some of the usage of this passes zero width (hairline divider, such as the Divider widget and between ListTiles), some passes 1.0 (noticeable much thicker line, such as in the Drawer header and and DataTable rows).
MergeableMaterial uses 0.5, though I think that may be a special case.
I suspect most or all of these should be consistent; according to our UX a 1.0 dp width is what the spec uses. Can this be made the new default value, or is there a use case for the 0.0 widths that I'm missing?

/cc @HansMuller


Version number showing v3.0.0-11 for 3.1.0 from release page

Go to https://github.com/linkedin/bluepill/releases/, download 3.1.0
bluepill --version
Bluepill v3.0.0-11-g5140bee

It's the right version (5140bee is the git Sha-1, which matches the tag), it's reporting it wrong because it was probably built before it was tagged. I stared working on automating releases to prevent that, so once we get those going this should not happen again.
Thanks for reporting, we should re-upload the binaries with the right version. @oliverhu can you rebuild after the tag?

It seems my reply before didn't get published for some reason.. anyway, it is fixed in latest https://github.com/linkedin/bluepill/releases/tag/v3.1.1


feat(Mathlib/Analysis/Analytic): continuously polynomial functions

This defines "continuously polynomial" (shortened to "cpolynomial") functions as a special cases of analytic functions: they are the functions that admit a development as a finite formal multilinear series. Then we prove their basic properties. The point of doing is this is that finite series are always summable, so we can remove the completeness assumptions in some results about analytic functions (differentiability for example). Examples of continuously polynomial functions include continuous multilinear maps, and functions defined by polynomials.

Ok oops, the file Polynomial.lean already existed in that directory. I'm surprised that vscode didn't protest when I tried to create a new file with that name.

The file was just merged into mathlib yesterday and wasn't in your local copy of mathlib, so no wonder VSCode doesn't complain ... It's about evaluations of Polynomial and MvPolynomial rather than MultilinearMap or Series, so you did the right thing to create a new file CPolynomial. I should have suggested the name CPolynomial earlier, because it's me who added the Polynomial file ... 😅

Ok oops, the file Polynomial.lean already existed in that directory. I'm surprised that vscode didn't protest when I tried to create a new file with that name.

The file was just merged into mathlib yesterday and wasn't in your local copy of mathlib, so no wonder VSCode doesn't complain ... It's about evaluations of Polynomial and MvPolynomial rather than MultilinearMap or Series, so you did the right thing to create a new file CPolynomial. I should have suggested the name CPolynomial earlier, because it's me who added the Polynomial file ... 😅

But do you understand why all the CI checks fail now ? My files compile locally, but now the "check that all files are imported" check fails, and then mathlib doesn't build. As I don't know what "check that all files are imported means", I am kind of stuck.
Also, the file Polynomial.lean should really prove that polynomials are CPolynomial, not just analytic, then you'll get infinite differentiability even if B is not complete.

Well I merged with master, I tried pushing again, CI still fails with an error message I don't understand...

Yeah it was the import in Calculus.FDeriv.Analytic, wasn't it. I finally figured it out, but looks like you got there just before me...

I think you also need to add the CPolynomial file to Mathlib.lean.

Oh, is that something that you need to do every time you add a new file ? Good to know.

If you click on the second "detail" you'll be led to this page which shows what need to be done. I think the bot used to suggest such fixes ...

I did click on the "details" page, but I didn't see any information that I could understand...

Yes it's not totally clear. It's presented in the form of a diff that you can apply, like the bot's suggestion.

Also, CI still fails, and the bot seems to be suggesting that I remove CPolynomial from Mathlib.lean, or switch the order? It's making several suggestions and they don't seem very compatible.

Yes it's not totally clear. It's presented in the form of a diff for you to apply, like the bot's suggestion.

Okay, well, now I see what you mean, but there was no way I was going to understand that on my own... I guess I'll know next time.

So let's see if I understand the suggestion this time. I think it wants me to move a line around. I thought the imports were in alphabetical order, but apparently not.

bors r+


Does not work with modules

Linter does not work well with modules. Is there a possibility of making that happen in forseeable future?

Can you share little bit more details? I have a feeling this is Lint.jl issue but it would be easier to comment if you can show by example what is exactly the problem.

I meant things like having code from a local module show up as undeclared symbols.

Try running Lint.jl from REPL. Do you get the same errors and warnings?
Your example is too small to understand your workflow. If you either include or import by using the local module before your functions usage it should work.

It works fine from the REPL

I still don't understand your problem. In your example each line have a red dot meaning there are some errors. Maybe you just don't see the error messages? What happens when you move cursor to any of the red dotted lines?

The dotted lines are where the functions and constants from ISCLib are used

Fixed in #119


Per-operation credentials (passed via OptionsBase instance) ignored.

Problem description
It seems that credentials passed via com.eventstore.dbclient.OptionsBase.authenticated(UserCredentials) are ignored and global client-level credentials (configured via com.eventstore.dbclient.EventStoreDBClientSettings.defaultCredentials) will always be used.
EventStoreDB servder version: 22.10.1 OSS
EventStoreDB-Client-Java version: 4.1.1
Demo code
    // create non-admin test/test user on target ES server
    @Test
    public void testPerOperationAuth() throws Exception {
        final EventStoreDBClient client = eventStoreDBClient();
        final StreamMetadata md = new StreamMetadata();
        final StreamAcl acl = new StreamAcl();
        acl.addWriteRoles("$admins");
        md.setAcl(acl);
        client.setStreamMetadata("test", md);
        // the append should fail with auth error from server, since "test" user is not part of "$admins" group
        // instead it works?!
        EventStoreUtils.append(client,
                               new UserCredentials("test", "test"),
                               jacksonObjectMapper(),
                               xenaEventStream("test"),
                               ExpectedRevision.any(),
                               DateTimeUpdatedImpl.builder().now(Instant.now()).build());
    }

    private static EventStoreDBClient eventStoreDBClient() {
        final String host = EventStoreConsts.ES_DEFAULT_HOST;
        final EventStoreDBClientSettings settings =
            EventStoreDBClientSettings.builder()
                                      .addHost(new Endpoint(host, ES_DEFAULT_HTTP_PORT))
                                      .defaultCredentials("admin", "changeit")
                                      .keepAliveInterval(Duration.ofSeconds(10).toMillis())
                                      .keepAliveTimeout(Duration.ofSeconds(10).toMillis())
                                      .discoveryInterval((int)Duration.ofSeconds(10).toMillis())
                                      .maxDiscoverAttempts(3)
                                      .gossipTimeout((int)Duration.ofSeconds(15).toMillis())
                                      .defaultDeadline(Duration.ofMinutes(3).toMillis())
                                      .dnsDiscover(false)
                                      .tlsVerifyCert(false)
                                      .buildConnectionSettings();
        return EventStoreDBClient.create(settings);
    }

Suggested modifications
From code inspection, it seems that following places should be modified:

com.eventstore.dbclient.GrpcUtils.configureStub(S, EventStoreDBClientSettings, OptionsBase<O>)

Instead of:
        UserCredentials credentials = null;

        if (options.hasUserCredentials()) {
            credentials = options.getCredentials();
        } else if (settings.getDefaultCredentials() != null) {
            credentials = settings.getDefaultCredentials();
        }

impl should be similar to this:
        UserCredentials credentials = options.getCredentials();
        if (credentials == null) {
            credentials = settings.getDefaultCredentials();
        }

com.eventstore.dbclient.GrpcClient.WorkItemArgs.getHttpConnection(OptionsBase<A>, EventStoreDBClientSettings, String)

Instead of:
                String creds = options.getUserCredentials();

                if (creds == null && settings.getDefaultCredentials() != null) {
                    creds = settings.getDefaultCredentials().basicAuthHeader();
                }

                if (creds != null) {
                    conn.setRequestProperty("Authorization", creds);
                }

something like this:
                UserCredentials creds = options.getCredentials();
                if (creds == null) {
                    creds = settings.getDefaultCredentials();
                }
                if (creds != null) {
                    conn.setRequestProperty("Authorization", creds.basicAuthHeader());
                }

com.eventstore.dbclient.OptionsBase.metadata

It seems to me that this field can be removed altogether. It is never actually used AFAICT, except in getUserCredentials() and hasUserCredentials(), which are only used in above-listed snippets. The only  other use is in OperationBase.getMetadata(), which is called from com.eventstore.dbclient.DeleteStream.execute() but only to be stored in unused local var (headers).
The dichotomy between OptionsBase.hasUserCredentials() and OptionsBase.getUserCredentials() on one hand and OptionsBase.credentials on the other is the root cause of this entire issue so it seems cleanest to remove OptionsBase.metadata altogether.
Note that I'm not familiar with entire codebase in great detail (this is all from one afternoon of debugging), so it is of course possible that I'm missing something.

Nice catch @jezovuk and thanks for reporting. PR is ready. As soon as the PR is merged, I'll cut a release on Maven Central.


MeteorJs

I would love to use this for Meteor JS.
How would I do so?

There is nothing meteor-specific about Brim.


Select the correct ratio for content images

Does the terminus data we have give us enough info to choose the editor selected ratio for content images?
see: https://github.com/abcnews/interactive-expandable-cards/pull/15#discussion_r1006438869

I'd like to know this too. It may help remove the need for config (mount) markers in Odyssey too.


swiper滑动到其他页面，此时通过请求后台接口改变swiper中的数据；再次滑动页面时，swiper的index出现负值；请问如何重新设置index，期望每次能够设置swiper的index从0开始

Which OS ?
Version
Which versions are you using:

react-native-swiper": "^1.5.12
react-native": "0.48.2

Expected behaviour
Actual behaviour
How to reproduce it>
To help us, please fork this component, modify one example in examples folder to reproduce your issue and include link here.
Steps to reproduce

<EMAIL_ADDRESS>没问题


Package request: fluentui-emoji

Project description

Default emoji set used in Windows 11 and up, it's open source apparently (news to me tbh). So there is no reason they can't be packaged in theory
Metadata

homepage URL: https://github.com/microsoft/fluentui-emoji
source URL: https://github.com/microsoft/fluentui-emoji
license: MIT
platforms: Should work on anything that can parse a .tff

Add a :+1: reaction to issues you find important.

However, does seem that a actual build script is WIP. So I guess we need to hold off on this until if and when that is merged
https://github.com/microsoft/fluentui-emoji/pull/131
So consider this made in advance


fix panic error

162

@Pantani  Use "close" word in from of PR link to close it automatically once PR merged.

@kolya182, thks for the tip. But this issue is inside the back-end repository.

@kolya182 the panic is about we try to call a method to log an error without an error object. But I fix in this PR

@kolya182, thks for the tip. But this issue is inside the back-end repository.

I know, that why i did not expose user request that cause a panic, was curious what was special about it ?


Add Rhocall viz

rhocall viz generates wig files and bed files of binned zygosity and RHO-regions. This can be used as input for chromograph to generate a visual representation of regions of homozygosity.

Done!


RemNote plugin marketplace

Could you add a README file to explain how the plugin works and re-upload it? Otherwise it's difficult for users to know how to use it.

terribly sorry! A submission was made by mistake before.
I have corrected the code and description and resubmitted. (I'm not sure whether the submission was successful, because the submission interface took a long time)
It is a simple plug-in that provides word count functionality in the sidebar.

Nice :) thanks! Just approved it now.
On Wed, Jul 17, 2024 at 2:47 PM Nova Nightfall @.***>
wrote:

terribly sorry! A submission was made by mistake before.
I have corrected the code and description and resubmitted. (I'm not sure
whether the submission was successful, because the submission interface
took a long time)
It is a simple plug-in that provides word count functionality in the
sidebar.
—
Reply to this email directly, view it on GitHub
https://github.com/bianyeyu/word-count-develop/issues/1#issuecomment-2233372441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN3UCAZRHAGSGTJ2P5WDWUTZMZYYVAVCNFSM6AAAAABK6BGBP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZTGM3TENBUGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


[console and router auth-plugin] Username is not showing for an authenticated connections

[standard address space with standard authservice]
[console]
In connections tab, there is an option to filter by user, however in the connections table there is no column for user, meaning that any filtering done for any usernames will always return no results, even when filtering for the username provided in an authenticated connection ( provided by creating a binding, the username is prefixed with "user-" )
[router auth-plugin]
after discussion with gsim, and observing outputs from following commands:
oc exec -n myproject-myspace -it qdrouterd-0 -- qdstat -c
oc describe pod/qdrouterd-0 -n myproject-myspace
oc exec -n myproject-myspace -it qdrouterd-0 -- qdmanage query --type connection
gsim suspects this could be a bug in router auth-plugin thats not exposing the username for authenticated connections.

#1576 approved

@RH-sdavey are you saying the issue is not fixed?

Sorry @grs , wrong tab!


Flaky-test - PulsarBoltTest failed to initialize producer

This exception occurs occasionally during builds. Here's one of the places it appeared: https://github.com/apache/pulsar/pull/6202/checks?check_run_id=428130764
[ERROR] Tests run: 15, Failures: 1, Errors: 0, Skipped: 11, Time elapsed: 57.314 s <<< FAILURE! - in org.apache.pulsar.storm.PulsarBoltTest
[ERROR] beforeMethod(org.apache.pulsar.storm.PulsarBoltTest)  Time elapsed: 34.056 s  <<< FAILURE!
java.lang.IllegalStateException: Failed to initialize producer for persistent://my-property/my-ns/my-topic1 : HTTP get request failed: Internal Server Error
	at org.apache.pulsar.storm.PulsarBolt.prepare(PulsarBolt.java:110)
	at org.apache.pulsar.storm.PulsarBoltTest.setup(PulsarBoltTest.java:83)
	at org.apache.pulsar.storm.PulsarBoltTest.beforeMethod(PulsarBoltTest.java:62)
	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:498)
	at . . .

This results in a test failure that looks like this:
[ERROR] org.apache.pulsar.storm.PulsarBoltTest.beforeMethod(org.apache.pulsar.storm.PulsarBoltTest)
[ERROR]   Run 1: PulsarBoltTest.beforeMethod:62->setup:83 » IllegalState Failed to initialize p...

I've attached the log archive.
logs_13139.zip

I also wasn't sure if this should be labeled as a bug since it's just in tests.

Here's another place where this occurred: https://github.com/apache/pulsar/pull/6202/checks?check_run_id=432617829


Architectural/Activity Diagram of litmus-go repository

Architectural/Activity diagram of the litmus-go repository which can cover/explain all the components of litmus-go repository and their connections. It can be used inside README.md. As Diagram can summarise things better.

The raw diagram will be provided soon.

I could take up this issue

I'm a bit confused from where to start. Could you please guide me a bit?


Android Basics: Dogglers app

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-project-dogglers-app?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-project-dogglers-app#2
In which task and step of the codelab can this issue be found?
Task "Get Started", step 3
Describe the problem
In the screen shot, the highlighted bit has had "horizontal" and "vertical" swap places in the id:  It says I need to inflate horizontal_vertical_list_item, however; in the blue tip box it is referred to as vertical_horizontal_list_item and the file  in the codelab raw source code is called vertical_horizontal_list_item.xml

Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Duplicate of #11 , #14

@Lunarclipper
Thank you for reaching out to us. The issue is fixed and will be published soon:


Add specific errors to content types

Motivation

Fixes #342.

Modifications

Add a ContentTypeError enum that has specific error cases used when content types are parsed.

Result

This will affect the ContentType struct.

Test Plan

Test it on an openapi document to make sure the errors are thrown as intended.

@swift-server-bot test this please

You're very welcome! Thank you for considering the changes, and I completely understand the priority of shipping 1.0.0 first. I'm thrilled to contribute, and eagerly looking forward to the exciting milestone of the first release.

Hi @PARAIPAN9 - the issue might not have been super clear, but the issue was about doing the work of walking the OpenAPI document, and validating all the content type strings that exist there, in the validateDoc function.
That way, we'd catch invalid content types at the start, instead of doing so later when we're trying to generate code.
The errors emitted today, when an invalid content type is encountered, are good enough, we just don't do the validation at the start, so it can lead to a more confusing debugging experience.
Does that make sense? If that task is not something you want to pick up, no worries at all, just let me know and I'd unassign you. 🙂

Ah, thank you for the clarification. It's ok, I'll continue working on it.

@swift-server-bot test this please

Perfect, You’re welcome!!


refactor(ui): migrate Reports to functional component and split files

Partial fix for #9810
Partial fix for #11740
Motivation

Use newer, modern functional components compatible with hooks instead of legacy class-based components

Use newer, modern async/await syntax instead of Promise chains

also remove a deprecated, upgrade-blocking usage of BasePage similar to #11793

Modifications

Follow a very similar process to #7035

state -> useState

this.props -> arguments

BasePage + this.queryParams -> location argument + URLSearchParams

componentDidMount -> useEffect

services.info.collectEvent -> useCollectEvent

saveHistory -> useEffect + historyUrl

Consumer -> useContext

merge renderReport into the main component

split out filters into a separate ReportsFilters component similar to CronWorkflowFilters, WorkflowFilters, etc

be consistent with how all these work
similar to #7035, refactor logic to have an onChange function and retrieve Workflows in a top-level effect (after all state has been set)
and similarly add an SCSS file with the appropriate styles

note: this may be worth refactoring as nearly all the filters have the same styles. comments in #7035 suggest making a shared filters component if possible too.

Promise chains -> async/await

refactor ReportsFilters logic to use async/await syntax
also refactor CronWorkflowList to use newer async/await syntax consistent with #11740

getters and setters to regular functions

split out a pure function into a util file

to make clear that it is pure and unrelated to React state, context, etc

Also some tiny optimizations similar to #11743:

use named functions instead of consts assigned to anonymous functions

this makes tracing a bit easier and source maps a bit clearer
(also technically a memory improvement as well)

Verification

Pure refactor, no real semantic changes
Test manually, seems to work, still very interactive with fancy animations

It was my first time using this page though, I literally didn't know it existed 👀

I believe it's not really documented either?

Screenshot below of it working:

Looks good, no need to worry about useCallback for now imo. [...] anyway I think the performance of the UI is not a worry for me regardless.

Agreed, figure I'd notate it at least. I used to write fully memoized components, but I'd forgotten some of the optimizations like useCallback (since React hasn't been my day job in years). Some of those UIs I wrote in the past were impressively performant, so it can make a difference at scale (scale with respect to number of rerenders and components).

Oh the only exception being the swagger docs page which is unusable but yes not related to this issue.

Yea it freezes everytime I try to use it. I can't even run a profiler on it. It may very well be on the swagger-ui side though, or due to the size of the swagger. I have not seen this problem in other codebases though (but idr if we used different renderers or not).


Failed to build with xcodebuild command

Hi I got some problem when using xcodebuild + SPM
Step to reproduce:

create a simple App project from Xcode
add promises dependency by SPM
open command line tool, type xcodebuild -target TestPromise -sdk iphonesimulator

Error log:
note: Using new build system
note: Planning build
note: Constructing build description
Build system information
unexpected service error: build aborted due to an internal error: unable to write manifest to '/Users/chao-chih/Library/Developer/Xcode/DerivedData/TestPromise-chzitmlovmvjsfhjzonnctocolks/SourcePackages/checkouts/promises/build/XCBuildData/f6d544bc94ade848d7e36d0c5727fe62-manifest.xcbuild': File exists but is not a directory: /Users/chao-chih/Library/Developer/Xcode/DerivedData/TestPromise-chzitmlovmvjsfhjzonnctocolks/SourcePackages/checkouts/promises/build

Looks like xcodebuild trying to write some files into build directory. However there is a BUILD file is promises package already. How can I resolve this issue?
Thanks.

Hi @dokinkon,
The BUILD file you see is the Bazel configuration. I guess we should rename that file to BUILD.bazel to avoid such conflicts.
@ykjchen can you confirm such rename would work with both the OSS Bazel and the internal Blaze?
Thanks.

I was able to reproduce this. Resolved by renaming BUILD -> BUILD.bazel as suggested above.
I confirmed that building a project depending on Promises using Bazel still succeeds.
The fix should be available in 1.2.12. Please reopen if you're still running into this.


feat: noir_wasm revamped: deps, cjs, esm, bundled, nicer API

Description
Merging the work done here:
https://github.com/AztecProtocol/aztec-packages/pull/3696
https://github.com/noir-lang/noir/pull/3781
https://github.com/noir-lang/noir/pull/3760
Plus some extras to make the API nicer.
Problem
Closes(?) https://github.com/noir-lang/noir/issues/3695
Summary
Makes noir_wasm easier to work with, including dependency resolution and bundling. This package can be used from both node and the browser with identical API leveraging a virtual filesystem.
Uses webpack for bundling, which is done in two steps:

rust -> wasm (cjs/esm)
TS + wasm (cjs/esm) -> universal package for web

Tests have been migrated to mocha and playwright.
Additional Context
I really want to test it here before merging, but it's in a state in which it can be reviewed before we commit to an API.
Even though the initial memFS-backed FileManager developed by @alexghr is still here, it is not used for the web version due to import problems. The way it works now, webpack uses memfs directly it to alias the node fs module (which seems to be its intended use case) and allows us to use the nodejs fs API everywhere.
Documentation
Documentation is required for usage, but should basically be:

// Node.js

import { compile, createFileManager } from '@noir-lang/noir_wasm'; // Rename!!

const fm = createFileManager(myProjectPath);
const myCompiledCode = await compile(fm);

// Browser

import { compile, createFileManager } from '@noir-lang/noir_wasm'; // Rename!!

const fm = createFileManager('/');
for (const path of files) {
  await fm.writeFile(path, await getFileAsStream(path));
}
const myCompiledCode = await compile(fm);

Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

2k line diff

Let's punt this until after christmas.

Hey, great work! This simplifies noir_wasm A LOT
QQ, for the usage on the browser, does the user have to specify all the dependencies in a files array? Or does it use the new noir_wasm resolver?
It's a bit unclear as tests are using playwright and I'm unsure how would it work for both server-side and client-side rendering.
Basically my concern is that one should have a somewhat consistent way to use this on a browser. Would be totally fine to have different ways to load files, as long as they would follow one or two patterns we can document and that would fit 90% of the use-cases

@signorecello I think that on the browser you just populate the virtual file manager with the files in your virtual file system. For example if you are doing a sort of "noir remix" you'd populate all the files in the user editor to the file manager and ask it to compile. All dependencies that are not local will be fetched from github automatically and added to the virtual filesystem, so everything that works with a local filesystem should work with a virtual one.

Oh I see, so you really treat it as fs, you mkDir stuff, you add files, etc. It's a polyfill.
Pretty cool. Could be worth having an external lib to do that with all files in a folder (probably there is already). My thing would be to have an abstraction such that the dev points to a folder with a Nargo project, and forget about how it actually works

Oh I see, so you really treat it as fs, you mkDir stuff, you add files, etc. It's a polyfill.
Pretty cool. Could be worth having an external lib to do that with all files in a folder (probably there is already). My thing would be to have an abstraction such that the dev points to a folder with a Nargo project, and forget about how it actually works

It's both! as you can see here:
// Node.js

import { compile, createFileManager } from '@noir-lang/noir_wasm'; // Rename!!

const fm = createFileManager(myProjectPath);
const myCompiledCode = await compile(fm);

While in nodeJS, you just point it to a folder with a nargo toml. It'll use fs internally.
// Browser

import { compile, createFileManager } from '@noir-lang/noir_wasm'; // Rename!!

const fm = createFileManager('/');
for (const path of files) {
  await fm.writeFile(path, await getFileAsStream(path));
}
const myCompiledCode = await compile(fm);

While in the browser you will get an empty file manager, since there is no filesystem to read from. So what you do is add the files from the data structures in your editor objects, etc.

Leaving a note here for after the holidays:
This is great work -- Lets try and pull out as many changes into smaller isolated PRs as possible -- we generally do not accept large PRs unless necessary as they are harder to review.
I haven't yet reviewed the PR in detail to see how feasible this is.

Lets try and pull out as many changes into smaller isolated PRs as possible -- we generally do not accept large PRs unless necessary as they are harder to review.

Yeah, this really got out of hand. Good news is that it's pretty self contained, AFAIK the only consumer of noir-wasm is noir-compiler back at aztec-packages, so things shouldn't break even if we sync the repos thanks to this interim PR: https://github.com/AztecProtocol/aztec-packages/pull/3696
Once we're happy with this, it would only be a matter of updating the CLI to consume this package directly (probably with another name?)

I don't really find a place where this PR could be cut to get smaller changes into master, sorry @kevaundray =/
At any rate there's a lot of files that are directly ported form noir-compiler and a bunch of test fixtures. Unless someone is strongly against it, I would ask to merge it as-is...and for extra peace of mind, I updated @signorecello's playground so it can be seen in use: https://github.com/signorecello/noir-playground/pull/32


Package Datadog Agent for DCOS to monitor mesos

Datadog can monitor mesos and marathon natively (this version only support mesos). It uses an agent that runs as a docker container and queries the stats endpoints.
Here is what it looks like when running on DCOS:

The conflict stems from index.json.gz being checked in, by the way.

@alq666 I'm checking this out... the installation fails due to licensing with the following message:
Registered docker executor on ip-10-0-0-206.us-west-2.compute.internal
Starting task datadog-agent.e2b9de24-4c04-11e5-87d3-521641bcf454
You must set API_KEY environment variable to run the Datadog Agent container

can you help me out here
is there a place where this is documented?

@alq666
is it expected that a user must create a json file to over these values at the time of installation?
Pass your API key as a package option (datadog.api_key).
Also pass the number of Mesos slaves as an option (datadog.instances).

@kensipe it is so I'm curious if there's a better way to do this.
The API key is an obvious one. The number of instances is until we can run as an admin task (and it gets deployed on all the agent nodes).

Tested... LGTM.  I'm running details past our internal team.

@alq666 here is what I would like to do... will you remove the default value for api_key in config.json?  By removing it,  the DCOS CLI will see that it is required but that it is missing... this will fail the installation until a key is put in a json file and included with the --options flag.     This will be a better experience IMO.   After that... I'm inclined to merge this.  nice work!

a bit usually... but I've tested the change I requested... so I will merge manual with the change and add this in the universe.

@kensipe thank you! :+1:


game runs incredibly slow to not at all

probably has to do with windows 11 is cet not compatible with widows eleven

Not related to windows 11, direct your support request to discord.


Show Issues tab

Please display the Issues tab alongside Source and Project Home, to make it
easier to report bugs.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Mar 2015 at 8:11

fixed with GH migration


demag_gui

This program fails to produce any plots if sites can not be associated with specimens.  it also seems to treat sample and site names that look like integers as integers and not strings.
Here is a folder  with data that "should" work but it doesn't.
Dubois_MagIC.zip
HELP!

Oops saw the email before the issue, here's a copy of my last message:
Okay I have applied a patch to master. Basically it checks for empty or improperly propagated fields in the necessary meta data of measurements and replaces the stuff that just didn't work out with the 'unknown' place holder. This allows the data to still be viewed but the data hierarchy is obviously not preserved and will make it basically impossible to extract some other data that needs that hierarchy. It seems as if the code was just not ready for meta data that was all numbers it seems to have caused havoc in the data propagation code of the Contribution object as well as in Demag GUI itself.
This is a non-ideal solution as it does mess up some functionality and doesn't really address the problem so much as circumvent it so hopefully it can just be temporary. I'd have done better, but it's 9:42 and I'm tired.
I'm cc-ing Lori now as if this can be fixed anywhere inside the Contribution object is the best spot as it handles the data read in for data model 3 and it seems it is pandas that is causing the files to read in the meta data as float or int objects. If anyone wants to mess with my patch it's lines 3352-3356 of Demag GUI and I commented out 3381-3390 as they don't do anything now.

Yes, I should be able to make sure names are read in as strings, which should be easy.  I hope to do this tomorrow.  Baby Eleanor has arrived, though, so no absolute promises :)

On Jun 28, 2017, at 7:46 PM, Kevin Gaastra<EMAIL_ADDRESS>wrote:
Oops saw the email before the issue, here's a copy of my last message:
Okay I have applied a patch to master. Basically it checks for empty or improperly propagated fields in the necessary meta data of measurements and replaces the stuff that just didn't work out with the 'unknown' place holder. This allows the data to still be viewed but the data hierarchy is obviously not preserved and will make it basically impossible to extract some other data that needs that hierarchy. It seems as if the code was just not ready for meta data that was all numbers it seems to have caused havoc in the data propagation code of the Contribution object as well as in Demag GUI itself.
This is a non-ideal solution as it does mess up some functionality and doesn't really address the problem so much as circumvent it so hopefully it can just be temporary. I'd have done better, but it's 9:42 and I'm tired.
I'm cc-ing Lori now as if this can be fixed anywhere inside the Contribution object is the best spot as it handles the data read in for data model 3 and it seems it is pandas that is causing the files to read in the meta data as float or int objects. If anyone wants to mess with my patch it's lines 3352-3356 of Demag GUI and I commented out 3381-3390 as they don't do anything now.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/232#issuecomment-311847260, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIs9XSPjZldv66REHb_LIHgcpUHGZks5sIxADgaJpZM4OIt8f.

thanks to all for helping track this down.  I thought it was something like the fact that Shelby’s samples are all numbers - of course we could change that, but i don’t think that is unusal and should be possible.
lisa

On Jun 28, 2017, at 8:12 PM, Lori J<EMAIL_ADDRESS>wrote:
Yes, I should be able to make sure names are read in as strings, which should be easy. I hope to do this tomorrow. Baby Eleanor has arrived, though, so no absolute promises :)

On Jun 28, 2017, at 7:46 PM, Kevin Gaastra<EMAIL_ADDRESS>wrote:
Oops saw the email before the issue, here's a copy of my last message:
Okay I have applied a patch to master. Basically it checks for empty or improperly propagated fields in the necessary meta data of measurements and replaces the stuff that just didn't work out with the 'unknown' place holder. This allows the data to still be viewed but the data hierarchy is obviously not preserved and will make it basically impossible to extract some other data that needs that hierarchy. It seems as if the code was just not ready for meta data that was all numbers it seems to have caused havoc in the data propagation code of the Contribution object as well as in Demag GUI itself.
This is a non-ideal solution as it does mess up some functionality and doesn't really address the problem so much as circumvent it so hopefully it can just be temporary. I'd have done better, but it's 9:42 and I'm tired.
I'm cc-ing Lori now as if this can be fixed anywhere inside the Contribution object is the best spot as it handles the data read in for data model 3 and it seems it is pandas that is causing the files to read in the meta data as float or int objects. If anyone wants to mess with my patch it's lines 3352-3356 of Demag GUI and I commented out 3381-3390 as they don't do anything now.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/232#issuecomment-311847260, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIs9XSPjZldv66REHb_LIHgcpUHGZks5sIxADgaJpZM4OIt8f.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/232#issuecomment-311850622, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDc8t7_sY6n8S0QaC1s6S5AK-56Xhks5sIxYPgaJpZM4OIt8f.

@Caoimhinmg, @ltauxe, I haven't been able to get to this.  If one of you wants to make sure that names are string-i-fied in the MagicDataFrame object, please go ahead!  We are having lots of busy baby times around here :)


Sometimes the operations are misunderstood, can you add more checks for this?

12+8(39) = 47.000000
136/5! = 27.200000

12 is ignored because there are not enough operations 8+39 = 47
factorial is ignored 136/5 = 27.2

Exceptions thrown for additional safety check added to the Model code.


about download file

do it support to download file by  using  HTML   ?
I tried, but it seems not work.

If the file is not too large, you can use the data URI scheme.
You add following in the head section:
<script>
  function saveContent(fileContents, fileName)
  {
      var link = document.createElement("a");
      link.download = fileName;
      link.href = "data:" + fileContents;
      link.click();
  }
</script>

and use following function, which takes as first parameter the path of the file to download, the name used for saving as second parameter, and the dom object as provided by the Atlas toolkit as third parameter.
sub send {
  my ($sourceFilename, $targetFilename, $dom) = @_;

  open my $in, '<', $sourceFilename or die;
  binmode $in;

  my $mime = "application/octet-stream";

  my $data = '';

  while (1) {
      my $success = read $in, $data, 1000, length($data);
      die $! if not defined $success;
      last if not $success;
  }

  close $in;

  $dom->executeVoid("saveContent('" . $mime . ";base64," . encode_base64($data,'') . "', '" . $targetFilename . "')");
}

You will find a working example at<EMAIL_ADDRESS>


After build routing does not work in chrome, but works in firefox

What problem does this feature solve?
After "npm run build" try to run index.html
Everything works in Firefox. But in google chrome when you click the link turns file:///C:/index ..
What does the proposed API look like?
import Vue from 'vue';
import VueRouter from 'vue-router';

Vue.use(VueRouter);

import Login from './pages/login';
import Index from './pages/index';

const routes = [
	{
		path: '',
		redirect: {name: 'Login'}
	},
	{
		name: 'Login',
		path: '/login',
		component: Login,
		props: { name: 'Авторизация' }
	},
	{
		name: 'Index',
		path: '/Index',
		component: Index,
		props: { name: 'Страницы' }
	}
];

export const router = new VueRouter({
	routes,
	mode: 'history'
});

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server, gitter or StackOverflow.

BTW, you first route should probably be '/'

Hello.
Replace
{ 	path: '', 	redirect: {name: 'Login'} }
with
{ 	path: '*', 	redirect: {name: 'Login'} }


Client: Refactor usage of proprietary Contexts (HeroContext, ...) and try to use CommonContext

All values across contexts are basically same, thus it's possible to simplify whole stuff by using one context (per view context is unique).
Create some helper "class":

method validate() to check if everything is filled
validate will be called on target site (thus forcing all values being filled)

/cib

Uff, this was quite worky, but it's done!


Safari Extension Update: Create Quicklinks from Safari

Description
Using the new CreateQuicklink action to make it easy to convert links to Quicklinks. I also ran npx @raycast/migration . to apply all migrations.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast

@thomaspaulmann Brilliant, just noticed the update with the CreateQuickLink addition!
TIL about npx @raycast/migration also, will run it on my other extensions
Thanks

@loris yeah it makes it a lot easier to migrate. Most changes where in v1.28.

🎉


Notifications UI

The stakeholder UX studies and recent feedback from Azure SDK engineering team (Travis, Catalina, Mayuri) is that the Power Apps default notification component is not sufficient.

The Power App notifications pop up as banners at the top of the screen and they do not remain visible for very long.  This makes the notifications very easy to miss. The user may not see them or if they do, they don't have time to read them and figure out what they mean.

The notification messages that are informing a user of a missing pre-requisite/requirement do not tell the user how to complete the action. (For example, notification that a user needs to schedule an information meeting but no link to the scheduing tool).

We have seen this feedback in the Scheduler Tool as well. We opened issue https://github.com/Azure/azure-sdk-tools/issues/3957 with some feedback and ideas

This is a dup of https://github.com/Azure/azure-sdk-tools/issues/4593. Closing in favor of the other that includes a proposal from our UX designer


Bug #316 fix

Hi,
We tried to fix bug #316, though we're not sure the problem was fixed, instead of avoided, since we couldn't find it's source. But at least it shouldn't crash anymore.
Thank you!

This indeed resolves the crash, but probably causes broken presents to lose their color in those cases. I suppose this would be two ways to fix the issue:
Use StackUtils, which makes sure the stack's tag compound exists
StackUtils.set(stack, (byte)colorInner, TAG_COLOR_INNER);
StackUtils.set(stack, (byte)colorOuter, TAG_COLOR_OUTER);
StackUtils.set(stack, skojanzaMode, TAG_SKOJANZA_MODE);
if (nameTag != null)
	StackUtils.set(stack, nameTag, TAG_NAMETAG);
StackUtils.remove(stack, "display", "color");
StackUtils.set(stack, "color", color);

Create and assign the compound if it is null
NBTTagCompound compound = stack.getTagCompound();
if (compound == null)
	stack.setTagCompound(compound = new NBTTagCompound());
compound.setByte(TAG_COLOR_INNER, (byte)colorInner);
// Etc...

(untested!)
Due to the large commit history, lots of other files being changed, the faulty fix (which I could fix with some work) and most importantly this version of the mod being discontinued, I won't accept the PR as is.
I really appreciate your work on continuing the project. Thank you!


Functional clock by Hari

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
[ ] Bugfix
[ ] Feature
What is the current behavior?

Added a functional clock
Issue Number: [Insert Issue Number (if applicable)]
What is the new behavior?

Other information

Please review my code and approve my PR

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Tab completion does not work if PowerTab module is loaded

C:\Users\aaronlev\chrome\chrome0\src [tab_states]> import-module PowerTab
PowerTab version <IP_ADDRESS> PowerShell TabExpansion Library
Host: ConsoleHost
PowerTab Enabled: True
C:\Users\aaronlev\chrome\chrome0\src [tab_states]> git branch      ## Shows popup with list of files for autocomplete
C:\Users\aaronlev\chrome\chrome0\src [tab_states]> remove-module PowerTab
C:\Users\aaronlev\chrome\chrome0\src [tab_states]> git branch    ## Cycles through branches
If there's a configuration option I'm missing, it wasn't clear to me -- search engines couldn't find me the answer either.

I got it to work by making sure PowerTab was loaded before the posh-git profile
import-module PowerTab
. 'C:\Users\aaronlev\Documents\WindowsPowerShell\Modules\posh-git\profile.example.ps1'
In case anyone else comes across the same issue.

Out of curiosity, does the posh-git expansion show up like the other PowerTab output?
Closing since there's a workaround...if someone wants to contribute proper integration, I'd be glad to pull it. :)


Add pod-run-netns-command script to run command from network-tools

container within existing pod netnamespace.
This command has multiple options and examples, please check
if they all make sense and work properly.
Signed-off-by: Nadia Pinaeva<EMAIL_ADDRESS>

/lgtm


ForwarderService commandlets aren't documented

[Enter feedback here]
Many of the commandlets in the module are not documented
Here is what's NOTdocumented
Add-ForwarderServiceAllowedClientCertificate
Add-ForwarderServiceAllowedHost
Get-ForwarderServiceAllowedClientCertificate
Get-ForwarderServiceAllowedHost
Get-ForwarderServiceConfig
Get-ForwarderServiceRelayProxy
Remove-ForwarderServiceAllowedClientCertificate
Remove-ForwarderServiceAllowedHost
Set-ForwarderServiceConfig
Set-ForwarderServiceRelayProxy
It seems that the commandlets have both ForwarderService and OMSGateway names. Would be really good to document when one name is to be used versus the other

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

ID: 093a2178-a60d-bcda-1243-c8fdd89b0252
Version Independent ID: cdf94d78-1aad-c2dd-4a8c-38ceccda0816
Content: Connect computers by using the Log Analytics gateway - Azure Monitor
Content Source: articles/azure-monitor/agents/gateway.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@sritodi Thanks for your feedback.
Assigning to content author @bwren  for review.

Thanks for this. That article needs some other updating as well. Creating a work item to get that done.
#please-close


Introduction of HasValue and IsKnown properties

When Qowaiv started, it was a considered decision to make .IsEmpty() ,.IsUnknown(), and .IsEmptyOrUnknown() methods, rather then properties of the SVO's. After all, they are only partly related to those SVO domains.
That being said, with the introduction of #C pattern matching it would be nice to check for those states via properties, in one way, or another.
To allow pattern matching, and not to change those long lived methods, this is the proposal:

.HasValue as the functional equivalent of !IsEmpty()
.IsKnown as the functional equivalent of !IsEmptyOrUnknown()

This can be used like this:
if (EmailAddress.TryParse(str) is { IsKnown: true } email(
{
    ...
}

and obviously other pattern matching scenario's.

Just a consideration, would HasValue lead to confusion compared to the Microsoft Nullable HasValue property? As that one can still have an 'empty' value, but this one is specifically not an empty value.


Hint when I have a show that I can remove

It would be great if Editor could show me "unused show" statements from imports.
Thanks for considering.

Done in c1963b3995765dc8c4a5a90a3d01dac079552bf0.


What is the license applied to this repo?

I cannot see any license in the repository.
Can you add one or mention what license it is released?
Thanks.

Thank you for issuing this, clarified the license as MIT.

Thank you to release it under a non-restrictive license.


Conditional says type extends base type, but cannot use as base type

TypeScript Version:  2.9.0-dev.20180516

Search Terms: conditional type, base, cannot be used to index
Example 1
Code
interface Item {
    field : number
}
type TryGetFieldValue<T, K extends string> = (
    K extends keyof T ?
        (
            T[K] extends Item ?
                //Expected:
                //Should work, T[K] extends Item.
                //So, it should have `field`
                //Actual:
                //Type '"field"' cannot be used to index type 'T[K]'.
                T[K]["field"] :
                never
        ) :
        never
);

//`x` is of type `4`, as expected
type x = TryGetFieldValue<
    {
        test: { field : 4 }
    },
    "test"
>

Expected behavior:
T[K]["field"] should work because T[K] extends Item
Actual behavior:
Type '"field"' cannot be used to index type 'T[K]'.
Playground Link: Example 1
Example 2
Code
interface Item {
    field : number
}
type TypeOfField<T extends Item> = T["field"];

type TryGetFieldValue<T, K extends string> = (
    K extends keyof T ?
        (
            T[K] extends Item ?
                //Expected:
                //Should work, T[K] extends Item.
                //Actual:
                //Type 'T[K]' does not satisfy the constraint 'Item'.
                TypeOfField<T[K]> :
                never
        ) :
        never
);

//`x` is of type `4`, as expected
type x = TryGetFieldValue<
    {
        test: { field : 4 }
    },
    "test"
>

Expected behavior:
TypeOfField<T[K]> should work because T[K] extends Item
Actual behavior:
Type 'T[K]' does not satisfy the constraint 'Item'.
Playground Link: Example 2
Related Issues:

A generic type is narrowed only in a distributive conditional type, you can refactor your code to use a type alias:
type FieldOf<T> = T extends Item ?
    T["field"] :
    never;

type TryGetFieldValue<T, K extends string> = (
    K extends keyof T ? FieldOf<T[K]> : never
);

See https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#distributive-conditional-types for some details about distributive conditional types.


Fix some trivial issues learned on real project

We are currently evaluating using spring-boot-migrator on our legacy applications and encountering some trivial exceptions. Hope this pull request helps make this project more robust.

@fabapp2 would you please approve me as a first-time contributor to run workflows?

@tan9 Welcome to Spring Boot Migrator. Thanks for the commit 🎉

Hi @sanagaraj-pivotal , the problem is that I can't manage to pass all tests in main on my local machine like:
[WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0 s - in org.springframework.sbm.jee.jsf.recipes.AddJoinfacesDependencies_MyFaces_Test
[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   AddJmsConfigTest.testAddJmsConfig:122 [

Here's the diff between 1. (---) actual and 2. (+++) expected:

--- package com.example.foo;

import javax.jms.ConnectionFactory;

import org.springframework.boot.autoconfigure.jms.DefaultJmsListenerContainerFactoryConfigurer;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;
import org.springframework.jms.annotation.EnableJms;
import org.springframework.jms.config.DefaultJmsListenerContainerFactory;
import org.springframework.jms.config.JmsListenerContainerFactory;

Am I missing something? I am using OpenJDK 17.0.5

@fabapp2  I have had rebase this branch onto main and passed all the tests, would you please provide some feedback :)

HI @tan9
thank you for your contribution 🚀
... and sorry for my bad responsiveness, I've been on vacation and got a pile of work.
Your PR made me write a test for
BuildFile.getDeclaredDependencies()
BuildFile.getRequestedDependencies()
and BuildFile.getEffectiveDependencies() with a multi-module setup which unfortunately surfaced some more issues.

Hi @tan9
You added a comment requested dependency from another module in this multi-module project won't be resolvable.
I am not sure what exactly your intent was but from your comment, it looks to me like getEffectiveDependencies would be the method you need maybe?

Hi @tan9
I deleted my previous comment as I misunderstood your problem.
My test were failing because of an invalid artifactId that I created but after fixing this dependencies to modules in the same multi-module project were properly resolved and
if (dependencies.isEmpty()) {
    // requested dependency from another module in this multi-module project won't be resolvable
    return d;
}

was not required.
Could you maybe provide an example that made you add this if block?
I would like to see if my test (not pushed yet, probably tomorrow) is no enough or if there's another reason that made you add this and if there's an issue my test doesn't surface.

@fabapp2 All I can remember right now was described in the comment:

requested dependency from another module in this multi-module project won't be resolvable

If we are working on ear packaging module to assemble one or more war packaging module, the war module somehow won't be resolvable from getPom(),findDendendepcies() call, hence returning empty dependencies, causing dependenceis.get(0) to throw  IndexOutOfBoundsException.
Please let me know if the information above is enough to spot the root cause, Or I can try to reproduce this issue and give more details later today or tomorrow.

Hi @tan9
That's valuable insight, thank you!
As we're mostly focussing on Boot applications at the moment we might just not have realized this problem.
Actually, there's a good chance that I'd need to discuss this with the people from OpenRewrite who are doing the heavy lifting here. I will try with a JEE setup using ear and war and see if I can reproduce this problem.
It is not the current focus (which is the Boot 3 Upgrade report) but I will create a new issue for this and look into it.
I am excited that you're using SBM with a real JEE application (they are hard to find publicly).
So, please keep us posted about issues and problems you might encounter.

I added #516 to progress on this

@fabapp2  I am glad that I can help.
I am going to migrate tons of Java EE projects (100+) that are all tight to Spring Framework 3 under Java 6. We tried using TypeScript to migrate our projects with little success. However, due to the leak of AST, we cannot do further refactoring in an effortless way. Until I read SBM from the Preparing for Spring Boot 3.0 blog post.
Thanks to all of you guys working on this project, I will try my best to make this project more robust against our legacy and fragile projects :)


News menu

The most recent news about fast reply should be accessible from the webapp.
The dedicated account @fast-reply with resteem all the interesting news (release, tutorial, ...) making them easy to access.
The feed of @fast-reply can be used to built this list of menu items

Currently a link to https://www.steemit.com/@fast-reply has been added to the menu.
This account will resteem all the Fast-Reply related news post on steem.
In the future the latest post could be retrieve directly from the blockchain using the blog feed of @fast-reply


[Discussion] How are you scaling your Dapr applications?

We're trying to understand how users of Dapr are scaling their applications and have some questions to ask:

Are you manually or automatically scaling?
Are your apps running in Kubernetes or another managed offering?
Are you using KEDA to scale or something else?
Are you mainly concerned with scaling 0<-->1 replicas or 1<-->N replicas, or both?
What types of Dapr components are you using that are involved in making scaling decisions?
What challenges have you faced when scaling Dapr-based applications?
What would make scaling Dapr applications easier?

Getting answers to these questions will help us understand how scaling intersects with Dapr and what can be done to make the experience better.  Thanks!

Are you manually or automatically scaling?

Automatic scaling.

Are your apps running in Kubernetes or another managed offering?

Kubernetes.

Are you using KEDA to scale or something else?

Using KEDA.

Are you mainly concerned with scaling 0<-->1 replicas or 1<-->N replicas, or both?

Both.

What types of Dapr components are you using that are involved in making scaling decisions?

AWS SQS/SNS.

What challenges have you faced when scaling Dapr-based applications?

None so far. It is good there is an extensive overlap between Dapr components and KEDA scalers.

Are you manually or automatically scaling?
Scaling automatically
Are your apps running in Kubernetes or another managed offering?
Running on AKS
Are you using KEDA to scale or something else?
KEDA
Are you mainly concerned with scaling 0<-->1 replicas or 1<-->N replicas, or both?
Both
What types of Dapr components are you using that are involved in making scaling decisions?
Azure Service Bus
What challenges have you faced when scaling Dapr-based applications?
The service bus component sometimes shows time out errors. probably not related to scaling
What would make scaling Dapr applications easier?
It's working good


Type Error

This isn't right, since the ref points to an instance of CustomTextInput. Thanks for the effort!


Compiler limit for expanded limit (only 127 allowed)

I tried the to expand the item limit to fit my needs.
I have an enum with ~1400 list members. Using MSVC I triggered C1112 (compiler limit:'1400' too many macro arguments, only '127' allowed.

Preprocessing or compiling using clang-cl may also work.
Closing this issue, as it's not a problem with Better Enums, and I don't think we can work around it in Better Enums.

Any updates on this bug?
got also in VS2015
fatal error C1112: compiler limit
after running and using generated enum_macros.h:
make_macros.py"  256 64 > "C:\work\GitLab\imageinjecttools\src\ImageInjectLib\enum_macros.h"

AFAIK this is a limitation in MSVC. If you are able to use a newer version, perhaps you can try to see if they have increased the limit.

I add an MSVC compile flag to reslove this problem:
    if (MSVC)
        set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /Zc:preprocessor")
    endif ()

Here is more info about MSVC compile limit.


CPU core requirements

[Requirements section doesn't specify a minimum of 2CPU cores, only memory. ]

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

ID: 944b7b97-efbf-e83d-9330-a825d901819d
Version Independent ID: 6c49c1c5-a178-0c9d-4768-fef1507b1a55
Content: Microsoft Defender for Endpoint on Linux
Content Source: microsoft-365/security/defender-endpoint/microsoft-defender-endpoint-linux.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Hi @cjkeist, is there a 2 cores minimum?
Can you point me in the right direction on where you found this information so I can add it via a PR?

Hello,
It’s in the mde_installer.sh script. It checks number of cores and fails if only one core is found. Along with number of cores it also checks for minimum of 2GB of RAM free and disk space.
--
CJ Keist
Assistant Director, CoSINe IT Services
Division of Arts and Sciences
Oregon State University
541-737-1066 | Strand Ag Hall 062C
From: Thomas Sjögren @.>
Reply-To: MicrosoftDocs/microsoft-365-docs @.>
Date: Tuesday, April 27, 2021 at 1:36 AM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: "Keist, CJ" @.>, Mention @.***>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] CPU core requirements (#4756)
[This email originated from outside of OSU. Use caution with links and attachments.]
Hi @cjkeisthttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcjkeist&data=04|01||6cf6e500911d48ac975908d909578c27|ce6d05e13c5e4d6287a84c4a2713c113|0|0|637551093881692695|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EBSAa2KeijFzrpEKoH2QfnF4qie2mIR8fo3xUaOxCeU%3D&reserved=0, is there a 2 cores minimum?
Can you point me in the right direction on where you found this information so I can add it via a PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-docs%2Fissues%2F4756%23issuecomment-827426429&data=04|01||6cf6e500911d48ac975908d909578c27|ce6d05e13c5e4d6287a84c4a2713c113|0|0|637551093881692695|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=iPjLnoE0ruxEUxhNP0NzCjzEZCoYrZOhC%2BD8Isl3LTY%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB2TLI55GC4LKBSXAXC44ZDTKZZQVANCNFSM43KYYO7A&data=04|01||6cf6e500911d48ac975908d909578c27|ce6d05e13c5e4d6287a84c4a2713c113|0|0|637551093881702652|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=LtE9VA35innQb5sWoaBSHnSgnU4GR50pWH1g4s%2FVHCQ%3D&reserved=0.

Yeah, that script doesn't really follow the memory requirements either.
https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/microsoft-defender-endpoint-linux?view=o365-worldwide:
Disk space: 1GB
Memory: 1GB
No CPU requirements.
https://github.com/microsoft/mdatp-xplat/blob/master/linux/installation/mde_installer.sh#L17-L19
Cores: 2
Memory: 2048
Disk space: 1024

@Dansimp Please share your opinion and help us in resolving the issue and PR. Thanks

Hi again @cjkeist, could you verify the shell script after the code merge?


modularize an app cause [$injector:nomod] errors due to folder strcture

My main app uses two modules: core and dashboard. Here are the declaration:
client/lib/app.js:
angular
	.module('app',[
		/* Shared modules */
		'app.core',

		/* Feature areas */
		'app.dashboard',
	]);

client/shared/core.module.js
angular
	.module('app.core', [
		'angular-meteor',
		'ui.router',
		'accounts.ui',
		'nemLogging',
		'ui-leaflet',
		'rzModule',
		'angular.filter',
		'checklist-model',
	]);

client/dashboard/dashboard.module.js
angular
	.module('app.dashboard', [
		'app.map',
		'app.widgets'
	]);

client/map/map.module.js
angular
	.module('app.dashboard', [
		'app.map',
		'app.widgets'
	]);

client/widgets/widgets.module.js
angular
	.module('app.widgets',[]);

At loading I have errors:
Uncaught Error: [$injector:nomod] Module 'app.map' is not available!
Uncaught Error: [$injector:nomod] Module 'app.core' is not available!
Uncaught Error: [$injector:nomod] Module 'app.widgets' is not available!

I think it is due to the load order of Meteor. It seems to not fit at all with Angular requirements. What folder structure is required for modularization in meteor angular?

I managed to load the app.map module by renaming it 0-map.module.js
Is there an other way, it is a bit crappy.

Why do you have two app.dashboard modules declared in two files client/map/map.module.js and client/dashboard/dashboard.module.js ?

@MilosStanic  Typo. Edited.

@dagatsoin here is the way Meteor depends the load order: http://docs.meteor.com/#/full/structuringyourapp
You can use naming or a certain folder structure.
But the best thing is, that with the new version of Meteor (1.3, a first beta should be out today or tomorrow) you can use ES2015 modules and they will determine the load order


Recursive template

I've tried to build a "category tree" using a recursive template
$scope.categories = [
    {"name":"Cat1","children":[
      {"name":"Cat2","children":[]}
    ]},
    {"name":"Cat3","children":[]}
];

but the children is not moved but is copied (inside the template) when I drag everywhere, the parent category instead is correctly moved (outside the template) when I drag into a children list.
I'm doing something wrong?

"category in category.children" overrides the category variable on your scope, so your calling splice on the wrong array. You could try $parent.category.children.splice, or fix the baning issue somehow.

You are right, I try your solution but I still have the same issue, I solved using $parent.$parent.category.children.

Sorry, It was not a problem of angular-drag-and-drop-lists, thx for the prompt response.

You could also use ng-init to create an "alias" of the parent

Can confirm that the alias solution works in 1.4.0 - you can probably close the issue :)