[BUG] ULS stops collecting

Describe the bug
We're seeing the ULS collector (running in a docker container) just stopping processing of events.  Restarting the container seems to make it start again.  When it stop it throws the following:
2023-11-02 15:00:56,332 ULS E UlsInputCli - CLI process [20] was found stale - Reason:  "2023-11-02 15:00:51,170 cli-etp MainThread E API call failed with HTTP/400: b'{
  "type": "https://problems.luna.akamaiapis.net/-/pep-authn/request-error",
  "title": "Bad request",
  "status": 400,
  "detail": "Invalid timestamp",
  "instance": "https://xxxx.luna.akamaiapis.net/etp-report/v3/configs/xxxx/threat-events/details",
  "method": "POST",
  "serverIp": "xxx.xxx.xxx.250",
  "clientIp": "xxx.xxx.xxx.215",
  "requestId": "xxxx",
  "requestTime": "2023-11-02T15:00:51Z"
}'

Traceback (most recent call last):
  File "/opt/akamai-uls/uls/ext/cli-etp/bin/akamai-etp", line 738, in <module>
    main()
  File "/opt/akamai-uls/uls/ext/cli-etp/bin/akamai-etp", line 669, in main
    fetch_events_concurrent(config, out)
  File "/opt/akamai-uls/uls/ext/cli-etp/bin/akamai-etp", line 376, in fetch_events_concurrent
    fetch_event_page(start, end, 1, concurrent_fetch, pool_futures, stats, output)
  File "/opt/akamai-uls/uls/ext/cli-etp/bin/akamai-etp", line 304, in fetch_event_page
    total_records = response_data["pageInfo"]["totalRecords"]
                    ~~~~~~~~~~~~~^^^^^^^^^^^^
"

To Reproduce
Run container and wait
Expected behavior
Container runs without dying, or at least self heals
ULS Version output
Akamai Unified Log Streamer Version information
ULS Version		1.7.1

EAA Version		0.6.3
ETP Version		0.4.5
MFA Version		0.1.1
GC Version		0.0.1-beta
LINODE Version		0.0.1-aplpha

OS Plattform		Linux-5.4.0-166-generic-x86_64-with-glibc2.36
OS Version		5.4.0-166-generic
Python Version		3.12.0
Container Status	True
RootPath 		/opt/akamai-uls/uls
TimeZone (UTC OFST) 	UTC (0.0)
Installation ID 	RFVMT1RGLTIwMjMxMDEyLTEuNy4x

Additional context
It's been running fine since implemented a number of months, don't know if the latest container version has introduced the bug, going to try downgrading it and seeing if we stabilise for now

Hi @lwhitworth,
thanks for raising this bug. We've already spotted this one and created a ticket on the underlying CLI.
You can track the CLI fix over here:
https://github.com/akamai/cli-etp/issues/14
Once the CLI has been fixed, you should be able to pull the fixced version in the ext directory and this bug should be gone.
I'll leave this ticket open until we have a confirmed fix on this.

@bitonio , please confirm but i think this bug will be fixed with the latest ETP CLI version which will result in the following ULS version: v1.7.3 (feel free to grab the current development branch that already should have the fix inside)

Hi @lwhitworth ,
sorry for the fix taking so long.
We just released ULS v1.7.3 which contains the fix for the bug.
Please make sure you update the CLI-ETP version also to 0.4.6 if you're NOT using the dockerized version.


Menu bug

There is a bug on the menu, it is not showing the links correctly with. The solution is on screen.scss:
.g-plusone-slot{
position: absolute;
right: 50px;
}
.navbar .container{
position:relative
}

I know this issue is a bit dusty, but if that's still case could you send a PR with the fix?

@bartoszmajsak Yeap, it is still there. This is an critical UI/UX bug. The menu is invisible on -md sizes making the website impossible to navigate.

In order to send a PR I'd need to install Awestruct and compile everything. It would take me more time than I have this week. It is a very simple fix for someone that has everything running. Just paste this:
position: absolute;
right: 50px;
}
.navbar .container{
position:relative
}```
If it can wait, then I'll try to install everything next week and send a PR.

Ok, cool :) I will take care of it, no worries. Thanks for investigating an issue and providing a solution.
I think it would be good to have some simple dockerized solution for contributors. I will try to provide something like that in the nearest future.

@bartoszmajsak thanks :) and ping in on IRC (my nick is agalante) if anything doesn't work as it should

Related to #116

I cannot reproduce this issue :) Can you share your browser/device details + resolution?

Just start resizing the window till to see no menu items and no navicon 😄

I've just tested it on FF, Chrome and Safari. Same result.

LOL :) Got it now. Resized to a too small window and got the "menu icon" instead. It seems to be messed up for the certain ranges. Thx.

.... I feel dumb now :)

@bartoszmajsak don't worry, it was a hidden bug. Let me know if I can help


Issue edit view button submit not submiting V1.2 8.6 umbraco

Not submiting when editing in view.

Let me know if you need more info.
Im just trying to insert a layout grid in doctype and render in edit view ,

function submit() {
            if ($scope.model.submit) {

                $scope.model.node.name = "Dtge Temp: " + $scope.model.node.key;
                $scope.model.node.variants[0].name = $scope.model.node.name;
                $scope.model.node.variants[0].save = true;

                // save the content as a blueprint, to trigger validation
                var args = {
                    saveMethod: contentResource.saveBlueprint,
                    scope: $scope,
                    content: $scope.model.node,
                    create: true,
                    action: "save",
                    showNotifications: true,
                    softRedirect: true
                };

SOLVED with this.
     $scope.model.close();

Where did you put the close call? Are you sure it doesn't have other implications?

I have tested the functions and it seems that everything works well, I understand that it is the submit function that goes through all the forms that the editor has already done.
The main js doctypegrideditor.controllers.js and function
`  function submit() {
            if ($scope.model.submit) {

                $scope.model.node.name = "Dtge Temp: " + $scope.model.node.key;
                $scope.model.node.variants[0].name = $scope.model.node.name;
                $scope.model.node.variants[0].save = true;

                // save the content as a blueprint, to trigger validation
                var args = {
                    saveMethod: contentResource.saveBlueprint,
                    scope: $scope,
                    content: $scope.model.node,
                    create: true,
                    action: "save",
                    showNotifications: true,
                    softRedirect: true
                };
                $scope.model.close();
                contentEditingHelper.contentEditorPerformSave(args).then(function (data) {
                    $scope.model.submit($scope.model);
                },
                    function (err) {
                        alert(err);
                    });
            } else {
                alert('no submit');
            }
        }`

I just put  $scope.model.close();
                `<umb-button` type="button"
                            button-style="action"
                            label-key="general_submit"
                            shortcut="ctrl+s"
                            ng-if="dialogMode == 'edit'"
                            action="vm.submit()">
                </umb-button>`

Finally I have not managed to get version 1.2 uploaded to production, it may be due to having the custom url or some plugin that is interfering with your addon- Angular js error, if you need more info let me know, in the meantime I will wait for another version to use it.

Hey Bardaji, I have exactly the same issue as you. But adding $scope.model.close tot the function submit() doesn't really help me out :/ Any other ideas ?
Thank you !

I can't reproduce this on a clean install of Umbraco 8.6 with a clean install of DTGE 1.2.
Can you provide steps to reproduce?

Maybe there is something similar with the Member Group Picker.
When pressing the submit button the window just close and nothing is inserted.

I think this has been fixed in 1.2.1 or 1.2.2. I still can't reproduce, so I am closing this.


Unexpected program hang after exception is raised while iterating a generator

Package Name: azure-servicebus
Package Version: 7.4.0
Operating System: GNU/Linux
Python Version:  3.9.9

Strangely, the following program does not terminate:
from azure.identity import DefaultAzureCredential
from azure.servicebus import (
    ServiceBusClient,
    ServiceBusReceiveMode,
)

def receive():
    namespace = 'xxx'
    topic = 'yyy'
    subscription = 'zzz'

    credential = DefaultAzureCredential()
    client = ServiceBusClient(
        fully_qualified_namespace=f'{namespace}.servicebus.windows.net',
        credential=credential,
    )

    with client as client:
        receiver = client.get_subscription_receiver(
            topic_name=topic,
            subscription_name=subscription,
            receive_mode=ServiceBusReceiveMode.PEEK_LOCK,
        )
        with receiver as receiver:
            for message in receiver:
                yield message

def main():
    messages = receive()
    for message in messages:
        raise Exception('Terminate program.')

if __name__ == '__main__':
    main()

It prints the exception and hangs:
Traceback (most recent call last):
  File "/home/indy/dm/sb-experiment/./sbexperiment/app.py", line 40, in <module>
    main()
  File "/home/indy/dm/sb-experiment/./sbexperiment/app.py", line 36, in main
    raise Exception('Terminate program.')
Exception: Terminate program.

But it works as expected when I modify main() to this:
def main():
    for message in receive():
        raise Exception('Terminate program.')

@Indy2222 Thanks for your feedback, we'll investigate asap.

hey @Indy2222 , I am able to reproduce the issue.
this is an interesting issue -- from the code itself I didn't see a major difference. I tried to debug and execute one line by one line, and they are of the same execution order.
I honestly am not clear on what happens when error is raised and how python exits the program, need to do some more research.
Are you blocked by this issue? I think adding a try except catch should help you bypass this hang issue.

Thank you for the investigation!
I am not blocked on the issue personally. However, I think that others might easily spend a some "painful" time figuring out what is happening (like I did) if they use the client in the same way. Therefore, I think this is worth fixing sooner rather then later.

Hey @Indy2222 - Appreciate your feedback on this! We have a few other issues taking priority at the moment, but we plan to address this as soon as possible. Thank you!

Hey. Is there any progress on the issue?

Hey @Indy2222 - apologies for the late update! I'll make sure to discuss this with the team this upcoming week and keep you posted on when we can address this. Thanks!

Hey guys, I encountered exactly the same issue. Any time estimate when this could be fixed? thanks

Hey all! Apologies for the long delay here. Unfortunately, it's been a very busy month and we were not able to find time to investigate this issue. However, this is definitely on our radar, and I will be bringing this up to the team again in a few weeks.

