[Bug]: a new filetype.json should not be created when cancel it.

Make sure you have done the following

[X] I have updated to the latest version of the plugin.
[X] I have read the docs, and in particular the FAQ.
[X] The problem is related to creating or editing snippets. (Expanding and using snippet is the responsibility of the snippet engine plugin, such as Luasnip or nvim-snippets.)

Bug Description
a new json will be created when I cancel creating snippet,this should be avoided because a new file with empty content is useless.
Also, an option to remove the json file automatically when the file contains no snippet would be helpful.
Relevant Screenshot
No response
To Reproduce
go to a new filetype ,create a snippet and cancel, an empty json file will be created.
neovim version
NVIM v0.11.0-dev-1043+ge4a74e986
Build type: RelWithDebInfo
LuaJIT 2.1.1727870382
Run "nvim -V1 -v" for more info

Technically, this is not a bug, since bootstrapping a new snippet file if there is currently none is the intended behavior.
I am not entirely sure whether the—in my view very rare—edge case of "created new snippet for filetype that has no snippet yet, but changed my mind midway" justifies adding an extra option to disable automatic bootstrapping. In the very rare case this happens, you are still able to manually delete the file (and its reference in the package.json).


How to do Integration Testing with FastEndpoints and .Net 8 WebApplicationBuilder?

I have a .net 8 minimal API setup:
var builder = WebApplication.CreateBuilder(args);
var app = builder.ConfigureApp();
app.Run();

// Reference for Testing Purposes of FastEndpoints
namespace BeeHive.Api
{
    public abstract class Program;
}

public static WebApplication ConfigureApp(this WebApplicationBuilder builder)
{
    builder = builder
        .ConfigureConfiguration()
        .ConfigureLogging()
        .ConfigureSwagger()
        .ConfigureDatabase()
        .ConfigureApi();

    var app = builder.Build();

    app.UseSwaggerConfig()
        .UseApi()
        .UseDatabase();

    return app;
}

Now I use the FastEnpoinds:
https://fast-endpoints.com/docs/integration-unit-testing#service-registration
With that I have a simple setup of my integration test:
public class PingPongEndpointTest(BeeHiveApiFixture app) : TestBase<BeeHiveApiFixture>
{
    [Fact]
    public async Task PingPongEndpoint_ExpectedBehaviour()
    {
        var (httpResponse, dataResponse) = await app.Client.POSTAsync<PingPongEndpoint, PingPongEndpointRequest, PingPongEndpointResponse>(
            new()
            {
                Text = "Test"
            });

        httpResponse.StatusCode.Should().Be(HttpStatusCode.OK);
        dataResponse.ReversedText.Should().Be("tseT");
    }
}

public class BeeHiveApiFixture : AppFixture<Program>
{
    protected override void ConfigureApp(IWebHostBuilder a)
    {
        // Doesnt overwrite the WebApplicationBuilder
    }
}

Now my goal is to now use the AddDbContext from my ConfigureDatabase, because this shows to a real database instead I want to overwrite this with a InMemory Database.
Normaly in the AppFixture I would like to overwrite this behauviour. But with being and IWebHostBuilder in the override from FastEndpoints and a WebApplicationBuilder in my setup this isnt overrideable (both get's called).
How to do this?

this is how you'd typically replace the db context for tests:
public class BeeHiveApiFixture : AppFixture<Program>
{
    protected override void ConfigureServices(IServiceCollection services)
    {
        // remove the real db context configuration
        var descriptor = services.SingleOrDefault(d => d.ServiceType == typeof(DbContextOptions<MyDbContext>));
        if (descriptor != null)
            services.Remove(descriptor);

        //add a test db context
        services.AddDbContext<MyDbContext>(o => o.UseInMemoryDatabase("TestDB"));
    }
}

if that doesn't work, can you attach a standalone/debuggable repro project which i can try to fix up for you?
thanks!

ill consider this issue sorted since you accepted the answer on stackoverflow.
feel free to reopen in case you need further help.
thanks!

If anyone arrives here like I did struggling to get an InMemoryDB to work I had to remove a few different services on top of the ones described in the solution above. I'm not 100% sure why but here is the code I used
`    protected override void ConfigureServices(IServiceCollection services)
{
var descriptor = services.SingleOrDefault(d => d.ServiceType == typeof(DbContextOptions));
if (descriptor != null)
services.Remove(descriptor);
    var descriptor2 = services.SingleOrDefault(d => d.ServiceType == typeof(AppDbContext));
    if (descriptor2 != null)
        services.Remove(descriptor2);

    var descriptor3 = services.SingleOrDefault(d => d.ServiceType == typeof( IDbContextOptionsConfiguration<AppDbContext>));
    if (descriptor3 != null)
        services.Remove(descriptor3);

    services.AddDbContext<AppDbContext>(x => x.UseInMemoryDatabase("TestDb"));
}`

@MarkTallentire
last i checked, just removing DbContextOptions worked fine.
i don't use entity framework regularly, so i'm not sure what's going on there.


Possibility to run all modes over multiple structures

Must be possible to run all modes over multiple structures. For MD simulations, trajectories with each label should be saved. If no labels exist in header, add arbitrary numbering based on index. Must also be possible for all modes to run over specific indices, e.g. specify 'structure index: i' as a mode variable. Default for traj files must be -1.
It would be neat if it was possible to specify:
structure index: 2 4 5 6
and have the program loop over just those indices in the input file.

Traj file should be saved for current running structure. Last finished structure minimisation should be appended to an xyz or pdb file.
It would be good if derivative of last 10% of points could be saved in the log file. This would allow one to asses whether or not a structure could be considered to have relaxed. The absolute difference between fmax and average of fmax could also be included here for extra clarity.

Traj file should be saved for current running structure when running a minimisation. Last finished structure minimisation should be appended to an xyz or pdb file.
It would be good if the derivative over the last 5%, 10% and 15% of points could be saved in the log file. This would allow one to assess whether or not a structure could be considered to have relaxed. This is a good measure of the relaxation state of the system. This should be the main metric to evaluate.
The absolute difference between fmax and average force over the last 10% of structures could also be included here for extra clarity. However, this will tell us very little if fmax is set too low. This metric is only useful if we want to know how close we are to this criteria being met. This is more useful if structure is sensitive to optimisation. This metric can be used for fine tuning if the user knows the system well.

The most efficient way would be to have a list and append the last value to it. If the length exceeds e.g. 15%, remove first index and append. Averages and derivatives could then be calculated on different portions of this list. One must keep track of indices at each coordinate dump sop that derivatives are evaluated over steps, and not dump interval.

Currently underway. Both SP and EMIN are currently working with multiple structures.

What remains is the ability to select indices if starting from traj file and the obtion of selecting certain structures based on indices in the input files.
It would be optimal if the same index variable could be used for both cases.

This has now been implemented and should work without problems.


questions about MOIRAI implementation

I noticed that the official implementation of moirai(or uni2ts) uses mixture distribution as the model output, but in this implementation I don't see the similar part. I am not sure if the code was wrong or I was my wrong.
again, I want to thank the author group for their contribution!

Since it is very difficult to unify the training framework, we have omitted some specific designs of the models. We added this note :)


⚠ Bofrost.de has degraded performance

In bff643a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1186 ms

Resolved: Bofrost.de performance has improved in 47280ca after 13 minutes.


Black screen on iPhone 7

Hi, how to get it running on iPhone 7?

Can you please check whether these online examples work on an iPhone 7 (I do not have access to one):

https://undecaf.github.io/zbar-wasm/example/
https://undecaf.github.io/barcode-detector-polyfill/example-bundled/
https://undecaf.github.io/barcode-detector-polyfill/example-loaded/
https://undecaf.github.io/vue-barcode-scanner/example/

No activity, therefore closing


Make ZPL Orientation configurable

Since the  merge of https://github.com/apple/cups/commit/29500e14c7c977ac56aa520be45092ef8ea766fe the orientation on zebra label printer is rotated by 180 degree.
   /*
    * Rotate 180 degrees so that the top of the label/page is at the
* leading edge...
*/

puts("^POI");

from the ZPL manual:
^POI - Print Orientation Inverted
^PON - Print Orientation normal
some years ago I created the merge request https://github.com/apple/cups/pull/4925
but the merge was canceld.
After some years later I am still fighting against this rotated labels and could not solve all problems.
the idea to change the orientation via -o orientation-requested=6 works mostly, but we are not in a perfect world.
Label-designer and Label-Developer are doing crazy stuff from time to time.
please extend the rastertolabel filter and the drv-file.
This would be very usefull for a lot of user.
regards
Franz

The point of this change was to make the Zebra output consistent with all other label printers and IPP in general.

Hi Michael,
why did you close the isssue with wontfix.
I understand your Idea to keep the output consistent and I like this idea.
But we do not always live in a perfect world and often we have to deal with real world issues.
When this option would we merged a lot of real world problem would be fixed.
When Zebra printers are used in a mixed environement  with raw zpl data stream and cups both datastreams disturbe each other.
When you are dealing with enterprise application like SAP it is not always possible to manipulate the datastream and orientation.
So a configuration options could be very helpfull and when the default is configured with ^POI nobody will have a problem with this and in special cases the administrator is able to fix this kind of orientation issues.
I still strongly hope you will care of this and implement the configuration option!
best regards
Franz

We’ve been down this road before about why the change was done. As for why this is “wontfix”:

CUPS drivers are deprecated, and LPrint is the “official” replacement printer application for Zebra label printer which offers a default orientation option you can configure
The proper way to get the orientation you want is through the orientation-requested option - set the default value or specify the value (6 == reverse-portrait) in the print command.
if you are sending raw ZPL from an application, it is responsible for setting the orientation.

Hello Michael,
I am back again ...

