Online version? Like bundlephobia.com

Clear and concise description of the problem
When exploring packages on the web, it's nice to be able to find this information without context switching or to share the results with others. For example I added several links to such services in https://npmhub.org and I regularly link to bundlephobia from my own packages’ readme.
Suggested solution
I think this is a good extension of bundlephobia, what do you think about either adding it to bundlephobia or creating a standalone website for it?
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think https://pkg-size.dev/ by @privatenumber is doing a much better job on it

Super cool site, but it looks more like an alternative to bundlephobia than export-size, it doesn't show the size of each export

It does, and also allows you to composable multiple exports to see the final bundle size. Or are you looking for a flat list of each export?


Update update-daemon-set.md

What type of PR is this ?
Improve documentation.
Which type of issue this PR fixes
fixes 31311(issue).

@mehabhalodiya I have changed update-daemon-set.md as per Tim's comment

/lgtm

Awaiting another sig docs reviewer/approver to sign off these changes.

Ah, a merge conflict? I can see the issue. It's not an actual conflict though.
/approve cancel
@MdSahil-oss do you know how to rebase? If so, please do. If not; please let SIG Docs know so we can suggest a different way forward.

@sftim Sorry I'm beginner and don't know how to rebase Please suggest me another way

We couldn't merge this as-is as there is a conflict that the automatic system does not know how to resolve.
 It's a great skill if you often work with source control; on the other hand, if you're mostly a writer or focus your skills in other areas than Git, it's probably not worth the effort (it's genuinely not that easy to learn the first time around).
Close this pull request, and open a new one based on the main branch that makes the same changes. In the new PR, mention that it's a replacement for PR #31354

Here are some references/tips for the following:

merge conflicts and rebasing
squashing commits

@mehabhalodiya made a lot of mistakes so let me open a new PR in replacement of this one


Thanks for the suggestion! I was thinking about adding that at some point but forgot to.
I'll check if it's easily doable and if so, I'll add this feature when I have time :)

Added in v2.2.0 GUI
Thanks to @MasonGuinn for the pull request, your time and code.


Add custom styles for redirect page

Feature Request
Summary
Currently the redirect page looks very different than the rest of tabi.
Motivation
I loved the custom styles for atom.xml and think this would be useful for branding and trust.
Detailed Description

Add a templates/internal/alias.html to style the redirect page similar to the rest of the theme.

Additional Context
https://github.com/getzola/zola/blob/v0.17.2/components/templates/src/builtins/internal/alias.html

I didn't even know this was a feature, to be honest.
I do agree it would be a good idea to add (minimal) styling to fit the theme.
On it!


Implementation Plan: Machine-generated tags on the frontend

Due date:

2024-05-24
Assigned reviewers

[ ] @AetherUnbound
[ ] @zackkrida

Both of you were chosen for your familiarity with the project and the frontend. Input from @fcoveram is also necessary from the design point of view: whether the proposed approach is feasible and good for the UX.
Description

Resolves #4039
Current round
This discussion is following the Openverse decision-making process. Information
about this process can be found
on the Openverse documentation site.
Requested reviewers or participants will be following this process. If you are
being asked to give input on a specific detail, you do not need to familiarise
yourself with the process and follow it.
This discussion is currently in the Clarification round.

The deadline for review of this round is 2024-05-17

@obulat I see there are a few places with missing text still, I can review this more deeply after those are addressed. I am also positive about the general approach outlined so far :)

@obulat I see there are a few places with missing text still, I can review this more deeply after those are addressed. I am also positive about the general approach outlined so far :)

@zackkrida, I updated the proposal, and it's ready for review

I will review this today.

Very clear document. I left comments related to showing how tags are generated.
In short, I agree with the content need and embrace the "giving all the details" approach, but I think we can introduce the info with more context more smoothly.

Drafting to update


configuring API Gateway resource policy

Description:
The new API Gateway private endpoint feature requires creating a resource policy that allows API requests coming from a VPC.
AFAICT there is no way to configure the Policy field on AWS::ApiGateway::RestApi via SAM. (Additionally, I observe my manually-added resource policies getting wiped on new SAM deployments.)
Steps to reproduce the issue:

Create an API via SAM, setting EndpointConfiguration: PRIVATE

Observed result:
No resource policy on the API.
Expected result:
No resource policy on the API until SAM allows configuring a resource policy for an API in the SAM template.

I'll add this as a feature request for the new APIGW feature.

This would be awesome! Until then private endpoints are pretty much useless?

Is this not implemented here? https://github.com/serverless/serverless/pull/5071

@simlu yes, until this is added there's no way to use private endpoints with SAM. The link (https://github.com/serverless/serverless/pull/5071) looks like a different project...

@rgarcia Yes. I thought you could just use cloudformation with SAM, but apparently not?

Yes, you can use Cloudformation with SAM. In this case, though, the Cloudformation field for an API's resource policy is contained within the result of the transformation that SAM applies to your template. Thus there's no way to modify the resource policy without changing how the SAM transformation behaves.

@rgarcia Thank you for the explanation!

Is it possible to apply the Resoure Policy after the main SAM Deployment, in another cloudformation script?  What ouputs from the SAM stack would I need.  My company is rolling out a cluster of API Gateways that i need secured to internal VPC access only.  Let me know your recommendation as I need this feature.

@lorddelicious workarounds I've thought about, but haven't tried:

If you're using API Gateway / SAM's support for putting your API definition in a swagger file, it looks like you can put the resource policy in the swagger definition: https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-resource-policies-create-attach.html#apigateway-resource-policies-create-attach-using-swagger
Custom cloudformation resource that makes the call to the API Gateway API to update the API's resource policy https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-resource-policies-create-attach.html#apigateway-resource-policies-create-attach-using-api
Some out-of-band, not-in-cloudformation (but in our deploy scripts) call to the API Gateway API to update the resource policy

^ We use (2) extensively for lots of things.  We like that everything is still part of the stack deploy and not "manual" api requests after the stack deploy is already completed

I'm currently working on a design doc for all API Gateway auth strategies and I'll create an RFC issue for community feedback soon (will crosslink here).
@simlu Could you list a few things that you're using Custom Resources for? Is this to work around SAM limitations specifically or something else? I'm also working on a doc to enable users to passthrough/override properties of the generated CloudFormation template. This will enable you to use all features (including Resource Policies) without us adding explicit support for it (though explicit support will provide a cleaner interface for defining things; the intent of this passthrough feature is to unblock you). I'll get an RFC out for that soon also.

Hi guys, wanted to update you on my progress.  I was able to use the workaround @rgarcia , the first one using the swagger definition file.  Mine is YAML formatted, and I need to secure the API access to within our VPC.  I was able to get this to go through cloudformation, after playing with some of the CF intrinsic functions.  It works and explicitly denies access to outside of VPC, but testing via curl on an EC2 within the VPC I am getting the same denied message.
{"Message":"User: anonymous is not authorized to perform: execute-api:Invoke on resource: arn:aws:execute-api:us-east-1:********somehashvalue/Path/GET/ with an explicit deny"}
swagger: '2.0' x-amazon-apigateway-policy:   Version: "2012-10-17"   Statement:     - Effect: "Allow"       Principal:         AWS:           Fn::Sub: "arn:aws:iam::${AWS::AccountId}:root"       Action:         - "execute-api:Invoke"       Resource: "execute-api:/*"     - Effect: "Deny"       Principal: "*"       Action:         - "execute-api:Invoke"       Resource: "execute-api:/*"       Condition:         StringNotEquals:           aws:Source:             Fn::ImportValue:               Fn::Sub: "${OurVPC}"

@brettstack We use if for everything that is not available through cloud-formation. E.g. setting up cognito triggers, service registration, db init / sync / update / reset on deploy. The goal was to have things happen on stack deploy and not have separate external calls from external - deploying the stack directly from the aws website should result in the same state as deploying it through a deploy script locally - so no, this is not SAM specific.

Another update, I was able to successfully configure the Private Gateway through the Swagger YAML.  As listed above, I had to add a VPCE endpoint for the execute-api in my environment https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-private-apis.html#apigateway-private-api-set-up-resource-policy, and every thing worked as expected.

Closing this in favor of #514. Please provide feedback over there if you're interested in Resource Policy implementation.

@lorddelicious could you please share the resource policy that worked for you?  I am getting the same error
{"Message":"User: anonymous is not authorized to perform: execute-api:Invoke on resource: arn:aws:execute-api:us-east-1:********somehashvalue/Path/GET/ with an explicit deny"}
I have tried specifying just * for the Principal but that doesn't seem to work.
Thanks!

@karlgoldstein I was having the same issue and now made it work. The issue caught me was, after you modified the API resource based policy and save, it won't actually take effect until you deploy the API. At first deployment I messed up "vpce-id". I was trying to fix the policy but couldn't make it work. Then I found the exported swagger file still has the non-updated policy.

@karlgoldstein Sorry for the delay, this works for our purposes, remember to set uyp your VPCE endpoint for execute api gateway in your env.  Should work.
x-amazon-apigateway-policy:   Version: "2012-10-17"   Statement:     - Effect: "Allow"       Principal: "*"       Action:         - "execute-api:Invoke"       Resource: "execute-api:/*"       Condition:         StringEquals:           aws:SourceVpc:             Fn::ImportValue:               Fn::Sub: "OurVPC"

@lorddelicious (or others), where in your Cloudformation serverless template can you specify an x-amazon-apigateway-policy? I've been struggling to see it show up after my template goes through the transform.
Thanks.

For the next person who comes here and sees this, the way to do this in SAM is to define it in the swagger defining your API.
Doc is here https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-resource-policies-create-attach.html#apigateway-resource-policies-create-attach-using-swagger
and it should look something like this
  SomeServerlessApiDefinition:
    Type: AWS::Serverless::Api
    Properties:
      StageName: Prod
      DefinitionBody:
        swagger: '2.0'
        x-amazon-apigateway-policy:
          Version: '2012-10-17'
          Statement:
            - Effect: Allow
              Principal: "*"
              Action: execute-api:Invoke
              Resource:
                - arn:aws:execute-api:*:*:*/Prod/someendpoint
        info:
          version: '2018-11-06'
          description: A description for the API
          title: Some Title
        schemes:
          - https
        paths:
...

if you are using Cloudformation to create PRIVATE Api use "Resource": !Join ['',["execute-api:/","*"]]
in policy AWS Will translate to proper ARN.


