Add the bioimageio resource description to deserialized stardist model

to keep the information stored in the bioimageio rdf.yaml available after loading the model.

Can you explain why this is useful?

Can you explain why this is useful? I would really help if I understood what this is supposed to be used for.

To access information stored in the rdf.yaml in a programatic way.
For the motivating use-case please see https://github.com/HenriquesLab/ZeroCostDL4Mic/pull/181#issuecomment-1072602021. And maybe @esgomezm can expand a bit on which properties she needs in the exact use-case.

To access information stored in the rdf.yaml in a programatic way. For the motivating use-case please see HenriquesLab/ZeroCostDL4Mic#181 (comment). And maybe @esgomezm can expand a bit on which properties she needs in the exact use-case.

Sorry, I really don't get it. Can you (or @esgomezm) please write the steps of a potential use case? E.g.

Import bioimage.io model
Fine tune weights on another dataset
...

Ok, I understand now why the changes proposed here don't make sense.
What do you think about updating import_bioimageio a bit so that it can be called several times and stores the rdf.yaml and other model related data.
This would also mean that import_bioimageio could be called multiple times.
This would be easy to change and I could propose these changes.

Sorry, I really don't get it. Can you (or @esgomezm) please write the steps of a potential use case? E.g.
1. Import bioimage.io model

2. Fine tune weights on another dataset

3. ...

Just very briefly, a use case would be:
1. import_bioimageio
2. access the `training_data` field of the model
3. download that training data for retraining

Hi @uschmidt83 and @constantinpape
Thank you for opening the discussion! The use case proposed by @constantinpape is actually very nice. The use-case with ZeroCostDL4Mic notebooks is as follows:

The user initialises the notebooks and sets up the architecture to be trained (or fine-tuned): the grid size and the number of convolutional layers.
The user specifies a pre-trained model from the BioImage Model Zoo.
The notebook tries to load the pre-trained weights. If the pretrained network does not match the specified parameters, the notebook outputs an error message informing the user about the discrepancy in the model architecture.

In step 3 we could suggest uploading the pre-trained model directly and keep training it, but then we would need to create an extra library or code to check that the architecture specified by the user, match the one of the pre-trained model.
Yet, some of the information contained in the model YAML is not contained in configuration or thresholds. I think (but maybe I'm wrong) giving the chance to access this info is not damaging and it can be helpful for the integration of StarDist in different workflows. Note also that the YAML file may contain a technical description about how to use the training data or some training parameters. Parsing this type of information and directly providing it makes the interaction smoother to building up self-contained workflows, in my opinion.
In any case, if this is a big issue, I can program the notebooks in a different manner.

Thanks for expanding on the use-cases, @esgomezm. To make my proposal from above more concrete, I would adapt the implementation of import_bioimageio as follows:

it downloads all the model data to output_folder, including the rdf.yaml (if it does not exist yet)
if the output folder with the correct files exists already it just loads the model
the deserialized bioimageio model description will be attached to the returned model (as is done here already)
calling import_bioimageio will still create the folder s.t. it can be called with the "normal" stardist model function afterwards.

@uschmidt83 let me know if this makes sense to you. If not we can work around it in zero-cost and other potential use-cases. (But I think the proposed solution would be more elegant, also because it allows calling import_bioimageio multiple times with the same argument)

What do you think about updating import_bioimageio a bit so that it can be called several times and stores the rdf.yaml and other model related data.

Sorry, I don't like the idea that import_bioimageio can be called multiple times and does different things depending on what files exist on disk.

I would adapt the implementation of import_bioimageio as follows:

it downloads all the model data to output_folder, including the rdf.yaml (if it does not exist yet)

This I can get behind, i.e. also saving the rdf.yaml file to the model folder.

if the output folder with the correct files exists already it just loads the model

How do I check whether the folder has the "correct files"? The files can change over time, e.g. when fine-tuning. New weights will then be added to the folder, which may be loaded instead of the weights from the BMZ. This makes it hard to reason in advance what will happen.
I propose the following:
>>> from stardist import import_bioimageio
>>> from pathlib import Path
>>> model = import_bioimageio("10.5281/zenodo.6348084", "./bio-model")
>>> print(Path(model.logdir / "rdf.yaml").exists())
True

Isn't it easy enough for you to then just load the rdf.yaml from disk?

The use-case with ZeroCostDL4Mic notebooks is as follows:

The user initialises the notebooks and sets up the architecture to be trained (or fine-tuned): the grid size and the number of convolutional layers.
The user specifies a pre-trained model from the BioImage Model Zoo.
The notebook tries to load the pre-trained weights. If the pretrained network does not match the specified parameters, the notebook outputs an error message informing the user about the discrepancy in the model architecture.

Wouldn't it be better to have the user select a pre-trained model and then just allow to change parameters that don't affect the architecture of the CNN?

I updated the PR to implement your suggestion @uschmidt83: import_bioimageio now downloads the rdf.yaml and the other relevant files to the output folder, but otherwise the behavior stays the same (i.e. it can still be called only once and I don't attach the bioimageio resource description to the stardist model object).
This solution would be totally fine for our use cases as well, since it's indeed very easy to now load the bioimageio resource description via bio_model = bioimageio.core.load_resource_description(outpat / "rdf.yaml").
Let me know if there are any more questions / issues with this or if we can merge it.

I've changed the code again. Ok like that?

Which comment?

The result of your changes is good, i.e. having a subfolder bioimageio with the rdf.yaml and other files.

I think it's cleaner that way.

I think there is a simpler way of achieving this without using the tmpfile, see my comment in the code.

Which comment?

Which comment?

I forgot to hit the review button, you should see it on line 436 now.

Thanks @uschmidt83. I also tested it locally for our use-case data now and this works as expected.
Do you think you could make a small release soon so that @esgomezm could also use this in https://github.com/HenriquesLab/ZeroCostDL4Mic/pull/181?

Do you think you could make a small release soon so that @esgomezm could also use this in HenriquesLab/ZeroCostDL4Mic#181?

I've just made a new release (0.8.1).


Add job for testing n-inference failures

This PR adds a JET test to ensure that we don't increase the number of JET failures in step!

bors r+


Customization on AWS perspective UI

Replacing the AWS Perspective name with Custom Name and log
Is it possible to replace the AWS Perspective Name from the Cognito login page & From Static Page ? If yes then please suggest the code and step to do the same
Is it possible to add the Custom logo (ex- Company Logo) ? If yes then please suggest the step to do the same

Thanks for opening an issue
You could change the detail on the login page by customising the text in source/frontend/src/App.js
Here is a link to the file
https://github.com/awslabs/aws-perspective/blob/f490977d44571f17089da64e0b6b83c380732da7/source/frontend/src/App.js#L30
You could add a custom logo in a similar manner. It depends where you wish to add the logo.  The Header component could be a potential candidate https://github.com/awslabs/aws-perspective/tree/master/source/frontend/src/components/Header
I hope this helps.

Hi @awsyadav ,
I've not heard back from you, so I assume you have received the help you need.
We don't intend to add further features to customise the Perspective UI at this time.
I intend to close this issue. If you require further help on this topic, feel free to reopen this issue.


HTML5 and CSS: Nest Many Elements within a Single Div Element

The text in this assignment is flawed.
The last two paragraphs read:

Try putting your opening div tag above your "Things cats love" p element and your closing div tag after your closing ol tag so that both of your lists are within one div.

and

Nest your "Things cats love" and "Things cats hate" lists all within a single div element.

The latter of these looks like the real task, while the first one is more like a hint on how to solve this task.

Created pull request for a fix: https://github.com/FreeCodeCamp/FreeCodeCamp/pull/7281

@textPreferred Please create a PR as per Our Contributing Guidelines

@textPreferred  Your PR just rearranges the lines, closing this. Not sure what you are insisting here. Please discuss this in the HelpChat. We can open this after confirmation.


Model Piece

A piece is an object that, in addition to having the static information about a card (essentially its rank and suit), also has dynamic information about its status in a game:

Location: waste, administration, or a position in the tableau.
Scored: whether it has been "flipped" by its direct descendent(s) being moved somewhere else.
Moveable: whether it has target(s) to be moved to.
Available: whether it can be a target for another moveable piece.

TODO: develop this issue to flesh out modeling.

Model the properties of a piece.


linux-58 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
linux-58 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Ethernet port network connection was disabled by itself, re-seated network cable to restart network service.
flutter@flutter-devicelab-linux-58:~$ uptime
08:56:17 up  7:23,  1 user,  load average: 0.05, 0.01, 0.00


go api code example

ı thing is correct place :)

@omerfruk Thanks so much! This wworks!

bro why don't merge this pr ?

@omerfruk Sorry for the delayed response. Can you please write this up for all routes? Currently, you've added only for all-countries.

Done!


Application installation Error

I've build a flutter application with all architecture
with flutter build apk and flutter build apk --**abi.
I have a phone that my application can't be installed
This is Architecte of the phone (Krait 400)

I've tried all version

Application works with another phone.

@gaetan1903
Can you provide flutter doctor -v and the entire error log you get while trying to install on that phone ?
Please provide the log in properly formatted text and not in screenshots.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


go: drop gccgo bootstrap, drop autoPatchelfHook from bootstrap

bootstrap binaries are static since 1.21
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@tie I've added you as co-author as this change would have been included in https://github.com/NixOS/nixpkgs/pull/256505.

@ofborg build pkgsMusl.go


✨ Adding infrastructure alerting for Production Servers

User Story
As a cloud ops engineer
I need to be alerted of any unhealthy status of the production environment
So that we can attend to any issues that may arise in a timely manner
Value / Purpose
To be alerted of the infrastructure
Useful Contacts
Ky, Rich B
Additional Information
No response
Definition of Done

[ ] To be alerted of CPU thresholds
[ ] To be alerted of memory thresholds
[ ] To be alerted of instances of not being online

Metrics are already being obtained, this will require the alerting of said metrics & thresholds.


does it loads image in NCHW or NHWC?

and what is the channel sequence i.e BRG or RGB

Data is loaded in HWC format and it is RGB ordered.


config: avoid prow's "Merging to branch *** is forbidden" on 8.2

avoid prow's "Merging to branch *** is forbidden" on 8.2

/lgtm


Indexing field of DocList type (=subindex)is not yet supported.

I am creating database of the following format, but unable to index/search
User
-- Folder (multiple folders)
-- Document (multiple documents)
-- Sentences ( document split into sentences)
For the datastore, I get the error Indexing field of DocList type (=subindex)is not yet supported. when using HNSWLIB

What alternative datastore can I use that currently support the case, or what alternate strategy I could use to implement the same
Is searching in sub-sub index possible? Is there a direct function or alternate strategy to use as of now?
I want to search for relevant sentences from all of the documents in one folder at once, so how can I search through all of them at once

Hello @munish0838 ,
We are about to release the feature to enable subindex search. I hope this will fix the issue.

This is released in 0.32.0 version


Implement "dsl_ode_action_capture_image_player_add/remove" services

Want to further simplify the process of playing a captured image by allowing a Player to be created and added to a Capture Action directly.
This should support multiple types of players... Render and RTSP to start with.

dsl_ode_action_capture_image_player_add
dsl_ode_action_capture_image_player_remove

this is dependent on #464

Propose API
/**
 * @brief Adds an Image Player, Render or RTSP type, to a named Capture Action.
 * Once added, each captured image's file_path will be added (or queued) with
 * the Image Player to be played according to the Player's settings. The Action can
 * have at most one Image Player.
 * @param[in] name unique name of the Capture Action to update
 * @param[in] player unique name of the Image Player to add
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_ODE_ACTION_RESULT otherwise.
 */
DslReturnType dsl_ode_action_capture_image_player_add(const wchar_t* name,
    const wchar_t* player);

/**
 * @brief Removes an Image Player, Render or RTSP type, from a named Capture Action.
 * @param[in] name unique name of the Capture Action to update
 * @param[in] player unique name of the Image Player to remove
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_ODE_ACTION_RESULT otherwise.
 */
DslReturnType dsl_ode_action_capture_image_player_remove(const wchar_t* name,
    const wchar_t* player);

Implementation has been merged into the v0.14.alpha branch, and the ode_occurrence_object_capture_overlay_image.py example has been updated to use the new service.


Updates app_config.yml.sample for unit tests

The unit tests expect session_domain to be '.localhost.lan' and subdomainless_urls to be false.  Therefore by setting these values in the "test" section of app_config.yml.sample it'll reduce the chances that users who overwrite this in other environments will have failures running the unit tests.
This address Issue #13218

Hey @juanignaciosl, this PR will improve our open source installation test runs. It's just setting the test env to always be non-submdomainless (an assumption tests already make). Without this, tests will start failing for subdomainless installations.
Could you review?
Thank you very much!

Fair enough!


.net 6.0.6 release notes link has preview in the name

Looking at https://github.com/dotnet/core/releases/tag/v6.0.6
The release notes link links to https://github.com/dotnet/core/blob/main/release-notes/6.0/preview/6.0.6.md whereas it should link to https://github.com/dotnet/core/blob/main/release-notes/6.0/6.0.6/6.0.6.md.

Thanks for reporting this issue @grantborthwick. I have fixed the broken link.


Map tile not loading in ios webview

Hi,
Im using react native.
This is my map.html file :

    <link rel="stylesheet"<EMAIL_ADDRESS>/>

and Im loading map.html into webview :
import React, { Component } from 'react';
import {
  WebView
} from 'react-native';

export default class MapScreen extends Component {
  render() {
    return (
      <WebView style={{flex:1}} javaScriptEnabled={true} source={require('./map.html')}/>
    );
  }
}

But the tile not loading in ios simulator, but its fine in android emulator.
This is what I see in ios simulator :

Hi, this issue involves React, WebView and the iOS simulator. The Leaflet team does not work with, or support these tools, so I'm afraid we can't help you with this one.
Given the very simple example, it's pretty apparent that the problem does not lie in Leaflet itself, but rather how it interacts with one of the other components.
We are open to making changes to Leaflet the make it work better with other tools, but unfortunately do not have the resources to work out issues in other frameworks or tools.
If you can provide a failing example using just Leaflet, JavaScript and HTML/CSS, we're of course very open to working with this.

Thank you for reply.
I was because of http requests, and when I disabled the ATS ( App Transport Security ) it works fine and the tiles are loaded. But as you know disabling the ATS is not the solution.
According to the @F1LT3R answer in the issue Streaming Leaflet Tiles over HTTPS / SSL #3186 we can use https for loading tiles and changing the osm to openstreetmap in the tile url ( http://{s}.tile.osm.org/{z}/{x}/{y}.png -> https://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png ).

@arminghm, thanks a lot, it works for me :)