You deprecate the the CUPS drivers, and LPrint the "official" replacement, but I have never seen such a application. I am still working with CUPS as a central Spoolserver for a lot of server applications like SAP ERP. Did you talk to SAP about LPrint? Is there a usefull documentation about this? Are there real live examples?
I set the default orientation via "lpadmin -p  -o orientation-requested-default=6. yes, this is working frequently, but not always, cause of formular developer. And is there a interface to show this settings. I extended the template to display this, but there have to be a standard. To manage this via web this is also not possible.
You, did not understand. In my scenario, the ZPL-printer is driven from two printserver. And when you have some experience with such thermotransfer printers, you know this devices are acting different to laserprinter. In a perfect World you are right, but the world is different.

In the end I do not understand why you are so strong against this extension to the ZPL. Modify the drv-file, so ROI is the default and nothing changes. In the cornercases admins are able to handle the issues.
In the end this is only a extension of 10 to 15 lines of code and this would be helpfull for a lot of use cases.


Fix for catch worker no usable ball loop

If we run out of pokeballs while in the catch worker, we were getting into an endless loop.
Now returning an error to stop the process.
Closes #4533

Wrong again.


Error message patterns improvemenet

To resolve issue #221

Coverage remained the same at 95.148% when pulling ea5ee06fa36eb148396bc36d27c49c5f2e5a714b on issue-221 into 0921f69ac4d059766d3e958ea91acc63e16fb7fa on master.


error: use of undeclared trait name hash::Writer