Is it possible to use Postgres/Mysql instead of Pinecone?

Just wondering if it's possible to use something other than Pinecone, to reduce the dependencies. Postgres has extensions for vector storage https://github.com/pgvector/pgvector

Pinecone is a vector database, PostgreSQL is not specifically designed for handling high-dimensional vector data, but I assume you should be able to store vector embeddings as arrays or binary data. Searching and retrieving similar vectors won't be as efficient as using a dedicated vector database though. Extensions like pgvector or pgroonga are for that indeed, and if you don't want to mess up with the backend for upsert & query you can expose Postgres using let's say stored functions via PostgREST

https://supabase.com/docs/guides/database/extensions/pgvector definitely possible, although Pinecone is specialized for the task.

Are there any updates regarding this topic?


How rebind C-x 0 to delete-window rather than the (+workspace/close-window-or-workspace)

Doom binds C-x 0 to (+workspace/close-window-or-workspace), which is not handy. because, the function closes not only the window but also the buffer as well.
For instance, when I open a eshell window and C-x 0 to close the window.  The eshell buffer is destroyed at the same time. You cannot strike C-x 4 b to invoke the previous eshell again.
So I want to rebind C-x 0 to delete-window just as vanilla emacs does.
(global-set-key (kbd "C-x 0") 'delete-window)
Unfortunately, it fails to works, while C-x 0 still bind to  (+workspace/close-window-or-workspace),
How could I rebind C-x 0 to delete-window?

the function closes not only the window but also the buffer as well.

False. +workspace/close-window-or-workspace does not kill buffers. The reason eshell dies is because of the :ui popup module cleaning up after itself. M-x eshell will produce a popup window.
Use SPC o e (+eshell/toggle) or SPC o E (+eshell/here) instead. The former creates a popup. The latter uses the currentw indow.
Otherwise, remove the :ttl property from +popup-defaults. ttl = time to live (for the buffer after the popup's been closed). The default of 5 means kill the buffer after 5 seconds. Though, removing this may have other side effects.
(plist-put +popup-defaults :ttl nil)

I want to rebind C-x 0 to delete-window just as vanilla emacs does.

Why you can't bind it.
To "fix" it, either:
(map! "C-x 0" (cmd! (delete-window)))

or
(map! :after persp-mode
      :map persp-mode-map
      [remap delete-window] nil)

(map! "C-x 0" #'delete-window)


To make the dockerfile build I need to create a dir before making it a volume.

See documentation: https://docs.docker.com/reference/builder/#volume
Tested under Virtualbox. Please consider to test the patch with parallels before.
This partly solves #75.

This setting was necessary when I did build the image isolated. However in the context of docker-compose it doesn't seem to be necessary. Closing the pull request.


Certain capabilities should be a fixed set of capabilities

We have a set of capabilities that if "dynamically" changed by adding or removing fundamentally breaks CPS.
Anything that affects the project tree via IProjectTreeSettingsProvider, is COM aggregation cannot be a dynamic capability (ie cannot come from targets):
HandlesOwnReload
AppDesigner
EditAndContinue

This is the cause of the second issue in https://github.com/dotnet/roslyn-project-system/issues/1103.

I'm going to add OpenProjectFile to this list. It's really a capability of the project system, not of the project type, and to move to the in-memory editor we'll need to use COM aggregation anyway.

Fixed in #1141.

I think we need another bug to track removing the default capabilities provider - that currently will cause us to load for all projects.

@davkean that exists, https://github.com/dotnet/roslyn-project-system/issues/559


Changeable windows

Add command line arguments to possibly select custom windows for testing purposes

This would look great in the testing section

Not important


Delete resource directly without checking old one

Referring to kubebuilder sample, simplify deletion logic to delete
resources without checking existing one.
Signed-off-by: Lan Luo<EMAIL_ADDRESS>

/test-multicluster-e2e
/skip-all


add icon for Twitter link

we're linking to Twitter only now. we still could/should add this.

Hi Cvan , if I am correct you want to replace the text links with Twitter icon links , right ??
I want to take it up , if you are okay with it.

thanks for the contiubtion, @deathping1994. we have since redesigned and removed the Twitter link. thanks though!


Added a message queue (handles HTTP 409), better debug messages

I see that we are now handling messages by API?


Inconsistent behavior on positive and negative Float/Double zeros

This library appears susceptible to the same problem as discovered on the hashable package
https://github.com/tibbe/hashable/issues/116

Thanks for pointing this out! I'll try to follow the discussion in that ticket

@glguy I haven't seen what hashable authors have been discussing on IRC, but I'm not convinced this is correct behavior. Those values have an identical Eq instance, but are in fact distinguishable (with isNegativeZero) and supposedly may have different behaviors under different operations.
What do you think? Do you know of any other hashing libraries that normalize negative zero? I definitely think a comment is warranted at least.

If you expect nub x == HashSet.toList (HashSet.fromList x), then they should hash to the same value.

Right, you can spell that all sorts of ways. But there are arguments to be made that instance Eq Float is broken or shouldn't even exist. And I'd be worried if I saw sort (nub (x::Float)) in real code.
At least a hash over the standard IEEE format bytes of a float is well-defined. Furthermore it's easier for users to newtype wrap a Float and write a normalized hash instance for that than the other way around.

Although it's worth considering whether we want to promise to match Eq (and tell users their instances should respect that as a law as well). Currently I don't promise that at all.

This may be resolved one way or the other looking at : #8 and  in particular https://github.com/jberryman/hashabler/issues/8#issuecomment-258643797 . It's a priority to me that we get the best compatibility between the same code compiled to JS and machine code.


Expose new admin decorators in django.contrib.admin

I forgot this important step in #37.

thanks!


Feature suggestion - colorized (per pod) output

I got here looking for an alternative to kubetail. One of it's limitations is that it doesn't see new pods as they are created.
However, one feature of kubetail that is missing in ktail is it's colorization support. Kubetail picks a different randomly assigned color for each container - so if you are watching multiple, you can easily see from one set of output to the next.
That would be a nice feature to see added here.

Implemented in 1.2.0.


inherit scikit-learn BaseEstimator object?

We can copy the BaseEstimator class from here (with proper attribution): https://github.com/mne-tools/mne-python/blob/master/mne/decoding/base.py
Then it should work with scikit-learn cv objects without having a dependency.

@jasmainak is this related to cvpyglmnet issue? should we merge the issues or classify this as version 0.2?

I can look into it early next week if nobody beats me to it :) It's not too difficult. I suppose it would be neat if you can show cross-validation results with the GLM object before the release. My idea is that for the first release, we can first show the simple use case where you can do cross-validation with a scikit-learn cv object. The cvpyglmnet object that you first proposed might be handy if you want to do grid-search with warm-starts -- doing it more efficiently than with scikit-learn. Since the details are not super clear in my (or anyone else's) head yet, I suggested waiting before we proceed in that direction ...

@pavanramkumar I totally agree with @jasmainak. If we inherit BaseEstimator, it will look very clean and later on we can easily apply cross validation from scikit-learn. +1 for this

@jasmainak I can work on this if you want

@jasmainak @pavanramkumar following the idea of using sklearn's classes, I was wondering if I should inherit from the ElasticNetCV class in https://github.com/scikit-learn/scikit-learn/blob/5876f885f04f1abbf46a9cdfb5cb1d5a2ff64f98/sklearn/linear_model/coordinate_descent.py#L1355 I would only need to redefine the function that produces the path (https://github.com/scikit-learn/scikit-learn/blob/5876f885f04f1abbf46a9cdfb5cb1d5a2ff64f98/sklearn/linear_model/coordinate_descent.py#L1511) and sklearn would do the rest.

@jasmainak @pavanramkumar following the idea of using sklearn's classes, I was wondering if I should inherit from the ElasticNetCV class

what is the least "copy-able" unit if we were to not create a dependency?
Actually I just realized this, the license for scikit-learn is BSD whereas ours is MIT. @pavanramkumar any particular reason for going with MIT? we would have to copy the BSD license clause in our LICENSE file just for the scikit-learn parts. Something like this : https://github.com/nilearn/nilearn/blob/master/LICENSE. Or if we switch to BSD, then I think one license file will do it.

@jasmainak I can work on this if you want

that's perfect! Feel free to make a "WIP" even before you have something that works perfectly. You can continue working on it by pushing to the same branch. It's always a good idea for everyone to look at the code early in the development path :)

@jasmainak The thing is that ElasticNetCV does the cross validation in a special way by understanding that there is warm start during the regularization path. The simplest inheretance would need to come from BaseEstimator and RegressorMixin (or ClassifierMixin for logistic and multinomial regression, not sure yet) I'll work on it as WIP

okay I see. I think it might be specially geared for their Lasso object but I'm not sure ...
Also, if we have a scikit-learn import in our pyglmnet.py file, then we introduce a dependency on scikit-learn. Whereas copying the relevant parts of the code will ensure compatibility without introducing dependence. I'm not sure which is better ... I'll look at your WIP code :)

@daniel-acuna do remember that inheriting two classes vs. writing your own class while conforming to API and style elements is probably equivalent in terms of effort!

@pavanramkumar If I inherit from sklearn classes then I can be used in all data analysis pipelines that sklearn has. Also, people will know better what our class is supposed to do

@daniel-acuna sure, agreed

@daniel-acuna let's see what your PR means in terms of code changes. We can continue the discussion once you make the PR :)

@jasmainak happy to go with BSD license.
apparently BSD license is for those who fear their name will be dragged through the mud, whereas MIT license is for those who fear their contributions will never be used :)
http://programmers.stackexchange.com/questions/82321/why-do-people-pick-the-mit-license-over-bsd-if-bsds-non-endorsement-clause-i

bah, I don't know the real difference between the two :\ BSD is used everywhere because it even allows commercial use -- maximum impact :) not sure if MIT allows that but MIT and BSD are pretty similar as far as I know. If most people are using BSD, it might be easier in terms of logistics I suppose?

@jasmainak At the end, I don't use scikit-learn code in the new PR #78

very cool @daniel-acuna !


Keys with namespaces doesn't display translated text.