Hi,
I have tested it with:
python = 3.7.9, 3.8.6, 3.9.12,
azure-servicebus = 7.6.1,
uamqp = 1.5.3,
OS: Ubuntu 20.04.4 LTS
With 3.9.12 it does not happen, with the others it does.

Hi @Indy2222  and @pavelm10,
I apologize for the extreme delay in providing an update to this matter.  We are currently working on moving azure-servicebus away from uamqp and on to a pure python based stack, which will address this issue. The preview is expected to be released in February. I'll leave this issue open and update yall when its available on pypy.

@kashifkhan Is this one released?

@xiangyan99 The beta for service bus pyamqp has been released, which should have the fix. GA is not released yet

@kashifkhan Thank you.
@Indy2222 Please try latest beta and let us know if it does not work.


Will we be getting add-on support using Angular like excel for word as well?

[Enter feedback here]

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

ID: 77e380a7-6e2e-ada8-d297-3fd5018a1958
Version Independent ID: f23490e8-6eeb-f2d5-6c94-69e3c9ed83f4
Content: Build your first Word add-in
Content Source: docs/quickstarts/word-quickstart.md
Product: office
GitHub Login: @o365devx
Microsoft Alias: o365devx

I will create a user request for this idea. In the meantime, you can use the Excel Angular Quickstart with a few simple changes. When you get to the question "Which Office client application would you like to support?" answer "Word" instead of "Excel". In the step where you modify src/app/app.component.ts, Replace "Excel" with "Word" in both places. Then replace the two lines
const range = context.workbook.getSelectedRange();
    range.format.fill.color = 'green';

With these lines:
const range = context.document.getSelection();
        range.font.color = "red";

Then after step 1 of the Try it out section, type some text in the document, highlight part of it, and press the button to see the font change to red.


[DOCFIX] Use correct punctuation in docs/cn/api/Clients-Go.md

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2647/
Test PASSed.

LGTM

alluxio-bot, merge this please


Using create-react-app gives errors when building

I am trying to use this with create-react-app similar to how it is done in ReactNativeMeteorBoilerplate.
I add meteor-client.js to node_modules/ and then I can write import 'meteor-client'; and everything works as expected when running npm start.
However in create-react-app  when I run npm run build I get the error
Failed to minify the code from this file:
 	./node_modules/meteor-client.js:5740

see here: create-react-app README npm-run-build-fails-to-minify.
It seems that the code needs to be compiled to ES5. I try and do this using babel like so (package.json and running npm run babel):
{
  "name": "meteor-client-bundler-module",
  "version": "1.0.0",
  "description": "module for meteor-client-bundler to compile to ES5",
  "main": "./build/meteor-client-es5.js",
  "scripts": {
    "babel": "babel --presets es2015 ./meteor-client.js -o ./build/meteor-client-es5.js"
  },
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "babel-cli": "^6.26.0",
    "babel-core": "^6.26.3",
    "babel-preset-es2015": "^6.24.1"
  }
}

I put the new meteor-client-es5.js in the node_modules/ directory  and now do import 'meteor-client-es5'; instead. Now npm run build compiles successfully. However when running in the browser it now gives the same browser console errors as running npm start:
ReferenceError: __meteor_runtime_config__ is not defined
...

Maybe babel is not compiling to ES5 properly? I don't know...
I created a repository showing what I have so far: https://github.com/vephinx/create-react-app-with-meteor

I guess babel adds use-strict on the top of file. I recommend to import meteor-client from node_modules.