Experiencing the following exception building with today's nightly:
rust-sdl2-8facfb557b6c1670/a36627dff1069e5ca6098dd3a39fbd6039794097/src/sdl2/keycode.rs:246:24: 246:36 error: use of undeclared trait name `hash::Writer`
rust-sdl2-8facfb557b6c1670/a36627dff1069e5ca6098dd3a39fbd6039794097/src/sdl2/scancode.rs:252 impl<S: hash::Hasher + hash::Writer> Hash<S> for ScanCode {

I've just updated to nightlys, and I'm having trouble reproducing.

Think this is fixed now.


Review GH repo secrets

let's create some service accounts

I think security-wise we're ok. we can punt on creating service accounts for now


Pol 206 new resource table 1

Description
New Resource Table -1
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

Danger has errored
[!] Invalid Dangerfile file: cost/azure/instances_log_analitics_utilization/azure_instance_log_analytics_utilization.pt:505: syntax error, unexpected keyword_end, expecting end-of-input. Updating the Danger gem might fix the issue. Your Danger version: 6.0.9, latest Danger version: 6.3.1
 #  from Dangerfile:99
 #  -------------------------------------------
 #  has_app_changes.each do |file|
 >    pp.parse(file)
 #    category = pp.parsed_category
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger

@SheikSena   see the Danger error, there is a problem with a file.  Looks like there could be a syntax error.

@cdwilhelm As of now, I'm displaying all the fields in the same order which were in the previous detail template block. And whenever "ID" is not required I'm hiding it. On priority, I'm working on AWS Unattached volumes after completing Once again I'll review all the policies and make all the necessary suggestions as suggested by you. Thanks

@SheikSena  After talking to the UI developers, we will not use the format hidden, only right, left, center.   If no format is found the browser default is used.

@cdwilhelm As per the PR #461, u have added format hidden for the volume id field. In order to hide the "ID" field, what should I use?. if we don't have the option to hide the "ID" field shall I make it as a last column in the incident table?. Thanks

@SheikSena   there isn't a way to hide the fields.  I'll update the docs and examples.  It can be the last field in the list.

@SheikSena  can you resolve the conflicts on this PR?


Error On Logdna Rust Agent(3.4)  Startup

Hi,
Each time I startup the logdna Agent I get the following ERROR at startup:
[2022-05-11T08:39:28Z INFO  logdna_agent] Received SIGTERM signal, shutting down
[2022-05-11T08:39:28Z INFO  logdna_agent] running version: 3.4.0
[2022-05-11T08:39:28Z ERROR config::raw] error encountered loading configuration: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" })
[2022-05-11T08:39:28Z ERROR config::raw] error encountered loading configuration: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" })
[2022-05-11T08:39:28Z INFO  config] using settings defined in env variables and command line options
[2022-05-11T08:39:28Z INFO  config] starting with the following options:
    ---
    http:
      host: logs.logdna.com
      endpoint: /logs/agent
      use_ssl: true
      timeout: 10000
      use_compression: true
      gzip_level: 2
      ingestion_key: REDACTED
      params:

I have the /etc/logdna.env file setup for the configuration, so I'm not sure as to why I get the error: ERROR config::raw] error encountered loading configuration: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" })
The agent starts up properly, reads in my configuration in /etc/logdna.env, and runs fine, but I can't see the root cause of the first ERROR message.
Am I missing a config file that gives rise to this error message in the log?

there was a fix in master recently related to that:
https://github.com/logdna/logdna-agent-v2/commit/1c59760e972c6f5d2d7fd9f21341d872d5788fa2

Thank you for the Reply.


Fatal Error: gpd_ros/GraspConfig.h: No such file or directory

Hello,

System: Ubuntu 18.04
Successfully Installed and tested gpd library.

Getting following error:
fatal error: gpd_ros/GraspConfig.h: No such file or directory
#include <gpd_ros/GraspConfig.h>
Is the file missing in the actual repository?
I tried running catkin_make multiple times as you have mentioned in #3

@shrirangsp I have the same issue, did you found the solution that you closed it?

I have the same error. Did you find a solution?


MacOS main menu bar activation fix.

On MacOS 10.14.6, after implementing a quit menu item (seen in pull request #362) and subsequently additional menu items, I found that upon initial App launch the Apple menu and the App's menus in the main menu bar do not respond to mouse clicks. Switching away from the App and then back again resolves the issue.
I found others having the same issue (MacOS 10.15, using Swift) on stack overflow along with a fix: (https://stackoverflow.com/questions/62739862/why-doesnt-activateignoringotherapps-enable-the-menu-bar). Moving the App activation into the function applicationDidFinishLaunching() seems to work great, after this change, the menus are clickable from first launch.

Good find! I thought I had an error in my application and was pulling my hair trying to track it down.

Oh interesting... that's surprisingly late in the initialization process. Not the first time I've seen on Cocoa where changing the order of initialization calls has surprising effects though :)
I'll give it a whirl in the evening and then merge to master (and also integrate the change into the multi-window branch I'm currently working on, which will change the initialization code quite a bit again (but mostly the window-related stuff which is moved out into separate functions).

Awesome, thanks for your time. And thanks for the quick reply.
For thoroughness, I found a few other posts around the web about the same issue. Here's another on stack overflow: (https://stackoverflow.com/questions/33345686/cocoa-application-menu-bar-not-clickable). And in this blog post (https://ar.al/2018/09/17/workaround-for-unclickable-app-menu-bug-with-window.makekeyandorderfront-and-nsapp.activate-on-macos/) the author goes to the trouble of programmatically toggling focus away from the app and back. It sounds like the bug is inconsistent across versions of MacOS, I haven't yet come across any official discussion from Apple.

... into the multi-window branch I'm currently working on...

Exciting stuff!!

Ok merged! I don't have samples with a "proper" macOS menu to test, but it doesn't behave weird either, so I guess it's ok ;)

Hmm, one question: does it matter if the two lines:
    NSApp.activationPolicy = NSApplicationActivationPolicyRegular;
    [NSApp activateIgnoringOtherApps:YES];

...are moved after the call to:
    [_sapp.macos.window makeKeyAndOrderFront:nil];

??
Because in my multiwindow-branch, the makeKeyAndOrderFront call is inside the new platform-create-window function, and without hacks I'd need to move the 'activation stuff' after the call to makeKeyAndOrderFront (otherwise, if the order matter I'll come up with a workaround).

NVM it seems to work. At least I can open the Apple menu when moving the activation code after makeKeyAndOrderFront (I actually never questioned why I can't open the Apple menu lol, but now it works both with your fix on the master branch, and on my wip multiwindow-branch by moving the two activation lines after makeKeyAndOrderFront (or rather: after the platform-create-window call).

Seems to work that way fine for me too... Looks great, thanks!


Decide some Seattle use case for an MVP blog post

The goal is to get a real blog post working end-to-end, integrating A/B Street somehow (maybe with static videos to start, but then running for real). But what particular idea do we want the post to cover? Here are a few ideas. CC @michaelkirk
Open up Broadmoor to people walking and biking

Create a new connection from the Udistrict and Montlake to Madison Park, by allowing foot/bike traffic through the gated community. Using Lake Wash Blvd or the Arboretum trails sucks because of hills and treacherous over-taking on Lake Wash Blvd.
Why would this be a good use case?

Potentially get the businesses in Madison Park onboard with this idea -- easier access means more customers.
Broadmoor does some dodgy stuff with taxes anyway; it's unlikely anything would come of this, but maybe there could political momentum built. https://twitter.com/pushtheneedle/status/1372950949472342021
The lakeslice map runs without gridlock. We'd likely want to clip a slightly different area for this study, but I bet we could get it to run pretty easily.

What would the article cover, or rather, what would it show with abst?

The problem: cycling/walking trips going a roundabout way to get there now.

If we get elevation data working soon, point that out. Climbing up Madison is tough, and there's just a shoulder for that section.
If we start measuring safety soon (specifically over-taking events and cycling/crossing next to high-speed traffic), point that out and compare to values elsewhere
Follow an individual cyclist with cars stacking up behind them on Lake Wash Blvd

An intervention: change the access restrictions through the neighborhood. Today, all traffic can only enter if the origin or destination is in the neighborhood. I propose allowing foot and bike traffic too.

Once we can build new roads, I propose opening a connection directly east/west from Foster Island to MGilvra.

Effects of the change

Individual: follow that same cyclist through the nice, quiet neighborhood instead. See impact on elevation gained, trip time, safety metrics.
Aggregate: Look at the trip table and summaries, compare trip times. I suspect vehicle traffic will also speed up with less bikes on Lake Wash Blvd, because we lack over-taking right now. Also reasonable this could happen in reality; over-taking does slow down cars.
Aggregate: Use the relative throughput layer to show how which roads get more/less car/bike/total traffic

It'd be important to point out the Soundcast 2014 data and the fixed demand data. No mode shift, and no increased demand to go to Madison Park with easier access. Future work.

West Green Lake cycletrack
https://twitter.com/GordonOfSeattle/status/1372290369459654657
Transform the third northbound lane of Aurora to a two-way cycletrack between N 63rd and West Greenlake Dr.
Similar pattern for what the article should use abst for -- show the problem individually and in aggregate, demonstrate the edits, show the effects individually and in aggregate.
This one's harder to start because of how the area is currently represented:

A few of the problems (all fixable with effort, of course):

The path around Green Lake is currently one-way counter-clockwise and foot-only, because of covid. That's not reflected in OSM. If it was marked as no bikes, then we actually wouldn't even import it yet.
There are some ped-actuated signal crosswalks over Aurora near 68th that aren't being imported properly, because of us not handling footpaths yet.
The 3rd NB Aurora lane is currently tagged in OSM as bus-only. We'd need to figure out a hack for transforming this one lane into a two-way cycletrack, ahead of the road-widening work.

After writing these up, they may be actually kind of hard to do. :\

Taking a lane on Fremont and Ballard bridge
https://github.com/a-b-street/abstreet/issues/273 has details. This is a cool idea, but both areas are kind of complicated, and the current import has lots of issues we'd have to figure out first. Also, the separate cyclepaths in OSM aren't associated with the main bridge way, and ideally we would snap them together and treat it as one big abst Road to model changes here.

More ideas at https://github.com/a-b-street/abstreet/discussions/490. I'm less clear on what we could show here short-term. The Northgate bridge would be another awesome "shortcut" use case like the Broadmoor one, but we'd need to first hack in a bit of support for building the new road. This one wouldn't be as hard, because it's in OSM (https://www.openstreetmap.org/way/842065364) and we could hardcode an exception to import it immediately.

To get the ball rolling without getting caught up in more technical work making the other proposals work, we're going with the first idea to start. I already got a map running without gridlock and the proposal made. Initially the results look awesome:

Lots of long walking/biking trips get non-negligibly faster. But already the technical problems creep back in. Looking at a particular faster trip, this was the route before the edits:

Since this trip ends in the private neighborhood, it was allowed before the edits, but it chose the wrong border to route into. I'll fix this and look at the results again.
Also, we'll need to do something better for #85. The relative throughput layer, one of the most interesting, looks like this:

Because of the current thresholds for coloring roads, some of the ones with significant change aren't colored at all, because of how the cut-off works.

Fixed the main routing problem. The other only affects drivers and will be fixed when I bring in fresh OSM with https://www.openstreetmap.org/changeset/101367067. The results continue to look like a net win, and now the example trips are more expected. Before:

After:

There's still some noise in here from things like parking (I might disable parking sim entirely for this map). I'll keep iterating and get the video demos up before Monday.

Alright, here's some unorganized dataviz to start to make the case for this. Along the way, I've of course discovered more bugs and technical gaps, but I'm going to try to not get too distracted, and still generate some initial material.
Everything's on the new arboretum map. Let's start with the commuter patterns -- how many people actually want to go from the Montlakeish area to Madison via bike or walk? Not that many, according to Soundcast:

There's around 30 from the Montlake bridge border and 70 snapped to the 23rd ave border. I may play around with the Soundcast importer and revisit the assumptions about border snapping.
Now let's point out the problem individually... through some manual effort (that I'll make easier by adding more filters to the trip table), I found person 6390 (Ctrl+J and warp to P6390). Around 8:05pm, they bike on Madison towards the trail through the Arboretum, and a few cars stack up behind them, wanting to over-take:

There are a few odd things about their route -- namely, they hop onto 23rd at Boyer. I would personally take the trail all the way to Foster Island and then cut over, but this is how the routing costs are currently doing it. Also, that trail through the Arboretum isn't a great commute option; it's got lots of people jogging. I would take Lake Wash Blvd in reality, because it's faster (but more stressful).
Anyway, fire up edit mode, click one of the pink "access restricted" roads in Broadmoor, and enable walk/bike traffic. The change applies to the whole area automatically. I've baked in a proposal called "broadmoor access" to do this:

Apply it and run the sim again. Following P6390, they choose to go through the neighborhood now -- much quieter:

But what about aggregate patterns? Trip time summaries:

Deselecting cars, you can see longer walk/bike trips had savings:

And opening the throughput layer, then saying compare shows what we'd expect -- more traffic through the neighborhood, less along Lake Wash:

If you mouseover roads, you get exact count comparisons. Currently the colors only show up beyond a certain percentage difference, which I don't think is a good color scheme after all, since it hides so much data.
This is kind of an MVP / first cut attempt for content. Any initial thoughts?

Blog's live, this is finally done! https://a-b-street.github.io/docs//proposals/broadmoor.html


testVM stuck in "Scheduling" Phase #3042

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug
/kind enhancement

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

empty duplicate, sorry


Leak hunt

some attempts at fixing the memory leak

@hoffmang9 I rebased this on top of master now. It passed with all the sanitizers. I suspect this will at least fix the python module, which I understand is what farmers use.

Once CI completes, I'll merge it and tag a beta into the dev-vdf-test branch on blockchain

I'm running 0.13.1b1 - which is the sanitize branch - on one of the farmers I saw leaks on and it's not leaking there - FYI.


Compress Virus Svg

Compress Virus Svg on Landing Page to Optimize the page and load it faster
Remove unused assets from assets folder

closes #42

looks good to merge!


Inconsistent default scrolling on the personal message section on Android that in turn hides the first line of texts, unlike mweb chrome

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging dot on mweb chrome
Go to any workspace and change the name to a longer name
Go to invite members and notice that the personal message texts are shown from the top itself
But now follow the same steps on android and notice that the first line of text is not shown properly since it's automatically scrolled to bottom

Expected Result:
Android and mweb chrome personal message in invite members should have consistent behavior and the first lines of texts should also be shown on android properly
Actual Result:
The First line of text in personal message on invite members page on android scrolls to bottom automatically and the first line of text is not shown properly. But behaves different in mweb chrome
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.94-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/229880245-37e0e802-0155-4c0c-9931-f54942d930a1.mp4

Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680599826934579
View all open jobs on GitHub

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@sonialiap Eep! 4 days overdue now. Issues have feelings too...

Reproducible

Triggered auto assignment to @luacmartins (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Making it external

Job added to Upwork: https://www.upwork.com/jobs/~010511c30c967553b5

Current assignee @sonialiap is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh (External)

Current assignee @luacmartins is eligible for the External assigner, not assigning anyone new.

We got one proposal from @Ollyws, but we have these outstanding questions.

@sonialiap @luacmartins @fedirjh this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@Ollyws have you had a chance to take a look at the questions?

Issue not reproducible during KI retests. (First week)

maybe we can close this one after the 2nd week?

Waiting to see if we can still reproduce this during the next KI retest

Still waiting on KI tests

Waiting on KI

Still waiting on KI

I couldn't reproduce this. Both platforms were consistent and had the cursor set to the end of the text field. Closing.


Hopper Score Amendment #2 (JIRA 536)

This is the first  of 5 improvements to the Hopper score, better dealing with numbers in text fields. It has been tested on internal branch.

Will change the brackets as codacy requested for next time


Change the logo in splash page

Overview (Required)

Currently the title logo in splash page is just DroidKaigi 2016 text.
I want to change it to the logo image as same as web.

I had already added img_logo, but it have white background color and black text.
We need to create new image file with the transparent background and white text.

This is the image on the web.
https://droidkaigi.github.io/2017/images/2017-textlogo.png


When new stable version will be available?

The stable dart sdk is 2.4.0. I would like to know when will be new stable fluttee version be available?

Stable was updated yesterday; closing.
In the future, please use the mailing list or a discussion forum rather than the issue tracker for questions like this.


tests is done for CommandList component

I will test each component individually because it will take time doing all at once
we had "testing" before and this is "testing1"

@Sky-De
This has a conflict with CommandList.js
Please fix this and recommit.  Thanks

@Sky-De
This has a conflict with CommandList.js
Please fix this and recommit. Thanks

sure, but let me know if it is ok with you I remove the commands array from the CommandList component and add above that :

then I can import commands array to test and as a result even if you change command array elements it will not broke test and it would be more dynamic  :

@Sky-De
That would be fine. There may be more commands to add so being able to add commands to the list will help that wont break the tests

@gbowne1 The second commit has been completed and the conflict has been resolved

-[x] checked out this change for testing and review for merge
This change adds tests for the commands
These appears to work as expected
Approving this PR for merge and merging
@Sky-De thanks for the opportunity to review your PR and for your contributions to this project

@gbowne1 You're welcome, and it's my pleasure to assist you. After going through your bio, I must say that I admire you.


[logs] Support UTF-16 encoded files

What does this PR do?
Adds UTF-16 support when tailing from file.
It can be enabled by adding encoding: utf-16-le or encoding: utf-16-be in a log configuration.
Motivation
MSSQL and other windows logs are UTF-16 encoded
Additional Notes
It aims to be as generic as possible.
Describe your test plan
Tail files encoded in UTF-16-LE and UTF-16-BE and ensure there correctly read ideally on windows as it is the main target for this feature.

Doc added https://github.com/DataDog/documentation/pull/8585


Improve documentation about deduplication

Use-case: https://cloud-native.slack.com/archives/CK5RSSC10/p1639588902066100?thread_ts=1639588902.066100&cid=CK5RSSC10
Good question about deduplication and (I think) a good answer as well. Let's create documentation as it's valuable information.
I will update this issue later with better context as I'm on mobile now.

@wiardvanrij I am new to opensource and would like to take up this issue. Can you please guide me in contributing if the issue is still open

@wiardvanrij I am new to opensource and would like to take up this issue. Can you please assign it to me if the issue is still open

Of course! Welcome :)

@wiardvanrij made the pull request. Please provide the feedback - https://github.com/thanos-io/thanos/pull/5004


Add HiveCacheContext into the CacheContext

What changes are proposed in this pull request?
This PR add HiveCacheContext into CacheContext
Why are the changes needed?
HiveCacheContext contains the hive-related information like schema, table, partition, this information later will be used to identify whether the corresponding file needs to be cached and its caching TTL.
Does this PR introduce any user facing changes?
N/A

Codecov Report

Merging #13823 (c7007a2) into master (3f343ec) will decrease coverage by 26.12%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #13823       +/-   ##
=============================================
- Coverage     43.64%   17.52%   -26.13%
+ Complexity     9093     2674     -6419
=============================================
  Files          1386     1387        +1
  Lines         80552    80584       +32
  Branches       9803     9809        +6
=============================================
- Hits          35158    14119    -21039
- Misses        42450    65216    +22766
+ Partials       2944     1249     -1695

Impacted Files
Coverage Δ

...main/java/alluxio/hadoop/LocalCacheFileSystem.java
0.00% <0.00%> (ø)

...rc/main/java/alluxio/client/file/CacheContext.java
0.00% <0.00%> (-48.28%)
:arrow_down:

...ain/java/alluxio/client/hive/HiveCacheContext.java
0.00% <0.00%> (ø)

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 663 more

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 e2571b8...c7007a2. Read the comment docs.

the failing test "alluxio.client.rest.S3ClientRestApiTest.listAllMyBuckets" is unrelated and has been fixed in master branch

alluxio-bot, merge this please


Mac Teams app -can't access settings

If you click on the gear icon on the Mac app, instead of opening "settings" it opens up "manage teams"

Could you please share some screen shots?

Sure!

Shouldn't this open settings, as it does on other versions of Teams, rather than Manage Teams? There are no settings (light/dark mode etc.) within the menu system.

@dkubes You could open Teams settings by clicking on the profile picture and choose Settings:

You will be able to change the light/dark mode from here:


How to use with vuex-persistedstate

Hi,
I'm using vuex-persistedstate and vuex-orm with nuxt.
Also I have some vuex store data outside of vuex-orm created manually (not by database.register).
While vuex store data persisted between page refreshes, vuex-orm data is not. How can I do that?
Many thanks,

I think I should asked the question in [vuex-persistedstate] repo, because it's onlu happening with data created in nuxtServerInit() and not seems to related to vuex-orm.

I reopened case, because non vuex-orm data is persisted even it is created in nuxtServerInit()
nuxtServerInit({ commit }) {
  commit("SET_LOCALE", "xyz"); // This is persisted
  Option.create({ data: sample.options });  // This is not persisted.
}

Any suggestion?

Oh, really...? I though nuxtServerInit would work since persisting data inside fetch works. Maybe it7s because vuex-orm is recreating the module state at client side...?

vuex-persistedstate also overwrites data on client side. To be compatible with nuxt server side rendering, vuex-persistedstate is registered as a nuxt plugin instead of vuex plugin as described here. According to their doc it is related to ordering.


dark icons - small chat changes

[x] name icons iconDark-whatever and run the script and it'll support dark mode

cc: @adamjspooner

@keybase/react-hackers bump


Property strings of extensions aren't translated

Describe the bug
Property strings of extensions are not translated in the help widget on the Properties sidebar of the designer. The same strings are translated correctly in the blocks for those same properties.
Affects

[X] Designer
[ ] Blocks editor
[ ] Projects Explorer
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

I think Designer's Properties sidebar and the Blocks Editor handle translations differently. While the Blocks Editor fetches property strings using a proper translation mechanism, the Designer seems to miss linking the property strings to the correct localization resources.

I think Designer's Properties sidebar and the Blocks Editor handle translations differently. While the Blocks Editor fetches property strings using a proper translation mechanism, the Designer seems to miss linking the property strings to the correct localization resources.

Yes, this is a known issue on my plate to fix. It might not have a github issue, though.


Update readme, update text formatter

Fixes https://github.com/arkadiyt/aws_public_ips/issues/16

Pull Request Test Coverage Report for Build 68

33 of 33 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 65:
0.0%

Covered Lines:
383

Relevant Lines:
383

💛  - Coveralls

Pull Request Test Coverage Report for Build 68

33 of 33 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 65:
0.0%

Covered Lines:
383

Relevant Lines:
383

💛  - Coveralls


Runtime versions

Goal
Introduce runtime version data to the report and session payloads.
Changeset
Added
device.RuntimeVersions
device.GetRuntimeVersions()
device.AddVersion

Changed

Session and report payloads.
Revel example README (fix typo)
Introduced OS Name in report payload as well (already existed in session payload)

Tests
Added some unit tests, and verified that the payloads show up as expected, including the new runtime version data.
Not great
It's unfortunate that we're using a global variable for the runtime versions, but this cached value simplifies things a lot, and is relatively harmless in this package. If/when we decide to upgrade to a v2 we should get rid of all global state anyway.
Linked issues
#121 is the first commit on this branch in order to get a passing build.
Review
For the submitter, initial self-review:

[x] Commented on code changes inline explain the reasoning behind the approach
[x] Reviewed the test cases added for completeness and possible points for discussion
[ ] A changelog entry was added for the goal of this pull request
[x] Check the scope of the changeset - is everything in the diff required for the pull request?
This pull request is ready for:

[x] Initial review of the intended approach, not yet feature complete
[x] Structural review of the classes, functions, and properties modified
[x] Final review

For the pull request reviewer(s), this changeset has been reviewed for:

[ ] Consistency across platforms for structures or concepts added or modified
[ ] Consistency between the changeset and the goal stated above
[ ] Internal consistency with the rest of the library - is there any overlap between existing interfaces and any which have been added?
[ ] Usage friction - is the proposed change in usage cumbersome or complicated?
[ ] Performance and complexity - are there any cases of unexpected O(n^3) when iterating, recursing, flat mapping, etc?
[ ] Concurrency concerns - if components are accessed asynchronously, what issues will arise
[ ] Thoroughness of added tests and any missing edge cases
[ ] Idiomatic use of the language

Good eye @bengourley -- that was a slip. I meant to say runtime versions instead of hostname, but the same applies (already) for hostname.


Automated checks improvement

To prevent issues that occur only when launching game, there should be extra automated check that runs minecraft client and sees if it works correctly.
This is especially prominent with problems in en-us.json file and is importand step to make sure all releases are working correctly.

Running the client might be too much, but at least ensuring the resource file can be read easily. This could be an automated test that is ran.


fix(ffe-icons): update kake-kvart for thinner stroke

Coverage remained the same at 80.028% when pulling c49c745fc530691e39a35c639d22d32a01675dba on update-cake-icon into ccff38e12177e12af68b183495ebc3de8db0bd68 on develop.


Fixed some bash script issues

the 'configure-skeleton.sh' script crashed on Win10 WSL  because it modified itself while running
fixed some legacy code in order to pass shellcheck -- $( do stuff ) instead of do stuff

Thanks!

If you're up to it, you could also send a similar PR to spatie/package-skeleton-laravel 👍


New project fix

Pull Request Test Coverage Report for Build 569235163

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.773%

Totals

Change from base Build 568357542:
0.0%

Covered Lines:
490

Relevant Lines:
640

💛  - Coveralls


calf plug for windows

Could you add the calf plug so us on windows can use these awesome plugin.

This is a little bit promising..

for macos, not windows. but shows we can build on other archs that are not linux.
still no gui, yeah.

closing, decided against gtk2 based plugins


TLSe cannot connect to some servers

TLSe apparently cannot connect to a lot of servers, e.g. Github, when using TLS version 1.2.
You can reproduce this by running https://github.com/eduardsui/tlse/blob/master/examples/tlsclienthello.c and changing google.com to github.com; when passing -DDEBUG, TLSe reports an error with the code 80, which seems to be an "internal error".
Note that I get a different result when using TLS_V13:
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Location: https://github.com/
connection: close

but this doesn't really help me out as well.
I'm on Windows 10, let me know if you need any more info.

On Debian, I get this with TLS_V12:
Initializing dependencies
Message type: 15, length: 2
ALERT MESSAGE
02 50 Consumed -12/7 bytes
ERROR IN CONSUME: -12
CANNOT SWITCH TO kTLS

and still the "moved permanently" thing with TLS_V13.

This issue seems to have been worsened by 3659130c770f95dcdd339c55417f4204407f8b79.
Now I also get the following for google.com on Windows (which worked previously):
Initializing dependencies
Message type: 15, length: 2
ALERT MESSAGE
02 32 Consumed -12/7 bytes
ERROR IN CONSUME: -12
CANNOT SWITCH TO kTLS

└─(21:33:22 on master ✭)──> ./examples/tlsclienthello                                                                                ──(Пт,сен06)─┘
HTTP/1.1 200 OK
Date: Fri, 06 Sep 2024 18:34:58 GMT
Expires: -1
Cache-Control: private, max-age=0
Content-Type: text/html; charset=ISO-8859-1
Content-Security-Policy-Report-Only: object-src 'none';base-uri 'self';script-src 'nonce-m1dOYCoTUAF-A-Mj1bJuiA' 'strict-dynamic' 'report-sample' 'unsafe-eval' 'unsafe-inline' https: http:;report-uri https://csp.withgoogle.com/csp/gws/other-hp
Accept-CH: Sec-CH-Prefers-Color-Scheme
P3P: CP="This is not a P3P policy! See g.co/p3phelp for more info."
Server: gws
X-XSS-Protection: 0
X-Frame-Options: SAMEORIGIN
Set-Cookie: AEC=AVYB7cplj1Fjt665C-QNAAOT3-nyudJYJvOrC125mKwOObaIIg37SvOslg; expires=Wed, 05-Mar-2025 18:34:58 GMT; path=/; domain=.google.com; Secure; HttpOnly; SameSite=lax
Set-Cookie: NID=517=4YEzW2CKcpNptfY5i2JhgiH81o67qJka3K04G1XSQVGEQEc7h-ZGaNMvryrMZ5gyrZJD7c7TiLVHyV4qnEDUTXQ_gdh8RI5qsN2OJMLltIQckbSNb-at-n9IeNTmALeN-g99KUsqKmPDaATvq2XHteKe5xYeTYDEKjcSWgD_nZMdzC13Z6qB8PNWZw; expires=Sat, 08-Mar-2025 18:34:58 GMT; path=/; domain=.google.com; HttpOnly
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
Accept-Ranges: none
Vary: Accept-Encoding
Connection: close
Transfer-Encoding: chunked
...

Personally, I do not consider HTTP 301 Moved Permanently as an error

@Wertzui123 ping

└─(21:33:22 on master ✭)──> ./examples/tlsclienthello                                                                                ──(Пт,сен06)─┘
HTTP/1.1 200 OK
Date: Fri, 06 Sep 2024 18:34:58 GMT
Expires: -1
Cache-Control: private, max-age=0
Content-Type: text/html; charset=ISO-8859-1
Content-Security-Policy-Report-Only: object-src 'none';base-uri 'self';script-src 'nonce-m1dOYCoTUAF-A-Mj1bJuiA' 'strict-dynamic' 'report-sample' 'unsafe-eval' 'unsafe-inline' https: http:;report-uri https://csp.withgoogle.com/csp/gws/other-hp
Accept-CH: Sec-CH-Prefers-Color-Scheme
P3P: CP="This is not a P3P policy! See g.co/p3phelp for more info."
Server: gws
X-XSS-Protection: 0
X-Frame-Options: SAMEORIGIN
Set-Cookie: AEC=AVYB7cplj1Fjt665C-QNAAOT3-nyudJYJvOrC125mKwOObaIIg37SvOslg; expires=Wed, 05-Mar-2025 18:34:58 GMT; path=/; domain=.google.com; Secure; HttpOnly; SameSite=lax
Set-Cookie: NID=517=4YEzW2CKcpNptfY5i2JhgiH81o67qJka3K04G1XSQVGEQEc7h-ZGaNMvryrMZ5gyrZJD7c7TiLVHyV4qnEDUTXQ_gdh8RI5qsN2OJMLltIQckbSNb-at-n9IeNTmALeN-g99KUsqKmPDaATvq2XHteKe5xYeTYDEKjcSWgD_nZMdzC13Z6qB8PNWZw; expires=Sat, 08-Mar-2025 18:34:58 GMT; path=/; domain=.google.com; HttpOnly
Alt-Svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000
Accept-Ranges: none
Vary: Accept-Encoding
Connection: close
Transfer-Encoding: chunked
...

Is this with TLS 1.2 or 1.3?
Actually, I think your PR has nothing to do with the issue, it's just a TLS 1.2/1.3 thing. Sorry for the inconvenience.
Nonetheless, the issue persists. And even if 301 isn't regarded as an error, something is still off, since the address suggested by the 301 is the very address the program originally queried.

So, I'm a little confused right now; these are the current results on Windows 10 for github.com:
with TLS_V12:
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Location: https://github.com/
connection: close

with TLS_V13:
Initializing dependencies
Message type: 15, length: 2
ALERT MESSAGE
02 32 Consumed -12/7 bytes
ERROR IN CONSUME: -12
CANNOT SWITCH TO kTLS

not sure if I mixed something up previously or if it really changed, but that's what I get right now.

So, the moved permanently thing can actually be resolved by adding a Host header like this:
GET / HTTP/1.1\r\nHost: github.com\r\nConnection: close\r\n\r\n

maybe we should add that header to the example (even though Google apparently does not seem to require it)?
I still get the error in V1.3 though.

Is this with TLS 1.2 or 1.3?

According to what I see in wireshark it is for sure TLSv1.2
Pull my develop branch from the fork and build the project up this way:
$ mkdir build
$ cd build
$ cmake ../ -DTLSE_COMPILE_DEFINITIONS="TLS_AMALGAMATION" -DBUILD_EXAMPLES=ON
$ cmake --build . --verbose -j$((nproc))
$ ./examples/tlsclienthello

Also having issue connecting to some servers.
The debug print from tlsclienthello.c with TLS_V12 & TLS_V13 when the sites I tested failed. (all of them print the same thing)
Initializing dependencies
Message type: 15, length: 2
ALERT MESSAGE
02 28 Consumed -12/7 bytes
ERROR IN CONSUME: -12
CANNOT SWITCH TO kTLS

https://bing.com, https://github.com & https://google.com works with TLS_V12, but not TLS_V13.
https://featherless.ai fails with both TLS_V12 & TLS_V13.


Intermittent 404 issue since upgrade from 2.2.1

There are several threads on the support forum, where people are getting intermittent 404 *which are fixed by  downgrading to 2.2.1 without changing the configuration. Examples:

https://community.containo.us/t/v2-https-domain-and-subdomain-error-404/6848
https://community.containo.us/t/v2-intermittent-404-errors-across-our-docker-containers/6853
https://community.containo.us/t/404-even-when-route-exists/6854

Since this is reported by different people within short time frame it is likely to be a genuine issue. No one has given a repro so far, but given intermittent nature of the problem it could be difficult.
I'm leaving it here in case the developers are interested into investigating this a bit more in case there is a genuine issue.

Same here. I checked all browsers (as my Firefox updated about the same time, so suspected that) and the issue happens everywhere.
Sometimes full reload (cmd + shift + r) helps, but the 404 happens very often.
Almost every visit to a page!

I opened a new thread trying to get a repro. Unsuccessful so far: https://community.containo.us/t/consolidated-404-issue-thread-in-versions-since-2-2-1/6865

related to https://github.com/containous/traefik/issues/7020.
I recommend to use v2.2.4

Closed in favor of #7020
Thanks you for reporting.


Add golanglint-ci action

Add golangci lint workflow
applyYamlFile is unused
createOwnerRef is unused
applyOneObject is unused
Drop cfg from suite_test.go
Simplify all the false boolean comparisons
Check return code when calling controllerutil.SetOwnerReference()
Catch error when calling getSubscription()

Tested this PR and got a multicloud gitops reconcile loop to complete successfully


UI improvement UI looks modern

Added box shadow

I hope You Like the Box part @suresh37

Hey @Ayushpanditmoto
Decrease the opacity of the names.
And add a little zoom-in effect on hovering over the names.


Python 3 fix for profiling utils

iterkeys and itervalues don't exist in Python 3.

Should't we use six here? Don't .keys and .values allocate a list rather than create an iterator in 2.7?

In principle that would be better... I'll refix and resubmit.

Sorry, going back on my previous statement if you're creating a list regardless common practise is just to go list(mydict.keys()).  This is what 2to3 will produce.
Any preference @mossblaser (randomly selected adjudicator!)

Yeah; in this specific case you're creating a list anyway so there's not an enormous reason to go to the trouble of carefully using an iterator. There may be some (weak) argument that using the iterator may be faster here anyway since you're going to be creating a list only to iterate over it a single time in Python 2....
Probably "go with your heart" etc...
Incidentally, -0.02% decrease in coverage(!) Tut tut :P ;)


