Get BAD working!

The last big big task is to get out Bayesian Adaptive Design working. We already have the experiment and trial loop infrastructure etc. We have models doing inference based on the designs and responses. We have classes for Kirby and Frye et al designs.

[x] We need a general Abstract Base Class for adaptive designs in general (both DARC and non-DARC) Call it something like AdaptiveABC. Then we can create a concrete class which inherits from both DARCDesign and AdaptiveABC.
[x] Actually do BAD, by calling optimisation.py. This will involve converting it to work with pandas dataframes.
[ ] Implement the heuristics to cut down the design space. This is a reasonably big task.
[ ] Confirm it's working with some parameter recovery simulation, see #6
[x] Update all the other models given that there was some change in getting BAD to work

Initial classes laid out and the structure basically works. Although this is not doing BAD yet. It does generate all possible designs. But at the moment we just randomly pick from the design space, and we have no implementation of heuristics to cut down the design space. But a good first start.
Next step is to actually do BAD, by calling optimisation.py. This will require sending some model based into back into the design along with the last response. This will also require some updates to the way how we do that, so that we have a consistent method call across all design types.

Ok. Problem found. I was having problems with the matrix algebra when predictive_y was being called from either log_likelihood or design_optimisation as the sizes of the parameters and data are different and it wasn't playing ball.
The problem is that I implemented log_likelihood incorrectly. That needs to iterate over trials, calculating p_chose_delayed, then calculating the product.

Currently, base on some parameter recovery simulations (#6), this is not working as expected. Try to track down the bug. First hunch is in the implementation of the log_likelihood() method.

Actually no, there IS a problem with design optimisation. Running the Frye task, we get very good inferences, so I think the inference code is working just fine. But the choice of design in the BAD is very poor. This is leading to broad posteriors, but that's the fault of the design selection


Search archive by started at between two dates and times

Allow users to search for archived workflows between two dates and times.
See #2321

Potentially this could play nicely with https://github.com/argoproj/argo/issues/2258


Path parameter interpreted as template literal when

I have a query parameter defined called "riskGroup" in my swagger yml like so:
/efficientPortfolio/getPortfolios/{riskGroup}:
      get:
        summary: Efficient Portfolio Parameters
        description: ''
        operationId: getEfficientPortfolios
        produces:
          - application/json
        parameters:
          - name: riskGroup
            in: path
            description: User's risk benchmark to which their request can be evaluated.
           ....
     x-swagger-router-controller: "getEfficientPortfolios"

However, once generated the controller in typescript interprets the path parameter as a template literal like so:

Any ideas on what I am doing incorrectly?

Issue addressed when upgrading the major version of dependency

Issue addressed when upgrading the major version of dependency

Hello @thuoe!
Which version of swagger-typescript-api it was ?

Hello @thuoe!
Which version of swagger-typescript-api it was ?

Hi @js2me it was an Incredibly old version I was using (1.4.1). Since upgrading to the latest version it has resolved this issue. Apologies

Hi @js2me it was an Incredibly old version I was using (1.4.1). Since upgrading to the latest version it has resolved this issue. Apologies


ZURB Template Feedback

We're working on some changes to the technical underpinnings of the ZURB Template (see zurb/foundation-sites-template#28), but we'd also like to collect some feedback on the featureset.
Here's what we're interested in knowing:

Are there any common problems you run into with the existing features? (For example, BrowserSync reloading before HTML templates are done compiling, which is resolved with the PR referenced above.)
Are there any new tasks you'd like to see in the current template? We're pretty happy with the core featureset (HTML, CSS, JS, images), but we're open to suggestions.

Current ideas:

Notifications
Accessibility audit
Linting
Deployment boilerplate (FTP, rsync, scp, etc.)

How about implementing gulp-notify for error notifications through the operating system in addition to the console?

@colin-marshall +1 . I like this a lot because my console is usually buried under like 20 windows (I know its a terrible personal problem).  It's very useful to a lot of people.

I recently added gulp-notify to FoundationPress and I would also be down to do this for the Foundation templates, and/or Foundation itself if @gakimball and the Zurb people want this feature added.

+1 for notify, I notice that if I have a syntax with SCSS browsersync loads the last working css file, so it's a fairly silent fail I have to check manually.
Having template/layout changes show up in browsersync without a page reload fixed is nice. :)
One question on loading a js file manually - I have webshim loaded via bower and can't seem to get it work. The obvious absolute/relative paths directly to it are 404s and if I load it via the gulp build process I'd have to manually include everything separate file the intitial polyfiller.js loads.

The updated template will fix the race conditions that currently exist with BrowserSync—it won't reload until the task is actually done. This is easier in Gulp 4 because you can be more explicit about running things in series vs. parallel.
Would that reduce the need for a notification? All of the compile steps are very fast, and if BrowserSync is working as intended then you don't necessarily need a reminder that it worked.

Notifications on success would be obnoxious, but I wouldn't mind them on fail. If Gulp 4 also solves Browsersync using the last good CSS file when SCSS has an error on compile I could live without them now.

Off-topic (and out of scope for you!), but the discussion Wednesday made me want something in between panini and a traditional SSG. Laying some more sugar on top of a layout engine (archive generation, jekyll collections, tags) but not worrying about everything else and having gulp plugins handle it makes sense. The benefits of having a tightly coupled system are fairly slim (aside from a few interesting plugins like https://github.com/jaredly/hexo-admin) vs not splitting communities.
Let me know how the bower build stuff ends up working out. :)

@erutan I'm definitely interested in exploring options beyond Panini. We looked at gulp-ssg, but the issue is it's not quite integrated enough. It still relies on a lot of extra plugins to get all the functionality. Tools like Hexo, Metalsmith, etc. are valuable because they can more smartly manage data, page collections, and so on. That's the extra value that Panini doesn't provide right now.
Super long term we've looked at reworking our CLI to be backed by Yeoman, so each template is a Yeoman generator. If we do that, we could make configurable build processes. For example, swap out Panini for Metalsmith, add some extra CSS packages out of the box, add a deploy to live server task, etc.

@gakimball yeah, gulp-ssg wasn't enough to make me bite, but it's a nice proof of concept for using things like gulp-wrap, gulp-template. I'm digging the idea of just having a gulp chain for a SSG, but it's missing that "more than a layout engine, less than a full on SSG" markdown/HTML set of pipes. Or ideally different groupings of pipes - not sure whether it'd be paring down current SSGs or just building out from templating engines with that in mind.
  Something like hexo-admin (it can even deploy from UI!) with https://github.com/yabwe/medium-editor instead of the typical dual pane markdown editing panel seems like it could work. Most brochure sites don't need to be calling a live DB, and hosting is such a pain. I could also see something like a large organization using something like Drupal 8 or Contentful as a data repository and having the context pulled into static sites for things like labs, teams, that tend to be pretty straightforward ala http://carrot.is/coding/static_cms
Does Slush allow for prompts ala Yeoman? It seems like it'd be fairly simple to do some component swaps.

@erutan One fundamental issue with the stream concept in this context is that every file is processed in isolation. I'm sure there are ways around this, but by default that's how it works. So, for example, we had someone ask if Panini can generate an array of pages to use in Handlebars. And it can't, because it doesn't know how many pages it's processing.
I realized this weekend that Assemble hit 0.6, which adds Gulp compatibility and a more robust API. In the Foundation 5 days, we used Assemble to build our documentation, and used it in our client template. Once they update their website to match the new library, I could see switching back to Assemble.

@gakimball Having tightly coupled modules along the lines of current SSGs that deal with content + templating (md/html/jade/etc) but keeping image / style / scripts / preview etc in gulp what I was thinking of, but the pipe limit is a good point. There are ways around it (chain very specific plugins) but that gets unwieldy. Hmmm.
Switching to assemble.io if it solves your needs makes sense - less maintenance, more functionality.

@erutan suggested incorporating gulp-accessibility, or a similar plugin, to allow for built-in accessibility audits in our template. Very cool idea.

I can do a pull request if you want it. It's REALLY verbose, so I'm not including it in any common task, right now I have it set to fire on npm start a11y.
The alternatives out there are:
https://github.com/mpezzi/gulp-a11y
https://github.com/dreamzmaster/gulp-pa11y

@erutan that sounds extremely useful.
:+1:

@erutan Still need to discuss it internally with the team. We probably won't release the revised template until 6.2 is out at the end of February, so we still have a decent amount of time to collect feedback and decide on what we want to add.

I didn't do a comprehensive audit of which tool to use, so having other people look at that before committing sounds like a solid idea.
It looks like scss-lint has a ruby dependency, so that's not ideal from a cleanliness standpoint. I have mixed feelings dealing with js hint/lint.

sass-lint (uses node.js) will be replacing scss-lint in this repo in the near future. Sounds like an ideal candidate for the ZURB template too when it's a little more mature.

I recently changed computer, and reinstalled nodejs latest 5.5 version. Got a bunch of warning and ERROR,  and bower not installed.
Most important I guess is to npm -g node-gyp marked bower and mark Python v2.7 as a pre-requisite as the version 3.x is not supported...

We have some longer-term ideas around how to extend these templates, and that involves Yeoman generators. There are good ideas here, but some of them aren't compelling enough to be a default. They could be optional, however.
After 6.2 ships, we're going to look at making a Yeoman generator out of the ZURB Template. Same structure and everything, but then we can make extras like linting, a11y audits, notifications, deploy processes, extra compilers, etc. all optional.
At that point the CLI also becomes a lightweight wrapper around Yeoman, which will make it much easier to maintain.

@gakimball I also want you fix the bug in magellan nav as well .

@hassanbinsafdar It would help to make sure there is an issue open that explains to the issue you are referring to. And we can continue the conversation there - thanks!

@rafibomb https://github.com/zurb/foundation-sites/issues/8062#issuecomment-182963541 here issue is discussed and i found out soulution too so you guys need to fix that too :)
thanks

Going to close this as 6.2 is shipping this week. Thanks again for all the feedback! :)
To reiterate what I said above, a lot of the features suggested here sound good as options in a Yeoman generator. That's the direction we want to take all of the framework templates—have good defaults, with optional Gulp plugins/tasks that can be added when you're first setting one up.

