[Question] geolocation example is not working

Hi, I'm testing this example but the result is the default values (see image)

I used my local environment and the option to deploy using vercel.com (https://geolocation-beta.vercel.app/)

Is there any configuration required to make it work?

I have the same issue, too. I followed the example and deployed it to Vercel, and I found req.geo.country was undefined.

As i understand the geo object is available only for paid plans on Vercel

@danestves WOW, I can confirm the example works correctly after I transferred it to my team project. Thank you.

Duplicate of https://github.com/vercel/examples/issues/24.
We're making geo available for all plans this week, sorry about this! Was not intended.

I have used request.geo on vercel and it is still empty and undefined. Although the header["x-vercel-ip-country"] is populated, but the objects (ip and geo) are still empty on vercel.


Allow 1/0 for True/False in inputs.boolean?

When creating a URL query parameter string in Objective C from a dictionary:
@{@"not_deleted": @YES}

The result looks like:
not_deleted=1

Which will raise ValueError when parsed by Flask-RESTful. Should the inputs.boolean parser be extended to accept 1 as True and 0 as False?
Perhaps additionally "yes" and "no"?

This is my code:
class UserListResource(Resource):
    request_class = user_parser

    def get(self):
        return {'list': USERS}, 200

    def post(self):
        args = self.request_class.parse_args()
        new_id = max(USERS.keys()) + 1
        USERS[new_id] = args
        print args
        return args, 200

I submit the data:
{
    "username": "Tony",
    "gender": 1,
    "email"<EMAIL_ADDRESS>}

This is my response fields:
user_fields = {
    'username': fields.String
}

I print the parse_args() and observe it:
{'username': 'Tony', 'gender': AttributeError("'int' object has no attribute 'lo
wer'",), 'email'<EMAIL_ADDRESS>
Finally, it will not raise any error but it returns the below:
TypeError

TypeError: AttributeError("'int' object has no attribute 'lower'",) is not JSON serializable

What happen? Is it bug?

ref the comments:
https://github.com/flask-restful/flask-restful/pull/341/files#diff-3dfb63fe71f79c86d5dbf637569d7374R203

I see. Oh my god, it really doesn't handle int 0/1.


Rollback Fitbit sensor client to previous version

Description:
Client update broke the component when refreshing tokens. Rollback client to previous version until permanent fix to sensor component can be made.
Related issue (if applicable): fixes #9152
Pull request in home-assistant.github.io with documentation (if applicable): N/A
Example entry for configuration.yaml (if applicable):
N/A

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Instead of rolling it back, I've sent the PR https://github.com/home-assistant/home-assistant/pull/9183 with a fix. Please check it out.

I agree with @tchellomello that a fix would be the way to go.

Agreed, thanks for the assist on this one!


By pass cache per request.

Hey,
We (our team) have a need to force refresh a request based on user input, (they pull scroll a view).
I'm purposing adding a variable to the request config.
ignoreCache: <true|false> or byPassCache <true|false> or forceRequest: <true|false>
I envision this working as.

skip getting items from the cache by pretending it was a cache miss.
request fresh data from the server
insert new data from the server.

My preference is either ignoreCache or forceRequest because the cache is ignored but still utilized.
let me know what you think.

Hey @jahead !
This is interesting, my only concern is that it could actually be achieved using the new invalidate method or even by manually removing the cache from the store with axiosInstance.cache.removeItem('key').
Adding yet another option seems like a bit of a burden. 🤔
Nonetheless, I can also understand that having this easily accessible option instead of setting up a complicated invalidate method is quite convenient 😄

So the reasoning here for not removing the cache is that we still want to use the cache for the new item for future loads. We just want to conditionally break through the cache.
We considered overriding the invalidate function but we weren't sure creating more dependency on the sequence of when invalidate function is run was a good thing.
Yea the number of options is quickly growing. Just a means to break through the cache is a common one. I'm not sure on how one could fix this growing options problem.
Maybe giving users a onError function. Let them decide what to do with the error. shrugs

From: Carl Ogren<EMAIL_ADDRESS>Sent: Friday, February 8, 2019 10:02:53 PM
To: RasCarlito/axios-cache-adapter
Cc: jahead; Mention
Subject: Re: [RasCarlito/axios-cache-adapter] By pass cache per request. (#68)
Hey @jaheadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjahead&data=02|01||5f086b42b66f42f0c64708d68db4f99a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636852205746783284&sdata=rubpMyZKoDQz7bXIo94B5y9SYmj4RU7qIlbNRvKbu4g%3D&reserved=0 !
This is interesting, my only concern is that it could actually be achieved using the new invalidate method or even by manually removing the cache from the store with axiosInstance.cache.removeItem('key').
Adding yet another option seems like a bit of a burden. 🤔
Nonetheless, I can also understand that having this easily accessible option instead of setting up a complicated invalidate method is quite convenient 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRasCarlito%2Faxios-cache-adapter%2Fissues%2F68%23issuecomment-461767558&data=02|01||5f086b42b66f42f0c64708d68db4f99a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636852205746793295&sdata=IUwNlGsIjar2vIE9N6yfqXIN0HHFRxpI%2BBNES7I6jsc%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE1egAqu2MPiGybYB6_euRdJ5NiHamP9ks5vLVldgaJpZM4amRg4&data=02|01||5f086b42b66f42f0c64708d68db4f99a|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636852205746803288&sdata=r0oVKkJEH02SbIT3KVhMLyOFS3ly1XvlN0vtaKGyJn8%3D&reserved=0.

Ok sounds reasonable 😄
I'll see how to minimize the number of parameters in v3.


Updated cloudant-http to 2.6.2

What
Upgraded cloudant-http to version 2.6.2 to consume upstream cookie handling fixes.
Testing
No new tests, existing tests pass.
Issues
Fixes #358

+1


Throttle the ledger rollover

Is your feature request related to a problem? Please describe.
High ledger rollover frequency may lead to zookeeper in heavy workload. In order to make zookeeper run more stable, we should limit the ledger rollover frequency.
Describe the solution you'd like
Add a rate limiter for the ledger rollover.

@codelipenghui HI, I'll be happy to participate.
Please check this more detailed solution.
Add a new parameter managedLedgerRolloverRateLimit, rate limit the amount of rollover per second in one broker. This parameter can be set as 0 by default meaning no rate limiting .
As ManagedLedgerImpl#currentLedgerIsFull is called before each ledger rollover, we can add a global rate limiter after that. Every ledger rollover can only happen when currentLedgerIsFull() and rolloverRatelimiter.tryAcquire() both return true.


Update Home Page Content

Remove duplicate links

[x] contact
[x] faq

Remove Search icon until further notice
add Top Community Projects

[x] leblad project

Update How To Contribute to dzCode.io
add Top Community Articles

[x] How To Add Your Own Article
[x] Send emails with Django and Gmail, a better way

Hi, for the search icon and redundant links i'm already on that ! :)

implemented #214


Create shivam.html

this is the created the second branch creating the for the open the pl request

this is the comment


GROOVY-8002: propagate source position to method in chain assignment

Is it better to set the position of method nodes in the parsers?

This is a common pattern, the original parsed AST is transformed and the new nodes need to have source ranges transferred.  In this case PoppingMethodCallExpression trn = new PoppingMethodCallExpression(receiver.transformExpression(transformer), setter, (TemporaryVariableExpression) tmp.transformExpression(transformer)); creates new nodes.  The entire expression does not have source position set.  In addition, the object expression and method expression may be new and lack source position.

Agreed.
Merged. Thanks!


Request: "cleaner" video player

Request: "cleaner" video player

You can use this user-script: Twitter native video player


Created new class. Searching in coder starts Spotter. Error when try to access the new class.

What I did:

Created new class in playground (example from Pharo by example):

Looked for a way, to add the methods. So I decided to open Coder from the GT main menu.
Then I looked for the new class, which opened Spotter.

when pressing the 2nd button from left, I get this error:

4. When pressing the hook, nothing happens.
So, how can I add methods to my class?

ok, When I use the tree browser in Coder, I can open the class, and add methods. But it is miles to scroll, as the package does not start with "A" or so ;)

Another note: The Spotter called from coder is not resizeable or scrollable. Difficult to navigate in it, if you are in a smaller window. -> another ticket?

Searching for a class works now.

For the resizable Spotter, we now have #1046.


Modify Prism nightowl theme to improve YAML syntax highlighting

Description & motivation
As requested in #44 , YAML highlighting currently does not differentiate between keys and values.
The docs site relies on Docusaurs, which uses Prism for syntax highlighting.
I tried a few of the different Prism themes, but most of them also had the same colors for YAML keys and values. Also, I thought changing the theme may be too large of a change for this issue.
So, the PR still imports the same nightOwl theme that was used before, but makes a small change to account for YAML keys. I chose a color that looked decent from the nightOwl theme palette
Before this PR on seed config page

With this PR

Would be helpful to have the following input before merging

[ ] Is the color chosen for YAML keys appropriate?
[ ] Is the Javascript written idiomatic? (I'm rusty in modern Javascript)
[ ] Is this approach sustainable? i.e. modifying the an existing Prism theme inline this way

I'm inclined to say that having up to two total inline modifications is fine, but beyond that, it's probably a better idea to have a separate theme file within the repo

@Mingweigu this is so cool!!
One thing to note is that we've modified the code rendered to support links in code blocks. My only real suggestion here would be to use a different color that helps users better differentiate between links and regular YAML keys. Maybe something like rgb(255, 203, 139) would work?
Tagging @clrcrl for her thoughts too :)

Hey @Mingweigu — thank you so much for this PR! What a joy to see this on a Monday morning 💥
I agree with Drew — it would be cool to have a different key color.

Is the Javascript written idiomatic? (I'm rusty in modern Javascript)
Is this approach sustainable? i.e. modifying the an existing Prism theme inline this way

@drewbanin — those are both for you!
(Also do you know why the build failed?)

@clrcrl yeah the JS looks good to me! I think we should merge this when we're happy with the color styles. I just rebuilt the PR - let's see if that helps

Just changed the color to rgb(255, 203, 139) and updated the screenshot.
How does that color look @clrcrl @drewbanin ?

I like it! After looking at this, I think we might want to take another pass at styling the links -- maybe we'd be better served by something more subtle like an underline?


The ad bar hides horisontal scrollbar

Without an adblocker:

With an adblocker:

Seconded. The ads themselves are fine, but obscuring page content/controls with them is frustrating.


Add an option to announce extra routes

Hi,
I'm wondering if you would like to add an extra configuration option where one can define the routes to announce. I am not fancy of the "auto-discovery" function built in the addon and I'd like to override it when needed.
Something in the area of:

when no option is set, autodiscover the local LAN where HA is running
when the option is set to "disabled" (whatever way), do not announce any route
when option is set, do not autodiscover and instead announce the routes that are set in the option

If you prefer I can open an issue in the official addon repo.

This is how it looks like in Tsujamin's add-on (https://github.com/tsujamin/hass-addons) that I use on the distant node:

I'm looking at the code of your addon and although I have found the auto-discovery function in usr/bin/subnet-routes I fail to see where you add the advertised routes to the executable.
Something like (From Tsujamin's code)
if bashio::config.has_value 'advertise_routes'; then
    TAILSCALE_FLAGS+=('-advertise-routes' "$(bashio::config 'advertise_routes')")
fi

that could be added for the 3rd bullet above

Yes, this is the exact same situation/solution that happened to samba add-on, to run not only on the HA Supervisor managed interfaces.
HA team insists on not dealing with networks that are not managed by HA Supervisor. Only interfaces listed by ha network info can be used (eg. eth0, wlan0).
It didn't get merged, I maintain a samba add-on fork basically only for this extra option, but at least I use this extra option.
I plan to finish maintaining this tailscale fork after the official add-on is released, so I prefer not to add config options that has 0% chance to be merged.
Though I leave this issue open now. After you manage to make your networking setup working without advertise_routes and snat_subnet_routes options available, and only these are what required, I maybe add them.

Ok, I see the point here, can't blame the HA team. I'll rethink my setup see how bad I need this.
Thanks!


Fix cupsoda simulator error on Tyson model

CupSodaSimulator was erroring on the models with rates consisting
of a single Parameter or Expression. Calling .args on these returns
a string, not a pysb.Component object, which fails when we try to
multiply together to get the rate. This commit introduces a check
and workaround in the CupSodaSimulator.
I've also removed the pycuda.autoinit import, which triggers GPU
initialization when pysb.simulator is imported, even if a
CupSodaSimulator is not instantiated. Instead, pycuda is initialized
on demand within the n_blocks function.

Coverage decreased (-0.03%) to 76.555% when pulling 01d76747a25066d4cc5e5c9557427d42b76d68f1 on alubbock:cupsodafix into bf80cc8633b7ee6ca2cf1309d371720279520da7 on pysb:master.


Refactor S3 Testutils (GSI-762)

Bumps version from 3.2.2 to 3.2.3 because the changed items were not in use by any other microservices and are low-impact. Strictly following SemVer would mean bumping the major version number to 4 because the delete_created_buckets method was deleted, and some other functions were moved into the s3 fixture instead of existing on their own (and another was deleted).
The fixtures were moved into s3/testutils/_fixures.py, the supporting functions moved into s3/testutils/_utils.py, and typical_workflow was moved into its own module in that same directory. The imports are all funneled into __init__.py to avoid disrupting consuming services.
A list of current buckets is retrieved using the object storage's s3 client. The bucket deletion process iterates through the buckets and deletes them one by one, so neither the object storage nor test fixture need to track which buckets have been created.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

219 of 226 (96.9%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 91.936%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hexkit/providers/s3/testutils/_fixtures.py
106
108
98.15%

src/hexkit/providers/s3/testutils/_utils.py
69
74
93.24%

Files with Coverage Reduction
New Missed Lines
%

src/hexkit/providers/s3/provider.py
1
79.42%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1733

Relevant Lines:
1885

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

217 of 224 (96.88%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 91.928%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hexkit/providers/s3/testutils/_fixtures.py
104
106
98.11%

src/hexkit/providers/s3/testutils/_utils.py
69
74
93.24%

Files with Coverage Reduction
New Missed Lines
%

src/hexkit/providers/s3/provider.py
1
79.42%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1731

Relevant Lines:
1883

💛  - Coveralls


Apply new mechanism to HbaseInterpreter

What is this PR for?
This PR applies the new interpreter registration mechanism to Hbase interpreter.
What type of PR is it?
(minor) Improvement
Todos

[ ] - Task

What is the Jira issue?
[ZEPPELIN-913]
How should this be tested?

apply this patch
build source
bin/zeppelin-daemon.sh start
5.Run a paragraph with hbase commands

Screenshots (if appropriate)
Questions:

Does the licenses files need update? => no
Is there breaking changes for older versions? => well it's not compatible with older static interpreter declaration
Does this needs documentation? => no

Just realized I kept the old static declaration in comments... Not really clean and should probably be removed.

LGTM

LGTM

@ggdupont Could you tell me your jira ID? I'll update you as an assignee

@jongyoul simply gdupont


ARROW-1310: [JAVA] revert changes made in ARROW-886

@elahrvivaz , @StevenMPhillips
Reverting the changes made for ARROW-886 -- https://github.com/apache/arrow/pull/591
(1) Don't explicitly reallocate the offsetVector in realloc() function of Variable Length Vectors.   If we call setSafe() on variable length vector, it will internally invoke setSafe() on the corresponding offsetVector and the latter function can decide whether to reallocate the offsetVector or not.
(2) Doing (1) will break the unit test added as part of PR 591 so we need to remove that as well.

Is there any way to assert the correct behavior in a unit test?

Looks like we have realloc related unit tests in TestVectorRealloc.java. But as a follow-up, I am planning to revisit realloc related tests and add to/enhance them.

OK, no problem


align develop with latest master changes (Master -> develop)

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.2-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: prepare release 2023-05-09

Summary
This PR has been created using the yarn release script. Once merged, the clients will try to release their new version if their version has changed.
Version Changes

javascript: 5.0.0-alpha.60 -> prerelease (e.g. 5.0.0-alpha.61)
java: 4.0.0-SNAPSHOT -> minor (e.g. 4.0.0-SNAPSHOT)
php: 4.0.0-alpha.58 -> prerelease (e.g. 4.0.0-alpha.59)
go: 4.0.0-alpha.7 -> prerelease (e.g. 4.0.0-alpha.8)
kotlin: 3.0.0-SNAPSHOT -> minor (e.g. 3.0.0-SNAPSHOT)

Skipped Commits
(None)

I didn't understand your comment on the other PR about the bumps 🤔

we are releasing the php client even though there are no changes, and we have messages like add Kotlin API client in the CHANGELOG, which has no business being here

it's because it's only based on the commit message, not the actual changes so if we are too broad on the commit message it will be included in each langages

Oh okay, I understand

lol you can start again this will break everything, we need to add more process when a release is awaiting

@millotp so, should I wait for the current build to finish before launching mine? 😏


Document for Multiple networks

We are having requirement to configure calico with different network example network A and Network B.
some set of pod will get ip from Network A and others get IP from network B also both networks should be isolated.
How to achieve this ?
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Calico version v3.5.6
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes v1.14.3
Operating System and version: Red Hat Enterprise Linux Server release 7.6 (Maipo)
Link to your project (optional):

@jebinjeb Have you taken a look at using ippools for this purpose?
https://docs.projectcalico.org/v3.9/networking/use-specific-ip

This document might help also:
https://docs.projectcalico.org/v3.9/reference/cni-plugin/configuration#using-kubernetes-annotations

Closing due to inactivity, please add a comment if this is still an issue and we'll re-open it.


Added some properties

Added properties:

elapsed_simple (simple progress, even when length is available)
total_time; shows the total time of a track (if applicable)

Added parameters:

time24hour; added a switch to control 24hr formatting

Not sure if it's the best and neatest way to do this, but I think you can judge that better than I can. ;)


Add TTF_GetFontScalable to distinguish bitmap vs outline fonts

closes #169
This is the freetype function used for bitmap font detection in pygame's freetype submodule. I've been recently looking at how feasible it is to replace the old pygame.freetype submodule with just our SDL_ttf wrapper pygame.font and one of the snags I noticed is bitmap font detection.
I looked over here and saw the issue so thought I'd have a stab at a PR.
My first SDL_ttf PR so I've almost certainly missed a bunch of things everyone is supposed to do. So, sorry about that.

seems ok to me. this was asked before! @slouken

This seems fine to me. I would probably rename the function TTF_IsFontScalable() to indicate it's returning a boolean value.

Merged, thanks!

Do we want this in the SDL2 branch too?

No, this will be an SDL3 API


Added Jose Maria College Foundation Inc.

Request to the school "Jose Maria College Foundation Inc."

@altair1001 First stage of verification passed, official domain for Jose Maria College Foundation Inc. is jmc.edu.ph as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@altair1001 Regrettably, this request has signs of fraud. I have to close it without merging.
It does not mean that something is wrong with the school. The issue is related to a combination of suspicious factors related to the domain itself.
In particular, the problem is that the website of the school is down. It may be a result of technical failure or another legitimate issue. Thus, you can make a new request to add the domain in future.
As for now, please use an alternative way of receiving free licenses:
Please enroll to GitHub Student Developer Program at https://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).
Thank you for your understanding!


Simplify integration test setup

I think with https://github.com/knq/chromedp and https://chromium.googlesource.com/chromium/src/+/master/headless/README.md we will be able to remove docker and selenium from our integration tests, simplifying things a lot.

https://github.com/knq/chromedp/issues/30 is making this harder – I will look into implementing this using JS.


jsii reserved keywords should be checked on interfaces

Describe the bug
This PR (https://github.com/aws/aws-cdk/pull/20382) failed to be packaged into Java.
Expected Behavior
I'm not completely sure, but I think this bug probably could have been caught by JSII at compile time.
Current Behavior
The TypeScript code misleadingly compiles successfully even though it will not result in valid Java code.
Reproduction Steps
I haven't made a minimal repro yet, but it should be possible to reproduce the error by checking out the branch of this PR and doing a full build and packaging: https://github.com/aws/aws-cdk/pull/20382
Possible Solution
It looks like we keep track of a list of reserved keywords here https://github.com/aws/jsii/blob/2ba2ec47af1b3007844c00bd44aa81ccc7729040/packages/jsii-pacmak/lib/targets/java.ts#L519, and it's used later to determine if a method name or property name is safe in Java. But perhaps it also should be checked against interfaces?
It's also possible this is just user error and there is a warning we just ignored in our build logs somewhere...
Additional Information/Context
No response
SDK version used
jsii v1.59.0
Environment details (OS name and version, etc.)
CodeBuild

Fixed by #3553 (thanks @rix0rrr !)


Feature Request: Apple/Google native sign-in

Apple requires app use native sign-in instead of browser-based OAuth authentication, is there any plan to support this?

Apple requires app use native sign-in instead of browser-based OAuth authentication

I'm not aware of such requirement. As far as I know, the only requirement is to have "Sign-in with Apple", but it doesn't necessarily need to use the native apple sign-in:

Apps that exclusively use a third-party or social login service (such as Facebook Login, Google Sign-In, Sign in with Twitter, Sign In with LinkedIn, Login with Amazon, or WeChat Login) to set up or authenticate the user’s primary account with the app must also offer Sign in with Apple as an equivalent option.

In any case, I'm not planning adding support for native Apple auth. Even the existing Apple OAuth2 integration was very cumbersome to implement as I don't have apple developer account.
Users contribution are welcomed.


Rename ElectronCashSLP.ElectronCashSLPInstaller to SimpleLedger.ElectronCashSLP

Rename ElectronCashSLP.ElectronCashSLPInstaller to SimpleLedger.ElectronCashSLP

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

I changed the folder structure to match the new name. I renamed it to make it more intuitive and because it should be clear that it's an installer. I would appreciate it if someone would help me with this and #15831, which is similar, just with the regular variant of the program.

Addition and deletion only permitted in the same package directory.

Ooop before you start fixing these, the Publisher have to be Electron Cash LLC and the PackageName have to be Electron Cash


Updating HDInisght configuration files

Could these configuration files be updated in the same way as they initially set? For example, if I apply ARM template with modified configuration settings will configuration files be updated?
Is it any way to update these settings via Azure APIs?

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

ID: 096913ec-b990-0da5-4be4-2cf1c91fa347
Version Independent ID: a03a31eb-1087-3ba4-9499-5e81b87948e3
Content: Customize Azure HDInsight cluster configurations using bootstrap
Content Source: articles/hdinsight/hdinsight-hadoop-customize-cluster-bootstrap.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

Thank you for your detailed feedback.  We are looking into the issue, and we will respond when we have more information.

#reassign: @dagiro

@kosinsky Thanks for reaching out with your question. I think that ARM templates are primarily for deploying new resources and not for modifying existing resources.
There may be some limited configuration capabilities through the various APIs, such as the REST API which you could view here: https://docs.microsoft.com/en-us/rest/api/hdinsight/. For all the APIs, check in the Table of Contents under 'Reference'.
For discussion of specific scenarios, you could get some support by emailing the HDInsight discussion alias at<EMAIL_ADDRESS>Thanks again.

Thank you @hrasheed-msft.
I'm seeing https://docs.microsoft.com/en-us/rest/api/hdinsight/configurations/update with "Configures the HTTP settings on the specified cluster. This API is deprecated, please use UpdateGatewaySettings in cluster endpoint instead."
Is it correct to assume that we can't update any configuration changes except gateway via Azure REST API?
My scenario is adding support for external metastore HDInsight clusters created via terraform: https://github.com/terraform-providers/terraform-provider-azurerm/pull/6145 Just wanted to confirm that it will be oneway only.

@kosinsky I dont have detailed knowledge on that issue. If you reach out to the discussion list, you may find someone who has worked on a similar scenario.

#please-close


cc-proxy: Fix systemd socket and service

Some paths provided in the cc-proxy.service and cc-proxy.socket were both relying on legacy path names from cc-oci-runtime. This directly led the cc-proxy service to be not started as expected when the proxy socket was used.

Coverage increased (+0.4%) to 71.053% when pulling bc3fe0a3e1d7cd483e15c3347ee24bac72d59e7e on sboeuf/fix_service into 398fd8b9d8d1e223fc7034ff4d7ab6db131fff30 on master.

Coverage remained the same at 70.614% when pulling 1d02d3ce72b0ea9f45466b868ab61e9f1e7bb9d4 on sboeuf/fix_service into 398fd8b9d8d1e223fc7034ff4d7ab6db131fff30 on master.

I'm absolutely fine with this

Coverage remained the same at 70.614% when pulling 8870d77daad18c133cbd1bdcb7a939137bbf05ce on sboeuf/fix_service into 398fd8b9d8d1e223fc7034ff4d7ab6db131fff30 on master.

@dlespiau Does that sound good now ?

I would squash the two commits together, you're changing the same strings twice in a row! but let's move on :)
lgtm


Fixing the docs headings

Hey @esethna have a look.

Merged, thanks Asaad, Corey fixed the build issue


#18701 Fix editor area node drag-n-drop

Please make sure the fix works on all platforms!!!
Acceptance criteria:

[ ] Dragging resource nodes (scripts, etc) to the desktop copies them
[ ] Dragging database nodes (tables, views, etc) to the editor area opens them
[ ] Dragging any navigator nodes to the text editor inserts their names
[ ] #17415

https://user-images.githubusercontent.com/84867187/231438605-dcadd677-086e-470f-9f90-0b6d3fb60c2f.mp4
everything below the node with connections does not open (bookmarks, scripts etc.)

https://user-images.githubusercontent.com/84867187/231689340-13f119de-cfb7-4898-901a-bcb7c8cc7b8c.mp4
now only scripts can be opened (on mac and windows behavior is the same)
event loop exception -error
org.eclipse.core.runtime.AssertionFailedException: assertion failed:  	at org.eclipse.core.runtime.Assert.isTrue(Assert.java:113) 	at org.eclipse.core.runtime.Assert.isTrue(Assert.java:99) 	at org.eclipse.ui.internal.ide.EditorAreaDropAdapter.asyncDrop(EditorAreaDropAdapter.java:130) 	at org.eclipse.ui.internal.ide.EditorAreaDropAdapter.lambda$0(EditorAreaDropAdapter.java:87) 	at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:40) 	at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:132) 	at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4029) 	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3645) 	at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$5.run(PartRenderingEngine.java:1155) 	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:338) 	at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1046) 	at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:155) 	at org.eclipse.ui.internal.Workbench.lambda$3(Workbench.java:643) 	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:338) 	at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:550) 	at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:171) 	at org.jkiss.dbeaver.ui.app.standalone.DBeaverApplication.start(DBeaverApplication.java:297) 	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:203) 	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:136) 	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104) 	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:402) 	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:255) 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) 	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) 	at java.base/java.lang.reflect.Method.invoke(Unknown Source) 	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:659) 	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:596) 	at org.eclipse.equinox.launcher.Main.run(Main.java:1467)