Stubbing out functions

I'm working on a small personal project and I am trying to stub out the call to hostname to always return the same thing (no matter the machine it is executed on).
Basically, in my test file I just redefine hostname to echo what I want, but when I actually run my code with run, that definition isn't seen and calls to hostname end up returning my machine's hostname.
Am I doing this incorrectly, or is this not supported?

Seeing this a while later, this is awesome -- good call @mislav Thank you!

A separate file in your path is not strictly required, @nherson. First, here's the system under test:
#!/bin/bash
hostname

Now, the test that makes your hostname a mock:
hostname() { echo 'mock-hostname'; }

@test 'it does not override the hostname command' {
    run "$BATS_TEST_DIRNAME"/example.bash
    [ 'dev-bishop-frontend2125' = $output ]
}

@test 'it overrides the hostname command' {
    export -f hostname # the secret sauce
    run "$BATS_TEST_DIRNAME"/example.bash
    [ 'mock-hostname' = $output ]
}

If your system under test needs elevated privileges, sudo -E will not pass the exported function (for reasons I have yet to explain). You will need to use su or the separate file method.
If you had a ton of functions you needed to export, you might use declare -pf to get the list of all prior defined functions, then dump those into the startup script (.bash_profile for example).


distillation weights

Thanks for your work! Can you provide the distillation weights, such as resnet18, mobilenet?

