Style/DoubleNegation: EnforcedStyle: disallow_in_conditionals

Is your feature request related to a problem? Please describe.
For similar reasons as allowed_in_returns, !!foo should be allowed in any circumstance where the distinction between false and nil (or true and anything else) might matter.  In rbs types, this is the difference between bool and bool? and boolish.
Describe the solution you'd like
Only disallow using !!val inside a conditional or a boolean expression, e.g. with if, elsif, unless, when, until, ?:).
EnforcedStyle: disallow_in_conditionals
# bad
foo if !!cond
foo if !!cond1 && cond2
!!cond and do_foo
val = (!!cond ? foo : bar)

if !!cond
  foo
end

while !!cond
  foo
end

# good
call(!!param)
val = !!(cond ? foo : bar)

Describe alternatives you've considered
Perhaps with another config attribute, it could also be disallowed for method parameters, if we know the parameter type is boolish.  Likewise, variables could be traced from assignment in case they are only used in a conditional context.
Or I can just add # rubocop:disallow Style/DoubleNegation.  Or I can replace !!cond with cond ? true : false, but that feels much worse than !!cond to me.

I don't really know what use this cop would be if it wasn't checking conditionals. Sounds like for your use-case it'd be better to just disable the cop altogether?

@dvandersluis I think the point @nevans makes is that Style/DoubleNegation is marked as SafeAutoCorrect: false but we could introduce an enforced style that only reports instances of !! where we can be sure that removing them makes no semantic difference. Thus effectively making autocorrect safe. I'm on board with that idea.

Yes, what @jonas054 said is exactly right.  However subtle or insignificant the distinction between bool and boolish might be, I want this cop to only make suggestions or corrections when the code's behavior is unchanged.  IMO, that should be the default.
I think maybe @dvandersluis read some of my examples to mean the opposite of what I intended?  Sorry if I was unclear.  It may have been unbalanced to give ten lines of disallowed cases (all of which are already disallowed by existing EnforcedStyles) vs only two lines of allowed (which exhibit the change I want).
I didn't mention before, but in addition to the standard conditional clauses (if, elsif, unless, case; when, until, ?:), I'd also disallow !!val in boolean expressions (expr and expr, expr or expr, arg && arg, arg || arg), but only when they are used purely for their short-circuiting properties.  I.e. when the boolish-vs-bool value doesn't "escape" the local context.
Maybe this is simpler to explain from a positive "only allow" perspective?  The current default style (allow_in_returns) is incomplete.  To keep the same semantics, we also need to allow it for method arguments and assignments.  Maybe name it allow_arguments_assignments_and_returns instead of disallow_in_conditionals? 🙂  I think these three cases cover every time the semantics would be changed; can you think of any others?  Maybe a better name would be something like EnforcedStyle: allow_semantically_significant?
For assignment to a local variable that is only ever used in conditionals, it might seem like there's no difference between boolish and bool.  But even in that case there are both communicative differences (to developers, and to LSP, typeprof, sorbet, etc) and behavioral differences: the value is accessible from binding and that prevents GC!
Just like the current behavior, I would still disallow the following.  They can all be safely auto-corrected.
foo if !!cond
!!cond ? foo : bar

def short_circuits_that_do_not_care_about_bool_vs_boolish
  !!cond1 and a
  !!cond2 &&  b
  !!cond3 or  c
  !!cond4 ||  d
  !!foo and
    !!bar and
    !!baz
  retval
end

stmt while !!cond

The following would all be allowed.  Changing them changes the semantics and is unsafe.
foo if bar = !!baz

foo(!!bar || baz)

while okay = !!next_values
  etc
  etc
end

def short_circuits_lvar
  val = !!query1 && !!query2
  other stuff that maybe uses var
end

def short_circuits_return_val
  !!foo and
    bar and
    qux
end

def short_circuit_a = !!cond && a
def short_circuit_b = !!cond || b
def short_circuit_c;  !!cond and c end
def short_circuit_d;  !!cond or  d end

foo(!!val)

The following might be considered an edge case.  I don't care if the first form is allowed or disallowed... there's a case to be made either way.  My preference is whichever is simpler code and executes faster. :)
# should these be simplified?
var1 = !!foo || !!bar || !!baz || !!quux
var2 = foo || !!bar || !!baz || quux

# semantically identical simplifications:
var1 = !!(foo || bar || baz || quux)
var2 = foo || !!(bar || baz) || quux


Credit card: Maestro

It would be nice to include in your regexp this maestro cards:
6759649826438453
6799990100000000019
thanx!

Maestro isn't a credit card, but a debet card.
I would prefer second method isDebetCard() and optionally a validator isPayCard() or somthing which would allow for both credit or debet cards in the validation.
Please do not include maestro in credit card by default though; that way the name of the method does not longer reflect the actual validation.  Thanks very much!

Technically you correct, but your cannot judge credit/debit only by card number. A lot of visa/master cards are debit and valid according to this validator.

Fair enough. I would prefer another validator name in that case though.

Do we use Luhn Algorithm to validate credit card numbers here within this library or something else?

@denjaland as far as I know only an issuing bank can tell you if a card is debit or credit, you cannot determine type just from the numbers; that being said, binlist lookup - identifies some card types for US and Canada;

@roydondsouza you are right, which is why I still would prefer a method "isPayCard" ;-)
a method "iscreditcard" does not do what it promises :-)

Anyone still needs this?

@profnandaa yes I would like it to be added

Some Brazilians credit card have the same issue.
Elo and some MasterCard like 5031 4332 1540 6351

+1 on Elo and some MasterCard. Same issue here.

I will take a look at this and probably raise a PR for the same.
Cc. @profnandaa.

@profnandaa Support for 19 digit card numbers was added in #1161. This was the main issue here. So it may be closed OR if we need to rename the isCreditCard method to isPayCard, I'd be happy to make a PR :)

Sure we can close this one, no need to rename. Thanks for spotting.


Feature/dashboard liam

This PR:

Creates a composable that queries diabetes data from Firestore "diabetesData.js"
added <Suspense> to <router-view> for Lazy-Loading (needed for data querying on load)
Gets diabetes data on dashboard load
made Computed property for integer of "Percentage of participants with diabetes" https://github.com/advweb-grp1/advanced-web-final-year-project/issues/29#issuecomment-1506735631
added Pie Chart for "Prevalence of diabetes" https://github.com/advweb-grp1/advanced-web-final-year-project/issues/29#issuecomment-1506735631

DON'T APPROVE! Last fix messed it up 😭

let { total, withDiabetes } = await getDiabetesData(); worked fine, just ESLint gave Parsing Error so build failed

Need to refactor so it makes use of pinia store

[ ] Remove Composable diabetesData.js


NGINX server for search data

We are using S3 bucket for Dug input datasets . And stars.renci.org for translator kgx files. Since we are looking into using other data repositories, I think we can look in that direction instead of setting up an nginx for that purpose.
@hhiles , I think we can close this please re-open if needed.


Compatibility upgrade to Python 3.11

I had to close and reopen to make it fetch the updated 3.11 CI workflow


How to install from private bitbucket ?

How to properly install a plugin from a private bitbucket repo ?

@albabosh I think you will have to basically create a new user in bitbucket and add their credentials to composer:
https://stackoverflow.com/questions/28625311/configuring-composer-json-with-private-bitbucket-mercurial-repository#37180271


Invalid data format: Invalid passphrase

[etc.txt](https://github.com/ETCDEVTeam/emerald-wallet/files/2810680/etc.txt)

JsonRpcError: Invalid data format: Invalid passphrase
    at JsonRpcError.ExtendableBuiltin (file:///C:/Users/Mitchy/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24080:30)
    at new JsonRpcError (file:///C:/Users/Mitchy/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24109:116)
    at file:///C:/Users/Mitchy/AppData/Local/Programs/EmeraldWallet/resources/app.asar/app/index.js:24146:19

I have this same exact issue! Always thought I had forgotten my password but sometimes I don't use new passwords for each app, service, website and had also written it down to no success. A fairly good amount of ETC too. If this issue plagued two users, then there are likely more.


Develop

//spacing: [] was not be passed through because isNaN would eval to true

I'll try again...


make Contact us section Functional: Interactive Phone Number and Email Integration

What happened?
Expected behavior
In the footer section of the website, the contact information is currently static and non-functional. To improve user experience and interaction, I propose the implementation of interactive functionality for the phone number and email. This feature will enhance usability by enabling users to easily interact with the contact details based on their device.
Phone Number Functionality:
On mobile devices, clicking the phone number will automatically open the dial pad with the number pre-populated for effortless calling.
On desktop devices, clicking the phone number will copy the number along with the contact name to the clipboard, facilitating easy access for users.
Email Integration:
Clicking the email address will automatically open a new email window in the default email client, with the sender's email address pre-filled for seamless communication.
Screenshots

How can we reproduce this bug?
To Reproduce
Steps to reproduce the behavior:
Go to 'footer'
Scroll down to '....'
Click on 'phone number or email'
something should occur like open dial pad or for email the Gmail window should open
Desktop Information (Optional)
No response
Urgency (Optional)
High priority

Will be removing that section, as phone number not needed now

ok


Remove version: Spacemesh.Spacemesh version 0.2.6

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response Error: The remote server returned an error: (403) Forbidden.
(Automated message - build 912)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 887.)

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 887)

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 887)

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 887)

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 887)

URL: https://storage.googleapis.com/smapp/v0.2.6/Spacemesh Setup 0.2.6.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 887)


Since the new update the Alex Launcher asks me to isntall every time my game starts?