i hit the same error! when I run
npm run build
I get the error Fail`d to minify the code from this file:
./node_modules/meteor-client.js:5853

i found this is because the meteor-client.js include e6, but create-react-app minify the code only in es5, so, i use uglifyjs-webpack-plugin which can use es6.

use web.browser.legacy is ok!

So you issue has been solved by settings arch to web.browser.legacy in bundler configuration, right?

yes


PULL_REQUEST_TEMPLATE.md: updated message for sha256 change without version

After seeing it live, it seemed a bit verbose/confusing still. Suggesting this small change.

@vitorgalvao @reitermarkus Just wondering, do you think it would be wise to add new tag for pull requests which still need to be verified (e.g. https://github.com/caskroom/homebrew-cask/pull/33570). Or will the "on hold" tag suffice?

@giannitm Currently, it should be on hold + upstream. By I wouldn’t oppose a new label.

@reitermarkus How about like this? We don’t need to repeat it’s a link. The amount of earlier commits was me trying to align the text, but apparently Github always has that small indentation.

The amount of earlier commits was me trying to align the text, but apparently Github always has that small indentation.

That was also driving me nuts yesterday. 😂


Passing certain maskers (e.g., NiftiLabelsMasker) will produce invalid results for IBMA estimators

Summary
Currently I believe we let users pass any nilearn Masker in at estimator initialization, and then internally use that masker to reduce the data. This works fine for certain estimators, but will lead to wonky/invalid inferences in other cases.
Additional details
Consider any of the PyMARE-based IBMA estimators that expect the input to be pairs of estimate and variance maps. Suppose the user passes a NiftiLabelsMasker as the mask. Then the expectation is presumably that the meta-analysis will be done separately for each discretely-labeled ROI in the mask. Naively, the masker will reduce the dataset by averaging over all voxels within each ROI—and it will do this (I think) for both the parameter estimate and the sampling variance maps. The former is not a problem, but the latter is problematic because (a) we need to scale the variance by the number of voxels in the ROI (which we could do ourselves, but it would be an extra step), and (b) voxels are dependent, and we have no idea what the covariance matrix looks like (nor any good way to estimate it). The net effect is that the variances being passed to PyMARE internally will be (heavily) biased, and hence so will p/z-values. (Credit to @nicholst for pointing this out in email a while ago.)
A related problem will arise for the combination tests (e.g., Fisher's or Stouffer's): I presume the naively applied masker will average z-scores within ROIs, but this is not at all what we want—we presumably would need to aggregate values within each ROI via some Fisher's-like procedure (but not using any of the methods currently in PyMARE, because they all assume independence).
Reproducing the bug
Almost any ROI-based IBMA analysis will suffer from this issue. The exceptions are the estimators that take parameter estimates but not variances as inputs (I think that's currently only the SampleSizeBasedLikelihood estimator, which frankly is kind of crappy).
Proposed solution
Disable ROI-based analyses entirely in the package until we can figure out a sensible solution to the problem (which is going to be... hard). Masking is fine so long as the user's just defining the valid voxel space, but as soon as there's aggregation of voxels, all bets are off.

@tyarkoni, do you have that email/could you paste it here? I would like to read more about the issue to understand it better.

This is a little depressing 😞(especially for the NiMARE paper) but on the bright side we can probably handle this fairly easily by checking the mask type in get_masker.

Yeah, it's a bummer.
This might be a good impetus to bring back the old "GLM" approach, which was basically just a t-test on the averaged estimates. That's equivalent to using WeightedLeastSquares with unit variances everywhere, so it wouldn't require any PyMARE changes; we could just re-use the code in NiMARE's WeightedLeastSquares.fit() with v=None in the PyMARE dataset. We could probably just call it LeastSquares. This would at least provide a valid and widely understood way to do ROI-level tests, even if performance is probably not great.

@tyarkoni, do you have that email/could you paste it here? I would like to read more about the issue to understand it better.

No, because it's a long thread with lots of in-line comments, but here are a couple of stats/math exchange threads giving more succinct explanations: 1, 2.

The only quick and dirty solution I can think of is using a simple mixed model with random intercept per subject to the ROI-level data.  The estimated mean (the population intercept, over subjects and ROI values) will be the outcome of interest and we can easily get the estimate and standard errors.
This is not exactly the 'right' model, as it will only capure compound symmetry correlation within each ROI (i.e. each voxel in the ROI equally correlated with each other), but will be a big step towards the highly biased SE from the naive approach.  Slightly more nuianced models could use a few spatial bases (e.g. linear trends, or tensor spline bases) as random factors which would account for more spatial structure in the ROIs.

Between this thread and conversations on Slack, it seems like the current easy solution is the following:

Prohibit non-NiftiMasker maskers for combination tests (Fishers and Stouffers).
Log a warning about biased variance for non-NiftiMasker maskers for non-combination, non-likelihood tests (Hedges, WeightedLeastSquares, and DerSimonianLaird).
Log a warning for likelihood-based tests (SampleSizeBasedLikelihood and VarianceBasedLikelihood).

This is separate because I think we are less certain about how averaging will impact these ones. Is that correct?

Leave PermutedOLS alone.

PermutedOLS just uses the contrast maps, so it should be fine, right?

Does that sound like a reasonable next step, pending progress in #475 or new estimators for the methods proposed by @nicholst?

I think that's right, except:

The SampleSizeBasedLikelihood should be just fine, as it doesn't depend on the variances at all, and the sample size is a constant for each study. (It will still be a shitty estimator that people should avoid if they can use one of the variance-using estimators, but that's true whether or not one is doing ROI analyses.)
We can probably give the same warning for all the estimators that depend on the variance; I'm sure they will each behave slightly differently, but I don't think we're in a position to say exactly how, absent a much more systematic analysis. It should be something that makes the user nervous but doesn't induce outright panic. Maybe something like "Note that the aggregation of variance estimates within ROIs can introduce bias and/or variance to the results. Preliminary analysis and simulation suggests the effect is likely to be small in most cases, but no guarantees are provided, and users are encouraged to conduct analyses at a whole-brain level when feasible."

Agree with your plan @tsalo as amended by @tyarkoni


appended digital ocenan's how to raise pull request link for user help

How Many Days Until Halloween? PR Request Template
Please note: Has this feature already been added? Sometimes, duplicate pull requests happen. It's worth checking the pull requests and issue page to see if the change you are requesting has already been made.
Descriptive name.
Your pull request should have a descriptive name.
Type of Change was Made?
What type of change does your code introduce? After creating the PR, tick the checkboxes that apply.

[ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Description of the Change Being Made.
It's helpful to outline what changes were made to which files so that I have an idea of what will be involved in reviewing the code and—hopefully—merging it into the codebase.
Issue Number
If your pull request is related to a specific issue, please included it in your description and or pull request name. It helps to keep changes linked. Any issues that are referenced in pull requests become part of the discussion history of the issue.
Potential Performance Issues
Does the PR have a potential impact on performance on the codebase? If so, to what degree and why does the PR warrant the performance hit?
Tests/Checks
What tests were conducted to ensure the PR functions and has no impact on previous functionalities of the code base
New Dependencies
Have new dependencies been introduced? Please list them with links to documentation and add installaion steps to the README

@sagarwala
It looks like your use of [link] is conflicting with other links on the page. For example, [link] is also used on lines 15 and 1. Can you make the corrections? Let me know if you need some guidance.


[2pt] Add unit tests for all implemented methods.

Test all methods in the Game class.
Test all methods in the Author class.

Hello @rachidelaid,
I have submitted the tests for the game and author class for review on the test branch


[Solutions] SAP 3tier

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/example-scenario/apps/sap-3tier-distributed.md
:warning:Warning
View
Details

docs/example-scenario/apps/media/sap-3tier/SAP-Infra-3TierDistributedNoHA_finalversion.png
:white_check_mark:Succeeded
View

docs/example-scenario/apps/sap-3tier-distributed.md

Line 23: [Warning] Invalid file link:(~/example-scenario/apps/media/SAP-Infra-3TierDistributedNoHA_finalversion.png).
Line 80: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed).
Line 88: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed-HA.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/example-scenario/apps/sap-3tier-distributed.md
:warning:Warning
View
Details

docs/example-scenario/apps/media/sap-3tier/SAP-Infra-3TierDistributedNoHA_finalversion.png
:white_check_mark:Succeeded
View

docs/example-scenario/apps/sap-3tier-distributed.md

Line 80: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed).
Line 88: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed-HA.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/example-scenario/apps/sap-3tier-distributed.md
:warning:Warning
View
Details

docs/example-scenario/apps/media/sap-3tier/SAP-Infra-3TierDistributedNoHA_finalversion.png
:white_check_mark:Succeeded
View

docs/example-scenario/apps/sap-3tier-distributed.md

Line 80: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed).
Line 88: [Warning] Invalid file link:(~/example-scenario/apps/sap-3tier-distributed-HA.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Pub/Sub is semi-broken?

AppVeyor has failed to build Pub/Sub, but GitHub Actions seem to be okay. I'm confused. Either way, we need to sort it.

GitHub actions has failed now too... although it passed in PR #5712 itself, which still concerns me. I believe it'll be easy to fix, but I want to understand it first.


ipWhitelist - cant be whitelisted

After updating to v2.1.0, i cant access this from remote location anymore. Normally accessing devices via VPN to home network, but gives me the default error message on the MM pi.
Also tried adding the IP from remote location without success.
Simply adding "::" or "<IP_ADDRESS>" in config.js doesnt give me access to it.. VNC works, so its running fine on localhost
Is there a way to disable this feature, or am i adding the ip wrong? (Its on double quotes im assuming?)

Check your node console output, it should show the blocked ip.

Access denied to IP address: ::ffff:<IP_ADDRESS>
(VPN adds the extra number "8" on my local ip pool)
But even trying to add "::ffff:<IP_ADDRESS>" or "<IP_ADDRESS>" its still displaying the same msg,

You need add ::ffff:<IP_ADDRESS> . Will not work if only put the Ip address in IPV4 format.

Tried this. And tried on my normal local network using ::ffff:10.0.0.(my device) and ::ffff:<IP_ADDRESS>/100 to make all local devices whitelisted, still nothing. Could this be a router issue? Otherwise, i dont know what else to try..

ipWhitelist: ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1","<IP_ADDRESS>","<IP_ADDRESS>"],
Access denied to IP address: ::ffff:<IP_ADDRESS> Access denied to IP address: ::ffff:<IP_ADDRESS> Access denied to IP address: ::ffff:<IP_ADDRESS> Access denied to IP address: ::ffff:<IP_ADDRESS>
why?

@xdker; was able to get access following some advice from the forum.
ipWhitelist: ["::fff:<IP_ADDRESS>/1", "::fff:<IP_ADDRESS>/1", "::fff:<IP_ADDRESS>/2", "::fff:<IP_ADDRESS>/3", "::fff:<IP_ADDRESS>/4", "<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1"]
This bacially allows all variations of IP pools to access the mirror. Im comfortable with having MM accessible on my network. So if you feel the same way, please copy and add to your config.js


feat: [#11] Split MiTi by owner

fixes #11

prepare: additional column Owner, only for Topic MiTi, based on a hardcoded list of descriptions
validation that we have no unwanted combinations, i.e. If Topic is MiTi, we need to have an Owner, if Topic is other than MiTi, we must not have an owner.
Use it in summary export to add columns MiTi_MiTi, MiTi_LoLa and MiTi_Total

prepare: additional column Owner, only for Topic MiTi, based on a hardcoded list of descriptions
validation that we have no unwanted combinations, i.e. If Topic is MiTi, we need to have an Owner, if Topic is other than MiTi, we must not have an owner.
Use it in summary export to add columns MiTi_MiTi, MiTi_LoLa and MiTi_Total


Go will include windows-only files when compiling

Issue description
Go will include windows-only files when compiling, and cause this error.
# github.com/bakape/hydron
/nix/store/360zh9xvnqzvzvl567b43960fhgq90dw-go-1.10.3/share/go/pkg/tool/linux_amd64/link: running g++ failed: exit status 1
/run/user/1000/go-link-113124839/000000.o: file not recognized: File format is ambiguous
/run/user/1000/go-link-113124839/000000.o: matching formats: coff-x86-64 pe-x86-64
collect2: error: ld returned 1 exit status

I have tested on another OS to ensure it wasn't the repo's problem, but instead this is Nix, or Nix's go expression that's the problem. Go will include a windows-only binary icon, and that will cause the above error.
Steps to reproduce
go get github.com/bakape/hydron
Technical details

system: "x86_64-linux"
host os: Linux 4.17.5, NixOS, 18.09pre145679.dae9cf6106d (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0.4
channels(root): "nixos-18.09pre145679.dae9cf6106d, nixpkgs-18.09pre146127.98792fe3515"
channels(okina): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

This was fixed at some point.


[Bug] api returns "f [Error]: 2" rather than the actual 404 response

Current Behavior
If API is used to fetch all resources, or used to fetch a resource translation from a resource they're not a translator on (but still part of the org/project), the API will return a very unhelpful f [Error]: 2 without any other details.
In order to find what the "real" error was, I had to use the online interactive API docs, which revealed the real error (which is also incorrect):
"status": "404",
"code": "not_found",
"title": "Object not found",
"detail": "No `ProjectLanguage` resource found with the identifier `p:xxx:l:xxx`"

Expected Behavior
The error returned should indicate that the user doesn't have permission/is not a translator for the requested language.
Steps to Reproduce
Using the latest @transifex/api package:
Scenario 1:
const { transifexApi } = require('@transifex/api');

const project = await transifexApi.Project.get({ organization: `o:${org_slug}`, slug: project_slug });
const resources = await project.fetch('resources');
const resource = await resources.get({ slug: resource_slug }); // *** Error thrown here ***

Scenario 2:
const { transifexApi } = require('@transifex/api');

const language = await transifexApi.Language.get({ code: 'es' });
filter = { resource: ..., language };
for await (const t of transifexApi.ResourceTranslation // *** Error thrown here ***
      .filter(filter)
      .include('resource_string')
      .all()) {

Possible Solution
API should expose the real HTTP error. Additionally, a 404 error should only be returned if the resource doesn't actually exist as per the HTTP spec, and should not be used if the user does not have the correct translator status.

@Foda We released version 3.0.0 that has more verbose errors exposed. I hope it fulfils the requirements of this issue.
Related PR: #124


Running Eval mode issue

Hi there,
Thanks for open sourcing this awesome project! I got an error when running the testModel.sh in eval mode, where when I want to process a .jpg image, it defaults to .png, even when I set the --imageFormat to jpg.
I dug into the codebase, and I believe the issue occurs in line 176, where you construct the dataReader.dataset class. The argument imageFormat = a.imageFormat is not set in the class construction, which results in line 89 of dataReader.py always defaulting to look for .png formatted images.
Replacing line 176 as shown below should resolve this issue:
data=dataReader.dataset(a.input_dir, imageType = a.imageFormat, imageFormat = a.imageFormat, trainFolder = a.trainFolder, testFolder = a.testFolder, inputNumbers = a.nbInputs, maxInputToRead = maxInputNb, nbTargetsToRead = a.nbTargets, cropSize=CROP_SIZE, inputImageSize=a.input_size, batchSize=a.batch_size, fixCrop = (a.mode == "test"), mixMaterials = (a.mode == "train"), fixImageNb = a.fixImageNb, logInput = a.useLog, useAmbientLight = a.useAmbientLight, jitterRenderings = a.jitterRenderings, firstAsGuide = False, useAugmentationInRenderings = not a.NoAugmentationInRenderings, mode = a.mode)

Hi,
Thanks for pointing this out and your interest in our work. The bug came from a miss in the "cleanup" I did before publication, leading to two properties trying to control the same thing. imageFormat should now be properly connected!

I incorporated the fix locally, and it looks good to me! I will close the issue :).


How can I add Button Hover Effect to Suggestions and to adaptive cards

How can i add hover and active effects to the adaptive cards and suggestions  buttons. I'm using React for building web chat

.ac-pushButton{
border-radius: 20px;
}
.ac-pushButton:hover{
background-color: #008AF9;
color: #fff;
border-color:#008AF9!important;

}
.ac-pushButton:active{
background-color: #00498e !important;
color:#ffffff!important;
}
mm, I did some trick using css , but can't i  get a direct styles from js file ?

Hi @keshansandeepa,
Presently, there are no styling options available via Web Chat for adding a hover effect to buttons. The method you have selected is your best option, at this time.
Please feel free to create a new issue as a feature request, if you feel this would benefit the greater community.


adding support for InsertUpdateRelationalTable in silverpop API

Also looks like the Travis build failed, not really concerned about Ruby 1.9 https://travis-ci.org/upworthy/silverpop/builds/156341123

Thanks for getting to this so quickly.
The Ruby 2.1.5 build is failing because of an error with the Bundler version.  I was able to reproduce and using a the newest 1.7.15 version fixed it.  Please add a command to that version's job to use a newer version of Bundler (info to do that here travis-ci/travis-ci#3531).
I would agree that the pre Ruby 2 versions probably don't need to be supported.  I would recommend adding 2.2.5 and 2.3.1 versions and bumping the 2.1.5 to 2.1.9 for the Travis jobs.


Consistent shuffling of recommendations in the feed

Papers should be shuffled for each user independently. The shuffling should be random, but consistent. So everytime a user views a feed from a specific cycle they should get the same ordering. Different users should get different random ordering.
Proposal on how to do it:
Each user gets allocated a secret random seed when they create their account (--> #37 ). The seed is created once on account creation, and is not modified afterwards. The seed is not visible to the user. Given a list of papers to display, we shuffle them based on the seed. For example, something like this might work:
https://stackoverflow.com/a/53758827
Or maybe this:
https://stackoverflow.com/a/28256743

@vtjong you could use the Chance package that seems to be able to take a string as a seed: https://chancejs.com/usage/seed.html


Larger titles in results + spacing

Perhaps a bit subjective, but I find it easier to scan the results if the heading pops a bit more. Trade-off: we lose a little bit on vertical space.
Before:

After:

Great, thank you.


Update submit page

Issue
Fixes #390
Description
Update submit page info, create new pages for extra info
Testing instructions
http://localhost:3000/submit
http://localhost:3000/submit/submission-process
http://localhost:3000/submit/faq

[ ] Link new pages
[ ] Think of adding pages to nav

Looks good!


Fixed search results not rendering.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Allow consumers to set always_include_developer_search_paths in swift_library

https://github.com/bazelbuild/rules_swift/pull/1162

Discussed offline and we agreed that instead of bumping rules_swift in rules_ios it provides a better DX for both consumers and maintainers to simply allow the flag always_include_developer_search_pathsto be set in kwargs and propagated to the swift_library. Letting consumers decide if they want to bump to a newer rules_swift version or not to start setting that flag in their BUILD files.

Shouldn't the flag be always_include_developer_search_paths instead of swift_ always_include_developer_search_paths?

Shouldn't the flag be always_include_developer_search_paths instead of swift_ always_include_developer_search_paths?

@mattrobmattrob From a mixed frameworks perspective I think we should explicitly convey that this is a Swift thing and not Obj-C. Mostly thinking of a rules_ios user that is not aware of all the underlying impl details of the rules and want it to "just work". In that scenario the names used in the API are the main way to interact with the rule set and IMO we should be very explicit here.
Open to change it though in case I am missing something here!

I do wonder if we should come up with a general approach for passing rule attributes through to this point vs. a one off flag. Doesn't seem like we have a pattern for it at this point though.

@mattrobmattrob +1, I would still prefer to land this change to unblock ongoing work but I'm open to contribute to an issue/proposal on that. Def something I thought about in the past too while working on the rules.

Shouldn't the flag be always_include_developer_search_paths instead of swift_ always_include_developer_search_paths?

@mattrobmattrob From a mixed frameworks perspective I think we should explicitly convey that this is a Swift thing and not Obj-C. Mostly thinking of a rules_ios user that is not aware of all the underlying impl details of the rules and want it to "just work". In that scenario the names used in the API are the main way to interact with the rule set and IMO we should be very explicit here.
Open to change it though in case I am missing something here!

Deleted my comment but wasn't fast enough. I thought you were passing it through to the swift_library with the prefix but realized I was wrong.


Support Named Token String Replacement

Support Named Token String Replacement
As a PowerShell developer, I want to allow variable name interpolation, to improve readability and maintainability of strings used as a template.
For example:
$HereString =
@'
use [{0}]
select * from dbo.table where Databasename like '{0}%'
'@

Right now we can do this via the example above, which means if you begin to pass in 5-6 variables, you have to maintain the 0 index position for them carefully.
More intuitive text similar to python's string formatting, and later c# string replacement might be:
$HereString =
@'
use [{DatabaseName}]
select * from dbo.table where DatabaseName like '{DatabaseName}%'
'@

I know we can use:
$HereString =
@"
use [$DatabaseName]
select * from dbo.table where DatabaseName like '$DatabaseName%'
"@

In most cases this works fine. However, if you are generating a text string like this that you want to pass to a parallel pipeline and iterate on then the improved syntax would help things be much more readable and intuitive.
This is a sample of where this syntax would be much more intuitive.
Sample code, not used for-eachobject paralle, so syntax might be slightly off
$HereString =
@'
use [{DatabaseName}]
select * from dbo.table where DatabaseName like '{DatabaseName}%'
'@

@('Database1','Database2','Database3') | ForEach-Object -Parallel {

$DatabaseName = $_
Invoke-Query -Query $HereString # this or some approach to include the variables by name mean I don't have to do string format replacement in this scope now

}

Note
I couldn't find any issue on supporting named tokens but I have to believe something exists. If so, please just link it and close this as duplicate and I'll add my 👍  to it. Thanks!

This will works:
@('Database1','Database2','Database3') | ForEach-Object -Parallel {

$DatabaseName = $_

$HereString =
@"
use [{DatabaseName}]
select * from dbo.table where DatabaseName like '{DatabaseName}%'
"@

Invoke-Query -Query $HereString # this or some approach to include the variables by name mean I don't have to do string format replacement in this scope now

}

You can use $ExecutionContext.InvokeCommand.ExpandString to expand a verbatim string as if it were an expandable one on demand:
$HereString =
@'
use [${DatabaseName}]
select * from dbo.table where DatabaseName like '${DatabaseName}%'
'@

$DatabaseName = 'foo'

$ExecutionContext.InvokeCommand.ExpandString($HereString)

The caveat is that you should only do that if you fully control or implicitly trust the content of the string template ($HereString) to not contain malicious commands, since it's possible to embed arbitrary commands with $(...).

It would be nice to surface this functionality as a cmdlet, say Expand-String.
Such a cmdlet could then support a switch, say, -VariablesOnly to prevent execution of embedded commands.


Custom Validation Rules - Not seem to be working?

const CONTACT_FORM = document.querySelector('#FormContact');

var validator = new FormValidator(CONTACT_FORM, [{
    name: 'contact-name',
    rules: 'required|valid_name'
}, {
    name: 'contact-email',
    rules: 'required|valid_email'
}, {
    name: 'contact-subject',
    rules: 'required'
}, {
    name: 'contact-message',
    rules: 'required|min_length[50]|max_length[500]'
}], function(errors, event) {
    if (errors.length > 0) {
        console.log(errors);
    }
});

validator
    .registerCallback('valid_name', value => /^[a-zA-Z ]{2,30}$/.test(value))
    .setMessage('valid_name', 'Please enter your full name.');

As it seem to me, it does not pick up custom validation rule. I tried hardcoding to always return false and still it passes like it is truthy...
validator
    .registerCallback('valid_name', () => false)
    .setMessage('valid_name', "You can't pass this rule.");

Oh and there's one more thing. The contact-subject has a rule set to required which is select element having disabled and selected default value, so I assume the expectation of validator is to say that field is not valid since it is pending non-disabled selection, right?

In the FormValidator definition, the rule needs to be callback_valid_name like this:
rules: 'required|callback_valid_name'

You can read more about custom rules here: http://rickharrison.github.io/validate.js/


Python3.8 support

Remaining problems:

tuple hash algo changed. https://bugs.python.org/issue34751
looplifting is broken due to CFG assumption changed.

Edit The fixes for the above unresolved py3.8 problems will be splitted to separate PRs.  This PR now contains the necessary changes to make Py3.8 mostly working.

Yeah! First CI all pass

New code path running on py3.7 now

Added py3.8 CI and skip test for looplifting on py3.8

Was wondering RE your comment above if the hash tests for tuple are disabled on 3.8, if they are I missed it?

The test are missed due to the test slicing.  numba.tests.test_hashing.TestTupleHashing shoud fail on py3.8

kicked CI

kick CI again

alrighty, going to merge

Good job @sklam :)


podMetadata and WorkflowTemplate do not work together

Summary
Meta information from podMetadata defined in WorkflowTemplate is not applied to pods from workflow.
Diagnostics
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: simple-wft
spec:
  entrypoint: whalesay
  workflowMetadata:
    labels:
      token: token
  podMetadata:
    labels:
      token: token
  arguments:
    parameters:
      - name: message
        value: hello world
      - name: token
        value: 123
  templates:
    - name: whalesay
      inputs:
        parameters:
          - name: message
      container:
        image: docker/whalesay:latest
        command: [cowsay]
        args: ["{{inputs.parameters.message}}"]

Workflow definition:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: example-workflow-
spec:
  arguments:
    parameters:
        - name: message
          value: hello world
        - name: token
          value: 123
  workflowTemplateRef:
    name: simple-wft

Pod description:
❯ kdp example-workflow-fxbqs
Name:         example-workflow-fxbqs
Namespace:    argo
...
Labels:       workflows.argoproj.io/completed=true
              workflows.argoproj.io/workflow=example-workflow-fxbqs

What Kubernetes provider are you using?
EKS
What version of Argo Workflows are you running?
3.0.2

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

it worked, thank you!


fix(minimal-slp-wallet): Updating to v5.5.3

:tada: This PR is included in version 4.0.3 :tada:
Your semantic-release bot :package::rocket:


make kube-dns main dns server

What this PR does / why we need it:
This PR makes kube-dns the default DNS server on Windows nodes.
This will enable resolution of service names.
Special notes for your reviewer:
This PR needs acs-engine PR#3104

@sharmasushant please review. Thanks!


Android support

hi,
this is agreat software solves alot of problems, so THANKS.
is there any chance for android support?

For the foreseeable future, anything other than a desktop OS will not beyond the scope of the project.


New style for widgets with pop-up input controls

This adds a pop-up control bar, and I put a little bit of effort into styling:

A few advantages:

The input field is hidden, not just disabled (less user confusion)
More horizontal room (can add explanatory text)
The "Restart" button now does something obvious (open the input box back up)

Looks great!


[SPARK-14873] [CORE] Java sampleByKey methods take ju.Map but with Scala Double values; results in type Object

What changes were proposed in this pull request?
Java sampleByKey methods should accept Map with java.lang.Double values
How was this patch tested?
Existing (updated) Jenkins tests

Test build #56797 has started for PR 12637 at commit 0fe43b4.

Test build #56797 has finished for PR 12637 at commit 0fe43b4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56797/
Test FAILed.

Jenkins retest this please

Test build #56799 has started for PR 12637 at commit 0fe43b4.

Test build #56799 has finished for PR 12637 at commit 0fe43b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/56799/
Test PASSed.

LGTM merging in master.


Disallow clicks on land

We want to do something similar to this: https://github.com/ua-snap/iem-webapp/pull/75 but where the ocean is the only portion allowed to be clicked.


metricKey on metric_events required when it ought not be

I'm creating a template for metric event definitions for my organization's applications using pulumi typescript.
I've found a discrepancy on defining a metric event for the type METRIC_SELECTOR, where the value  metricKey on the MetricEventsQueryDefinition type is required, even when it should not be (eg, type would be METRIC_KEY rather than METRIC_SELECTOR when it ought be required).
It's unclear to me if I'm misconfiguring the below, however I believe there should be a change to set metricKey as optional on this resource. I see on the corresponding terraform object that this specific key is labeled optional: https://registry.terraform.io/providers/dynatrace-oss/dynatrace/latest/docs/resources/metric_events#metric_key
        const metricQueryDefinition: MetricEventsQueryDefinition = {
            metricKey: metricEventData.queryDefinition.metricKey,
            type: metricEventData.queryDefinition.type,
            metricSelector: metricEventData.queryDefinition.metricSelector.replace(/REPLACE_ME_HOST_ID/g, host),
            queryOffset: metricEventData.queryDefinition.queryOffset,
        }

Error when the metricKey is set on a METRIC_SELECTOR type event:
  dynatrace:index:MetricEvents (ix-test-example-Mem Usage):
    error: 1 error occurred:
        * Given property 'metricKey' should not be set as it does not satisfy the following precondition: type = 'METRIC_KEY'

Error when the metricKey is not set, on a METRIC_KEY type event:
  dynatrace:index:MetricEvents (ix-test-example-REPLACE_ME_PROJECT_NAME Mem Usage):
    error: 1 error occurred:
        * Given property 'entityFilter' with value: 'null' does not comply with required NonNull of schema
    Given property 'metricSelector' should not be set as it does not satisfy the following precondition: type = 'METRIC_SELECTOR'
    Given property 'aggregation' with value: 'null' does not comply with required NonNull of schema```