Thanks @arminghm, it worked for me too !


Deploy and replace existing pythonsd.org site

@davidfischer: The proxy site is done, and if we're fine with Heroku free tier hosting is ready to receive requests. You're keeping up the domain registration right?
Heroku deployment hosted here: http://pythonsd.herokuapp.com/

I am the one who has the domain registration.
I haven't been keeping up with this repository whatsoever. Can you explain what this does differently? I also don't know how I would deploy a new version of this app.

I think we should pay for Heroku. They only allow the site to be live for a certain number of hours per day which means it will take a long time to load unless it's visited every hour or so and if we did have something ping it to keep it alive then it would have about 6 hours per day that it would be offline.

On the way this new app functions, it falls back on a proxy view for any patterns it doesn't have (in pythonsd/urls.py). Currently aside from the admin and that proxy view, there's nothing here yet.
What that means for you is that you would continue to use the old repository the way you are now until we move those portions of the site. This is what I meant by a hybrid site before. Ideally the proxy would be happening in something like nginx, I know, but this reduces some of the deployment complexity and makes it more obvious to anyone hacking on it what's going on, I think.
Currently the project is hosted by Heroku under my personal account. I should probably add at least one other collaborator so we have some redundancy, and it appears I can do that. For deployment we can either manually deploy the way I have it now, or have CI deploy on successful build on master. I'm fine with either.

Interesting
"ping it to keep it alive" is that robust and "industrial" enough ?
I have done that before, not for very long! :-)
Note that the Heroku free tier requires that "the instance must sleep 6
hours in a 24 hour period"
Which 6 hours per 24 hours would we choose? do we want to do that ?
Are we expecting to remain remain with Heroku free rate for a long time?
Otherwise, when and if we do scale:  do we want to pay the Heroku rates ?
How important would it be to benefit from the  the automated on-need
scaling that heroku provides when we decide to start paying ?
How important is it to pay nothing while we don't have much traffic?
Digital Ocean costs $5 fixed per month for this spec:  512MBMemory, 1
CoreProcessor, 20GBSSD Disk, 1TBTransfer
Would these requirements be enough for our initial traffic ?
It looks like the first paying tier on Heroku ("hobby") which does not need
to sleep 6 hours, has min cost of $7 per month but estimated monthly cost
of $57
Heroku has ease of deployment. Is that the main reason to go with Heroku ?
We could use Digital Ocean + Dokku which gives us a very similar deployment
process. Here are a few pointers
http://rogerstringer.com/2015/05/13/make-your-own-heroku/
https://dokku-alt.github.io/
https://github.com/mikexstudios/dokku-nginx-alt
Micah, I 'd be happy to set this up and show you. Interested ?
Also I'd be happy to be your backup on Heroku if needed
Thanks
--Alain
On Wed, Dec 9, 2015 at 6:40 PM, Micah Denbraver<EMAIL_ADDRESS>wrote:

On the way this new app functions, it falls back on a proxy view for any
patterns it doesn't have (in pythonsd/urls.py
https://github.com/pythonsd/pythonsd-django/blob/6561679bbd94525c3d8eb2bea0b184e3fdff28e5/pythonsd/urls.py#L11).
Currently aside from the admin and that proxy view, there's nothing here
yet.
What that means for you is that you would continue to use the old
repository the way you are now until we move those portions of the site.
This is what I meant by a hybrid site before. Ideally the proxy would be
happening in something like nginx, I know, but this reduces some of the
deployment complexity and makes it more obvious to anyone hacking on it
what's going on, I think.
Currently the project is hosted by Heroku under my personal account. I
should probably add at least one other collaborator so we have some
redundancy, and it appears I can do that. For deployment we can either
manually deploy the way I have it now, or have CI deploy on successful
build on master. I'm fine with either.
—
Reply to this email directly or view it on GitHub
https://github.com/pythonsd/pythonsd-django/issues/9#issuecomment-163467777
.

--
--Alain
619-884-5450

@alaindomissy / @davidfischer: I have no problems deploying the app wherever. I have some experience with digital ocean with a home-rolled solution using fabric, and I'm sure Dokku wouldn't be too hard to set up. In the grand scheme of things even the 'hobby' tier will probably be fine. I don't think we'll get enough traffic to incur the higher charges.
Let's not get too hung up on the deployment / hosting. I would much rather seen energy put into helping people work on it and developing the app itself.

It seems we are all on board with this. From what I can see, it looks like the dyno has been switched to hobby ($7/mo - no sleeping). I'm going to see if I can switch the DNS today.

Alright, I believe I have this worked out. pythonsd.org and pythonsd.com are supposed to forward to www.pythonsd.org. www.pythonsd.org points to Heroku with a CNAME. I'm pretty sure that pythonsd.org is forwarding to www.pythonsd.org but it hasn't propagated yet as it involved an A record change. I'll check back in an hour.
Heroku apparently doesn't do A records which I thought was weird.

All looks to be working to me... Thanks @davidfischer.
Let me know if any of you want to be added as collaborators so you can deploy new versions of the app.

I checked all the forwarding and it seems to be correct. You have already added me as a collaborator.


Changes in Externally Add Consequence

a) Changes in Externally Add Consequence according to new Recreated Tree.
b) Need To Create Add Externally FCA Feature

Done


setInteractive on Container

Issue with V3.9.0 . setInteractive() on Container disregards its position and dimensions.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

have you called setSize  on container before setInteractive()?

@RollinSafary yes

@kanthi0802, it seems you're right ) Click on container's body doesn't work on whole body
var config = {
    type: Phaser.AUTO,
    width: 800,
    height: 600,
    backgroundColor: '#010101',
    parent: 'phaser-example',
    scene: {
        preload: preload,
        create: create
    }
};

var game = new Phaser.Game(config);

function preload ()
{
    this.load.image('buttonBG', 'assets/sprites/button-bg.png');
    this.load.image('buttonText', 'assets/sprites/button-text.png');
}