Hi. Thanks. You can find pretrained nanosam2 checkpoints here. I also updated the readme.md Those are trained for ~10 epochs so you maybe training them for longer will slightly improve the results even more.


Allow hidding MediaLibraryPickerField in admin list view

All MediaLibraryPickerFields are displayed in admin content list. It's ugly when there are many fields / medias.
It could be a good idea to add setting in ContentPartFieldDefinition to show/hide it in SummaryAdmin display.

FYI - You can do this yourself using a placement.info file in a custom module. I almost always have a module for this so I can customize exactly what's shown in admin lists for each content type.
<Place Fields_MediaLibraryPicker_SummaryAdmin-MyMedia="-"/>

The preferred way to overcome this is by using Placement as mentioned by @joshberry . A config for this could be done but I'd thus consider it as a very low priority improvement.
And if we do it for specifically this field, then what about other fields? How about parts? How would this play together with Placement?

Creating a fake module with a placement.info file is ok if you have one site per installation, and of course if you know how to do it.
I host various customers web sites on one installation with multitenancy, and I don't want to create a new fake module per customer only for SummaryAdmin placement.
@Piedone you should have the same situation on DotNest ?
It would be even better to do something for all parts and fields, like in /Admin/ContentTypes/EditPlacement.

And how would this play together with Placement?

It could override default Placement (defined in Module) for SummaryAdmin DisplayType only.
It could be limited to show/hide, or really define a new placement.

How we can call MediaLibraryPickerField with filter string? for select a type of media.


use closest fixtures

Codecov Report

Merging #11 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   96.26%   96.35%   +0.09%
==========================================
  Files          13       13
  Lines         535      549      +14
  Branches       90       91       +1
