Support custom authorizers

Custom authorizers should also be supported by the plugin, i.e. if a service defines custom authorizers, they have to be assigned to the correct stage and either the referenced alias lambda has to be set as target or, in case a full qualified arn has been given, this has to be set.
#21 Is caused by the missing feature.

+1

The referenced lambda function has to be completed with the alias stage variable - myAuth-func:${stageVariables.SERVERLESS_ALIAS}
This will work in the first place. It has to be checked, if deployments from different aliases with changed authorizer configurations (like header spec etc.) will cause unwanted behaviors.
Otherwise the authorizer should be renamed by the plugin and deployed separately per alias.

Authorizers do not support APIG stage variables. The authorizer is renamed to include the deployed alias and points to the correct aliased function version.
Verified the functionality with a test project and by creating a Swagger definition of the deployed stage from the AWS APIG console.

@vkkis93 Could you try if it works for you?

@HyperBrain don't work for me.
I can fix it only by hand if I set func:${stageVariables.SERVERLESS_ALIAS} in Lambda function field

@HyperBrain Why previous<EMAIL_ADDRESS>for example has correct authorizer function
They have func:${stageVariables.functionAlias}
I don't understand why we can not set func:${stageVariables.SERVERLESS_ALIAS}

Ok. I will check how exactly the 0.5 version created the authorizer - I'll try to use the same parameters then and see if it works.

Can we do like this ?
const aliasedName = ${name}+':${stageVariables.SERVERLESS_ALIAS}';

Can you try this locally and check if the deployment works with the API console and a deployed API?
If it works, I have no objections against this solution.


[Bug] Trying to fetch loader script get 403/421 cloudflare cdn link is probably broken

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Link
https://microsoft.github.io/monaco-editor/playground.html?source=v0.41.0#example-creating-the-editor-hello-world
version 0.41.0 gets 403 when trying to access the cdn
after testing a bit with other versions I see that all versions under the latest version doesn't work at all
Monaco Editor Playground Code
No response
Reproduction Steps
just try and load the editor.. you get this fetch error
Actual (Problematic) Behavior
unaccessable editor...
Expected Behavior
No response
Additional Context
No response

Seems like the CDN is down

I see that jsdelivr status page saying that some systems experiencing some issues... can find any other details.. and especially no ETA for a fix... this is a production issue on my end.. super not fun

There is temporary solution here:
https://www.reddit.com/r/webdev/comments/1ci43x4/jsdelivr_cdn_is_down/

@Danhiler thanks taking a look

Seems like jsdelivr updated their certificates, it is working well now


Integration Test - Release - 2019/03/15 00:15:47

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


WIP Kim mb 5318 refactor contact info page to use formik

Description
This converts the Contact Info page in the customer profile to use Formik instead of Redux forms.  It includes validation of telephone numbers, email, and requires a 'preferred contact method' to be checked.
Reviewer Notes

WIP ISSUES to resolve:
• The telephone field Required validation message isn't showing as expected upon blur the way email is.  It does show up after some clicking in and out or removing the value.
• 4 tests in ContactInfoForm.test.jsx are failing. 2 are due to not finding a labelText Alt. phone, though I don't know why it can't find it.  The other two are not showing the fields as invalid- not sure if this is a test issue or an actual issue with the component.
• I'm getting the this warning Warning: An update to Formik inside a test was not wrapped in act(...). in the ContactInfo.test.jsx, although I'm using async and await.

This PR includes the back button workaround (first implemented in this PR. Thread about it here.
Setup

Login as a new customer user.  Fill the profile in until you get to the Contact info page.  Enter phone numbers.  Check that Best contact phone and personal email show  Required on blur when empty.
Check that an invalid or incomplete telephone or email cannot be created and you see a validation message when invalid.
Check that the back and next buttons do indeed save all field values when valid.
Check that the back and next buttons do not save and do show the validation errors when clicked.

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
[this Yup documentation](git push --set-upstream origin kim-MB-5318-refactor-contact-info) explains more about the approach used for the preferred contact method validation.

Screenshots

Warnings

:warning:

It looks like you are attempting to bypass a linter rule, which is not within
security compliance rules.
** Contains a rule that is not in the permitted eslint list. You are free to disable only: (
no-underscore-dangle
,prefer-object-spread
,object-shorthand
,camelcase
,react/jsx-props-no-spreading
,react/destructuring-assignment
,react/forbid-prop-types
,react/prefer-stateless-function
,react/sort-comp
,import/no-extraneous-dependencies
,import/order
,import/prefer-default-export
,import/no-named-as-default
) **
Please remove the bypass code and address the underlying issue. cc: @transcom/Truss-Pamplemoose

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against f6328b89697ffebf26b9162969e2de764975ebea


Width of the fields when grouping

Hi there,
I tried to use the material templates, but when I try to use it grouping fields to get like 3 fields in the same row, I don't see how can I set the width for the fields, so when it renders, I get this:

and this is my configuration:
{
                elementAttributes: {
                    "layout": "row"
                },
                fieldGroup: [
                    {
                        key: 'existId',
                        type: 'select',
                        templateOptions: {
                            label: "¿Presenta documento de identificación?",
                            placeholder: 'Seleccione..',
                            options: [
                                {key: 'default', value: labelsContact.options.defaultSelect},
                                {key: '1', value: labelsContact.options.yes},
                                {key: '2', value: labelsContact.options.no},
                                {key: '3', value: labelsContact.options.process}
                            ],
                            valueProp: 'key',
                            labelProp: 'value',
                            "required": true

                        },
                        validators: selectDefaultValidator
                    },
                    {
                        key: 'idType',
                        type: 'select',
                        templateOptions: {
                            label: "",
                            placeholder: 'Seleccione..',
                            options: [],
                            valueProp: 'key',
                            labelProp: 'value',
                            "required": true
                        }
                    },
                    {
                        key: 'idNumber',
                        type: 'input',
                        templateOptions: {
                            label: labelsContact.fields.idNumber,
                            "required": true
                        }
                    }
                ]
            }

How can I handle  this with material templates?
Thanks.

@ColadaFF I'm going to look it up in few days.

@ColadaFF Take a look at this http://angular-formly.com/#/example/other/advanced-layout-flex
I prepared playground of your example.
http://codepen.io/kamilkisiela/pen/mVqjZP
The problem is related to ngMaterial.
<md-input-container> has two children: <label> and <md-select>.
<label> (which contains templateOptions.label) has the same width as <select>.
So you can try to add custom css.

I have a similar problem :
http://codepen.io/anon/pen/JGxLGr
how do I make field1 50% and field2 50% of the width ?

anyone got any ideas ?

ok, 17 days without any response on this or another question - so I have to ask - is this project dead ? If so, please just say so that we don't waste time evaluating this.

@jmls It's not dead. I've got lot of stuff happening and now I'm back.
ngMaterial changed something in css so it has to be fixed now. I'm going to work on it at the end of today.

you da man ;) thanks


Flash messages should not be escaped

The Flash messages from session are automatically escaped, causing chars like single quotes to be shown as ' . Use {!!  !!} to prevent escape.

@stefano73it Hi,
Legit concern but shouldn't users be aware of these type of situations as this can occur at many place other than session flash also.


Add backend support matrices for different platforms

@wphicks  Can you verify the accuracy for FIL backend support across platforms?
@szalpal  Can you do the same for DALI backend?

FIL info is correct.


bug when you press a key while going out of tab

the game does not pause and hero keep going in the direction pressed

This is because when the tab is not focused, it does not get the "keyup" event.


settings are not saved

Settings.ini is not correctly used when installed via pip. This appears to be due to a relative path issue.

Relative Path was missing in the "Save settings" option.


Onsave Event Not Dispatching (Jspreadsheet V8.7 and V9)