Apologizes if this is just how the current version works or if this has been answered before.
So I reinstalled XIV Alex, put in the new op codes, and it seems to be working (I can triple weave barely on east coast).
However every time I launch the game (Dalamud launcher) a big pop up from XIVAlex shows up asking me how I would like to install or just run the game or uninstall etc, the screen that originally pops up.
I have to X out of it every single time I run the game, the previous update the Alex would just go straight to my minimize tray on my PC and I didnt have to close anything. Is this supposed to happen? Its not the end of the world just a mild annoyance. Thank you for your time.

Can you screencap FFXIVQuickLauncher's settings page on autostart stuff? -a load should still work.

I think you're talking about FFXIVQuickLauncher by "dalamud launcher".

Probably this:

If XIVAlexander is installed, then there is no need to have it launched via XIVLauncher. Otherwise you may get that window. (if I'm reading this correctly)

OOOOOO yes i have XIVloader checked on my Dalamud launcher, so I just need to uncheck that?

Try it and see if it works. Those are my settings and it works fine, without manually loading it via the launcher.

That fixed the issue, it opened straight into the tray of my PC without prompting me to install again. Thank you so much !! Really appreciate the fast response and your help.


[REQ] Ability to synchronise req-cnts between wiki and code

mantra must have a command to automatically update ref-cnts in the wiki.
Updating these cnts manually would not be feasible.
Preferably a GitHub action should be available that updates the wiki on every push to the branch the action is set to.
However, it should also be possible to synchronise ref-cnts for local projects and wikis.

Wiki pages are simple markdown files in GitHub, so there are no property fields counts can be attached to.
As a result, some syntax rules must be adhered to by users for mantra to correctly find and update ref-cnts in the wiki.
The best way I could come up with so far is a bullet list with **References:** above it, which is directly below the heading of the requirement.
Example:
# my_req: Some Title

**References:**

- in branch main: 3
- in branch stable: 1

The branches should also have links pointing to the branch in the repository.

Requirements will be traced using a requirements ID set in the wiki section.
To allow hierarchical requirement structures, the ID of sub-requirements must follow the form parent_reg_id.sub_req_id.
As consequence, a . in an ID is only allowed if requirement IDs exist for all parent ID-parts.

Syntax for references on the code side will be [req:your_req_id].
This should be distinctive enough from common expressions in most programming languages.

This issue was partially implemented in v0.1.0.
The GitHub action will take more time, and is moved to the separate issue #11


Cuda version

I am seeing the following error
RuntimeError: Expected object of scalar type Long but got scalar type Int for sequence element 1 in sequence argument at position #1 'tensors'
Can you please tell me which version you were using for
cudatoolkit and cudnn
Thanks
abhishek

Thanks for your interest in our work. The CUDA version is 11.3. The cuDNN version is 7.6.2.
Best,
Yu


Support non-public cloud environments in the Azure Storage Queue and Azure Storage Blob scalers

Related PR in the keda repo: https://github.com/kedacore/keda/pull/1863
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)
[x] A PR is opened to update the documentation on our docs repo

Signed-off-by: amirschw<EMAIL_ADDRESS>

Thank you! Can you fix DCO so we can merge please?

Of course! Fixed


Moving Startup class to a library project breaks my site.

From @sam-wheat on December 19, 2016 21:43
This question refers to beta I am using .net core 1.1:
http://stackoverflow.com/questions/33183357/move-startup-cs-to-class-library-package-project-asp-net-5
I moved my Startup class to a library project and now my site does not work.  Only error I get is 404.
I checked ContentRootPath and it appears to point to the correct directory.
I see no errors in the output window.
IIS Express says my site is running on the correct port specified in launchSettings.json.
Startup class is exactly the same in library project as site project.  Only difference is namespsce.
I don't believe my question is environment related:
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/startup?highlight=startup#the-startup-class
Program.cs:
public static void Main(string[] args)
    {
        var host = new WebHostBuilder()
            .UseKestrel()
            .UseContentRoot(Directory.GetCurrentDirectory())
            .UseIISIntegration()
            //.UseStartup<xStartup>() works
            .UseStartup<Application.AppComponents.Startup>() // broken
            .Build();

        host.Run();
    }

Startup.cs (exactly the same in library / application projects):
public class Startup
{
    public Startup(IHostingEnvironment env)
    {

        var builder = new ConfigurationBuilder()
            .SetBasePath(env.ContentRootPath)
            .AddJsonFile("appsettings.json", optional: true, reloadOnChange: true)
            .AddJsonFile($"appsettings.{env.EnvironmentName}.json", optional: true)
            .AddEnvironmentVariables();
        Configuration = builder.Build();
    }

    public IConfigurationRoot Configuration { get; }