==========================================
+ Hits          515      529      +14
  Misses         10       10
  Partials       10       10

Impacted Files
Coverage Δ

django_north/management/commands/migrate.py
96.87% <100%> (+0.15%)
:arrow_up:

django_north/management/commands/flush.py
89.28% <100%> (+0.26%)
:arrow_up:

django_north/management/migrations.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix leak in SqlStageExecution

Task scheduling and stage cancellation race against each other. In
particular, scheduleSourcePartitionedNodes() may take a long time if it
has to wait for free nodes, and a LIMIT query could finish after just
one of those tasks starts running. In this case the stage will be
cancelled while scheduling is still in progress, which will cause tasks
to be leaked, as they're created after the stage already has
transitioned to the CANCELLED state.

looks good


Bug in 0.13 when PUT with ?force=true

Since Marathon 0.13, when I PUT my app using ?force=true
PUT=${PROTOCOLE}://${MARATHON_USER}:${MARATHON_PASSWORD}@${MARATHON_ADDRESS}/v2/apps${ID}?force=true
curl -X PUT -H "Content-Type: application/json" ${PUT}<EMAIL_ADDRESS>
The application get stuck in deployement at RestartApplication.
The previous version doesn't stop.
I am using this method to deploy jenkins over jenkins, could you restore the old behavior ?

It was my slave that hasn't enough resources to start two container in parallel.


Delete board support crates

I've moved the remaining ones into their own repos in this org.
Fixes https://github.com/nrf-rs/nrf-hal/issues/141

LGTM! Thanks a ton for doing the work I was too lazy to do!


Normalize family names

I'm not sure how many special cases you want to add in to Gilda, but there are lots of times when text comes with the suffix "family" like "RAS family" that would be easy to normalize to an appropriate term like fplx:RAS
I'm manually curating PID right now for bio2bel/bio2bel#27 in this spreadsheet that has some examples.

Yes, this could be handled in two ways: either when generating lookup strings, for an input like "RAS family", we could also look up "RAS". Alternatively, * family and * complex could be added to all FamPlex entries as synonyms.

Hmm i think adding family and complex into the synonyms would probably be the simplest and also most informative


Drop use of the Q.allSettled()

The old Q promise library was being used solely for the allSettled() function [1].  This allows multiple async function / promises to run in parallel and collect the full results of each.  Use of the Q library just for this function is no longer necessary.  The standard Promise object contains the same allSettled() function with the same signiture [2].
[1] - https://github.com/kriskowal/q#combination
[2] - https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/allSettled
The libraries were remove by running:
npm remove q -ws packages/legacy
npm remove @types/q -ws packages/legacy

Any changes to other package.json files were done by npm during the remove operation.
Signed-off-by: Scott J Dickerson<EMAIL_ADDRESS>

Nice 👍


Pulled Staging / pomordora. No conflicts

Description
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[x ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

Looks good from what I can tell, lack of testing makes it hard to be sure. I would focus on having everyone do a snapshot test. Some of the colors are out of date with figma and there are some minor margin issues that should be addressed. Other than that all looks good from a visual non braking perspective.


Make Quickstart docs consistent

What type of PR is this:
/area documentation
/kind documentation

What does this PR do / why we need it:
Which issue(s) this PR fixes:

Fixes part of #6143
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[X] Documentation

How to test changes / Special notes to the reviewer:

/cc @feloy @valaparthvi

/approve
Thanks @SD-13 for your help

/override OpenShift-Integration-tests/OpenShift-Integration-tests
Doc-only changes.


Addressing "Neighbor table overflow" issue.

Changing gc_thresh settings to address "Neighbor table overflow" issue. For an example discussing this see this blog post: https://openai.com/blog/scaling-kubernetes-to-2500-nodes/
Values were chosen to match defaults recommended by AKS: https://docs.microsoft.com/en-us/azure/aks/custom-node-configuration#socket-and-network-tuning

Removed the sunrpc tcp_max_slot_table_entries fix because it doesn't survive reboots unless we add a service to modprobe sunrpc. Should we add that?

I have decided to modify the PR to add in a systemd service to start sunrpc at startup just to show what that would look like.


Fix missing quotation mark in README

Coverage increased (+2.3%) to 99.642% when pulling 6e96663ae8a19caccec1f0c37aff5a64c18a12d4 on lidimayra:fix-quotation-mark-in-readme into 89166b03907a36f84d3f87493cd352dce58950a1 on ruby-grape:master.

Coverage increased (+2.3%) to 99.642% when pulling 6e96663ae8a19caccec1f0c37aff5a64c18a12d4 on lidimayra:fix-quotation-mark-in-readme into 89166b03907a36f84d3f87493cd352dce58950a1 on ruby-grape:master.

Coverage increased (+2.3%) to 99.642% when pulling 6e96663ae8a19caccec1f0c37aff5a64c18a12d4 on lidimayra:fix-quotation-mark-in-readme into 89166b03907a36f84d3f87493cd352dce58950a1 on ruby-grape:master.

Coverage increased (+2.3%) to 99.642% when pulling 6e96663ae8a19caccec1f0c37aff5a64c18a12d4 on lidimayra:fix-quotation-mark-in-readme into 89166b03907a36f84d3f87493cd352dce58950a1 on ruby-grape:master.

Coverage increased (+2.3%) to 99.642% when pulling 6e96663ae8a19caccec1f0c37aff5a64c18a12d4 on lidimayra:fix-quotation-mark-in-readme into 89166b03907a36f84d3f87493cd352dce58950a1 on ruby-grape:master.

thanks @lidimayra


Set up mobile pick with detic and contact graspnet

Motivation and Context

Changelog

How Has This Been Tested

Types of changes

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

This is an empty file: src/home_robot/home_robot/mapping/voxel.py


Orthogonal initializer

This PR adds an orthogonal initializer based on the paper Exact solutions to the nonlinear dynamics of learning in deep linear neural networks.

Can one of the admins verify this patch?

ping for @zheng-xq

Or ping for @rmlarsen

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please

The failing tests on Windows emit error messages of the form
11:16:03 ======================================================================
11:16:03 ERROR: testGain (__main__.OrthogonalInitializerTest)
11:16:03 ----------------------------------------------------------------------
11:16:03 Traceback (most recent call last):
11:16:03   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1021, in _do_call
11:16:03     return fn(*args)
11:16:03   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 999, in _run_fn
11:16:03     self._extend_graph()
11:16:03   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1048, in _extend_graph
11:16:03     self._session, graph_def.SerializeToString(), status)
11:16:03   File "C:\Program Files\Anaconda3\lib\contextlib.py", line 66, in __exit__
11:16:03     next(self.gen)
11:16:03   File "C:\Program Files\Anaconda3\lib\site-packages\tensorflow\python\framework\errors_impl.py", line 469, in raise_exception_on_not_ok_status
11:16:03     pywrap_tensorflow.TF_GetCode(status))
11:16:03 tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'Svd' with these attrs.  Registered kernels:
11:16:03   <no registered kernels>
11:16:03
11:16:03 	 [[Node: Svd = Svd[T=DT_FLOAT, compute_uv=true, full_matrices=false](random_uniform)]]

Unfortunately, I'm not quite sure what to make of that.

@mrry Derek, do you understand why this failure occurs on Windows but not Linux or Mac?

Yes, IIRC there was a problem compiling the SVD op, so it was originally excluded from the (CMake) build. I can look into it later, or you could try commenting out this line in this CL and running the tests again....

@mrry Thanks for the explanation, Derek. I think we should probably get the SVD op working on Windows before merging this PR then?

@rmlarsen Sounds good. I'll look at getting the SVD op compiled today.

@tensorflow-jenkins test this please

@tensorflow-jenkins test this please

Our Mac tests are rather flaky at them moment :-1: . I'll keep running the tests until they pass at least once...


fix: cert-authn_plugin: there is the problem, during login;

cert-authn_plugin: there is the issue/bug, during login - authenticator.authenticate with step == 2 is not called;

Resolved, after fixing of navigation rules initializing in Janssen;


Exiting full screen doesn't restore the correct window size

Environment

OS: Windows 10
scrcpy version: 1.13
installation method: Windows release
device model: pixel 3 XL
Android version: 10

I run scrcpy using the following parameters: --always-on-top --turn-screen-off --max-size 800
When I go fullscreen (ctrl+f) then back to windowed, the initial size of the window is lost (ie. the resolution is still correct but the window takes the whole screen).
INFO: scrcpy 1.13 <https://github.com/Genymobile/scrcpy>
<snip>\scrcpy-server: 1 file pushed. 6.3 MB/s (27694 bytes in 0.004s)
[server] INFO: Device: Google Pixel 3 XL (Android 10)
INFO: Renderer: direct3d
WARN: Trilinear filtering disabled (not an OpenGL renderer)
INFO: Initial texture: 800x392
[server] INFO: Device screen turned off

Sceenshot before fullscreen (for scale)

Screenshot after entering then exiting fullscreen:

Thank you for the detailed report.
This bug has been introduced in v1.13 on Windows and is fixed on development branch.
See #1346 (there is a link to Windows binaries in the comments).

No worries, and sorry for the dup' (I didn't use the right keywords to find the other bug).


Issue#75 sa triangle

Hello @lasofivec! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tofu/tests/tests01_geom/test_04_sampling.py:

Line 377:9: E722 do not use bare 'except'
Line 381:5: E303 too many blank lines (2)
Line 418:23: E262 inline comment should start with '# '
Line 511:80: E501 line too long (80 > 79 characters)

Shall we wait for this PR before the deployment of 1.4.11?
It has to be done before 13h today, what do you think?

Hum... the tests for the 1.4.11 are still failing because of openadas, so it looks like you have more time


FSA-5759: Implemented custom pagination for product assignments

Because of our specific implementation of assigned and potential product list, pagination is implemented completely on frontend side. With one call to the backend, complete list of products is fetched and later divided into pages. It's the easiest way to handle second list which is dynamically created in reducer.

