_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 23s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  7s
Maven dependency ordering for patch

-1 :x:
hadolint
0m  2s
The patch generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 23s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

+1 :green_heart:
hadolint
0m  1s
There were no new hadolint issues.

+1 :green_heart:
shellcheck
0m  1s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

2m  9s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2807/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2807

Optional Tests
dupname asflicense hadolint shellcheck shelldocs

uname
Linux f5710090a487 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d963342f8a

Max. process+thread count
52 (vs. ulimit of 30000)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2807/2/console

versions
git=2.17.1 maven=3.6.3 shellcheck=0.4.6 hadolint=1.17.5-0-g443423c

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Add Node information including devices

Is your feature request related to a problem? Please describe.
I've been working with Nomad Devices a lot recently, in making a this plugin.  In debugging that, I noted that the Nomad GUI itself doesn't have a good way to explore list the devices on a node.
Describe the solution you'd like
Wander could have a Node page like Nomad's ui/clients/<client_id>  that showed data from the Nomad "Read Node" API.
The information I was interested in  was Devices, but there's also the exposed network interfaces and other information that could be shown in stanzas.
To reach this view, there might have to be a Client/Server list page, like Nomad's /ui/clients and ui/servers.  Then they drill down to this view from a list.   One might want to reach it from a Job's view too ("what is the Node Info for this Job?"
Describe alternatives you've considered
I use a combination of Docker and the Nomad CLI.
Additional context
Really enjoy wander as is, it has helped me in some jams... =)

Hey @neomantra :)! This sounds useful! I won't get to it immediately as I have less time for wander these days, but I'll shoot for sometime in the coming months, and marking it as "Help wanted" in case someone else wants to submit a PR in the meantime.


feat: ensure records getting are working fine for Arbitrum resolver

Feature Request
Describe the Feature Request
As we are introducing Arbitrum resolvers, we want to guarantee that Arbitrum stored domains datas are safely get by our dApp.
Definition of done

Register a name with Arbitrum resolver and set text records to it: check if text records are get correctly after;
Use the previous registered name to create subdomains with Arbitrum resolver: check if subdomain is included in subdomains list;

This issue is blocked by L2 Metadata API

duplicated by #198


AuthP support for UserNames?

If you try to add a UserName to a User in Example3AppAuthSetupData.UsersRolesDefinition, you'll get an exception during the AuthDBContext migration as follows...
"AuthPermissions.BaseCode.CommonCode.AuthPermissionsBadDataException: 'Index X: The user XXXXX didn't have a userId and the IFindUserInfoService couldn't find it either.
XXXXX'"
It appears that the AuthP code is creating IdentityUsers with UserName defaulting to Email but allowing a UserName to be specified for AuthP users in the Bulk Load facility.
As a side issue, the password for new users defaults to the user's email address. Would be better to be able to set a default password from dev settings or in production randomize the initial password and force reset on initial sign-on.

I think the problem is that you have added / changed some of the users in the Bulk Load, but you haven't added the ASP.NET Core individual user account users.
These individual user accounts are added via the StartupServicesIndividualAccountsAddDemoUsers, which looks for the DemoUsers in the appsettings.com file.
Of course, Bulk Loading and auto-adding individual user accounts is only there so that you can easily try the various examples - in a real application these features wouldn't be used.

@ccnicholls99 I encountered the same issue initially. However, AuthP allows you to specify a UniqueUserName value as well as a UserName. The UniqueUserName takes priority and is passed to the FindUserInfoService implementation.
Here is my bulk load:
        public static readonly List<BulkLoadUserWithRolesTenant> UsersWithRolesDefinition = new()
        {
            new ( email<EMAIL_ADDRESS>userName: "Joe Bloggs", nameof(AppRoles.SysAdmin), uniqueUserName: "example1@example.com"),
            // ...
        };

I think the problem is that you have added / changed some of the users in the Bulk Load, but you haven't added the ASP.NET Core individual user account users.
These individual user accounts are added via the StartupServicesIndividualAccountsAddDemoUsers, which looks for the DemoUsers in the appsettings.json file.
Of course, Bulk Loading and auto-adding individual user accounts is only there so that you can easily try the various examples - in a real application these features wouldn't be used.

Sorry. only just got back around to revisiting this issue.
The code you indicated is....
`        public async ValueTask ApplyYourChangeAsync(IServiceProvider scopedServices)
{
var userManager = scopedServices.GetRequiredService<UserManager>();
var config = scopedServices.GetRequiredService();
var demoUsers = config["DemoUsers"];
        if (!string.IsNullOrEmpty(demoUsers))
        {
            foreach (var userEmail in demoUsers.Split(',').Select(x => x.Trim()))
            {
                //NOTE: The password is the same as the user's Email
                **await userManager.CheckAddNewUserAsync(userEmail, userEmail);**
            }
        }

    }`

As I mentioned, this code is assuming (a) Demo Users are identified by Email Address and (b) the initial password is set to email address. The problem I see is that the demo accounts in appsettings.json are added based on the email address, yet the code in SeedRolesTenantsUsersIfEmpty that looks up the User Account uses the UniqueUserName as the lookup key.  As stated, when username = email address then the lookup works. If not, you get the error I originally pointed out.
"in a real application these features wouldn't be used." - well, actually yes it could be part of an automated site provisioning process that includes pre-defined Demo Accounts.
All this implementation is buried in the core code (AuthPermissions.AspNetCore.StartupServices) so it's hard to see how I can work around it or override it at the moment. Perhaps a better idea might be to allow clients to plug-in their own Account Setup Service to....
(a) Define Identity Users using their own identification policy (Username/Email)
(b) Set initial passwords and/or force reset on first logon
....might already be possible, I haven't gotten that far yet.

hi @ccnicholls99,
I thought about this and I decided the best way is for you create your own code to setup users, which cover below. I only envisioned that Bulk Load code be used to setup the SuperUser when deploying a new application and I think is the best way. That's because the user part of the Bulk Load feature won't work with all the ASP.NET Core authentication providers, while AddRolesPermissionsIfEmpty will always work.
My recommended approaches are to setup the SuperUser when deploying a new application and use AddRolesPermissionsIfEmpty to add the Roles, then use:
1. Manual setup of users - best for adding a few app admin users
You use the admin features to add users manually, either via the normal "Add User" pages etc. (see this article), or use the IAddNewUserManager feature, which is designed to handle any ASP.NET Core authentication provider (see Add New User adapter).
2. Manual invite users - best to add tenant users
There is a "invite a user" feature, where you can define what roles, tenants, etc. and then send invite to a new user. They use the invite log in, and they are ready to go.
Also consider using a "tenant admin" who is in charge of a tenant (see this section in the Administration article), which moves the admin burden to someone in the each tenant).
2. Create code to add users - best for adding lots of users
If you want to add lots users, e.g. adding all the people in a company using the AzureAD  authentication provider, then use the AuthUsersAdminService directly to add users. You will be creating the same type of user Bulk Load data, but your code will be focus on your specific needs.


load init_net.pb and predict_net.pb into same graph for caffe2

I'm trying to view the caffe2 quantized mobilenetv2 model from this link:
https://github.com/caffe2/models/tree/master/mobilenet_v2_quantized
Is there a way to load in the predict_net.pb and init_net.pb into the same viewer?
https://caffe2.ai/docs/zoo.html

Loading up a pre-trained model to do things like predictions such as object detection is very simple in Caffe2. You need two files: 1) a protobuf that defines the network, and 2) a protobuf that has all of the network weights. The first is generally referred to as the predict_net and the second the init_net.

I can load the two files separately without error.  Both are in my downloads folder on Windows 10.  However, the init_net.pb doesn't appear to be loading when I open predict_net.pb, as all the weight and bias tensor values are empty.  I'm using the 2.7.7 build installed today.