    // This method gets called by the runtime. Use this method to add services to the container.
    public IServiceProvider ConfigureServices(IServiceCollection services)
    {
        // Add framework services.
        services.AddMvc();
        // Autofac

        ContainerBuilder builder = new ContainerBuilder();
        builder.RegisterModule(new Application.AppComponents.IOCModule());

        builder.Populate(services);
        var container = builder.Build();
        return container.Resolve<IServiceProvider>();
    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
    {
        loggerFactory.AddConsole(Configuration.GetSection("Logging"));
        loggerFactory.AddDebug();

        app.UseMvc();
    }
}

Copied from original issue: aspnet/Mvc#5623

If you run it without IIS Express what do you see in the console logs?

//.UseIISIntegration()

Same chatter about loading symbols. No error messages from build or Debug.

It's not an exception, it's just that something is returning 404 right? What request is returning a 404 that shouldn't be? You said you checked the content root, can you double check? Is it razor views that can't be found? If you turn logging up to the debug level:
loggerFactory.AddConsole(LogLevel.Debug);

Also change the run tab in visual studio to pick the application instead of IIS express. This will run on Kestrel directly:

It's not an exception, it's just that something is returning 404 right?
Correct.
What request is returning a 404 that shouldn't be?
Every request.
You said you checked the content root, can you double check?
Looked at env.ContentRootPath in a watch window and it is correct.
Is it razor views that can't be found?
No, .html page.
I think I misunderstood Tratcher's question about running without IIS Express.
When running under kestrel and startup is in library project:
dbug: Microsoft.AspNetCore.Hosting.Internal.WebHost[3]
	  Hosting starting
dbug: Microsoft.AspNetCore.Hosting.Internal.WebHost[4]
	  Hosting started
Hosting environment: Development
Content root path: C:\Git\AdaptiveClientDemo\AdaptiveClientDemo\src\RESTServer
Now listening on: http://localhost:5000
Application started. Press Ctrl+C to shut down.
dbug: Microsoft.AspNetCore.Server.Kestrel[1]
	  Connection id "0HL1C2L6N3NVR" started.
dbug: Microsoft.AspNetCore.Server.Kestrel[1]
	  Connection id "0HL1C2L6N3NVS" started.
dbug: Microsoft.AspNetCore.Server.Kestrel[6]
	  Connection id "0HL1C2L6N3NVR" received FIN.
dbug: Microsoft.AspNetCore.Server.Kestrel[10]
	  Connection id "0HL1C2L6N3NVR" disconnecting.
dbug: Microsoft.AspNetCore.Server.Kestrel[7]
	  Connection id "0HL1C2L6N3NVR" sending FIN.
dbug: Microsoft.AspNetCore.Server.Kestrel[8]
	  Connection id "0HL1C2L6N3NVR" sent FIN with status "0".
dbug: Microsoft.AspNetCore.Server.Kestrel[2]
	  Connection id "0HL1C2L6N3NVR" stopped.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
	  Request starting HTTP/1.1 GET http://localhost:5000/api/values/index
dbug: Microsoft.AspNetCore.Builder.RouterMiddleware[1]
	  Request did not match any routes.
dbug: Microsoft.AspNetCore.Server.Kestrel[9]
	  Connection id "0HL1C2L6N3NVS" completed keep alive response.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
	  Request finished in 305.7897ms 404

When running under kestrel and startup is in app project:
Hosting environment: Development
Content root path: C:\Git\AdaptiveClientDemo\AdaptiveClientDemo\src\RESTServer
Now listening on: http://localhost:5000
Application started. Press Ctrl+C to shut down.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
	  Request starting HTTP/1.1 GET http://localhost:5000/api/values/index
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[1]
	  Executing action method RESTServer.Controllers.ValuesController.Index (RESTServer) with arguments ((null)) - ModelState is Valid
info: Microsoft.AspNetCore.Mvc.ViewFeatures.Internal.ViewResultExecutor[1]
	  Executing ViewResult, running view at path /Project_Readme.html.
info: Microsoft.Extensions.DependencyInjection.DataProtectionServices[0]
	  User profile is available. Using 'C:\Users\Sam\AppData\Local\ASP.NET\DataProtection-Keys' as key repository and Windows DPAPI to encrypt keys at rest.
info: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[2]
	  Executed action RESTServer.Controllers.ValuesController.Index (RESTServer) in 1971.8227ms
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
	  Request finished in 2371.1935ms 200 text/html; charset=utf-8

I image you guys will want to see project.json:
Web app project:
{
  "dependencies": {
	"Autofac": "4.2.1",
	"Autofac.Extensions.DependencyInjection": "4.0.0",
	"Microsoft.AspNetCore.Mvc": "1.1.0",
	"Microsoft.AspNetCore.Routing": "1.1.0",
	"Microsoft.AspNetCore.Server.IISIntegration": "1.1.0",
	"Microsoft.AspNetCore.Server.Kestrel": "1.1.0",
	"Microsoft.Extensions.Configuration.EnvironmentVariables": "1.1.0",
	"Microsoft.Extensions.Configuration.FileExtensions": "1.1.0",
	"Microsoft.Extensions.Configuration.Json": "1.1.0",
	"Microsoft.Extensions.Logging": "1.1.0",
	"Microsoft.Extensions.Logging.Console": "1.1.0",
	"Microsoft.Extensions.Logging.Debug": "1.1.0",
	"Microsoft.Extensions.Options.ConfigurationExtensions": "1.1.0",
	"Microsoft.NETCore.App": {
	  "type": "platform",
	  "version": "1.1.0"
	},
	"Application.Domain": { "target": "project" },
	"Application.Model": { "target": "project" },
	"Application.Services": { "target": "project" },
	"Application.AppComponents": { "target": "project" },
	"LeaderAnalytics.AdaptiveClient": "0.0.4"
  },

  "tools": {
	"Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-preview2-final"
  },

  "frameworks": {
	"netcoreapp1.1": {
	  "imports": [
		"dotnet5.6",
		"portable-net45+win8"
	  ]
	}
  },

  "buildOptions": {
	"emitEntryPoint": true,
	"preserveCompilationContext": true
  },

  "runtimeOptions": {
	"configProperties": {
	  "System.GC.Server": true
	}
  },

  "publishOptions": {
	"include": [
	  "wwwroot",
	  "**/*.cshtml",
	  "appsettings.json",
	  "web.config"
	]
  },

  "scripts": {
	"postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  }
}

Library project:
{
  "version": "1.0.0-*",

  "dependencies": {
	"Autofac": "4.2.1",
	"Autofac.Extensions.DependencyInjection": "4.0.0",
	"NETStandard.Library": "1.6.1",
	"Microsoft.AspNetCore.Mvc": "1.1.0",
	"Microsoft.DotNet.InternalAbstractions": "1.0.0",
	"Microsoft.AspNetCore.Routing": "1.1.0",
	"Microsoft.AspNetCore.Server.IISIntegration": "1.1.0",
	"Microsoft.AspNetCore.Server.Kestrel": "1.1.0",
	"Microsoft.Extensions.Configuration.EnvironmentVariables": "1.1.0",
	"Microsoft.Extensions.Configuration.FileExtensions": "1.1.0",
	"Microsoft.Extensions.Configuration.Json": "1.1.0",
	"Microsoft.Extensions.Logging": "1.1.0",
	"Microsoft.Extensions.Logging.Console": "1.1.0",
	"Microsoft.Extensions.Logging.Debug": "1.1.0",
	"Microsoft.Extensions.Options.ConfigurationExtensions": "1.1.0",
	"NLog": "5.0.0-beta03",
	"Application.Core": { "target": "project" },
	"Application.Domain": { "target": "project" },
	"Application.Model": { "target": "project" },
	"Application.Services": { "target": "project" },
	"Application.RESTClient": { "target": "project" },
	"Application.WCFClient": { "target": "project" },
	"LeaderAnalytics.AdaptiveClient": "0.0.4"
  },
  "frameworks": {
	"net462": {
	  "dependencies": {
	  }
	},
	"netstandard1.6": {
	  "imports": [ "dnxcore50" ]
	}
  }
}

@rynowak Does the MVC controller discovery take into account the location of the Startup class?

Yes, it does. Specifically, it uses the ApplicationName as the "entry point" assembly. The controller scan starts from that "root" assembly.
@sam-wheat Do you have the Startup in a class library and the Controllers in another class library?

OMG you guys are working on Christmas Eve!  Merry Christmas!

Do you have the Startup in a class library and the Controllers in another class library?
No, controllers are in web app.

You can workaround it by doing the following in Program.cs
public static void Main(string[] args)
    {
        var host = new WebHostBuilder()
            .UseKestrel()
            .UseContentRoot(Directory.GetCurrentDirectory())
            .UseIISIntegration()
            .UseStartup<Application.AppComponents.Startup>()
            .UseSetting(WebHostDefaults.ApplicationKey, typeof(Program).GetTypeInfo().Assembly.FullName) // Ignore the startup class assembly as the "entry point" and instead point it to this app
            .Build();

        host.Run();
    }

Works, thank you!

I ran into this today, in my case because I had an alternate Startup class in my test assembly. Great workaround @davidfowl!


Dictionary to Map

Dictionary is a nice format, but the LV-Data package could provide APIs to more easily list keys, retrieve data according to key, convert to map
Explore use of VIMs?

released v3.4


Recording Rules take more time in Prometheus 2.3.0 after upgrade

Proposal
Use case. Why is this important?
Prometheus recording rules should not take more time
Bug Report
What did you do?
Upgraded Prometheus 2.2.1 to 2.3.0. The same set of recording rules were taking less time with previous version.
What did you expect to see?
Alarms based on metrics (recording rule evaluation) should not have gotten fired.
What did you see instead? Under which circumstances?
We are observing alarms firing.
Environment

System information:
Linux 4.1.12-124.19.7.el7uek.x86_64 x86_64

Prometheus version:
prometheus, version 2.3.0 (branch: heads/v2.3.0, revision:
290d71791a507a5057b9a099c9d48703d86dc941)
build user:       teamcity@tca-hv05-agent018-default
build date:       20181030-04:34:03
go version:       go1.11.1

Can you share the recording rules and the definition of the firing alerts?

We've looked at this as part of our bug scrub, and lacking information we're presuming this is no longer an issue as we're up to 2.17.1. If you've more information, please let us know.


Remove guava dependency from the API artifact.

Copied the BaseEncoding class for base16 from Guava (including tests).
/cc @saturnism

Codecov Report

Merging #1393 into master will increase coverage by 0.02%.
The diff coverage is 94.28%.

@@             Coverage Diff             @@
##             master   #1393      +/-   ##
===========================================
+ Coverage     84.17%   84.2%   +0.02%
- Complexity     1459    1469      +10
===========================================
  Files           227     228       +1
  Lines          6750    6775      +25
  Branches        630     635       +5
===========================================
+ Hits           5682    5705      +23
  Misses          886     886
- Partials        182     184       +2

Impacted Files
Coverage Δ
Complexity Δ

api/src/main/java/io/opencensus/trace/TraceId.java
95.45% <100%> (-0.2%)
24 <4> (-1)

api/src/main/java/io/opencensus/trace/SpanId.java
94.59% <100%> (-0.28%)
21 <4> (-1)

...census/trace/internal/LowerCaseBase16Encoding.java
93.1% <93.1%> (ø)
12 <12> (?)

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 884015c...fd3be3c. Read the comment docs.

Nice!  I think this can be removed now: https://github.com/census-instrumentation/opencensus-java/blob/884015cffa807a168e26ce36e8ae6d5ab426d8bb/buildscripts/import-control.xml#L57-L58


Expression does not seem to have any effect.

This expression does not seem to have any effect. If the convergence centroids assumed to be sorted based on the number of pixels drained, then this may have to be eventually fixed.
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L670
Similarly (it is done properly a few lines below, and it may not introduce any unexpected behavior anyways):
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L680
Also, the boundary condition may not have any effect:
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L695

IMO the whole loop here:
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L682
is probably dead code since it updates pix[k] (that is a copy of the seeds) in line
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L699
with the pixels in newpix (containing pix[k] and neighbors of pix[k]) having h value > 2. This can only be satistied by pix[k] because in line
https://github.com/MouseLand/cellpose/blob/5fb74fde155c35dafcd096fd66118b3cfdd174a0/cellpose/dynamics.py#L666
we selected the seeds such that none of them can have first and second order neighbors.

will be fixed in new release


User

Added navbar-animations project to web dev projects folder.

hey @ron2111 I have added projects to web dev projects folder #428 and #430. Please check and merge.


feat: Add ignore_exceptions to Client

Problem
Currently CordCutter works on all exceptions, even the ones we handle with the @command.error decorator.
A new argument that can accept several exceptions will ignore typed exceptions.
Additional context
from nextcord import Client, Interaction
from nextcord.ext.application_checks import has_permissions, ApplicationMissingPermissions

from cordcutter import Cordcutter

client = Client()
cordcutter = Cordcutter(client)

@client.slash_command(name="test")
@has_permissions(manage_roles=True)
async def test_command(interaction: Interaction) -> None
    """
    Currently CordCutter will turn on for two people.
    For the person who has permissions but the command is broken
    and for the person who does not have permissions to use this command.
    Because the command will raise an `ApplicationMissingPermission` exception.
    Ignoring individual exceptions will avoid this situation
    """

    raise RuntimeError("Shit. My command is broken :(")

@test_command.error
async def test_command_error(interaction, error):
    if isinstance(error, ApplicationMissingPermissions):
        await interaction.send("u dont have permissions")

I also thought about sending a warning to the logger with information about ignoring the exception, but I'm not sure.

hey @xXenvy, this feature has been implemented in 56c35f8ed2199d971e0d575d0883bc80ad48cccc

hey @xXenvy, this feature has been implemented in 56c35f8
take a look at the updated example

Yo! Sorry for the late response, but I've been a bit busy. Your implementation is a little different from what I wanted to achieve, however, it is still cool.
But I don't think it solves what I wanted to fix, because I don't think it's possible to globally ignore ApplicationMissingPermissions ? This exception can block the command even if someone has it handled in the error handler. Well, unless something has changed recently (I haven't used it in a long time). I'm betting that a similar effect can be achieved by the function-cooldowns library, which also probably triggers exceptions.


[Artist Rail] Fix right margin cutoff of cards by adding spacer view #trivial #tiny

Fix #309
Cards now bleed to the edge:

There's still additional space at the end of the rail:

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/components/home/artist_rails/artist_rail.js