@kyleian I've brought this inline with the upstream TF provider. Did this fix your issue?


fix samples.json with proper github URL

See https://github.com/Microsoft/openshift-container-platform/blob/master/azuredeploy.parameters.json
and https://github.com/Microsoft/openshift-container-platform/blob/master/azuredeploy.parameters.sample.json
It contains
"_artifactsLocation": {
"value": "https://raw.githubusercontent.com/haroldwongms/openshift-containerplatform/master"
},
In that case, the deployment will fail complaining about diagStorageAccount and Role not being valid inputs in the template.
Even though both files are just samples, for the sake of first time users, please update the github URL to point to this repo ("https://raw.githubusercontent.com/Microsoft/openshift-container-platform/master") or delete the variable altogether. Otherwise, the error is very esoteric and hard to troubleshoot.
Thanks

It seems to me that this one thing is a bit messy, it is possible to make this dynamic, so that it will point at the repo from which the things are fired off from?


OkHttpClient.Builder refers to the missing type Interceptor

Getting an error when try to use this:
Exception in thread "main" java.lang.Error: Unresolved compilation problems:
The method interceptors() from the type OkHttpClient.Builder refers to the missing type Interceptor
The method addInterceptor(Interceptor) from the type OkHttpClient.Builder refers to the missing type Interceptor
at com.binance.api.client.impl.BinanceApiServiceGenerator.createService(BinanceApiServiceGenerator.java:38)
at com.binance.api.client.impl.BinanceApiRestClientImpl.<init>(BinanceApiRestClientImpl.java:37)
at com.binance.api.client.BinanceApiClientFactory.newRestClient(BinanceApiClientFactory.java:58)
at binance.BinanceScapper.main(BinanceScapper.java:13)