Hi guys! I'm a JSpreadsheet Pro subscriber and I'm attempting to hook into the onsave event. As far as I can tell, it isn't firing for me on both V9 and V8.7.
I'm attempting to send back an ID from my server to the client in order to keep my front and back ends in sync. The idea is to process the server response during this event, but it looks like the event isn't getting dispatched.
I can provide my spreadsheet config if that would be helpful.
Thanks!

Hi 😊
for pro support, you can open issue on https://github.com/jspreadsheet/pro/issues
For your questions : to dispatch event onsave, you need define persistence property with url of your server. If you want manage your system of save without persistence. You need to use onchange/onbeforechange/onafterchanges


has_optional_field implementation may lead to unexpected results

has_optional_field(field) verifies whether the value of field is not None. But, some attributes are sometimes initialized as some default/empty object, like an empty list. In that example, has_optional_field(field) will always return True, even if the list is empty (when I think it should return False because there is actually no information).
I suggest: return bool(field)

Can I work on this issue ?


Fix login issue with complex passwords

UrlEncodes the password, which is required when you have certain characters in your password (e.g. &, ^, ?).
Should work with any password, but I can't test that. It works on my PC I guess.
This commit does introduce a 'security issue' because the encoded password is stored in a managed string. You could work around that by using HttpUtility.UrlEncodeToBytes and Buffer.BlockCopy to merge the resulting arrays. In that case you won't have a managed string, I guess?
Feel free to deny this PR if you feel like this is an issue.
On an unrelated note, I would recommend not using SecureString in the future. I saw that you were rewriting this mod manager, so you could probably remove it there. Why not use SecureString? Furthermore, Avalonia and .NET Core don't even support it. And it's a pain to use.

Thanks!
I am no longer actively developing here but I will carry this over to MMF2 for sure. I already got rid of SecureString because .Net Core does indeed not support it and that's what I'm primarily targeting now. In Reality it doesn't even provide that much more security anyway, String is fine as long as users don't have a Virus running on their machine.


TypeError: 'PretrainedCfg' object is not subscriptable

Hello, I have TypeError Issue for training with main cifar100-convprompt
pretrained_custom_load='npz' in pretrained_cfg['url'],
~~~~~~~~~~~~~~^^^^^^^
TypeError: 'PretrainedCfg' object is not subscriptable
could you check it?

Hi, this seems like an error with the pre-trained config for vit. Could you please share the entire log containing line numbers and file names?