function create ()
{
    var bg = this.add.image(0, 0, 'buttonBG');
    var text = this.add.image(0, 0, 'buttonText');

    var bg2 = this.add.image(0, 80, 'buttonBG');
    var text2 = this.add.image(0, 80, 'buttonText');

    var container = this.add.container(400, 200, [ bg, text, bg2, text2 ]);

    container.setInteractive(new Phaser.Geom.Circle(0, 0, 60), Phaser.Geom.Circle.Contains);

    bg2.setInteractive();

    container.on('pointerover', function () {

        bg.setTint(0x44ff44);

    });

    container.on('pointerup', function () {
        bg.setTint(0x8e8e8e)
    })

    container.on('pointerout', function () {

        bg.clearTint();

    });

    bg2.on('pointerover', function () {

        this.setTint(0xff44ff);

    });

    bg2.on('pointerout', function () {

        this.clearTint();

    });

    //  Just to display the hit area, not actually needed to work
    var graphics = this.add.graphics();

    graphics.lineStyle(2, 0x00ffff, 1);

    graphics.strokeCircle(container.x, container.y, container.input.hitArea.radius);
}

@RollinSafary That's exactly what i've mentioned. :) . We'll have to wait till V3.10 then

What exactly is the issue being reported here? because Container hit areas are working as intended in 3.9 and this won't be changing in 3.10. Please post some code demonstrating the problem.

@photonstorm . Here is an example. Just click on the ListView at the bottom of the demo. The clicks does not work on the entire container. They work in the empty area just above Container and also only on certain area of the container. Not the full container.

This is because the hit area shapes are centered on the Container. So in your code, you create a hit area rectangle that is 360 x 640, but the center of that rectangle is the Container x/y, which is why clicks only "work" in the top-left of your list view, because all of your children are positioned as if the Container x/y is the top-left.
To solve it you can do this in your measure function:
this.input.hitArea.x += this.w / 2;
this.input.hitArea.y += this.h / 2;


BUG: Don't modify tensors in place

Tensorflow graphs are inefficient or give incorrect results when tensors are modified in place.  This pull requests creates new tensors (or lists/dictionaries of tensors) in functions that could be used in Tensorflow graph mode.

Also, creating a new function strategyExample().
Also, tensorflow.range(0,-1,1) is not an empty range but an error, so putting in logic to avoid the error.
Also, avoiding use of the non-existent uint32 in dtype=tensorflow.uint32.

@Leengit looks good to me. Will want to hear more about what you observed exactly with in-place modification when we meet.


Update from angular/cli1.0.0-rc4 to angular/cli 1.0.0 does not update angular to version 4

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.0.0
node: 6.9.5
os: linux x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/flex-layout: 2.0.0-rc.1
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0
@angular/compiler-cli: 2.4.10
Repro steps.

Update angular/cli from 1.0.0-rc4 to 1.0.0.
In package.json are old angular versions. I expected angular 4.
Output after run ng -v:
@angular/cli: 1.0.0
node: 6.9.5
os: linux x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/flex-layout: 2.0.0-rc.1
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0
@angular/compiler-cli: 2.4.10
The log given by the failure.

Desired functionality.

Mention any other details that might be useful.

Updating angular-cli version will not update your packege.json, you should create new app with
ng new MyAngularApp

I have updated my angular to version 4.0( description from angular blog page):
npm install @angular/{common,compiler,compiler-cli,core,forms,http,platform-browser,platform-browser-dynamic,platform-server,router,animations}@latest typescript@latest --save
Is it the same as updating with ng new MyProject?

While there is currently no upgrade suggested plan I find it easier to simply create a new project and copy/compare both package.json files, take it consideration that some of the other files may have changed for example the tsconfig changed from the src folder to the project root.

Yeah we have a 'real' update command planned but we're not there yet. There's an upgrade guide you can follow though: https://github.com/angular/angular-cli/wiki/stories


cargo 3ds new and init commands

Since cargo new is a convenient way to start a new project, I thought it might be useful to have a cargo 3ds new as well, to try and make it even easier to get started. This definitely isn't a necessary feature, but it might be nice to have to simplify the setup.
We could probably piggyback off plain cargo new --bin (forwarding arguments), then just modifying main.rs and Cargo.toml to add the dependencies on ctru and do some basic setup of Gfx and a Console to get a hello-world going.
Same all applies for cargo init, I think. Finding the directory would be slightly trickier but probably not too bad.

Might also want to add a rust-toolchain/rust-toolchain.toml file so it uses nightly.


Fix bug with date/datetime comparisons are in different units

Ensure date/datetime comparisons are in the same unit when doing comparisons. Dates are converted using the local timezone (see util methods for overrides)
  File "/home/allen/gcal_sync/gcal_sync/timeline.py", line 119, in __iter__
    heapq.heappush(heap, (event.start.value, event))
TypeError: can't compare datetime.datetime to datetime.date

Codecov Report

Merging #116 (4e4226b) into main (6b1efed) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   96.35%   96.35%
=======================================
  Files          10       10
  Lines         714      714
=======================================
  Hits          688      688
  Misses         26       26

Impacted Files
Coverage Δ

gcal_sync/timeline.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


GitHub Action failed  Render and Publish / build-deploy (push)

@ateucher Can you please help me? I don't know why this failed / how to resolve
https://github.com/Openscapes/quarto-clinic/actions/runs/9261286684/job/25476395285

🤔 This repo / site for a Quarto + GitHub Clinic we're developing and will reuse and expand. I think we'll want R code to execute so we can show folks we can run R code in a quarto file.
So I guess I want rmarkdown installed in the GHA

We can do that, or just render it locally and commit the _freeze directory?
quarto render
# check git pane in RStudio, the `_freeze` directory should show as added/changed
git add _freeze/
git commit -m "Render and add freeze"
git push

ah thanks I'll try the freeze soon

Thank you. My recent PR included _freeze and rendered just fine.


Ownership of nrf51::Peripherals

@therealprof I am having some trouble utilizing 2 of your libraries, nrf51, nrf51-hal.  I am working off a nrf51-dk board and trying to implement some of the examples from your microbit repo for the nrf51.  I've gotten your gpio_hal_blinky.rs running fine and  modified your gpio_hal_printbuttons.rs to work as follows (although its extremely slow to respond and I'm not sure whats causing that)
#![feature(used)]
#![feature(const_fn)]
#![feature(use_extern_macros)]
#![no_std]

extern crate cortex_m;
extern crate nrf51;
extern crate nrf51_hal;
extern crate cortex_m_semihosting;

use nrf51_hal::prelude::*;

use cortex_m::peripheral::Peripherals;
use cortex_m::interrupt::Mutex;
use cortex_m_semihosting::hio;

use core::cell::RefCell;
use core::fmt::Write;

static GPIO: Mutex<RefCell<Option<nrf51::GPIOTE>>> = Mutex::new(RefCell::new(None));

fn main() {
    if let (Some(p), Some(mut cp)) = (nrf51::Peripherals::take(), Peripherals::take()) {
        cortex_m::interrupt::free(move |cs| {
            let mut stdout = hio::hstdout().unwrap();
            /* Enable external GPIO interrupts */
            cp.NVIC.enable(nrf51::Interrupt::GPIOTE);
            cp.NVIC.clear_pending(nrf51::Interrupt::GPIOTE);

            /* Split GPIO pins */
            let gpio = p.GPIO.split();

            /* Configure button GPIOs as inputs */
            let _ = gpio.pin17.into_floating_input();

            /* Set up GPIO 17 (button A) to generate an interrupt when pulled down */
            p.GPIOTE.config[0]
                .write(|w| unsafe { w.mode().event().psel().bits(17).polarity().hi_to_lo() });
            p.GPIOTE.intenset.write(|w| w.in0().set_bit());
            p.GPIOTE.events_in[0].write(|w| unsafe { w.bits(0) });

            *GPIO.borrow(cs).borrow_mut() = Some(p.GPIOTE);

            writeln!(stdout, "Main ran").unwrap();
        });
    }
}

/* Define an exception, i.e. function to call when exception occurs. Here if we receive an
 * interrupt from a button press, the printbuttons function will be called */
nrf51::interrupt!(GPIOTE, printbuttons);

fn printbuttons() {
    /* Enter critical section */
    cortex_m::interrupt::free(|cs| {
        if let Some(gpiote) = GPIO.borrow(cs).borrow().as_ref() {
            let mut stdout = hio::hstdout().unwrap();
            writeln!(stdout, "A Button was Pressed").unwrap();

            /* Clear events */
            gpiote.events_in[0].write(|w| unsafe { w.bits(0) });
        }
    });
}

I am now trying to combine the two and cause the lights to turn on when a button is pressed but I'm struggling.  I cannot figure out how to get access to the nrf51::Peripherals within the printbuttons function without causing ownership errors.  Do you have any recommendations or insights into how I can do this?  Sorry if this is a trivial question I'm new to Rust and embedded programming.

No idea what causes your slowness, could be the semihosting output which is really slow.
In order to allow the interrupt handler to access the LED you'll need to setup the LED pin as in the gpio_hal_blinky example, define a static LED Mutex<RefCell<...>> thingymabob as in the example you cited above and assign it the led pin and then access it from the interrupt handler. The implementation should be pretty similar to the gpio_hal_printbuttons example but with the serial setup and output substituted with a LED toggle.

The issue isn't a matter of "Slowness" but rather the interrupt is being hit intermittently due to a floating ground. The microbit is designed to work with floating input whereas the nrf51dk needs a pull up.
So the proper way to handle the interrupt is to set
 let _ = gpio.pin17.into_floating_input();

to
let _ = gpio.pin17.into_pull_up_input();

and then set the interrupt event to occur on a pull up from lo_to_hi
p.GPIOTE.config[0].write(|w| unsafe { w.mode().event().psel().bits(17).polarity().hi_to_lo()});

to
p.GPIOTE.config[0].write(|w| unsafe { w.mode().event().psel().bits(17).polarity().lo_to_hi()});


Pick typography – both free and $$ options

Likely Inter 4.0 as part of the free option. Pick a nice serif to pair it with.
Commercial Type for fancy options.
Maybe Blanco by Foster Type

If we want to recommend a paid font, it might be best to self-host the free option and then provide documentation for how to swap it out.