Prettier is failing for PaginationHelper.spec.ts


Revert "Fix the repo for CaaSP"

This reverts commit 45dc75ece636f7b64bb34aa0879d6d644694c771.
Fixes: bsc#1039453

You are right, let me find a proper solution

Maybe we can use the create attribute of the managed file:

create
: True
If set to False, then the file will only be managed if the file already exists on the system.

This could be set to False when the os != openSUSE.
Just my 2cents


Generate offline map

@nCastle1 @saratchandrakarumuri
Please review when you can. Thanks!

@nCastle1 - This is ready for another look. I addressed the comments you made and updated the code to clean up previous outputs as best as possible. Thanks.

Thanks @nCastle1
I changed the PR to merge with master and also made the typo fix.


Invoke-RestMethod does not respect error action preference.

Steps to reproduce
Invoke-RestMethod -ErrorAction SilentlyContinue -ErrorVariable RestError -Uri http://nowhere

Expected behavior
Error suppressed for custom error handling. RestError variable is created and error inserted into collection.

Actual behavior
Error thrown in current runspace and passed to RestError

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.0-beta
PSEdition                      Core
GitCommitId                    v6.0.0-beta.5
OS                             Darwin 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I repro'd this on OS X as I'm working on cross-platform support for a project using PowerShell, but this same behavior also occurs on Windows 10, PowerShell v5.1.

The error you're provoking is a statement-terminating one, whereas the common -ErrorAction parameter only applies to non-terminating errors.
To catch a terminating error, you must use either a try / catch or a trap statement.
This dichotomy between non-terminating and terminating errors has always existed, but was never explained well; see Our Error Handling, Ourselves - time to fully understand and properly document PowerShell's error handling.

P.S: What you're looking for is:
$RestError = $null
Try {
  Invoke-RestMethod -Uri http://nowhere
} Catch {
  $RestError = $_
}

Thanks. I remember now that I've come across this before, but was clearly blanking on the terminating errors piece.


stdout is dropped in sbuild

Hello,
Package build in debian needs to patch away the following assertion:
--- rpassword.orig/tests/piped.rs
+++ rpassword/tests/piped.rs
@@ -28,8 +28,4 @@ fn piped_password() {

     let out = out.wait_with_output().unwrap();
     assert!(out.status.success());
-
-    let stdout = String::from_utf8(out.stdout).unwrap();
-    println!("stdout: {}", stdout);
-    assert!(stdout.ends_with("Password: secret\n"));
 }

Arguably, this is perhaps more related to sbuild, but forwarding for awareness.

To be clear, i think this report is about https://bugs.debian.org/975479, where the reported error during the test is:
test piped_password ... FAILED

failures:

---- piped_password stdout ----
stdout:
running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

thread 'piped_password' panicked at 'assertion failed:
stdout.ends_with("Password: secret\n")', tests/piped.rs:34:5
stack backtrace:
   0: std::panicking::begin_panic
             at /usr/src/rustc-1.47.0/library/std/src/panicking.rs:497
   1: piped::piped_password
             at ./tests/piped.rs:34
   2: piped::piped_password::{{closure}}
             at ./tests/piped.rs:8
   3: core::ops::function::FnOnce::call_once
             at /usr/src/rustc-1.47.0/library/core/src/ops/function.rs:227
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

It's not clear to me why this particular patch is the right solution.
@edneville  can you explain what you think is at issue here?  I don't mind commenting out failing tests where there's a good, well-understood reason to do so.  But i don't understand the reason here yet.

As above, stdout is lost (stdout:). I don't currently know a work around for this, I don't think it is a fault in rpassword, but I'm forwarding the commentary mostly for awareness.

What causes the stdout to be lost?  Are you saying that you think this is a flaw in sbuild?  Or the debian build environment? Or somewhere else?
I see this same failure by unpacking the source for rust-rpassword 5.0.0-1 on debian, and then from the new directory, running the standard debian test with all features:
apt source rust-rpassword
cd rust-rpassword-5.0.0
/usr/share/cargo/bin/cargo-auto-test rpassword 5.0.0 --all-targets --all-features

This is not using sbuild.  (the above commandline is derived from the debcargo-generated test suite)
But I also see the same error if i do:
git clone https://github.com/conradkleinespel/rpassword
cd rpassword
git checkout v5.0.0
cargo test --all-targets --all-features

I do not see the errors if i omit --all-targets from the cargo test invocation.
So, it looks like this problem comes up with --all-targets.  I admit that I don't really understand that flag for cargo test, or why it might cause this particular failure.  Maybe this is worth investigating further?
upstream folks (@conradkleinespel?) can you confirm that the tests work for you when using --all-targets ?

I found something odd:
cargo test --all-targets --all-features

$ cargo test --all-targets
    Finished test [unoptimized + debuginfo] target(s) in 0.00s
     Running target/debug/deps/rpassword-d876ea8b112128d8

running 1 test
test mock::tests::can_read_from_redirected_input_many_times ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/no_terminal-80dfd3b50166e775

running 1 test
test can_read_from_redirected_input_many_times ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/piped-18bd2072bbd83245

running 1 test
test piped_password ... FAILED

failures:

---- piped_password stdout ----
stdout:
running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

thread 'piped_password' panicked at 'assertion failed: stdout.ends_with("Password: secret\n")', tests/piped.rs:34:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    piped_password

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

error: test failed, to rerun pass '--test piped'

The test calls an example with pipes to the child.  Let's see what it executes:
$ strace -f -e trace=execve target/debug/deps/piped-18bd2072bbd83245
execve("target/debug/deps/piped-18bd2072bbd83245", ["target/debug/deps/piped-18bd2072"...], 0x7fff57676ef8 /* 49 vars */) = 0

running 1 test
strace: Process 3997473 attached
strace: Process 3997474 attached
[pid 3997474] execve("/home/teythoon/repos/pep/rust/rpassword/target/debug/examples/read-password", ["/home/teythoon/repos/pep/rust/rp"...], 0x7fffaed30158 /* 49 vars */) = 0
[pid 3997474] +++ exited with 0 +++
[pid 3997473] --- SIGCHLD {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=3997474, si_uid=1000, si_status=0, si_utime=0, si_stime=0} ---
test piped_password ... FAILED

failures:
[pid 3997473] +++ exited with 0 +++

---- piped_password stdout ----
stdout:
running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

thread 'piped_password' panicked at 'assertion failed: stdout.ends_with("Password: secret\n")', tests/piped.rs:34:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    piped_password

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

+++ exited with 101 +++

Ok, let's call it and see what happens:
$ /home/teythoon/repos/pep/rust/rpassword/target/debug/examples/read-password

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

