[HAC-1170] Initial Cypress configuration to be able to run e2e tests

To run tests:

yarn install
Start the servers for the sample plugin and the sample app:
npm run http-server  within packages/sample-app and packages/sample-plugin
npm run e2e-test

Some considerations:

The cypress documentation recommends the cypress/* folder structure I went with it but we can change it if necessary and point the configuration files to our custom directory names.
I left some boilerplate code in related to cypress support and plugins that can help us extend the internal behavior of Cypress. These folders are auto-generated when we run cypress unless we explicitly disable them in the Cypress configuration. The only reason I kept this is so that we know it's an option to use going forward, but I can disable them too and we can enable these features as needed going forward. @florkbr what do you think?
I intentionally disabled the setting for capturing videos for now because I suspected that would probably trigger the out of memory issue we'd seen earlier on Prow. We can turn this on if needed once we have more tests (Screenshots get captured by default on errors)

shall we add support to trigger e2e tests upon every PR creation?

shall we add support to trigger e2e tests upon every PR creation?

@yapei This story and PR are specifically for the cypress configuration to enable us to start writing E2E tests. Triggering the e2e tests will require some more discussion and will be covered in https://issues.redhat.com/browse/HAC-1412.

@vojtechszocs I've pushed up a commit to address your comments. The reason I kept cypress.json in the e2e-sample-app package rather than e2e-common is that cypress.json does not have an extends option so it seemed like it's better to place it in the package that triggers the e2e tests and this way we can specify package specific configs like baseUrl and screenshots directory. Let me know if you have any concerns with this.
Please re-review . Thanks!
cc: @florkbr

This PR looks good to me. I checked it out and the E2E seems to pass locally:

We will sync as a team this week to decide where we want our E2E to live.

@florkbr Thanks, Bryan. It appears that this is an open issue still with cypress not providing a clean way to disable the auto-generation of directories - https://github.com/cypress-io/cypress/issues/501 (duplicate issue https://github.com/cypress-io/cypress/issues/619 is still open).
Since I don't see a use for the fixtures directory as yet, I've pushed up a commit to disable it in the config for now. So you should no longer see that generated directory for test-e2e-open.

/lgtm

/approve


GetCostAndUsageWithResources Daily Granularity saying hourly?

When I run the following code, I get an error saying that 'start date is too old for hourly, the max supported days for hourly granularity is 14 days.'  You'll notice in the code that the granularity is DAILY, not hourly.  Also, the response data is daily as well.  I'm assuming this is a bug, but maybe I'm doing something wrong?
@ashishdhingra
Thanks!
            GetCostAndUsageWithResourcesRequest request = new GetCostAndUsageWithResourcesRequest();
            request.TimePeriod = new DateInterval();
            request.TimePeriod.Start = DateTime.Now.AddDays(-15).ToString("yyyy-MM-dd");
            request.TimePeriod.End = DateTime.Now.ToString("yyyy-MM-dd");

            request.Granularity = Granularity.DAILY;

            request.GroupBy = new List<GroupDefinition>();
            request.GroupBy.Add(new GroupDefinition { Type = GroupDefinitionType.DIMENSION, Key = Dimension.RESOURCE_ID });
            request.GroupBy.Add(new GroupDefinition { Type = GroupDefinitionType.DIMENSION, Key = Dimension.OPERATION });

            request.Filter = new Expression() { Dimensions = new DimensionValues() { Key = "SERVICE", Values = new List<string> { "Amazon Elastic Compute Cloud - Compute" } } };

            request.Metrics = new List<string>();
            request.Metrics.Add(Metric.AMORTIZED_COST);
            request.Metrics.Add(Metric.BLENDED_COST);
            request.Metrics.Add(Metric.UNBLENDED_COST);
            request.Metrics.Add(Metric.USAGE_QUANTITY);
            request.Metrics.Add(Metric.NORMALIZED_USAGE_AMOUNT);

            GetCostAndUsageWithResourcesResponse cost = client.GetCostAndUsageWithResources(request);

Hi @webecker25272,
Good morning.
Looking at the SDK code base, the error and response are returned by the service API. The method GetCostAndUsageWithResources is only a wrapper around the service API call. Looking at the service API documentation https://docs.aws.amazon.com/aws-cost-management/latest/APIReference/API_GetCostAndUsageWithResources.html, it appears that TimePeriod start and end date range must be within the last 14 days (the start date cannot be earlier than 14 days ago). Hence you are getting that date range error.
Hope this helps.
Thanks,
Ashish

Hi @webecker25272,
Good morning.
Looking at the SDK code base, the error and response are returned by the service API. The method GetCostAndUsageWithResources is only a wrapper around the service API call. Looking at the service API documentation https://docs.aws.amazon.com/aws-cost-management/latest/APIReference/API_GetCostAndUsageWithResources.html, it appears that TimePeriod start and end date range must be within the last 14 days (the start date cannot be earlier than 14 days ago). Hence you are getting that date range error.
Hope this helps.
Thanks,
Ashish

@ashishdhingra This does not resolve the issue.
If setting Granularity by 'MONTH' or 'DAY' is an option, why would the time period have to fall within the past 14 days?
I want to get my costs from the past x Months.

@ashishdhingra This does not resolve the issue.
If setting Granularity by 'MONTH' or 'DAY' is an option, why would the time period have to fall within the past 14 days?
I want to get my costs from the past x Months.

Hi @garyholiday13,
The restriction is imposed by service, not by the SDK. SDK provides wrapper classes for making service API calls.
Thanks,
Ashish

Im also getting this error on the GetCostAndUsage command with DAILY granularity even though the service
https://docs.aws.amazon.com/aws-cost-management/latest/APIReference/API_GetCostAndUsage.html doesn't say that there is a 14 day limit.

I also have the same. The error message says that it is the limitation for hourly granularity. But the granularity in the request is DAILY, so I believe it is the bug.

I also have the same. It looks like aws ce is not relevant tool for usage.

Also running into the same issue with the following:
response = client.get_cost_and_usage_with_resources(         TimePeriod={             'Start': '2022-06-01',             'End': '2022-06-30'         },         Granularity='MONTHLY',         Filter = {             'CostCategories': {                 'Key': 'string',                 'Values': [                     'string',                 ],             }         },         Metrics=[             'NetUnblendedCost'         ]     )
I have it set to monthly but getting an error message for hourly

I also have the same issue.

Same problem. I have it set to daily but getting an error message for hourly

Running this PowerShell query and getting error for hourly granularity even though I am using Daily/Monthly granularity. Can anyone suggest whats the issue?
I am using the latest version of AWS.Tools.CostExplorer [Version: 4.1.398]
$TimePeriod = (New-Object Amazon.CostExplorer.Model.DateInterval -Property @{Start='2023-08-22'; End='2023-07-10'})
$BlendedCost = Get-CeCostAndUsage -TimePeriod $TimePeriod -Granularity DAILY -Metric 'BlendedCost'

[Error] Get-CECostAndUsageWithResource : Hourly timePeriod is not supported for this request

Same issue. Please update the documentation to specify --granularity is not an option, or allow for MONTHLY and DAILY to be used.

Thank you for the feedback.  We have reopened this issue to open a ticket against the service team or to provide a solution.

We have opened a ticket with the service team and will provide an update.

V1275484116

Running this PowerShell query and getting error for hourly granularity even though I am using Daily/Monthly granularity. Can anyone suggest whats the issue?
I am using the latest version of AWS.Tools.CostExplorer [Version: 4.1.398]
$TimePeriod = (New-Object Amazon.CostExplorer.Model.DateInterval -Property @{Start='2023-08-22'; End='2023-07-10'})
$BlendedCost = Get-CeCostAndUsage -TimePeriod $TimePeriod -Granularity DAILY -Metric 'BlendedCost'

[Error] Get-CECostAndUsageWithResource : Hourly timePeriod is not supported for this request

@50ur48h I guess this is the same issue where I posted an update https://github.com/aws/aws-sdk-net/issues/3036#issuecomment-1688809665 from the service team. Please refer note Resource-level data granularity is an opt-in only feature. You can be enable this feature from the PAYER account's Cost Explorer Settings page. and check of you have enabled granularity at your end.
Thanks,
Ashish

We have opened a ticket with the service team and will provide an update.

Did you get any update from service team? We literally have no clue why APIs are behaving oddly.


:sparkles: return optimal text and line arguments

It's sometimes useful to be able to store the optimal text boxes positions.
For instance, I use it to pre-compute optimal label positions once and then reuse-cache to draw at the same positions with different colors (via a GUI and a color-picker)
It won't break previous usage since nothing was returned.

I've applied your suggestion
You're right for the optimal adjective :smile:

Merged and updated pypi package


Change milliseconds to seconds

Abstract
Convert all timestamps to be in seconds rather than milliseconds.
Motivation
Millisecond precision is too fine for time keeping, we can afford to be a bit more coarse by using seconds. Block intervals are produced every 3 seconds, but we don't actually care about the milliseconds when the block was created.
Specification
Time expiry needs to take the millisecond to second conversion into account. Any arbitrary arithmetic with time needs to be updated to use seconds as well.

Closed by fcd2e83


⚠ ClamAV has degraded performance

In 6aaa606, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 88 ms

Resolved: ClamAV performance has improved in 7274a9d after 542 days, 8 hours, 40 minutes.


Issues with Charset

Hi,
Don't understand why got this problem. Data are stored in utf8_bin in database. When i query with node, got special chars instead of accentued ones.
no problems quering with another client got correct chars.
Impossible to get correct encoding characters while quering with mariasql node client.
That's bad cause ont functionality i like is to use objects with queries in data injection.

The connection charset is wrong. You can work around it by sending SET NAMES 'utf8' upon connecting. I think this should be done properly by the library, though.

I attempt to do lot of things, SET NAMES 'utf8' doesn't change Anything.
I replaced the node MariaSQL client with Mysql Node client, and everything worked properly. So sad.

i had issues with persian characters, result only show "?" as every characters
after couple of hours, it's solved
before executing any queries , i run SET NAMES 'utf8' ,
c.query('SET NAMES \'utf8\'', function (err, rows) { 	if (err) 		throw err; 	console.dir(rows); });
c.query('select * from chert', function (err, rows) { 	if (err) 		throw err; 	console.dir(rows); });


Get out in front of trademark issues with wrangler jeans and wrangler jeep

It's no secret that the wrangler 2 release is going to be big. Bigger than big. In fact, I don't think it's too much of a stretch to say it will change the way that software is developed and deployed on a huge scale. Like, on the same level as git, or CI/CD, or debugging.
As a result, there are bound to be lawsuits incoming. Lawyers clamoring at Cloudflare's door with spurious claims like "You can't monopolize the entire serverless market with your ridiculously sleek, top-of-the-line, dev-friendly CLI!" and "We're suing for damages because our team was unable to work for a month straight after your project became so popular that it single-handedly took down GitHub."
These should be easily dealt with by Cloudflare's legal team. However, there is a potential issue regarding the naming of our CLI tool. Once wrangler becomes popular enough (which it will), and our SEO gets juiced enough (which it will), searches for "wrangler" will lead to our CLI first, and jeans and Jeeps second.
This PR aims to address these complaints before they happen, by incorporating both of those products into our own. Embrace, Extend, Extinguish. Eventually, of course, it will be buying "Cloudflare Jeans" and "Jeep Cloudflares", but until the inevitable merger between the three companies it's probably best to throw a bone their way.
context

I've been told not to discuss the Cloudflare-Wrangler-Jeep merger any further, including changes to the wrangler codebase, until we've put out a blog post about it. Closing.

🤣 @caass


[Enhancement] Background color for Different Sections

What happened?
:Screenshot:

:Description:
As you can see all the background color of the sections have the same background color and does not look good, What I want to change is the background color for maybe 1 or 2 section accordingly, to make it something with the website theme. Will choose each section and see what looks best and accordingly do the changes.
Pl. assign me this issue
How can we reproduce this bug?
N/A
Desktop Information (Optional)
Windows
Chrome
Urgency (Optional)
Medium priority
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Not sure if it will look good or not, depends on your UI then will be merging that

hello sir, I would like to work on this can you assign this issue to me

can you assign me this issue


Prepare new release

See #29

@kba I think this is ready. Can we release it?

Looks good, just needs to rebased against current master so the to-text transformations are actually available.

@kba, no need for a rebase, merge knows how to handle this.

BTW: You can create release zip/tar.gz with make release. I've added them to the release.


OverflowError: Python int too large to convert to C long in pdfminer.six

$ pdftotree -vv MOTOS03160-1.pdf
--- Logging error ---
Traceback (most recent call last):
  File "/home/lwhsiao/repos/pdftotree/pdftotree/utils/pdf/pdf_utils.py", line 117, in analyze_pages
    interpreter.process_page(page)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 852, in process_page
    self.render_contents(page.resources, page.contents, ctm=ctm)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 862, in render_contents
    self.init_resources(resources)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 362, in init_resources
    self.fontmap[fontid] = self.rsrcmgr.get_font(objid, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 194, in get_font
    font = PDFType1Font(self, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 588, in __init__
    PDFSimpleFont.__init__(self, descriptor, widths, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 547, in __init__
    self.cid2unicode = EncodingDB.get_encoding(name, diff)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 61, in get_encoding
    cid2unicode[cid] = name2unicode(x.name)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 21, in name2unicode
    return six.unichr(int(m.group(0)))
OverflowError: Python int too large to convert to C long

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.5/logging/__init__.py", line 980, in emit
    msg = self.format(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 830, in format
    return fmt.format(record)
  File "/usr/lib/python3.5/logging/__init__.py", line 567, in format
    record.message = record.getMessage()
  File "/usr/lib/python3.5/logging/__init__.py", line 330, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/home/lwhsiao/repos/pdftotree/.venv/bin/pdftotree", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/lwhsiao/repos/pdftotree/bin/pdftotree", line 82, in <module>
    args.favor_figures, args.visualize)
  File "/home/lwhsiao/repos/pdftotree/pdftotree/core.py", line 56, in parse
    if (not extractor.is_scanned()):
  File "/home/lwhsiao/repos/pdftotree/pdftotree/TreeExtract.py", line 104, in is_scanned
    self.parse()
  File "/home/lwhsiao/repos/pdftotree/pdftotree/TreeExtract.py", line 79, in parse
    for page_num, layout in enumerate(analyze_pages(self.pdf_file)):
  File "/home/lwhsiao/repos/pdftotree/pdftotree/utils/pdf/pdf_utils.py", line 119, in analyze_pages
    log.exception(oe, ', skipping page', page_num, 'of', file_name)
Message: OverflowError('Python int too large to convert to C long',)
Arguments: (', skipping page', 0, 'of', 'MOTOS03160-1.pdf')

...

pdftotree -vv MOTOS03160-1.pdf
[ERROR] pdftotree.utils.pdf.pdf_utils - Python int too large to convert to C long, skipiping page 0 of MOTOS03160-1.pdf
Traceback (most recent call last):
  File "/home/lwhsiao/repos/pdftotree/pdftotree/utils/pdf/pdf_utils.py", line 125, in analyze_pages
    interpreter.process_page(page)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 852, in process_page
    self.render_contents(page.resources, page.contents, ctm=ctm)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 862, in render_contents
    self.init_resources(resources)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 362, in init_resources
    self.fontmap[fontid] = self.rsrcmgr.get_font(objid, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 194, in get_font
    font = PDFType1Font(self, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 588, in __init__
    PDFSimpleFont.__init__(self, descriptor, widths, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 547, in __init__
    self.cid2unicode = EncodingDB.get_encoding(name, diff)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 61, in get_encoding
    cid2unicode[cid] = name2unicode(x.name)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 21, in name2unicode
    return six.unichr(int(m.group(0)))
OverflowError: Python int too large to convert to C long
[ERROR] pdftotree.utils.pdf.pdf_utils - Python int too large to convert to C long, skipiping page 1 of MOTOS03160-1.pdf
Traceback (most recent call last):
  File "/home/lwhsiao/repos/pdftotree/pdftotree/utils/pdf/pdf_utils.py", line 125, in analyze_pages
    interpreter.process_page(page)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 852, in process_page
    self.render_contents(page.resources, page.contents, ctm=ctm)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 862, in render_contents
    self.init_resources(resources)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 362, in init_resources
    self.fontmap[fontid] = self.rsrcmgr.get_font(objid, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdfinterp.py", line 194, in get_font
    font = PDFType1Font(self, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 588, in __init__
    PDFSimpleFont.__init__(self, descriptor, widths, spec)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/pdffont.py", line 547, in __init__
    self.cid2unicode = EncodingDB.get_encoding(name, diff)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 61, in get_encoding
    cid2unicode[cid] = name2unicode(x.name)
  File "/home/lwhsiao/repos/pdftotree/.venv/lib/python3.5/site-packages/pdfminer/encodingdb.py", line 21, in name2unicode
    return six.unichr(int(m.group(0)))
OverflowError: Python int too large to convert to C long
[ERROR] pdftotree.utils.pdf.pdf_utils - Python int too large to convert to C long, skipiping page 2 of MOTOS03160-1.pdf

Any update on this one? :\

What is causing it and is there a workaround?

I tried to reproduce this but was successful...
macOS: 10.15.6
python: 3.7.3
pdfminer.six: 20200726
pdftotree: 0.4.0
I did a quick search and found this: https://github.com/pdfminer/pdfminer.six/issues/177
Seems like the issue was fixed by https://github.com/pdfminer/pdfminer.six/pull/263 at version 20191020 of pdfminer.six.
@lukehsiao if you can confirm it too, this issue can be closed.

Confirming I don't see this issue anymore, even with this specific document. Closing the issue.


File Search won't show results.

Can't seem to make File search plugin to work. It doesn't show anything. Tried to restart my Computer and still nothing. Thanks!

What are your preferences set to? Mine, for example, looks like this:

@drnhzn did the steps above resolve your issue? Do you have more information on this? Thanks.

Closing as no response.


How can i turn the checkbox to be clickable?

close is you can found here the solution https://github.com/chmac/action.tools/commit/6867ae907ba83b95d95cd93e011ffe60d656a832


new skills and balance for JiNiTaiMei

Plz add more description about ur work

Add a new skill: Explosion with One More Glance.

With 15s-CD
Generates a new type of bullet that can penetrate obstacles

Change the implementation of the original bullet and the E skill.

Now, the normal attack can be rebounded on the boundary of the grassland
The E skill is now implemented by declining its damage linearly instead of exponentially, making it more powerful in the long distance and less powerful in the short distance.

Modify some previous problems.


Refactor Elegy using Treex

Elegy is now based on Treex 🎉
Changes

Remove the module, nn, metrics, and losses from Elegy, instead Elegy reexports these modules from Treex.
GeneralizedModule and friends are gone, to use Flax Modules use the elegy.nn.FlaxModule wrapper.
Low level API is massively simplified:

States is removed, since Model is a pytree all parameters are tracked automatically thanks to Treex / Treeo.
All static state arguments (training, initializing) are removed, Modules can simply use self.training to pick their training state and self.initializing() to check whether they are initializing.
Signature for pred_step, test_step, and train_step now simply consists of inputs and labels, where labels is a dict that can contain additional keys like sample_weight or class_weight as required by the losses and metrics.

@alexander-g This version uses a new Module system + layers from Treex and hooks are gone.
Resnets and the GAN example needs fixing if you want to give it a try.

Will merge this for now. Docs need updating.


Non-python version of operating-system

This implements the version of operating-system that we came up with a couple weeks ago that doesn't depend on python being installed. This is nice, as it removes one less dependency for some of the modules (e.g the terraform-kubernetes-helm repo).

I am closing this. I thought this would work, but it has a nasty behavior on destroy where the data source won't be calculated because the null_resource is gone.

Can you provide more context on what error you hit on destroy?

The main issue is with the pex data source test. The external data source is supposed to be able to run whenever, whether it be apply, plan, or destroy. The problem is that the pex code makes use of the path-join module, which uses the operating-system module.
Now previously, everything was implemented using external data sources so there was no problem: there were no resources, so all the relevant properties were preserved. However, in this new implementation, the local_file data source depends on a null_resource, which is deleted on destroy. When that happens, the local_file data source refuses to compute the output of the file because I have the depends_on link. This causes a chain reaction that ultimately causes an error in the pex data source on destroy call because the path separator is not available.


fixed pip install version for next release

FIxes https://github.com/ranjanrak/OptionChainStream/issues/1#issue-853096888

Thanks for the PR. This is fixed in the new release(which introduces additional field IV).


Transparency Issue with All Filters

Hi, very excited to use the framework but I'm running into an issue. I'm using AVPortraitEffectsMatte to generate a "cutout" of a picture. It uses the CIFilter below

Then I apply a filter like such:

The cutout works great but when I go to add a simple filter like Brightness, the background no longer stays transparent, as seen with the images below. I've tried using a UIImageView as well as a RenderView.
Before:

After:

Is there anything I can do on either end (masking or adding filters) to keep that transparency? I'm planning to do some real time filtering as well as one time outputs. Edit: This also happens with any PNG photo that has a transparency.

https://github.com/BradLarson/GPUImage2/issues/298#issuecomment-488699644

any solution? same problem.


Added aarch64 workflow

What
Given that the vast majority of our target devices use the aarch64 ISA, it would be beneficial to run a build and test using that architecture. Since Github runners use x64, we have to use qemu to emulate both the build and test environment. There is a signifact performance penalty when using qemu. It might be possible to do the build using the aarch64-unknown-linux-gnu toolchain on x64 and just run the unit tests in the qemu environment, but it would require the sysroot and rootfs to be binary compatible.
Why
Difference in types (especially when using FFI functions) can cause compilation errors. This ensure that future changes must pass both x64 and aarch64 workflows inorder to be considered for a merge into master.
How
By using qemu, we are able to confirm that Ripple can compile on aarch64, the target architecture.
Test
Verify that the new Github action passes.
Checklist

[x] I have self-reviewed this PR
[x] I have added tests that prove the feature works or the fix is effective

@brendanobra Two things to note.

This workflow does take 50 mins to complete. Up to the Ripple team if that is too long to make a blocking merge.
This workflow will create an image of the docker container used to build and run the workflow on aarch64 and save it to this repo's packages. This speeds up the process as subsequent builds don't have to completely rebuild the image. No concerrns just something the team should be aware of. You can read more about it here: https://github.com/uraimo/run-on-arch-action#usage


When returning to compound list after viewing compund selected filters and compound list may be insconsistent

When I filter compounds, say be deselecting Show MPS and Show TCTC, there are 68 compounds in the list. If I view one, then come back to the list with the back arrow, the filters are still shown (i.e. Show MPS and Show TCTC are still unselected, but the whole list of 372 compounds is shown. This could be misleading. I understand the problem with returning to a page with a selection, however, if I refresh the page, the filter selections go back to the default of all selected, and then at least the page is consistent. Is there a way to refresh the page on return as a quick solution to this problem? Can we make the default filter setting Group specific (i.e. if I am in Taylor_EPA, only Show EPA is selected)?

Related: https://github.com/UPDDI/mps-database-server/issues/385
The bug where filters are incorrectly applied should be fixed on stage, but it looks I will have to make some more fixes before merging into master.
I will look into changing the filters automatically depending on the chosen group, but in order for that to work we must either include a designation (like 'EPA') in the group names or have some other way of indicating what filters to use.

I think we can make sure the EPA group names include ‘_EPA’ and the TCTC group names include ‘_TCTC’.
-Bert
From: Luke-Malchow-Bergenthal<EMAIL_ADDRESS>Sent: Sunday, January 08, 2017 4:29 PM
To: UPDDI/mps-web-client<EMAIL_ADDRESS>Cc: Gough, Albert H<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [UPDDI/mps-web-client] When returning to compound list after viewing compund selected filters and compound list may be insconsistent (#50)
Related: UPDDI/mps-database-server#385https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FUPDDI%2Fmps-database-server%2Fissues%2F385&data=01|01|gough%40pitt.edu|b004fc9ddf8c4371acc908d4380d4c5e|9ef9f489e0a04eeb87cc3a526112fd0d|1&sdata=3tmdUk6t4CUdalZtWW6e5ze1bn5ce8GR8cPuNTdFEe4%3D&reserved=0
The bug where filters are incorrectly applied should be fixed on stage, but it looks I will have to make some more fixes before merging into master.
I will look into changing the filters automatically depending on the chosen group, but in order for that to work we must either include a designation (like 'EPA') in the group names or have some other way of indicating what filters to use.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FUPDDI%2Fmps-web-client%2Fissues%2F50%23issuecomment-271180887&data=01|01|gough%40pitt.edu|b004fc9ddf8c4371acc908d4380d4c5e|9ef9f489e0a04eeb87cc3a526112fd0d|1&sdata=QU%2B1PBvnBtl8GL0%2Ba3p3hJPAyEQHwgbeXINpvIbiqxk%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAG_cswSqXfFgsRQ1ZyhgzvNLI_JbsINaks5rQVUBgaJpZM4Ldu2U&data=01|01|gough%40pitt.edu|b004fc9ddf8c4371acc908d4380d4c5e|9ef9f489e0a04eeb87cc3a526112fd0d|1&sdata=bsWfXO49%2Bcb0taCdgfvnzPP8IN1i0zgZwpgIy4W4Ubw%3D&reserved=0.

Old issue, but I will note that Project associations are now deprecated regardless.


[explore] datasource dropdown should only contain the datasources a user has access to

@mistercrunch is #883 what you had in mind?

Closing as fixed in #883


pin docs to compatible apm-server version

Codecov Report

:exclamation: No coverage uploaded for pull request base (0.7@6a11961). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             0.7     #309   +/-   ##
======================================
  Coverage       ?   72.71%
  Complexity     ?     1055
======================================
  Files          ?      117
  Lines          ?     4087
  Branches       ?      392
======================================
  Hits           ?     2972
  Misses         ?      929
  Partials       ?      186

Continue to review full report at Codecov.

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


Add to defaults parmas all Operator and Sensor params, not only args

Defaults section only works for args, but not only args may be defaults.
This PR adds support for defaults class, callback_args, callback.

@tsvetkov-ii Could you please fix a typo in commit message?

@tsvetkov-ii Could you please fix a typo in commit message?

@kxepal BTW, you can change it yourself by choosing "Squash and Merge"  when merging.

@tsvetkov-ii Could you please fix a typo in commit message?

DId it

Thanks!


mtbserver process still running after deletion

It's surprising to me that while I still uninstalled Xamarin studio, I just realized that mtbserver is listening on my 5000 port.
Could you guys make it stop after Xamarin was uninstalled?

Could you report this to bugzilla.xamarin.com if still an issue, not related to xamarin.mobile pack!


i created binary search tree code

Please merge my file.

what's inside this pull request?

There is binary search tree code

example for binary search tree

yes


Update Fonts in "Read More" Section "gallery" page

[x] Change the font for the H2 headings in the "Read More" section to Montserrat.

[x] Update the font for the buttons to Montserrat as well.

Adjusted the font-normal to font-sans on the h2 and buttons for Montserrat. The width and height using padding p-3 and px-8. I also adjusted the text size to text-2xl.


Extend report of deprecated object usage with possible remove date

ref OAMG-3757
Add to the deprecation report the note that the functionality is deprecated since .... and it might be removed in 6 month
State before (example):
----------------------------------------
Risk Factor: medium
Title: Usage of deprecated Model "TMPTargetRepositoriesFacts" at /usr/lib/python2.7/site-packages/leapp/models/__init__.py:113
Summary: The model is temporary and not assumed to be used in any other actors.
Since: 2020-09-01
Location: /usr/lib/python2.7/site-packages/leapp/models/__init__.py:113
Near:         return cls(init_method='to_model', **data)

----------------------------------------

State after:
----------------------------------------
Risk Factor: medium
Title: Usage of deprecated Model "TMPTargetRepositoriesFacts" at /usr/lib/python2.7/site-packages/leapp/models/__init__.py:113
Summary: The model is temporary and not assumed to be used in any other actors.
Deprecated since: 2020-09-01
May get removed after: 2021-03-01
Location: /usr/lib/python2.7/site-packages/leapp/models/__init__.py:113
Near:         return cls(init_method='to_model', **data)

----------------------------------------

@bocekm @vinzenz @pirat89 any wishes to the description of the ticket?

Possible remove since: 2021-03-01 sounds strange.
I'd write:
Deprecated since: 2020-09-01
May get removed after: 2021-03-01

It looks little inconsistent: Since and Possible remove since. As well, from the previous discussion (maybe with @vinzenz ?) we concluded it could be more confusing for people. They are really supposed to read the documentation about the deprecation. Putting there the possible remove (again without reading documentation) could lead to point that people will panic.
E.g.: we deprecate something on 2020-03-01, we do release on 2020-07-01. Does it mean we remove the stuf on 2020-09-01? no. we will probably keep it in the way that it will dissapear in 2021-02-01 in earliest from the user POV.

"They are really supposed to read the documentation about the deprecation." No, please no :)
"Putting there the possible remove (again without reading documentation) could lead to point that people will panic." - I proposed adding the date after which the entity could get removed on Cabal and there was no voice against.
I don't see why should anyone panic. Above, I suggested rewording it to "May get removed after: ". The word "may" is the key here.
In the example you mention (we deprecate something on 2020-03-01, we do release on 2020-07-01), we may release leapp asynchronously on 2020-08-08 and in that release, the deprecated entity can be gone.

"Putting there the possible remove (again without reading documentation) could lead to point that people will panic." - I proposed adding the date after which the entity could get removed on Cabal and there was no voice against.

No problem then.


503 Service Temporarily Unavailable on https://charts.deliveryhero.io/

This is breaking us.  Please fix!

Seems to be working again

Seems to be working again

Sorry about that. Like @joshuastern said, it should be working now. We had some issues with chartmuseum that are not resolved.

Sorry about that. Like @joshuastern said, it should be working now. We had some issues with chartmuseum that are not resolved.


[OTBN] Idle signal feedback to clkmgr

The clock manager needs an "idle" indication so that it can ensure the clock is not manipulated while operations are ongoing.  This is primarily to ensure software cannot inject clock glitches into a running design.
It would be preferable is such a signal were level.

The current design has such a signal provisioned (https://github.com/lowRISC/opentitan/blob/bbfb0e02a34fe7898970096d43c54d916f43886b/hw/ip/otbn/data/otbn.hjson#L39). It's rather underspecified at the moment, so maybe we can take this issue to flesh things out.
For a start: Does "idle" only mean the core itself is idle, i.e. the bus device register interface doesn't matter? I.e. the clock manager takes care not to shut down the clock in the middle of a bus device transaction?

yes that's right. In my mind this is more related to when software starts
otbn on some operation, it cannot shut off the clock until otbn is done.
So a level signal that drops on a "start" and asserts on your "done" should
be more than enough.
Let me know what you think.
On Fri, Jun 26, 2020 at 10:41 AM Philipp Wagner<EMAIL_ADDRESS>wrote:

The current design has such a signal provisioned (
https://github.com/lowRISC/opentitan/blob/bbfb0e02a34fe7898970096d43c54d916f43886b/hw/ip/otbn/data/otbn.hjson#L39).
It's rather underspecified at the moment, so maybe we can take this issue
to flesh things out.
For a start: Does "idle" only mean the core itself is idle, i.e. the bus
device register interface doesn't matter? I.e. the clock manager takes care
not to shut down the clock in the middle of a bus device transaction?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/2646#issuecomment-650308800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSR3JBYKWVS5FG6JHHLRYTMTTANCNFSM4OJQYNHQ
.

Sounds good, that's what I've implemented so far in https://github.com/lowRISC/opentitan/pull/2641. I'll update the spec to make it slightly clearer what the idle signal is about.

thanks! feel free to take yourself off the assign and i'll make sure to
hook it up before closing.
On Fri, Jun 26, 2020 at 10:50 AM Philipp Wagner<EMAIL_ADDRESS>wrote:

Sounds good, that's what I've implemented so far in #2641
https://github.com/lowRISC/opentitan/pull/2641. I'll update the spec to
make it slightly clearer what the idle signal is about.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/2646#issuecomment-650313272,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSR75ERQGJHJ73FFIKTRYTNYHANCNFSM4OJQYNHQ
.

since this signal is already added, this issue is resolved.


Update pgvector to 0.8.0

DO NOT MERGE YET
See performance regression investigation in this slack thread

The regression was resolved (was related to autoscaling, not pgvector). Merging.


Gender prediction example missing DLL

Hi @takuya-takeuchi
I’m attempting to run the Gender Training example. When I build the project, I’m missing the files referenced. DlibDotNetNative.dll***, DlibDotNetNativeDnn.dll and **DlibDotNetNativeDnnGenderClassification.dll
Any suggestions?
The build works fine without errors.
Thanks,
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/tree/master/examples/GenderTraining

latest nuget package <IP_ADDRESS> contains these library. extract these from it.
Or you can build it from source code.
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/tree/master/src/DlibDotNet.Native.Dnn.AgeClassification
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/tree/master/src/DlibDotNet.Native.Dnn.GenderClassification

I was able to get DlibDotNetNative.dll and DlibDotNetNativeDnn.dll from nuget however DlibDotNetNativeDnnGenderClassification.dll wasn't present in nuget. I also attempted to build it following the instructions on this page however that was unsuccessful. Screenshot attached.

no. latest package is FaceRecognitionDotNet.

No Response

I ran into the same problem, just wanted to point out, these are in your local .nuget directory normally C:\User\Username\.nuget and then in the path \packages\facerecognitiondotnet\<IP_ADDRESS>\runtimes\win-x64\native.


Should handle "Combining schemas" and "Applying subschemas conditionally"

I know it's super late. But just now released support for allOf. A bit tricky to support anyOf and oneOf in this use case. Check examples in our Online Editor.


touching date select element on mobile devices print errors in console

General information

SDK version: 3.12.1
Environment: mobile devices
Browser and OS: Safari, iOS

Issue description
steps to reproduce:

open hosted field form with select elements for expiration dates.
touch on of the select elements.
error will be printed in console. Everything else works fine.

Cause of the bug:
Look at base-input.js, line 147:
You're calling element.select() function on touchstart event, but .select() only available on input elements, select element doesn't have that functionality.
Screenshots:

Thanks for the report. Working on fixing this now.

This is fixed in 3.15.0.
Thanks for the report!


feat(cli): new report-definitions update command

Summary
Support updating existing custom report definition from the CLI with the new command:
lacework report-definitions update

How did you test this change?

Testing Prereq:
Only custom report definitions can be updated. System created definitions cannot.
Either create a new custom report definition or use F5B4F0B0124F4C9E3547DCC6E62A8C6D774E45E9610999FE5FA517EF on tech-ally
Manual

[x] lacework rd update <id> -> update from editor -> Y
[x] lacework rd update <id> -> update from editor -> N
[x] lacework rd update <id> --file customReport.json
[x] lacework rd update <id> --file customReport.yaml

Tests

[x]  TestReportDefinitionsPromptUpdate
[x]  TestReportDefinitionsUpdateFromFile

Issue

https://lacework.atlassian.net/browse/GROW-1475

check windows dsjflkfjlajfldkld


Make symbol and params attribute always available for all kinds of Symbols

The behavior of all kinds of Symbols will be more consistent, make it easier to extend the library.

Can you elaborate with a bit more detail on why and how this would be used?

I'm trying to add the tooltip support, but notice that the symbol attribute of Condition and InputOutput is always undefined so I cannot add event listener to it


Release 1.19.0

This is a draft of changelog for the 1.19.0 release.
Here you can find all related translations changes: 1.18.4...master.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release. You can read more about the process on forum.

Show release notes preview
1.19.0

Menambahkan dukungan untuk ekstensi:

foskym/flarum-issue-tracking-youtrack
foskym/flarum-issue-tracking
mattoid/flarum-ext-store-auto-check-in
mattoid/flarum-ext-store-check-in
mattoid/flarum-ext-store-invite
mattoid/flarum-ext-store
xypp/forum-quests
xypp/pay-to-read
xypp/sticky-card
xypp/store-auto-checkin
xypp/store-doorman
xypp/store-group
xypp/store
xypp/user-decoration

Terjemahan terbaru untuk ekstensi:

clarkwinkelmann/flarum-ext-who-read
fof/drafts
gbcl/userip

Semua perubahan: 1.18.4...1.19.0.

Success! Now you can announce new release on forum:
## Versi [`1.19.0`](https://github.com/flarum-lang/indonesian/releases/tag/1.19.0)

**Menambahkan dukungan untuk ekstensi**:

* [`foskym/flarum-issue-tracking-youtrack`](https://github.com/FoskyM/flarum-issue-tracking-youtrack)
* [`foskym/flarum-issue-tracking`](https://github.com/FoskyM/flarum-issue-tracking)
* [`mattoid/flarum-ext-store-auto-check-in`](https://github.com/Mattoids/flarum-ext-store-auto-check-in)
* [`mattoid/flarum-ext-store-check-in`](https://github.com/Mattoids/flarum-ext-store-check-in)
* [`mattoid/flarum-ext-store-invite`](https://github.com/Mattoids/flarum-ext-store-invite)
* [`mattoid/flarum-ext-store`](https://github.com/Mattoids/flarum-ext-store)
* [`xypp/forum-quests`](https://github.com/zxy19/flarum-forum-quests)
* [`xypp/pay-to-read`](https://github.com/zxy19/flarum-pay-to-read)
* [`xypp/sticky-card`](https://github.com/zxy19/flarum-sticky-card)
* [`xypp/store-auto-checkin`](https://github.com/zxy19/flarum-store-auto-checkin)
* [`xypp/store-doorman`](https://github.com/zxy19/flarum-store-doorman)
* [`xypp/store-group`](https://github.com/zxy19/flarum-store-group)
* [`xypp/store`](https://github.com/zxy19/store)
* [`xypp/user-decoration`](https://github.com/zxy19/user-decoration)

**Terjemahan terbaru untuk ekstensi**:

* [`clarkwinkelmann/flarum-ext-who-read`](https://github.com/clarkwinkelmann/flarum-ext-who-read)
* [`fof/drafts`](https://github.com/FriendsOfFlarum/drafts)
* [`gbcl/userip`](https://github.com/GBCLStudio/userip)

Semua perubahan: [1.18.4...1.19.0](https://github.com/flarum-lang/indonesian/compare/1.18.4...1.19.0).

Untuk memperbarui, jalankan perintah ini:

```console
composer update flarum-lang/indonesian
php flarum cache:clear
```


Problem when slide slide is loaded for the first time

short description of the bug / issue, provide more detail below.

I am making a page using the slick slider. However, when the page is loaded for the first time, Slick is piled up as a block element, and after a certain period of time, Slick is executed. Then there is a phenomenon that the scroll animation cannot find the height. Has anyone solved this case?

refer to #4210


Make model dump/load symmetric for aliased fields

When using field aliases, I expected to be able to dump a model to a dict and then load that result back into the same model, but field aliases aren't considered when populating a model. This changes that behavior to allow this; in other words, MyModel(**MyModel(a="foo").dict()) now works if a has an alias.
The field alias is considered first to maintain as much of the existing behavior, but this change does have the potential to break user code, as data that previously failed validation could now pass.
Thanks for this library, @samuelcolvin! It's a very nice tool to have in the toolbox. 😄

I can't accept a change like this as it alters current behaviour too much.
Better to add a kwarg to dict use_aliases which defaults to false, but when true uses the aliases as keys.

Understood; I’ll get this updated as an opt-in behavior. Thanks!

WDYT of this approach? I'd rather be able to opt-in on a model-by-model basis, rather than having to hunt down every call site for dict(), which seems error-prone.
If this looks reasonable to you, I'll update the documentation as well. Thanks!

see #164, I think this is a cleaner way of doing it with fewer potential unexpected consequences.
If you agree I'll close this and merge #164.

#164 looks like the implementation of your initial feedback, I think. I understood what you were suggesting, but I'm not convinced it's cleaner. I'll share a bit more of my use case in hopes it might highlight why I think it might be cleaner to define this in the model config instead.
We've implemented common models for a large number of objects that we are ingesting from a third-party API. That API provides a reasonable structure for the data, but in numerous instances, we've needed to alias field names so that we can use these objects in downstream services in a more developer-friendly way. These objects, after being loaded from the third-party JSON and validated, are then serialized and placed on a message queue, where other consumers are able to receive the serialized data and load it back into the exact same pydantic model (with the model distributed as shared code). In many cases, those consumers then re-serialize all or part of the models they've loaded from messages they've received, and they send them to other components.
Thus, as I mentioned, we have a growing number of dict() calls already. Not only would updating them be a hassle, but later on, it would be an easy mistake for a developer to forget use_aliases when calling dict()--a mistake they may only learn about when another system consumes their serialized message and the message fails validation.
By having the model itself configured to allow aliases, this problem is avoided. The model remains the canonical definition of how it will serialize/deserialize itself--this is already the case for other aspects of serialization/deserialization; in my mind, this is a very close sibling to allow_extra and ignore_extra.
I hope this provides some useful context for why I added the option to the model config. I'm also curious what the unexpected consequences you have in mind are; I'm probably blind to them, having not seen the library evolve, and I definitely don't want to cause any!
Thanks for your consideration. 😄

#164 looks like the implementation of your initial feedback, I think.

It is, in the end implementing another PR to show what I meant was easier than a comment with loads of code.
I've come round to your suggestion, if you still think this is required once you've read through my explanation below I'll merge this. (documentation and history will need to be updated first)

I'm also curious what the unexpected consequences you have in mind are

Let's say we have
class Foobar(BaseModel):
    card_number: str
    expiry: date = date(2020, 1, 1)

    class Config:
        allow_population_by_alias = True
        fields = {
            'card_number': {
                'alias': 'cardNumber'
            }
        }

That model expects something like {"cardNumber": "4242..."} but you've added allow_population_by_alias = True so your model works with some backend. Now there's a weird case where someone can also submit {"card_number": "4242..."} which you didn't intend for the public API but will work. This will be fine with pydantic but could mess up other components.

Thus, as I mentioned, we have a growing number of dict() calls already.

You could just have a base model you use everywhere like
class OurBaseModel(BaseModel):
    def dict(self, *kwargs):
        kwargs.setdefault('use_aliases', True)
        return super().dict(**kwargs)

or use_aliases default value could be added to Config.

are then serialized and placed on a message queue, where other consumers are able to receive the serialized data and load it back into the exact same pydantic model

This would be much better accumplished using pickle. Pickling and unpickling models will be much more performant as validation doesn't have to be rerun upon every deserialisation, it also gets round this very problem.
I setup pydantic to play well with pickle for this very case, see here.

Thanks for going over this--I appreciate your thorough response. 😄
I agree with what you're saying w.r.t. similar, but ultimately undesired, values being considered valid. I think this concern can/should be strongly stated in the documentation for this new config field, for the reasons you've laid out. I'll have an update to the docs and history shortly.
Perhaps the root of my confusion is in my interpretation of "alias;" I'd understood it as setting up an equivalence between the field names, rather than a one-way transformation, and it was that confusion that makes me think having this noted and configurable could improve the DX for others. OTOH, sometimes I can't words well, so it might just be me!

This would be much better accomplished using pickle.

It would definitely be faster, but I don't mind re-validating when the payload is coming off the wire...and I've had some bad experiences with pickle that still sting a bit. Appreciate the suggestion, though; it was a selling point when looking at pydantic that you'd ensured compatibility with pickle. Not every author can be bothered to do so, I think. 😁

I agree that alias isn't a perfect term for this, but then again I can't think of a better one, input_field_name is uuuuuugly. Let's just document it better and stick to "alias"

It's also worth noting that inheritance should "work" (aka, do something magic and cause massive confusion) between different models.
So you could have
class FrontendFoobar(BaseModel):
    ...
    class Config:
        foobar = 'xxx'
        fields = {'card_number': 'cardNumber', 'expiry_date': 'expiryDate'}

class BackendFoobar(FrontendFoobar):
    class Config:
        fields = {}

Here BackendFoobar.Config.foobar == 'xxx' because of magic config inheritance, but also `data = frontend_foobar.dict() backend_foobar(**data)``` should work because you've got rid of the aliases. Perhaps this is an uglier sollution than all the others, not sure.

@bendemaree what's your intention on this?

@samuelcolvin I will have the remaining changes up tomorrow. My apologies; I forgot that I was going on vacation for a few weeks a day or two after committing to this. ☹


Add OIDC remote sign-out to the sample.

#423 @brentschmaltz @tushargupta51 @PinpointTownes @muratg @HaoK @Eilon
This is one way to implement http://openid.net/specs/openid-connect-logout-1_0.html as an application endpoint. Incorporating this into the middleware has a number of problems:
A) it's another endpoint to listen on.
B) the spec is still pretty new, it's unclear how much it may change.
C) to avoid xsrf issues you need to flow and verify the session id. The middleware could only do this for the simple OIDC + cookies scenario. If you used Identity you'd have to manually flow the session id and implement your own endpoint anyways.
D) an app probably wants to do other things when the user logs out like clear session state, etc..
I recommend leaving it as a sample for now.

Hi @Tratcher, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

HttpLogoutSupported = false. Closing this PR for now as we're lacking an implantation for interop testing.

Maybe you should use an openid connect spec compliant and certified server to test with -
you can find a selection here: http://openid.net/certification/
just sayin' ;p


Add buttons to put cards into specific statuses

The different listed buttons should only show up based on the status that a card is in. We want the ability to:

[ ] Send a card back to Claim Cards
[ ] Send a card back to Submit Cards

Finished


Automatic export 'When Idle' doesn't pause

Log ID: 7NJTUZ4T-euc
Automatic export 'When Idle' behaves like 'When change'. As I edit one or several  items, BBT exports, pauses for 2-3 seconds and then exports again. This has been happening since the day I installed BBT. Exports freeze Zotero for a few moments every time and really disrupts editing Zotero entries. To make things worse, it looks like an export is triggered even when the field I am editing would not affect the Citation Key and the change should ideally be ignored.

This is a duplicate of #2105, can you please subscribe to that issue?


White screen on iOS 8 devices and simulator

Hey, i'm getting a white screen on devices that run iOS8. The rootViewController is set but not presented.
After i'm setting the rootViewController i make my LGSideMenuController subclass a delegate of the rootViewController, this causes the view not the show
Is there any solution for this?
Thanks,

It seems you forget UINavigationController.
This code is worked for me:
UINavigationController *navigationController = [self.storyboard instantiateViewControllerWithIdentifier:@"NavigationController"];
self.rootViewController = navigationController;
_leftViewController = [self.storyboard instantiateViewControllerWithIdentifier:@"LeftViewController"];
[self setLeftViewEnabledWithWidth:285.f
                            presentationStyle:LGSideMenuPresentationStyleSlideAbove
                      alwaysVisibleOptions:LGSideMenuAlwaysVisibleOnNone];

self.leftViewStatusBarStyle = UIStatusBarStyleLightContent;
self.leftViewStatusBarVisibleOptions = LGSideMenuStatusBarVisibleOnAll;
self.leftViewLayerShadowColor = [UIColor colorWithWhite:0 alpha:0.8];
_leftViewController.tintColor = [UIColor whiteColor];
[_leftViewController.tableView reloadData];
[self.leftView addSubview:_leftViewController.tableView];

Please see the demo apps


Update kernel version to v4.19.177-cip44

Purpose of pull request
Update kernel version to v4.19.177-cip44

Which is correct?

v4.19.177-cip44
v4.19.177-cip47

I'm sorry. I made a mistake.
v4.19.177-cip44 is correct and has been fixed.

v4.19.177-cip44 is correct and has been fixed.

Got it. Thanks!

Thanks!


AttributeError: 'ShapedArray' object has no attribute 'ndims'

Not a huge deal, hit an issue ( in haiku )
class Filter(Module):
    "call a layer on a subset of inputs, then update those inputs"

    def __init__(self, genotype, name=None):
        super().__init__(name=name)
        self.keep_percent = choose(f"{name}_keep_percent", PERCENTAGES, genotype)
        self.controller = Conv1D(1, 1)
        self.attn = Attention(genotype, name=f"{name}_attn")

    def __call__(self, inputs, init=False):
        unbatched = inputs.ndim == 2
        if unbatched:
            inputs = jnp.expand_dims(inputs, 0)
        outputs = vmap(self._call_one)(inputs)  # <--- this vmap
        if unbatched:
            outputs = jnp.squeeze(outputs, 0)
        return outputs

    def _call_one(self, element):
        assert element.ndims == 2 # <--- AttributeError: 'ShapedArray' object has no attribute 'ndims'
        width = element.shape[0]
        n_keepers = round(self.keep_percent * width)
        weights = self.controller(element)
        _, indices = top_k(weights, n_keepers)
        keepers = element[indices[1], :]
        updates = self.attn(keepers)
        return element.at[indices[1], :].set(updates)

fixed without that assert. we could potentially add
@property

Hey, thanks for raising this! Does element.ndim == 2 work?

ah! damn i'm dumb, yes, that's it. I thought it was plural!

I do stuff like that all the time :)


RTL support and other languages ?

Hello , it seems that the model library doesn't support the Arabic language .. not sure if this due to the model-qa script or because the inference - runtime doesn't support it ?
it does actually output  Arabic prediction but it doesn't respond if there's  question mark "؟"  at the end for a Q&A  test.

Thank you @PrestigeDevop! We will look into this for you

Hi @PrestigeDevop
We have a streaming detokenization issue for which a fix is currently in progress.
Can you please try running your input with the model-generate.py script also in the examples/python folder? This will output the generated text in one operation rather than streaming one token at a time.
python model-generate..py -m <path to your model> -pr <your prompt>

Hi again @PrestigeDevop,
We have made a number of fixes in the decoding functionality. Can you please try again with the latest release onnxruntime-genai 0.2.0?

Hi again @PrestigeDevop,
We have made a number of fixes in the decoding functionality. Can you please try again with the latest release onnxruntime-genai 0.2.0?

HI there, thanks for your support ..
running phi3-qa.py seems work just fine (using the same model too) so it seems the issue with  model-qa.py  (still occur )

Hi @PrestigeDevop, can you share the model-qa.py command line you used?

Hi @PrestigeDevop, can you share the model-qa.py command line you used?

I knew it was due to the parameter  context length  2048 , anyway here is the run command
python model-qa.py -m cpu -l 2048


Cluster API Add-on Provider for Helm (CAAPH) integration

Hi
Currently, the CAPI Visualizer just shows ClusterResourceSet Addons.
I think integrating with CAAPH and showing their resources (HelmChartProxy&HelmChartRelease) would be a good addition.

@Preisschild This is something I would like to add as well. It's a similar use case to #42 as the issue is this app uses the clusterctl client to fetch the resources and CAAPH is not included in the clusterctl discovery. This is b/c it'd be a circular dependency since CAAPH imports CAPI, so we'd need a workaround to list external CRDs. One solution I was thinking of would be to add a label to other CRD types and query those instead.

Should be done with #43!


Correct our handling of MapIt responses

MapIt's postcode route returns the following statuses:

200 if a postcode exists
404 if it is a valid postcode but MapIt doesn't know about it
400 if it is not a valid postcode (postcode validity is handled here)

We currently handle 404/410 errors but MapIt doesn't return 410s and does return 400s.  This amends the exception handling to include all 4xx errors.
We don't treat missing postcodes (that is seemingly valid postcodes that we don't have in our dataset) any differently from invalid postcodes at the moment.  (We do for some formats in Frontend, but not places)

Looks good! Could we add another test for when the /places routes is hit with a invalid postcode param as that's the majority of 500 errors that we were seeing last week.

That's a good plan, thanks. Have added a test to places_api_test.rb and renamed one of the existing ones as it used "invalid" when it meant "missing".

Great 👍


Also load 'Content' in Kemono post pages if there are images within it

Source name
Kemono
Source language
All
Describe your suggested feature
There's this lil field 'Content' on most if not all artists post page, and usually those fields only have text and DDLs, but there are times where instead the field has images within it, so it would only make sense for the extension to load them since they do are images
Other details
e.g. (NSFW):  https://kemono.su/patreon/user/3828531/post/22839927
https://kemono.su/patreon/user/3828531/post/17969613
I was only able to find out abt it bc of the image inside 'Files' on that post that the extension did load, and if it wasn't for that I would never have known there was an image on the artist and so I would've missed it
I've come across similar cases before, even in comics, where the stuff inside the 'Content' field has better image quality than the one in 'Files', I can guess why is that
I'm using Yōkai
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version 0.15.3.
[X] I will fill out all of the requested information in this form.

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

@lalalasupa0 Kemono extension is working for you?

@lalalasupa0 Kemono extension is working for you?
For me its not #5883

Nop, I'm getting the same issue


authentication.enabled=false Interception should ignore authentication and athn

check with authentication.enabled=false
Probably interceptor is applied for legacy apis also, verify the access.
Originally posted by @dileepbapat in https://github.com/Sunbird-RC/community/discussions/125#discussioncomment-1558233

Closing with the commit


SSL Support

Hey guys, as the title says I would love SSL support. I'm currently running BookStack behind a nginx reverse proxy but when I force it to https the page loads incorrectly and I get a "Mixed Content" issue when inspecting the elements. I have linked a picture of what it looks like when served over https.
https://imgur.com/a/AnGXg

I use BookStack with Apache and forced SSL without any issues.
Maybe a php artisan cache:clear may help?

Hi @moonjazzz, Sorry to hear you're having issues with SSL and BookStack.
One thing to check is the APP_URL parameter in the .env file. Double check this value and ensure it is your domain and that it starts with 'https'. If not default, BookStack uses this as the base url for all content so forcing it to your URL may help.
If that does not work press F12 to bring up browser development tools, Click the 'console' table and refresh the page. Warnings should pop up in there about what requests are insecure.

Hey @ssddanbrown, that seemed to do the trick, we're running smooth as butter over here now! Thanks for the help!

@moonjazzz Awesome! Glad to hear it's now working 😃

Hey guys,
I love BookStack, but I just couldn't make it work on the SSL mode.
I installed BookStack via the official script for auto-installing it. So it runs on Apache. When I edited 000-default.conf to the bookstack location it still displayed the same Apache2 Ubuntu Default Page.
We're running it on Ubuntu 18.04 and we have the Cloudflare SSL.
Any help would be highly appreciated.
Thanks and cheers,
Nedim.

Hi @Nedimko123,
Did BookStack work on standard HTTP after the install script?
The install script adds a apache config file at /etc/apache2/sites-available/bookstack.conf. This would generally be used instead of 000-default.conf.
By the way, Might be worth opening a new GitHub issue here to deal with your specific issue since it's hard to keep tabs on an old closed issue such as this.

Hey @ssddanbrown,
Yes, it did work and it still works.  Any idea how to edit it in the bookstack.conf, I've tried
`Listen 443 http
Listen 80
NameVirtualHost *:80
NameVirtualHost *:443
ServerName *:80
<VirtualHost *:443>
ServerName docs.mni.me
ServerAdmin webmaster@localhost
DocumentRoot /var/www/bookstack/public/
SSLEngine on
SSLCertificateFile /Users/myusername/certs/server.crt
SSLCertificateKeyFile /Users/myusername/certs/server.key

<VirtualHost *:80>
SSLEngine off
ServerName docs.mni.me
ServerAdmin webmaster@localhost
DocumentRoot /var/www/bookstack/public/
`
But, then I cannot start the apache server. Any tips on how to enable SSL?
Thanks.

I think it would be a good idea if we open a new ticket @Nedimko123
On Wed, Jun 26, 2019 at 8:08 AM Nedimko123<EMAIL_ADDRESS>wrote:

Hey @ssddanbrown https://github.com/ssddanbrown,
Yes, it did work and it still works. Any idea how to edit it in the
bookstack.conf, I've tried
`Listen 443 http
Listen 80
NameVirtualHost *:80
NameVirtualHost *:443
ServerName *:80
<VirtualHost *:443>
ServerName docs.mni.me
ServerAdmin webmaster@localhost
DocumentRoot /var/www/bookstack/public/
SSLEngine on
SSLCertificateFile /Users/myusername/certs/server.crt
SSLCertificateKeyFile /Users/myusername/certs/server.key
<VirtualHost *:80>
SSLEngine off
ServerName docs.mni.me
ServerAdmin webmaster@localhost
DocumentRoot /var/www/bookstack/public/
`
But, then I cannot start the apache server. Any tips on how to enable SSL?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BookStackApp/BookStack/issues/272?email_source=notifications&email_token=AADHKL3DUY3WVHSI6CVLRODP4MBWPA5CNFSM4C4PMRI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSOABI#issuecomment-505733125,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADHKL7LIBWP4IAD2C2PVILP4MBWPANCNFSM4C4PMRIQ
.

I think it would be a good idea if we open a new ticket @Nedimko123
…
On Wed, Jun 26, 2019 at 8:08 AM Nedimko123 @.***> wrote: Hey @ssddanbrown https://github.com/ssddanbrown, Yes, it did work and it still works. Any idea how to edit it in the bookstack.conf, I've tried Listen 443 http Listen 80 NameVirtualHost *:80 NameVirtualHost *:443 ServerName *:80 <VirtualHost *:443> ServerName docs.mni.me ServerAdmin<EMAIL_ADDRESS>DocumentRoot /var/www/bookstack/public/ SSLEngine on SSLCertificateFile /Users/myusername/certs/server.crt SSLCertificateKeyFile /Users/myusername/certs/server.key <VirtualHost *:80> SSLEngine off ServerName docs.mni.me ServerAdmin<EMAIL_ADDRESS>DocumentRoot /var/www/bookstack/public/  But, then I cannot start the apache server. Any tips on how to enable SSL? Thanks. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#272?email_source=notifications&email_token=AADHKL3DUY3WVHSI6CVLRODP4MBWPA5CNFSM4C4PMRI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSOABI#issuecomment-505733125>, or mute the thread https://github.com/notifications/unsubscribe-auth/AADHKL7LIBWP4IAD2C2PVILP4MBWPANCNFSM4C4PMRIQ .

Done!

@mikulabc Done!


Deprecated new_name in Undelete

new_name is no longer supported on the service side for undelete_container() and undelete_filesystem()

Can we update the API comments to indicate that the parameter does not work if the value is different than the original container?

So we want to keep the parameter even though the only applicable value is if it matches the old one?
In this PR I just removed the parameter altogether and surfaced a deprecation warning if the user passes it - then we let service throw.

Can we update the API comments to indicate that the parameter does not work if the value is different than the original container?

So we want to keep the parameter even though the only applicable value is if it matches the old one?
In this PR I just removed the parameter altogether and surfaced a deprecation warning if the user passes it - then we let service throw.

isnt' that a breaking change?

Can we update the API comments to indicate that the parameter does not work if the value is different than the original container?

So we want to keep the parameter even though the only applicable value is if it matches the old one?
In this PR I just removed the parameter altogether and surfaced a deprecation warning if the user passes it - then we let service throw.

isnt' that a breaking change?

Well technically in Python they can still pass it if they want to and currently we ARE parsing it like we did before - its just now we surface a warning (if the cx passes it) and let service throw accordingly. However since this kwarg is no longer supported thought it would be best to get removed from the docstrings as an accepted kwarg. Do you feel like it should be kept in the docstrings as an accepted kwarg and add a deprecation comment besides it?


Responsive target

Issue description:  This is a feature request. I have noticed that the responsive option is referred to the width of the window and not to the with of the slider container. However, in my case the slider is in between two side bars that can appear/disappear base on the window with. For some width intervals, this makes the slider to become larger while the window gets smaller.
Request: Would it be possible to have a "responsiveTarget" option (default to "window") where to set the element whose width is observed for responsiveness?
Demo link/slider setting:
Tiny-slider version:   2.6.0
Browser name && version:  All
OS name && version:   windows

There is an issue related to this, please check out #100

If the slider need to watch the changes of the container rather than the window, a plugin would be added to this task. Because there is no container queries supports right now in the browsers and the slider is only rely on media queries.


Failed to build SwiftKit 1.3.2 with SPM

SwiftKit Environment

SwiftKit version: 1.3.2
macOS version: 10.15.7
Xcode version: 12.1
Installed via (mint, brew): brew + mint

What did you do?

brew tap SvenTiigi/SwitfKit
brew install swiftkit

What did you expect to happen?

successful installation of SwiftKit

What happened instead?

==> Installing swiftkit from sventiigi/swiftkit
==> Downloading https://github.com/SvenTiigi/SwiftKit/archive/1.3.2.tar.gz
==> Downloading from https://codeload.github.com/SvenTiigi/SwiftKit/tar.gz/1.3.2
#=#=#
==> swift build --disable-sandbox -c release

'XcodeProj' /private/tmp/swiftkit-20201104-23612-a6gsum/SwiftKit-1.3.2/.build/checkouts/xcodeproj: error: Source files for target XcodeProj should be located under 'Sources/XcodeProj', or a custom sources path can be set with the 'path' property in Package.swift
==> Formula
Tap: sventiigi/swiftkit
Path: /usr/local/Homebrew/Library/Taps/sventiigi/homebrew-swiftkit/swiftkit.rb
==> Configuration
HOMEBREW_VERSION: 2.5.8
ORIGIN: https://github.com/Homebrew/brew
HEAD: 1c822206a05cd7fffe526b8e0fa2f45365be72a5
Last commit: 8 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: f8c50a382156306b36d413eb5f8f98d464ebc9e9
Core tap last commit: 69 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit skylake
Clang: 12.0 build 1200
Git: 2.24.3 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
Java: 15
macOS: 10.15.7-x86_64
CLT: <IP_ADDRESS>.1.1602137645
Xcode: 12.1
==> ENV
HOMEBREW_CC: clang
HOMEBREW_CXX: clang++
MAKEFLAGS: -j8
CMAKE_PREFIX_PATH: /usr/local
CMAKE_INCLUDE_PATH: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers
CMAKE_LIBRARY_PATH: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries
PKG_CONFIG_LIBDIR: /usr/lib/pkgconfig:/usr/local/Homebrew/Library/Homebrew/os/mac/pkgconfig/10.15
HOMEBREW_GIT: git
HOMEBREW_SDKROOT: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk
ACLOCAL_PATH: /usr/local/share/aclocal
PATH: /usr/local/Homebrew/Library/Homebrew/shims/mac/super:/usr/bin:/bin:/usr/sbin:/sbin
Error: sventiigi/swiftkit/swiftkit 1.3.2 did not build

Hey @alexanderpuchta
Thanks for your detailed bug report.
It seems like the dependency XcodeProj which is used by SwiftKit fails to build due to an updated requirement of SPM that the name of a Swift Package must match the directory name inside the Sources directory.
I have updated the dependency on the develop branch.
Please let me know if this resolve your issue:
mint install SvenTiigi/SwiftKit@develop

Hey, thank you! I was able to install the SwiftKit!

Perfect 👍
The bug fix is now available with the new version 1.3.3 of SwiftKit


Android - Settings-On tapping options in settings page,all options greyed out

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: V9. 0.70-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Issue reported by: Applause Internal Team
Action Performed:

Launch site in both mweb and Android
Tap settings
Tap profile and navigate back
Tap workspace

Expected Result:
On tapping options in settings page, behavior must be consistent in mweb and android.
Actual Result:
On tapping options in settings page, all options are greyed out in android but not in mweb.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/e0e4f3c5-7de3-4f94-a6cc-bf336d1b4840

View all open jobs on GitHub

Hm, this seems to be working correctly? I'm not seeing any obvious bug in the video, but I tested the repro steps on both platforms and they seem to be consistent.


does triton.language.sum have reproducibility?

Is the operator triton.language.sum reproducible? Specifically, can it guarantee that the results are completely consistent when executed multiple times with the exact same environment and inputs?

Yes it is deterministic for a fixed kernel. However if there is the kernel changes elements may get accumulated in a different order and change the result

I don't understand the question. The user writes the kernel, so they would know when it changes. If the kernel is automatically generated by some tool (e.g., torch inductor), then that's the question for this tool


Basic auth for backend ssl service.

I have an ingress setup with the nginx ingress controller version 0.90beta2 and I can't seem to make basic auth work on this interface.  It seems to go right past  as if no authorization attempt is being made.  i have a docker registry v2 running in the cluster via a real cert ( not self signed).  Works just fine.  It is fronted by an ingress.
Here is my ingress:
   90	---
    91	apiVersion: extensions/v1beta1
    92	kind: Ingress
    93	metadata:
    94	  name: kube-registry
    95	  namespace: kube-system
    96	  annotations:
    97	    kubernetes.io/ingress.class: "nginx"
    98	    ingress.kubernetes.io/ssl-passthrough: "true"
    99	    ingress.kubernetes.io/secure-backends: "true"
   100	    ingress.kubernetes.io/auth-type: basic
   101	    #the auth-secret has to come from a file called "auth"
   102	    ingress.kubernetes.io/auth-secret: registry-auth
   103	    ingress.kubernetes.io/auth-realm: "Authentication Required - for the registry"
   104	spec:
   105	  rules:
   106	  - host: registry.ops.example.com
   107	    http:
   108	      paths:
   109	      - path: /
   110	        backend:
   111	          serviceName: kube-registry
   112	          servicePort: 443
   113	  tls:
   114	  - hosts:
   115	    - registry.ops.example.com

Logs from the nginx controller look "normal" nothing to really report.  I've logged on to the container and this is the generated config.  At least a snippet of it
 server {
        server_name registry.ops.tropo.com;
        listen [::]:80 proxy_protocol;
        listen 442  ssl http2;
        # PEM sha: 146d6a1a176900b4af00faf1b177c82d6c7b39ae
        ssl_certificate                         /ingress-controller/ssl/system-snake-oil-certificate.pem;
        ssl_certificate_key                     /ingress-controller/ssl/system-snake-oil-certificate.pem;

        more_set_headers                        "Strict-Transport-Security: max-age=15724800; includeSubDomains; preload";
        location / {
            set $proxy_upstream_name "kube-system-kube-registry-443";

            port_in_redirect off;

            # enforce ssl on server side
            if ($scheme = http) {
                return 301 https://$host$request_uri;
            }
            auth_basic "Authentication Required - for the registry";
            auth_basic_user_file /etc/ingress-controller/auth/kube-system-kube-registry.passwd;
            proxy_set_header Authorization "";

            client_max_body_size                    "1m";

            proxy_set_header Host                   $host;

            # Pass Real IP
            proxy_set_header X-Real-IP              $remote_addr;

            # Allow websocket connections
            proxy_set_header                        Upgrade           $http_upgrade;
            proxy_set_header                        Connection        $connection_upgrade;

            proxy_set_header X-Forwarded-For        $proxy_add_x_forwarded_for;
            proxy_set_header X-Forwarded-Host       $host;
            proxy_set_header X-Forwarded-Port       $pass_port;
            proxy_set_header X-Forwarded-Proto      $pass_access_scheme;

            # mitigate HTTPoxy Vulnerability
            # https://www.nginx.com/blog/mitigating-the-httpoxy-vulnerability-with-nginx/
            proxy_set_header Proxy                  "";

            # Custom headers

            proxy_connect_timeout                   5s;
            proxy_send_timeout                      60s;
            proxy_read_timeout                      60s;

            proxy_redirect                          off;
            proxy_buffering                         off;
            proxy_buffer_size                       "4k";

            proxy_http_version                      1.1;
            proxy_pass https://kube-system-kube-registry-443;
        }
    }

Oh and I've checked this file:
  auth_basic_user_file /etc/ingress-controller/auth/kube-system-kube-registry.passwd;
does exist and is valid.

@snoby you need to remove ingress.kubernetes.io/ssl-passthrough: "true" because when you use it invalidates all the other available annotations


BUG report

echo_client.cc
        send_num = my_mmap<int>(send_num_file.c_str(), false, &error_msg);
        recv_num = my_mmap<int>(recv_num_file.c_str(), false, &error_msg);

may be
        send_num = my_mmap<int>(send_num_file.c_str(), use_shm, &error_msg);
        recv_num = my_mmap<int>(recv_num_file.c_str(), use_shm, &error_msg);

send_num and recv_num  are numbers mmap-ed to local files, so false means to use normal file instead of a shared memory file.


Update 02_firewall.yml

task "enable ufw" fails if no direction is provided.
https://github.com/ansible/ansible/issues/53854

Thanks, do you know if that's new behavior?
Haven't had any issues afaik, and it's not labeled as mandatory in the docs (or in the examples).
https://docs.ansible.com/ansible/latest/modules/ufw_module.html

Ah, just read the issue you linked, I test with the oldest common version.


Removing a vote sends an API request

Resolves jjcm/nonio#139

Whoa! I just caught up on this. You were literally the first person ever to add a PR to the project unbidden so I wasn't even on the lookout for this.
Seriously, huge thank you for your contributions here. Do you have an account on nonio I can tag in the updates post/thank you post? Otherwise I can use your github username.

https://non.io/user/wbb
I am also fine with including references to the PR for preserving release history on the site itself.
I simply enjoy when the things I make are used by others, so I am simply happy to have my code included.
Non.io is an interesting concept and I'm curious in how it might develop


Add copy constructor of List and Map (BREAKING).

I found that there is no way in pocketlang to copy a list or map.
So adding some copy ctors seems a good idea.
However, this make conflict with List(n) constructor.

close temporary


Tfile runtime only supported on Linux and not MacOS

Describe the bug
Python Error: WARNING:root:Tried to import the tflite runtime, but it was not found. Trying to switching to onnxruntime instead, if appropriate models are available.
tflite_runtime is only supported on linux and not MacOS.

MILES does switch to the onnxruntime, but the python error causes MILES to not show conversation history in electron.
To Reproduce
Steps to reproduce the behavior:

Run through the full setup on MacOS.
npm start

Expected behavior
Miles should start up and show the conversation history as i talk to it.
**Laptop **

Mac OS M1 Pro

Additional context
My fix was to change the code in main.py:
if platform.system() == 'Windows' or 'Darwin':
    MODEL_PATH = "Miles/miles-50k.onnx"
    INFERENCE_FRAMEWORK = 'onnx'
    DETECTION_THRESHOLD = 0.01

I didn't want to do a pull request incase i'm just missing something.

The tflite wake word model file performs better on Mac rather than the onnx file type.
Each file type also highly depends on your audio setup and environment, onnx might work better for you specifically, or there might be no difference, but in my testing in a few different environments, tflite always had less unintended activations and more intended activations than the onnx file type.
Another issue I had with the different file types is that the detection thresholds are completely arbitrary when switching between them.
Typically the issue you're describing is mitigated by the error message just going away after you talk to Miles, for that specific error, and the history is resumed. In my testing on windows and Mac it always uses tflite on Mac, and onnx on windows without issues or error messages so I'm not sure why history doesn't resume for you, I assume it's a Mac issue because you're missing the tensor flow lite package.
If you have any other issues or questions I'd be glad to help!

Follow-up, I read the issue more carefully and I see you mentioned tensorflow lite is not a package for Mac, I assumed the full Tensorflow library included it as a module, but I know I also installed the source file directly for tf lite, so I'll update the code later to check for tf lite, and it not available I'll have it use onnx.
Thank you for reporting this!


Fix null handling in manifest

Fix #3522.

The fix is confirmed for testnet up to 4372513.

Mainnet is OK up to 5759367.

NeoFS mainnet is OK up to 6080412.

NeoFS testnet is OK up to 3967861.


Support for variable fonts

Support for variable fonts is now available in both FreeType and SheenFigure. It should also be added in Tehreer-Android along with a working demo.

Fixed in v2.4


HexagonArt

This is my art piece "HexagonArt." It is a series of hexagons lined up uniquely. The code also allows you to make it more unique by increasing the hexagons and or increasing the sizes of the hexagon!
While coding, the process was very difficult, but after multiple attempts I finally nailed it!
Overall, this is a great art piece!

Hey there! I am closing this PR due to inactivity, please create a new PR once you have updated your art!


updated cmip6 catalog

Added back in RAW stores
Updated daily_data_uri to MSFT bucket location
Removed 4 gard sv missing stores

pyramids are looking good (and loading pretty quickly) for me!

same here! looks like one prettier issue (maybe @katamartin can help fix?) but otherwise good to go i think


[BUG] CoreLightning: paying invoice returns error but payment goes through

Describe the bug
With CLN v23.05rc1 as funding source, paying an invoice returns an error but the payment goes through.
'msatoshi_sent'
500 INTERNAL SERVER ERROR

Additional context
Probably related to msats being strings is now deprecated.
https://github.com/ElementsProject/lightning/pull/5986
Desktop (please complete the following information):

LNbits version: Commit version: 6f194fe2f8efa13d67bfb5fd3e18d91d896699cd
Database: sqlite

Which version of CLN are you using this with?

CLN v23.05rc1


fixed Insiders: "undefined -  (Microsoft)" displayed for MS Account

This PR is with respect to issue #142776
@TylerLeonhardt I have implemented all the suggested changes ,
please review them and let me know if any more changes are required.

Right now, you're working off of the branch you used before, but now you should make a new branch for the fix.
Also, I think the chain of || is getting really hard to read. Can you use some if statements instead?
if we have an email and a name, use the combined string. else do the || of the rest of the options.
Btw, instead of using || you can use ?? which is slightly more correct.

@Yash621 ohh okay thanks :)


Adapter can panic on interface conversion

When the adapter fetches the proxy config from 3scale, it uses the client to do so which returns a custom error type of ApiErr. This line of code is susceptible to causing the adapter to panic.
We should use a type switch in this instance to attempt to  parse the status code, and default to returning the error message from any other type.
The case that flagged this for me was an invalid certificate on the system route (self-signed). @unleashed do we want to support allowing the deployer of the adapter to skip certificate verification?

The answer would be "make it configurable", so the responsibility is on the admin/deployer. Perhaps we can pass on the settings to the http client.
Re that line of code, yes, we should try to avoid the panicking in a known potential condition and return a suitable error (ie. when invalid certificates are not allowed).

Sure. I'll fix the bug.
We can also pass down the allow insecure flag to the client and that will fix this particular issue if that is something we want to do. I believe, with the way we have structured this, there won't be a need to change the underlying 3scale client library.


Type checker bug

[fact.age]
  encoding = "int"

feature age
  ~> let top = (filter value >= 18 ~> filter value < 25 ~> count value)
  ~> top / count value

gives this type error:
Check error:
  Cannot discharge constraints at 54:108:test/cli/sortings/input/dictionary.toml
  Constraints:
    54:108:test/cli/sortings/input/dictionary.toml  Cannot unify: Possibly
                                                    With type:    Int
                                                    These types were required to be equal, but are not.

Will have quick look at this to see if can be fixed before release today


[Enhancement] Typed View Controller from Storyboard

Hi, 😀
I had an idea and I would like to present it before I attempt a PR so we can discuss it first.
I am really not fan of that line:
let colorViewController = self.storyboard?.instantiateViewController(withIdentifier: "identifier") as! ColorViewController

So having a bit of play I came up with an idea (which is pretty simple and swifty .. 😀)
extension UIStoryboard {
    func instatiate<T: UIViewController>(type: T.Type, with identifier: String) -> T {
        return instantiateViewController(withIdentifier: identifier) as! T
    }
}

So the following can be possible:
let colorViewController = storyboard?.instatiate(type: ColorViewController.self, with: "identifier")

Also must of the times I use the same name of the class as a storyboard identifier, so we can improve it like that:
extension UIStoryboard {
    func instatiate<T: UIViewController>(type: T.Type, identifier: String? = nil) -> T {
        let identifier = identifier ?? "\(type.self)"
        return instantiateViewController(withIdentifier: identifier) as! T
    }
}

and can be used like that:
let colorViewController = storyboard?.instatiate(type: ColorViewController.self)

As a result a protocol conformance is no longer required. 😀
This is just a draft of my thoughts but please let me know if you believe that Reusable can facilitate this approach.

This approach seems nice, thanks for the suggestion 👍 I also don't like as! and prefer generics as you propose :wink:
You can see in  some techniques on how to avoid that and that it made me come up with Reusable. But basically that's a similar technique as what you present in your solution, using both generics and type inference to do some magic.
The whole point of Reusable is to get rid of the String-based APIs, including getting rid of String identifiers, because e.g. with your solution:

you can still call let colorViewController = storyboard?.instantiate(type: ColorViewController.self, identifier = "foo") somewhere and let colorViewController = storyboard?.instantiate(type: ColorViewController.self, identifier = "bar") somewhere else, which will probably not make sense as the storyboard identifier to use is probably always the same for a given ViewController in a given Storyboard, right?
you could improve your solution by adding a default value of = T.self to your type: T.Type parameter. See my talk/slides "Mixins over Inheritance" where I explain exactly this technique in Reusable among others to make all that even safer.

So basically, Reusable already allows to do something similar to what you're proposing, especially just using let colorViewController = ColorViewController.instantiate() and it will do all the magic, using the right storyboard and the right identifier automatically (specified in the type, with a default value derived from the name of the class but you can specify a different storyboard and identifier in case they are not named the same)
So, that said, I wonder if what already exists in Reusable (see also this article explaining all the steps and little magic stuff like generics behind Reusable and what you can do with it) wouldn't be sufficient for you?
Basically, what you suggested is indeed cool and uses the right bit of Swift magic, but I wonder if let colorViewController = ColorViewController.instantiate() wouldn't be even better, or if you have a specific need for still providing the identifier on every call like in your implementation? I mean, I'm totally ok to add your implementation if the implementation in Reusable is limited to you and doesn't address some cases you need, but given what you've presented, I wonder if it isn't in fact already covered by Reusable itself?

Thanks @AliSoftware  for that. 😀
Well, my suggestion doesn't cover something that is not already covered in Reusable, it just provides a different way without protocol conformance. The trade off though (in the case the identifier is same as the class name, func instatiate<T: UIViewController>(type: T.Type = T.self, identifier: String? = nil) -> T 😉 ) is that it always needs a storyboard instance .. It might come handy in cases that some view controllers are not called more than once I assume or are called from within the same storyboard of the caller. In any other case ColorViewController.instantiate() seems more appropriate. 👍🏻
That said, I really appreciate your time and your input on that 😀, I really enjoyed it. 👍🏻

I see. If you wish to still have a way to do it your way, specifying the identifier at call site every time or needing a storyboard instance, that can still be added, but tbh I quite dislike (and it's quite dangerous to, actually) adding extensions on types that are part of Cocoa and that don't use any way of disambiguation.
What I mean by that is that is basically in these cases, you'll need a prefix like func ps_instantiate instead of just func instantiate to ensure that there is no name collision in the future.
Because, sure, now Swift has the concept of modules, and if you declare a class with a given name and a class of the same name is later added to UIKit, you can always dsambiguate with the full name ModuleName.ClassName. But for extensions there's no way to do that, so an extension on a type that is part of UIKit and that adds a method that only make use of UIKit and Foundation parameters can totally conflict with a future implementation that Apple would add to UIStoryboard. The only case when you can safely create an extension on some type provided by Apple is if your extension adds functions that themselves use custom types from your own module, because that way you don't risk conflicting with an hypothetical future function of the same name added by Apple.
That's one of the main reason why I prefer Mixins and adding protocol conformance instead of adding an extension directly on UIStoryboard. Because then that change is only opt-in and won't risk propagating to every UIViewController (including the ones in other modules/frameworks you might depend on) and it only applies to what the consumer of the library choose to.
The use of the library can still totally write extension: UIViewController: StoryboardSceneBased {} somewhere in its own source code so that all UIViewControllers have that instantiate() method by default, but at least that's their choice at that point.
That's the same reason I didn't extend UITableViewCell to mark all of them as Reusableso they have a default reuseIdentifier. Because some people might not want all their cells to use that API (in case you're progressively migrating to Reusable and your old cells still don't use it but only your new ones for example) and because I didn't want to risk affecting UITableViewCells used by the system in some standard UIViewControllers or affecting cells that are part of dependent frameworks. People can still write extension UITableViewCell: Reusable {} in their own code if they want that anyway.
Note that for the part of the Reusable framework that deals with cells, I did make extensions on a UIKit type (extension UITableView and extension UICollectionView, but that's OK because those extensions only add functions that contains params or generics that depend on my own type, so there's no risk of conflicting with hypothetic future implementations added by Apple or by extensions added by other frameworks in your project.

Thanks for the explanation about what driven you to make that design decision. I really like what you have done and I find your approach really cool. 👍🏻 Truth is my suggestion doesn't fit to the current approach that you have chosen and it might feel weird to mix these two different approaches. So, I believe it will be better not to include it in Reusable.
Thanks for the time and the effort and that you put on this thread 😀,  I really appreciate it. 👍🏻


Swipe Gestures on iOS9 / Swift 2

I've had to disable the swipe gestures as I've found them not to redraw the card stack properly. For example, swiping down a card can result in it disappearing. If I then drag up the 'invisible' card it suddenly appears.
I'm not having any problems with the Pan gesture.

Thank you for reporting this issue, we will address it as soon as possible

Yes, I am experiencing a similar issue at the moment.

fixed in https://github.com/tispr/tispr-card-stack/pull/10