Cool, that sounds good. Self-hosting seems simpler than whatever cloud hosting individual typeface foundries would offer. Most I've bought just give you a zip file download.


Cannot Scale Background Image to Canvas Dimensions

Hey,
thanks again for integrating FabricBackground.
I am struggling hard to scale the image to the given Canvas Size
<FabricCanvas :height="500" :width="500" :ref="'canvas'+index">
              <fabric-background :ref="'background'+index" :url="view.image" :width="500" :height="500" :scaleX="1" :scaleY="1"></fabric-background>
</FabricCanvas>

If the image itself is 1000x1000 px for example, I only see the top left quarter of the image.
Can you help?
Normally it would be achieved by scaleX: canvas.width / img.width
But I could not access the image width here.

Ive added the event @image-loaded to the most recent release for each type of image from url component.
From this you can now access the img width etc.
v0.3.70
`
<fabric-background-image
:id="345667"
:scaleX="400 / imgWidth"
:scaleY="400 / imgHeight"
@image-loaded="imageLoaded"
>
...
imageLoaded(img) {
  this.imgHeight = img.height;
  this.imgWidth = img.width;
},`

assuming 400 is the width and height etc

Hello, thanks for your quick answer.
It is getting close. The only problem left now, is that the background of the canvas is only updating its scaling, when I touch an element in the canvas, and force a rerender.
I tried it in the imageLoaded function with
this.$refs.canvas0[0].requestRenderAll()

But it is not working here. It is working on other parts of the code, tough

Okay I got it working, I first had a mindfuck but doint it exactly as you wrote worked.
Thanks a lot!


Alternate compiler files > Per version its own compiler.

Just create compiler and basic html templates that compile according to their version. Then include all those in the framework. Done.

Done


Swift APIView: update MacOS deployment target

Specifies minimum MacOS level 10.14 for supporting XCode 10.2.

/check-enforcer override


NO DOCS?

Version of the software
0.2.4
Operating system
OpenBSD
Browser / Native
LoliChrome 101
What happened?
A bug happened!

Relevant LoliCode if needed
No response

PPLEASE MAKE DOCS!


Improve the buffering of simple wires

Describe the bug
The new buffer_varying_lengths test for rsz shows cases where the buffering needs improvement.  Two-pin wires are increasing length are generated and buffered:

The buffering is far from optimal with some wires showing one buffer right next to the driver.
Expected Behavior
Optimal buffering for all lengths.
Environment
Head of master

To Reproduce
Run the test
Relevant log output
No response
Screenshots
No response
Additional Context
No response

For example moving instance wire15 to the middle (x=450um) reduces the slack from -0.14 to -0.12


Better defaulting for scroll view primary-ness

Improved defaults for scroll view primary-ness

Vertical scroll views default to primary:true.
Horizontal scroll views default to primary:false.
If a scroll view is primary and it got a non-null inherited primary
scroll controller, it introduces a primary scroll controller inherited
with a value of null for its descendants.

ScrollController now multiplexes writes to all registered positions;
reads of position continue to assert that only one position is
registered.

Followup to discussions in #7444

@Hixie

Followups in #8534.


How to install? Posting the link the JSON fails in FoundryVTT setup menu.

Will this auto-update once installed correctly in Foundry? And it will pull from this repo and not the old one?
Thanks for making it work again.

@Sonny-Crockett to install this fork, you first will have to uninstall the old module, then install based on this manifest url:
https://raw.githubusercontent.com/League-of-Foundry-Developers/simplefog/master/module.json

@Sonny-Crockett , All your fog should remain, event after uninstalling and re-installing the new fork.


Could UnifiedNlp.apk be also present at F-droid?

i can only use this version because i have google version also present on my sistem. i use an xposed module so that your app could work instead of google's.
thanks!

I think it can be done.
I will need help with the summary/description of that package & the other one so that users won't get confused when they see two of them.

I just read through your description as added to the F-Droid repo. I guess it's better to put the "To be able to use this package, you will either need the" paragraph before the backend list, so it's more likely for users to see it. And you can remove the "WARNING: Depending on your hard-/software configuration" paragraph. I think you can also change the title to something like "UnifiedNlp (Xposed version)", as this is the only real use case right now (although your description is technically right).
As an interesting side-note: I plan to integrate the Xposed module into the UnifiedNlp.apk version longterm, so no additional apk is needed.


Replace centered transforms with non-centered versions

The centered transforms are not recommended to be used because the
transforms are over parameterized, or having dependent
parameters. This makes the optimization more "difficult" and less
efficient.
In the ITKv4 examples, their usage is replaced with the regular
transforms.

Are these examples involved in any regression tests? I guess CI infrastructure will answer that 😄

It does not look like any of the CI is testing the Examples.

Some examples are used in tests, but none of these I suppose.

It looks like BUILD_EXAMPLES is off for all CI.

2018-11-23T19:30:01.4074180Z The following tests FAILED:
2018-11-23T19:30:01.4075090Z 	2762 - ImageRegistration14Test (Failed)

Please see the corresponding update to the Software Guide: InsightSoftwareConsortium/ITKSoftwareGuide#106

@blowekamp VERY NICE! and thanks for addressing the software guide too.


Remove remote URL loading feature from Waterbug

It's just never going to work properly because of cross-origin issues.

Okay okay okay, here it is. It works as an Electron app because native. If you deploy to a fileserver/S3, this probably isn't going to work. So do we conditionally enable it?

Removed entirely.


Using RemoteIO with microphone

Hi,
When playing instruments over TwilioVideo, the voice processing seems to degrade the quality of instruments like guitar and piano pretty badly. From what I understand, RemoteIO in CoreAudio is a possible solution to disable the extra processing thats happening.  Our app users will be playing one at a time, so we're not concerned with cancelling echo from additional participants.
I was checking out https://github.com/twilio/video-quickstart-ios/tree/master/AudioDeviceExample as it uses RemoteIO, but I see its only for audio file playback and not microphone. Is there an example of the same, but for use with the built-in mic or any thoughts or concerns around doing this?

Hey @snobear,

Is there an example of the same, but for use with the built-in mic or any thoughts or concerns around doing this?

We don't have this exact use case covered in example code, but if you're looking to use RemoteIO in place of VoiceProcessingIO, you could take ExampleCoreAudioDevice and modify it. You would need to:

Enable the input bus for the RemoteIO audio unit. The flag is here.
Add a rendering callback for the input bus. Like this example but without the extra mixing logic that you don't need.
Implement TVIAudioDeviceCapturer methods to claim support for capturing.

I hope this is helpful, but I understand that what I just described is pretty low level and might be difficult depending on your knowledge of real time audio.
Best,
Chris

Hi @ceaglest, thanks for the reply and the info. Audio programming is certainly not in my wheelhouse, but I'll take a crack at it nonetheless. I may come back with a question or two :).

Thanks! Let us know if you have any more questions.


Fax machines can print from text file

About the PR

Adds functionality from https://github.com/space-wizards/space-station-14/pull/22278 to the fax machine.
You can now print use the fax machine to print a text file from your computer.
You can choose between normal or office paper.
Why / Balance

Less alt tabing and copy pasting, more qol.
Technical details

Media

https://github.com/space-wizards/space-station-14/assets/86210200/15778f44-6035-45ee-8217-491d3afe70d6

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Fax machines can now print text files from your computer.

Heads Of Personnel everywhere rejoice

Based

What security vulnerabilities could be introduced by this feature :hmm:

From what I've read, apparently none.

same vulnerability from copy and pasting large files or whatever

ok i think now its good and can be moved to printer when it will be in game

This is so legitimately useful


Remove pd_legal from base_select and derive it like any other legality where it is needed

It's just inelegant to have a separate case for this even though it's our most-used legality.

Sticking points here are:

ORDER BY pd_legal DESC, name (in two places)
HAVING pd_legal = 1 (commented out, in set_legal_cards)
To get the rank for update_fuzzy_matching

I've decided that as this is derived from wonderful normalized data it's not really an issue. Grabbing it out in the ORDER BY seems uglier than just having it available as it is generally useful. WONTFIX.


Caffe Model to CoreML Model

Greetings,
This may be off topic, but I'm just trying to find some help.
I'm trying to get 300 FCN model to run in my xCode project.  I'm converting .coffemodel to .mlmodel with coremltools:
coreml_model = coremltools.converters.caffe.convert(caffe_model,                 image_input_names='data',            is_bgr = True,            red_bias = -104,            blue_bias = -123,            green_bias = -117,            image_scale = 1)
As far as I understand the input image is in BGR color space, with above mentioned biases.
After conversions when I read the model description with coremltools:
input {   name: "data"   type {     imageType {       width: 300       height: 300       colorSpace: BGR     }   } } output {   name: "score"   type {     multiArrayType {       dataType: DOUBLE     }   } } metadata {   userDefined {     key: "coremltoolsVersion"     value: "3.3"   } }
The output has no shapes.
When I add the model to Xcode project, I run the model by passing CVPixelBuffer as input
 let input = buffer(from: userSelectedImage_UI)             guard let prediction = try? model.prediction(data: input!) else {                 return             }
the output of the model is MultyArray.
let output = prediction.score
How can I convert it to  CVPixelBuffer, if there're no shapes.
I've tried using MultiArray converters to no avail, the output is just black image
I've tried this and this methods.
If anybody knows how to get this working in CoreML I'd really appreciate it

Dear @malemo,
Did you find how to implement to CoreML yet?
Hope to hear you soon.


improved FireboltProperties: removed of() factory method, added constructors

The annoying factory method of() is removed. Now we have constructor that accepts Properties instead. Constructor can guarantee that all final members are initialised.

Can you comment on what this improves? The change is fine IMO, but it's not clear what's the benefit. Also, I'd suggest adopting a better PR/commit name. Perhaps refactor FireboltProperties to use AllArgsConstructor or even better refactor FireboltProperties to <insert the reason for this change>.

Done


Lacking consistency on determining what are manifests

On package.json, it is required to have at least three elements for it to allow jsonValidation/yamlValidation. But in the code, we allow at least two elements.

Indeed, I wonder if we could provide such integration using actual code instead of package.json, this way - we could potentially only make it usable for actual detected manifest


the precision of detection

In the process of my reproduce, the precision of detection is relatively low。Is there some problem？
Looking forward to your reply！Thanks!