Hi @kernelogic, could you please provide the full stacktrace and your project's pom file?

It might be an issue with my Eclipse. I didn't do anything, just download your repository and import into Eclipse. Immediately see this error.

Oh ya never mind. It turned out this particular class was corrupted during maven download. I cleaned up .m2 folder and it's fine now.


✨[feature Request]: Missing Animation with Hover Shadow in Login Page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The login card and "Login to Save your Data!" heading can have an animation effect on hover to make the design more interactive. You can add a hover effect to slightly increase the shadow and change the card's position, making it look dynamic.
Use Case
By using a hover animation on the login card and interactive heading text, the user is visually guided to the input fields, making the login process more engaging and easy to focus on. This can also be useful in highlighting important actions (like Login) on your page.
Benefits
The animation and shadow effects will draw attention to the login form, making the page feel responsive and modern. The subtle design improvements will encourage interaction while maintaining a professional appearance.
Add ScreenShots

it contains slitter shadow but not contain hover effect to slightly increase the shadow and change the card's position, making it look dynamic
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Anjaliavv51
Hii assign this part to me


support verifying against multiple PEM certs

Google OAuth 2.0 uses two certs. Would be awesome if we could supply a concatenated string of all of them, and node-jsonwebtoken checked each in turn.
EDIT: I need to check them all because the PEM doesn't include the kid. If you guys supported JWK, the problem wouldn't exist.
EDIT2: I realized google supplies kids even for their PEM certs. But I guess the issue might be helpful to others.

^ Maybe this should be added as a remark in docs ?

Here is how I solved this back then:
https://github.com/auth0/connect-google-jwt/blob/master/index.js
I haven't use this code in a while, it might be outdated.

So, for the case of google, you should first get the kid by using jwt.decode and then use the correct PEM to validate the token.


feat(aztec-nr): initial work for aztec public vm macro

Performs initial work for "aztec-vm" macro.
Current implementation is NOT the final form, but it is moving us in that direction.

Adds indirect (hardcoded as 0 for now) to the transpiler
Renables the transipler test in the avm directory

fixes: https://github.com/AztecProtocol/aztec-packages/issues/4269

Current dependencies on/for this PR:

master

PR #4400   👈

