Unable to activate The following constructors are ambiguous

`System.InvalidOperationException: Unable to activate type 'reCAPTCHA.AspNetCore.RecaptchaService'. The following constructors are ambiguous:
Void .ctor(reCAPTCHA.AspNetCore.RecaptchaSettings)
Void .ctor(Microsoft.Extensions.Options.IOptions`1[reCAPTCHA.AspNetCore.RecaptchaSettings])
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.CreateConstructorCallSite(ResultCache lifetime, Type serviceType, Type implementationType, CallSiteChain callSiteChain)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.TryCreateExact(ServiceDescriptor descriptor, Type serviceType, CallSiteChain callSiteChain, Int32 slot)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.TryCreateExact(Type serviceType, CallSiteChain callSiteChain)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.CreateCallSite(Type serviceType, CallSiteChain callSiteChain)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.<>c__DisplayClass7_0.<GetCallSite>b__0(Type type)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteFactory.GetCallSite(Type serviceType, CallSiteChain callSiteChain)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngine.CreateServiceAccessor(Type serviceType)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngine.GetService(Type serviceType, ServiceProviderEngineScope serviceProviderEngineScope)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetService[T](IServiceProvider provider)
   at reCAPTCHA.AspNetCore.Attributes.ValidateRecaptchaAttribute.CreateInstance(IServiceProvider services)
   at Microsoft.AspNetCore.Mvc.Filters.DefaultFilterProvider.ProvideFilter(FilterProviderContext context, FilterItem filterItem)
   at Microsoft.AspNetCore.Mvc.Filters.DefaultFilterProvider.OnProvidersExecuting(FilterProviderContext context)
   at Microsoft.AspNetCore.Mvc.Filters.FilterFactory.CreateUncachedFiltersCore(IFilterProvider[] filterProviders, ActionContext actionContext, List`1 filterItems)
   at Microsoft.AspNetCore.Mvc.Filters.FilterFactory.GetAllFilters(IFilterProvider[] filterProviders, ActionContext actionContext)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvokerCache.GetCachedResult(ControllerContext controllerContext)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvokerProvider.OnProvidersExecuting(ActionInvokerProviderContext context)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ActionInvokerFactory.CreateInvoker(ActionContext actionContext)
   at Microsoft.AspNetCore.Mvc.Routing.ActionEndpointFactory.<>c__DisplayClass7_0.<CreateRequestDelegate>b__0(HttpContext context)
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.Invoke(HttpContext httpContext)
--- End of stack trace from previous location where exception was thrown ---
   at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
   at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.StatusCodePagesMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)`

I have the same problem

Happens when using
services.AddRecaptcha(options =>
{
    options.SecretKey = "Your secret key";
    options.SiteKey = "Your site key";
});

instead of
services.AddRecaptcha(Configuration.GetSection("RecaptchaSettings"));

Thanks, this has been reproduced. Will be fixed when time permits or if @mo-esmp can address it as it's from the feature they added.

I will fix it @TimothyMeadows. Sorry for that.

I merged the fix for now in case someone needs to compile it out of band in an emergency. However i am not sure if removing IOptions was the correct solution. I will validate this weekend and update if a better solution is not found.

Any update on this?

This should be updated in the next day, or two. Including the fix for #61 Apologize for all the delays, crazy times these days.

This should now be resolved in 3.0.4


Add a user feedback mechanism. Perhaps direct to Discord?

Add a button for the user to provide feedback. When tapping it launches discord in a product feedback channel.

Think it can just be an invite link to a channel
Ser, please


Steam bot starting but not accepting offers

So i was testing my bot and it doesn't seem to accept my offers it doesn't come with errors and i have disabled email verification thing.
Here's a screenshot:
http://prntscr.com/90uoof
And i also have a issue with the ip thingy.

Not a help desk.


Add topolgySpreadConstraints support to helm

Add topologySpreadConstraints to Helm Charts
This PR introduces topologySpreadConstraints to the Helm charts to improve availability and fault tolerance in multi-zone Kubernetes clusters. The changes ensure that pod replicas are evenly distributed across zones using the topology.kubernetes.io/zone label, minimizing the risk of zone-specific failures.
Key Changes:
Topology Spread Constraints: Evenly distributes replicas across zones with a configurable maxSkew.
These changes are optional and configurable via values.yaml, providing better fault isolation without breaking existing deployments.

You're welcome @gianlucam76
Yes, the problem is precisely what you mention.
In my scenario, I have a multi-zone cluster. If I lose one zone, I want the cleaner to work without any disruption.
So, I increased the replicas and used affinity and topologySpreadConstraints. It's working fine now(I know it depends on the instance and could cause problems for some cleaner instances).
So, I opened this PR and hope it works for someone else until the implementation of sharding or leader-election.
Thanks!


Get active equities

Take a look at #34, let's use that format (TypedDict) for the Tastytrade JSON objects that don't need extra functionality, so TickSizes and DestinationVenueSymbol.

I'm going to go ahead and merge this so I can get the other PR in


Add a tutorial to the Calculate activity

Add a tutorial to the Calculate activity following tutorial 9 in Sugarizer development tutorial: https://github.com/llaske/sugarizer/blob/dev/docs/tutorial_step9.md
The tutorial:

[ ] Should display a welcome message to explain the activity (could reuse the help message describing the activity in Sugarizer activities List view)
[ ] Should explain usage of each button in the toolbar
[ ] Should be localized
[ ] Should respect the Sugarizer UI, as explained in tutorial

Fixed in https://github.com/llaske/sugarizer/pull/537


Change promote branch name from - to env//.

This means that there can now be one PR pending per app per environment, where there could previously be only one PR per app. Put differently, a change that has been promoted will no longer be replaced by a later new.
Fixes #135. See for details.

I don't quite trust this change yet; additional testing will be needed.

@bittrance should we enable this behind a flag - disabled by default? Then at a later stage after we have been running it for a while in production, we can enable it by default.

@simongottschlag Indeed, because rolling it out may require manually deleting or renaming branches.

We are using tags on all our pipelines so if users don't want to update they don't have to.
As we all know we are using many templates on templates features, by using feature flags we will have to update templates not once but 2 times.
I think it will create two much administration to put this change behind a feature flag but that is my 2 cents.

In order to get this PR to pass, I have had to update gitops-promotion.yaml in the respective testing orgs to include prflow: per-env so as to be able to get working e2e tests.


[PT Run] Fix web link with ports support (#14260)

URL in the format of domain:port now directs to default browser

Add tests to verify web link with ports scenario

Fix test case and scenario where mismatching schema and port for IPv6 does not result in correct output

Summary of the Pull Request
Fixes issue #14260, and additionally fixes an unnoticed bug when IPv6 address has a port, the port number will be dropped from the URI

PR Checklist

[x] Closes: #14260
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[x] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
No manual validation steps, automated tests.

Added more tests for tel since in monkey testing I realised it actually breaks. Will attempt to fix.
Specifically tel:xxxx will break if xxxx is not preceded by a plus, and is more than 65536.

Added more tests for tel since in monkey testing I realised it actually breaks. Will attempt to fix.

okay

Specifically tel:xxxx will break if tel:xxxx is not preceded by a plus, and is more than 65536.

Oooh. Crazy.

The text for open default browser makes no sense. We could update it to "Open default browser":

I'll change it in the following commit.

Do you think it will confuse the users that default ports (443, 80, 21, ...) are cut of:

Maybe we should avoid changing the user's input? Not just less confusing but also less likely to have breaking issues. Additionally, if I'm not wrong this can be done by just removing two lines of code.

Do you think it will confuse the users that default ports (443, 80, 21, ...) are cut of:

~Maybe we should avoid changing the user's input? Not just less confusing but also less likely to have breaking issues. Additionally, if I'm not wrong this can be done by just removing two lines of code.~
Edit: No, the default behaviour for UriBuilder is to ignore the port number, it seems. Not sure how it can be changed. Even when explicitly declaring port in UriBuilder, when the UriBuilder.toString() is called the port will be dropped from the string conversion.

I think we should leave it as is. The protocol changes based on port the the functionality isn't affected.

mySite:80 => http://mySite
mySite:443 => https://mySite
ftp://myFTP:21 => ftp://myFTP


Silk.NET backed Skia

Just a basic Silk.NET backed Skia demo on OpenGL. We could consider adding this as a demo, instead of a Lab app.
@Perksey, thoughts?

Yeah making it an official demo seems fine, will review it accordingly though.

If you don't want to graduate this from the Lab you can discard all of the above comments, just want to ensure we keep a high standard for our tutorials.

I have no idea why, but it automatically requested a review from the codeowners :thinking:

Oh just noticed you moved this into the lab. Happy to merge once conflicts are resolved.

Just me or is GitHub refusing to merge this without a reason?

Nah, the conversations must be resolved. It's a new requirement they've added, and I enabled it as I figured it'd make us look at all the comments again to make sure they're actually resolved. I'll resolve them real quick as this is the lab and we don't care.


feat: test template from profile examples

Description

This PR add ability to prepare test file for specific profile and provider. Result for prepare:test computer-vision/face-detection google-apis:
import { SuperfaceTest } from '@superfaceai/testing';

describe('computer-vision/face-detection/google-apis', () => {
  let superface: SuperfaceTest;

  beforeEach(() => {
    superface = new SuperfaceTest({
      profile: 'computer-vision/face-detection',
      provider: 'google-apis',
    });
  });

  describe('FaceDetection', () => {
    // specify test case name
    it('should map error', async () => {
      const result = await superface.run({
        useCase: 'FaceDetection',
        input: {
          code : '14',
          message : 'We can not access the URL currently. Please download the content and pass it in.',
        }
      })

      expect(result.isErr()).toBe(true);
      expect(result).toMatchSnapshot();
    });

    // specify test case name
    it('should perform successfully', async () => {
      const result = await superface.run({
        useCase: 'FaceDetection',
        input: {
          imageUrl : 'https://upload.wikimedia.org/wikipedia/commons/c/c3/RH_Louise_Lillian_Gish.jpg',
        }
      })

      expect(result.isOk()).toBe(true);
      expect(result).toMatchSnapshot();
    });
  });
});

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have updated the documentation accordingly. For updating Oclif commands documentation use oclif-dev.
[ ] I have read the CONTRIBUTION_GUIDE document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I would also consider specifying test instance in constructor since it's already generated for jest.


[Brother] Color density is low

The standard combination of CUPS and the Brother drivers for the HL-L3230CDW and the HL-1110 printers, is that the BLACK text is in fact, not black, it's a bit too far on the pale size of gray, and the colour images on the CDW printer are quite faded, instead of being STRONGLY COLOURED.
AND what really pisses me off is that everyone seems to have overlooked this, AND there is NO colour density control for the CDW and sort of not much control or variation in the 1110 printers.
It's all stuck in "economy / ink saver mode".
It's nice that the printer runs - but it's not particularly great that the print is very pale.
AND now today when it really counts - on the kind of highest end colour documents - there IS not adjustment on the CDW printer drivers / cups.
I will help if I can, to get this operating properly...

Hi @Phuxxi,
what is your OS, CUPS version, cups-filters version? Can you attach your PPD here with .txt suffix?
If your driver is classic (non-IPP everywhere or driverless) one from Brother and using non-standard PPD keyword for density or any other density setting is missing, then we can't help much - such driver takes care of the setting by itself.
If the printer uses IPP Everywhere/AirPrint, then we probably can influence the output - however IIRC cups-filters loads color profiles from color manager as well, which might have influence on it too.
@michaelrsweet how do we actually handle color density? Is it related to color spaces which we get from urf-supported and pwg-raster-document-types-supported?


Support multi ranges

Each postgres range type supports whats called a multirange

Every range type has a corresponding multirange type. A multirange is an ordered list of non-contiguous, non-empty, non-null ranges. Most range operators also work on multiranges, and they have a few functions of their own.

E.g.
SELECT '{}'::int4multirange;
SELECT '{[3,7)}'::int4multirange;
SELECT '{[3,7), [8,9)}'::int4multirange;

We should decide if and how we want to support multi ranges, probably under another module prefix, like EctoRange.Multi.Date

@davydog187 sounds like a good idea to me 👍


Square FontAwesome icons exemption and custom icons

At present the theme has a list of non-square icon exemptions and all others are assumed to have a square version. With the addition of the feature to choose a custom icon as the third parameter in the SOCIAL variable, this breaks too often as the vast majority of icons do not have a square version. I am trying to use graduation-cap for Google Scholar for example. Wouldn't it make more sense to flip the priority now, so there's a check for a list of icons for which a known square exists, while the others are all passed as-is? Otherwise the ability to specify an icon is pretty much useless. I'm suggesting the following inversion, which I am happy to submit as a pull request if you think it makes sense:
{% if name_sanitized in ['behance', 'bitbucket', 'facebook', 'foursquare', 'git', 'github', 'google-plus', 'linkedin', 'pinterest', 'reddit', 'rss', 'tumblr', 'twitter', 'vimeo', 'youtube'] %}
    {% set iconattributes = '"fa fa-' ~ name_sanitized ~ '-square fa-lg"' %}
{% else %}
    {% set iconattributes = '"fa fa-' ~ name_sanitized ~ ' fa-lg"' %}

Hi!
First of all, thanks for your interest in this theme! It's with heavy heart that I have decided that I can no longer maintain this theme due to lack of time and changing interests. To makes sure the theme lives on however and not goes to waste, I've donated it to the Pelican community. It can now be found in the official pelican-themes repository. I'd suggest you re-submit your issue  there.
Sorry for the inconvenience!


Support dependencies parameter for the binary targets

Motivation
As it is described well in this swift forum thread, there is currently no straightforward way to define dependencies of a binaryTarget in a Package.swift file. This seems to be an obvious feature to have as it is available for other target types and clearly a binary target with some dependencies is a common use-case.
Proposed solution
An obvious solution would be to expose an optional dependencies: [PackageDescription.Package.Dependency]? = nil parameter when creating a binary target. I might be wrong, but there should not be any significant problems in reusing the implementation that the usual target uses.
Alternatives considered
No response
Additional information
The swift forum thread I've mentioned before contains a description for a workaround:
Considering that a binary target BinaryFramework has a dependency to SomeOtherFramework,
it is possible to define an auxiliary non-binary AuxiliaryTarget that will depend on both BinaryFramework and SomeOtherFramework, this way a client can define a dependency to AuxiliaryTarget so that both the BinaryFramework and SomeOtherFramework will be added.

@MaxDesiatov Please transfer this.

Please avoid pinging non-admins of these repositories with transfer requests, as only admins have transfer permissions. Thanks!

I attempted (poorly) to propose this a while ago https://forums.swift.org/t/accepting-swiftpm-binary-target-dependencies/60619

Please avoid pinging non-admins of these repositories with transfer requests, as only admins have transfer permissions. Thanks!

I am sorry to have bothered you. GitHub documentation states that you need write (not admin) access in both repositories. I genuinely was not expecting there to be something more to those words. Could you at least tell me exactly who am I supposed to be asking?

Apparently there are additional permissions required for transfers. @shahmishal would you be able to transfer this?


Remove Istio stackdriver metrics from XDS

Please provide a description of this PR:
Istio Stackdriver metrics are implemented using the OpenCensus SDK. As OpenCensus is deprecated, this PR removes the XDS programming of the Istio Stackdriver metrics from Istio.
Istio Stackdriver logs are not implemented using the OpenCensus SDK and are not removed in this PR.
Related issue: https://github.com/istio/istio/issues/50808.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@lei-tang: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test release-notes

unit-tests-arm64_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test release-notes

unit-tests-arm64_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test unit-tests-arm64

unit-tests_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

The difficult part is not removing it but making it so that it can be plugged back in a vendor fork. Can the code be made modular so that xDS programming is done only on a vendor fork?

The difficult part is not removing it but making it so that it can be plugged back in a vendor fork. Can the code be made modular so that xDS programming is done only on a vendor fork?

FWIW, perhaps flags/code comments/environment variables, etc may be used for vendor-specific xDS programming on a vendor fork.

Easiest (from maintenance POV) is just to embed envoyfilters into the filesystem. This is how it has been done by some in the past

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
8f0135b7c4c302f78f1972440911c60ec33c5cfe
link
true
/test unit-tests

unit-tests-arm64_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests-arm64

unit-tests_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests-arm64

unit-tests_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@lei-tang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests

unit-tests-arm64_istio
5b60aa8959348cb97540d55496beb09b31580857
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest-required

@lei-tang: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
5b60aa8959348cb97540d55496beb09b31580857
link
unknown
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test integ-pilot-multicluster


Add support for Polygon

What is this feature about? (1 sentence)
This makes the Safe available on Polygon.
Why is it needed? What is the value? For whom do we build it?

As part of our roadmap we decided to make the Safe available on more L2 networks.
Polygon is an L2 that got quite some attention recently.

High-level overview of the feature

The Safe on Polygon is available at https://polygon.gnosis-safe.io/app/.
Enabled wallets: Metamask only.
Enabled Safe apps:

Tx builder
WalletConnect (Needs gnosis/safe-react-apps#128)

Network info:

Name: Polygon
chain id: 137
Native coin name: Polygon
Native coin symbol: MATIC
Native coin decimals: 18

Network color in our interface: #8B50ED
Block explorer: https://explorer-mainnet.maticvigil.com/
Use a new Google analytics project with the following app params (cf. #1510)

app id: io.gnosis.safe.polygon
app name: Gnosis Safe Multisig (Polygon)
Property id: TODO (contact @tschubotz once this ticket is about to be implemented)

Addresses of deployed contracts can be found in https://github.com/gnosis/safe-deployments/tree/main/src/assets/v1.3.0

Deployment
Adjust the GitHub Actions to build the frontend bundle for Polygon and ask devops to deploy it.
Misc

Doc with more technical info about nodes etc: Doc

Here are some o f the necessary details to configure Polygon network
https://docs.matic.network/docs/develop/network-details/network/

I just saw that Infura also supports Polygon!

Could you please provide me these parameters ?

clientGatewayUrl
txServiceUrl
safeAppsUrl

Thanks!

I would assume that we use the same style url like on other networks.
i.e. for prod:

clientGatwayUrl: https://safe-client.polygon.gnosis.io
txServiceUrl: https://safe-transaction.polygon.gnosis.io
safeAppsUrl: https://apps.gnosis-safe.io (<- same for all networks)

@matextrem since TX Builder and WC are fixed for L2, let's re-enable Safe Apps.

And need to disable the spending limit feature as per Franco's comment.

[ ] Time to time there are 503 errors in console when I am working with the safe on polygon


Fix and update requirements.txt

Preparation for Release 0.14.3

@phschaad please use this pr to also change version.py


implement the pr label feature

this commit lays the ground work for adding labels to PRs depending on the CI Run Status.
The config type was already implemented but the actual labeling action was not.

?brawl try

?brawl try

?brawl try

?brawl try

?brawl try

?brawl try

?brawl try

?brawl try

?brawl merge


Accounting for phylogenetic relation between samples in the formula?

Hey!
I have two questions-

I have a biom file of microbial KEGG pathways for 100 host samples which are phylogenetically related. I was wondering if I can account for phylogenetic relation between host samples in the co-variates via patristic distance?

For example,
--formula "Depth+Temperature+patristic(phy)"

Are the other flags (as given below) default in songbird command?
--epochs 10000
--differential-prior 0.5
--summary-interval 1 \

Question 1 : See our paper on phylogenetic methods. It sounds like you want
something like phylogenetic GLS - but it's not clear to me.
I suggest looking at phylofactor : https://github.com/reptalex/phylofactor
Songbird does not support phylogenetic methods, and won't for the
foreseeable future.
Question 2 : the defaults should be in the command. Does it show when you
run it on the command line?
On Tue, Nov 12, 2019, 1:31 AM Jigyasa3<EMAIL_ADDRESS>wrote:

Hey!
I have two questions-

I have a biom file of microbial KEGG pathways for 100 host samples
which are phylogenetically related. I was wondering if I can account for
phylogenetic relation between host samples in the co-variates via patristic
distance?

For example,
--formula "Depth+Temperature+patristic(phy)"

Are the other flags (as given below) default in songbird command?
--epochs 10000
--differential-prior 0.5
--summary-interval 1 \

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/issues/97?email_source=notifications&email_token=AA75VXP3ZBY4OVZATAZJI4TQTJETRA5CNFSM4JL64RE2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HYTII2Q,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA75VXK6I6OG7ZJBUST4PP3QTJETRANCNFSM4JL64REQ
.


Tutor move damage updates

Expanding Force
Serebii and Bulbapedia disagree on the damage for expanding force.  According to Serebii, the base power is boosted by 1.5x in Psychic Terrain.  According to Bulbapedia, the terrain modifier is replaced by a 2x modifier.
However, a base power increase of 1.5 results in 1.5 * 1.3 = 1.95 total damage modifier — which is very close to 2x.  Until we have clear data on exactly which is correct, I opted for 1.5 damage modifier under terrain.  This would be consistent with other moves in the Isle of Armor release, and is much closer to accurate than the calculator is returning now.
https://bulbapedia.bulbagarden.net/wiki/Expanding_Force_(move)
https://www.serebii.net/attackdex-swsh/expandingforce.shtml
Misty Explosion
Serebii and Bulbapedia both now agree that Misty Explosion is boosted by 1.5x under terrain.
https://bulbapedia.bulbagarden.net/wiki/Misty_Explosion_(move)
https://www.serebii.net/attackdex-swsh/mistyexplosion.shtml

The Expanding Force mechanics are not confirmed, and serebii and bulbapedia are not reliable sources for nitty-gritty mechanics.

The 1.5x boost on top of the 1.3x terrain boost is what should be implemented for now.

Great, that's what I've done in this pull request.

Sure, sounds good


Add resource tags at a global/resource level to all resources deployed by CDK EKS Blueprints

Describe the feature
A way to define AWS resource tags on a per-resource level within a cluster template. These tags are then added to all resources (at least the ones that can have tags) that are deployed by CDK EKS Blueprints.
Use Case
To keep track of different resources from different apps, to monitor and manage them, customers want the ability to add custom tags to all resources that are part of a specific application. This is necessary when wanting to properly monitor and manage these resources. Not being able to do this makes it hard to work with monitoring software that depends on specific tags being applied to all resources in an AWS account.
This applies to resources that are defined by the user within the stack, as well as resources automatically generated by CDK EKS Blueprints to deploy the environment (ex: Lambda functions, Roles, ...).
Proposed Solution
The user gets to define their preferred tags on a per-resource level. This would basically mean that the "tags" argument is a valid argument that can be added to all applicable EKS Blueprints methods that generate resources. For example, if the user wants to add the LoadBalancer Controller AddOn, one would be able to define tags which are added to the related roles and other resources created by EKS Blueprints for this addon to function. Example:
new blueprints.addons.AwsLoadBalancerControllerAddOn( { "key1":"val1", ... } ),

In this example, to cover all resources that are implicitly created by EKS Blueprints (such as Lambda functions, SSM, etc... ), the user would also be able to define "blueprintGlobalTags" which add tags only to these implicit resources. Example
const stack = blueprints.EksBlueprint.builder()
    .blueprintGlobalTags( { "key2":"val2", "key3":"val3", ... } )
    .build(app, 'eks-blueprint');

Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request

CDK version used
2.133.0 (build dcc1e75)
EKS Blueprints Version
1.14.1
Node.js Version
v20.11.0
Environment details (OS name and version, etc.)
Win11Pro22H2

Especially need to add resource tags in LaunchTemplate(LT) that can tag ASG launched EC2 instances with desired tags. Currently, the LT has no tags under the Resource Tags tab.

This is very interesting and particularly useful in those cases where a cloud-migration-oriented discount agreement exists, where the customer needs to tag all resources with a specific tag value provided by AWS, in order to be eligible to the discount.
Thankfully CDK provides an easy way to tag all resources in a stack, via the Tags aspect. For example, to tagg ALL cloud resources of specific types:
import * as cdk from 'aws-cdk-lib';
const app = new cdk.App();
cdk.Tags.of(app).add(key, value, {
    includeResourceTypes: [
      'AWS::EC2::VPCEndpoint', 'AWS::EC2::NatGateway', 'AWS::EC2::InternetGateway',
      'AWS::KMS::Key',
      'AWS::Lambda::Function', 'AWS::StepFunctions::StateMachine',
      'AWS::SQS::Queue',
      'AWS::S3::Bucket',
      'AWS::SecretsManager::Secret', 'AWS::SSM::Parameter',
      'AWS::RDS::DBInstance'
    ]
  });

Furthermore, some cloud resources are indirectly created by Kubernetes controllers, instead of CDK/CloudFormation itself, and each controller/addon has a slightly different mechanism (when available) to specify tags for provisioned resources (e.g. EC2 instances, EBS volumes, and NLB load balancers).
For example, Karpenter allows to set tags on launched instances by setting the spec.tags field in the EC2NodeClass:
apiVersion: karpenter.k8s.aws/v1beta1
kind: EC2NodeClass
metadata:
  name: default
spec:
  amiFamily: AL2
  tags:
    key: value

EBS CSI drivers allows to propagate tags to provisioned EBS volumes, via parameters in the StorageClass:
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: ebs-sc
provisioner: ebs.csi.aws.com
parameters:
  tagSpecification_1: "key1=value1"
  tagSpecification_2: "key2=value2"

AWS Load Balancer Controller has a way to inject tags into ALB/ELB/NLB by setting defaultTags in its helm chart's Values:
# values.yaml
defaultTags:
  key: value


Trying to run har-to-openapi on large .har file returns nothing

Log
PS C:\Users\blusc\AppData\Local\Temp\psdetest> ls

    Directory: C:\Users\blusc\AppData\Local\Temp\psdetest

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----          03/09/2024    04:23                har2openapi
-a---          03/09/2024    04:31           2525 index.mjs
-a---          03/09/2024    04:06      706007128 input.har
-a---          03/09/2024    04:32            214 openapi.json
-a---          03/09/2024    04:32            113 openapi.yaml
PS C:\Users\blusc\AppData\Local\Temp\psdetest> node index.mjs
(node:25684) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
{
  spec: {
    openapi: '3.0.0',
    info: { title: 'HarToOpenApi - no valid specs found', version: '1.0.0' },
    paths: {},
    servers: [ [Object] ]
  },
  yamlSpec: 'openapi: 3.0.0\n' +
    'info:\n' +
    '  title: HarToOpenApi - no valid specs found\n' +
    '  version: 1.0.0\n' +
    'paths: {}\n' +
    'servers:\n' +
    '  - url: /\n',
  domain: undefined
}

Source
import { generateSpec } from "har-to-openapi";
import * as fs from 'fs/promises';

// read a har file from wherever you want - in this example its just a root json object
const har = await fs.readFile("input.har");

const openapi = await generateSpec(har, {
    // if true, we'll treat every url as having the same domain, regardless of what its actual domain is
    // the first domain we see is the domain we'll use
    // forceAllRequestsInSameSpec: false,
    // // if true, every path object will have its own servers entry, defining its base path. This is useful when
    // // forceAllRequestsInSameSpec is set
    // addServersToPaths: false,
    // // try and guess common auth headers
    // guessAuthenticationHeaders: true,
    // // if the response has this status code, ignore the body
    // // ignoreBodiesForStatusCodes: [],
    // // whether non standard methods should be allowed (like HTTP MY_CUSTOM_METHOD)
    // relaxedMethods: true,
    // // whether we should try and parse non application/json responses as json - defaults to true
    // relaxedContentTypeJsonParse: true,
    // // a list of tags that match passed on the path, either [match_and_tag] or [match, tag]
    // // tags?: ([string] | [string, string] | string)[] | ((url: string) => string | string[] | void);
    // // response mime types to filter for
    // mimeTypes: ['text/json','application/json'],
    // // known security headers for this har, to add to security field in openapi (e.g. "X-Auth-Token")
    securityHeaders: ['X-Origin-Integrity'],
    // // Whether to filter out all standard headers from the parameter list in openapi
    // filterStandardHeaders: true,
    // // Whether to log errors to console
    // logErrors: true,
    // // a string, regex, or callback to filter urls for inclusion
    // // urlFilter: /https:\/\/www\.pietsmiet\.de\/api\/.*/,
    // // when we encounter a URL, try and parameterize it, such that something like
    // // GET /uuids/123e4567-e89b-12d3-a456-426655440000 becomes GET /uuids/{uuid}
    // attemptToParameterizeUrl: true,
    // // when we encounter a path without a response or with a response that does not have 2xx, dont include it
    // dropPathsWithoutSuccessfulResponse: false,
  });
const { spec, yamlSpec } = openapi;
console.log(openapi);
fs.writeFile("openapi.json", JSON.stringify(spec, null, 4));
fs.writeFile("openapi.yaml", yamlSpec)
// spec = { ... } openapi spec schema document
// yamlSpec = string, "info: ..."

I ran a python script to count the number of print("entries:",len(content["log"]["entries"])):
PS C:\Users\blusc\AppData\Local\Temp\psdetest> python .\test.py
entries: 3417

Can you share the har? Are you passing in the right path to the har? No valid specs found means you probably aren't passing in the right path to the har file.

Can you share the har? Are you passing in the right path to the har? No valid specs found means you probably aren't passing in the right path to the har file.

Its very big so i have no idea where i should share it?
Also yeah the path is correct as seen in the logs + code

Here's a version pre-filtered to the base api url through a python script: https://workupload.com/file/THAyctYneks (pw: g)

Got it working! I was forgetting to JSON.parse inbetween reading the file and trying to generate the spec. I just yoinked that from your tests and now it works:
import { generateSpec } from "har-to-openapi";
import { fileURLToPath } from "url";
import path, { dirname } from "path";
import * as fs from 'fs/promises';

const __filename = fileURLToPath(import.meta.url);
const __dirname = dirname(__filename);

const dataDir = path.join(__dirname, "data");

const readHar = async (name) =>
  JSON.parse(await fs.readFile(name), { encoding: "utf8" });

// read a har file from wherever you want - in this example its just a root json object
const har = await readHar("filtered.har");

const openapi = await generateSpec(har, {
    // if true, we'll treat every url as having the same domain, regardless of what its actual domain is
    // the first domain we see is the domain we'll use
    forceAllRequestsInSameSpec: true,
    // // if true, every path object will have its own servers entry, defining its base path. This is useful when
    // // forceAllRequestsInSameSpec is set
    addServersToPaths: true,
    // // try and guess common auth headers
    guessAuthenticationHeaders: true,
    // // if the response has this status code, ignore the body
    // // ignoreBodiesForStatusCodes: [],
    // // whether non standard methods should be allowed (like HTTP MY_CUSTOM_METHOD)
    relaxedMethods: true,
    // // whether we should try and parse non application/json responses as json - defaults to true
    relaxedContentTypeJsonParse: true,
    // // a list of tags that match passed on the path, either [match_and_tag] or [match, tag]
    // // tags?: ([string] | [string, string] | string)[] | ((url: string) => string | string[] | void);
    // // response mime types to filter for
    // mimeTypes: ['text/json','application/json'],
    // // known security headers for this har, to add to security field in openapi (e.g. "X-Auth-Token")
    securityHeaders: ['X-Origin-Integrity','psde_auth3'],
    // // Whether to filter out all standard headers from the parameter list in openapi
    filterStandardHeaders: false,
    // // Whether to log errors to console
    logErrors: true,
    // // a string, regex, or callback to filter urls for inclusion
    urlFilter: /https:\/\/www\.pietsmiet\.de\/api\/.*/,
    // // when we encounter a URL, try and parameterize it, such that something like
    // // GET /uuids/123e4567-e89b-12d3-a456-426655440000 becomes GET /uuids/{uuid}
    attemptToParameterizeUrl: true,
    // // when we encounter a path without a response or with a response that does not have 2xx, dont include it
    dropPathsWithoutSuccessfulResponse: false,
  });
const { spec, yamlSpec } = openapi;
console.log(openapi);
fs.writeFile("openapi.json", JSON.stringify(spec, null, 4));
fs.writeFile("openapi.yaml", yamlSpec)
// spec = { ... } openapi spec schema document
// yamlSpec = string, "info: ..."

Kinda sad that it was able to "parameterize" the /articles/{id} but not the categories/{id}

Great! I'll add a test but yes I just got it working as well.
I just added a new option minLengthForNumericPath that should let you parameterize that path too https://github.com/jonluca/har-to-openapi/commit/73b8f6ff460a2bac09eac7850ee863c1ee93c821


Icons are broken in firefox

Materialize icons are not rendered properly

The issue seems to be repro on in some old Firefox browsers that are no more supported. Issue will be closed.


Add Company model

Closes #43

Codecov Report

Merging #44 (45bc7bf) into main (f9323b3) will increase coverage by 3.70%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   18.33%   22.04%   +3.70%
==========================================
  Files           5        6       +1
  Lines         180      186       +6
==========================================
+ Hits           33       41       +8
+ Misses        138      135       -3
- Partials        9       10       +1

Impacted Files
Coverage Δ

pkg/scraper/bruneida_scraper.go
6.55% <0.00%> (+0.20%)
:arrow_up:

pkg/scraper/jobcenter_scraper.go
8.33% <0.00%> (+0.33%)
:arrow_up:

pkg/model/model.go
80.00% <80.00%> (ø)

Continue to review full report at Codecov.

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


Docs Need Update

Documentation Changes Needed:

Move cell information to a more visible location (and maybe multiple locations or redirect with header link)

Seems like their is a lot of information regarding cells and how to make them in docs/Usage/NotebookFiles.md, this should be moved somewhere else. Not everyone may be importing and exporting. A good place could be in the "Gettting Started" page.

Update the Troubleshooting Doc with the issues linked and all the different solution types.

Issues: #1684, #1665, #1550, #1517, #1500, #1405. The solutions may need to be sorted and then throwaway if they are too complex or have no benefit over another.

Add a Doc for Hydrogen Settings and what they do.

While we do provide a description, sometimes that is not enough and can lead to a lot of confusion. It may be helpful to provide examples of our settings.
This stems from the languageMappings setting, which seems to confuse people regularly even our contributors sometimes.

If this Issue still needs attention, I'd like to contribute

@Purukitto
Thanks for the message !
Yes we DO need the help.
In my opinion, we especially need an update for

Move cell information to a more visible location (and maybe multiple locations or redirect with header link)

Seems like their is a lot of information regarding cells and how to make them in docs/Usage/NotebookFiles.md, this should be moved somewhere else. Not everyone may be importing and exporting. A good place could be in the "Gettting Started" page.

In addition to the examples @wadethestealth mentioned above, the new documentation for upcoming features like:

completion descriptions: https://github.com/nteract/hydrogen/pull/1727
support for vega-3: https://github.com/nteract/hydrogen/pull/1724
inspection datatip: https://github.com/nteract/hydrogen/pull/1710
import results: https://github.com/nteract/hydrogen/pull/1715

are really welcome.

In my opinion, we especially need an update for

Move cell information to a more visible location (and maybe multiple locations or redirect with header link)

Seems like their is a lot of information regarding cells and how to make them in docs/Usage/NotebookFiles.md, this should be moved somewhere else. Not everyone may be importing and exporting. A good place could be in the "Gettting Started" page.

I'll start with this then 👍

Cool, looking forward to your PR !
xref:

https://github.com/nteract/hydrogen/pull/1628
https://nteract.gitbooks.io/hydrogen/docs/Usage/NotebookFiles.html

Cool, looking forward to your PR !

Only pushing update in the docs/Usage/NotebookFiles.md file would be enough or parallel changes are needed on some other files?

Also I feel that adding a separate page under usage for Cells should be a good idea

This sounds better. Maybe adding something like docs/Usage/Cells.md.
Additionally mentioning that in docs/Usage/GettingStarted.md also could be helpful.

@aviatesk
I thought of mentioning it in Getting started but later thought better off it. Knowledge about 'Cell Markers' is necessary as such to get started so I didn't find it fitting.
Also the Getting Started page already references "Export" which in turn references "Cells" so it should be fine


Change HONOR links to ids on TOS page

@stvstnfrd, settings don't seem to do the right thing, so here's a commit to do it in theme.

@caesar2164 What settings don't do the right thing?
We've been burnt by hardcoding (even parts of) marketing links, so I'm fairly opposed to reverting to that approach..

So, I can't seem to be able to define HONOR key as tos#honor in MKTG_URL_LINK_MAP.

Also, I'm just hardcoding the thing we're hardcoding already in tos.html, @stvstnfrd

Ok @stvstnfrd, now there is no hardcoding at all. The same variable that's used to define the id in the tos.html template is also used to define the redirect.
This also doesn't affect platform or tests at all because the honor template exists like normal.

New plan: redirect both tos and honor to tos_and_honor.


(Nodejs) REST service to fetch the full content of a test result file of a specified trained model

AS AN Architect
I WANT TO have a REST service to fetch the full content of a test result file of a specified trained model
SO THAT there wouldn't be duplicate services for it
Acceptance Criteria

[ ] Use POST endpoint /rasa/training/results/files
[ ] Output the content of /results/<test name>/<file> in the format of base64
[ ] Output must contain name of the file
[ ] Output must contain mime-type of the file

Sample input for POST /rasa/training/results/files
{
  "trained_model": "First_Bot",
  "file_name": "RegexEntityExtractor_errors.json"
}

Pull request https://github.com/buerokratt/Training-Module/pull/189


chore(master): release 4.2.8

:robot: I have created a release beep boop
4.2.8 (2024-08-30)
🐞 Bug Fixes

fix dynamic thread pool full event emission rate (52021de)

✨ Polish

move dynamic pool only getters to its own class (21e6b0e)

🧪 Tests

refine pool event tests expectation (6a30766)

📚 Documentation

publish documentation (bc407dc)
README.md: fix worker example export (a5844a0)
README.md: use top level await syntax in example (3b4d090)
refine pool ready event emission condition (3abc7fe)

🤖 Automation

deps-dev: bump the regular group with 3 updates (d707c4d)
deps: bump the regular group across 11 directories with 2 updates (5c2e6e8)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v4.2.8
:sunflower:


Disable the Add More Bed Types option in Facility Page When All Bed Types are fulfilled

Is your feature request related to a problem? Please describe.
Right now, the Add More Bed Types button opens a modal to add more bed types even when all types are filled. It's better to disable it and show a tool tip saying "All types are already filled"

Describe the solution you'd like
Disable the button if all types are filled and show a tool tip on hover for user

@Jacobjeevan @rithviknishad I'd like to work on this and raise a quick PR

#8956 is reworking the bed and staff capacity logic to make use of Form components; as part of this, this is also being implemented 👍

Fixed in #8956


idl-daemon with config file fails

Getting the following error when running idl-daemon --config-file=config.json
(masked exact git repo with ...)
failed to process thrift file ~ { err2:
   { [Error: spawn /bin/sh ENOENT]
     code: 'ENOENT',
     errno: 'ENOENT',
     syscall: 'spawn /bin/sh',
     path: '/bin/sh',
     spawnargs: [ '-c', 'git remote --verbose' ],
     cmd: 'git remote --verbose' },
  remote:
   Remote {
     repository<EMAIL_ADDRESS>     branch: 'master',
     directoryName: 'test-idl',
     fileName: 'test-idl.thrift' } }
/usr/local/lib/node_modules/idl/node_modules/debug-logtron/backends/debug-log-backend.js:118
        throw new Error(logRecord.fields.msg);
        ^

Error: failed to process thrift file
    at DebugLogStream.write (/usr/local/lib/node_modules/idl/node_modules/debug-logtron/backends/debug-log-backend.js:118:15)
    at DebugLogtron._log (/usr/local/lib/node_modules/idl/node_modules/debug-logtron/index.js:57:18)
    at DebugLogtron.error (/usr/local/lib/node_modules/idl/node_modules/debug-logtron/index.js:89:10)
    at onProcessed (/usr/local/lib/node_modules/idl/repository.js:189:29)
    at onServiceName (/usr/local/lib/node_modules/idl/repository.js:217:20)
    at onVerboseRemote (/usr/local/lib/node_modules/idl/service-name.js:40:24)
    at exithandler (child_process.js:209:5)
    at ChildProcess.errorhandler (child_process.js:221:5)
    at emitOne (events.js:90:13)
    at ChildProcess.emit (events.js:182:7)

Here is the config.json (masked exact git repo with ...)
Your documentation is not update to date, some of the fields in config were required by code, but was not updated in the documentation.
{
    "upstream"<EMAIL_ADDRESS>    "repositoryDirectory": ".../idl-registry/repo",
    "cacheLocation": ".../idl-registry/cache",
    "fileNameStrategy": "lastSegment",
    "fetchInterval": 10, //not sure what this is
    "remotes": [
        {
            "repository"<EMAIL_ADDRESS>            "branch": "master"
        }
    ]
}

After a bit of going through your code, the error seems to be occurring at line 30 exec in service-name.js. Please help.

@malandrew any ideas?

None off the top of my head. I'll add this to my backlog, but in the meantime pull requests are welcome.


prisma format ends the file with a single CRLF on windows

Bug description
On windows, I run prisma format and note the unusual file ending. The lines are all LF, but the very last line is CRLF.
This causes issue on my Linux CI where it formats it ending in LF's only, causing a diff to occur and the build to fail.
How to reproduce

On windows do prisma format
Open in HxD or similar
See attached:

Expected behavior
LF's only - no CR's to exist at all
Prisma information

Environment & setup
Node 16
Windows/Linux
Prisma Version
$ C:\p\pab\monorepo\node_modules\.bin\prisma -v
prisma               : 2.26.0
@prisma/client       : 2.28.0
Current platform     : windows
Query Engine         : query-engine 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at<EMAIL_ADDRESS>Introspection Engine : introspection-core 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at<EMAIL_ADDRESS>Default Engines Hash : 9b816b3aa13cc270074f172f30d6eda8a8ce867d
Studio               : 0.408.0

I just ran into the same problem. I check out files as-is, so without git's auto-clrf setting, even on windows.

Hey @janpio, Guillermo (@GmoZen) and I are working on a fix for this bug as part of our CodeDay project (https://labs.codeday.org/).
I'm wondering if we can fix this by just changing the os.EOL line in this file to \n,  that way the file would consistently have the same line endings on every line, as opposed to having the platform-specific line ending on the last line?
https://github.com/prisma/prisma/blob/96dc3444b1439e3ab166a69c8290aec06c61bfba/packages/cli/src/Format.ts#L65
Or is the goal here to have every line have the correct platform-specific EOL? If so, would this require a change in the prisma engines Rust code, somewhere here?
https://github.com/prisma/prisma-engines/blob/main/prisma-fmt/src/format.rs
Thank you!

Would it be enough to get one sample of a line ending from the file and then using that instead of os.EOL?

@DanielZavala11 Figuring that out is part of the work here - I do not know what the real expected and correct behavior would be. That depends very much on what problems the current behaviour is creating, and if the new would avoid these. So right now we would need to understand further (optimally with a reproduction(before we can fix.

Thanks for the advice Jan!
@Miladiir, can you describe a bit more the problem this causes for you? Is the issue that you (1) are editing a file on one OS (let's say Windows), and you push that, (2) another project member edits the same file on another OS (let's say Mac) and edits it (changing one line) and runs prisma format, (3) git then shows that ALL lines have changed, because the format command changed all the line endings?
If so, we're thinking that maybe your suggestion of sampling a line ending from the file is a good solution. Or alternatively always just sticking to one type of line ending (let's say just LF) but I'm not sure if that has other consequences. We'll try to research that a bit more.

This all get more complicate when a team setting and Git come into play:

You might have two developers working on Linux and Windows - one creates the file initially, the other later formats it. Which line ending to use?
Git has configurable behavior on Windows to locally change the line endings to the Windows default, but read and write the Linux default instead to avoid exactly situations like listed in 1.
If an existing file already has different line endings when you start, things get even messier.

@janpio  Is this issue available to work on?

Generally yes, but we have a previous attempt here: https://github.com/prisma/prisma/pull/12401 I am not sure why that did not progress further.
If you want to take another shot, a first step should to be to confirm and understand the problem and offer an overview of the problem and a potential solution here in the issue.

With @tomhoule we think this could be already fixed, so @Nike682631, check if you can reproduce with the latest Prisma version or not first, if it's fixed we can close this 😄

I stumbled on where this is happening by chance: https://github.com/prisma/prisma/blob/a1b1f3cb3780c4214f125911791225a8f7d48685/packages/cli/src/Format.ts#L82 — it looks like the CLI is adding this.

This issue is already solved

Thanks for looking into this @fvckDesa! 🙌
We can extend the scope of this issue to allow you to work on the bounty and create a meaningful contribution, also considering the work you've already put into it. We suggest the following:

confirming via our tests that all operating systems indeed use the same line ending consistently
cover other commands besides prisma format, e.g prisma init or prisma migrate diff, which also create schema files

@jharrell can help you if you have any questions about the details.

@petradonka It's fine for me, I start working on it as soon as I finish with another commitment.
/attempt https://github.com/prisma/prisma/issues/8548

I can't reattempt the bounty, in the case I can't work on it I'll leave a comment

@nabby27 created a $50.00 reward using Opire
How to earn this reward?
Since this project does not have Opire bot installed yet 😞 you need to go to Opire and claim the rewards through your programmer's dashboard once you have your PR ready 💪

Hi there @nabby27,
we are not accepting bounties at this time. Thank you for your interest, however!

Hi @jharrell, do you not accept rewards for this issue or for any issue? Just to know if I could contribute financially to any issue (even if it is a small one).

@nabby27 👋
TL;DR - we don't accept rewards for any issues.
We trialed a few different bounty systems with a small set of issues, you happened to find one of them 😁 We're still thinking about this though, just want to find the best way to approach it in order to do right by our community.


avoid computing 2_u32.pow(OV::BITS) in a loop

compute the modulus once (and not per sample), and add more tests for all BA types.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


OpenBLAS should be resilient to reported zero-size L2 cache lines

On current versions of the "crostini" virtual machine on Google's Pixelbooks, the hypervisor erroneously reports all 0s for CPUID 0x80000006, including a zero-size L2 cache line. This causes openblas to spin forever on most operations:
https://bugs.chromium.org/p/chromium/issues/detail?id=848488
The bug here is clearly in the hypervisor, but I'd like to request that OpenBLAS could still be better-behaved here, and either die with an error or assume a default cacheline size.

Not sure what a sensible default would be, perhaps 256k ?. Building with an appropriate TARGET option such as TARGET=HASWELL should work around this. (Misconfigured hypervisors are known to cause problems, e.g. qemu not reporting a feature set corresponding to any real cpu model)


Returns 0 instead of throwing when EOF is encountered by channel

Return 0 when EOF.
Return 0 when ECANCEL
Returm 0 when EABORT

#57


Update to build docs locally

In response to #278

thanks for trying @znicholls, but this doesn't work as expected on readthedocs, see https://pyam-iamc.readthedocs.io/en/docs-requirements-makefile/install.html
(I pushed your branch to https://github.com/IAMconsortium/pyam to have it built on RTD).

I pushed your branch to https://github.com/IAMconsortium/pyam to have it built on RTD

Beauty thanks will work on that then!

Coverage decreased (-5.9%) to 85.067% when pulling e7fe5217db0c8a9019e2c4025a85cec9d3c7b9a4 on znicholls:docs-requirements-makefile into ac7caa01f2c51e51cdb9f1d457de6854beb6be46 on IAMconsortium:master.


Notebooks: Find widget is broken

Verifying https://github.com/microsoft/vscode/issues/99371

Open our verification.github-issues.
Ctrl F, type config, press Enter
Ctrl Backspace to delete, type `label, press Enter

🐛 Only one label is found
🐛 config is still highlighted as a match

I was also able to get into a terrible state in which NPEs appeared in dev tools and I was even able to surface one of the text editor's find widget, by focusing it and pressing Ctrl F. I can't repro this so I'll have to keep an eye on it.

Already fixed in https://github.com/microsoft/vscode/commit/dab68ed88523db7e7161f69da791b8c7e87c99c8.


AEM-17 is claiming replace method is not mutable?

Hi
I'm trying to get rid of this code smell but it appears this sonar rule believes the replace method is not mutable

Hi @holdsworthr, thanks for reporting this. Looking https://github.com/wttech/AEM-Rules-for-SonarQube/blob/master/src/main/java/com/cognifide/aemrules/java/checks/ModifiableValueMapUsageCheck.java#L63, the rule only recognises put, putAll and remove as mutable methods.
I'm guessing that the implementation follows the ModifiableValueMap Javadoc, which says:

The modifiable value map is only changeable through one of these methods

#put(String, Object)
Map.putAll(java.util.Map)
Map.remove(Object)

This is odd, because other methods inherited from java.util.Map have the effect of changing the values of the properties of the ValueMap's underlying resource.
Unless there's something I'm missing, both the AEM-17 Rule, and the Sling Javadoc could need a change. There may be a reason the Javadoc does not recommend using replace or replaceAll. If there's a good reason not to use those, we may be looking at a new rule. Let me search the Apache Sling Jira to find out if this has been discussed.

https://issues.apache.org/jira/browse/SLING-11878 has been raised

@holdsworthr it looks like these methods are valid. I think we'll improve AEM-17 to no longer report them. In the meantime, I would suggest resolving these issues as a won't fix  or false positive  on your Sonar.

https://github.com/apache/sling-org-apache-sling-api/pull/48 has been merged. This means we should update the rule to allow the following methods:

#replace(Object, Object)
#replace(Object, Object, Object)
#replaceAll(java.util.function.BiFunction)
#compute(Object, java.util.function.BiFunction)
#computeIfAbsent(Object, java.util.function.Function)
#computeIfPresent(Object, java.util.function.BiFunction)
#merge(Object, Object, java.util.function.BiFunction)


code coverage badge not in line with coverage reports in PRs

The badge in the README sometimes shows a wrong number (currently 80% instead of 88%)
https://github.com/devitocodes/devito
what is going on ?

Not sure, all PR show it's putting it back to 88 no idea why ref is now at 80, looking into it

Fixed .


[core] Add a workflow listener extension

This extension should react to workflow events (changelog, checksum, etc) whenever they are started/finished/failed.
Also support fine grained events for:

announce
assemble
deploy
download
upload
release
distributions
packagers

Required by #967 and #646

Released in v1.3.0 -> https://github.com/jreleaser/jreleaser/releases/tag/v1.3.0


mate.caja: 1.21.1 -> 1.21.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/caja/versions.
These checks were done:

built on NixOS
/nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/caja passed the binary check.
Warning: no invocation of /nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/caja-file-management-properties had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/caja-autorun-software had a zero exit code or showed the expected version
/nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/caja-connect-server passed the binary check.
/nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/.caja-wrapped passed the binary check.
Warning: no invocation of /nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/.caja-file-management-properties-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/.caja-autorun-software-wrapped had a zero exit code or showed the expected version
/nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2/bin/.caja-connect-server-wrapped passed the binary check.
4 of 8 passed binary check by having a zero exit code.
0 of 8 passed binary check by having the new version present in output.
found 1.21.2 with grep in /nix/store/29qp0b60aa3w4v4wvwdxy81qk64krqc0-caja-1.21.2
directory tree listing: https://gist.github.com/2709b44f62bdb6a5d626484e7b4f24ae
du listing: https://gist.github.com/a862e36a066610421b23ee38f984d40b

meta.description for mate.caja is: '"File manager for the MATE desktop"'.
cc @romildo for testing.
This build is cached with Cachix. To use the cache follow these instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout afdbeb5b1a06a4a38f96025d2410c4fbb25ebea5 && nix-shell --pure -I nixpkgs=. -p mate.caja

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mate.caja
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mate.caja
Partial log (click to expand)
shrinking /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/bin/.caja-wrapped
shrinking /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/bin/.caja-file-management-properties-wrapped
shrinking /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/bin/.caja-connect-server-wrapped
shrinking /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/bin/.caja-autorun-software-wrapped
gzipping man pages under /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/lib  /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2/bin
patching script interpreter paths in /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2
checking for references to /build in /nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2...
/nix/store/x1h509xf8mfkdgnnah8dwyrky5bjhxh7-caja-1.21.2

Success on aarch64-linux (full log)
Attempted: mate.caja
Partial log (click to expand)
shrinking /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/bin/.caja-file-management-properties-wrapped
shrinking /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/bin/.caja-autorun-software-wrapped
shrinking /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/bin/.caja-connect-server-wrapped
shrinking /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/lib/libcaja-extension.so.1.4.0
gzipping man pages under /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/lib  /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2/bin
patching script interpreter paths in /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2
checking for references to /build in /nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2...
/nix/store/4z5jcxjif86ln7lmqvijlm0a8xqbk6v6-caja-1.21.2


Change Password/Update Profile Page Revamp

I'm down to work on this one.

I'm down to work on this one.

Assigned!


[#169034053] Type-check IPA public administration info before upserting

This PR aims to fix an internal server error occurring when performing an administration search.
Some email strings from IPA info contain whitespaces at their endings, so their format is invalid for the decoder. In this PR all the strings from IPA are trimmed and a type check on the whole administration info is performed before upserting, in order to ensure that all the data saved in the database have a valid format.

Affected stories

🐞 #169034053: I dati da iPA devono essere salvati in un formato valido

  Generated by :no_entry_sign: dangerJS against 9e429d6bd7e3085e8282244ef7971acb54b909fd

iiuc  you are using PUT for updating a resource with a partial representation.
If that's what we're doing  it's a bad practice: see https://twitter.com/fielding/status/275471320685367296?s=19
PUT is intended to replace the whole representation with another one.
We can fix this in a different PR though
Sorry for not noting that before.


Add support for building on Mac

After Linux support, we need to add support for building on Mac (under mono).

TravisCI has a waiting list for mac build runners; I'd go ahead and contact them to get on the list. Took us about 45 days to get our repo(s) activated for mac tests.

@nathanaeljones the existing Jenkins CI already has mac and linux workers.

Nice! Does it build pull requests automatically?

Yes: http://dotnet-ci.cloudapp.net/job/microsoft_msbuild_prtest/ (it's just not hooked up to the mac and linux workers yet)

Is there a guide for people who want to run CI against their own fork?

@nathanaeljones I don't understand?

Of course, but in practice it can be difficult to get a CI server set up to mirror the official version. The beauty of supporting Travis and AppVeyor is that the user only needs to flip a switch on their accounts, and their fork is automatically building. While my company could use a dedicated CI server, our users are better served by us maintaining .travis.yml and appveyor.yml files, as they can fork and experiment without losing the crucial benefit of CI. Of course, all of our projects are public OSS, so there aren't many downsides.

Hmm, yeah that's a valid point, but in practice I think it's not that difficult to set up your own CI as this repo has very few dependencies (i.e. cloning the repo and running build.cmd should be enough).

MSBuild team triage: Closing because Mac support is now in. Eventually we'd like to get Mac CI support, but Linux will come first.


Add link to map under travel directions heading

I seemed to recall we intentionally didn't want a reference to directions on the about page, in order to fail something? (I had intentionally removed the address from About when I finished off the Map PR.) That being said, I don't think I ever added anything regarding this to the list of failures, either...
@rachaelbradley, thoughts?

The new sentence fails Link Purpose (AAA); this should be noted in the meta page.


onCentered event

Is it posible to add onCentered event? Would it be the same to use onPlaceChanged?

yes, or at least I think so (without checking the code) you can add hooks anywhere in the code literary :)
I don't have time right now to do an update, but if you check the code, you can mod it yourself and add the hooks you want


Override show_in_app?

I've read over this: https://github.com/sferik/rails_admin/wiki/Actions
And it doesn't have any example of overriding the show_in_app action.
My route has get :blah, as: something and rails_admin falls down when the show_in_app button is clicked.
I'd like to change the action to be custom. Is this possible?
Thanks!

@davidwparker Did you ever find a solution to this? I'm facing a similar issue now where show_in_app isn't able to find an object looked up by its slug.

@ianbayne - I switched to ActiveAdmin, which is infinitely more powerful.

Overriding controller option of show_in_app is the answer.
  config.actions do
    ...
    show_in_app do
      controller do
        proc do
          redirect_to main_app.url_for(@object)
        end
      end
    end
  end


the UIPageControl color can not change

how can i change the Indicator color

The delegate method does not work, but try this:
[self.buttonBarView.selectedBar setBackgroundColor:[UIColor whiteColor]];

@darkbreed :+1:

@darkbreed 's line should definitely be added to the main Readme.


Add all Providers to NotificationSources which class parent is EmailSource

Proposed Functionality
by default, all Providers will be added as related Providers for NotificationSources which parent class is EmailSource, as it's the usual use-case, where all the notifications are received in one email box.
Obviously, if the Provider don't have an email configured, it will be skipped from the loop.
Use Case
This simplifies the usability of the plugin with this assumption.

Implemented in #63


remove jisc mailer sync workers

e.g. remove these jisc mailers / convert them to sync with mailman

I don't think Mailman has an email command analogous to Listserv's quiet add command, and I don't think it can do authenticated requests via email. The jisc mailers should just be removed completely in favour of #1889.

Sounds like a plan @astopy thanks for the feedback 👍


Update 0000-01-02-kzwkt.md

created a branch, created a file and made a commit, and opened a pull request


Fix Travis builds for Mistral

Synchronize Mistral Installation script for Travis with recent st2_deploy changes.
PR for st2 master branch now should use  MISTRAL_STABLE_BRANCH="master"
Otherwise use MISTRAL_STABLE_BRANCH="st2-0.13.0"

Fixes #1925

Tests pass so I'm going to merge this for now.
Eventually, we should come up with a better and more maintainable plan.

Yeah, I'm for having at least some information/feedback from such "open for everyone" CI tests, instead of having nothing.
Especially useful for OSS when someone creates PR.

Merged, thanks.


Error Message at "Generate init images"

Hello,
I'm trying to use this, but unfortunately I'm getting stuck at Generate Init images. The following error message appears:
NameError                                 Traceback (most recent call last)
 in
39         prompt_starts = OmegaConf.to_container(prompt_starts)
40     # update prompt_starts if any changes were made above
---> 41     if not np.all(df_pre.values == df.values):
42         df_pre = copy.deepcopy(df)
43         for i, rec in enumerate(prompt_starts):
NameError: name 'df' is not defined
Can you help?

i think this is fixed, lemme know if it's still a problem

if it's still an issue, let's move discussion here: #117


Misconfigured LOG var results in AttributeError: 'function' object has no attribute 'error'

@jlongstaf
What issues does this address?
Fixes #685
What's this change do?
This is a port of @busterswt change in PR #686 to the liberty branch. Also, a test was added to check the functionality. I also grepped the agent codebase to check for other instances of this problem and did not find any.
Where should the reviewer start?
Any background context?

@busterswt: thanks for the assistance.

Sure thing! Thanks for the backport.

@jlongstaf: the build finished with success, but we're still getting double status messages reported. I'll look at this in test jenkins, but we're ready to merge when you've reviewed.


Double mirrored?

The double negatives are gone now but I know for sure the Y is inverted and I believe the X is as well.

I agree Y is mirrored though it looks like X is correct to me.

It looks like the canvas draws from top down in the y direction. I have a PR coming.


Add a way to exclude columns from samples

Is your feature request related to a problem? Please describe.
There does not seem to be a way to exclude columns from the samples section in the yaml. This could be useful for cases when someone wants sample data but cant send an email address to Soda Cloud due to PII reasons.
Describe the solution you'd like
If we could add an exclude_column to the samples section that would be great.
samples:
  table_limit: 10
  failed_limit: 10
  exclude_column: email

Duplicate: https://github.com/sodadata/soda-sql/issues/63


Bundles don't work anymore in Meteor 1.8.2

When using nathantreid:static-assets, Meteor behaves fine in the development mode using meteor run.
During rollout into production a bundle is generated using meteor build direct from the repository clone. Opening the new application in the browser will throw a JavaScript error Uncaught Error: Cannot find module '…'
When generating the bundle a second time without modification of any of the files, it works. No JavaScript errors are shown in the browser.
See https://github.com/qnipp/meteor-double-build-bug to reproduce the bug.
The bug was observed on Linux, using Meteor 1.8.1 and Node version 8.15.1 (which is the corresponding Node version for this Meteor version).
On 1.8.2 and 1.9-beta.3, even a second build does not help, the error remains.
The bug is reported to Meteor as well: https://github.com/meteor/meteor/issues/10780

Using static assets from NPM packages works well with symbolic links, as explained in https://guide.meteor.com/using-npm-packages.html#npm-assets
Therefore, I've changed my application to do it as explained in the guide and removed this package.


Why is there no body.fixed-top-nav-bar rule?

If I'm using a  navigation system, I have to go add break pointed padding to the body tag.
body {
  padding-top: 60px;   // For me this # is 56px
}
@media (max-width: 979px) {
  body {
    padding-top: 0px;
  }
}

It seems like this should be included in bootstrap as classes that could be applied to the body, and it seems so obvious needed that I'm wondering if it is impossible to do.  I'm not the only one that seems to think it should be built in based on this SO question and answer and comments
I'm thinking something similar to:
body.fixed-nav-bar {
  padding-top: 60px;  // Actual size computed based on variables from the navbar
}
@media (max-width: 979px) {
  body.fixed-nav-bar-expand-md {
    padding-top: 0px;
  }
}
...

Seems like the variables could be introduced so that as the navbar height is adjusted over time, they would track the padding at the top of the window.  I think we might need different -md, -xl, -sm variants for where the menubar hiding takes place.  Has this been tried and abandoned?  Not wanting to waste time implementing it if it has already been proven impossible to do.
One other show stopper here is that the navbar height does not appear to be specified at all, which of course complicates things....
The best alternative I've found is:

    Navbar

      Navbar

This causes a "shadow" navbar to be under the fixed-top navbar.
Where this fails is with screen readers which will now present 2 navbar-brand elements. This points at the need for a not-for-sr class to prevent that element from showing up for screen readers. However that class does not exist for screen readers
I've tried to compensate for the screen reader issue with aria-hidden="true" but this guide seems to indicate this will probably not work when the screen reader is in focus mode which is of course the only time you actually need it to work...

Hi @mark0978, I think you have a good point. To me, it's a bit more complicated.
The navbar doesn't have a height by design. If we set up a fixed height  size then it would break when text string on the navigation items are too long causing text wrapping. As a general rule, to build a flexible UI no component should have a height.
Since the Bootstrap doesn't know how high the navbar is, then I don't see how we could calculate the body class you are suggesting just with CSS.
In the docs there is a note about adding the padding to the body:

Fixed navbars use position: fixed, meaning they’re pulled from the normal flow of the DOM and may require custom CSS (e.g., padding-top on the ) to prevent overlap with other elements.

In our apps, we use .sticky-top because we don't want to deal with a header with a fixed height. However, it only works with browsers that support the sticky position, no stickiness for old browers like IE10 or IE11.

Closing as a won't fix for now.

thanks 😍😍😍😍😍😍😍😍


Fix "proofs.proto" path

Summary
The proofs.proto file is in the /third_party/proto root folder while it was usually in /third_party/proto/confio.
I modified the project proto files to be consistent but it would be more convenient to have common a standard within all cosmos projects.
The SDK uses the buf schema registry now:  https://github.com/cosmos/cosmos-sdk/pull/10669
I have opened https://github.com/confio/ics23/issues/67 to support this.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Same issue with v3

closed by https://github.com/cosmos/ibc-go/pull/2705
Updating ics23 to use BSR can be tracked separately


Added support for emcee in arviz

Wanted to let you know we have added support for emcee to the arviz library - see https://github.com/arviz-devs/arviz/pull/302 for a description of the change.
Roughly, the plan is for arviz to replace plotting and diagnostics for pymc3 (so we do not have to rewrite it for pymc4), by converting data to a common data format (netCDF/xarray datasets), and then implementing all the plots and diagnostics on that object. emcee is the third library to have support (pymc3 and pystan), after hearing so many astronomers saying nice things about it at a conference.
In addition to the plots you can see in the pull request above, you can also calculate things like the effective sample size and the gelman rubin statistic. The summary function below shows a number of these.
Some questions I had -

we store sample_stats from pystan and pymc3. Is there a way to access whether a sample was accepted at a per-draw basis in emcee?
Are there particular diagnostics or plots you would like to see?

import arviz as az
from arviz.tests import helpers

sampler = helpers.emcee_linear_model(None, 500, 10)  # this is the sampler from http://dfm.io/emcee/current/user/line/

# This converter turns the data into a netCDF object that can be serialized and shared
data = az.from_emcee(sampler, var_names=['ln(f)', 'b', 'm'])

# all arviz functions should work with this object
az.summary(data).to_dataframe()

ln(f)
b
m

metric

mean
-0.901523
4.186736
-0.493168

standard deviation
0.083640
0.387793
0.146651

mc error
0.002052
0.008238
0.003727

hpd 3.00%
-1.061138
3.448212
-0.759357

hpd 97.00%
-0.742526
4.923531
-0.201992

effective samples
1429.248983
1408.863122
1287.373414

gelman-rubin statistic
1.030000
1.030000
1.040000

Hey Colin, thanks for this!
We don't currently store per-draw stats for emcee, but I would like to include it at some point. It probably won't make it into v3.0, but hopefully soon! I'll keep you posted.
The plot that I always struggle to make with pymc3 in a consistent way is a nice corner plot (like your pair_plot but using corner.py). I haven't looked at arviz in detail, but an interface that could work easily with that would be awesome. (The problem occurs when some of the variables have non-scalar shapes - something that you must have an interface to deal with anyways!)
Finally - it's worth noting that the chains produced by emcee are not independent so some care must be taken when doing convergence analysis. I don't think that this is necessarily a big problem, but we might want to include some sort of disclaimer so that people don't take the G-R statistic/n_eff without the proper caution.

Quick question @dfm : I'm currently using PyMC3's effective_n function to estimate the effective sample size for samplers of both emcee and ptemcee. Is this not allowed? Or is it a reasonable estimate with some caveats?

Thank you Dan. If I understand what is stated in that post then I should simply estimate the effective sample size as N/tau, where tau must be obtained with emcee's (or ptemcee's) own autocorr functions. Is this correct?

Closing since this was just informative (though thanks for pointing out the effective_n calculation is wrong for this sampler!)

Hey @dfm, I have been doing some experiments (posted on my blog) comparing autocorrelation/ess values using the algorithm in ArviZ and the one in emcee, and it looks like they both converge to the same value (and that the condition on N/tau seems to hold for both).
Could you indicate which assumptions are not satisfied in the Gelman-Rubin algorithm? Maybe it affects only R_hat values? I tried to read the references, but I could not figure out the assumptions that must be satisfied for the MCMC samples for the algorithms to be applied, in fact both of them seem to incorporate empirical/practical cutoffs or approximations without any proof.
Any help or hints would be greatly appreciated. Thanks!

@OriolAbril: That post looks great! All I meant by that is that the walkers in the stretch move are not formally independent so you'll be overly confident about your convergence if you treat them as independent. I'm sure it can still be a useful diagnostic!

Yes! I understand this point, which is why I expected ArviZ version to underestimate the autocorrelation time. Thus, the effective sample size would be overestimated, and we would end up being overconfident with convergence. But what happens in all the examples I have looked into is the opposite of this. That is why i am having trouble trying to understand what exactly is going on.

This may also be relevant to my previous question: https://discourse.mc-stan.org/t/does-effective-sample-size-estimation-require-independent-chains/

Yes - that response there is exactly what I was saying! The chains do communicate at every step so they don't satisfy the assumptions in that work. It is interesting that the method works well, but it's not obvious that it will work well in less trivial experiments...

I know, what still confuses me is why do both methods converge to the same value and even worse, why does "Gelman" version overestimate the autocorrelation time instead of
underestimating it.
I will try to work out the maths in both approaches to see exactly where they differ and see if I can get why my intuition is not working.

Both algorithms are doing the same, with only two differences, one of them I think is not relevant. My notation is N indicates draws, M indicates chains/walkers and their lowercase version is their loop variable; c_f^m(tau) indicates the autocovariance of chain/walker m (I can write a proper blog post of latex document and send it to you if you prefer).
The differences are the following:

The normalization of the autocovariance/autocorrelation is different: in emcee, the normalization is done with c_f^m(tau) / c_f^m(0) and then average over each chain m (autocorrelations are used) whereas the other algorithm normalizes with 1 - (W - C_f(tau)) / var_plus (an overconservative version of autocorrelation is used).

W = N / (N-1) * 1/M * sum^M(c_f^m(0)) (an average of each chain variance)
C_f(tau) = 1/M * sum^M(c_f^m(tau)) the average of the autocovariances of each chain
var_plus = C_f(0) + 1/N B being B the between chain variance, thus, var_plus >= C_f(0)

As expected, both algorithms truncate the autocorrelation sum, but they choose this threshold to truncate differently. I think this difference only has numerical implications, as they try to estimate the same values with the minimum noise. If you think this is relevant I can comment this in more detail.

I think that the normalization difference has no implication on whether the chains used for the autocovariance/autocorrelation computation must or must not be independent, so I am more confused than ever. I mean, I have understood why "Gelman" algorithm gives autocorrelation times larger or equal than emcee's version, but how is the dependence taken into account (if taken into account)? Or what is the same, where is independence required (if required)?
Thanks for all your time! I hope I am not too annoying.

Hello,
I seriously need help with emcee. I am trying to fit model spectra to observed ones. After working hard on code, the results are very strange. The chain values of parameters are the same for each walker. I mean the initial values do not change over the chain. Do you have any clue? If needed, I can send the code.


Updated repo to resolve #117

Added following things

Data for work with
Updated config.yml
Added leaflet.html to see data
update dive doc to mention the config we need to add

Not sure where we use the airports.csv file?

I was planning to use it in "Indian-centric" data sets


vite.config.ts Print in process.env The. Env address is not found in the configuration file

Do NOT ignore this template or your issue will have a very high chance to be closed without comment.

Describe the bug
vite.config.ts Print in process.env The. Env address is not found in the configuration file
A clear and concise description of what the bug is.
Reproduction
nothing
Please provide a link to a repo that can reproduce the problem you ran into.
A reproduction is required unless you are absolutely sure that the the problem is obvious and the information you provided is enough for us to understand what the problem is. If a report has only vague description (e.g. just a generic error message) and has no reproduction, it will be closed immediately.
System Info

required vite version: 1.0.0-rc.9
required Operating System: windows
required Node version: 12.13.1
Optional:

npm/yarn version 6.12.1
Installed vue version (from yarn.lock or package-lock.json) 3.0.0
Installed @vue/compiler-sfc version 3.0.0

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

vite.config.ts Print in process.env The. Env address is not found in the configuration file and my .env  file is exist VITE_

I'm not sure get what your said. Can you give more clear descripition for this? (It's better if you give a reproduction).

my .env file
VITE_API_DOMAIN = '<IP_ADDRESS>:8080i'

my vite.config.ts file
console.log(process.env.VITE_API_DOMAIN)

I want to be in vite.config.ts The address in the. Env file is obtained from, but cannot be obtained

@underfin

Duplicate of #1096
https://github.com/vitejs/vite/issues/1096#issuecomment-729077386


map es5 support draft

Hi guys! This is a draft solution for es5 map support. What do you think about this approach?
It should works with babel preset, also it may works in IE11 ;)
Now map draft tests passes and I'm planning to add Set support, patches, etc

Hmm, CI failed, I guess according this https://github.com/immerjs/immer/commit/017de43395044faa703c22eef441aa3b717a42d7

Awesome! I'll hope to review in detail hopefully somewhere next week

Need to decide

Should it work in environment without Map and Set?
If yes, I need to add a checking
Should it work in IE11 without babel?
IE11 has partial support without iteration methods and difference between Map.set method and does not support new Map(iterable)

And I have doubts about a lot of copy-paste between implementations proxy and es5

No, as they can be polyfilled, which is the responsibility of the consumer of this package. A nice error message about the need of this would be great though.
I think it is fine sticking to the normal web api's, and if something doesn't work on any browser, it is up to the host to set up some polyfill. So I'd write the code as if it is targetting a modern browser.
I can imagine that having only an ES5 implementation might work, that is, a class based approach, and use that one in the Proxy case as well? I mean, the fact that Proxies are available, doesn't mean that we must use them :). I'd opt for code-base maintainability here.

No, as they can be polyfilled, which is the responsibility of the consumer of this package. A nice error message about the need of this would be great though.

We have some checking here
https://github.com/immerjs/immer/pull/437/files#diff-de99f837249e5adf62716789c7b9e80eR40
Imagine, some consumer no uses Map or Set if he updates Immer, it will fail in ES5 browsers without Map and Set

@mweststrate I think I've done. But I don't understand one thing. Should I update markChangesRecursively? It skips Map and Set, should it be an explicit?

Awesome! I don't know the answer to that question from the top of my head,
but if a test in which you modify an entry in a map, that is in a map, that
is in an object, and that works, it's correct, as the mechanism is used to
ensure that changes propagate through unmodified parents. I'm gonna try to
carve out some time tomorrow to review in detail!
On Mon, Oct 21, 2019 at 5:49 PM Andrey Boldyrev<EMAIL_ADDRESS>wrote:

@mweststrate https://github.com/mweststrate I think I've done. But I
don't understand one thing. Should I update markChangesRecursively? It
skips Map and Set, should it be an explicit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/immerjs/immer/pull/437?email_source=notifications&email_token=AAN4NBAEPDGWUTRUD5VZ67DQPXMSDA5CNFSM4I5VFPDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB3AHYI#issuecomment-544605153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN4NBHOQ2UIXRWYHWGUNSLQPXMSDANCNFSM4I5VFPDA
.

Superficially this looks great! I'll merge it into the v4 branch, and to further local investigation there. I'll also send a maintainer invite. That will allow you to directly commit to that branch, which avoids the whole song-and-dance of maintaining your own fork :)

@mweststrate Glad to hear that! Thanks a lot for invitation!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hyperparameters

Hi, I'm trying to train imagenet with EfficientnetB3. Would you please be kind enough to mention the correct combination of hyperparameters.

I have not been training this models, just convert weights, look at original paper.


fulltext search fuzziness Auto

Hi
When doing fulltext search with fuzziness, AUTO seems to not be recognized any longer. It works with a fixed number but not w Auto any longer.
Auto worked in ver 55.6
error in 1.02:
"message": "SQLActionException[UnhandledServerException: java.lang.NumberFormatException: For input string: "[41 55 54 4f]"]",
"code": 5000
41 55 54 4f  = AUTO
https://www.elastic.co/guide/en/elasticsearch/guide/current/fuzziness.html
Ex.
select xx from tableA where
match (xx_ft, 'yy') using most_fields with (prefix_length=2, fuzziness='AUTO', minimum_should_match='1<67% 5<3')
_score >= 0 order by _score desc
crateDB docker 1.0.2
JVM Host - openjdk 1.8.0_111
host ubuntu 1604

Thanks for reporting, we'll take a look at it.

This has been fixed (https://github.com/crate/crate/pull/4714). The fix will be in the next 1.0. hotfix release


[Feature Request]: Add per-worker labelled/histogram metrics to BigQuery IO for the Dataflow Legacy Harness.

What would you like to happen?
Get better observability on BigQuery IO Operations for user workers using the Java Legacy Harness.

Support a new metrics path from User Worker -> Windmill Worker by piggybacking metrics GetData heartbeats.

This path will be used to support per-user worker metrics in Dataflow

Instrument BigQueryIO's storage write API with metrics that track RPC response status, request latency, request size.

Latency/Request size metrics will be supported with histogram metrics.

Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[X] Component: Google Cloud Dataflow Runner

.take-issue


expose component status

Currently with dev-workspace I can create a workspace and I've access to:

devworkspace object that has spec.template with devfile content listing components/commands that I want
workspaceRoutings object saying which endpoint are exposed.

But where to grab component status ?
(with che server it was within workspace.runtime object)
It seems for now I've to look at pod object and compare some container name with component/name but it looks very fragile.

For example on pods we have
status.containerStatuses
so it would be a good fit to have status.componentStatuses

Closing assuming that https://github.com/devfile/devworkspace-operator/pull/318 provides endpoints URL and all commands, components as flattened devfile.
@benoitf please create another issue if it's not enough, or you have strong need to have that info under DevWorkspace CR status

well status should still expose stuff (not a separate or mounted file)

you have strong need to have that info under DevWorkspace CR status

Seems to me that having this info in the Status would allow other components, that do not run inside Workspace POD containers, to have access to the result of the flattening (== the complete and annotated structure of the resulting workspace). Just think of the Dashboard trying to graphically represent the various components of a Workspace.
Storing it in the status would also allow keeping it even when the workspace is stopped (the fact that the workspace is stopped doesn't really impacts the result of the flattening).
TBH it seems to me that not putting it in the status is just doing more work for a less useful result.
Are there real problems raised by adding the annotated flattened devworkspace in the status, apart from having a CR that is  longer ?
It is already the case because of the last-applied metadata added by kubectl for example.

I would say putting a flattened devworkspace in the status is a real problem. I already had to figure out a bash alias to strip out managedFields when I'm looking at resources.
The flattened workspace can (and will be if theia is involved) 3-4 times longer than the spec. That's too much info for a status IMO.

I believe we agreed we are not going to expand devworkspace status with much details. Instead we may add link to flattened devfile from devworkspace status. But we agreed we won't add it until we see the necessity.