Sorry, I don't know! For the reprodece, the other evaluation indicators are not much different from those given by the author, except for the precision not sure. In my custom dataset, less than 0.1 in the case of multiple categories. So I didn't try again。

In the process of my reproduce, the precision of detection is relatively low。Is there some problem？ Looking forward to your reply！Thanks!

Do you know why now?

因为_C.TEST.NMS_CONF_THRESHOLD  = 0.001 置信度太低了，导致非常多的bbx，P就很低，R很高

I haven't seen it for a long time. I only remember these place to be modified：
lib/dataset/bdd.py，line76:
if obj['category'] in id_dict_single.keys():
the value of id_dict_single need  modified in lib/dataset/convert.py
lib/dataset/bdd.py, line8: single_cls = False
@wangsigong

Thank you for your reply! @Yaoxingtian

在我的复现过程中，检测的精度比较低。有什么问题吗？期待您的回复！谢谢！

现在知道为什么了吗？

因为_C.TEST.NMS_CONF_THRESHOLD = 0.001 置信度太低了，表示非常多的bbx，P就很低，R很高兴

您好，那应该设置多少比较合适呢


latest Zanata translation changes in kie-wb-common from 2016-07-07

Return of the slash..... (luckily it's a non-issue now).
+1 to merge.

(Waiting on Jenkins before merging)

@manstis are these slashes not anymore "dangerous" for the line endings?

@mbiarnes AFAIK errai was changed to handle the slashes..


Bug: crafting recipes and inventory sort do not update inventory

When using a crafting book recipe, or using a "sort inventory" function (not in the first beta release but will be in a future one) doesn't seem to update. This is because of a conflict with the "inventory pause" mod. It behaves like a visual bug. Clicking on any inventory slot, even an empty one, causes it to update to accurately reflect everything.

I like inventory pause, even though it can feel a little like cheesing the game. However, it does not work with all modded inventory GUIs. In addition, it obviously cannot work on a multiplayer world. I might leave it in, disabled by default, as an option for singleplayer worlds, but I might also remove it in the future.
If you're having an issue or dislike the way it behaves, feel free to disable it or delete it from your copy of the pack. It is not a dependency for any other mod in the pack until I start uploading new versions of the pack and decide how I want to deal with this.

Inventory Pause will be removed in stability release 1.1.0


How to change only selected  tab text color

I need to  change the only selected tab text color with remaining tab text should be same color

I made an update to the library. Automatic text highlight is done when using basic text tabs, if you set the parameter tab_highlight_text=true the text color will have the indicator color. You can set a different color by also setting the param tab_highlight_text_color to your desired value.
Please update to version 1.0.7 to use this.
compile 'com.hold1:pagertabindicator:1.0.7'
If you are using custom tab views using TabProvider interface, then you should do your own logic based on the example of CustomAnimAdapter in my sample app.


Translations failing  when page is idle for couple of hours

🐛 Bug Report
in next-i18next version 13.2.2, When translations are rendered using getServerSideProps (next 13.5-pages router)  , translations doesn't happen after couple of hours. I can only see key names .
To Reproduce
next-i18next.config.js
const path = require('path');
module.exports = {
  debug: false,
  i18n: {
    defaultLocale: "en",
    locales: ["en", "es"],
  },
  localePath:
    typeof window === 'undefined'
      ? path.resolve('./public/locales')
      : '/locales'
};

index.tsx
export const getServerSideProps: GetServerSideProps<Props> = async ({
  locale,
})=>{

  return {
    props: {
      ...(await serverSideTranslations(locale ?? 'en', [
        'common',
        'randomns'
      ]))
    }
  };
}

Expected behavior
Translations shouldn't fail intermittently
Your Environment
"i18next": "^22.4.13",
"next": "^13.5.6",(pages router)
"next-i18next": "^13.2.2",

Hearing this for the first time...
Maybe a problem of your deployment environment?
Or is this reproducible also locally?
Please provide more information.

Yes, I can reproduce it locally. Similar comment was addressed here. Can you confirm if this is needed whenever we need to get translations from getServerSideProps
https://github.com/vercel/next.js/issues/46730#issuecomment-1489654441

seems the way Next.js handles this... yes

@PreethiVS21 Have you figured out how to fix it? I have also encountered this issue.

I've found I can reproduce this locally fairly easily.  In our case, we are using a custom Docker image.  Simply rebuilding the image and redeploying it is enough to break the translations.
Steps:

Deploy the app in prod mode and open any page.
Rebuild / redeploy the image.
Soft-navigate from the already-opened page to another page.  Translations will be broken.

I can see that Next is fetching a translations file on every page transition.  For instance, every time I navigate to Page A, it will fetch /_next/data/tfbRnBfT3Y6OXf7Afbrm1/en/PageA/6632778e658b8005459f8503.json, and every time I go to Page B, it will fetch /_next/data/tfbRnBfT3Y6OXf7Afbrm1/en/PageB/99125246-21fc-4742-876c-e7d3b9a52bcc.json.  But after a redeploy, those two URLs return an empty translation list: {}.  Presumably this is because the Next BUILD_ID has changed.  If I do a full reload, the pages fetch slightly different URLs, like: /_next/data/eCtPEywIx-3IoSgG-JHVF/en/PageA/6632778e658b8005459f8503.json.  (Note that the old BUILD_ID tfbRnBfT3Y6OXf7Afbrm1 has been replaced with eCtPEywIx-3IoSgG-JHVF).
Anyways, it seems I need a way to detect that there's a new BUILD_ID in play and force next to do a full-page-load transition.

@PreethiVS21 @Sarawut-keng any updates guys? same issue on the same next version 13.5.6 (pages router)

@adrai getServerSideProps totally doesnt work after couple of hours, its ok with getStaticProps

@adrai getServerSideProps totally doesnt work after couple of hours, its ok with getStaticProps

If that's the case you may ask the Next.js team...

@BrilliantDecision after reviewing the case, I have figured out that @jgregory-apogee  ’s comment was correct. After my CD re-deployment, the language broke. I am not sure if this occurs only on the page router. Right now, we do not have the best solution yet. We have just briefed users that we have updated to a new version and that they need to refresh the screen.

@Sarawut-keng and how do you implement this notification, what do you use for this? how do you understand that you need to refresh the page? some error catcher?

We have the exact same problem here. i18n store is empty on after some deployments even the translation files are exist.


Add Size getter for pixels and megapixels

Number of pixels is used in some UI logic to sort available sizes. It's tedious to multiply it on each occasion.
Megapixels count is useful for presentation, the same as aspect ratio.

Hey @ezaquarii, thanks !
Size is meant to not have units. We actually use it everywhere as Resolution with pixels as units. Renaming this to Resolution will create breaking changes. Renaming docs only (about the units) and keeping the Size name will feel incomplete.
So let's keep only the width * height operation under the name getArea().
Fun fact:
According to google 1MP = 1,048,576 pixels :D

Fair enough. Here you go.

Thanks


Move consts/functions to the other existing files and update imports

Hi!
First of all, thanks a lot for this plugin, file rename is amazing. :slightly_smiling_face:
In the codebase I'm working on there's multiple import options when I use code action, so if done automatically, it will most certainly mess things up by importing the wrong thing.
When refactoring, I usually move several functions, types and consts to the other file, and looks like I can't automatically move imports they use with them..
How can I help you implement that? :slightly_smiling_face:

Hmm, I wonder – this behavior isn't part of the LSP specification, so there's only so much we can do. "Watching" exports would be very tricky, but I could potentially imagine hijacking the built-in Move to a new file code action to create a command to move an export from one open buffer into another and update references. You'd have to manually trigger it every time, though, but it could be useful.
I've also thought about creating a version of TSLspImportAll that only adds imports if they're exported from an open buffer. That could help improve accuracy when automatically running code actions.
Let me know if either of those options sounds useful. Beyond that, I'll give it some thought, but since the plugin ultimately depends on LSP functionality, I'm not sure how close we can get to what you describe.

Hijacking Move to a new file seems like the way to go. :slightly_smiling_face:
I see following steps:
df - destination file which we want to move stuff to
nf - file created by Move to a new file

Use Move to a new file to create nf in the same directory as df
Move everything from nf to df
Fix all imports in the opened and unsaved buffers that point to the nf
Use reorganize imports or some sort of LSP-provided autofixing to possibly deduplicate imports in the df without importing anything new

As to TSLspImportAll, I think it will be much more useful if it could look for the imports in the opened buffers in addition to the exports, and prefer exports.
Because there could be imports like EditIcon from common/icons, but common/icons is not opened. And when you use autoimport, it could add it from something like material instead.
Maybe it would be wise to ask tsserver authors if it's a feature they would be interested in having. :thinking:
Thanks for the response. :slightly_smiling_face:

Right, I agree that ultimately big features like this should be implemented at the language server level. For the first idea, let me look into it some more and get back to you re: feasibility. For the second, I just opened up #42, so let me know there if you have any thoughts / ideas.

So I looked into this, and my conclusion is that I'm not at all confident we could do this in a way that would a) provide a good user experience, and b) not break all the time. It's a cool idea, but ultimately it's too far from LSP behavior to implement in a satisfactory way.
For the priority idea, I've put together an implementation in #43 that implements your ideas, so let me know if you have any feedback there.

How do you approach the problem of moving functions between files and updating imports? :thinking:
Is there some solution which I'm not aware of, or do you update imports by hand like I'm doing currently?

Honestly I'm not sure if there is a better solution than just doing it manually (if there is, I'd love to know, too). I've only worked with Neovim and VS Code, but as far as I know, there's no editor / extension that has that kind of functionality.
Your question gave me an idea: I could create a new function to detect broken imports from tsserver diagnostics, delete them, and run :TSLspImportAll. It would then be possible to either run the function manually or set it up so that it runs automatically when opening / focusing a buffer. This wouldn't be as convenient as fully automated updates, but it would take away some of the pain from going through the project and updating broken imports.
Let me know if you'd be interested in that and I can put something together.

That sounds really cool. I'd love to test something like that.
The only thing that makes me avoid using :TSLspImportAll in existing files is that it reorders imports, thus making extra noise in diffs when merging. :thinking:
I use it all the time in the new files though.
Also could we batch interactive import selection? I would imagine if we run :TSLspImportAll for a 10 files, every time there will be options to import from?
What do you think about this workflow:

move const/function to the other file, save both
hover on moved const/function and type use something like :TSLspPointBrokenImportsHere
profit? :grin:

Can we get import path from tsserver by hovering on a const/function?
Cases where we already have grouped import might be annoying to deal with though. :thinking:
Another way which comes to mind:

save a list of references to a hovered const/function somewhere
move it to the other file by hand
update imports from that list using some mechanism like :TSLspImportAll

Thanks. :slightly_smiling_face:

Batching imports is possible. I could make it possible to specify the buffer(s) you want to run it on, e.g. :TSLspImportAll App.tsx, or run it on all open buffers via something like :TSLspImportAll all.
I don't think it's possible to request an import path from tsserver, but saving references is a really interesting idea. I think the most reliable way to do this would be to make sure all the files that refer to the moved function are loaded, then run :TSLspFixBrokenImports (or whatever we decide to call it) on each.
I opened #46, #47, and #48 to discuss potential improvements, so we can talk more there if you have more ideas.

Can we somehow run regular code action on a const/function with missing import?
For example, when I import a const/function from a file, which is already used for some imports tsserver says this:

Add 'someFunction' to existing import declaration from "somepath"

And by doing it this way I never produce noisy diffs. :thinking:
Would be cool to use this automagically for fixing several files though. Could it work with saved references?

I'm not sure I understand completely, but :TSLspImportAll simply gets and runs code actions, so if that's the available code action, that's what it'll do. The issue is that when it adds two or more imports from the same source, each code action is (at that point) independent, so each import will get a separate import statement.
I'm not that thrilled with the current need to organize imports afterwards (and it's not how VS Code does it), but manually merging imports is complicated, and when we can completely offload that complexity to tsserver, I prefer to do so. I'll see if I can think of something else though.


fix-default-branch-name: to main

what
Fixes default branch name references.
why
The default branch is now main instead of master.
references
#90 #89

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/bjd2385/dynamic-continuation?version=dev:abf442d8e0ab91afd5be5abc3b8cf7d7b6fcbbcd


clarified LONG_NAME of "snow depth" variable

Trying to correct a long-standing problem with users misunderstanding of the GEOS snow mass and snow depth output.   See "snow depth" entry in MERRA-2 FAQ:
https://gmao.gsfc.nasa.gov/reanalysis/MERRA-2/FAQ/#
This has been an issue for years, with multiple inquiries about "funny" MERRA-2 output.  This past summer, we narrowly avoided two peer-reviewed papers by ECMWF and Meteo-France showing very unhelpful MERRA-2 results.  Ask me if you want to know more.
By fixing the F90 src code, we have at least a chance of getting this critical information into new versions of GEOS file specs documents.
Similar edits may be good to have for snow depth over glaciated surfaces, which is why I added @lcandre2 and @zhaobin74 to the list of reviewers.  (Richard Cullather isn't on Github.)

Let's verify with Rob what impacts this has on file-specs in GEOS-FP before approving.

For the record, I'm adding an email from Matt to the discussion below.
In response, without knowing much at all about CF conventions, I think Matt's email is head-on.  My only comment would be that the "snow depth" from the GEOS "lnd" Collection is the mean over "snow-on-land" only, and I don't know how that would be expressed in CF-compliant language.  (That is, the mean excludes the zero snow depth values over the snow-free land portion and also excludes non-land areas such as sea-ice, regardless of whether they are snow-covered.)
For now, I think we should still go ahead with the PR and not wait for the big switch to CF compliance.

Snow Depth and CF Conventions
From: Thompson, Matt
Thu 19-Dec-19 10:48 AM
To: Reichle, Rolf
Rolf,
This is a random question, but your PR about the snow depth change made me go on a tangent to CF. I did that because one of my (perhaps "our") long standing wants is to convert GEOS to use CF Convention standard_names instead of using short_name everywhere (and where long_name is the actual human-readable description).
So I wondered how would one make your change under CF? This is something we might have to think about soon-ish.
I think/hope I'm safe saying that our "snow_depth" would be "surface_snow_thickness" in the conventions table. But then the help for that says:
The surface called "surface" means the lower boundary of the atmosphere. Unless indicated in the cell_methods attribute, a quantity is assumed to apply to the whole area of each horizontal grid box. Previously, the qualifier where_type was used to specify that the quantity applies only to the part of the grid box of the named type. Names containing the where_type qualifier are deprecated and newly created data should use the cell_methods attribute to indicate the horizontal area to which the quantity applies.
Thus, your change made me go on the cell_methods tangent. I stared at:
http://cfconventions.org/Data/cf-conventions/cf-conventions-1.7/cf-conventions.html#statistics-applying-portions
and wondered would this then need to have a cell_method of:
surface_snow_thickness:cell_methods="area: METHOD where snow"
where "METHOD" is the right entry from here:
http://cfconventions.org/Data/cf-conventions/cf-conventions-1.7/cf-conventions.html#appendix-cell-methods
mean, median, whatever the code does.
Does this track with your understanding? I'm just trying to think of all the bits and pieces Atanas/Ben/Tom might need to implement in MAPL in the future (if we go the CF route).
Matt

Should be ready for approval now.
Adding Rob's email below for the record:
Date: Tue, 24 Dec 2019 00:14:36
From: Rob Lucchesi
Subject: Re: action items re. mis-interpretation of MERRA-2 (and GEOS) snow output
Rolf, Bill,
[...]  Changing the long-name should not be a big deal.  Is the intent
to have this in for the current FPP going operational after the 1st of
the year?  I think it is reasonable to just mention the change in the
release notes for the system.
[...]
Thanks, Rob


FW-449. Fixing overflow in DAGrank calculation

In the calculation of the tentativeDAGrank in
neighbors_updateMyDAGrankAndNeighborPreference an overflow was possible.
This results in occasional choosing a preferred parent with MAXDAGRANK.

There is something strange with the Travis CI build... Any idea?

Yes, I noticed that. It seems something wrong to download the texlive-binaries from the server.

@koalo , could I ask you to change the line to the following which should be more easy to understand? Thanks!
tentativeDAGrank = (uint32_t)neighbors_vars.neighbors[i].DAGrank+(uint32_t)rankIncrease;

Yes of course! The patch is updated.

This is perfect! Thanks a lot!


[feature request] specify the name and location of the recording

Would it be possible to specify what the recording should be called, and where to store it, via obs-cmd?
many thanks!

I didn't investigate if it is possible via API. I leave the issue open for now, if somebody else is interested


scrollbar dont show up on hover, but works and show up when scrolling?!

Hi mate,
First of all well done! a very nice and useful script u made right here.
there is an issue I have with the perfect scrollbar. I have a display:none div that shows up when someone click on a button. I embedded Perfect-Scrollbar on that div.
the issue is: when i click the button and hover over the div, the scroll bar does not show up. but it actually works when I scroll through mouse wheel and show up. and then every time I click the button it shows up. until I refresh the page.
so basically it only shows up when I scroll the wheel, else it remains hidden.
By the way, in normal divs (without display:none) it works fine
Regards

I had this same problem when using Angular. The scrollbars would only appear after I scrolled in the div, but not on initial hover over the div. Here's how I fixed that issue:
I imported the library with import PerfectScrollbar from 'perfect-scrollbar';
I created the following function on my controller:
initializeScrollbars(): Observable<PerfectScrollbar> {
  return observableOf(new PerfectScrollbar('#id-of-div-to-make-scrollable'));
}

Then I called that function from ngAfterViewInit like:
this.initializeScrollbars()
  .pipe(
    take(1),
    delay(1500)
  )
  .subscribe(ps => ps.update());

jQuery slideToggle() is an async function and if you want to do something after the element is fully toggled you should use a callback function. Just calling the update function in the next line of the statement doesn't help.
As you can see here: https://api.jquery.com/slideToggle/, there's a complete parameter. It's callback to do something when the toggle is completed. So, you should do like this:
$(".more-list").slideToggle(200, function () {
  $('#scroll').perfectScrollbar('update');
});

Thanks :D

Hello @utatti  ,
I also have the same issue as @bigpapa86. The solution you have provided here is some how animating the contents of my the particular div in which I have integrated the perfect-scrollbar/always available feature. Please provide a solution.


az webapp create returns Error : There was a conflict. AlwaysOn cannot be set for this site as the plan does not allow it.

Describe the bug
when you create an ASP on Shared SKU and then create a web app on that ASP it tries to enable AlwaysOn. Returns Error : There was a conflict. AlwaysOn cannot be set for this site as the plan does not allow it.
To Reproduce

install AZ CLI  2.0.66
az group create -l wewstus -n MyTestRG
az appservice plan create --name "MyDemoASP" --resource-group MyTestRG --location westus --sku SHARED
az webapp create -g MyTestRG -p MyDemoASP -n MyDemoApp

Expected behavior
create the webapp and not enable AlwaysOn
Environment summary
MSI install of latest version 2.0.66
Windows 10

PowerShell Core 6.0
Additional context
This appears to happen with any version greater than 2.0.61

@chNorman thanks for reporting this. Shared SKU doesn't support Always ON & is a regression on our end. For not you can create an app with a different SKU & Scale Out to Shared. Sorry for the inconvinience.

https://github.com/Azure/azure-cli/pull/9323 fixes this


Add full working example, proto file and source

Resolves #452

The travis build appears to be failing for reasons unrelated to my change... but I could be wrong!

Merged with removing .travis.yml.
Thanks!


feat(docs): ability to set security scheme

All we need to make possible authorize from swagger — its define security schema and set it globally or per endpoint.

I've added some comments. Just cosmetic changes. After that I will merge it and make a release.


Make lines drawn the same frame they are called

While working on some updates to Rusty Engine I started using this plugin to draw debug lines representing the colliders on my sprites. Since the sprites were in motion, I noticed that the debug lines were consistently drawn 1 frame later, which was annoying.
After poking around a bit, I found that if you move the systems from CoreStage::Last to CoreStage::PostUpdate, then they were drawn in the same frame!  This is much nicer.
I have tested this only with the release version of Bevy 0.5.

Thanks for this, I should have considered that.  Just sorting out #10 first which touches a lot of the same code and then I'll merge/change this :).

Cool, thanks!

Just gonna push this through now as #10 is gonna take a while.

Cool, thanks! When do you plan on issuing the next release?

I'll try to push a release tonight with this fix (sorry if I forget!)

That sounds great, thanks! (no worries if you forget, life is busy 😄)

Just published 0.3.4 :)