Describe the bug
with the config
  "i18n-ally.localesPaths": "translation/locales",
  "i18n-ally.sourceLanguage": "en",
  "i18n-ally.keystyle": "nested",
  "i18n-ally.namespace": true,
  "i18n-ally.enabledFrameworks": ["i18next"],
  "i18n-ally.pathMatcher": "{locale}/{namespaces}.{ext}",
  "i18n-ally.displayLanguage": "en"

🈶 Activated, v2.2.9

――――――

💼 Workspace root changed to "[path to project]"
🌞 Enabled
🧩 Enabled frameworks: i18next
🧬 Enabled parsers: json

🚀 Initializing loader "[path to project]"
📂 Directory structure: dir
🗃 Custom Path Matcher: {locale}/{namespaces}.json
🗃 Path Matcher Regex: /^(?<locale>[\w-_]+)\/(?<namespace>.+)\.json$/

📂 Loading locales under [path to project]/translation/locales
	📑 Loading (de) de/common.json [1595579598497.0623]
	📑 Loading (en) en/common.json [1595579246887.7266]

👀 Watching change on [path to project]/translation/locales
✅ Loading finished

――――――

In the editor items seem to be not getting highlighted at all. By Hover no tooltip appears:

(tried also i18n.t('common:first-name'))
If I remove:   "i18n-ally.enabledFrameworks": ["i18next"],
It seems to work again, but still can't fetch the text. If I edit the translation file (via UI Editor), it correctly sets the text in the JSON file.

Extension Version
2.2.9
LanguageID
typescriptreact
Framework/i18n package you are using
react-i18next
Device Infomation

OS: Mac OSX
Version: Catalina
VS Code Version: 1.47.2

Tried multiple things... including setting of deprecated i18n-ally.keyMatchRegex. Interesting observation: After setting of enabledFrameworks, highlighting disappears completely.

Tried this?
"i18n-ally.enabledFrameworks": ["i18next", "react"],

🈶 Activated, v2.2.9

――――――

💼 Workspace root changed to "[project-path]"
🌞 Enabled
🧩 Enabled frameworks: i18next, React
🧬 Enabled parsers: json

🚀 Initializing loader "[project-path]"
📂 Directory structure: dir
🗃 Custom Path Matcher: {locale}/{namespaces}.json
🗃 Path Matcher Regex: /^(?<locale>[\w-_]+)\/(?<namespace>.+)\.json$/

📂 Loading locales under [project-path]/translation/locales
	📑 Loading (de) de/common.json [1595579598497.0623]
	📑 Loading (en) en/common.json [1595579246887.7266]

👀 Watching change on [project-path]/translation/locales
✅ Loading finished

――――――

Yes, just tried, same effect. No reaction on t('{key}') elements

It's a bug and should be fixed in 2.2.10, thanks

Thanks for such quick reaction & resolution. It's super helpful extension.

Works like a charm!

Hey,
Just opening back this issue, I had the same problem

{
  "i18n-ally.localesPaths": ["modules"],
  "i18n-ally.pathMatcher": "{namespace}/domain/i18n/{locale}.json",
  "i18n-ally.namespace": true,
  "i18n-ally.keepFulfilled": true,
  "i18n-ally.sourceLanguage": "fr",
  "i18n-ally.keystyle": "flat"
}

When I add  "i18n-ally.enabledFrameworks": ["i18next", "react"], it works, looks like i18next isn't recognized by default
@antfu Any idea? I might help? looks like it's looking at package.json for i18next but as u can see I have it


Why isn't the normalized name used when adding roles to the claimsidentity?

https://github.com/aspnet/Identity/blob/eb3ff7fc32dbfff65a1ba6dfdca16487e0f6fc41/src/Microsoft.Extensions.Identity.Core/UserClaimsPrincipalFactory.cs#L143
As ClaimsPrincipal.IsInRole() calls down to ClaimsIdentity.HasClaim() which uses an Ordinal comparison on the value it seems logical to save the normalized name for the role. Or if it's important to be able to display the original name to return a version of ClaimsIdentity that has an overridden HasClaim method with an OrdinalIgnoreCase comparison, perhaps just for the default role type.
Otherwise why normalize the role name at all?

Normalization is there for index comparisons/lookups/uniqueness. When it goes into a claim we use the real value, because fidelity is important. What ClaimsIdentity does isn't something under our control.


Reclaim button

Reclaim sold item
After click:

Remove sold money from campaign
Remove sold for from item
Set sold to false

Done. But the field 'hero' remains nul after reclaim an item. I figured, I will add new variable that stores name of last owner and after reclaim sets it as hero.

As I commented in your commit - this should be an association.
You're still implementing this feature. So it is not closed yet ;)


report-v2 errors: project %in% d_projects$project; Can't convert y  to match type of x <pq_geometry>

Warning: Error in filter: ℹ In argument: `project %in% d_projects$project`.
Caused by error in `mutate()`:
ℹ In argument: `across(where(is.character), na_if, "NA")`.
Caused by error in `across()`:
! Can't compute column `geometry`.
Caused by error in `fn()`:
! Can't convert `y` <character> to match type of `x` <pq_geometry>.
  164: <Anonymous>

removed geometry from projects output with https://github.com/marineenergy/apps/commit/cab065349f9aa6fd13f9288f08ffb8861865c44d#diff-3490322c3f0ea3493b574aca72b1352d290c3f624b00ef631fcaaf3e32c5ec0fR163


Swapping items into offhand bypasses inventory conditions

I have an inventory condition which I apply to players to prevent them from moving items in their inventory:
public static final InventoryCondition readOnlyInventoryCondition =
    (player, slot, clickType, result) -> result.setCancel(true);

This works, except items in the hotbar can be moved by being put in the offhand (using the keybind), then by moving the held slot, and swapping back.
The swap should first check against the player's inventory conditions and respect the result.
This would affect PlayerDiggingListener#swapItemHand (line 153)
A temporary workaround for anyone else with this issue is to listen for PlayerSwapItemEvent and cancel it.
EDIT: More experimentation shows this bug also applies to armour and the elytra when right-clicked.

Will be closed by #1992


Feature request: delay option

It would be nice to have a delay option, which will only show the progress bar if .done was not already called within the delay time:
NProgress.configure({delay: 80}) // Only show if not done within 80ms

https://github.com/este/next/blob/aa41364278e7d01a4a77319111b6fdf78935ecc3/components/loading-bar.js

you can implement this with something like lodash.debounce:
const debounce = require('lodash.debounce')

const start = debounce(NProgress.start, 80)

On second thought, that doesn't really cover it!

Looking forward for this great new feature! Use case:
A singlepage-app may have pages with zero loading time but the progress bar is shown anyway which is kind of confusing to the user. Would be nice to have a delay exactly as @fridays described: Only show the progress bar if NProgress.done() has not been called within this time.

I've extended @fracz idea to avoid patching the library, in case anyone else finds it useful:
let progressBarTimeout = null;

const clearProgressBarTimeout = () => {
  if (progressBarTimeout) {
    clearTimeout(progressBarTimeout);
    progressBarTimeout = null;
  }
};

const startProgressBar = () => {
  clearProgressBarTimeout();
  progressBarTimeout = setTimeout(() => {
    nprogress.start();
  }, 200);
};

const stopProgressBar = () => {
  clearProgressBarTimeout();
  nprogress.done();
};

just use startProgressBar and stopProgressBar instead of nprogress.start/done

@msurdi I believe you could simplify that to:
let progressBarTimeout = null;

const startProgressBar = () => {
  clearTimeout(progressBarTimeout);
  progressBarTimeout = setTimeout(nprogress.start, 200);
};

const stopProgressBar = () => {
  clearTimeout(progressBarTimeout);
  nprogress.done();
};

For typescript users:
let progressBarTimeout: number | undefined = undefined

const startProgressBar = () => {
  clearTimeout(progressBarTimeout)
  progressBarTimeout = setTimeout(nProgress.start, 200)
}

const stopProgressBar = () => {
  clearTimeout(progressBarTimeout)
  nProgress.done()
}

This works in React/NextJS:
import NProgress from "nprogress";
import { useEffect } from "react";

export function Progress() {
  NProgress.configure({
    showSpinner: false,
  });

  useEffect(() => {
    const timeout = setTimeout(NProgress.start, 250);

    return () => {
      clearTimeout(timeout);
      NProgress.done();
    };
  });

  return;
}


capybara-screenshot does/does not/does make screenshots

rspec spec/features/auth/signin_spec.rb
FFFFFF

Failures:

  1) Sign in process should not allow a sign in without a password
  Failure/Error: log_in(user)
  Capybara::ElementNotFound:
    Unable to find link or button "#logoutMenu"
  # ./spec/test_auth_helpers.rb:13:in `log_out'
  # ./spec/test_auth_helpers.rb:4:in `log_in'
  # ./spec/features/auth/signin_spec.rb:7:in `block (2 levels) in <top (required)>'
Finished in 7.64 seconds
6 examples, 6 failures

Failed examples:

rspec ./spec/features/auth/signin_spec.rb:17 # Sign in process should not allow a sign in without a password
rspec ./spec/features/auth/signin_spec.rb:43 # Sign in process should allow a user to log in with the correct credentials
rspec ./spec/features/auth/signin_spec.rb:29 # Sign in process should not allow a sign in with an incorrect username
rspec ./spec/features/auth/signin_spec.rb:23 # Sign in process should not allow a sign in without a username
rspec ./spec/features/auth/signin_spec.rb:36 # Sign in process should not allow a sign in with an incorrect password
rspec ./spec/features/auth/signin_spec.rb:12 # Sign in process should not allow a sign in with no data

When I try to do "screenshot_and_open_image" in my test-helper I get:
 Launchy::ApplicationNotFoundError:
    No application found to handle '/Users/.../development/.../tmp/capybara/screenshot_2014-10-29-10-19-32.772.png'

Which is absolute correct, bacause a 'ls -l' shows not a single file in the directory.
The funny thing: I can't tell when why it is working sometimes and most of the time not...
oh: used a old version, updated to 1.0.3, nothing changed. capybara is 2.2.1

@AntonTrapp, apologies for not replying to this, is this still an issue?  Did you find a solution?

Wow. Some time ago... ;) - Tnx for the answer.
Let me thing about it. When I remember correct the error occured before the first visit command. So everybody just has to use it in a "sane" way. However I would throw a "visit a page before you try to generate a screenshot" error instead of a silent return. Would be more user friendly and save many people a lot of time. Not sure so, quite some time and I am old and forget many things :D