This stack of pull requests is managed by Graphite.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (8): 110 (+575%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 8569e7c0 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,636
180,516
720,036

l1_rollup_calldata_gas
222,564
870,552
3,464,112

l1_rollup_execution_gas
315,051
988,292
3,683,769

l2_block_processing_time_in_ms
1,151
4,342 (-1%)
17,250

note_successful_decrypting_time_in_ms
301 (-1%)
961 (-1%)
3,608

note_trial_decrypting_time_in_ms
:warning: 110 (+575%)
88.5 (-11%)
147 (+5%)

l2_block_building_time_in_ms
18,104 (+2%)
70,161
281,641

l2_block_rollup_simulation_time_in_ms
13,276 (+1%)
51,684
207,997

l2_block_public_tx_process_time_in_ms
4,777 (+3%)
18,406
73,401

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,461 (+4%)
27,361 (+4%)

note_history_successful_decrypting_time_in_ms
2,319 (-3%)
4,552 (-5%)

note_history_trial_decrypting_time_in_ms
157 (+67%)
211 (+18%)

node_database_size_in_bytes
17,322,064
33,001,552

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
310
44,481
26,193

private-kernel-ordering
192 (-1%)
44,513
14,929

base-rollup
1,406
128,970
881

root-rollup
82.3
4,088
677

private-kernel-inner
414
71,640
26,193

public-kernel-private-input
239
32,615
26,193

public-kernel-non-first-iteration
238
32,657
26,193

merge-rollup
7.61
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,575
37,974

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
545
1,214

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.477 (-23%)
570


update outdated example code

@hf-kklein : Thanks for your contribution! The author, @rloutlaw, has been notified to review your proposed change.

The proposed changes are reflected in the article on the live site, so we'll close the pull request.


fix(image): scale image after it was rotated (#111)

Closes #111

:tada: This PR is included in version 1.18.8 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[NETBEANS-1074] Module Review j2ee.clientproject

no external library
checked Rat report: changed license headers of five file
skimmed through the module, did not notice additional problems

Thanks.

@thilina01 It seems that application-client-1.4.xml e.t.c. are template files. So please remove thier license heders, then add them to licenseinfo.xml.
see https://github.com/apache/incubator-netbeans/blob/c8c697418cc0bef6ba070b3e157b5b67cc4080a4/j2ee.clientproject/src/org/netbeans/modules/j2ee/clientproject/api/AppClientProjectGenerator.java#L162-L174

Thank you @junichi11
Just realized how to identify template files. :+1:


Web Audio Error

Affecting chrome 43 when loading in a game.

I guess it throws now instead of doing nothing if there wasn't anything to disconnect.

Commenting the line fix the issue for now

I love web tech! It's so bad! Cheap fix pushed, I don't care to figure out why the exception is happening in the first place.

It's still broken, you have to add the try{} on the line 37 for fixing Chrome also.

Does it need a try catch anyhow? Thought testing for truthiness was enough on the node to disconnect. Anyhow that's a very minor nitpick. Thanks for updating it


Edit button

add edit button in contest.html

+1 besides the comment.

fixed

+1, going to merge


Creating new tree from seed node removes node from original tree

Hey, thanks for your work on this library!
I'm running into an issue that I don't quite understand when creating subtrees. I'm not sure if it's a problem with the code or if I'm just taking the wrong approach.
I have one large tree as my dataset, and in my analysis I am finding certain nodes, and then creating another tree using their parent node as the root. I wanted to create a straight copy of the structure, without any filter functions, and with all nodes intact. The base dataset is huge, so I didn't want to use the extract_* methods to filter through all the nodes that way. The problem is that once I have done this, all nodes in the new tree seem to have disappeared from the original dataset. Here is the line that causes the issue:
node = tree.find_node_with_taxon_label(id)

while True:
    node = node.parent_node
    if len(node.child_nodes()) > 1:
        break

subtree = dendropy.Tree(seed_node=node)

Now, if I try and find a node in the original tree, that was within the bounds of that subtree, it fails:
# Same ID as before, returns `None`
node = tree.find_node_with_taxon_label(id)

This seems a bit strange, since semantically I would have assumed that Python would treat the seed node assignment by retaining a reference to the original node, rather than moving everything. I'm by no means a Python expert though, so I could well be wrong...
Deep copying the node works fine:
subtree = dendropy.Tree(seed_node=copy.deepcopy(node))

Am I just making the wrong assumption on the behavior of this method here, or is there something going wrong?
If it's just me, I'd be happy to make a PR with a method that performs a deep copy of the seed node. This unfortunately stumped me for an embarassingly long amount of time!

Thanks for the report! Just added a warning on any implicit splice operations. The deepcopy is a good workaround, calling dendropy.Tree(seed_node=node.extract_subtree()) should work too.


e2e test where we call a private method not via an account contract

Description
Fixes #1011
Checklist:

[ ] I have reviewed my diff in github, line by line.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[ ] The branch has been merged or rebased against the head of its merge target.
[ ] I'm happy for the PR to be merged at the reviewer's next convenience.

Currently blocked because get_note call is broken (callback not implemented). Leila is just working on a fix in https://github.com/AztecProtocol/aztec-packages/pull/1084 and she is about to merge it soon so i decided to wait for the fix to be in before continuing on the task.


misspelling "sensor"

"List of blocks that will trigger the senor."

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

ID: a09d8820-84ca-d267-1fc6-5592d9eb51fc
Version Independent ID: afc98587-78e9-5054-5139-a00faaf372b6
Content: Entity Documentation - minecraft:block_sensor
Content Source: creator/Reference/Content/EntityReference/Examples/EntityComponents/minecraftComponent_block_sensor.md
Service: minecraft-bedrock-edition
GitHub Login: @JimSeaman42
Microsoft Alias: mikeam

Thanks for your feedback! The spelling error has been addressed, and will be updated with the next release.


Constant folding

This proposes a single optimization, that improves two things at once:

constant complex expressions that work in python are pre-evaluated and will work in OpShin too (i.e. bytes.fromhex("0011") and list(range(0,10,3)))
runtime is improved because at a high level constant values are precomputed (i.e. 2**24 is not computed on-chain but at compile time)

One final thing that needs to be addressed here is that we need to keep track of user-defined scopes (potentially re-assigned variables)

I just checked the examples and they have converged to a state where there is almost nothing that could be constant folded (except for some PlutusData constants). Might be a results of this feature missing so far though.

Since this only adds 0.1s to the testbench, I will merge this feature. Notably, proper constant folding is still a desired feature and may follow later (potentially even based on this implementation).
Hopefully users won't get too confused about the fact that certain statements work as long as they are constant (i.e. bytes.fromhex("00") but not when they are dynamic (i.e. bytes.fromhex(a)). The fact that the writer of the contract now does not have to worry about the cost of evaluating constant statements should greatly improve readability though.


css problem when horizontal scrolling

Original table.

After scroll to right.
Need to set display:block or something?

i'm got same issue, when i tried using with bootstrap tab, please help

try using
$(document).ready(function(){
$(".freeze-table").freezeTable({...});
});


whats the difference between FactorizerBlock and FactorizerBlockV2?

V2 use learnable DimWisePosEmbed, while V1 do not?

Hi @earsonlau! FactorizerBlock and SegmentationFactorizer correspond to the original version that can be used to reproduce the results presented in the paper. FactorizerBlockV2 and SegmentationFactorizerV2 are internal trials related to my attempts at making improvements over the original version of Factorizer and achieving a modified version, i.e., FactorizerV2. This work is still in progress, and I am currently performing ablation experiments to obtain a better config and design for the 2nd version.  FactorizerV2 with major updates and improved results will be released soon🤞, Stay tuned!


Issue on rowRender's TypeDefinitions?

Had an issue using Typescript while trying to pass style on rowRenderer. TypeDefinitions for rowRenderer may be wrong.
rowRenderer: (info: { index: number, isScrolling: boolean }) => React.ReactNode;
Shouldn't that be:
rowRenderer: (info: { index: number, isScrolling?: boolean, style: any, key: any }) => React.ReactNode;

Sounds like an issue that should be filed against DefinitelyTyped/DefinitelyTyped (for @types/react-virtualized). I don't maintain that project (or use TypeScript). 😄 Sorry!


feat(diag): add mongo enhanced query collection

Adds experimental (opt-in) configuration option on Config to opt into sending more detailed query information for autocollected dependencies. Currently only mongodb queries will work. Support for other libraries will be added in separate PRs (and future releases).
This feature is intended to be experimental in that the format of the data or the actual data could change in future releases. Other libraries will also be supported, which would cause more telemetry volume to be sent if this feature is enabled.

Now that this is rejected, is there an alternative way to get more detailed logs from Mongo DB in Application Insights? In particular I am looking for getting the name of the collection being queried/operated on.


How to retrieve correct image when image url did not change but image changed such as Twitter or GitHub?

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to reoprt a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
How to retrieve correct image using Kingfisher when image url did not change but image changed such as Twitter or GitHub.
SDWebImage has a option [SDWebImageRefreshCached] to solve the problem.
   /**
     * Even if the image is cached, respect the HTTP response cache control, and refresh the image from remote location if needed.
     * The disk caching will be handled by NSURLCache instead of SDWebImage leading to slight performance degradation.
     * This option helps deal with images changing behind the same request URL, e.g. Facebook graph api profile pics.
     * If a cached image is refreshed, the completion block is called once with the cached image and again with the final image.
     *
     * Use this flag only if you can't make your URLs static with embedded cache busting parameter.
     */
    SDWebImageRefreshCached = 1 << 4,

I am afraid this is not supported in Kingfisher now. Kingfisher is not using NSURLCache system so there is no way to refresh the cache in that behavior.

A new option fromMemoryCacheOrRefresh was added  in version 4.3.0 as a "similar" solution in Kingfisher cache. Instead of checking the cache setting for URL response (in the NSURLCache), this option will always download the image and cache it to memory. Once it is in memory, the same image will not be downloaded again (as the behavior of  forceRefresh) until it disappears from memory cache.
Since Kingfisher is not using the URL cache system, it also ignore the cache policy provided by the response. If you want a behavior like that, to respect the cache control tag of response header, you might want to just use a plain URL session and do not do anything additional on the data.

A new option fromMemoryCacheOrRefresh was added in version 4.3.0 as a "similar" solution in Kingfisher cache. Instead of checking the cache setting for URL response (in the NSURLCache), this option will always download the image and cache it to memory. Once it is in memory, the same image will not be downloaded again (as the behavior of forceRefresh) until it disappears from memory cache.
Since Kingfisher is not using the URL cache system, it also ignore the cache policy provided by the response. If you want a behavior like that, to respect the cache control tag of response header, you might want to just use a plain URL session and do not do anything additional on the data.

I encountered the same problem, is the current version optimized for this situation？


Oil tab missing

Looks to me like the only thing missing is the oil tab, not really clear who needs to do what here. Is this a task for @georgeslabreche or @IronRunes?

Duplicate of #14, closing.


Image size in project pages

Hi there,
Thanks for this amazing theme!! I was able to make this site in 2 days: https://www.lukeholman.org/. Much nicer than my old site, which took forever.
I have 2 questions, not sure if they are solvable within the current theme:

Is is possible to change the title of the "Education" section to something like "Career history", and remove the little graduation caps for stuff that is not education? If you look at my page, you'll see what I'm aiming for.

Is there a way to adjust the size of the images when you click on a project and enter the specific project page? The images are massive and fill the whole screen, which looks nice up to a point, but they're so big that people might not know that they can scroll down to see the content.

Thanks heaps!
Luke

Bah, I'm dumb - just noticed that the image cleverly changes size with the width of the page. It works great on a phone, and not so well on a massive desktop screen. I guess maybe what I need, then, is a way to set the maximum width, to stop it getting too huge on big screens?

Thanks for the feedback :)

Refer to the paragraph beginning "To edit the interface text" in the documentation at https://sourcethemes.com/academic/docs/language/ .

Customization of the CSS image style would be required to change image size/layout - see https://sourcethemes.com/academic/docs/customization/#customize-style-css to get started and then the support forums if you want further guidance.


[general] return error early to avoid nil pointer usage

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
Related #1684
Special notes for reviewers:

Release note:

NONE

thanks for submitting this PR @jichenjc, but I don't think this PR is actually fixing the issue #1684, so please just use related #1684 in the description.

/lgtm

@lingxiankong updated, thanks~

I don't know why we have the error check before, but the change seems to be good to me.
/lgtm

@ramineni can you help review this? Thanks

I too don't understand why we have that check. Change looks reasonable to me.
/lgtm
/approve


Meeting 2023-05-18

Agenda

Welcome / Roll call
Action Items Update

(Chassin) Transfer code from slacgismo to arras-energy
(Le) Seek new members from California IOUs/POUs

Work progress update

HiPAS GridLAB-D 4.3.1 status
Open pull requests
Current issues
Dockerhub automation for build of images
Arras-energy dockerhub account

New topics
Approval of minutes
Action items

(Chassin) Transfer code from slacgismo to arras-energy
(Le) Seek new members from California IOUs/POUs

Adjourn

@Allen-Le-CEC, @anjames16, @dpinney, @ftuffner, @iut62elec, @jmertic: reminder that the May 2023 TSC meeting is today at 10am PDT. Your attendance will help us make a quorum.
If you do not have an invitation, please contact @jmertic.

No quorum. Meeting notes are for information only.


Generator Proposal

I think Faker need a Brand Generator, like something "Ford","Chevrolet","Nokia","Samsung",etc.

Great idea! I suggest you create it as a standalone provider, host it in your own account, and send me a PR so that I can link it in Faker's README.

Perfect


Ignore party field in public API GetExchangeRequest.

This also exposes the Exchanges and RecurringExchanges services in the public API server. Since these services were never exposed, this is not a breaking change.

This change is

See https://github.com/world-federation-of-advertisers/cross-media-measurement-api/issues/138


add items/item depencencies

This PR adds dependencies for both /items and /items/{itemID} endpoints
This will enable custom backend which could allow more/less query parameters

🤔 I think this is over-engineered if user need to override endpoint input it will be much easier to re-write most of the endpoints like in https://github.com/developmentseed/tipgstac/blob/main/tipgstac/factory.py


Make test names match

Names of tests in the code vs #6 differ. Either the in-code names need to be updated, or the issue names.

Ship Tests

[ ] #10 or shipMove
[ ] #11 or shipFires
[ ] #12 or gainMoney
[ ] #13 or gainPoints
[ ] #14 or progressTasks

Building Tests

[ ] #17 or buildingFires
[ ] #18 or buildingCapturable

Game State Tests

[ ] #15 or winGame
[ ] #16 or loseGame
[x] #19 is same in code

Decisions

Ship Tests

Used shipMove and refactored to use a Ship instead of a Player.
Used shipFires as this function uses a Ship.
Used playerMoney, playerPoints and playerProgressTasks as these tasks pertain to the Player specifically.

Building Tests

Used buildingFires as it matches naming scheme of shipFires.
Used buildingCapturable as it matches other new names.

Game State Tests

Used winGame and loseGame as these tests pertain to the state of the game over that of the Player.


Version Conflict for scikit-learn - Bump to 1.3.2 possible?

Hi thank you very much for this helpful package.
Refering back to a recently successful closed ticket: https://github.com/AutoViML/featurewiz/issues/84
would it again be possible to might bump the sklearn depency to the latest 1.3.2
followed by a new release of featurewiz? This would be highly appreciated.

Hi @Buedenbender 👍
This is likely possible but I believe it will break a few things. I will test it and let you know
If you can test it and tell me where it breaks, that can also speed up my implementation.
Thanks,
Auto Vimal.


Update slayer-wiki

I re-created this plugin from scratch, so many files might have changed. I kept the same (updated) logic for the plugin itself
Bugs have not been replied to for years. No commit in years. Felt that taking the ownership is the right thing to do.
Tagging @ffalor just in case.

@guillaume009 works for me, sorry about abandoning. Work just started taking all my free time.

It's preferred that you keep the history in your fork so we can document the transfer.

Modified my repo to be a fork of the original


Unable to reactivate LND Static Channel Backup on Nextcloud

RaspiBlitz v1.7.2   LND 0.14.2-beta
I've been having a few issues reactivating LND static channel back up on Nextcloud after the latest update.
It used to work well for a few months, then it stopped working.
I used the free nextcloud service provided at https://free.files.cnow.at/
I tired to get it working again by firstly de-activating through the SSH menu, and then activating the LND Static Channel Backup.
After activating, I am then prompted to enter the server URL, then user name, then password.
It accepts and then runs and flashes briefly on the screen the following:
...
Nextcloud Setting changed ..
rm: cannot remove '/var/cache/raspiblitz/.tmp' : Operation not permitted
sed: -e expression #1, char 59: unknown option to ‘s’
err=’Nextcloud credentials are missing’
...
No files are written to the server and the LND Static Channel Backup on Nextcloud option does not have a * in the menu
I then also restarted the node and tried again to no avail.
I have also tried two other nextcloud servers
https://use05.thegood.cloud/
https://nextcloud05.webo.cloud/
But again to no avail.

Maybe related and fixed in issue #3116 which will be part of upcoming v1.8.0 release. Please test if an update to v1.8.0 is the fixes this once its available and report back.

@benniblitz  is this still relevant in 1.9.0? Closing for now, if it's still relevant, please re-open


refactor + test

Get rid of the hasattr call and cover with a (very) basic test; better solution for #308

Current coverage is 57.40%

Merging #475 into master will increase coverage by +0.04% as of 0bf9e6c

@@            master   #475   diff @@
=====================================
  Files           49     50     +1
  Stmts         9143   9154    +11
  Branches      1543   1544     +1
  Methods          0      0
=====================================
+ Hit           5245   5255    +10
- Partial        515    517     +2
+ Missed        3383   3382     -1

Review entire Coverage Diff as of 0bf9e6c

Powered by Codecov. Updated on successful CI builds.


Blazor Server, how much memory does it user per connect client on server ?

[EDIT by guardrex to add the metadata]
As per various document i have read , it is mentioned that Blazor Server manages state for each connected client.
How can we do sizing for Blazor Server side, How do we get a number on how many concurrent user can a Blazor Server side support?
What kind of memory does it use for each connected user what does it store on server ?
Can timeout be configured for longer or shorter duration than the default ?

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

ID: 7367a651-2b27-0a57-e7d0-95686cc97f1c
Version Independent ID: 8c769648-728d-72ce-ebf6-2a06e5076aa2
Content: ASP.NET Core Blazor configuration
Content Source: aspnetcore/blazor/fundamentals/configuration.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @nssidhu ... You can find information on SignalR (which Blazor Server is based on) in the SignalR doc set. There's some information on scale in the SignalR: Host and scale node on these subjects. But before looking at SignalR docs, we have some Blazor-specific guidance in the Blazor area of the doc set:

There's information on what's stored in a Blazor Server circuit in the Blazor: State management topic.
There's information on Blazor Server's scale, limits, and connections in the Blazor: Threat mitigation topic.

The amount of memory consumed per connected user can't be stated precisely because it depends on the app and components. There might be some rough minimum for a minimal component, but I don't know what that value is. It also likely changes a bit from release to release. It's probably been getting smaller each release as the framework engineers improve Blazor.
One thing that I'd like to do now that we have a new Blazor Configuration topic is add a section there on Blazor Server configuration, including the circuit options described in Blazor: Threat mitigation: Denial of service (DoS) attacks with a cross-link to that DOS content.
Leave this issue open. It will close automatically when the PR merges. Thanks for commenting.


feat: replace mailhog with mailpit

Hi,

Mailpit is a rewrite of Mailhog.
Mailpit has same ports as Mailhog
Work as SMTP

Read more here
https://github.com/axllent/mailpit

You already had laravel sail docker image on your machine and mailhog container was running?

All good now :)