The issue is that the DND mechanism picks the first appropriate transfer and proceeds with it, ignoring all other potential transfer candidates. Let's imagine the following situation:

We told Eclipse that the navigator could provide these transfers: files and editor inputs
If we drag a database node, it would pick file transfer because it was first, yet the node doesn't represent a resource

Do you see the issue? And we can't do anything about it.
I was thinking about manually constructing FileEditorInputs for all files that represent resources. It might do the job.


Broken Link

The link on the Trial (listing) for the sample app is broken: https://contoso.com/trial/saas-app

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

ID: 4c0ce426-a720-24d2-35e6-6c63f68e47a8
Version Independent ID: e993bc40-5837-3b76-ea5a-fe07002d3c2c
Content: Create a new SaaS offer in the Commercial Marketplace
Content Source: articles/marketplace/partner-center-portal/create-new-saas-offer.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @ChJenk
Microsoft Alias: v-chjen

Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

That URL isn't meant to be a link but is listed as an example for how partners can list their offers to customers and they can include a link to a free trial. As the description reads:
"List your offer to customers with a link to a free trial by providing a valid URL (beginning with http or https), where they can get a trial through one-click authentication by using Azure Active Directory (Azure AD). For example: https://contoso.com/trial/saas-app.  Offer listing free trials are created, managed, and configured by your service and do not have subscriptions managed by Microsoft."