I'm running into the same issue.
I have a single page Javascript app running on a remote server and the page being tested is the root URL. When any of the tests fail, it does not take a screenshot because capybara.current_path is empty (even though the page was loaded successfully).
Are there any workarounds? Alternatively, is there a better way to check that the page was loaded? I'd happily create a PR.

I don't have this issues since I stopped using after hook at all.

Are there any workarounds? Alternatively, is there a better way to check that the page was loaded? I'd happily create a PR.

A PR would be welcome, but are you sure you're using the latest version as we've recently had quite a few updates?


Get deleted features by delta

This PR allows to

Get a customized number of deltas (<100)
Choose what deltas to consider to look for deleted features
Get a file with features that have been deleted in each delta

@samely thanks for putting this together! Can I ask that you refactor this to follow the schema defined in lib/
That allows the feature to be used as a library or as part of the CLI.
If you could build out the interface to be able to list deltas and also a second function to get a specific delta which has a filter option that could a couple different things like:
filter: [ 'delete']

filter: ['delete', modify']

filter: ['modify', 'create', 'restore']


Add before and after hooks only for Features and Scenarios

This commit adds four functions to context: beforeEachFeature,
beforeEachScenario, afterEachFeature and afterEachScenario. They set
hooks that would only be executed on nested Features and Scenarios
respectively.
This is convenient to setup all the Features and Scenarios without
requiring a before or after for each of them.

First of all, thanks in advance for your time reviewing this.
I'm working in a project which adopted mocha-cakes-2 recently and last week we've missed a feature. We were setting a bunch of Scenarios inside a feature, and we wanted to provide a clean state (in this case, a clean database) for each Scenario, but share that state between all the tests inside the Scenario.
We ended up adding the same before hook on each Scenario. I've found that solution not quite satisfactory so I tried to implement a beforeEachScenario hook that would solve our problem in a rather elegant way.
For the same price, I've also implemented beforeEachFeature, afterEachFeature and afterEachScenario. I'm not sure about the order in which the hooks should executed, so maybe it's worth having an extra look at that.
I'm not that familiar with the mocha ecosystem, so maybe there's a better way to implement this. In any case, this would be a nice feature for us and I hope you find that it has a place in this module.

Is this project active?

Hi!
Thank you very much for your PR, it's definitely a feature I've been wanting to implement.
I just got back from a long vacation, so very sorry for taking so long to get to this. I will evaluate your PR as soon as I can.

@binarymist Unfortunately beforeEach and afterEach are executed around every it, regardless of nesting. At least that was the case last time I checked. Not sure if it has changed for Mocha 4

Thanks @iensu, welcome back :)
And yes, I don't think there's a way to do this in Mocha. There's probably no reason to in the interfaces they ship because there's no named levels of indentation.

@meis Thanks! I have had a look at your PR and it works as expected. Great work!
I understand the symmetry reason for implementing beforeEachFeature and afterEachFeature. However, I can't come up with a good use-case for the hooks since they would be applied globally to all features in the test suite. I'm worried that it might be a bit confusing. I would like to hear your opinion here :)
Also before merging it would be awesome if you could add a description of the functions in the README.

I understand the symmetry reason for implementing beforeEachFeature and afterEachFeature. However, I can't come up with a good use-case for the hooks since they would be applied globally to all features in the test suite. I'm worried that it might be a bit confusing. I would like to hear your opinion here :)

The only thing that comes to mind (and how I intend to use this) is as a common setup step. We do initialize a database on each scenario, but I can see that done on each feature too. If you think that might be confusing, we could leave them out. They should be trivial to implement anyway.

Well, let's leave the functions in. As you say, it might be that someone has the need for them and the name should be clear enough to understand the consequences of using it on the top scope.
I'm going to merge this now. Again thanks a lot for your PR!

Awesome @iensu, thanks to you!
Do you have any plans to release this to npm?

I'm going to try to get a new release out beginning of next week.

@meis version 3.1.0 is out now!


Would It Be Possible To Add Support For a Small Widget?

It would be the same as the medium widget except the line graph would only show the current weather and the weather for the next 3 hours, instead of the next 9 hours.

Sorry not at the moment.


Add: Solution for Day 11, 12, 13 and 14 in Swift language

Add: Solution for Day 11, 12, 13 and 14 in Swift language.

@ham118 request you to update Commit message as per the Contribution Guidelines.you can go through this link: https://help.github.com/en/articles/changing-a-commit-message for reading about changing commit message. commit message can be Add: Solution for Day 11 and Day 12 in Swift language

@sagarchoudhary96 Please review this pull request

@sagarchoudhary96 Please review this pull request.

@ham118 day 13 is not available to solve in swift language on hackerrank, can you also confirm from your side too. you will need to just create PR with changes for Day 11,  Day12 , Day13. you can create separate PR for them. let me know will close this PR then accordingly and then you can update new one.

@sagarchoudhary96 I have verified a day 13 solution in XCode playground and it is working fine. Also, you can check with various book's title (string), author (string) and price (integer) values to put instead of 'readLine()!' function.
Although, If not consider day13 solution then I'll create PR with only day11,12,14 solutions in swift language and I'd like to add day15 solution in swift language. Please let me know so I can do accordingly.

@ham118, i would request to create PR for 11, 12, 14 and you can take day15 next to solve.

@sagarchoudhary96 Please check this PR and another separate PR will send you soon for day15 solution in swift language

Hi @sagarchoudhary96 Please review this pull request. As per our conversation removed day13 solution and added day15 solution in swift language.

great work @ham118 👏

Thanks 👍


Add July meeting notes

A couple of issues got fixed shortly after the meeting -- should they be mentioned here?

Yes, if they were discussed.

Also would it be helpful to mention who brought up certain issues? (e.g. I wrote about the diataxis guidelines, but the list included here doesn't have any context or way to follow up.)

Please add your name to the notes if it's important. But even better, put up the draft guidelines :)

@encukou Since its almost time for our next meeting, are there any blockers to merging this now?


alpha.js allow colors without alpha components to be processed

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=453

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


Create Accept/Confirm landing page

Create a new landing page for confirming assignments without being based on a team.  Similar to the main home landing page, it'll pull up pending acceptance stuff and if you only have items in one team pending acceptance it'll redirect you straight there /team/Confirm/Confirm, otherwise it'll show you the teams in which you have pending stuff and make you choose which to go to.
This new page would then be the landing page for notification emails and possibly a future revision of the team home approvals header

In the confirm controller, create a new method Landing() which does the following:

If the Team property isn't empty, just redirect them there, ex: return Redirect("/" + Team + "/Confirm/MyStuff");

If the Team property is empty, create the HomeViewModel just like we do in https://github.com/ucdavis/Peaks/blob/master/Keas.Mvc/Controllers/HomeController.cs#L29.  If that object only comes back with one team with pending assignments, redirect them to that team's confirm/mystuff page.  Otherwise, make some UI which lists the teams with pending assignments, and each button takes them to that team.  Basically you can copy what the home/index action I linked does.


PR for #303  Add animated upload button

I have created animated  upload button with html and css github username Gurnav224

@Gurnav224  Amazing work!
Looking forward for more contributions from you. 🧑💻
Thank you so much for your contribution.🌟
Make sure to ⭐ and follow me.

I would love to connect with you all ❤

https://www.linkedin.com/in/om-gawande/


Fix typo in *_items_summary.item_classifiaction

item_classifiaction should be item_classification in all the x_items_summary views.

@duncandewhurst FYI this has been fixed without adding a new field (as per your comment on Trello) so queries will need to be modified.

Ah, I hadn't seen the Trello comment (not sure which card it is as not attached here). Anyhow, it should be easy to identify the queries to fix (and to fix them), as they will fail on newly created views/schemas.


Google authentication prompt pops up too often

It is quite annoying to see the following authentication prompt many times per day:

It's strange, I've been using StackEdit for about a year and just started having this happen. It happens like every hour. Before, I was never prompted.

Google now says this is a security risk since the app has an "Unverified developer"

It sounds like the developers have not gone through the app verification process.
However I was able to verify that for these sensitive oauth2 scopes, the refresh interval is required to be 45 minutes. Which is a real bummer. I wish I had control over this, but Google's a big company. @benweet please let me know (DM) if you need expedited Google OAuth App Verification.
It's possible that if the app refreshes the token before the 45 minutes expires, then it won't need to prompt the user to be prompted and that would be a helper.
I moved away from StackEdit for keeping a scratchpad a few weeks ago. b/c the re-auth prompts were so annoying.

I actually do not need the sync feature at all. i find this to be the most light way local OS markdow editor and converter (besides needing pandoc CLI table scripture software for the other forms like odt, or doc) it can do md, pdf, and html.
all others are either fremium or heavy on the web tethering.  I thought this app  could stick to the local browser space and not need to have any web dependency.  I might have mistakenly done some sync. and now I am fighting or trying to figure out how to wean it from any syncing to the web..  I could use google account permission nuking perhaps. but is there an another issue about this for more surgical within app, rollback to local OS strorage (whitin browser of course)?


Invalid byte sequence for encoding "UTF8": 0x00

Fluent attempts to save strings that contains the NULL character, it seems to be linked to JSON serialised values as this repo demonstrates:
https://github.com/adtrevor/FluentPosgresNullCharBug
The resulting error is:
ERROR:  22021: invalid byte sequence for encoding "UTF8": 0x00
LOCATION:  report_invalid_encoding, wchar.c:2017

Thank you

Where does the null character (0x00) appear in the JSON?
If it's appearing in a value string, that's not valid according to spec. Null characters should be represented as \u0000.
{"string": "foo\u0000bar"}

See the spec for value here:
      string = quotation-mark *char quotation-mark

      char = unescaped /
          escape (
              %x22 /          ; "    quotation mark  U+0022
              %x5C /          ; \    reverse solidus U+005C
              %x2F /          ; /    solidus         U+002F
              %x62 /          ; b    backspace       U+0008
              %x66 /          ; f    form feed       U+000C
              %x6E /          ; n    line feed       U+000A
              %x72 /          ; r    carriage return U+000D
              %x74 /          ; t    tab             U+0009
              %x75 4HEXDIG )  ; uXXXX                U+XXXX

      escape = %x5C              ; \

      quotation-mark = %x22      ; "

      unescaped = %x20-21 / %x23-5B / %x5D-10FFFF

It turns out the problem was attempting to submit TIMESTAMPTZ formatted data to a query expecting JSON formatted data. A failure here is expected.
However, this does bring up a possible enhancement to postgres nio:
https://github.com/vapor/postgres-nio/issues/53


isMoreAvailable not working for mediaComments feed

I have spotted a very weird behavior about mediaComments feed and it's method isMoreAvailable,
For posts that have been posted after June 22nd of 2019, the method isMoreAvailable is working,
for those that were posted after, it always returns false.
Do you have any clue of what is happening ?
Did instagram changed something during this period ?

Actually, I wanted to build the insta bot by using instagram private api, but I haven't any clue

This thread has been inactive for a long time now, so it will be closed. If this is still an issue, reopen the it.


className prop

Is it possible to give the ColorPicker component a className, otherwise what is the best method to override the components inline style in the actual colorpicker?

Ah, yeah this is totally do-able. Would you open up a PR with one? You can also create a custom color picker with whatever styles you want: http://casesandberg.github.io/react-color/#create


Feature/generic api

Codecov Report

Merging #146 (c4274b9) into main (43ce97e) will increase coverage by 0.18%.
The diff coverage is 79.88%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
+ Coverage   77.25%   77.44%   +0.18%
==========================================
  Files          54       55       +1
  Lines        3003     3006       +3
==========================================
+ Hits         2320     2328       +8
+ Misses        483      479       -4
+ Partials      200      199       -1

Components
Coverage Δ

garden-app
77.44% <79.88%> (+0.18%)
:arrow_up:


[REQUEST] Mixin implementations

What is the feature you're proposing?
With the ever-growing complexity of context functionality being used in our commands and components, we're in a situation where we need to abstract and break apart the code written for it in more simplified methods that don't involve dunder abstractions. Because of this, we're looking into metaclassing other classes that purely add methods as "mixins."

EventMixin: a mixin that can build event data, this will theoretically run on top of DictMixin but for Gateway-unique events that do not work off of objects.
ContextMixin: a mixin that will serve the purpose of simple super().method(**kwargs) method implementations in various context managers, e.g. send and edit methods.
ErrorMixin: a mixin for base support of the ErrorFormatter class that allow custom message, id and extended_message attributes. extended_message should be designed to represent the nested information of a dictionary to a key -> value arrow tree in the form of a string.

Additional Information

[ ] My feature request is related to an existing Issue.

Issue (if referenceable):

#681

Marking this as resolved since mostly implemented and lack of activity


Added RedStone oracles docs

thanks for the PR. I wasn't aware of Redstone, but it looks really interesting. Do you have standard reference contracts deployed to each EVM chain, or do dapps just need to use the PriceAware library?
Also curious about what dapps on other EVM chains are using your data feeds

thanks for the PR. I wasn't aware of Redstone, but it looks really interesting. Do you have standard reference contracts deployed to each EVM chain, or do dapps just need to use the PriceAware library?

They just need to extend the PriceAware contract for now. Actually, it will be converted to library soon :)