Add support for gp2 EBS volume type

@irvingpop @scalp42 this constant does not appear to be used, so adding this value will have no effect, however you can just pass an options hash with the key :volume_type set to 'gp2' to create gp2 ebs volumes

This is old now and was related Chef aws cookbook being based on right_aws (and not having control over it without forking). Everything has been moved to aws-sdk since then @dallasmarlow


Support lazy message evaluation

A simple example could be:
import log from "loglevel"
log.debug(()=>JSON.stringify(value));

I belive this feature will make loglevel very efficient.

This is an interesting idea, but isn't something the core of loglevel is ever going to do. Any logic that needs to run at log time necessitates clobbering all stacktraces. Reliably giving you simple working logic without breaking anything (including traces from your log messages) is one of the core features of the library.
Extensions like this are the perfect case for plugins though. Take a look at https://github.com/pimterry/loglevel#writing-plugins in the docs for a bit more background on why this can't really be built in, and how you can set up your own plugin to build that on top of loglevel anyway.
I'm going to close this since I'm not happy to put it into the core library. Do let me know if you end up writing and releasing a plugin though, and I'll happily add a reference to it in the README.


Fix hack/relprep.sh script to match KEDA repo

Previously the relprep script assumed we always wanted the latest go toolchain, but that might not be the case for point releases. Now we will explicitly match what KEDA used for a given release.
Same goes for sigs.k8s.io/controller-runtime, sigs.k8s.io/controller-runtime/tools/setup-envtest and sigs.k8s.io/controller-tools.
Checklist

[X] Commits are signed with Developer Certificate of Origin (DCO)

Makes sense -- if we ever need to skew on those deps for a good reason we still can, we'll just be more explicit about it. Looks good to me!


Cannot set more than one cookie to a single response

Caveat: I'm limited in how much code I can show due to work restrictions.
Express: 4.16.3
Serverless: 1.30.0 (I just checked and see how out of date this is)
I have an express application served via Serverless. Serveless is effectively acting as a proxy for all requests with a function definition (serverless.yml) as follows:
service: service-lambda

provider:
  name: aws
  runtime: nodejs8.10

plugins:
  - serverless-offline

functions:
  index:
    handler: index.handler
    events:
    - http: ANY /
    - http: 'ANY /{proxy+}'

And the Index.js is simply:
const serverless = require('serverless-http');
const app = require('./app');

module.exports.handler = serverless(app);

App defines the express app as standard. I'm using cookieParser and it is set up prior to all my routes. All the routes check for the presence of a specific cookie and if it is not there the user is sent to okta for authentication then redirected back to a specific url.
For testing purposes that route is simply called /authenticated. Upon returning from okta and authenticating, this route authenticates the access token and attempts to apply cookies that include information returned as part of the verified JWT.
It then renders an ejs view called "authenticated".
I have 3 cookies I'm trying to set thusly:
res.cookie('oscp', oscp[0]);
res.cookie('ocid', ocid);
res.cookie('accessToken', at);

I was expecting to see all three tokens but I'm only getting the last one.
I did some googling and this bug previously existed but was fixed when Connect was removed I believed. I've tried setting them all at once via setHeader but that didn't work as well.
I can also confirm that the cookies are being sent out with the response via the logs as you can see here:
Serverless: [200]
{
  "isBase64Encoded": false,
  "statusCode": 200,
  "headers": {
    "set-cookie": "oscp=openid; Path=/",
    "Set-cookie": "ocid=0oaghi5emxJaZojD90h7; Path=/",
    "sEt-cookie": "ouid=00ufev7ma0qtC3pep0h7; Path=/",
    "content-type": "text/html; charset=utf-8",
    "content-length": "1418",
    "etag": "W/\"58a-iDjFIYcKwbQzooCSb8pKYrQ6X4w\""
  },
  "body": "<!DOCTYPE html>\n\n\n    <meta charset=\"utf-8\">\n    <meta name=\"viewport\" content=\"width=device-width, initial-scale=1, shrink-to-fit=no\">\n    <link rel=\"stylesheet\" href=\"https://stackpath.bootstrapcdn.com/bootstrap/4.1.3/css/bootstrap.min.css\" integrity=\"sha384-MCw98/SFnGE8fJT3GXwEOngsV7Zt27NXFoaoApmYm81iuXoPkFOJwJ8ERdknLPMO\" crossorigin=\"anonymous\">\n    <title>more than one</title>\n\n\n
\n
\n
\n
TEST AUTHENTICATION
\n
\n
\n
Query Code: LbujxbMv8MeQQ0V_-Gwm
\n
Query state:state-6a0cd73c-e8a7-415f-abff-e3ede4885461
\n
Go Home
\n
\n
\n
\n\n\n\n\n"
}