Ah, that is so much nicer. Thank you! ❤


0.6.0a1 breaks namespace packages

# pip install titiler==0.5.1 --no-cache
$ ls -la /Users/vincentsarago/Dev/venv/dev/lib/python3.8/site-packages | grep "titiler"
drwxr-xr-x    5 vincentsarago  staff       160 May 11 16:33 titiler
drwxr-xr-x    9 vincentsarago  staff       288 May 11 16:33 titiler-0.5.1.dist-info
drwxr-xr-x    7 vincentsarago  staff       224 May 11 16:33 titiler.application-0.5.1.dist-info
drwxr-xr-x    7 vincentsarago  staff       224 May 11 16:33 titiler.core-0.5.1.dist-info
drwxr-xr-x    7 vincentsarago  staff       224 May 11 16:33 titiler.mosaic-0.5.1.dist-info

# pip install titiler==0.6.0a1 --no-cache
$ ls -la /Users/vincentsarago/Dev/venv/dev/lib/python3.8/site-packages | grep "titiler"
drwxr-xr-x    5 vincentsarago  staff       160 May 11 16:31 titiler
drwxr-xr-x    9 vincentsarago  staff       288 May 11 16:31 titiler-0.6.0a1.dist-info
drwxr-xr-x    6 vincentsarago  staff       192 May 11 16:31 titiler_application-0.6.0a1.dist-info
drwxr-xr-x    6 vincentsarago  staff       192 May 11 16:31 titiler_core-0.6.0a1.dist-info
drwxr-xr-x    6 vincentsarago  staff       192 May 11 16:31 titiler_mosaic-0.6.0a1.dist-info

$ python -c 'from importlib.metadata import version; print(version("titiler"))'
0.6.0a1

$ python -c 'from importlib.metadata import version; print(version("titiler.core"))'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/importlib/metadata.py", line 531, in version
    return distribution(distribution_name).version
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/importlib/metadata.py", line 504, in distribution
    return Distribution.from_name(distribution_name)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/importlib/metadata.py", line 177, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: titiler.core

links:

https://github.com/pypa/flit/issues/442
https://github.com/pypa/warehouse/pull/10072
https://github.com/pypa/warehouse/issues/10030

I don't think there is another choice than to move back to setup.py here 😭


Support JSON response bodies on validation error(s)

The default validation error response content-type is a text/html. I am wondering if you have considered adding an option to support JSON response bodies?
use case: I am creating a pure JSON api and I have many application level responses that are other status codes besides 400 that return JSON. I would like the 400 status code responses, which are handled by the middleware, to also be JSON so they could be more easily displayed in a single page application to the end user.

I believe what you have happening here is that the enforcer is generating an error which it then passes on to the next error handling middleware. If you don't have a custom defined error handling middleware then the Express fallback error handler middleware that runs will generate an HTML document that contains the error message and stack.
In your case you can create a custom error handler middleware that will provide an appropriate JSON response. Here is an example of how that can be done. The custom error handler middleware is near the bottom of the example, right before the server starts to listen.
Also, if the error received by the custom error handler middleware was produced by the enforcer then there will be an exception property on the error.
'use strict'
const express = require('express')
const Enforcer = require('openapi-enforcer-middleware')
const path = require('path')
const statuses = require('statuses')

const app = express()
app.use(express.json())

const enforcer = Enforcer(path.resolve(__dirname, 'openapi-v2.yaml'))

// this middleware will handle explicit mock requests
enforcer.mocks(path.resolve(__dirname, 'mock-controllers'), false)
  .catch(console.error)

// this middleware will handle non mock requests
enforcer.controllers(path.resolve(__dirname, 'controllers'))
  .catch(console.error)

// this middleware will automatically run fallback mocks
enforcer.mocks(path.resolve(__dirname, 'mock-controllers'), true)
  .catch(console.error)

// add the enforcer middleware runner to the express app
app.use(enforcer.middleware())

// add custom error handling middleware
app.use((err, req, res, next) => {
  if (err.statusCode >= 400 && err.statusCode < 500 && err.exception) {
    res.status(err.statusCode)
    res.json({
        message: err.message,
        statusCode: err.statusCode
    })
  } else {
    const statusCode = err.statusCode || 500
    console.error(err.stack)
    res.json({
      message: statuses[statusCode],
      statusCode: statusCode
    })
  }
})

const listener = app.listen(3000, err => {
  if (err) return console.error(err.stack)
  console.log('Server listening on port ' + listener.address().port)
})

If this isn't what you're looking for then please continue to comment here, but I'll close this for now.

thank you! forgot i could do this 😮


Recipe Feature - Mostly Details Page

List page work to follow in another PR.

recipe details is so 🔥


display help in browser

this is just a minor change to get help displayed in a separate tab in the browser

I think this setting would maybe better be put in your local gaprc on your computer. If the jupyter notebook runs on a machine like https://cloud.gap-system.org this setting has no effect (or rather, it will error, because we cant execute a browser there).
Needless to say that I would quite like a better help system integration...

Agree!


Pass the model appends in the new intances

I think it is good if we also pass the appends so that we can set appends before querying the models something like:
(new User)->append(['full_name', 'age'])->get(); // this will return users with appends of full_name and age

Please forgive me if I am not sure if I created this PR correctly, please let me know if something needs here to be corrected

This would probably be considered a breaking change so would have to be considered on the master branch.

Hi @taylorotwell. Yes sorry the appends really break and I experienced it now in my local. I should have use setAppends() instead, and I achieve the behavior I intended. Do you think it is good if we change this to setAppends() or it is still possible that it will be a breaking change? For the meantime, I just override the method newInstance in my models and add $model->setAppends($this->appends) there.

Hi @taylorotwell. Currently, there are two ways to override the default appends.

Create another model class extending to the class to override the $appends property.

class UserWithAppends extends App\User {
  protected $appends = [ 'full_name', 'age' ];
}

Call setAppends on retrieved models

public function index() {
  return User::get()->map(function($user) {
    return $user->setAppends([ 'full_name', 'age' ]);
  });
}

If we call the setAppends() in the Illuminate\Database\Eloquent\Model::newInstance(), we can achieve it like this:
(new User)->append(['full_name', 'age'])->get();

what are your thoughts on this?


Should we do separate repositories for the different APIs?

@e-lo asked:
Where does API / Examples / Gists live?
Preferably in separate repositories, tagged with OMX version #.  Currently the APIs of all code are in one place and move forward in commits together. I personally find this messy.
What do you all think about having:
•	OMX.r
•	OMX.py
•	OMX.cpp
•	OMX.java
•	etc...
all as separate repositories?  There are good and bad things about this, but the main issue i see is one of the APIs moving forward w/out volunteers to move forward the other APIs and ended up with stale parts of a repo.  The major disadvantage I see is consolidated issue tracking/version control.

I'm fine (at least for now) with everything in one repo since we don't have a lot of activity.

@bstabler @e-lo @bstabler @toliwaga
I think it's time to split these API's into separate repos. I used to think one repo was fine since it's low-volume, but I've changed my mind. Reasons:

I want to tag release versions with meaningful and simple labels, without having to resort to using labels like "python-0.3.2"
Each language API should move forward without affecting the other languages.
I only want notifications for the file format itself and for Python API bugs/issues

I can post this to the omx mailing list if that is more appropriate.

yes!

@bstabler  @e-lo
So if there is no objection can I just do this? Here's my proposal:

Have a main repo for the OMX file format and the general wiki
Have separate repos for each language API and related wiki documentation
All of these should live under one "umbrella" org for OMX. I don't think osPlanning makes a lot of sense anymore... and I've already checked and see that http://github.com/omxdata would do nicely. I've grabbed that org name just in case we decide to do this.
Notify the omx mailing list that we're moving things around.

What do you think? I can do the pruning on each repo and I think it's pretty easy to transfer the wiki contents as well.

Well, I'm not sure.  For VisionEval, we started to put everything into separate repos, but that made it difficult for everyone (RSG, FHWA, ODOT, OSA, and Volpe) to keep track of the overall effort.  So I just recently moved everything back into one repo.  I'm concerned about the same issue here.  How about we discuss? I agree with a lot of what you want to do, so maybe we just need to talk through a plan?

I think having it in the same namespace, github/omxdata, mitigates that somewhat and you can maintain a Jekyll landing page in the same place [ which for some people is the only place they will really need to go ]

@billyc and I chatted and here's the plan:

Keep this as the main repo for the OMX file format and the general wiki
Have separate repos under osPlanning for each language API and related wiki documentation
Consider a future organizational home for this project, such as ActivitySim, Zephyr, or whatever else may emerge in the near future

Python code and Python API docs moved to https://github.com/osPlanning/omx-python -- and the sidebar there still links to the main OMX home page.
I can split the other API's off as well, or the primary maintainers can...? Probably makes sense for me to do it all, right @bstabler ?

Yep

Done.

  I think we're done with this task.


[FEAT] 3D movement of image on landing page.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
The image should be having some moments to looks like 3d.
Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.

Can you explain in detail or show it?


Notify if config breaks the code

I uploaded a config with the following:
todo:
  keyword: ["@todo", "TODO", "dump("]