Also curious about what dapps on other EVM chains are using your data feeds
We are just finalising our contracts audits and our initial infrastructure configuration. We are working on integration with:

Moola (lending protocol on Celo)
Cybertime (NFT marketplace on Celo)
Deltaprime
TracerDAO

We also have few implemented a and tested POC apps like for example https://github.com/redstone-finance/komodo-celo
We are providing data via HTTP API for about a year now and we have ~400000 GET requests daily. But we are just launching with providing data on EVM compatible blockchains.
Thinking now, maybe it makes sense to add a note to the docs, that before using in production apps with high volume, we would like to finalise the audits and help with the correct integration :)

very cool!

Thinking now, maybe it makes sense to add a note to the docs, that before using in production apps with high volume, we would like to finalise the audits and help with the correct integration :)

Do you want to add this and then I will approve?

very cool!

Thinking now, maybe it makes sense to add a note to the docs, that before using in production apps with high volume, we would like to finalise the audits and help with the correct integration :)

Do you want to add this and then I will approve?

Just added :)


Milestone 4 - Web bridge Interface

Modify the Cartesi wallet React app from MugenBuilders and make this the standard bridge interface for CarteZcash

Currently being worked on by @ayad318

Closed by #31


Color output in console can conflict with output_runner regex

It would probably be best to have output_runner strip any color codes from the output before trying to match the regex.

@DavidANeil, the idea here, I presume, would be to strip ANSI escape codes and just pass it through? Do you have any sample input and output pairs?