One thing that'd be handy, having an option to build and push the contents of /dist to gh-pages branch.
The one thing I really miss about using Jekyll as my prototyping tool was that I'd get a preview built automatically a few seconds after pushing up code. Right now I just built from master, switch over to gh-pages where I have a .gitignore ignoring a bunch of dirs, then copy over the contents of /dist/ into the root. It's not horrible, but not ideal.

@erutan That could be interesting! What's the series of Git commands you run to do that? You could make it into a Gulp task using Node's exec() function.

@gakimball I, uh, use Tower (GUI client) for most of my git these days.
/hides
Good idea though. :)


performance issue/slow

After adding the Jodit element, typing in all other fields in the page are slow and sometime even the typed text is displayed with delay of 10 seconds. Strangly, this happens when I add some config parameters.

Show me your code please

(https://user-images.githubusercontent.com/6502834/189671860-69b7ba85-4318-4f5f-ac41-3e2ffd75380c.png)


Revert AssemblyInfo.cs removal; file needed for Azure OpenAI support

This [InternalsVisibleTo] attribute likely appeared to be a development-time accidental artifact. It's the interim mechanism that the Azure.AI.OpenAI package will use for Azure OpenAI Service support, though, so we'll need it back.

After we enabled signing, we removed this because we need a full strong name key for this to work. So, if this is needed, we also need to include the key. However, I'd like to understand what our plan is for this.


BUG:we caaan add realtime subscribe

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
i click on subscribe then nothing to happen
we can add : user type her email and click on subscribe then go to notification that email id

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@PatilHarshh ,please assign me


FindNextPrinterChangeNotification SafePRINTER_NOTIFY_INFO is invalid

Describe the bug and how to reproduce
Having an issue with the 'FindNextPrinterChangeNotification' method within the "WinSpool". The 4th parameter, out SafePRINTER_NOTIFY_INFO pointer to a buffer that receives the printer information for the change notification, isn't set as it should be and keeps being invalid.
Below is the result of FindNextPrinterChangeNotification(hChange, out chg, default, out ppi) on the ppi:

Name
Value
Type

ppi
0x0000000000000000
Vanara.PInvoke.WinSpool.SafePRINTER_NOTIFY_INFO

IsClosed
false
bool

IsInvalid
true
bool

IsNull
true
bool

handle
0x0000000000000000
System.IntPtr

What code is involved
This behavior is happening within the test StartWriteEndDocPagePrinterTest. Below is my own implementation:
SafeHPRINTER hprnt;
var printerName = "MyPrinterName";
if (!OpenPrinter(printerName, out hprnt))
{
    Console.WriteLine($"Failed to open printer {printerName}. Error: {GetLastError()}");
    return;
}

using var hChange = FindFirstPrinterChangeNotification(hprnt, PRINTER_CHANGE.PRINTER_CHANGE_ALL, PRINTER_NOTIFY_CATEGORY.PRINTER_NOTIFY_CATEGORY_2D);
while (true)
{
    var kernalStatus = Kernel32.WaitForSingleObject(hChange, 200);
    if (kernalStatus == Kernel32.WAIT_STATUS.WAIT_OBJECT_0)
    {
        SafePRINTER_NOTIFY_INFO ppi;
        PRINTER_CHANGE chg;
        bool foundPrinterNotification = FindNextPrinterChangeNotification(hChange, out chg, default, out ppi);
        if (foundPrinterNotification && !ppi.IsInvalid)
        {
            PRINTER_NOTIFY_INFO pi = ppi;
            Console.WriteLine($"{chg}: {string.Join(",", pi.aData?.Select(d => d.Field))}");
        }
    }
}

When running this program, I just request the printer to print a test page. Everything goes as expected until it hits the FindNextPrinterChangeNotification method and checks if the PPI is valid, which it is not.
Expected behavior
I expected that within the 'FindNextPrinterChangeNotification' the ppi 'SafePRINTER_NOTIFY_INFO ' should receive the printer information for the change notification.

See updated test in commit referenced above. You have to provide a PRINTER_NOTIFY_OPTIONS parameter value with the data you wish to see.


[PowerToys Run] steam app Binding of Isaac Rebirth does not launch correctly

Microsoft PowerToys version
0.35.0
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

Open program that crashes via PowerToys Run (for me Binding of Isaac Rebirth's exe "isaac-ng.exe" crashes reliably)
Program crashes

✔ Expected Behavior
Program runs as expected without issues.
❌ Actual Behavior
Program crashes, and in some cases has other issues
Other Software
I've had this happen to me with some programs, but can only think of one off the top of my head.
The game "The Binding of Isaac: Rebirth" crashes without fail when run via PowerToys Run.
I also attempted to do some additional testing, and the game runs fine when run directly, as well as via shortcut and even from Command Prompt. PowerToys is the only thing that seems to cause this issue.

How was the game installed via steam?

Installed via steam, I also tried with and without some of the DLCs. Even tried removing the steam DRM to see if that was the issue (some steam games don't allow you to run via exe), but it didn't work.

could be a commandline arg issue ... I literally just uninstalled Among Us / steam on this computer too

its quite possible, especially considering the fact that running it via cmd with no args does not cause any issues

slight update, when I attempted running it with the latest PowerToys release, I now got this error

(it also complains about the game data folder in general)
My guess is that it checks the folders where the exe is being run from, and for some reason running via PowerToys makes it assume that the PowerToys folder is where the exe is?

is this issue still relevant with version 0.79? /needinfo

Wow, I totally forgot I even opened this issue, it seems that at some point this bug was fixed, closing!


Model "Lead" - Multifurcating key support [Was "Lug"]

As a curator I want to developer, or record historical, bifurcating keys.
They bifurcating engine in mx, and the corresponding interface was simple, and worked well.
I have a branch where model work is begun.

Leads can also be composed of Observations.  ObservationLead is required.

Background:

We're porting the "Clave" model that 'mx' used into TaxonWorks:

https://github.com/mx3/mx/blob/master/app/models/clave.rb
https://github.com/mx3/mx/tree/master/app/views/claves (views, for reference only, will be re-done in VueJS, maybe as a stand-alone plugin sensu Distinguish)

@kleintom has graciously agreed to take a stab at scaffolding this up to (at this point) the corresponding key building task that will be done in Vue (@kleintom you're welcome to stab at that too, but just getting the basis is the initial scope here. @jlpereira is the UI guru, so he'll be involved at some level).

Intermediate steps

We use Rails scaffold to start a process like this.  While we work on Lead we want to update our documentation to reflect the current best practices.  This means:

[ ] inspecting https://github.com/SpeciesFileGroup/taxonworks_doc/blob/archive/development/SCAFFOLDING.md

[ ] Ensure this points to the most recent Rails docs on scaffolding

[ ] updating, and moving it to docs.taxonworks.org (under https://docs.taxonworks.org/develop/coding.html)

Updating the scaffolding should tune the parameters to exclude generating certain files that we do not use (e.g. certain stylesheets, js, and specs files) and perhaps review for adding ones we might be overlooking

Model
We largely follow mx, with some updates.
Lead
  id
  parent_id (references self)
  otu:references
  text:text (was `couplet_text`)
  origin_label:string (was manual_id, i.e. if the couplet was given a # in print, that number, if provided assume there is an origin citation)
  description:text (only used on the starting couplet, to describe the overal key; was head_annotation in mx)

  redirect_id (references self IIRC)

  link_out:text (URL)
  link_out_text:string (URL string)

  position:integer (uses acts_as_list)

  is_public:boolean

  project:references
  TW housekeeping

We are removing from the mx model:

pub_annotation for Notes that are public
edit_annotation for Notes that are not
l,r for LeadHierarchy
ref_id for Citations

We need a second index (I can't remember if we can scaffold this with the gem tool) that references closure_tree (has_closure_tree).
LeadHierarchy
ancestor_id
descendant_id
generations


[HYP-75] [Clinic] Add Detail dialog for each course

Jira: HYP-75

Add detail course dialog

– [x] Artifacts Added

Please fix the button alignment when user has no course.

LGTM

you can use "npm run build" to check it build fail. @iionayne25


Bug: HDG command accuracy depends on simulation time step size

The result of the HDG command seems to depend on the size of the timestep used for the simulation. For small time steps the final heading is equal (or at least close) to the value commanded by the user, however for larger time steps the final heading can differ from the commanded heading by multiple degrees.
The following scenario results in a final heading of 090 degrees (rounded to the nearest integer).
0:00:00.00>PAN 0.0,0.0
0:00:00.00>DT 0.05
0:00:00.00>CRE AC001 B744,0,0,360,36000,280
0:00:00.00>HDG AC001 090

The next scenario results in a final heading of 088 degrees (rounded to the nearest integer).
0:00:00.00>PAN 0.0,0.0
0:00:00.00>DT 0.5
0:00:00.00>CRE AC001 B744,0,0,360,36000,280
0:00:00.00>HDG AC001 090

And this scenario results in a final heading of 087 degrees (rounded to the nearest integer).
0:00:00.00>PAN 0.0,0.0
0:00:00.00>DT 1.0
0:00:00.00>CRE AC001 B744,0,0,360,36000,280
0:00:00.00>HDG AC001 090

Thanks! fixed in 18f8346.


k2cli tool command should use correct version of tools

there are now going to be several versions of kubecl and helm installed onto the k2-tools image.  when attempting to execute the binary inside the image we should be certain to use the correct version as specified by the config file.

There's many k8s components in hyperkube (kubelet, apiserver, controller-manager, proxy, scheduler) - arguably we could have a different kubelet version than apiserver.  Should we pin this out intended apiserver version?

should be pinned to the version specified in the master config block.  this is a bit unspecific but we don't have the control plane components split out so its the best we can do.

I have been informed that GKE lacks master entirely.  David has suggested that we should take the latest version specified anywhere.  ???? @coffeepac

To clarify, I am suggesting the latest version specified by a nodePool within the same cluster.

Ok.  So we should use whatever the version of Kubernetes the API server is running.  In our system the only way to know that is by looking for  nodePools.name == master.  We need to codify the special names in our system...especially master.   That is the correct way to be forward looking..in case we have different versions running on each node (I hope not).
However, I agree, perhaps it is adequate to take the first  deployment.clusters[0].nodePools[*].kubeConfig value that you can find and get the version from there.

I am not convinced that we need a special name for nodes which have the role of master.

nodePools.name == master does not exist for GKE, so the nodePool name is not sufficient.

The AWS provider role does not assume a special nodePool named master (when it matters we check if nodePool.apiServerConfig is defined). This was a conscious design choice.

It is possible to have two different master nodes running different versions of k8s (c.f. rolling upgrades, testing, etc.)

If it turns out that the design choice was wrong, I'd rather invent a new property (say nodeType) and continue allowing names to be chosen by the user.

ok, I like the nodeType idea... it is implicit in the system right now.

GKE has a master node but it is provided by GKE and we have no control over it.
nodeType is most definitely a different issue.  I have been assuming that the master nodePool would always exist because most people would take a generated config and alter it.  A sloppy assumption but it seemed sufficient for now.
Taking the first nodePool with a kubeConfig is probably sufficient but it would be preferable to interrogate the whole list of nodePools and find the kubeConfig with the most recent version.

Will take the highest present.

Blocked on https://github.com/samsung-cnct/k2/issues/498.

The implementation work on samsung-cnct/k2#498 (https://github.com/samsung-cnct/k2/pull/526) diverged substantially from initial plan.  samsung-cnct/k2#498 is now an unrelated feature request.

To Reaffirm for me as I get on this:  k2cli tool kubectl will use the highest major.minor version it can find in the nodepools to select which version of kubectl to use.

Yes. I believe we should always use the latest kubectl specified.
Medium term we need more CI testing in this area.
There are a few more bread crumbs here.

need to fix comupted_kubectl.sh as it only works if called without an absolute path.  If you have a path on the call to the script the call to mktemp uses the entire application call so we need to either a: strip directories off of $0 before using it in mktemp or don't use $0 in the shell script.

Hate to ask...but the actual implementation of this feels really convoluted... i.e.:  bash script -> bash script -> ansible -> write file -> pull in file -> return in pipe.  I expect there are some reasons for this, would like them to be more obvious in the script/code.
Other observations.. the 2nd bash script assumes arg $1 will be supplied, but doesn't do any validation of that.

I won't disagree.  The issue its hitting is not having config validation exist in more than one codebase and making the version information available in a general way (ie:  output to a file).  I believe the best long-term solution is moving config parsing to an external go lib that can be imported to either k2 or k2cli.  No issue for that though, I can file one if that will cover your first observation.
I can add some validation for $1 as part of this one.

Ok.

config validation issue:  https://github.com/samsung-cnct/issues/issues/26

update PR to check that the variable is passed in and points to a file.  should be good?


create /var/run/redis after redis user

Can't create it before because the owner is the redis user.

Whoops... Thought I checked all of those. Thanks @RyanPineo.


Upgrade foreground-child, remove unnecessary arg munging

Foreground-child is now using cross-spawn-async in order to be able to
properly spawn shebangs and .cmd/.exe files.  This avoids the extra
cmd.exe that win-spawn throws into the mix, since that causes problems
when there is no PATH environ and a binary is being called explicitly
via a full path name.
Cross-spawn-async is a lot more code than win-spawn, but the approach it
takes is more surgical and well-tested.
Because of this, it's no longer necessary to unshift the node/io.js
binary onto the argument list when spawning a shebanged javascript file.
So, doing 'nyc mocha ...' on Windows will now work, and it's no longer
necessary to do 'nyc ./node_modules/mocha/bin/mocha.js ...' or any other
extra manual path munging.

LGTM!

It looks like this might only be part of the solution.
I'm getting a NYC_REQUIRE=undefined in the environment on Windows.  This diff seems to fix it, but idk why:
diff --git a/bin/nyc.js b/bin/nyc.js
index b63b5ee..0132a79 100755
--- a/bin/nyc.js
+++ b/bin/nyc.js
@@ -122,11 +122,14 @@ if (process.env.NYC_CWD) {
     if (argv.all) nyc.addAllFiles()
     if (!Array.isArray(argv.require)) argv.require = [argv.require]

-    sw([__filename], {
+    var env = {
       NYC_CWD: process.cwd(),
-      NYC_REQUIRE: argv.require.join(','),
       NYC_CACHE: argv.cache ? 'enable' : 'disable'
-    })
+    }
+    if (argv.require.length) {
+      env.NYC_REQUIRE = argv.require.join(',')
+    }
+    sw([__filename], env)

     foreground(nyc.mungeArgs(argv), function (done) {
       if (!argv.silent) report(argv)

But now, I get this:
Error: Cannot find module 'c:\tap\--silent'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:289:25)
    at Function.Module.runMain (module.js:467:10)
    at Function.runMain (c:\tap\node_modules\nyc\node_modules\spawn-wrap\index.js:248:10)
    at Object.<anonymous> (c:\tap\node_modules\nyc\bin\nyc.js:19:6)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)

so it looks like it's not properly chopping off the arg when doing nyc --silent <some command>?

@isaacs
Your branch works for me (without your diff or any other modification) on Windows 7, with Node 4.2.1 and 5.3.0.
// foo.js
var tap = require('tap');

tap.is(1, 1);

command:
node .\bin\nyc.js --silent tap foo.js

also works when npm linked:
nyc --silent tap foo.js

(only tested that on Node 0.4.2).

Your branch also seems to work just fine when I npm link into the node-tap test suite (I get the same errors using<EMAIL_ADDRESS>and your branch).
One thing to note: you want to do npm run clean in the nyc directory before npm linking. Otherwise you end up using files instrumented for nyc self coverage, which can cause problems.

That was totally a legit bug in spawn-wrap, fixed on 1.1.1 (tagged next in npm for now).

On Windows 7 I now get this slightly different error:

Switching to just nyc --cache mocha --timeout=4000 --check-leaks fixed the yargs test for me.
(I do not have mocha installed globally, so it's not that it's picking up a global install. It is honoring the path environment set by npm).
I do get a single test failure, but it looks like it is related to newline separators, and occurs when I just run node_modules\.bin\mocha.

@jamestalmage cool, confirmed myself having simply mocha rather than the full path does work -- makes me think this will be an easy fix.

Yeah, the --silent thing was definitely spawn-wrap's fault.  That's published on its @next tag.
I'm most of the way done with making tap's tests all pass (and generate coverage!) on Windows.  The hardest one (and where I'm calling it a night) is tap's test/test.js that runs 43 output tests and verifies their output.  Turns out that a LOT of tap's output has a lot of file names in it, so you get errors like this (note the path slashes):
not ok 8 - should match pattern provided
  ---
  found:
    at:
      file: "test\\test\\assert-todo-skip.js"
      line: 7
      column: 5
    source: |
      t.ok(false, 'expected', {todo: 'implement a thing'})
  pattern:
    at:
      column: 5
      file: test/test/assert-todo-skip.js
      line: 7
    source: |
      t.ok(false, 'expected', {todo: 'implement a thing'})

If I live for a thousand centuries, I might never forgive Microsoft for their unfathomable continued use of back slashes.

OK,
With the following hack to nyc/node_modules/spawn_wrap/index.js makes the yargs test suite work
- var spawn = ChildProcess.prototype.spawn
+ var __spawn = ChildProcess.prototype.spawn
+
+  function spawn(options) {
+    console.log('spawn called: ', options.args[3]);
+    options.args[3] = options.args[3].replace('"/bin/sh" ', '')
+    __spawn.apply(this, arguments);
+  }

Somehow a "/bin/sh" is leaking into the arguments. Here is that log statement (before I did the options.args[3].replace line):

If you're seeing tap+nyc fail on Windows, can you try this?
npm i isaacs/node-tap#wsup

That should pull in this nyc PR, plus the windows support I just landed in tap, @next versions of foreground-child and spawn-wrap.

yargs uses mocha, and that fails with this PR. I haven't seen tap fail yet.

I'd argue that the /bin/sh is actually correct behavior, since yargs is explicitly providing a path to a sh script.
c:\yargs>cat node_modules/.bin/_mocha
#!/bin/sh
basedir=`dirname "$0"`

case `uname` in
    *CYGWIN*) basedir=`cygpath -w "$basedir"`;;
esac

if [ -x "$basedir/node" ]; then
  "$basedir/node"  "$basedir/../mocha/bin/_mocha" "$@"
  ret=$?
else
  node  "$basedir/../mocha/bin/_mocha" "$@"
  ret=$?
fi
exit $ret

Then cross-spawn finds that the script does indeed exist, reads the shebang, and tries to execute it as intended.
It's only cmd.exe that uses PATHEXT rather than the explicitly provided script name.  So, when win-spawn put an extra cmd in the process stack, then cmd munged node_modules/.bin/_mocha into ./node_modules/.bin/_mocha.cmd, even though that's not what we provided.
It smells to me like a hacky workaround to specify a test script of ./node_modules/.bin/_mocha rather than simply mocha.  Yargs PR coming shortly.

Also, spawn-wrap 1.1.1 fixes the failing-to-wrap bug that was the cause for doing _mocha instead of mocha in the first place.

@isaacs my testing went quite well (I'm quite happy to stop referencing the absolute path to the bin in yargs). Mind rebasing this and we'll land?

merged in #125

I'd argue that the /bin/sh is actually correct behavior, since yargs is explicitly providing a path to a sh script.

Hmm. I didn't realize that. Still, this seems like a potential point of confusion. The error message was particularly unhelpful ("The system could not find the path specified"). My initial assumption on reading that was that it wasn't finding the nyc or _mocha files.
I'm assuming nyc isn't the place to improve the logged error. Would the right place be spawn-wrap or cross-spawn-async?


Bug just after power on

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
WinGet
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
Just power on the computer and powertoys load automatically then the bug arise. Log is as follow:
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 2023/10/25 11:18:33
Exception:
System.Collections.Generic.KeyNotFoundException: The given key 'C:\Users\****\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Visual Studio Code\Visual Studio Code.lnk' was not present in the dictionary.
at System.Collections.Concurrent.ConcurrentDictionary`2.ThrowKeyNotFoundException(TKey key)
at System.Collections.Concurrent.ConcurrentDictionary`2.get_Item(TKey key)
at Wox.Infrastructure.Image.ImageCache.get_Item(String path)
at Wox.Infrastructure.Image.ImageLoader.LoadAsync(String path, Boolean generateThumbnailsFromFiles, Boolean loadFullImage)
at Wox.Infrastructure.Image.ImageLoader.<>c.<b__16_1>d.MoveNext()
--- End of stack trace from previous location ---
at System.Threading.Tasks.Task.<>c.b__128_1(Object state)
at System.Threading.QueueUserWorkItemCallback.Execute()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()
✔ Expected Behavior
No pop-up window.
❌ Actual Behavior
A pop-up window with bug reported appeared
Other Software
None.

Fixed in 0.72.0
/dup https://github.com/microsoft/PowerToys/issues/25848


noisy counts vs. counts of noise?

Right now, the sample code bins the values, and then applies noise to the counts. An alternative would be to apply noise to the values before binning. Is either clearly better or worse? The approach does influence what "contributions" means: is it the maximum number of values, or the maximum number of bins that an individual can contribute to? Since an individual can't contribute to more bins than they have values, there is a bound, but it might be inefficient.

Good question! There's a tendency where the later in the data pipeline that noise/randomization happens, the better your utility gets. This is why, on the same data, typically local-dp has more noise than shuffle-dp has more noise than central-dp. However, what you get from it is that since you're adding noise earlier in the pipeline, you might be able to apply the noise locally instead before collecting the data, resulting in a lower trust requirement from the user (satisfying local or shuffle dp instead of central).
In local dp, we typically assume an individual could completely change their answer, without bound. If you clamp their scalar response to a range (a, b), then you could claim their sensitivity is b - a, and privatize the clamped scalar with the laplace or gaussian mechanism. Note that in order to avoid introducing bias, you'll want your bounds a and b to be the lower and upper bin edges, which will be pretty wide, resulting in a lot of noise, as is typical in local-dp. While this would satisfy privacy, you might get better utility by using randomized response to pick one of the k bins, biasing toward the bin the user was in.
It might be possible to analyze under central-DP as well, but you have to consider the randomization and aggregation together, which could make the analysis tricky...

Confirmed this morning with Mike that adding noise to the counts makes sense in this context, and that an additional note on the unit of privacy in the generated notebook shouldn't be necessary. (I had been a little confused at the start of this process and ended up with this approach because it was easy, rather than correct... but it is also correct.) Closing.


Duplicate item after copying and replacing

I also have this bug. I copied the same photo from one folder (WhatsApp images) to another (Camera) couple of times. The second time I tried to copy Aves asked me if I want to replace - I said yes. But as a result now Aves shows me 2 thumbnails for the very same photo in "Camera" folder. Both photos details are identical (including path). Rescanning or app restarting doesn't get rid of that duplicate. File manager shows me there's just a single file.
Aves version: 1.6.13-play (Build 79)
Flutter version: 3.3.0-0.5.pre (Channel beta)
Android version: 12 (SDK 31)
Android build: SP1A.210812.016.G970FXXSEGVA9
Device: samsung SM-G970F
Mobile services: ready
System locales: en_GB, ru_RU
Aves locale: system -> en
Installer: com.android.vending

Originally posted by @kha84 in https://github.com/deckerst/aves/issues/107#issuecomment-1248210810

@kha84 This is fixed for the next version. If you want to clean the duplicates, you can select all of them and move them to another album (if it asks to rename/replace/skip, choose skip).

Awesome - that move forth and back did a trick, I got rid of that duplicate. And happy to know this is fixed in the upcoming version. You're the star!


Owner field bug

Private message:
had person 1 set as his and person 2 set as her. It looks like you only appended “her” to the person 2 accounts.  His accounts has no label. How do you handle the naming of “ours” or joint accounts?  Maybe label his and hers and leave the third case without “ours”?
My reply:
I think I have a bug with owner…if it isn’t showing up, set it to another value, and then back to the value you want.

yes, "ours" doesn't get added to an account...it is the default, only accounts that are owned by one person should have the identifier. I believe this is consistent with the bogleheads portfolio review format.


Fix Veldrid texture uploads allocating new command buffers and resources

Closes #5895

GraphicsDevice.UpdateTexture is very inefficient for dynamic textures, since it allocates both a local command buffer and a staging texture for perform the upload.
This PR comes with two changes (skipping the minor rename commit), the first one which changes texture uploads to be performed all on one command buffer (BufferUpdateCommands), and the second one caches staging textures to cut the allocation overhead.
The final numbers for Metal don't seem to be as good as they should be, still investigating that.
Profiling with this test case shows the following:

master:

After using BufferUpdateCommands:

After caching staging textures:

Not totally sure why CopyTexture shows different numbers, but the number is consistent per commit anyway.

m2:

Before
After

windows:

Before
After


feat: add blame status bar item

Description

Add status bar item that displays blame info for

The first selected line
The latest commit if no lines are selected

Refactor to deduplicate common logic between status bar item and text document decorations

Screenshot

Codecov Report

Merging #175 (c57286e) into master (40e874a) will increase coverage by 8.18%.
The diff coverage is 77.50%.

@@            Coverage Diff             @@
##           master     #175      +/-   ##
==========================================
+ Coverage   39.39%   47.58%   +8.18%
==========================================
  Files           4        4
  Lines          99      124      +25
  Branches       23       25       +2
==========================================
+ Hits           39       59      +20
- Misses         60       65       +5

Impacted Files
Coverage Δ

src/extension.ts
0.00% <0.00%> (ø)

src/blame.ts
84.37% <96.87%> (+7.10%)
:arrow_up:

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 40e874a...c57286e. Read the comment docs.

Publishing as it works on 3.25.2 now


Incorrect latest version of Docker Engine v23.x.x

Link to product page on endoflife.date
https://endoflife.date/docker-engine
Details of incorrect and correct details you have found
The latest version of Docker Engine V23 is 23.0.6 as mentioned here. endoflife.date shows it as 23.0.10 which doesn't seem to exist.
What is the source website for the product and for its version information?
https://docs.docker.com/engine/release-notes/23.0

@jamesburriss we kind of using moby to do the docker engine updates, see relationship between moby and docker enginee, https://github.com/moby/moby?tab=readme-ov-file#relationship-with-docker cc @endoflife-date/everyone

Should we switch from tracking https://github.com/moby/moby/tags to https://github.com/docker/cli/tags perhaps?

Looks like a case of missing documentation (on docs.docker.com), but it does have release notes for v23.0:
https://github.com/moby/moby/releases?q=v23.0&expanded=true
The docker-engine.md page is tracking the server version, so only checking the client (cli) seems "wrong"?

This seems like an upstream issue, so I've filed https://github.com/docker/docs/issues/20505
Closing this.


flag --short is no longer defined, remove it from docs

When I run kythe help --short, I get the following:
flag provided but not defined: -short
help [<subcommand>]:
	With an argument, prints detailed information on the use of
	the specified subcommand. With no argument, print a list of
	all commands and a brief description of each.

And the command returns status code 2.
What should happen
We should not try using the --short flag in our docs, it seems it was removed.

/gcbrun

Looks like the command output was out of date also, updated that too.

go/gcbrun


⚠ Bofrost.lu has degraded performance

In ea4b789, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1710 ms

Resolved: Bofrost.lu performance has improved in cab8fbf after 11 minutes.


Function NC_check_nulls crashes with segfault for more than 1 var

ParaView crashed for me while working with netcdf files. I was able to track this down to an error in libnetcdf in this line:
https://github.com/Unidata/netcdf-c/blob/98e63846524db726ad4dc0c7de7210fcd5b98379/libdispatch/dvar.c#L705
stride is a pointer to a list of integers and not a list of pointers to integers. So we need to dereference the pointer first with braces:
(*stride)[i] = 1;

After changing this locally for me, everything works fine with ParaView. :smiley:
I will open a PR for this.

Yes, this is already covered as issue #1265. Sorry for any inconvenience, this was my fault. ;-(
A fix is pending in PR #1266.

Ah, glad that someone else noticed it, too. I was surprised that I seemed to be the first one, but I wasn't able to find your issue or pull request. Thanks for your work. :wink:

Could you possibly use your influence to get various package managers to move to 4.6.3?  Apparently, macports just moved to 4.6.2 on March 9 and now lots of GMT users (who depend on the netcdf library) get various crashes.  Is there any workaround other than getting 4.6.3 or reverting to 4.6.1?

Any specific crash? We might be able to mitigate some of them.

Looking at the code, It appears that if you specify a stride array (as all ones),
it should bypass the error.

Looking at the code, It appears that if you specify a stride array (as all ones),
it should bypass the error.

Yes, here is a typical backtrace:

ERROR: Caught signal number 11 (Segmentation fault) at
0   libnetcdf.13.dylib                  0x000000010d0d22b2 NC_check_nulls + 158
1   ???                                 0x00000002e3071590 0x0 +<PHONE_NUMBER>6
Stack backtrace:
0   libgmt.6.dylib                      0x000000010cb9d824 sig_handler + 580
1   libsystem_platform.dylib            0x00007fff775b3b3d _sigtramp + 29
2   ???                                 0x000000011bed1ccc 0x0 +<PHONE_NUMBER>
3   libnetcdf.13.dylib                  0x000000010d0d39e7 NC_get_varm + 117
4   libnetcdf.13.dylib                  0x000000010d0d3ac4 nc_get_varm_float + 14
5   libgmt.6.dylib                      0x000000010cc35b81 gmtnc_io_nc_grid + 641
6   libgmt.6.dylib                      0x000000010cc35495 gmt_nc_read_grd + 1893
7   libgmt.6.dylib                      0x000000010cc00c97 gmtlib_read_grd + 199
8   libgmt.6.dylib                      0x000000010cbc678b api_import_data + 9019
9   libgmt.6.dylib                      0x000000010cbb2fab api_get_data + 315
10  libgmt.6.dylib                      0x000000010cbaac47 GMT_Read_Data + 4007
11  libgmt.6.dylib                      0x000000010cefecc8 GMT_grdvector + 5880
12  libgmt.6.dylib                      0x000000010cbbb564 GMT_Call_Module + 628
13  gmt                                 0x000000010cb8fede main + 1070
14  libdyld.dylib                       0x00007fff773c8ed9 start + 1

presumably NC_get_varm ens up calling NC_check_nulls which sets of the bug as described above.  If I downgrade to 4.6.1 the problem goes away.

I can see we are calling nc_put_varm_float with a stride of NULL. So are you saying we should pass a size_t stride[5] = {1, 1, 1, 1, 1} instead?

pass a size_t stride[5] = {1, 1, 1, 1, 1} instead?
Yes, you might try that since that error apparently is only
invoked when stride is null.

Also, I am curious. Do you actually need to use put_varm instead of put_vars?

Not 100% sure (the guy who did that part is in Germany) but think so.  At any rate, passing the ones avoids the crash so I will go with that.  Thanks!

BTW, you only need a single vector of ones of length NC_MAX_VAR_DIMS
(see netcdf.h) and you can use it everywhere for the stride vector.

Thanks, this problem is happening when reading 2-d slices from grids and they are of max 5 dimensions (same as size and count arrays) so I can get away with only 5.

Macports updated to netcdf 4.6.3 on March 23 from ticket 58212.  This resolves the original issue with NC_check_nulls.  I think you can close this issue.
It would be helpful for users needing Macports fixes and updates to start their own ticket, or to add a comment to an existing ticket.  The Macports organization tends to be slow to respond, and a show of interest helps.


Selectors autocomplete feature in CSS files

Issue Type: Feature Request
I'd like to autocomplete selectors while writing in CSS files. I don't know if there's an extension with a similar feature or not.
I've tried the 'IntelliSense for CSS class names in HTML'  and it makes the opposite as it caches the used selectors in the CSS file to be written in the HTML file, but I want to cache the used selectors in the HTML files and use them in CSS files.
VS Code version: Code 1.24.1 (24f62626b222e9a8313213fb64b10d741a326288, 2018-06-13T17:52:25.282Z)
OS version: Windows_NT ia32 6.1.7601

duplicate of #28442


[WIP] Add editable to dialogue field model

Checks dialogue field's visibility and read_only status before running validations.

@miq-bot add_label wip
@miq-bot add_label refactoring
@miq-bot assign @gmcculloug

@d-m-u The idea here was to deal with validate_field_data methods that are checking the fields when the dialog is submitted.  The thought is if the field is not visible or read-only the user should not be warned about validation errors since they cannot correct those fields.


Fix kfp1.4.0

Error Received
kfp 1.4.0 requires kfp-pipeline-spec, which is not installed.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


[Feature] Support "GET" operations for the knowledge base

Is this your first time submitting a feature request?

[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing functionality

Describe the feature
The knowledge base module. supports upsert and delete commands, but I do not see any option to retrieve ("GET"). It would be great if you can support such a feature. Ideally, I should be able to do the retrieval based on document IDs or specified metadata filters.
Describe alternatives you've considered
It seems you can potentially do this in a non-pythonic way using https://docs.pinecone.io/docs/get-record-ids. However, there does not seem to be a native Python way of doing this (via the pinecone or canopy package directly). Moreover, the records need to be manually combined or filtered which is suboptimal.
Who will this benefit?
Everyone who wants CRUD API endpoints to manage their canopy/pinecone indexes
Are you interested in contributing this feature?
Sure, but would appreciate guidance (pair programming maybe?)
Anything else?
No response

I like the idea, let me loop in internally and get back to you about this feature request!

+1 it would be really useful to be able to see what's in the knowledgebase for debugging. I'm dealing with an issue where things I don't think are part of the knowledgebase apparently are, and things that I thought would be apparently aren't.


Dockerising the bot

PROBLEMS
When bot needs to be tested or run by the user, they need:

Python installed on their system
To activate virtual environments
To install dependencies through pip

and then they can run the bot.
SOLUTION
Docker can be used to containerize the bot inside a Linux image.
This will also speed up development time because the environment of the project remains the same, irrespective of machine.
All the suggested changes have already been done and mentioned in the fork. Kindly review.

Hi there!
This is within the scope of #38, which I am co-authoring with @kry0sc0pic
You can follow it for notifications.

I will then. Thank you for letting me know!


fetchadc: removed

At Nixcon hackathon during a documentation initiative we found that the function isn't being used anywhere. This PR removes the function from all-packages.nix and pkgs/build-support/fetchadc.
@copumpkin

🎉


add table of contents

Is it possible to add an Easy-to-use and self-expanding table of contents, it is really useful！Example website：https://hugoloveit.com/theme-documentation-basics/

Sure. I can add one. Just need some time.

@GaHoWong I have added TOC in this PR: https://github.com/hossainemruz/toha/pull/33
Here, is what its going to look like: https://deploy-preview-3--toha.netlify.app/posts/writing-posts/markdown-syntax-guide/#code-blocks
Let me know if you have any comment.

Just some quick feedback, some display issues on mobile devices.  I'm using a Samsing Galaxy S10+
[Image]
[Image]
Best,
David

From: Md. Emruz Hossain<EMAIL_ADDRESS>Sent: Sunday, July 12, 2020, 11:26 AM
To: hossainemruz/toha
Cc: Subscribed
Subject: Re: [hossainemruz/toha] add table of contents (#34)
@GaHoWonghttps://github.com/GaHoWong I have added TOC in this PR: #33https://github.com/hossainemruz/toha/pull/33
Here, is what its going to look like: https://deploy-preview-3--toha.netlify.app/posts/writing-posts/markdown-syntax-guide/#code-blocks
Let me know if you have any comment.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/hossainemruz/toha/issues/34#issuecomment-657258145, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACN5SNKTBAQ7CAYGEF63RRTR3H535ANCNFSM4OUJ4EKQ.

@daveops1 The PR is still work in progress. I haven't fixed responsiveness yet.

@GaHoWong I have added TOC in this PR: #33
Here, is what its going to look like: https://deploy-preview-3--toha.netlify.app/posts/writing-posts/markdown-syntax-guide/#code-blocks
Let me know if you have any comment.

Great! But it seems that the area is too large, blocking part of the content, which is worth optimizing


chore/devops

Changes

added github actions:  codeql.yml and tests.yml
added BuilderOption on SetMap on InsertBuilder and UpdateBuilder

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add Postgres query retry mechanism to handle transient errors

Description
There is really no free way to do this. I have added the logic and wired it into searches, gets and deletions. Upserts are in the stores and will be added in a different PR because that will modify autogen and muddy the PR. Effectively, if Postgres is unavailable with a transient error then we should try to be successful in the background while still respecting contexts
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
All postgres tests should pass and will add some more chaos-like tests in a follow up

@erthalion Any of the PG codes you think that we should not retry on?


[T4A4][F10-B3]Lee Wenwei Johnervan

Ready For Review.

@johnervan Some comments added. Please close the PR after reading comments. Good job! :)


Add back original installation text (the one which includes emojis from Carlos's script) as we have already validated it with users

/assign


Specify declarative yaml format

We need to determine whether the manifest format in Swarm2:

is identical to Compose v2 format
implements a subset (most commonly used features) of Compose v2
is a completely breaking change with Compose v2

Related issues to consider:

there are a lot of ServiceConfig options -- which would we support?
@stevvooe's idea of adding an extra containers level to the manifest:

web:
    [...]
    container:
      image: nginx

what team will own this file format and the parser long term
what do users want/expect

Follows up on conversation in PR #136

Considering this done


Question: Is it also hardcoded that some of the UI elements are blurred?

In this screenshot of sm64, the press start button is all pixelated via a setting in project 64, however it does not make every texture pixelated like one of the players in mario kart.

is there any sort of setting like that in your code?

I'm not aware of what setting that is, what is it called in project 64?

Maybe deinterlacing?

It's in Texture enhancement tab, and its called Enhancement. It is a drop-down list.

If you click "None" that gives you the result of pixelated UI elements.


Hi, Cannot get the voltage to increase

Hi Jeff, first off thanks so much for your content, only just been introduced to your channel.
I am new to Linux so please forgive my ignorance however I am used to overclocking on mainstream chips. I have a problem though which you may be able to help with?
I have followed your guide/code and others as well as many videos but I cannot get my Pi to overvolt, I am at the point where I have edited the .txt file and found my max freq with 0.05v increase that you can achieve with the firmware voltage limit in place and after using your firmware hack everything seems as if it should work and that I should be able to overvolt beyond the 50000 limit but I can't.
I will try to explain as best I can as to where I am up to;
I have cloned the repository but had to cd into the cd pi-overvolt to be able to compile without an error, not sure why because && cd pi-overvolt was already in the code, now when I run compile I do not get anything to tell me that it has worked it just blinks and then returns me to pi-overvolt, is this normal? I looked at the code and there is no printout so I guess yes? but if not then maybe its not compiling correctly or I am doing something wrong?
I believe I have successfully created the slowcpu systemd service so I think that is all fine.
Now, assuming it compiled correctly the next step I have taken is to go back into the .txt file, increased the OVD=55K, rebooted and then run;
cd pi-overvolt && sudo ./removelimit && vcgencmd cache_flush
echo 3300000 | sudo tee /sys/devices/system/cpu/cpu*/cpufreq/scaling_max_freq
echo performance | sudo tee /sys/devices/system/cpu/cpu*/cpufreq/scaling_governor\
Then in a separate terminal run;
watch -n 1 ./stats.sh
This is where the problem is! the voltage reports 1V and even if I run a stress test it stays locked to 1V and if I repeat the steps above and increase the OVD to 60K, 70K etc it stays locked to 1V.
Please can you help? what am I doing wrong? is there another way to edit the ovd? is doing it in the .txt wrong?

What is the output of all the other commands, especially cd pi-overvolt && sudo ./removelimit && vcgencmd cache_flush ?

Hi Jeff,
Here is the output;
VC_MEM_IOC_MEM_PHYS_ADDR = (nil)
VC_MEM_IOC_MEM_SIZE =<PHONE_NUMBER>
VC_MEM_IOC_MEM_BASE = 0x553fc00000
vc = 0x7fb0500000
found A6 4a... replacing
3300000
performance

After that, then—can you paste the contents you have in /boot/firmware/config.txt? And you're running Pi OS 12 Bookworm, correct?


make-builds-unique-for-each-run: mark localhost and pr builds as unique builds (current name only option does not make them unique)

Issue: Currently localhost builds are named as fromLocalhost and the PR builds takes name from PRs. However, when I am running tests, if even one of the previous tests is failed, it is marking the whole build as failed. So dont club them all together under same name but make builds unique.

fixed by PR builds will be unique now. #131


🐛 Weird effect when a ComboBox is inside a SizedBox

Describe the bug
Put a ComboBox in a SizedBox, the content is based on the bigger element inside, instead of the real size of the combobox.

Can you provide a reproducible snippet?

          SizedBox(
            width: 200,
            child: ComboBox(
              value: 1,
              items: const [
                ComboBoxItem(
                  value: 1,
                  child: Text('Val 1'),
                ),
                ComboBoxItem(
                  value: 2,
                  child: Text('Val 2'),
                ),
              ],
              onChanged: null,
            ),
          ),

Give this:

I would expect to have Val 1 on the left, and the chevron at the right.
Is it okay with that or you need a full reproductible project ?

same issue

Okay my bad, I wanted to fix that but I saw the parameter isExpanded, when I change it to true, that fix this issue.
I don't know, maybe there is a way to detect automatically if the expand should be applied or not. But at least I can fix the render in my app by using isExpanded: true.

Okay, so since the parameters isExpanded fixed my problem. I think we can close this.


Search Page Stop Facets from Disappearing When Checkbox is Selected

Is your feature request related to a problem? Please describe.
On Search page: https://dev.dockstore.net/search?entryType=workflows&searchMode=files
Currently the side panel disappears and reappears when a checkbox is selected to prevent users from selecting two impossible checkboxes.
We wish to remove this effect.
And with our current situation, we might run into some integration test problems because the table content(the results on the right) is not rendered on time(slowness due to the API), so we should consider some of the solutions below:
Describe the solution you'd like
We can have a loader for the table contents, or grey out the facets to prevent them from being touched
Describe alternatives you've considered
Leave it as it is
Additional context
Some useful resources for design ideas:
https://portal.gdc.cancer.gov/repository (this is what we have in mind)
https://dcc.icgc.org/repositories
https://stackblitz.com/edit/angular-gpjdoz?file=src%2Fapp%2Fapp.component.html
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.12
┆Issue Number: DOCK-1776
┆Sprint: Backlog
┆Issue Type: Story

➤ Melaina Legaspi commented:
Gary Luu did you fix this already?

nope


destroyed is triggered before afterLeave

Vue.js version
2.1.10
Reproduction Link

https://codepen.io/weizhenye/pen/mRwoqE
Steps to reproduce

Open DevTools.
Click Toggle Button.

What is Expected?
Should outputs
beforeDestroy
beforeLeave
leave
afterLeave
destroyed

What is actually happening?
It outputs
beforeLeave
leave
beforeDestroy
destroyed
afterLeave

In Vue v2, beforeDestroy is triggered after leave, and destroyed is triggered before afterLeave
In Vue v1 document, it mentions

Note if there is a leaving transition, the destroyed hook is called after the transition has finished.

And I also write a demo with Vue v1, it works as expected.

It's probably intended but I may be missing something

Yes, this is intended behavior in 2.x.

Could anyone clarify why is this intended behavior and some possible solutions to wait for the transition to complete before destroying listeners in a component?
My particular use case is waiting for a page transition to finish before deregistering some child components from mouse position updates.


fix popup menu offset when using useRootNavigator in PopupMenu

fix issue #144669
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@joodo Could you help with the conflicts? Then we are good to go!


anyone got issue w/ blazor wasm please read this: https://github.com/dotnet/aspnetcore/issues/17754

[Enter feedback here]

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

ID: c07e57dc-526f-9479-4695-de7d8f06eb9a
Version Independent ID: 8226ed72-69b2-3c93-a931-218bd11cf9c8
Content: Get started with ASP.NET Core Blazor
Content Source: aspnetcore/blazor/get-started.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @zhou-xb ... They resolved the issue for the upcoming 3.2 release with https://github.com/dotnet/aspnetcore/pull/18207. We don't document every bug fix. We only usually call out bugs/fixes/workarounds when they affect many users.


Language server can't load project references after upgrade to .NET Core 3.1.400

Describe the bug
When opening a Q# project in Visual Studio Code or Visual Studio, the language server outputs errors and is unable to load project references:

[Info  - 14:24:15] workspace folder: c:\Users\samarsha\Desktop\QsProject
[Error - 14:24:16] MSBuild error in C:\Program Files\dotnet\sdk\3.1.400\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(59,5): Unerwarteter Fehler bei der ProcessFrameworkReferences-Aufgabe.
System.IO.FileNotFoundException: Could not load file or assembly 'NuGet.Frameworks, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. Das System kann die angegebene Datei nicht finden.
File name: 'NuGet.Frameworks, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'
   at System.Linq.Enumerable.WhereEnumerableIterator`1.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Microsoft.NET.Build.Tasks.ProcessFrameworkReferences.ExecuteCore()
   at Microsoft.NET.Build.Tasks.TaskBase.Execute()
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask)

[Error - 14:24:16] Failed to restore project 'c:\Users\samarsha\Desktop\QsProject\QsProject.csproj'.
[Error - 14:24:16] MSBuild error in C:\Program Files\dotnet\sdk\3.1.400\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.FrameworkReferenceResolution.targets(59,5): Unerwarteter Fehler bei der ProcessFrameworkReferences-Aufgabe.
System.IO.FileNotFoundException: Could not load file or assembly 'NuGet.Frameworks, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. Das System kann die angegebene Datei nicht finden.
File name: 'NuGet.Frameworks, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'
   at System.Linq.Enumerable.WhereEnumerableIterator`1.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Microsoft.NET.Build.Tasks.ProcessFrameworkReferences.ExecuteCore()
   at Microsoft.NET.Build.Tasks.TaskBase.Execute()
   at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
   at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask)

[Error - 14:24:16] Failed to resolve assembly references for project 'c:\Users\samarsha\Desktop\QsProject\QsProject.csproj'.
Discovered Q# project 'c:\Users\samarsha\Desktop\QsProject\QsProject.csproj'.
Loading project 'c:\Users\samarsha\Desktop\QsProject\QsProject.csproj'.
Done loading project 'c:\Users\samarsha\Desktop\QsProject\QsProject.csproj'

System information

VS Code extension 0.12.20072031, VS extension 0.12.2007.2031
Windows 10, .NET Core 3.1.400
VS Code 1.47.3, VS 16.7.0

Additional context
Reported first by @iitaiy: https://github.com/microsoft/qsharp-compiler/issues/556#issuecomment-670195256

@bettinaheim, I think you are familiar with something like this that happened before?

I have the same problem after update .NET Core to 3.1.4 "Could not load file or assembly 'NuGet.Frameworks, Version=<IP_ADDRESS>"

Hi @Gambero81, our current plan is to include a fix for this bug in the next release at the end of August. The pull request for that is #566.
Unfortunately, in the meantime, I think the only solution is to downgrade .NET Core to the last 3.1.3xx release (3.1.302).

Hi @samarsha , the issue is not only in qsharp-compiler when you applied the fix.
I use roslyn libraries to analize source code and i need to use MSBuildLocator.RegisterDefaults() so i tried the fix:
VisualStudioInstance vsi = MSBuildLocator.RegisterDefaults();
AssemblyLoadContext.Default.Resolving += (assemblyLoadContext, assemblyName) =>
{
string path = Path.Combine(vsi.MSBuildPath, assemblyName.Name + ".dll");
if (File.Exists(path))
{
return assemblyLoadContext.LoadFromAssemblyPath(path);
}
return null;
};
`
but it not solve for me.. the method LoadFromAssemblyPath throw an exception because the assembly version trying to load does not match the version requested

Hi @Gambero81 ,
This issue was opened to track the error above on the Q# Compiler Language Server during initialization. With the fix that Sarah mentioned above, we're now able to start correctly after an upgrade to the SDK v3.1.400.
Are you encountering the problem above in a project unrelated to qsharp-compiler? If that's the case, then you might be interested in following the discussion on the original issue that motivated this workaround, link below:
https://github.com/microsoft/MSBuildLocator/issues/86

Resolved by pull request #566 .


Fixed line break issue in filter challenge

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue : Closes #8595

Description

Removed the line break and updated text to match image in the original issue post.

:sparkles: :sparkles: LGTM :+1: :sparkles: :sparkles:
:tada: Thanks for contributing @jaydenwindle and congratulations on your first PR submission to FreeCodeCamp! :tada:


Query: SelectExpression Cache hit for InExpression coming from parameter

Currently when we have Enumerable.Contains in SelectExpression, the Enumerable comes in as a parameter and we generate a list of constant for InExpression. This means for each parameter we have to generate different SQL. (No second level query caching).
According to the idea from @divega,
Instead of putting a list of constant which would change every time, we can put a list of parameters in fixed number (let's say 10) and generate 1 SelectExpression. If the Enumerable has 10 elements then we just set parameter values and use SelectExpression from cache. If enumerable has lesser than 10 elements then we can copy the last value in remaining parameters just to fill in the values (it would have no effect on results). In case if the Enumerable has more than 10 elements then we can generate SelectExpression of higher number of params in InExpression or just use it without caching like we do today.
This could give us performance boost especially when SQL is large. Though at the same time plumbing of cache key would be slight complex.

Would there be a practical limit of any of the thresholds? i.e. could we have caching for up to a 1000 item List<Guid> ?

@bmarder - Practical limit is number of parameters you can add in one command. (SqlServer has that limit). At that point using parameter variation is impossible and we have to write values in SQL explicitly. We can have caching upto 1000 items. This still needs some design in terms of what is the initial threshold or what should be step size to generate more SelectExpession. Using 100 parameters when list has only 2 elements is also inefficient.

Could it be possible to switch to a inner join after a threshold? Either as json,xml, or UDT? This would have to be on a per provider bases though.


The Public IP addresses can not display the VM name

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION
CloudStack <IP_ADDRESS>

Network name is empty -> public ip is used by VPC or system vms
VM name -> public IP is not used as Static NAT.

Hi @xuanyuanaosheng
The vm name will be displayed only if the public ip address is having static nat
https://cloudstack.apache.org/api/apidocs-4.17/apis/listPublicIpAddresses.html

@weizhouapache @kiranchavala
Yes, as you said. may be this sould be a  Enhancement Request.
I mean the VM name can display the  system vms's name or just add a type note or add a hyperlink.

So we can known this IP is used by cloudstack system or user.


Can't pickle local object 'default_module_loader..default_module_loader_mgr' while autotuning in Windows 10

Hi,
I have successfully built TVM (latest 0.8 source from git) on windows 10 x 64. However, while trying to optimize a model (pretrained, insightface arcface r34_v1) through autotune, i encountered
AttributeError: Can't pickle local object 'default_module_loader..default_module_loader_mgr'
Issue seems to be in tvm/autotvm/measure/measure_methods.py: Method default_module_loader(pre_load_function=None) which contains an inner function def default_module_loader_mgr(remote_kwargs, build_result). Can we rewrite default_module_loader so that it doesn't contain an inner function, thus making it picklable.
Complete error stack is as under:-
python ..\tune_relay_x86.py
Extract tasks...
[20:45:17] C:\Jenkins\workspace\mxnet-tag\mxnet\src\nnvm\legacy_json_util.cc:209: Loading symbol saved by previous version v0.12.1. Attempting to upgrade...
[20:45:17] C:\Jenkins\workspace\mxnet-tag\mxnet\src\nnvm\legacy_json_util.cc:217: Symbol successfully upgraded!
Tuning...
[Task  1/17]  Current/Best:    0.00/   0.00 GFLOPS | Progress: (0/800) | 0.00 s
[INFO] Tracker binded to <IP_ADDRESS> 9000    # Print message added by me to confirm if tracker is being binded successfully
[INFO] Server binded to <IP_ADDRESS> 9001     # Print message added by me to confirm if server is being binded successfully
Traceback (most recent call last):
File "..\tune_relay_x86.py", line 142, in
tune_and_evaluate(tuning_option)
File "..\tune_relay_x86.py", line 86, in tune_and_evaluate
tune_kernels(tasks, **tuning_opt)
File "..\tune_relay_x86.py", line 68, in tune_kernels
autotvm.callback.log_to_file(log_filename)])
File "D:\miniconda3\envs\insight-kera\lib\site-packages\tvm\autotvm\tuner\tuner.py", line 133, in tune
results = measure_batch(inputs)
File "D:\miniconda3\envs\insight-kera\lib\site-packages\tvm\autotvm\measure\measure.py", line 266, in measure_batch
results = runner.run(measure_inputs, build_results)
File "D:\miniconda3\envs\insight-kera\lib\site-packages\tvm\autotvm\measure\measure_methods.py", line 314, in run
module_loader,
File "D:\miniconda3\envs\insight-kera\lib\site-packages\tvm\autotvm\measure\local_executor.py", line 156, in submit
process.start()
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\process.py", line 112, in start
self._popen = self._Popen(self)
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\context.py", line 223, in _Popen
return _default_context.get_context().Process._Popen(process_obj)
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\context.py", line 322, in _Popen
return Popen(process_obj)
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\popen_spawn_win32.py", line 89, in init
reduction.dump(process_obj, to_child)
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\reduction.py", line 60, in dump
ForkingPickler(file, protocol).dump(obj)
AttributeError: Can't pickle local object 'default_module_loader..default_module_loader_mgr'
Done.
(insight-kera) D:\code\tvmmodels>Traceback (most recent call last):
File "", line 1, in
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\spawn.py", line 105, in spawn_main
exitcode = _main(fd)
File "D:\miniconda3\envs\insight-kera\lib\multiprocessing\spawn.py", line 115, in _main
self = reduction.pickle.load(from_parent)
EOFError: Ran out of input
complete script is as https://gist.github.com/lunar-walker/48e969934663e1634d852f000ec4d73c
Thanks for help...

Tuning is not currently supported on windows. Sorry.

Thanks for asking the question, you are more than welcomed to continue the topic on https://discuss.tvm.apache.org/. You can however, setup an RPC server on windows and run tuning from linux via cross compilation, please see https://discuss.tvm.apache.org/t/cross-compilation-for-windows/7698 for the an example


Add option for move input to top

Description

Screenshots (if appropriate)
Types of changes

[ ] Chore (tooling change or documentation change)
[ ] Refactor (non-breaking change which maintains existing functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

This feature will be implemented as part of #1166 but under the config name singleModeForMultiSelect. It works on select-one and select-multiple and provides the same behavior.


[Feature request] Make "Empty query string" error less intrusive

When trying to execute a query from two lines below the query itself, the popup "Empty query string" appears.
Good, right.
But i think it would be good to have the option to reduce it to a small message below data grid, or even better, not open at all, and just show/do nothing...

Good point.
I'll show warning in active tab status bar.

Verified


I cannot quit Atom

I've tried to quit Atom but it hasn't been working.
I'm pretty sure that I've tried everything:
Command + Q
and selecting quit from the dropdown menu.
I'm using a Mac.
If anyone has any solutions, or this problem has been fixed, please tell me.

Have you tried :q!? 🤣
Just kidding. It seems like it might be frozen. Are you able to kill it from the task manager?

same here on osx 10.13.6
have to kill it always whit the task manager!

looks like a duplicate of #17672

If anyone has any solutions, or this problem has been fixed, please tell me.

You can use the "Force Quit` entry in the Apple Menu as a workaround.
Like @UziTech said, dupe of #17672

I am having the same problem. I believe this issue is caused by a package you have installed. Luckily for me I only have about a half a dozen different packages installed, so it was easy to find what package(s) was the culprit. You can view your installed packages under the Settings View, found under the menu Packages > Settings View > Open. Once opened, click Packages from the left panel to see the list of package you have installed. From the Settings View, you can disable a package, kill the app, re-open Atom, then try quitting. You have to do it one by one until you found the offending package. I recommend starting with any recently installed packages first. Luckily for me it was the most recent package I installed, so it didn't take long for me to find. In my case, it was the "pigments" package. I don't really need that package anyway, so uninstalled it. Good luck!

Still having this issue.
@dquessenberry No matter what, there is no kind of package that should be able to prevent Atom from quitting.
Atom should be updated such that no matter what packages the user has installed, the Quit function always works!!!!

@dquessenberry No matter what, there is no kind of package that should be able to prevent Atom from quitting.
Atom should be updated such that no matter what packages the user has installed, the Quit function always works!!!!

I agree with you. Just thought I would share what helped me, at least for a short term fix.

I am having the same issue since some weeks... I haven't installed any packaged recently though. Anyway, I do force quit.

+1 on this issue. I always have to force quit.

I'm having the same issue as well :/

same issue! no solutions?

Same here, I have to force quit every time, really annoying.

Same here :(
Can't figure it out

Anyone not even able to force quit?
There are no atom processes in the activity monitor, which is weird. It does show up in the Force Quit Applications window, but force quit attempts don't do anything. In the dock, there is no white circle indicating the app isn't running, but right-clicking on the app icon has the message "application not responding". Finally, attempts to restart fail, with the OS saying it can't restart because Atom can't be closed. I have to physically shut down via the power button.
Atom v1.50.0
MacOS Mojave 10.14.6

I can reproduce this issue on my Mac running 10.14.6. The first version of Atom that has this issue for me is 1.47. Maybe the upgrade to Electron 5.0.12 is relevant?

Same issue here with Atom 1.50.0 under Mac 10.15.6.

I'm also having this issue. Didn't happen prior to a week ago. No new packages installed, upgraded to Atom 1.50.0, but that didn't fix it.
Force quit works, but when I open it next time, it always opens to the same project, but not the one I was using previously.

I was also having this issue. Didn't happen prior to a week ago. No new packages installed, upgraded to Atom 1.50.0, but that didn't fix it. Also just figured out how to get around it, and what it takes to reproduce the issue.
Whenever I opened Atom, it would open to the same project. Let's called that Project A. If I close that window, leaving no Atom windows open, and then open a different project(Project B), then I can't quit. When I force quit, then reopen atom, it opens with Project A again.
Now, if I open Project B while Project A is still open, and then close Project A, suddenly I can quit normally without issue.
Can anyone else confirm if this works for them too?

@HCLarsen Can confirm. Open one project (A), can still quit. However, if I close all windows for Project A and open another project (B), I can no longer quit. Have to use Activity Monitor.
If I still have Project A open when I open Project B, I am still able to quit, even if I close all Project A windows first.

Was still experiencing this issue even with all packages disabled. I got so sick of it that I deleted my ~/Library/Application Support/Atom folder entirely, which seems to have at least temporarily fixed the issue, at the cost of loosing my "Reopen Project" history. I checked, and my Themes, Packages, Config etc seem to be intact, which is consistent with my understanding they're stored in ~/.atom
Not advising this as a fix necessarily, but seems to be working for me at least.

@HCLarsen Can confirm. Tested on Atom 1.51.0, macOS 10.13.6 High Sierra

Confirmed, Atom 1.51.0, macOS 10.14.6 (18G6032) Mojave

I solved the issue by changing the core setting "Restore Previous Windows on Start" to "No".

I have this problem too

@HCLarsen Yes, seems to be the case for me too

Jup, also got this issue on my macbook

This should be a reliable workaround
in your init.coffee add
atom.commands.add '*', 'application:delayed-self-murder', (options) ->
    # first try normal quitting
    atom.commands.selectorBasedListenersByCommandName["application:quit"][0].didDispatch()
    waitTime = 500 #half a second
    # after a certain amount of time, murder atom
    setTimeout ->
        process = require("process")
        childProcess =  require("child_process")
        childProcess.exec("kill -9 #{process.ppid}")
    , waitTime

in your keymap.cson add
'*':
    'cmd-q': 'application:delayed-self-murder'

Reload the window (or restart Atom) and now cmd+q should always work (note: unsaved work/undo buffer probably not guaranteed)
If you don't care about unsaved work, you can change the waitTime = 500 to waitTime = 0

MacOS 10.15.7
Atom 1.52.0
same issue
Atom works well but can killed only by a "Force Quit" from activity monitor.

Same for me
MacOS 10.15.7
Atom 1.52.0 x64
I need to force quit to exit

I have the same issue on MacOS 10.15.7 and Atom 1.53.0 and can confirm @HCLarsen except that it doesn't seem to matter if it's the same project. I usually use Atom from the command line, but I can certainly reproduce the issue by doing the following:

Click on Atom in the dock -> last project opens
Click on the "x" so that Atom remains open w/o any open windows - can still quit by command + q
Go to "File" -> "Reopen last project"
command + q closes the window but the app remains open and can only be force-quit

Same for me
MacOS  10.15.6
Atom 1.53.0 64

Same for me
MacOS 10.15.6
Atom x64 1.53.0

Same for me
MacOS 11.0.1
Atom 1.53.0 x64

Same

I have only core packages installed. The inability to quit has come up randomly over about the past year. I don't remember experiencing it before then. Concurrently, it has started being random about weather it quits when all windows have been closed.

Same problem here, I'm using Atom 1.53.0 x64 on macOS HighSierra 10.13.6. I often need to force quit the application.

Also, when it fails to quit, it can actually prevent shutdown and you can't Force Quit it either.  I have had to physically power off my Mac laptop.  This is not ok.

Also, when it fails to quit, it can actually prevent shutdown and you can't Force Quit it either.  I have had to physically power off my Mac laptop.  This is not ok.

Same problem here. 1.53.0 x64 MacOS, Catalina 10.15.7.
I have to force quit all the time.

Same

I made a PR that fixes this issue.
https://github.com/atom/atom/pull/21843
Anyone having this issue can go to this link and request the core maintainers to merge the PR. This would facilitate their review process.

I made a PR that fixes this issue.
https://github.com/atom/atom/pull/21843
Anyone having this issue can go to this link and request the core maintainers to merge the PR. This would facilitate their review process.

for my case, whenever i "Reopen Project" a project from project history, atom won't quit normally. have to force quit.
if i use "Open" and select a project from file dialog, atom quits normally.
there is a clear project history selection in "Reopen Project". try clearing that does not help. in fact, the project history is not clearing. that seems to show a problem.
read on the internet to start "atom --clear-window-state"
after that, clearing project history works. all previous project entries are gone.
now, when opening projects, the projects are added back to project history.
with the new project history list, using Reopen Project to open a project does not lead to atom won't quit anymore.
no need to force quit atom anymore

for my case, whenever i "Reopen Project" a project from project history, atom won't quit normally. have to force quit.
if i use "Open" and select a project from file dialog, atom quits normally.
there is a clear project history selection in "Reopen Project". try clearing that does not help. in fact, the project history is not clearing. that seems to show a problem.
read on the internet to start "atom --clear-window-state"
after that, clearing project history works. all previous project entries are gone.
now, when opening projects, the projects are added back to project history.
with the new project history list, using Reopen Project to open a project does not lead to atom won't quit anymore.
no need to force quit atom anymore

I'm still having this issue on Big Sur when Atom is open and I do a Restart or Shutdown without first shutting down Atom.

Me too. This is so annoying. No document open in atom. Quit does not work so I always have to force shut down atom. So annoying.

Hi folks. The fix for this is included in Atom Nightly (as of Atom Nightly 1.56.0-nightly1, released on the 8th of January), and should be in Atom Beta (1.56) soon. Then Atom stable 1.56.0 some time after that.
(Usually there is about a month or so between new minor releases on each channel).
You can get Atom Nightly here with the fix: https://atom.io/nightly although it's not as stable as Beta or stable releases, and it's possible it might not work so well for a day or two if you leave automatic updates turned on.
Originally posted by @DeeDeeG in https://github.com/atom/atom/issues/17672#issuecomment-774806049

Sorry to say this but I'm on 1.58 MacOS Catalina up to date, and I have this problem as well.

its fixed in atom nightly

Sorry I meant
Atom    : 1.58.0
Electron: 9.4.4
Chrome  : 83.0.4103.122
Node    : 12.14.1

Atom version : 1.60.0
Mac OS : Monterey 12.2.1
Still could not quit Atom with Cmd+Q

This issue still exists in the latest version when run in the latest version of MacOS. This issue should be reopened.

Because of this problem I switch to VS Code. I never looked back. I don’t know why the dev team is ignoring it this tedious bug in Atom while the VS Code team does not have this problem.

Experiencing this too.


Not sure how to use the local package

In my expressjs server that I'm using with isolate to deploy on GAE I do imports like
import app from "@/app"
with in the tsconfig:
"@/*": "./src/*"
but after compiling, and running isolate, I have this bug when deploying:
2023-12-01 13:17:04 default[20231201t131511]  Error: Cannot find module '@/app'
2023-12-01 13:17:04 default[20231201t131511]  Require stack:
2023-12-01 13:17:04 default[20231201t131511]  - /workspace/dist/server.js
2023-12-01 13:17:04 default[20231201t131511]      at Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
2023-12-01 13:17:04 default[20231201t131511]      at Module._load (node:internal/modules/cjs/loader:922:27)

is there a way for this to work?
here is my tsconfig.json (the lib-gouach-db-schemas is my local monorepo lib that I'm including in the project)
{
  "references": [{ "path": "../lib-gouach-db-schemas/tsconfig.cjs.json" }],
  "compilerOptions": {
    "target": "ES2020",
    "module": "CommonJS",
    "moduleResolution": "Node",
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "strict": true,
    "sourceMap": true,
    "allowJs": true,
    "skipLibCheck": true,
    "lib": ["esnext.asynciterable"],
    "outDir": "dist",
    "rootDir": "src",
    "baseUrl": ".",
    "paths": {
      "@/*": ["./src/*"],
      "@gouach/db-schemas/*": ["../lib-gouach-db-schemas/dist/cjs/*"]
    },
    "pretty": true,
    "composite": false
  },
  "compileOnSave": true,
  "include": ["src", "src/**/*.js", "src/**/*.ts"]
}

and there is actually a dist/app.js which is there and correctly compiled...

@maelp The typescript compiler does not convert your path aliases. I know, it was surprising to me too when I first encountered this.
You will need to run something after it tspath (not recommended) or use something like tsup to build and bundle the code. I recommend using tsup. It's very easy and you can have a look at my mono-ts example to see how to configure it.

Going to close this since it is unrelated to isolate-package, but feel free to continue the conversation.


ENH: Add affine_tolerance flag to MergeSeries

This should help clear up the error found in https://github.com/nipreps/fmriprep/issues/2706

Hello @mgxd, thank you for submitting the Pull Request!

In the file niworkflows/interfaces/tests/test_nibabel.py:

Line 232:1: E302 expected 2 blank lines, found 1

To test for issues locally, pip install flake8 and then run flake8 niworkflows.

Codecov Report

Merging #706 (78e3f5b) into master (46bcbf6) will increase coverage by 0.02%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #706      +/-   ##
==========================================
+ Coverage   49.15%   49.18%   +0.02%
==========================================
  Files          49       49
  Lines        5859     5866       +7
  Branches      834      837       +3
==========================================
+ Hits         2880     2885       +5
- Misses       2871     2872       +1
- Partials      108      109       +1

Flag
Coverage Δ

reportlettests
?

unittests
49.18% <75.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

niworkflows/interfaces/nibabel.py
70.96% <75.00%> (+<0.01%)
:arrow_up:

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 46bcbf6...78e3f5b. Read the comment docs.

@mgxd sorry to do this, but we're going to need this in maint/1.4.x to get it into fmriprep 21.0.x. Or do we just want to tell these users to upgrade to 22.0?

Sounds good - I changed the PR to target 1.4.x

Thanks. It did pass, so I'm going to go ahead and merge.


[📝CUSTOM] Contact Page Responsiveness and Form Validation

Description 📄
The contact page currently has the following issues:

The layout is not fully responsive on mobile devices, making it difficult to use and visually unappealing on smaller screens.
The "Send Message" button can be clicked even when the required fields (Name, Email, and Message) are left empty, resulting in incomplete or invalid submissions.

Expected behavior ✅

The contact page should adjust its layout properly on mobile devices, ensuring a seamless user experience.
The form should validate input fields, requiring the user to fill in all mandatory fields before enabling the "Send Message" button. If any field is left empty, an appropriate error message should be displayed.

Actual behavior 🚫

The contact page layout breaks or does not display correctly on smaller screens like phones.
The form allows users to click "Send Message" without filling in the required fields, leading to invalid submissions.

Possible Solution 💡
1 .Use CSS media queries to optimize the layout for different screen sizes, ensuring proper spacing, alignment, and readability on mobile devices.
2. Implement front-end validation using JavaScript or a form library to check if all required fields are filled before submission. Add error messages to notify users about missing inputs.
Additional context 💬
This issue is critical to ensure a better user experience and prevent invalid form submissions. I would like to work on this issue as part of SWOC. Please assign this to me.

please assign this issue to me under the SWOC label.
I will start contribute this from January 1st.

Can u assign me this issue I will work on it

https://github.com/user-attachments/assets/2911da3b-8f4d-47f0-aa36-7c57168b29d6
I have resolved the mentioned issue, and I’ve recorded a video to showcase the work. I will push the changes after January 1.
Let me know if you need further adjustments!

good to go.

i have created a seperate branch with name contact_page , is this ok ?

okay.