oh, I see, so even some init_net.pb values, from that file loaded alone, are going to be incorrect, since your pull is fixing some new data fill cases.
I haven't tried to use caffe2 protobuf before, but I see they also support int64 and double_data storage types.  Maybe those would need to be handled as well.
https://github.com/pytorch/pytorch/blob/master/caffe2/proto/caffe2.proto

ok, I see you already handle int64, also it looks like this should be opening the init_net.pb,  so maybe some error is occurring, and I'm just not seeing it. I'm attaching a screen capture.

            caffe2.Metadata.open(host, (err, metadata) => {
                context.request('init_net.pb', null, (err, data) => {
                    var init = null;
                    if (!err && data) {

2.7.7 doesn't include 6ee3620. Are you running from TIP?
git clone https://github.com/lutzroeder/netron.git
cd netron
npm start

Yes, the changes fix the load of the filter and bias values.  If the Y_zero_point parameters are the quantization offsets, they seem believable.  If the Y_scale values are supposed to be the quantization scale values, then they probably are not correct, since values are 0.  Thanks for the quick fix.


[🐛 Bug]: Cannot access to non-public env variables in .env.local from RSC

next-on-pages environment related information
No response
Description
I have 3 .env files,
.env.local
.env.production
.env.development
I have some sensitive variables defined in .env.local however after building my app they become undefined in RSC but on dev mode I have access to them, So I assume that it might be related to next-on-pages,
Reproduction
add something like RECAPTCHA_SECRET_KEY to .env.local and build app using next-on-pages and run it and log it in a RSC. it's undefined.
Pages Deployment Method
Pages CI (GitHub/GitLab integration)
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[ ] Would you like to help fixing this bug?

Hi @anonymouscatcher 👋 , thanks for the issue 🙂
the Vercel CLI (that we use in the build process) doesn't seem to handle different env files very well,
as it always uses the production variant, as you can see here: https://github.com/cloudflare/next-on-pages/issues/161
I think this is the same issue you're facing?

Not sure if this is same, the values which are public are defined after building the app, only the secret variables are undefined

@anonymouscatcher could you share a minimal reproduction? 🙂

I'm closing this issue in favour of just keeping #604
(as I suspect the most sensible approach here is to improve our documentation rather than making assumptions and generate files for users when they might not want or expect that)


upload SBOM to TMA

per: https://github.com/github/package-security/issues/1355
able to generate intoto format for sbom and upload to tma
still need to work on the tests

Going to split this capability into a separate action


video_url is not returning with its value from getMediaByUrl method

Hi everyone,
I guess, Instagram is changing some requests and responses.
Yesterday, while I am working with this scraper everything was fine and working.
But today it has got some problems about getMedia method(s).
When you wanted to request with getMediaByUrl for a video post or sidecar posts including videos, it returns like below;
Response includes like that:
[video_url] => https://static.cdninstagram.com/rsrc.php/null.jpg
I checked getMediaByUrl($mediaUrl) method and print the JSON response and it seems like that.
This problems have got only includes the video post or sidecar video posts.
I am working from localhost and not using with Instagram::withCredentials() instance.
Has anybody got this issue like that?
Best Regards

I have the same problem

@zaivst yes I saw your issue. I wrote my opinions under the your issue.


Add utility class for building responses

Add utility class for building responses for the game client to avoid the use of creating and injecting hash maps, which make difficult to understand how the responses are built.
Responses are sent in a very similar format to JSON, with some exceptions:

Strings don't include " at the beginning, neither at the end
Colons (:) are substituted by equals (=)

Example:
Take the following response, which is used to tell the game client what is the Shared Object name, as an example:
{soName=nombredelso, call={room=lovy}}

No longer needed, since it is possible to use ObjectMapper from Jackson XML.


add sbomqs to dracon

Expected behaviour
sbomqs enables checking built-SBOM for compliance with NTIA-Minimum-Elements, some common sense rules and SCVS2 (coming soon). The results are in JSON and can be consumed by variety of tools.
It would be great to see dracon support integration with sbomqs for the SBOMs produced within the pipelines.

Hi, this is a good idea, we will work on this!


feat(mockUi): add initial UI for create/update/delete mock data

https://user-images.githubusercontent.com/6326478/154800775-4eec327e-69b4-4dd5-bc82-c71dfbbeaef1.mp4
Adding very basic UI for managing mock. The mock dashboard is hosted under apps/web and the actual UI is embedded in iframe. They communicate through window.postMessage API.
This UI will compare local mock JSON file data against recorded API responses and show the status of the mock accordingly. (inSync, outOfSync, notReady etc...)
Using File System API to access filesystem from browser.
potentially closes #8

inSync, outOfSync, notReady etc...

this makes me really want to incorporate xstate just for fun ^^

inSync, outOfSync, notReady etc...

this makes me really want to incorporate xstate just for fun ^^ (never used this before, just been following it on twitter)

Sure, I'll use it from next PR :)

I got this error when I click on "update mock directory", is this expected?

Abort directory picker is not handled yet. We can do it in this PR or in a future one, not a blocker

Some early discussion about mocks and application location.
Do we want to give the users the option to select where they want to save the mocks? I think we can make it simpler and predictable for all users if they were in a fixed path.
It could be in web/.mocks/ or in the future when bleublanc is standalone-able in the same way as Cypress, it could be in a /bleublanc/mocks folder potentially.

Currently it only records the mock without hijacking the response back to sample-ui right?

Abort directory picker is not handled yet. We can do it in this PR or in a future one, not a blocker.

Handled!

Some early discussion about mocks and application location.
Do we want to give the users the option to select where they want to save the mocks? I think we can make it simpler and predictable for all users if they were in a fixed path.
It could be in web/.mocks/ or in the future when bleublanc is standalone-able in the same way as Cypress, it could be in a /bleublanc/mocks folder potentially.

In case of cypress I think mock file location is defined in fixturesFolder in cypress.json, I'm wondering if we can refer to that value somehow. Let's see :)

Currently it only records the mock without hijacking the response back to sample-ui right?

It actually does in here

Oh but you need to explicitly map the mocks to the endpoint right. Btw after taking a look at the actual POC, I think hijacking the response might not be required, if we let the users use it themselves in cy.intercept. What bleublanc can help here maybe is to provide the code snippet for cy.intercept with the correct mock path for all the APIs that are recorded.


[AGORA-1729] Emit User Id with all API Calls

PR-Codex overview
This PR adds OpenTelemetry instrumentation to server components, enhances authentication with user ID, and improves logging with span attributes.
Detailed summary

Added OpenTelemetry instrumentation to server components in next.config.js
Enhanced authentication with user ID in src/app/api/v1/delegates/route.ts
Improved logging by setting span attributes in src/app/lib/logging.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

If you add the full ticket number (AGORA-...) Linear will autolink it to the actual issue

If you add the full ticket number (AGORA-...) Linear will autolink it to the actual issue

sorry! might've liked it to the wrong issue


[NEW] Create channel

Closes https://github.com/RocketChat/Rocket.Chat.Android/issues/857 and https://github.com/RocketChat/Rocket.Chat.Android/pull/891
I'd like to say thanks to @aniketsingh03 for his (great) idea of using chips to show the selected member.
In this PR I'm removing the usage of the Float Action Button (action is "called" from the nav drawer now) and using a fragment instead of activities (users will be able to add the members from the same view which it is created a channel (respecting the user experience) - also after create a channel successfully we'll show the chat list with the recent created channel).
ATTENTION: There are some bugs with our API that returns some users with null usernames and that does not work when using the dot (.) character in the query. For more info see https://github.com/RocketChat/Rocket.Chat/issues/11091  and https://github.com/RocketChat/Rocket.Chat/issues/11092
Depends on https://github.com/RocketChat/Rocket.Chat.Kotlin.SDK/pull/178
Preview:

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: aniketsingh03:x: filipedelimabritoYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: aniketsingh03:x: filipedelimabritoYou have signed the CLA already but the status is still pending? Let us recheck it.

you guys should be using the spotlight endpoint to search users, just like the web client does, this way we'll have the same results on both clients

@sampaiodiego I think we'll change to use the directory API, how about that? We're using on iOS already.

@rafaelks I think using directory endpoint is fine. they're almost the same on the implementation.

@rafaelks @sampaiodiego Ok, we have that flexibility. But I'll end up using the spotlight endpoint that is already present in our SDK.
Android is hugging the web client. But don't feel alone on the iOS side. I'm sorry @rafaelks 😁

Oops, is the web client moving to directory too? Then I think that is better to user that API, so the web client and all mobile client will be using the same, right?
Give me some advice 🙏

@filipedelimabrito spotlight has limited resources (returns 5 results maximum) and I don't think we should be using for this situation.

@rafaelks Got it! Thanks for your advice.
But before this discussion about what API should we use, I was searching for the maximum of 5 users (See here)
Because we are using a small view to show the results (see on the image on this PR description), then I think that's fine in this case.
Do you agree? Let me know.

@filipedelimabrito Yes, the way you're doing today on Android makes sense to keep on 5. 👍

just to clarify, I wrongly thought the web client was using spotlight but it is not.. on the web we're using a subscription called userAutocomplete .. 🤷♂


chore(deps-dev): downgrade conventional-changelog-conventionalcommits

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.89%. Comparing base (d2db3d8) to head (6544081).
Report is 2 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next     #114   +/-   ##
=======================================
  Coverage   95.89%   95.89%
=======================================
  Files           3        3
  Lines          73       73
  Branches        7        7
=======================================
  Hits           70       70
  Misses          3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: run duplicate pre-hooks just once

With https://github.com/erc6900/reference-implementation/pull/29, which removes the ability for plugins to depend on other plugin's hooks, the only cases where there will be duplicate hooks applied to a selector are:

When a plugin's manifest incorrectly applies the same pre-hook twice to a selector.
When different plugins apply ManifestAssociatedFunctionType.PRE_HOOK_ALWAYS_DENY to the same selector.

This makes running duplicate pre-hooks multiple times unnecessary.
Note that duplicate post-hooks may still run multiple times if they are attached to different pre-hooks, since the pre-hook return data that are passed to them may be different. And since they will be configured from the same plugin, any duplicate runs will be intentional.

Edit: to expand, in cases where a plugin might want to apply {preHook: A, postHook: B} and {preHook: A: postHook: C} to a selector, AND they want A to be executed twice and not once, this can be addressed by collapsing the logic in B and C into a single post-hook.

Thanks for the simplification! Do you think if it's still worth of storing duplicated preHooks if they're only run once now? (It seems like HookGroup still records the count of dupes)

@huaweigu Yes, but it can be simplified further to reduce costs. There is still the possibility of the magic values like PRE_HOOK_ALWAYS_DENY being set multiple times for a given function, so we need to keep track of the number of times it has been set to facilitate proper uninstalls.

Thanks for the simplification! Do you think if it's still worth of storing duplicated preHooks if they're only run once now? (It seems like HookGroup still records the count of dupes)

@huaweigu Yes, but it can be simplified further to reduce costs. There is still the possibility of the magic values like PRE_HOOK_ALWAYS_DENY being set multiple times for a given function, so we need to keep track of the number of times it has been set to facilitate proper uninstalls.

Thanks, I was thinking even if PRE_HOOK_ALWAYS_DENY being applied to the same selector multiple times, the result from running them would be the same(please correct me if I'm missing anything)? If that's the case, could we consider just installing/storing one preHook and uninstalling it later?

Thanks for the simplification! Do you think if it's still worth of storing duplicated preHooks if they're only run once now? (It seems like HookGroup still records the count of dupes)

@huaweigu Yes, but it can be simplified further to reduce costs. There is still the possibility of the magic values like PRE_HOOK_ALWAYS_DENY being set multiple times for a given function, so we need to keep track of the number of times it has been set to facilitate proper uninstalls.

Thanks, I was thinking even if PRE_HOOK_ALWAYS_DENY being applied to the same selector multiple times, the result from running them would be the same(please correct me if I'm missing anything)? If that's the case, could we consider just installing/storing one preHook and uninstalling it later?

Correct, the result of running them would be the same. When uninstalling though, you'd want to make sure that the plugin you're uninstalling applied the last instance of that hook to that function before removing it. If two plugins applied that hook, and one of them is uninstalled, you'd still want to keep that hook applied instead of removing it.

Thanks for the simplification! Do you think if it's still worth of storing duplicated preHooks if they're only run once now? (It seems like HookGroup still records the count of dupes)

@huaweigu Yes, but it can be simplified further to reduce costs. There is still the possibility of the magic values like PRE_HOOK_ALWAYS_DENY being set multiple times for a given function, so we need to keep track of the number of times it has been set to facilitate proper uninstalls.

Thanks, I was thinking even if PRE_HOOK_ALWAYS_DENY being applied to the same selector multiple times, the result from running them would be the same(please correct me if I'm missing anything)? If that's the case, could we consider just installing/storing one preHook and uninstalling it later?

Correct, the result of running them would be the same. When uninstalling though, you'd want to make sure that the plugin you're uninstalling applied the last instance of that hook to that function before removing it. If two plugins applied that hook, and one of them is uninstalled, you'd still want to keep that hook applied instead of removing it.

Thank you Jay for your explanation. It makes sense for PRE_HOOK_ALWAYS_DENY since plugin address doesn't reallly play a role here in the hook resolving process. If it's just for this special case, I feel like maybe there's some optimization we can come up to cut a potential sstore.


[DNM] test:test ci

What problem does this PR solve?
test ci in order to accelerate

 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.

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests


Booze

Previously:

5l of gin
tonic for the gin
10kg of ice
4l of rum (different ones, mid-tier)
a lot of beer (more than 50l I think)
a lot of ciders (more than 20l I think)

@curusarn do you want to take care of this again?

Yeah 👍

Something could be ordered through kosik.cz.
Buying hard booze ahead of time and ordering tonic, beer, and cider should work.

GT2.0:

24.-27.9. -> 4 days (3 nights)
18 people

GT3.0:

2.-10.9. -> 9 days (8 nights)
people

wants to go: 25
reality: ???

GT3.0 requires 2.2 - 2.5 more booze than GT2.0

Latest summary:
coz vlastne znamena
* ginger beer
* kafe - to stejne budeme jeste resit nejak
* vodka
* limetky
* (cukr)
* (khalua)

s tim ze pocitam s:
* rum
* gin
* tonic
* led

pokud budeme mit i nejakej dzus tak s tim uz namichas fakt hodne veci (sours, mules)


Feature/add default template parameters to web response

WebResponse does not know about additional options, which exist in GenericRestClient

export interface ApiCallOptions extends WebRequestOptions {
   backendUrl?: string;
   excludeEndpointUrl?: boolean;
   eTag?: string;
}

https://codesandbox.io/s/v6jyl7n1m0

@a-tarasyuk - thanks for making the requested changes. We need to take a bit of time to validate these changes upstream in our client that heavily uses SimpleRestClients - it's a good canary to ensure nothing regressed (typings or behaviour)


New questions to Github Actions

PR Type

[ ] Adding new question(s)

Warning: We do not support the inclusion of questions directly copied from official GitHub certification exams. Please only submit original questions and content that you have created.

[ ] Other content changes (updating questions, answers, explanations or study resources)
[ ] Code changes (non-content)
[ ] Documentation changes
[ ] Other

What's new?

Related issue(s)

Screenshots

Hi @FidelusAleksander ,
I am planning to take GitHub Actions Certification exam and learning it intensively, I hope if you don't mind if I bring more questions and topics here as part of my learning and contributing to this great project

Hi @FidelusAleksander ,
I am planning to take GitHub Actions Certification exam and learning it intensively, I hope that you don't mind if I bring more questions and topics here as part of my learning and contributing to this great project

Go ahead! That's the whole idea behind the project 😎🚀

Hi @FidelusAleksander ,
I am planning to take GitHub Actions Certification exam and learning it intensively, I hope that you don't mind if I bring more questions and topics here as part of my learning and contributing to this great project

Go ahead! That's the whole idea behind the project 😎🚀


Update app-service-mobile-xamarin-forms-get-started-push.md

Added line missing, The reason last commit. Need to Merge

@ktoliver, Still my PR waiting for review.

@elamalani Could you review the proposed changes? Thanks.

@ktoliver Kristine Toliver (Aquent LLC) Vendor, Still my PR waiting for review.

Hi @susairajs - The pull request has been assigned to the listed author, who needs to review and approve the PR. Thank you.


Change variant name from plain-dangerous

Description
The word dangerous in the plain-dangerous variant of VButton gives the idea that the button would be used for destructive actions and not that its use is discouraged.
See an example of the confusion here: https://github.com/WordPress/openverse-frontend/pull/1142#discussion_r834307513
Goal

The intent of developers not using this variant can be conveyed more clearly by using synonyms for "danger" e.g. "beware" and "careful" and my favourite "avoid". I recommend renaming the variant to "plain--avoid". The "--avoid" suffix can add that little bit of friction to discourage it's usage while not confusing with the "danger" convention we use for destructive actions.
Resolution

[ ] 🙋 I would be interested in resolving this bug.

Hey, I just went over to the discussion. It would be better to update the variant to variant= "plain--avoid" . Can I take up the issue?

Sure @shreyanshmalvya! I'll assign it to you.


With version 3.1.0, the HintManagerHolder.clear method is executed each time SQL is executed?

With version 3.1.0, the HintManagerHolder.clear method is executed each time SQL is executed?
When using HintShardingAlgorithm, you need to set HintManagerHolder before executing SQL. I execute multiple SQL in Dao, or execute multiple Dao methods in a service. I want to set HintManagerHolder in the form of AOP interception before calling the method. Execute clear after the end, there is no way to achieve

@cherrylzhao just check it is duplicated or not

we have enhanced it in 4.0.0-RC1, you can see #2164

4.0 When is the official version issued?

maybe within 2 months.


Drop PHP 5.3 support

This depends on how many people are still using PHP 5.3. PHP 5.3 is EOL or really near EOL.

CakePHP 3.x requires at least PHP 5.4.16 or greater. (http://book.cakephp.org/3.0/en/installation.html)

Apps that won't upgrade to 5.4 should have already started a refactor process. Software should never get to EOL terms, especially web applications. People will stay on 5.2 and 5.3 until they are forced to move. I wish all these shared hosting services would force that.

In the year+ since I last commented, we have been able to move to PHP 5.6. Even if that hadn't happened, though, I would have been totally fine with removing PHP 5.3 support.

Cool, Good to know!

On 28 Aug 2015, at 1:47 pm, Joel Boonstra<EMAIL_ADDRESS>wrote:
In the year+ since I last commented, we have been able to move to PHP 5.6. Even if that hadn't happened, though, I would have been totally fine with removing PHP 5.3 support.
—
Reply to this email directly or view it on GitHub.

FYI - Nearly 90% of users voted to drop PHP 5.3 support in this year's Phinx product survey. Therefore instead of waiting for 2.0.0 I'm going to drop PHP 5.3 support in 0.6.0. At the moment I'm contemplating whether to skip 5.4 also, but we need to take into account dependant projects such as CakePHP.

https://github.com/cakephp/cakephp/issues/5602

closing this as we no longer support 5.3. Will open another issue to drop 5.4 later this year.


[Feature] Collapsible mod list

Hi, I'm trying to play modded HK on the Steam Deck, which is working great so far.
However, because of the smaller screen the list of installed mods when in the pause screen overlays a lot of the other mod UIs like benchwarp.
It would be great if there were some option to collapse the mod list so it only shows some info text to expand that list again in the top left.
I would like to even contribute this myself, however I was not able to find out where this mod list is implemented (if this is even the correct repository for this)

It's implemented at ModVersionDraw and the calls using it in ModLoader fwiw.

HideModList on your favorite mod installer implements this, along with provisions that make it so that SRC verifiers can be certain it's not being used to sneak banned mods into speedruns

Ok... somehow I wasn't able to find that mod at all before because in all the posts people always complained to questions about the poster "wanting to hide mods for fake speedruns" etc.


Opotion to hide the card when no packages to track

Can we have an option to hide the card when there's no packages to track? Currently the card displays "no packages to track at the moment" which imo adds unnecessary clutter to my dashboard.
Maybe something like "hide_when_no_packages" or "hide_when_empty"
which when set to "true" would hide the card when you're not expecting for any packages.

Hey! As this is something already achievable with Home Assistant's built-in conditional card, I'm not keen on adding the same functionality to the card itself. You do need to create a template sensor, but with that it's quite simple. See the example below.
Template sensor definition for getting the total number of packages:
{{ state_attr("sensor.matkahuolto_XXXXXXXX", "packages")|count + state_attr("sensor.posti_XXXXXXXX", "packages")|count }}
Conditional card configuration:
type: conditional
conditions:
  - condition: numeric_state
    entity: sensor.lahetysten_kokonaismaara
    above: 0
card:
  type: custom:package-tracker-card
  entity:
    - sensor.posti_XXXXXXXXXX
    - sensor.matkahuolto_XXXXXXXXXXX
  max_events: 3
  title: Lähetykset
  show_destination: false


Turn off logging while unit testing

Hi.
I'm running unit tests, I was unable to turn off logging.
Here is similar issue: https://github.com/nestjs/nest/issues/3246
I found that for regular run we can do the following:
  const app = await NestFactory.create(AppModule, {
    logger: ['error', 'warn'],
  });

but for unit tests, we cannot do so.
Maybe it's worth adding a logger to option to DrivineModuleOptions ?

Verified that drivine correctly honors log_level.

@jasperblues "Verified that drivine correctly honors log_level" what does it mean ?
how we should turn of logging for tests vs production vs development ?
how you doing it in your application?

https://docs.nestjs.com/techniques/logger
^-- Just using NestJS facilities @myflowpl for test vs production and other logging concerns @myflowpl. I showed @alexgarbarev and I believe it Is working well for him now too.

For tests, I'm using EmptyLogger
import { LoggerService } from '@nestjs/common';
export class EmptyLogger implements LoggerService {
  log(message: string): any {}
  error(message: string, trace: string): any {}
  warn(message: string): any {}
  debug(message: string): any {}
  verbose(message: string): any {}
}

Then in test:
const app = await Test.createTestingModule(module).compile();
app.useLogger(new EmptyLogger());

@jasperblues I'v tried it all
Nest recommends to use Dependency Injection for injecting the  logger, in order to overwrite it in unit tests, but Drivine is not using the DI for it:
https://github.com/liberation-data/drivine/blob/master/src/logger/StatementLogger.ts
import { Logger } from '@nestjs/common';
import { QuerySpecification } from '@/query/QuerySpecification';

export class StatementLogger {
    private logger = new Logger(StatementLogger.name);// <<<--- should be injected

    constructor(readonly sessionId: string) {}

    log(query: QuerySpecification<any>, hrStart: [number, number]): void {
        const hrEnd = process.hrtime(hrStart);
        const uSec = Math.round(hrEnd[1] / 1000);
        this.logger.verbose({
            ...query,
            sessionId: this.sessionId,
            elapsed: `${uSec} µsec`
        });
    }
}

Nest docs recomends something like this:
constructor(private myLogger: MyLogger) {
    this.myLogger.setContext('CatsService');
  }

That's why I can't disable it for tests
I'v tried the static method to disable logLevel, and it is not working:
import { Logger } from '@nestjs/common';
Logger.overrideLogger(['error'])

Thanks for your help.
Drivine is so cool, good work !!!

ok, it looks like Drivine should have @nestjs/common and @nestjs/core as peerDependencies
now it is placed in dependencies:
"dependencies": {
        "@nestjs/common": "^6.8.3",
        "@nestjs/core": "^6.8.3",
        "@nestjs/platform-express": "^6.8.3",
}

so if you use divine in project where you have the nestjs version 6,  then the static method will work
in my case I'm using nestjs version 7, so static method Logger.overrideLogger(['error']) will not work, be course drivine and my project will receive different references

@jasperblues do you use nestjs v7 or v6 in your applications ??

Up until now NestJS 6, but I prefer to stay on the latest versions, and recommend that in general.
If we set it up so that it is easy for all users that is preference.
Second preference is to make it easy/default for 7 and possible for 6.
Do you have a suggestion?

@jasperblues do you use nestjs v7 or v6 in your applications ??

Up until now NestJS 6, but I prefer to stay on the latest versions, and recommend that in general.
If we set it up so that it is easy for all users that is preference.
Second preference is to make it easy/default for 7 and possible for 6.
Do you have a suggestion?

@jasperblues  yes, quickest fix would be to move the nestjs inside package.json  from dependencies section to peerDependencies section.
Even without updating the nest version to 7, but I recommend to update the nest version to latest and run all the tests to see if it's working
In this case, all projects using v6 will work with no problems
and all projects using v7 will also work, but only with smal warning during npm install saying that drivine pear deps are wrong, but it will work without problems on run time.
package.json NOW
    "peerDependencies": {},
    "dependencies": {
        "@liberation-data/agensgraph": "^0.1.0-rc",
        "@nestjs/common": "^6.8.3",
        "@nestjs/core": "^6.8.3",
        "@nestjs/platform-express": "^6.8.3",
        "class-transformer": "^0.2.3",
       ...
    },

AFTER FIX
    "peerDependencies": {
        "@nestjs/common": "^6.8.3",
        "@nestjs/core": "^6.8.3",
        "@nestjs/platform-express": "^6.8.3"
    },
    "dependencies": {
        "@liberation-data/agensgraph": "^0.1.0-rc",
        "class-transformer": "^0.2.3",
       ...
    },

Alright, let's do that. Will do asap.
Or if you're so inclined, feel free to send a PR!

@jasperblues yes, I can do PR for this one, and I also found some other issues so I'll add them and if possible also do PR.
I'll do it in next 2 days, while I'm developing my app i'll also test the solution.
May be you can reopen this issue or should I post new one ?

Thank you Piotr!

@myflowpl Btw, as you're among the first contributors, along with Khoak, Aleksey and Jeet, you're welcome to be featured on the website. If you'd like that pls send photo and few sentence bio to<EMAIL_ADDRESS>

@jasperblues thanks Jasper, that is very nice of you, lets do that after my PR's will be merged ;)
I'm just migrating from OrientDB to Neo4j, and Drivine draw my attention while I was looking for solution in Node.js, so far it's so cool, and only found minor bugs.
Today I'm trying to do my first deployment do testing servers with Neo4j+drivine, and hope to have time tomorrow to do some work on drivine
Good work :)

I have been struggling with this for several hours now on a very simple jest unit test, and I can't believe how massively frustrating it is.
Tried mocking, spying on console, global console, replacing the property, mocking, spying on Logger, Logger.overrideLogger, nothing works.
I guess I might be venting out a bit, and I might find a solution in the next minute.
But this should not be so hard. Feels like a massive time waste. Makes me sad to the core.
Will report back if I find any meaningful/minimally elegant solution.

Did you set the DRIVINE_LOG_LEVEL? Here's an example if you're configuring
via an env file:
https://github.com/liberation-data/drivine-inspiration/blob/master/.env.sample
Drivine has its own logger so you can turn it off and on. The issues you're
reading are stale threads in relation to why people wanted that.

Message ID: @.***>

hey @jasperblues I had to vent off but in my frustration I didn't notice that the link I followed on NestJS repo actually took me to drivine. My issue was with NestJS not drivine. Thanks for the response anyway!!


test: SelectARelatedEntityInALookup failing

This test is currently being ignored but is failing due to an EasyRepro issue.
Please can we:

Raise with EasyRepro
Attempt to resolve the issue in our codebase
Contribute back the solution to EasyRepro

This bug has also been fixed. I ran the test and all works fine.


Scan() doesn't work if used with Where()

I'm on v1.1.1.
Using the code:
var playersIDs []uint64

err = tx.NewSelect().Column("id").Model((*Player)(nil)).Where("?TableAlias.team_id = ?", id).Scan(ctx, &playersIDs)

playersIDs is nil.
Using the same code without .Where() it works!
err = tx.NewSelect().Column("id").Model((*Player)(nil)).Scan(ctx, &playersIDs)

Both the queries work on SQL: records are there!

I was totally wrong. Sorry.


Implement list_volumes() API for VMAX driver

This PR implements list_volumes() API for the Dell EMC VMAX driver.
Functionality is verified using VMAX250F backed.
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

LGTM

Codecov Report

Merging #70 into master will decrease coverage by 0.76%.
The diff coverage is 4.00%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   29.98%   29.21%   -0.77%
==========================================
  Files          59       59
  Lines        3739     3710      -29
  Branches      414      416       +2
==========================================
- Hits         1121     1084      -37
- Misses       2601     2609       +8
  Partials       17       17

Impacted Files
Coverage Δ

dolphin/drivers/api.py
37.25% <0.00%> (-0.75%)
:arrow_down:

dolphin/drivers/dell_emc/vmax/vmax.py
47.05% <0.00%> (ø)

dolphin/drivers/dell_emc/vmax/client.py
19.27% <4.54%> (-5.32%)
:arrow_down:

dolphin/drivers/driver.py
56.52% <0.00%> (-11.23%)
:arrow_down:

dolphin/wsgi/common.py
53.57% <0.00%> (-4.50%)
:arrow_down:

dolphin/api/validation/validators.py
21.60% <0.00%> (-2.44%)
:arrow_down:

dolphin/api/validation/__init__.py
72.72% <0.00%> (-2.28%)
:arrow_down:

dolphin/api/v1/storages.py
40.35% <0.00%> (-2.03%)
:arrow_down:

dolphin/api/common/wsgi.py
22.60% <0.00%> (-1.09%)
:arrow_down:

dolphin/api/common/__init__.py
28.35% <0.00%> (-1.06%)
:arrow_down:

... and 12 more

LGTM


dependabot incorrectly reported already fixed security issue

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
github-actions
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
No response
dependabot.yml content
No response
Updated dependency
No response
What you expected to see, versus what you actually saw
In the weekly "GitHub security alert digest" (aka "Your Dependabot alerts for the week") email, I got notified that some of the repos I manage were affected by a security issue in actions/download-artifact:

I checked the security issue and noticed that it says that "No security update is needed":

It also suggests to just use @v4 which is what we are already using.
Initially I thought that the report was created while the latest 4.x.x release was vulnerable, and that it was left open after a new release that fixed the issue came out, in order for the maintainers to review and dismiss the alert manually.
However<EMAIL_ADDRESS>was released ~5 months ago, on Apr 24th.
Upon further investigation, I noticed that even though the issue has been fixed long ago, the security report was published by @joshmgross just last week:

https://github.com/actions/download-artifact/security/advisories/GHSA-cxww-7g56-2vh6

So it would appear that when Josh submitted the security report, @dependabot erroneously detected that my repos were using an affected version (maybe it thought that 'v4' < 'v4.1.17' or that all versions were affected until a new release with a fix came out?) and created the security issues.

In addition, I also found this somewhat confusing log in a failed GitHub action run:
Click to see the logs

updater | 2024/09/04 14:46:19 INFO <job_879335831> Latest version is 4
updater | 2024/09/04 14:46:19 INFO <job_879335831> no security update needed as actions/download-artifact is no longer vulnerable
  proxy | 2024/09/04 14:46:19 [038] POST /update_jobs/879335831/record_update_job_error
  proxy | 2024/09/04 14:46:19 [038] 204 /update_jobs/879335831/record_update_job_error
  proxy | 2024/09/04 14:46:19 [040] PATCH /update_jobs/879335831/mark_as_processed
  proxy | 2024/09/04 14:46:19 [040] 204 /update_jobs/879335831/mark_as_processed
updater | 2024/09/04 14:46:19 INFO <job_879335831> Finished job processing
updater | 2024/09/04 14:46:19 INFO Results:
Dependabot encountered '1' error(s) during execution, please check the logs for more details.
+----------------------------+
|           Errors           |
+----------------------------+
| security_update_not_needed |
+----------------------------+
Error: Dependabot encountered an error performing the update

I think @dependabot was trying to create a PR to update actions/download-artifact but it realized that it was not necessary and "failed" with the "security_update_not_needed" error.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

I'm encountering the same issue. After merging the Dependabot PR, the security alert remains open, stating, "No security update is needed as ..... is no longer vulnerable."


Backport missing webhook attributes

I'm still using version from git from https://github.com/dkd/paymill-ruby
because of https://github.com/dkd/paymill-ruby/commit/3f18fd706909aad1b994a4c8d56ab2f085657e62
Can you please backport this commit?

@nikoloff I can open PR with this backport if you willing to merge it.

Hi @morgoth
we have the attributes app_id and active in this version. Are you using paymill_ruby gem?

Yes, it's there - I was searching by commit history and not code. Thanks.


Add ability to hide not planned tests

A fair share of tests are not planned for specific release (~190 for 2024.2) and as such are not planned, however, they still show at the list, which creates visual clutter and can be severely confusing for a newcomer (like me).
I propose adding an option to hide not planned tests and groups that contain only not planned test either as user option or a general button on a site

@pehala
you are referring to "not planned" ones ?

Yes, sorry for the confusion, I updated name and the description


Implement JWT system.

In GitLab by @doddzilla on Sep 10, 2019, 21:52
See the Users & Tokens chapter of the guide.

Done.


Enable profiling of project libraries and executables

I need a way to build and run an executable with profiling, and currently can't because the profiling libraries aren't being installed for my package dependencies.
E.g.:
nix-shell -A shells.ghc --run "cabal new-build <executable_to_profile> --enable-profiling --enable-library-profiling"

...

src/Model/User.hs:12:1: error:
    Failed to load interface for ‘Control.Lens’
    Perhaps you haven't installed the profiling libraries for package ‘lens-4.15.4’?
    Use -v to see a list of the files searched for.

Also, there should be better documentation on what passing profiling = true to obelisk's default.nix actually does, because it didn't seem to do anything for the project build.

Okay, just tried this again and was able to get it working. I probably wasn't actually passing the profiling parameter to obelisk before.
In default.nix:
{ profiling ? false }:
  (import ./.obelisk/impl { inherit profiling; }).project ./. ({ pkgs, ... }: { ... }

In backend.cabal:
executable backend-profile
  main-is:             ProfileMain.hs
  hs-source-dirs:      app
  build-depends:       base
                     , backend
                     , common
  default-language:    Haskell2010
  ghc-options:         -fprof-auto "-with-rtsopts=-p -s -h -i0.1"

And then this should just work:
nix-shell -A shells.ghc --arg profiling true --run "cabal new-run exe:backend-profile --enable-profiling"


Does dotnet restore not support authenticated (private) feeds at all

For more detail: https://inedo.com/support/questions/7319

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

ID: 0b256dc4-c4ad-c115-f4e2-40310bc3505a
Version Independent ID: 0b993779-65a2-6092-4047-2fc2ec16170b
Content: dotnet restore command - .NET Core CLI
Content Source: docs/core/tools/dotnet-restore.md
Service: unspecified
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

@dsplaisted @KathleenDollard @livarcocc I believe you can specify private feeds via source no? But how does it work with authenticated private feeds?

@rohit21agrawal @rrelyea

@kexxxfeng
This is something we're actively working on.
Please follow https://github.com/NuGet/Home/issues/6486 for updates.

Closing as duplicate of https://github.com/NuGet/Home/issues/6486


Disable horizontal bouncing?

Is there a way to disable swiping past the left/right open value to disallow any white space from appearing? Currently I am able to continue swiping the entire row while my open value is only 75.

Hey @mmckinley8, stopLeftSwipe and stopRightSwipe props should do the trick for you

Closing as inactive, please reopen if you still have issues


wrong info related to managed identity

Managed identity does not utilize environmental variables. when running your code inside an instance which utilize managed identity it should invoke web request to something like http://<IP_ADDRESS>/metadata/identity/oauth2/token?api-version=2018-02-01&resource=https://management.azure.com/

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

ID: f486180b-8851-6bf0-c3a6-8d3823312c23
Version Independent ID: f486180b-8851-6bf0-c3a6-8d3823312c23
Content: Authenticating Azure-hosted apps to Azure resources with the Azure SDK for Python
Content Source: articles/python/sdk/authentication-azure-hosted-apps.md
Product: azure-python
GitHub Login: @diberry
Microsoft Alias: diberry

@scottaddie I'll defer to you on this. Whatever you decide should be across all lang-MI docs.

@rufathpe Are you referring to the following sentence at the bottom of the doc?

When the above code is run on your local workstation during local development, it will look in the environment variables...

In this context, "it" refers to the DefaultAzureCredential. @diberry, Can you please update the doc accordingly?

Content merged - will publish later today.


File is damaged and cant be opened M3 Pro Macos Sonoma

tried the last 3 versions, none of them worked and show as damaged

Hiya @DerpcatMusic! sorry for the late response.
Yeah this issue happens because Apple has increased restriction with Gatekeeper on the enwer versions of MacOS. Luckily, there is a way to still make it work.

Open Terminal:

You can open Terminal from the Applications > Utilities folder, or press Cmd + Space, type "Terminal," and hit Enter.

Navigate to the App Location:

Use the cd command to go to the folder where your app is located. For example, if the app is in the Downloads folder, you would type:cd ~/Downloads

Remove the Quarantine Attribute:

Run the following command, replacing Comet.app with the actual name of your app:xattr -d com.apple.quarantine Comet.app

This removes the quarantine flag that macOS automatically sets on downloaded applications, which is what blocks it from opening.

Open the App:

Now that the quarantine attribute has been removed, you should be able to open the app by double-clicking it in Finder.

Alternative: If You’re Using a Path
If your app is in a custom location or has spaces in the name, you may want to use the full path. Here’s an example:
xattr -d com.apple.quarantine "~/Downloads/Comet.app"

Verification
You can check if the quarantine attribute was successfully removed by running:
xattr Comet.app

If com.apple.quarantine no longer appears in the output, the quarantine flag has been successfully removed.
Let me know if you run into any more issues!

Thank you I really appreciate it! Would be great to add it in the document of comet incase it's not there :)
Best,
Daniel Elmaliah.
On Nov 2, 2024 at 8:06 PM +0200, Jerome Thayananthajothy @.***>, wrote:

Hiya @DerpcatMusic! sorry for the late response.
Yeah this issue happens because Apple has increased restriction with Gatekeeper on the enwer versions of MacOS. Luckily, there is a way to still make it work.

Open Terminal:
• You can open Terminal from the Applications > Utilities folder, or press Cmd + Space, type "Terminal," and hit Enter.

Navigate to the App Location:
• Use the cd command to go to the folder where your app is located. For example, if the app is in the Downloads folder, you would type:
cd ~/Downloads

Remove the Quarantine Attribute:
• Run the following command, replacing Comet.app with the actual name of your app:
xattr -d com.apple.quarantine Comet.app
• This removes the quarantine flag that macOS automatically sets on downloaded applications, which is what blocks it from opening.

Open the App:
• Now that the quarantine attribute has been removed, you should be able to open the app by double-clicking it in Finder.

Alternative: If You’re Using a Path
If your app is in a custom location or has spaces in the name, you may want to use the full path. Here’s an example:
xattr -d com.apple.quarantine "~/Downloads/Comet.app"
Verification
You can check if the quarantine attribute was successfully removed by running:
xattr Comet.app
If com.apple.quarantine no longer appears in the output, the quarantine flag has been successfully removed.
Let me know if you run into any more issues!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Type provider fails on empty tables

Description
The type provider becomes unusable for tables that contain no entries.
Repro steps
Create a table, add some entries
Use the type provider to great success
Remove all the entries
No more great success; methods like GetPartition disappear.
Expected behavior
If a table exists, I believe it has metadata available, describing what types each column is supposed to contain. I presume the Type Provider uses actual entries, and not the metadata, to generate types, would it be reasonable to use that as a default?
Actual behavior
Known workarounds
Keep a row in any existing table, no matter what?
Related information
I realize this is probably not a verycommon case - tables will typically contain data. Yet, in early design phase, it's not uncommon for me to run code, see how the data looks, then wipe out the data to change a couple of things.

Sadly, your assumption regarding table metadata isn't correct AFAIK. Azure Tables hold row level metadata only. There's no such thing as a table schema.
This is something I have on the backlog but need to define a schema definition etc.. - I've created #98 to track this.

That's interesting - I based my assumption on the fact that, in the Azure Storage Explorer, you can display columns, and some type information shows up there. So tables also infer types on rows then? Anyways, I trust that your knowledge on the matter is deeper than mine, and I think the approach you suggest (providing a sample/static file) sounds quite reasonable. Thank you! And this might make a good up-for-grabs topic, perhaps?

Indeed - #98 has the up-for-grabs tag :-)
Actually if someone wanted to just write the schema definition and a simple domain model for it (e.g. discriminated union for the different EDM property types etc.) that would be a massive help after which I can just integrate it with the TP.
We took this approach for the schema support for blobs and it worked well.


mkdocs: fix typo in title

This PR fixes a typo in the title of the contribution section in the documentation.

Thanks!


Glossary translation

Hello all, I need some help here, there are some words that I don't feel comfortable to translate, because I'm from México and we are use to use these words as its in english.
Some words are:
re-render
arrow function
template literals
controlled component
uncontrolled component
props
state
camelcase
lowercase
callback
event handler
I need your opinion on what would be a suitable translation for these words.
Related with #1

Of course, all these terms sound familiar to us and we may be tempted to let them as is. This is not always the best solution as it may create a conundrum out of the abuse of mixing the two languages.
Here are my proposals for the terms:

re-render -> re-renderizado
arrow function -> función flecha (This is a term I've seen several times (MDN and Wikipedia use it) and it is a well-formed translation)
template literals -> literales de cadena o plantillas de cadena (as MDN). The translation of string to cadena or cadena de texto is widely used in many technical docs as well  as the translation of template to plantilla.
controlled component -> componente controlado
uncontrolled component -> componente descontrolado
props -> props or propiedades
state -> estado
camelcase -> camelcase
lowercase -> minúscula
callback -> callback
event handler -> manejador de eventos (It is a translation I've seen used several times in technical documentation and it sounds natural in Spanish.)

Last, but not least. I know many people use librería as a translation of library. I already gave my opinion in one PR #3. I am goin to repeat it:
Although the use of «librería» as a translation for «library» is adopted by many people is also discouraged by others. I think the position of the Royal Academy on this matter should be considered.
Also, «biblioteca» is the preferred term by many official docs, check out:

https://developer.android.com/topic/libraries/support-library/?hl=es-419
https://developers.facebook.com/docs/reference/php?locale=es_ES

@dmoralesm how is this going? Could you respond to the changes people have requested?

Hey @carburo @alejandronanez, I updated this PR. Can you take a look? Thanks.

It still feels weird to me that we translate most of the technical terms to Spanish template literals or arrow functions for example, but if that's what everybody agreed on, I'm ok with that.
We can revisit after the first release and see what other teams are doing!
Sounds good @tesseralis @dmoralesm @carburo ?

Sounds good to me.

Haha, I don't really have a stake in what sounds better in Spanish, as long as you're following the glossary consistently.


How can I create a new empty document?

How can I create a new empty document? Thanks!

You can create empty org files from the first menu page by clicking on + icon. Local files will live inside your browser. Maybe we may add downloading that file into your local computer.


ModalBottomSheetRoute: useSafeArea puts gap below sheet

Steps to Reproduce

Execute flutter run on the code sample, choosing a recent iPhone with non-zero insets including a camera notch (such as an iPhone 14 Pro)
Rotate the device so the notch is on the left (90° counterclockwise)
Tap the first button, "Modal bottom sheet with useSafeArea: true"
Bug: useSafeArea correctly makes the sheet avoid the notch on the left, but it leaves a gap at the bottom, so the sheet doesn't open from the bottom.
Close the bottom sheet by tapping outside it, then tap the second button, "Modal bottom sheet with useSafeArea: false"
The gap at the bottom is gone, but now the sheet isn't protected from the notch on the left, and content is obscured

Screen recording:

Expected results: I want to use the API to render a bottom sheet so it doesn't extend into the top or horiztonal insets, but it's still a bottom sheet (i.e., it opens from the bottom edge of the screen, and the sheet itself consumes the bottom inset).
Actual results: It doesn't seem possible with the current API (see steps above).
Code sample: https://github.com/chrisbobbe/modal-bottom-sheet-use-safe-area-repro
Link to ModalBottomSheetRoute.useSafeArea doc: https://api.flutter.dev/flutter/material/ModalBottomSheetRoute/useSafeArea.html

Logs
(No crashes or exceptions expected or observed.)
$ flutter analyze
Analyzing modal_bottom_sheet_use_safe_area_repro...
No issues found! (ran in 2.0s)

$ flutter doctor -v
[✓] Flutter (Channel main, 3.8.0-17.0.pre.21, on macOS 13.0 22A380 darwin-x64, locale en-US)
    • Flutter version 3.8.0-17.0.pre.21 on channel main at /Users/chrisbobbe/.local/lib/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 06952ba254 (3 hours ago), 2023-02-27 19:13:00 +0000
    • Engine revision 4bba1b958e
    • Dart version 3.0.0 (build 3.0.0-276.0.dev)
    • DevTools version 2.22.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/chrisbobbe/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • ANDROID_HOME = /Users/chrisbobbe/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.75.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • Chris’s iPhone (mobile) • 00008110-000924383632801E            • ios            • iOS 16.1 20B82
    • iPhone 14 Pro (mobile)  • B10DA4B8-480B-4FE1-A651-4DFBE39ABF71 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
    • macOS (desktop)         • macos                                • darwin-x64     • macOS 13.0 22A380 darwin-x64
    • Chrome (web)            • chrome                               • web-javascript • Google Chrome 110.0.5481.177
    ! Error: Chris’s Apple Watch needs to connect to determine its availability. Check the connection between the
      device and its companion iPhone, and the connection between the iPhone and Xcode. Both devices may also need to
      be restarted and unlocked. (code 1)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

The bug is fixed with the following small change; would this be appropriate?
--- packages/flutter/lib/src/material/bottom_sheet.dart
+++ packages/flutter/lib/src/material/bottom_sheet.dart
@@ -860,7 +860,7 @@ class ModalBottomSheetRoute<T> extends PopupRoute<T> {
     // If useSafeArea is false, the bottom sheet is aligned to the bottom of the page
     // and isn't exposed to the top padding of the MediaQuery.
     final Widget bottomSheet = useSafeArea
-      ? SafeArea(child: content)
+      ? SafeArea(bottom: false, child: content)
       : MediaQuery.removePadding(
           context: context,
           removeTop: true,

Thanks for the report and potential fix @chrisbobbe
I was able to replicate this using latest versions as below:

Also, this isn't specific to material 3, as using material 2, we see same behavior:

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.3, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.3 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (4 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.8.0-17.0.pre.38, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.8.0-17.0.pre.38 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1d17caed66 (38 minutes ago), 2023-02-27 23:11:38 -0500
    • Engine revision 002b3d7a24
    • Dart version 3.0.0 (build 3.0.0-277.0.dev)
    • DevTools version 2.22.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


V2 PowerApp different OnStart

Description
In the new V2 of the Power App, OnStart does not contain the Admin Group information as indicated in the steps.
So either the V2 requires new documentation for this specific step or there is a bug in the uploaded app package.
Steps to reproduce
Step 3 point 8 is not correct for V2
Expected results
That admin group could be added
Actual Results
No admin group could be added
Solution component
Power App
Operating system (environment)
Windows
Additional Info
No response

Please update, got it too

@ZAJE-BDK @Vvand-jkra I've raised this. Will come back to you soon.

Second this. Anyone with Members permissions can approve their own Teams Requests.

@ZAJE-BDK @Vvand-jkra  @dcrockin01 this is now fixed, please download the latest v2 Power App where the code has been reinstated in the OnStart method. Thanks


Channel isn't loading past videos

Bug
When going to a channel lbry sometimes doesn't load all videos.
To Reproduce
Steps to reproduce the behavior:

Go to any channel with multiple videos (not sure what the cutoff value might be)
Scroll down until you can see pulsing thumbnail placeholders for videos and wait...
Wait a little longer...
Videos/Thumbnails do not load.

Expected behavior
Videos/Thumbnails should load.
Screenshots

System Configuration

Tried to repro on 2 RCs on windows and linux. Can you post a channel you find it happens on?

For example: lbry://@disaffected#a
But it's complicated. This happens on different channels on different times. Is there any way to debug this? Should I run it from the console to get some output? It is not happening right now.


cargo-llvm-lines: 0.4.31 -> 0.4.32

Diff: https://github.com/dtolnay/cargo-llvm-lines/compare/0.4.31...0.4.32
Changelog: https://github.com/dtolnay/cargo-llvm-lines/releases/tag/0.4.32
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

cargo-llvm-lines


<__NSArrayM: 0x60000327cff0> was mutated while being enumerated

NSGenericException *** Collection <__NSArrayM: 0x283821110> was mutated while being enumerated.
    Frameworks/CoreFoundation.framework/CoreFoundation ___exceptionPreprocess
    /usr/lib/libobjc.A.dylib _objc_exception_throw
    Frameworks/CoreFoundation.framework/CoreFoundation ___NSFastEnumerationMutationHandler
    /Users/runner/runners/2.158.0/work/1/s/node_modules/react-native-iap-3.2.3/ios/RNIapIos.m:305:5 -[RNIapIos productsRequest:didReceiveResponse:]
    Frameworks/StoreKit.framework/StoreKit ___27-[SKProductsRequest _start]_block_invoke_2
    /usr/lib/system/libdispatch.dylib __dispatch_call_block_and_release
    /usr/lib/system/libdispatch.dylib __dispatch_client_callout
    /usr/lib/system/libdispatch.dylib __dispatch_queue_override_invoke
    /usr/lib/system/libdispatch.dylib __dispatch_root_queue_drain
    /usr/lib/system/libdispatch.dylib __dispatch_worker_thread2
    /usr/lib/system/libsystem_pthread.dylib __pthread_wqthread
    /usr/lib/system/libsystem_pthread.dylib _start_wqthread

We see several instances of this error occurring in the wild. It might be us calling getProducts more than once in rapid succession, but I wonder if there's some defensive coding we could put in place, to prevent this error?
Version of react-native-iap
4.2.0
Version of react-native
0.61.4
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
No error
Actual behavior
Race condition
Tested environment (Emulator? Real Device?)
Real device/Emulator
Steps to reproduce the behavior
Unkown

Thanks for the issue. I hope we could have minimal reproduction.

I've noticed the same issue but only in release iOS versions, I can't replicate it at all in debug with the IAP sandbox, it seems to happen randomly and doesn't consistently crash and it's not affecting every user.
@kennym are you calling getAvailablePurchases() at all? As that seems to be what is causing the crash for me.
@hyochan I think for me it happens when I check if the user has an active subscription using the snippet from this thread: https://github.com/dooboolab/react-native-iap/issues/275#issuecomment-433582389
(I'm currently running react-native-iap 4.0.7)

Hey guys, how are you? I'm also currently experiencing this issue in production and, given that I'm not an Objective-C developer, I tried to look into the code to see what's happening.
-(void)productsRequest:(SKProductsRequest *)request didReceiveResponse:(SKProductsResponse *)response {
    for (SKProduct* prod in response.products) {
        [self addProduct:prod];
    }
    NSMutableArray* items = [NSMutableArray array];

    for (SKProduct* product in validProducts) {
        [items addObject:[self getProductObject:product]];
    }

    [self resolvePromisesForKey:RCTKeyForInstance(request) value:items];
}

The issue happens at the line
for (SKProduct* product in validProducts) {
and for what I have read, it has to do with multiple threads accessing the validProducts array at the same time, maybe because the method is being called multiple times in succession or something like that. On Stack Overflow, people have given 2 different solutions:

Use synchronised to ensure thread safety.
Copy the validProducts array before using it.

Do you think this is correct? Sorry if I'm being completely wrong about it, and thanks in advance.

I am also facing the same problem. Could anyone help me with this

In version<EMAIL_ADDRESS>same issue. Occurs on various devices (iPhone 6S, X, 11 pro max, 7, 8 etc) and OS versions 13. *


Email sending speed and stats

We tried sending a newsletter to 50K users via AWS SES. A couple of questions:

Will Parcelvoy keep updating the sending stats on campaigns? While sending this newsletter, it was stuck on 3899/48K for a long time and finally showed the entire campaign as completed.

How do we ensure that Parcelvoy is obeying the AWS SES sending rate limit? As of now, it shows the entire 48K emails sent at the same time.

This is how the CPU utilization looks for the EC2 instance. Will it stay the same (wont go beyond 80%) even if we send the newsletter to 600K recipients instead of 50K? As of now we are using a t3.large instance.

Will the server use lesser resources if we turn off the LOG_COMPILED_MESSAGE setting since we don't want each message to be logged anyways?

Thank you for your help!

Stats (the numbers up top) are only updated every five minutes so there may be a delay in what you are seeing. If you refresh the page however you should see the status of the send updating and showing the status immediately
You set the rate limit for SES inside of the provider itself. The value is in messages per second. You'll see messages enter a throttled state if they aren't able to immediately send.
By default jobs are processed at a concurrency of 10 simultaneously processing messages per worker container (which is typically more than enough to send thousands of messages a second). If you want to adjust this you can add the variable REDIS_CONCURRENCY to docker compose and set that value up or down. If your CPU hits a ceiling, it'll just slow down the rate of send and take longer to process sending over multiple minutes for example. If you want to increase this, you would increase your instance size and add an additional worker container to help with concurrency. Overall what you are most likely seeing is MySQL and the workers competing for CPU resources here, if your send size continues to grow we would very much recommend moving MySQL over to RDS such that you can grow the installation beyond a single instance.
It may improve resource utilization, but I would not expect it to shift dramatically


local files gopen

I have recently encountered an error when trying to iterate through a locally-stored dataset. I found this recent issue on StackOverflow and have posted a temporary solution there. Is there any better and more stable way to solve this issue?
In short: urlparse on local addresses does not return an object with .scheme = 'file' but the name of your disk, which is not recognised by the gopen_schemes dictionary and thus returns the default gopen_error instead of trying to open the url with gopen_file.

How about
file:d:/some/path/train-{000000..000999}.tar
You can also use Windows symlinks that include drive letters in the target.
I find that the most convenient and even use it on Linux (that is, I always refer to "./data" regardless of machine and just symlink it to wherever the data happens to be on the current machine).