  Generated by :no_entry_sign: danger

I thought that we wouldn't require changelog changes for a #trivial pr? :o

maybe wasn't added to the Dangerfile?

I think it's #tiny in emission. After deadline I'll look into making them consistent.

:dancing_men:


Bl vectorized map estimates

The map estimate methods can now take in multiple images at once

the .get_metrics method in sleep now uses the vectorized functions, and the tests pass on my machine. Feel free to merge after a review


⚠ Minecraft has degraded performance

In 99861aa, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1155 ms

Resolved: Minecraft performance has improved in f49df7b.


This closes #494 at the runner level to give "just enough protection"

This closes #494 at the runner level to give "just enough protection"
to allow using self-hosted runners with public repos.
This PR could do with a bit of a tidy up (some strings should likely become constants/enums, and a few things should possibly be added as methods on the GitHubContext class) but I wanted to get a :+1: from the approach first.
(Also C# is not my "native" language so there may be some oddities in here)
By default, the current behaviour is unchanged -- all jobs passed to the runner are executed.
If the .runner config file has this block added to it:
  "pullRequestSecurity": {}

Then by only PRs from "CONTRIBUTORS" (as defined by the field in https://docs.github.com/en/free-pro-team@latest/graphql/reference/objects#pullrequest -- nothing for us to have to work out ourselves.)
It is also possible to explicitly list users that are allowed to run jobs on this worker:
  "pullRequestSecurity": {
    "allowedAuthors": ["ashb"]
  }

Or to only allow the given users, but not all contributors:
  "pullRequestSecurity": {
    "allowContributors": false,
    "allowedAuthors": ["ashb"]
  }

Owners of the repo are always allowed to run jobs.
Originally posted by @ashb in https://github.com/actions/runner/pull/783

Ok

Ok

Good

Do me a favour?
Cool.
Thanks.


Add h3rald.com to webring

I added the icon to the webring on the footer of every page.

Hi,
Nice website, I was wondering do you have other projects than code(audio/visual) documented on the site? I try to not focus on scripts and collect websites that have an interest in arts. Let me know, I wouldn't want to have missed it. Also, do you have an about page, other than the home page?

Welcome to the webring :) Looking forward to watch your website grow.


Create CODE_OF_CONDUCT.md

Description
Add code conduct
Status
READY
Type of change

Documentation (adds documentation)

How Has This Been Tested?
Read documents and i dont find any errors.
Checklist:

[x] I have made corresponding changes to the documentation (if applicable)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

It's all right


feat(cli): add --expo option to CLI for creating Expo projects

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
This PR adds support for initializing Expo projects using the --expo option in the CLI. It enhances projectType and framework selection based on user input.
Detailed summary

Added --expo option to initialize Expo projects
Updated framework selection based on --expo option
Updated CLI help text for --react-native option to include Expo reference

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

#3455  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on  Graphite


High Order Types

Given the new this return type and F-bounded polymorphism I was a little surprised that I still could not express a functor interface
Example of Problem
interface IFunctor<A> {
  /** Maps `A` into `B` */
  map<B>(f: (a: A) => B): IFunctor<B>;
}

// A class with more than just IFunctor in it.
class Maybe<A> implements IFunctor<A> {
  constructor(init: A) { this.value = init; }
  isEmpty(): boolean { return false; }
  map<B>(f: (a: A) => B): IFunctor<B> { return new Maybe(f(this.value)); }
}

function toString<F extends IFunctor<any>>(a: F) {
  return a.map((v: any) => '' + v); // no way to define that this returns F<string>
}

let a: Maybe<number> = new Maybe(100);
let b: Maybe<string> = toString(a);
// error TS2322: Type 'IFunctor<string>' is not assignable to type 'Maybe<string>'.

Ideal Solution:
Ideally we should be able to define that F needs to have a generic type, which we can change
interface IFunctor<A, F<A> extends IFunctor<> {
  /** Maps `A` into `B` */
  map<B>(f: (a: A) => B): F<B>;
}

or we can define the functions generics
function toString<A, F<A> extends IFunctor<A>>(a: F<A>): F<string> { ... }

Can you fix up the repro so it produces the error you're describing (and no other errors)? I wrote this, trying to fill in the gaps:
interface IFunctor<A> {
  /** Maps `A` into `B` */
  map<B>(f: (a: A) => B): IFunctor<B>;
}

// Mark javascript arrays as Functor
declare interface Array<T> extends IFunctor<T> {}

// Create another functor like structure
class Maybe<A> implements IFunctor<A> {
	map<B>(f: (a: A) => B): IFunctor<B> { return undefined }
}

let a: Maybe<number>; // = Maybe.just(100)
let b: Maybe<string> = a.map((x) => '' + x)

and didn't get an error

Hi Ryan, sorry about that - I've updated the example code which now throws an error at compile

This (below) compiles. You put an explicit type annotation on Maybe#map that said it returned a less-specific type, then tried to convert that to a more-specific type on the final line -- the compiler can't know that that's safe.
interface IFunctor<A> {
  /** Maps `A` into `B` */
  map<B>(f: (a: A) => B): IFunctor<B>;
}

// Mark javascript arrays as Functor
declare interface Array<T> extends IFunctor<T> {}

// A class with more than just IFunctor in it.
class Maybe<A> implements IFunctor<A> {
  value: A;
  constructor(init: A) { this.value = init; }
  isEmpty(): boolean { return false; }
  map<B>(f: (a: A) => B) { return new Maybe(f(this.value)); }
}

let a: Maybe<number> = new Maybe(100);
let b: Maybe<string> = a.map((x: number) => '' + x);

duplicate of https://github.com/Microsoft/TypeScript/issues/5999?

dupe of #1213 (although the title of it is sort of strange)

a bit of a rant @RyanCavanaugh's been giving some tough love there, how many people do you guys know who can go into your code base and outline how HKT would look like right there in code? 3? 4?

how many people do you guys know who can go into your code base and outline how HKT would look like right there in code? 3? 4?

Correct (if not fewer). The scarcity of such resources is exactly the source of its current prioritization. We have a large number of features that can only be done correctly by a few experts, so it's important that those people are working on the features with the highest value. No one's saying it wouldn't be a cool thing to have (I :heart: FP features too), but it addresses fewer scenarios than other things currently in the pipe.

Sorry I was trying to make a simple example of what I've been trying to do and missed a key point.
I've updated the example code again, trying to show a failure which I can't see how to get around without high order types.
#1213 is very similar but takes a different approach to the issue (separate classes for the monads, rather than building it into the classes directly).
For us it's not a priority as we've worked around it, but would be nice to have in the future

closing in favor of #1213


Unexpected level error

The following code
variables {a r : Type}

definition f a := Πr, (a -> r) -> r

lemma blah2 (sa : f a) (k : (a -> r)) :
  sa r k = sa r k :=
  sorry

gives a level error (I can't figure out how to copy paste the exact message from the web editor). Yet if I inline the definition of f into the type of blah2, it compiles.

Thanks for reporting the problem. I'm investigating. In the meantime, we can workaround this issue by providing the universe levels explicitly. Example:
definition f (a : Type) := Π r : Type, (a → r) → r

lemma blah2.{l₁ l₂} {a  : Type.{l₁}} {r  : Type.{l₂}}  (sa : f.{l₁ l₂} a)  (k  : a → r) :
    sa r k = sa r k :=
    sorry

Or
definition f (a : Type) := Π r : Type, (a → r) → r

universe variables l₁ l₂
variables {a  : Type.{l₁}} {r  : Type.{l₂}}

lemma blah2 (sa : f.{l₁ l₂} a) (k  : a → r) : sa r k = sa r k :=
sorry

Thanks for the quick fix. Is there a way for me to take advantage of it in the browser? My actual program is a bit more complicated than this minimal example, and I'm struggling to provide the right annotation.

Is there a way for me to take advantage of it in the browser?

I've just updated the web version.


Add contributing.md file

This is a file that provides a potential contributor with a short guide on how to contribute to this project quickly, and to know what standards are accepted inorder to contribute.

.take


Is there any way to use Easegress etcd cluster storage api in my own custom business filter?

I am developing new filter using Easegress，in other api gateway sometime I need to use cluster cache like redis to cache expired token，but in Easegress it already has etcd embedded and I don't want to introduce new outer cache component， so is it possible to use Easegress own cluster storage api do something like this, or is it a proper way?

Please check if the CustomData feature meets your requirement (the PR has been merged, but the feature has not been released yet, please get the latest code from the main branch).
Or, you can use the APIs in the cluster package directly for data storage.

Thanks, CustomData cannot support expired data store. I investigate the source code，currently the only way to to initialize cluster storage api for client user is to get command opt params and call cluster.New(opt) like the way in cmd/server/main.go.  We should support something like factory pattern in pkg/cluster/cluster_interface.go  so client user can get cluster instance conveniently.

@jthann  You don't have to initialize new cluster instance, because Easegress creates one instance at startup and it's reused everywhere. Here's example how you access to the global cluster instance from Filter.Init function.

Thanks，I get it.


HTTP Custom Route Example Code does not work

Bug Report
Current Behavior
When placing the example code from the documentation in index.js and navigating to https://localhost:3000/my-app/hello-world, there is no response. A GET request from Postman also does not work.
module.exports = (app, { getRouter }) => {
  // Get an express router to expose new HTTP endpoints
  const router = getRouter("/my-app");

  // Use any middleware
  router.use(require("express").static("public"));

  // Add a new route
  router.get("/hello-world", (req, res) => {
    res.send("Hello World");
  });
};

Expected behavior/code
When navigating to the page: https://localhost:3000/my-app/hello-world to access the endpoint, the server should respond with "Hello World."
Environment

Probot version(s): v11.0.5
Node/npm version: v14.15.4
OS: Windows 10

Possible Solution
None.
Additional context/Screenshots
Log output:
> probot run ./index.js

INFO     (server): Running Probot v11.0.5 (Node.js: v14.15.4)
INFO     (server): Forwarding https://smee.io/xxxxx to http://localhost:3000/
INFO     (server): Listening on http://localhost:3000
INFO     (server): Connected

Can you share the log output from your server?

@gr2m Added the log output in the issue 👍

I tried to reproduce the problem but to no avail, it works as documented for me

$  LOG_LEVEL=trace npx probot run ./debug.js
INFO (server): Running Probot v11.0.6 (Node.js: v14.15.4)
INFO (server): Forwarding https://smee.io/Xs7emdfD6u2i7Svz-wip-production to http://localhost:3000/
INFO (server): Listening on http://localhost:3000
INFO (server): Connected
INFO (http): GET /hello-world 304 - 7ms
INFO (http): GET /hello-world 200 - 1ms
INFO (http): GET /favicon.ico 404 - 2ms

It might be a windows problem, unfortunately I don't have a windows machine to test with myself

Turned out to be an issue with my computer - I restarted my machine and it works now.


feature(api): Weather v2

Description
This PR introduces the Weather API v2 which returns the day value as weekday on forecast data and we're able to return some values ( temperature and wind ) based on weather units.
This endpoint use the wttr JSON response because it's tricky to get the weekday parsing the HTML as the v1 does for other fields.
To make it backward compatible there is a new endpoint /v2/weather/. The v1 keeps untouched.
Examples
Get weather (returning weekday on forecast )
➜ curl http://localhost:3000/v2/weather/Curitiba | jq
{
  "temperature": "26 °C",
  "wind": "7 km/h",
  "description": "Partly cloudy",
  "forecast": [
    {
      "day": "Thursday",
      "temperature": "15 °C",
      "wind": "6 km/h"
    },
    {
      "day": "Friday",
      "temperature": "19 °C",
      "wind": "5 km/h"
    },
    {
      "day": "Saturday",
      "temperature": "22 °C",
      "wind": "5 km/h"
    }
  ]
}

Get weather in USCS (used by default in US)
Obs: unit M (used by default everywhere except US) is default.
curl http://localhost:3000/v2/weather/Curitiba\?unit=u | jq
{
  "temperature": "79 °F",
  "wind": "4 mph",
  "description": "Partly cloudy",
  "forecast": [
    {
      "day": "Thursday",
      "temperature": "59 °F",
      "wind": "4 mph"
    },
    {
      "day": "Friday",
      "temperature": "65 °F",
      "wind": "3 mph"
    },
    {
      "day": "Saturday",
      "temperature": "72 °F",
      "wind": "3 mph"
    }
  ]
}

Fixes #18 Fixes #17

Thank you very munch for you contribution @marcosinger ! I really appreciate it :rocket:  I'm reviewing it right now :smile:


Allow users to delete their own accounts

We have the API endpoint for this. Includes revised wording to indicate when you're deleting your own account.
NOTE: Untested! If someone can pull this down and confirm functionality, that'd be amazing. Otherwise I'll do so on Saturday or so.

Per https://github.com/joindin/joindin-api/issues/621, closing this one.


Stuck in grabbing mode on load

when button mappings are customized

Need to separate the concepts of "no longer trying to start an interaction" from "terminate an existing interaction" for better experience when the start and stop buttons are differnent (e.g. the sticky example)

Fixed #37


/midi cc ... doesn't map values

Apparently /midi cc ... is not mapping values correctly. It just maps input to input.linlin(0,127, 0.0,1.0)

Apparently it does map values...


Integrate Google Gemini and AWS Bedrock

Any plans to integrate Gemini and Bedrock? Independent of DSPy I'm working on testing different models, similar to what anyscale is doing in Building RAG-based LLM Applications for Production. I already have my own code to do this and many others have written similar code, but as I'm starting to use DSPy, I'm getting confused where the boundary of the library should be. It seems to me that DSPy would benefit from standardizing a bit more, but I'm not sure if that is the vision for the library.
Personally I'd prefer the library to allow me to more easily swap out models, chunking strategies, etc (again very similar to what is done in the blog post shared above). I see the library as an obvious replacement for langchain like libraries and many other variants attempting to do more or less the same thing. Except as it currently stands, I'm not sure exactly how I would integrate DSPy into my own workflow without adding some of the complexity which I'm trying to avoid. Any thoughts?

I agree that internals need a more explicit standard, especially around adding support for new models.
LMs can be modeled after this class, though probably they can be shorter: https://github.com/stanfordnlp/dspy/blob/main/dsp/modules/gpt3.py
For chunking strategies, that feels outside the scope. It should be straightforward to handle that in user code? Or is there an argument for making this a built-in thing?

I do agree chunking feels outside of the scope (I hesitated to add it in the first place). But I ended up adding it because it's hard for me to separate all of these topics into buckets once I get to the implementation stage. It feels a bit like the microservices vs monolith debate to me, and I think my current bias is to think of these LM systems as a single thing vs isolated modules with single responsibility.
I think it's challenging during implementation because there are pretty strong dependencies between completions, prompts, context, and embeddings and it's quite difficult to know up front which strategies will work best for a specific use case (basically this is the problem I'm facing now thinking to myself "should I implement this myself or should it be in the library?"). Which leads me to wanting more flexibility for easily swapping out all of the pieces. I don't think that necessarily implies DSPy should be taking on all the responsibility though.
I think what I need to do before considering adding more to DSPy is create a full application that uses DSPy and do my own experimentation of swapping things in and out and see what happens.

@jhyearsley can you point out in the dspy library where this was implemented?


Fix empty pod or node list resulting in an error

Fixes #147
Tested this on my cluster and works as expected. Technically I don't see how the node list should ever be empty, but I fixed the case anyways. Also let me know if you prefer nil or empty lists rather, I have no strong opinion on either.
@DirectXMan12 @s-urbaniak @squat @metalmatze

I don't have a strong opinion on this either.
Maybe it's better for the caller to return with empty list and not nil three times.

This looks good to me as is :)

If the caller asserts len([]provider.TimeInfo) == 0 and len([][]metrics.ContainerMetrics) == 0, then both will be true if return nil, nil, nil is being used, that's quite idiomatic here.
I am :+1: on the change.


[FLINK-32945][runtime] Fix NPE when task reached end-of-data but checkpoint failed

What is the purpose of the change
Fix NullPointerException when executing TopSpeedWindowing example with checkpointing enabled.
The bug happens when some vertices have reached the end of data but do not transform to FINISHED because their final checkpoint fails and causes a failover. The VertexEndOfDataListener would remove the end-of-data vertices from its state, however, in such case these vertices would be restarted and try to access the VertexEndOfDataListener again, which causes the NPE.
Brief change log

Restore the state of VertexEndOfDataListener when some tasks are going to be restarted.

Verifying this change
This change added tests and can be verified as follows:

Added test that validates that the state of VertexEndOfDataListener would be restored when failover happens.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable )

@lindong28 Could you help review this PR?

@flinkbot  run azure

@flinkbot run azure

Thanks for the update. LGTM.


tests: add base tests for commands

Description
This PR adds basic tests for commands.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

If anyone has an idea what is going on with rector, I would appreciate any hint.
Locally everything is passing on my machine 🤷♂

This is rather a temporary solution. Thanks to @kenjis we already have very good Mocking for CLI: https://github.com/codeigniter4/CodeIgniter4/blob/4.5/system/Test/Mock/MockInputOutput.php, but it will be released with v4.5.

Ah, ok, nice! I missed that one.

@michalsn

Installing rector/rector (0.15.25): Extracting archive

The version is too old.


[FIXING URL] Notre Tombeau

Just a minor update to fixing picture URL to point to Roll20 Github.

Character Sheet Info Roll20 Internal Use only.


Module level Client forcefully loaded when in PyCharm debugger

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
If I run the following script simply via Python
from openai import OpenAI

openai = OpenAI(api_key="MY KEY")

print("We got to the end!")

It works fine. However, in PyCharm's debugger, it seems like the _module_client is being forcefully initialized, so I get this error raised when trying to run on the from openai import OpenAI line
# in _client.py
        if api_key is None:
            raise OpenAIError(
                "The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable"
            )

The workaround that worked for me is conditionally loading the module client like this in __init__.py at the end of the file:
import sys

gettrace = getattr(sys, 'gettrace', None)

if gettrace is None:
    # We're not in debug mode, load libraries
    from ._module_client import (
        beta as beta,
        chat as chat,
        audio as audio,
        files as files,
        images as images,
        models as models,
        batches as batches,
        embeddings as embeddings,
        completions as completions,
        fine_tuning as fine_tuning,
        moderations as moderations,
    )

To Reproduce
Follow instructions from the description.
Code snippets
No response
OS
Windows 11
Python version
3.12
Library version
1.23.2

This is a duplicate of https://github.com/openai/openai-python/issues/1108 - but the tl;dr is that we view this as a bug in PyCharm and would very much appreciate it if you'd file a ticket with them.


Freebayes AF values different from calculated AF values

Hello Erik,
I just started using Freebayes and I have a small question that may have already been asked before (apologies).
In my multisample VCFF file, I have noticed that the AF values given by Freebayes are not the same as the values I get when I compute AF values directly from AO and RO fields for each sample. Therefore also the AF spectrum histograms look different when I plot them out using the Freebayes-given AF values and computed values.
Would you be able to explain why this is so? I'd appreciate it much to understand this and know which of two approaches/values to work with. Kindly let me know.
Thanks!
David.

The AF values are determined by the number of alleles in called genotypes. The AO and RO fields reflect the number of read observations for each alternate and the reference allele respectively.


Animated GLB with mesh point animation - crash Windows Explorer

Animated GLB with mesh point animation - crash Windows Explorer.
Only this type of GLB animation - crash Windows 10 Explorer.
Other animated GLB & FBX - is - ok !
Example of my glb- in zip is attached.
Flag TelegramLooP80.zip

Yes, same here, did you find a work-arround ?

Not previewer? but viewer...
https://github.com/f3d-app/f3d


More projects (Node.js and Python)

The first of probably quite a few issues branching off of #1.
Add support for a skeleton project for Node.js and Python. Maybe start with a package.json for node?

What kind of python project support do we want?


Stream Enrich: propagate pii validation to other platforms

Atm only Kinesis is validated wrt to the pii stream

I was wrong closing


Synchronize certs in case they're missing

The run_once only sets the variable for the first host in the play,
which usually is the first master, which already has the certs. Ensure
we set it for all hosts in the current play, so they get certs
synchronized properly.

I'm not sure the run_once is the cause of your issue.
Could you please paste (again) your inventory ? i'll do some tests.
Thanks

@Smana this is my inventory:
scratchbox1
scratchbox2

[kube-master]
scratchbox1

[etcd]
scratchbox1

[kube-node]
scratchbox1
scratchbox2

[k8s-cluster:children]
kube-node
kube-master

If i understand run_once correctly this task will only be run for the first host in the current play, resulting in other potential master hosts won't get the sync_certs variable set correctly.

Hrm, a simple test seems to contradict that though -- didn't really expect that.

Okay, sneaking suspicion that run_once with delegate_to is pretty terrible.. Looks like it will set the variable on all current play_hosts to whatever the delegated hosts value is.

Oh, this is actually intended in this case -- would probably help if i actually understood the intent of the code :D

Hello @davidreuss, are you still facing the issue ? because indeed i don't think this is related to the run_once

We can just close this, i'll reopen if i can get some more evidence/proper reproduction of the failure going.
Sent from my iPhone

On 23. maj 2016, at 18.40, Smaine Kahlouch<EMAIL_ADDRESS>wrote:
Hello @davidreuss, are you still facing the issue ? because indeed i don't think this is related to the run_once
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

ok :)


[PDI-16422] runTrans web service does not apply transformation parameters

[PDI-16422] runTrans web service does not apply transformation parameters
clearing parameters was removed

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change:  + 7.14%
Complexity Change:  + 7.69%

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change: -.53%:small_red_triangle_down:
Complexity Change: -.56%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change:  + 1.00%
Complexity Change:  + 1.41%
Instruction Change:  + .30%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.28%:small_red_triangle_down:
Line Change: -3.87%:small_red_triangle_down:

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change:  + .66%
Complexity Change:  + .93%
Instruction Change:  + .13%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change:  + .31%
Instruction Change:  + .06%
Line Change: -.23%:small_red_triangle_down:
Method Change: -1.02%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3.12%:small_red_triangle_down:
Complexity Change: -3.16%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.18%
Complexity Change:  + 1.35%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change:  + 2.04%
Complexity Change:  + 2.04%
Instruction Change:  + .32%
Line Change:  + .81%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change:  + 1.79%
Complexity Change:  + 1.19%
Instruction Change:  + .07%

org.pentaho.di.trans.Trans

Branch Change:  + .20%
Complexity Change:  + .41%
Instruction Change:  + .22%
Line Change:  + .43%
Method Change:  + .90%

org.pentaho.di.trans.TransConfiguration

Complexity Change:  + 25.00%
Instruction Change:  + 12.90%
Line Change:  + 20.00%
Method Change:  + 25.00%

org.pentaho.di.trans.TransExecutionConfiguration

Complexity Change:  + 1.82%
Instruction Change:  + .45%
Line Change:  + .92%
Method Change:  + 2.99%

org.pentaho.di.www.RunTransServlet

Branch Change:  + 40.91%
Complexity Change:  + 12.55%
Instruction Change:  + 56.44%
Line Change:  + 61.36%
Method Change:  + 14.55%

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -.78%:small_red_triangle_down:
Complexity Change: -1.03%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change:  + .53%
Complexity Change:  + .56%
Instruction Change:  + .07%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change:  + .66%
Complexity Change:  + .93%
Instruction Change:  + .13%

org.pentaho.di.trans.step.StepMeta

Branch Change: -.68%:small_red_triangle_down:
Complexity Change: -.58%:small_red_triangle_down:
Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.26%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change:  + 2.00%
Complexity Change:  + 1.89%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.35%:small_red_triangle_down:
Complexity Change: -2.70%:small_red_triangle_down:
Instruction Change: -.18%:small_red_triangle_down:
Line Change: -.48%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change:  + .18%

org.pentaho.di.trans.Trans

Branch Change:  + .20%
Complexity Change:  + .41%
Instruction Change:  + .22%
Line Change:  + .43%
Method Change:  + .90%

org.pentaho.di.trans.TransConfiguration

Complexity Change:  + 25.00%
Instruction Change:  + 12.90%
Line Change:  + 20.00%
Method Change:  + 25.00%

org.pentaho.di.trans.TransExecutionConfiguration

Complexity Change:  + 1.82%
Instruction Change:  + .45%
Line Change:  + .92%
Method Change:  + 2.99%

org.pentaho.di.www.RunTransServlet

Branch Change:  + 40.91%
Complexity Change:  + 12.55%
Instruction Change:  + 56.44%
Line Change:  + 61.36%
Method Change:  + 14.55%

org.pentaho.di.www.TransformationMap

Branch Change: -9.09%:small_red_triangle_down:
Complexity Change: -6.15%:small_red_triangle_down:
Instruction Change: -10.03%:small_red_triangle_down:
Line Change: -10.00%:small_red_triangle_down:

@AliaksandrShuhayeu please add SP case number to the commit message

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/www/RunTransServlet.java
  engine/test-src/org/pentaho/di/www/RunTransServletTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.auth.core.impl.DefaultAuthenticationConsumerFactory

Branch Change: -7.14%:small_red_triangle_down:
Complexity Change: -7.69%:small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -.68%:small_red_triangle_down:
Complexity Change: -.58%:small_red_triangle_down:
Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.26%:small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -2.08%:small_red_triangle_down:
Complexity Change: -2.11%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .18%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change:  + 3.33%
Complexity Change:  + 1.85%
Instruction Change:  + .28%

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change:  + .53%

org.pentaho.di.trans.Trans

Branch Change:  + .20%
Complexity Change:  + .41%
Instruction Change:  + .22%
Line Change:  + .43%
Method Change:  + .90%

org.pentaho.di.trans.TransConfiguration

Complexity Change:  + 25.00%
Instruction Change:  + 12.90%
Line Change:  + 20.00%
Method Change:  + 25.00%

org.pentaho.di.trans.TransExecutionConfiguration

Complexity Change:  + 1.82%
Instruction Change:  + .45%
Line Change:  + .92%
Method Change:  + 2.99%

org.pentaho.di.www.RunTransServlet

Branch Change:  + 40.91%
Complexity Change:  + 12.55%
Instruction Change:  + 56.44%
Line Change:  + 61.36%
Method Change:  + 14.55%

org.pentaho.di.www.TransformationMap

Branch Change:  + 9.09%
Complexity Change:  + 6.15%
Instruction Change:  + 10.03%
Line Change:  + 10.00%


Command-key to toggle Open in Background preference

It would be nice if clicking a web link with the command-key toggled the Open in Background setting (so clicking the link could open in normally, and command-clicking could open it in the background).

Good point. I wonder if the callback API tells me anything about modifier keys. (I suppose I could inspect the current NSEvent.)

I suspect it would be safe enough to just use NSEvent.  It's unlikely to be that time critical.

This should be fixed in #2100 (for Mac 5.1); feel free to reopen if it's not.


Move ModuleIdentifier to Symbols

Bug: http://b/243520787

I see how this change is motivated: You need to access ModuleIdentifier in the Symbols component. Event though this works perfectly fine, there is a downside of doing this.  These modules are: FakeClient, DataViews, CodeReportTest and ClientData (OrbitGl and OrbitQt needs Symbols anyways). This ties these components tighter together than they need to be.
To not have this downside, I propose the following: Either, make a new component that only contains ModuleIdentifier, or see if the component ModuleUtils makes sense for ModuleIdentifier

I see how this change is motivated: You need to access ModuleIdentifier in the Symbols component. Event though your current change works perfectly fine, there is a downside of doing this.  These modules are: FakeClient, DataViews, CodeReportTest and ClientData (OrbitGl and OrbitQt needs Symbols anyways). This ties these components tighter together than they need to be.
To not have this downside, I propose the following: Either, make a new component that only contains ModuleIdentifier, or see if the component ModuleUtils makes sense for ModuleIdentifier

Yes. Henning also mentioned the problem of llvm dependency in Symbols and suggested adding an independent component.
I'm going to create a separated component.


Create a separate config file in settings

We want to put all usernames and passwords into a separate file and prevent submissions go github, plus we need to update the documentation to let the users know how to setup the the project with these new configure.

This looks like the solution to implement:
http://stackoverflow.com/questions/3605866/hide-password-when-checking-config-file-in-git


Update constants.js to add new H7106 fan

Resolves #781, resolves #784
Tested with actual H7106 versionHard 1.02.00, versionSoft 1.00.10
Fan on/off and speed settings working.
Noticed issue with oscillate switch in Apple Home app, similar to issue #521. The oscillate toggle in the Home app turns oscillation on and off but:

Immediately turns to off after being turned on, even though the fan continues to oscillate.
Often shows the wrong state.

By the way, I've also noticed that when I turn this fan on at 11% via the Home app, it turns on at 100% and then throttles back to 11% after a couple of seconds. Is this expected? Afraid I have no other Govee fans to test with!

Thanks!
will look at the other comments you wrote in an issue I will make now


Adding mutators and major code cleanup/documentation/test expansion

This is a fairly major patch that adds mutators to URI (see issue #27) and also fixes how the query-form is returns (see issue #33).
It adds new class and types for maintaining the correctness of the URI while modified and to provide piecewise encapsulation of data. This includes adding a number of subsets for validating the components and the URI::Query, URI::Authority, and URI::Path classes for maintaining state specific to those components of the URI.
The most controversial decision I think I made is a preference for what I call "heavy mutators" which are defined as methods on the class rather than as assignment. For example,
my $u = URI.new("mailto:bob@example.com");
$u.path("steve@example.com");

This is due to the fact that most of the mutators have to do some amount of specialized validation, conditional object construction, and cross-checking with other fields in order to keep the state of URI pure. The only way to achieve this with assignment is to use a lot of Proxy objects, but I do not believe this to be a good practice. (Historically, I have done just this on other work I've done and I believe that has often been a mistake.)
The other controversial decision I made is to deprecate query-form and make hash-like use of query data always work with lists of strings rather than a mixed form (see #33). However, because of how lists work in Perl 6, I believe there's a very good chance most existing uses of query-form will work correctly despite this. If not, there is a hash-format option for quickly recovering the old behavior. I will also say that I'd be willing to modify the code so that the deprecated query-form method always returns the mixed form and recommend everyone update their code to use query in the future.
I have expanded the test suite quite a bit and have attempted to thoroughly document everything URI is capable of doing going forward.

It appears to me that the README.md was not updated ...

Please add:
Ilya Belikin(ihrd)
To the list of contributors.  He wrote a regex version that comprised the first four commits of the repository.  Thanks.

I tried to run some of the examples from the README and a few did not work.
For example setting the userinfo:
ron@ws-ubuntu:~/perl6/zostay-uri$ perl6 -Ilib -MURI -e 'my $u = URI.new(q!http://example.com/foo/bar?tag=woow#bla!); .userinfo("bob") with $u.authority'
Too many positionals passed; expected 1 argument but got 2
  in method userinfo at /home/ron/perl6/zostay-uri/lib/.precomp/7A5C2A15488B7F2C467D33F04E36BD2EE0BCA9C1.1499026043.26079/CB/CB371C93C5AA0E62198EFD303AE2C17474416D1A line 1
  in block <unit> at -e line 1

The current userinfo setter method signatures all specify a URI object as invocant not an Authority.  My best guess is that there should be a setter methods with both invocants, with the userinfo setter for URI just calling something like .authority.userinfo($new_userinfo) for its authority property.
In updating the query later, the pair lookup and list index seem backwards:
ron@ws-ubuntu:~/perl6/zostay-uri$ perl6 -Ilib -MURI -e 'my $u = URI.new(q!http://example.com/foo/bar?tag=woow#bla!); with $u.query {.query("foo=1&foo=2"); say .query-form<foo>[0] }'
Type Array[Pair] does not support associative indexing.
  in block <unit> at -e line 1

ron@ws-ubuntu:~/perl6/zostay-uri$ perl6 -Ilib -MURI -e 'my $u = URI.new(q!http://example.com/foo/bar?tag=woow#bla!); with $u.query {.query("foo=1&foo=2"); say .query-form[0]<foo> }'
1

After fixing the indexing above, the example of .query-form.push: 'bar' => 'ok'; doesn't seem to change the topic object.  You might want to go through and retest your documentation examples.  When you are done let me know and I will take another look.
Thanks

Hi,
I've moved this into https://github.com/perl6-community-modules/uri/tree/zostay-mutators so we can move it on a bit.  All looks good.


More resilient error handling.

If you have bad authentication, the callback can be fired without the error, this PR addresses it.

I think travis may have failed because it was using the older architecture. I just tested updating the build with sudo false, and it seems to pass. Will you rebase your PR?

Still seems to fail for me with the credentials.

Looks like travis won't allow environment variables when it's a pull request.
I fetched and checked the tests locally, and there is an error deleting time entries. Here is the relevant CI entry, https://travis-ci.org/log0ymxm/node-harvest/jobs/83727748
Do your tests pass locally? Do you know what's causing this to fail?

I was unable to run the unit tests as I don't have a harvest account that I can use for testing, I had to test manually. I think this may fix it though. Sorry for the broken test! It's annoying I can't rely on CI to fix this - I took a look at the travis docs, but I don't see a good solution for this.

This looks correct, thanks for the update. Harvest does have a free account tier which can be used for testing, if you wanted to test locally.


Feature Request: do not transform build-in require function

Since nuxt.config.js is not packaged, the require will throw an error
const ROOT_DIR = rootDir || process.env.ROOT_DIR || process.cwd()
- const config = require(path.join(ROOT_DIR, 'nuxt.config.js'))
+ const config = __webpack_require__(7083)(path.join(ROOT_DIR, 'nuxt.config.js'))

Can we keep the built-in require?
For example by adding such a comment
const config = /*ncc-build-in-require*/ require(path.join(ROOT_DIR, 'nuxt.config.js'));

Although this job seems to be webpack
But if do it with a plugin, that will be great.

Fixed in #378


TypeError: redirect() takes 0 positional arguments but 1 was given

Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 2073, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1518, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1516, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1502, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
File "/binance/main.py", line 51, in index
return redirect("/register")
TypeError: redirect() takes 0 positional arguments but 1 was given
[2022-01-10 06:19:43,514] ERROR in app: Exception on / [GET]
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 2073, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1518, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1516, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1502, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
File "/binance/main.py", line 51, in index
return redirect("/register")
TypeError: redirect() takes 0 positional arguments but 1 was given
Environment:nginx

Python version:3.8.10
Flask version:last version

I can't reproduce this issue with the information given.

you're probably importing redirect from elsewhere or redefine it locally

you're probably importing redirect from elsewhere or redefine it locally

from flask import render_template, Flask, session, redirect, request, g, url_for


macos utf8 bug

E5108: Error executing lua ...chenxu/.local/share/nvim/lazy/jieba-lua/lua/jb_utils.lua:2: m
odule 'lua-utf8' not found:
        no field package.preload['lua-utf8']
cache_loader: module lua-utf8 not found
cache_loader_lib: module lua-utf8 not found
        no file './lua-utf8.lua'
        no file '/opt/homebrew/share/luajit-2.1/lua-utf8.lua'
        no file '/usr/local/share/lua/5.1/lua-utf8.lua'
        no file '/usr/local/share/lua/5.1/lua-utf8/init.lua'
        no file '/opt/homebrew/share/lua/5.1/lua-utf8.lua'
        no file '/opt/homebrew/share/lua/5.1/lua-utf8/init.lua'
        no file './lua-utf8.so'

after upgrade lua this problem disable


ObsoleteAttribute inheritance

Nice source generator. Thanks.
If i want delegate implementation to interface with Obsolete methods, i have compilation warnings CS0618.
https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/compiler-messages/cs0618?f1url=%3FappId%3Droslyn%26k%3Dk(CS0618)
For example, i want delegate Nlog.ILogger implmentation.
NLog source coge:
    /// <summary>
    /// Writes the diagnostic message at the <c>Debug</c> level using the specified parameters.
    /// </summary>
    /// <param name="message">A <see langword="string" /> containing format items.</param>
    /// <param name="args">Arguments to format.</param>
    [MessageTemplateFormatMethod("message")]
    void Debug([Localizable(false)][StructuredMessageTemplate] string message, params object[] args);

    /// <summary>
    /// Writes the diagnostic message and exception at the <c>Debug</c> level.
    /// </summary>
    /// <param name="message">A <see langword="string" /> to be written.</param>
    /// <param name="exception">An exception to be logged.</param>
    /// <remarks>This method was marked as obsolete before NLog 4.3.11 and it may be removed in a future release.</remarks>
    [Obsolete("Use Debug(Exception exception, string message) method instead. Marked obsolete with v4.3.11")]
    [EditorBrowsable(EditorBrowsableState.Never)]
    void Debug([Localizable(false)] string message, Exception exception);

    /// <summary>

Generated code:
    void NLog.ILogger.Debug(string message, params object[] args) => this._nlogImpl.Debug(message, args);

    void NLog.ILogger.Debug(string message, System.Exception exception) => this._nlogImpl.Debug(message, exception);

I think, we want nesting Obsolete attribute.

As I understood the real cause of the problem, the ObsoleteAttribute has Inherited=false and a copy of that attribute should be added to the generated source so compiler would not generate warning.
It seems the correct aproach would be to add an additional attribute PassthroughAttributes="Obsolete" (that is the list of attributes). In that case Obsolete would be forwarded by default, but anyone could change that behaviour for other attributes.
I don't have enough free time at the moment and will try to solve this and another issue by the end of this year.

I've made nuget prerelease package 1.0.30-pre that should fix this issue.

thanks. It work for me.


fabrik ik integration

I'm also interested in using the new model constraints published for FABRIK with vsr_chain. Any chance those are present or could be added relatively easily?

Extending FABRIK with model constraints paper
Overview of IK methods, including some FABRIK model constraints details
FABRIK IK algorithm creator's site.

In my case I just have 7dof revolute joints and need to set joint limits.
There is also an interesting thread related to constraints, and another on math.stackexchange
the versor library provides an implementation, which I may be able to use:
https://github.com/wolftype/versor/issues/4

won't be doing this, we have settled on https://github.com/jrl-umi3218/Tasks


Fix flake8 warnings in cockroach_settings.py

(Could we add PR checks for flake8 and isort?)

are those separate plugins for github, or do they just need to be added to the CI build script?

I'd suggest to add them as separate jobs so that each PR has three checks (tests, flake8, isort). That makes it easy to see what's failing. The commands:
pip install flake8
flake8

and
pip install isort
isort --recursive --check-only --diff

I'm going to merge this, I have another PR out that is going to add checks for flake8 and isort.


slot in child component

Hi,
I know this is somewhat strange but I need nested components which pass their content on.
//comp1 template
.comp1
  comp2
    slot

//comp2 template
.comp2
  slot

//usage
comp1
  comp3

the generated dom turns out correctly (which is amazing :smile:)
comp1
  comp2
    comp3

but the comp3 will be in $children of comp1 and not of comp2
(in my case it is even more complicated as comp2 and comp3 have v-fors and so I can't even search the right comp3 out of the $children of comp1)

I tried to work around by basically doing this:
// comp1 template
.comp1
  slot(:data="data" v-for="data in dataset")
// usage
comp1
  comp2

but comp2's data prop won't be bound..

This is working as intended, so I think this is more like a question. Please move it to the forum instead.
Also please clarify what are you trying to achieve - access comp3 from comp2?

I made a small fiddle
I'm trying to build a sophisticated v-for with Clusterize.js functionality in form of a component.
Basically I have 4 clusters (own components - only used within the main component) with each a few (depending on content and overall height) rows (also own components, managed by the corresponding cluster component).
The clusters move on scrolling and the rows get filled with the appropriate data.
I got this already working but only when I create the rows from a fixed template.
here is my code more detailed:
##clusterize.vue
<template lang="jade">
.clusterize
  // ...
  clusterize-cluster(v-for="cluster in 4"")
    slot
 // ...
</template>
<script lang="coffee">
module.exports =
  components:
    "clusterize-cluster": require "./clusterize-cluster"
   # ...stuff to fill clusterize-cluster.data
</script>

##clusterize-cluster.vue
<template lang="jade">
.clusterize-cluster
  slot(:data.sync="d" v-for="d in data")
</template>
<script lang="coffee">
module.exports =
  props:
    "data":
      type: Array
      default: -> []
</script>

##clusterize-row.vue
<template lang="jade">
.clusterize-row(v-show="data != null")
  slot
</template>
<script lang="coffee">
module.exports =
  props:
    "data":
      default: null
</script>

I want it used like this:
// in template
clusterize(data="[{name:'something'}]")
  clusterize-row
    {{name}}

But I can't get the data within 'clusterize-row' or access the right 'clusterize-row' within 'clusterize-cluster'.
(later on I want to make it a small datatable (like ag-grid) where I define the content by columns, so clusterize-row need to be exchangeable, a buggy version of my first try is here)

I uploaded my current state of vue-clusterize


Improve support for "Platforms:" in LID file

This would avoid having to list all the platforms and would allow us to write the registry entry in the registry/generic/.  It's also nice for self-documenting purposes compared to "here's a list of platforms; you decide if it's complete".

I would like to add support for these special platforms keywords:

all -- like it says on the tin. This allows writing the registry entry in registry/generic
macos -- currently would be equivalent to x86_64-darwin
windows -- currently would be equivalent to win32
unix -- currently would be equivalent to x86_64-darwin + x86_64-linux + x86_64-freebsd + x86_64-netbsd + all the 32-bit versions of those unixen.

When this is done, or even partly done, update the documentation here: https://opendylan.org/library-reference/lid.html#standard-lid-keywords

Also, would it make sense to write a registry file for each covered platform if the spec is not Platforms: all?  That is, if it's Platforms: unix, write a registry file to registry/x86_64-linux, registry/x86_64-darwin, etc.
Currently deft only writes a registry file for the current platform, in the platform-specific registry directory.


Feature/replayer

This change finishes the json consumer and replay consumer implementation.
It also finishes the slang-replay standalone app.

https://github.com/shader-slang/slang/issues/4764 is created to resolve the comment in https://github.com/shader-slang/slang/pull/4750#discussion_r1697442288.
I will just go ahead to merge this change for now.


Reduce CTA arrow spacing, bring closer to text

Without theme:

With RH theme:

Design spec

@starryeyez024 Can you add a changelog when you get a chance?

Testing instructions
Be sure to include detailed instructions on how your update can be tested by another developer.

Browser requirements
Your component should work in all of the following environments:

[ ] Latest 2 versions of Edge
[ ] Internet Explorer 11 (should be useable, not pixel perfect)
[ ] Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
[ ] Firefox 60.7.2 (or latest version for Red Hat Enterprise Linux distribution)
[ ] Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
[ ] Latest 2 versions of Safari
[ ] Galaxy S9 Firefox
[ ] iPhone X Safari
[ ] iPad Pro Safari
[ ] Pixel 3 Chrome

Ready-for-merge Checklist
Check off items as they are completed.  Feel free to delete items if they are not applicable to your PR.

[ ] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[ ] Tests have been updated to cover these changes.
[ ] Browser testing passed.
[ ] Documentation (README.md, WHY.md, etc.) updated or added.
[ ] Link to the demo recording:
[ ] Approved by designer.


cli: show output of docker pull

similar to #785 but for docker pull

Hello! My group and I also commented on issue #541. We would also like to work on this issue.


Fix a mismatch of lightbulb highlight group name

Change the texthl argument for the lightbulb sign from LspSagaLightBulbSign to LspSagaLightBub so that it match the default highlight group definition in plugin/lspsaga.vim.

Thanks @chmnchiang , merged temporary in https://github.com/tami5/lspsaga.nvim

@chmnchiang Add to Readme :)


need a basename for printJson.sh

Hi @oksuzian,
You have proposed changes to files in these packages:

CampaignConfig

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for b6f7b43fe7aea8426b1bccf3ccf749d3f67e0df1: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at b6f7b43fe7aea8426b1bccf3ccf749d3f67e0df1.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged b6f7b43fe7aea8426b1bccf3ccf749d3f67e0df1 at 3139985ff2384f40cbd1f2a4e63fd718c8bc3137

build (prof)
:white_check_mark:
Log file. Build time: 08 min 26 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at b6f7b43fe7aea8426b1bccf3ccf749d3f67e0df1 after being merged into the base branch at 3139985ff2384f40cbd1f2a4e63fd718c8bc3137.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


Summary of the presentations, what needs to be done

Before the summer school

[ ] Add installation instructions

Intro to Theano

First session:

[ ] Slides: intro to Theano (1h)
[ ] Notebook: logistic regression, SGD (30 min)

Fuel, Deep MLP

Second session:

[ ] Slides: intro to Fuel (30 min)
[ ] Notebook: Fuel (30 min)
[ ] Notebook: DAE, deep MLP (30 min)

Debugging, ConvNets

Third session:

[ ] Slides/Notebook? Debugging tools in Theano (30 min)
[ ] Notebook: Convolution (30 min)
[ ] Notebook: ConvNets + Fuel (30 min)

Scan, RNN

Fourth session:

[ ] Slides/Notebook? Scan (30 min)
[ ] Notebook: RNN, LSTM (1 h)

On the debugging tools, we have slides and notebook with exercies.


error 403

i get error 403 please help me
votify "https://open.spotify.com/track/18gqCQzqYb0zvurQPlRkpo"
[INFO     11:38:22] Starting Votify
[INFO     11:38:23] (URL 1/1) Checking "https://open.spotify.com/track/18gqCQzqYb0zvurQPlRkpo"
[INFO     11:38:25] (Track 1/1 from URL 1/1) Downloading "Never Gonna Give You Up - 2022 Remaster"
[ERROR    11:38:30] (Track 1/1 from URL 1/1) Failed to download "Never Gonna Give You Up - 2022 Remaster"
Traceback (most recent call last):
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/utils.py", line 8, in check_response
response.raise_for_status()
~~~~~~~~~~~~~~~~~~~~~~~~~^^
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/requests/models.py", line 1024, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://gew4-spclient.spotify.com/playplay/v1/key/b7098e5bb8c2c1639bb44e0c17549e6d9f24bb3f
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/cli.py", line 597, in main
downloader_song.download(
~~~~~~~~~~~~~~~~~~~~~~~~^
track_id=media_id,
^^^^^^^^^^^^^^^^^^
...<4 lines>...
playlist_track=index,
^^^^^^^^^^^^^^^^^^^^^
)
^
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/downloader_song.py", line 137, in download
self._download(*args, **kwargs)
~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/downloader_song.py", line 213, in _download
decryption_key = self.get_decryption_key(stream_info)
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/downloader_audio.py", line 98, in get_decryption_key
decryption_key = self.downloader.get_playplay_decryption_key(
stream_info.file_id
)
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/downloader.py", line 381, in get_playplay_decryption_key
playplay_license_response_bytes = self.spotify_api.get_playplay_license(
file_id,
playplay_license_request.SerializeToString(),
)
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/spotify_api.py", line 287, in get_playplay_license
check_response(response)
~~~~~~~~~~~~~~^^^^^^^^^^
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/utils.py", line 10, in check_response
_raise_response_exception(response)
~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
File "/Users/abolfazl/Documents/melobot/crack/Votify spotify downloader/venv/lib/python3.13/site-packages/votify/utils.py", line 14, in _raise_response_exception
raise Exception(
f"Request failed with status code {response.status_code}: {response.text}"
)
Exception: Request failed with status code 403:
[INFO     11:38:30] Done (1 error(s))

duplicate of #5


Could not get subdomains: invalid status code received: 401

I received my beta API key yesterday , and while testing it i am not able to fetch the subdomains
chaos.exe -d tesla.com -key somekey -silent
Response
[←[1;31mFTL←[0m] Could not get subdomains: invalid status code received: 401

Hey @Droidzzzio,
Can you try with chaos -d uber.com -silent -key KEY_HERE

Tested and still not working

When i got the API key in the mail , there was something written in brackets that the API key will be usable in next 30 min , so may be the key is not usable still. I am just throwing a guess.

Should be fixed now @Droidzzzio.

thank You , the issue is fixed now