I believe that the ( broke the regex, since it was not escaped, but it took me a few commits and some time to figure it out.
I think it would be useful if it could open an issue / write a comment in the PR, if the config somehow breaks the todo app.

Thanks for opening this @zoispag, and I'm sorry about the super long delay. This is a larger problem in Probot - here's one discussion, but there have been a few. Since this is outside of the scope of TODO and better left to Probot to handle, I'm going to close this issue - but it's still on my radar for improvements to the framework. Cheers!


Alt or Meta-hjkl work in neovim but not in vim with same config

I have this in my .tmux.conf file
is_vim="ps -o state= -o comm= -t '#{pane_tty}' | grep -iqE '^[^TXZ ]+ +(\\S+\\/)?g?(view|n?vim?x?)(diff)?$'"
bind-key -n 'M-h' if-shell "$is_vim" 'send-keys M-h' 'select-pane -L'
bind-key -n 'M-j' if-shell "$is_vim" 'send-keys M-j' 'select-pane -D'
bind-key -n 'M-k' if-shell "$is_vim" 'send-keys M-k' 'select-pane -U'
bind-key -n 'M-l' if-shell "$is_vim" 'send-keys M-l' 'select-pane -R'

and in my .vimrc I have
let g:tmux_navigator_no_mappings = 1
nnoremap <M-h> :TmuxNavigateLeft<cr>
nnoremap <M-j> :TmuxNavigateDown<cr>
nnoremap <M-k> :TmuxNavigateUp<cr>
nnoremap <M-l> :TmuxNavigateRight<cr>

This works fine when I am using Neovim but doesn't work with Vim. If I change the alt/meta key with control key then both work. I am trying to have the same configs work with both vim and neovim and want to use alt key since Ctrl-h, j, l do different stuff in bash terminal.
Any suggestions about how to make vim work with alt keys?

Hi @kvngvikram, can you clarify what "doesn't work with Vim" means? Are you still able to navigate in tmux, but not vim? Can you move within vim, but not out of vim and back to tmux? Is there any error message in Vim? Does mappy anything else with the meta key in vim, e.g. if you set nnoremap <M-r> :echo "works!"<cr> and then press <M-r> combination, does vim echo "works!"? Similar question for neovim.

I opened Vim in a terminal and not in tmux and tried nnoremap <M-r> :echo "works!"<cr>. It doesn't work. There are no error messages, no change. Instead of <M-r> if I use <C-r> or <Leader>r it works. So there is a problem with alt mapping in Vim alone and nothing to do with tmux.  Whereas Neovim can work with the same <M-r> mapping.
Other than that, tmux works fine without any problem, I can move in to a vim pane. I cannot get out of the Vim pane or move inside the Vim splits when I am using <M- mapping. I don't see any errors.
So I think my issue has noting to do with this plugin.
It would help if anyone can tell me why is <M- mapping not working. Thanks for your reply.

Set the maps from an independent vim session (i.e. that isn't nested within a tmux session) and insert the literal control characters using <C-v>. See :h i_CTRL-V if you're unfamiliar. On my setup, this produces <M-h> et. al. from a tmux nested vim session, but ^[h otherwise.
tmux next-3.3
VIM - Vi IMproved 8.2 Huge Included patches: 1-716
NVIM v0.5.0-dev+1105-gfb2adadc9

@pkfm
Thanks for the suggestion. Now I tried to map using literal character. In terminal outside tmux (byobu) Vim is working with <M-j> and other keys as well. One problem is that the literal character looks like ^[j and the mapping works for <Esc>j as well. Vim looks the both as same similar to <C-j> and <CR>. Anyway when I try switching between splits outside tmux, Alt/Meta keys work as expected and also Esc as well. But when I try in tmux, only Esc mappings work, Meta/Alt mappings do not work.
So no new improvement.


ARM64 - hash-table replace is very expensive on arm64

@Alan-Jowett, can you ask the customer that is requesting this and we can discuss this the following week with that information.

ARM v8.0 is required.
So the proposed solution is to do CPU check at runtime and fallback to functions that are compatible with hardware as needed. This is what the current behavior in PR #3838 .


Fix timestamps in the frontend UI, make it work on OS X

Fix timestamp rendering in the UI
Add changes to start/stop scripts to support OS X
Add log directory contents to .gitignore

Looks Great. Thanks for your contribution!


decouple find InsertValuesSegment with insert statement

For #4342.

Codecov Report

Merging #4360 into master will not change coverage.
The diff coverage is 83.33%.

@@            Coverage Diff            @@
##             master    #4360   +/-   ##
=========================================
  Coverage     53.88%   53.88%
  Complexity      351      351
=========================================
  Files          1047     1047
  Lines         19288    19288
  Branches       3550     3550
=========================================
  Hits          10393    10393
  Misses         8238     8238
  Partials        657      657

Impacted Files
Coverage Δ
Complexity Δ

...nerator/impl/ShadowInsertValuesTokenGenerator.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rator/impl/ShardingInsertValuesTokenGenerator.java
86.95% <100%> (ø)
1 <0> (ø)
:arrow_down:

...keygen/GeneratedKeyInsertValuesTokenGenerator.java
90% <100%> (ø)
1 <0> (ø)
:arrow_down:

...erator/impl/EncryptInsertValuesTokenGenerator.java
75.34% <100%> (ø)
1 <0> (ø)
:arrow_down:

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 00f0db6...00e1a56. Read the comment docs.

Pull Request Test Coverage Report for Build 9641

5 of 6 (83.33%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 57.341%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shadow-core/shadow-core-rewrite/src/main/java/org/apache/shardingsphere/shadow/rewrite/token/generator/impl/ShadowInsertValuesTokenGenerator.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

Totals

Change from base Build 878:
-0.005%

Covered Lines:
11060

Relevant Lines:
19288

💛  - Coveralls


⚠ Fetchers has degraded performance

In b7eb3fc, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 501 ms

Resolved: Fetchers performance has improved in bab79a2.


Backend initialization required when using terragrunt_validate prehook

Describe the bug
Here is my precommit config:

When using this docker command;
docker run --rm -v $(pwd):/lint -w /lint ghcr.io/antonbabenko/pre-commit-terraform:latest run -a
I get a lot of this error:

Everything works fine when I'm executing:
terragrunt run-all validate
Any idea why ?

Hook looks fine
https://github.com/antonbabenko/pre-commit-terraform/blob/ae5cbe0342617310b0491146d51573f2f4c93705/hooks/terragrunt_validate.sh#L60
Theoretically, you could need to provide AWS credentials inside the container by volume or environment variables.
If you need any future guidance, please provide details specified in the issue template, which you ignored. And prefer to use text over images whenever possible.


[Event-processor-host] Missing readme headers

As part of readme verification, we would want all headers to have required sections. The package event-processor-host is missing the following sections:
^Getting started$
^Key concepts$
^Troubleshooting$
^Next steps$
^Contributing$
Refer issue : #1583
The packages should have following headers:
Azure .+ client library for JS
^Getting started$
^Key concepts$
^Examples$
^Troubleshooting$
^Next steps$
^Contributing$

This is an older package that is now deprecated, so we won't be making any updates here


Provide build script

for installing dependencies.

Is it mostly applying https://github.com/wkearn/RasterIO.jl/pull/11 to this repo, or would that not suffice?

Yup, applying what we did for RasterIO to this repo. This repo will then go into RasterIO's REQUIRE file.


UserIdentity without Migration

After you put something wrong in the register page, the view changes. Please make sure it allways follows the concept

After registering this comes up. Please remove the whole register conformation logic or implement it (would do the first because it can be hard todo and is not needed for our usecase)

Username wants email??? Please decide if you login with email or username (best case the unique value)

Size of the container seems to change if there is a wrong value. Please fix this

To many values, specially the ones which are duplcates. What is the diffrent between Name and UserName for example?
Is the UserId supposed to be a varChar? In the concept its a INT

Why is there a username which is my email when there is a name?
Please make sure its like the concept. Thx
P.S: One thing you did, which is much better than in the concept, is that you only store the password hash. If this works its much better than storing the clear password and I would like to encourage you keeping the password hash instead of the cleartext password of the concept (if it works)

Container gets longer if you put in wrong credentials, this souldnt happen

After login:

Maybe following contains the fix:
https://stackoverflow.com/questions/46761397/invalidoperationexception-cannot-create-a-dbset-for-role-because-this-type-is


Lifecycle: Add new systemd targets

Add the systemd targets: focused, unfocused and lazy.
These targets will later be used to integrate the GDP-lifecycle.
Signed-off-by: Viktor Sjölind<EMAIL_ADDRESS>

As we discussed in person, we'll create a PR once we actually have implementation of lifecycle that does something.

I agree with @zeenix. Let's apply the PR wih real implemention.


Fix grammars to support OCaml

Removes "Command not configured for OCaml!" issue, since the grammars had a typo that made it not recognize OCaml files.

Hello, @markbao!
Thanks for the PR!


Implement WaitForNodes measurement

One of the use-cases for this is to allow running simple ClusterAutoscaler tests.

/assign @wojtek-t

PTAL

/lgtm


Add scheduler AND SciML tests

There is some test coverage for the SciML operations but there is currently zero coverage for the service-side


Upgrade gRPC to the version 2.43.

Compatibility with netStandard 2.0 is possible.
Change Dapr.Client from .netCore3.1 to netStandard 2.0 and 2.1.
Allow Dapr.Client to be used with .NET 4.6.1 and greater!
Description
The current version of Grp.Client was not compatible with netStandard 2.0. This limit the usage of DaprClient to .netCore3.1 and greater. But the latest version of gRPC.Client is able to run on netStandard2.0. Allowing Dapr.Client to run also on .NET 4.6.1 and greater. This is an added value to bring Dapr in combination with MAN Sidekick project.
The Dapr.Client now target frameworks netStandard2.0 and netStandard2.1.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:
No new unit tests were needed and they pass with sucess.

@GFlisch - Thanks for submitting this issue!
Unfortunately, based on current feedback and trends, we don't have any plans to support netstandard2.0/2.1.
Out of curiosity, what is blocking you here?

Hi,
We have a lot of legacy app in .Net 4.6.1. We will use the nuget package Man.Dapr.Sidekick to let us integrate our legacy apps with our new ones.
I need homogeneity when communicating with bindings and pubsub. The other scenario to integrate old applications doesn’t interest me.
With a Dapr.Client built for netStandard2.0 I can do this.
Another question. Why do you promote Man.Dapr.Sidekick in your community call if this is not to use it in a write scenario?
Thanks,
GIlles
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: Monday, February 28, 2022 10:39 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [dapr/dotnet-sdk] Upgrade gRPC to the version 2.43. (PR #844)
@GFlischhttps://github.com/GFlisch - Thanks for submitting this issue!
Unfortunately, based on current feedback and trends, we don't have any plans to support netstandard2.0/2.1.
Out of curiosity, what is blocking you here?
—
Reply to this email directly, view it on GitHubhttps://github.com/dapr/dotnet-sdk/pull/844#issuecomment-1054690109, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABNE2KNQDDCEO6LKD55PZPDU5PTPZANCNFSM5PQNPBBQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

I'm going to close this, as we've updated past 2.43 by now and are maintaining our course to not support netstandard2.0/2.1.