Yes, I believe removing those escape codes would be the most general solution.
I am seeing output from rules_typescript that looks like:
path/to/my/file.ts:1:48 - error TS2307: Cannot find module 'library/file'.
Which I am currently matching using the regex (\\x1B\\[96m)?(.*?\\.ts).*?(\\x1B\\0m)?:(\\x1B\\[93m)?\\d+(\\x1B\\[0m)?:(\\x1B\\[93m)?\\d+(\\x1B\\[0m)? - (\\x1B\\[91m)?error(\\x1B\\[0m)?(\\x1B\\[90m)? TS2307: (\\x1B\\[0m)?Cannot find module '(.*?)'\\.
For my use cases the regex simply replacing /\x1B\[\d*?m/g with nothing would work for the colors, I'm not sure but I think /\x1B\[[0-9;]*[mGKHF]/g would match all of the escape codes.
I also added a .*? because I was sometimes seeing a backspace character being output as well.  Would it make sense to also remove those?


Streaming a table with a silly name crashes Postgres

Postgres allows creating tables with arbitrary names by enclosing them in double quotes:

Quoted identifiers can contain any character, except the character with code zero. (To include a double quote, write two double quotes.) This allows constructing table or column names that would otherwise not be possible, such as ones containing spaces or ampersands.

Creating such a table with the Bottled Water extension enabled crashes the Postgres backend when it tries to stream an insert.
Steps to reproduce:
CREATE TABLE "flobble-biscuits?whatisthetime/hahahaha" (thing SERIAL NOT NULL PRIMARY KEY);
INSERT INTO "flobble-biscuits?whatisthetime/hahahaha" DEFAULT VALUES;

This will crash Postgres, as observed by the Bottled Water client logs:
Writing messages to Kafka in JSON format
Created replication slot "bottledwater", capturing consistent snapshot "000002B0-1".
Snapshot complete, streaming changes from 0/1725898.
/usr/local/bin/bottledwater: Could not receive replication data: server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.

At minimum Bottled Water should ignore such tables instead of crashing; preferably it should do a best effort to transform their names so as to work as normal despite the name.
A similar problem exists for columns with silly names:
CREATE TABLE silly_column ("flobble-biscuits?whatisthetime/hahahaha" SERIAL NOT NULL PRIMARY KEY);
INSERT INTO silly_column DEFAULT VALUES;

This may be a duplicate of #53.

Closing as a duplicate of #53.


Asynchronous downloads and encoding

Still trying to figure out how to do this, I've tried putting together a download function with aiohttp multiple times and its failed in some way each time.
Currently attempting it again. There is so much mixed information in regards to downloading multiple files with aiohttp it gets a little crazy trying to figure out what actually works.
I know I need:

[ ] Function to "watch" the download process and keep the user informed
[x] Generator to create a list of just track urls
[ ] Download function

Must have a call to start an encoding process after each file which should run independently of the download process.

[ ] Initialization function

Edit: As an aside I will be going on a short hiatus after the release of 0.0.7 to catch up on other things. Once 0.0.7 is released bandcamp-dl will be pretty solid feature wise and lots of redundancy has been added. Its actually pretty hard to break it at this point:

You can shut the script down forcibly and it will just resume on next run

(without you needing to retype the commands, just run bandcamp-dl by itself)

You can interrupt a download in any way and it will re-download on next run

And if there are network issues it will retry a few times

You can shut the script down while encoding and it will just skip downloading and finish encoding on next run

..Note to self: add a check for byte-range downloads and split the process up into chunks.

I'm getting better at this, been doing some other projects (and even my own API) with async to get my head around it and so far so good, should be able to start making the codebase async relatively soon.

After some experiments there doesn't seem to be much of a performance benefit at least on the downloads side, may be a limitation of downloading from Bandcamp itself.
Haven't attempted to make the parser async yet but I don't anticipate much of a performance increase as we only make a single request (to create our single BeautifulSoup object)
Now if/when multiple album, artist, discography downloading is a thing it would be different as we would then need to request the individual bandcamp pages themselves which aren't throttled of course.

Not going to happen, after multiple branches trying various speedups I found no solution that did not require using proxies to download individual files at the same time without being limited.


openomf - compile updated binary, fix gptokeyb mappings and remove log.txt

Tested on my Odroid Go Ultra and Game Consoie R33S, both running JELOS

Draft for now, needs testing on some other devices

Testing thread: https://discord.com/channels/1122861252088172575/1209828793074720808

Updated gptokeyb mappings to match default 'Right keyboard' control scheme, now works OOTB. Made script and binary executable and cleaned up superfluous gamedata files as per https://github.com/omf2097/openomf/blob/master/INSTALL.md#data-files


Create jekyll-docker.yml

try this

Now that I know what I need to use Jekyll 4.3.4, this is abandoned!
Because github-pages >= 230 depends on jekyll = 3.9.5
and Gemfile depends on jekyll = 4.2.2,
github-pages >= 230 cannot be used.
So, because Gemfile depends on github-pages ~> 231,
version solving has failed.


Node example doesn't work, handshake timeout

I get this error when running the Node example:
{"MessageId":"e61ab986-c9c4-4b9c-a557-0c7fe13ee4c5","CreatedDate":"2021-10-01T09:24:34.917Z","DestinationId":"9eb2aa6d-936c-44a9-80aa-2c163fd558d8","SessionId":"xxxxxxxxxxxx-00326d985ae62b3af","MessageType":"channel_closed","SchemaVersion":1,"Output":"\n----------ERROR-------\nEncountered error while initiating handshake. Handshake timed out. Please ensure that you have the latest version of the session manager plugin."}
I've updated the session manager plugin to the latest and the issue is the same.

I was starting SSM with DocumentName: "AWS-StartSSHSession", removing it fixes the issue


Fix Typo

Description 📖
Everything else on the page says it should be "dev", not "serve", so I switch this one to match the others.

Consistency is nice, thanks Joe!


Default Props are included in output

These should not be there!

:rocket: Issue was released in v7.3.4 :rocket:


#1 make creation scripts idempotent

For individual proc and table creation scripts.

My first pull request! (After deleting a few, heh.)
To see what changed, click on the "Files changed" tab at the top of this screen. The left side shows the files as they used to be, and the right side shows the new lines I added to those files. If it looks okay, click the Back button in your browser, and then when you're back on this page, click the "Merge pull request" button.
Anytime someone sends you a pull request, make sure it says "Merging can be performed automatically" - if it doesn't say that, then you should leave them a comment asking them to update their branch. It usually means they worked on their own branch for too long of a time, and you made changes since. They can sync on their side pretty easily, but it's not fair to ask you to do that work (because it can be pretty painful to fix conflicts on the maintainer's side.)

Strange...  Github is treating all 5 changed files as binary so won't show diffs.  Is that happening for anyone else here?   (yes I have a dark theme.  no that isn't the cause {hoping there's going to be no egg on my face!})

Yeah, it's due to this: https://stackoverflow.com/questions/28145687/why-does-git-think-my-sql-file-is-a-binary-file

What would be the drawbacks to converting the scripts to UTF8? That would solve the diff issues and shouldn't have any impact on executing the scripts from SSMS.

Yeah, totally agreed - that's how we handle our own stuff, too. You wanna do a PR for that?

Created. :) PR #7

@dsolodow thanks sir! If he accepts that one, I'll close this and redo it.

Closing this pull request because I need to redo the code with the new UTF8 encoding.


[TE9][TC-OO-2.4] wrong TH implementation when DUT has to be rebooted

In TC-OO-2.4 there are several test steps where the DUT should be rebooted. This is wrongly implemented in the TH. It tries to reboot the Raspi.
A better solution would be to ask the User to reboot the DUT instead.

Same as https://github.com/CHIP-Specifications/chip-test-scripts/issues/74


Alias preprocessor define is working?

I tried to use the alias function names define, and as best I can tell it didn't do anything. Looking at the code, it isn't clear to me how it's supposed to work. I am on the latest 27 Sep commit.
#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# define printf_    printf_
# define sprintf_   sprintf_
# define vsprintf_  vsprintf_
# define snprintf_  snprintf_
# define vsnprintf_ vsnprintf_
# define vprintf_   vprintf_
#endif

// ... some stuff here ...

#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# undef printf_
# undef sprintf_
# undef vsprintf_
# undef snprintf_
# undef vsnprintf_
# undef vprintf_
#endif

I temporarily hand-edited my local printf.h file and went back to way function name aliasing worked in the previous 17 Aug commit. This works, and I understand why it works (unlike the current commit).
#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# define printf    printf_
# define sprintf   sprintf_
# define vsprintf  vsprintf_
# define snprintf  snprintf_
# define vsnprintf vsnprintf_
# define vprintf   vprintf_
#endif

// ... some stuff here ...

I did read the issue #14 notes, but they didn't explain what I'm experiencing. Regarding the 27 Sep commit, am I missing something about how to make it work and/or understanding how it should work?
Thanks,
-Chris

Hmm. I think I might have gotten that wrong. In the recent commit, change the first set of directives to:
#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# define printf_    printf
# define sprintf_   sprintf
# define vsprintf_  vsprintf
# define snprintf_  snprintf
# define vsnprintf_ vsnprintf
# define vprintf_   vprintf
#endif

and that should do it.

I did mess that up. Working on it.

@thestumbler : Can you please try it again now?

I just tested it, the latest commit on the develop branch. No joy.
First of all , my compiler (gnu arm-none-eabi-g++ v 6.3.1 Atmel build for Atmel/Microchip SAM-E5x chip) doesn't like the preprocessor #if statement without a conditional (the second line from the snippet below)
#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
#if PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# define printf_    printf
# define sprintf_   sprintf
# define vsprintf_  vsprintf
# define snprintf_  snprintf
# define vsnprintf_ vsnprintf
# define vprintf_   vprintf
#endif
#endif

Even when commenting this out in all three places (twice in the h file and once in the c file), it doesn't actually work.
I got it to work by basically putting it back like before:
(1) swapped all of the definitions around at the top of the header file, e.g...
WAS: # define printf_    printf
 IS: # define printf    printf_

(2) removed the #undef block at the end of the header file, and removed the duplicate? block at the top of the c file.

"commenting this out in all three places..." I mean like this:
#ifdef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
//#if PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
# define printf_    printf
# define sprintf_   sprintf
# define vsprintf_  vsprintf
# define snprintf_  snprintf
# define vsnprintf_ vsnprintf
# define vprintf_   vprintf//
//#endif
#endif

First of all , my compiler (gnu arm-none-eabi-g++ v 6.3.1 Atmel build for Atmel/Microchip SAM-E5x chip) doesn't like the preprocessor #if statement without a conditional (the second line from the snippet below)

Ok, I'll change it to
#ifndef PRINTF_ALIAS_STANDARD_FUNCTION_NAMES
#define PRINTF_ALIAS_STANDARD_FUNCTION_NAMES 0
#endif

Even when commenting this out in all three places (twice in the h file and once in the c file), it doesn't actually work. I got it to work by basically putting it back like before:

The thing is, I can't do that. We can't go around replacing print's in people's source code, outside the library, with printf_. Those replacements will happen in comments, in C++ namespaces... can't have that. The macros must be undefined at the end of the header.


Some improvements

Added some keywords;
Removed unused keywords;
Add typetag support (new Anything:variable);
Improvements for directives (tryinclude, if defined, elseif, undef,
pragma);
Removed some C99 stuff;

Thanks for your contribution.
Don't mind the confusion i made (releasing two versions at once), i've forgotten how to use apm publish.


Cleans notebook to plot number of docstring errors

Updates #142 by cleaning the notebook code for better readability.
Closes #139.

Looks really cool, much better now. Not very important, but couple of things:

inplace will probably be removed in the future, I wouldn't use it in general, and by not using it you'll be able to add those calls to the pipe as the others
In PEP-8, the spacing is foo = 1, but foo(bar=1) (you have spaces around the operator in the second case
pandas has an assign method that would let you create columns in a pipeline. If you wanted to join the 4 first cells in one, and have a single pipe, I think that would be possible

Thank you, @datapythonista ! I tried removing inplace from the code. Though I was able to call the function in the pipe, the output deviated from the expected. I am not sure about the work around without inplace but shouldn't the output remain unaffected if it is removed in future?
I will try to make the code better with your suggestions until then.

If the output of df.whatever(inplace=True) is different than df = df.whatever() that's something wrong in pandas IMO. But I don't think that's the case. If you want to give it another try, and find a minimal example to replicate the problem, that would be useful.


Error with Default run

I have installed the fresh onioningestor from git and on first run it gave following error:
(it also gave error about monitoring.txt not there but I created a file with single onion link)
$ python -m onioningestor --config onioningestor.yml
[23 Jul 2020 18:38:05] - INFO - Starting OnionScraper
[23 Jul 2020 18:38:05] - INFO - Loading config file
[23 Jul 2020 18:38:05] - INFO - Creating Elasticsearch mapping
[23 Jul 2020 18:38:05] - INFO - Onions will be saved synchronously
[23 Jul 2020 18:38:05] - INFO - Initializing simple-html
[23 Jul 2020 18:38:05] - INFO - Initializing onionscan-go
[23 Jul 2020 18:38:05] - ERROR - __init__() takes 2 positional arguments but 4 were given
Traceback (most recent call last):
  File "/home/user1/OnionIngestor/onioningestor/__init__.py", line 66, in __init__
    for name, operator, kwargs in self.config.operators()}
  File "/home/user1/OnionIngestor/onioningestor/__init__.py", line 66, in <dictcomp>
    for name, operator, kwargs in self.config.operators()}
TypeError: __init__() takes 2 positional arguments but 4 were given

Current onioningestor.yml:
# This is an example ThreatIngestor config file with some preconfigured RSS
# sources, feeding extracted artifacts into a CSV file.

general:
    # Run forever, check feeds once an hour.
    daemon: True
    sleep: 10
    onion_validation: ([a-z2-7]{16,56}\.onion)
    blacklist: porn,cvv
    interestingKeywords: leak,deface,hack,ddos,exploit
    save-thread: no         # Use a separate thread to save onions
    TorController:
        port: 9051
        password: mysecreatetorpass

monitor:
    filename: monitoring.txt

sources:
    # A few threat intel blogs to get you started!
    - name: simple-text-file
      module: simplefile
      filename: onion_master_list.txt

    - name: source-gist
      module: gist
      url: https://gist.github.com/search?l=Text&q=.onion

    - name: source-reddit
      module: reddit
      url: https://api.pushshift.io/reddit/search/comment/?subreddit=onions&limit=1000000
      feed_type: messy

    - name: pastebin
      module: pastebin-account
      url: https://gist.github.com/search?l=Text&q=.onion
      feed_type: messy

    - name: hunchly-report
      module: gmail-hunchly
      url: https://gist.github.com/search?l=Text&q=.onion
      feed_type: messy

    - name: onionland-search
      module: collect-onions
      url: http://3bbaaaccczcbdddz.onion/discover
      feed_type: messy

    - name: torch
      module: collect-onions
      url: http://xmh57jrzrnw6insl.onion
      feed_type: messy

operators:
   - name: simple-html
     module: html
     timeout: 300
     retries: 2
     interestingKeywords: leak,deface,exploit,hack
     socks5:
         http: 'socks5h://<IP_ADDRESS>:9050'
         https: 'socks5h://<IP_ADDRESS>:9050'

   - name: onionscan-go
     module: onionscan
     binpath: /home/user1/go/bin/onionscan

   - name: simple-screenshot
     module: screenshot
     screenshots_path: /home/user1/screenshots/

#  - name: yara-rule
#    module: yara
#    filename: categories.yar
#    base_score: 50

database_Engines:
    # Simple telegram notifier
    - name: telegram-notifer
      module: telegram
      chat_id: YOUR-TELEGRAM-CHAT
      token: YOUR-TELEGRAM-TOKEN

    - name: elasticsearch
      module: elasticsearch
      index: onioningest
      port : 9200
      host : <IP_ADDRESS>

#  - name: email
#    module: send_email
#    alert: no             # Enable/disable email alerts
#    from<EMAIL_ADDRESS>#    to<EMAIL_ADDRESS>#    server: <IP_ADDRESS>     # Address of the server (hostname or IP)
#    port: 25              # Outgoing SMTP port: 25, 587, ...
#    tls: no               # Enable/disable tls support
#    username: ''          # (optional) Username for authentication. Leave blank for no authentication.
#    password: ''          # (optional) Password for authentication. Leave blank for no authentication.
#    subject: '[onioningestor] - {subject}'
#    size-limit: 1048576   # Size limit for pastie, above it's sent as attachement

Hi,
The error is thrown because only the simple-text-file module is currently tested and supported!
I'm still working on other sources at the moment. To fix the issue I suggest commenting out all under-development sources in the config file and remember to create a list of onion links and name the file onion_master_list.txt as in the config file.
onion_master_list.txt file should look like this:
onionlinkaddress1.onion
onionlinkaddress2.onion
...
etc

For operators simple-screenshot is still under development so please comment it also in the config file. The onionscan-go module works fine. Have you already installed it? Before using it as a module I suggest running a test scan using onionscan.
onionscan --webport=0 --jsonReport --simpleReport=false YOUR-onionlinkadress.onion
There is an issue on the go binary. I haven't created documentation for this module yet but you can try it out yourself. The issue is that onionscan's link validation regex does not support V3 .onion address that is 56 char long. To fix this check out their issues
Finally, If you are not using telegram-notifer comment it out and the collected items should be indexed only on elasticsearch.
Hope this helps.


Updating segwit_wallet_dev info

Found a few typos and updated information about segwit activation and Bech32 native address types.

utACK 0c6b1fd946071a775d632d7e39c49925260e9c5b

Thanks @MarcoFalke, would be great to get an actual review for the automated pull-request tester.

Thanks the author, the improvements of code are improvements of life

Thanks for the feedback @harding -- all taken care of now.


Compatibility with Rails >= 4.0

Hi. Do you have any plans to make dj_mon compatible with rails 4.0 and higher? Thanks

Would love Rails 4 support!

any update? would also love rails 4 support!

Any fork you know which supports Rails 4.2? And what about Rails 5?

No commits in almost 2 years :( fear there won't be updates.
I got it to work in Rails 4 - I think the last gem doesn't support Rails 4 but if you checkout master and put it in lib and import with
gem 'dj_mon', :path => 'lib/dj_mon'
it works


chore(CategoryTheory): Make terminology for final functors consistent

Removes the remaining mentions of "cofinal functors".

Thanks!
bors merge


Add town token for Nucleus integration.

I've not tested this yet, will do so when I have a little more time.
This will add a {{town}} token to Nucleus to add the name of the town you are in.
(Please don't pull yet unless someone tests it first!)
Fixes #667

Is this active in the latest release?

Is this active in the latest release?

No, since the PR hasn't been merged yet.

Nice, thank you!


Fixed AutoSync error for Web resources

Fixed Issue #276 where a change in the Case of a file caused the logic to try to delete the Web Resource. Also changed some of the logic to throw the SparkleTaskException.

@scottdurow There isn't a good way to setup a unit test for this scenario. Right now, the setup for Web Resources just has the Web Resource project and no tests. It would almost need multiple file trees and a scripted execution folloed by a check for status to see if changes were applied or not.

This released as https://www.nuget.org/packages/spkl/1.0.370


Handle no-op step-repeat blocks better

The line %SRX1Y1I0.0J0.0*% appearing before anything has been drawn shouldn't affect anything, but it actually does change the SVG output by creating a wrapping group and <use>ing it. This, in turn, increases the complexity of the SVG file unnecessarily and messing up outline masks downstream in pcb-stackup (see tracespace/pcb-stackup#48).
This may be a problem here or in gerber-plotter.

Now tracking in tracespace/tracespace#81


1.16.0 new features

Hello, I have added some new code that will handle the notifications on Chill&chat, the notifications should be working once the API version with the notifications is up and running.
Please review soon.
Thanks!

Oh yes! I have also changed some config for app store stuff as outlined in code!


Compute relative paths to support sources outside a spec's directory

Closes #318
Previously, XcodeGen would determine the relative path to a source by removing the project's basePath from the source's path, but this doesn't work when the source's path is outside the base path. For example, given a base path /a/b and a path to a source ../c, XcodeGen would unsuccessfully try to remove "/a/b" from "/a/c" and wind up creating a "relative" file reference to "/a/c".
This change adds a method to Path that computes the relative path between two paths, inspired by methods like Pathname#relative_path_from in Ruby, and uses it to determine the paths for top-level references in the project.
I'm trying to decide whether I should really be contributing this to PathKit, since this solves a generalizable path problem, but since it fixes a specific bug in XcodeGen I wanted to put it up here. I'm happy to open a PR on PathKit in addition to this one, so that you don't get stuck maintaining a pathing algorithm 🙂

Thanks! It was fun to write :) Changelog updated.


What is RN50*4?

Hi, nice work！
I am confused about the backbone RN50*4 used in the paper. Is it a RN50 network with 4 stages FPN? And what is the difference with the RN50-FPN in ViLD?
Thank you for your kindly help.

@YifanXu74 Thanks for your interests. We didn't use any FPN in our backbones. It was a backbone used in CLIP. It's deeper and wider ResNet (see details of blocks in each stage at this link).

I got it! Thanks a lot!


add github sponsors

Add github sponsors, there may be someone from the community who donates money to keep the project active and running: https://github.com/datawan-labs/pg/pull/3

sorry for late response, not active at this project atm. thanks for help


Frontend Not connecting to VS Code

I'm not able to see new changes or commit changes in VSCODE, but my colloborators  are able to.

Fixed this issue!


Keep the description field intact on route schemas.

Otherwise, using this plugin destroys the ability to describe the endpoints when defining routes.

Superseded by https://github.com/turkerdev/fastify-type-provider-zod/pull/14


[FLINK-31348] Fix unclosed shorcodes (/tabs) in the new HUGO-based do…

https://issues.apache.org/jira/browse/FLINK-31348

CI fails with https://issues.apache.org/jira/browse/FLINK-31351


HTML code formatting broken by <% .. %>

Sample code

<% print('template code') %>

Select this code in LightTable, press Shift+Tab to auto-format.
Expected

        <% print('template code') %>

Result

        <% print('template code') %>

Reproduced in the atom-shell branch, which intended for the 8.0 release. This is probably a problem related specifically to CodeMirror, which LightTable uses for a lot of its editor functionality. I get similar (but not the same) results in this demo:
http://codemirror.net/2/demo/formatting.html
Are the <% %> template tags? CodeMirror does appear to support mixed mode formatting, but it probably needs a specific parser for whatever is in the tags. I'm not at all familiar with it's capabilities, however, nor it's integration with LightTable

I've verified this is a bug because we aren't providing file types for this mode. Fix coming soon

Next release will have 4 file types that handle this correctly


bring button stories back

The type of this PR is: Bugfix

This PR resolves MOPLAT-120
Description
The issue was with Wrap. It was returning children as a string, not {children} for the actual children.
@patrinoua check if its running locally for you too please 🙌.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

#nochangelog
Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against 8a01c7a338039a83726491bf02e18f36439542dc

lets #squashonmerge


Add example of using gcloud to run the samtools pipeline

Adds a script "poll.sh" which can be used to monitor the completion status of an operation.
Also includes some minor corrections to comments in other examples.

Nice update!  LGTM

LGTM


Error with PowerDNS ports in docker-compose.override.yml

I am having an error come up when I try to run docker-compose I copied my config file from the sample as stated in your guide and changed all the locations of the files and domains, am I missing something here.
ERROR: The Compose file './docker-compose.override.yml' is invalid because:
services.powerdns.ports is invalid: Invalid port "53/udp:53/udp", should be [[remote_ip:]remote_port[-remote_port]:]port[/protocol]

Hello thanks for using my little side-project.
It will probably work if you change it to 53:53/udp (https://docs.docker.com/compose/compose-file/compose-file-v2/#ports). This could be a change, introduced in some version of docker-compose, but I will definitely update the sample file.
Thanks for reporting this
Philipp

Ok thanks for that it's now working and thanks for making the project public.

I've updated the sample docker-compose.override.yml in ba7d494


Enable GZIP compression for JSON-RPC calls

Describe the feature
Introducing gzip compression for JSON-RPC calls can significantly reduce the payload size of the requests and responses, leading to faster communication and lower bandwidth usage. This enhancement can be particularly beneficial for applications making frequent or large JSON-RPC calls to the Ethereum network.

I believe Geth already does it by default https://github.com/ethereum/go-ethereum/blob/380688c636a654becc8f114438c2a5d93d2db032/node/rpcstack.go#L396-L404, sounds like a good idea.

That sounds like a plan :shipit:


Pond - Share XBB.1.5. SARS-CoV-2 NCBI list with Mazumder Team

Share list with @Jgergely11 and @steph-sing.

[ ]  @Jgergely11 will filter the list for accessions links to fastq files, which can be run through our respective ngsQC protocols, can be used to build new assemblies, and we can run the fasta's through our assemblyQC protocol.
[ ] @Jgergely11 and @stephenshank can annotate the list to track which accessions have fastq data

Once we have an idea of how much data we're working with, we can come up with a plan to QC/assemble new genomes.

@Jgergely11 The list of GISAID accessions and corresponding SRA accessions is here. I also have a FASTA with genomes corresponding to the GISAID  accessions that I can share privately (they are pretty sensitive about others sharing the data publicly).

Complete - Share annotated list when ready


Operators not supported by Lite routine

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
# Copy and paste here

Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Please fill in template, add the description of the issue in the message and use a much shorter title

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing as there was no further activity, template was not filled.


Where is the Guide folder?

Describe your problem and how to reproduce it:
I am trying to find the guide folder, but it that I cannot locate it. Was it remove temporarily or relocated to a different section?
Add a Link to the page with the problem:
Just take a look on the freeCodeCamp repository main section and and the guide folder is missing.
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version: Version 74.0.3729.169 (Official Build) (64-bit)
Operating System: Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Yes, the guide has been removed from this repo completely. See: https://github.com/freeCodeCamp/freeCodeCamp/pull/36709
The current production guide at guide.freecodecamp.org, will soon be moved over to our forum as a wiki.
The changes that were made over the past months will also be added to the forum wiki post migration.
We are no longer accepting any pull-requests for the guide on this repository.
Contribution Guidelines for the same will be made available soon. We request your patience.
I am closing this thread, but pinning this for visibility.


HTTP specializations for non-blocking

It is likely possible to provide overloads for asynchronous HTTP read and write algorithms which accept basic_socket streams to work in non-blocking mode.

The implementation can check the socket and see if it is in non-blocking mode, and switch to a non-blocking algorithm.

This is done, and it turned out to be slightly worse performance-wise, on Windows at least. Code for it is here:
https://github.com/boostorg/beast/blob/develop/include/boost/beast/core/detail/impl/read.hpp#L253


add options for build idris package

Motivation for this change
It could be helpful to provide additional options to idris commands while building, testing and install an idris package.
For my specific use case, I need to add a --log option to idris commands.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ x ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ x ] Ensured that relevant documentation is up to date
[ x ] Fits CONTRIBUTING.md.

It might be better (though less convenient) to pass options as a list of strings instead of a string directly, such that users don't need to worry about proper escaping (can be done in the code with lib.escapeShellArgs).

@Infinisil that seems like a nice suggestion. I updated the code and the docs accordingly

Could you squash the last commit into the first two? This looks good to merge otherwise

done! Thanks @Infinisil for your feedback!


data migration - automatic migration scripts

Write scripts to automatically generate exports from the old database on a nightly basis, and save them to an accessible location.
Automate a process to determine what values need to be updated in the new system and handle modifying/adding data automatically.

This is no longer applicable, the data migration doesn't require automated scripts.


implement verify commands - useful iff cassandra-5791 is accepted

If the submitted patch for https://issues.apache.org/jira/browse/CASSANDRA-5791 is accepted, this patch will add corresponding functionality to ccm.

Thanks Jeff! I will merge this in once 5791 is committed. Until this is in, you can always run ccm node nodetool verify as a longer syntax command to do the same operation.

@ptnapoleon : CASSANDRA-5791 went in, this is probably ready (at your convenience)

Sorry I missed that, I thought I was watching that ticket, but apparently not.


Deliver fails to create a new app version on App Store Connect

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Deliver fails to create a new app version on App Store Connect
Command executed
lane :submit_review do   deliver(     submit_for_review: true,     automatic_release: false,     force: true, # Skip HTMl report verification     skip_metadata: true,     skip_screenshots: true,     skip_binary_upload: true,     run_precheck_before_submit: false,     submission_information: { export_compliance_uses_encryption: false,                               add_id_info_uses_idfa: false }   ) end
Complete output when running fastlane, including the stack trace and command used
[17:42:54]: Cannot find edit app store version... Retrying after 10 seconds (remaining: 4)
[17:43:06]: Cannot find edit app store version... Retrying after 10 seconds (remaining: 3)
[17:43:17]: Cannot find edit app store version... Retrying after 10 seconds (remaining: 2)
[17:43:27]: Cannot find edit app store version... Retrying after 10 seconds (remaining: 1)
[17:43:38]: Cannot find edit app store version... Retrying after 10 seconds (remaining: 0)

and fails with

[17:43:48]: Cannot update languages - could not find an editable version for 'IOS'

Trying to upload a new version of an app to App Store Connect, Deliver fails when it tries to upload metadata because it didn't create a new app version successfully on App Store Connect.

Environment
Fastlane 2.217.0

Same here, started happening somewhere around yesterday noon (EST). We release updates daily stopped working today.

I had the issue this morning but not anymore. Probably a temporary issue with App Store Connect? @ArthurAlvarez can you try again?

Same issue. When I try to create the version in App Store Connect manually, I get an error that the version number is already in use. After a page reload, the new version magically appears.

My pipelines still fail consistently

This could be a legitimate issue with App Store Connect API, e.g. http 500 or timeouts. I've been monitoring ASC API on my own an noticed that it's been partially degraded for the past ~9 hours, see: https://x.com/rogerluan_/status/1734206562418258125
My monitoring system didn't detect anything prior to 9 hours ago though (e.g. 3-4 days ago everything seemed fine on their end 😬) so it could be a different issue that is impacting you guys!
Just sharing what I got 😅

Still happening. Is there any workaround or fix?

FWIW I'm still seeing intermittent issues (HTTP 500) with App Store Connect API calls. You can monitor them here:
https://status.statused.com
Or more specifically here (for further downtime details): https://stats.uptimerobot.com/1kzByfnDjz/795501684

Same issue here. Is there a way to increase the retry interval or number of retries?

Same issue here. We're delivering builds via Transporter while this is going on. Is this an Apple or a fastlane issue? Any idea of what the resolution would be?

Looks like an issue on Apple servers

Hi @rogerluan but the chances of Apple fixing their code anytime soon are pretty low so maybe fastlane can work around it somehow? They did their 10 second retries so maybe something else needs to be incorporated?

In an attempt to mitigate this, we have updated our release flows to first trigger an independent call to
ensure_version!(version_string, platform: nil, client: nil),
then we actually build the app and trigger the upload.
Our assumption (that might be totally wrong) is that between the call that Fastlane does to create a new version in the App Store and the upload itself, there is not enough time for ASC to properly propagate the changes. We hope that by introducing a few minutes delay between the 2 operations the release will be successful.

Hi @dcacenabes, thank you, is is this in the latest fastlane or do we need to call a special version somehow? I'm a bit of a newbie.

for this you need to work with the app store connect API directly.
Spaceship::ConnectAPI.token = "this_is_your_token"
app = Spaceship::ConnectAPI::App.get(app_id: "your_app_id_in_app_store_connect)
app.ensure_version!("X.Y.Z", platform: Spaceship::ConnectAPI::Platform::IOS) # X.Y.Z is the semantic version of the new release for your app

You can find information on how to authenticate in the official docs:
https://github.com/fastlane/fastlane/blob/master/spaceship/docs/AppStoreConnect.md#login

Thanks @dcacenabes!

I was able to mitigate the issue simply by increasing the timeout time to 60 seconds instead of 10. You can check PR #21740
I hope Apple fixes the issue on their side soon. For now, that's the easiest solution.

Hi @rogerluan but the chances of Apple fixing their code anytime soon are pretty low so maybe fastlane can work around it somehow? They did their 10 second retries so maybe something else needs to be incorporated?

I've opened a PR that should resolve the issues you guys are seeing: https://github.com/fastlane/fastlane/pull/21742
@ everyone, if you wanna help (and increase the chances of your CI succeeding 😇), you can help test this PR of mine in the wild 🙇 this would be tremendously helpful and I'd be super grateful! ☺
Looking forward to hearing feedback from the community whether the proposed solution works 🙏

@ArthurAlvarez I reviewed your PR and I proposed a different solution (see PR above), I hope you're okay with that! 🙇 thank you for taking a stab at fixing this issue, though! Super helpful! ❤

This is still a significant issue for us.  I build hundreds of apps using Fastlane and approx. 50% of my builds fail.  I'm using Fastlane version 2.219.0.

Just ran into this issue for the first time, running my first release train of the year. Our CI release flow uploads a release build to TestFlight for internal QA and then holds on an approval job - once approved it runs the last CI job that actually submits the build for review using deliver. I ran that today and it failed with the error mentioned above. I had to run it three times before it was successful.

I experienced the same problem today too.

I wonder when this will be fixed, because half of the assemblies simply don’t work....

Currently there is no working solution to this problem. #21742 adds some retries with exponential backoff to increase the chances of your lane succeeding but it may make your CI/CD task take a long time to complete and even then it may still fail.
Personally, whenever I release a new version of my app on the App Store, I am now manually creating the next app version on App Store Connect and then any subsequent lanes that upload that version will work as always.

This issue is still not fixed (at least for Github Workflow. I'm using Fastlane version 2.219.0 (latest as of now) and I still get Cannot update languages - could not find an editable version for 'IOS'

had the same issue as well with the latest fastlane version

Still happening for me - every time I run our workflow to submit for review, it fails this way and has to be retried. The job succeeds the second time.


How to add Interceptor/EventListener to all OkhttpClient

Maybe your project have serveral OkhttpClients，you need to add your custom   Interceptor/EventListener to every OkhttpClients one by one. But some OkhttpClients come from 3rd library, and you can't add your custom  Interceptor/EventListener to them.
Is there any way to add  add Interceptor/EventListener to all OkhttpClients conveniently.

Unfortunately not, you should file an issue with those libraries to allow you to share clients.  As well common interceptors, it should provide benefits of sharing connections on the device.

Consider using dependency injection? It's my favorite way to make configuration uniform within an application.

We also need this feature to do accounting and monitoring logic. Add interceptors case by case is error prone, not future proof, and sometimes not doable.

Hi, recently, I develop a plugin based on asm and gradle transform api to insert global interceptor for every okhttpclient, you can find it here https://github.com/Leaking/Hunter#okhttp-plugin
You can have a try.

Consider using dependency injection? It's my favorite way to make configuration uniform within an application.

Is that possible when the third party library that use OkHttpClient do not use DI and we can not change that situation ?

We also need this feature to do accounting and monitoring logic. Add interceptors case by case is error prone, not future proof, and sometimes not doable.

Hi, recently, I develop a plugin based on asm and gradle transform api to insert global interceptor for every okhttpclient, you can find it here https://github.com/Leaking/Hunter#okhttp-plugin
You can have a try.

Yes, this is the ideal way to do it, but a little overkill. Will try it

Doesn't work quite as well when the code creating the OkHttpClint is in another APK does it? (I.e, Tachiyomi.)
In particular, suppose you're want to use something like Stetho that needs to be set up centrally but also needs an interceptor on each client; for this to work, you'd have to add something to the main APK's ABI, and then use that in extension APKs to set up the interceptor(s), which would make those APKs depend on the new main APK, which can be a pain when testing on a phone you use external builds of Tachiyomi on because then all of a sudden that extension can't load in the release build and you have to switch back to a release build of the extension to read your manga :-(. (Or, you know, migrate your state to a self-built "release" APK with the new ABI also.)
I was really hoping for a dynamic instrumentation option. (Sure would be nice if most of the work on dynamic instrumentation for Android didn't assume you wanted to instrument someone else's APK.)
I guess the simplest approach would be a way to set a list of interceptors to add to all new clients (and then remove when creating a builder from those clients, I guess, to avoid extra registrations). Then you could just add Stetho's interceptor to the list during app startup and be good to go.

(Obviously, it could just as well be any other diagnostic facility with a central component and an interceptor.)