#sign-off
Please see comment above. If there are any suggestions in making the example look less like a link, I'd appreciate it. Thanks!

#please-close


Issue & Update on how to use the library with app router

What version of this package are you using?
tried canary and last official
What operating system, Node.js, and npm version?
20 and 10
What happened?
I am writing this issue after desperately trying to make the library work with the app router since 2 week already. I somehow managed to make everything work locally with the dynamic rendering (static ssr would only build the default locale).
I tried a consequent amount of different configurations, but not a single one worked so far. All example that i could find didn't worked either. Canary version of the library (that need to be implemented with npm i --force) doesn't seems to help either. So i though that i would put here all the configurations that  have here and let's see if someone would have an idea to how debug it.
If i fix my issue i offer myself to build an advanced usage example for this library including multi vendor and dynamic page routing.
1. The configuration that almost worked
1.1 package.json
here i used "next-translate": "2.6.2" and "next-translate-plugin": "2.6.2"
1.2 next.config.js
In this case i used the i18n routing with:
const { domains, defaultLocale, localeDetection, locales } = require("./i18n")

module.exports = nextTranslate({
  // OTHER STUFF HERE
  output: "standalone",
  i18n: {
    locales,
    defaultLocale,
    localeDetection,
    domains
  }
}

i made sure to have all the domains etc coming from the i18n file as it is a nice place to centralize all this information.
1.3 i18n.js
here also i tried with the [lang] parameter and it didn't helped so i ended removing it
const defaultDomains = () => {
  return [
    {
      domain: "www.example.dev",
      defaultLocale: "us"
    },
    {
      domain: "www.example.dev.br",
      defaultLocale: "br"
    }
    // etc
  ]
}

module.exports = {
  locales: ["us", "in", "gb", "es", "fr", "dk", "br", "pt", "mx"],
  defaultLocale: "us",
  localeDetection: false,
  domains: defaultDomains(),
  pages: {
    "*": ["nav", "error"],
    "/": ["home"],
    "/[profession]": ["home", "architect", "partner", "how", "error"],
    "/[profession]/[city]": ["home", "architect", "partner", "how"],
    "/blog": ["blog"],
    "/not-found": ["architect", "partner", "error"],
    // etc
  }
}

1.4 middleware
Make sure to have the middleware inside the src
import { defaultLocale } from "@i18n"
import { NextResponse } from "next/server"

export function middleware(request) {
  const locale = request.nextUrl.locale || defaultLocale
  request.nextUrl.searchParams.set("lang", locale)
  request.nextUrl.href = request.nextUrl.href.replace(`/${locale}`, "")
  return NextResponse.rewrite(request.nextUrl)
}

This is the one i used that was suggested by @aralroca in some of the examples i could find.
1.5 folder architecture
My folder architecture would look that this:

where the layout would make usage of the const { lang } = useTranslation() to get the lang for metadata and html lang.
As you can see there is no [lang] folder in this solution as all my attempts to make this work with a [lang] folder resulted in a fail.
1.6 The issue
this configuration worked perfectly in development but unfortunately in production it only worked partially. If the defaultLocale match the defaultLocale from a specific domain then it work but if it doesn't most of the pages at the root level would not work correctly. The only way to make then work is to add the /br/us that correspond to the /domainDefaultLocal/defaultLocal in the path and it somehow mysteriously work.
i tried to custom the middleware to redirect the pathname to that specific case on some url:
 if (request.nextUrl.pathname === "/") {
    request.nextUrl.pathname = `${pathname}${locale}/${defaultLocale}`
    return NextResponse.rewrite(request.nextUrl)
  }

but then i get an error of

2. Asking for help
I really tried all examples, with different type of  structure with the [lang] wrapper. Different middlewares. The new canary versions. removing manually some config from the next.config.js such as the i18n = 'undefined' and non of all trial could even work a single time in production.
Most of the time i had a 404 in the home page but i couldn't make the translation work in the other pages anyways.
So here i am, a bit lost, asking for a good soul to try to indicate me the path to follow :)