Here's the full log
File "/home/dbice/workspace/CL/ConvPrompt/main.py", line 206, in
main(args)
File "/home/dbice/workspace/CL/ConvPrompt/main.py", line 53, in main
model = create_model(
^^^^^^^^^^^^^
File "/home/dbice/anaconda3/lib/python3.11/site-packages/timm/models/_factory.py", line 117, in create_model
model = create_fn(
^^^^^^^^^^
File "/home/dbice/workspace/CL/ConvPrompt/vision_transformer.py", line 1079, in vit_base_patch16_224_in21k
model = _create_vision_transformer('vit_base_patch16_224_in21k', pretrained=pretrained, **model_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dbice/workspace/CL/ConvPrompt/vision_transformer.py", line 842, in _create_vision_transformer
pretrained_custom_load='npz' in pretrained_cfg['url'],
~~~~~~~~~~~~~~^^^^^^^
TypeError: 'PretrainedCfg' object is not subscriptable

Sorry, I couldn't reproduce the error. Also, there is no PretrainedCfg in the code.
Did you by any chance make any changes to the code?
Please see if  git checkout . helps.

I hope the issue has been solved. I am closing it.


Make editor compatible with browser back/forward buttons

This is part of DataStewardshipWizard/dsw-common#58.


Authorization issue

What steps will reproduce the problem?
Simply you can see this short screen video http://i.de-v.net/1478867817.webm
It seems that the session is not being flushed properly after logout.
What is the expected result?
After login it should redirect me to the prev page.
What do you get instead?
I redirected to the logout url after login
Additional info
I using starter kit project based yii2 advanced

Q
A

Yii version
2.0.10

PHP version
5.5.9

Operating system
Ubuntu 14.04.5 LTS

Could you reproduce it using basic project template?

Can not reproduce neither on the local yiisoft/yii2-app-basic installation, nor on the demo installation of trntv/yii2-starter-kit (https://github.com/trntv/yii2-starter-kit#demo)


Installing itk-js

Hi,
When I try to install itk through node & webpack, this is failing. The folder created is strange, it contains every module on the root, no package.json, no build file, nothing usefull.
In unpkg.com itself an error is displayed<EMAIL_ADDRESS>Cannot find "/dist/itk.js" in<EMAIL_ADDRESS>Is it the normal behaviour? Am I missing the whole thing?

Install the itk.js node package with, and:
npm install --save itk

This installs the built package. It contains a package.json, LICENSE, README.md, the JavaScript modules that will be used by dependent project sources, along with Emscripten module image IO, mesh IO, related web workers. The latter are pre-built, so you only need the standard node and npm to use the package in a dependent project.
That is not the correct path in unpkg, but the package would not be used in this way from unpkg regardless.
For more information, please see the documentation:
https://insightsoftwareconsortium.github.io/itk-js/examples/


Upgrade Django version

Upgrading from 1.7.x to 1.9.x here.
I am opening this pull request to get some feedback (it is a work in progress).
Some observations:
1 - It needs a rebase.
2 - It needs a cleanup on settings.
3 - Python requirements need to have their proper PyPI packages. Working on it.

It should fix #129


Game Error

游戏初期不会发生，但是当生产规模变大就会。一直玩游戏不退出不会发生，但是某一次的读档就会
won't happen at early stages of the game when there were no mass pruduction.
won't happen if continue playing the game without quitting. But sometime you try to load the savegame will cause it.

这个我自己也遇到了，应该在1.0.5版里修复了，原因是插件没有正确处理物流塔的删除。


[Bug] Exception output when Cuda Graph is enabled for Qwen2.5-Coder

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 5. Please use English, otherwise it will be closed.

Describe the bug
I'm using sglang to launch an OpenAI-compatible server with Qwen2.5-Coder-7B-Instruct, one of the best code models. I then use OpenAI library to send concurrent requests for data generation. However, I noticed some abnormal outputs in the generated results (dozens of exception outputs for thousands of requests, others seem to work well), similar to the following (I provide user content and assistant content is returned by sglang):
[
    {
        "role": "user",
        "content": "Develop a class in Python that accepts three arguments, \"username\", \"birthdate\", and \"email\"."
    },
    {
        "role": "assistant",
        "content": "Here can create to and"
    }
],
[
    {
        "role": "user",
        "content": "Create a Node.js module for validating email addresses."
    },
    {
        "role": "assistant",
        "content": "Here static1\n2\n\n"
    }
]

Initially, I thought the issue was with the tokenizer, but after careful examination, I found nothing unusual. I then tested LLama3.1 and Deepseek-Coder, and their outputs were correct, which ruled out any environmental issues.
Finally, using --disable-cuda-graph resolved the issue. In large batch scenarios, disabling Cuda graph has minimal impact, but given that it’s enabled by default, I hope it can be fixed.
Reproduction
The command causes exception output:
python -m sglang.launch_server --model-path Qwen/Qwen2.5-Coder-7B-Instruct --port 10086 --schedule-conservativeness 0.3 --mem-fraction-static 0.8

The command works well:
python -m sglang.launch_server --model-path Qwen/Qwen2.5-Coder-7B-Instruct --port 10086 --schedule-conservativeness 0.3 --mem-fraction-static 0.8 --disable-cuda-graph

I use LLMs to detect unusual output, so providing a quick repro script is a bit complicated, if this is confirmed as a bug, I will try to provide it.
 It has been fixed by #1779, #1782, and #1789
Try 0.3.4.post2!


Is there an example where we can create a Bacnet virtual device and add read and write capabilities?

Since I am new to BACnet, I need an example showing how to create a BACnet device and read and write data.

This library is right now more a bacnet client than a server ... So it can interact with devices, but not offer a device

Thank you for responding so quickly.
Another question, do we have any other library where we can achieve the above functionality in nodejs?


Update gem to 0.8.1

Improves the styling of the new homepage by making the buttons appear below the main content, rather than on the sides of the screen.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.066%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/spina/conferences/primer_theme/version.rb
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
227

Relevant Lines:
242

💛  - Coveralls


Edge Add-ons Publish API is now live!

Discussed in https://github.com/microsoft/MicrosoftEdge-Extensions/discussions/13

Originally posted by nagachaitanyalokam May  6, 2022
With this addition, you can integrate APIs directly into your build pipeline and deploy packages and publish updated versions without visiting the Partner Center.
To learn more, you can check the below documentations:
Using the Microsoft Edge Add-ons API
Microsoft Edge Add-ons API Reference
Read more about changes made to streamline the process of submitting a Microsoft Edge extension here

Hi @JohnnyDa5 , the image you have shared is broken. Could you re-send the image or give us more details about the trouble you are facing? This would help our team understand what exact problem you are dealing with and come up with a solution.

@JohnnyDa5 - thank you for celebrating the add-ons API release. Since this is not a bug, closing it for now. Happy to hear back from you.


AutoSizeBox not re-rendering when url changes

Hi, I'm using version 1.7.6 and AutoSizeBox and it's not rendering new image when the url changes. I only seem to be seeing this though in the Compose for Desktop client.  It should be possible to see this happening by running that client in https://github.com/joreilly/Confetti.  The following is how it's used so maybe something I'm not doing correctly here (in SessionDetailsViewShared.kt)
                AutoSizeBox(speaker.photoUrl) { action ->
                    when (action) {
                        is ImageAction.Success -> {
                            Image(
                                rememberImageSuccessPainter(action), null,
                                modifier = Modifier.size(64.dp).clip(CircleShape),
                                contentScale = ContentScale.Crop,
                            )
                        }
                        is ImageAction.Loading -> {
                            CircularProgressIndicator()
                        }
                        is ImageAction.Failure -> {
                            Image(
                                painter = painterResource(Res.drawable.ic_person_black_24dp),
                                contentDescription = speaker.name,
                                contentScale = ContentScale.Fit,
                                modifier = Modifier.size(64.dp).clip(CircleShape)
                            )
                        }
                    }
                }

It’s seem similar to my previous fix of AutoSizeImage, on desktop invalidateMeasurementwill not triggerinvalidateDraw. Can you help me to try AutoSizeImagecan run fine?AutoSizeBox` could also be for a more similar reason, but needs further investigation.

Yes, AutoSizeImage works fine.


Binding expressions don't appear to work with Python code

Within the "Trigger - example" section, it states, "The string {name} in the blob trigger path samples-workitems/{name} creates a binding expression that you can use in function code to access the file name of the triggering blob. For more information, see Blob name patterns later in this article."
However, this appears to be misleading as {name} only works as it is an attribute of the InputStream class, whereas if you use other binding expressions, they do not work and throw an error when the Azure function executes (i.e. "Exception: AttributeError: 'InputStream' object has no attribute '<binding-expression>'", where <binding-expression> is the binding expression used in the function.json file.)

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

ID: fff7ae2f-4f02-7ed9-c2ed-a2715e407975
Version Independent ID: 542016c1-3c25-f6af-cc83-0c5b6218e0ce
Content: Azure Blob storage bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-storage-blob.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@zegor Thanks for your comment. We are actively investigating and will get back to you shortly.

@zegor I have assigned this to content author for further review and update as appropriate.

Relevant post: https://stackoverflow.com/questions/59181302/blob-input-binding-with-azure-function-in-python-is-not-working-inputblob-name

To be more specific the relevant post I found is this one:https://github.com/microsoft/azure-docs/blob/master/articles/azure-functions/functions-bindings-storage-blob-input.md.
this exact statement
                 f"Name: {myblob.name}\n"
                 f"Blob Size: {myblob.length} bytes")

returns something like this
Name: None
Blob Size: None bytes

I checked the corresponding SDK[1] as well as the class mentioned,  we have the corresponding attributes available and even the code which I got by the default installation from vscode samples.

class InputStream(io.BufferedIOBase, abc.ABC):
    """File-like object representing an input blob."""

    @abc.abstractmethod
    def read(self, size=-1) -> bytes:
        """Return and read up to *size* bytes.

        :param int size:
            The number of bytes to read.  If the argument is omitted,
            ``None``, or negative, data is read and returned until
            EOF is reached.

        :return:
            Bytes read from the input stream.
        """
        pass

    @property
    @abc.abstractmethod
    def name(self) -> typing.Optional[str]:
        """The name of the blob."""
        pass

    @property
    @abc.abstractmethod
    def length(self) -> typing.Optional[int]:
        """The size of the blob in bytes."""
        pass

    @property
    @abc.abstractmethod
    def uri(self) -> typing.Optional[str]:
        """The blob's primary location URI."""
        pass

I wonder how come it doesn't work.
[1] https://docs.microsoft.com/en-us/python/api/azure-functions/azure.functions.inputstream?view=azure-python

#reassign: @stefanushinardi


Support for DuckDB v1.1.0

Writing to notify that at this moment in time this extension do not successfully build or test against duckdb v1.1.0.
Extension will keep being available on duckdb v1.0.0, but can't be distributed on v1.1.0 until the problem is sorted out.
Problem, as detected by the test infrastructure is as follow: https://github.com/duckdb/community-extensions/actions/runs/10773618982/job/29873889918#step:16:5385
[3/10] (30%): /Users/runner/work/community-extensions/community-extensions/te...
Query unexpectedly succeeded! (/Users/runner/work/community-extensions/community-extensions/test/sql/scrooge/test_eth_scan.test:71)!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
================================================================================
unittest is a Catch v2.13.7 host application.
Run with -? for options

-------------------------------------------------------------------------------
/Users/runner/work/community-extensions/community-extensions/test/sql/scrooge/
test_eth_scan.test
-------------------------------------------------------------------------------
/Users/runner/work/community-extensions/community-extensions/duckdb/test/sqlite/test_sqllogictest.cpp:219
...............................................................................

FROM read_eth(
    'USDT',
    'Transfer',
<PHONE_NUMBER>9,
<PHONE_NUMBER>99
);

More information / example of what other extension needed to change at https://github.com/duckdb/community-extensions/issues/85.
Thanks for being involved with the DuckDB community extensions!

Hi Carlo, thanks for all the hard work! I've updated scrooge and bumped in the community extensions repo!
I guess that should be it?

Yep, https://github.com/duckdb/community-extensions/pull/110 looks good, test passed.
I will wait for CI to go through and then merge it.
I think this can then be closed, and thank you!


fix: parse WITH command as SELECT

Parses WITH statements as queries and allows other whitespace characters than only ' '.
Fixes #5857

LGTM. But can you add an integration test that uses WITH? Just to make extra certain.

I've added an integration test case to the existing ReadTests class.

LGTM. But can you add an integration test that uses WITH? Just to make extra certain.

I've added an integration test case to the existing ReadTests class.


Fix heartbeat returning too fast (#5973)

Pull request opened by the merge tool on behalf of #5973

Merged into branches iso6 in 8f8222cc4d05b0361b09fac4120be1181b30618a


Clean up, Travis testing, and SFTP support

Hi Oleksii,
I have applied clean-up changed by @eudoxia0, and a modified version of his .travis.yml.
Rewrote part of examples.lisp as stefil tests, which need an SSH server and a test user. For details of how integration testing works, please look at .travis.yml.
My main change is of course the sftp support which currently consists of sftp-list-directory and sftp-get.
In addition, I introduced some condition types to allow for meaningful use of handler-bind (like in run-all-tests in test/package.lisp).
As I found myself adding a lot of (format t "....") to understand the error-handling, I thought that logging would be a good idea and added a minimal amount of logging with hu.dwim.logger.
Should you have any questions, I will be happy to discuss them.
Cheers
Kambiz

What I forgot: if you want to see the Travis tests in action, you have to

sign on to http://travis-ci.org (using your github account)
activate your cl-ssh2 repo
push a change to the repo

After pushing a change, a build is scheduled which you can follow on travis-ci.org.


ORACLE Onboarding - 1. Validate Test Files [ETOR]

Acceptance Criteria

[x] Sample file from sender received by RS team
[x] Sample files saved to Keybase by assigned RS engineer
[x] Sample file ingested into RS using validation UI tool by assigned RS engineer
[ ] Feedback provided to sender by assigned RS engineer  (including instructions on what to modify in order to get validated)*
[ ] Sender modifies and re-sends sample file to get file validated by RS engineer*

*Steps can continue until RS can ingest the file from sender
To Do

[ ] RS engineer: update this ticket with comments with every interaction with sender
[ ] RS team member who originates this ticket: add any dependencies, as needed
[ ] RS team member who originates this ticket: make sure the following are included when creating this Issue.

[ ] Pipelines: Onboarding & Operations (New)
[ ] Labels: "Onboarding-ops" and "Sender"
[ ] Epics: [select the epic for this API onboarding sender]

@oslynn to confirm status with Flexion and determine if ticket should be closed

We are still waiting for status from Flexion on ORACLE work

Flexion moving this work out to early 2025

Not going to use ETOR any longer.


All frames after frame 32 are empty every time i save/load the piskel file.

Bug description
In my current sprite, I have created frames after frame 32 multiple times, saved them through all available means, and tried loading them through all available means. Each time the result is that all frames after 32 still exist but are fully blank/erased.
Steps to reproduce the bug
save sprite. load sprite.
Environment details

operating system: Windows 10
browser (or offline application version): Chrome

Sprite details

number of frames: 32 + blank/deleted frames
sprite resolution: 256X256
session duration: Several  Hours

Feel free to include a screenshot of the bug, a .piskel file of the sprite or anything else that can help us investigate.

You should save to your desktop 🤦🏻♂. You have to do that because their server can't that big sizes of sprites.


Add search bar styling, PDB Redo filter, auto-suggestions

:pushpin: References

Issue: Closes

Add filter to show only entries with "PDB-REDO"
Redesign filters in covid page
Add autosuggestion in covid search

:memo: Implementation

Implemented new search bar style and filters style. Filters will now be in a mini-menu and filter selections will be chips that are inside the search bar area.
Added PDB-Redo as a filter with Arnau's help. Because of the way the PDB-redo info is extracted on the fly, only the rendered rows have PDB-redo info. So I can only filter by PDB-redo on the page that users see.
Added autosuggestion and changed the search bar to an Autocomplete Material UI Lab component

:art: Screenshots

:fire: How to test it? (If there is any special consideration or the reviewer does not know how to test it)
:bookmark_tabs: Others

[ ] Any change in the API repo? If so, what branch/PR?
[ ] Any change in the GUI library? If so, what branch/PR?

Thanks @jvfdunkley for the changes! @tokland I am fine with merging this branch. Let me know if there is anything pending on your side. As I said, there are some minor things I would like to refine in terms of styling but I will work with @p3rcypj on them. Once we merge his branch

@jvfdunkley I think we should take care of @tokland second comment at least:
If the user enters a search string and then clicks out of the input text, the search is lost. <- It should stick
I truly think it is important to fix it to improve the user experience. Can you take care of this tomorrow?


aws:s3:BucketObjectv2 should only trigger an update when my code changes

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Even when I don't change my code, the aws:s3:BucketObjectv2 goes through an Update cycle when I run pulumi up. I expect it to only do that when code changes. Does pulumi perform any hashing and caching to avoid redundant work like this?

Can you share the inputs you are passing in before and after the initial update?

Added to epic https://github.com/pulumi/home/issues/2191


Add Connor Phee to Contributors list

@connorphee, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

Roshan,
Thank you for the great message and interest.
I am currently a senior in college studying CS. The past two summers I have interned at a large company as a software engineer intern, and have accepted a position there, starting when I graduate.
Because of this, I have a good amount of experience with git and PR's. Despite this, I have been very intimidated to get involved in open source. I feel very eager to get involved, and to help grow the community, but have been more of a passive follower. I ran into something promoting Hacktober fest and figured now was as good a time as any to dive in.
The tutorial and docs for your project were great, easy and clear to follow. Learning my PR was merged was a great feeling, I feel as if a gate has been lifted and I can continue forward.
Sincerely, Connor Phee

From: Roshan Jossey<EMAIL_ADDRESS>Sent: Sunday, October 1, 2017 1:16:41 PM
To: Roshanjossey/first-contributions
Cc: Phee, Connor R; Mention
Subject: Re: [Roshanjossey/first-contributions] Add Connor Phee to Contributors list (#592)
@connorpheehttps://github.com/connorphee, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Roshanjossey/first-contributions/pull/592#issuecomment-333391692, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ade6L1tBqQgblvQjYraifB_pRHkUvxEUks5sn8j5gaJpZM4Pp9Xr.

@connorphee, I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


Update tdr-generated-graphql to 0.0.374 in scala-steward-dependencies

About this PR
📦 Updates uk.gov.nationalarchives:tdr-generated-graphql from 0.0.372 to 0.0.374
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "tdr-generated-graphql" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "tdr-generated-graphql" }
}]

labels: library-update, early-semver-major, semver-spec-patch, commit-count:1

Superseded by #30.


macOS CI erroring on gmpy backend missing

See e.g. https://travis-ci.org/scipy/scipy/jobs/410512300. Multiple PRs are failing on this.  No new mpmath or gmpy2 releases on PyPI, but given that the failure is macOS specific, maybe some change in one of the packages in Homebrew. Needs investigating.

Related to #8326?

See also review comments on #6427.
(not a fan of homebrew myself).

Why does it matter what the mpmath backend is, so long as a calculation can be done?

IIRC that's because without the gmpy2 acceleration, mpmath is so much slower that we'll get CI timeouts after 50 minutes.

Not sure that those two issues explain very much. Probably best to go through and do what .travisci does for macOS locally to replicate the problem.

I've just tried disabling the gmpy assert in the travis setup. It won't make the linux builds take any longer because they typically don't have a problem with installing gmpy2. The macOS test doesn't fail and completes well within the timeout limit.


Winbar error when moving around

Describe the bug
I see this stack trace:
Error detected while processing CursorMoved Autocommands for "<buffer=1>":
Error executing lua callback: ...k/packer/start/lspsaga.nvim/lua/lspsaga/symbolwinbar.lua:258: Vim:E36: Not enough room
stack traceback:
        [C]: in function '__newindex'
        ...k/packer/start/lspsaga.nvim/lua/lspsaga/symbolwinbar.lua:258: in function 'render_fn'
        ...k/packer/start/lspsaga.nvim/lua/lspsaga/symbolwinbar.lua:313: in function 'init_buf_symbols'
        ...k/packer/start/lspsaga.nvim/lua/lspsaga/symbolwinbar.lua:335: in function <...k/packer/start/lspsaga.nvim/lua/lspsaga/symbolwinbar.lua:334>

It seems like it triggers with the Matchup plugin shows an off screen paren match (I'm using the matchup_matchparen_offscreen = {method= 'popup'} setting).
Steps to reproduce
Use nvim with the following plugins enabled (packer config here):
  use { 'andymass/vim-matchup', config = function()
    vim.g.matchup_matchparen_hi_background = 2
    vim.g.matchup_matchparen_offscreen = {method= 'popup'}
  end}

      require('lspsaga').setup({
        symbol_in_winbar = {
          enable = true,
          show_file = false,
        },
        lightbulb = {
          enable = false,
          saga_winblend = 20,
        }
      })

Open a file that has some paran matching of some kind. For instance:
    package "Intell 2800 - Simple Version" {
      [ComputerCommunications] -up- USB
      [ComputerCommunications] - [MainComputerProcess]
      [KinectProcessing] -down-> KINECT
      [KinectProcessing] - [MainComputerProcess]
      [VideoProcessing] -down-> Camera
      [VideoProcessing] - [MainComputerProcess]
      [ComputerCommunications2] -up- [MainComputerProcess]
      [ComputerCommunications2] -down- WiFi
      [ComputerCommunications2] -down- Bluetooth
    }
    EXTRATEXT

Position the cursor at EXTRATEXT and then use  to scoot the first line off screen. j to move to the '}' character.
The matchup off screen text is displayed (the first line), and a stack trace appears for the winbar plugin.
Expected behavior
no stack trace.
Neovim version (nvim -v)
NVIM v0.9.0-dev-589+gcce736218-dirty
lspsaga commit
79e1f5b81e9356b27e52b162dbfd1d5e242b8c7d
Terminal name/version
wezterm

does this file have lsp server? and if have which server is.

Yes...yamlls and null-ls

the matchup is builtin plugin or vim-matchup ?

sorry about that - vim-matchup!

does that file is yml file ? not looks like normal yaml file .  I think the reproduce step should more clear ..and I guess this is neovim bug

Yeah, sorry I only have you a snippet. Its actually a yaml file that looks more like:
---
- key: |-
    some other code {
       ...
       ...
    }

I've reproduced the error in a Lua and TypeScript file as well.

can not reproduce i don't use that plugin

I use a min config for test and it works well.

This happens to me as well.
Screencast from 01-19-2023 09:13:20 PM.webm

did you read error message?

Sorry, my bad. I just read the errors and those are identical.


(feature) auto w tap (suggestion)

what is w tap?
w tap is a strategy that requires you to run and hit massive knockback to your opponents.
how does this module works?
everytime you hit an enemy while running the knockback will be very high. Therefore, everytime you hit the opponent it will reset your run and you can hit him again (and the cycle goes)

this is just a concept/suggestion i dont mind if you dont want to add this. :v

will look into this

added in 1.4.


错误

Problem encountered on https://dotnet.microsoft.com/zh-cn/learn/dotnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Can you give us more information about the error you're having?


Request for Documentation: Clarify the role's/tradeoffs/recommendations on using mmctl versus the legacy cli

Mattermost user stu.doherty from https://community-daily.mattermost.com has requested the following be documented:
We need to put appropriate callouts in the docs for `cli` versus `mmctl` (which is in BETA).

E.g.,
* If using CLI with an HA environment admins need to be aware that any configuration changes will require a service restart.  `mmctl` does not have this requirement

We should put clear callouts under the `cli` that `mmctl` is more appropriate for HA environments (assuming this is what we believe)

See the original post here.
This issue was generated from Mattermost using the Doc Up plugin.

Also might need to have a call out on this page:
https://docs.mattermost.com/administration/config-in-database.html
Making note that "HA Deployments should use mmctl for modifying configuration" in the cluster... we need to confirm this line, but this is the conclusion I'm coming to.

Documentation updated via: https://github.com/mattermost/docs/pull/4326


Modify docs 1129 v3

PR types

Others
PR changes

Docs
Description

1.删除./llm/config中各模型的README文件
2.将./llm/config各模型README中的信息更新至主页README，并删除主页README中指向已删除文件的超链接
3.删除./llm下README中的超链接

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


update readme

Small documentation patch, so that people can try it out more easily, by copy pasting.

Thanks for fixing this.  Can you add this fix to the other examples in the readme?  I'll merge it then.


Basic Bandit example in Documentation does not work - gives error of unclear significance. Solution?

If you make a project as per:
https://hexdocs.pm/plug/readme.html
https://hexdocs.pm/bandit/Bandit.html
You can get:
#PLUG MODULE FOR BANDIT
defmodule MyPlug do
  import Plug.Conn

  def init(options) do
    # initialize options
    options
  end

  def call(conn, _opts) do
    conn
    |> put_resp_content_type("text/plain")
    |> send_resp(200, "Hello world")
  end
end

#MAIN MODULE
defmodule ServerTest do

  def start_http_server do
    IO.puts("Try Start HTTP Server");
    webserver = {Bandit, plug: MyPlug, scheme: :http, port: 4000 }
    supervisorOpts = [strategy: :one_for_one ]
    {:ok, _} = Supervisor.start_link([webserver], supervisorOpts)
    IO.puts("Plug now running on localhost:4000");
    #Process.sleep(:infinity)
  end

end

If you add this to a project and then run ServerTest.start_http_server(), then go localhost:4000 you will get a “Hello world” back. This is fine. However, you will also get this:

21:11:37.278 [error] GenServer #PID<0.554.0> terminating
** (stop) :econnaborted
Last message: {:tcp_error, #Port<0.3>, :econnaborted}
State: {%ThousandIsland.Socket{socket: #Port<0.3>, transport_module: ThousandIsland.Transports.TCP, read_timeout: 60000, silent_terminate_on_error: false, span: %ThousandIsland.Telemetry{span_name: :connection, telemetry_span_context: #Reference<0.267163659.1006895106.157280>, start_time: -5764607406509056, start_metadata: %{telemetry_span_context: #Reference<0.267163659.1006895106.157280>, parent_telemetry_span_context: #Reference<0.267163659.1006895106.157220>, remote_address: {127, 0, 0, 1}, remote_port: 63311}}}, %{opts: %{http: [], http_1: [], http_2: [], websocket: []}, plug: {MyPlug, []}, handler_module: Bandit.HTTP1.Handler, http_1_enabled: true, http_2_enabled: true, requests_processed: 2}}

21:13:32.629 [error] ** (Bandit.HTTPError) Header read socket error: :closed

I can find no information on what this means. With such simple code I am not creating any GenServer myself so this is obviously a GenServer that is running inside the Bandit code.
Is this a bug or is there something missing in the documentation to avoid this? What is the fix?

There's nothing wrong with how you've set up Bandit; it's running and serving your plug just fine. The example is correct.
The issue you're seeing is an error indicating that Bandit errored attempting to read headers from your client. Typically this is a client-side or network issue, where the complete client request does not make it to the server (hence the error). The reality of network facing logic is that there's always remote issues of one type or another; you'll never be rid of them entirely.
That being said, this does suggest a misbehaving client of one type or another. What client were you using?

I have run this as follows:

Run iex.bat -S mix from powershell in Windows 11
In Firefox on same computer/OS running, go to localhost:4000.

I note the errors are not related. There are two:
EConnAborted

17:26:22.812 [error] GenServer #PID<0.545.0> terminating
** (stop) :econnaborted
Last message: {:tcp_error, #Port<0.5>, :econnaborted}
State: {%ThousandIsland.Socket{socket: #Port<0.5>, transport_module: ThousandIsland.Transports.TCP, read_timeout: 60000, silent_terminate_on_error: false, span: %ThousandIsland.Telemetry{span_name: :connection, telemetry_span_context: #Reference<0.2506072894.74711041.114195>, start_time: -5764604723200000, start_metadata: %{telemetry_span_context: #Reference<0.2506072894.74711041.114195>, parent_telemetry_span_context: #Reference<0.2506072894.74711045.114199>, remote_address: {127, 0, 0, 1}, remote_port: 51484}}}, %{opts: %{http: [], http_1: [], http_2: [], websocket: []}, plug: {MyPlug, []}, handler_module: Bandit.HTTP1.Handler, http_1_enabled: true, http_2_enabled: true, requests_processed: 2}}

Header Timeout

17:10:21.466 [error] ** (Bandit.HTTPError) Header read timeout

These do not occur at the same time, but randomly. I so far cannot identify what is causing them or how to trigger them reliably. Sometimes I get them, sometimes I don't. But they keep happening.
Is there any way to debug further the cause or handle these errors in some way?
I can appreciate that perhaps things may never be perfect, but this is the simplest proof of concept for the Bandit server system I can imagine and I am already being spammed with errors, which is not ideal.
I have also tested as per the documents WebSocket system which provides the same errors:
#//=====================
#//HTTP SOCKET SERVER //https://hexdocs.pm/websock/0.5.3/WebSock.html
#//=====================
defmodule EchoServer do
  def init(options) do
    {:ok, options}
  end

  #handle_in function //WebSock will call the configured handler's WebSock.handle_in/2 callback whenever data is received from the client
  def handle_in({message_in, [opcode: :text]}, state) do

    IO.puts("RECEIVED SOMETHING");
    case message_in do

      "ping" ->
        IO.puts("PING IN REQUEST RECEIVED");
        {:reply, :ok, {:text, "pong"}, state}

      _ ->
        IO.puts("RECEIVED: " <> to_string(message_in));
        {:reply, :ok, {:text, to_string(message_in)}, state}
    end

  end
end

defmodule ServerTest do

  def start_web_socket do

    webserver = {Bandit, plug: Router, scheme: :http, port: 4001}
    supervisorOpts = [strategy: :one_for_one ]
    {:ok, _} = Supervisor.start_link([webserver], supervisorOpts)

  end
end

Is there some way to gather more information about these errors? It seems they are all being processed and outputted in your code, not anything I have access to. Is that correct? Or can I do something with handle_info or something to catch them or see what is happening or suppress them or what?
Thanks for any help.

I tested the code with Cowboy instead. So far, since switching to Cowboy I have seen neither of these two errors again.
Thus I do believe this is Bandit bug. I would prefer to use Bandit, because as you say it is more efficient. But error free basic function is necessary as well. If I am getting errors from the most ideal simple basic test on a single computer, this does not bode well for a large scale application.
I would be very interested in any fix if you can think of anything to try.
I posted a bug project here:
https://github.com/jonmdev/bandit_bug
In Application.ex you can comment in or out the Bandit or Cowboy version:
        #httpServer = {Plug.Cowboy, plug: MyPlug, scheme: :http, port: 4000 }
        httpServer = {Bandit, plug: MyPlug, scheme: :http, port: 4000 }
        #webSocket = {Plug.Cowboy, plug: Router, scheme: :http, port: 4001}
        webSocket = {Bandit, plug: Router, scheme: :http, port: 4001}

Seems to work smoothly and no errors for either with Cowboy. While Bandit recurrently (though unpredictably) provides the two errors in this thread.
Thanks for any help or thoughts.

You may also be experiencing the same problem as https://github.com/mtrudel/bandit/issues/407, insofar as you may not be rebinding conn everywhere. This will fail silently in Cowboy but produce errors similar to what you're seeing

You may also be experiencing the same problem as #407, insofar as you may not be rebinding conn everywhere. This will fail silently in Cowboy but produce errors similar to what you're seeing

Can you explain more what you mean by this? I have reviewed the linked threads including:
https://github.com/mtrudel/bandit/issues/361
However, I do not know what you mean when you say "rebind conn" or "everywhere".
I posted a bug project here:
https://github.com/jonmdev/bandit_bug
The entire project is just these two files which are based on the documentation code:

https://github.com/jonmdev/bandit_bug/blob/main/lib/http_server.ex
https://github.com/jonmdev/bandit_bug/blob/main/lib/web_socket.ex

This is directly copied as you can see from the official documentation:

https://hexdocs.pm/plug/readme.html#hello-world-websockets

Is there a problem with the documentation code or my copy/paste from it? Can you explain what? Thanks for any further clarification.

Thanks for the repro - it's not the same issue at #407, so I guess it stands.

I'm unable to reproduce this on macOS or Linux, with a variety of clients. Given how transparently simple and correct your example code is, if this was a general bug we'd be hearing about it constantly.
I'm going to close this until / unless we can get a more reproducible repro case


Create a Icon class for jewel that contains both path and the class.

We should also generate a Jewel icon object starting from known icons from the platform and icons put inside src/main/resources/icons/**.svg/png. We can use a Gradle Plugin inside our repo and a gradle plugin to be used by our users.

Duplicate of #73


RFC: separate initial and refresh auth

Summary
In the auth-exchange, we currently have one function - getAuth that handles fetching both initial auth as well as refreshing the auth in case of errors.
This proposal is to separate this function into two separate functions: one to handle the initial auth only, and the other to handle all subsequent calls.
These should be separated because currently users will have to keep track of their own auth state to know when to fetch the token from storage and when to trigger the refresh flow, and it's easy to make mistakes.
Furthermore, "get initial auth" and "refresh auth" are very different actions from a library user's point of view andit would reduce the mental load to keep them separated.

Proposed Solution
Separate the getAuth into two functions:

getInitialAuth: called only once when the exchange first loads. It should be used to get any initial auth state (e.g. form a cookie or async storage)
refreshAuth: called when willAuthError returns true, or when a request fails with an auth error (as determined by didAuthError)

Requirements

The proposed changes can be seen in this PR: https://github.com/urql-graphql/urql/pull/2814

I am a fan of separating these out, I do have some thoughts about the naming in general as I was thinking of it in the sense of setup and on(Auth)Error as the refresh only gets called when will- or didAuthError returns true. I do recognise that onError might sound a bit severe 😅

I'd also like an option to getAuth() for every request. I previously did this by setting willAuthError: () => true, but that ends up sending double requests every time.
Firebase Auth doesn't expose the token expiration time, so there's no reliable way to know if auth will error ahead of time. However, it does expose a getToken() function which returns the existing token. I'd ideally like to call this prior to each request. However, I've had to revert back to didAuthError() and just intercept a custom error response.

Thanks for the feedback @nandorojo that's interesting. willAuthError: () => true would be the correct way to achieve calling getAuth() every time and it shouldn't be sending double requests. I'll have a look into it.

I see that this PR was meant to address just that: https://github.com/urql-graphql/urql/pull/1849
However, I've found that removing willAuthError is the only way around the double requests for me.

@nandorojo: I'm not quite sure this is subject to this issue, so if #1825 is an issue for you and you have a reproduction, I'll have to ask you to create a new issue please 😅

I gave this RFC some thought, and I think I also have a proposal for an alternative API. Essentially, I think the issue is that a "setup phase" and a "refresh phase" is hard to differentiate, but we're entering territory here of adding a looooot of API 'hooks' for these different checks and phases, and I'd love to reduce them.
I think, we can potentially make the setup phase very explicit if we had a single entrypoint into the setup phase. I've also seen some confusion from people around the authState, since it's only local, but the timing aspect in it isn't actually as confusing to people than its mere naming.
So, I'd love to submit a second solution to this problem:
// the first function is called on initial setup i.e. when the first request is made
// I'd pass utilities here, and potentially make a helper around `fetch` based on the client's options
authExchange(async ({ mutate }) => {
  // we're free here to initialise state as local state
  // or make extra requests already
  let token = localStorage.getItem('token');
  let refreshToken = localStorage.getItem('refreshToken');

  // we then return the config for future requests
  return {
    // this now takes no arguments but the operation
    willAuthError: (operation) => !!token && !!refreshState,

    // both of these only take a single argument, since type hints are sufficient
    // other arguments don't seem needed anymore
    didAuthError: (result) =>
      result.error.graphQLErrors.some(e => e.extensions?.code === 'FORBIDDEN'),

    // this now becomes a simple mapping function as well
    addAuthToOperation(operation) {
      if (!token) {
        return operation;
      }

      const fetchOptions =
        typeof operation.context.fetchOptions === 'function'
          ? operation.context.fetchOptions()
          : operation.context.fetchOptions || {};

      return makeOperation(operation.kind, operation, {
        ...operation.context,
        fetchOptions: {
          ...fetchOptions,
          headers: {
            ...fetchOptions.headers,
            Authorization: authState.token,
          },
        },
      });
    },

    // this now takes essentially no arguments at all and can return a Promise
    // because only some functions allow (in their type and impl.) to return a
    // promise, we automatically guide the user towards this function
    async refreshAuth() {
      const result = await mutate(refreshMutation, { refreshToken });

      if (result.data?.refreshLogin) {
        localStorage.setItem('token', token = result.data.refreshLogin.token);
        localStorage.setItem('refreshToken', refreshToken = result.data.refreshLogin.refreshToken);
      }

      localStorage.clear();
      logout();
    },
  };
}),

The idea here is essentially:
authExchange(async (utilities) => {
  return {
    willAuthError: (operation) => false,
    didAuthError: (result) => false,
    addAuthToOperation: (operation) => operation,
    refreshAuth: async () => {/*...*/},
  };
}),

This allows users to:

Rely on local state variables
Imply that calling out to other async functions in refreshAuth or the setup call is fine
Allow conditional returns and local functions to be more easily reusable
Make it less awkward to share state with "external auth sources"

@kitten huge fan of getting rid of the authState - even in my own real world experience, it doesn't actually get used to store the JWT. It only gets used to determine whether getAuth needs to fetch the auth state from storage or not, but if we separate the getAuth to getInitialAuth and refreshAuth, this requirement goes away completely.
As for initialising the auth outside of the exchange. I'd argue that a lot of engineers would read this api as "localStorage.getItem('token') gets called every time the a request comes in" even though that's not the case. But if we added code examples with variable names that make this more explicit, that should be okay.

@kadikraman I also had to look a bit hard for this, but we planned a while ago to add a mapExchange in core that is called on every request and can return a promise or a non-promise mapped operation. That was meant as a replacement for fetchOptions and simple cases.
But I think we never got around to that. It would cover other cases though.
To show the overarching theme here, in general, I'd argue that we have some patterns where exchanges call a function repeatedly per operation, and other cases where we don't (Graphcache has a single config as well), so I'd say, we could even present all code samples as:
authExchange(async function setup() {
})

and it'd be clear what the intention here is 🤔
In general, we'll also need to work on our TSDoc comments

I've spent a few months now heavily using the auth exchange and I'd love to see the refreshAuth / getInitialAuth as 2 functions, as it clearly explains the purpose of each one.
I think that having the setup() could work, but IMO it's less clear on the intention than having a getInitialAuth which is very explicit for developers using the exchange, and the change would be more simple to adopt for the old exchange users, as it's only splitting the old getAuth into 2 hooks refreshAuth / getInitialAuth (using a the new exchange would require minimal effort in existing codebase)

To be a little more precise, the intention of the setup function isn't to make migration easier. And that's a more or less important distinction for me, because I'm always very hesitant to optimise API design for few known cases. If we were to optimise this to either match up more with the old design, and put more weight on what feels more similar/easier for people who have seen (and gotten familiar with) the old API, then we risk accepting a subpar option 😅
The reason why I've basically found the setup elegant is that it gets around a problem that the authExchange already had in my opinion: API surface bloat.
We're talking about an exchange that could then potentially have the following callbacks:

an initialiser (called once; async)
a refresh (called repeatedly on error)
will auth error (a pre-request, non-async predicate)
did auth error (a post-request, non-async predicate)
add auth to operation (a transform function)

And preserving all else, the concept of authState and having multiple object-parameters passed to all of these functions.
I've personally struggled in the past to explain people three concepts here:

the auth state (and syncing this to external singleton state stores)
the timing of getAuth/refresh
why it's important to have will/did auth and have this option
what happens to operation after or during any of these processes

So, I've taken the opportunity to propose the setup because it particularly helps to reduce this, by:

removing the concept of auth state altogether
mentally and obviously segregating the initialisation (since it has no name and is just a function returning the config, there's a lot of clarity in the timing and what happens; i.e. it's not a lifecycle step people have to think about)
simplifying parameters in the process (because we'll have one place to pass utilities and all other predicates and the transform function only take and return one object from here on)

So, the proposal is a subtle distinction (after all, current getAuth functions can still be refactored in little time to a setup/refresh pattern or init/refresh pattern; I don't expect this to take more than 15min)
But, it does, at least in my view, reduce the amount of hidden concepts we've introduced here in the past

This examples would greatly simplify the flow!
When using cookies for token storage, you don't even need initial setup or addAuthToOperation, just didAuthError and refreshAuth. Current auth implementation is confusing to use in this case, but still works.
Love both proposals, @kitten more. Hopefully one of them will be added not so far in future ❤

@kadikraman: I finally got around to implementing the initializer-based proposal. I know it was a little hard to visualize just from my notes, but I hope the PR will help, and if we can get this signed off on by all interested parties and get this ready, awesome! If not, happy to take more time or circle back on different solutions :v:
cc @JoviDeCroock


Added  Meme Generator

Hey , i  have added  the meme generator . can u please check and let me know if nay further changes u required .
Thanks

Thanks for contributing in my repo . Please  add  the content   in the readme  .
Thanks  :smile:

Looks Great to me . Looking forward to see more contribution from your side .


Linux version?

Like others I'm really interesting by a linux version. Is there any chance that you integrate a linux version in future releases?

Its written for a unix-like (apple bsd) and uses QT for the interface, i havent personally tested this yet but in theory it should just compile and run under linux as the network stack isnt any different and qt has x11 support


update Rails to 5.2.6

This PR will fix https://github.com/antleaf/nims-mdr-development/issues/465 .

This PR was superseded by https://github.com/nims-dpfc/nims-hyrax/pull/406 .


Australasia Reward Points

Hi, please create an issue so we can make an icon that respects the guidelines.


Fix broken installation of NLopt

Tested locally and worked as expected.

Great! Well done for figuring this out @chuan-wang! :)

:) Hopefully it will work!

Build is passing! 🎉

Yeah! Seems like the old version of bowtie2 caused the problem.


OSX browser package has around 4 crust config files with various names in multiple locations

The following is extracted from a discussion in our forum: https://forum.safedev.org/t/trying-to-build-a-global-network/2266/19:
"SAFE Browser release packages needs better resource bundling. Currently browser package has around 4 crust config files with various names in multiple locations thereby needing other means to identify the valid config file. Should really be pruning dev config files/… before release packaging."
I just checked v0.11.0 linux package and I can see only two, still, the one at resources folder shoouldn't be needed in the package
$ find . -type f -iname "*crust.config"
./safe-browser-v0.11.0-linux-x64/safe-browser.crust.config
./safe-browser-v0.11.0-linux-x64/resources/SAFE Browser.crust.config

In OSX, the latest release non-dev has a few more:
./SAFE Browser.app/Contents/safe-browser.crust.config
./SAFE Browser.app/Contents/Resources/SAFE Browser.crust.config
./SAFE Browser.app/Contents/SAFE Browser.crust.config
./SAFE Browser.app/Contents/Frameworks/SAFE Browser Helper.app/Contents/MacOS/SAFE Browser Helper.routing.config
./SAFE Browser.app/Contents/Frameworks/SAFE Browser Helper.app/Contents/MacOS/SAFE Browser Helper.crust.config

This is sorted on master.
QA: in a folder with the release on macOs run find . -name "*crust*"

Tested by building package on OSX (VM), the release folder contains only one crust.config file at this location ./SAFE Browser.app/Contents/Frameworks/SAFE Browser Helper.app/Contents/MacOS/SAFE Browser Helper.crust.config.

QA verified


Where can I find the dataset?

I clicked the link for the dataset but it says the site is under construction. Is there another way in which I can get the dataset?

Please download the dataset from here.

thank you

Hello author, the testfg folder is missing in this dataset. Can you provide a complete dataset? I ran to 20000iter and reported an error that the validation set could not be found.

@liuxingyu123 the testfg folder can be found on kaggle under the ground truth folder->test folder through this linkhttps://www.kaggle.com/datasets/balraj98/berkeley-segmentation-dataset-500-bsds500?resource=download.


@proxy object support in strong mode

in strong mode, we need to mark these as dynamic calls. This is not currently happening.

https://codereview.chromium.org/2223343002/


Access resources only as resource owner?

So let's say I just want to get the character names of an anime and that there is no need at all to involve any clients in this, I just want to get the character names, as the API key holder.
So I see you guys in the docs say that if I need to do so just use "Authorization: Bearer access_token". But in order to get this access token I need to do post to "auth/access_token" every hour for the access token. Why do I need to do this if I'm only accessing as the resource owner, is there not any way for me to just obtain the character names by using the Client ID and Client Secret?
If no other way, would I have to to obtain a new access token every hour..? Could I just get the access token every time I need to do a request because I wouldn't be sure how to create a system that get's a new access_token every hour without having to put checks of time in the PHP scripts.

Your PHP script is a client so it needs to use its' client id and client secret to get the access token (the API key, if you will) to access API endpoints.
The access token idea is not new to AniList but is a standard of OAuth, so if you want to know more you can research into that. There's some good reasons supporting access tokens here
You do not need to obtain a new access token as soon as the old one expires, nor do you do you need to wait for your current access token to expire before requesting a new one.
Ideally you'd want to store the access_token and expires values on the server (file_put_contents / file_get_contents, local database, etc) then start out each script like this:
if ($expires < time()) {
  $access_token = get_new_token();
} else {
  $access_token = get_current_token();
}

@Kazakuri I thought I might have to do that, it just seemed really inconvenient to have to have code that checks the expiration of the access token every time I use it.

Kazakuri has pretty much said everything that needs to be said so I'll close this.
I'm afraid it is all standard oAuth and I have no plans on changing it. I don't really see the expiry check as all that big of an inconvenience, and you could alternatively run an hourly cron-job if you don't wan't to worry about it on every request. There are also plenty of oAuth client packages out there that should handle all of the oAuth stuff for you.


How to open two pages?

browser = playwright.chromium.launch()
context = browser.new_context()
page0 = context.new_page()
page = context.new_page()
How to open two pages using pytest?
Thank you！

The pytest plugin provides you a default page. A second or additional pages you can create based on the context fixture. Example:
def test_foobar(page, context):
    page2 = context.new_page()
    page.goto("https://example.com")
    page2.goto("https://github.com")

The pytest plugin provides you a default page. A second or additional pages you can create based on the context fixture. Example:
def test_foobar(page, context):
    page2 = context.new_page()
    page.goto("https://example.com")
    page2.goto("https://github.com")

thank you！！！

thank you！！！

It would be better to have docs about playwright pytest plugin on the docs
website https://playwright.dev/docs/test-runners?_highlight=test with some
examples @mxschmitt
On Sun, Jan 31, 2021, 19:31 zhonghai9967<EMAIL_ADDRESS>wrote:

thank you！！！
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/playwright-pytest/issues/40#issuecomment-770386959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGYUZUJCAJQB5RDP5OB5F3S4VPDRANCNFSM4WYBAWQQ
.

It would be better to have docs about playwright pytest plugin on the docs
website https://playwright.dev/docs/test-runners?_highlight=test with some
examples @mxschmitt
On Sun, Jan 31, 2021, 19:31 zhonghai9967<EMAIL_ADDRESS>wrote:

thank you！！！
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/playwright-pytest/issues/40#issuecomment-770386959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGYUZUJCAJQB5RDP5OB5F3S4VPDRANCNFSM4WYBAWQQ
.

The pytest plugin provides you a default page. A second or additional pages you can create based on the context fixture. Example:
def test_foobar(page, context):
    page2 = context.new_page()
    page.goto("https://example.com")
    page2.goto("https://github.com")

Hey, I don't think this strategy works? I just tried that, and both pages had the same cookies...it didn't create different contexts. How would I create 2 pages, compliantly through pytest playwright fixtures, but, do it with different contexts/cookies?


Duplicate plugin name with OnlinePlayer class message

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28004957-duplicate-plugin-name-with-onlineplayer-class-message?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github).

Resolved locally.


Make optional use consistent throughout the API

Make Optional usage consistent (frequencyCap/duration is not Optional yet clearly should be)

Deciding to not push Optional and instead we eventually want to switch to object parameters.


./install_kaldi.sh fails with wave-reader.cc:254:10: error: call to 'abs' is ambiguous

Hello I am trying to install pykaldi on macOS 10.15.3 (Catalina) and it has been a rather frustrating uphill battle the whole way through. I was able to get through ./install_clif.sh by adding macOS.sdk/usr/include path to the build.ninja with the help of this suggestion  (i needed to -isystem commandlinttool's sdk usr/include libraries) but then I ran into the following issue while running ./install_kaldi.sh:
g++ -std=c++11 -I.. -I/Users/kankawabata/Documents/auralanalytics/analytics_engine/pykaldi/tools/kaldi/tools/openfst/include -O1  -Wall -Wno-sign-compare -Wno-unused-local-typedefs -Wno-deprecated-declarations -Winit-self -DKALDI_DOUBLEPRECISION=0 -DHAVE_EXECINFO_H=1 -DHAVE_CXXABI_H -DHAVE_CLAPACK -msse -msse2 -pthread -g  -fPIC -Wno-mismatched-tags -isystem /Library/Developer/CommandLineTools/usr/include/c++/v1 -isystem /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/   -c -o wave-reader.o wave-reader.cc

wave-reader.cc:254:10: error: call to 'abs' is ambiguous
      && std::abs(static_cast<int64>(riff_chunk_read) +
         ^~~~~~~~
/usr/local/include/stdlib.h:132:6: note: candidate function
int      abs(int) __pure2;
         ^
/Library/Developer/CommandLineTools/usr/include/c++/v1/math.h:765:1: note: candidate function
abs(double __lcpp_x) _NOEXCEPT {return ::fabs(__lcpp_x);}
^
/Library/Developer/CommandLineTools/usr/include/c++/v1/math.h:769:1: note: candidate function
abs(long double __lcpp_x) _NOEXCEPT {return ::fabsl(__lcpp_x);}
^
1 error generated.

Here is the corresponding line in the kaldi source code. the error went away when I monkey patching the line from:
std::abs(static_cast<int64>(riff_chunk_read) +
                  static_cast<int64>(data_chunk_size) -
                  static_cast<int64>(riff_chunk_size)) > 1)

to:
std::abs(**(int)**(static_cast<int64>(riff_chunk_read) +
                  static_cast<int64>(data_chunk_size) -
                  static_cast<int64>(riff_chunk_size))) > 1)

But I don't have much experience with c++ and I'm not sure if this actually solves the underlining issue on why it failed. I would appreciate on some feedback as to why it's failing.
Thank you.

What is really weird here is std::abs(int64_t) or something like it is not listed among candidate function for overload resolution. The compiler considers abs(int), abs(double) and abs(long double). You mention you had to add an include path to the compilation, and this is what may have caused this. I know very little about Macs to be of help, but this is a clue.

Turned out this error was because of -I/usr/local/include messing up the header file search order. A while back I used this solution here to fix missing header files for macOS Catalina which probably messed up the #include_next commands in the header files.
Removing the files in /usr/local/include seems to have worked and seems safe to do so


odd check in yii\filters\AccessRule::matchRole

in yii\filters\AccessRule::matchRole
if (!isset($roleParams)) {
    $roleParams = $this->roleParams instanceof Closure ? call_user_func($this->roleParams, $this) : $this->roleParams;
}

!isset($roleParams) will is always true

Q
A

Yii version
<IP_ADDRESS>

This check is inside of foreach - it will be true only for first iteration which reach this statement, but finally it may happen multiple times.
https://github.com/yiisoft/yii2/blob/292a545642eaa947a5169d7c6366a6dca486d571/framework/filters/AccessRule.php#L231-L248