I have a workaround (basically re-verify the access token as the only cookie I can save) but I'm not a fan of it and checking with some coworkers (and the googling) I should be able to set multiple cookies in a row.

Thanks for the report! So there is a lot going on there, but mainly I'm confused as to what you're saying the issue is with express. You showed above that all the cookies are in the response. That's all express does when you use res.cookie: it puts the cookies in the response headers, which you show above.
Can you describe the issue, exactly? Also, with so much going on, it's hard to know where is exactly an issue is if there is one. For example does the issue happen when you remove serverless thing? Or when you run the express app locally?

So this actually isn’t an issue with Express, but however you wrap Express. It’s caused by the fact that Lambda doesn’t support multiple headers of the same name. There’s a workaround which involves doing mixed casing per header as a hack (e.g. https://github.com/blakeembrey/node-bit-string-mask).

So I'm going to close this issue seems it seems to have just gone stale. If there is something Express team should be looking into, let us know and we can reopen.


exteding ban user behaviour

when clicking on ban for a user all huggs that this user requested to the banning user should be rejected and all huggs that the banning user did should be added the banned user info

closed.
when blocking:
add blockedProfileID to all exising huggs from the user that requested the block
decline all huggs from the requesting user that the blocked user has answered
when unblocking:
remove blocked user from all existing huggs from the requesting user


Is there Hazard Function in lifelines ?

Hi! Is possible to implement Hazard Function in python with lifelines like in R and SAS languages?
Thanks

Hi @DaniBre99, like you want to implement your own hazard function? Something like this?

Hi @CamDavidsonPilon! Thanks a lot for the answer! I'm doing a survival analysis with KaplanMeierFitter (lifelines) and I'd like to add a graph like this:
https://bookdown.org/rwnahhas/RMPH/survival-hazard.html
Or like this:
https://stats.oarc.ucla.edu/sas/seminars/sas-survival/

(chapter: 1.2.4. Background: The hazard function)
In python I can manually calculate the hazard ratio like this:
https://allendowney.github.io/SurvivalAnalysisPython/02_kaplan_meier.html
(chapter: Estimating the hazard function)
I was wondering if these calculations to find the hazard ratio were already included in the lifelines library or if it was possible to insert them and then create the graph. I saw the link you kindly sent me, maybe are they the same in the ThreeParamInverseTimeHazardFitter() function? I don't understand very well how this function works.
Thank you very much!

Thank you very much!!


Unique prefix matching?

Something that is to my knowledge new as-of https://github.com/c-blake/cligen is accepting any prefix like --opti for a long option as long as it is "unique" in its namespace.  Tools like the Mercurial version control system and long before it gnuplot and debuggers have had this kind of thing for keywords forever (and I also use it for enum's or subcommand names), but it's also quite nice for long-options.  It's like TAB-completion without having to hit TAB (or to either A) program completions for every shell under the sun or B) always have to type the full string).  Nim's stdlib CritBitTree makes it really easy, too.

I think I understand. You want for instance --peri to match --peripheral etc?

As long as there are no conflicts.

Yeppers.  Sounds like you get it.
  Then when the CL user presents you some string you can use iterator keysWithPrefix*[T](c: CritBitTree[T], prefix: string; longestMatch = false): string to "expand it"/see if it expands uniquely.  Couple more details you can see in proc lengthen in https://github.com/c-blake/cligen/blob/master/cligen/parseopt3.nim

(And, of course, you could also get this done with regular old sort and binary search, but that's probably a bit more coding effort.)

Made an initial implementation of this. Just did some manual checks. https://github.com/madprops/nap/blob/master/src/nap.nim#L301

I only read your impl, did not try anything out, but I think you want startsWith not contains.  Contains will let a person typing --dofoo match an expected --foo.  It's admittedly a bit of a judgement call since we're already being "flexibly accepting", but that feels a little too loose to me.
And yeah, even a linear re-scan is a viable impl as long as you check for an exact match first (which I think you do) because one key could be the prefix of another (as in --foo --foobar without any abbreviation tricks) and you need the exact match to prevail.
I'm honestly surprised this has not been a more popular keystroking/CLI framework feature.  It's almost as obscure as Nim itself.  Trying to help change that in my own little way.  It both lets CL authors/script authors document things with long identifiers but also does not force heat-of-the-moment keystrokers to use the whole thing or to remember the short options.  Indeed, it basically creates a "discrete continuum of abbreviations" starting with "full long options" and ending with "as short as the CL author can muster via de-conflicting prefixes".  One might theorize that a "short-option-less" command-options system would be only a minor burden if all the long-option systems did this.  The dash-dash could even fall away, as in, e.g. gcc's options syntax.
{Personally I have good recall for 1-letter options and also like/frequently use "combining syntax" like, e.g. "du -scBG", but I understand I am rare. Honestly, liking that sort of combining syntax so much was one of the things that initially motivated me to write cligen, but I learned a few things along that journey.}

Yeah it definitely needs to be startsWith, not contains.
What I like about long options is that it's obvious what it's doing, no need to memorize. But combining short options like -snB is very powerful. Recently I added support for "alts" to be able to do this.

Yeah, the continuum of abbreviation gives users a wide choice rather than forcing explicitness.  What is ideal definitely varies between "interactive sessions" vs. "scripted invocation".  Anyway, feel free to close this out once you get it all working/tested.


Custom algorithm of CertificatePinner

Hi, I feel awkward to pin Certificate.
What I have now is a hex string which equals to sha256(x509Certificate.getEncoded()).
While I found it is sha256(x509Certificate.getPublicKey().getEncoded()) in CertificatePinner.
if (pin.hashAlgorithm.equals("sha256/")) {
    if (sha256 == null) sha256 = sha256(x509Certificate);
    if (pin.hash.equals(sha256)) return; // Success!
}

static ByteString sha256(X509Certificate x509Certificate) {
    return ByteString.of(x509Certificate.getPublicKey().getEncoded()).sha256();
}

Do you have plan to support custom algorithm ?

No plans. Don’t pin the whole certificate; that won’t work if you issue a new certificate with the same keys and a new expiration date.


Update default value of Jenkins version

#1409 set the default set the value of this to 2.356 in docker-bake.hcl, but the individual Dockerfiles still have an old version. This PR makes the version consistent throughout the repository.

Looks like there is a test failure in the last three tests of tests/runtime.Tests.ps1.

Looks like the same fix to the Get-Csp-Value and Get-Timezone-Value functions as was done in https://github.com/jenkinsci/docker/commit/f471a2f18e9ead4d984c70c2e1dad63b7863380f needs to be done to the corresponding copypasta in runtime.Tests.ps1.

I tried to port the corresponding logic from https://github.com/jenkinsci/docker/commit/f471a2f18e9ead4d984c70c2e1dad63b7863380f to runtime.Tests.ps1, but this is the first time in my life that I have ever written a .ps1 script, so I would be shocked if I got it right on the first try.


feat: For movies mock db, query should be populated with default values

Description
When we create movies mock DB, and create queries on top of it, then:
If datasource schema is available, then we need to populate the query with defaults as shown below:

When schema is not available, query would not be populated and shown in blank state as below:

PR fixes following issue(s)
Fixes #25320

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

New feature (non-breaking change which adds functionality)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] Jest
[x] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/build-deploy-preview

/ok-to-test sha=0a90b1c

/ok-to-test


fix: postCss searches folders recursively #68

Alaska Airlines Pull Request
Updating postCss to recursively search folders.
Testing
Tested in two repositories:

auro-input
auro-radio

Process

Copied the file locally into /node_modules/@aurodesignsystem/auro-library/scripts/build and renamed it to testPostCss.mjs
Updated the path in package.json to temporarily reference the updated testPostCss.mjs file:

"postCss:component": "node<EMAIL_ADDRESS>

Ran npm run build which ran the build:sass task:

"build:sass": "npm-run-all build:sass:component postCss:component sass:render"

Changes to package.json
build:sass:component
Based on the example present in the auro-radio's package.json:

Changed the build:sass:component command  in auro-input from:

"build:sass:component": "for file in src/*.scss; do npx sass --no-source-map \"$file:${file%.scss}.css\"; done",

to:
"build:sass:component": "sass --no-source-map src:src",

sass:render
Updated sass:render to check recursively, and added the /components/ folder:
"sass:render": "sass-render 'src/**/*.css' 'components/**/*.css' -t<EMAIL_ADDRESS>
sweep

Updated the run sweep command to check recursively
Added the ./components folder

"sweep": "find ./demo ./dist ./src ./components -type f \\( -name \"*.css\" -o -name \"*-css.js\" \\) -delete"
  },

Confirming changes

Using npm run sweep, any previous .css & .js files are cleaned from the repo
Ran npm run build and observed the expected files being built:

auro-input

Created test subdirectories in /src/ to further check recursive functionality:

Before Submitting this pull request:

Link all tickets in this repository related to this PR in the Development section
note: all pull requests require at least one linked ticket
If this PR is Ready For Review, all ticket's linked under Development must have their status changed to Ready For Review as well

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I have performed a self-review of my own update.

After further testing, the required changes can be managed in package.json as opposed to editing the postCss.mjs file.
Will close this PR and associated issue, and will be making the required edits to wc-generator and any respective, individual Auro repos.


Updated fetching InStoreMetadata according to new changes in SDK v3.0.2

This Pull Request has been merged with #28 into #37

This Pull Request has been merged with #28 into #37