That's odd, looking at the example it should prompt for a password:
$ cargo run --example read-password
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target/debug/examples/read-password`

Password: arst
$ /home/teythoon/repos/pep/rust/rpassword/target/debug/examples/read-password

Password: arst

And now it does!
Really weird, maybe a cargo bug, screwing up file descriptors somewhere?

To make it explicit, the test runs target/debug/examples/read-password but cargo miscompiles that for some reason, replacing it with what appears to be a test executable with zero tests.

Here is a diff between running target/debug/deps/piped-18bd2072bbd83245 as part of cargo test vs cargo test --all-targets.
i note that the --all-targets test run dynamically loads libm, though the other test run does not.  i really don't understand this.

looks to me like when i run cargo test cargo produces: ./target/debug/examples/read_password-420acc0b3e81a06f
but when i run cargo test --all-targets, cargo produces: ./target/debug/examples/read_password-43972f448c6487b3 instead.
(i'm running cargo clean in between these tests)

We think that cargo compiles the example in cfg(test).  This is either a bug in cargo, or a feature, in which case this crate shouldn't expect the example to work.
I guess it is a feature in cargo.  Your example can have tests, and you would expect cargo test to run them.
I made an experiment.  I tucked #[test] fn test_in_example() {} into examples/read-password.rs and indeed:
[...]
     Running target/debug/examples/read_password-43972f448c6487b3

running 1 test
test test_in_example ... ok
[...]

So yeah, the test is broken and needs fixing.
As a short-term workaround for packaging, simply remove the tests/piped.rs file.  The patch proposed by @edneville papers over the issue, suggesting that the test is still run, when in fact it is completely gutted.

Hey ! Thanks for looking into this and sorry the late answer. Actually, after reading the code again, it seems that these tests related to pipes were already in tests/tests-unix.sh. So I could just remove tests/piped.rs if that's OK for you guys. Just means that there'd be less testing done on cargo test, but the CI here on Github would run the proper tests before merges.

Hey ! Thanks for looking into this and sorry the late answer. Actually, after reading the code again, it seems that these tests related to pipes were already in tests/tests-unix.sh. So I could just remove tests/piped.rs if that's OK for you guys. Just means that there'd be less testing done on cargo test, but the CI here on Github would run the proper tests before merges.

Or maybe I could let the test from piped.rs run only on Windows and tests/tests-unix.sh would run tests for unix. If I just remove piped.rs, we'd lose Windows testing.

Or maybe I could let the test from piped.rs run only on Windows and tests/tests-unix.sh would run tests for unix. If I just remove piped.rs, we'd lose Windows testing.

Thanks @dkg @teythoon and @edneville for looking into this, your comments helped me understand where to look. I just pushed a fix as 5.0.1 to crates.io. Hopefully this will make the test pass on Debian.

Thanks @dkg @teythoon and @edneville for looking into this, your comments helped me understand where to look. I just pushed a fix as 5.0.1 to crates.io. Hopefully this will make the test pass on Debian.

I finally uploaded 5.0.1 to debian.  sorry for the long delay!

That's great, thanks a lot @dkg 😃


Add alloc-tls: custom thread-local variable implementation

@ezrosent and I have verified that this change does not adversely affect performance. Unfortunately, getting to that state required the addition of the somewhat-unergonomic macro alloc_tls_fast_with - calling the with method directly significantly degraded performance. This probably reflects a missed optimization opportunity that could be reported to the rustc folks.


refactor: Sync volar 0.27.8

@johnsoncodehk
There seems to be an error in CodeLens's setup sugar disable (unuseSetupSugar.ts?).
coc.nvim:
[coc.nvim]: Error on "codeLensAction": Request workspace/executeCommand failed with message: Cannot read property 'startTagEnd' of null

VSCode's volar extension:
[Error - 9:50:26] Request workspace/executeCommand failed.
  Message: Request workspace/executeCommand failed with message: Cannot read property 'startTagEnd' of null
  Code: -32603

Will fix it in 0.27.9, thanks!


"Use the provided code three times. One for each slot" doesn't check as complete.

Challenge http://www.freecodecamp.com/challenges/waypoint-give-your-javascript-slot-machine-some-stylish-images has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
I setup the random images for each slot and it works as intended when I click "Go". The first checkpoint "Use the provided code three times. One for each slot." is never marked as completed. The other three do get checked.

Never mind. It's working for me now. I think maybe the problem was that I was starting with double quotes rather than single ones.

can you tell me hows you achive that......? screen short please!!


fix: localized sitemaps include all routes on nuxt generate

Describe the bug
Running nuxt generate and npx serve .output/public shows for /en-sitemap.xml

Especially does it show / and /en, while only /en should be shown (as it does with nuxt dev).

Maybe it should also not show false, but that's another topic.
Reproduction
https://stackblitz.com/edit/github-segxhh-fy9uaf?file=package.json
System / Nuxt Info
------------------------------
- Operating System: Linux
- Node Version:     v16.20.0
- Nuxt Version:     3.7.1
- CLI Version:      3.8.3
- Nitro Version:    2.6.3
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      modules, i18n
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

You're too fast! 🔥

Thanks for catching this one :+1: Available in 3.2.8
Also I pushed a fix so that it correctly asserts a site url is set if you're prerendering (fixes the fase) issue.


Remove legacy repositories

Different repository types have multiple unique fields. To make them more manageable, we split repository resources by package_type.
We will remove the following legacy resources: artifactory_local_repository, artifactory_remote_repository and artifactory_virtual_repository.
These resources will be replaced by more specific resources, declaring the package type in their name. For example:
artifactory_local_alpine_repository,
artifactory_remote_docker_repository ,
artifactory_virtual_alpine_repository
and so on.
This will be a breaking change and will be released as a major version update.

I don't see artifactory_remote_vcs_repository. So how could I migrate from artifactory_remote_repository with package_type = "vcs" ?

@jcoste-orange Thanks for bringing that to our attention. We will create a new PR for this.

VCS added in v6.0.1


SpotList in home menu needs more information.

Users want to include more information about spots, like tags and other data the spot possess, under the "spots list"  tab, under the home screen.

@nicolasmroberts Can you please provide a wire frame or some way to let us know how exactly this is being visualized to be developed.

Added with commits 8c7a745f69d383238dbad7fc88de31ce7e3bcd58 and 8c7a745f69d383238dbad7fc88de31ce7e3bcd58
In the next version the Spot list will show the same info as SS1:

an icon for the type of geometry
'pages' that have data (text in SS1 but icons here)
applicable tags as subtitles

I can update if there is a different way you want to visualize @nicolasmroberts

@jessicagood The spots list page is looking very good. I created transparent versions of the icons to show what data is within the spots, and uploaded them to the ui design slack channel, and again here. The titles are e.g. Tags_transparent.png, Photo_transparent.png, etc.
[
StraboIcons_v6.zip
](url)


Show the watch / fork count

I didn't find any option to re-enable the watch and fork counts.
Is there any way to show them?

It says it should be hidden "on smaller screens" but it occurs for me on my 2560×1440 screen. I'd also like to have them visible.

Yeah, mine is a 1440p too

https://github.com/refined-github/refined-github/issues/5406 has CSS to re-enable it

Undo the CSS:
.pagehead-actions :is(#repo-network-counter, #repo-notifications-counter) {
	display: inline-block !important;
}

Duplicate of https://github.com/refined-github/refined-github/issues/5406


Feature: TLS

Goals:

[ ] Working tls handshakes
[ ] TLS RSA encryption
[ ] TLS ECC encryption

Currently it just times out without sending anything

Found the possible cause of the timeout: The net.socket is never connected. Seems like .pipe doesn't connect it.

Now I'm getting Error:<PHONE_NUMBER>45696:error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol:../deps/openssl/openssl/ssl/s23_clnt.c:797:

Reference: #54.

Reference: https://github.com/ZeroNetJS/zeronet-js/issues/1.

So far TLS works, but interop fails.

Closing in favor of #88


std.experimental.allocator.make does not respect private class constructors

joeyemmons reported this on 2016-06-14T10:53:32Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=16171
Description
Allocator make with no extra args works on classes even when default construction is not available.

import other;
import std.experimental.allocator;
import std.experimental.allocator.mallocator;
void main(){
	// auto t1 = new test(); // Error
	auto t = Mallocator.instance.make!test(); // Works
}

...

module other;
class test{
	private this(){
	}
}

Worse even is it will still work even if class test does not have a default constructor at all.

module other;
class test{
	private this(int i){
             // No default constructor...
	}
}

b2.temp commented on 2016-06-14T12:23:18Z
(In reply to joeyemmons from comment #0)
>
> Worse even is it will still work even if class test does not have a default
> constructor at all.
>
> module other;
> class test{
> 	private this(int i){
>              // No default constructor...
> 	}
> }

please make another report for this one. This is another issue.

joeyemmons commented on 2016-06-14T13:04:46Z
(In reply to b2.temp from comment #1)
> please make another report for this one. This is another issue.

Issue 16172


refactor(ast): rename the visited_node marker to ast.

closes #4282
I went with #[ast(visit)] but we can change it to 2 separate markers as suggested by @overlookmotel in the issue.

#4289  👈
main

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

Join @rzvxa and the rest of your teammates on  Graphite

@rzvxa Is this stack ready for review? (or at least bottom 3 PRs of it?)

@rzvxa Is this stack ready for review? (or at least bottom 3 PRs of it?) Or you still working on it?

Yes, the bottom 3 are ready to be reviewed.


[Mobile]

Can support to split page?

No

OK, Thanks.  And How to get the length of content with a fix size quill view?


nixt: 2.2.0 -> 2.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nix-community/nixd/releases.
meta.description for nixt is: Supporting library that wraps C++ nix
meta.homepage for nixt is: https://github.com/nix-community/nixd
meta.changelog for nixt is: https://github.com/nix-community/nixd/releases/tag/2.2.1
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.1 in filename of file in /nix/store/ndqqcacsndqd3d1r3xbp8d3xdhd3sdip-nixt-2.2.1

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

nixd
nixf
nixt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ndqqcacsndqd3d1r3xbp8d3xdhd3sdip-nixt-2.2.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nixt https://github.com/r-ryantm/nixpkgs/archive/64a5be0eea20703c52b0849994b7a9bc7f4e66ae.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/64a5be0eea20703c52b0849994b7a9bc7f4e66ae#nixt

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ndqqcacsndqd3d1r3xbp8d3xdhd3sdip-nixt-2.2.1
ls -la /nix/store/ndqqcacsndqd3d1r3xbp8d3xdhd3sdip-nixt-2.2.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

5 packages built:

nixd
nixf
nixf.dev
nixt
nixt.dev

Maintainer pings
cc @inclyc @Ruixi-rebirth @Aleksanaa for testing.

There's a nixf update.

Result of nixpkgs-review pr 320794 run on x86_64-linux 1

5 packages built:

nixd
nixf
nixf.dev
nixt
nixt.dev


brew install command for bosh-cli incorrect

The tracker story shows:

it should say

brew install cloudfoundry/tap/bosh-cli

This commit should fix it: https://github.com/pivotal-cf/onboarding/commit/927aa4e29ae7c6de7ccacc9d426e388fa684da13


setup: configure NAT on Neutron router

Update neutron_router_gateway from upstream

This will need to be rebased on the kvm bump.

but otherwise looks :+1:

@j2sol how does this conflict with KVM bump? A rebase will be required to merge with #959, but not anything KVM related.

All tests will fail because qemu-kvm package is hardcoded, and the old version is no longer in the repo. So unless the test is ran with the new version in the config it will fail.
I'm not 100% sure what's what failed the test, because I broke the test on accident in Jenkins itself. A re-push here to trigger jenkins again might be necessary.


Add GKE package

A kpt package to configure a GKE cluster with a default node pool and a dedicated subnet.

@karlkfi looks like lint is failing. make docker_fix_lint should try to auto fix for you.


Point debugger URL to redot Github resources

This changes the engine debugger to reference https://github.com/Redot-Engine/redot-engine instead of https://github.com/godotengine/godot so that issues prompted to be reported and submitted are not sent to the other project.

The change seems to be failing on Linux with Mono. Maybe there is a check to ensure the link doesn't change? Not sure

seems fine, needs to be moved to the redot-merge branch. it's failing for Linux tests though so there's likely more related files.

Thanks so much! I think I found what you saw, I pushed another changed to ProjectSettings.xml and it seems to pass.  Waiting on the checks to complete and then will request a review.


How to append quill-mention dropdown to body?

same issue here is there any fix for this ?

i used this but didn't change anything


Use a LocalObjectReference for Medusa bucket secret

What this PR does:
Changes the type of Medusa's Storage Secret Ref from a string to a LocalObjectReference.
Which issue(s) this PR fixes:
Fixes #401
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[x] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

StorageSecretRef is not set as required, thus it is optional. It's not ever checked actually if it's there, everything is just assumed. Is this always a correct approach? What happens if I don't have "local", but do not have that field set? Is the default value alright to not break anything?

Reconciliation should then fail because the secret won't be mounted correctly.
I can add a check prior to that though and fail the reconciliation with a more specific message.


removing typos

updated the docs little bit, added punctuation removed grammatical errors.

Too many errors and false positives. Sorry!

Hi @tannerlinsley,
in my next pull can you approve this ?