Update: I made it work in production finally... It's a bit flacky but it does the work.
1. Remove the reference to the domains in your i18n.js and next.config.js
This was the main blocker for me. It somehow remove the weird url redirect that was not working correctly.
2. Update the middleware to look like this:
import { Domains } from "some-file-where-you-have-a-list-of-domain-and-their-locals"
import { NextResponse } from "next/server"

export const middleware = async request => {
  const { searchParams } = request.nextUrl

  // GET DEFAULT LOCALE
  const host = request.headers.get("host")
  const { defaultLocale: locale = "us" } = Domains?.find(({ domain }) => domain === host)

  // APPLY LANGUAGE
  searchParams.set("lang", locale)

  return NextResponse.rewrite(request.nextUrl)
}

export const config = {
  matcher: ["/((?!api|_next/image|favicon.ico).*)", "/"]
}

where the Domains object look like
const Domains = [
    {
      domain: "www.example.dev",
      defaultLocale: "us"
    },
    {
      domain: "www.example.dev.br",
      defaultLocale: "br"
    }
]

What i am doing here is to get the host from the header to manage to know what language attribute and from there setup the language with the searchParams.set("lang", locale)
currently in prod right now for

https://www.archsplace.com
https://www.archsplace.es
etc


Support downsampling for /series.

Signed-off-by: Aleksey Sin<EMAIL_ADDRESS>

Use-case
Grafana need to get variables from /series. It do not work when we have only downsampled data (we using other retentions for every resolutions).
Changes
Support downsampling for /series API method. Based on defaultInstantQueryMaxSourceResolution
Verification
Grafana can get data for variables.

This is kind of weird as series does not fetch chunks... I think the fix here is to actually look up all blocks even downsampled for all series? Instead of this?
What if instead put maxSourceResolution = max.Int64`? I think that should include all series... (:
What do you think?

Yes, I think it good idea! PR Updated)

@bwplotka Any comments?)

I have tested this out - works properly and now returns all data. Thanks!


TypeError: Cannot read property 'id' of undefined

expo init
✔ What would you like to name your app? … test-app
✔ Choose a template: › blank                 a minimal app as clean as an empty canvas
✔ Downloaded and extracted project files.
🧶 Using Yarn to install packages. You can pass --npm to use npm instead.
✔ Installed JavaScript dependencies.
✅ Your project is ready!
To run your project, navigate to the directory and run one of the following yarn commands.

cd test-app
yarn start # you can open iOS, Android, or web from here, or run them directly with the commands below.
yarn android
yarn ios
yarn web
cd test-app
eas build --platform android
✖ Creating @beyondcode/test-app on Expo
TypeError: Cannot read property 'id' of undefined

eas login
Log in to EAS
✔ Email or username … username
✔ Password … *************
TypeError: Cannot destructure property 'sessionSecret' of 'body.data' as it is undefined.

eas login
Log in to EAS
✔ Email or username … username
✔ Password … *************
TypeError: Cannot destructure property 'sessionSecret' of 'body.data' as it is undefined.


Install error Xcode Version 6.1.1

Description
After Downloading source code and installing it there was an error.
Operation
$ xcode-select -p
/Applications/Xcode.app/Contents/Developer
$make
Expected behavior
Environments

Xcode Version [6.1.1 (6A2008a)    ]
XVim branch and revision [   master commit 2af06c6cc7514d511e6e00c7ecdb3c74d3b47371  ]
Keyboard Language [     ]
Input source language [    ]

Crash log (if any)
/Users/honu_mac_air/Downloads/XVim-master/XVim/XVim.m:218:111: error: no known
instance method for selector 'menuKeyBindingSet'
...preferenceSetsManager] currentPreferenceSet] menuKeyBindingSet];
^~~~~~~~~~~~~~~~~
1 error generated.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

It has Already been solved


【Hackathon 7th No.22】NO.22 在 paddle.audio.functional.get_window 中支持 bartlett 、 kaiser 和 nuttall 窗函数

 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.

https://github.com/PaddlePaddle/Paddle/pull/68268#discussion_r1791494013
中文文档也需要同步修改

PaddlePaddle/Paddle#68268 (comment)
中文文档也需要同步修改

是指这些吗

已经好了，请review @luotao1


Add support for dweet.io

Add support for dweet.io, which provides a free cloud service for devices to publish messages and information,  This would make it easy to see the most recent sensor readings, information about the device itself (e.g., WiFi RSSI and IP address) and even error/status messages.    Uploads happen via an HTTP POST and JSON, which is easy to add given we already have HTTP connectivity support in the app.

completed and in master


Ubuntu 22.10 (Kinetic Kudu) EOL

Ubuntu 22.10 (Kinetic Kudu) reaches End of Life on July 20 2023. Remove its build files.

Why are we using Git? :thinking:

wdym?

wdym?

I meant that for

I'm thinking they might be useful in the future, just in case we need to exceptionally patch something.

there's no need to rename and keep the files around just in case. They are in Git.

you forgot to remove 22.10 from the test matrix

Ooops. Forgot to (force) push. Done now.

wdym?

I meant that for

I'm thinking they might be useful in the future, just in case we need to exceptionally patch something.

there's no need to rename and keep the files around just in case. They are in Git.

y sure I know. My thinking is:

this is a public repo, and we might have users coming here looking for the images' source. Keeping the unsupported Dockerfiles is useful:

for having an easy indicator that a specific base has been deprecated
so that less proficient Git users avoid navigating the commit history
for having a quick way to rebuild old images

The rockcraft*yaml files OTOH, I agree. They are only useful for CI purposes, and thus if we ever need them again, we (maintainers) can fetch them back from Git. Let's go ahead and remove them :+1:

Removed the files.


Update install-release.sh for new release structure

Attempt to fix #81
I don't have any armv5, armv6, riscv64 setups to test with. So I am not sure what 'uname -m' actually returns on those platforms.

BTW 我们是不是应该考虑修改 https://install.direct/go.sh 指向 fhs 的安装脚本了。
旧的安装脚本应该逐步替换掉，其中第一件事就是让新调用 https://install.direct/go.sh 的用户受到警示，表明这个文件的内容即将与 fhs 安装脚本同步，然后过段时间再真的把 fhs 安装脚本同步过来。
现在这样新旧脚本都得维护，不是特别好，要尽快合二为一。

@IceCodeNew @dctxmei Feel free to edit this patch or even summit new PR with changes the team prefers. I am neither a v2ray nor a bash expert whatsoever.
The script is currently in a broken state and there was supposed to be a hotfix for this. I am assuming that is affecting lots of users as majority of them rely on https://install.direct/go.sh to install or upgrade v2ray.
Not to mention the issue I posted as #82 . I believe the use of awk -e is incorrect there which makes the script fails on any system that not taking gawk as the default awk utility.

I recommend deprecating now

Should indeed be deprecated immediately.

i remove it now


⚠ Thumbnails API Endpoint has degraded performance

In 536b08d, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1213 ms

Resolved: Thumbnails API Endpoint performance has improved in 537aedc after 9 minutes.


Permission denied while creating screenshots: can't change docker image user without errors

I'm trying to run tests with the docker image. It works great on my local machine but fails on GithubCI
Problem is that screenshots are created by "user" (PID 1000) which is defined in your Dockerfile while my screenshots folder is mounted with my host user as owner (PID 502). As a consequence, screenshots file creation ends up with a Permission Denied error (on my local machine the screenshots is mounted with "user" as owner, everything works fine)
To fix that, I want to specify which user actually run the tests (and creates the screenshots):
docker run -v $(pwd):/tests -it -u 502  testcafe/testcafe -s takeOnFails=true,path=/tests/screenshots chromium
What is your Test Scenario?
docker run -v $(pwd):/tests -it -u $UID  testcafe/testcafe -s takeOnFails=true,path=/tests/screenshots chromium
What is the Current behavior?
I get errors before my tests begans:
dbus[7]: Could not get password database information for UID of current process: User "???" unknown or no memory to allocate password entry

dbus[7]: Failed to start message bus: Memory allocation failure in message bus

What is the Expected behavior?
I'd expect my tests to be run with UID 502
Steps to reproduc
Run the docker image with the -u flag with anything else than "1000"
Your Environment details:
Local machine:

platform and version:         Mac OS 11.2.1 (20D74)
docker:                                Docker version 20.10.6, build 370c289
docker image:                     8fca81fb890c

Github runner:

platform and version:  Debian 4.9.210-1
docker:  Docker version 19.03.6, build 369ce74a3c
docker image:                     8fca81fb890c

Hi. I don't see this as a question: I'd like to be able to run the tests with another user than "user" which today is not possible

I used the '-u' flag to specify the user in the container and it works:

According to the Docker docs regarding this topic, a specified user does not have to exist in the container. However, some programs that are required for TestCafe won't work correctly with the user that does not exist (as you can see in the error messages from the dbus program which failed to start because of an unknown user). I suggest you change the rights on the mounted directories from your host and run the container as usual (without specifying the user).

Thanks for your answer.
Even if I chmod 777 my host screenshot folder, the files themselves will be owned by "user" uid which is 1000. I have a user with this uid on my host but it is not related at all to my usage here. Typically, I'm running the docker image with a user of uid 1002. As a consequence I can't delete the screenshot without a sudo rm.
What would be great is for the docker image to allow to override "user" uid. The entrypoint script could look for an env var (USER_ID) for instance and  do usermod -u $USER_ID user before running the test. In my case, I would call the image with docker run -e USER_ID=$(id -u) ... testcafe/testcafe.

Thank you for your feedback. I don't know if this feature can be implemented, so it can be closed sometime later, but for now, let's keep it as a suggestion.
Meanwhile, you can set the SETGID bit on your screenshot directory to make any new files and subdirectories to inherit the group ownership from the base directory: https://serverfault.com/a/93942


Bugs in provided methods with static parameters

The TP:
[<TypeProvider>]
type Provider1 (_config: TypeProviderConfig) as self =
    inherit TypeProviderForNamespaces()
    let ns = "TypeProviderSandbox"
    let asm = Assembly.GetExecutingAssembly()
    let rootType = ProvidedTypeDefinition(asm, ns, "Provider1", Some typeof<obj>, HideObjectMethods = true)
    let cache = new MemoryCache("HexProvider")
    let rootParams = [ProvidedStaticParameter("GeneratingMethodName", typeof<string>)]
    do rootType.DefineStaticParameters(rootParams, fun typeName args ->
        lazy
            let rootTy = ProvidedTypeDefinition(asm, ns, typeName, Some typeof<obj>, HideObjectMethods = true)
            let innerTy = ProvidedTypeDefinition("InnerType", Some typeof<obj>, HideObjectMethods = true)
            let methodName = args.[0] :?> string
            if methodName.Length < 3 then failwith "Method name must be at least three letters long"
            let m = ProvidedMethod(methodName, [], innerTy, IsStaticMethod = true)
            let methodParams = [ProvidedStaticParameter("WhatToReturn", typeof<string>)]
            m.DefineStaticParameters(methodParams, fun methodName methodArgs ->
                let valueProp =
                    match methodArgs.[0] :?> string with
                    | "now" -> ProvidedProperty("Value", typeof<DateTime>, IsStatic = false, GetterCode = fun _ -> <@@ DateTime.Now @@>)
                    | "year" -> ProvidedProperty("Value", typeof<int>, IsStatic = false, GetterCode = fun _ -> <@@ DateTime.Now.Year @@>)
                    | _ -> failwithf "WhatToReturn must be either 'now' or 'year'"

                innerTy.AddMember valueProp
                let m2 = ProvidedMethod(methodName, [], innerTy, IsStaticMethod = true, InvokeCode = fun _ -> <@@ obj() @@>)
                rootTy.AddMember m2
                m2
            )

            rootTy.AddMember innerTy
            rootTy.AddMember m
            rootTy
        |> cache.GetOrAdd typeName)

    do self.AddNamespace(ns, [rootType])
       self.Disposing.Add <| fun _ -> cache.Dispose()

[<assembly:TypeProviderAssembly>]
do ()

Root type accept a string static parameter and generates a static method with the same name, which returns an inner generated type InnerType.
The method accepts another string static parameter and adds a property Value to InnerType. The return type and value of this property depends on the static parameter value: "now" => DateTime, DateTime.Now, "year" => int, DateTime.Now.Year. If the argument has other value, then an exception is raised.

Issues:

If a unexpected string is passed to the provided method's static parameter, then VS shows generic error instead of our exception's Message: "The type provider 'TypeProviderSandbox.Provider1' reported an error: Exception has been thrown by the target of an invocation."

Compare it with error which is shown if we pass wrong string to the type static parameter:

VS and FSI cache Value property and does not change it according to its static parameter:

(resetting FSI helps, but VS is stuck with int as return type for Value, until a file is reopened)

(the bug is not caused by the cache, removing it does not solve the problem)

I guess this the same as
https://github.com/fsprojects/FSharp.TypeProviders.StarterPack/issues/63

The first part of this (the bad error message) was fixed some timme ago
The second part (the property caching) is by design - the line
            innerTy.AddMember valueProp

ends up adding two properties with the same name to innerTy and returns a different method in two cases associated with that property.  Instead a different innerTy should be provided for each of the two cases, where the name of innerTy incorporates methodName


Feature request: Return last inserted ID for MySQL INSERT

It would be really nice if there was some way where, using MySQL as the database of choice, the sql.Result struct was returned when INSERT queries are executed. Currently, the sql.Result struct is being ignored.

do you use anotation :exec ? if it yes, you should change it with :execresult. execresult will return (sql.Result, error)

Thanks for the answer @dee-el. The :execresult command is meant for this exact situation.

Thank you everyone


Method of getting string input?

I am trying to make a TUI for discord, and I cant find any other non-blocking key input like this.
In every other library it blocks code from continuing while looking for input, instead of passing None.

You'll need to write the text input widget yourself, handling individual keypresses.


Close of inproc does not notify peer

While working on the fix for #286 I discovered that the inproc transport does not handle close of the pipe properly.  In particular, the writeq is never closed, so that the peer pipe never gets a chance to discover that the peer has gone away.
Probably we never thought to test inproc this way -- a program generally doesn't assume that itself is going to die -- but there are paradigms where using inproc and closing inproc channels can be useful -- e.g. to notify a condition, or for example when using the patterns to distribute work to oneself.
The fix is fortunately quite easy -- we just need to close the write side of the channel, taking care to ensure that we don't leave ourselves open to attempting to double close the wq, or to attempt to send on it.   (We can ensure the sender is woken by closing the closeq.  We do need to set the pipe wq to nil first so that if the writer loops quickly or another one comes it, it the read on the closed cq wins.)

This turned out to be trickier than I thought -- a few issues.
a) apparently even closing the cq was not enough to guarantee the writer (blocked) got woken up and avoided the panic.   Solution to that was simply to add the peer's closeq to the list of queues we sleep on, and not close the write q.
b) the new logic, that is pretty aggressive about getting messages out, actually caused messages to get replayed during test setup, because the tests didn't wait for slow start to settle.  The duplicate/retry messages wound up breaking various test assumptions.  The best solution to that was simply to wait a bit for the connections to establish.


Add versionbot workflow

@klutchell to clear/modify branch protection rules (e.g.):
org=balenaci
repo=vb-test
branch=master

curl -X PUT https://api.github.com/repos/$org/$repo/branches/$branch/protection \
  -H "Accept: application/vnd.github+json" \
  -H "Authorization: Bearer $GITHUB_ACCESS_TOKEN" \
  https://api.github.com/repos/$org/$repo/branches/$branch/protection \
  -d '{"required_status_checks":{"strict":true,"contexts":[]},"enforce_admins":false,"required_pull_request_reviews":{"dismissal_restrictions":{"users":[],"teams":[]},"dismiss_stale_reviews":false,"require_code_owner_reviews":false,"required_approving_review_count":1,"bypass_pull_request_allowances":{"users":[],"teams":[]}},"restrictions":{"users":[],"teams":[],"apps":[]},"required_linear_history":false,"allow_force_pushes":true,"allow_deletions":false,"block_creations":false,"required_conversation_resolution":false}'

See, https://docs.github.com/en/rest/branches/branch-protection#update-branch-protection

@balena-ci retest

@balena-ci I self-certify!

@ab77 we need something to be required for bulldozer to run. Can we mark just VersionBot/generate-version as required for now? I wish we had admin access like I requested.

@balena-ci rebase

@balena-ci retest


getCompilerOptionsDiagnostics didn't report when use compiler API

here is a example .
// a.ts
var a = (
HelloWorld

when I run tsc a.ts in terminal , it will report
a.tsx(2,9): error TS17004: Cannot use JSX unless the '--jsx' flag is provided.

but when I compile it with compiler API ( from https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API ) , it will report nothing . ( by the way , seems that only getCompilerOptionsDiagnostics report nothing , other diagnositics report right )
here is the example code
function logErrors(fileName: string) {
        let allDiagnostics = services.getCompilerOptionsDiagnostics()
            .concat(services.getSyntacticDiagnostics(fileName))
            .concat(services.getSemanticDiagnostics(fileName));

        allDiagnostics.forEach(diagnostic => {
            let message = ts.flattenDiagnosticMessageText(diagnostic.messageText, "\n");
            if (diagnostic.file) {
                let { line, character } = diagnostic.file.getLineAndCharacterOfPosition(diagnostic.start);
                console.log(`  Error ${diagnostic.file.fileName} (${line + 1},${character + 1}): ${message}`);
            }
            else {
                console.log(`  Error: ${message}`);
            }
        });
    }

Do you have complete code (or a minimal repro) that you could share so we can diagnose the issue?

@WanderWang any update on this?

@DanielRosenwasser    sorry . I will make a simple example tonight

" tonight " means that 10 hours later because of I'm in Beijing ,China

Hello @DanielRosenwasser
Here is a simple example https://github.com/WanderWang/TypeScriptIssue6174
I got a mistake . When I haven't set jsx field to  compilerOptions , it will report  error TS17004: Cannot use JSX unless the '--jsx' flag is provided.
But when I input a error jsx value ( such as '100' or 'react' ) ， it will not report a " Unknown jsx value " exception  so I thought that it 's right in a mistake.
Maybe you should check the compilerOption as a Defensive programming . But it 's out of range in this issue


Test MQTT driver with Kepware IoT Gateway egress.

TODO Describe test scenario.

@MRIIOT not sure what this is referring to - is this an old clickup issue? the mqtt driver (provider/subscriber) works now, but could develop some unit tests for it.

This was in regards to parsing across Kepware IoT payload array.


Why in the world is this program described as a GUI application?

The program clearly uses a TUI.
TUI is different from CLI.

Hi, this program is not described as a GUI application.
The first line of the README.rst file says Giterm is a ncurses-based terminal GUI for Git.
Though I agree that GUI would be an abusive term for Giterm, it seems that "terminal-based GUI" is an acceptable definition for such a rich TUI.
Moreover there is no intention to lie about its nature:

in addition to the README file, its Pypi's long_description tells the same
the Pypi classifiers explicit Environment :: Console :: Curses
there is a screenshot on the project's home page.

Please feel free to suggest an improvement if you have any, but take care not to make an opinion picking a single word in a full-text. I would also recommend not starting your GitHub experience by attacking projects before having carefully read their description.


formulation and reference of "this" in understudied-languages.tex

This is correct, but confusing … Anyway. Author does not respond.


400 error during Setup Wizard after reinstalling plugin (re-using same tenant)

If you install the plugin and connect it with a particular tenant, and then reinstall it, the following error occurs during the Setup Wizard using the default settings:
WP_Auth0_Api_Client::create_user    N/A    {"statusCode":400,"error":"Bad Request","message":"Payload validation error: 'Expected type string but found type null' on property connection (The connection the user belongs to).","errorCode":"invalid_body"}

This seems to be caused by the Users Migration option being disabled after reinstalling (it's usually enabled by default). If you enable it, you get an error:
There was an error enabling your custom database. Check how to do it manually HERE.
However, if you try it again, you can now go through the Setup WIzard successfully. You don't need to update the connection manually in the Auth0 dashboard as described in the Users Migration option either.
I'm not sure why the migration is disabled by default during subsequent installs, but it would be good to make the cause of this problem more clear to users.

Backlogging internally with #408 for a future release.


Handled undefined or non-array 'sitemap' in API response

Have you found why the sitemap might not be defined in the response?
I’d remove the console body logging, that’ll be very noisy as it’s outside the condition


Fixed link to Known Issues for SSMS Download

Link to Known Issues section of SSMS ReleaseNotes was pointing to 1831.  Updated to 184

@DaveDustin : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.
@DaveDustin Thank you for submitting this link fix.


What is status column in Conversations table for?

I'm interested in knowing if a conversation is replied to or not, do i have to do find it by checking the last message or is there any other implemented method already as i cant find it in documentation.

Sorry these feature is not available now.
The status column is used for block/mute conversation

Can you tell me is there any way within Talk to know who updated the status column?  I am wanting to use "status" to show if a user has blocked the conversation.  That user can then "un-block" if necessary.  In order to do that, I need to know who initially blocked it.  Does the package support this?  Thank you!


Add dynamic prompt support via ExternalPrinter

rustyline doesn't currently support changing the prompt while in the core readline loop.  There are a number of open PRs and issues for this functionality, but all of them appear to be stalled for more than a year.
Looking at #696 and 4ec26e8, the traditional appoach to this is to provide a reference to a trait object (Prompt or ToString), but with that appoach there's no way to cause the prompt to be redrawn for a change without user input.  This means for these appoaches the prompt could change without being displayed to the user.
There's an existing mechanism to allow another async task/thread to push input into the core readline loop, the ExternalPrinter.
In this commit, I expand ExternalPrinter to add set_prompt().  With various plumbing, this function results in wait_for_input to return Cmd::SetPrompt(String).
One of key change here is State.prompt changes from &str to String.  There is a performance hit here from the copy, but rustyline would need to prompt and receive input hundreds of times per second for the copy to have a noticable performance inpact.
Added examples/dynamic_prompt.rs to demonstrate the functionality.
Closes #417
Related #208, #372, #369, #417, #598, #696

Note, this is only the unix implementation.  I do not have access to a windows development environment to add the implementation for that side.  It's not a complicated change, but for now it's marked as unimplemented!().

See https://github.com/kkawakam/rustyline/issues/126#issuecomment-1887685910

removing / replacing ExternalPrinter by the clever / simple linenoise solution is appealing.

And another issue related to the prompt: #348
And a more fundamental issue: https://github.com/kkawakam/rustyline/blob/master/Incremental.md

Ideally, we would like to redraw only impacted row(s) / cell(s).

So, for me, when only the prompt (Cmd::SetPrompt) is changed, we should not have to redraw the whole line ( s.refresh_line()) or at least we should not have to invoke Hinter::hint() / Highlighter::highlight_char / Highlighter::highlight (only Highlighter::highlight_prompt).
I apologize for the stalled situation induced by some bad design choices.

TL;DR: Pushing a change to reduce processing when prompt changes.  Linenoise approach is out-of-scope for this PR, but I am looking at how to do something similar.
Reduce computation for Cmd::SetPrompt

removing / replacing ExternalPrinter by the clever / simple linenoise solution is appealing.

I dug into the linenoise code to look at this note.  Linenoise does not directly support changing the prompt.  Although the prompt is stored in the struct linenoiseState, the only way to get the prompt to be redrawn is by calling linenoiseShow() at minimum.  This causes a full redraw of the prompt.  Which leads to...

[...] when only the prompt (Cmd::SetPrompt) is changed, we should not have to redraw the whole line [...]

I don't think this is advisable when you consider that the length of the prompt may change.  In that case you'd have to redraw the rest of the line as all of the characters would need to be shifted.   You can absolutely implement this, but it's added complexity for sub-millisecond performance gain on user keyboard input.

when only the prompt (Cmd::SetPrompt) is changed, [...] or at least we should not have to invoke Hinter::hint() / Highlighter::highlight_char / Highlighter::highlight (only Highlighter::highlight_prompt)

This makes sense as the hint() function may do blocking things like checking networked filesystems.  I made this change by adding Refresher::refresh_prompt() as a cut down version of refresh_line().  I could instead make the call to State::refresh() in State::set_prompt(), but it looked like you wanted to maintain a clear separation.
Remove ExternalPrinter
I agree with you that the linenoise approach where the caller only calls into rustyline when there is input to be processed is a much better approach.  I've already hit some additional limitations with ExternalPrinter in my project, so I'll see if there's a simple way to implement that for rustyline.  No guarantees on my timeline here.
That said, I think adding that interface is out of scope for this PR.  It's an architectural shift in rustyline, and not directly related to adding dynamic prompt support.

Hey, would love to see progress here!


Add portfolio and photo

I have included Tan Wei Jie's photo to my PR, photo name and type corrected.

Portfolio contents and photo has been transferred and updated on PR #56, can close this PR without merging.


Dynamic ui-sref in directive

I've written a breadcrumb directive which takes an array of crumbs, each with a name and sref property. I'm trying to bind the ui-sref directive ($StateRefDirective) to use the evaluated attribute, but it's not working. When the ui-sref directive's link function is called, the attrs.uiSref value is undefined, but immediately after the running of the function becomes available as the correct value.
angular.module('myApp').directive 'breadcrumb', ->
  scope: { crumbs: '=' }
  replace: true
  template: '

      {{crumb.name}}

  '

Inside the $StateRefDirective link method, logging attrs gives
Attributes {… uiSref: undefined …}

Running the same log after a 100ms timeout gives:
Attributes {… uiSref: "apps.show({appId: crumb.app.id})" …}

Am I doing something wrong, or are dynamic ui-sref attributes not supported?
Note: Running v0.2.0

If anyone would like to implement this, please submit a PR and reference this issue, thanks.

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


HDDS-6049. IllegalStateException: zip file closed in HDDSLayoutVersionManager

What changes were proposed in this pull request?
Fix intermittent IllegalStateException: zip file closed by creating a single instance of Reflections for HDDSLayoutVersionManager.  This avoids repeated (concurrent) scanning of the classpath.
https://issues.apache.org/jira/browse/HDDS-6049
How was this patch tested?
https://github.com/adoroszlai/hadoop-ozone/actions/runs/4900659031

@errose28 would you like to take a look?

Thanks @sodonnel for the review.


How can I read the XYZ position of the spindle

I want to read the spindle position of heidenhan cnc, but I can't find function.

Maybe so, but I'm not sure
        payload = bytearray()
        payload.append(0)
        payload.append(0x16)
        result = self._send_recive_block(CMD.R_RI, RSP.S_RI, payload)

result: [bytearray(b'\x00\x050.0000\x000.0000\x00500.0000\x000.0000\x000.0000\x00X\x00Y\x00Z\x00B\x00C\x00')]

Hello, Could  you tell me how to download tnccmd? Thank you

Great find!
I checkt with TNC 640 and iTNC programming station and both work with the command you found. It was a bit tricky to decode the result but it worked out in the end.
The second byte is the number of axes followed by the actual values followed by the axes letters in order. We are missing info about the first byte which is always zero but it dosen't seem to be necessary :-)
The new function returns a dictionary where the keys are the axes letters and the values are the numerical values.
Pull request #27 contains the implementation. If it works for you I will include it in the next release.

function is part of release v0.7.4

Test OK in Heidenhain 530

I used TNC dialog function SYSREAD


GitHub Actions

I have no experience using GitHub Actions, but I'm guessing they can be setup to do all the packing for us. Would be nice to update the snapshot build every commit and the actual packages on tags.

https://docs.github.com/en/actions/publishing-packages/publishing-java-packages-with-maven

An action to publish to Maven Central is done. Next up will be actions to build commits and pull requests.


📃: Gravity Drops (VanillaJS)

:red_circle: Title : Gravity Drops
:red_circle: Aim : An application to visualize the gravity with balls and create patterns with them
:red_circle: Tech Stack Used :  HTML, CSS and Javascript
Screenshots 📷

https://github.com/user-attachments/assets/b30c71dc-adc4-4228-b401-7df864bc2e76

:white_check_mark: To be Mentioned while taking the issue :

Full name : Kosuri Indu
What is your participant role? GSSOC'24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney
Please assign this issue to me


Keep docsgen from updating protected branches

Changes introduced with this PR
This PR adds a check to the docsgen reusable workflow which skips the step where it updates the branch if the branch is protected (like main), because attempting the update will result in an error which will cause the workflow (and the rest of the CI) to fail.
We're not supposed to need updates to protected branches, as the updates are supposed to have been applied to PRs before they are merged, so, under normal circumstances, this change should have no impact. (And, if an update is actually required, it should be detected and applied to the next PR to come along, anyway.)
However, under unusual circumstances (such as when a PR is merged too quickly for the bot to respond, or when the bot is producing an unstable result (https://github.com/arcalot/arcaflow-plugin-fio/pull/39)) this change will prevent the main branch CI from failing.

By contributing to this repository, I agree to the contribution guidelines.

Testing complete:

Performs the update when branch is not protected (log; rerun with debug output)
Skips update when branch is protected (log)

Yeah, OK; I suppose all the repos must at least have main protected.

Yeah, I contemplated filtering based on whether the target was a branch (as opposed to a PR), but our practice has drifted toward using upstream branches for PRs; I considered excluding particular names (e.g., main), but that seemed cumbersome and unreliable; and, I considered ruling out tags (either by name or as a class).  I finally concluded that, since the harm only arises when the bot is unable to push the commit, and that happens only when the branch is protected, I sould use that as the basis of the filter, since GitHub conveniently offers a test for it.
And, yeah, the main branch should be protected.  (And, if it's not, then I guess the bot is free to update it....)


Fix Microsoft.Wpf.Interop.DirectX_dxsdk.sln

Fixing build of Microsoft.Wpf.Interop.DirectX_dxsdk.sln

Hi @xmedeko, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

You are referencing the wrong project.
I think Microsoft.Wpf.Interop.DirectX_dxsdk.sln should be referencing Microsoft.Wpf.Interop.DirectX_dxsdk.vcxproj
and not Microsoft.Wpf.Interop.DirectX_winsdk.vcxproj like you did
For that we already got Microsoft.Wpf.Interop.DirectX_winsdk.sln

@xmedeko, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;


Why are the docs blank?

The new docs have been blank for some time now. I think the old ones should be kept till the new one isfinished.

I am working on fixing it. Due to a physical injury, I haven't really been on key with it.

Sorry to hear it hope you get better.

Me too! I have begun the update!

All good. I have pushed. Getting started somehow is acting weird but it may just be cached.


Show only specific region(s)

Provide a plugin command to allow the user to black out all regions expect a specified one then undo the same thing, remembering which of the regions were dark before.

Probably won't do this.


🔥 announcement 🔥: introducing user registration section

Will be abandoning the registration inside portal section. Instead the registration will have its own configuration section.
This change will be released in: v1.1.2
Note if you are not using user registration, then this is not a "breaking" change.
Previously, your  registration config was a part of authentication portal.
{
  security {
    authentication portal myportal {
      registration {
        dropbox {env.HOME}/.local/caddy/registrations.json
        title "User Registration"
        code "NY2020"
        require accept terms
        require domain mx
        admin email<EMAIL_ADDRESS><EMAIL_ADDRESS>      }
    }
  }
}

Now, it has its own section. Here, the user registration is tied to an email messaging provider, localhost-smtp-server.
{
        security {
                user registration localdbRegistry {
                        dropbox {$HOME}/.local/caddy/registrations.json
                        title "User Registration"
                        code "NY2020"
                        require accept terms
                        require domain mx
                        admin email<EMAIL_ADDRESS><EMAIL_ADDRESS>                        email provider localhost-smtp-server
                        identity store localdb
                }

                local identity store localdb {
                        realm local
                        path {$HOME}/.local/caddy/localdb/users.json
                }

                authentication portal myportal {
                        identity store localdb
                }

                credentials root@localhost {
                        username root
                        password foobar
                }

                messaging email provider localhost-smtp-server {
                        address <IP_ADDRESS>:1025
                        protocol smtp
                        passwordless
                        sender root@localhost "My Auth Portal"
                        credentials root@localhost
                        bcc greenpau@localhost
                }
        }
}

@greenpau - is this a breaking change? Will my Caddy instance break if I download this plugin without updating my config file first?
SemVer isn't about versioning your API endpoints (I don't, but I do follow SemVer). It's about communicating to users when you're breaking a previously-establish contract they depend on. That includes changing a config file parser.
Without that, you're expecting users like me to check your GitHub Issue board for an Announcement post with every single release. I don't have time to do that for the 30+ Docker services I run, not to mention all of the other software I use daily. I don't have time to post a support ticket either and deal with the back-and-forth. Clear communication and documentation is critical.
Instead of releasing these breaking changes one-off like this, why not get them all working on a v2 branch, get your docs updated as well, and merge and release all of that change as a major release? Make a big announcement of it, celebrate the progress. That flow makes the change a lot easier for a user like me to process, and let's me celebrate along with you. 🤷

@greenpau - is this a breaking change? Will my Caddy instance break if I download this plugin without updating my config file first?

@tpansino , if you are not using user registration, then it is non-breaking.

Instead of releasing these breaking changes one-off like this, why not get them all working on a v2 branch, get your docs updated as well, and merge and release all of that change as a major release?

When you build caddy, you can lock a specific version of a specific plugin. This way, your will be locked to whichever version you choose.
https://github.com/greenpau/caddy-security/blob/059a464f90b0de31d352dfa6bbfb3eee2daa2908/Makefile#L17-L19
It works the same when you use Caddy download page.

You can lock your builds to a specific version.

Without that, you're expecting users like me to check your GitHub Issue board for an Announcement post with every single release.

Yes, I do expect people to skim through the announcements. If a person is running an authentication server, the person needs to keep it up-to-date and understand recent changes.
Side note: You did mention another authentication project in another issue. I don't appreciate the mentions like that. That's why I deleted your comment. Also, I will delete these comments too because they are not useful.

The examples should be also updated:
https://raw.githubusercontent.com/greenpau/caddy-security/main/assets/config/Caddyfile
Of course configurations changes from time to time. But it looks like it's a bigger refactoring. I think it's my 3rd reconfiguration in just some weeks. ;) Do you have a road map when it's finished?

Of course configurations changes from time to time. But it looks like it's a bigger refactoring. I think it's my 3rd reconfiguration in just some weeks. ;) Do you have a road map when it's finished?

@helmut72 , this would be the last one big refactoring for some time! 😃 Hopefully, I will finish it this weekend. It is just super tedious work converting stuff to be interface-driven.
Next, I will be working on redesigning UI. There will be no configuration changes there.

Thanks for info 😀

I think the documentation here needs to be updated: https://authp.github.io/docs/authenticate/local/user-registration
With the new registration section, is it required to use an email provider? The old example didn't use an email provider, but it seems it is required now. When I try to set it up without an email provider, I get this error:
reload: adapting config using caddyfile: parsing caddyfile tokens for 'security': user registration configuration for "localfileregistry" instance failed: email provider is not set

@bdr99 , thank you for pointing it out. Please create an issue in https://github.com/authp/authp.github.io/issues/new for tracking

Also, with the new registration section, is it required to use an email provider?

Yes, please see complete example:
https://github.com/authp/authp.github.io/blob/main/assets/conf/local/registration/Caddyfile#L38

@bdr99 , thank you for pointing it out. Please create an issue in https://github.com/authp/authp.github.io/issues/new for tracking

Thanks! Will open an issue in the website repo.

Also, with the new registration section, is it required to use an email provider?

Yes, please see complete example:
https://github.com/authp/authp.github.io/blob/main/assets/conf/local/registration/Caddyfile#L38

Is there a reason why it was changed to be tied to an email provider? This seems like a bit of a step backward, since it was possible before to configure registration without an email provider. It's not a big deal to me, but I'm just curious of the reasoning that went into this change.


Configure winsqlite3 from connection string

Ideally the setting (something like UseWinsqlite3=True;) could handle the "if windows 10 do this, else do that" for users so we wouldn't need different connection strings for Linux, Windows 7 and Windows 10.
This is an alternative to calling SqliteEngine.UseWinSqlite3()

The way I understand this is working only makes it possible to choose the SQLite native assembly before you load it and once made loaded you cannot change it.
The connection string on the other hand can easily be different per connection instance.
Don't you think then that the mismatch would make it confusing?

Don't you think then that the mismatch would make it confusing?

Could be, but they have different semantics. The SqliteEngine call currently means “Only use winsqlite3.dll”.
I’m suggesting we add a parameter that means “Use winsqlite3.dll when it exists otherwise use the packaged version”.

We don't have any customer use cases where this would be wanted. If folks ask for it then we can reconsider in the future.


[WIP] Refactor viz backend

Description

Development notes

QA notes

Checklist

[ ] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

Legal notice

[ ] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Closes in favour of https://github.com/quantumblacklabs/kedro-viz/pull/432/files


devtool Identifier 'global' has already been declared

setImmediate.js
第三行：
const global = (Function('return this'))();
请把const改成var。
不然devtool开启时会报错。
报错信息如下：
devtool Identifier 'global' has already been declared

Thanks!
Has been done new version<EMAIL_ADDRESS>

I,m sorry, I installed 1.7.0.
but also 'const'.

Yes but you will not have the error anymore .
the probleme was that node define alrady global so i change the name of our
variable to _global


How to get 640 x 480 pixels raw images

I want to get 640 x 480 pixels raw images in Unity.
I implemented in the following way and got images of 640 x 240 pixels.
public class GetLeapImage : MonoBehaviour{

    private LeapServiceProvider _provider;
    private Controller _controller;

    private void Awake()
    {
        _provider = GetComponent<LeapServiceProvider>();
        _controller = _provider.GetLeapController();
        _controller.ImageReady += OnImageReady;
    }

    private void OnImageReady(object sender, Leap.ImageEventArgs args)
    {
        Debug.Log(args.image.Width + ", " + args.image.Height); // 640, 240
    }
}

Using LeapUVC, it seems that I can get 640 x 480 pixels images.

I don't believe this is possible currently with the our LeapC API through Unity. I do expect some changes down the pipe for the API in the coming months, but I don't have any specifics on that right now.

@nickjbenson
Ok, thank you for the answer!


Fail to build in step 2: cannot move location counter backwards (from 20090e84 to 20040000)

Running on Adafruit Feather RP2040 Adalogger
In step 2, when I try to build, the following error comes up after reaching 100%
[100%] Linking CXX executable ../bin/featherrp2040/LedBlinker.elf
<command-line>: note: this is the location of the previous definition
/home/xyz/.arduino15/packages/rp2040/tools/pqt-gcc/4.0.1-8ec9d6f/bin/../lib/gcc/arm-none-eabi/14.2.0/../../../../arm-none-eabi/bin/ld:/home/xyz/Downloads/code/fprime/led-blinker/build-fprime-automatic-featherrp2040/arduino-cli-sketch/memmap_default.ld:235 cannot move location counter backwards (from 20090e84 to 20040000)
collect2: error: ld returned 1 exit status
gmake[2]: *** [LedBlinker/CMakeFiles/LedBlinker.dir/build.make:340: bin/featherrp2040/LedBlinker.elf] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:7420: LedBlinker/CMakeFiles/LedBlinker.dir/all] Error 2
gmake: *** [Makefile:136: all] Error 2
[ERROR] CMake erred with return code 2

settings.ini
[fprime]
project_root: .
framework_path: ./fprime
library_locations: ./fprime-arduino:./fprime-baremetal
config_directory: ./config
deployment_cookiecutter: https://github.com/fprime-community/fprime-arduino-deployment-cookiecutter.git

default_toolchain: featherrp2040

default_cmake_options:  FPRIME_ENABLE_FRAMEWORK_UTS=OFF
                        FPRIME_ENABLE_AUTOCODER_UTS=OFF

current connected board:
/dev/ttyACM0 serial   Serial Port (USB) Adafruit Feather RP2040 Adalogger rp2040:rp2040:adafruit_feather_adalogger rp2040:rp2040

It seems to be some kind of memory issue, since nothing has been added yet, not really sure what can possibly be wrong.
Update:
Tried w/ Pico board, got same result. default_toolchain: rpipico

Hello,
Was anything in the config/ directory changed? Did you copy the config/ directory from fprime/config/ or another source?

copied config directly from fprime/config as suggested, i'll try not copy later today

Use the config/ from the deployment cookiecutter since the default fprime/config/ is too large for majority (if not all) of the Arduino boards after the recent updates.
I will update the tutorial docs to reflect this.

Try copy the LedBlinker/config/ directory into the project root instead, and change the settings.ini to be
config_directory: ./config

build cache seems to always place the config/ path under build-fprime-automatic-x instead of build-fprime-automatic-x/LedBlinker, hence why the include path couldn't be found.
I'm not sure the reasoning behind this.. probably some underlying linking stuff FPrime is doing within their cmake files.
@LeStarch is this a bug? Or were custom config/ files not intended to go in other file paths other than the project root?

Configs must be synonymous with a project. This is because it is invalid to have different configs between different items in the same build.  A build and a project are one to one.

That being said, configs should be allowed to be placed anywhere in a project.  That bit might be a bug.


Currecy Convertor

I would be using HTML, CSS, JS to complete this task.
In this, the user will enter the amount, to currency and from currency, and according to the options chosen the value will be displayed.
@urvashi-code1255 @Vishal-raj-1 I would love to work on this issue as a part of GSSOC 2021 and I will make a PR for this when the contribution period starts.
I have completed the earlier issue also and will make the PR for the same when the contribution period starts.
Please assign me this issue.

@Sapna2001 go for it

Hii, I am interested to work on this project. So pls assign this to me !


Why "target" attributes are not allowed for anchor "A" elements?

I can't seem to find any reason documented for this. Is there some reason "a::target" is blacklisted?

Because specific values of target= other than "_blank" are a namespace in the host page. If you open two links with the same target= value then they overwrite each other, so if the guest can set arbitrary target= then it can interfere with the functionality of the host or other guests in the same page.
I don't immediately see a reason why we couldn't rewrite them to have a guest-specific prefix/suffix in the same way we do ids and classes, but currently such methods have to be implemented ad-hoc and supporting this has never been requested before.

@kpreid thanks, this makes sense. The value of _blank is actually what my current problem is with. Basically what happens right now is that guest specifies _blank, but it's removed by the sanitizer and HTML defaults it to _self. So, in a way, it creates a security issue. I think some other values might be more controversial and require a deeper look, but _blank should be a good idea to whitelist. WDYT?

It should be already whitelisted — there's a configuration option which defaults to allowing it. Sounds like there's a bug. If you want to dig into the code, look for identifiers optTargetAttributePresets and targetAttributePresets.

@kpreid I've been using src/com/google/caja/plugin/html-sanitizer.js and I don't think those options are exposed there. Could you please advise if this is not something I should be using?

The sanitizer should simply have target whitelisted. The relevant part of the default (only) whitelist is in src/com/google/caja/lang/html/html4-attributes-whitelist.json.

Do you mean that it should be already whitelisted by default? I'm looking into the compiled html-sanitizer-bundle.js, where it's inlined as:
html4.ATTRIBS = {
  'a::target': 10,
}

From what I can tell, the value of 10  (FRAME_TARGET) is filtered out in the sanitizeAttribs method in the html-sanitizer.js.

Sorry, your analysis is correct. We've got a little too many hardcoded policies...

@kpreid Do you think this could be fixed?

@kpreid Friendly ping. Have you given this issue a thought?

42626651

Oude3c


Support HTTP01 DNS resolvers for cert-manager

There are two ways for Cert Manager to do the certificate resolution:

DNS01: cert manager gets a token created from the DNS provider that allows it to put a record that solves the TLS challenge
HTTP01: this is in case the DNS provider isn't supported by cert manager, a file from the gateway should be exposed on a specific path that the ACME provider can then use to solve the TLS callenge.

Currently we've a workflow that can work with DNS01 resolvers, as these are automatic.
As a user, I'd like to be able to use HTTP01 certificate resolution with Kusk Gateway.
Reference:

https://letsencrypt.org/docs/challenge-types/
https://cert-manager.io/docs/configuration/acme/

I'll expand on this further but have you considered https://zerossl.com/.